jax.config.update("jax_enable_x64", True) does not work for joblib.

Description
Briefly, even I set jax.config.update("jax_enable_x64", True) ahead , joblib will work under jax.config.update("jax_enable_x64", False).
from jax.config import config
import jax
import jax.numpy as jnp
import joblib

jax.config.update("jax_enable_x64", True)

key = jax.random.PRNGKey(0)
subkeys = jax.random.split(key, num=100)
N = 5

def norm_gen(key_id):
    return(jax.random.normal(subkeys[key_id],shape = (N,)))

norm_gen(0)
# Array([-2.1415603 , -1.38104703, -0.91724264, -0.14136964,  2.0047816 ],      dtype=float64)
norm_gen(1)
# Array([ 0.81512986, -0.17840127,  1.00610674,  1.57187942,  0.14101336],      dtype=float64)

Joblib gives the float32 results, indicating jax.config.update("jax_enable_x64", True) not working.
# joblib
numbers = range(2)
results = joblib.Parallel(n_jobs=5)(joblib.delayed(norm_gen)(num) for num in (numbers))
results

#[Array([-0.90704215, -1.548955  ,  0.79437876, -0.4385296 ,  1.5684816 ],      dtype=float32),
# Array([-0.47438177,  1.4778413 , -1.123923  ,  1.6738842 , -2.7033079 ],      dtype=float32)]

However, if I set jax.config.update("jax_enable_x64",False), then it gives the same results as to joblib.
jax.config.update("jax_enable_x64",False)

norm_gen(0)
# Array([-0.90704215, -1.548955  ,  0.79437876, -0.4385296 ,  1.5684816 ],      dtype=float32)
norm_gen(1)
# Array([-0.47438177,  1.4778413 , -1.123923  ,  1.6738842 , -2.7033079 ],      dtype=float32)

System info (python version, jaxlib version, accelerator, etc.)
jax:    0.4.22
jaxlib: 0.4.22
numpy:  1.25.0
python: 3.11.0 (main, Oct 24 2022, 18:26:48) [MSC v.1933 64 bit (AMD64)]
jax.devices (1 total, 1 local): [CpuDevice(id=0)]
process_count: 1
$ nvidia-smi
Thu Mar 28 02:00:11 2024
+-----------------------------------------------------------------------------------------+
| NVIDIA-SMI 551.86                 Driver Version: 551.86         CUDA Version: 12.4     |
|-----------------------------------------+------------------------+----------------------+
| GPU  Name                     TCC/WDDM  | Bus-Id          Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |           Memory-Usage | GPU-Util  Compute M. |
|                                         |                        |               MIG M. |
|=========================================+========================+======================|
|   0  NVIDIA GeForce RTX 3070 ...  WDDM  |   00000000:01:00.0  On |                  N/A |
| N/A   36C    P8             19W /  150W |     658MiB /   8192MiB |      0%      Default |
|                                         |                        |                  N/A |
+-----------------------------------------+------------------------+----------------------+
+-----------------------------------------------------------------------------------------+
| Processes:                                                                              |
|  GPU   GI   CI        PID   Type   Process name                              GPU Memory |
|        ID   ID                                                               Usage      |
|=========================================================================================|
|    0   N/A  N/A      4816    C+G   ...oogle\Chrome\Application\chrome.exe      N/A      |
|    0   N/A  N/A      5604    C+G   ...wekyb3d8bbwe\XboxGameBarWidgets.exe      N/A      |
|    0   N/A  N/A      6372    C+G   ...5n1h2txyewy\ShellExperienceHost.exe      N/A      |
|    0   N/A  N/A      8256    C+G   ...ekyb3d8bbwe\PhoneExperienceHost.exe      N/A      |
|    0   N/A  N/A      8512    C+G   ...siveControlPanel\SystemSettings.exe      N/A      |
|    0   N/A  N/A     10040    C+G   C:\Windows\explorer.exe                     N/A      |
|    0   N/A  N/A     10128    C+G   ...nt.CBS_cw5n1h2txyewy\SearchHost.exe      N/A      |
|    0   N/A  N/A     11016    C+G   ...2txyewy\StartMenuExperienceHost.exe      N/A      |
|    0   N/A  N/A     11520    C+G   ...on\122.0.2365.92\msedgewebview2.exe      N/A      |
|    0   N/A  N/A     12188    C+G   ...tionsPlus\logioptionsplus_agent.exe      N/A      |
|    0   N/A  N/A     14724    C+G   ...CBS_cw5n1h2txyewy\TextInputHost.exe      N/A      |
|    0   N/A  N/A     16360    C+G   ...GeForce Experience\NVIDIA Share.exe      N/A      |
|    0   N/A  N/A     16908    C+G   ...les\Microsoft OneDrive\OneDrive.exe      N/A      |
|    0   N/A  N/A     18296    C+G   ...__8wekyb3d8bbwe\WindowsTerminal.exe      N/A      |
|    0   N/A  N/A     19316    C+G   ...crosoft\Edge\Application\msedge.exe      N/A      |
+-----------------------------------------------------------------------------------------+

Well, this isn't really a bug in JAX: we didn't promise anything about joblib.
That said, you can think of the config option like a global variable: it apparently isn't being set in the subprocesses joblib is using. Perhaps there's a way to do some extra initialization as part of the joblib subprocesses so you can set that config option? Or you could presumably set it as part of the function you are parallelizing, as the first thing you do.
What do you think?

Yes, you are right. I just found that setting
def norm_gen(key_id):
    jax.config.update("jax_enable_x64", True)
    return(jax.random.normal(subkeys[key_id],shape = (N,)))

can solve this issue. Thanks for pointing out!


Add AppVeyor build badge to readme

Add AppVeyor build badge to readme

Coverage remained the same at 90.342% when pulling 1bceb6ef77bdea867a7fe11e8a24c2339c2919e3 on cs/badge into 88708d0089a49b99e6004f960cc7955c30026c79 on master.


Support complex types in map key in type parser

Summary:
Support complex types in map key in type parser. The current
implementation restricts parsing to simple types.
Differential Revision: D52145611

Lazy review +1.


Container's volumes not deleted with the containers itself

I used Postgres container and I also created a new type (Vault container).
each of the containers mounted a volume to the disk, after the tested finished and the container deleted, the volume was still in the disk and I needed to use docker system prune for deleted them.
To Reproduce
Steps to reproduce the behavior:

Run Postgres test container
Finish the UT gracefully saw the dispose will call
go to the terminal and run docker volume ls -f dangling=true
and see the unused volumes

I tried to override the dispose method in the container class I created but the details of the containers and the used volumes are private and I couldn't access it for calling the remove after the docker removed.
also I could use your docker api client since its internal so anyway I need to do the system call separately.
there is other way to do it, by calling the container remove with the flag -v that delete the unnamed volumes with it (only unnamed volumes, I'm not sure what kind of volumes you using)
Desktop (please complete the following information):

Server: Docker Engine - Community
Engine:
Version:          19.03.5
Nuget version - 0.0.9

I guess we need to set the ContainerRemoveParameters property RemoveVolumes in:
https://github.com/HofmeisterAn/dotnet-testcontainers/blob/fc9e6566acbef849ce215b801406852f3395d811/src/DotNet.Testcontainers/Client/TestcontainersClient.cs#L64
Would you like to create a pull request?


Data as Placeholder

I would like to reuse a same model for (a lot of) different data sets, but I believe in the current implementation GPflow needs compilation every time the data has changed.
( is it correct?)
I think TensorFlow itself has a framework to prevent such an unnecessary compilation,  placeholder.
Do you guys have a plan to update GPflow to handle data as placeholder?

This is directly related to #80 where I have described one idea how to include tensorflow placeholders.
This should help reduce the number of recompilations.
@fujiisoup thank you for the feedback. Could you describe some scenarios, ideally via short examples, that you believe would benefit from such an improvement?

A scenario I could think of would be to define some regression model over a gene expression time course, and change the Y-data to be different genes :)

Thank you for your replies.
In my study field (nuclear fusion experiment), a bunch of multi channel data (typically 100 points per 1 data set and 100,000 data sets per day) are being generated and we need to estimate their latent functions and derivatives.
The recompilation time will be significant.
I will try to contribute (though I'm not very familiar with TensorFlow), if there are no known concern about using the placeholders.

@jameshensman I think @vals and @fujiisoup make a compelling case from the applications end. What do you think?

Thanks all.
I implemented this and #80 features into GPflow.
see notebook.
I tested it manually only for GPR and VGP.
Please let me how should I test the code and send pull_request.

Hi @fujiisoup
This sounds great. Let's have a discussion about getting this into GPflow. To start, I have

merged your changes into GPflow/Placeholder,
Created a pull request #108 , so that we can get this into the master branch,
invited you to become a GPflow contributor.

Looking at the PR, the tests seem to be failing at the moment. If you accept our invitation to contribute, you'll be able to push code to GPflow/Placeholder, and tests/coverage will automatically run over at #108. Let's continue this discussion over there.
Thanks for your efforts so far!
James.

@fujiisoup @jameshensman are you happy to close this issue now?


Rewrite ARTSerializablePath to use class

ARTSerializablePath.js is written using legacy helper from art/core/class. We should refactor it to use ES6 class syntax.
class ARTSerializablePath {
	...
}

If anyone's interested, I started the work on https://github.com/react-native-community/art/tree/chore/path-as-class branch, but I'm likely not gonna continue and I'm also not convinced this is cleaner approach :D We'll need to override typings anyway, since art/core/path is untyped.


added NO_SSLv3 to ssl_options and disabled FIPS mode

Updated default attributes, template and readme.

Hi @paulrossman
Thanks for sending in this pull request. DNSimple has adopted this cookbook from Heavy Water and I've cleaned up the master branch to get the test suites working correctly again. If you would like to get these changes into master, please rebase against master and we can look at merging this in once again.

Hmm, it looks like @damacus removed the SSL_OPTIONS flag in https://github.com/sous-chefs/chef-stunnel/commit/01a9be11f96a3d3714943d8f4fafc34844bc2383
I'm going to assume this isn't able to be merged. If either of you disagree, we'll need to merge this in manually.


remove CreateContainer annotation and DockerContainer

This test code is not used elsewhere in the project and is not really
maintained. It seems like it might be useful for some testing, but since
it is not used (and not distributed since it is not in src/main/java), I
rather remove it.

Codecov Report

Merging #861 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #861      +/-   ##
============================================
- Coverage     65.92%   65.89%   -0.04%
+ Complexity      737      736       -1
============================================
  Files           166      166
  Lines          3140     3140
  Branches        357      357
============================================
- Hits           2070     2069       -1
+ Misses          914      913       -1
- Partials        156      158       +2


query:thanos-query duration mismatch with maxsourceresolution

Signed-off-by: t00350320<EMAIL_ADDRESS>
[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
when query duration or step paramater mismatch with max_source_resolution, an error will pop up
Verification
query like :
rate(foo[15s])&max_source_resolution=1h, an error will return.

hi,@bwplotka
pls help to review again.
thanks!

Hi @t00350320,

when query duration or step paramater mismatch with max_source_resolution, an error will pop up

can you share an example of the returned error? Like a screenshot?

Hi @t00350320,

when query duration or step paramater mismatch with max_source_resolution, an error will pop up

can you share an example of the returned error? Like a screenshot?

OK，the test case like this:
v1_test.go
// Use default step when missing.
{
endpoint: api.queryRange,
query:
url.Values{
"query": []string{"sum(max_over_time(irate(mysql_global_status_com_select{DATABASE='xxx',TOKEN='xxx',cluster_mark='<IP_ADDRESS>_33066'}[30s])[7d:]))"},
"start": []string{"1626773208"},
"end":   []string{"1626773298"},
"step":  []string{"60s"},
"max_source_resolution":[]string{"1h"},
},
go test , the error like this:
v1_test.go:117: Unexpected error: bad_data: duration or step mismatch with maxSourceResolution


popovers: Prevent text-overflow in user-card-popover.

[x] Prevents text overflow in user-card popover on narrow screens.
[x] Refactor the CSS nesting of user-card popover.

Fixes: CZO Thread

Screenshots and screen captures:

Before:
After:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@zulipbot add "area: popovers"

Hey @alya , could you please review the PR, thanks :)

Looks good to me in manual testing, so you can tag @sahil839 to take a look once the commit message has been tweaked.

Please update the commit message, as "Refactor..." indicates non-functional changes.

@alya , I used the word `Refactor" because  I did not add any new functionality. I simply reorganized the code to fix the issue. If you still want me to update the commit message, please let me know. Thank you.

@palashb01 we should use the term "refactoring" correctly (https://en.wikipedia.org/wiki/Code_refactoring). If there are any functional changes, including fixing a bug, the change is not a refactor.

@sahil839 , could you please review the PR, thanks.

Looks good to me. @timabbott this is ready for your review.

@sahil839 , it's been some time. Maybe an integration review would be much better, as I recently encountered this issue again :)

I had a look at this again. Wouldn't it be better to instead set width to 240px to the .popover-inner element irrespective of the screen sizes.

Closing since this is no longer reproduceable.


 Missed one letter in the attribute.

Damn, I missed that one too. :D Thanks for fixing! Released in https://github.com/maragudk/gomponents-htmx/releases/tag/v0.5.0


⚠ Bofrost.es has degraded performance

In 012207a, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1416 ms

Resolved: Bofrost.es performance has improved in 9083e09 after 6 minutes.


Add new pattern: Replace in order for with for

This PR adds the new pattern and includes to a test to assert:

Replace in order for with for

Closes #11

Codecov Report

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

@@          Coverage Diff          @@
##           master    #12   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          26     26
=====================================
  Hits           26     26

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 c9d53e9...c3d834e. Read the comment docs.

Thanks! Released as 5.1.0!


Permit only valid IMAP tag characters rather than rejecting invalid

@jryans I realised that even with the tweak I suggested to the new IMAP tag matcher, it still allows invalid tags (e.g., anything beyond the basic 128 ASCII characters). That is still better than what was used before, but I think it's worth fully matching the formal specification, so the version in this pull request takes an allowlist-based approach instead.
Could you test this change with your client that uses non-alphanumeric tags?

@simonrob Ah, good catch! I can confirm this version also works well for me. 😄

Great - will merge this change too. Thanks again for the original fix.


[native filter] Cascading filters should show layer in filter indicator on native filter panel

Cascading filters should show layer in filter indicator on native filter panel, now is shown in parallel.
Expected results
cascading filters should show in parent/ child layer in filter indicator
Actual results
cascading filters now show in parallel in filter indicator
Screenshots

How to reproduce the bug

create cascade filter
go to native filter panel
open indicator for cascade filter
See error

Environment

superset version: master

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

It's a recent regression from fixing the forever enlarging popover window 😀, i think. @villebro @kgabryje

I am having a look

We have discussed this with @michael-s-molina and @villebro. This is a conscious choice as you can read in Michael's PR #15390.
We might have an unlimited number of filters. Eventually, the filters won't have enough space available.
Another reason was the input size being different as we move down the hierarchy, which made the UI look poorly designed.
CC @jinghua-qa @junlincc

@geido the changed made in #15390 was not approved or reviewed by product, it was done when I was on PTO. Sorry Diego for the confusion. please do add the indentation back 🙏

@geido the changed made in #15390 was not approved or reviewed by product, it was done when I was on PTO. Sorry Diego for the confusion. please do add the indentation back 🙏

@junlincc @geido Can we talk about the motivations for the change and try to approve it before reverting?

https://github.com/apache/superset/issues/16532 raised a bunch of other concerns about this screen. We decided to change the design of it to improve the user experience. Closing this issue.


Update color-adjust to print-color-adjust

Is your feature request related to a problem? Please describe.
PostCSS is throwing a warning on fonts utilizing the color-adjust shorthand.
Describe the solution you'd like
Update to use recommended print-color-adjust shorthand if and when it makes sense.
Describe alternatives you've considered
Do nothing. It's only a deprecation warning.
Additional context

https://github.com/postcss/autoprefixer/issues/1454

I don't think this has anything to do with Fontsource considering I've never about this CSS property, much less be able to code it in.
This is the CSS that you are loading when importing it.

Right you are. The error message is pointing at the wrong thing. Looks like this issue is within Tailwind. Sorry for the noise!


Duplicate toast when replacing by fee

I wanted to add more stx to the fee for a given transaction, and it seems there are two toast that get fired:

I correctly replace-by-fee'ed, and feel like this toast was fired by mistake.

Closing this off as its outdated


Fix nonce mismatch caused by rooms waiting overhead upon sending messages

Fixes #2229
Short description
This was a little bit hard to debug, but it seems some overhead upon explicitly queueing messages and waiting for peer to join the room with us caused some overhead which prevented the persistence callbacks from firing at proper time.
This PR was tested using BF6, and as the presence bug was kicking in, it also includes a workaround for it, by explicitly requesting peer's presence upon presence events, instead of relying on event's payload, which can be wrong.
Definition of Done

[x] Steps to manually test the change have been documented
[x] Acceptance criteria are met
[ ] Change has been manually tested by the reviewer (dApp)

Steps to manually test the change (dApp)

Test bug doesn't occur
High-load scenarios pass

@ezdac Please, test if this fixes your bug. I'd suggest:

Ensure dependencies are up to date, with pnpm install --prefer-frozen-lockfile on repo's root
Build SDK: pnpm run build --filter raiden-ts
Serve dApp: pnpm run serve --filter raiden-dapp
Use a fresh account, or close+settle channels and clear IndexedDB before reloading dApp
Test


Maintaining compatibility with xclim

Hello!
I just wanted to open an issue about some things that would make compatibility much easier to maintain for xclim. As of now, we have clisops installed in our Travis CI load-out for the following versions and builds:

python = 3.6, 3.7, 3.8
OSes = Windows, Linux (Ubuntu Xenial), macOS

Clisops is built into Xclim such that when importing xclim.subset with clisops in the installation environment, xclim exposes all the processes listed in __all__ of clisops.core.subset. It would be good to ensure that new functions are listed in __all__ and tested. Presently, these are:
__all__ = [
    "create_mask",
    "create_mask_vectorize",
    "distance",
    "subset_bbox",
    "subset_gridpoint",
    "subset_shape",
    "subset_time",
]

I understand that there may eventually be decisions that change the way clisops is run, so in order to ensure that these changes don't take us by surprise, it would be good to see some of the following:

DeprecationWarnings and FutureDeprecationWarnings on major functions.
Documentation for the API and/or usage examples (notebooks) of clisops.
Release notes in tagged versions or in the documentation that let us know of new/upcoming changes.
Warnings about any major refactoring efforts, so we can update our API and let our users know of any changes coming down the line to the subset utilities.

I am always available to help clarify any of the practices we use and can help with implementing standards as needed. Just let me know where I can help out.

Hi @Zeitsperre , I am going to organise a meeting with @huard, yourself and people my end so that we can talk about various issues relating to our collaboration. Apologies that things may have appeared erratic our end. I thought you would be concerned mainly with the releases but that was naive of me. Let's meet and talk about all these issues so that we can improve our practices and make sure we are aligning our approaches.

Hi @agstephens Thanks for checking in with me yesterday and today. I meant to respond but yes, xclim was (and still is to a degree) under very active development, but we didn't have to worry too much about API and userbase until much later. I am indeed curious to see where this is going and don't want to seem in the way. If we can synchronize on a handful of dev practices, it would be make development simpler on our side.
Thanks!

Hi @Zeitsperre, I'm very happy for us to improve our procedures this end. You will be doing us a favour by applying some gentle pressure on that front, and I expect we'll learn new things that will make us more efficient in future :-)


Fix all ts-check issues in concept exercises

I found out my editor did not show me the ts-check issues at all. I fixed the editor setting and then fixed all the typing issues.
This PR contains

removing env.d.ts files as they are not needed anymore
fix typing issues
fix typing import paths in exemplar.js files

/format


Audio single result page does not show default artwork if no thumbnail is available

Description

The search results show default artwork for audio results that do not have a thumbnail. The single result page shows a broken image.
Reproduction

Go to single audio result.
See error: the broken thumbnail in the audio details section. Note that related audio correctly shows the default artwork.

Screenshots

Expectation

For audio without thumbnails, single result page should show default artwork.
Resolution

[ ] 🙋 I would be interested in resolving this bug.

This issue occurs because VAudioDetails does not use the VAudioThumbnail component but rather a plain <img> tag.


[Experimental TitleBar] Collapse TitleText when empty

Description

When the TitleBar TitleText is empty make sure to collapse the TitleText Textblock so that it does not take any extra space.

Add title property changed handler
Add visual states for title visibility
Apply visual state change if the title is empty or not

Motivation and Context

Allows TitleBar consumers to have responsive UI and remove the extra space that an empty title was adding.
How Has This Been Tested?

Tested via the MUXExpirementalTestApp.

/azp run


part: cv.py ImgCrop class reduces width regardless of inputs

donkey/donkeycar/parts/cv.py
in the code below the width and height values are in the wrong order, the img_array.shape call returns height first.. so the width of the output img_arry is always set to the input images height, typically 120
class ImgCrop:
    """
    Crop an image to an area of interest.
    """
    def __init__(self, top=0, bottom=0, left=0, right=0):
        self.top = top
        self.bottom = bottom
        self.left = left
        self.right = right

    def run(self, img_arr):
        width, height, _ = img_arr.shape  # width and height are reversed, resulting in the issue
        img_arr = img_arr[self.top:height-self.bottom,
                          self.left: width-self.right]
        return img_arr

Since you know how to fix this, could you submit a pull request? Thanks.

hey, i think I got it right..
Forked the code
fixed the issue
pushed to my forked repo
create pull request.. is that how its done?

I'm removing opencv from the repo. This can be implemented with a keras layer or tensor operation.


Telegram error 409

Home Assistant release (hass --version):
0.65.5
Python release (python3 --version):
3.6.3
Component/platform:
telegram_bot
Description of problem:
Since upgrading from 0.64 to 0.65 the telegram bot is flooding my log file with errors. This message appears 5 or 6 times per minute:
2018-03-16 06:57:04 ERROR (MainThread) [homeassistant.components.telegram_bot.polling] wrong status 409

BTW, notifications are working. But it itches me seeing this error.
Expected:
No errors
Problem-relevant configuration.yaml entries and steps to reproduce:
################################################
## Telegram Bot
################################################

telegram_bot:
  - platform: polling
    api_key: !secret telegram_bot_api_key
    allowed_chat_ids:
      - !secret telegram_bot_chat_id_robert

################################################
## Notify
################################################

notify:
  - name: telegram
    platform: telegram
    chat_id: !secret telegram_bot_chat_id_robert

Traceback (if applicable):

Additional info:
Using hassio on rpi 2b
Possibly related to multiple other telegram issues since 0.65 issued at github and/or the community forums.

The issue is occurring since the same time that #12774 started.

Telegram error 409 is

another webhook is active

so perhaps I am accidentally using the api key somewhere else. Will look into this. Strange though that this is only showing up recently.

Don't know what was wrong, but I changed the api key for the bot and added that to my config. This way I was sure the key could not be used somewhere else. Error is gone. So not caused by HA.


improve: Linear Search

⭐ Summary
Hi @Utkarsh1504
I would like to improve documentation and code for Linear Search in Searching & Sorting Section
Please assign it to me.
Thanks
💻 Location
title: Linear Search
section: Searching & Sorting
order: 5A
filename: linearsearch.md

❌ Additional Info
#hacktoberfest2021

please assign this to me?


Workflow marked legacy?

After reading the docs thoroughly, I can't understand why the workflow section is marked as Legacy. I'm looking to have several steps iteratively gather information from several different responses before taking some action and I don't see a way to do that with simple actions passing values to bolt besides building some type of data store locally. Workflows seemed to be the perfect fit as I can gather several inputs before taking an action on the final set of data but I don't want to build on something that is marked Legacy. I don't see a note on why it was marked Legacy or what the alternative would be so could someone explain why that section is marked so in the docs and if workflows are still a reasonable thing to use in to build new apps?

Hi @JoshuaLyle, thanks for writing in!
The section is marked as legacy because our next-generation platform feature (still in beta) will provide a better way to implement the same in the near future. Here are a few resources about the beta platform feature:

https://slack.dev/bolt-python/future/concepts
https://api.slack.com/future

The newer version of Workflow Builder is under development, so the next-gen platform does not provide the same functionality with the existing custom step feature yet. But, in the near future, your "function" in a next-gen platform app can work as a step in any workflow. You should probably be curious about the timeline for the next-gen's GA release, but sorry I cannot tell you the details now.
As for the existing "Steps from Apps" feature, it will be maintained for a certain period. However, with that being said, we encourage developers to consider the next-gen platform functions as the primary option for new app development.
I know that the current state of the next-gen platform may not be a good fit for your need today, but I hope my reply here is helpful to you.

Thanks Kazuhiro!
This is great information and I appreciate the details in the response!
One thing I may suggest is to add a note or link to the intended replacement feature (or lack thereof) in a deprecation notice in the doc. Most cases I've seen in docs with legacy features mentions have a link to the stable version of the feature nearby. One other thing I find peculiar is that is was marked legacy before its replacement was out of beta. I would imagine they'd coexist as main features before deprecating the older legacy feature.
I hope this don't come off as criticism! This is an excellent library and I just hope to avoid any questions like mine in the future from others.


remove async from trustroot trait

Note: this PR builds on top of https://github.com/sigstore/sigstore-rs/pull/348 and https://github.com/sigstore/sigstore-rs/pull/349. I'll rebase it once those are merged.
This PR makes the methods defined inside of the sigstore::trust::TrustRoot trait non-async. This fixes https://github.com/sigstore/sigstore-rs/issues/346

Closing, this is done as part of https://github.com/sigstore/sigstore-rs/pull/311


OCP 4.8 Java lab - trigger not working

I tried the current 4.6 lab with the amended pipeline and trigger yamls for 4.7 on a  4.8 install and everything worked except the trigger step.
I used this GIST to accelerate through
https://gist.github.com/mandibuswell/c1d3d61f9413cf849146ca6e1798b4eb
[mbuswell-redhat.com@bastion agnosticd]$ oc create -f https://raw.githubusercontent.com/openshift-roadshow/nationalparks/master/pipeline/nationalparks-triggers.yaml
triggertemplate.triggers.tekton.dev/nationalparks created
triggerbinding.triggers.tekton.dev/nationalparks created
route.route.openshift.io/el-nationalparks created
Error from server (BadRequest): error when creating "https://raw.githubusercontent.com/openshift-roadshow/nationalparks/master/pipeline/nationalparks-triggers.yaml": admission webhook "webhook.triggers.tekton.dev" denied the request: mutation failed: cannot decode incoming new object: json: unknown field "name"

In addition of course, the imagery will need updating.

I have confirmed that by setting the name to ref in template this works
`---
apiVersion: triggers.tekton.dev/v1alpha1
kind: TriggerBinding
metadata:
name: nationalparks
spec:
params:

name: git-repo-url
value: $(body.repository.clone_url)
name: git-repo-name
value: $(body.repository.name)
name: git-revision
value: $(body.after)

apiVersion: triggers.tekton.dev/v1alpha1
kind: EventListener
metadata:
name: nationalparks
spec:
serviceAccountName: pipeline
triggers:

bindings:

ref: nationalparks
template:
ref: nationalparks

apiVersion: route.openshift.io/v1
kind: Route
metadata:
labels:
app.kubernetes.io/managed-by: EventListener
app.kubernetes.io/part-of: Triggers
eventlistener: nationalparks
name: el-nationalparks
spec:
port:
targetPort: http-listener
to:
kind: Service
name: el-nationalparks
weight: 100`


Cannot start configurator add-on

I have problems with hass.io in Edge and Chrome:
When trying to start the configurator add-on in the Edge browser (latest version of hass.io and browser), nothing happens and the following errors pop up in the browser console. There are no errors in the hass.io system log.
HTTP404: NOT FOUND - The server has not found anything matching the requested URI (Uniform Resource Identifier).
GET - http://<IP_ADDRESS>:8123/api/hassio/build-temp/util.js

 HTTP400: BAD REQUEST - The request could not be processed by the server due to invalid syntax.
(XHR)POST - http://<IP_ADDRESS>:8123/api/hassio/addons/core_configurator/start

In Chrome I get:
Uncaught (in promise) DOMException: Only secure origins are allowed (see: https://goo.gl/Y0ZkNV).     core_configurator:1
Failed to load resource: the server responded with a status of 404 (Not Found)    util.js
POST http://<IP_ADDRESS>:8123/api/hassio/addons/core_configurator/start 400 (Bad Request)

It turns out that the default config for the configurator has a null password but the addon requires a password to be set.

I ran into this shortly after installing Hass.io. Without the configurator, I can't install either the Samba or SSH add-on, so I don't even know how to work around this.
@wind-rider, can you point me to the configuration file's path and describe what change to make to get this to work?

In the addon section of hass.io open the Configurator settings.
Below the panel where you can e.g. restart the Configurator, you can configure its basic settings like the port to run on or the password.

The addon section (clicking on Hass.io in the menu) remains blank, perhaps because util.js can't be loaded?
I thought that was the bug you were reporting here. :D

/move home-assistant/home-assistant-polymer


Order URL in attack-simulation-training-get-started.md

Sort attack simulation training URL in ascending order

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree


MSBuild Error DEP1000: Cannot copy file

🐛 Bug Report
Followed the Getting Started for an existing Blazor Hybrid app.
dotnet add package Microsoft.FluentUI.AspNetCore.Components
Added this to the index.html as it contained the blazor.webivew.js from the VS template.
<script app-name="MyAppName" src="./_content/Microsoft.FluentUI.AspNetCore.Components/js/initializersLoader.webview.js"></script>
🤔 Expected Behavior
Project builds and can start debugging
😯 Current Behavior
DEP1000: Cannot copy file "C:\Users\me\.nuget\packages\microsoft.fluentui.aspnetcore.components\4.6.0\staticwebassets\Microsoft.FluentUI.AspNetCore.Components.lib.module.js.LEGAL.txt" to layout "C:\Users\me\source\reposMyAppName\src\MyAppName\bin\Debug\net8.0-windows10.0.19041.0\win10-x64\AppX\wwwroot\_content\Microsoft.FluentUI.AspNetCore.Components\Microsoft.FluentUI.AspNetCore.Components.lib.module.js.LEGAL.txt". DirectoryNotFoundException - Could not find a part of the path 'C:\Users\me\source\repos\MyAppName\src\MyAppName\bin\Debug\net8.0-windows10.0.19041.0\win10-x64\AppX\wwwroot\_content\Microsoft.FluentUI.AspNetCore.Components\Microsoft.FluentUI.AspNetCore.Components.lib.module.js.LEGAL.txt'. [0x80070003]

## 🌍 Your Environment

Windows 11, Visual Studio 17.9.5
* .NET and Fluent UI Blazor library Version [e.g. 8.0.2 and 4.4.1]

    <PackageReference Include="Microsoft.Maui.Controls" Version="8.0.14" />
    <PackageReference Include="Microsoft.Maui.Controls.Compatibility" Version="8.0.14" />
    <PackageReference Include="Microsoft.AspNetCore.Components.WebView.Maui" Version="8.0.14" />
    <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="8.0.0" />
    <PackageReference Include="Microsoft.Maui.Essentials" Version="8.0.14" />
    <PackageReference Include="MudBlazor" Version="6.19.1" />
    <PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="7.5.0" />

Did you copy the error message? If so, there seems to be a '' missing in the path after layout.
Not sure if that is a library issue or just something in your configuration.

Closed due to inactivity

I think this is related to https://github.com/TobiasBuchholz/Plugin.Firebase/issues/199 as looking at the file path it's over 400 characters long including the project name I stripped out of the bug report.


[WFLY-19577] Remove usage of AbstractAddStepHandler.attributes from u…

…ndertow ServerAdd
#18076 backporting to 33.x
Jira issue: https://issues.redhat.com/browse/WFLY-19577

Thank you @yersan


IntelliSense window leaves the editor when it should open to the top

I had the panel open and triggered IntelliSense in the lower area of the editor. I would expect the widget to position to the top to not draw under the panel:

This looks pretty bad.

@bpasero I think this might be a workbench problem.
The suggestWidget already has a z-index of 40 that makes it:

render on top of the next editor (first screenshot)
render on top of normal text following the standalone editor (second screenshot)
I have tried a value of z-index up to 50k -- does the workbench use more than that?
I don't know how the workbench layouts the things in the bottom area in such a way that z-index has no effect on them.

@isidorn I am sure you did something very powerful there ;)?

For some reason z-index=1 was too strong, so now I set it to initial. I did not change this for the sidebar, only panel (suggest widget already goes nicely on top of sidebar)


[WIP] Focused tests

This is a work in progress, to implement #185.

Coverage increased (+0.06%) to 89.329% when pulling<PHONE_NUMBER>ac4923454376e36d95398b76f19b13 on focused-specs into 01fa66e2517bfca839bf42fd9655f643c6bc8e09 on master.

Closing this to submit it to the feature branch


508 System Component Needs Header

Required Fix

[x] RF-14

Context
Table Headers Missing TH Tags (RF-14)
The table row or column header cells are coded as ordinary data cells.
Use header from other tables
See EPA Monitoring Plans Accessibility Report for how to fix and additional details.

Confirmed that the table now has visible header cells and coded as TH.


Add Lab Guide

Purpose
Lab guide in progress. Current exercises worked on

[x] Setup and Aspire
[x] Semantic Kernel (intro to SK Open AI chat completions)
[x] Context Window
[x] RAG Pattern (intro to SK Open AI embedding generation and SK NoSQL Vector connector)
[x] Semantic Cache

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

@jcocchi this is really great!!!! I only had small suggestions other than my comments on the max_tokens.
Maybe I'm confused now on what this value is for. Originally I thought it was for Total Tokens but maybe it only drives the tokens used to generate the completion text. If so we should document this in the lab and sample.
@aayush3011 let us know on whether max_tokens actually works or if it just controls completion text generation.

@markjbrown the max_tokens specifies the max number of tokens to generate in the completion.
However, if the sum of tokens in prompt + max_tokens exceeds the context length of the model, the request will be considered invalid and you’ll get a 400. The context_length for the model we are using is 4096.

@markjbrown the max_tokens specifies the max number of tokens to generate in the completion.
However, if the sum of tokens in prompt + max_tokens exceeds the context length of the model, the request will be considered invalid and you’ll get a 400. The context_length for the model we are using is 4096.

I was noticing some weird 400 errors when I ask certain questions- maybe this is why! Although I was getting 401 unauthorized which I thought was strange because we're using DefaultAzureCredential() and auth shouldn't go in and out across requests in the same app


Reduced RAM usage when generate row name for data writing

RAM usage reduced from 70 to 66 Mb in my example (table with 50 columns and 2000 rows).
MemoryProfiler statistics before:
Total allocated: 70070873 bytes (1341650 objects)
Total retained:  4274328 bytes (189 objects)

allocated memory by gem
-----------------------------------
  64518379  write_xlsx/lib
   5527315  rubyzip-2.3.2
     12005  fileutils
     10774  other
      1800  delegate
       600  tmpdir

MemoryProfiler statistics after:
Total allocated: 66166872 bytes (1244050 objects)
Total retained:  4274328 bytes (189 objects)

allocated memory by gem
-----------------------------------
  60614378  write_xlsx/lib
   5527315  rubyzip-2.3.2
     12005  fileutils
     10774  other
      1800  delegate
       600  tmpdir

Execution time did not changed (check rake test).

thanks.
PR #113 and #114 have been merged.


ACM-5861 and ACM-5865: Missed translations

https://issues.redhat.com/browse/ACM-5861
https://issues.redhat.com/browse/ACM-5865

/cc @Randy424


PHP Parse error

PHP Parse error:  syntax error, unexpected 'class' (T_CLASS), expecting identifier (T_STRING) or variable (T_VARIABLE) or '{' or '$' in PHPCap/src/RedCapProject.php on line 2013

Hi Devin,
Thanks for the feedback. I have looked into this issue, and I can't see a problem with the code.
To figure out the cause of the problem, I need more information.
What version of PHP are you using? PHPCap requires version 5.6. If you are using an earlier version, it appears you might get this error.
Jim

Thanks for the Note Jim!
You are quite right, I was on a machine with an even older version of PHP when I got that error.
Many thanks for your work to maintain this repo—it’s been incredibly helpful to my research team!
Best,
Devin
On Oct 8, 2018, at 6:03 AM, Jim Mullen <notifications<EMAIL_ADDRESS>wrote:
Hi Devin,
Thanks for the feedback. I have looked into this issue, and I can't see a problem with the code.
To figure out the cause of the problem, I need more information.
What version of PHP are you using? PHPCap requires version 5.6. If you are using an earlier version, it appears you might get this error.
Jim
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faarenson%2FPHPCap%2Fissues%2F11%23issuecomment-427807754&data=02|01|devin.r.cowan%40dartmouth.edu|dc2caf02329d436f1a2208d62d161a81|995b093648d640e5a31ebf689ec9446f|0|0|636745970308827151&sdata=g%2BDH323Mrosgh5ggMnJQ64esnCnPPMNLcnkl6FHof84%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FARGncfaq-SUjmJEkW0HJqmokT-0B-tCFks5uiz8igaJpZM4XL-lx&data=02|01|devin.r.cowan%40dartmouth.edu|dc2caf02329d436f1a2208d62d161a81|995b093648d640e5a31ebf689ec9446f|0|0|636745970308827151&sdata=cAbWlhi2Bx7T%2FraEKdd5OPbjrjTCCPXTwjyB%2BcnD%2Fwk%3D&reserved=0.

Devin, glad it's proving useful. We're always interested in hearing about how people are using PHPCap. Feel free to email us details about how you're using it at<EMAIL_ADDRESS>and<EMAIL_ADDRESS>.


Feat/add logging docs

This PR adds docs about customisation in Claudie on the manifest level.
Closes #684

Shouldn't we also make the Builder, Scheduler and frontend port customisable ?

Builder, Scheduler and Frontend are services that act as a client. I am not sure why the port variables were in .env but AFAIK the only place where these ports are used is in health check probes and I think its okay if these will not be customisable.


Define a deprecation policy for Falcon's API

We should review how other projects are handling the deprecation of API methods and document one for us.
To be defined

[ ] Grace period for deprecated items: how much releases will still have the deprecated item
[ ] Requirements to deprecate an API item: an item should not be deprecated if there is not a clear migration path for developers.
[ ] Mechanism for notification: raise warnings during the grace period? Raise exceptions afterwards? Document deprecated items in release notes or is a separated deprecation schedule needed?.

Policies to Review

[ ] Django: https://docs.djangoproject.com/en/1.8/internals/release-process
[ ] Rallydev: https://help.rallydev.com/web-services-api-deprecation-policy
[X] Raven: http://raven.readthedocs.org/en/latest/#deprecation-notes
[X] PBR's SemVer Deprecation Guidelines: http://docs.openstack.org/developer/pbr/semver.html#how-should-i-handle-deprecating-functionality

raise warnings during the grace period
Raise exceptions afterwards
Document deprecated items in release notes or is a separated deprecation schedule needed

I think semantic versioning would be a good idea. This project is still young but is starting to gain traction and use, so it would probably be a good idea to jump to 1.0.0 soonish, and then use the major version number to indicate backwards incompatible changes, and use minor releases to deprecate features.
Semantic versioning has it's problems, but for a library used only by developers (as opposed to end users) a consistent and "standardized" (oh well) versioning scheme could go a long way in complementing a proper deprecation scheme.
Note that according to semantic version anything is allowed to happen before 1.0.0. In practice however, falcon is already being used in production even though it's just 0.3.0, so it might make sense to be a little careful but make the jump sooner rather than later.

I'm with @richardolsson on having deprecation duration be integrated with version numbering.
semver or not, binding deprecation to version numbers allow an easy way to freeze if you are not yet ready to update.
I'm thinking something like this would be a nice way to go about it:
Before any major release, mark feature as deprecated in docstring (and write at what version the feature will stop working), write in changelog AND a dedicated deprecation schedule, AND do a minor release with the deprecation.
Apart from the docstring and documentation, nothing has changed yet.
Then for the following major releases (major, as in semver):

Start emitting warnings (via logging, prererably muteable) about use of deprecated function/feature, but otherwise still functioning. Emitted message should contain some hint as to how to move away from the deprecated feature.
Raise an error with directions on how to migrate. Feature no longer functioning.
Remove completely. Devs will get NameError and the likes.

So, if a feature is deprecated in major version 1, it will continue as always for the duration of 1, emit warnings in major version 2, and raise errors in 3. In 4 it will be completely removed.
Personally I like semver, but it is religion. Any well-defined versioning is fine.
Imo. Django's versioning is horrible for anything but time-defined releases, and it doesn't seem like falcon will ever have a pre-defined release cycle (?).
tl;dr: Imo: deprecations should keep functioning for the duration of the major release they were made in, emit warnings in the next. Then break.

+1 @richardolsson & @tbug I think semver is the way to go.

+1 @richardolsson and @tbug All for semver.

Closing this as our documentation does lay out a policy that is clear enough for the end users IMHO:

Reliable. We go to great lengths to avoid introducing breaking changes, and when we do they are fully documented and only introduced (in the spirit of SemVer) with a major version increment. The code is rigorously tested with numerous inputs and we require 100% coverage at all times. Falcon does not depend on any external Python packages.

Informally, we do also follow what @tbug has suggested above, although exceptions were sometimes made for small breaking changes before a major release, like when having to accommodate ASGI in 3.0. I'll add a note on https://github.com/falconry/falcon/issues/473.


request help: encounter problem in setting up a ingress controller

Issue description
Hi, i have setup a kubernetes cluster using kind and here's my config
To test my kind cluster, i run:
$ kubectl apply -f https://raw.githubusercontent.com/pkking/homelab/main/ingress/nginx-deploy.yaml
$ kubectl apply -f https://raw.githubusercontent.com/pkking/homelab/main/ingress/nginx.yaml

and then curl -v localhost -H "Host: local.httpbin.org"  will return the httpbin page:
$ kubectl get ingress
NAME                 CLASS    HOSTS               ADDRESS     PORTS   AGE
httpserver-ingress   <none>   local.httpbin.org   localhost   80      63s

But then i tried deploy a apisix controller, it seems not works:
$ kubectl apply -f https://raw.githubusercontent.com/pkking/homelab/main/ingress/apisix-apiroute.yaml

Then curl -v localhost -H "Host: local.httpbin.org"  will just return 404;
and have no idea why
$ kubectl get apisixroute
NAME               HOSTS                   URIS     AGE
httpserver-route   ["local.httpbin.org"]   ["/*"]   27m

Any suggestions appreciated!!
Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long):

$ helm list -A
NAME    NAMESPACE       REVISION        UPDATED                                 STATUS          CHART           APP VERSION
apisix  ingress-apisix  1               2023-07-18 17:15:09.356819359 +0800 CST deployed        apisix-2.1.0    3.4.0

your Kubernetes cluster version (output of kubectl version): K8s Rev: v1.27.3
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a):  5.15.0-58-generic 64-Ubuntu SMP Thu Jan 5 11:43:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

In fact, all e2e test cases of this project use Httpbin as the backend, which you can refer to.
https://github.com/apache/apisix-ingress-controller/blob/e809cfbaaad6f6c3cbd23f5ec8e93bc1d87f543d/test/e2e/suite-plugins/suite-plugins-general/echo.go#L34-L58

Misconfigured ApisixRoute: The ApisixRoute configuration might be incorrect. You can check the configuration file to ensure that it is correct and that the local.httpbin.org host is correctly specified 1.
Incorrect Apisix version: The Apisix version you are using might be outdated or incompatible with your Kubernetes cluster. You can try upgrading to the latest version of Apisix to see if that resolves the issue 1.
Misconfigured Ingress: The Ingress configuration might be incorrect. You can check the configuration file to ensure that it is correct and that the local.httpbin.org host is correctly specified 1.


Request to help: Tejiri and Hardik

Hello @juergba, Hardik Patel and I would like to help with anything that would be a good first issue. We have support from @joesepi, Here is a link to a new issue we ran into while trying to create a new issue in your forked mocha repo https://github.com/mochajs/mocha/issues/4966.
Can we help with this issue for a start?

Welp, coming across this 10+ months later: juergba isn't maintaining Mocha anymore. But if you folks are still interested in contributing to Mocha then great! Anything marked as status: accepting prs that doesn't yet have an open PR is fair game.


Merge coverage package

Merging package:coverage into this repo.
This is the "raw" output of using https://github.com/mosuem/merger, specifically running
dart run bin/merger.dart --input-name coverage --branch-name master --input-path ~/projects/coverage/ --target-path ~/projects/tools/ --git-filter-repo /home/mosum/tools/git-filter-repo

on my local machine, and some small cleanups afterwards.

[x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

571 of 609 (93.76%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-6.2%) to 93.76%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkgs/coverage/lib/src/chrome.dart
60
61
98.36%

pkgs/coverage/lib/src/util.dart
55
56
98.21%

pkgs/coverage/lib/src/formatter.dart
64
66
96.97%

pkgs/coverage/lib/src/resolver.dart
78
82
95.12%

pkgs/coverage/lib/src/run_and_collect.dart
9
13
69.23%

pkgs/coverage/lib/src/collect.dart
177
190
93.16%

pkgs/coverage/lib/src/hitmap.dart
128
141
90.78%

Totals

Change from base Build<PHONE_NUMBER>0:
-6.2%

Covered Lines:
571

Relevant Lines:
609

💛  - Coveralls

Can you rerun your merger tool? This is missing https://github.com/dart-lang/coverage/pull/502 (and https://github.com/dart-lang/coverage/pull/503 but I see you did a similar change yourself).


-c as the default?

There are 2 options:
    -c, --continue           resume partially downloaded files
    --no-continue            do not resume partially downloaded files (restart from beginning)

Why have both? -- wouldn't it make sense for -c to be the default and to just have --no-continue? -- Why would you want to restart a download by default?

There are two options because we want users to explicitely opt-in or opt-out of continuation. By default, youtube-dl does the sensible thing, whatever that is. Since there have been no breakage reports of continue, it is now (again) the default in 2012.02.26.


dorsal.done should be passed all wired elements

Whenever we resolve the promise from a wire we should pass all elements wired and their associated plugins back.

It looks like this is handled by: https://github.com/eventbrite/dorsal/blob/master/src/dorsal_deferred.js#L30
I saw the .resolve line in src/dorsal.js and didn't see anything indicating that we were passing over the instances. We should be fine.


feat: LogEvent attributes

@rish691 Pl. add a description and associated issue ticket of supporting embedded log events.


ResearchOps refresh and initial recommendations

User Story
As a researcher on ReportStream, I want to conduct a synthesis and refresh of previous ResearchOps documents, so I can eventually centralize, update, and create new ResearchOps materials for our current scrum teams and project.
Background
Prior ResearchOps documents are disjointed and spread among different teams and projects. Likewise, there is not one source of truth for ResearchOps strategies, and as such, any previous strategies are not widely practiced within the design team. Revisiting the ResearchOps documents will be helpful to understand which guidelines are still relevant for our team, what gap(s) in knowledge and/or strategy are occurring, and what new process(es) may need to be made.
Risks/Impacts/Considerations

Our research initiatives are too spread out in the timeline and/or too different from one another to be appropriately centralized in ResearchOps
Different projects/initiatives have differing needs that require separate ResearchOps - this one doesn't adequately cover them all
Too general/not specialized enough for adoption purposes

Open Questions

Which part(s) of the various ResearchOps documents are unused/stale?
Which parts are still pertinent and usable?
Which parts are helpful, but need to be updated to be relevant?
What new processes need to be described?
What connections should be made with Product, Engineering, and other teams? What strategies should these m
What is the best way to form these ResearchOps so they become widely adopted and practiced?

Links

WIP PRIME ReportStream Design Research Operations Guide
ReportStream Research Norms

Acceptance Criteria

[ ] Conduct background research and synthesize prior ResearchOps materials
[ ] Develop recommendations for the ResearchOps "refresh"
[ ] Share-out of ResearchOps synthesis to Design team and Platform scrum team, as appropriate

Out of Scope

Creating new documentation or any new ResearchOps material (yet)

WIP ResearchOps document started here.

I shared out the ResearchOps document with the design team and with @jillian-hammer for feedback. The feedback seems to be moving in a good direction, allowing me to begin working on the actual ResearchOps materials themselves. As such I will close out of this ticket and make a new one! CC @Andrey-Glazkv @jsutantio


Test points functions

Codecov Report

Merging #409 (34b5a2c) into master (8b224b2) will increase coverage by 0.41%.
The diff coverage is 0.00%.

:exclamation: Current head 34b5a2c differs from pull request most recent head 6f8a606. Consider uploading reports for the commit 6f8a606 to get more accurate results

@@            Coverage Diff             @@
##           master     #409      +/-   ##
==========================================
+ Coverage   75.47%   75.88%   +0.41%
==========================================
  Files         176      176
  Lines       28491    28495       +4
==========================================
+ Hits        21504    21624     +120
+ Misses       6987     6871     -116

Impacted Files
Coverage Δ

resqpy/grid/points_functions.py
66.78% <0.00%> (+19.17%)
:arrow_up:

resqpy/model/_xml.py
88.34% <0.00%> (-0.28%)
:arrow_down:

resqpy/rq_import/__init__.py
100.00% <0.00%> (ø)

resqpy/olio/vector_utilities.py
79.66% <0.00%> (+1.66%)
:arrow_up:

Continue to review full report at Codecov.

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

@andy-beer Added the tests asked for. If you would like to check them then merge if they are fine


Please move from suds to something like pysimplesoap

As reported in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=788086 ,
suds is not maintained for long time since, so will be removed from Debian
in next release. Then, psphere package will also be removed.
Bug reported suggests replace subs with other implementation. For example,
pysimplesoap is maintained and python3 ready.
Could you consider to move it from depending suds?

@henrich - Yes I will review pysimplesoap and determine if a move is possible. Thanks for the heads up!

:-)


Select notes higher/lower than octave range

Is your feature request related to a problem? Please describe.
I can't import most MIDIs not meant for NBS with the restriction, so I've resorted to using Double Bass and Bell for the extra 2 or so octaves. Doing this is tedious, however.
Describe the enhancement you'd like
2 buttons when you right-click the track that say "Select all notes above octave limit" and "Select all notes below octave limit". They select said notes.

If you click the "Not compatible" indicator at the top, it should tell you exactly how many blocks are outside the 2 octave range and allow you to select them. The only problem is that there's no distinction between blocks that are lower and higher, so you still need to do some manual filtering.
The only workaround I can think of is to "Increase octave" until all lower notes are within range. Then, when you click "Select all notes outside the octave range" again, it will only select the higher notes. Then you can change the instrument, fix the octave, and do the same for the lower notes. You can use layer locks to filter which layers you want to select at a time.
Indeed, this is a ridiculous solution for something that should be simple to solve. I'll see if in the next version I can split that button into higher and lower notes only. :)


refactor(base64dec): make the out buffer  null-terminated

Signed-off-by: pingkai<EMAIL_ADDRESS>

Codecov Report

Merging #399 into develop will decrease coverage by 4.26%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #399      +/-   ##
===========================================
- Coverage    68.55%   64.29%   -4.27%
===========================================
  Files          216      216
  Lines        11746    11746
===========================================
- Hits          8053     7552     -501
- Misses        3693     4194     +501

Impacted Files
Coverage Δ

framework/tests/dataSource/ipList.h
0.00% <0.00%> (-100.00%)
:arrow_down:

framework/tests/demuxer/demuxerUnitTest.cpp
0.00% <0.00%> (-100.00%)
:arrow_down:

framework/tests/dataSource/dataSourceUnitTest.cpp
0.00% <0.00%> (-93.85%)
:arrow_down:

framework/tests/decoder/decoderUnitTest.cpp
0.00% <0.00%> (-93.62%)
:arrow_down:

framework/tests/demuxer/demuxerUtils.cpp
0.00% <0.00%> (-82.67%)
:arrow_down:

framework/data_source/ffmpeg_data_source.cpp
6.38% <0.00%> (-51.07%)
:arrow_down:

framework/demuxer/AVBSF.cpp
25.27% <0.00%> (-42.86%)
:arrow_down:

framework/data_source/curl/CURLShareInstance.cpp
69.44% <0.00%> (-27.78%)
:arrow_down:

framework/data_source/curl/CURLConnection.cpp
52.28% <0.00%> (-12.86%)
:arrow_down:

framework/data_source/ffmpeg_data_source.h
80.00% <0.00%> (-10.00%)
:arrow_down:

... and 9 more

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 ec2e184...a803c2d. Read the comment docs.


add suggestive fallback messages on unavailable fonts

as per https://github.com/jwilm/alacritty/issues/39

@jwilm Ok, done. I've done that as a new commit, but if you'd rather I squash the commits together, I can do that as well.

Yeah squashing sounds right. Thanks!

@jwilm done


Aligned textual output on bug reporting in all tools

Add a base text to ltsmin-standards.h

The 2.0 release made a step in he right direction by aligning error outputs.
The next step is to have all tools output setup, progress, and end statistics, in the same way and order.
I agree that the output strings should be recorded in ltsmin-standard.h


CIP-0021: Canonical CBOR serialisation

A CIP defining canonical CBOR serialisation for Cardano transactions. The resulting format is based on the needs of HW wallets and Slack discussions with @dcoutts.
Nothing is set in stone, this PR should mainly spark discussion on the topic of CBOR canonicalisation so we can agree on a format.
Open questions can be found in the comments.

Closing this one as the revised version has changed significantly and is oriented more towards HW wallets. (we will create a new PR shortly)


Added a test for cloning arrays

@evilsoft I think @HenriqueLimas may have been correct in adding in support directly for Arrays. It appears that they don't clone well. It all looked fined until I went to use it in head and it didn't work. It appears that the context for the iterator is lost and so when calling next you just get { value: undefined, done: true }. We can get around this by putting the original code back in for array and string but i thought it best to re-open this discussion.
You can remove the array code in cloneIterable to see the failure

Coverage remained the same at 100.0% when pulling b6def63fafcde1a6fe2bb7739725cd633a366a86 on check/cloneIterable into e1d2484255500de813676de8988b8efd661397c1 on master.

We should stick with using Array directly even when in the iterable functions.
So for things like take on an Array that would do an m.slice(0, num - 1) when it is an Array.
What are some of the cases that we would want to clone an Array?

I'm not sure we ever will want to clone it. This came more from the side of treating all iterables as the same so that way we didn't have to have code that checked for string or array then checked iterable but after playing with it a bit and doing research i'm quite happy to leave as is and do the check for array/string then iterable in each part. Although it might be worthwhile creating a branching helper for that given we'll be doing it in a few spots

Agree, I think we should separately check it. Probably it will be repetitive, but at least we would have a consistent way of deal with that. Sometimes we need a copy like map, but others we do not.


Move WebSub unit tests under respective modules

Purpose

$Title

Codecov Report

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

@@             Coverage Diff              @@
##             master   #12992      +/-   ##
============================================
+ Coverage     59.33%   59.34%   +<.01%
  Complexity      659      659
============================================
  Files          2111     2111
  Lines        103333   103333
  Branches      13183    13183
============================================
+ Hits          61315    61321       +6
+ Misses        36828    36823       -5
+ Partials       5190     5189       -1

Impacted Files
Coverage Δ
Complexity Δ

...java/org/ballerinalang/bre/bvm/BLangVMStructs.java
50% <0%> (-1.67%)
0% <0%> (ø)

...a/org/ballerinalang/bre/bvm/WorkerDataChannel.java
79.62% <0%> (-1.24%)
0% <0%> (ø)

...a/org/ballerinalang/bre/bvm/GlobalMemoryBlock.java
80.19% <0%> (-1%)
0% <0%> (ø)

...inalang/net/http/serviceendpoint/InitEndpoint.java
62.42% <0%> (+0.63%)
0% <0%> (ø)
:arrow_down:

...ain/java/org/ballerinalang/net/websub/hub/Hub.java
60.6% <0%> (+1.51%)
0% <0%> (ø)
:arrow_down:

...llerinalang/stdlib/socket/tcp/SelectorManager.java
71.79% <0%> (+1.7%)
0% <0%> (ø)
:arrow_down:

...t/websub/compiler/WebSubServiceCompilerPlugin.java
82.14% <0%> (+3.57%)
0% <0%> (ø)
:arrow_down:

...allerinalang/test/util/client/ResponseHandler.java
80.76% <0%> (+15.38%)
0% <0%> (ø)
: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 cd57561...1c4bbb2. Read the comment docs.


Sorting Albums - Manual type

Helloz,
I would like to suggest an option to sort albums not only by date,name or so but also by manually by users. Nice there is bookmark for Apps like FB Mess Tg and so on. But here and below that bookmark i would like to choose which album gonna be first and which one would be at the middle and which at the end. I would like to have most important for me albums like DCIM Videos SS at the but like the other albums like photo of docs,memes,ss from games/browsing at the bottom while I don't use em a lot but I cannot (or don't see option for that) sort them as I wish to.
Best Regards, NIGHTZNERO.

You can pin albums to the top (select them and use the "Pin" action) but no manual sorry yet.
Duplicate:

#231

You can pin albums to the top (select them and use the "Pin" action) but no manual sorry yet.
Duplicate:

#231


Support Chromium based Microsoft Edge

I took the liberty of creating this issue so we can track and centralize discussion around supporting the upcoming Chromium engine based Microsoft Edge.
Microsoft's official  announcement (archive.org copy) (PDF copy) (Gist markdown copy)
What

Today we’re announcing that we intend to adopt the Chromium open source project in the development of Microsoft Edge on the desktop to create better web compatibility for our customers and less fragmentation of the web for all web developers.

When

Over the next year or so, we’ll be making a technology change that happens “under the hood” for Microsoft Edge, gradually over time, and developed in the open so those of you who are interested can follow along.

My take
This is potentially great news for Lorca on Windows if future Edge provides the features that Lorca needs. Current version of Edge on Windows 10 spawns a private Edge session for me when running:
start shell:AppsFolder\Microsoft.MicrosoftEdge_8wekyb3d8bbwe!MicrosoftEdge -private https://github.com

But my understanding is that Lorca also relies on Chrome DevTools Protocol. I hope to see that available in the new Edge.

Me too. Would make developers loves much easier

Update
Microsoft made available a preview build of Microsoft Edge based on Chromium.
Announcement: https://twitter.com/MSEdgeDev/status/1115283353731600384
Download available at: https://www.microsoftedgeinsider.com

Might also be possible to support Firefox: https://github.com/GoogleChrome/puppeteer/tree/master/experimental/puppeteer-firefox

I was able to get it working by just installing the edge dev channel and modifying the LocateChrome function
func LocateChrome() string {
	var paths []string
	switch runtime.GOOS {
        [...]
	case "windows":
		paths = []string{
			"C:/Program Files (x86)/Microsoft/Edge Dev/Application/msedge.exe",
			"C:/Users/" + os.Getenv("USERNAME") + "/AppData/Local/Google/Chrome/Application/chrome.exe",
			"C:/Program Files (x86)/Google/Chrome/Application/chrome.exe",
			"C:/Program Files/Google/Chrome/Application/chrome.exe",
		}

not passing tests, but is working when I open a locally served html5 application, so gives good indication that it shouldn't take much rework. When I have time I'll look into the tests and do a PR

PR submitted: https://github.com/zserge/lorca/pull/137, using the release path for MSEdge

PR submitted: https://github.com/zserge/lorca/pull/137, using the release path for MSEdge

Thank you very much @chenilim! Since your PR was merged I think it's sensible to close this issue.


Refactor state latex drawer

Summary
This PR builds on #8273 (fixes #8073, fixes #8169, fixes #7513) and #8461 (fixes #8460).
In this PR I have:

(1219531a09233315981ecdcb12552c8d2562ece6, c4debbdf2b92289ffd01e743df5808559808bc32, 72e4fb7ec9e41453f1d91af8482e033daebf8931) Removed duplicated code that converts amplitudes to latex strings
(visualization.array._num_to_latex and visualization.state_visualization._num_to_latex_ket)
(21d4b6bd2c288bc67b22a7ab9877c326cdb520aa, 1126a4fa735b3db03f05e06af74159544ab4d3ee) Reduced the amount of code in the drawers and hopefully cleared up the logic a bit
(bd64399c2295b910b107e0051b7ad47c7cf0d4c4) Cleaned up the output latex (fixes most of #8317)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

39 of 41 (95.12%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 84.039%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/visualization/array.py
23
25
92.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
56249

Relevant Lines:
66932

💛  - Coveralls

Is this intended to supersede/obsolete #8273 and #8461?  If so, was it in discussion with @1ucian0 and @ryanhill1?

@1ucian0 - The fixes in this PR overlaps with some of the code in the 'noisy' PR #8174.
If this PR is allowed, then my original PR (linked above) shouldn't have been closed in the first place. It could've been linked to multiple issues and re-worded.


Fix a mistake in example

Simple mistake in a worked example

This issue is already fixed. This PR should be closed.


Api without web enabled

As I can occupy the api, in web disabled, is it possible ??
If that's what changes I have to do, thank you.

It's not possible because it's a web API.

Perfect, thank you very much.
Greetings.


Add the calendar name of each entity

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
If we have several calendars to display in event mode, it is not possible to distinguish them (except by icon)
(I have tried showCalendarName but it seems not working…)
Expected behaviour
Would it be possible to add the following option :

display the calendar name of each entity

Thank you in advance for your help
Possible Solutions
Displayd Prior to the event name or below
Mode
Event (default)
Context / Reason
Provide an better readable and understandable event list when we are using calendar event mode with several calendars.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Setting showCalendarName: true will show the calendar name at the right side of the card.
I have this setup on my HA and it is working. If you could provide your configuration and a screenshot of what you currently see that would help me troubleshoot

Hello @marksie1988
I have already test it in event mode but nothing is displayed on the right side.
Indeed, if showCalendarName is already implement, may be it is a bug.

my configuration :

Home Assistant 2023.8.4
Supervisor 2023.08.1
Operating System 10.5
Interface utilisateur : 20230802.1 - latest
Atomic Calendar Revive v 7.6.1

I would need the configuration from the card, if you edit the card, click show code editor in the bottom left and paste the yaml on GitHub (feel free to change calendar names to something else if they are sensitive but don't remove them)

Ok your entity configuration is incorrect, try this:
entities:
        - entity: calendar.perso
          name: XXX
          icon: mdi:human-male
        - entity: calendar.us
          name: Us
          icon: mdi:human-male-female
        - entity: calendar.family
          name: Family
          icon: mdi:human-male-female-child


OPNsense 23.7-amd64 / OpenVPN new Instances pages creation doesn't save 'Enforce local group' setting

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
When creating or editing an OpenVPN instance from the new instances pages, the "enforce local group" selection is not saved.
No error message displayed.
To Reproduce
Steps to reproduce the behavior:

Go to VPN/Instances [new];
Create or edit an instance;
Scroll down to 'Enforce local group' and select an existing group;
Click on 'Save';
Click on 'Apply';
Open the instance created or edited. The 'Enforce local group' returned to 'None'.

Expected behavior
Saving the instance must save the value indicated in 'Enforce local group'.
Describe alternatives you considered
n/a
Screenshots
n/a
Relevant log files
n/a
Additional context
n/a
Environment
OPNsense 23.7-amd64.

It looks like the form/JS doesn't render... the value seems to be there and selected in the API response:
        "local_group": {
            "": {
                "value": "none",
                "selected": false
            },
            "1999": {
                "value": "admins",
                "selected": 1
            }
        },

Fixed via 68ae7bd, thanks a lot for the quick report!
# opnsense-patch 68ae7bd

Cheers,
Franco


Case insensitive URL matches

Would it be possible to support case insensitive URL matches?  I've tried via regex with (?i) or /i flags, but still doesn't seem to work.

Ah, looks like this is a duplicate of https://github.com/sylouuu/chrome-tab-modifier/issues/120

Ah, looks like this is a duplicate of https://github.com/sylouuu/chrome-tab-modifier/issues/120


Update README.md

The config file has default 'keyName' => 'X-Authorization',

Change the config is maybe better

I think this was better :)


Document and make web-controllable the default_streams feature

DefaultStream is a table that allows us to configure the list of default streams for a given Zulip realm when new users join. It's currently only accessible via a management command (./manage.py set_default_streams) and ideally would be something that has a web UI for configuring it in the Administration page.  We actually even already have API routes for controlling this (that I'm pretty sure nobody has used), so I think this could be entirely a frontend project.
While looking into this I also noticed that we incorrectly use PATCH rather than PUT in the API definition in zproject/urls.py; we should fix that as part of this project.
    url(r'^default_streams$', 'rest_dispatch',
        {'PATCH': 'add_default_stream',
         'DELETE': 'remove_default_stream'}),

Hmm, I take back the comment about frontend-only; we'll need to add the current list of DefaultStream values to fetch_initial_state_data, put it into page_params, and probably add sending an event for DefaultStream updates processed in apply_events.
The new feature tutorial on ReadTheDocs I think documents how to do most of that; happy to answer any questions on how to do this.

I just discovered I actually have an old (non-working) branch that has some partial progress towards this; I posted it as https://github.com/timabbott/zulip/tree/default-streams-rebase


Add -k for short notation when using dapr init

There need to be -k for kubernetes
Usage:
dapr init [flags]
Flags:
-h, --help                     help for init
--install-path string      The optional location to install Dapr to.  The default is /usr/local/bin for Linux/Mac and C:\dapr for Windows
--kubernetes               Deploy Dapr to a Kubernetes cluster
--network string           The Docker network on which to deploy the Dapr runtime
--runtime-version string   The version of the Dapr runtime to install. for example: v0.1.0-alpha (default "latest")

Closed with #282


Declare ARPHRD_CAN on the linux_like module

Supersedes #2077, closes #2077

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit f77d49b818d38742f65a8256b9e15a0199516ef8 has been approved by Amanieu

:hourglass: Testing commit f77d49b818d38742f65a8256b9e15a0199516ef8 with merge 1d5190618101005058d8d0e3eb8fd1e990be6882...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing 1d5190618101005058d8d0e3eb8fd1e990be6882 to master...


Fix issue with math.ceil() and math.floor() output

Addressing issue #759
math.ceil()  and math.floor() are supposed to return int inside numba_dpex.kernel, but they are returning float64. This PR should fix that issue.

[x] Have you provided a meaningful PR description?
[x] Have you added a test, reproducer or referred to an issue with a reproducer?
[x] Have you tested your changes locally for CPU and GPU devices?
[x] Have you made sure that new changes do not introduce compiler warnings?
[x] If this PR is a work in progress, are you filing the PR as a draft?

Please note this is a difference between c++/sycl/ocl and python.
In c++/sycl/ocl ceil always returns floating pointer type. In python it returns int

Please note this is a difference between c++/sycl/ocl and python. In c++/sycl/ocl ceil always returns floating pointer type. In python it returns int

Exactly, ceil(),floor() in all c/c++ implementations out there do float --> float or double --> double, including SYCL. We are discussing with inria what we should do, whether we follow the python construct or c/c++ , we are discussing here in issue #759

@chudur-budur I can confirm that on this branch math.ceil returns an int.
But, looking back at https://github.com/IntelPython/numba-dpex/issues/759 , the most troublesome point was that math.ceil and math.floor do not to work with devices that do not support float64 compute (such as igpus), and this PR does not solve that point.
Even if casting everything float32 beforehand, e.g:
import dpctl.tensor as dpt
import dpnp
import numba_dpex as dpx
import math

N = 10

cst = dpnp.float32(0.5)

@dpx.kernel
def func(a):
    i = dpx.get_global_id(0)
    i = math.ceil(dpnp.float32(i + cst))
    a[i] = a[i] + i

a = dpt.ones(N, dtype=dpnp.float32)

func[dpx.Range(N)](a)

print(dpt.asnumpy(a))

gives a SyclProgramCompilationError: which is the error we're used to for float64 kernels on float32-only  hardware.

In fact forget what I just said, this works:
import dpctl.tensor as dpt
import dpnp
import numba_dpex as dpx
import math

N = 10

cst = dpnp.float32(0.5)

@dpx.kernel
def func(a):
    i = dpx.get_global_id(0)
    i = math.ceil(dpnp.float32(i) + cst)
    a[i] = a[i] + dpnp.float32(i)

a = dpt.ones(N, dtype=dpnp.float32)

func[dpx.Range(N)](a)

print(dpt.asnumpy(a))

so the remaining issue in this example was that a + b fails to compile if a is int64 but b is float32, explicitly casting a to float32 before summing fixes the issue. Without the explicit casting, it seems the JIT casts a from int64 to float64.
Knowing this is probably enough to write numba_dpex kernels that can work on all devices but one must think of casting explicitly to have compatibility with igpus.

so the remaining issue in this example was that a + b fails to compile if a is int64 but b is float32, explicitly casting a to float32 before summing fixes the issue. Without the explicit casting, it seems the JIT casts a from int64 to float64.
Knowing this is probably enough to write numba_dpex kernels that can work on all devices but one must think of casting explicitly to have compatibility with igpus.

Most likely it's an issue with type inference. Let's tag @diptorupd he might can give us more pointers.

In fact forget what I just said, this works:
import dpctl.tensor as dpt
import dpnp
import numba_dpex as dpx
import math

N = 10

cst = dpnp.float32(0.5)

@dpx.kernel
def func(a):
    i = dpx.get_global_id(0)
    i = math.ceil(dpnp.float32(i) + cst)
    a[i] = a[i] + dpnp.float32(i)   # <<<< change is there

a = dpt.ones(N, dtype=dpnp.float32)

func[dpx.Range(N)](a)

print(dpt.asnumpy(a))

But I have another question,  i = math.ceil(dpnp.float32(i) + cst) is supposed to give an indexing error at the last value of i. If a = [0, 1, 2], then i is 0 <= i <= 2. At i = 2,  i = math.ceil(dpnp.float32(i) + cst) should evaluate to i = math.ceil(2.0 + 0.5) = 3. But then why the kernel is not complaining about that?

@fcharras

Knowing this is probably enough to write numba_dpex kernels that can work on all devices but one must think of casting explicitly to have compatibility with igpus.

We are aware of this issue and it is going to be addressed in future. Unfortunately I can't provide you any estimations at the moment.

@chudur-budur

But I have another question,  i = math.ceil(dpnp.float32(i) + cst) is supposed to give an indexing error at the last value of i. If a = [0, 1, 2], then i is 0 <= i <= 2. At i = 2, i = math.ceil(dpnp.float32(i) + cst) should evaluate to i = math.ceil(2.0 + 0.5) = 3. But then why the kernel is not complaining about that?

Numba doesn't do boundary checks as it has devastating effect on performance.

@fcharras

Knowing this is probably enough to write numba_dpex kernels that can work on all devices but one must think of casting explicitly to have compatibility with igpus.

We are aware of this issue and it is going to be addressed in future. Unfortunately I can't provide you any estimations at the moment.

The solution I have in mind for a while is to split up DpexKernelTargetContext that is now used to compile all kernels into two separate contexts: DpexFP32KernelTargetContext and DpexFP64KernelTargetContext. Once we have two separate contexts we will also have two separate registries for math functions. Then, at the point of compiling we can pick the right context based on what device is targeted.
I have not yet worked out all the details, but it should not be a huge effort to prototype. Also, the approach may not solve all issues around lack of fp64 support. I hope to get it in for the next release.

The PR is out of date and bitrotten and as stated in https://github.com/IntelPython/numba-dpex/pull/960#issuecomment-1480435881 the approach needs to be better thought out to solve the underlying issue.
Closing the PR.


Clicking on ground is targeting square right next to it

Experiencing an issue where clicking on the ground causes character to walk to an adjacent square. The click target seems to be off, and also prevents picking up dropped items

check this issue here for a solution, i believe it is the same we had https://github.com/SolastaMods/SolastaUnfinishedBusiness/issues/3452

yep, same.


Can't login

nothing happens when pressing login (open in Browser), no tab in browser pops up

@snokey https://chatterino.com/client_login

the link Felanbird posted can be used for now, however to be able to try to diagnose the issue we need more information from you about your system.
What's your Operating System?
What's your default browser?
Are you able to click links in Chatterino and have them open in your browser?

In addition to what pajlada said, also consider using the bug report template when reporting future issues.

I cannot reproduce this in Chatterino Nightly 2.1.7 (commit 6d5ba0c44, 24.05.2020)
Can someone please confirm if they can reproduce this?


RDC errors

After creating the resets on the points suggested by the integration guide, got a lot of SETUP_RESET_ASSERT_MISSING errors, one example is:

This is the schematic view, with brown signal defined as reset point.

are there any RDC reference settings available? I am using vc_static for RDC.

@amullick007 can you provide any guidance here?

@amullick007 can you provide any guidance here?

Did you apply this constraint?
https://github.com/chipsalliance/caliptra-rtl/blob/3ab518de05d6c056e268b13162eb7ec804b6afcf/docs/CaliptraIntegrationSpecification.md?plain=1#L801

Yes.

From: Avirup Mullick @.>
Sent: Friday, June 7, 2024 4:54 PM
To: chipsalliance/caliptra-rtl @.>
Cc: Lei Lin (H10 CAPITAL) @.>; Author @.>
Subject: Re: [chipsalliance/caliptra-rtl] RDC errors (Issue #521)
@amullick007https://github.com/amullick007 can you provide any guidance here?
Did you apply this constraint?
https://github.com/chipsalliance/caliptra-rtl/blob/3ab518de05d6c056e268b13162eb7ec804b6afcf/docs/CaliptraIntegrationSpecification.md?plain=1#L801
—
Reply to this email directly, view it on GitHubhttps://github.com/chipsalliance/caliptra-rtl/issues/521#issuecomment-2155704539, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BIIIZMSZ5OHATLYW3RQNQLTZGJB3JAVCNFSM6AAAAABIEO26NOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJVG4YDINJTHE.
You are receiving this because you authored the thread.Message ID: @.***>

Can you try adding a constraint on scan_mode to set it to 0? Looks like I missed adding this to the documentation since these are taken care of globally in our internal flows.

I did some clean up with updating some of the reset grouping. The previous error disappeared. There are still 2 major violations left:

from RISC_VEER_CORE_RST to RISC_VEER_DBG_DM_RST:
[cid:62459c81-74e0-4920-b3a7-16713c201c30]
2.
From CPTRA_NON_CORE_RST to   RISC_VEER_CORE_RST:
[cid:097486d1-3d5e-43b8-a015-a3d160ab4cd9]
I did all set_case_analysis.
Any other settings I will need to clear these 2 errors?

From: Avirup Mullick @.>
Sent: Friday, June 7, 2024 7:13 PM
To: chipsalliance/caliptra-rtl @.>
Cc: Lei Lin (H10 CAPITAL) @.>; Author @.>
Subject: Re: [chipsalliance/caliptra-rtl] RDC errors (Issue #521)
Can you try adding a constraint on scan_mode to set it to 0? Looks like I missed adding this to the documentation since these are taken care of globally in our internal flows.
—
Reply to this email directly, view it on GitHubhttps://github.com/chipsalliance/caliptra-rtl/issues/521#issuecomment-2155759630, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BIIIZMUKFRF7MPYKL2XTEH3ZGJSFDAVCNFSM6AAAAABIEO26NOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJVG42TSNRTGA.
You are receiving this because you authored the thread.

The images are not rendering for me in the comment. Can you please reupload.

OK, the 2 major violations are between RISCV_VEER_DBG_DM_RST and RISCV_VEER_CORE_RST. Examples are:
(1) SrcObject : rvtop/veer/dbg/dbg_data1_reg/genblock.genblock.dff/dout/Q[31:28]
DestObject: rvtop/veer/dma_ctrl/GenFifo[1].fifo_prty_dff/genblock.dffs/dout/CLK
(2) SrcObject:  rvtop/veer/dec/tlu/halt_ff/genblock.dff/dout/Q[13]
DestObject: rvtop/veer/dbg/dmcontrol_wrenff/dout/CLK
What are the settings I should apply on these 2 resets?


I would like to join Hexo

Hi, I am TriDiamond, an active open source web developer. I have made two themes for Hexo, Obsidian and Aurora.
I have submitted the new theme's PR and I have notice Hexo's PR review is not as active as before. Since I am active on Open Source right now. I would like to join Hexo as a member and help out on the Hexo theme side of the development. Probably will work on some of the features in Hexo core.
I am not sure where to look for an Admin on the website, but I see a message in the about section, stating posting an issue if one have question about joining Hexo.
Hope this get to one of the Admin in Hexo team, thanks.

CC @hexojs/core


chroots/containers support

At initramfs time, /usr and /etc belong to the initramfs image, while the actual root is being prepared somewhere else (e.g. /sysroot). In this case, os-release path looks like /sysroot/usr/lib/os-release

See https://github.com/ashcrow/osrelease/issues/1

#4 resolves this issue.


fixed multitouch panning

The "panmove"-Event only fires when using a single finger.
More fingers may be added while panning.
But if the panning ist started with multiple fingers, the "panmove" event doesen't fire.
Added "rotate.recognizeWith(pan);" to fix this.
Now the "panmove" event is fired when starting with multiple fingers.

Looks good to me! Sorry for the long delay.
Thanks for your contribution!
Erik


Add a widget showing a chart of the recent trend of code coverage percentage

Add a widget "Code Coverage History" showing a chart (using Chart SDK) of the recent trend of code coverage percentage. This would be similar to the built in widget for build history.

Attaching mockup of what widget could look like.


[Window Management] Jump window between monitors

Extension – Window Management
Author: RayCast
Describe the feature and the current behavior/state.
If you have two monitors side by side, it would be great to jump the window from one monitor to the other when you press hotkey to go from half left to half-right, twice.
Magnet app does that.
Who will benefit from this feature?
All people with two or more screens.

Hi,
I think this could be implemented!

also need, like rectangle

We could not find the extension related to this issue. Please update the issue with the link to the extension.

@raycastbot reopen


React to required attribute

Added code to make recaptcha field conditionally non-required, for example using expressions 'ng-required="!isAuthenticated" '.

Closing this as it was implemented in #107


[Question] Access Mat from BoxedRef without cloning

I was going through and updating the opencv dependency in my Rust project. Due to some breaking changes, functions that previously returned a Mat now return a BoxedRef. How would I return the underlying Mat without cloning? Here is an example couple lines of code:
pub fn crop(input: &Mat, x: i32, y: i32, width: i32, height: i32) -> opencv::Result<Mat, RipsError> {
    let cropped = Mat::roi(input, Rect::new(x, y, width, height))?;
    Ok(cropped)
}

Now returns the error:
Type mismatch [E0308] expected `Mat`, but found `BoxedRef<Mat>`

Is there any way to "unwrap" the Mat out of the BoxedRef? I'm trying to avoid cloning if possible, but I'm a bit confused on how to do this with how the crate currently functions.
Opencv Version: 0.91.3
OS:                     Ubuntu 22.04
Edition:               2021
Cargo Version:   1.77

Not being able to extract Mat to not lose lifetime information was one of the goals of introducing BoxedRef so it’s not possible. To fix the issue you’re encountering you’ll need to update the signature of crop function to take &impl MatTraitConst instead of &Mat to match what roi function expects.


Add documentation to document-execute/os_usage module

TITLE
Description
Added comments to document-execute/os_usage.rs
Checklist

[x] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.
[x] Documentation has been added/updated.

Codecov Report

Merging #897 (0a69804) into main (5763f6b) will decrease coverage by 0.16%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #897      +/-   ##
==========================================
- Coverage   89.69%   89.54%   -0.16%
==========================================
  Files          50       50
  Lines       14351    14351
==========================================
- Hits        12872    12850      -22
- Misses       1479     1501      +22

Files
Coverage Δ

src/execution/os_usage.rs
99.64% <ø> (ø)

... and 2 files with indirect coverage changes


fix issue in BrokerToControllerChannelManager

Problem introduced by https://github.com/confluentinc/kafka/commit/a60fce7b0df1b887feba35abfe84cde37d0fa185
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

LGTM


Removed a11y warning on Gradient Header

As per https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-level the proper level will be inferred from the tag being h1-6 or explicitly defined by the end dev.  Just added an ignore for the warning so as to not be a false positive.
This fixes #183

@niktek Do you think we go one step further and still provide the an a11y prop for level? In the off chance someone sets a non-heading tag they would still have the ability to set this value manually. I'd say we add that to the component/docs -or- just give the recommendation that to keep a11y you should use headings. The former seems more flexible though.

Look like I already thought of this, so I'll go ahead and merge. Thanks!


Second Time if Access Same TWitter Integration its Showing Error Domain=STHTTPRequest Code=401 "HTTP Status 401

Hi Mrs Nicolas Seriot..,
i am using STTwitter every thing woking  fine. But some times i got this issue "" -- error: Error Domain=STHTTPRequest Code=401 "HTTP Status 401: Unauthorized" UserInfo=0x7bf21cd0 {NSLocalizedDescription=HTTP Status 401: Unauthorized} "" , When means first time register via twitter that time i got user id, ScreenName,  if again i try means its  showing this  Error ? if stop the App then i will try means its  woking fine. Any solution  pls?

What do you mean by "register twice via Twitter"?
If you can send me a code that doesn't work and I can reproduce the issue, I'll be happy to fix it.

Please send more details demonstrating the issue.
I'll close the issue meanwhile since I do not understand the issue and cannot reproduce it.
Thank you.


Update elementaryos.svg (767 Bytes => 477 bytes)

Use circle and simplified path.

Brilliant stuff! You can shrink a few more bytes from it by following the guide in the CONTRIBUTING files.