kafka-node Producer doesn't emit Ready event

We are trying to create a kafka-node Producer and send message using it. However, the 'Ready' event is not emitted by the Producer and hence cannot proceed with the send message code.
Environment Information :
OS: Windows 10
Node Version : 10.16.3
NPM Version : 6.9.0
kafka-node version: 5.0.0
Steps to Reproduce :
Following is the POC that we are trying.
const Kafka = require('kafka-node')

const brokerList = 'broker-0-xxxx,broker-1-xxxx,broker-3-xxxx,broker-5-xxxx,broker-4-xxxx,broker-2-xxxx'
// tried providing a proper array here, but throws: [ERR_SOCKET_BAD_PORT]: Port should be >= 0 and < 65536. Received NaN error

const username = 'xxxxxx'
const password = 'xxxxxxxxxxxxxxx'

const _getProducerInstance = function () {
  const client = new Kafka.KafkaClient({
    kafkaHost: brokerList,
    sasl: {
      mechanism: 'plain',
      username: username,
      password: password
    },
    sslOptions: { rejectUnauthorized: false }
  })
  const prod = new Kafka.Producer(client, { requireAcks: 1 })
  return prod
}

const _init = function () {
  const producerInstance = _getProducerInstance()
  console.log('---------- Producer created---------------')
  // Tried explicit connection too
  // producerInstance.connect()
  producerInstance.on('ready', () => {
    console.log('---------- In Ready---------------')
    producerInstance.send([{ topic: 'xxx', partition: 1, messages: ['Hello World'] }], (err, result) => {
      if (err) {
        console.error('------------Error--------------', err)
      } else {
        console.log('--------------Result-----------', result)
        producerInstance.close()
      }
    })
  })

  producerInstance.on('error', function (err) {
    console.log('------------- error event----------------', err)
  })
}

_init()

I have faced a similar issue, i don't have a proper solution for this, but it turns out that usually, the code will work if the constructor of the producer is called in the same scope as the actual producer.on('ready',CB) call. I don't know why this is the case, but that's just how it is
In your case, I believe, if you replaced the call to the _getProducerInstance() function with the actual code, it will probably work

I am also facing the same issue. Is there any solution for that ?
The above workaround is also not working for me.

I am also facing the same issue. Is there any solution for that ?
The above workaround is also not working for me.

I actually found out the proper solution to this, a few days ago
change the producer.on('ready', CB) to
if(producer.ready){
// do some work
}
Apparently, every time the constructor for the producer is called, it emits a ready event, which is what producer.on('ready') looks for.
However, this event is only emitted once, hence the producer.on('ready') doesn't work in a different scope from the constructor apparently.
The proper solution is quite simple. once the event is emitted at first, it sets the ready property of the producer object to true, which should then be used as the condition to check if Kafka connection is live and then you may use producer.send({topic, payloads}) or whatever else it is you need to do

I am still not able to publish the data. This is my code structure. I am getting timeOutError in send method callback.
`
// adminService
import { KafkaClient, HighLevelProducer } from 'kafka-node';
export class Service {
private static instance: Service;
private kafkaClient: any;
private producer: any;
private constructor() {}
static getInstance() {
    if (!Service.instance) {
        Service.instance = new Service();
        Service.instance.kafkaClient = new KafkaClient({
            kafkaHost: SOME_KAFKA_HOST,
            sasl: { mechanism: 'plain', username: 'some_user', password: 'some_password' },
            sslOptions: {
                rejectUnauthorized: false
            }
        });
        Service.instance.producer = new HighLevelProducer(Service.instance.kafkaClient);
    }
    return Service.instance;
}
get _kafkaClient(): KafkaClient {
    return this.kafkaClient;
}
get _producer(): HighLevelProducer {
    return this.producer;
}
initProducer = () => {
    this._producer.on('ready', () => {
        console.log('Producer is ready');
    });
}

}
// publish method:
const ServiceInstance = Service.getInstance();
const kafkaClient = ServiceInstance._kafkaClient;
const producer: any = ServiceInstance._producer;
const publish = async (req: any) => {
const { topic, message = '' } = req;
const payloads = [
{ topic: topic, messages: message }
];
if(producer['ready']) {
producer.send(payloads,  (err: any, data: any) => {
console.log('If any error: ', err);
console.log('Published data: ', data);
});
};
}
`
I am getting this Error in send method:
If any error:  TimeoutError: Request timed out after 30000ms     at new TimeoutError (......./node_modules/kafka-node/lib/errors/TimeoutError.js:6:9)     at Timeout._onTimeout (......./node_modules/kafka-node/lib/kafkaClient.js:1012:16)     at listOnTimeout (internal/timers.js:549:17)     at processTimers (internal/timers.js:492:7) {   message: 'Request timed out after 30000ms' }
Please suggest what could be the solution here.

Commenting further on this issue, I work with @amit-kumar27 and we've troubleshooted the issue.
We'd used IPs instead of hostnames in our connection string, resulting in Kafka never completing the handshake. Our configuration is SASL+SSL, in which Kafka should care about the hostname.
Interestingly server does show SSL connection failure in the case wrong hostname. But it never shares why the SSL connection failed.
The issue also happened in another instance when we used the wrong port. It seems Kafka leaves the connections hung up, instead of closing them with a message or dropping the connection. But here we get no message in the server logs obviously.
Thanks for helping out guys! 👍

@prafiles I am facing kind of similar issue.  When ssl handsake fails, it neither emit error event, nor emit ready event. Hope someone take a look at this.


about ip

What should I do if I want to add one to get which GPU comes from which IP? I tried to add one in my_gpustat and the corresponding one in the front end, but they seem to be unable to connect, could you help me, thank you

Hello there ! I unable to connect  either


When test on 'Synthetic_v3_InstanceSegmentation' test dataset, break out Segmentation fault (core dumped) without any more imformation

Hi, I am working to test on 'Synthetic_v3_InstanceSegmentation' test dataset. However, no matter how to split data to small blocks, it awalys come out "Segmentation fault (core dumped)" without any more imformation to fix it. what can I do to reslove [it?]
(isbnet) root@autodl-container-690911b33c-4a0c7e92:~/ISBNet-master# python tools/test.py configs/stpls3d/isbnet_stpls3d.yaml head_stpls3d_val.pth --out result/test
2023-07-16 10:37:20,083 - INFO - Load state dict from head_stpls3d_val.pth
2023-07-16 10:37:20,229 - INFO - unexpected key in source state_dict: criterion.empty_weight
2023-07-16 10:37:20,233 - INFO - Load test dataset: 100 scans
Segmentation fault (core dumped)

Hi, I am working to test on 'Synthetic_v3_InstanceSegmentation' test dataset. However, no matter how to split data to small blocks, it awalys come out "Segmentation fault (core dumped)" without any more imformation to fix it. what can I do to reslove [it?]
(isbnet) root@autodl-container-690911b33c-4a0c7e92:~/ISBNet-master# python tools/test.py configs/stpls3d/isbnet_stpls3d.yaml head_stpls3d_val.pth --out result/test 2023-07-16 10:37:20,083 - INFO - Load state dict from head_stpls3d_val.pth 2023-07-16 10:37:20,229 - INFO - unexpected key in source state_dict: criterion.empty_weight 2023-07-16 10:37:20,233 - INFO - Load test dataset: 100 scans Segmentation fault (core dumped)

Hi, have you solved the problem of testing on the stpls3d26-28 dataset, I trained the whole model and I don't know how to test on unlabeled data now!

Hello, thank you for your answer, last time under Mask3D also you answered my question, I looked at your homepage, it seems that you are also Chinese, I still have doubts about this issue, may I trouble you, add your contact information to consult you? Thank you very much! My wechat is:<PHONE_NUMBER>3. if you feel inconvenient, it does not matter, anyway, thank you very much for your reply!I will try your method, thanks again!

Hi, you need to add label for val not for test. The easiest way is to add 2 columns, like this x,y,z,r,g,b,100, 100.

I apologize for my rude request, what you mean is that when I executed test.py, the files inside my val folder were semantically labeled and instance labeled, and the txt file for the test inside the test folder, simply had xyzrgb, and I executed the task python tools/test.py . /configs/plant/isbnet_stpls3d_test.yaml ... /work_dirs/stpls3d/exp200/latest.pth
--out
result/stpls3d , Since my input test data doesn't have sem_label and ins_label, he is just wrong in calculating the AP, and when running the program, I debugged it and found that the pred_insts are not simply a list of only ins_label, please ask you to implement test.py's for the test file to output their prediction results yet, thank you very much for your answer!

Hi, maybe I need to explain. I did not read the code carefully so I did not kown the reason. The solution is the result of trying luck. I have not work on the code yet, so sorry that I can not help you more.
Best!

嗨，我正在努力在“Synthetic_v3_InstanceSegmentation”测试数据集上进行测试。但是，无论如何将数据拆分为小块，它都会出现“分段错误（核心转储）”，而没有任何更多信息来修复它。我能做些什么来重新爱上它？
（isbnet） root@autodl-container-690911b33c-4a0c7e92：~/ISBNet-master# python tools/test.py configs/stpls3d/isbnet_stpls3d.yaml head_stpls3d_val.pth --out result/test 2023-07-16 10：37：20,083 - INFO - 来自 head_stpls3d_val.pth 的加载状态字典 2023-07-16 10：37：20,229 - INFO - 源 state_dict 中的意外键：criterion.empty_weight 2023-07-16 10：37：20,233 - INFO - 负载测试数据集：100 次扫描 分段故障（核心转储）

嗨，你有没有解决在stpls3d26-28数据集上测试的问题，我训练了整个模型，我现在不知道如何在未标记的数据上测试！

Hello and thanks for your great work. I have trained the stpls3d dataset using the code to get the model, I would like to know how to test the stpls3d unlabeled test data, which has only 6 columns of data, which are xyzrgb, could you please take the time to answer my question, I would appreciate if you can help me with my problem!


Temperature incorrectly passing to HomeKit

The temperature being passed from the nest add on through the homebridge is incorrect. It's set to 72°F but shows 169°F in the Home app...any ideas it shows up correctly in Home Assistant.

I searched logs and found this:
2017-10-12 20:43:09 ERROR (MainThread) [homeassistant.core] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/lib/python3.6/asyncio/tasks.py", line 181, in _step
result = coro.throw(exc)
File "/usr/lib/python3.6/site-packages/homeassistant/core.py", line 1025, in _event_to_service_call
yield from service_handler.func(service_call)
File "/usr/lib/python3.6/site-packages/homeassistant/components/climate/init.py", line 331, in async_temperature_set_service
yield from climate.async_set_temperature(**kwargs)
File "/usr/lib/python3.6/asyncio/futures.py", line 331, in iter
yield self  # This tells Task to wait for completion.
File "/usr/lib/python3.6/asyncio/tasks.py", line 244, in _wakeup
future.result()
File "/usr/lib/python3.6/asyncio/futures.py", line 244, in result
raise self._exception
File "/usr/lib/python3.6/concurrent/futures/thread.py", line 55, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/lib/python3.6/site-packages/homeassistant/components/climate/nest.py", line 158, in set_temperature
self.device.target = temp
File "/usr/lib/python3.6/site-packages/nest/nest.py", line 510, in target
self._set('devices/thermostats', data)
File "/usr/lib/python3.6/site-packages/nest/nest.py", line 168, in _set
response = self._nest_api._put(path=path, data=data)
File "/usr/lib/python3.6/site-packages/nest/nest.py", line 1590, in _put
return self._request('PUT', path, data=data)
File "/usr/lib/python3.6/site-packages/nest/nest.py", line 1582, in _request
raise APIError(response)
nest.nest.APIError: blocked

Hi @erfind,
This seems like an issue with the Nest component (may be initiated by Homebridge). This issue is not within the domain of this add-on. I suggest you'd open an issue at the Home Assistant or Homebridge repository.


32 bit GDScript performance is 15-20x slower than 64 bit

Godot version:
3.1.1-stable
OS/device including version:
Windows 10 1903
Issue description:
GDScript performance is heavily affected by whether or not it's running in 64 bits.
Slowdown occurs both in the 32 bit editor and in exported builds.

Left to right: 64 bit, 32 bit
Steps to reproduce:
for x in range(10000000): 1 + 1
Minimal reproduction project:
32-64 Performance Test.zip

use valgrind to analysis which function cost the most of time

In case you didn't know, there're no iterators in GDScript and range returns an actual array. I'd be curious how much of the runtime was allocating and filling those 10000000 numbers. The speed difference may be due to auto-vectorization.

In case you didn't know, there're no iterators in GDScript and range returns an actual array. I'd be curious how much of the runtime was allocating and filling those 10000000 numbers. The speed difference may be due to auto-vectorization.

Oh. That's helpful. I'll update the issue.

GDScript ints and floats are both 64-bit data types, which can be slow on 32-bit CPUs. Additionally, 64-bit CPUs have more instruction sets than 32-bit ones. There is probably a bug with this high amount of performance degradation, but do note that you shouldn't expect parity.

Fixed by #40598. 32-bit and 64-bit timings are now 0.5 and 0.6 seconds, respectively.
Though, it's an overall increase over the original 64-bit timings. Oh well.


splitting order emits into 2 separate emits

there is now separate  orderhistory and openorder emit events

forgot to commit the change in the readme, a new request will reflect that as well


[Cider] NoSuchMethodError: The method 'readIndex' was called on null.

NoSuchMethodError: The method 'readIndex' was called on null.
Receiver: null
Tried calling: readIndex()
#0      LinkedElementFactory.elementOfReference (package:analyzer/src/summary2/linked_element_factory.dart:295)
#1      LinkedElementFactory.buildExportNamespace (package:analyzer/src/summary2/linked_element_factory.dart:55)
#2      LibraryElementImpl.exportNamespace (package:analyzer/src/dart/element/element.dart:4963)
#3      NamespaceBuilder._getExportMapping (package:analyzer/src/dart/resolver/scope.dart:315)
#4      NamespaceBuilder.getImportedElements (package:analyzer/src/dart/resolver/scope.dart:212)
#5      new PrefixScope (package:analyzer/src/dart/element/scope.dart:201)
#6      new _LibraryImportScope (package:analyzer/src/dart/element/scope.dart:314)
#7      new LibraryScope (package:analyzer/src/dart/element/scope.dart:107)
#8      LibraryElementImpl.scope (package:analyzer/src/dart/element/element.dart:5230)
#9      LibraryAnalyzer._resolveFile (package:analyzer/src/dart/micro/library_analyzer.dart:694)
#10     LibraryAnalyzer.analyzeSync.<anonymous closure>.<anonymous closure> (package:analyzer/src/dart/micro/library_analyzer.dart:148)
#11     _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:397)
#12     LibraryAnalyzer.analyzeSync.<anonymous closure> (package:analyzer/src/dart/micro/library_analyzer.dart:147)
#13     OperationPerformanceImpl.run (package:analyzer/src/util/performance/operation_performance.dart:137)
#14     LibraryAnalyzer.analyzeSync (package:analyzer/src/dart/micro/library_analyzer.dart:146)
#15     FileResolver.resolve.<anonymous closure>.<anonymous closure>.<anonymous closure>.<anonymous closure> (package:analyzer/src/dart/micro/resolve_file.dart:382)
#16     _rootRun (dart:async/zone.dart:1354)
#17     _CustomZone.run (dart:async/zone.dart:1258)
#18     _runZoned (dart:async/zone.dart:1788)
#19     runZoned (dart:async/zone.dart:1710)
#20     NullSafetyUnderstandingFlag.enableNullSafetyTypes (package:analyzer/dart/element/null_safety_understanding_flag.dart:42)
#21     FileResolver.resolve.<anonymous closure>.<anonymous closure>.<anonymous closure> (package:analyzer/src/dart/micro/resolve_file.dart:381)
#22     OperationPerformanceImpl.run (package:analyzer/src/util/performance/operation_performance.dart:137)
#23     FileResolver.resolve.<anonymous closure>.<anonymous closure> (package:analyzer/src/dart/micro/resolve_file.dart:380)
#24     PerformanceLog.run (package:analyzer/src/dart/analysis/performance_logger.dart:32)
#25     FileResolver.resolve.<anonymous closure> (package:analyzer/src/dart/micro/resolve_file.dart:365)
#26     PerformanceLog.run (package:analyzer/src/dart/analysis/performance_logger.dart:32)
#27     FileResolver.resolve (package:analyzer/src/dart/micro/resolve_file.dart:336)

Add debug information: https://dart-review.googlesource.com/c/sdk/+/182400

Closing as stale; please re-open if you see this crash on a recent version of Dart.


Add support for absolute values

I love this expression evaluator, it's very fast and the api is well designed!
I'm just wondering if it's possible/planned to add the feature of evaluating absolute values, like |-1.5| as 1.5.

found out about abs() facepalm


Add volume workload

Description
A new Volume workload has been added to ZOS and grid-client. It should act the same as ZMounts with some extra enhancements in performance as it go through less layers of virtualization.

Note: raw images are not supported

Interface
The new workload is of type volume and has the same structure as ZMount:
{
    "size": <size-in-bytes>
}

duplicated https://github.com/threefoldtech/terraform-provider-grid/issues/928


Adding Solution for Bellman Ford Algorithm

Fixing Issue #3487
Description
This pull request implements the Bellman-Ford algorithm in C++. The Bellman-Ford algorithm computes the shortest paths from a single source vertex to all other vertices in a weighted graph. It is capable of handling graphs with negative weight edges, making it a versatile tool for various applications.
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Security enhancement
[ ] Other (specify): _______________

Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have followed the code style guidelines of this project.
[x] I have checked for any existing open issues that my pull request may address.
[x] I have ensured that my changes do not break any existing functionality.
[ ] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.
[x] I have read the resources for guidance listed below.
[x] I have followed security best practices in my code changes.

Additional Context
The Bellman-Ford algorithm is a fundamental algorithm used in graph theory and is particularly important for educational purposes. This implementation includes functionalities for initializing the graph, computing shortest paths, and detecting negative weight cycles.
Resources for Guidance
Please read the following resources before submitting your contribution:

[x] Code Harbor Hub Community Features
[x] Markdown Guide

Related Issue
None
Additional Context
This implementation is valuable for educational settings where students are learning about graph algorithms, especially those dealing with negative weights. It provides a practical example of the Bellman-Ford algorithm's application.
Resources for Guidance
Please read the following resources before submitting your contribution:

[x] Code Harbor Hub Community Features
[x] Markdown Guide

@Ajay-Dhangar sir, kindly review and let me know if any changes required.


:package: feat: Init example-beta package

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #73   👈

This comment was auto-generated by Graphite.


Expand docker-compose with common dependencies used in integration tests

Privileged issue

[X] I am a LangChain maintainer, or was asked directly by a LangChain maintainer to create an issue here.

Issue Content
The goal is to make it easier to run integration tests.
Acceptance Criteria:

[ ] Update docker compose file here: https://github.com/langchain-ai/langchain/blob/master/docker/docker-compose.yml to include a database that is used during integration tests and can be spun up locally via docker compose.
[ ] Use non standard port -- just increment by 1 from the previous service in that file
[ ] Update any integration tests that use the given service to use a matching port

This is a good first issue for folks with experience in dev ops.
Consider git-grepping for existing yml files that contain service configuration.

I would like to work on this issue.


Fixed S390X CI builds not running in forked repos.

Since MSAN is not able to run in emulated environment, I have just made it to become a Compat test in forks. Without this PR, then workflows in forks will not finish and will timeout.

Looks like the S390X docker images don't have sudo, so the tests fail there now.

Hmm there doesn't appear to be a way to install sudo.

Successful build in fork here: https://github.com/nmoinvaz/zlib-ng/actions/runs/7515952515

Oops, merged this as part of #1642

Sorry, I thought this had been merged, but only one of two commits was merged.

Rebased.


Problem compiling ESP32 + TFT_eSPI + FT6206

Hello,
I tried to find other people that had the same issue but no luck.
I'm trying to use the library with a huzzah32 (ESP32) using the TFT_eSPI library and the FT6206 capacitive library. I have the 2.8" TFT LCD with Cap Touch.
But I can not get it to compile, it gives me the error:
lib/GUIslice/src/GUIslice_drv_tft_espi.cpp:822:21: error: 'class TFT_eSPI' has no member named 'getTouch'
I got the TFT_eSPI working before, so I know that my configuration is OK under the "User_Setup.h"
Even if I try to disable touch on "GUIslice_config_ard.h" the error still persists. It's always trying to include and compile this section of code inside the GUIslice_drv_tft_espi.cpp

/ Confirm that TOUCH_CS has been defined otherwise the
// m_disp.getTouch() call won't be defined in TFT_eSPI
#if defined(DRV_TOUCH_TFT_ESPI)
  #if !defined(TOUCH_CS)
    #error "To use DRV_TOUCH_TFT_ESPI, TOUCH_CS needs to be defined in TFT_eSPI User_Setup"
  #endif
#endif

bool gslc_DrvGetTouch(gslc_tsGui* pGui,int16_t* pnX, int16_t* pnY, uint16_t* pnPress)
{

  if ((pGui == NULL) || (pGui->pvDriver == NULL)) {
    GSLC_DEBUG_PRINT("ERROR: DrvGetTouch(%s) called with NULL ptr\n","");
    return false;
  }

  // Use TFT_eSPI for touch events
  uint8_t     bPressed = 0;
  uint16_t    nX=0;
  uint16_t    nY=0;
  int16_t     nOutputX, nOutputY;

  bPressed = m_disp.getTouch(&nX,&nY);

  // Perform any requested swapping of input axes
  if( pGui->nSwapXY ) {
    nOutputX = (int16_t)nY;
    nOutputY = (int16_t)nX;
  } else {
    nOutputX = (int16_t)nX;
    nOutputY = (int16_t)nY;
  }

  // Perform any requested output axis flipping
  if( pGui->nFlipX ) {
    nOutputX = pGui->nDispW - 1 - nOutputX;
  }
  if( pGui->nFlipY ) {
    nOutputY = pGui->nDispH - 1 - nOutputY;
  }

  // Assign coordinates
  *pnX = (int16_t)nOutputX;
  *pnY = (int16_t)nOutputY;

  if (bPressed > 0) {
    *pnPress = 1;
  } else {
    *pnPress = 0;
  }

  return true;
}
It's very likely that I'm missing something, could you give a hint?

Hi @vitormhenrique --
In the GUIslice config, I presume you are selecting DRV_TOUCH_ADA_FT6206 (which matches your capacitive touch driver). Since you are loading an external touch driver (ie. outside TFT_eSPI's built-in XPT2046 driver), we should probably comment out #define DRV_TOUCH_IN_DISP if you haven't already. (Looks like I need to make this clearer in the config file)
If possible, could you attach your GUIslice_config_ard.h and User_Setup.h files?

Lastly, could you confirm that you are compiling with the Arduino IDE, and that you are using the latest TFT_eSPI (eg. version 0.18.21)?

Thanks!

@vitormhenrique I am currently attempting to get an ESP32 working with an Adafruit 3.5 inch TFT display. Would you be willing to share your configuration and .h files? My code seems to compile but not display properly.

Thank you so much. I appreciate the files for reference. Once I test things out and get more to report I'll let you know how they were able to help me.

Hi @robertknorton -- can you let me know a little more about your setup?
I understand you are using an ESP32 with the Adafruit 3.5" TFT.

Is your display the "touchscreen breakout board" with resistive touch? (Adafruit PID: 2050)
Are you using the DRV_DISP_TFT_ESPI mode in GUIslice_config_ard.h?
Can you confirm that the TFT_eSPI library examples are working for you (both display and touch)? You should find these in the TFT_eSPI library under:

examples/480x320/Keypad_480x320/Keypad_480x320.ino

examples/480x320/TFT_graphicstest_one_lib/TFT_graphicstest_one_lib.ino

If the TFT_eSPI examples are running fine (display and touch), what do you see when you run the GUIslice examples? (eg. using @vitormhenrique config, perhaps modified as shown below)

Note: Your 3.5" display may require some changes from Vitor's config since I think you are using a different display driver and touch driver than the 2.8" setup he has. In particular, I think you'd want:
User_Setup.h:

Uncomment #define HX8357D_DRIVER
Comment #define ILI9341_DRIVER

GUIslice_config_ard.h:

Uncomment #define DRV_TOUCH_ADA_SIMPLE
Comment #define DRV_TOUCH_ADA_FT6206

Thanks

Hello @ImpulseAdventure
I tried messing around with the TFT_eSPI library examples and they are failing to compile.
I have made the recommended changes except for the touchscreen. I am currently not using touch, just button controls.
Here is some code from my current GUIslice_config_ard.h
// =============================================================================
// DISPLAY CONFIGURATION - SELECT
// =============================================================================
// Specify the graphics driver library
// - Uncomment one of the following graphics drivers DRV_DISP_*
//   applicable to the device type in use
// --------------------------------------------------------------
// Arduino / ATmega / AVR / Cortex-M0 / nRF52:
//  #define DRV_DISP_ADAGFX           // Adafruit-GFX library
// --------------------------------------------------------------
// ESP8266 / ESP32 / NodeMCU:
//  #define DRV_DISP_ADAGFX           // Adafruit-GFX library
#define DRV_DISP_TFT_ESPI         // Bodmer/TFT_eSPI library
//  #define DRV_DISP_M5STACK          // m5stack/M5Stack library
// --------------------------------------------------------------
// STM32:
//  #define DRV_DISP_ADAGFX           // Adafruit-GFX library
//  #define DRV_DISP_ADAGFX_AS        // Adafruit-GFX-AS library
// --------------------------------------------------------------
// =============================================================================
// TOUCH CONFIGURATION - SELECT
// =============================================================================
// Specify the touchscreen driver
// - Uncomment one of the following touchscreen drivers DRV_TOUCH_*
//   applicable to the controller chip in use
#define DRV_TOUCH_NONE            // No touchscreen support

//  #define DRV_TOUCH_ADA_STMPE610    // Adafruit STMPE610 touch driver
//  #define DRV_TOUCH_ADA_FT6206      // Adafruit FT6206 touch driver
//  #define DRV_TOUCH_ADA_SIMPLE      // Adafruit Touchscreen
//  #define DRV_TOUCH_TFT_ESPI        // TFT_eSPI integrated XPT2046 touch driver
// =============================================================================
// DISPLAY CONFIGURATION - DETAILS
// - Graphics display driver-specific additional configuration
// =============================================================================
// -----------------------------------------------------------------------------
#if defined(DRV_DISP_ADAGFX)
// The Adafruit-GFX library supports a number of displays
// - Select a display sub-type by uncommenting one of the
//   following DRV_DISP_ADAGFX_* lines
//#define DRV_DISP_ADAGFX_ILI9341         // Adafruit ILI9341
//#define DRV_DISP_ADAGFX_ILI9341_8BIT  // Adafruit ILI9341 (8-bit interface)
//#define DRV_DISP_ADAGFX_ST7735        // Adafruit ST7735
//#define DRV_DISP_ADAGFX_SSD1306       // Adafruit SSD1306
#define DRV_DISP_ADAGFX_HX8357        // Adafruit HX8357
//#define DRV_DISP_ADAGFX_PCD8544       // Adafruit PCD8544
#elif defined(DRV_DISP_ADAGFX_AS)
// The Adafruit-GFX-AS library supports a number of displays
// - Select a display sub-type by uncommenting one of the
//   following DRV_DISP_ADAGFX_* lines
//#define DRV_DISP_ADAGFX_ILI9341_STM     // Adafruit ILI9341 (STM32 version)
#endif
#if defined(DRV_DISP_ADAGFX) || defined(DRV_DISP_ADAGFX_AS)
// For Adafruit-GFX drivers, define pin connections
// - Define general pins (modify these example pin assignments to match your board)
// - Please refer to "docs/GUIslice_config_guide.xlsx" for detailed examples
#define ADAGFX_PIN_CS    14   // Display chip select
#define ADAGFX_PIN_DC    15   // Display SPI data/command
#define ADAGFX_PIN_RST   16   // Display Reset (some displays could use pin 11)
#define ADAGFX_PIN_SDCS   4   // SD card chip select
#define ADAGFX_PIN_WR    13   // Display write pin (for parallel displays)
#define ADAGFX_PIN_RD    12   // Display read pin (for parallel displays)
// Use hardware SPI interface?
// - Set to 1 to enable hardware SPI interface, 0 to use software SPI
// - Software SPI may support the use of custom pin selection (via ADAGFX_PIN_MOSI,
//   ADAGFX_PIN_MISO, ADAGFX_PIN_CLK). These pin definitions can be left blank in
//   hardware SPI mode.
#define ADAGFX_SPI_HW     1
// Define custom SPI pin connections used in software SPI mode (ADAGFX_SPI_HW=0)
// - These definitions can be left blank in hardware mode (ADAGFX_SPI_HW=1)
#define ADAGFX_PIN_MOSI
#define ADAGFX_PIN_MISO
#define ADAGFX_PIN_CLK
// Set Default rotation
// - Note that if you change this you will likely have to change
//   ADATOUCH_FLIP_X & ADATOUCH_FLIP_Y as well to ensure that the touch screen
//   orientation matches the display rotation
#define GSLC_ROTATE     1

Hi @robertknorton --
When you indicate that you tried the "TFT_eSPI library examples and they are failing to compile", can you confirm:

Whether you were using TFT_eSPI alone or combining TFT_eSPI with GUIslice? If combined, could you let me know if the TFT_eSPI standalone examples (eg. TFT_graphicstest_one_lib.ino) work?

Can you list some of the errors that were reported during compile?

Thanks!

Dear Calvin,
I'm running into problems using the Arduino IDE compiling the first example (gslc_ex_01_ard) for ESP32 and ILI9431. The code compiles but it seems to me the code from the TFT_eSPI library does not get implemented (The display doesn't do anything at all except the backlight is on):
I added some code to the example to check for the connections which should be established in the eTFT library:
#if defined(TFT_MISO)
  Serial.print ("TFT_MISO: " + String(TFT_MISO)  + "\r\n");
#else
  Serial.print("TFT_MISO not defined !!!\r\n");
#endif

And the output at runtime shows me that TFT_MISO has not been defined.
Then, I inserted tracers into GUIslice_drv_tft_espi.cpp
// Compiler guard for requested driver
#define CHECKPOINT_2  // by ws
#include "GUIslice_config.h" // Sets DRV_DISP_*

#if defined(DRV_DISP_TFT_ESPI)
#define CHECKPOINT_3 // by ws
// check if we get here at all

Which have not been defined if I check them in my code
#if defined(CHECKPOINT_2)  // coded into GUIslice_drv_tft_eSPI.cpp
  Serial.print("CHECKPOINT_2 GUIslice_drv_tft_eSPI.cpp reached\r\n");
#else
  Serial.print("CHECKPOINT_2 in GUIslice_drv_tft_eSPI.cpp NOT REACHED !!!\r\n");
  #endif

#if defined(CHECKPOINT_3)  // coded into GUIslice_drv_tft_eSPI.cpp
  Serial.print("CHECKPOINT_3 in GUIslice_drv_tft_eSPI.cpp reached\r\n");
#else
  Serial.print("CHECKPOINT_3 in GUIslice_drv_tft_eSPI.cpp NOT REACHED !!!\r\n");
#endif

However, DRV_DISP_TFT_ESPI gets defined in GUIslice_config_ard.h, as intended.
GUIslice_drv_tft_eSPI.h is found, as the query for the define of GUISLICE_DRV_TFT_ESPI_H is positive, CHECKPOINT_4 in GUIslice_drv.h is reached:
#elif defined(DRV_DISP_TFT_ESPI)
  #include "GUIslice_drv_tft_espi.h"
  #define CHECKPOINT_4  // by wo 20181006

To make it more confusing, the compiler reports that TFT_eSPI has been used and the libarary also is compiled
Compiling library "TFT_eSPI"
Using previously compiled file: /tmp/arduino_build_522274/libraries/TFT_eSPI/TFT_eSPI.cpp.o

Restarting the IDE which is deleting the tmp files doesn't change the outcome.
Do you have any clues what's going on and how to fix this?
If it should matter, I'm running
Ubuntu 18.04.1 LTS
gcc (Ubuntu 7.3.0-27ubuntu1~18.04) 7.3.0
arduino-1.8.3
Thanks for your help!
Wolfgang

Hi Wolfgang -- thanks for the report. Could you confirm a couple items for me?

Are you using a M5Stack? If so, you might want to update to the latest version of GUIslice (0.10.3) as I addressed a M5 compilation issue in the past day.
When you run the TFT_eSPI's graphics test example (from the Arduino IDE's Examples -> TFT_eSPI -> TFT_grapicstest_one_lib), I assume the display is working properly?
When running the gslc_ex01_ard example, if you enable the Serial Monitor and reset, is anything reported? (may need to adjust the Arduino IDE monitor's baud rate to match your board's serial output).

thanks!

Hi Calvin,
thanks for getting back to me. I'm working with an ESP32 dev board [1].  Meanwhile, I have found it does not generate the clock signal and some other I/Os also seem dead, so I'll have to wait with further checks until a new board arrives.
However, with an NodeMCU (ESP8266), your example code is working well. And I have learned that #define does not work across libraries, so most of the testing I have reported in my previous post, was garbage.
Sorry for the confusion and thanks again!
Wolfgang
[1] https://www.aliexpress.com/item/ESP32-Development-Board-WiFi-Bluetooth-Ultra-Low-Power-Consumption-Dual-Cores-ESP-32-ESP-32S/32799057508.html

Thanks Wolfgang for the update indicating that it may have been a board issue. Hopefully the replacement works better (and doesn't take too long to arrive!). Good to hear that the NodeMCU/ESP8266 examples ran fine. Let me know if any further help is needed.

Hi Calvin!
Got a new ESP32 (NodeMCU-32S) and tested the examples (ILI9341 w/o touchscreen, waiting for an ILI9488 with touchscreen to arrive) and did some tests:
Examples 1..4 compile and seem to run ok as far as I can see.
Example 2 has warnings:
/home/wo/Arduino/libraries/GUIslice/arduino/gslc_ex02_ard/gslc_ex02_ard.ino: In function 'void setup()':
/home/wo/Arduino/libraries/GUIslice/arduino/gslc_ex02_ard/gslc_ex02_ard.ino:57:19: warning: unused variable 'bOk' [-Wunused-variable]
   bool            bOk = true;
                   ^

Examples 1..4 have these warnings:
/home/wo/Arduino/libraries/GUIslice/src/GUIslice_ex.c: In function 'gslc_ElemXCheckboxFindChecked':
/home/wo/Arduino/libraries/GUIslice/src/GUIslice_ex.c:792:18: warning: comparison is always true due to limited range of data type [-Wtype-limits]
     if (nCurType != GSLC_TYPEX_CHECKBOX) {
                  ^
/home/wo/Arduino/libraries/GUIslice/src/GUIslice_ex.c: In function 'gslc_ElemXTextboxDraw':
/home/wo/Arduino/libraries/GUIslice/src/GUIslice_ex.c:2187:21: warning: variable 'bEncUtf8' set but not used [-Wunused-but-set-variable]
   bool              bEncUtf8;
                     ^

Example 5 yields an error:
gslc_ex05_ard:63: error: 'gslc_tsXSelNum' does not name a type
 gslc_tsXSelNum              m_sXSelNum[3];
 ^
/home/wo/Arduino/libraries/GUIslice/arduino/gslc_ex05_ard/gslc_ex05_ard.ino: In function 'bool InitOverlays()':
gslc_ex05_ard:152: error: 'm_sXSelNum' was not declared in this scope
   pElemRef = gslc_ElemXSelNumCreate(&m_gui,E_ELEM_COMP1,E_PG_MAIN,&m_sXSelNum[0],
                                                                    ^
gslc_ex05_ard:153: error: 'gslc_ElemXSelNumCreate' was not declared in this scope
     (gslc_tsRect){160,60,120,50},E_FONT_BTN);

Example 6 yields errors, too:
/home/wo/Arduino/libraries/GUIslice/arduino/gslc_ex06_ard/gslc_ex06_ard.ino: In function 'bool CbDrawScanner(void*, void*, gslc_teRedrawType)':
gslc_ex06_ard:118: error: narrowing conversion of '(((int)nX) + -30)' from 'int' to 'int16_t {aka short int}' inside { } [-Werror=narrowing]
   gslc_DrawFrameRect(pGui,(gslc_tsRect){nX-30,nY-20,60,40},GSLC_COL_BLUE_DK2);
                                           ^
gslc_ex06_ard:118: error: narrowing conversion of '(((int)nY) + -20)' from 'int' to 'int16_t {aka short int}' inside { } [-Werror=narrowing]
   gslc_DrawFrameRect(pGui,(gslc_tsRect){nX-30,nY-20,60,40},GSLC_COL_BLUE_DK2);
                                                 ^
gslc_ex06_ard:123: error: narrowing conversion of '(((int)nX) + 1)' from 'int' to 'int16_t {aka short int}' inside { } [-Werror=narrowing]
   gslc_DrawFillRect(pGui,(gslc_tsRect){nX+1,nY+1,10,10},GSLC_COL_RED_DK2);
                                          ^
gslc_ex06_ard:123: error: narrowing conversion of '(((int)nY) + 1)' from 'int' to 'int16_t {aka short int}' inside { } [-Werror=narrowing]
   gslc_DrawFillRect(pGui,(gslc_tsRect){nX+1,nY+1,10,10},GSLC_COL_RED_DK2);
                                               ^
gslc_ex06_ard:124: error: narrowing conversion of '(((int)nX) + 1)' from 'int' to 'int16_t {aka short int}' inside { } [-Werror=narrowing]
   gslc_DrawFillRect(pGui,(gslc_tsRect){nX+1,nY-10,10,10},GSLC_COL_GREEN_DK2);
                                          ^
gslc_ex06_ard:124: error: narrowing conversion of '(((int)nY) + -10)' from 'int' to 'int16_t {aka short int}' inside { } [-Werror=narrowing]
   gslc_DrawFillRect(pGui,(gslc_tsRect){nX+1,nY-10,10,10},GSLC_COL_GREEN_DK2);
                                               ^
gslc_ex06_ard:125: error: narrowing conversion of '(((int)nX) + -10)' from 'int' to 'int16_t {aka short int}' inside { } [-Werror=narrowing]
   gslc_DrawFillRect(pGui,(gslc_tsRect){nX-10,nY+1,10,10},GSLC_COL_BLUE_DK2);
                                          ^
gslc_ex06_ard:125: error: narrowing conversion of '(((int)nY) + 1)' from 'int' to 'int16_t {aka short int}' inside { } [-Werror=narrowing]
   gslc_DrawFillRect(pGui,(gslc_tsRect){nX-10,nY+1,10,10},GSLC_COL_BLUE_DK2);
                                                ^
gslc_ex06_ard:126: error: narrowing conversion of '(((int)nX) + -10)' from 'int' to 'int16_t {aka short int}' inside { } [-Werror=narrowing]
   gslc_DrawFillRect(pGui,(gslc_tsRect){nX-10,nY-10,10,10},GSLC_COL_YELLOW);
                                          ^
gslc_ex06_ard:126: error: narrowing conversion of '(((int)nY) + -10)' from 'int' to 'int16_t {aka short int}' inside { } [-Werror=narrowing]
   gslc_DrawFillRect(pGui,(gslc_tsRect){nX-10,nY-10,10,10},GSLC_COL_YELLOW);
                                                ^
cc1plus: some warnings being treated as errors

Example 7 compiles and runs,
Example 10 compiles and runs.
Best regards,
Wolfgang


add block_page_enabled

Sorry for the delay. Your PR looks good, but could you please set BLOCK_PAGE_ENABLED to 0 in .env.example? I think it's better to keep the block page disabled by default, since it might possibly carry some security risks and puts a bit too much trust on Cloudflare.


refactor: remove driver-go dependency

Codecov Report
Attention: Patch coverage is 92.39075% with 296 lines in your changes missing coverage. Please review.

Project coverage is 77.37%. Comparing base (ddba0c7) to head (3b36a59).

Files with missing lines
Patch %
Lines

driver/wrapper/stmt2.go
86.86%
70 Missing and 25 partials :warning:

driver/wrapper/stmt.go
85.66%
45 Missing and 43 partials :warning:

driver/common/serializer/block.go
87.92%
38 Missing and 19 partials :warning:

driver/common/parser/block.go
94.49%
9 Missing and 4 partials :warning:

driver/common/parser/raw.go
89.07%
10 Missing and 3 partials :warning:

driver/wrapper/row.go
83.33%
7 Missing and 2 partials :warning:

driver/wrapper/setconfig.go
59.09%
8 Missing and 1 partial :warning:

driver/common/change.go
72.72%
6 Missing :warning:

driver/wrapper/field.go
91.66%
2 Missing and 1 partial :warning:

driver/wrapper/cgo/handle.go
90.47%
1 Missing and 1 partial :warning:

... and 1 more

Additional details and impacted files
@@            Coverage Diff             @@
##              3.0     #362      +/-   ##
==========================================
+ Coverage   72.91%   77.37%   +4.46%
==========================================
  Files         111      142      +31
  Lines       13806    18069    +4263
==========================================
+ Hits        10066    13981    +3915
- Misses       3058     3298     +240
- Partials      682      790     +108

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Small fix blog post network upgrade 17th October 2020

Change "final user" to "end user". Or remove final.

#1167


Re-render html to change colors

Hello all
I have been using this lib a while now. It does very well the work. But now i have a requirement that is to change the colors of based on the theme.
For that, for now, i have my styles on classStyles and tagStyles props that will adapt to the theme that i have applied. In order for the styles to be used i have to re-render the complete html that depending on the html size will or not be noticed by the user.
Do you think there is a more efficient solution for that?
Thank you very much.

In version 4.2.2, HTML component will re-render on tagsStyles and classesStyles changes, if this is what you meant.
You can test a pre-release version now, see #385.


Fix typo of cuda-z-beta

Single quotes caused this error::
Error: Cask 'cuda-z-beta' definition is invalid: 'url' stanza failed with: bad URI(is not URI?): https://downloads.sourceforge.net/project/cuda-z/cuda-z/Beta/CUDA-Z-#{version}.dmg

Thanks for fixing that :+1:


151

QUESTION 151
Hotspot Question
For each of the following statements, select Yes if the statement is true. Otherwise, select No.
NOTE: Each correct selection is worth one point.


Disable blank lines around top level definition

Operating system: windows10
Python version: 3.6.5
Black version: 18.6b4
I need to use only one blank line or whatever how many blank lines before top level of class/def I want,
How black would help in this case?

How black would help in this case?

It wouldn't. It's opinionated and it standardizes the amount of blank lines to adhere to PEP 8.


Warning C6328 is not detected

It seems like warning C6328  is not detected:
https://godbolt.org/z/bMb4PnYT4
(The function f2 was just added to check that warnings in general are detected and displayed.)

Sure enough, it doesn't. Looking into it.

We are in the process of moving away from GitHub issues for C++ documentation (we still accept pull requests). For details, please see Announcing a new way to give feedback on Microsoft Learn This issue will be closed after it is copied to an internal tracking system.


The beam_method "KatBeam" in simulate beam raises a NameError when called

I tried to do a primary beam simulation like it is shown in the function test_beam in test_beam.py, but I used "KatBeam" as a beam_method. I get the following error

I think one needs to add a self. when calling get_meerkat_uhfbeam.
After I did this I got a new error message:

What is f? I guess one needs to choose a value for f.

Yes, I see it's a bit broken. It needs to be BeamPattern.get_meerkat_uhfbeam(..) but I am unsure what f is supposed to be:

It seems that f is some sort of frequency?

@rohitcbscient can you help?


copying API key WIP

Copying not working and errors in console.

@mgrauer this should be good now

Great, thanks @JellyWang-3 and @AlmightyYakob . I tested this in preview and it works.


DTXProfiler framework not found

Description
After installing everything, when I launch the app it just crash. I have installed Detox Instruments with brew, include the script and add the FLAGS
Steps to Reproduce
Follow the README installation steps
Expected Behavior
The app to launch
Crash Dump
dyld: Library not loaded: @rpath/DTXProfiler.framework/DTXProfiler
  Referenced from: /Users/brian/Library/Developer/CoreSimulator/Devices/DF7A7EBE-F0C0-4E65-8128-F24F3F17658C/data/Containers/Bundle/Application/47784207-2F62-4D6C-B2B4-7AA24FB580C7/Perlego.app/Perlego
  Reason: image not found

Environment

Model: MacBook Pro 2018
OS: macOS 10.15.4
Xcode: 11.4.1

iOS Device (please complete the following information):

Model: Simulator
OS: iOS 13.4.1
Xcode: 11.4.1

me too

Thanks guys, I’ll look soon

Can you post the contents of the resulting .app folder? Specifically, in the Frameworks folder. Thanks

I've just released a new version. Please try again. Since you haven't responded to my request, I am closing the issue. If the issue persists, please post the requested information, as well as your build log from Xcode, and I'll reopen. Thanks


restart apache

I need to restart apache after deploy because it seems symlinks are still cached for a little while (has no one noticed this?? - at least when you use .htaccess rewriting).
So, my current deploy is as the user. At the final step I need to su to root so that I can restart apache. How can I do that? (password should be typed in). thanks.

It's the same like restart PHP FPM.
See https://github.com/deployphp/docs/blob/master/examples.md

You muck config sudo passwordless for deploy user.
After that, make a task like restart php_fpm and run it after deploy finished.
https://www.google.com/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=sudo+passwordless


Making Apache Kafka bootstrap servers configurable in the Docker image

In part it's already true because we have a bridge.properties configuration file for that


Settings - when updating explorer url is not updated in settings after reopen

Wallet: v0.12.1
Steps to reproduce:

Login to wallet.
Click Settings in top right corner.
Change explorer link from "http://explorer-96x.symboldev.network/" to "http://explorer-96x.symboldev.network2/" and save settings.
Now when you I click "Open in explorer" link in account it opens http://explorer-96x.symboldev.network2/account/TDBVCFJCOMSPFRE6AIEXLC2M425VVXLCVJQPNHA which is correct.
When I click again Settings in top right corner explorer link is presented as "http://explorer-96x.symboldev.network/" instead "http://explorer-96x.symboldev.network2/" which is used in the wallet.

Still exists in v0.13.1

Still exists :)

Video: https://share.getcloudapp.com/rRu0LRkW

It still doesn't work. Bug as in the video: https://share.getcloudapp.com/rRu0LRkW

Looks like fixed in 0.13.8

Looks like fixed in 0.13.8


Google plus button now loads after load event

Update to the Google plus social button.
The button will now load once the "onload" event has been fired.
I have also remove some unnecessary code.
*I should also mention that the button pulls in a Google font, Roboto

:+1:

:+1:

:+1:


Load resources json files inside component

I'm submitting a ...  (check one with "x")
[ ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[x ] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

What is the motivation / use case for changing the behavior?
I need to load the json file with the resources only when the app is already booted because I need to make a resquest to the server do some specific project informations.
At this moment I'm loading the json files like the exemples:
export function createTranslateLoader(http: Http) {
    return new TranslateHttpLoader(http, './assets/i18n/', '.json');
}

@NgModule({
    imports: [
        BrowserModule,
        HttpModule,
        TranslateModule.forRoot({
            loader: {
                provide: TranslateLoader,
                useFactory: (createTranslateLoader),
                deps: [Http]
            }
        })
    ],
    bootstrap: [AppComponent]
})
export class AppModule { }

How can I loaded it inside a component?

The translations will only be loaded when you call the .use(lang) method, you can call this anywhere you want.


How to catch error throw by FileTools.ReadFile?

like https://github.com/BabylonJS/Babylon.js/pull/10209
will it possible to catch error in FileTools.ReadFile?
when parse an unlegal file, it will throw bad hdr format error, but i cannot catch the errors by onError or wrap it with try catch.

As indicated in the template, please first report the problem to forum for discussion.
do not hesitate to create a PR logging the error as done in #10209 ? I can do it if you prefer.


fix bug issues for sonar scan.

sovles #23577 of sonar scan.
This pr includes bug issues from sonar scan.

API change check for com.azure.spring:spring-cloud-azure-autoconfigure
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-autoconfigure

API change check for com.azure.spring:spring-messaging-azure-servicebus
API changes have been detected in com.azure.spring:spring-messaging-azure-servicebus. You can review API changes here

close this pr for this is split into two prs.


Rename job to jobId in DeploymentGroup

To be consistent with other classes like Deployment

@rohansingh @gimaker

:+1:


Error in the 4th condition

Challenge http://www.freecodecamp.com/challenges/waypoint-wrap-many-elements-within-a-single-div-element has an issue. Please describe how to reproduce it, and include links to screen shots if possible.
When I create the div tag and assign it an ID, it gives an error for the 4th condition of the challenge.

So I tried it this morning and this will work:

Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

But this:
 <--- Note the space after div *****
Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

Results in;
Cannot read property 'length' of null
If you put an ID on it as mentioned  by the reporter it will the same result as above. It's the space after the word div before the ending > that causes the error condition.  Looks like it's because the  last test is trying to do an exact match  and trying to use length when there isn't an dom object that matches causing it to error out.

This challenge has since been removed.
Happy Coding!


Add exclude option

I really want to put my specs as close as I can to what they are testing.  Is there currently a way to exclude files from the glob?  If not, it would be great to have an exclude option which would also have glob syntax and would exclude files from the group.

This is actually a great idea. How would you suggest this works?
I am actually leaning to expose the underlying glob-functionality and just pass on any options that are not require-globify-specific to node-glob.

I see that node-glob options has an ignore parameter that would work for what I want.
I suggest you just add an options parameter to your current object and pass whatever is set, which is less confusing than trying to distinguish parameters.

Fair enough, that also adds the benefit of eliminating the risk op future naming collisions (if/when they might occur). I'll implement it like that. Expect it to be in the next version which should come out somewhere in the next 48 hours. I'm using this time to also handle the other issues that have been piling up.

Sweet, thank you!

Took a while longer than expected, but should now be implemented.
I'm deploying the new version when TravisCI comes back up (it's momentarily down for maintenance) and it gives the all-green.

This is great news, thank you!  I will give it a try tomorrow.

I added this new feature to my project, and it worked like a charm.  Thank you!  I am now able to keep my spec's right next to what they test.


How to support define options.

Because the vite use esbuild define option at optimizer, the rollup has [@rollup/plugin-replace](https://github.com/rollup/plugins/tree/master/packages/replace#rollupplugin-replace) to do it,  it using string match to replace, it is performance friendly but other ecosystem using last to do it, eg swc/webpack.
I think we can use @rollup/plugin-replace to do it at milestone 1. It means we need to support some plugin API for the plugin, it is easy to implement.
But we need re-think it how to support it at rolldown for better performance and correct replace result.

I am interested in this issue and making the plugin work.
we consider performance, I think the plugin should be implemented on the Rust side.
However, we would ideally like to keep it compatible with the existing rollup plugin.
Do you have in your team about a plugin plan, how to implemetation idea?

It seems that this feature has already been supported in PR #1882?

Yeah. But we haven't exposed the options from nodejs. Will closed this issue once it's done.


Data Glossary Tags: order Tag suggestions by Level and Alphabet

Is your feature request related to a problem? Please describe.
Example. I have a Data Glossary Term Earth. It has 7 Children (continents) and each of them have Country children.
Use cases:

I only want to tag continent Europe to a table. I type in Europe. I get the list of 45 suggestions. As Europe is part of FQN name for Continent and Countries. And it doesn't seem that is ordered in any specific order.

This is a real case example, where I need to tag a parent term, but suggestion lists all the children, and children of children. And I have maybe 50 of them.  What's worse - I only see the beginning of FQN, but I think there is a bug for that already.

Describe the solution you'd like
If I type Europe in the Tag, I first get parent terms that contain Europe. And only then Children Terms.
Describe alternatives you've considered
I tried typing in Geo.Continent.Europe but it didn't work.
Additional context
Add any other context or screenshots about the feature request here.

@harshach and @chirag-madlani, we should have tag suggestions that include the hierarchy as we do in the Add User page as shown below:

Yes. the tree hierarchy - would make the discovery of tags so much easier!

hello, do we have any plans for this enhancement?

hello, do we have any plans for this enhancement?

We'll handle for 1.4

Hi @pmbrull @DovileKr
Can you confirm if current design for Tag Selection will be good here? I will add Alphabet sorting to this.
My view for not going to hiearachy approach is:

It will add indentation which will consume some space and we are already at crunch of space there
To avoid long fqn as a line, We separate actual term and FQN with two different lines
Search results will be more clear instead it will be hard in case of hierarchy view (May have horizontal scroll in case of n level of nesting)

Hierarchy view fits better if we have full width to use
cc: @sureshms @harshach

@chirag-madlani we don't need to expand by default right? only when the user searches for it the term needs to be expanded.
The goal here is to make it easy for users to understand the hierarchy and making sure the term is selected from the right glossary.


Add TestEvaluator helper module

This PR introduces TestEvaluator test helper, which will be needed to properly test operators precedence (as discussed on Discord). Some usage examples can be seen in the documentation I inserted in the code itself.

@emilsoman You can see evaluating an expression not only tests the parser, but also the type checker and the Erlang translation, just like Elixir does (except for the type checking obviously). I wonder whether it's something you want or not.


scihub acq ingesters blocked by bad creds

The following jobs are blocked by what appears to be a credentials misconfiguration:

job-acquisition_ingest-scihub:develop
job-acquisition_ingest-scihub_daily:develop
job-acquisition_ingest-aoi:develop

Error message:
Traceback (most recent call last):
  File "/home/ops/verdi/ops/scihub_acquisition_scraper/acquisition_ingest/scrape_apihub_opensearch.py", line 557, in
    ESA_USER, ESA_SECRET = read_creds()
TypeError: cannot unpack non-iterable NoneType object

Need to check end points and configuration in read_creds function.

The read_creds function is pulling from /export/home/hysdsops/verdi/etc/settings.yaml on factotum. Not seeing any mention of anything ESA in there.

Updated settings.yaml on factotum and mozart. Daily scihub scrapers are now failing due to missing entry in datasets.json -- need to add entry on factotum and mozart.

tweaked regex in dataset.json for daily scrapers

The PGE is now running (and doing all the work correctly), but is failing with a 500 error when trying to publish it's dataset to GRQ.


Why LiveAuthRequestUtility.RefreshTokenAsync() is not public?

Hi,
Why LiveAuthRequestUtility.RefreshTokenAsync() is not public?
Here http://msdn.microsoft.com/en-us/library/dn631817.aspx (Windows desktop apps documentation) says "Tip  If the wl.offline_access scope is specified, a refresh token is also provided.
Your app can read the authorization code from the returned URL, and use it to retrieve an access token by making a call that passes a URL similar to this one."
The SDK is doing it, in LiveAuthRequestUtility.RefreshTokenAsync()  but this class is internal.
Why it is not public? It will allow custom code to simply call LiveAuthRequestUtility.RefreshTokenAsync() to get a fresh valid session object.
Thanks

I don't see any reason to keep it internal if you want to use it.  Submit a pull request after you get it doing what you need any I'll happily take a look


Add dependencies to fix example

Thanks - apologies for delay, been off on paternity leave.  We shifted all the internal use of this to Express, but forgot to tidy up the examples, this will mean it at least works for everyone again.

:+1:


ItalyNorth location not working

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform: 1.5.7
azure provider: 3.75.0
module: 4.2.0
Description
Tests using provided examples, for example https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/wiki/[Examples]-Deploy-using-multiple-module-declarations-with-remote-state, fail when the new ItalyNorth region is specified as location
Describe the bug
locals.geo_codes.tf.json does not contain ItalyNorth and its geocode

Steps to Reproduce

Prepare Terraform folders and files according to example https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/wiki/[Examples]-Deploy-using-multiple-module-declarations-with-remote-state
Edit connectivity/variables.tf  by setting italynorth as primary_location
Execute terraform plan
the following error is generated:

│ Error: Invalid index
│
│   on .terraform/modules/alz/modules/connectivity/locals.tf line 1448, in locals:
│ 1448:       "privatelink.${local.lookup_azure_backup_geo_codes[location]}.backup.windowsazure.com"
│     ├────────────────
│     │ local.lookup_azure_backup_geo_codes is object with 134 attributes
│
│ The given key does not identify an element in this collection value.
Screenshots

Additional context

fixed by #834


Is not possible to change the editor code font family on Settings

Is not possible to change the editor code font family on Settings. (In Linux - Ubuntu Gnome 16.4)

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


[ENHANCEMENT] When/how do we get assets about new Teams features like breakout rooms?

When will we get assets about the new features in MS Teams that were rolled out end of 2020, like breakout rooms and the option to poll attendees?
Or do we have to do something from our end to pull them in?
I did find this page online, but I could not find it within Learning Pathways: https://support.office.com/en-us/client/7de1f48a-da07-466c-a5ab-4ebace28e461?embed=true

New content is added in batches so when the next content update happens, whatever additions the Learning Pathways team makes to the content packs will automatically available in your Learning Pathways instance. I see you have the URL formatted correctly to add manually as an asset now too. I just tested in my environment and that page works beautifully :)

New content is added in batches so when the next content update happens, whatever additions the Learning Pathways team makes to the content packs will automatically available in your Learning Pathways instance. I see you have the URL formatted correctly to add manually as an asset now too. I just tested in my environment and that page works beautifully :)

We'll be publishing a list of content updates on March 4th during Microsoft Ignite on our blog at https://aka.ms/DriveAdoption When we publish new content as mentioned above it will simply stream into your experience but your custom playlists will not be disrupted.

Content focused question
User received an answer, no follow up responses
Closing issue


[qtbase] Build time when cross-compiling

Is your feature request related to a problem? Please describe.
When cross compiling, the qtbase package needs a copy of itself built for the host as well, presumably to use the Qt build tools. This package is built with all the features of Qt enabled and so takes a significant amount of time to build, only to not use the majority of the features that were just built.
Proposed solution
Use a more minimal set of features for the qtbase package built for the host if possible.
Describe alternatives you've considered
No response
Additional context
No response

https://github.com/microsoft/vcpkg-tool/pull/177

Looks pretty abandoned. Is there any chance this will actually happen?
Building all of qtbase for the host just to use its tools inflates our CI build times with a clean cache to 45 minutes...

It is not abandoned. The vcpkg owners just put priority on other issues.
Basically it is already possible to limit the host qtbase to core by adding this request to the command line or to the user project manifest. But it breaks if a reverse dependency forgot to declare the "default-features": false.

Assuming it does work for qtbase, is it documented somewhere how I can change this in my manifest?

In the dependencies section:
  {
    "name": "qtbase",
    "host": true,
    "default-features": false
  }

Great, thanks for the tip! This speeds the builds up massively.

@nadiaholmquist Is your problem solved?

Yes, for my specific use case it is - and with the linked PR already covering the more general case this issue can probably be closed.


Question：Flowdroid cannot build path between flow1 and flow2.

public class Flow {
public static int data1=-1;
public static int flow1(int count){
data1++;
if(count<1){
count++;
}
return count;
}
public static int flow2(int count){
count=count+data1;
if(count<2){
count++;
}
return count;
}
public static void entryMethod(){
int flowdatat1=0;
data1=flow1(flowdata1);
flow2(flowdata1);
}
}
hi aoli, i see that your work use flowdroid for local primitive taint analysis.
The reason is that flowdroid cannot find flow path between flow1 and flow2, right?
Flowdroid can transfer taint variable via assign|call parameter|call return value instead of shared memory data1, right?
So, you use phosphor to trace the taint path between flow1 and flow2 via data1.
My understanding is correct? If i misunderstand your work, please correct me, thanks.

The reason we use FlowDroid to only track local primitive is: 1) we don't want to do any taint analysis online. Taint analysis is expensive and we can't afford it in production environment. 2) Static data-flow analysis is fairly accurate for local primitives.
In your example, yes, FlowDoird/Soot can perform really bad if the program contains complex data dependency between heap objects and local primitives. For ExChain, it does not use track anything dynamically. It only adds exception information to heap objects.

Thanks.
Why Exchain does not track anything dynamically?
Exchain firstly infers affected state variable and responsible state variable.
For heap objects, Exchain taints affected state variable to responsible state variable by Phosphor.
This procedure is not online.
So you add exception information to heap objects and record them. Then you analyze information to get dependency, right?
For example,
P1：
try{
responsible variable
}catch(Exception e1){
write heap_obj1
}
P2:
heap_obj2=func(heap_obj1)
heap_obj3=func(heap_obj2)
P3
try{
if(check(heap_obj3){
throw e2;
}
}catch(Exception e2){
handleE2();
}
For e1, affected state variable is heap_obj1
For e2, responsible state variable is heap_obj3
taint path is heap_obj1->heap_obj2->heap_obj3.
Exchain does not analysis this taint path online, right?
Exchain just add exception information to heap_obj1,heap_obj2,heap_obj3 for offline dependency analysis, right?
If like this, should exchain add exception information to which heap objects?

Because the overhead introduced by taint analysis is too high.
In your example, if func copy heap_obj1 to heap_obj2, ExChain cannot track this propagation since it requires tracking the propagation among heap objects.

Thanks.
I am curious that how to add exception information to heap objects for heap-to-heap analysis.
I did not find it in the paper.
If i miss something, I would so thankful for mentioning it.

Yes, current implementation supports that! It uses phosphor to perform full dynamic taint analysis (will introduce 10X overhead).
You can do it by
python3 runner.py wicket_6908 build
python3 runner.py wicket_6908 instrument
python3 runner.py wicket_6908 run --type dynamic

The main difference is how to update and propagate taint information: https://github.com/aoli-al/exchain/blob/main/runtime/src/main/kotlin/al/aoli/exchain/runtime/analyzers/AffectedVarDriver.kt#L167 and https://github.com/aoli-al/exchain/blob/main/native/src/cpp/affected_result_processor.cc#L147

Thanks! It helps me a lot.
I will close this issue.


Fix spurious leading spaces in output of DynStringSetValue.String().

The array is allocated with the same size as the map, instead of having the capacity of the map but size zero. This results in unintended leading empty string elements in the array.

Can you add a test? :)


fix(INJI-39): skip request for PHY upgrade to PHY2

reason:

if a preferential PHY is not requested in a connection, the BLE 5.0 chips which have issues with 2M PHY can also connect using older PHY and the responsibility of choosing a PHY is left to the firmware/OS and other entities

concerns on speed: https://github.com/mosip/tuvali/issues/52
Link: https://mosip.atlassian.net/browse/INJI-39

clone of #56 being sent to develop


Create a UUID for loaded configs if necessary

What this PR does / why we need it:
When loading a config without a UUID, we don't generate one and it
remains empty. It is necessary for the config to have a UUID as it is
used to create a uniquely named directory for storing results. Without
this, we don't correctly create a child directory to store the results.
We create a tarball within a directory that we also try to add to that
tarball, effectively adding the created tarball to itself resulting in a
write error.
Signed-off-by: Bridget McErlean<EMAIL_ADDRESS>Which issue(s) this PR fixes

Fixes #806

Release note:
Fixed a bug where a config without a UUID would result in an error when writing results

Codecov Report

Merging #807 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #807      +/-   ##
==========================================
+ Coverage   42.79%   42.84%   +0.04%
==========================================
  Files          71       71
  Lines        4229     4232       +3
==========================================
+ Hits         1810     1813       +3
  Misses       2311     2311
  Partials      108      108

Impacted Files
Coverage Δ

pkg/config/loader.go
63.52% <100%> (+1.33%)
:arrow_up:

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 700ac79...9a1fddb. Read the comment docs.


Add gtfslib-python library.

See https://github.com/afimb/gtfslib-python for the library.

Thanks!


Grey map on iOS

I'm submitting a ...  (check one with "x")

[ ] question
[x] any problem or bug report
[ ] feature request

If you choose 'problem or bug report', please select OS:  (check one with "x")

[ ] Android
[x] iOS

cordova information:  (run $> cordova plugin list)
com.googlemaps.ios 2.7.0 "Google Maps SDK for iOS"
cordova-plugin-device 1.1.7 "Device"
cordova-plugin-googlemaps 2.3.11 "cordova-plugin-googlemaps"
cordova-plugin-ionic-webview 1.2.1 "cordova-plugin-ionic-webview"
cordova-plugin-splashscreen 4.1.0 "Splashscreen"
cordova-plugin-whitelist 1.3.3 "Whitelist"
ionic-plugin-keyboard 2.2.1 "Keyboard"

Current behavior:
Hello , i downloaded your example ionic project from https://github.com/mapsplugin/ionic-googlemaps-quickdemo and run it by command "ionic cordova run ios". Application was started fine, but i have a grey screen. I've read this (https://github.com/mapsplugin/cordova-plugin-googlemaps-doc/blob/master/v1.4.0/TroubleShooting/Blank-Map/README.md) help, but no one method not helps me. The same problem in device (iPhone 6)
Expected behavior:
Expected to see the map in application
Screen capture or video record:

Related code, data or error log (please format your code or data):
https://github.com/davayd/googlemapsiosproblem

https://github.com/mapsplugin/cordova-plugin-googlemaps-doc/blob/master/troubleshootings/graymap/README.md

Your key is valid for JS API and geocoding API, but your key is NOT valid for iOS SDK.

how can I make it valid?

Regenerate api key for ios.

I've generate new api and past in config.xml

In Xcode searching this API on whole project

-- no one find
same behavior

Xcode console

White page and gray map is completely different meaning and behavior.
Gray map means your api key is invalid.
White page means you execute code before platform.ready().
As the evidence, Ionic Native tried accessing the GoogleMaps plugin but it's not installed. warning.

Here
https://docs.google.com/presentation/d/e/2PACX-1vScoho1ensbR4qCI9AIuQN55BZVvK73pAjI7sumDvW3CrxxHnrmpXWUjx2-8CpFibqU1EjLKCRhuthJ/pub?start=false&loop=false&delayms=3000&slide=id.g292c767148_0_24
And here
https://docs.google.com/presentation/d/e/2PACX-1vScoho1ensbR4qCI9AIuQN55BZVvK73pAjI7sumDvW3CrxxHnrmpXWUjx2-8CpFibqU1EjLKCRhuthJ/pub?start=false&loop=false&delayms=3000&slide=id.g292c767148_0_24

ping

Issue was solved with help gmap docs (https://developers.google.com/maps/documentation/ios-sdk/start#step_3_get_an_api_key). I'm just generate new API key and set the Credentials. Anyway, thx for help!


fixed plugin datetimepicker

So.... what is this fixing...?

dupe of #1562 and #1563?

At the moment, the plugin datetimepicker does not work in the master branch, I fixed it

It looks like the "fix" isn't a fix, it's a replacement to use a different plugin.  Was there an issue that this is fixing? I have no context here.  What was the original issue?


docs: update server.md

withing -> within

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

Project coverage is 89.86%. Comparing base (c48b9be) to head (8e9f13a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #619   +/-   ##
=======================================
  Coverage   89.86%   89.86%
=======================================
  Files          19       19
  Lines        1125     1125
=======================================
  Hits         1011     1011
  Misses        114      114

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


sha/asm/sha512-armv8.pl: adapt for kernel use.

I have tested the resulting sha512-armv8.pl with the Linux kernel patch I intend to propose once this change is merged, and it works as expected.
Thanks!

@levitte, last minor adjustment in commentary needs formal re-confirmation.

Yup


Fix SASL_CB_GETCONF(PATH) detection

Fix for https://github.com/memcached/memcached/issues/365
On RHEL(5|6|7) the name of the defined constant in sasl.h is SASL_CB_GETCONFPATH and not SASL_CB_GETCONF, this adds support for HAVE_SASL_CB_GETCONFPATH in configure.ac and then uses it in sasl_defs.c

nice, thanks! looks good, I'll test it and merge when I get a chance

seems to work, I've staged it for the next release! if you want bonus points, reviewing the related sasl tests and improving the wiki docs around sasl are great things to do :)

Thanks.  I can improve https://github.com/memcached/memcached/wiki/SASLHowto if you can grant me access to edit it.

I invited you to an org team, which has 'read' access on memcached.. which I think gives you wiki edit access. give it a shot?

hmm... I'll have to try again. Just poked through and I swear the docs said 'read' access to a repo from a team member would allow editing the wiki... you can also just write up a markdown page into gist to get things started. :/ sorry, I have to be a bit careful with write access.

No problem.  I created:  https://gist.github.com/thatsafunnyname/e76cd73831ed43315d3c23c9d83d1f2e

updated, thanks!

released in 1.5.7


Update README.md: sin(num) ~> sin($num).

:memo:

Thanks for fixing all this documentation... :+1: I was writing them in the middle of night :D

Glad you like it. 😉


⚠ Samsung has degraded performance

In d9f7c70, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2804 ms

Resolved: Samsung performance has improved in c353620.


TimeCopDateTime should extend DateTime

As new DateTime returns an instance of TimeCopDateTime it breaks all methods that expect instance of DateTime.
This only happens when I execute php through php-fpm. Inheritance works fine in CLI

The base of problem is the TypeHinting:
function dump_date(DateTime $date) {
  var_dump($date);
}
dump_date(new DateTime());

But this issue can be closed because TimecopDateTime is an instance of DateTime from v1.1.0.
var_dump(new TimecopDateTime instanceof DateTime);
var_dump(new TimecopDateTime instanceof DateTimeInterface);


Fix for sequential execution failure.

These ipmi tests are large in number, up to now these ipmi
tests are executing sequentially, due to this if any test fails
initially all other remaining tests are not getting executed.
In order for effective execution below procedure helps us.
1. Store alll the tests in a list.
2. Run tests in a for loop.
3. If any failure occurs, increase the failure count
and continue with the next test.
4. Finish all the tests from list.
5. Report failure count and failed test cases at the end.
Now this fix make sure run all the ipmi tests which are stored in a list
and notifies the user at the end, how many failures happenned during execution.
By this way it resolves sequential execution failure
Signed-off-by: Pridhiviraj Paidipeddi<EMAIL_ADDRESS>

This change is

some more commits are pushed for few fixes in this PR.

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion.

common/OpTestIPMI.py, line 183 [r1] (raw file):

    def ipmi_power_on(self):
        output = self._ipmitool_cmd_run(self.cv_baseIpmiCmd + 'chassis power on')
        time.sleep(BMC_CONST.LONG_WAIT_IPL)

why did you remove the sleep?
It was intended to wait for the system to power on before executing any other commands

Comments from Reviewable

This sleep is causing missing hostboot isteps while waiting for working state in ipl_wait_for_working_state---> It will call ipmi_sol_capture. We need to log complete boot console messages into host_sol.log otherwise we will be missing important boot time failures. Any how we are giving enough sleep time in ipl_wait_for_working_state which will block the user to execute any host commands.

After ipmi_power_on , we are always using this function to wait for working state.
def ipl_wait_for_working_state(self, timeout=10):
    ''' WORKAROUND FOR AMI BUG
     A sleep is required here because Host status sensor is set incorrectly
     to working state right after power on '''
    sol = self._ipmi_sol_capture()
    time.sleep(60)   <------we have a 60s selep

    timeout = time.time() + 60*timeout

    ''' AMI BUG is fixed now
     After updating the AMI level the Host Status sensor works as expected.
     '''
    cmd = 'sdr elist |grep \'Host Status\''
    while True:
        output = self._ipmitool_cmd_run(self.cv_baseIpmiCmd + cmd)--->This command also takes more time to execute
        if 'S0/G0: working' in output:
            print "Host Status is S0/G0: working, IPL finished"
            break
        if time.time() > timeout:
            l_msg = "IPL timeout"
            print l_msg
            raise OpTestError(l_msg)
        time.sleep(5)

So we need to call  _ipmi_sol_capture as early as possible after ipmi power on, so that we can capture entire boot messages.

Reviewed 3 of 4 files at r1, 4 of 4 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Paidipeddi Pridhiviraj<EMAIL_ADDRESS>writes:

After ipmi_power_on , we are always using this function to wait for working state.
def ipl_wait_for_working_state(self, timeout=10):
    ''' WORKAROUND FOR AMI BUG
     A sleep is required here because Host status sensor is set incorrectly
     to working state right after power on '''
    sol = self._ipmi_sol_capture()
    time.sleep(60)   <------we have a 60s selep

what bug is this? Has it been fixed in a new AMI build?
Which one?
A sleep isn't good and is prone to breaking or hiding something.
Could we mandate a recent AMI build with this bug fixed (i.e. did this
ever make it into a released build?)
--
Stewart Smith
OPAL Architect, IBM.

it is fixed, as AMI is setting the "Host Status" to S0/G0: working at right time.
i Just now checked.
In system is in powered off--->
Host Status      | 04h | ok  | 35.0 | S5/G2: soft-off
When host-boot boots---->
Host Status      | 04h | ok  | 35.0 | Legacy ON state
When skiboot about to finish and starts linux kernel---->
Host Status      | 04h | ok  | 35.0 | S0/G0: working

But with my fix https://github.com/open-power/op-test-framework/pull/67/commits/e09c0a3b33caa89ba6f95000ac093a594511374e removing the sleep before capturing(__ipmi_sol_capture) , we are able to capture entire boot messages right after ISTEP 6.5
08:39:09 [SOL Session operational.  Use ~? for help]
08:39:09  41.37571|HWAS|PRESENT> DIMM[03]=AAAAAAAAAAAAAAAA
08:39:09  41.37572|HWAS|PRESENT> Membuf[04]=CCCC000000000000
08:39:09  41.37572|HWAS|PRESENT> Proc[05]=C000000000000000
08:39:09  55.25784|ISTEP  6. 6
08:39:09  55.30492|ISTEP  6. 7
08:39:09  55.30531|ISTEP  6. 8
08:39:09  55.33265|ISTEP  6. 9
08:39:09  63.29926|ISTEP  6.10
08:39:09  63.34515|ISTEP  6.11
08:39:09  64.71156|ISTEP  6.12
08:39:09  64.71280|ISTEP  6.13
08:39:09  64.71360|ISTEP  7. 1
After starting sol capture 60s sleep is required as saqibh mentioned we need to stop the user to execute any host related accesses to execute while host boots.
But i think we can remove that sleep also as "Host Status" is sensor setting to working state at right time.


Created proper error logging

I explain in a comment why i didn't use the enum.
Let me know if you have changes/suggestions, tomorrow I'll implement them, goodnight.

Ahahahahahah


Make checked vs enclosing error mismatch a warning

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #19529 into next-release will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           next-release   #19529   +/-   ##
=============================================
  Coverage         15.73%   15.73%
=============================================
  Files                47       47
  Lines              1265     1265
  Branches            197      197
=============================================
  Hits                199      199
  Misses             1053     1053
  Partials             13       13

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 ca55ebe...69a7031. Read the comment docs.


Bug report: black app screen after sending file from macOS

When I run the development version (https://github.com/localsend/localsend/commit/d0e864ddf6ebe1c55188b33f8fdccd534d203c75) on macOS and send a file, after sending, as soon as I press "Done", the app screen goes black:

And I have to kill the app and reopen.
In version v1.15.4 the problem does not exist.
flutter: 2024-10-26 22:17:50.337064    [INFO] [Refena]  - Next: []
flutter: 2024-10-26 22:17:50.337147    [INFO] [Refena]  - Rebuild (1): [ViewProvider<SendTabVm>]
flutter: 2024-10-26 22:17:50.337175    [INFO] [Refena] └────────────────────────────────────────────────────────
flutter: 2024-10-26 22:17:50.337333    [INFO] [Refena] Action finished:   [SelectedSendingFilesNotifier.ClearSelectionAction]
flutter: 2024-10-26 22:17:50.337999    [INFO] [Refena] Action dispatched: [SelectedSendingFilesNotifier.ClearSelectionAction] -> [ClearCacheAction] (global)
flutter: 2024-10-26 22:17:50.343765    [INFO] [Refena] ┌────────────────────────────────────────────────────────
flutter: 2024-10-26 22:17:50.343897    [INFO] [Refena]   Rebuild by [ViewProvider<SendTabVm>] triggered by [ChangeEvent(SelectedSendingFilesNotifier)]
flutter: 2024-10-26 22:17:50.343950    [INFO] [Refena]  - Prev: Instance of 'SendTabVm'
flutter: 2024-10-26 22:17:50.343985    [INFO] [Refena]  - Next: Instance of 'SendTabVm'
flutter: 2024-10-26 22:17:50.344017    [INFO] [Refena]  - Rebuild (1): [ViewModelBuilder<SendTabVm>]
flutter: 2024-10-26 22:17:50.344045    [INFO] [Refena] └────────────────────────────────────────────────────────
flutter: 2024-10-26 22:17:50.382704 [WARNING] [ClearCacheAction] Failed to clear cache: ParallelWaitError: UnimplementedError: clearTemporaryFiles() has not been implemented.
flutter: 2024-10-26 22:17:50.394395    [INFO] [Refena] ┌────────────────────────────────────────────────────────
flutter: 2024-10-26 22:17:50.394540    [INFO] [Refena] Provider disposed: [ViewProvider<SendTabVm>]
flutter: 2024-10-26 22:17:50.394600    [INFO] [Refena] └────────────────────────────────────────────────────────
flutter: 2024-10-26 22:17:50.402617    [INFO] [Refena] Action finished:   [ClearCacheAction]

Fixed in 7cc185373fbd769c5440833f16d9a8664b69ed50

@Tienisto Now I noticed that there is an error in the language selection button

Good catch! I fixed this also


Add command to validate rollapp state against hub state

In order to validate snapshot validity pre running the rollapp we should have the ability to validate the rollapp state against the hub state and alert the user in case of a mismatch in terms of app-hash.
[edit]
upon extracting the snapshot, we get access to the database. Open the database, check for the latest state and compare it with the hub state

current thoughts are that it's not MVP material as this can be done manually by the operator and if it doesn't match the hub state it will be detected anyway using detection of comparing to hub state later on.


hotfix: VS Code badges in README

Not sure what needs to happen for this to propagate to https://marketplace.visualstudio.com/items?itemName=Iterative.dvc .

Nice, thanks!


[Azure DevOps] Updated to use encodeURIComponent

Hey, I just made a Pull Request!
Small updated based on this comment: https://github.com/backstage/backstage/pull/20729#discussion_r1405804594
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Thanks @Rugvip and also to @freben for the comment about this 🚀


Clear any pooled proxy connections

Based on the explanation of the bug in #3090 this PR closes any pooled connections for any active ProxyManager object associated with the session.

This looks great to me. @bodgit could you add some tests to ensure this doesn't regress?

I'm wondering what/how I can test this. I had a look through tests/test_requests.py but I could only see one proxy-related test. Do you have any suggestions?

Create a dictionary with values which are mocks and then assert that clear() was called once on each one?

To be clear, my code review is +1 on this too. @sigmavirus24 has the best testing approach here, I think we just need to mock it out.

The suggestion makes sense, however I don't know how to do this with pytest. I'm trying to find an existing example I can use as a starting point...

I've managed to come up with the following:
    def test_session_close_proxy_clear(self, mocker):
        proxies = {
          'one': mocker.Mock(autospec=True),
          'two': mocker.Mock(autospec=True),
        }
        session = requests.Session()
        mocker.patch.dict(session.adapters['http://'].proxy_manager, proxies)
        session.close()
        proxies['one'].clear.assert_called_once_with()
        proxies['two'].clear.assert_called_once_with()

I had to pull in pytest-mock. It passes with my change and fails when I revert it so it looks like it does the job.

I think that looks fine to me. If you want to go ahead and update this pull request (you can just push your new commits to the branch), we'll review more formally. =)

Fixed.

Thanks @bodgit! :tada:


Fix check script exit code.

Refactor to wait for all checks to finish.
Exit with 1 on failure so CI check runs will fail.

@BigBlueHat The time sorting code won't work for checkouts.  I think adding all the contexts will work fine?  If it's a performance issue in the future this can be revisited.


Does flatgeobuf uses http range requests?

Hi,
I was wondering if this library uses httprange requests to fetch header and features. Mainly something similar to pmtiles.

It does if you are using the bbox API to select a subset of features. Search the readme for "Range" and there's a little bit about it.
The rust client actually uses range requests for the "select all" case as well as it's buffering strategy.


<ComboBox menuTrigger="focus" />

Provide a general summary of the issue here
<ComboBox /> doesn't close completely when I click outside the window
screen-capture.webm
https://react-spectrum.adobe.com/react-spectrum/ComboBox.html#trigger-options

🤔 Expected Behavior?
Expected Behavior?
😯 Current Behavior
Current Behavior
💁 Possible Solution
No response
🔦 Context
No response
🖥 Steps to Reproduce
Steps to Reproduce
Version
3.31.1
What browsers are you seeing the problem on?
Chrome
If other, please specify.
No response
What operating system are you using?
W11
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

https://github.com/adobe/react-spectrum/assets/44729383/09468f23-f56b-4c85-b338-461f278002b4
Here's what I'm seeing on MacOS 13.5.2 Chrome where the behavior seems to be a bit better. It looks like in Windows 11 that when you close a combobox outside of the browser window and then click back on the page, the menu doesn't reopen visually making it quite confusing that you are able to make selections.

Actually, it might be that what is happening is the menu is opening up and mouseUp is just happening really quickly so that we don’t see it. Can you try selecting an item in the combobox, clicking outside of the browser window to close it, then clicking to the side of the combobox where the menu would not be and let us know what happens then?

Just checking, but you mean this right?

If i click the side of the  it closes completely.

-> If i click the side of the  it stays closed.


Vite 3.0 support?

Vite 3.0.0 was released today, is it possible that this plugin could be updated to support that?
It appears that the plugin works just fine with vite 3.0.0 but the dependency just needs to be updated to include ^3.0

@bcorcoran we will absolutely be support Vite 3 - but as it only shipped today we need to test it with the plugin. Support will likely be in the next release.


Add force-graph library for collection visualization in admin dashboard

I have added force-graph library for collection visualization in admin dashboard. Suggestion in issue #4457

Thank you but there is no need for this at the moment.
There are plans to add an ERD visualization, not just a graph, but remains a low priority as the Admin UI will be refactored in the near future.

ok :( Let me know whether I can help with anything, I would love to help


Convert Api class into angular.js service (provider)

This allows for injecting angular objects like $http which:
a) other changes will depend on.
b) currently used plain XHR should be replaced with it.

Enable Hide whitespace changes in github's diff view to make diff easier to review.

@resoai BTW, related to future changes, any particular reason code doesn't use promises but instead uses callbacks? I belive angular.js polyfills promises if they are not available.

I think that it was simply an old habit :) I don’t mind promises.


Investigate sysbox, and try to bring up containers inside

https://github.com/nestybox/sysbox

Dear Professor,
The issue has been solved and the solution is uploaded in Markdown documentation format as you had suggested.
Please find the attached file of solutions and instructions.
Conatiners with sysbox.md


Add while loops

This PR adds support for while loops, including break, continue and return statements in the loop body

The error message for inifinite loops run on the node REPL can be improved:


A little polish on docs

Prepare for a new release.
Add configuring page, with more explicit docs and examples for each option.
Remove configuration stuff from creation page.
Comparisons are ugly, so remove them from the index page.

Whew, nice 🎉


Update state param to base64

Coverage decreased (-0.2%) to 78.661% when pulling d5747540cf9e9f840e1dadb875de884527a04775 on stateValidation into 032b1b4070949546d1872198c16783f39ac3c246 on development.


build: release candidate

...

:tada: This PR is included in version 1.18.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


nupac.json sha256 mismatch

Describe the bug
Expected: 88944f8c0a15ffb3917701aaf7c6090ac48bce9c68294b3f3f42cee8be28f1a1
Actual: 8419b6dd4dec2989af65eb9946093257ea63694c802b47a0a566ebeab3b8f08d
How to reproduce

use installer.nu with uncommented checksum validators

Expected behavior
Checksums don't mismatch
Screenshots
No response
Nu configuration

key
value

version
0.67.0

branch
metadata

commit_hash
33674d3a98dd7510a4e4784c079dfdaecfd2e0d6

build_os
linux-x86_64

build_target
x86_64-unknown-linux-gnu

rust_version
rustc 1.63.0 (Arch Linux rust 1:1.63.0-1)

cargo_version
cargo 1.63.0

pkg_version
0.67.0

build_time
2022-08-16 22:50:03 +00:00

build_rust_channel
release

features
database, dataframe, default, trash, which, zip

installed_plugins

Nupac configuration

key
value

author
[skelly37,Yethal]

checksum
c50d8e3f81ead7fc0e4f87aab09ed97a36b87ef464d7d70ea13bccd615321d00

installer

keywords
[package,packages,management,manager]

long-desc
Nupac is a package manager written in nu and for nu. Source on https://github.com/skelly37/nupac

name
nupac

nu-dependencies
[]

os
[android,linux,macos,windows]

post-install-msg

pre-install-msg

raw-url
https://raw.githubusercontent.com/skelly37/nupac/metadata/modules/nupac/nupac.nu

short-desc
package manager for nushell

url
https://github.com/skelly37/nupac/blob/metadata/modules/nupac/nupac.nu

version
0.1.0

Additional context
No response

Should be solved as a part of re-adding checksum validations after merging https://github.com/skelly37/nupac/pull/85


Update the User Guide

The User Guide is not up to date with the new commands implemented.
The User Guide should be up to date and comprehensible.
Let's update the User Guide to make it more complete and accurate.

Codecov Report

Merging #188 (2e2183a) into master (56ac362) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 2e2183a differs from pull request most recent head 87c927a. Consider uploading reports for the commit 87c927a to get more accurate results

@@            Coverage Diff            @@
##             master     #188   +/-   ##
=========================================
  Coverage     48.18%   48.18%
  Complexity      584      584
=========================================
  Files           130      130
  Lines          2596     2596
  Branches        322      322
=========================================
  Hits           1251     1251
  Misses         1256     1256
  Partials         89       89

Impacted Files
Coverage Δ

...u/address/logic/commands/task/ReminderCommand.java
0.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM


Relocate locally.md to the main Kubernetes repo under local-up-cluster.md

Based on @bgrant0607 feedback on PR https://github.com/kubernetes/kubernetes.github.io/pull/454, we need to get docs/getting-started-guides/locally.md moved to kubernetes/kubernetes/docs/dev/local-up-cluster.md in the main Kubernetes repo. We should point people at minikube for local setups.

https://github.com/kubernetes/kubernetes.github.io/pull/454#issuecomment-237171172


Use definition of DTypeLike from Numpy if found

This allows application of numpy ufuncs to DataArrays to typecheck correctly with Numpy 1.20
E.g. without this fix:
import xarray as xr
import numpy as np

def applyufunctodataarray() -> xr.DataArray:
    a = xr.DataArray(np.array([-1,2,3]))
    a.__array__
    b = np.abs(a)
    print(b)
    return b

will result in:
test.py:9: error: Argument 1 to "__call__" of "ufunc" has incompatible type "DataArray"; expected "Union[Union[int, float, complex, str, bytes, generic], Sequence[Union[int, float, complex, str, bytes, generic]], Sequence[Sequence[Any]], _SupportsArray]"
Found 1 error in 1 file (checked 1 source file)

While it will type check without issues with this fix.
I have not yet documented this or added tests.
Please let me know if this change is acceptable and I am happy to do that.

[ ] Tests added
[x] Passes pre-commit run --all-files
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst

Hello @jenshnielsen! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file xarray/core/npcompat.py:

Line 86:1: E302 expected 2 blank lines, found 1

IIUC the existing code may break type-checking, but the code will still run on numpy<1.20; please let me know if that's not the case.

Yes the existing code which is now the fallback could be considered wrong as it does not match the definitions that numpy ships.
We could consider syncing that with numpy but that seems like more trouble than its worth since no one is likely to typecheck this against the definitions of numpy functions without running numpy 1.20 or newer.
The numpy definition is:
DTypeLike = Union[
    np.dtype,
    # default data type (float64)
    None,
    # array-scalar types and generic types
    type,  # TODO: enumerate these when we add type hints for numpy scalars
    # anything with a dtype attribute
    "_SupportsDType[np.dtype[Any]]",
    # character codes, type strings or comma-separated fields, e.g., 'float64'
    str,
    _VoidDTypeLike,
]

So one could add str None and type but should probably omit the rest in order to not replicate too much of numpys internal type definitions

Thanks. Looks good. IMHO typing is a nice-to-have far-from-complete part of xarray & I wouldn't bother about backwards compatibility (of the typing). Up to you.

Great, merging. Thanks @jenshnielsen !

Thanks @max-sixty  @mathause