refine HDFS version installation

At present it's rather hard to understand how to build HDFS version, e.g. which env. variables and paths should be specified before compiling.
The minimum goal is to update the corresponding section in installation guide. Also, it'll be good to adopt CMake FindHDFS module for LightGBM.
Original discussion:

@StrikerRUS:

left only guide for compiling HDFS version on Linux and marked it as untested (to be honest, I'm not sure that this version can be compiled at all. For instance, we don't provide FindHDFS CMake module like this one https://github.com/dmlc/dmlc-core/blob/master/cmake/Modules/FindHDFS.cmake and HDFS_CXX_LIBRARIES variable appears in Google results only in LightGBM repo (see screen below));

@ebernhardson:

@StrikerRUS I'm not sure how to test hdfs on macos, even if i borrowed a macbook for a few days I don't have any hdfs cluster that could be connected to by that macbook.
I didn't see in your linked message what the actual error reported was, only that a variable both created and used in that file isn't seen anywhere else.
For the acutal implementation, indeed this doesn't use a FindHDFS package. I suppose it could, i wasn't aware of one at the time. The current cmake code primarily works via find_package(JNI_REQUIRED). That sets up all the things regarding java which may give trouble. The rest is looking for hdfs.h and the related shared library which is very boilerplate and all paths should be externally providable. I have indeed only tested this on debian and ubuntu systems running cloudera hadoop and wouldn't be surprised if the build step didn't work elsewhere.
The linked cmake package in dmlc-core certainly looks more robust, converting to it would be a reasonable goal imo.

@StrikerRUS:

@ebernhardson Thank you very much for your response!
I tried to compile HDFS version at Appveyor but with no luck. As I remember, the errors were about that HDFS_INCLUDE_DIR cannot be found. After I set this variable manually, error about HDFS_LIB appeared, and I stopped experiments :-) .
The linked cmake package in dmlc-core certainly looks more robust, converting to it would be a reasonable goal imo.
It will be great if you can take this! Or at least if you have no time, please provide some instructions about what should be done in order to compile HDFS version (if you know any special platform-dependent requirements for compiler, CMake, etc., please include them too). You can either do it here in comment (I'll add them in this PR), or in your PR (in this case, base branch should be macos due to significant rework of the installation guide structure).
Thanks in advance!

Please help, @ebernhardson

I'm not great at documentation, but perhaps a (hopefully) working demo will do the trick?
https://gist.github.com/ebernhardson/c1fbe7c7c2425afe5131d8d6130572f8
I put together a dockerfile that will pull appropriate dependencies into a vanilla debian jessie image, compile lightgbm with hdfs, and install the python library with it. I added additional instructions for how to connect that image to cloudera's quickstart image to see training data load from hdfs into lightgbm and be utilized.

Thank you very much!
I'll definitely add your Gist as an example in the guide!
And what about FindHDFS CMake module? Will it be useful to adopt this module for LightGBM?

BTW, don't you want to contribute your dockerfile from Gist to LightGBM's docker folder under new hdfs directory?

ping @ebernhardson

Another attempt to ping @ebernhardson

I gave the dmlc FindHDFS module a shot, it unfortunately completely fails to build on my cdh cluster.  They try to link the .a instead of .so, which for unknown reasons causes a relocation error related to -fPIC. They also explicitly do not check /usr/include for hdfs.h, but that's where cdh installs it. I can start changing things, but without a system that works that way I can't verify anything in particular.
Overall it seems like replacing the current setup with dmlc FindHDFS simply breaks one configuration to work on another. I'm not sure that's worthwhile? At best perhaps document that there are two paths that might have to be explicitly set:
HDFS_INCLUDE_DIR: Path containing hdfs.h
HDFS_LIB_DIR: Path containing libhdfs.so
If someone wants to try and collect containers for various hadoop distributions, and attempt to make a FindHDFS that works on all of them that would be great, but the existing version in xgboost (which looks quite similar to the impala version) isn't working for me.

Thanks a lot @ebernhardson for coming back here! And thank you very much for giving it a try! I think that it's better to leave the current setup as is and abandon the idea about adopting dmlc FindHDFS module.
I'll definitely document PATH tips for users and the fact that the current implementation works fine on CDH.
And what about your Gist? Would you mind creating a PR with it as a new entry in docker folder?
BTW, there are several open issue in the LightGBM repo connected with Hadoop/HDFS. We'll really appreciate if you take a look at they when you have a free time. Thanks!

Closed via #1890.


Siblings of html tag

Just from inspection (i.e., I didn't test), it appears that the comments in
<!DOCTYPE html>

won't make it into the xml tree.

Just from inspection (i.e., I didn't test), it appears that the comments in
<!DOCTYPE html>

won't make it into the xml tree.


Document Platform APIs

The cql-evaluator needs documentation around the APIs a platform needs to implement in order to support CQL execution and the FHIR operations. These are roughly:

FHIR Resource Crud
FHIR Searching/Batching/Transactions
Terminology Services
CQL Services (Caching / Configuration)

Additionally, specific operations could have their own requirements, like job scheduling for PlanDefinition/ActivityDefinition $apply.

Related: The current interfaces are CQL-centric requiring a fairly deep understanding of what the DataProvider, LibraryLoader, and so on do. Ideally, the Platform APIs would be FHIR-centric with most of the code to utilize them correctly contained internally in the cql-evaluator. This would simplify integration on new platforms. Need to solicit community feedback on what those should look like so that they are as easy to implement as is possible, and also remain generally stable going forward.

Discussion ongoing here: https://github.com/DBCG/cql-evaluator/discussions/90


Ethics Templates

Perhaps the hardest part in sharing a dataset is to get the Ethics part approved and on point. If we were to provide templates/examples on how to get a study approved in the CONP portal, somewhere, it would facilitate the sharing of a dataset.

Bonjour ! Here is our shiny new ethics toolkit - https://github.com/CONP-PCNO/Ethics_and_Data_Governance


ios-plus doesn't seem to be exist.

I also don't seem to see this font in the ionicons.js file.

It's called add in v3, see here: http://ionicframework.com/docs/v2/ionicons/


Android - Room - Room background is severely cut off when created offline

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

Version Number: 1.4.76-0
Reproducible in staging?: Y
Reproducible in production?: N
Found when executing PR : https://github.com/Expensify/App/pull/35045
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Go to staging.new.expensify.com
Go offline.
Go to FAB > Start chat > Room
Create a room.

Expected Result:
The background image in room will not be severely cut off.
Actual Result:
The background image in room is severely cut off.
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/4032b9a3-240a-47ea-93bc-1ff46e36c879

View all open jobs on GitHub

@mallenexpensify I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think this issue might be related to the #vip-vsb.

Production

This feels like a minor UI issue. I think it doesn't need to block the deploy.

I agree this doesn't need to be a deploy blocker. Looks like @luacmartins removed the label so I'm bumping to Daily.
Tried multiple times to reproduce on Android via BrowserStack with no luck.

Sorry, what is cutoff? I don't see it 😕

The bug in the OP is this

and the way it's supposed to look is to have the full background image, like here

Waiting for proposals!

I'm from Software Mansion agency and I'll work on it

I can reproduce the issue. Moreover when I go offline on Android many SVGs are missing, for example:

Do you also see this problem?

@jnowakow Thanks for looking into this! I do see the same issue with the missing SVGs.

Assigned to @jnowakow and added to #vip-vsb cuz it doesn't have to do with expense.
@jnowakow , please try to find additional instances that you can fix at the same time (since you already found some)

Thanks @jnowakow , I bumped it to weekly, it's not a high priority issue.

PR approved by me.

The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/23710
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/23710/files#r1671779151
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
Determine if we should create a regression test for this bug. Yes
If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Go offline
Create new room chat
Verify that the chat's background image is displayed correctly without any cut-off

Do we agree 👍 or 👎

Contributor+: @jjcoffee paid $250 via Upwork.
Thx for the TestRail steps, issue created

https://github.com/Expensify/Expensify/issues/411945


Command Class Wishlist

All users of this library: here's your chance to influence which CCs I will implement next.
The rules are simple:

Pick one of the un-implemented CCs
Vote for it:
a. Search this issue. If there's a comment for your CC, vote for it using the thumbs up button.
b. If there's no comment yet, please add one.

Unrelated comments and comments that don't follow the rules will be ignored and deleted.

These CCs are next:

this could be your CC!

Already requested:

Application Status - Won't be supported. Supervision is better in every regard and we can't match the Application Status commands to the requests they belong to.
Association Command Configuration
Energy Production
Meter Table Monitor
Schedule
Transport Service - Receiving is supported in v8.0.0
Window Covering

I want to get a vote in for Color.  I could try to implement it too, but I haven't been able to find the location of the spec docs.

Color Switch
I would like to try implementing it, but so far I have been unable to find the spec docs.

That's the Color Switch CC, to be found in https://github.com/AlCalzone/node-zwave-js/blob/master/specs/SDS13781-Z-Wave-Application-Command-Class-Specification.pdf on page 148ff. If you would like to make a PR, go ahead.

"Alarm Sensor CC"
16:08:05.075 CNTRLR « [Node 013] TODO: no handler for application command
16:09:17.290 DRIVER   Dropping message because it could not be deserialized: The command class Alarm Sensor is not implemented
Thank you!

Association Command Configuration

Scene Actuator Configuration

Scene Controller Configuration

Thermostat Fan Mode

Thermostat fan state

I am interested in Schedule one, to be used with new devices supporting scheduling
Schedule Comman Class. I am also happy help implementing it

Barrier Operator

I'd love to finally be able to move my scene controller, an Enerwave ZWN-SC7, off of the Vera controller that it resides on. This device does not support the Central Scene Command class (91 or 0x5B). Instead, it uses Z-Wave Group Associations along with the Scene Controller Configuration Command Class (45 or 0x2D). This is described in the user manual. You use this command class to a map an association group, which is mapped to a physical button, to a scene ID. For example, for this device it looks like Group 1 is mapped to “Button 1”. The scene ID can be any number 1-255. I'm certain to be moving to zwavejs as I use Zwave2mqtt and it's moving to zwavejs.

@MYeager1967 please upvote https://github.com/zwave-js/node-zwave-js/issues/729#issuecomment-730310951 to vote for Scene Controller Configuration

Entry Control, literally no DIY keypad solution on the market at the moment!

Application Status
I'm seeing some reports of it being dropped due to not being supported. Has something to do with "device being busy". Sounds simple :)

Application Status
I'm seeing some reports of it being dropped due to not being supported. Has something to do with "device being busy". Sounds simple :)

Irrigation, given that summer is coming :)

summer is coming

I'll use that as the release name

Sound Switch, please.  I have a couple of the Zooz MultiSirens that I use with the legacy HA Zwave implementation that use parameter 3 via a service call, Sound Switch would make life easier!

I'm currently working on Scene Controller Configuration and will do Scene Actuator Configuration afterwards.

Being able to move my last z-wave device off the old Vera controller would be awesome!!!!

Reminder:

Unrelated comments and comments that don't follow the rules will be ignored and deleted.

Powerlevel would be very useful to debug networks

Started implementing Entry Control in https://github.com/WizKid/node-zwave-js/commit/7648f2de5669df450976023870a529d71e6cd8ac . Still more to do which I'm planning to do. But thought I post in case someone else is working on it.
Especially handling the notifications and doing something useful with them. But I'm testing it with Ring Keypad and overall it seems to work. Pushing in a code and clicking Arm or any of the other buttons works. In the sense that the messages are parsed and I can see in the log that we get the right data.

Powerlevel CC would be very useful to debug networks and would integrate nicely with the new node graph.

I'm implementing PowerlevelCC now.

Put up a pull request for Entry Control at #1838 .

I’m voting for MultiCommandCCCommandEncapsulation since I have three Philio PST02-1A that are ignored when they send motion and door-open data... (temperature and illuminance are OK):
14:38:55.358 DRIVER « [Node 024] [REQ] [ApplicationCommand]
                       └─[MultiCommandCCCommandEncapsulation]
14:38:55.365 DRIVER   Received a command that contains multiple CommandClasses.
                      This is not supported yet! Discarding the message...
14:38:55.367 CNTRLR « [Node 024] TODO: no handler for application command

https://github.com/zwave-js/node-zwave-js/blob/master/packages/zwave-js/src/lib/driver/Driver.ts#L2006

Energy Production

Window Covering

Hi,
Thanks for asking the masses what we need.
I'm voting for Transport Service as I've loads of messages on my log stating "Dropping message because it could not be deserialized: The command class Transport Service is not implemented".
I think they are being generated while trying to communicate with my Aeotec Water Sensor 7, so maybe this will also help pushing for that device to be supported soon. ;)

Door Lock Logging

Humidity Control Mode/Operating State/Setpoint

Humidity Control Mode/Operating State/Setpoint

Meter Table Monitor
I've an "Eneco - Meteradapter" by "Prodrive B.V." PN: 6599-1500-0201 that sends this command. It is connected to the P1 port of my electricity meter.
Z-Wave JS logs show two of these every 10 seconds:
SERIAL « 0x0122000400081c600d06003d0d000111000007e601090d1d0060006fbc9d67000 (36 bytes)
         0013b0c
DRIVER   Dropping message because it could not be deserialized: The command class Meter
          Table Monitor is not implemented (ZW0303)
SERIAL » [ACK]

I expect that this contains the kWh counter(s) of the meter.

Schedule Entry Lock V3
E.g to be able to set temporary/time limited user codes on locks.
I'm aware it's deprecated, but guessing many locks that exist today use this CC.

I would like to see support for the Screen Meta Data Command class.  This would allow for setup of Button Labels on the Nexia NX1000 scene controller.

Command Class: Door_Lock_Logging

@danielbrunt57 you keep requesting that, but it is supported already.

@danielbrunt57 you keep requesting that, but it is supported already.

I have had this Kwikset 910 since about 2008 and it supports these command classes:

Antitheft V2
Association Grp Info
Association V2
Battery
Configuration
Device Reset Locally
Door Lock Logging
Door Lock V2
Firmware Update Md V3
Manufacturer Specific V2
Notification V4
Powerlevel
Schedule Entry Lock V2
Security
Time Parameters
User Code
Version V2
Zwaveplus Info V2

From 2000 to 2021 I used HomeSeer and had logging in 2008 when I acquired the lock (that's 15 years ago btw!)
yet I see nothing in Z-WaveJS resembling logging...

am I missing or not understanding something???

Not all CCs are exposed via the value API. Some are just too complex for that, some don't make tons of sense.
https://zwave-js.github.io/node-zwave-js/#/api/CCs/DoorLockLogging
AFAIK this CC also doesn't operate in push-style but would have to be polled sporadically/on demand.
You can open an issue in https://github.com/zwave-js/zwave-js-ui and request support for a dedicated UI for Door Lock Logging.

I've been in contact with iBlinds tech support, and they are telling me that many of the issues I have with iBlinds are due to the command class that ZWave JS is using (Multi-level switch).  They told me that there is a newer command class for Window Coverings, that isn't being implemented by many hubs, that is more appropriate for Horizontal blinds.  Is this command class implementation on the roadmap any time soon?
If it is, what would need to be done to change the device integration?

@tamorgen : Maybe I'm misunderstanding the specification. But according to page 159 of the specification https://raw.githubusercontent.com/zwave-js/specs/d6c115ec1c46dd1db9f3dca3fc2106df577fac9c/Z-Wave Device Class Specification.pdf it says:

It is not RECOMMENDED to use this generic device class for new devices. Instead, it is
RECOMMENDED to base devices of this category on the Multi-level switch generic device class with
the Multi-position Motor specific device class

To me that sounds like ZwaveJS is doing the right thing.

@tamorgen : Maybe I'm misunderstanding the specification. But according to page 159 of the specification https://raw.githubusercontent.com/zwave-js/specs/d6c115ec1c46dd1db9f3dca3fc2106df577fac9c/Z-Wave Device Class Specification.pdf it says:

It is not RECOMMENDED to use this generic device class for new devices. Instead, it is
RECOMMENDED to base devices of this category on the Multi-level switch generic device class with
the Multi-position Motor specific device class

To me that sounds like ZwaveJS is doing the right thing.

No, it should not.  The problem here is that while the generic multi-level command class works fine for roller shades or curtains, where Open is at one end of the spectrum, and close is at the other, Horizontal blinds have an open at 50% (in the middle).
To quote their CFO, with whom i've been interacting:

Horizonal blinds are relatively new to the smart home environment. Z-Wave just recently implemented a specific command class for Window Coverings which includes horizontal blinds. However, to date, there are no hubs that have implemented the new command class. Instead, all hubs are still using the Z-Wave Multilevel Switch command class to support window coverings. While this works well for roller shades and curtains, it is not ideal for horizontal blinds where 50% is fully open and as you are aware can cause problems for our implementation.

@tamorgen : Maybe I'm misunderstanding the specification. But according to page 159 of the specification https://raw.githubusercontent.com/zwave-js/specs/d6c115ec1c46dd1db9f3dca3fc2106df577fac9c/Z-Wave Device Class Specification.pdf it says:

It is not RECOMMENDED to use this generic device class for new devices. Instead, it is
RECOMMENDED to base devices of this category on the Multi-level switch generic device class with
the Multi-position Motor specific device class

To me that sounds like ZwaveJS is doing the right thing.

Also, the link you provided does not have a date on it.  I googled the same document, and the one's I found have specific dates listed on the first page of the PDF.
https://sdomembers.z-wavealliance.org/document/dl/638

@tamorgen : Maybe I'm misunderstanding the specification. But according to page 159 of the specification<EMAIL_ADDRESS>(raw) it says:

You are mixing up Device Types vs. Command Classes. The Window Covering Generic Device Class is discouraged, not the command class. The Window Covering Command Class is alive and well, a requirement for Window Covering Device Type for the Device Type V2 Specification.
Anyways, this is a wish list issue. The WC class is already specifically called out as requested. There's been no statement it would not be implemented. Hit the thumbs up in the comment and wait, or even better, implement it!

Thank you for your feedback.  I've voted for it.

Receiving MultiCommandCCCommandEncapsulation is supported since v10.10.0

Energy Production CC support is added in #5677

Hi, I've a Quby / Engie Meter Adapter
See here : https://devices.zwave-js.io/?jumpTo=0x0130:0x0002:0x0001:0.0
or
https://github.com/openhab/org.openhab.binding.zwave/blob/main/doc/quby/en00_0_0.md
And I receive this error when trying to get data from it:
The command class Meter Table Monitor is not implemented (ZW0303)


"az acr helm repo add" fails on "'NoneType' object has no attribute 'basic'"

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az acr helm repo add
Errors:
The command failed with an unexpected error. Here is the traceback:

'NoneType' object has no attribute 'basic'
Traceback (most recent call last):
  File "/Users/orsamuel/code/venvs/k8s-lab/lib/python2.7/site-packages/knack/cli.py", line 206, in invoke
    cmd_result = self.invocation.execute(args)
  File "/Users/orsamuel/code/venvs/k8s-lab/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 603, in execute
    raise ex
AttributeError: 'NoneType' object has no attribute 'basic'

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

az acr helm repo add --name {} --subscription {}

Expected Behavior
Environment Summary
Darwin-18.7.0-x86_64-i386-64bit
Python 2.7.16
Shell: bash

azure-cli 2.0.51 *

Additional Context

@orensam-msft Could you upgrade your az cli to the latest version? I cannot reproduce the bug in the version 2.0.70 and the latest version 2.0.71 while I do find that the version 2.0.51 has issues in AAD login.

Hi, upgrading to 2.0.71 did the trick. Thanks!


Run the integration tests on CI

Issue #, if available:
Description of changes:
Current CI only checks if the project is formatted and compiles. With this patch, the integration tests are also run
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Tests must fail due to #14

Since there have been no updates for a while for this draft pull request, please let me close it for now.


Hot exit took Untitled-2 as Untitled-1

VSCode Version: c2418f46d20464a893dc5db6449e9d79766eadc4
OS Version: macOS 10.11.6

This should be fixed by https://github.com/Microsoft/vscode/issues/14067#issuecomment-255542395


⚠ Secret Website 2 has degraded performance

In 1bc15d4, Secret Website 2 ($SECRET_WEBSITE_2) experienced degraded performance:

HTTP code: 200
Response time: 5404 ms

Resolved: Secret Website 2 performance has improved in 186b15c.


Byref analysis on record of functions FS0412 and FS0437

Errors FS0412 (A type instantiation involves a byref type. This is not permitted by the rules of Common IL) and FS0437 (A type would store a byref typed value. This is not permitted by Common IL) are reported if a record contains a function with a byref argument.
Using a type alias for the function type works
Repro steps
open System
open System.Buffers.Binary

module X =
    type Nope = {
        Funcn : ReadOnlySpan<byte> -> int
        }

    type FuncType = ReadOnlySpan<byte> -> int
    type Works = {
        Funcn : FuncType
        }

    let f = BinaryPrimitives.ReadInt32BigEndian
    let works = { Works.Funcn = f }
    let nope =  { Nope.Funcn = f }

Expected behavior
Both Nope and Works behave the same
Actual behavior
Nope produces FS0412 and FS0437
Known workarounds
Create a type alias like FuncType
Related information
.net standard 2.0
VisualStudio.15.Preview/15.8.0-pre.3.0+27825.0
Visual F# Tools 10.2 for F# 4.5   <IP_ADDRESS>.  Commit Hash: ac667ab00ebb5ef25ca1611ded25254b799877e0.

Yes, it should give an error on the type abbreviation. I suspect the check didn't strip type equations.

Milestone is 16.0 for now, but if we can get a quick fix in for 15.8 then I'll retarget

I'll look at this tomorrow.

I have a fix for this coming. It was pretty trivial.

Closing as completed.


Shared data content when used with FastAPI backgroundtasks and client.beta.threads.runs.create_and_poll

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
I have a FastAPI API that I have one endpoint in /execute. This endpoint creates a FastAPI backgroundtask to do a call to the assistants API. In here I make a request to client.beta.threads.runs.create_and_poll, there are no issues starting the request and getting the expected outcome. However, if another request comes in to the API, when the first request background task is still waiting for the create_and_poll, both requests will complete at the same time and the result out from create_and_poll will be based on the first request it received.
I am creating the OpenAI client in the backgrountask method to not use a global one.
I am a beginner in python and still trying to learn and understand how things are isolated, could this be a usage error or is there no thread isolation for the client.beta.threads.runs.create_and_poll method?
To Reproduce

Code snippets
No response
OS
Ubuntu
Python version
Python 3.10
Library version
openai 1.20

My instinct is that this is a bug in your code with how the background
worker is being used. I don't think it's likely this library would have a
concurrency bug of the nature you describe.
On Thu, Apr 18 2024 at 10:58 PM, Robert Karlsson @.***>
wrote:

Confirm this is an issue with the Python library and not an underlying
OpenAI API

This is an issue with the Python library

Describe the bug
I have a FastAPI API that I have one endpoint in /execute. This endpoint
creates a FastAPI backgroundtask to do a call to the assistants API. In
here I make a request to client.beta.threads.runs.create_and_poll, there
are no issues starting the request and getting the expected outcome.
However, if another request comes in to the API, when the first request
background task is still waiting for the create_and_poll, both requests
will complete at the same time and the result out from create_and_poll
will be based on the first request it received.
I am creating the OpenAI client in the backgrountask method to not use a
global one.
I am a beginner in python and still trying to learn and understand how
things are isolated, could this be a usage error or is there no thread
isolation for the client.beta.threads.runs.create_and_poll method?
To Reproduce

Code snippets
No response
OS
Ubuntu
Python version
Python 3.10
Library version
openai 1.20
—
Reply to this email directly, view it on GitHub
https://github.com/openai/openai-python/issues/1345, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFL6LURAUFSAKVK6UJW7C3Y6CB3XAVCNFSM6AAAAABGOMGTCGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TCOJXGU4DKNY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


LBRY for Raspberry Pi

Could we get the build system to build aarch64 & armhf binaries for Raspberry Pi users?

I think one user already tried this in the past, but it runs quite slow.

@belikor even on the Pi 4?
Do you know if there's anything special you need to do to build lbrynet on arm64?

lbrynet is a Python program, so you only need the Python interpreter, which works in the Raspberry Pi without issues.
Exactly how the Python code becomes a single executable, I don't know. But if you want to run it, you only need to follow the instructions in the INSTALL.md from lbry-sdk.
Basically
python3 lbry/extras/cli.py start


Refactor symbol resolve API

Introduce ReferenceResolution to avoid nesting to many non-nominal
types.

bors r+


Feature: Add share button in Notes which creates a unique link for anyone to view Note

Issue Template
Description

Add share button in Notes which creates a unique link for anyone to view Note

@yashmandi : please assign it to me


Keep logical foreign keys on ERD

We can drag-n-drop tables from other schemas and databases to ERD for the logical foreign keys creation. But we need to keep these linked new tables and links after the reload and reconnect.
Tables from other sources (schemas and databases) have to be marked by another color border. If we have tables from a few different data sources each new data source has to have own color.

Verified


Version 2.7 can see the UI of version 2.5

Describe your issue here
I am using rancher2.7 version, using docker single-node deployment, I do not know how to access a link, and found that it is very similar to the UI of the 2.5 series

Useful
Info

Versions
Rancher v2.7.1 UI: v2.7.1-dev

Route
undefined

2.5.x UI:  https://xxxx/p/c-xxx:p-xxxxx/workloads
2.7.x UI : https://xxxx/dashboard/c/xxxx/explorer/apps.deployment


suggesstion: use v1.1.1 format when release version

ping @taowen is it possible to add a new v1.1.1 tag to assist projects that switched to go mod (which requires the v prefix)?

Also experiencing build errors from gofmt output on a project which depends on this library after switching that project to use go modules.

@taowen ptal

Ping  to add v and go.mod.


Crash when "reset values to previous state" while editing agricultural tower

Problem Description
The mod Factory Planner (2.0.2) caused a non-recoverable error.
Please report this error to the mod author.
Error while running event factoryplanner::on_gui_click (ID 1)
factoryplanner/ui/dialogs/machine_dialog.lua:118: attempt to index field 'limit_textfield' (a nil value)
stack traceback:
factoryplanner/ui/dialogs/machine_dialog.lua:118: in function '?'
factoryplanner/ui/base/modal_dialog.lua:377: in function 'handler'
factoryplanner/ui/event_handler.lua:169: in function <factoryplanner/ui/event_handler.lua:121>
I know there is nothing to edit in this window so you will never need to press it, but it being located in the top left traditionally reserved for X to close in most other windows (outside this mod) I found myself accidently pressing it and crashing out. its is consistent 10/10 attempts with multiple recipes. I assume its attempting to reset something that has no data to index and there is no exception to that in the function.
Reproduction
set some sort of Gleba factory chain using the agricultural tower
Edit agricultural tower machine
click to reset values to their default state
click Are you sure
Say F to your unsaved progress

Appreciate the report, this is fixed for the next release!


fix(#1574): '@angular/http' module not exist

I signed it!

@jelbourn whether there's a better solution or not as a temporary fix it's better than what's already published. I will make that fix you mentioned above and I suggest in the near future some of you will come up with a better solution.

LGTM, thanks!


testing for rebuild

Test-run for rebuild of keeper-jespen

This is an automated comment for commit bdcd23c2d4f0e86938afd2ab8da84fc3fb670985 with description of existing statuses. It's updated for the latest CI running
The full report is available here
The overall status of the commit is 🟢 success
Check nameDescriptionStatus
Push to DockerhubThe check for building and pushing the CI related docker images to docker hub🟢 success


Invalid documentation link for build-info-extractor-jar

Describe the bug
When performing maven build actions with the jf cli tool, when the build-info-extractor jar isnt already present, there is a log message that refers to documentation around the behavior of this jar being downloaded.
However, that URL does not exist (anymore?) and redirects to here, which doesn't contain any relevant information. The referred link also is not available on the wayback machine.
The link over here in the documentation project is also old.
Current behavior

19:02:37 [Info] The build-info-extractor jar is not cached locally. Downloading it now...
You can set the repository from which this jar is downloaded.
Read more about it at https://jfrog.com/help/r/jfrog-cli/downloading-the-maven-and-gradle-extractor-jars

Reproduction steps
No response
Expected behavior
A link to valid documentation, or even a PR so that I can better understand how to configure a CI pipeline to not need public internet access.
JFrog CLI-Core version
dependency of jf cli
JFrog CLI version (if applicable)
jf version 2.64.1
Operating system type and version
jfrog/jfrog-cli-v2-jf:2.64.1
JFrog Artifactory version
No response
JFrog Xray version
No response

Hi @holysoles ,
Thanks for letting us know. The documentation is available here.

@RobiNino Thank you! I can close this issue if appropriate based on the PR getting merged

The fix is merged and released, you can close, thanks


Fix up empty namespaces JSON unmarshalling error

At present, if the namespaces endpoint returns 0 bytes, there is still
an attempt to unmarshal it, resulting in an unexpected end of JSON input
error being passed down.
Signed-off-by: Paul Mundt<EMAIL_ADDRESS>
Description

Motivation and Context

[ ] I have raised an issue to propose this change (required)
[ ] My issue has received approval from the maintainers or lead with the design/approved label

How Has This Been Tested?

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Hi thank you for your interest.

How Has This Been Tested?

Please give before and after and versions being used so we can reproduce the error. Thank you.

I hit this while developing a new connector (using the MQTT connector as a base of reference), but the same issue occurs with the in-tree tester, as well (the build of which is currently broken for me, for other reasons).
Before the fix:
$ go run main.go
2019/12/14 13:41:33 Topic: country-change	Broker: http://adaptant-lab-02:8080
INFO[0000] New 'country-change' subscription: 1b9fdd6f-8b8a-4a66-b0f0-62276389d3b6
2019/12/14 13:41:33 unexpected end of JSON input
exit status 1

After the fix:
$ go run main.go
2019/12/14 13:44:40 Topic: country-change	Broker: http://localhost:8080
INFO[0000] New 'country-change' subscription: fb25150c-ee24-49a0-b7ca-7f8900fbe9e0
2019/12/14 13:44:52 Invoking (http://adaptant-lab-01:8080) on topic: country-change, value: {
	"driverId": 1,
	"entering": "at",
	"leaving": "de",
	"timestamp": "2019-12-12T20:11:12+01:00"
}
2019/12/14 13:44:52 Invoke function: figlet
...

It took a while to figure out exactly where this JSON response was coming from, but I ultimately tracked it down to the namespace unmarshalling in the types package. A curl request on the endpoint confirmed that this was returning 0 bytes, so we've made that check explicit prior to any unmarshalling attempts, which has resolved the matter.
As far as the FaaS Gateway - I've used the latest version from git (as of yesterday) and simply ran the the deploy_stack.sh with no further modification.

So are you using swarm? @Waterdrips can you check faas-swarm is updated correctly to return an empty set and that the stack is up to date? Cc @LucasRoesler

What's your connector for btw?

Released in https://github.com/openfaas-incubator/connector-sdk/releases/tag/0.5.5

So are you using swarm? @Waterdrips can you check faas-swarm is updated correctly to return an empty set and that the stack is up to date? Cc @LucasRoesler

Yes, this was on Docker Swarm.

What's your connector for btw?

We have developed a connected car services platform and a number of companion apps (https://knowgo.io), which we are gradually preparing to launch, and are now simply trying to get some of the interfacing components out of the way (OpenFaaS being one of them).
Our current pub/sub interface exposes adaptation events for both our risk and compliance monitoring, as well as for service providers using the platform that need to trigger run-time reconfiguration of either the application or the service deployment. We are likely to expose driver-specific events in this way as well but are more likely to route these through something like beehive or IFTTT (both of which are being tested at the moment).
We are ultimately planning on using OpenFaaS for triggering Edge-based functions within the vehicle, federating this together with fleet and cloud-level OpenFaaS instances, and then begin looking at cross-instance orchestration and deployment (this is largely a byproduct of work we are doing in the SODALITE H2020 project).
I'll send you a better write-up of this for the ADOPTERS document when I have some time.

https://github.com/openfaas/faas-swarm/pull/63
@alexellis ^ PR for the fix in faas-swarm here

Thank you both


does this library off the same context.trace.fetch

does this library off the same context.trace.fetch as the original library [workers-honeycomb-logger] ???
I need to wrap the 'fetch" calls  (https://ross-hagan.com/blog/remix-cloudflare-and-honeycomb)

One of the main selling points of creating this library based on the OTel spec and the new AsyncLocal functionality in the Workers is that we don't need to treat fetch any differently. Just use the regular fetch and the traces will automatically show up :)


Why not add fields to user schema explicit way instead of using pow_user_fields() function?

Ok, why? I like that Ecto is enforcing defining schema with all fields. I come from the Rails world where everything is hidden. And when I look at that newly generated schema by pow I'm missing the fields it generated for me. Because of it, the explicitness has disappeared and I feel like I'm back in Rails.
I know that counter argument could be that routes are done similar way, but there I can use mix phx.routes and then I can see everything.
Wdyt?

It's a good point.
The biggest reason is to make it simpler to enable extensions:
use Pow.Extension.Ecto.Schema,
  extensions: [PowExtensionOne, PowExtensionTwo]

schema "users" do
  pow_user_fields() # This will also have fields from PowExtensionOne and PowExtensionTwo

  timestamps()
end

Furthermore, the pow_user_fields/0 macro has a similar purpose to the timestamps/1 macro, and the implicit :id field, as there will be a third party library that populates these fields. By letting developers manage the fields right away (and in particular virtual fields that definitely should stay virtual) it could make it more difficult to ensure everything works as expected.
Instead I prefer that developers expand on the fields themselves when necessary. It's possible to define some fields that Pow will use, and let the rest be handled by the macro:
schema "users" do
  field :password_hash, :string, source: :encrypted_password

  pow_user_fields()

  timestamps()
end

This is how I like Pow to be most of the time. Rather than writing out everything from the start explicitly, Pow makes it possible to extract certain parts that will be explicit, and easy to test. Templates are another example of this thinking.
In case you want to see how it looks under the hood:
https://github.com/danschultzer/pow/blob/356a610c34f44e7f37bf71e455b0efe4e1e2a11d/lib/pow/ecto/schema.ex#L146-L162
https://github.com/danschultzer/pow/blob/356a610c34f44e7f37bf71e455b0efe4e1e2a11d/lib/pow/ecto/schema/fields.ex#L7-L12

Another way to think about it is that the changeset is handled in the Pow library, so the fields should be handled by it too. But it's hard to find the right balance, so please let me know if you have any suggestions how this could be improved.
I definitely want to make Pow as explicit as possible, and let Pow documentation help developers think in that way too 😄

Hmm, thanks for the answer. I see your point, if someone is gonna use pow they may not need that information at all, because they trust pow to take care of it. After reading it I guess I understand my problem a bit more. That timestamps/1 analogy helped. I think what I struggled with was an easy way to find out (preferably from docs) what fields that macro generates.
Take a look at timestamps/1 docs. It lists what kind of fields there will be generated.
Maybe just adding proper information via docs, would be enough.
Btw. Really liked the code so far! There are a lot of things taken care of and it makes it pretty easy to adjust it to my needs, at least so far :smile:

That's great to hear! Don't hesitate to write about any other things in Pow that's confusing or could be improved. It helps a lot :smile:
I've opened #96 to improve the docs on pow_user_fields/0 and Pow Ecto fields.

Looks very cool! Thanks for such a quick response. If I get any more feedback or questions, I will surely open an issue.


FIX mark with attrs are missing in nested mark rule parse

Fix missing case in #51

I did not notice that Mark create without attrs will directly return same instance which case Mark being replace in applyPennding.
Prev commit didn't cover this case.
I am realy sorry for my carelessness.

Thanks for following up. Merged.


Run tests automatically on new PRs

and perhaps show that they pass on the master branch too.

https://github.com/atn832/cloud_firestore_mocks/pull/39#pullrequestreview-374891408

I wonder if eventually, it would be possible to run the full battery of unit tests from cloud_firestore_mocks/test/* on the mock library and emulator/real firebase as part of #19.

Nice idea. The challenge is resetting the state of emulator and real firebase. For cloud_firestore_mocks, reseting its state is easy; just creating a MockFirestore instance. For emulator and real firebase, I don't know how to reset their state (collection and documents) via Dart.

Reference docs:

https://flutter.dev/docs/deployment/cd#other-services
https://github.com/marketplace/actions/flutter-action


TakeSnapshotView does not work as expected

I am experiencing several issues with the TakeSnapshotView:

AttributeGraph Cycle Detection
=== AttributeGraph: cycle detected through attribute 33664 ===

State Modification During View Update (Xcode 14+).
In the new Xcode version (Xcode 14+), the line image.wrappedValue = newImage in TakeSnapshotView causes the following error:
Modifying state during view update, this will cause undefined behavior.

Intrinsic Size Propagation:
The TakeSnapshotView does not propagate the intrinsic size correctly. To clarify, consider the following view hierarchy:

GeometryReader { proxy in
     ScrollView(showsIndicators: false) {
         Text("Some long\n random text").screenshotOnAppear(to: $captureImage)
             .fixedVerticalSize()

         Color.red.height(100)
     }
}

In this case, I want the Text view to propagate its size to the ScrollView.

@denandreychuk can you upload an encapsulated project I can run to reproduce this? I'm low on bandwidth, doing so would help me debug + close this issue faster.

@vmanot sure!
Hope this helps
https://github.com/denandreychuk/TakeSnapshotViewIssue

Also it would be really nice, if we could capture a screenshot of the view that isn't visually displayed on the screen.

@denandreychuk I think I've fixed the runtime warning. I don't have the bandwidth to investigate + fix the height issue atm, but one good place to start could be conditionally using ImageRenderer on iOS 16 to mitigate it.

Hey @vmanot,
Thanks for the quick fix on my issue! You rock! 🚀
I'll be able to check the updates to TakeSnapshotView in a few days and investigate the height issue. If I find a solution, w8 for a PR :)

Closing this for now as it's been inactive for a bit, please feel free to reopen if there's an update.


Upgrade to TypeScript 2.8

Opening issues for the 1.0 milestone. TypeScript 2.8 seems like a good target: https://twitter.com/typescriptlang/status/974394621601095680. This needs to happen in a semver major, so 1.0 seems like the right time to do this.

Fixed by #140.


allow object oriented usage

:wave: hey there I'm working on a Sass like preprocessor and was hoping to use Khroma for color logic. The issue tho is that i really need to be able to create reusable Color objects to e.g. store as the values of a variable declaration. THe problem here is that Khroma reuses the same Channels instance for all work so it doesn't seem possible to store an intermediate representation of the color channels since any other color will mutate the same data. I see why this is the case, but any chance of getting a version of the API that allows creating reusable Color instances from strings?

Sure, I could add a way to output non-reused Channels instances, although this is a bit of an implementation detail and I'd prefer not to expose it if possible, why do you need this? Is this purely for performance reasons?

Maybe a nice quick way to fix this would be to add a "clone" method to the "Channels" class. However you could potentially already clone Channel classes already manually.


Add Rohde-Schwarz vendor MIBs

Hi!
I need to upload the following MIBs from Rohde-Schwarz:
TLU9_mibs.zip
Thanks.

Hello @Odrork, the mibs were released with mibserver v1.15.12.


New tagged release?

Any chance we could have a new tagged release please, or confirmation that the recommended way to install is to do so from source?
https://github.com/geerlingguy/ansible-role-mailhog/issues/3 has been waiting on a MailHog 0.2.0 tagged release since Oct 2015, and as I'm waiting on the change in that issue, figured I 'd ask :)

I'm in the same boat. Waiting for a fresh release tag as well.

Done, sorry it took so long!

Thanks Ian, and thanks for writing and maintaining such a fantastically useful piece of software!


Nature of Code track video preview images

This is a small thing but I'm adding links from natureofcode.com to here (see: https://github.com/nature-of-code/noc-book-2/pull/985) and noticed that the NOC videos are just using the default site image and it would be nice to update them at some point!

Non-challenge videos rarely have distinct thumbnails on the site, probably because they are never displayed on the site and only when sharing links?
We could fix this pretty easily by downloading the YouTube video thumbnails to the codebase. They have a predictable URL structure on their CDN: https://i.ytimg.com/vi/{id}/maxresdefault.jpg
ex: https://i.ytimg.com/vi/Rob0pbE7kks/maxresdefault.jpg

Actually, since #1310, the image should fall back to the track's cover image, if the video itself doesn't gave a thumbnail.
For example: Video from ml5 track, doesn't have any thumbnail image associated with it, falls back to ml5 track's cover image.

However, there is an issue with this specific video (Vector Math, and a few others from NOC), these videos actually have a thumbnail defined (probably an artifact from the design/testing phase), which is in fact the last fallback placeholder image for videos, which is in this case being used for the preview

cc @fturmel

Ah nice to know they'll fall back to the track image properly. I also noticed these placeholder thumbnails were manually copied and not the actual fallback, a bit strange.
So, I guess we have some options for the preferred behavior:

Always use the track image by deleting the non-challenge video images currently in the repo
Use the YouTube thumbnails for all non-challenge videos (still falls back to track images if any would be missing in the future), essentially #1646
A mix of both? We could start from option 2 and selectively delete images for the videos that should use the track image

I do wonder, if adding all these images in the repo, even though they are nowhere used on the site, only for link preview purposes is worth it ... Could we directly set the YouTube's thumbnail url as the og:image during build time?

I do wonder, if adding all these images in the repo, even though they are nowhere used on the site, only for link preview purposes is worth it ... Could we directly set the YouTube's thumbnail url as the og:image during build time?

That did cross my mind, but the high res thumbnail is not always available so we have to check for 404s and downgrade accordingly to find the best available match. See https://stackoverflow.com/questions/18029550/fetch-youtube-highest-thumbnail-resolution

Thanks for the work on this! I'm fine either way, this isn't really a high priority for the reason @fturmel states that they aren't ever displayed on the site. But this has revealed some helpful info for what is my priority right now over here! https://github.com/nature-of-code/noc-book-2/pull/985

I have created #1648 as an alternative to #1646, corresponding to "Option 1" as discussed above.
Unrelated, here is a demo of using directly the maxres yt thumbnail url as the meta image: (using an extension to preview the image) - if video defines a thumbnail image then that is used (i.e. for challenges), otherwise the youtube thumbnail is used (i.e. for track videos)
https://github.com/CodingTrain/thecodingtrain.com/assets/59444569/07153d11-8de9-4348-a734-a3a32ef56968

@dipamsen Nice! It would be a great approach if the maxres image was more predictably available.
Here's an example of a 720p video without the maxres, we have to fallback to a lower resolution image:
https://thecodingtrain.com/tracks/git-and-github-for-poets/git/1-fundamentals/8-pages
https://i.ytimg.com/vi/bFVtrlyH-kc/maxresdefault.jpg <- 404
https://i.ytimg.com/vi/bFVtrlyH-kc/hqdefault.jpg <- best available

Thanks for all of this! I prefer having the specific YT thumbnail for the shared image but I don't mind having a low-res one or if it's not available just using the track thumbnail. In this case would it make sense to just maxres thumb as the preview?
Another pro is that I am slowly updating all YT thumbnails in which case I wouldn't have to remember to re-run the import script whenever a thumb changes.
The con, however, is that it's nice to have a back-up / reference of all of my YT thumbnails here in this repo.
Not sure why I am having so much trouble deciding on this! Any votes?

I have some additional info:

sddefault is actually larger than hqdefault, here's the whole list of what is available with dimensions:

const thumbnailResolutions = [
  'maxresdefault', // 1280x720 - may not be available for 720p videos
  'sddefault', // 640x480
  'hqdefault', // 480x360
  'mqdefault', // 320x180
  'default' // 120x90
];

I tested all 444 videos on the site and the sddefault image is always available. This means that referencing the image from the YT CDN directly as demo'd by @dipamsen could reliably work and might be the best/simplest approach for the NoC site as long as it's referencing sddefault resolution or lower. (I will comment in the NoC PR in a minute)

Is using a custom proxy server which returns the highest available version of the image, viable? Here's one I just created:

https://yt-thumb.deno.dev/Rob0pbE7kks.jpg (has maxres)
https://yt-thumb.deno.dev/bFVtrlyH-kc.jpg (does not have maxres, uses sd)

Is using a custom proxy server which returns the highest available version of the image, viable? Here's one I just created:

https://yt-thumb.deno.dev/Rob0pbE7kks.jpg (has maxres)
https://yt-thumb.deno.dev/bFVtrlyH-kc.jpg (does not have maxres, uses sd)

Yes, that's definitely a possible approach. Netlify Edge Functions are just white-labeled Deno Deploy IIRC so the demo is on point. I think it's also possible to cache the response on the CDN with the proper response headers to reduce the amount of function invocations.
Could we even just turn this into a redirect service instead? HEAD requests to the YT CDN to determine the highest res available, then respond with a 302 to the YT CDN URL directly instead of eating the bandwidth cost?
Some considerations:

It's ripe for abuse if we don't set CORS response headers and make sure we only respond for actual Coding Train videos
It might be more infra cost, but if we tune things right it's probably negligible - even perhaps cheaper if YouTube actually serves all of it with redirects?

Thanks for all of this discussion! I think we should (a) delete the non-challenge video images currently in the repo (which will cause the fallback to be the track image and (b) use video thumbs when available either via the YT CDN or @dipamsen's proxy server or whatever you both recommend! I think I should go ahead and merge #1648 and then we can proceed from there? What do you all think?

Works for me! I just closed #1646.


Fix PDOStatement::fetchColumn() return type

The return type should be mixed, not string. In particular, it returns false when there are no more rows.
See: http://php.net/manual/en/pdostatement.fetchcolumn.php

It's better to use string|bool as a return type

I agree, but it's also good to keep the stubs in sync with the PHP documentation, and they document it as mixed.
This would need to be submitted to the PHP docs as well in this case!

Unfortunately, PHP docs are often inaccurate and so there are many cases in our stubs where they don't follow the docs exactly. In this particular case we still believe that string|bool is better.

Actually, the return type is not string|bool. It can return null for a nullable column.
And when using the mysqlnd driver, it can return native types such as integer as well. Try the following:
$pdo = new PDO('mysql:host=localhost', 'root', '');
$pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
$pdo->setAttribute(PDO::ATTR_STRINGIFY_FETCHES, false);
var_dump($pdo->query('SELECT NULL')->fetchColumn()); // NULL
var_dump($pdo->query('SELECT 1')->fetchColumn()); // int(1)

So it would be more something like @return string|bool|null|int, so I guess the mixed type in the PHP documentation makes sense here.

OK, let's make it mixed


[Infra][Java] Repackaging Java library under the Apache namespace

Proposed changes

packaging from com.alibaba to org.apache
update the code directory structure from com/alibaba/graphar to org/apache/graphar
update related document

related to issue #427

@lixueclaire @Thespica This pull request may needs your review.

Try to replace Java_com_alibaba_graphar_arrow_ArrowTable_1Static_1cxx_10x58c7409_nativeFromArrowArrayAndArrowSchema by Java_org_apache_graphar_arrow_ArrowTable_1Static_1cxx_10x58c7409_nativeFromArrowArrayAndArrowSchema in ffi code
and replace jni_com_alibaba_graphar_arrow_ArrowTable_Static_cxx_0x58c7409.cc by jni_org_apache_graphar_arrow_ArrowTable_Static_cxx_0x58c7409.cc in cmake file.

Try to replace Java_com_alibaba_graphar_arrow_ArrowTable_1Static_1cxx_10x58c7409_nativeFromArrowArrayAndArrowSchema by Java_org_apache_graphar_arrow_ArrowTable_1Static_1cxx_10x58c7409_nativeFromArrowArrayAndArrowSchema in ffi code, the file name also needs to be change from jni_com_alibaba_graphar_arrow_ArrowTable_Static.cc to jni_org_apache_graphar_arrow_ArrowTable_Static.cc.
and replace jni_com_alibaba_graphar_arrow_ArrowTable_Static_cxx_0x58c7409.cc by jni_org_apache_graphar_arrow_ArrowTable_Static_cxx_0x58c7409.cc in cmake file.

The CI is passed. Please review again, thanks~


Monkepyatch TQDM Progress Bars when Streamed over the network

TQDM Progress Bars streamed over the network do not display until a \n is written. In effect, this caused the progress bars not to show, until they were finished. This behavior defeats the purpose of progress bars. This issue has been documented here: https://github.com/tqdm/tqdm/issues/1319
This PR fixes this issue by attempting to detect whether we are in a K8S environment, and if so, then automatically write \n each time the progress bar is updated.

Wait, this just repeats the progress bar multiple times, right? I'm exploring if we should just use a FileLogger instead to solve the same problem with a better UX, and then explore a deeper solution next week.
Additionally, can we do sys.stdout.isatty() to detect whether we need to monkey patch? This would also create newlines when using interactive instances, or am I missing something?

Wait, this just repeats the progress bar multiple times, right? I'm exploring if we should just use a FileLogger instead to solve the same problem with a better UX, and then explore a deeper solution next week.
Additionally, can we do sys.stdout.isatty() to detect whether we need to monkey patch? This would also create newlines when using interactive instances, or am I missing something?

Correct, this would repeat the progress bar multiple times. sys.stdout.isatty() is true when doing mcli run, so unfortunately cannot do that. Let me include an example output.

Going to close this PR for now and see if we can fix the underlying issue with log streaming.

Hello @ravi-mosaicml , do you know if there are any updates on this ?


Saying its the "fastest" inheritance library is disingenuous

Its certainly faster than some, but calling it the fastest either means you're not trying to be truthful, or you just didn't do your research. Here's a perf test someone that knows how to write them did that includes augment: http://jsperf.com/js-inheritance-method-calls/7

At the time that I wrote this library it was indeed the smallest and the fastest. Now, I don't really care whether I'm being 100% truthful. The fact of the matter is that my library is pretty fast, and considering the size of my library (30 source lines of code, 10 if you don't consider the boilerplate) I would say that it is worth every byte.
Now compare that to the size of the ClassManager library created by the “someone” who wrote the tests that you referenced. It's 962 source lines of code. That's 32 times as big as augment and in terms of speed it is only twice as fast. Hence, per byte my library is 16 times faster.
Finally, when it comes to writing real world code I don't think object creation or method call performance would ever be a bottleneck. The real bottleneck would be related to the problem that you're trying to solve and the algorithm that you use. Trying to optimize classes is usually just a waste of time because the payoffs are very small.
To quote Donald Knuth, one of the most famous programmers:

Programmers waste enormous amounts of time thinking about, or worrying about, the speed of noncritical parts of their programs, and these attempts at efficiency actually have a strong negative impact when debugging and maintenance are considered. We should forget about small efficiencies, say about 97% of the time: premature optimization is the root of all evil. Yet we should not pass up our opportunities in that critical 3%.

Optimizing classes in JavaScript is the perfect example of premature optimization.
In fact, I don't even use augment in my code anymore even though its just seven lines of code. Instead, I use the following defclass function to create classes which don't inherit from other classes:
function defclass(prototype) {
    var constructor = prototype.constructor;
    constructor.prototype = prototype;
    return constructor;
}

// Usage:

var Person = defclass({
    constructor: function (name) {
        this.name = name;
    },
    greet: function () {
        alert("Hello " + this.name + "!");
    }
});

// Object creation and method call:

var person = new Person("John Doe");
person.greet(); // Hello John Doe!

If I ever need to inherit from another class (which I usually don't) then I make use of the following extend function:
function extend(constructor, keys) {
    var prototype = Object.create(constructor.prototype);
    for (var key in keys) prototype[key] = keys[key];
    return defclass(prototype);
}

// Usage:

var Man = extend(Person, {
    constructor: function (name) {
        this.name = name;
    }
});

// Object creation and method call:

var man = new Man("John Doe");
man.greet(); // Hello John Doe!

In practice, I have never needed to call overridden base class methods. Ever. In fact, if I had to make a chart of all the object-oriented operations I had to use then it would look like this:

defclass 99% of the time.
extend 1% of the time.
super 0% of the time.

For three lines of code, defclass is a very useful function. No premature optimization. No need to read the documentation of libraries like ClassManager. I can directly focus on the task at hand and optimize those things that really need optimization.
P.S. The only reason that I still maintain augment is because other people use it. Otherwise, I really don't care about it.


docker run -v does not recognize absolute path (seems to add leading whitespace)

Description

I am trying to run my docker container with a mounted volume. I used the jbpm-workbench-showcase-image and tried using the persistent git-repository as described on it's DockerHub-page. Therefore I used the
docker run [...] -v /home/***/jbpm-workbench-docker/wb-git:/opt/jboss/wildfly/bin/.niogit [...]
command to start my container.
As a result, the docker-daemon complained about being unable to execute the create command, because " /home/***/jbpm-workbench-docker/wb-git" contains invalid characters for a local volume name, even though it is an absolute path on my host system.
Steps to reproduce the issue:

Pull the jbpm-workbench-showcase-container (6.5.0.Final-1)
Run the container using docker run -p 8080:8080 -p 8001:8001 -v /home/***/jbpm-workbench-docker/wb-git:/opt/jboss/wildfly/bin/.niogit -d --name jbpm-workbench jboss/jbpm-workbench-showcase:latest

Describe the results you received:
When running the command shown above, I receive to following error message:
docker: Error response from daemon: create  /home/***/jbpm-workbench-docker/wb-git: " /home/***/jbpm-workbench-docker/wb-git" includes invalid characters for a local volume name, only "[a-zA-Z0-9][a-zA-Z0-9_.-]" are allowed. If you intented to pass a host directory, use absolute path.
See 'docker run --help'.
json: cannot unmarshal array into Go value of type types.ContainerJSON

Describe the results you expected:
I expected the container to start up normally and mount the specified folder into the given path of the container.
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
Client:
 Version:      1.13.1
 API version:  1.26
 Go version:   go1.7.5
 Git commit:   092cba3
 Built:        Wed Feb  8 06:50:14 2017
 OS/Arch:      linux/amd64

Server:
 Version:      1.13.1
 API version:  1.26 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   092cba3
 Built:        Wed Feb  8 06:50:14 2017
 OS/Arch:      linux/amd64
 Experimental: false

Output of docker info:
Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 13
Server Version: 1.13.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 15
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: aa8187dbd3b7ad67d8e5e3a15115d3eef43a7ed1
runc version: 9df8b306d01f59d3a8029be411de015b7304dd8f
init version: 949e6fa
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.4.0-62-generic
Operating System: Ubuntu 16.04.2 LTS
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 5.825 GiB
Name: marl
ID: FAKJ:CSC6:ZI4P:5XKQ:4PJL:K7ZV:7LSZ:NNQ4:7GJQ:W6MN:DI4L:LCUP
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Additional environment details (AWS, VirtualBox, physical, etc.):
Docker runs directly on an Ubuntu 16.04 host (no additional virtualization)

Can you give a simple example, with hopefully without a redacted user name that reproduces this issue?

I failed to reproduce the error on a simpler case. It looks like there is another problem in my bash-script which produced the docker run command. Hard coding all the paths on the command-line actually works as expected.
Sorry to bother you.

Thanks, do let us know if you can reproduce later!


admin/deposits/btc URL give error when run in production using passenger and nginx

When i run in production admin/deposits/btc URL give error when run in production using passenger and nginx
i have got following error log in nginx logs
NoMethodError (undefined method end_with?' for nil:NilClass): App 29004 output:   app/routers/crud_router.rb:19:in action'
App 29004 output:   app/routers/admin_deposits_router.rb:9:in `call'
can anyone help in this regard while this work fine when run using puma

But i am getting this error on puma
undefined method `email' for nil:NilClass

13 14 15 16 17 18
<% end %>     <% t.column 'Member email' do |x| %>       <%= link_to x.member.email, url_for([:admin, x.member]), target: '_blank' %>     <% end %>     <% t.column :amount do |x| %>       <%= x.amount %>

Hi,
I fixed it
In app/routers/crud_router.rb
replace the env['REQUEST_PATH'] with env['REQUEST_URI']

@prashantomer no your steps not help me to fixed!
I was struggling with a similar type of problem and found that by changing config.consider_all_requests_local to true in config/environments/production.rb, I could get the error message to display in production and see what the actual problem was. Obviously, change it back once you've fixed the problem.

@prashantomer
it worked for me, thanks


Uncaught rejected promise when attempting to fetch flags while offline

Hi there 👋 I'm seeing an uncaught rejection when calling Flagsmith.init while offline.

It seems to be coming from this call:
https://github.com/Flagsmith/flagsmith-js-client/blob/f2c29640991df6428c2603d98da7f847276dd4fd/flagsmith-core.ts#L447-L449
It doesn't cause any real issues as flags are still resolved correctly from local storage, and since the user is likely offline when this happens, it may not affect our telemetry. However, since it's not an error that can be caught in user land, perhaps it'd make more sense to handle the rejection and only reject from init() if the cache fails.
Please let me know if you need more details, thank you!

Hi @frankieyan, thanks for raising this. I think your suggestion here makes sense

handle the rejection and only reject from init() if the cache fails

Would you be willing to submit a PR for this?


Multiplexing Temperature Inputs

Test the CD405 in combination with an STM32 in order to multiplex the inputs and have room for expansion, need be.

https://www.ti.com/lit/ds/symlink/cd4051b.pdf?ts=1695018939057&ref_url=https%253A%252F%252Fwww.google.com%252F

TODO

Get a stable supply (probably Starelec or Mouser, depends which is cheaper)
Test functionality with something simple at first (maybe just applying a voltage) in combination with C code
After code can reliably switch between inputs asynchronously , try hooking it up to LM35DZs
If all tests pass, design circuit for PCB


Pass data to updating/updated methods

Is your feature request related to a problem? Please describe.
Less a problem than an inconvenience. With components that contain a lot of properties you can end up writing many identical/similar updatedFoo methods.
Describe the solution you'd like
I would like the generic updated / updating methods in a livewire component to receive data, so that logic can be performed conditionally depending on the name or value of the updated property
Describe alternatives you've considered
The updatingFoo and updatedFoo methods already receive $value so you can work round this just by writing a specific method for each property
Additional context

Example (pseudo-code)

public function updated($data)
{
    // ... do some validation.

    $this->model()->persistToDatabase($data['name'], $data['value']);
}

This has been merged and added to the framework (will be in the next release):
public function updated($name, $value)
{
    // ... do some validation.

    $this->model()->persistToDatabase($name, $value);
}


DatePicker executing onDismiss even on OK click

DatePicker is executing onDismiss event even on clicking OK button. For some strange reason, this is only happening on my production environment and not on my local. The only difference between the two env's is uglification using webpack uglify plugin.
However, I tried generating bundle locally and it works fine. Its just my prod env which is executing onDismiss event on OK click. Here's my code
<DatePicker
     ref="eventDate"
     style={{display: 'none'}}
     shouldDisableDate={(date) => {
            return (moment(date).diff(moment().startOf('day'), 'days') < 0)
     }}
     onDismiss={() => this.onEventCalendarDismiss()}
     onChange={(event, date) => this.setState({eventDate: date})}
/>

so, when I select a date and click ok, it executes onChange method, but also, onDismiss method, which I don't want it to do.
Material-ui version : 18.0
webpack version : 2.5.1

I think that this behavior was introduced by #6764.

@oliviertassinari If that's required, then is there a way where I can handle cancel/dismiss and ok clicks independently? I have a checkbox to uncheck if the calendar is dismissed and upon clicking ok, it should remain checked. And this behaviour was handled by onDismiss and onChange respectively.

Did you tried to use the onAccept callback? Eww, looks like we did the opposite with the TimePicker 🙈 #5213.

Haven't tried onAccept as I don't see it in the documentation. So, you mean onChange should be replaced with onAccept?

DatePicker isn't accepting onAccept method. I get an unknown-prop error.

@oliviertassinari I've reverted my material-ui dependency to 18.0 from 18.1 and that works for me.
Will be waiting for a fix in the new release.

Yes, indeed, onAccept isn't exposed. It's forwarded with a onChange event. The real question is what behavior do we want. The current one of the DatePicker or the TimePicker. People will want to change it based on their custom use case. I think that we should keep the DatePicker one as the change value/dismiss of the Dialog can potentially be two independent events. It's conceptually simpler and unlock the possibility to propose users to change the value with an input and no only the calendar interface.

Exactly. Independent events will make things a lot easier.

Seems like a bug as there is currently no way to do an action only when the user cancels.

@nihaux point is interesting. I think that we should be reverting #6764 PR of @NewOldMax.

@oliviertassinari

onDismiss Callback function that is fired when the Date Picker's dialog is dismissed.
So after clicking on OK button DatePicker is logical dismissed, right? Maybe we should have another one event like onClose or something...
As you said, people always want to do something that you unexpected (like, my case requires onDismiss when DatePicker is closed by any reason, and case of some person - don't)

Maybe we should have another one event like onClose or something...

I agree, that's how I think we should be handling the problem regarding the next branch where we focus our effort. But regarding the master branch where we try to stabilize things, I think that reverting would be the best local optimum.


How to return null with Illuminate\Http\Resources\Json\JsonResource?

I want to return null if model has some invalid conditions, but it is always return [].For example,
$productTemplate['gift'] = new GiftTemplate($gift);
use Illuminate\Http\Resources\Json\JsonResource;

class GiftTemplate extends JsonResource
{
    public function toArray($request)
    {
        if ($this->expired_at < now() || $this->is_disabled || $request->user()->age > 20)  {
            // I want to return null here for $productTemplate['gift'],
            // but it is always return [] when resource is null in JsonResource@toArray.
            // The [] is not friendly for frontend.
            return null;
        }

        $this->setVisible([
            'id',
            'name',
        ]);

       $data = parent::toArray($request);
        return $data;
    }
}

so,how to reurn null with JsonResource? Thanks for your answer.

Hi there,
Welcome to Laravel and we are glad to have you as part of the community.
Unfortunately this GitHub area is not a support area for general application issues. This is only for issues/bugs with the framework code itself.
I will be closing your ticket here. Instead please try asking your question on one of the many great community support areas that will likely give you a better answer more quickly:

Laravel Slack (https://larachat.co/)
Laravel.io Forum (https://laravel.io/forum)
Laracasts Forum (https://laracasts.com/discuss)
StackOverflow (http://stackoverflow.com/questions/tagged/laravel)

If you feel I've closed this issue in error, please provide more information about how this is a framework issue, and I'll reopen the ticket.
Thanks in advance.


fix(apps/prod/jenkins-beta): upgrade chart

use official chart
Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


./timberborn_linux_bepinex.sh: 63: [[: not found

Hi, I'm trying to run bepinex; I've followed the steps in the readme, and I'm getting this error in the log:
[original command]: /home/<me>/.steam/debian-installation/ubuntu12_32/reaper SteamLaunch AppId=1062090 -- ./timberborn_linux_bepinex.sh (it will be discarded)
arguments (0 is the base command):
 ./timberborn_linux_bepinex.sh
./timberborn_linux_bepinex.sh: 63: [[: not found
 1: /home/<me>/.steam/debian-installation/ubuntu12_32/reaper
./timberborn_linux_bepinex.sh: 63: [[: not found
 2: SteamLaunch
./timberborn_linux_bepinex.sh: 63: [[: not found
 3: AppId=1062090
./timberborn_linux_bepinex.sh: 63: [[: not found
 4: --
./timberborn_linux_bepinex.sh: 63: [[: not found
 5: /home/<me>/.steam/debian-installation/ubuntu12_32/steam-launch-wrapper
./timberborn_linux_bepinex.sh: 63: [[: not found
 6: --
./timberborn_linux_bepinex.sh: 63: [[: not found
 7: /home/<me>/.steam/debian-installation/steamapps/common/SteamLinuxRuntime_sniper/_v2-entry-point
./timberborn_linux_bepinex.sh: 63: [[: not found
 8: --verb=waitforexitandrun
./timberborn_linux_bepinex.sh: 63: [[: not found
 9: --
./timberborn_linux_bepinex.sh: 63: [[: not found
 10: /home/<me>/.steam/debian-installation/steamapps/common/Proton - Experimental/proton
./timberborn_linux_bepinex.sh: 63: [[: not found
 11: waitforexitandrun
./timberborn_linux_bepinex.sh: 63: [[: not found
 12: /home/<me>/.steam/debian-installation/steamapps/common/Timberborn/Timberborn.exe
ERROR: command is not found. Either script didn't get expected arguments or regular expresion in it is wrong. Check the list of args above.
- regular expression was: '\/proton$' (refer to https://regex101.com/)
Terminating.

As a quickfix you can modify the first line of the ./timberborn_linux_bepinex.sh file from: #!/bin/sh to #!/bin/bash.

Why are you running the script in the first place? As the top of readme says it should not be needed anymore - your just add dll and then bepinex loads.

As a quickfix you can modify the first line of the ./timberborn_linux_bepinex.sh file from: #!/bin/sh to #!/bin/bash.

I guess I've introduced bash syntax there and sh won't work anymore? I guess we could just use it as "fix" since I'm not sure if script is useful anymore and most distribution will have bash anyway

yeah go for it...still the whole BepInEx mumbo jumbo is not working. Have you tried your instructions with the recent BepInEx and Timberborn releases? For me, it seems that the whole stuff leading to exec "$cmd" $@ comes not at the right place since the code for injecting the doorstop libs comes afterward (LD_LIBRARY_PATH, LD_PRELOAD).

Yes, I don't think there is any need to inject anything. Initially script case for Steam just appears to remove some launch arguments and I've assumed they've just skipped some Steam wrappers that were getting in the way, I've just made changes so arguments are discarded in more "clever" way. Not sure what changed and where but for some time now it launches just as is. No idea about non-steam versions of the game.

I've tried without the script in the launch instructions, I also might have put the cart before the horse and it was just loading the base game. I'll try @aruscher's workaround once I get the chance though.

@MarkKocherovsky Before you start messing with the script again, you should check whether a log file was created in your BepInEx directory, since you don't get a console under Linux.

@aruscher I've got no log file, and I've tried the bash workaround, but I've still not got the mod manager option in the main menu. I made a plugins directory independently and put the necessary files there as normal.

@MarkKocherovsky when everything works fine, the plugin directory will be created automatically and a log file should appear.
Do you have the winhttp.dll in your directory and have you configured the DLL override in protontricks for `winhttp?

I do not, but I did the protontricks directions as written

I do not, but I did the protontricks directions as written
![image](https://user-images.githubusercontent.com/67556945/274422813-8e940901-dc20-4f4a-97d8

Where are doorstop files? They are the key that makes DLL injection in Unity games as far as I understand.
That's how content of latest BepinEx for Timberborn archieve looks like, all of it (give or take the .sh file) should be in your game root dir.

-e3190bf13759.png)

I've updated script with suggested shebang directive changed to use bash and added some clarification and dir screencap to the readme, I wonder if it solves the issue?

Alright, this seems to have worked!


Ability to search the docs site

Some of the detailed reference information for Spin is found in-line in the existing articles. It would be great to have a search feature on the docs website to search for specific terms, and not rely on the navigation pane only.

I believe we can close this one out now that Spin docs are hosted at https://developer.fermyon.com/, which has search courtesy @karthik2804 🎉 .  If so, I'll let @karthik2804 or @mikkelhegn do the honors 😄


Optional "lazy" bridge subscribers

Optional "lazy" bridge subscribers
Summary
This allows for the bridge to be created in such a way that it is "lazy".  In this case "lazy" means:

The publication (output) side of the bridge is always on and actively looking for subscriptions.
The subscription (input) side of the bridge is only turned on in the case that there are subscriptions on the output side.

This means that in the case of IGN->ROS2:

If there are no ROS2 subscribers, the bridge won't subscribe to the IGN topic
If there are one or more ROS2 subscribers, the bridge will subscribe.
If all subscriptions go away, the bridge will unsubscribe.

This is especially helpful when interfacing with ign-gazebo, where sensors will be skipped if there are no active subscribers to them.
Current limitations

Ignition transport can't tell if a connection is local or not, so in the case of a bidirectional bridge, the output side of the bridge detects the input side of the bridge.  This can be overcome by changing the ignition topic names on both sides of the bridge.

This could likely be solved via https://github.com/ignitionrobotics/ign-transport/issues/263

"laziness" is evaluated in a timer loop from ROS2 at 1 second intervals.  There is a potential data race where a subscriber comes online and misses up to one second of data.  For this reason, laziness is disabled by default.

Test it
Lazy ros subscriber
# In the first window
ros2 run ros_ign_bridge lazy_static_bridge

# In a new window
ign topic -e -t /chatter

# In the first window you should see
[INFO] [1647638695.035819089] [test_node]: Lazy Mode Enabled and subscriptions found, starting
[INFO] [1647638695.035944431] [test_node]: Created ROS Subscriber: chatter

# After Ctrl-C the topic echo
[INFO] [1647638722.035932655] [test_node]: Lazy Mode Enabled and no subscriptions found, stopping

Lazy ign subscriber
# In the first window
ros2 run ros_ign_bridge lazy_static_bridge

# In a new window
ros2 topic echo /chatter

# In the first window you should see
[INFO] [1647638483.034489736] [test_node]: Lazy Mode Enabled and subscriptions found, starting
[INFO] [1647638483.034533126] [test_node[INFO] [1647638484.034472832] [test_node]: Lazy Mode Enabled and subscriptions found, starting
[INFO] [1647638484.034615154] [test_node]: Created ROS Subscriber: chatter
]: Created IGN Subscriber: chatter

# Ctrl-c in the second window and you should see
[INFO] [1647638525.034771030] [test_node]: Lazy Mode Enabled and no subscriptions found, stopping

Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

I still need to add more tests for the lazy publishing capabilities, this was mainly to verify that it didn't break any of the existing behavior.

nice the lazy_Static_bridge works for me. Do you plan to add a lazy_parameter_bridge or an arg to parameter_bridge to enable laziness?

Yes, I think it would be nice to add on a per-topic basis, but I haven't figured out the best way to add that to our current syntax.

an arg to parameter_bridge to enable laziness?

I went for a parameter that will enable/disable "laziness" for an entire bridge executable.  If we find the need for something more nuanced in the future, I can add it.


added new tool: spot detection in 2D image sequence

@bgruening Hi Björn, could you please take a look why the auto check did not start (I saw the message "Workflow runs completed with no jobs")? I did something wrong?
Thanks!

Travis is really out of date and changed their service. I added GitHub based testing now. Hope that works.

The downside or upside is that we linting your python code now as well :) Can you fix the flake8 issues?

The downside or upside is that we linting your python code now as well :) Can you fix the flake8 issues?

Thanks for prompt response! I am not familiar with flake8. Do I simply need to fix all listed issues of coding standards (for example, missing whitespaces)?

The downside or upside is that we linting your python code now as well :) Can you fix the flake8 issues?

Thanks for prompt response! I am not familiar with flake8. Do I simply need to fix all listed issues of coding standards (for example, missing whitespaces)?

yes, you can also pip install flake8 and use that for linting your code locally. Your editor has probably a flake8 plugin as well.

The downside or upside is that we linting your python code now as well :) Can you fix the flake8 issues?

flake8 issues fixed.
Now the check stops at "Check outputs".

Go to this page https://github.com/BMCV/galaxy-image-analysis/pull/25/checks?check_run_id=3122707432

Download the "All tool test results" and you will see this error:

Go to this page https://github.com/BMCV/galaxy-image-analysis/pull/25/checks?check_run_id=3122707432
Download the "All tool test results" and you will see this error:

I did not notice that I can download the results. All problems solved! Thanks!
I will add more new tools in the following days.

I did not notice that I can download the results. All problems solved! Thanks!

No worries, its a new feature :)

I will add more new tools in the following days.

That is pretty cool! Are you using them? Should we get those all on the European Galaxy server?

That is pretty cool! Are you using them? Should we get those all on the European Galaxy server?

Yes, we need to get these tools on usegalaxy.eu. (I will edit the imaging.yaml file and create a PR soon.) We and our biological partners will use the tools and develop some new workflows.
Shall I worry about the failure of Slash Command Dispatch before merging the this PR?
BTW, since there have been a number of imaging tools, shall we (or is it possible to) sort these tools on usegalaxy.eu (or in "imaging.yaml") in an alphabetical order for easy access?

BTW, since there have been a number of imaging tools, shall we (or is it possible to) sort these tools on usegalaxy.eu (or in "imaging.yaml") in an alphabetical order for easy access?

Yes, I could sort them with your preferred order.

I need to set some github credentials, but I'm not admin here?

I need to set some github credentials, but I'm not admin here?

Yes, I am. I just added you as the maintainer. Could you try it again?

I feel the power ;)

Mh, but its not enough, I can not see the "secrets" tab in the settings :(

Mh, but its not enough, I can not see the "secrets" tab in the settings :(

OK, now you must be able to do anything you want, ;) Is it related to Github Actions?

Yes, the GitHub Actions are now loading the tools automatically to the Galaxy Tool Shed and I needed to put the API key as "secret" into the repo. You can remove my privileges now again if you like.
Thanks @qgao-hd!


hppts status 404 not found

hello everyone,
what do I wrong.
after installing homebridge deconz i get the message https not found while i have entered ip address and port and i think i can't do more??

Please attach the log file.

this is de log of homebridge

This is an issue with your deCONZ gateway: it doesn’t expose /groups/0.  I’m not sure what’s causing this, nor how to fix it.  Best ask in the deCONZ support forum.


Rings across branches.

If I'm not mistaken, the current version of Deepsmiles cannot handle the ring in smiles such as CCCC(CC1)C(CC1)CCCC directly. (The example smiles is valid although not canonical.)
If I input the example smile into the current code, the output will be CCCCCC))CCC2))CCCC which makes no sense at all.
The reason for this may lie in the DFS constraint you mentioned in  #6.
I am wondering whether this issue is worth fixing and whether this pattern will appear in canonical smiles.

As you've found, it's possible to construct a string that looks like DeepSMILES which the code doesn't handle. I don't regard this as a bug. It only supports reading DeepSMILES that are consistent with SMILES strings generated by a DFS. It not a question of canonical SMILES; no SMILES writer will ever write that string - why would it stop to make a ring closure at the first "1"? From the preprint: "Since the successive atoms in a SMILES string occur in depth-first traversal order and thus form a tree, for any atom there is a single path back to the first atom; thus the ring size can be used as a distance along that path to unambiguously identify the location of the ring opening." If the string violates this assumption/requirement then the behaviour is undefined. The fix is simple though; just run it through a SMILES reader/writer.

Actually the paper describes this exact case: "To generate a DeepSMILES string we transform a SMILES string into an alternative syntax that
preserves atom order but modifies ring closure symbols and parentheses (see Table 1). The
transformation process does not require chemical interpretation of SMILES but only involves string
processing of the syntax. As such, it makes certain assumptions about the form of the SMILES string
that will be met if the SMILES string was generated by a standard cheminformatics toolkit, but may
not be met otherwise. For example, the code assumes that the SMILES was generated as a depth-
first traversal of the molecular graph. For example, this assumption is not met by the SMILES string
CC(C1)CCCC1 which cannot be encoded as DeepSMILES (at least, while preserving the atom order).
The solution is simply to roundtrip such SMILES through such a cheminformatics toolkit to generate
the form required."

Thank you very much! I must have missed that paragraph when reading the paper. Sorry for the bother.


Add docs for PAR feature

Describe the issue:
Related to: https://github.com/wso2/product-is/issues/9110
With IS 6.2-alpha, the PAR specification was onboarded. This is to track the inclusion of documentation related to the feature.

Gdocs:

https://docs.google.com/document/d/1bLn3VhDkPHloXOzCaIRm_qP-DF4GZiOsVGFiSWeFu7c/edit
https://docs.google.com/document/d/11eYSljNmNOa-o0MxP4Xl2-yz9shYOaksn7lk5b526q0/edit

PAR docs were added with

https://github.com/wso2/docs-is/pull/4162
https://github.com/wso2/docs-is/pull/4174


PYIC-4933: Changed stubData to be accessed from experian-kbv-stub stack

Proposed changes
What changed
Changed stubData to be accessed from experian-kbv-stub stack
Why did it change
The second stage of the changes required to move from kbv-stub to experian-kbv stub. This should not be merged until the infrastructure changes are successfully merged and deployed
Issue tracking

PYI-4933

Checklists
Environment variables or secrets

[ ] No environment variables or secrets were added or changed

[ ] Documented in the README
[ ] Added to deployment repository
[ ] Added to local startup repository

Other considerations

[ ] Update README with any new instructions or tasks

Do not merge until infra changes have been deployed and tested


windowruls center move size not work for Intellij idea popup floating window

it's odd that idea popup window will be resized after a few repeats. so I want to set some windowrules to fix this, but it seems certer, size and move not working for that xwayland popup window(override methods action, whose default shortcut is ctrl-o).
https://user-images.githubusercontent.com/16650765/233554083-c1118386-1b14-42d5-9408-b1f8b717887f.mp4

you open the window -> it immediately requests a move
is my guess, because x11 is x11

you open the window -> it immediately requests a move
is my guess, because x11 is x11

a probable guess, which is mine too.

a tricky&dirty solution
#!/bin/bash

function fixJetbrains() {
	echo "$1"
	[ "$1" = 'Select Methods to Override/Implement' ] && centerAndResizeActiveFloat 800 800
}

centerAndResizeActiveFloat() {
  sleep 0.1
  hyprctl dispatch resizeactive exact "$1" "$2"
  hyprctl dispatch centerwindow
}

socat -u UNIX-CONNECT:/tmp/hypr/"$HYPRLAND_INSTANCE_SIGNATURE"/.socket2.sock - | while read -r event; do
	if [[ ${event:0:12} == "activewindow" && ${event:0:14} != "activewindowv2" ]]; then
		string=${event:14}
		class="${string/,*/}"
		title="${string/*,/}"
		[ "$class" = 'jetbrains-idea' ] && fixJetbrains "$title"
	fi
done

This issue is happening to me on version 0.26.0

may help ones who encounter the same issue, a tricky solution:
#!/bin/bash

function fix_jetbrains() {
	if [ "$1" = 'Select Methods to Override/Implement' ]; then
		sleep 0.1
		resize_and_center_active_float 800 900
	fi
}

function resize_and_center_active_float() {
	hyprctl --batch "dispatch resizeactive exact $1 $2; dispatch centerwindow"
}

socat -u UNIX-CONNECT:/tmp/hypr/"$HYPRLAND_INSTANCE_SIGNATURE"/.socket2.sock - | while read -r event; do
	eventType="${event/>*/}"
	if [ "$eventType" = 'activewindow' ]; then
		class="${event/,*/}" && class="${class##*>}"
		title="${event#*,}"
		if [[ "$class" =~ ^jetbrains-.*$ ]]; then
			fix_jetbrains "$title"
		fi
        # Here you can do some other tricks
	# elif [ "$eventType" = 'monitoradded' ]; then
	# 	~/.config/hypr/scripts/detect_monitor.sh "$eventType"
	fi
done

exec-once this script, and be sure you have these rules:
windowrulev2 = forceinput,class:^(jetbrains-.*),title:^Select Methods to Override
windowrulev2 = windowdance,class:^(jetbrains-.*)

Note: do not use the forceinput rule without title, like:
windowrulev2 = forceinput,class:^(jetbrains-.*)

do so every IDEA popup will request focus, include cursor hover(on some icons, e.g. git, debug) popup windows, which will result in a infinite create->lose focus->close ->create loop when cursor is hovering on that kind of icons, and would be a performance issue in IDEA new ui.
UPDATE(2023-05-06): these hack stuffs for Jetbrains IDEA is not needed any more since hyprland fixed these, except windowrulev2 = windowdance,class:^(jetbrains-.*). But may be still a useful temporary solution for some other issues.

Not working for me, when I want to search files with double shift, the window is over the screen, it oversized to top.

@septianinet For IDEA popups resize issue, this script is not needed any more. Double shift popup is working well for me, I just set this rule for IDEA: windowrulev2 = windowdance,class:^(jetbrains-.*)$, no anything else.

@YinAqu I already set the windowdance rule, but its still not working.

this my window rules
#-- Window Rules -----------------------------------------------
windowrule = float, foot-float
windowrule = float, yad|nm-connection-editor|pavucontrolk
windowrule = float, xfce-polkit|kvantummanager|qt5ct
windowrule = float, feh|Viewnior|Gpicview|Gimp|MPlayer
windowrule = float, VirtualBox Manager|qemu|Qemu-system-x86_64
windowrule = float, title:File Operation Progress
windowrule = float, title:Confirm to replace files
#windowrulev2 = unset,class:(jetbrains-phpstorm)
#windowrulev2 = float,class:^(jetbrains-phpstorm)$,title:^(Confirm Exit)$
#windowrulev2 = size 300 150,class:^(jetbrains-phpstorm)$,title:^(Confirm Exit)$
#windowrulev2 = center,class:^(jetbrains-phpstorm)$,title:^(Confirm Exit)$
#windowrulev2 = monitor 0,class:^(jetbrains-phpstorm)$
#windowrulev2 = size 500 500,title:^(Trust and Open Project)$
#windowrulev2 = float,floating:0,class:^(jetbrains-.),title:^(win.)
#windowrulev2 = float,class:^(jetbrains-.),title:^(Welcome to.)
#windowrulev2 = center,class:^(jetbrains-.),title:^(Replace All)$
#windowrulev2 = forceinput,class:^(jetbrains-.)
windowrulev2 = windowdance,class:^(jetbrains-.*) # allows IDE to move child windows
#windowrule = size 60% 64%, Viewnior
#windowrule = center, Viewnior
windowrule = animation slide down,foot-full
windowrule = animation slide up,wlogout
#windowrulev2 = opacity 0.7 0.7,class:^(foot)$
windowrule = opacity 0.8 override 0.5 override,^(foot)$ # set opacity to 1.0 active and 0.5 inactive for kitty
windowrulev2 = opacity 1.0 1.0,class:^(wofi)$
windowrulev2 = opacity 0.8 0.8, title:^(lazygit)$
windowrulev2 = opacity 1.0 0.5,class:^(Ray)$
windowrulev2 = opacity 0.85 0.4,class:^(xfce4-terminal)$
#windowrule = float,WINDOW
#windowrule = tile,WINDOW
#windowrule = fullscreen,WINDOW
#windowrule = nofullscreenrequest,WINDOW
#windowrule = move 69 420,WINDOW
#windowrule = size 420 69,WINDOW
#windowrule = minsize 420 69,WINDOW
#windowrule = maxsize 420 69,WINDOW
#windowrule = center,WINDOW
#windowrule = pseudo,WINDOW
#windowrule = monitor 0,WINDOW
#windowrule = workspace 12,WINDOW
#windowrule = opacity 1.0,WINDOW
#windowrule = opaque 1.0,WINDOW
#windowrule = animation slide left,WINDOW
#windowrule = rounding 10,WINDOW
#windowrule = noblur,WINDOW
#windowrule = nofocus,WINDOW
#windowrule = noborder,WINDOW
#windowrule = noshadow,WINDOW
#windowrule = forceinput,WINDOW
#windowrule = windowdance,WINDOW
#windowrule = pin,WINDOW
#windowrule = noanim,WINDOW
#windowrule = bordercolor color color,WINDOW
#windowrule = idleinhibit none,WINDOW
#windowrule = unset,WINDOW
#windowrule = nomaxsize,WINDOW

@septianinet because you have a second monitor plugged in, which has its 0 position above this monitor's 0 position?

@septianinet because you have a second monitor plugged in, which has its 0 position above this monitor's 0 position?

No, i'm not plugged my second monitor.
If I plugged in my monitor (1920x1080), the popups will flickering from monitor 0 to 1.

@septianinet had a test, if I move or resize my ctrl-o popup, and close and then reopen, it keeps its last size and position. But double shift popup dost not keep its last size and position, seems it appears at the center position every time, try use watch -n 1 "hyprctl activewindow" to monitor this. Hyprland may have a special rule for this double shift window, I guess this kind windows will always  be center when open. So, maybe this is your problem, hyprland get a wrong center position from your envrionment when it automatically center your double shift popup window.


Crash Report, add, then remove story in list view

go to list view
add story, any kind
remove same story directly after creation in detail view

The issue is due to how mapView is keeping stories.
When deleting a story in listView, mapView is not updating its collections in time and even without being ontop in Layout hierarchy, mapView is re-creating its collectionData and references to the deleted object.
In ListView I have experimented in using Realm data more directly, accessing the storyDataProvider directly from VC.
This works fine and follows the idea to provide story data from the storyDataProvider as "single source of truth for stories". As the story data is not held anymore in the ViewModel,

I don't know: is this a violation of MVVC Pattern?
It saves a lot of update and passingThrough boilerplate

I will push the example to branch feature/list-realm


Use testcentric signing key for all assemblies

We are currently using the NUnit signing key for some assemblies. This should be entirely removed from the project.

:tada: This issue has been resolved in version 2.0.0-beta3 :tada:
The release is available on:

GitHub Release
NuGet Package


Merchant detail view update/patch (2h)

detail view with update/patch of the merchant needs to be checked

Macht Wolf!


PIcking up a trial spawner with the mod carryon and then doing /home crashes the server

as the title say, the trial spawner while carrying it and doing /home crash the server. Its worth testing if doing /home while carrying other things crash the server

has been added to blacklist to prevent this from happening.