Binded Listview inside another binded Listview populates extremely slow and crashes app

Greetings, i know some people will probably suggest using a DataGrid for what i need, but i actually need to use this system for a very specific purpose (populating more "columns of data" for specific "rows" and less "columns of data" for other "rows").
Therefor, the first listview populates data verticaly, which is binded using x:Bind to an ObservableCollection of a class.
Inside that class theres another ObservableCollection which is then binded by the child listview which in turn populates data horizontaly.
This way i can easely achieve an assymetrical datagrid, with more columns where its needed and less where it isnt.
The master ListView only contains 1 textblock and a child listview inside its datatemplate. usualy has 20 "rows"
The child listview only contains a combobox inside its data template with custom comboboxitems, each usually has around 30 "columns"
The problem is, even at such low ammount of data, populating 600 combobox selected indexes is extremely slow, it can take up to 10 seconds and sometimes the app crashes with a generic error not handled by visual studio, usually pointing to app.g.i.cs
at line 68
#if DEBUG && !DISABLE_XAML_GENERATED_BREAK_ON_UNHANDLED_EXCEPTION             UnhandledException += (sender, e) =>             {                 if (global::System.Diagnostics.Debugger.IsAttached) global::System.Diagnostics.Debugger.Break();             }; #endif
Ive been using different builds and even the latest stable and latest experimental, however the issue still persists.
Is there anything i can do to speed up the performance of these listviews and is there anything i can do to avoid this type of crash?
I know 600items might sound alot but they are actually quite compact and preferably displayed in this way.
Thanks in advance for all your help!

I have tried changing from combobox control to textblocks and the performance is much faster and no longer crashes even with alot more data on screen, that leads me to believe the fault here was not the list inside a list but that the comboboxes are very slow and takes a huge hit to performance. I have redesigned a special control to replace the functionality of the comboboxes and im now pretty satisfied with the performance.


Add the ko prefix to prevent the notice message during building

What this PR does / why we need it:
During build there's message like:
➜  kfserving git:(master) ✗ make deploy-dev
/Users/pugang/go/bin/controller-gen "crd:maxDescLen=0" paths=./pkg/apis/serving/v1alpha2/... output:crd:dir=config/crd
/Users/pugang/go/bin/controller-gen rbac:roleName=kfserving-manager-role paths=./pkg/controller/inferenceservice/... output:rbac:artifacts:config=config/rbac
.....................................
2020/09/02 15:34:34 Publishing docker.io/pugang/logger-254effe41b2d12817e512614f7349610:latest
2020/09/02 15:40:55 Published docker.io/pugang/logger-254effe41b2d12817e512614f7349610@sha256:e593d7a8f7d4cbc0b2640342a672bfbfbf8d8dcc5db18670cdcda60638015cf2
2020/09/02 15:40:56 NOTICE!
-----------------------------------------------------------------
We will start requiring ko:// in a coming release.  Please prefix
the following import path for things to continue working:

   github.com/kubeflow/kfserving/cmd/batcher

For more information see:

   https://github.com/google/ko/issues/158

-----------------------------------------------------------------
2020/09/02 15:40:56 Using base registry.access.redhat.com/ubi7/ubi-minimal:latest for github.com/kubeflow/kfserving/cmd/batcher
2020/09/02 15:41:01 Building github.com/kubeflow/kfserving/cmd/batcher

I saw there's for logger and manager there's already "ko://" prefix added, so will do the same for batcher to prevent the message above.

thanks @pugangxa !
/lgtm
/approve


twitch.tv || ERROR: Unable to download JSON metadata: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED]...

Checklist

🗹 I'm reporting a broken site support
🗹 I've verified that I'm running youtube-dl version 2021.06.06
🗹 I've checked that all provided URLs are alive and playable in a browser
🗹 I've checked that all URLs and arguments with special characters are properly quoted or escaped
🗹 I've searched the bugtracker for similar issues including closed ones

Verbose log

$ youtube-dl -v https://www.twitch.tv/videos/1091046304
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.twitch.tv/videos/1091046304']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.06.06
[debug] Python version 3.8.8 (CPython) - macOS-10.13.6-x86_64-i386-64bit
[debug] exe versions: ffmpeg present, ffprobe present
[debug] Proxy map: {}
[twitch:vod]<PHONE_NUMBER>: Downloading stream metadata GraphQL
ERROR: Unable to download JSON metadata: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1125)> (caused by URLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1125)')))
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 634, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2288, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/urllib/request.py", line 525, in open
    response = self._open(req, data)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/urllib/request.py", line 542, in _open
    result = self._call_chain(self.handle_open, protocol, protocol +
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 2735, in https_open
    return self.do_open(functools.partial(
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/urllib/request.py", line 1357, in do_open
    raise URLError(err)

Description

I get this error when trying to download twitch VODs for archival. I get the same error when I try different twitch VODs. I have found somewhat similar issues in the bugtracker so I tried the solutions suggested there (e.g. reinstalling SSL) but no luck, and I couldn’t find this specific issue anywhere (apologies if it is and I missed it). I’ve tried with and without my VPN. I’m on MacOS 10.13.6 but get the same error on a Fedora VM (I haven’t tried on Windows yet as I can’t get the .exe to work). I tested with a youtube link but got a different error so it seems at least partially site specific. Not sure what else to try.

This isn't really "broken site support".
It works for me (Linux/x86 Python 3.5).
Typically this error means that the root certificate for the remote site is not available, because of some problem with the integration of the platform root certificate store, OpenSSL and Python.
Presumably --no-check-certificate works around the failure?
Is the Fedora VM on the Mac?
Might you have some SSL proxy interfering with the transfer?
Did this start to happen recently (if so, when, what had you done just before)?

--no-check-certificate that worked perfectly, thank you! The Fedora VM was on the mac yes, I guess that means it had hte same problem. It started recently but I haven’t used youtube-dl for a few months so it could be a lot of things. I’ve also got youtube-dl working on windows so all’s well. I can close this now, unless you have any advice on the certificate integration thing. Thanks for your help!

--no-check-certificate restores an old and undesirable behaviour that you shouldn't normally use.
Have a look at issues such as #5317 (and other issues linked from a search URL there) to resolve the SSL issue.


dev(hansbug): add optional function and method in units

Codecov Report

Merging #6 (24946b9) into main (f488765) will decrease coverage by 0.49%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              main       #6      +/-   ##
===========================================
- Coverage   100.00%   99.50%   -0.50%
===========================================
  Files           22       22
  Lines         1415     1425      +10
===========================================
+ Hits          1415     1418       +3
- Misses           0        7       +7

Flag
Coverage Δ

unittests
99.50% <100.00%> (-0.50%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

argsloader/units/operator.py
100.00% <ø> (ø)

argsloader/units/__init__.py
100.00% <100.00%> (ø)

argsloader/units/base.py
100.00% <100.00%> (ø)

argsloader/units/common.py
100.00% <100.00%> (ø)

argsloader/units/func.py
88.37% <0.00%> (-11.63%)
:arrow_down:

argsloader/units/string.py
97.22% <0.00%> (-2.78%)
:arrow_down:

argsloader/units/numeric.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


[SPARK-49685][CONNECT] Increase CONNECT_GRPC_MARSHALLER_RECURSION_LIMIT from 1024 to 2048

What changes were proposed in this pull request?

Increase the CONNECT_GRPC_MARSHALLER_RECURSION_LIMIT that was introduced in this PR.
Why are the changes needed?

While developing a Spark Connect Plugin, I ran into this error

com.google.protobuf.InvalidProtocolBufferException: Protocol message had too many levels of nesting.  May be malicious.  Use CodedInputStream.setRecursionLimit() to increase the depth limit.

Increasing the above limit will solve the issue.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Existing UTs.
Was this patch authored or co-authored using generative AI tooling?

No.

There are two community recommendations for that case, @longvu-db . Please choose one which you want to prefer.

You can use GitHub Draft feature.
You can use your fork to run the same testing before making to the PR.

Hey @dongjoon-hyun, my PR wasn't ready for review yet as I was testing the limit of this configuration, setting a high limit will cause a java.lang.StackOverflowError this test, but thank you for your feedback.

For this one, I converted this to Draft for you because the PR was already created.

I see, thank you for this kind feedback @dongjoon-hyun!


Trying to run V4.5 after Windows update ruined my V3

OK, here we go,
Since the last update for Windows 10 (2004) my V3 style2paints got *****
So I decided to format my computer and try to use V4.5
The guide says "only download and double click as a video game"
I'm getting unable to run the V4.5, brand new Windows 10 installation hardware setting:
CPU: Xeon 1220 v3
GPU: GTX 1060 6GB
RAM:  8GB DDR 3
Syslog:
Traceback (most recent call last):
File "Style2PaintsV45.py", line 595, in
File "Style2PaintsV45.py", line 33, in main_exec
File "g:\s2p_env\env_s2p\lib\site-packages\PyInstaller\loader\pyimod03_importers.py", line 627, in exec_module
File "site-packages\cv2_init_.py", line 3, in
ImportError: DLL load failed: The specified module could not be found.

Are you using 64bit windows 10 or 32bit windows 10? It seems that "DLL load failed" error happens in 32bit systems.

64 bits, after the post I've downgraded my Windows 10 to 1903v2 version, the V3 is working properly and the V4.5 either.
If I would recommend a version for stability would be Windows 10 1903v2

Possible Reason:

Several months ago, I remembered 360 AntiVirus Software's Cloud Scanning Engine report some .pyd files in Style2Paints V4.5 as virus. And auto delete the file. (I can't remember the filename clealy, perhaps it is assets\cv2\cv2.cp36-win_amd64.pyd)

I know it is FALSE POSITIVE! and restore the file. It is not the first time it report some development files/library as virus.
Infact, it is a bug of the Cloud Scanning Engine. I can't reproduce it under virus total. After several days, the Cloud Scanning Engine bug seems to be fixed. I scan it again and the result is ok without virus.
Cloud Scanning Engine may be based on big data, and the executable files that owned by very few persons may be judged as suspicious.
AntiVirus Software's Cloud Scanning Engine is continuously updating, and its bug is hard to reproduce.

By the way, I noticed that 360 AntiVirus Software sometimes has a bug that it will auto detete the detected files without any popup message. The files are found in Antivirus Quarantine. I'm not sure this bug has been fixed now.

update:
auto detete the detected files (or set the file as "access denied")  without any popup message.

Some strange bugs I encountered with 360 AntiVirus earlier (I'm not sure they have been fixed):
(1) Occasionally, the exclusion list doesn't work.
(2) Occasionally, "temporarily disable the real-time protect" can't prevent it from setting files as "access denied". Only completely EXIT can unlock the detected files.
So, I suggest completely CLOSE the antivirus software, before debugging some strange bugs of Style2Paints.


Enable static libraries

The new ENABLE_SHARED, ENABLE_SHARED_EXECUTABLES, and ENABLE_STATIC cmake flags give you full control over whether to build shared and/or static libraries, and whether to link any executables against shared libraries or static.

+1 to merge


Support for Visual studio online

I just tried to install the neovim plugin on vsonline and I got the following error message:

Cannot install 'Neo Vim' because this extension has defined that it cannot run on the remote server.

Related #262

I think they are identical issues. You could close this one as a duplicate in favor of #262 as it seems it has more comments and interactions to it than this issue.

Duplicate of #262 👍


Installing on mac

Can I install it on macOS High Sierra?

@negarl I haven't tested on macOS before unfortunately. Feel free to try & post any questions here!

Hi again,
I have too much trouble installing PCL on mac. I got this error, Can you help me with it?
"clang: error: linker command failed with exit code 1 (use -v to see invocation)"
What if I install PCL with homebrew instead of the one you used ? would it be any different?

@negarl Hi, the core TEASER++ library does not require PCL. You should try to just compile with BUILD_TEASER_FPFH=OFF

Closing this issue for now. Feel free to reopen if you have more questions.


Phone number shouldn't be required

Steps to reproduce:

Went to the Profile page
Entered my name
Clicked "Update your profile"

Result: Profile didn't update until I entered a phone number
Expected: Phone number shouldn't be required

😄  I just ran into this, too.

Fixed in 54e2d5f83.


Handel errors processing offline orders

Coverage decreased (-0.04%) to 26.592% when pulling 58ebc097f9fb194a369149f128c5588ce22992c5 on errors into cd89be2a9f8e796dfe68a1f1059c7e62f4fa8003 on master.

@cpacia What can I do to make an order go into the PROCESSING_ERROR state, so that I could see what the server is sending the client, including the error string?

Don't we need to update POSTOrderCancel to allow the buyer to cancel unmoderated payments in this state?

Similar to above, wouldn't we need to update POSTOpenDispute to allow a buyer to open a dispute if the order is moderated and in a PROCESSING_ERROR state?

@cpacia Currently, the server will not allow the buyer to cancel a partially funded order. Is that by design?
Currently, it fails with order must be PENDING or partially funded and only a direct payment to cancel.
Either the error message or the behavior are incorrect.

Currently, the server will not allow the buyer to cancel a partially funded order. Is that by design?

Correct

Currently, the server will not allow the buyer to cancel a partially funded order. Is that by design?

Correct

Ok, can we update the inaccurate error message, which implies that "partially funded" can be cancelled.
"order must be PENDING or partially funded and only a direct payment to cancel"

I think this addresses all those issues. Anything else @rmisio ?

I'll get back to you. I need to test the order flow when an order is canceled / disputed after being in the PROCESSING_ORDER order state and make sure there are no surprises.

order must be PENDING or PROCESSING_ERROR and only a direct payment to cancel

Based on what I'm observing, if we want the error message to reflect the behavior, it should say:
order must be PENDING or a partially funded PROCESSING_ERROR and only a direct payment to cancel.

All tests pass. Going to merge.


how to develop new stages on spinnaker?

Page: /setup/install/
Feedback:  Hello, do we have more guides on how to develop new stages on spinnaker?

Duplicate of https://github.com/spinnaker/spinnaker.github.io/issues/313


How to change mujoco model parameters to pytorch tensors?

I want to construct a neural network in pytorch which outputs appropriate values of mujoco model parameters. But this requires that mujoco model parameters are converted into pytorch tensors and mujoco environments run well with these converted model parameters. I don't know how to achieve this goal. Anyone know how to do that?

Hi @ZJP-c4cld, closing this. Please use the discussion dashboard for these sorts of questions.


microcourse 1.5

For me, this course was one of the more difficult one, because until now, I was completely unfamiliar with any kind of AWS services. This makes the learning curve quite steep, which is nice, but I wonder, if it is really worth it, focusing so much energy on something which is practically only a workaround. To give you an example of the level of knowledge I started with and what kind of errors I made, I'll give you a collection of aspects, which I needed to learn because I was doing them wrong in the first place:

The mongodb cluster needs to be made available to all IP adresses, to be reachable: <IP_ADDRESS>
After writing the lambda function, it has to be deployed
I could not find any pymongo layer for the lampda function, so I had to build it locally on my laptop and upload it to aws
The connection string for pymongo needs to contain the username and password for the DATABASE USER and not the global Atlas credentials. This should probably be pointed out explicitly, because I guess many people will do it wrong when touching this system the first time
To have the API gateway working with the API key, the "API key required" must be toggled under API Gateway -> API -> ressources -> POST -> method request -> API key required
In main.py the import time needed to be added. Not sure, if this was on purpose from your side
For the use of the API key, a usage plan needs to be created and associated with the API key and the API stage
I spent lots of time trying to figure out, how the body of the requests needs to be formatted. To someone new to the field this kind of formatting is quite confusing. Some extra explanation would be very helpful here (see the code for main.py, which finally worked for me below)

connection string: database user (not global atlas user credentials)

API Gateway - API - ressources - POST - method request - API key required

missing import time in main.py

invoke url must be copied from the POST method and include the /data directory

create usage plan and associate both, API key and API stage with it

Modify main.py
`from netman import connectWiFi
import urequests
from my_secrets import SSID, PASSWORD, COURSE_ID, AWS_API_GATEWAY_URL, AWS_API_KEY
import time
import json
connectWiFi(SSID, PASSWORD, country="US")
headers = {"x-api-key": AWS_API_KEY}
document = {
"body": json.dumps({"course_id": COURSE_ID})
}
print(f"Sending document to AWS Lambda")
num_retries = 3
for _ in range(num_retries):
response = urequests.post(AWS_API_GATEWAY_URL, headers=headers, json=document)
txt = str(response.text)
status_code = response.status_code
if status_code != 200:
    print("Retrying in 5 seconds...")
    time.sleep(5)

print(f"Response: ({status_code}), msg = {txt}")

response.close()

if status_code == 200:
    print("Added Successfully")
    break

print("Retrying...")

`

Here a loose collection of screenshots about steps, which I found to be important, because I had not done these properly during my first try:

I am still thinking about, how the setup of this course could be simplified, even though the data api of mongodb is now disabled. Two ideas, I have now in my mind:

Make a collection available for every user of the microcourse in a central mongodb provided by AC. For course 1.5 then only the connection credentials for the AWS Gateway would need to be provided and the student could focus on the actual communication and data handling. Drawback: Student does not learn, how to setup everything by himself. Pro: Less risk of loosing motivation and quicker success. I think this would be a fine way, considering, that this is still the hello world course
Setup a selfhosted mongodb on a second raspberry pi. I have my own mongodb instance running on one of our institutes VMs and managing it via mongsh, is not really userfriendly in my opinion. But overall, if I had the choice between learning to selfhost my db and getting to know all these new tools and words around the AWS universe, selfhosting would probably be easier and less confusing, if the guide is well written. Basically, its installing mongodb on the pi, creating a user, creating a database and creating a connection string. The biggest drawback here is, that this Pi would have to be reachable from the internet, which is a security issue when people play around at home. On the other hand: If they have all their communication running in their local network, this would not be a problem

Thanks for the great suggestions. Definitely a lot going on here.

Make a collection available for every user of the microcourse in a central mongodb provided by AC. For course 1.5 then only the connection credentials for the AWS Gateway would need to be provided and the student could focus on the actual communication and data handling. Drawback: Student does not learn, how to setup everything by himself. Pro: Less risk of loosing motivation and quicker success. I think this would be a fine way, considering, that this is still the hello world course

Reasonable approach. Might try this out. Alternative would be to drop this idea of direct microcontroller to database uploading and instead focus on handling data from the orchestrator.
Maybe you could also look over https://github.com/AccelerationConsortium/ac-microcourses/issues/45 and lmk if you have any follow up thoughts?

If there were a step by step video walkthrough for the AWS and MongoDB setup for module 1.5, do you think that would help address this?


Build and run iOS application

I'm unable to build and run an iOS application on a Mac:
Error message received:
"/Users/donjayamanne/Desktop/Development/PythonStuff/beewareSample/folders/appname/venv/lib/python3.6/site-packages/briefcase/ios.py", line 150, in build_app     build_settings_str = ['{}={}'.format(x for x in build_settings)] IndexError: tuple index out of range
Information:

MacOS HighSierra Version 10.13.4 (17E199)
Python 3.6.5 (virtualenv)
XCode Version 9.3 (9E145)
I'm able to develop, build and run iOS apps (in Objective C)

Steps to reproduce:

Use cookiecutter to create project (use defaults)
Create virtual env virtualenv --python=3.6 venv
Install beeware (pip install beeware)
Build iOS project, here's the output (success)

$ beeware build ios
Running Briefcase...
>>> python setup.py ios

running ios

 * Updating user code...
 * Installing requirements...
No requirements.
 * Installing platform requirements...
Collecting toga-ios==0.3.0.dev8
  Using cached https://files.pythonhosted.org/packages/67/6e/d0aff2728c47e530516ac7d1f331d1679b8664b00ec74248090d11a30d84/toga_iOS-0.3.0.dev8-py3-none-any.whl
Collecting rubicon-objc>=0.2.10 (from toga-ios==0.3.0.dev8)
  Using cached https://files.pythonhosted.org/packages/35/b0/aa6d839057126991d16a591c74e24c075797db7436d6c0d6e655a3e15b3a/rubicon_objc-0.2.10-py2.py3-none-any.whl
Collecting toga-core==0.3.0.dev8 (from toga-ios==0.3.0.dev8)
  Using cached https://files.pythonhosted.org/packages/a4/2e/4757cadb2206e2f6bb6136fda395dbec2d84bda86b77b5713f435b0fdb60/toga_core-0.3.0.dev8-py3-none-any.whl
Collecting travertino>=0.1.0 (from toga-core==0.3.0.dev8->toga-ios==0.3.0.dev8)
  Using cached https://files.pythonhosted.org/packages/56/3f/f03efe63c2bdc72763ac39777c9ebbab98605ced5c74aefe48b4fa0abb4d/travertino-0.1.2-py3-none-any.whl
Installing collected packages: rubicon-objc, travertino, toga-core, toga-ios
Successfully installed rubicon-objc-0.2.10 toga-core-0.3.0.dev8 toga-ios-0.3.0.dev8 travertino-0.1.2
 * Installing project code...
Processing /Users/donjayamanne/Desktop/Development/PythonStuff/beewareSample/folders/appname
Building wheels for collected packages: appname
  Running setup.py bdist_wheel for appname ... done
  Stored in directory: /private/var/folders/22/0kj01mp12l3dsd1vw9vhfs6c0000gn/T/pip-ephem-wheel-cache-g4e3hwuo/wheels/83/35/3f/9792c1aeedb6706d827a27b802299fd76b879bf8b35fca4e98
Successfully built appname
Installing collected packages: appname
Successfully installed appname-0.0.1
 * No icons defined - using default...
 * No splash screen defined...

Installation complete.

Now run using beeware run ios, here's the output (this fails)

$ beeware run ios
Running Briefcase...
>>> python setup.py ios -s

running ios

 * Updating user code...
 * Installing requirements...
No requirements.
 * Installing platform requirements...
Collecting toga-ios==0.3.0.dev8
  Using cached https://files.pythonhosted.org/packages/67/6e/d0aff2728c47e530516ac7d1f331d1679b8664b00ec74248090d11a30d84/toga_iOS-0.3.0.dev8-py3-none-any.whl
Collecting rubicon-objc>=0.2.10 (from toga-ios==0.3.0.dev8)
  Using cached https://files.pythonhosted.org/packages/35/b0/aa6d839057126991d16a591c74e24c075797db7436d6c0d6e655a3e15b3a/rubicon_objc-0.2.10-py2.py3-none-any.whl
Collecting toga-core==0.3.0.dev8 (from toga-ios==0.3.0.dev8)
  Using cached https://files.pythonhosted.org/packages/a4/2e/4757cadb2206e2f6bb6136fda395dbec2d84bda86b77b5713f435b0fdb60/toga_core-0.3.0.dev8-py3-none-any.whl
Collecting travertino>=0.1.0 (from toga-core==0.3.0.dev8->toga-ios==0.3.0.dev8)
  Using cached https://files.pythonhosted.org/packages/56/3f/f03efe63c2bdc72763ac39777c9ebbab98605ced5c74aefe48b4fa0abb4d/travertino-0.1.2-py3-none-any.whl
Installing collected packages: rubicon-objc, travertino, toga-core, toga-ios
Successfully installed rubicon-objc-0.2.10 toga-core-0.3.0.dev8 toga-ios-0.3.0.dev8 travertino-0.1.2
 * Installing project code...
Processing /Users/donjayamanne/Desktop/Development/PythonStuff/beewareSample/folders/appname
Building wheels for collected packages: appname
  Running setup.py bdist_wheel for appname ... done
  Stored in directory: /private/var/folders/22/0kj01mp12l3dsd1vw9vhfs6c0000gn/T/pip-ephem-wheel-cache-50wz_1en/wheels/83/35/3f/9792c1aeedb6706d827a27b802299fd76b879bf8b35fca4e98
^[[1;10DSuccessfully built appname
Installing collected packages: appname
Successfully installed appname-0.0.1
 * No icons defined - using default...
 * No splash screen defined...

Installation complete.
Traceback (most recent call last):
  File "setup.py", line 79, in <module>
    'toga-django==0.3.0.dev8',
  File "/Users/donjayamanne/Desktop/Development/PythonStuff/beewareSample/folders/appname/venv/lib/python3.6/site-packages/setuptools/__init__.py", line 129, in setup
    return distutils.core.setup(**attrs)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/distutils/dist.py", line 955, in run_commands
    self.run_command(cmd)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/Users/donjayamanne/Desktop/Development/PythonStuff/beewareSample/folders/appname/venv/lib/python3.6/site-packages/briefcase/app.py", line 466, in run
    success = self.build_app()
  File "/Users/donjayamanne/Desktop/Development/PythonStuff/beewareSample/folders/appname/venv/lib/python3.6/site-packages/briefcase/ios.py", line 150, in build_app
    build_settings_str = ['{}={}'.format(x for x in build_settings)]
IndexError: tuple index out of range

Ah - yes - someone reported this during the sprints, but evidently didn't submit a patch.

I'll patch this and get a release out RSN - it's a pretty glaring error...

Awesome,
fyi, I've ironed out a few edhes of the vscode beeware extension and managed to get debugging working for macos and Linux apps.
Will work on Windows next. Then a wiki page.
Will let you know when you can publish the next version, hopefully on Monday.


static => public

/static folder has been renamed/public

Codecov Report

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

@@           Coverage Diff           @@
##           master     #184   +/-   ##
=======================================
  Coverage   88.42%   88.42%
=======================================
  Files          13       13
  Lines         190      190
  Branches       46       46
=======================================
  Hits          168      168
  Misses         16       16
  Partials        6        6

Continue to review full report at Codecov.

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


Issue with sensor voltage

Using the PulseSensor_BPM example, I was able to get the BPM effortlessly at first, however, after a few days I did the same thing again, but this time the values outputted by the program were nonsensical.

Here is the data from when the sensor stopped working properly. When it was working it looked exactly like those provided in the tutorials.

@HarrisL2
You are posting this issue in the wrong repository.
https://github.com/WorldFamousElectronics/PulseSensorPlayground
In order to help you, I need to know what Arduino hardware you are using and also send a picture of your Pulse Sensor so that I can verify it is legit hardware.


[5.9][Features] Allow ExtensionMacros to be enabled in noasserts builds.

Explanation: The @Observable macro has adopted -enable-experimental-feature ExtensionMacros. The Observation module is built in noasserts configurations in CI, but this flag is not available in noasserts builds, hence breaking CI. This change allows the flag to be enabled in noasserts builds.
Scope: Only impacts the enable-experimental-feature ExtensionMacros feature flag.
Risk: Extremely low due to tiny scope.
Testing: noasserts CI configurations have successfully gotten past building the Observation module after the main PR was merged:
Main branch PR: https://github.com/apple/swift/pull/67208

@swift-ci please test


Added dynamic drawing capability to line features.

Added a verticesPerFeature() method.
Refactored the code which generated the pair of triangles per line segment to use an array of information.  This allows a caller to determine which vertices of the triangles belong to which end of the line.  I expect that I will need this to dynamically change one end of a line without changing the other.
Added code to avoid computations on hidden lines.

Thanks @manthey. Looks good to me :+1:


fix(electric): @migrations version capture failing intermittently

Let's have a reviewer on this PR and merge the reviewed version into kevindp/vax-1180-expose-proxy-port to avoid having to rebase on every change.


how is the CMakelists.txt file generated

Hi,
I am working on similar plugins. My doubt is how did you generate this CMakeLists.txt file? I am getting this error
CMake Error at CMakeLists.txt:10 (find_package):
  By not providing "Findcatkin.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "catkin", but
  CMake did not find one.

  Could not find a package configuration file provided by "catkin" with any
  of the following names:

    catkinConfig.cmake
    catkin-config.cmake

  Add the installation prefix of "catkin" to CMAKE_PREFIX_PATH or set
  "catkin_DIR" to a directory containing one of the above files.  If "catkin"
  provides a separate development package or SDK, be sure it has been
  installed.

Configuring incomplete, errors occurred!

Also, how to generate the .so library file for loading the plugin?
Any help is appreciated. Thanks

This repository is still quite experimental, and it is part of a larger ecosystem than Ignition.
The objective of this project is storing custom plugins for Ignition Gazebo that can exploit the ScenarIO interface, part of the gym-ignition project. In few words, beyond other features, ScenarIO allows using Ignition Gazebo from Python. It also allows loading World and Model plugins, like those included in this repo, directly from code.
If your aim is to develop regular Ignition Gazebo plugins, i.e. you're not interested in Python usage, I'd recommend to visit the upstream documentation about how to develop plugins.

My doubt is how did you generate this CMakeLists.txt file

This is hand written. It seems trivial because most of the logic is hidden inside ScenarIO. The target ScenarioGazebo::ScenarioGazebo allows including all what a plugin needs to build against the active version of Ignition Gazebo. Furthermore, the build and distribution process is performed with common Python packaging tools, and this differs considerably from how a plain Ignition Gazebo plugin should be managed.

This repository is still quite experimental, and it is part of a larger ecosystem than Ignition.
The objective of this project is storing custom plugins for Ignition Gazebo that can exploit the ScenarIO interface, part of the gym-ignition project. In few words, beyond other features, ScenarIO allows using Ignition Gazebo from Python. It also allows loading World and Model plugins, like those included in this repo, directly from code.
If your aim is to develop regular Ignition Gazebo plugins, i.e. you're not interested in Python usage, I'd recommend to visit the upstream documentation about how to develop plugins.

My doubt is how did you generate this CMakeLists.txt file

This is hand written. It seems trivial because most of the logic is hidden inside ScenarIO. The target ScenarioGazebo::ScenarioGazebo allows including all what a plugin needs to build against the active version of Ignition Gazebo. Furthermore, the build and distribution process is performed with common Python packaging tools, and this differs considerably from how a plain Ignition Gazebo plugin should be managed.

Thanks for clarification.


Define dependencies in one place

This is to promote consistency and make updating dependencies easier.
see
https://discuss.gradle.org/t/what-is-the-proper-way-to-make-dependencies-consistent-in-a-multi-project-setup/17383/12
Also:
Define the common dependencies in the root build.gradle file.
        compile deps.log4j
        compile deps.guice
        compile deps.slf4j

        runtime deps.slf4jLog4j

Introduce common versions in the ext.versions map.

The build failed again.
azkaban.execapp.FlowRunnerTest2 > testFailedFinishingFailure3 FAILED
java.lang.AssertionError: Wrong status for [jobb] expected:<FAILED_FINISHING> but was:
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:834)
at org.junit.Assert.assertEquals(Assert.java:118)
at azkaban.execapp.FlowRunnerTestBase.assertStatus(FlowRunnerTestBase.java:117)
at azkaban.execapp.FlowRunnerTest2.testFailedFinishingFailure3(FlowRunnerTest2.java:520)
:azkaban-solo-server:check
:azkaban-solo-server:build
:azkaban-common:check
:azkaban-common:build
74 tests completed, 1 failed, 22 skipped
@juhoautio I think we should disable this test file until it is stable. What do you think?

Can you get the console output for the failed test on that Travis build somehow? Otherwise it's much harder to fix, when this can't be reproduced locally.. For example I ran both FlowRunner tests 2 x 50 times on Travis and there were no failures (see https://github.com/azkaban/azkaban/pull/1186). I also ran them 1000 x locally, no failures..
Last time that there was a Travis failure, the failed tests were:
azkaban.execapp.FlowRunnerTest > execAndCancel
azkaban.execapp.FlowRunnerTest2 > testFailedFinishingFailure3 (this is the same that you reported as failed now again)
I would add @Ignore only on those two test methods, not on entire files.

Here, maybe one last try before ignores :}
https://github.com/azkaban/azkaban/pull/1207

Unfortunately, I tried and couldn't get the console output from Travis. The best I was able to do was to get the stack trace when a test failed.
Please contribute the knowledge if you find a way.

@juhoautio I found a way to log test console output
see https://gist.github.com/HappyRay/84119110f957dc114402178258723632

Are you going to enable it for Travis builds? Otherwise it can be hard to catch problems if they occur rarely.. But I guess there haven't been more random failures after the latest stability fixes?

No. The output is too noisy.
It's a tool, however, to debug issues when needed. It can be enabled on a per pull request basis.
In any case, the test should be reliable and repeatable (locally or in travis ) by design. I would prefer simplifying the current DAG engining design and tests.


Added section on factory functions

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: gusthoff:x: raph-amiardYou have signed the CLA already but the status is still pending? Let us recheck it.


convert key formats

can nak convert a key between different formats, such as raw and nsec? did not find anything on that :/

nak encode/nak decode

thanks! would be helpful to have the help be more elaborate on that for those who don't know the meaning of nip numbers by heart ;)


Removing requirements.txt to force snyk to read the Pipfile instead

Right now the Snyk github action picks up the file requirements.txt under the portal directory, but this file is not used.
We are using the Pipfile to generate the python packages, so removing this file then force Snyk to re-import the project and pick up the Pipfile, when the file is in the master branch.

Merging this closes #209


Add WSL support to launchEditor utility / error overlay functionality

In WSL (Bash on Ubuntu on Windows), Windows executables can be launched from within the shell, however setting EDITOR or REACT_EDITOR to a Windows editor and feeding launchEditor() an absolute path currently results in it trying to open a file at e.g. C:\mnt\c\...\foo.js (usually resulting in a new file with that save path), rather than just opening C:\...\foo.js like it should do. Thankfully, the interop functionality can actually translate between the WSL and Windows version of a path (when it's located on the Windows file system), as long as it's provided as a relative path, which is what this change does.
Just to make it clear, these are the possibilities when using this utility under WSL after this change:

Linux-based editor(e.g. nano)Windows-based editor(e.g. VS Code)

File in "Linux" FS (not accessible from Windows)
Works (no difference)
Doesn't work (no difference). Current WSL behavior is to print "Unable to translate current working directory. Using C:\Users\Whoever" in terminal, which at least gives a hint to users why it won't work.

File in Windows FS (and accessible from within WSL via /mnt/wherever)
Works (path will be unnecessarily transformed to relative, but no practical difference)
Works now! (most common scenario)

In other words I don't believe there's any sort of regression with this change. I've tested all 4 scenarios with various editors.
Potential improvements

In the scenario of Linux FS + Windows editor, it would be nice to simply disallow it and display a custom message, but it depends on detecting that the editor is Windows-based - you could do this by checking extensions and/or using the file command, but even then the Windows versions of VS Code and Atom come with bash script launchers. You could also check where it resolves to (e.g. which $EDITOR), but that comes with some caveats as well (see below).

There's a bit of an edge-case where the file could be located under /mnt/, but not on a Windows FS volume, in which case it would be given a relative path but still fail if you tried to open it in a Windows editor. A more accurate, but more complicated way to detect Windows volumes is to check if the filesystem type of the mount is "drvfs" - I can't find an easy way to do this from Node, but you can, for example, run mount | grep /mnt/c or grep /mnt/c /proc/mounts or findmnt -P /mnt/c, then parse it.

Anyway I'm not sure how much you guys care about supporting WSL, but it seems to be gaining popularity for those of us using Windows for one reason or another, so I thought it would at least be worth some consideration.

I don’t fully understand it but looks good to me.
If there are any issues with this in the future I’ll be sure to ping you.
Thanks!


Download button create error and nothing happen

The bug occur on every save type, beta or not, you can modifi everything but as soon as you try to Save/Download it create an error related to weather time

Ive mannaged to found something , for some reason if i DISABLE the Time of day Cycle and i put sunny weather and 60 for weather switch time ( who is not use anymore) i can now download with no error . i dont know of that could hep a bit

iam having the same issue ..make changes to map and it will not download... i will however try what you have done and see if that works

This is likely a new property related to weather. Please attach a zip of a save file so that I can reproduce the issue and fix it.

Error started since your last commit ( the "do not store property types" one ) - same save would download fine previous to that. This one loads fine but errors on a download attempt.


Don't drop AsyncWrite in IntoAsyncRead

This implements AsyncWrite for IntoAsyncRead when the inner stream implements it,
forwarding all method calls.
This allows working on Types that keep both impls, and allows framing
them with codecs where those require both traits (like tokio codec).
r? @cramertj

This seems like a fine change to me, but did you mean to have a different commit message? The commit message and PR name seem to suggest a different thing than what is implemented here.

@cramertj We can change the commit message to whatever you like. From the user perspective it's like you have a Type which implements AsyncWrite and when you call into_async_read, you lose the AsyncWrite functionality. That's why I called it like this. This doesn't actually provide any new functionality, just makes the inner AsyncWrite available on the wrapper to avoid losing the functionality, and to be able to work with API's that require both. Especially since IntoAsyncRead will consume the inner stream.
Which part does not correspond to the code? I'm a bit  confused. It looks like the first paragraph describes the implementation exactly.

"Add AsyncWrite implementation for IntoAsyncRead" or "impl AsyncWrite for IntoAsyncRead" seems good commit message to me. (refs: #1575, #1503)

I changed the pull request title and the commit message.

I changed the pull request title and the commit message.


Incompatibility with Google Drive Filestream.

Explain the bug
I'm triyng to use Google Drive in Files in the same way i used to use in Explorer, it something like the "Files by Demand" in OneDrive, i use this in work, but, the firt one folders in Google Drive are "hidden" in Files, but i can access those folders by copyng the directory from Explorer, however, some folders crashes the app, making it close itself, that problem not happen in Explorer.
To reproduce

Go to "Your Computer" section
Click on 'Google Drive' folder, notice that doesnt show anything
Copy directory link from Explorer, paste on Files than it opens some folders
Some folders inside that one that has open by copyng the directory from Explorer crash the Files.

Expected behavior
In Explorer i can access all these files and folders almost as i use the OneDrive, that "Files by demand" feature. But i can tell that even on Explorer Google Drive crashes sometimes when the folder is too heavy, but it only take some seconds to recover.
Logfile
debug.txt
debug_fulltrust.txt
Video showing the bug and comparison with what happens in Explorer
https://user-images.githubusercontent.com/79664514/117831128-aaee4c80-b24a-11eb-9c40-1c146238e3b3.mp4
System Information

Edição	Windows 10 Home
Versão	20H2
Instalado em	21/10/2020
Compilação do SO	19042.964
Experiência	Windows Feature Experience Pack 120.2212.2020.0

App version: <IP_ADDRESS>

Additional comment
Its a long time issue, i thought it was a Google Drive problem, but it has many updates and never change this issue, so i'm assuming it as a Files bug or some incompatibility

Apparently 1.5.2 was broken you should have an update in the store or msixbundle on GitHub to fix it

Thank you guys for the attention, but how do i download this update on GitHub? Sorry, i'm kinda new in here 😅

You might have to uninstall the one you have currently or use the Microsoft store as it will likely reset your settings but https://github.com/files-community/Files/releases/download/v1.5/Files-1.5.Msixbundle  download that then click on it. it should open a windows you can click install

As you said, apparently this is a 1.5.2 bug, the 1.5 works perfectly, thank you so much! 😄 Its even faster now.


Show number of steps to go when rebasing/merging

This fixes #691
VCS_INFO has showing the number of steps in a rebase already builtin. Just need to activate and define the format (documentation):

Caveat:
We can only apply one actionformat, thus this information is also printed on other actions (like merge), where it is rather useless..
Secondly, this change now prints if there are incoming/outgoing changes when merging/rebasing (See 207874b21cedc0fa54ca11863a4ad660a05b35d0).
I haven't changed the tests yet, so I expect a couple of failures there. Just wanted to discuss the feature first.
//cc @cybercussion

This is really cool! Thanks so much, @dritter!
Looks good to me =)


Use edited thumbnails in partial-publish.xml

Intended to resolve #5541.
While thumbnails created in the (new) editor are stored in Opencast, they currently go unused. This changes partial-publish.xml (aka the default editor workflow) to use thumbnails created in the editor for publication.
Discussion points:

Currently this uses the thumbnails from the editor as is. Maybe they should be encoded in some way?
This only touches upon the thumbnails for the engage player. Other thumbnails (for search, feed cover) are ignored. Should they also be changed?

Not sure if this is the right place to ask this, but I'm running an 15.3 and this doesn't seem to work? The workflows in my installation are almost the community version, with only two unrelated (very sure about that) changes. When I upload a new thumbnail in the editor, click "save" and then "process changes", the workflow runs successfully, but the two steps added in this PR are skipped. And I end up with a completely black 16px * 9px thumbnail.
Any ideas?


EMBL database size issue

Hi there,
I am trying to download the plant portion of the EMBL database. I left the download running overnight because it's so large, it takes a long time. It failed because I ran out of space on my computer. I've now made more space on my harddrive, and moved my crabs folder so that it syncs to a cloud drive.
Is there a way to expedite the download, so that the --download-embl command skips downloads if they're already present in the --output folder? Or is there a way to identify which EMBL accessions have not yet been downloaded, and then specifically download just those? I had to re-start the process, but it's overwriting files I had already downloaded yesterday. Also, this way, if it fails again due to running out of space, I could begin again without losing progress.
Thank you!
Sara

Hello @saraswaminathan,
When CRABS downloads the EMBL files, there should be a number associated with them. For example:
crabs --download-embl --output embl_mam.fasta --taxon 'mam*'

Downloads 10 files from the EMBL database. These files follow the format STD_MAM_#.fasta.gz, where # represents a number between 1 and 10.
Once all 10 files are downloaded, CRABS will combine the 10 files and write them to embl_mam.fasta. This marks the successful end of the code. The reason CRABS downloads 10 files, is that the EMBL database is set up in a way, so that each file contains 10,000 sequences. Once the first STD_MAM_1.fasta.gz file contains 10,000 sequences a new file (STD_MAM_2.fasta.gz) is created on the database for newer barcodes. We know the number of files for MAM is 10, as CRABS reports how many files are being downloaded. For PLN, this should be 50 (?) files.
Unfortunately, EMBL doesn't allow for easy checking on which sequences were previously downloaded and skip those. (There is a way using the lists provided on the EMBL database, but it would require quite extensive coding to accomplish). However, we can make use of the numbers in the files that were previously downloaded and focus on downloading all other ones.
Let's say that we wanted to download the 10 mammal files on the EMBL database, but that our connection failed during the download of the fourth file (i.e., STD_MAM_4.fasta.gz). I recommend deleting this fourth file, as the download most likely wasn't finished/successful. Once the fourth file is deleted, we can write a for loop to tell CRABS to download the last 7 files. Example code:
for i in {4..10}; do crabs --download-embl --output STD_MAM_${i}.fasta --taxon mam_${i}; done

In this for loop, please note that I'm specifying new output files that contain the number of the download. This stops CRABS from overwriting the output files when new files are downloaded in the for loop. Please also note that I'm using an underscore (_) after mam and before specifying the number, which is essential to match the names.
For the files that were previously downloaded by CRABS before the download was interrupted, you can simply unzip them yourself, which is what CRABS normally does automatically. Example code:
for gz in *.fasta.gz; do gunzip ${gz}; done

If you follow these steps, you should end up with 10 files, from STD_MAM_1.fasta to STD_MAM_10.fasta`.
Once everything is downloaded and unzipped, you can import the files into CRABS separately and merge them using the --import and --merge functions, respectively. Example code:
crabs --download-taxonomy

for i in {1..10}; do crabs --import --import-format embl --names names.dmp --nodes nodes.dmp --acc2tax nucl_gb.accession2taxid --input STD_MAM_${i}.fasta --output STD_MAM_${i}.txt --ranks 'superkingdom;phylum;class;order;family;genus;species'; done

crabs --merge --output 'embl_mam.txt' --uniq --input 'STD_MAM_1.txt;STD_MAM_2.txt;STD_MAM_3.txt;STD_MAM_4.txt;STD_MAM_5.txt;STD_MAM_6.txt;STD_MAM_7.txt;STD_MAM_8.txt;STD_MAM_9.txt;STD_MAM_10.txt'

Please change the code accordingly for your use-case. I hope this helps.
Best wishes,
Gert-Jan

In the next CRABS update, I will incorporate regex support for the --taxon parameter of the --download-embl function, as well as the --input parameter of the --merge function to simplify the code.
Best wishes,
Gert-Jan

Hi Gert-Jan,
Thank you so much! Your solution worked perfectly. However, because of the shear magnitude of the EMBL plant database, pairwise global alignment with my gene of interest is taking forever (only 2% completed in 24 hours). Is there anything I can do to speed this process up?
The code I am using is
crabs --pairwise-global-alignment --input /Volumes/Photos/SARA/23S_merged_NCBI_EMBL.txt --amplicons /Volumes/Photos/SARA/23S_inSilico_withEMBL.txt --output /Volumes/Photos/SARA/23S_aligned_withEMBL.txt --forward GGACAGAAAGACCCTATGAA --reverse TCAGCCTGTTATCCCTAGAG --size-select 10000 --percent-identity 0.85 --coverage 0.8
Thank you so much for all your thoughtful feedback - I really appreciate your help!
Sara

Hello @saraswaminathan,
Great to hear this worked!
For speeding up the --pairwise-global-alignment step we have 4 tips and tricks:

To run --dereplicate and --filter prior to --in-silico-pcr. This should significantly reduce the number of sequences that would otherwise already be removed during these steps for --pairwise-global-alignment. Please see section 5.4.1 in the README document for more information.
Since --pairwise-global-alignment supports multithreading, running this code on a cluster or supercomputer will significantly increase the speed of execution.
Invoke the --size-select parameter, which removes sequences longer than N from the analysis. So, if --size-select 1000 is provided to CRABS, all sequences longer than 1,000 bp will be discarded for the --pairwise-global-align function. The reasoning behind this is the following: let's assume that when both primer-binding regions are in the sequence, the --in-silico-pcr function has extracted the amplicon for us. This means that what remains in the original file are (i) sequences that do not cover the barcode, (ii) sequences that cover the barcode but miss one primer-binding region, and (iii) sequences that cover the barcode but miss both primer-binding regions. Since we want to extract (ii) and (iii) from the file using --pairwise-global-align and we assume barcodes are created using Sanger sequencing, we can restrict the number of sequences we investigate by setting a length threshold of 2 x the max Sanger sequencing length (when a barcode was build using forward and reverse sequencing and used one of your target primers to generate the barcode). This is of course an approximation and some of our assumption will not always hold true, but it would speed up the code without the risk of losing too many barcodes.
If your aim is not to extract all barcodes to capture all possible intra-specific variation for each species, you can subset the initial file by removing all sequences for species for which you have retrieved a barcode from the --in-silico-pcr function. This is currently not yet supported in CRABS, but I will make this a filtering parameter for --pairwise-global-alignment in the next CRABS update. For now, you can extract the species names from the --in-silico-pcr file and use this list in the --subset function.

I hope these tips help.
Best wishes,
Gert-Jan


Datatable Buttons

I am using datatables with Buttons plugin, I am using latest floatThead plugin
I tried to fix header , search box and button to be fixed but somehow it seems that only table headers are fixed.
Is there any way I can fix search box and buttons?
Thanks
Ankur

I cant provide help related to 3rd party datatables plugins, especially if you cant provide a jsfiddle.


Allow customization of all icons via config

Description
Some icons on the input component are not customizable, while other icons can be changed. Please make it so that all icons can be changed.
Icons that cannot be changed at the moment:

The icon for password visibility cannot be modified via the config.
The icon to clear the input field cannot be modified via the config.
Perhaps others that I missed?

The current icons are hard-coded in the components, see the references below.
Password visibility:
https://github.com/oruga-ui/oruga/blob/5e7a1ae7175e1b651cd88c541ad3cdf2d403a682/packages/oruga/src/components/input/Input.vue#L235-L237
Clearing input field:
https://github.com/oruga-ui/oruga/blob/5e7a1ae7175e1b651cd88c541ad3cdf2d403a682/packages/oruga/src/components/input/Input.vue#L212
Why Oruga need this feature
All icons should be customizable in case one whishes to change the default icon pack.

Thanks for your reply! I have read the documentation, and have a custom icon component set up in order to support the 'material-symbols-outlined'. My custom component is based on your suggestion from this issue: https://github.com/oruga-ui/oruga/issues/359.
In my case, I want to replace the eye icon everywhere, so the possible issue you point out does not affect me.
// my-custom-icon-component.vue

<template>
  <span :class="iconPack">
    {{ iconName }}
  </span>
</template>

<script setup lang="ts">
import { computed } from "vue";

const props = defineProps<{
  icon: string[];
}>();

const iconPack = computed(() => {
  return props.icon[0];
});

const iconName = computed(() => {
  return props.icon[1];
});
</script>

// main.ts

app.use(Oruga, {
  iconComponent: "my-custom-icon-component",
  iconPack: "material-symbols-rounded",
  statusVariantIcon: {
    success: "check",
    danger: "error",
    info: "info",
    warning: "warning",
  },
  ...
}

This works great. However, the icons I am referring to are the 'hard-coded' ones and get passed to my custom component by their original name and with iconpack='material-symbols-rounded'.
In the example of the eye icon, this cannot be mapped to an icon and I just see the string eye in my UI. If the original iconpack was passed to my component perhaps I could fix this, but that doesn't seem to be the case.
How would this work with multiple icon packs? Can this be combined with a custom icon component at all? 🤔

In you example is simple to replace eye, just add if/switch to iconName computed 😉

In you example is simple to replace eye, just add if/switch to iconName computed 😉

That's more like a workaround than a structural fix. 🙅
Especially since a lot of the other icons can be customised via the config... Why not make them all customizable like that?

I don't see it as structural fix, btw you can do the same thing replacing the icon mapping as i told you (intenalIcons field)


ci: automatically fix ESLint errors and format using Prettire upon committing

Let's use #131 instead to ensure Vercel deployment preview is triggered.


Fixes navigation item states

This fixes the active and focus states in the navbar on the site. Addresses Issue #190
Before:

After:

Thanks @cameronoelsen 👍


How to watch for specific strings using regex?

Hey there,
Nice package. Is there a way to fire the callback only when a specific string of characters is written in the watched files? Any leads will be greatly appreciated.

If a string is doge491, then: chokidar.watch("directory", {ignored: /(?!doge491)/})


Updating the client_config_resource

Description
Per this bug: https://chefio.atlassian.net/browse/CHEF-7885, we had previously taken a fix to correct the issue. However, that fix only addressed a problem with the Ohai property and not the policy_persist_run_list property. This fix updates that.
Related Issue

Types of changes

[ ] 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)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] If Gemfile.lock has changed, I have used --conservative to do it and included the full output in the Description above.
[ ] All new and existing tests passed.
[ ] All commits have been signed-off for the Developer Certificate of Origin.

you also need to add to the property default: false https://github.com/chef/chef/blob/main/lib/chef/resource/chef_client_config.rb#L196


Update Migration doc for Identity

Fixes #6953
Review URL

@javiercn  please review. The scaffolder is a bit kludgy but this approach is growing on me.  See #6953 for my alternative approach.

I think it looks great!


Improve naming

This PR improves method names, eliminating get prefix in non-generated API.
Also SerializableConverter abstract class is introduced.

@yuri-sergiichuk, PTAL.

Codecov Report

Merging #355 into master will increase coverage by 0.08%.
The diff coverage is 89.79%.

@@             Coverage Diff              @@
##             master     #355      +/-   ##
============================================
+ Coverage     81.56%   81.65%   +0.08%
- Complexity     2576     2582       +6
============================================
  Files           390      391       +1
  Lines          9894     9903       +9
  Branches        592      593       +1
============================================
+ Hits           8070     8086      +16
+ Misses         1648     1642       -6
+ Partials        176      175       -1


Allow modification of keys with custom reference time

This allows us to get rid of Thread.sleep() in the tests, which should speed those up significantly, while allowing for greater test flexibility in terms of time frames to test.
Further, this PR also makes use of our own SubkeyBindingSignatureBuilder to bind subkeys when using SecretKeyRingEditor.addSubkey().

Fixes #310


add proper batching to the palette encoder

@jamieparkinson
looks like the numpy functions in question (kmeans, histogramdd) don't actually play nicely with stacked inputs, and trying to warp things internally isn't going to give us a meaningful increase in speed after all. Happy to go with the existing solution


Add metrics support for grpc client.

Also fixes an issue where the client is decorated twice with the same decorators.

Let me wait for two other teammates' reviews. (Yes, we added two full time members!)


Cleanup NetBeans license header for module java.api.common

Review and cleanup the license header in java.api.common
Oracle donated NetBeans to the Apache Software Foundation.
Your task now is to replace the old license header that has Oracle (and/or Sun Microsystems
and/or NetBeans) in it with the standard Apache Software Foundation license header.
For an empty file that supports headers you must add the Apache license header.
Note that you have to use the proper license header for Java, .properties or HTML files.
Look at the nearby files for templates and copy-paste the license header.
When done commit the changes with a commit message that starts with first line
"[NETBEANS-54] Module Review " and a message body describing your review.
See a longer explanation about what we are doing here: https://cwiki.apache.org/confluence/display/NETBEANS/List+of+Modules+to+Review
Please announce in a comment when you have started work.
Files that need to be looked at in java.api.common:
java.api.common/src/org/netbeans/modules/java/api/common/project/ui/customizer/vmo/gen/CommandLine.tokens
java.api.common/test/unit/src/org/netbeans/modules/java/api/common/queries/MultiModuleGroupQueryImplTest.java

Reserved for @mirkoperillo


MSID: 95521 Version: 1 DOI: 10.1101/2<IP_ADDRESS>2935

MSID: 95521
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>2935
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=95521
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>2935

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/95521v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>2935
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/95521/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Structural issues need resolving:


PI-8929: Changed about translation in en-US

Pull Request Test Coverage Report for Build 1214

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

Totals

Change from base Build 1210:
0.0%

Covered Lines:
1760

Relevant Lines:
3647

💛  - Coveralls


[api-minor] Remove the deprecated onUnsupportedFeature functionality (PR 15758 follow-up)

This was deprecated in PR #15758, which has now been included in three official PDF.js releases.
While PR #15880 did limit the bundle-size impact of this functionality on e.g. the Firefox PDF Viewer, it still leads to some unnecessary "bloat" that these changes remove.
Furthermore, with this being deprecated there'd also be no effort put into e.g. extending the UNSUPPORTED_FEATURES list when handling future error cases.

Please note: The Windows "failures" are caused by a previously failing makeref, hence those can be ignored.


a state of consumer

if offset.Timestamp-previousTimestamp < (storage.app.Config.Lagcheck.MinDistance * 1000) {
clusterOffsets.consumerLock.Unlock()
return
}
here, is there need exclude the state when offset.Timestamp-previousTimestamp==0  ?
if consumer did not commit offset for a while,  and the ring is not full yet, then the evaluateGroup methd will not check out the true state of the consumer.

If the consumer has not committed offsets in a while and the ring is not full, then the state of the consumer is incomplete as far as Burrow is concerned and this is reflected in the response (with complete = false).
I'm working separately on some tuning for how ZK offsets are added to try and make slow committers who are caught up work better.

in this way, you must alert when the ring is not completed, but when you application startup, it will alert for a while, but other ring is really uncompleted, so users may not distinct this stituation, thank you for your reply


[WFLY-8161] Don't change a system property just to alter an output value

https://issues.jboss.org/browse/WFLY-8161

Brad Maxwell is doing something else.


Return User answer and correct answer from useNextStep hook

Is your feature request related to a problem? Please describe.
Currently, ResponseBlock must be mounted to view and validate the answers.
As a study designer, I may want to get rid of ResponseBlock, and implement my custom version of the Responses within the stimuli component.
Describe the solution you'd like
Shift the validation logic to useNextStep hook which returns participants' responses as well as correct answer if available.

I don't think the answer saving is an issue with our current implementation. It's possible to hide the response blocks and the answer can be supplied from the react components, because we give the react components a saveTrialAnswer prop that allows the stimulus to define the answer.
We might need to allow for custom validation within the stimulus, though


[<IP_ADDRESS>] Incorrect logrotate config permissions with Debian package installation

Influx ver: <IP_ADDRESS>
OS ver: Ubuntu 14.04.2 LTS
Package: http://influxdb.s3.amazonaws.com/influxdb_<IP_ADDRESS>_amd64.deb
Hi folks,
I noticed I was getting mammoth logfile usage and it wasn't being rotated as expected. I ran logrotate as debug and it failed, giving me a permissions error:
user@xxx:~$ sudo logrotate -d /etc/logrotate.d/influxd
Ignoring /etc/logrotate.d/influxd because of bad file mode.

Handling 0 logs

The permissions:
user@xxx:~$ ls -la /etc/logrotate.d/influxd
-rw-rw-r-- 1 root root 113 Sep 29 17:51 /etc/logrotate.d/influxd

The following permission change corrected the problem, and a force run of logrotate on the config ran successfully.
user@xxx:~$ sudo chmod g-w /etc/logrotate.d/influxd

Thanks for all the ongoing work! Phil.

Duplicate of #4446, resolved in #4447

🙏Appreciated, look forward to the next release.


Fix Cloud Bigtable YCSB benchmark

Use netty-tcnative-boringssl in place of ALPN boot. This should give
better performance and is simpler to configure.
Bump client to 0.2.4.

@hildrum, could you PTAL?

LGTM.  I was able to run the benchmark.


Interaction controller (hand) gets stuck in soft contact state

Hi,
as mentioned in this thread the interaction hand gets stuck in soft contact state, when the hand is inside an interactive object and the tracking is lost (e.g. due to simply looking away). I attached the minimal Unity project setup.

Open Minimal scene and start (debug controller gizmos are enabled)
Rotate the object with your hand tracked by leap motion
You will see it keeps the soft contact mode while staying inside the object
Keep your hand inside the object and look away
Now the hand is stuck in soft contact state

SoftContactBugMinimal.zip

We've been able to reproduce so I'll add this to our backlog for 1.2. If I'm able to find a quick fix, I'll respond here with it so you can copy/paste a patch in.
Thanks for the report!

I also made another observation on this issue.
If the hands gets contact again with the very same object, when the tracking was lost, the hands state toggles again as it should.
Maybe it is a subscribe/unsubscribe thing. I hope it helps!

Any updates?

Finally got to this issue. Fix wound up being pretty simple once I was able to figure out what was going on. Thanks for the example scene! It's a one-line fix so check out the diff in the PR and feel free to copy-paste the solution into your project if you need the fix now; this PR is targeted for develop so it won't be public until our next IE release.
https://github.com/leapmotion/UnityModules/pull/906


XIC

Currently, I’m using MZmine 4.4.3, mainly in GC-EI low-resolution mode. For example, I’m unable to export the Extracted Ion Chromatogram (XIC). I’d like to view the XIC for three specific m/z values, but I can’t seem to figure out how to do it. Is this a feature that already exists, and I’m just unaware of it? Or, if not, would it be possible to add this feature? Thank you!

I need to export EICs. Instead of an m/z range, I need something where, for example, I can select only specific m/z values like 93, 121, and 136. I want to view these three values exclusively and export the results as a CSV file.

Apologies, but I still didn’t quite understand. If I were looking at just m/z 93, you would be correct, but I want to view all three m/z values (93, 121, and 136) simultaneously. How can I do that?


Remove old environment variable from Python artifact build

It appears GRPC_PYTHON_USE_CUSTOM_BDIST is no longer used anywhere: it was added in https://github.com/grpc/grpc/commit/5c04760d46f2628a83106d048e732c997fed8801 but reading it was removed in https://github.com/grpc/grpc/commit/f747409f6f528c95f39297ebf3aea02085119285#diff-2661f7832fd616ffcef52579b626c053

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[microbenchmarks] No significant performance differences

Thanks for the reminder - merged!


Add a staging env

Add staging enviornment, perhaps from development branch
Then we can use a gitflow framework

The staging branch is now deployed on https://staging-app.nowcasting.io/
There is still some issue with Auth0, didn't investigate yet.

Ive moved this to 'development' branch and https://development-app.nowcasting.io/, to be slightly more inline wiht gitflow workflow

@all-contributors please add @flowirtz for code

@all-contributors please add @dantravers  for ideas


Updates test coverage for bibliographic_controller#bib_jsonld

closes #911

Coverage remained the same at 77.454% when pulling 2a52361ab12445a43f0f698f0b35b88b1785331a on 911-jsonld-spec into eb799c5ae1f917d64816a452714d9765404661a4 on alma.


Missing sound for adding in loom interface.

Sound works when creating, but missing sound for when adding.
https://github.com/Wilyicaro/Legacy-Minecraft/assets/39068731/cb7e34c7-423f-4fae-b816-e55bd72f8540
And this is the sound it's supposed to make when adding in the loom interface in PS4 Edition, which is also present in JE when adding.
https://github.com/Wilyicaro/Legacy-Minecraft/assets/39068731/24426e38-75d8-4e5c-9f9b-2be3b5e8baba

Added in version 1.6.2a, thanks for reporting.


[EDMTP-435] Added ExpressionInjector.

I have read the CLA Document and I hereby sign the CLA

Already implemented.


Replace secrets in value files

When passing --values valueFiles it would be good to have secrets there and replace them at run time.

Now that secrets are handled inline in Drone, this functionality might become very useful


Add optional void* argument to osdp_set_command_complete_callback

I've been using the library in C++ using OOP approach, but I faced a problem to deal with the command_complete_callback_t type, it can't be a member function pointer.
So, to solve this I want to propose to add an extra void* arg to osdp_set_command_complete_callback, so at this way I can give the object instance ("this" pointer) to the callback and then I will be able to access to class members inside the callback function,
So, the function pointer will be something like this typedef void (*osdp_command_complete_callback_t)(void* arg, int id) and function signature to set it would be void osdp_set_command_complete_callback(osdp_t *ctx, osdp_command_complete_callback_t cb, void* arg)
Also, we'll need to add the argument variable in the osdp struct.

That sounds reasonable. And you have described everything needed to be done to support it (except that the python bindings also need to be updated). Would you like to create a PR for it? If not I'll get around to it soon-ish.
PS: this kind of change requests are easier described as a PR, you'd spend almost the same time making the change as you did describing it :)

That sounds reasonable. And you have described everything needed to be done to support it (except that the python bindings also need to be updated). Would you like to create a PR for it? If not I'll get around to it soon-ish.
PS: this kind of change requests are easier described as a PR, you'd spend almost the same time making the change as you did describing it :)

I'll work on it, I need to fix this to integrate it in my project.

Sorry I clicked wrong button, Is there an special rules I need to follow for PR? Should I create a branch from master branch? Anyways, I'm working in a fork.

You can create a PR from any branch in your fork, just be sure to create the branch from the most recent master here (or rebase to it).

You can create a PR from any branch in your fork, just be sure to create the branch from the most recent master here (or rebase to it).

Got it, I see thanks!

PR #90 merged!


Allow acceptance of pre-created IServiceProvider instance

Currently Asp.Net 5 is solely in charge of instantiating an IServiceProvider instance to be used.  While this works perfectly fine in Asp.Net websites, this is not ideal in circumstances where you are embedding Asp.Net 5 in another application (service or console application for example).   For example, a console application that is running in the background might use Asp.Net 5 to expose http endpoints for easy communication with external systems (health checks, event notifications, etc...)
In this circumstance, Asp.Net is a relatively minor piece of infrastructure for the application as a whole but the current implementation of Asp.Net's dependency injection strategy requires that Asp.Net is the sole creator of the ioc container.  This means that if you want a single ioc container for the whole application you must gather your service descriptors, pass them to Asp.Net, start the Asp.Net host, then call IWebApplication.ApplicationServices in order to retrieve the ioc container so it can be used in the non-Asp.net portions of the application.  Without doing this, any service that's registered as an instance or singleton risks not working as expected (since each ioc container can now creates it's own instance).
This seems to me to cause my application to be too tightly coupled to Asp.Net for dependency injection, and means that if any other minor infrastructure makes the same assumption (requires it to be the creator of ioc containers) it becomes impossible to have consistent and sane dependency injection strategy.
It seems like we should at least have the option to allow calling classes to request services that Asp.Net needs to register, let the original class create the IServiceProvider instance, then pass it to the WebApplicationBuilder for usage.
An example of how this could look:
    public class WebHost
    {
        private IHostingEngine _hostingEngine;
        private IApplication _application;

        public IServiceCollection GetServices()
        {
            // WebHostBuilder already has BuildHostedServices().  If this was decoupled from
            // the call to execute the configureServices delegate inside then we should be able to
            // somehow get a list of all services that Asp.Net wants to register by default
            var services = WebHostBuilder.GetDefaultServices();
            services.AddMvc();
            return services;
        }

        public void Start(IServiceProvider serviceProvider)
        {
            _hostingEngine = new WebHostBuilder(GetConfig())
                .UseServer("Microsoft.AspNet.Server.Kestrel")
                .UseServiceProvider(serviceProvider)
                .UseStartup<WebHost>()
                .Build();

            _application = _hostingEngine.Start();
        }
    }

........

        public static void Main(string[] args)
        {
            var services = GetApplicationServices();

            using (var host = new WebHost())
            {
                var webServices = host.GetServices();
                foreach (var webService in webServices)
                {
                    services.Add(webService);
                }

                var container = CreateIocContainer();
                host.Start(container);
            }
        }

Then (theoretically) the WebHostBuilder.Build() checks if a IServiceProvider was given, if so then it ignores the building of services, gets the resolution it needs and passes it into WebApplication's constructor. I do see that WebApplication adds to the collection but that can also be abstracted away by an IWebApplication..GetRequiredServices() method.

Is there a timing for this? I have this exact use case, and I found the suggested solution will be of great help.
There are different evidences that this is a friction point for those who want to have full control of the building of the DI container while integrating ASP.NET Core as part of an application.
See:
http://stackoverflow.com/questions/38916620/asp-net-core-with-existing-ioc-container-environment
https://github.com/autofac/Autofac/issues/811#issuecomment-262736919

In terms of implementation, would it be a case of having a configurable override for:
https://github.com/aspnet/Hosting/blob/a2962d54f18dece7ec57ee42f3e565f3727168e1/src/Microsoft.AspNetCore.Hosting/WebHostBuilder.cs#L171
...and then deciding an API which is in harmony with the rest?  E.g. builder.UseServiceProvider((IServiceCollection c) => MakeProviderFromCollection(c))?
Are there any landmines or complexities I'm blissfully unaware of, or is it worth me attempting a PR?

Unfortunately, I don't think this is feasible as hosting needs to add services to something, the only reasonable thing we could do is make it so that hosting can optionally support providing an IServiceCollection.

I realize this expands the scope of the request a bit but Autofac's solution to this problem is to allow you to create a child scope from an existing provider, allowing you to add additional resolutions to the child scope only.  That would be the ideal solution here as I believe the goal is to get existing dependencies and scopes in, not to expose ASP.NET services to the root scope.  This enables you to host multiple WebHosts with different sets of services without them interfering with each other.

I realize this expands the scope of the request a bit but Autofac's solution to this problem is to allow you to create a child scope from an existing provider, allowing you to add additional resolutions to the child scope only

Our DI system has no feature like that and will likely never have a feature like that. The IServiceProvider isn't editable once it's made. You can only append to the IServiceCollection.

That would be the ideal solution here as I believe the goal is to get existing dependencies and scopes in, not to expose ASP.NET services to the root scope.

Scopes aren't really child containers, so it doesn't have the same effect. We used to do container chaining way back when, but it was just too hard to support in all of the other DI containers and it was removed.

This enables you to host multiple WebHosts with different sets of services without them interfering with each other.

That's already the case today, you just don't get to control the starting point, as we always make a collection for you.

@davidfowl  Should this be closed by #1325 ?

This should probably be closed in general for the reasons I stated above. We've added support to allow overriding the host service provider. If you don't consume the IServiceCollection that was passed in, things won't work so you still can't just pass an IServiceProvider ignoring everything else.


Finer control on utilities responsiveness

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal
I realise that I never got to post the thoughts for utilities improvement I mentioned in #36029.
The idea would be to provide finer control over the responsiveness of utilities and let users control:

which breakpoints the utilities get generated for, allowing the responsive option to accept either:

a list of breakpoints, with the subset of breakpoints for which to generate the utility
a breakpoint => (list of value identifiers) map, allowing to not only subset breakpoints, but which values of the utility get set for them

For example:
(
  "max-width": (
  property: max-width,
  class: mw,
  responsive: (md), // Generates only `mw-100` and `mw-md-100`
  values: (100: 100%)
),

This may call for a special default key for the list to enable configuring what's generated for the class without breakpoint (or even disable its generation altogether by associating it to null or  an empty list ().

which values the utility take at specific breakpoints, by setting its value as a breakpoint => actual value map. I've got a bit less faith of this one, especially if there's a way to have fine grained utilities, but that could save stringing a few utilities together

Motivation and context
The motivation is mostly to avoid adding many breakpoints when switching a utility to be responsive. This could also let developers further optimise their build, but it'll likely be less burden to use something like PurgeCSS for that (though not every user will have access/be willing to use it or a similar tool as part of their build).

I think this would be a rad update for the utilities API. Any interest in exploring in a PR?

I would be keen to give it a shot. I'm not sure I'll have a chance to look at it before the end of this year though, as a heads up 😊
I think it'll be easier to build it with confidence on top of #36029, which adds the sass-true tests, so there's a safety net for the current functionalities.

@romaricpascal Whenever you want to dive back into this, let's talk a bit more about the utility API's future. Plenty of opportunities to improve things here as we move toward a v6 :).

@mdo That's still something I'd be keen to explore, but the time I can spend on Bootstrap is turning out to be very very limited. I don't want this held back if someone else is keen to move it forward 😊


Fix mangled license headers.

Looks like these were edited by a botched find-and-replace somewhere.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

LGTM, Thanks for fixing this, @robertwb.


Enable Active Directory On Premise

I hope somebody can help with this....Our client have many applications and they want to integrate it to just log in once (single sign on), but they want that using active directory on premise (NOT AZURE AD AND NOT WS-FEDERATION) they have their own server.They want something like this image https://ibb.co/nyHOia .... we decide to evaluate identityserver so we found this https://identityserver4.readthedocs.io/en/release/topics/windows.html and in that link says that identitserver 4 support active directory but here we dont see where we are going to put the LPAD//... route to connect to the active directory... so we are with doubts if identityserver supports or not Active directory or Maybe we cant undertand it very well. Thank you.

This seems to be a general question about IdentityServer - not a bug report or an issue.
Please use StackOverflow for that. This has the advantage that questions and answers can be easily found by search engines, and that there are more people answering questions than just us.
For IdentityServer3
https://stackoverflow.com/questions/tagged/?tagnames=identityserver3&sort=newest
For IdentityServer4
https://stackoverflow.com/questions/tagged/?tagnames=identityserver4&sort=newest
For commercial support options - visit
https://identityserver.io


feat(nip-64): Display Chess (PGN) messages

In order for NIP-64 to be approved, it needs to be implemented in two clients.
Would nostrudel be interested in displaying kind := 64 as chess board in the web interface?
The spec is quite simple and straightforward–so it should be rather easy.
It would need to include a dependency on a library capable of drawing chess boards.
If you are interested, I'll try to come up with a PR. Please, let me know :pray:

Hey @hzrd149!
That's great news–nice to hear.

Also if its to play a live games id like to get your thoughts on this https://github.com/hzrd149/chessnut I built it a while go and its far from perfect but I think it has a good event structure to create a chain of "game-states"

Wow.. damn.. chessnut seems nice. Seems to be very well thought out and sophisticated. Will definitely dig into it as soon as time allows. Just wow. :rocket:

I am interested in adding chess to the app but Id like to make sure its interoperable Could NIP-64 and PGN format be used to play a live game over nostr or is it simply to stored recorded games on nostr?

This is really just the most simple solution I could imagine to display messages as chess boards in clients. I thought about adding the game mechanics as well, but that would have been too challenging for any client to adopt and so I decided to not pursue it any further. JesterUI currently still uses plain FEN strings and can easily be gamed–not optimal. But to answer your question: I am sure it can be used to actually include more advanced game mechanics and play live games as well… however, the idea of the NIP is just for other clients to display "finished" games, e.g. chessnut, JesterUI and any other chess app on nostr could submit kind := 64 messages to be displayed in nostrudel, damus, primal, etc.. The actual games will probably use replaceable, ephemeral or expiring events (or any other useful nostr base feature)–and I did not want to force this on any downstream app or developer. Hope you can somehow follow my thoughts. What do you think about it?

If its just to display published games then that should be really easy to add. although where do you imagine displaying them. It wouldn't be hard to add them to the main timeline but that might get distracting and difficult to find old games

I could imagine, for example, that clients do not dump games into a main timeline, but rather display a small chess piece on pubkeys that sent kind :=64 messages in the last ~14 days or so. But I think there is such a great amount of possibilities that I really have no suggestions for clients on how they do it. The most important thing–I think!–is that they are capable of drawing chessboards and display games.

Update: NIP-64 has been merged :muscle:


Search Space produces better results with package.name checks

The underlying issue seems to still not be solved with adding more package specific names https://github.com/AppThreat/dep-scan/issues/23.
When adding new packages to requirements.txt, this seems to fail. As an example, scan now fails to detect the following packages as an example:
tensorflow
ansible

Searching in terms of package name seems to produce better results at least for requirements.txt. This shouldn't break other language dependencies in lock files ideally.

Hi @torque59,
Thanks for this PR. Unfortunately, removing vendor would lead to large number of false positives since the same package name might appear under different languages. The caller of vdb can pass None for vendor to force a package name based lookup.
So dep-scan for example could create a variation for a package with vendor as None which should either work or we could make it work.

This is handled in depscan as part of https://github.com/AppThreat/dep-scan/commit/3c0123a1ab86a06f7b109ae4482bc6dbdf3c3306
https://github.com/AppThreat/dep-scan/blob/master/depscan/lib/normalize.py#L79

Also note that NVD refers to tensorflow package as:

tensorflow
tensorflow-cpu
tensorflow-gpu

and so on. The package name really depends on the mood of the person filing the issue :)

@torque59 Did you get a chance to test with the latest version of scan?

@prabhu yes the latest version looks good. But i did see the commit, would the whitelist sort approach work in the long run, like we would still have to keep this extending the list right for new vendors ?

@torque59 for python, it is currently doing both vendor alias search and package-based search. So, there might be new false positives which might require tuning at some point, but false negatives should be reduced to an extent.
It's a known industry-wide problem. Large companies have resorted to human curation and updates for vulnerabilities or licensing the dataset from commercial providers. The resulting database unfortunately is not free and comes with restrictive licensing clauses and so on. In the past, I have tried looking into the description text as well started a project https://github.com/AppThreat/cve-annotations to improve the NVD database.

@torque59 can this be closed. Are you happy with the fixes made to depscan?

@prabhu yes this can be closed, apologies for leaving it open. The fixes look good now with some of the packages i tested. I'll open up issues incase i run into any of them. Thank you @prabhu for the effort :)


Record transformer cannot parse json object named as "time"

Record transformer cannot parse json object named as "time"
Tried to parse the time object of  json , but unable to do it.
With given input :
{"log":{"msg":"Local Cache Cleared"},"stream":"stdout","time":"2023-06-19T08:10:50.063648794Z"}
I used this record transformation:
<filter diagnostic_event>
  @type record_transformer
  enable_ruby true
  <record>
    message ${record['time']}|${record["log"]["msg"]}
  </record>
</filter>

When I print record to stdout, I got following result:
diagnostic_event: {"log":{"msg":"Local Cache Cleared"},"stream":"stdout","message":"|Local Cache Cleared"}
It seems to  ${record['time']} return as null.
Is time keyword have special meaning? And "time" keyword should not contains in the inputs.
To Reproduce
input for the source plugin:
echo "{"log":{"msg":"Local Cache Cleared"},"stream":"stdout","time":"2023-06-19T08:10:50.063648794Z"}" >> /var/log/1.log
Expected behavior
Stdout should print result as
2023-06-19T08:10:50.063648794Z|Local Cache Cleared
Your Environment
- Fluentd version: fluentd:v1.15.3
- TD Agent version:
- Operating system: linux
- Kernel version: 5.15.0-73-generic

Your Configuration
<source>
  @type tail
  path /var/log/*.log
  pos_file /var/log/td-agent/httpd-access-debug.log.pos
  tag diagnostic_event
  <parse>
    @type json
  </parse>
</source>

<filter diagnostic_event>
  @type record_transformer
  enable_ruby true
  <record>

    message ${record['time']}${record["log"]["msg"]}
  </record>
</filter>
<match diagnostic_event>
  @type copy
  <store>
    @type stdout
  </store>
</match>

Your Error Log
No error log, Unexpected behavior.

Additional context
No response

@yemreavci This is not a bug.
You are using parser_json, and it parses time key as the time of record by default.
If you still need time key value of record after parsing, you can set keep_time_key true to the parser setting.
<source>
  @type tail
  path /var/log/*.log
  pos_file /var/log/td-agent/httpd-access-debug.log.pos
  tag diagnostic_event
  <parse>
    @type json
+   keep_time_key true
  </parse>
</source>

For details, please refer https://docs.fluentd.org/configuration/parse-section#parse-parameters.

Sorry, in this case, perhaps you should use time key of the event directly.
You don't need to use keep_time_key of the parser.
<filter diagnostic_event>
  @type record_transformer
  enable_ruby true
  <record>
    message ${time}${record["log"]["msg"]}
  </record>
</filter>

Thanks a lot.

If i apply the keep_time_key true option in the <source>, the date will be example.19700101.json when extracted as a file via match.
How can I solve this problem?

Note #4240


fix aggregate_id search syntax

fix aggregate_id search syntax
this is related to the issues https://github.com/Yelp/elastalert/issues/2820, https://github.com/Yelp/elastalert/issues/1656 and https://github.com/Yelp/elastalert/issues/284

@Qmando / @pdscopes / @danielpops   please review.

Thank you

@balusarakesh
Any way you could push this PR to https://github.com/jertel/elastalert ?

@balusarakesh
Any way you could push this PR to https://github.com/jertel/elastalert2 ?

sorry, for the late reply...looks like it is pushed to elastalert2 here


Add ZHA fan tests

contains #20710 will need a rebase after that is merged.

Coverage is 98.46% on the reorg branch. Don't have the dev #. I'll pull them all from coveragerc at the same time after the reorg is merged.