Lack of documentation

This project is something I am looking for, but I am missing some documentation and/or place to discuss things. Personally, I would like to help with the development and/or documentation, so I share things that I have learned about the problem setuptools_scm is trying to address:
Connecting SCM with versioning stuff is difficult. There are two major use cases when you want the package to know its version:

The moment when you release (release-time) the package and
the moment when an unsuspecting user installs it without any knowledge of SCM at all.

During the release-time, thigs are easy. It is not that difficult to guess the descriptive version string of the package from the SCM. This is mainly because the developer has all tools he needs and the source code is tagged.
However, when one releases the package, users that are going to install it don't have the advanced tools and the package should be well-aware of which version it is (a good practice is that package foo has foo.__version__ attribute that is the semver version string). So, the version has to be somehow injected to the package and the package has to be able to pick it up.
Some quick examples:

numpy writes the version they infer from git to numpy/version.py at release-time. However, they keep the version number in setup.py along some boilerplate code.
setuptools maintain a setuptools/version.py containing a version string in SCM
matplotlib maintains version string in lib/matplotlib/__init__.py and has relatively convoluted code that extracts it.

Generally, there are two ways of how to solve this:

Store the version string somewhere and introspect it from setup.py using more or less convoluted code you have to write yourself.

Generate the version string during release-time and inject it somehow to the package in a smart way.
The cons of this approach is that it is an unusual approach and one can forget to perform the injection step. Having it as a hook of setup.py [sb]dist would be so much cleaner.
Also, there would be the ordering problem - first you tag, then you release and the release gets modified ex-post when it gets the right version string. So the package should be tagged afterwards or the part with the version string should not be revision-controled.

Now, let's get to setuptools_scm.
AFAIK, users will be obliged to get it when they get in touch with package that requires it, which I regard as sub-optimal, but if parts of it get integrated into setuptools, it would dissolve this weakness.
On the other hand, it seems to me that it is introspecting PKG-INFO, which I think is the right way to go. In my opinion, generating PKG-INFO is essentially the injection step and people in charge of their package's setup.py just have to have good possibilities of getting it out of the file.
So what would I like to see in this package:

Better documentation. Everybody needs under-the-hood documentation because of inevitable leakage of abstractions. A tutorial is also a must.
Integration of PKG-INFO parsing to setuptools, a pattern of using setuptools_scm in a way that end-users don't have to worry about having it installed.

Bitbucket: https://bitbucket.org/pypa/setuptools_scm/issue/24
Originally reported by: Matěj Týč
Originally created at: 2015-02-27T14:10:43.625

please ping me in the #pypa channel on freenode, im usually around, keep in mind i have a normal work week in CET

Original comment by: Ronny Pfannschmidt

after discussion on #pypa-dev the conclusion was to add version module support to setuptools_scm and a pkg_resources helper function

Original comment by: Ronny Pfannschmidt

a a note, setuptools should grow a pkg_resources utility function that can be used for
__version__ = __import__('pkg_resources').get_distribution_version_of_package(__name__)

Original comment by: Ronny Pfannschmidt

Issue #25 was marked as a duplicate of this issue.

Original comment by: Ronny Pfannschmidt

resuffle the READMe, fixes #24
→ <<cset 76f9f38fa608>>

Original comment by: Ronny Pfannschmidt

just missed you on irc

Original comment by: Ronny Pfannschmidt


Override notification URL

Hi,
The snap docs state that we are able to override/append default notification url by modifying header in the charge request. Can we do that using this library?
Since from what I understand, the Http request is taken care of by the library using cURL.
Thanks!

Not for now.
We are planning to add that new capability to set HTTP headers soon.

I see. Alrigth then, thanks for the answer. Looking forward into that!

Yes we are trying to make it happen as soon as we are availble, will keep this issue open as reminder. Thanks!

Hi, I get an error when request status
$status = \Midtrans\Transaction::status($order->no_order); var_dump($status);

Hi @imiro sorry for the delay, we have implemented this on latest v2.3
https://github.com/Midtrans/midtrans-php/releases/tag/v2.3.0
Thanks!

Usage instruction:
https://github.com/Midtrans/midtrans-php#override-notification-url


Unable to input wide characters

I have noticed any wide characters can't be input.
Both of them can't success.

Paste emoji 😄 from clipboard.
Input Japanese characters using macOS Japanese Input Method

Hello,
This covers two different things: paste large characters, which probably involves a "bracketed paste" functionality that I have yet to implement (the last thing that is to be implemented in this library.)
For the Japanese characters, I must look into it, as I thought the character handling library I'm using (github.com/rivo/uniseg) would handle those just fine.
I will look into this soon,
thanks

See here for the details concerning this:
https://github.com/rivo/uniseg?tab=readme-ov-file#monospace-width


[Feature Request] Function calling

Do you plan supporting the function calling feature? Would you be open to accept PRs on that regard?
Currently, some functions such as DuckDuckGo search or Wolfram Alpha could greatly extend the model's capabilities.

function call is for developers, not end users.
Currently the only promising form of extending model functionality is gpts. Technically, it is possible to implement a command line version of gpt through function call, but at this stage, there are neither users nor suppliers, so there is no chance.

I am not you what do you mean by only for developers. Certainly I can use both I mentioned as a user, because the model directly decides which function to call during a standard conversation.

It seems to me that the best use of function calling is to make the call via cli and pipe in the results to aichat. One use case that I hope evolves is the ability to use aichat as a remote control over an ollama-type system with enhanced tools capabilities.
Is this in line with your comment @gilcu3 ?

@cboecking That is one possibility, although the usage I was suggesting is much simpler. When interacting with the bot it might realize it needs to search something online, or solve some math formula, and in such cases it automatically uses function calling to reply to the user. In the background aichat would call (via cli or using a library) the corresponding functions specified by the AI.

tools are something you pass to the base model in a structured form, e.g. for OpenAI https://platform.openai.com/docs/api-reference/chat/create#chat-create-tools, and the API handles whether to return a chat message or request a tool invocation. it would make the most sense to allow the user to define tools and their interfaces in config.yaml, specify if the current model supports tool usage (which most OpenAI compatible API wrappers support, e.g. Huggingface TGI: https://huggingface.co/docs/text-generation-inference/en/basic_tutorials/using_guidance#chat-completion-with-tools), and if the completion requests a tool call then invoke the tool after prompting the user.
example config.yaml, in pseudocode:
- tools:
  list_directory:
    shell-command: `ls %input`
    interface: {"input": "Path of directory to list"}
  convert:
    shell-command: `convert %source %destination`
    interface: {"source": "Source image name", "destination": "Target image name"}
- models:
  openai:gpt-4o:
    features:vision,tool-usage

in a chat:
temp) Convert image in current dir to jpg

*Run tool `list_directory .`?* (y/N): y
Output of `list_directory .`: foo.tiff doc.txt

Thanks for providing the file names. foo.tiff appears to be the image you're interested in.

*Run tool `convert foo.tiff foo.jpg`?* (y/N): y
Output of `convert foo.tiff foo.jpg`:

I have obtained the filename of the image in the current directory and converted it to jpg as per your request.

temp)

#514 has implemented this function, you can try it. Welcome feedback and suggestions.

#514 has implemented this function, you can try it. Welcome feedback and suggestions.

Feature working perfectly, thank you! Will you accept PRs to add more example tools to llm-functions? I tested specifically with Wolfram Alpha factoring a big number...

I welcome you to submit a PR @gilcu3

I would go for either of the first 2 options, leaning towards the second. For the moment I cannot imagine any other type of function. For retrieve functions then it would make sense to keep their inputs and outputs as part of the session conversation.

AIChat now fully supports the function calling feature.

[x] Supports two function types: retrieve and execute
[x] Supports multi-calls and multi-steps
[x] Works within a role/session
[x] Have function filter
[x] Executes function calls in parallel


Is it possible to define defaultLocale at runtime for production?

Hi,
I tried to find a solution on stackoverflow, github, discord, but my question doesn't seem to be covered.
The initial problem I'm trying to solve
I need to define defaultLocale at runtime because I need to have translated urls without prefix.
The solution I found
I can launch one instance per locale with a custom defaultLocale.
The problem I'm facing
It works great in dev, process.env overwrite the settings. But after I build the app, process.env can't override the defaultLocale.
Is it possible to do it?
Is it a nuxt problem?
Should I also make one build per locale?
Thanks for your help

defaultLocale can affect the generation of routes which has to happen at build time.
So even if it would be possible to change it at runtime, it would likely make something not work as expected.

So the only workaround is to make one build per locale?

Currently I would say so, yes.

Thanks for making things clearer.


Md 4498 drill calcite1.16.0

@amansinha100 please review the PR. Thanks!

@amansinha100 I have addressed your review comments. Please take a look.

+1.  Before merging, pls change the commit message to something more meaningful for any_value.


showing as err even with the correct code.

Challenge Manipulate Arrays With unshift has an issue.
User Agent is: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

// Example
var ourArray = ["Stimpson", "J", "cat"];
ourArray.shift(); // ourArray now equals ["J", "cat"]
ourArray.unshift("Happy");
// ourArray now equals ["Happy", "J", "cat"]

// Setup
var myArray = [["John", 23], ["dog", 3]];
myArray.shift();
myArray.unshift(["paul", 35]);

// Only change code below this line.

@maheshareddy I think you need to change the last line to have a capital 'p' myArray.unshift(["Paul", 35]);

@maheshareddy The tests are case sensitive, so you need Paul, as has been pointed out.
In future, please visit the Help Rooms first for coding help.

@maheshareddy,
Thanks for reporting this. But you got the typo in the code. like @Manish-Giri say it.
These GitHub Issues are for reporting bugs on the freeCodeCamp website only.
Please use the HelpJavaScript chat room for getting challenge related help.
Happy Coding.


Make all backgrounds and borders partially transparent

When pressing on a message, the highlight wasn't looking good when there were elements with solid backgrounds and borders.
This PR makes sure they look the same as before, but use transparency.
Now the highlight looks sweet instead.

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!

I haven't had a chance to test this yet. My concern is that transparency could really slow the app down (especially if it's inside a ScrollView)

The color is drawn under the view, so no other way.
Also, only specific elements are transparent, not the whole views, so a low percentage of it is drawn as transparent.
As long as there is nothing in the background, it is possible that the perf is the same.
At least on the web, it is now relatively common to use transparency in that way. For example on Medium the text color is rgba(0, 0, 0, 0.8) and is not an issue.

I'm wary of making this change because scroll perf is so important to the app.
From https://developer.apple.com/library/content/documentation/2DDrawing/Conceptual/DrawingPrintingiOS/DrawingTips/DrawingTips.html:
"Compositing a view whose contents are opaque requires much less effort than compositing one that is partially transparent."
I'll test it and see if there's an impact but until then not approving this for merge.

Going to go ahead and merge. If it slows down perf we can always undo the changes


References to owner tabatkins should be replaced

This repository and the rendered documentation at https://speced.github.io/bikeshed/ includes references to tabatkins.github.io/bikeshed and github.com/tabatkins/bikeshed, which should be replaced with speced.github.io/bikeshed and github.com/speced/bikeshed (respectively).

I swear that I replaced all of these already, but apparently I hallucinated that. Fixed, thanks.


Code License

This repository should have a license file in the top level directory if it is in fact open-source as @debracleaver claims.

The MIT license has been added


3.x: Upgrade libs

Description
No real code changes, upgrading lib and plugin versions + moving to Java 17
Related Issue
https://github.com/Nosto/playcart/pull/31614
https://github.com/Nosto/playcart/pull/31470
Motivation and Context
Jackson, when solving some CVEs, had to migrate some classes and Beanie requires that the previous version is used (this upgrade will be done in a separate PR)
How Has This Been Tested?

Documentation:

Checklist:

[ ] I have updated the documentation accordingly.
[ ] I have assigned the correct milestone or created one if non-existent.
[ ] I have correctly labeled this pull request.
[ ] I have linked the corresponding issue in this description.
[ ] I have requested a review from at least 2 reviewers
[ ] I have checked the base branch of this pull request
[ ] I have checked my code for any possible security vulnerabilities

done as part of https://github.com/Nosto/beanie/pull/5


aezeed/errors: add missing space in error message

Made the asked/suggested changes.

@parth-choudhary Should also be squashed into one single commit, with the title
aezeed/errors: add missing space in error message

Squashed :)


Update Set-TransportConfig.md

This request is via Content Idea 78803:
The parameters, InternalDSNMaxMessageAttachSize and ExternalDSNMaxMessageAttachSize cannot be applied to ExO tenants.
We get the error "A parameter cannot be found that matches parameter name 'ExternalDsnMaxMessageAttachSize'." when we run it to online tenant.
According to the existing bug, these parameters don't work with online tenants: https://o365exchange.visualstudio.com/DefaultCollection/IP Engineering/_workitems/edit/119182
Please update the content to say those parameters are only for on-prem.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/mail-flow/Set-TransportConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/mail-flow/Set-TransportConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

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


Re-subscribe after blocking and deleting a subscription

Problem
When we try to resubscribe again after deleting a subscription that was blocked, we get a generic error "Error occurred during subscription" without mentioning the exact reason why it failed.
Solution
Change the  generic error message to a meaningful error similar to "Your subscription has been blocked, please contact your Adminstrator"
Affected Component
APIM
Version
4.1.0
Implementation
No response
Related Issues
No response
Suggested Labels
apim

The public fixes for this issue are as follows.

https://github.com/wso2/carbon-apimgt/pull/12299
https://github.com/wso2/apim-apps/pull/570


Feat/rk4

Coverage decreased (-0.8%) to 84.517% when pulling 1e2188f521181d64b9c91cf5559c43b7a2424e6b on feat/RK4 into b15343c9292014441cd010742b89aa3343850766 on master.

Hi Stas, the CARLsim3 "firingRateCPUandGPU_EULERvsRUNGE_KUTTA" test in coba.cpp is missing. That or an equivalent test should be included. Is there any more RK4 test that you wrote for CARLsim3?

Coverage increased (+0.2%) to 85.435% when pulling 7bd66f79a130710cac73349a4136aaf109eb6740 on feat/RK4 into b15343c9292014441cd010742b89aa3343850766 on master.

I've added the CUBA tests. I do not have data necessary to write TEST(COBA, firingRateVsData). If nobody minds I'd like to merge and move onto compartment model.


Add Aries Framework Go to Vendor Interoperability Tests

We may want to dockerize / cli-ify the client side of the interop tests... so that we can test N clients against M server implementations... today we are relying on 1 client (DBs edv client) and 1 server (Transmute's server).
How might we start adding support for aries framework go side by side the edv client?

If we had the edv-client as a dockerized CLI, it would be easy to substitute one implementation for another in the vendor interop tests.


HPX_ASSERT problem

When running this modified intersim benchmark against these modified composable guard files, something strange happens. The HPX_ASSERT(false) on line 113 allows the program to continue running, bypassing the abort() on line 114. This happens for RelWithDebugInfo, but not Debug. The breakme() function is there to give gdb an easy target to attach to.
HPX_ASSERT.zip

Except that the assert does do something, even though it's supposed to be disabled in that mode. Note above, the HPX_ASSERT(false) causes the code to bypass the abort() on the next line.

@stevenrbrandt I can't reproduce your problem, sorry. Could you give a more precise description of how to reproduce it?

I'll close this as being resolved. Please feel free to reopen if the problem persists.


Added full usage into README; Fixed minor markdown formatting issues

I found having to execute waifu2x-converter-cpp.exe --help every time to view the help page quite inconvenient. Therefore I propose migrating the command line help page into the README page.
It's nothing mission-critical, but maybe consider it as an option for enhancement
Changes:

Added all command line help page contents into README.
Minor edits to make the markdown format follow the conventions.

use ``` ``` and put the entire help in it.
The way it's currently done wastes a ton of space and looks kinda ugly.
Like:

   waifu2x-converter-cpp.exe  [--list-opencv-formats] [-l] [-f <png,jpg
                              ,webp,...>] [-c <0-9>] [-q <0-100>]
                              [--block-size <integer>] [--disable-gpu]
                              [--force-OpenCL] [-p <integer>] [-j
                              <integer>] [--model-dir <string>]
                              [--scale-ratio <double>] [--noise-level <0|1
                              |2|3>] [-m <noise|scale|noise-scale>] [-s]
                              [-r <bool>] [-o <string>] -i <string> [--]
                              [--version] [-h]

Where:

   --list-opencv-formats
     dump opencv supported format list

This is how waifu2x-caffe does it, and so is it how I do it.
I think the format is easy to look at, thus it gets the job done well.
If I were to just post the --help output .... what's the point writing it in markdown?

You can just add a help.txt on this repo too.  The markdown layout you posted just wastes 80% space

I would argue that including a full easy-to-read usage is beneficial, but alright.
I'll withdraw the changes.


Profiles follow users

Reusing the grid from projects view for main feed, which displays followed projects and users. Also in profile view, which shows the user's card as well as that user's owned projects. follow buttons work on profile cards and projects cards. We can restyle and display the data differently next week, but all the information is there and available to work with in each view/component

Added in with the Search PR. Please review and approve that one.


[NVSHAS-5843] Adding Documentation for Federated ConfigMap YAML

Adding documentation for the Federated ConfigMap primary and managed clusters. Also updated the latest ConfigMap link to https://raw.githubusercontent.com/neuvector/manifests/main/kubernetes/5.4.0/initcfg.yaml, and updated the sample initcfg.yaml. Related to NVSHAS-5843.

Who will update https://open-docs.neuvector.com/ about the changes in the PR?

Who will update https://open-docs.neuvector.com/ about the changes in the PR?

Thank you for reviewing! I've addressed the feedback in this commit: https://github.com/neuvector/docs/pull/134/commits/150fcfa6f883e013091dc3098f80d9f5c261c08a
For clarification this PR is handling the content update for the documentation at https://open-docs.neuvector.com/, and I will be handling the port of this PR to the product docs (https://documentation.suse.com/cloudnative/security/5.4/en/overview.html) as well once approved.


ListAssistants returning 400 error because of content-type

I'm getting the following error when calling ListAssistants
error, status code: 400, message: Unsupported content type: 'application/json; charset=utf-8'. This API method only accepts 'application/json' requests, but you specified the header 'Content-Type: application/json; charset=utf-8'. Please try again with a supported content type.

It looks like removing ; charset=utf-8 from the content type header inside the sendRequest() function in client.go fixes it.

Got the same issue with CreateAssistant method. I've also found this topic, it seems to be common problem today with community openai libs.

This issue should be resolved because it has been fixed there #659


Include nltk assets in conda env

It would be great if we could have the deployed conda environment include useful nltk assets downloaded by default.
e.g.
punkt
maxent_treebank_pos_tagger
average_perceptron_tagger
universal_tagset
vader_lexicon
stopwords
maxent_ne_chunker
etc

thanks for logging this. Marking as 'enhancement' as this is under consideration.

This request is likely in the same bucket as enabling download of arbitrary conda and pip packages as well. The mechanism is likely the same - privileged operations to download the packages or data payloads prior to any user code executing in the environment.

Updated. See https://support.microsoft.com/en-us/office/open-source-libraries-and-python-in-excel-c817c897-41db-40a1-b9f3-d5ffe6d1bf3e for details. Thx.


int4 type support for matmul

Hello,
I am working on hardware-accelerated matmul with int4 input tensors and get back an int16/int32 tensor.
I would like to be able to run matmul and emit MHLO with example below (or let me know if there are better options):
x=jnp.array(100).reshape((1,1)).astype(jnp.int4)
jnp.matmul(x,x)

Thanks!

cc @hawkinsp

This is fixed these days!
In [5]: import jax, jax.numpy as jnp, numpy as np
In [8]: x=np.array(100).reshape((1,1)).astype(jnp.int4)
   ...: print(jax.jit(jnp.matmul).lower(x,x).as_text())
module @jit_matmul attributes {mhlo.num_partitions = 1 : i32, mhlo.num_replicas = 1 : i32} {
  func.func public @main(%arg0: tensor<1x1xi4> {mhlo.sharding = "{replicated}"}, %arg1: tensor<1x1xi4> {mhlo.sharding = "{replicated}"}) -> (tensor<1x1xi4> {jax.result_info = ""}) {
    %0 = stablehlo.dot_general %arg0, %arg1, contracting_dims = [1] x [0], precision = [DEFAULT, DEFAULT] : (tensor<1x1xi4>, tensor<1x1xi4>) -> tensor<1x1xi4>
    return %0 : tensor<1x1xi4>
  }
}

(There's no guarantee any given JAX backend knows how to compile that, but producing the stablehlo is no problem.)


[FR]: Whisper int4 support

Proposal Summary
intel int4 quantization
https://huggingface.co/Intel/whisper-large-onnx-int4
Intel supports whisper int4 quantization using the MatMulFpQ4 operator of onnxruntime 1.16.0, and I saw that llama supports using OnnxMatMul4Quantizer to complete int4 quantization. Can whisper also support OnnxMatMul4Quantizer to complete int4 quantization?
What component(s) does this request affect?

[X] OliveModels
[X] OliveSystems
[X] OliveEvaluator
[X] Metrics
[X] Engine
[X] Passes
[X] Other

I did not compare the latency with fp32/fp16, but the int4 should work for olive whisper example, you can just change the int8 dynamic quantization as above config.

@trajepl Thanks I will try it later

Hey it works with a really great result. Whisper-int4

Hey. I made rud test
Driver Description:          NVIDIA GeForce GTX 1660 Ti with Max-Q Design
Video Memory:                6144 MBytes of GDDR6 SDRAM [Micron]
1272-141231-0002.mp3
File duration                                   : 13s 428ms
whisper-large-v2-int8
Transcribe duration avg			: 16s 713ms
realtime ratio 0.803458437
whisper-large-v2-int4
Transcribe duration avg 			: 2s 963ms
faster in 5.640482619 times
realtime ratio 4.531893351
I tested in C# demo app

Wow, great to see this big improvements!

A simple experiment:
model: whisper-medium
experiment: 15s audio running 10 times.
device	bit  running time  cpu core/gpu util	memory
cpu 	int8 45s 	   i9 24 core 		4.1G
cpu 	int4 63s 	   i9 24 core 		2.9G
gpu 	int8 57s 	   4090 39%     	2.7G
gpu 	int4 7s  	   4090 80~90%     	3.8G
in conclusion:
Running int4 on the CPU seems to be limited by the CPU processing speed, resulting in 40% slower than int8 and 1G less memory.
Int4 on the GPU runs 7 to 8 times faster than int8. At the same time, the computing resource usage is close to 100%. The memory has increased by 1G. Int8 cannot fully utilize the performance of 4090.
Overall, Int4 should be faster but requires more computing resources.

I ran the inference of the original model (GPU) through the following command. The speed of audio inference for 5 minutes is similar to that of int4 onnx (GPU), and the memory usage is about twice as much. Is there any problem with the int8 onnx model (GPU) that prevents full use of gpu resources?
gpu inference (4090 utilization 90%):
time whisper 1.wav --language zh --model medium
real 0m28.003s
user 0m40.866s
sys 0m3.852s

Thanks for the comparison! Currently, the int8 is not supported very well in onnxruntime gpu. But fortunately, the supporting is coming.
I do not know the detailed reason whey int8 does not fully utilize the gpu memory. But I suppose it might be caused by some operations(quantization) are not optimized in gpu.
Also, we have similar experiences before, that leads following convention:

For gpu, int4, fp16 are better, and avoid apply int8 quantization as of now.
For cpu, avoid fp16. For int4 there are several passes to get the int4 model, we need to search for better performance.

"blockwise_quant_int4":{
"type": "OnnxMatMul4Quantizer",
"disable_search": true
},
"bnb_quantization": {
"type": "OnnxBnb4Quantization",
"config": {
"quant_type": "nf4",
"save_as_external_data": true,
"all_tensors_to_one_file": true
}
},
bnb_quantization fp4 has better performance than nf4 and blockwise_quant_int4. Compared with blockwise_quant_int4, the performance is improved by 40%.
15s audio running 50 times.

bit
model size
running time
running memory
gpu util

fp4
636M
22s
3.62G
95%

nf4
636M
24s
3.67G
95%

int4
681M
34s
36.4G
85%~95%

Excellent! Will try to add the official example in Olive later.

The whisper variants I currently use include faster-whisper, whisper.cpp, and olive-whisper. If sorted by inference speed (gpu nvidia 4090), these three are almost the same. If sorted by memory usage (from small to large), faster-whisper->whisper.cpp->olive-whisper.
The memory footprint of whisper.cpp or faster-whisper is half that of olive-whisper.
Is there a way to optimize the memory footprint of olive-whisper?
I prefer using the onnx model and don't want to give up the olive whisper model.

I have no idea about how to optimize memory footprint.
It might not be a fair comparison? The factors like input data size, batch size, or memory cache may affect the memory footprint.

Closing issue since int4 quantization is already supported.
Memory footprint is out of scope for Olive and handled my onnxruntime https://github.com/microsoft/onnxruntime.
There have been multiple int4 improvements made in ORT so it might have already been addressed by them.

@jambayk  Can you provide a complete example of int4 at https://github.com/microsoft/Olive/tree/main/examples/whisper ? I tried to quantize whisper-tiny with int4 myself, but the ASR results were not satisfactory with some wrong. The core configuration is as follows:
"blockwise_quant_int4": {
"type": "OnnxMatMul4Quantizer",
"disable_search": true
}"


Bugfix: Fixes go install due to replace directive

Signed-off-by: yue9944882<EMAIL_ADDRESS>

@qiujian16 i moved the dirname from cmd to clusteradm on purpose, otherwise the installed binary in the gopath will be named cmd instead of clusteradm


IE11 Does not support classList on SVG

Been seeing a few errors in IE11 caused by missing classList support on SVG elements:
dom-helpers.js
export function isNodeFound(current, componentNode, ignoreClass) {
  if (current === componentNode) {
    return true;
  }
  // SVG <use/> elements do not technically reside in the rendered DOM, so
  // they do not have classList directly, but they offer a link to their
  // corresponding element, which can have classList. This extra check is for
  // that case.
  // See: http://www.w3.org/TR/SVG11/struct.html#InterfaceSVGUseElement
  // Discussion: https://github.com/Pomax/react-onclickoutside/pull/17
  if (current.correspondingElement) {
    return current.correspondingElement.classList.contains(ignoreClass);
  }
  return current.classList.contains(ignoreClass);
}

I guess the quick fix is to just check for "classList" support, and return false if not supported.
Otherwise you'd need a IE11 specific class lookup method.

https://github.com/Pomax/react-onclickoutside#ie-does-not-support-classlist-for-svg-elements
However, I will prepare a new version of the library soonish, which might not have this problem on default settings. No promises yet though.
In order to fix this in my application I'm using this dirty hack:
if (!('classList' in SVGElement.prototype)) {
  Object.defineProperty(SVGElement.prototype, 'classList', {
    get() {
      return {
        contains: className => {
          return this.className.baseVal.split(' ').indexOf(className) !== -1
        },
      }
    },
  })
}

Ah, sorry - Missed that note. Just thinking it would be better to ignore the "ignoreClass" on IE11, rather than throwing an unhandled error.
Thanks for the shim, i'll use that to fix it for now. 👌


cannot find symbol

Seems related to #6, but the suggested fixes don't seem to work with latest react-native 0.61.2.
Error: Command failed: ./gradlew app:installDebug -PreactNativeDevServerPort=8081
/home/app/android/app/build/generated/rncli/src/main/java/com/facebook/react/PackageList.java:78: error: cannot find symbol
      new HotspotModule()
          ^
  symbol:   class HotspotModule
  location: class PackageList
1 error

Tried adding package manually to MainApplication.java without success:
@Override
    protected List<ReactPackage> getPackages() {
      @SuppressWarnings("UnnecessaryLocalVariable")
      List<ReactPackage> packages = new PackageList(this).getPackages();
      // Packages that cannot be autolinked yet can be added manually here, for example:
      packages.add(new HotspotPackage());
      return packages;
    }

Any suggestions?

did you import the package name at the top of MainApplication.java before adding it to packages?!

Yup @assemmohamedali!

I don't think you add it correctly, could you please re-try? read this as well https://github.com/react-native-community/cli/blob/master/docs/autolinking.md

Got it to work! :tada:
Disabled autolinking as documented here.
Details on manually adding it can be found here.

This worked for me as well running react-native 0.61.5, thanks!


sql queries should be constructed using querybuilder object

All sql queries should be created using a query builder from pypika

https://github.com/kayak/pypika

Also explore: https://www.sqlalchemy.org/


Basketball Template fixes

NBA
Yes/No
Add NBA:  before all NBA Templates
Multiple choice- add NBA: (to the beginning of)
Which team will win: [Team A] vs [Team B], Estimated schedule start time: [Date time]?
[Team A] vs [Team B]: Total Points scored; Over/Under [Whole #].5, Estimated schedule start time: [Date time]?
Which Player will have the most [Action] at the end of the the [Year Range] regular season?
Scalar: Add NBA:(before Total...)
Total number of wins [Team] will finish [Year Range] regular season with?
Need to add added resolution details: (no check and checkbox)
Regular Season win totals are for regular season games ONLY and will not include any play-in, playoffs, or championship games
We need to remove resolution details boxes from all template markets:
Remove check and checkbox from all added resolution detail
NCAA BB
Need to add NCAA BB: (in front of all yes/no, and multiple choice NCAA templates)
Create input pre-pop picker list from
https://www.ncaa.com/standings/basketball-men/d1
WNBA tab need to be added
use https://www.espn.com/wnba/teams for dropdown list
Same templates as NBA.....Same Yes/No markets but add WNBA: (before every Yes/no template and first 4 multiple choice market....Within the templates wherever it says NBA change to WNBA.
Use all same Added Resolution rules as NBA templates
Remove check and checkbox for all added resolution rules
Remove resolution details from every template

Yes/No For the template Will [Name] win the [Year] [Award] award?
Need to add Added resolution Rules (no check or checkbox)
In the event of an award given to more than 1 player. If the player mentioned in the market is one of the players who wins the award, the market should resolve as yes.
Multiple choice
Which NBA player will win the [Year Range] [Award] award?
Need to add Added resolution Rules (no check or checkbox)
In the event of an award is given to more than 1 player.  The player who averaged the most points per game (determined to the first decimal place, meaning a player averaging 36.1 points per game would win over the player averaging 36 points per game) for the regular the season will be the tie breaker.  In the event of an additional tie, The regular season Field Goal % will be the final tie breaker.
***currently has an incorrect added resolution rule that needs to be removed
please remove 'includes regulation and overtime from this template
Which Player will have the most [Action] at the end of the the [Year Range] regular season?
***currently has an incorrect added resolution rule that needs to be removed
please remove 'includes regulation and overtime from this template
Need to Add Added Resolution Rule
Results are determined to the first decimal place, meaning a player averaging 10.6 rebounds per game is higher than a player averaging 10.0.
In the event of a tie between two players these rules will be used to determine the winner:
For points scored, field goal % should be used as the tie breaker.
For the most rebounds, the player with the total most offensive rebounds, should be used as a tie breaker.
For most total Assists, The player with the "Least" amount of turnovers should be used as the tie breaker.
For most made 3-pointers, the player with the highest 3 point %, should be used as the tie breaker.


Intermittent delay of minutes between ESP publish and receipt by mosquitto in a pi

I'm seeing an intermittent 6 minute or so delay between a publish on an ESP32 S3 and the message showing up in wireshark on the destination Raspberry Pi. My original program published a set of environmental data every minute. The program at the bottom is a  modification of that program intended to focus in on this problem. It runs a loop and publishes a loop counter and some timing info to help debugging.
Specifically what I am seeing is an intermittent delay of all published messages of about 6  minutes. Then all the messages show up at once in wireshark. The publishes are typically delivered in one packet. The delay is often 6 min 22 sec but not always. Sometimes it is a single batch that is delayed with normal delivery after the one oversized and late packet. Sometimes there is a sequence of delayed oversized packets.
My setup is an ESP32-S3 for data collection. Mosquitto (v1.5.7) and Nodered are running on a raspberry pi 400. Using a pi zero to collect the data and publish it at the same frequency doesn't have a problem. Publishing to a Pi 4 with Mosquitto 2.0.11 doesn't have the same problem. Other ESPs do have the same problem. I'm suspecting a Mosquitto configuration problem on the pi, but I think I'm running a vanilla version of Mosquitto.
Here's wireshark output of a normal publish followed by one that was delayed. These are the messages from the ESP to the Pi captured on the Pi.  Each cycle thru the loop publishes 4 times. The cycle time is 2 minutes. So the second batch would be expected as 12:18;11. Instead the publishes for times 12:18:11 and 12:20:11 show up at 12:21:21.  And messages that were generated 2 minutes apart show up in the same packet. I have instrumented the code and have verified that the loop starts are exactly as expected.
Time             len      message
12:16:11.611	83	Publish Message [loop count ESP to 128]
12:16:11.613	217	Publish Message [t1], Publish Message [t2], Publish Message [debugstring]
12:21:21.851	83	Publish Message [loop count ESP to 128]
12:21:21.895	405	Publish Message [t1], Publish Message [t2], Publish Message [debugstring], Publish Message [loop count ESP to 128], Publish Message [t1], Publish Message [t2], Publish Message [debugstring]
I have more data. For example the code captures the duration of the first 3 publishes in the loop. That duration drops significantly (about half for the first publish) for messages that are delayed.
Hopefully someone has seen this before and knows what is happening.
Here is a version of the code. I've used 1 minute, 2 minute, 5 minute and 10 minute loop durations with no significant difference.
mqtt_delay_debug_r2.txt

I am seeing a different intermittent variable delay of the publishes from the ESP to a Raspberry pi zero mosquitto server. I'm using wireshark on the pi zero to capture the MQTT protocol messages on the pi zero. The code is essentially the same as above. I have tested and confirmed that the ESP loop runs at intervals of just over 60 seconds. But the arrival of the messages at the pi zero interface has periods when the message is delayed by up to half a second.
The plot below shows about 90 minutes of execution. Sometimes there is no delay (10 to 17) and all messages arrive within a few milliseconds.  When there is a delay, the arrival time follows a systematic pattern.

This delay I can live with. The earlier delay of up to 15 minutes I'm beginning to suspect is something in the Mosquitto configuration running on a Pi 400. I've tried other combinations without seeing the 15 minute delay (multiple Espressif ESP32-s3, Seed XIAO with ESP32-s3 as sources and mosquitto running on a Pi400, a Pi zero and a Pi 4 with newer version of mosquitto).   I'm going to upgrade that pi to the current OS. That will get me a fresh configuration and hopefully I won't see the problem again.

Found a separate quirk. The one above I have only seen with Mosquitto running on my Raspberry Pi 400. When Mosquitto runs on  a Pi zero or a Pi 4 (later version of mosquitto) I do not see the multi minute delay.
But I do see a delay of up to 0.6 seconds that follows a strange pattern. The program is a minor derivative of the one above. It publishes 4 topics at 60 second intervals. Several tests confirm that the loop timing is stable. When use Wireshark on the platform that is running Mosquitto I see the following pattern in the time of the MQTT protocol message containing the publish.


MissingPluginException(No implementation found for method checkPermission on channel flutter.baseflow.com/geolocator)

MissingPluginException(No implementation found for method checkPermission on channel flutter.baseflow.com/geolocator)
getting this issue in the production build

I too got this in production release.

@mvanbeusekom look into this

I got it too.
I dropped the version of geolocato_android to 3.1.5 and it was OK.

That is a bit strange. Do you add platform specific dependencies?
All you would need in general is to depend on the geolocator package and it will pull in the necessary platform packages. This should also prevent versioning problems.

I dropped the version of geolocato_android to 3.1.5 and it was OK.

Solved the problem for me. Thanks!

geolocato_android

I am not using a native plugin, I used the flutter package geolocator: ^8.2.0

Any solutions @patildarshan66

Doing more detailed research we found out the this exception occurs when calling the getCurrentPosition from a background task (either the Workmanager or a separate Isolate).
This reason this is happening is two fold:

We refactored the code to comply to the latest architectural concepts for Flutter plugins (fully federated architecture) where platform specific packages (geolocator_android, geolocator_apple, etc..) are responsible for maintaining their own communication layer between Dart and the native platform.
When running a background task there is no Flutter engine and therefore platform specific plugins are not registered with the native platform.

Note: it is good to realize that the main (a.k.a. app-facing) geolocator package internally depends on the platform specific packages. So even if the app only depends on the geolocator package, internally a dependency is created on the geolocator_android, geolocator_apple, etc... packages. To verify the exact versions used check the contents of the pubspec.lock file.

On pre-Flutter 2.11, the workaround for this is to manually register the native platform implementations:
//callback function
void callbackDispatcher() {
  Workmanager().executeTask((task, inputData) async {
   if (defaultTargetPlatform == TargetPlatform.android) {
     GeolocatorAndroid.registerWith();
   } else if (defaultTargetPlatform == TargetPlatform.iOS || defaultTargetPlatform == TargetPlatform.macOS) {
     GeolocatorApple.registerWith();
   } else if (defaultTargetPlatform == TargetPlatform.linux) {
     GeolocatorLinux.registerWith();
   }

   await LocationService.getCurrentLocation();
    print("Native called background task-1: $task");

    return Future.value(true);
  });
}

Alternatively if you are running Flutter 2.11+, you can use the new DartPluginRegistrant.ensureInitialized() method to ensure all packages are registered correctly:
//callback function
void callbackDispatcher() {
  Workmanager().executeTask((task, inputData) async {
   DartPluginRegistrant.ensureInitialized();

   await LocationService.getCurrentLocation();
    print("Native called background task-1: $task");

    return Future.value(true);
  });
}

More information can be found here and here.

I dropped the version of geolocato_android to 3.1.5 and it was OK.

this solved my issue, just check on your pobspec.lock then change the version

I dropped the version of geolocato_android to 3.1.5 and it was OK.

this solved my issue, just check on your pobspec.lock then change the version

This is not the best solution, please check out the comment above and ensure you register the platform specific implementations. Going forward this will be the stable solution and new versions will rely on this.
Of course if you are fine using an outdated version and pin versions in your pubspec.yaml, feel free to do so. However I strongly suggest refraining from doing so and follow instructions in the mentioned comment.

This is not the best solution, please check out the comment above and ensure you register the platform specific implementations. Going forward this will be the stable solution and new versions will rely on this.
Of course if you are fine using an outdated version and pin versions in your pubspec.yaml, feel free to do so. However I strongly suggest refraining from doing so and follow instructions in the mentioned comment.

for your info, for my problem ... i didnt put any geolocator process on background or isolated task.
but yeah thank you for your suggestion

Check these lines of codes if they are present in your code or not. Adding them will resolve your issue:

android/app/src/main/AndroidManifest.xml

    <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
    <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />

ios/runner/info.plist

NSLocationWhenInUseUsageDescription
This app needs access to location when open.
NSLocationAlwaysUsageDescription
This app needs access to location when in the background.

3. In your getLocation() add these 2 lines:

await Geolocator.checkPermission();
await Geolocator.requestPermission();

After doing all these steps do a Hot Restart or Stop all and Run again.


Add thewchan as maintainer

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

noarch: python recipes are required to have a lower bound on the python version. Typically this means putting python >=3.6 in both host and run but you should check upstream for the package's Python compatibility.

For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@zhaofeng-shu33
@conda-forge/core
Another package need an older version of this package as a dependency, please add me as a maintainer so I can create that version on the feedstock


Add agenda for Cranelift meeting on 2021-08-09.

Others please feel free to add items to the agenda as desired!
cc @abrown @jlb6740 @fitzgen @uweigand @bnjbvr @bjorn3 and any others who plan to attend

Merging without CI (I wonder if we could configure GitHub to skip it when only meetings/ changes?).


Add hlint to CI to do static code analysis for consensus code

HLint can be added to shell.nix
CI check can be done in buildkite (see ouroboros-network/.buildkite/pipeline.yml)

Close as done


source points to the wrong directory in MSYS2 when Cygwin is installed

I have both Cygwin and MSYS2 installed on my PC.  I'm unable to launch oh-my-zsh after it has been installed successfully on one versus the other.
wayne@0620-DESKTOP-PC MSYS ~/development/hifi-fork
$ zsh
/c/Users/wayne/.zshrc:source:69: no such file or directory: /cygdrive/c/Users/wayne/.oh-my-zsh/oh-my-zsh.sh
\[\e]0;\w\a\]\n\[\e[32m\]\u@\h \[\e[35m\]$MSYSTEM\[\e[0m\] \[\e[33m\]\w\[\e[0m\]\n\$
\[\e]0;\w\a\]\n\[\e[32m\]\u@\h \[\e[35m\]$MSYSTEM\[\e[0m\] \[\e[33m\]\w\[\e[0m\]\n\$ exit

If I perform these steps I can get it working.

Is it because there can only be one or the other linked to oh-my-zsh?

It's possible the $ZSH variable wasn't set correctly, you should see its value at the top of the .zshrc file.


fix: improve parsing of Package.gz metadata, matching Debian parser logic

Problem
On Ubuntu 24.04, and likely other distributions  - a Package is able to be published to Ubuntu repositories with non-standard formatting.
 In this case, the improper formatting here implies an empty value set for X-Cargo-Built-Using and then tries to do key/val parsing logic on the list of dependencies used to compile the package in question.
This results in the following error when running:
INFO: Repository ubuntu_noble_resolution instantiated at:
  mypath/WORKSPACE:3:10: in <toplevel>
  /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/index.bzl:65:17: in deb_index
Repository rule deb_resolve defined at:
  /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl:135:30: in <toplevel>
ERROR: An error occurred during the fetch of repository 'ubuntu_noble_resolution':
   Traceback (most recent call last):
        File "/home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl", line 74, column 33, in _deb_resolve_impl
                pkgindex = package_index.new(rctx, sources = sources, archs = manifest["archs"])
        File "/home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/package_index.bzl", line 73, column 33, in _create
                _parse_package_index(state, rctx.read(output), arch, url)
        File "/home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/package_index.bzl", line 36, column 26, in _parse_package_index
                (key, value) = line.split(": ", 1)
Error: too few values to unpack (got 1, want 2)
ERROR: mypath/WORKSPACE:3:10: fetching deb_resolve rule //external:ubuntu_noble_resolution: Traceback (most recent call last):
        File "/home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl", line 74, column 33, in _deb_resolve_impl
                pkgindex = package_index.new(rctx, sources = sources, archs = manifest["archs"])
        File "/home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/package_index.bzl", line 73, column 33, in _create
                _parse_package_index(state, rctx.read(output), arch, url)
        File "/home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/package_index.bzl", line 36, column 26, in _parse_package_index
                (key, value) = line.split(": ", 1)
Error: too few values to unpack (got 1, want 2)
ERROR: no such package '@@ubuntu_noble_resolution//': too few values to unpack (got 1, want 2)
ERROR: /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/ubuntu_noble/BUILD.bazel:2:6: @@ubuntu_noble//:lock depends on @@ubuntu_noble_resolution//:lock in repository @@ubuntu_noble_resolution which failed to fetch. no such package '@@ubuntu_noble_resolution//': too few values to unpack (got 1, want 2)
ERROR: Analysis of target '@@ubuntu_noble//:lock' failed; build aborted: Analysis failed
INFO: Elapsed time: 17.174s, Critical Path: 0.14s
INFO: 1 process: 1 internal.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

Solution
This PR complements the issue I've opened up here: https://github.com/GoogleContainerTools/rules_distroless/issues/42
This PR sanitizes package metadata when parsing the Packages.gz archive on a given Debian/Ubuntu repositiory to allow for newlines and other characters which may follow a : in keyval definition.
Verification Evidence
rules_distroless$ git diff origin/main -- apt/
diff --git a/apt/private/package_index.bzl b/apt/private/package_index.bzl
index 620a021..191d18c 100644
--- a/apt/private/package_index.bzl
+++ b/apt/private/package_index.bzl
@@ -28,7 +28,17 @@ def _parse_package_index(state, contents, arch, root):
                 pkg[last_key] += "\n" + line
                 continue

-            (key, value) = line.split(": ", 1)
+            # This allows for (more) graceful parsing of Package metadata (such as X-* attributes)
+            # which may contain patterns that are non-standard. This logic is intended to closely follow
+            # the Debian team's parser logic:
+            # * https://salsa.debian.org/python-debian-team/python-debian/-/blob/master/src/debian/deb822.py?ref_type=heads#L788
+            split = line.split(":")
+            key = split[0]
+            value = ""
+
+            if len(split) == 2:
+                value = split[1]
+
             if not last_key and len(pkg) == 0 and key != "Package":
                 fail("do not expect this. fix it.")

$ bazel run --sandbox_debug --override_repository=rules_distroless=$HOME/git/alexconrey/rules_distroless  @ubuntu_noble//:lock
INFO: Analyzed target @@ubuntu_noble//:lock (91 packages loaded, 742 targets configured).
INFO: Found 1 target...
Target @@ubuntu_noble_resolution//:lock up-to-date:
  bazel-bin/external/ubuntu_noble_resolution/lock
INFO: Elapsed time: 3.277s, Critical Path: 0.13s
INFO: 5 processes: 5 internal.
INFO: Build completed successfully, 5 total actions
INFO: Running command line: bazel-bin/external/ubuntu_noble_resolution/lock external/ubuntu_noble_resolution/lock.json

Writing lockfile to ubuntu/24.04/packages.lock.json

Run the following command to add the lockfile or pass --autofix flag to do it automatically.

   buildozer set lock @@//ubuntu/24.04:packages.lock.json WORKSPACE.bazel:ubuntu_noble

I've just signed the CLA on behalf of myself for my GitHub user. Let me know if there's anything I can do to get this moved along!

Could you provide more information on why this is necessary?

@thesayyn certainly! I have updated the PR to contain more details around the nature of this fix. I did include a bit more information in my issue that I am happy to copy/paste over here for completeness.

Okay, ignoring the lines start with X sounds like workaround. looks like this is successfully parsed by other parsers, we should fix the parser to allow there to be nothing after the :.
https://salsa.debian.org/python-debian-team/python-debian/-/blob/master/src/debian/deb822.py?ref_type=heads#L788
https://regex101.com/r/BrBMqo/1

Something like this should work:
split = line.split(":")
key = split[0]
value = ""
if len(split) == 2:
   value = split[1]

Thanks @thesayyn ! That was an immense help. I was able to get a successful run against my lockfile with those changes. If you wouldn't mind glancing at this again, it would be appreciated! The only addition I added to your logic was the stripping of leading whitespace. I thought it was causing issues when running locally, but that was not the case. It seems like a good practice to strip that out, but happy to remove that logic if necessary.

Looks great, stripping sounds good but i am afraid it might break some other code path as parsing doesn't have a unit test for edge cases. Can we remove that part?

Removed!

Some buildifier errors on CI

It does seem that I am able to reproduce the whitespace issue consistently (making me think I fell victim to caching on my prior comment.) I am now cleaning my workspace as a means of verification:
without whitespace trim:
$ bazel clean --async --expunge; bazel run --sandbox_debug --override_repository=rules_distroless=$HOME/git/alexconrey/rules_distroless  @ubuntu_noble//:lock
INFO: Starting clean.
INFO: Output base moved to /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4_tmp_444983_6dfe942f-cdb5-494e-93d1-ebd46e153625 for deletion
Starting local Bazel server and connecting to it...
INFO: Repository ubuntu_noble_resolution instantiated at:
  mypath/WORKSPACE:3:10: in <toplevel>
  /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/index.bzl:65:17: in deb_index
Repository rule deb_resolve defined at:
  /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl:135:30: in <toplevel>
ERROR: An error occurred during the fetch of repository 'ubuntu_noble_resolution':
   Traceback (most recent call last):
        File "/home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl", line 92, column 21, in _deb_resolve_impl
                fail("Unable to locate package `%s`" % dep_constraint)
Error in fail: Unable to locate package `ncurses-base`
ERROR: mypath/WORKSPACE:3:10: fetching deb_resolve rule //external:ubuntu_noble_resolution: Traceback (most recent call last):
        File "/home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl", line 92, column 21, in _deb_resolve_impl
                fail("Unable to locate package `%s`" % dep_constraint)
Error in fail: Unable to locate package `ncurses-base`
ERROR: no such package '@@ubuntu_noble_resolution//': Unable to locate package `ncurses-base`
ERROR: /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/ubuntu_noble/BUILD.bazel:2:6: @@ubuntu_noble//:lock depends on @@ubuntu_noble_resolution//:lock in repository @@ubuntu_noble_resolution which failed to fetch. no such package '@@ubuntu_noble_resolution//': Unable to locate package `ncurses-base`
ERROR: Analysis of target '@@ubuntu_noble//:lock' failed; build aborted: Analysis failed
INFO: Elapsed time: 27.970s, Critical Path: 0.21s
INFO: 1 process: 1 internal.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

with whitespace trim:
$ bazel clean --async --expunge; bazel run --sandbox_debug --override_repository=rules_distroless=$HOME/git/alexconrey/rules_distroless  @ubuntu_noble//:lock
INFO: Starting clean.
INFO: Output base moved to /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4_tmp_443909_835417bd-715d-4781-91b6-8ff2b659b67e for deletion
Starting local Bazel server and connecting to it...
DEBUG: /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/package_resolution.bzl:141:22: Warning: optional dependencies are not supported yet. https://github.com/GoogleContainerTools/rules_distroless/issues/27
DEBUG: /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl:96:22: the following packages have unmet dependencies: ,awk
DEBUG: /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl:96:22: the following packages have unmet dependencies:
DEBUG: /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl:96:22: the following packages have unmet dependencies:
DEBUG: /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/package_resolution.bzl:141:22: Warning: optional dependencies are not supported yet. https://github.com/GoogleContainerTools/rules_distroless/issues/27
DEBUG: /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl:96:22: the following packages have unmet dependencies: ,,,
DEBUG: /home/ubuntu/.cache/bazel/_bazel_ubuntu/c72e88c157bca566b19afcd933de7ae4/external/rules_distroless/apt/private/resolve.bzl:96:22: the following packages have unmet dependencies: ,
INFO: Analyzed target @@ubuntu_noble//:lock (91 packages loaded, 742 targets configured).
INFO: Found 1 target...
Target @@ubuntu_noble_resolution//:lock up-to-date:
  bazel-bin/external/ubuntu_noble_resolution/lock
INFO: Elapsed time: 38.654s, Critical Path: 0.18s
INFO: 5 processes: 5 internal.
INFO: Build completed successfully, 5 total actions
INFO: Running command line: bazel-bin/external/ubuntu_noble_resolution/lock external/ubuntu_noble_resolution/lock.json

Writing lockfile to ubuntu/24.04/packages.lock.json

Run the following command to add the lockfile or pass --autofix flag to do it automatically.

   buildozer set lock @@//ubuntu/24.04:packages.lock.json WORKSPACE.bazel:ubuntu_noble

Alright, we should be good for (another) review. Thanks for the whitespace spot, I would have been here for a few more hours 😝
Buildifier is showing no changes after making these updates, CI should hopefully reflect the same.

Thank you!


FreeplayState capules being horrible on low fps

Describe your bug here.
the title and I'm doing mobile port btw but I also tested on low fps in my laptop it does the same thing
https://github.com/user-attachments/assets/ce0a4750-fa0c-44c0-9a05-3d942c006b7d
Command Prompt/Terminal logs ()
No response
Are you modding a build from source or with Lua?
No modding
What is your build target?
Neko, HashLink, or other build system
Did you edit anything in this build? If so, mention or summarize your changes.
No, I didn't

I was able to fix it in an upcoming release


feat: Added a scrollView in recover wallet

Fixes #103
Added a scrollView to the recover wallet and brought wallet recovery word list in one column.

Done with #109.


fix: suppress deprecated warning for production

Types of changes

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

Description
This PR is a fix to this issue.
We should not need this warning in the production build.
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[x] All new and existing tests are passing.

Security Audit is failed. Is there anything I can do for that?
https://circleci.com/gh/nuxt/nuxt.js/31459?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

Codecov Report

Merging #5137 into dev will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              dev   #5137      +/-   ##
=========================================
- Coverage   95.08%     95%   -0.09%
=========================================
  Files          72      72
  Lines        2421    2421
  Branches      614     614
=========================================
- Hits         2302    2300       -2
- Misses         99     101       +2
  Partials       20      20

Impacted Files
Coverage Δ

packages/vue-renderer/src/renderer.js
94.81% <0%> (-1.04%)
:arrow_down:

Continue to review full report at Codecov.

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

Hey 👋 Thanks for the contribution Left you some comments :)

LGTM ☺

Thank you 🙏 please merge it.


Error updating the App Service from 3.0 to 3.1.

Have you added the GENERATE_SOURCEMAP to your application settings in the app service as outlined in the upgrade instructions?
Reference Link: https://github.com/OfficeDev/microsoft-teams-emergency-operations-center/wiki/Upgrade-to-Latest-Version#5-add-new-application-setting-generate_sourcemap-in-the-app-service

I believe I have, the panel is different now than the documentation.  Application Settings no longer seems to be under the Configuration blade.
[cid:a6e693bf-a48a-4f2e-b23b-aee8dd307a7a]

From: v-ajaysahu @.>
Sent: Tuesday, August 13, 2024 02:18
To: OfficeDev/microsoft-teams-emergency-operations-center @.>
Cc: Trevor Thorsteinson @.>; Mention @.>
Subject: Re: [OfficeDev/microsoft-teams-emergency-operations-center] Error updating the App Service from 3.0 to 3.1. (Issue #231)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
@ToC-Tthorhttps://github.com/ToC-Tthor , thanks for reporting this issue.
Have you added the GENERATE_SOURCEMAP to your application settings in the app service as outlined in the upgrade instructions?
Reference Link: https://github.com/OfficeDev/microsoft-teams-emergency-operations-center/wiki/Upgrade-to-Latest-Version#5-add-new-application-setting-generate_sourcemap-in-the-app-service
—
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/microsoft-teams-emergency-operations-center/issues/231#issuecomment-2285646004, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BKPOLCVKIYWXMYQ6M6GFCQLZRG6O7AVCNFSM6AAAAABMMN6NUCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBVGY2DMMBQGQ.
You are receiving this because you were mentioned.Message ID: @.***>

@ToC-Tthor , If you cannot locate Application Settings within the Configuration blade, they should be available under the Environment Variables blade. Please verify and confirm.

That's correct, and that is where I have the Generate_sourcemap = false configured.

From: v-ajaysahu @.>
Sent: Tuesday, August 13, 2024 08:43
To: OfficeDev/microsoft-teams-emergency-operations-center @.>
Cc: Trevor Thorsteinson @.>; Mention @.>
Subject: Re: [OfficeDev/microsoft-teams-emergency-operations-center] Error updating the App Service from 3.0 to 3.1. (Issue #231)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
@ToC-Tthorhttps://github.com/ToC-Tthor , If you cannot locate Application Settings within the Configuration blade, they should be available under the Environment Variables blade. Please verify and confirm.
—
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/microsoft-teams-emergency-operations-center/issues/231#issuecomment-2286434904, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BKPOLCSBHWGBNSG2POFSWQ3ZRILSXAVCNFSM6AAAAABMMN6NUCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBWGQZTIOJQGQ.
You are receiving this because you were mentioned.Message ID: @.***>

@ToC-Tthor ,
Please reset your Node version to 16.13.0 and attempt the process once more. If the issue persists, kindly provide your email address here on GitHub along with your available times (note: our availability is from 10 AM to 7:30 PM IST only).

Thank you @v-ajaysahu, I have reset to Node 16.13.0 and re-run the sync, however it still errors as before.   My email is<EMAIL_ADDRESS> Friday the 23rd I am available all day.

@ToC-Tthor ,
I have scheduled a meeting on 23rd at 7 PM IST.

Hello, the last redeploy from our testing errored out as before.
FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
1: 0104343B v8::internal::Heap::PageFlagsAreConsistent+3099
2: 01037744 v8::internal::Heap::CollectGarbage+2116
3: 01035D3F v8::internal::Heap::AllocateExternalBackingStore+1551
Failed exitCode=134, command=npm run-script build
An error has occurred during web site deployment.
npm WARN EBADENGINE Unsupported engine {\r\nnpm WARN EBADENGINE   package: @.@.',\r\nnpm WARN

From: v-ajaysahu @.>
Sent: Thursday, August 22, 2024 08:01
To: OfficeDev/microsoft-teams-emergency-operations-center @.>
Cc: Trevor Thorsteinson @.>; Mention @.>
Subject: Re: [OfficeDev/microsoft-teams-emergency-operations-center] Error updating the App Service from 3.0 to 3.1. (Issue #231)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
@ToC-Tthorhttps://github.com/ToC-Tthor ,
I have scheduled a meeting on 23rd at 7 PM IST.
—
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/microsoft-teams-emergency-operations-center/issues/231#issuecomment-2304752447, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BKPOLCXGXNYZGKIZ2AM6FVDZSXVMXAVCNFSM6AAAAABMMN6NUCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBUG42TENBUG4.
You are receiving this because you were mentioned.Message ID: @.***>

@ToC-Tthor ,
A meeting has been scheduled for Monday, 26th August 2024, at 7 PM IST to address and troubleshoot the issue.

@v-ajaysahu looks like changing the app service plan back to S2 did solve the issue.  I have now successfully upgraded to 3.1.

@ToC-Tthor ,
The issue arose from downgrading the APP Service plan to Basic and was resolved after upgrading back to the Standard S2 plan. As this issue is resolved, I am closing it. If you have any queries, feel free to comment here.


Support to build kyuubi-spark-authz-shaded into kyuubi-spark-sql-engine module

:mag: Description
In this PR, I add a profile spark-authz-compile to make kyuubi-spark-authz-shaded module built into kyuubi-spark-sql-engine module.
It can make the authz integration more easily for kyuubi-spark-sql-engine.
Issue References 🔗

This pull request fixes #
Describe Your Solution 🔧
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Types of changes :bookmark:

[ ] Bugfix (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)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklist 📝

[ ] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

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

Comparison is base (e9e2d18) 61.11% compared to head (338d36d) 61.06%.
Report is 3 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5994      +/-   ##
============================================
- Coverage     61.11%   61.06%   -0.06%
  Complexity       23       23
============================================
  Files           622      622
  Lines         37035    37043       +8
  Branches       5023     5025       +2
============================================
- Hits          22634    22619      -15
- Misses        11964    11973       +9
- Partials       2437     2451      +14

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

With the new profile activated, is the kyuubi-spark-authz-shaded shipped as a jar with kyuubi server or just embedded inside the spark SQL engine ?

this may not be a good idea. engine's plugins use lots of engine's internal APIs which can not work across the engine's versions, we usually leave the engine's plugin separately.


Work with multiple selection

I really like this script but it currently doesn't work with dmenu multi-selection patch . Could you think of an easy way to implement this? Otherwise I will make a PR

Hi @RayZ0rr , just found this message by accident. Haven't ever checked the issues page on this repo :D. Sure, go ahead. I have no clue about this patch and am frankly pretty busy with another project. So, have at it if it still bugs you.


fix(SystemApp.ts): System app wrapped with global to fix react native storybook loading for dev env.

System app wrapped with global to fix react native storybook loading for dev env.
What I did
System app wrapped with global to fix react native storybook loading for dev env.
How to test
Is this testable with jest or storyshots?

[ ] Yes
[x] No

Are any of the existing tests are failing ?

[ ] Yes
[x] No

Does this need an update to the documentation?

[ ] Yes
[x] No

Does it have breaking changes ?

[ ] Yes
[x] No

Checklist:

[ ] If this PR is a new feature, please reference an issue where a consensus about the design was reached (not necessary for small changes)
[ ] Make sure all of the significant new logic is covered by tests
[ ] Documentation updated

Please make sure to include all above in your PR.
@bjavaid @amnajunaid please review the changes and merge them.

:tada: This PR is included in version 3.15.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


⚠ Bofrost.lu has degraded performance

In 9f2815a, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1212 ms

Resolved: Bofrost.lu performance has improved in bc663e2 after 6 minutes.


RFE: Don't show if a word is valid until I hit enter

When I'm solving a puzzle, I don't want to know if the current word is valid until I hit enter. It'd be nice to be able to have a toggle for this behavior.
Thanks!

Interesting, can you explain your reasoning behind this preference?

It significantly reduces the difficulty when I see that a word exists while I'm working through different possible solutions. In a recent puzzle (and in times past) this has happened where an acceptable word was stumbled on while trying to find a valid answer.


Internal Server Error, opentapioca down?

Hi. The /annotate endpoint has been resulting in an internal server error for me, and I see that the https://opentapioca.org/ is not responding either. Is this project still being maintained? Any chance it will get fixed? Thanks!

I recently moved the service to a smaller server but it does not seem to handle the load.
Maybe I need to somehow start fundraising to cover the costs of this service (50€/month), as I would not like to bear them alone. Any idea how to do that?

I just started a crowdfunding page that I linked to from the main page of the service:
https://en.liberapay.com/OpenTapioca

Ah, I see. Thank you for the quick reply. That's quite unfortunate, I had used it extensively a few months ago.
I see you have found your answer to the fundraising issue.  Hopefully, it will get support from the community.

@t-kuculo consider donating even 1$ per month, that's important

Liberapay requires a long-term, weekly donation which is worrying for me. But I would like to make one-time donation.  Is there any way to do this? And thank you for your great api.

You could set up a donation and cancel it immediately after, perhaps?

Done.


[T4A6][T16-A2]YipMunKitBernard

Ready for review.

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


dont do something

i try this script on linux mint after chrome appear and login bot dont run the code

no error showing in terminal

could you explain a bit more with some code examples?


[GEARPUMP-377] Add TwitterSource and examples

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

[ ] Make sure the commit message is formatted like:
[GEARPUMP-<Jira issue #>] Meaningful description of pull request
[ ] Make sure tests pass via sbt clean test.
[ ] Make sure old documentation affected by the pull request has been updated and new documentation added for new functionality.

Codecov Report

Merging #247 into master will decrease coverage by 0.14%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #247      +/-   ##
==========================================
- Coverage   69.07%   68.92%   -0.15%
==========================================
  Files         192      193       +1
  Lines        6166     6189      +23
  Branches      540      541       +1
==========================================
+ Hits         4259     4266       +7
- Misses       1907     1923      +16

R: @huafengw

What will happen if the app has multiple source tasks? Will each task have the same output?

Yes, almost identical tweets output. Any suggestions here ?

+1


Productionize cross build actions

https://packages.ubuntu.com/search?suite=focal&searchon=names&keywords=crossbuild-essential

https://wiki.debian.org/CrossCompiling
https://wiki.ubuntu.com/MultiarchCross
https://wiki.debian.org/Multiarch/HOWTO

Cross compiling on debian is easy? Apparently there are "crossbuild-essential" packages for everything.

https://github.com/taiki-e/upload-rust-binary-action


Clean unwraps

Some places marked as TODO perform unwraps without returning a proper error.

Closing with the latest PR.


Fix error when GraphQL error locations is missing

Issue
In the official GraphQL specification, the locations attribute is a required field in error responses. However, not all GraphQL implementations strictly adhere to this requirement.
For example, when dealing with malformed GraphQL queries, such as the one described in  https://github.com/spiceai/spiceai/issues/1714, GitHub API returns an error response without including the locations attribute.
{
  "message": "Unexpected end of document"
}

When locations attribute is missing, the code in format_query_with_context will fail, specifically this line as 0 - 1 = -1 which is out of bound for usize.
https://github.com/spiceai/spiceai/blob/a0b4686e9829f0d75e99742c76f515bef3441c1f/crates/runtime/src/dataconnector/graphql.rs#L353
This error will cause server to crash, so this potentially closes https://github.com/spiceai/spiceai/issues/1714.

Solution
The proposed solution is to avoid formatting query when locations attribute is missing.

Amazing, thank you, @phungleson !


Update datasets/{dataset_id}/genesets POST route

Need

Curators need to create genesets for a specified dataset

Approach

Update datasets/{dataset_id}/genesets POST route -- reuse code from collections/{collection_id}/genesets POST route (including request body format)
if the dataset is already linked to genesets delete those genesets and create genesets contained in request body
On creation/update recreate the associated tidy csv stored in s3

DOD

Return a 201 and a response body containing the geneset information on success
If the dataset belongs to a public collection return a 403
If the user is not the owner of the collection the dataset belongs to return a 405

Closing. Per the 2022 Q1 plan, the design would embed gene sets in datasets.


docs: change dropdowns examples, adding the right markup

This PR dds the proper markup to dropdown examples, in order to avoid misleading errors & issues
Probably is not the best solution, as the existing doc examples are very descriptive, but as these examples lack of toggle button, using their functionality, causes console errors

[x] Bug fix (non-breaking change which fixes an issue)
[x] Refactoring (non-breaking change)

Checklist

[x] I have read the contributing guidelines
[x] My code follows the code style of the project (using npm run lint)
[x] I have updated the documentation accordingly

Live previews

https://deploy-preview-37488--twbs-bootstrap.netlify.app/docs/5.2/components/dropdowns/#disabled

Related issues
closes #37428

This will need to be addressed if we want to ship it. I don't mind just showing the menus on the page fwiw.

TBH I prefer the version we currently have in the documentation 😬 I find it disturbing to have already opened dropdown menus.
Regarding the issue, pressing "Esc" at this moment in the documentation doesn't make sense except if this is an a11y issue for folks navigating in the page with screen readers and maybe who are not able to understand why this dropdown menu is here.

To be honest I prefer the existing version too. However I think it is the second issue related to this markup.  Maybe we can alternatively, add a note, below each example

alternatively...can we add some guardrails around the dropdown JS so that it doesn't throw an unsightly TypeError when somebody presses Esc on those "not really dropdowns" we currently have in the docs?
dropdown.js:98 Uncaught TypeError: Cannot read properties of undefined (reading 'parentNode')
    at new ci (dropdown.js:98:34)
    at ci.getOrCreateInstance (base-component.js:65:41)
    at HTMLUListElement.dataApiKeydownHandler (dropdown.js:419:31)
    at HTMLDocument.n (event-handler.js:118:19)

and yes, I prefer the existing docs version as well, as they hone in on the point we're trying to make more clearly

see https://github.com/twbs/bootstrap/pull/37755

Closing it, as all of us prefer the existing version


Adding support for no address and extended address modes

Adding support for more address modes, no address and extended address. Also adding support for PAN id compress and version zero (0).

@blueluna Thank you for the PR! Sorry for the delay, I was away from GitHub over the weekend.
I don't have time to review right now, but I'll most likely get to it later in the week.

@blueluna Looks like I won't have a chance to review this week, but I should get to it next week. Sorry, busy times! :-)

@hannobraun No worries. I found some issues during the week, and I have yet to encode and send frames.

I have used the frame encoding and decoding in receive and transmit with a 802.15.4 coordinator.

@blueluna

I have used the frame encoding and decoding in receive and transmit with a 802.15.4 coordinator.

Good enough for me :-)
Thank you again for the contribution, @blueluna. It's really appreciated!


sequence of jupyter notebooks / other types of documentation for model creation

Summary
a sequence of online tutorials (notebooks or other) going from creating simple to more complicated models in pybamm.
Motivation
Try to ease new users into the pybamm framework, focusing in particular on users interested in model development
Potential list of tutorials
(a) create an ode model
(b) create a pde model
(c) create a model class (maybe better as a video / screen capture rather than a notebook?)
(d) create a new submodel

(e) how to make a reduced order model and compare against the full model

FYI I've started to add some notebooks based on the exercises from the last training workshop. These will roughly be

ODE model (introducing the general model structure and solving)
PDE model (introducing geometry, mesh, etc.)
More complicated PDE based on diffusion in a particle (linked to a physical example, including parameters etc.)
Comparing the PDE for the particle problem to the ODE you get in the fast diffusion limit (introducing comparing solutions, and to act as an example of comparing the "full model" (PDE) to a "reduced order model" (ODE))
A modified version of the existing SEI model notebook. This is more complicated and introduces non-dimensionlisation etc. so that people's models can fit in with the existing framework.

Then possibly notebook on turning a model in a class and creating submodel classes, but as @martinjrobins  suggests, this might be better as a screen capture. As we discussed the other day, the submodel structure might be a bit intimidating for new users, so I think at this point in time it would be better for the notebooks to focus on creating a model all in the same script/notebook.

Yeah, I agree that the focus should be on creating a model all in a single script. Maybe even including parameters values in that too (instead of using standard_parameters_lithium_ion, at least for the examples, maybe even for when we do a full SPM and/or DFN in a single script).
So once we have a full DFN class for example, anyone wanting to add some new physics can just make a copy of that class with the new physics. And when they have finished, we take care of merging it into the submodel structure for them and adding an option to call that submodel, which shouldn't be too bad with all the different parameters and variables already defined (just moving things around).


Fix install, Dockerfile, CI

Add missing jinja2 dep
Fix Dockerfile
Ensure Dockerfile builds in CI

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


Business Rules Referral Prescription 0.19

Business Rules ReferralPrescription V0.19 (1).docx

No more open issues for this document.


cmd/atlascmd: add supported format for dbmate

to use github.com/amacneil/dbmate easily
Re-open #899

Tests fail on 1.17 because it does not recognize the go.work file and does not use the replace directive there.
I'm OK with running the unit tests only on 1.18 only. @rotemtam, @masseelch - WDYT?


[Feature Request] select checkpoints via file browser

It would be nice to have a custom checkpoint loader node that allows you to select checkpoints via file browser ( like the image loader node ).
This would be useful for people that have lots of models, since the drop down list on the native checkpoint loader can get very long.
But the main advantage for me, would be to load checkpoints from multiple drives.
My main drive is only 512 GB, and SDXL checkpoints are very large.
I have another internal 512 GB SSD where I keep images and checkpoints used less often.
But to use these other checkpoints I have to delete checkpoints on my main drive, because I want to keep a decent amount of free space, which is very inconvenient.
I'm not sure why Comfy UI does not work like this by default.
It may not be technically possible ?
Or is it just to prevent user error, when selecting the files ?
If possible, this would be one of the best Comfy UI usability improvements, given the large size of the models we need.
Thanks

I've just found that loading models from another drive it's already possible, by editing the [ extra_model_paths.yaml ]
https://github.com/comfyanonymous/ComfyUI/blob/master/extra_model_paths.yaml.example
Instructions are in this discussion:
https://github.com/comfyanonymous/ComfyUI/discussions/72
This solves the main problem for me.
But if you have any ideas on how to improve those massive drop down lists, you can keep this open.

Yea, I was thinking about this earlier. Yes, extra_model_paths.yaml is how ComfyUI expects you to handle this situation, though I do think a file explorer would be much easier.
Unfortunately, I think ComfyUI expects everything to be in/relative to the paths in extra_model_paths.yaml when its handling the path values, so a File Explorer would likely be too incompatible.
Subfolders was my next idea too, but if other popular extensions already handle it, I think I'll just leave it to them... unless I could get it in the standard menu so we don't have to use special nodes...

I heard that the extension developer ( pythongosssss ) was working at Stability AI now.
So, I guess he will end up adding this, if it does not complicate the native code too much.
The native loader is already able to recognize when a model is in a folder, which is helpful ( e.g. SDXL folder ).

I added a feature just now to do it globally over all combo lists. It's off by default since I can't test all possible combos easily, but give it should work well for this Load Checkpoint case.
Turn it on in the rgthree-comfy settings (right-click on an empty portion and choose "rgthree-comfy > Settings") by enabling "(Menu) Auto Nest Subdirectories". You can be my beta tester.

This is awesome!
It seems to work for all native loaders.
Those custom nodes were just for checkpoints and loras.
This is very useful to keep all the original file names.
It does not work for IPAdapter models, but it's OK, that's a custom node.
I have not tested all of these, but I will report if I find any issues.
I'm on Windows 10.

Ah, I forgot to mention the logic. I only trigger it if there's more than 20 total items. So, that IPAdapter one may work, but since there's only 14 I figured it may not be useful.
WDYT, I can remove that restriction..

Yes, having a limit is a good idea.
People may want to create folders, just to reorder the files, and also see all the files, if there are few of them.
But, maybe expose this option to the user, since you also have a way to input numeric values in your settings.
You probably already have some hidden limits for these input values, to prevent user error.

When I have this setting enabled, I can't change the [ sampler_name ], in the KSampler node, via click select in the dropdown list.
I can only change it, via click select in the triangle arrows.
If I disable this setting, it all works again.
This only happens for [ sampler_name ], not the other dropdown lists of the KSampler node
The [ sampler_name ] list also has a search field at the top, like the loader dropdown lists where subfolders are used.
Maybe this issue is related to that ?

Thanks for catching. Fixed and also added the threshold to the options to make it configurable from the UI.


Fullscreen not working with the 2.1

Software Versions

Python: 3.10.2+
OS: ubuntu mate 22.04
Kivy: 2.1
Kivy installation method: pip

Describe the bug
Only a black screen appears when I try to start the app in fullscrren mode ('auto' and 1 modes)
Expected behavior
A clear and concise description of what you expected to happen.
An application in fullscreen
To Reproduce
Code and Logs and screenshots
from kivy.config import Config
Config.set("graphics", "fullscreen", "auto")
from kivy.app import App
from kivy.uix.label import Label

class TestApp(App):
    def build(self):
        return Label(text="Works ?")

app = TestApp()
app.run()

Context
These seems to works only with 2.1. with the 2.1rc1, I can get a fullscreen working window.
EDIT: these appears either when the nvidia driver (510 on a RTX2060) or the amd driver are working.

Historically this has happened for the wheels and the solution was to install kivy from source.

That's what I did. I installed kivy from pip+git to get the master. It's strange that the wheel doesn't work.
Thanks.
Should I let this issue opened ?

That's what I did. I installed kivy from pip+git to get the master. It's strange that the wheel doesn't work.

Probably need to compile SDL2 with some option to fix it. But I didn't see anything obvious there.

Should I let this issue opened ?

@matham are you talking about https://github.com/kivy/kivy/issues/6651 ?

@matham are you talking about https://github.com/kivy/kivy/issues/6651 ?

yes!

I have this issue as well, even installing kivy from source as in the instructions. Using Ubuntu 20.04. Solution for me was to install specifically kivy 2.0.0 from source via python -m pip install "kivy[base] @ https://github.com/kivy/kivy/archive/dedcb6bcabe3d8d6758dcee607e8c33b174d782b.zip" .

Is there a permanent solution coming without building everything from source? I am getting tired of fixing this problem again and again. I believe Python's wheels have flat tires at the moment, always something 'ey

am currently facing this issue with kivy 2.1.0, how did u resolve this bug

Hi @Noor-Kalibbala,
For me, normal pip (binary) install gave either a black screen or sometimes a crash (on Mac vs Raspberry Pi ... I forget now which did what ...) in full screen mode (which I wanted to use for app deployment on Raspberry Pi touchscreen).
The above (in my previous comment) pip source code installation ('--no-binary') worked for me, though when I later did a clean re-install I found that it doesn't include the window manager (e.g. sdl2) that was included in the binary install. I assume that first time around the window manager had already been added during kivy binary installation, so was still there when I re-installed kivy from source. So I then re-did the binary install followed by source ('--no-binary') reinstall, and that gave me both working fullscreen and also the required window manager.

Hi @colin0brass !
Can you please confirm that 2.2.0rc1 wheels are fixing this issue also for you?
To install the pre-release (2.2.0rc1) version:
pip install --pre kivy

Thank you for confirming that!
Closing as fixed, @olivier-boesch feel free to confirm that or ping me and ask to re-open the issue.

Thanks @misl6 , yes that worked. I did the following (on Raspberry Pi 4):
# uninstalled previous --no-binary install (2.1.0 source compile)
pip uninstall kivy

# tested with current standard binary to check it still failed
pip install --no-cache kivy
# Downloading Kivy-2.1.0-cp39-cp39-manylinux_2_17_aarch64.manylinux2014_aarch64.whl (30.5 MB)
# sure enough my kivy app froze during start-up in fullscreen mode

# uninstalled that version
pip uninstall kivy

# installed --pre version
pip install --no-cache --pre kivy
# Downloading Kivy-2.2.0rc1-cp39-cp39-manylinux_2_17_aarch64.manylinux2014_aarch64.whl (19.1 MB)
# that worked! App successfully started and functioned correctly in fullscreen mode

Was having the same black fullscreen problem on a Raspberry Pi 4, the rc1 fixed for me as well! Thanks!


Updated EO to 0.29.5

PR-Codex overview

The following files were skipped due to too many changes: tests/org/eolang/runtime-tests.eo

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@yegor256 I updated new EO version. Please, take a look

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 13min)


Enable Component Governance for each build

We need to enable the Component Governance task for each of our build pipelines. See https://docs.opensource.microsoft.com/tools/cg.html for more information.

[ ]  azure-sdk-for-java
[ ]  azure-sdk-for-net
[ ]  azure-sdk-for-js
[ ]  azure-sdk-for-python

This is a duplicate to: https://github.com/Azure/azure-sdk/issues/40


Issue #195 - update Travis to Trusty due to gcc/clang strtod() bug

This pull request updates Travis to use the Ubuntu Trusty distribution instead of Precise.  This is required due to the bug in strtod() that is uncorrected in Precise builds of the standard library but fixed in Trusty.  This bug causes a single bit rounding error that may occur in as many as 1 in 4096 of the possible double values.
Note that currently this PR still uses the llvm-toolchain-precise* repositories because the  llvm-toolchain-trusty* ones have not been added to the Travis white-list.  This works because clang is not the problem, rather it is the standard library.
There is an open issue on Travis APT source whitelist request for llvm-toolchain-trusty #199 to add the repositories to the white-list at which point .travis.yml can be further updated.  It is suggested that perhaps this pull be merged but issue #195 be left open until the trusty llvm repos can be added as well.

Thanks!


feat: add flag for router IP local run

Reference: issue #350
Summary:

adds local_ip_address crate to be able to grab local router IP
add new flag --router_ip to be able to use router IP instead of localhost, which allows other devices on the network to access the local deployment.
relevant testing has been added to account for new feature

Notes:
local_ip_address was added for this feature implementation as it looks to be a stable crate that doesn't appear to be going inactive any time soon and appears to work across OSes with no code change required (tested using Ubuntu and Win11). In the event that it does (or there's some breaking issue that renders it unusable), it looks like a method would have to be implemented per OS to get the router IP.
Windows users who are using this flag in Bash/WSL may have problems getting other devices to get this to work if they are not aware of the fact that there are extra steps^1 required to be able to connect external devices to a locally deployed run in Bash/WSL due to the fact it uses the WSL virtual ethernet. Not a huge problem as it seems most people who use shuttle are currently running Linux, but it's something to keep in mind should any issues regarding this pop up.
The new test also uses the same example as one of the other tests (the "hello world" Rocket example), which may or may not break the tests. Not entirely sure because trying to run all integration tests is broken on my laptop due to memory issues, but although the test passes in isolation it's probably something to be aware of.

Ah that makes a lot more sense, I didn't realise you could bind it to <IP_ADDRESS> - that would probably be more optimal since it eliminates the need to add a dependency and would eliminate WSL-specific issues which would be helpful for compatibility. Looks like I still have a lot to learn about using IPs and networking 😅

This looks cool Josh! It looks like you need to do a cargo fmt to make the CI happy, and maybe clippy. And dont worry about squashing commits, we'll do that when we merge. 🙂

It's ok, I appreciate the checks to make sure everything is optimised before pushing through - I'm personally more concerned about solving the circleCI cargo shuttle test failing since that seems to be a pressing issue. It looks like it's running into the same issue I had in local which is that it failed on trying to wait for start up

The "rocket/postgres" has intermittent issues starting up, so don't worry about it too much. If your new test works locally then I'm fine merging this even if the other tests fail on the CI


Allow setup timezone in cluster config or config file.

Right now pinot time zone is decided by the local timezone, this caused a lot of confusion especially during ingestion time or use datetrunc function etc.
The way to set timezone for pinot is either using environment variable or JVM argument: https://docs.pinot.apache.org/basics/getting-started/frequent-questions/general#how-to-change-timezone-when-running-pinot
Ideally, we should make a config to set time zone for pinot component.
Default should be UTC always.

Hey @xiangfu0. Can I take this up?


ESLint: Add prefer-const rule

Require const declarations for variables that are never reassigned after being declared. If a variable is never reassigned, using the const declaration is better. const declaration tells readers, “this variable is never reassigned,” reducing cognitive load and improving maintainability.

/retest


Added links to workflow defintions

I also tried with a redirect icon next to the copy, but it takes too much space.