[BUG] Unable to play sound on headset

Describe the bug
When trying to set the audio output using the device config, my headset isn't listed in "Available System Devices".
To Reproduce
Steps to reproduce the behavior:

Right-click with "Device config" item
Scroll through "Available System Devices"
Notice "Headset" isn't there

Expected behavior
I should be able to output sound directly to my headset.
Screenshots
If applicable, add screenshots to help explain your problem.

Here I tried manually setting the output to my headset, but it didn't work. Also there is no more than 6 available system devices, 1 and 2 being integrated audio and 3 to 6 being hdmi/displayport audio) :

But in Minecraft's options I can see my Headset :

Environment (please complete the following information):

OS: [Arch Linux 6.9.1]
Mod Versions: [3.1.5 and 4.0.8] (screenshots are from 3.1.5)
Minecraft Version: [1.20.1]
Forge Version: [1.20.1 - 47.2.0]

Additional context
Output of alsactl info, notice "Headset" (card 2) is listed here:
#
# Sound card
#
- card: 0
  id: PCH
  name: HDA Intel PCH
  longname: HDA Intel PCH at 0x4402110000 irq 174
  driver_name: HDA-Intel
  mixer_name: Realtek ALC897
  components: HDA:10ec0897,10438814,00100402
  controls_count: 48
  pcm:
    - stream: PLAYBACK
      devices:
        - device: 0
          id: ALC897 Analog
          name: ALC897 Analog
          subdevices:
            - subdevice: 0
              name: subdevice #0
        - device: 1
          id: ALC897 Digital
          name: ALC897 Digital
          subdevices:
            - subdevice: 0
              name: subdevice #0
    - stream: CAPTURE
      devices:
        - device: 0
          id: ALC897 Analog
          name: ALC897 Analog
          subdevices:
            - subdevice: 0
              name: subdevice #0
        - device: 2
          id: ALC897 Alt Analog
          name: ALC897 Alt Analog
          subdevices:
            - subdevice: 0
              name: subdevice #0
  hwdep:
    - device: 0
      id: HDA Codec 0
      name: HDA Codec 0
      iface: 16
#
# Sound card
#
- card: 1
  id: NVidia
  name: HDA NVidia
  longname: HDA NVidia at 0xa1080000 irq 17
  driver_name: HDA-Intel
  mixer_name: Nvidia GPU a5 HDMI/DP
  components: HDA:10de00a5,10de17ff,00100100
  controls_count: 28
  pcm:
    - stream: PLAYBACK
      devices:
        - device: 3
          id: HDMI 0
          name: VG27B
          subdevices:
            - subdevice: 0
              name: subdevice #0
        - device: 7
          id: HDMI 1
          name: HDMI 1
          subdevices:
            - subdevice: 0
              name: subdevice #0
        - device: 8
          id: HDMI 2
          name: HDMI 2
          subdevices:
            - subdevice: 0
              name: subdevice #0
        - device: 9
          id: HDMI 3
          name: HDMI 3
          subdevices:
            - subdevice: 0
              name: subdevice #0
  hwdep:
    - device: 0
      id: HDA Codec 0
      name: HDA Codec 0
      iface: 16
#
# Sound card
#
- card: 2
  id: Headset
  name: PRO X Wireless Gaming Headset
  longname: Logitech PRO X Wireless Gaming Headset at usb-0000:00:14.0-6, full speed
  driver_name: USB-Audio
  mixer_name: USB Mixer
  components: USB046d:0aba
  controls_count: 6
  pcm:
    - stream: PLAYBACK
      devices:
        - device: 0
          id: USB Audio
          name: USB Audio
          subdevices:
            - subdevice: 0
              name: subdevice #0
    - stream: CAPTURE
      devices:
        - device: 0
          id: USB Audio
          name: USB Audio
          subdevices:
            - subdevice: 0
              name: subdevice #0

Update: installing the package pipewire-alsa made the "Automatic" option in Device Config play the sound on my headset.


⚠ Piwigo has degraded performance

In 694fce9, Piwigo ($SERVER_BASE/piwigo/) experienced degraded performance:

HTTP code: 200
Response time: 27974 ms

Resolved: Piwigo performance has improved in 53f3998.


MM-46004/MM-46274 Add Module Federation and Boards as a POC Product

This is a proof of concept of the work required to ship Focalboard as a "product" using Webpack Module Federation.
What works:

Boards is ran as part of the web app without being a plugin
In development, the web app checks for Focalboard's dev server on startup to know whether or not Focalboard should be loaded on startup. The web app runs both ways without any errors!
The Focalboard plugin is loaded and appears to run despite going through this new system.
As far as I can tell, it's loading shared libraries through module federation properly, so initial load should be slightly sped up

Short term TODOs (needed for 7.4 release):

Images and other public files don't load. Their path appears correct (eg. http://localhost:8065/plugins/boards/public/app-bar-icon.png), but the MM server doesn't know that the
Focalboard needs to actually be tested. I've just done some smoke testing of making sure the menu item for Boards appears, the Boards RHS appears, and that there's no errors in the console related to it on page load.
I haven't done the part to actually ship a product with a release of MM, but this covers the development case well enough.
There are some warnings from module federation to sort out around package versions. I'm not certain where they're coming from, but they seem harmless at the moment.

Longer term TODOs (after 7.4 release):

Having a new interface for a product rather than just instantiating the Focalboard plugin in a new way
Finding a way to standardize some of the plugin/product setup logic since parts of the Webpack config are going to be shared with the web app and other products.

Ticket Link
https://mattermost.atlassian.net/browse/MM-46004
https://mattermost.atlassian.net/browse/MM-46274
Related Pull Requests
https://github.com/mattermost/mattermost-server/pull/20888
https://github.com/mattermost/focalboard/pull/3748
Release Note
NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

For reviewers, I kind of threw a bunch of people at this to make sure at least a few different teams have visibility into what's changing here.
As I probably should've highlighted above, this shouldn't introduce any changes to people who don't have the product version of Boards (or any other future products) installed and running.

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/238430

/e2e-tests

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/240020

@jgilliam17 For QA testing this, it shouldn't include any user-facing changes at the moment, so this probably just needs some smoke testing. This new code for loading Boards as a product only applies to development currently, so Boards will appear as a plugin as it has before.

Thanks @hmhealey
Smoke tested, looks good to merge.
E2E report - no PR related failures.


TAJO-2145: Error codes based on errno.h need prefix

How about the prefix 'SYS'? Actually, those error codes were designed to indicate system call errors.

+1 LGTM!


Tests for the refactored multisig functionality

See https://github.com/input-output-hk/cardano-node/pull/1880

https://github.com/input-output-hk/cardano-node-tests/pull/155


Simplify cell merging?

Considering some changes which would "demote" cell merging behaviour to a CellProperty, e.g.
#r "nuget: ClosedXML"
#r "../FsExcel/bin/Debug/netstandard2.1/FsExcel.dll"
let savePath = "/temp"

open System
open FsExcel
open ClosedXML.Excel

[
    Cell [
        String "Hello"
        Merge(3, 4)
        VerticalAlignment Middle
    ]
]
|> Render.AsFile (System.IO.Path.Combine(savePath, "LocalDemo.xlsx"))

This has a few advantages:

simplifies the expression of ranges, which has got quite complicated
may mean that we don't have to have a separate concept of merged cell borders. (Current implementation puts borders around all populated cells, whether merged or not.)
might simplify the code and documentation generally.

@emigajda  As cell merging was your valuable addition originally, I'd value your thoughts on this proposal.  Either here or privately on LinkedIn or gmail (kit dot eason) as preferred.  Thanks!


cargo looks for wrong URL

hello, me again...
i think the following problem started when the crates.io mirror changed its format (but i am not sure about that).
running cargo update on an internal machine works fine - the crates info seems to be updated. but when i try to run cargo build (or something similar) this happens:
error: failed to download from `http://panamax.internal/crates/bitflags/bitflags-1.3.2.crate`

Caused by:
  failed to get 200 response from `http://panamax.internal/crates/bitflags/bitflags-1.3.2.crate`, got 404

what does exist on my local mirror is:
http://panamax.internal/crates/bi/tf/bitflags/1.3.2/bitflags-1.3.2.crate

on the external machine i built pamamax manually, upgraded everything to the latest version and ran panamax sync:
$ git pull origin master
$ cargo build --release
$ ./target/release/panamax sync mirror/

versions:
panamax 1.0.0
cargo 1.54

any advice?

The storage format changed recently, but that's still strange it's not getting the right URL.
Are you using panamax serve on the serving side, or nginx?
If you're using nginx, you'll likely need some rewrite config settings:
	# Rewrite the download URLs to match the proper crates location.
	rewrite "^/crates/([^/])/([^/]+)$"                     "/crates/1/$1/$2"         last;
	rewrite "^/crates/([^/]{2})/([^/]+)$"                  "/crates/2/$1/$2"         last;
	rewrite "^/crates/([^/])([^/]{2})/([^/]+)$"            "/crates/3/$1/$1$2/$3"    last;
	rewrite "^/crates/([^/]{2})([^/]{2})([^/]*)/([^/]+)$"  "/crates/$1/$2/$1$2$3/$4" last;

from https://github.com/panamax-rs/panamax/blob/master/nginx.sample.conf
If you already have those in your nginx config, or you're using panamax serve and it's still not working, that's indeed strange.

thanks a lot for your QUICK reply!
i use nginx directly - without panamax serve.
the rewrite rules helped although i had to tweak them a little in order to take the version number of the crates into account:
        rewrite "^/crates/([^/])/([^/]+)-([\d\.]*).crate$"                     "/crates/1/$1/$3/$2-$3.crate"          last;
        rewrite "^/crates/([^/]{2})/([^/]+)-([\d\.]*).crate$"                  "/crates/2/$1/$3/$2-$3.crate"          last;
        rewrite "^/crates/([^/])([^/]{2})/([^/]+)-([\d\.]*).crate$"            "/crates/3/$3/$4/$3-$4.crate"          last;
        rewrite "^/crates/([^/]{2})([^/]{2})([^/]*)/([^/]+)-([\d\.]*).crate$"  "/crates/$1/$2/$1$2$3/$5/$4-$5.crate"  last;

i did not run exhaustive tests but it did work for a few simple cases (obviously i'm no regex wizard...).
would it be beneficial to run panamax serve?
thanks again and best regards!

Also, this PR, I have removed the need for URL rewriting.

@k3d3

If you already have those in your nginx config, or you're using panamax serve and it's still not working, that's indeed strange.

I'm using panamax serve on macos and am getting a similar issue. Following the README tutorial with panamax serve and using installing ripgrep as a dummy and get the following output from cargo:
cargo install --config "source.crates-io.replace-with=\"panamax\"" ripgrep
    Updating `panamax` index
warning: spurious network error (3 tries remaining): [6] Couldn't resolve host name (Could not resolve host: panamax.internal)
warning: spurious network error (2 tries remaining): [6] Couldn't resolve host name (Could not resolve host: panamax.internal)
warning: spurious network error (1 tries remaining): [6] Couldn't resolve host name (Could not resolve host: panamax.internal)
error: failed to download from `http://panamax.internal/crates/ri/pg/ripgrep/13.0.0/ripgrep-13.0.0.crate`

Sorry to keep updating but now I've meddled in mirror.toml and now can't get the same error. Now I get
cargo install --config "source.crates-io.replace-with=\"panamax\"" ripgrep
    Updating `panamax` index
warning: spurious network error (3 tries remaining): unexpected http status code: 404; class=Http (34)
warning: spurious network error (2 tries remaining): unexpected http status code: 404; class=Http (34)
warning: spurious network error (1 tries remaining): unexpected http status code: 404; class=Http (34)
error: failed to update replaced source registry `crates-io`

Caused by:
  failed to fetch `http://localhost:8080/git/crates.io-index`

Caused by:
  network failure seems to have happened
  if a proxy or similar is necessary `net.git-fetch-with-cli` may help here
  https://doc.rust-lang.org/cargo/reference/config.html#netgit-fetch-with-cli

Caused by:
  unexpected http status code: 404; class=Http (34)

With the server outputting
Not a git repository: 'crates-mirror/crates.io-index'Not a git repository: 'crates-mirror/crates.io-index'Not a git repository: 'crates-mirror/crates.io-index'Not a git repository: 'crates-mirror/crates.io-index'

Ok now I got it back to the problem with panamax.internal somehow.

 I for some reason didn't figure out that the base_url would have to be actually what cargo would connect to. Should have read the docs more thoroughly. Sorry.


Overwriting layers produces Issues for Visibility

From @topowright on December 9, 2016 13:55
Currently the Visibility add-in is configured to overwrite the same four layers for LLOS and the same two layers for RLOS.
Expected Behavior
Layers overwrite and tools run properly
Current Behavior
Layers do not write to TOC
Layers do not write to the default database
Possible Solution
Have unique names for all layers being written from each tool.
RLOS - Observers
RLOS - Targets
as an example
Steps to Reproduce (for bugs)

1.Run the LLOS tool
2.Run the RLOS tool
3.
4.
Context

Your Environment

Version used:
Environment name and version (e.g. Chrome 39, node.js 5.4):
Operating System and version (desktop or mobile):
Link to your project:

Copied from original issue: Esri/military-tools-for-arcgis#82

What do other analytical tools in ArcGIS do?  Do they overwrite the output?  The user is creating this feature for a reason, not because they're interested in what the tool does.  They want to create a feature and persist it, use it somewhere, share it with someone.  Can they do this if the tool automatically overwrites?

@topowright, please update the Expected Behavior for this bug to reflect what we discussed on Friday where the layers will not be overwritten, but new layers will be added after each run.  These layers will be placed in a new group layer after each run of the tool, but the group layers are not currently working properly so the group layer functionality will be added later.

This is no longer an issue based on the change that was made where each of the tools writes unique layers each time the tool is run.


Cannot find module './build/default/binding'

Every time I clone a repo with this module in it - I get this error on start on a script. I have to npm install rbytes to fix it. I use a bunch of other node modules and rbytes is the only one that does this.
Complete error message:
        throw e; // process.nextTick error, or 'error' event on first tick
        ^
Error: Cannot find module './build/default/binding'
    at Function._resolveFilename (module.js:322:11)
    at Function._load (module.js:267:25)
    at require (module.js:351:19)
    at Object.<anonymous> (/home/user/repo/node/node_modules/rbytes/index.js:1:77)
    at Module._compile (module.js:407:26)
    at Object..js (module.js:413:10)
    at Module.load (module.js:339:31)
    at Function._load (module.js:298:12)
    at require (module.js:351:19)
    at Object.<anonymous> (/home/user/repo/node/session.js:4:14)```

@marook... thanks ... works for me, node v0.6.3

i get this error on node 0.6.5, directly from npm install on linux.
marook's workaround also works for me.

Use node's built-in crypto.randomBytes() instead.

Oh, nice, I didn't know it was added in the crypto module. Thanks you.
One question, though - why is it wrapped in try/catch block in the example?


Update steelseries-engine to 3.11.8

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Thank you.


Generate metadata yaml

metadata yaml can be downloaded from https://github.com/signalfx/splunk-otel-java/actions/runs/7411564164/artifacts/1147926232

This seems fine, thanks for doing all this cumbersome leg work. Did you just pull it all together by hand? Is this something that we should document in CONTRIBUTING.md as part of the regular process of keeping docs up to date?
The intent is for us to just maintain this by hand, right? If so, is it better than just maintaining the yaml file itself? I mean, I appreciate that the version and deps come from the build itself, but the rest of it just seems like a bunch of hard coded values hiding in code. 🤷🏻

This seems fine, thanks for doing all this cumbersome leg work. Did you just pull it all together by hand? Is this something that we should document in CONTRIBUTING.md as part of the regular process of keeping docs up to date?

This was created by hand. It may be possible to automate checking whether the source files have changed to make keeping it up to date easier.

The intent is for us to just maintain this by hand, right? If so, is it better than just maintaining the yaml file itself? I mean, I appreciate that the version and deps come from the build itself, but the rest of it just seems like a bunch of hard coded values hiding in code. 🤷🏻

As the structure of this yaml may still change generating it with code will make such changes easier.
I'm a bit skeptical about the usefulness of this and wouldn't spend time on improving it before it is shown that this yaml can actually be used.

@theletterf see the update yaml https://github.com/signalfx/splunk-otel-java/actions/runs/7462323350/artifacts/1156871225


How to stop the Proxy?

how can i stop the Proxy because i edited the config

CTRL + C should do its job.


Feature: mv & ls support  KIT and SYNTH references

Done when

[ ] list samples shows settings in KITs
[ ] list samples shows settings in SYNTHs
[ ] mv sample repairs XML in KITs
[ ] mv sample repairs XML in SYNTHs

NB KIT XML has different form...
/Music/DELUGE/TEST/02/KITS/KIT014.XML:				<fileName>SAMPLES/DRUMS/Kick/CR78 Kick.wav</fileName>
SYNTH XML is similar to SONG:
/Music/DELUGE/TEST/02/SYNTHS/SYNT991A.XML:		<fileName>SAMPLES/CLASSIC/DX7/SYNTH-MedivalSynzzz3.wav</fileName>


Parser.lean - make theorem smth : smth := by work

The reason paperproof doesn't display the initial state of the theorem is it errors out on "initial goal is expected for theorem". I imagine we can make it work?

This is slightly connected https://github.com/Paper-Proof/paper-proof/issues/13.
Would be amazing if you could take a look at the 13th issue too, but it seems harder/maybe not possible to do.


Infinite loop @ Tumblr

Here's an example URL of a Tumblr blog loop redirect that always goes to the same address:
http://t.umblr.com/redirect?z=https%3A%2F%2Fwww.humblebundle.com%2Fstore%2Fpromo%2Fhorror-sale%3Futm_source%3DFacebook%26utm_medium%3DLink%26utm_campaign%3DHorror_Sale_083117&t=YTNkYjZhZjVmNmRjN2RjOTJjYWE4ZWI0ZDgwYmQ0M2I2Yjc2ODA2MCxiNlNxdVVXYQ%3D%3D&b=t%3ALz353Gt9EBZmj6EJog4SOg&p=http%3A%2F%2Fblog.humblebundle.com%2Fpost%2F164829217715%2Fhorror-sale-humble-store-get-in-the-spooky-mood&m=1

Not for me? It goes straight to: http://blog.humblebundle.com/post/164829217715/horror-sale-humble-store-get-in-the-spooky-mood Any way, if you get problems, you should add the site to the blacklist.

So you too have the same issue!
The link you posted is the website the link originates from. The address that should be redirected to should be https://www.humblebundle.com/store/promo/horror-sale which is housed in the early part of the URL.

Other people have real redirect loops, that's why I thought it's not an issue. Any way, nothing I can do, you've got to add it to the blacklist.


iboss: Fixed Package

Fixed OfferID

Removed zip files 1.0.0 and 2.0.0 from the folder
Required items, please complete
Change(s):

Fixed OfferID

Removed zip files 1.0.0 and 2.0.0 from the folder
Reason for Change(s):

Fixed OfferID

Removed zip files 1.0.0 and 2.0.0 from the folder
Version Updated:

N/A

Testing Completed:

N/A
Checked that the validations are passing and have addressed any issues that are present:
N/A

@v-dvedak
iboss

Fixed OfferID
Removed zip files 1.0.0 and 2.0.0 from the folder


Thchs30 mfa

PR types

New features
PR changes

Models | APIs
Describe

add MFA example for THCHS30
here is the result for THCHS30 A11_0.wav
phone -> phone:

syllable - > phone:

text -> phone:

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


Develop

Shcema.org issue is fixed

Coverage increased (+0.002%) to 98.618% when pulling 5c2e733f5fd54fc8da4e03b903c9ac75e54db875 on rajaram5:develop into 031baed11b78cfbea7c918e86cc670c17d359a9f on DTL-FAIRData:develop.


Honor sampling ratio of 1 by setting to AlwaysOn

Motivation and Context
Trying to set 100% sampling still honors the parent's sampling which isnt ideal. We would like to be able to sample without having to use the with_new_root feature.
Marketing Changelog
NA
Checklist

[x] run make test
[ ] PR title must follow conventional-commit-standard
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[ ] commit message and code follows the Code of conduct

Hi @vjsamuel, I'm closing this one because you accidentally opened it in the wrong repository.


Pytorch QAT

Hi,
I am trying to use Pytorch's native QAT instead of pytorch_nndct and then use Vitis AI's Quantization and Compilation for a VCK190. Is there a way to do this? If not would the new ONNX compatibility enable a QAT fp32 Pytorch model to be run on the ONNX runtime in Vitis AI 3.0?
The reason I am exploring this is we need to run both on GPU and FPGA devices with QAT trained models without having to modify the model for pytorch_nndct.

Hi @ash8327 ,
I am sorry that there is no way to do this yet. The quantization should correspondence to hardware but Pytorch's native QAT cannot do it.


FilterBox visualization error with empty results from Druid datasources

I am testing 0.36 with two backported patches of mine:

https://github.com/apache/incubator-superset/commit/a52b9ee8fffb123d98e111c26ca82714deae49de
https://github.com/apache/incubator-superset/commit/5ab5457522a141139958a52c88e021c3e5a50ad7

I noticed that when using FilterBox with Druid datasources (so not via SqlAlchemy) if the time range returns an empty result then I see:
An error occurred while rendering the visualization: TypeError: Cannot read property 'map' of undefined

That expands to:
    in q
    in n
    in div
    in me
    in Se
    in je
    in div
    in div
    in C
    in Re
    in Connect(Re)
    in div
    in r
    in div
    in t
    in div
    in t
    in Uncontrolled(t)
    in ForwardRef
    in div
    in bn
    in div
    in div
    in div
    in nr
    in Connect(nr)
    in div
    in i
    in ir

Expected results
A nice "no results" message that is less confusing.
Actual results
What written above, that is not super easy to decrypt for users. We are currently migrating to SQLAlchemy-based Druid tables, but the migration will be long so it would be great to get a patch for this in the meantime :)
Screenshots

How to reproduce the bug

Select a FilterBox with a Druid Datasource (not one defined via SQLAlchemy, but via the "old" way)
Select one filter that you want.
Run a query with a time range that is not available in Druid segments.
See error

Environment
(please complete the following information):

superset version: 0.36 (plus two commits mentioned above)
python version: 3.7
node.js version: 10.19
npm version: 5.8

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] 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
On Superset 0.35.2 the same workflow leads to another error:
 Traceback (most recent call last)
   File "/srv/deployment/analytics/superset/venv/lib/python3.7/site-packages/superset/views/base.py", line 120, in wraps
     return f(self, *args, **kwargs)
   File "/srv/deployment/analytics/superset/venv/lib/python3.7/site-packages/superset/utils/decorators.py", line 69, in wrapper
     return f(*args, **kwargs)
   File "/srv/deployment/analytics/superset/venv/lib/python3.7/site-packages/superset/views/core.py", line 1092, in explore_json
     viz_obj, csv=csv, query=query, results=results, samples=samples
   File "/srv/deployment/analytics/superset/venv/lib/python3.7/site-packages/superset/views/core.py", line 1013, in generate_json
     payload = viz_obj.get_payload()
   File "/srv/deployment/analytics/superset/venv/lib/python3.7/site-packages/superset/viz.py", line 381, in get_payload
     payload["data"] = self.get_data(df)
   File "/srv/deployment/analytics/superset/venv/lib/python3.7/site-packages/superset/viz.py", line 1832, in get_data
     utils.get_metric_name(metric), ascending=flt.get("asc")
   File "/srv/deployment/analytics/superset/venv/lib/python3.7/site-packages/pandas/core/frame.py", line 4719, in sort_values
     k = self._get_label_or_level_values(by, axis=axis)
   File "/srv/deployment/analytics/superset/venv/lib/python3.7/site-packages/pandas/core/generic.py", line 1706, in _get_label_or_level_values
     raise KeyError(key)
 KeyError

So in the current Superset, the error seems to be in JS, meanwhile before was in viz.py.

ping :)

ping :) Anybody willing to review the above? Should be very quick to fix :)

Didn't test if this is fixed in 0.37, will do!

There have been many changes since this was opened, closing. Feel free to reopen if it's still a problem.


While having bloxstrap - installer hangs

Since no one reported, I'm lost

Well, after I ran one time bloxstrap player in background, roblox with bloxstrap closes and I have this.

You're using a pre-release of Bloxstrap that is not supported by Bloxshade. Please use Bloxstrap version 2.5.4.

https://github.com/pizzaboxer/bloxstrap/releases/tag/v2.5.4

You're using a pre-release of Bloxstrap that is not supported by Bloxshade

makes sense, ty


When full with always showing the last if others card are in 1/3 or 2/3

many thx

Hi there @issour.
If I understand correctly you're saying that if one card is full width, it's always being displayed last.
This is the current Nova behaviour and not specific to this package.
There are some issues on the Nova repo related to this (https://github.com/laravel/nova-issues/issues/1895, https://github.com/laravel/nova-issues/issues/2053).
You can check another package (https://novapackages.com/packages/ericlagarda/nova-text-card) that does hacks this problem if you need a rapid solution.
That being said, it is possible we might add this feature in the future.
Will be closing this issue since is not really an issue from this package, but from the framework itself. Feel free to continue the discussion if you want.

can anyone give proper answer on above questions


Tron

<EMAIL_ADDRESS>

Sorry, your issue will be closed as you did not submit your information in the correct format.


When autoload from composer, __autoload from config stops working

I tried to install a library via composer. When I autoload the composer's, your __autoload stops working.

I found the answer here: http://stackoverflow.com/questions/15801705/composer-breaks-exisiting-autoload-in-codeigniter

__autoload is the old, deprecated way of doing autoloading, because you can have only one.
You should register your autoloader using spl_autoload_register. e.g.:
function customCIAutoload($class)
{
 if(strpos($class, 'CI_') !== 0)
 {
  @include_once( APPPATH . 'core/'. $class . EXT );
 }
}

spl_autoload_register('customCIAutoload');

This way your autoloader and composer's will coexist happily.


azurerm provider change

[Enter feedback here]
Need to change the client_id to use the sp Application (client) ID instead of the Display name. This took me over an hour of digging and researching to figure out.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d35c94ec-c9ad-0233-4115-3ddc43ddcd38
Version Independent ID: 6b231b6b-523b-445d-2b15-7dfc04da7ef9
Content: Quickstart - Configure Terraform using Azure Cloud Shell
Content Source: articles/terraform/get-started-cloud-shell.md
Service: terraform
GitHub Login: @TomArcherMsft
Microsoft Alias: tarcher

@bigjon440 Apologies for the confusion. This was fixed in the documentation last week and we're waiting on a build to publish the article live. The updated doc should be live within the next day or two. I'm going to close this as a dupe of PR #444.


2.4 on spigot 1.15.2

Describe the bug
the holograms are broke,
they change the item meta injection from what i read.
NPCLib usage option
last version available
NPCLib JAR name
npclib-plugin-2.4-SNAPSHOT
Server version
103 paper , 1.15.2
To Reproduce
Steps to reproduce the behavior:
u see the armor stand above the npc
Expected behavior
like before it shows hologram with text and not the armor stand.
Screenshots
https://imgur.com/a/S74Oo4w
Additional context
as i said from what i read:
https://www.spigotmc.org/threads/holograms-break-on-1-15.408675/
https://www.spigotmc.org/threads/entityarmorstand-doesnt-work.417113/
hope its help

Can confirm that this issue is happening to me too.

I have fixed the issue you're having (finally). I'll release a new version of NPCLib within a few days.

Enjoy! https://github.com/JitseB/NPCLib/releases/tag/2.4.1-SNAPSHOT


Switch between hide and view password in LoginActivity

Currently there is no option to view the password whenever it is entered. There should be an option to view the password which is entered.
@timabbott @kunall17 Can i work on this? This is a very small issue. Will send the PR soon.

@saketkumar95 :+1:

Closed https://github.com/zulip/zulip-android/pull/266


Fix Initializing Client syncronously in measure tool

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2758

Relevant Lines:
2803

💛  - Coveralls


:sparkles: feat: Create the content development page hero section

Description
Created the content development page hero section.
issue number #75.
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Screenshot (optional)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

Good attempt closing this pr as it is already done, please pick another issue


[Bug]: recoil crosshair breaks when no aim punch is enabled

[X] read this document fully before sending the issue off.
[X] are running the latest commit from this repository clean without any modification.
[X] and I have installed all requirements for this project as required to run. (even if you have gdb/gcc/g++, you need at the least the version we have selected in the appropriate requirement installation commands in the README.)

What happened?
when no aim punch is enabled the recoil crosshair doesnt move
Relevant log output or backtraces.

Gamesneeze > aimware


fix: transaction type in TxpoolContent

Motivation
Fixes #1843
Solution
As suggested by @mattsee, this just replaces the TxpoolTransaction type with the normal Transaction type.
I updated the test with new data I gathered today. The test would have failed with the old data because it's missing some transaction fields that are now required.
I cheated a tiny bit: Geth encodes the addresses used as keys in the mappings (but not those used in the transaction as from and to) using the checksum format. We can deserialize this just fine, but when we serialize it again in the test to compare the result we encode them as lowercase. This would make our test fail. Therefore I opted to just manually make the addresses in the test data lowercase.
PR Checklist

[x] ~Added~ Updated Test~s~
[ ] Added Documentation
[ ] Updated the changelog

I'm guessing Anvil is using the other transaction type

that's it. do you want to migrate this as well @jannikluhn ?

awesome lmk if you're facing issues. it should simplify things a bit I think


onnxruntime issue on raspberry pi zero 2w

Hi,
I was trying to install EfficientWord-Net on raspberry pi zero 2W.
I am having trouble installing the prerequisite - onnxruntime !
$ python test.py
Traceback (most recent call last):
File "/home/aneka/test.py", line 2, in
from eff_word_net.streams import SimpleMicStream
File "/home/aneka/ewn/lib/python3.9/site-packages/eff_word_net/streams.py", line 4, in
from eff_word_net.engine import HotwordDetector
File "/home/aneka/ewn/lib/python3.9/site-packages/eff_word_net/engine.py", line 8, in
from eff_word_net.audio_processing import First_Iteration_Siamese, ModelRawBackend, Resnet50_Arc_loss
File "/home/aneka/ewn/lib/python3.9/site-packages/eff_word_net/audio_processing.py", line 8, in
import onnxruntime as rt

ModuleNotFoundError: No module named 'onnxruntime'

$ pip install onnxruntime

Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple

ERROR:` Could not find a version that satisfies the requirement onnxruntime

ERROR: No matching distribution found for onnxruntime

Any suggestions?

Soon we will have a tflite model with much lower sizes, once that is created the current models would be deprecated, stay tuned

Thank you,
I am not sure if you plan to continue resnet based detection or not, if yes, the current version, its functionality does not work for new wakeword training. If the resent is going to exist in tflite based approach, please take a look at the associated issues so that it works too..! Thanks for noting!

let me do one thing, will try to incorporate tiny classifiers on top of the embedding model to increase the robustness of the prediction, this should be fairly simple to make , will update soon

let me do one thing, will try to incorporate tiny classifiers on top of the embedding model to increase the robustness of the prediction, this should be fairly simple to make , will update soon

Hi, is there any progress?


Whitelisting paths/methods

I'd like to have granular path control, in order to whitelist some path and method combination.
Example:
Allow GET on /foo
Allow GET and POST on /bar
Deny everything else
It doesn't seem implemented by available plugins. Did I miss something or do I have to write my own plugin?

waiting for the #505

Closing as duplicate of #505.


additional doc

Hello,
Thanks for your action!
Here's a PR for the README instructions to use it with the latest GitHub actions.

Hi @stefanoeb, thanks a lot !!! See ya :)


[BUG] Stock allocation quantity

Describe the bug
Before the recent update when you go to "Allocate Stock Item" Quantity field was filled up by default, now it shows max stock quantity.
Now you have to first check the quantity entered in the order and then manually enter it, and if you are entering it from more than one stock record you have to calculate the remaining amount, in the previous versions this was done automatically.
In addition to this, you can enter quantity greater than the one entered in the order, I believe this was not the case before.
This seems to be a reoccurring bug as I pointed out this before here:
https://github.com/inventree/InvenTree/issues/2183
To Reproduce
Steps to reproduce the behavior:

Create a new sales order
Enter a new part
Click the allocate stock icon
See error (Default quantity is max quantity instead of the actual quantity in sales order)
Expected behavior
By default it should show the actual quanity required in the sales order, and if you have to enter quantity from more than one stock record it should acutomatically calculate the remaining quantity.

Screenshot

:
Deployment Method
Bare Metal
Version Information
0.6.0 dev
Development Version

Thanks for the bug report @irameez
Please check once https://github.com/inventree/InvenTree/pull/2446 is merged :)


Update to latest version of Component-Detection Libraries.

Codecov Report

Merging #436 (6c8e728) into main (8add17b) will decrease coverage by 4.93%.
Report is 3 commits behind head on main.
The diff coverage is 55.17%.

@@            Coverage Diff             @@
##             main     #436      +/-   ##
==========================================
- Coverage   67.47%   62.55%   -4.93%
==========================================
  Files         240      240
  Lines        7603     7616      +13
  Branches        0      878     +878
==========================================
- Hits         5130     4764     -366
  Misses       2473     2473
- Partials        0      379     +379

Files
Coverage Δ

...Sbom.Api/Executors/ComponentDetectionBaseWalker.cs
57.65% <100.00%> (-10.17%)
:arrow_down:

....Sbom.Api/Utils/ComponentDetectorCachedExecutor.cs
73.68% <100.00%> (-15.79%)
:arrow_down:

....Api/Utils/ComponentDetectionCliArgumentBuilder.cs
77.92% <69.44%> (-3.83%)
:arrow_down:

src/Microsoft.Sbom.Api/Utils/ComponentDetector.cs
0.00% <0.00%> (ø)

... and 97 files with indirect coverage changes


Add a font selection menu to the hamburger menu

Allow users to switch fonts for the target language and change the font of the fields on the target side.
DoD:
User can change fonts and the target side updates to display in that font for both TSVs and MD files.

Please hide this feature for the 1.8 release.

Verified in Firefox for Arabic fonts. This is how it looks for Scheherazade New and Awami Nastaliq.

Available in translationCore Create - v1.9.0 build 207-408b416 and looks good.


arrayOfFloats -> floatChannelData?

Change signature to match pcmBuffer for consistency and descriptiveness.
Encourage nil checking rather than passing around empty arrays.
Pre allocate our arrays before we use them to reduce memory thrashing and increase perf.

Much better, not even sure "float" is even needed in the variable name, but this is definitely an improvement.


Get cursor pos

This PR adds a new function, get_cursor_pos to Window. It is meant to call the underlying system and request the cursor position relative to the window origin. I made this because currently, on Windows at least, it is impossible through glutin to get the current cursor position.
I tried to implement all systems, however I cannot test if this works everywhere, only on Windows.

Previous discussion (https://github.com/tomaka/glutin/issues/753) mentions that wayland has no way of directly querying the mouse location, which would mean it would have to be tracked by glutin/winit.

Also see Travis for errors on the OSX/Linux builds.

Ok, should all be fixed now

Maybe not :p

I'd like some thought given about the prototype of the added function:
Current proposal is pub fn get_cursor_position(&self) -> Result<(i32, i32), ()>
I guess the returned numbers are location of the pointer, relative to window coordinates. Now what is the semantics of the error case ?
To be explicit about my question: in wayland, you cannot know where the cursor is if it isn't in the window, as you only receive update events for its location when it is on your window. Even if the glutin app has several windows, knowing the position of the cursor in one window tells nothing about its location relative to other windows, as you cannot know the location of a window regarding the others.
Thus, in wayland, the Err(..) case would trigger quite often for non-fullscreen windows.
Wouldn't using an Option<(i32, i32)> be more accurate, with a clear documentation that depending on the platform, it is not always possible to query the location of the pointer if the pointer is not on the window, and that in such cases the function returns None?

That's probably a good idea. Should the unimplemented! functions return None as well?

Also, I can't seem to find a "get cursor position" type function in the wayland api. Is this because we have to process all the events to get the most recent position? Will there have to be a timeout until the function returns None, or will it just return the most recent position?

Using a Result means that we can differentiate between "the cursor is outside of the window", "the window has been closed", "there's no cursor" (for Android/iOS), and "an error happened".
I also think that the function should return "the cursor is outside of the window" for all platforms when the cursor is outside of the window, instead of returning negative coords.

@trolleyman Yeah, as said previously by @fkaa , wayland do not provide you any way to query the cursor position, it's mandatory to process the events as they come.

Also, I completely agree with @tomaka regarding the return type, makes a lot of sense this way :+1:

So the return type should be Result<Option<(u32, u32)>, ()> to allow differentiation between all these cases?

No, it should be explicit: Result<(u32, u32), GetCursorError>
Where GetCursorError is an enum of each potential problem.

To add to @fkaa 's comment, wayland doesn't even have the notion of only one cursor, but it supports multiple cursors a time: https://www.youtube.com/watch?v=WO2L_ihO_rI


Export/import data for dev tools

Function to export the data either as text (might run into problems when to long) or as a file.
Vice versa there should be an option to import this data again.

Commit 7ab179334e86393e14c2f2d5fcdae37e0dcdcabc added the option to copy the content of an exercise to the clipboard.

Added an option to paste data from the clipboard. Currently, this is buggy as only one entry is inserted instead of all entries.


Should support being asked nicely

I like to talk nice to robots (maybe they will remember me when they take over the world)
When I ask:
@dependabot rebase please

I expect the rebase to start, but unfortunately rebasing will only work when I comment @dependabot rebase

Sorry for the delay replying here - I've updated our regexes to allow that!


Generate safety assertion in non-member functions

Part of #174

Still thinking of other way to check when use skip_assert_initialized

Removed assert_skip for now, as it possible depend on nullables and #176

Seems to pass the check_init_asserts checks so this should be good enough for a while. Thanks!
@homu r+

:pushpin: Commit 2286abf has been approved by gkoz

:zap: Test exempted - status


BUG- Weird interaction between clients with invoke long callback

Description
When two applications run an invoke_long_callback; there are strange behaviors with the client.
TaipyGuiWarning: Empty session id, using global scope instead.
TaipyGuiWarning: Empty session id, using global scope instead.
TaipyGuiWarning: Empty session id, using global scope instead.
TaipyGuiWarning: Empty session id, using global scope instead.

How to reproduce
Run this code and observe the value of each client. The number should always rise in both.
from taipy.gui import Gui, invoke_long_callback, notify, navigate
import time

page = """
<|{value}|>
"""

def update(state, status):
    state.value += 1

def iddle():
     while True:
        time.sleep(1)

def on_init(state):
    invoke_long_callback(state, iddle, [], update, [], 1000)

value = 0

if __name__ == "__main__":
    gui = Gui(page=page)
    gui.run(title='Raising number')

Expected behavior
Each context should be kept from the others.
Screenshots

Runtime environment
Taipy GUI 3.0.dev4

I have tested this one. I could not replicate the issue and it might be on your side due to the missing session id. Please reopen the issue if the error still persisted on your side. Here is the code that I used.
from src.taipy.gui import Gui, invoke_long_callback
import time

page = """
<|{value}|>
"""

def update(state, status):
    state.value += 1

def iddle():
    while True:
        time.sleep(0.5)

def on_init(state):
    invoke_long_callback(state, iddle, [], update, [], 500)

value = 0

if __name__ == "__main__":
    gui = Gui(page=page)
    gui.run(title="Raising number", server_config={"socketio": {"ping_interval": 0.5}})

Not working with this code. I don't think this is on my side. It is replicated even on Taipy Cloud. Have you created two different clients and have yet to see the weird changes?

Thanks for reporting. We will have a look into it. What I suspect that the bug appears when the session id is not properly initialized.


Some recipe tabs are no longer showing

EMI version 1.1.12
Certain PNC recipe views are not showing in EMI only. Tested in a limited instance with fresh installs of PNC, JEI, and EMI to confirm.
https://github.com/TeamPneumatic/pnc-repressurized/issues/1344

(PNC author here)
I see this logged in my client when recipes are processed:
java.lang.AbstractMethodError: Receiver class dev.emi.emi.jemi.impl.JemiRecipeSlotBuilder does not define or inherit an implementation of the resolved method 'abstract mezz.jei.api.gui.builder.IRecipeSlotBuilder addRichTooltipCallback(mezz.jei.api.gui.ingredient.IRecipeSlotRichTooltipCallback)' of interface mezz.jei.api.gui.builder.IRecipeSlotBuilder.
	at TRANSFORMER/pneumaticcraft@8.1.1/me.desht.pneumaticcraft.common.thirdparty.jei.JEIMemoryEssenceCategory.setRecipe(JEIMemoryEssenceCategory.java:62)
	at TRANSFORMER/pneumaticcraft@8.1.1/me.desht.pneumaticcraft.common.thirdparty.jei.JEIMemoryEssenceCategory.setRecipe(JEIMemoryEssenceCategory.java:49)
	at TRANSFORMER/emi@1.1.12+1.21+neoforge/dev.emi.emi.jemi.JemiRecipe.<init>(JemiRecipe.java:55)
	at TRANSFORMER/emi@1.1.12+1.21+neoforge/dev.emi.emi.jemi.JemiPlugin.register(JemiPlugin.java:263)
	at TRANSFORMER/emi@1.1.12+1.21+neoforge/dev.emi.emi.runtime.EmiReloadManager$ReloadWorker.run(EmiReloadManager.java:185)

Using JEI <IP_ADDRESS> and EMI 1.1.12.  Looks like it's related to the JEI tooltip callback API, which has changed in recent JEI builds (19.9+ I think?).

This is related to implementing new JEI APIs in 1.20.1 and 1.21, and has been resolved in EMI 1.1.13


SECOAUTH-177: Switch to Mockito

Updates for https://jira.springsource.org/browse/SECOAUTH-177

Patch applied, thanks.


Sequence component

Hey, I just made a Pull Request!
[WIP] Need to adapt tests
:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[ ] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

Shouldn’t this be exported in the component index.tsx?


Raven Crash for Atom 1.0 feed

Describe the bug
I am trying to add RSS feed using "Kill the Newsletter" service. I have the XML file which is validated. But Raven Reader crashes and it fails to add.
To Reproduce
Steps to reproduce the behavior:

Open 'Raven Reader'.
Click on 'Add'.
Enter https://www.kill-the-newsletter.com/feeds/d6s8iyxebya5iygcybbu.xml and press Enter.
See error.

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10

Additional context
The feed is validated here.

Fixed in https://github.com/mrgodhani/raven-reader/releases/tag/v0.4.3


Java dependency upgrade to version 0.6.0

With node v4.1.0 this package crashed because of node-java.
Check: https://github.com/joeferner/node-java/issues/250

Thank you ;)

Thanks for filing the issue!

FYI this isn't published on NPM yet. Could you bump the version and publish when you get a chance? Thanks!

Bumped and published! Thank you for pointing that out @rossj


[E2E Test Failure] RoundTripTable does not handle PATCH per spec

PATCH https://.../tables/RoundTripTable/guid with content

There is special code within the PATCH operation of the RoundTripTable that isn't included in the E2EServer

Fixed in #183


Remove django-toolbelt dependency

This is a code-less dependency that just depends on other dependencies.
Unfortunately it doesn't have a defined license so we must remove it.
Dependencies:

[ ] https://github.com/apiaryio/polls-api/pull/41

Moved to https://github.com/apiaryio/polls-api/pull/42


perf: investigate the performance of DataChunkBuilder

Proposed by @BugenZhao  @st1page

Any specific directions or existing evidence that this can be further optimized:

For instance does it show up on q17 flamegraph?
Or if a benchmark doesn't currently exist, but we suspect DataChunkBuilder might not be that performant, what should be the spec of the benchmark?
Or some other observation?


Drag and Drop not working on Firefox 45.0.2

It seems that drag and drop feature does not work in Firefox 45.0.2 (this security update was patched for me somewhere around 12.4.2016).
When you try to drag a file in to the drop zone, the cursor state remains as rejecting the operation.
Referring to this example:
http://jsfiddle.net/danialfarid/s8kc7wg0/400/
Works correctly in chrome, but not with the most recent version of Firefox.

Related: #1470.

Demo is working for me on mac firefox 45.0.2.

Could be a temporary browser bug.


when posted on heroku it redirects to <IP_ADDRESS>:8000

when posted on heroku it redirects to <IP_ADDRESS>:8000

Can happen due to misconfiguration (feedjack site added as http://<IP_ADDRESS>:8000), but I don't know how stuff works on heroku, and never tested the app there, so can be something else entirely.
In any case, not likely to look into it myself, unfortunately.

Hello,
How can I reconfigure the site so that feedjack site is added properly? In the settings.py file?

You can add/remove sites via Django admin app.
Can also admin feeds and such there, some management commands are also available for these.
My memory is a bit fuzzy (haven't touched this project in a while), but iirc on first request to feedjack, initial site should be added automatically, which I think might be a problem if heroku does some "ping" request from localhost.
See also: https://github.com/mk-fg/feedjack/#configuration
I think that section is a copy-pasted bit from old feedjack.org, which is now likely to be only available through archive.org wayback machine, but maybe had more such info.


[Putty Module] o at the beginning of the username

Hey! I have been using sheepl for test purposes, but I found that the Putty Module would not work because of a very strange "bug"
Any username specified in the interactive prompt end up being typed in Putty as "o"+, eg:
[in interactive sheepl prompt]
-> username: mika
etc.....
[in putty shell]
-> login: omika
-> password: ####
Connection to omika@server failed
(obviously, as this server as no "omika" user)
I found the culprit in the putty module .py file line 448, removing it solved the issue, however as I can't figure out what the "Send(!o)" (and "send(!n)") do I will not submit a pull request
(in W10 virtual machine, up-to-date at 09 August 2019. Terminal used for sheepl interaction: cmder)

Thanks for the issue submission. Apologies for not picking this up sooner. Those are keyboard combinations, like ALT + o. I will check the module and replicate the issue.

So when testing this, i managed to get the same condition once. It only seemed to appear after the first loop and appeared to be a race condition. The Send("!o") is sending the Alt + o combination to open the putty window. However, this is redundant code as the previous line sends an 'enter' after adding in the computer target.
I have now removed the Send("!o") line and pushed this fix.
Thanks for the bug report.

You are welcome :-)
On Sat, Sep 14, 2019, 09:02 Matt Lorentzen<EMAIL_ADDRESS>wrote:

So when testing this, i managed to get the same condition once. It only
seemed to appear after the first loop and appeared to be a race condition.
The Send("!o") is sending the Alt + o combination to open the putty window.
However, this is redundant code as the previous line sends an 'enter' after
adding in the computer target.
I have now removed the Send("!o") line and pushed this fix.
Thanks for the bug report.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lorentzenman/sheepl/issues/6?email_source=notifications&email_token=AKQZPGAYLT2IZ4W2Z6GE3UDQJSEBVA5CNFSM4IKUPCY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6WWBJI#issuecomment-531456165,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKQZPGCUFCJQA2ILAZG3EVTQJSEBVANCNFSM4IKUPCYQ
.


feat: add List.enum_eq_nil and List.mapIdx_eq_nil

Adds List.enum_eq_nil and List.mapIdx_eq_nil.
These are analogous to List.map_eq_nil.

I ran into the need for these lemmas here: https://github.com/dwrensha/compfiles/blob/4e9c73e01a7a1633e7d88cbba8b4afc542aa251d/Compfiles/Imo1968P2.lean#L25-L31

maintainer merge

bors d+

bors r+


Allow override of Traefik default port

On my server, port 8080 is already taken. This is causing Traefik container to fail while initial configuration phase.
I would like to be able to override this setting.

@patuchov
This is good one. Will be added in next build (in few hours)

Implemented, readme updated
https://github.com/carrier-io/carrier-io/commit/6fa5fe52a1ca9d9034b05c008d92f0eefe657e53


Bug on installation

I'm having a bug while installing the package. I don't know if it's a problem on my machine or the package has a bug, so I'll post this here.
$ pip install pptx2md
WARNING: Ignoring invalid distribution -ip (c:\python310\lib\site-packages)
WARNING: Ignoring invalid distribution -ip (c:\python310\lib\site-packages)
Collecting pptx2md==0.7.9
  Downloading pptx2md-0.7.9-py3-none-any.whl (7.3 kB)
Requirement already satisfied: python-pptx in c:\python310\lib\site-packages (from pptx2md==0.7.9) (0.6.21)
Requirement already satisfied: rapidfuzz in c:\python310\lib\site-packages (from pptx2md==0.7.9) (1.9.1)
Requirement already satisfied: pillow in c:\python310\lib\site-packages (from pptx2md==0.7.9) (8.4.0)
Requirement already satisfied: lxml>=3.1.0 in c:\python310\lib\site-packages (from python-pptx->pptx2md==0.7.9) (4.7.1)
Requirement already satisfied: XlsxWriter>=0.5.7 in c:\python310\lib\site-packages (from python-pptx->pptx2md==0.7.9) (3.0.2)
WARNING: Ignoring invalid distribution -ip (c:\python310\lib\site-packages)
Installing collected packages: pptx2md
  Attempting uninstall: pptx2md
    WARNING: Ignoring invalid distribution -ip (c:\python310\lib\site-packages)
    Found existing installation: pptx2md 1.0.0
    Uninstalling pptx2md-1.0.0:
      Successfully uninstalled pptx2md-1.0.0
  WARNING: Failed to write executable - trying to use .deleteme logic
  Rolling back uninstall of pptx2md
  Moving to c:\python310\lib\site-packages\pptx2md-1.0.0.dist-info\
   from C:\Python310\Lib\site-packages\~ptx2md-1.0.0.dist-info
  Moving to c:\python310\lib\site-packages\pptx2md\
   from C:\Python310\Lib\site-packages\~ptx2md
ERROR: Could not install packages due to an OSError: [WinError 2] O sistema não pode encontrar o arquivo especificado: 'C:\\Python310\\Scripts\\pptx2md.exe' -> 'C:\\Python310\\Scripts\\pptx2md.exe.deleteme'

WARNING: Ignoring invalid distribution -ip (c:\python310\lib\site-packages)
WARNING: Ignoring invalid distribution -ip (c:\python310\lib\site-packages)

The same error happens on commands:

pip install --upgrade pptx2md
pip install pptx2md==0.7.9

I cannot reproduce this error, might be some kind of permission issue (try install in a shell with admin privilege?).
Please provide steps to reproduce this.

I solve it by running on PowerShell as admin, thanks.


Safari dont get any tweak injection

safari browser app will always be like in no jailbreak state
To Reproduce
Steps to reproduce the behavior:

Go to 'safari'
See error

Device:

iOS Version: 12.4
iOS Device: 11,6 iphone xs max
unc0ver Version: 3.7.0 b1

Place an "x" between the brackets if true:

[x ] this is a bug others will be able to reproduce
[x ] this issue is present with all tweaks uninstalled(except for default packages) or disabled
[ ] this issue is present after a rootfs restore
[ x] this issue is present on the latest version of unc0ver

Can confirm. Safari Plus functionality not present when safari is opened.

Pwn has tweeted about this and confirmed it will be fixed in the next release


Update contains-duplicate.cpp

What type of PR is this? (check all applicable)

[ ] 🚀 Added Name
[ ] ✨ Feature
🌟 stared the repo
[ ] 🐛 Grammatical Error
[ ] 📝 Documentation Update
🚩 Other

Description
Added comments to the code making it easy to understand
Add Link of GitHub Profile
https://github.com/RIKI-05

Please merge the PR, Added comments to the Leetcode solution making it easier to understand.


[event-hubs] use core logger

Fixes #2845 and replaces #5401
Event Hubs currently creates debugging namespaces for nearly every class. These changes bring us in-line with the Azure SDK guidelines by consolidating namespaces to the package name with 4 log levels.
I did not remove any of our existing logs. While we should consider if our logging is too verbose, these changes update the existing logging to use the new azure:event-hubs:[log-level] namespaces while taking into consideration the SDK guidelines for which log levels to use.
ToDo:
update README troubleshooting section

/azp run js - eventhubs-client - tests

Looks like some tests are actually intercepting logs to validate certain codepaths were hit. I'll have to look at updating those tests to make CI pass.

/azp run js - eventhubs-client - tests

/azp run js - eventhubs-client - tests

Create a new PR to update the version on core-amqp and to use this new version on event-hubs

core-amqp is already set to an unreleased version (preview.7) and event-hubs depends on core-amqp-preview.7, so I don't think a new PR is needed.

ah! never mind


Implement blockPermutation hint

@MaksymMalicki  @cicr99  @rodrigo-pino  this is a first draft.
I changed all the code to use KeccakF1600 as specified by Rodrigo. This function doesn't return anything but alters the value of the variable the pointer passed as argument points to. This result is then written to memory starting at keccakPtr address.
There is a check in the python hint to revert if   keccakStateSize >= 100 , but keccakStateSize is defined as a constant value (25). I'm wondering if this check could be removed?
Closes #282

@MaksymMalicki  we loop over inputValuesInRange to fill the keccakInput []uint64 array.
And inputValuesInRange is calculated as follows:
			inputValuesInRange, err := hinter.GetConsecutiveValues(vm, readAddr, int16(keccakStateSize))

or in the python hint :
memory.get_range(ids.keccak_ptr - _keccak_state_size_felts, _keccak_state_size_felts)

It seems obvious that keccakInput will never contain more than keccakStateSize elements, do you confirm?

@cicr99  @MaksymMalicki  @har777  ok so i'm having real difficulty to test in an isolated way this hint. I always get errors like
Error:          Expected nil, but got: &fmt.wrapError{msg:"segment 1, offset 7: no builtin: reading unknown value"`

when trying to run the test without any check. I have no idea how to create a setup to run this hint separately at the moment.

@MaksymMalicki  thank you so much for the help !

@MaksymMalicki  i think it doesn't work. The hint will read from memory at location keccakPtr , up to 25 elements. It will then write at cell keccakPtr + 25, for 25 more slots.
So i guess initialized ap, ap+ 1.... ap +24 doesn't solve the issue?

lgtm! 🚀

@cicr99  @rodrigo-pino  waiting for your review to merge


add new tinymce support

Codecov Report

Merging #222 (157e5ae) into master (b6a5b3f) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #222      +/-   ##
==========================================
+ Coverage   89.19%   89.24%   +0.04%
==========================================
  Files          27       27
  Lines        2129     2129
==========================================
+ Hits         1899     1900       +1
+ Misses        230      229       -1

Impacted Files
Coverage Δ

mapentity/registry.py
96.82% <100.00%> (ø)

mapentity/forms.py
76.00% <0.00%> (+0.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 b6a5b3f...157e5ae. Read the comment docs.


launch configuration: support to delete variables from environment

In the terminal config, you can set "foo": null to "delete" a variable from the environment, we should support the same in launch configs for node-debugging.
The corresponding DAP change is https://github.com/Microsoft/vscode-debugadapter-node/issues/138

Relabeled to 'debt' since this aligns treatment of env variables with value 'null' with other parts of VS Code (integrated terminal).

@roblourens I've added support for this in VS Code and node-debug.

Object.keys(e).filter(v => v === null).forEach(key => delete e[key] );
Should that line be like
Object.keys(e).filter(k => e[k] === null).forEach(key => delete e[key] );?

@roblourens oops, thanks for spotting this!


Update pado

Hi. Made some changes to the documentation, please review

closing as this was fixed separately in #905


charm-6.7.1.tar.gz untars to charm

Original author: Jim Phillips (@jcphill)
Original issue: https://charm.cs.illinois.edu/redmine/issues/1161

The file distrib/charm-6.7.1.tar.gz file untars to the directory charm, which is not what the user would expect.

Original date: 2016-08-17 21:56:36

I changed it to untar to a directory named 'charm-6.7.1'

Original date: 2016-08-29 16:00:19

Marking 'closed' so that it doesn't show up under 'open' issues.


Superfluid: Treat severe Gamm faults in a unified way with asset removal

Severe faults include:

Pool can't be found
No osmo PoolAsset in pool
Governance removes a pool asset.

In this situation, all synthetic lockups of this denom should start unbonding, and the superfluid asset should get deleted gracefully.
Implementing the fix for all synthetic lockups is not v7 blocking, but making these all share a common method is. (Perhaps call it BeginDeleteSuperfluidAsset?

This was done pre-v7


6974 add lib implementation

This PR extract only implementation of the PR #7452 (which include symbol baselines that make it hard to review). Symbol baselines will be updated in a separate PR which will merge in here once both are approved to be merged into master branch.

:+1:

@yuit would you be opposed to publishing individual definitions to DefinitelyTyped? We could use it on angular2 to be compatible with new versions of RxJS that require Symbol and Iterable to be present.
CC @alexeagle


Add script to generate a module map from a podspec

To enable the the zip builder to be more robust across CocoaPods versions

@wilhuff Thanks for the quick review. I've addressed all of your comment either in code or comments. PTAL.

Done. The two comments are interdependent and require each other.


IsEmptyObject method moved to Utils

Tested and it look good to me.

But note that this branched needs to be updated before merge.


Remove dead code from the code base

Several parts of the code base (either internal or external) do not build at all.
Investigate if either :

they must be fixed
they are not used anymore and should be removed

Do you have a list of examples?

Do you have a list of examples?

Updated the description with a non-exhaustive list of problems.

This issue is currently investigated by Gaga.

I have written a list of all the build errors here

We removed all the code we know we want to delete, and not revive. Created #11717 to at some point revisit the code we might want to bring back.


Jars on https://bintray.com/typesafe/ivy-releases/sbt/0.13.17 failed to download

steps

Visited https://bintray.com/typesafe/ivy-releases/sbt/0.13.17#files/org.scala-sbt%2Fsbt-launch%2F0.13.17
Clicked to https://bintray.com/typesafe/ivy-releases/sbt/0.13.17#files/org.scala-sbt%2Fsbt-launch%2F0.13.17
Clicked sbt-launcher.jar to download the jar

problem
the redirected URL https://dl.bintray.com/typesafe/ivy-releases/org.scala-sbt/sbt-launch/0.13.17/sbt-launch.jar gave me "Forbidden!"
expectation
sbt-launcher.jar was downloaded successfully.
notes
It has been working until 2-3 hours ago (~7AM PST, Apr 26, 2021).

Same problem here +1

Looks like it is a scheduled maintenance and after May 1st all users should use a different URL to download jars: https://status.bintray.com/
"We recommend that JFrog Cloud customers continue using jcenter.bintray.com, gocenter.io, and repo.chartcenter.io as Artifactory remote repositories going forward."

We no longer use Bintray.
https://eed3si9n.com/bintray-to-jfrog-artifactory-migration-status-and-sbt-1.5.1


Add "New File" as an option

Is your feature request related to a problem? Please describe.
With the inclusion of support for Powershell via the extension, there is no easy way to create a new Powershell script from within AzureDataStudio.  While you can open, edit and save an existing powershell file, creating a new one does not readily appear to be possible.
Describe the solution or feature you'd like
A user should be able to select "New File" from the file menu, and/or an appropriate keyboard shortcut.  AzureDataStudio should create a new tab in the current workspace, but with no additional controls (Query, Dashboard, Notebook etc.) bound to it.
The problem would then be that the application would not know what behavior to apply so therefore Powershell or SQL extensions would not activate, meaning syntax highlighting, intelisense and other features would not be usable.
User would have to either save the file with appropriate extension, or chose document type by some means (such as the status bar in VS Code)
Describe alternatives you've considered
You can use New Connection to get a new tab, and then save that file, changing the extension to .ps1, at which point the connection specific controls are removed from the tab.

I agree that it is an awkward experience now that we have a PowerShell extension. Let's set time to review this experience.

Another consideration is you create a new file, and based on your settings ADS defaults to the language format you desire. VS Code handles it this way with "untitled.xxx"; that xxx is based on your setting under Files: Default Language (files.defaultLanguage). If I set that to powershell then any new file I create will be a PowerShell format. If I set it to sql then it is a query file. (In VS Codes this does not require saving the file either.)
That way if in future additional languages get added for support with Azure or R in SQL Server you can allow it to expand based on the configuration instead of having to always update the menu options.

I almost filed this exact item/request myself.  While I am experiencing the New-PowerShell-file issue multiple times each day, I think there needs to be 3 solutions.

"New File" that allows you to create a file of any extension type I choose.
"New [PowerShell/MDX/JSON] file, basically a list of the most popular file types we know customers are working with in ADS.
An easy to modify list of file extensions that we can add to the list in item #2 by simply checking a box.  This list would be comprehensive and also allow someone to add a file type which isn't there.

I can mock up an image of what I'm thinking #3 would look like if that would help?


DEP 4.11 dev requirements

Issue https://github.com/silverstripeltd/product-issues/issues/531

Travis still failing on PHP 8.0 and 8.1 - I suspect because silverstripe/silverstripe-travis-shared#40 wasn't merged in yet.


PP-3441 Reinstated endpoints previously deprecated

The new endpoints constraining all product operations by gateway accounts introduced in c49ca1e0abaac1386e3f4b2471a277a4fcda1b53, also deprecated the original endpoints. As it became apparent that the original endpoints were also needed, this change reinstates them and adds missing tests.
NOTE
To be merged only after:

https://github.com/alphagov/pay-selfservice/pull/703
https://github.com/alphagov/pay-products-ui/pull/67

@ckalista are we planning to remove the "unsafe" endpoints, e.g. findProductByExternalId

@daniele-occhipinti No, we need to keep all the endpoints. The reason is we only have an gateway account id for the selfservice journeys. When it comes to paying user journeys we don't know anything about geteway accounts, all we have is an product external id or a payment external id in most situations hence why we need to keep the "unsafe" endpoints.

@ckalista I see. Maybe it is a bit confusing for a user of the products API understand why they should use one over the other (passing gatewayId vs not passing it). We need to mention the security rationale at least in the docs? Btw, the README file needs to be updated for the new endpoints, right?

I have updated the README, thanks for reminding of that. Not sure there is anything to explain regarding the rationale of adding new endpoints though. Depending on what is needed people have the choice.

That's cool.
What I meant is this. Say in one year time, we are not around, and somebody needs to use the products API to delete a product, from selfservice or any other microservice. They look at the docs and they see 2 endpoints which do exactly the same thing - one of them requires an extra param gatewayId. They will probably just use the simpler one, and so they will not use the security protection, because they don't see the point of using the more complex one, unless we clearly write in the docs why the more complex is available.
Am I making any sense? Maybe not?

Yes it makes sense @daniele-occhipinti. I have added a line to give people a clue as to why we have added these two new endpoints.


There is no proxy flag

I am developing an atom package for work and cannot use electron-rebuild behind a proxy because there is no option for the command

@Lgueg28  May I know that why this ticket closed ?


Multicursor is broken in insert mode in Neovim v0.10.0-dev

[x] Did you check Readme and existing issues?

Describe the bug
I needed to switch to latest Neovim version and now suddenly this plugin only inserts at 1 location instead of all locations. I thought that I broke something, or there is a bug in recent update, or I need to update NvChad or I need to remove some other plugins. After 2 minutes, I finally realized that it has to do with a new dev version of Neovim. I have no idea what causes this bug, but if the non-dev version will still behave the same way, then this plugin won't be compatible with Neovim v0.10.0. You probably should look into this.
To Reproduce
You have to have docker and docker compose commands available on your host OS (and possibly just).

cd "$(mktemp -d)"
Download multicursors.zip
unzip multicursors.zip
cd multicursors
just (j alias) or the shell code line inside the justfile
wait until you are in Neovim and already selected 4 words
press ia

Expected behavior
a  should appear in all selected location instead of just one.
Screenshots
End of step 6:

i:

a :

Nvim version: 0.10.0-dev (v0.10.0-dev-64c2c7c)
Multicursor version: 0.10.1
Additional context
This bug also causes an auto-save plugin to stop auto saving. Only nvim re-open will fix this.

I can't reproduce this in latest nvim nightly release.

@smoka7, try following an updated reproduction instruction.

Our test are passing in the nvim nightly and stable https://github.com/smoka7/multicursors.nvim/actions/runs/6957221416 .

You know that test don't always represent reality. Try my steps and tell why manual/realistic test fail, but automatic/headless test succeed.
tests/custom.lua
local utils = require 'multicursors.utils'
local search = require 'multicursors.search'
local insert_mode = require 'multicursors.insert_mode'
local config = require 'multicursors.config'
local api = vim.api
local assert = require 'luassert'

local paragraph = {
    'word',
    'word',
    'word',
    'word',
}

describe('inserts mode', function()
    before_each(function()
        vim.cmd [[enew]]
        api.nvim_buf_set_lines(0, 0, -1, false, paragraph)
        local buffer = api.nvim_buf_get_lines(0, 0, -1, false)
        search.find_all_matches(buffer, 'word', 0, 0)
        vim.cmd [[w! before]]
    end)

    after_each(function()
        vim.cmd [[w! after]]
        vim.cmd.bdelete { bang = true }
    end)

    it('inserts text', function()
        local selections = utils.get_all_selections()
        assert.equal(#selections, 3)

        insert_mode.insert(config)
        vim.cmd 'normal! ia '
        insert_mode.exit()

        local buffer = api.nvim_buf_get_lines(0, 0, -1, false)
        utils.clear_selections()
        search.find_all_matches(buffer, 'a word', 0, 0)
        selections = utils.get_all_selections()
        assert.equal(#selections, 3)
    end)
end)

# Doesn't fail
nvim --headless --noplugin -u tests/minimal_init.lua -c "PlenaryBustedFile tests/custom.lua"

In Dockerfile I added version argument:
ARG version=nightly # nightly|stable
RUN wget https://github.com/neovim/neovim/releases/download/$version/nvim.appimage \

If I use nightly (v0.10.0) then the test fails, if I use stable (v0.9.4) then the test succeeds. I literally only change the version of Neovim and result changes, therefore your test doesn't represent real UX (even though I like how automatic tests are run/executed).

Your test passes on my machine and  even on the ci https://github.com/smoka7/multicursors.nvim/pull/71.

Your test passes on my machine and even on the ci #71.

Of course it passes, I said so myself:

# Doesn't fail
nvim --headless --noplugin -u tests/minimal_init.lua -c 'PlenaryBustedFile tests/custom.lua'

This is an automatic test that works on both versions. The only test that doesn't pass is the manual one, source for which is in the multicursors.zip (included in the OP).

I meant both tests also. Nothing I can do about this.

I meant both tests also

So you are saying that you (just to be 100% clear):

downloaded the zip;
run the shell oneliner that builds Docker image and runs/creates the container;
inside the container and inside the automatically opened Neovim with the simple test case pressed ia ;
and in the end all 4 lines were modified (not just one)?

This sounds impossible, since the whole point of creating the Docker image + container is to eliminate all variables. So we both have to have the same experience inside the container. I specifically re-build the image with --no-cache just now, and the issue still persists with version=nightly.

I updated the zip archive. Now you can either write just or just nightly (or just n) to open the file in Neovim nightly version. Or you can write just stable (or just s) to open the file in Neovim stable version. With just the plugin does not work as expected. With just stable the plugin does work as expected.
Here is a screen record:
https://github.com/smoka7/multicursors.nvim/assets/37143421/4ec25583-567e-4b17-95f8-e0ec6678b39d

I'm having the same issue with the nightly version

I reproduced this today, for a fix update hydra to latest commit https://github.com/smoka7/hydra.nvim  or disable hints.


Message Encryption in web app

Description:
Encrypt text messages to ensure privacy and security.
Problem:
Messages are not encrypted so there is no privacy. Encrypted text messaging is a method of secure communication that converts your messages into code.
Additional Context:
I am a GSSOC'24 contributor and would like to work on the issue.

@SatwikMohan please see your discord.

@Ijaiswalshivam yes I have already message you


Feature/update contributor and readme

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

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

Add note about rewrite in REAME and setup for contributors with yarn plus explicit note about good commit messages.
Relates to: #184
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information: N/A

Note: Signed the CLA -> https://cla.js.foundation/webpack/karma-webpack?pullRequest=185

Hi @d3viant0ne
Very good point, I have updated the PR with the changes you've requested.

Hi @d3viant0ne @MikaAK do we have a special workflow to merge a PR ? for example X amount of approvals before merge?


Training files missing for hugginface models

Hi,
I've been using the CRDNN-based model from Huggingface: https://huggingface.co/speechbrain/asr-crdnn-rnnlm-librispeech/blob/main/hyperparams.yaml
I wanted to find a train.py in the LibriSpeech recipe, however it's not there: https://github.com/speechbrain/speechbrain/tree/develop/recipes/LibriSpeech/ASR. In the transducer folder, the hparams are not the same!
Where can I find it?

Hi, I am not sure to understand. What is the problem exactly ? The hparams of the seq2seq is available on ASR/seq2seq/hparams.

Is https://huggingface.co/speechbrain/asr-crdnn-rnnlm-librispeech/blob/main/hyperparams.yaml equivalent to https://github.com/speechbrain/speechbrain/blob/develop/recipes/LibriSpeech/ASR/seq2seq/hparams/train_BPE_1000.yaml?
They seem to have a lot of differences thus I am not sure if the training file in the Librispeech folder matches the Huggingface model.

Issue related to the #1383 and will be tracked there. Closing but feel free to reopen if needed


Virtual text at end of line not truncated

Steps to reproduce

Run vim --clean
Source the following file:

40vsplit
call setline(1, repeat('a', 40))
call prop_type_add('test', {})
call prop_add(1, 0, {'type': 'test', 'text': 'bbb', 'text_wrap': 'truncate'})

first line wraped

Expected behaviour
virtual text should not render
Version of Vim
9.0.1562
Environment
MacOS 13.4
terminal: wezterm 20230408-112425-69ae8472
shell: fish 3.6.1
Logs and stack traces
No response

It looks like this is caused by this code
https://github.com/vim/vim/blob/315cd1fbcbba7c44ec8743f03645bfcaef58bd55/src/drawline.c#L628-L630
where PROP_TEXT_MIN_CELLS is 4 and your virtual text has only a length of 3.  If the virtual text is longer, then you will see it truncated when there is space for 4 or more characters.  Also, if wrapping is turned off in general with set nowrap, then you should also see the virtual text get truncated properly, even if there is room for less than 4 characters.
That makes it seem like the behaviour may be intentional, although using git log -p -G 'space < PROP_TEXT_MIN_CELLS' src/drawline.c I found that commit 04e0ed1ddf399d609dbcb7dbf19e531da1fe6172 seems to have changed this behaviour from being exclusively for text_align: 'right' virtual text.  The behaviour for text_align: 'right' virtual text is also different, since if less than 4 characters of that text fits, then it wraps from the start of that virtual text to the next line.  In contrast to text_align: 'after' from your example splits up the text by trying to put the first 1-3 characters of the virtual text on the first line if there is room.
Given that context, I wonder it was intentional that this PROP_TEXT_MIN_CELLS behaviour affects virtual text other than text_align: 'right' or whether that was an unintentional bug.

I looked for where the behaviour was introduced and although it moved around a bit, it looks like it has been that way since virtual text truncation was first implemented in commit 398649ee44edeb309c77361de697320378104b70


Update README.md for Blink.cmp instructions

This PR is created to update documentation for blink.cmp users following latest update to lazyvim distro using blink as default
See #959 for corresponding issue

I find these instructions really confusing. What files am I supposed to change? Where do put the various snippets, in the Avante opts, or in the blink.cmp opts?


Hide Sidebar Heading

Hi,
I would like to know if hiding sidebar heading is possible when sidebar is too small or with an options in ST preferences?

Actually the sidebar heading (only the text) become invisibile when sidebar is too small.


Make tool pip-installable

Shouldn't be difficult, you need a setup.py and account of Pypi
You can have a look at this repo of mine that can be installed via Pypi as a command line tool (if you install it, the command becomes automatically available in Path!)
https://github.com/corneliusroemer/fasta_zstd_sqlite/blob/master/setup.py

That sounds like a great idea! I will probably do it soon(ish).
I think I want to read a bit about distutils and setuptools first so that I understand what I'm actually doing.

This would be so useful! It's really not hard ;)

I just added a setup.py but then I realized that sc2rf needs some data files to run, which is possible.
It may also modify one of those data files at runtime (virus_properties.json), which is a problem. So it's not as trivial as I hoped it would be. I think I have to leave it in this half-baked state for a short while…

Is it now on pypi? :)


Ray Out Of Memory Error

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
python main.py --async --config config.r2d2.atari --env.name breakout
python main.py --async --config config.muzero.atari --env.name qbert
Expected behavior
RayOutOfMemoryError
Screenshots

Development Env. (OS, version, libraries):
Linux python 3.7.11 jorldy:0.3.0
Additional context
Add any other context about the problem here.
https://stackoverflow.com/questions/60175137/out-of-memory-with-ray-python-framework
https://github.com/ray-project/ray/issues/5572
It seems that GC for ray shared memory doesn't work properly.

It seems that the cause of this issue is not "RayOutOfMemoryError", but the size of the replay buffer is too large.

RayOutOfMemoryError occurs when learning with seq=20, n_step=3, burn_in=10 as in the config above.
However, it was confirmed that if only the buffer size was reduced from 2M -> 0.5M in the same config, it was learned without any problem.

When I measured the size of one transition stored in the buffer, it is approximately 5MB in size.

In the end, this issue seems to be caused by the size of the transition and buffer, so close it.


Launching iconify 2.0.2 immediately crashes Sketch 70.1 running on M1 mac

Launching iconify 2.0.2 immediately crashes Sketch 70.1 running on M1 mac. All other plugins are working fine.

Unfortunately I can't replicate that. There is nothing in plug-in that could crash Sketch. All it does at start is it retrieves saved configuration (if it exists) and opens a web view.
Safari could crash Sketch. It currently does crash Sketch when Safari is set to automatically close when closing plug-in, so there is something fishy going on in Sketch communication with Safari.
Can you check if attached plug-in crashes on your computer? It is a basic plug-in created from Sketch template, which uses same web view version (3.4.4) that is used by Iconify plug-in.
my-plugin-name.sketchplugin.zip

Hey, thanks for that. Yes, your plugin also crashed Sketch in the exact same way. I'm not really sure what the relationship with Safari is though, I don't use Safari at all on my laptop. Anyhow, hoping my confirmation of the crash helps in some way.

Sketch uses Safari to display user interface for plug-ins.
Thank you very much for testing, it helps a lot. It confirms that issue is with web view, not with Iconify plug-in. That plug-in is created with Sketch template using skpm create my-plugin --template=skpm/with-webview with sketch-module-web-view updated to version 3.4.4.
Can you post a Sketch crash log in https://github.com/skpm/with-webview/issues/25 and any details you can think of? I think it might help developers figure out what's causing crashes.

Just posted there, thanks.

this is the issue i see on big sur and version 70 in case it might help you

Thanks for report, it is a bug in plug-in, but it is different from this issue. I suspect its a bug that I've fixed in Icon Finder earlier today, but haven't pushed to Sketch plug-in yet.
I'll publish update soon.

@digiklug Try 2.0.3, I think it might fix that bug.

unfortunately not and now there is another issue

Thanks for reporting! Can't replicate it, but will try to figure it out. I've copied your post to https://github.com/iconify/iconify-sketch/issues/17 because this is a different issue from issue reported by @u-i-1.


Closed notes showing as Open

Some notes that are closed are displaying as open. Need to investigate.

This is fixed. Was a problem with the parser not over-riding status of the note if nothing else had changed about it.


About training step

Thank you for sharing your work!  I have a question about the training steps:
When train the whole network, whether the parameters of the Degradation encoder network are fixed or not?

Hi @DTM3302, thanks for your interest in our work.
The parameters of the degradation encoder are also updated during training the whole network.


Hassio auth

Description:
Allow supervisor to use the user backend from home assistant to provide a SSO for Add-ons.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

Add a test with not-JSON and add a test with JSON that doesn't contain username

@balloob done


[Select] Fix data-value value

Select value currently passed

We're using some custom Select Item and we have to acess original Item's value

@a-x- Thank you :)


'add child to list' is not working when using multiple lists in one document

There is a bug when using multiple lists in one document. It is not possible to create items for the second list. Instead of creating a element in the second list when clicking on add child and then save, no changes are visible. When opening the add child dialog of the first list, the name is marked as missing, so i´m suspecting that doku was trying to create an element in the first list.
steps to reproduce:

create a new document
add two lists to template
Try to create an element in the second list

template:
<html>
<head>
  <title>{{ title }}</title>
  <meta charset="utf-8">
</head>
<body>
  {% for element in first_list %}
  	{{ element }}
  {% endfor %}

  {% for element in second_list %}
  	{{ element }}
  {% endfor %}
</body>
</html>

Fixed in f38ff5e27fb3e12bd6770a29e3f9bee511a0bb09


Add crumbs and misc nodebalancer formatting

Coverage increased (+0.005%) to 83.281% when pulling 30fd7d97caaa9838f4fa86aefed1de455c9350c6 on eatonphil:add-crumbs into 737f07906e52408f01723e35c44999fc50a37f2f on linode:master.

Coverage increased (+0.02%) to 83.295% when pulling 3df41616827689fa92bfc37c6afec475d21490ce on eatonphil:add-crumbs into 737f07906e52408f01723e35c44999fc50a37f2f on linode:master.

Coverage increased (+0.02%) to 83.295% when pulling 3df41616827689fa92bfc37c6afec475d21490ce on eatonphil:add-crumbs into 737f07906e52408f01723e35c44999fc50a37f2f on linode:master.

thank you!


proxy: Add TLS support to endpoint metadata and wire it through to Connect::new

Depends on #1006. Closes #386.
This PR adds a supports_tls field to the endpoint Metadata struct. It is
currently set to true if and only if the meshed="true" label was set by
the control plane, as described in https://github.com/runconduit/conduit/issues/386#issuecomment-391800605.
I changed the ctx::transport::Client context struct to hold a Metadata,
rather than just the labels, so the TLS support determination is always
available. In addition, I've added it as an additional parameter to
transport::Connect::new, so that when we create a new connection, we will
know whether or not TLS is supported.

In general, this LGTM, however I had a hard time reviewing it against #1006 because I think one or the other of these had other changes merged into it.

@briansmith that's strange, as far as I can tell, this branch is based directly on #1006.

this PR was based on master but should be based on eliza/proxy-rebind-on-meta-changes to show the correct diff.

Ah, sorry about that!

Now that #1006 has landed, I've rebased this branch onto master and changed the base branch for the PR.

Commits e813118244ef0959da45889c70a50eb404f5de6e and b669eaf51cad750a46a9a268ad8c4959ad5f7f48 change this branch to use the approach I described in  https://github.com/runconduit/conduit/issues/386#issuecomment-392948046.
/cc @briansmith, @olix0r.

At @olix0r's suggestion, I've factored out the proto change into a separate PR #1041.


configuration_script_sources subcollection

For this BZ, it is necessary to filter on region_number of playbooks that belong to a repository.
While this can be achieved via
GET http://localhost:3000/api/configuration_script_payloads?expand=resources&filter[]=configuration_script_source_id=10000000000001&filter[]=region_number=10

adding it as a subcollection makes it possible to filter on the configuration_script_sources collection directly:
GET http://localhost:3000/api/configuration_script_sources/10000000000001/configuration_script_payloads?expand=resources&filter[]=region_number=10

@miq-bot add_label enhancement, api
@miq-bot assign @abellotti
cc: @h-kataria @gmcculloug

@miq-bot add_label fine/yes

This LGTM!! Thanks @jntullo for the enhancement.  👍

Fine backport details:
$ git log -1
commit 5f807ccfd0d01dbec2169e4062b8a343cb3ee1bd
Author: Alberto Bellotti<EMAIL_ADDRESS>Date:   Wed May 17 11:23:03 2017 -0400

    Merge pull request #15070 from jntullo/enhancement/csp_subcollection

    configuration_script_sources subcollection
    (cherry picked from commit 632c2d0c272e13fcd3f91d8334f4e402dcab6f32)

    https://bugzilla.redhat.com/show_bug.cgi?id=1459986


FR TN Fixes

What does this PR do ?

The % sign is now accepted by French TN (this issue). This is generally implemented as part of the Measures class, which French currently lacks. As a stop-gap measure, the % was whitelisted. It should be properly implemented as part of Measures.

Note:  This fix was tested with pyTest only.  It needs to be tested on Sparrowhawk as well.  I was unable to do that since my hardware doesn't have enough memory.
This PR does not address the following:

The issue with 2h not normalizing to deux heures in FR. Since the abbreviation h is subject to declension, this problem should be addressed with the implementation of the Measures class.

 Use git commit -s to sign.
[x] Do all unittests finish successfully before sending PR?

pytest or (if your machine does not have GPU) pytest --cpu from the root folder (given you marked your test cases accordingly @pytest.mark.run_only_on('CPU')).
Sparrowhawk tests bash tools/text_processing_deployment/export_grammars.sh --MODE=test ...

[x] If you are adding a new feature: Have you added test cases for both pytest and Sparrowhawk here.
[x] Have you added __init__.py for every folder and subfolder, including data folder which has .TSV files?
[x] Have you followed codeQL results and removed unused variables and imports (report is at the bottom of the PR in github review box) ?
[x] Have you added the correct license header Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved. to all newly added Python files?
[x] If you copied nemo_text_processing/text_normalization/en/graph_utils.py your header's second line should be Copyright 2015 and onwards Google, Inc.. See an example here.
[x] Remove import guards (try import: ... except: ...) if not already done.
[x] If you added a new language or a new feature please update the NeMo documentation (lives in different repo).
[x] Have you added your language support to tools/text_processing_deployment/pynini_export.py.

PR Type:

[ ] New Feature
[x] Bugfix
[ ] Documentation
[x] Test

If you haven't finished some of the above items you can still open "Draft" PR.

Both pyTests and Sparrowhawk tests passed.


iwd: don't include testing tools by default

Same intent as #105559, but use separate output to achieve the same goal.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc @romildo who added the testing tools in https://github.com/NixOS/nixpkgs/commit/902105abb9abbcc363a992e017d36d90ccc37d5e

Result of nixpkgs-review pr 105666 run on x86_64-linux 1

1 package built:

iwd


Automate opening docs issues

resolves https://github.com/dbt-labs/actions/issues/111
docs dbt-labs/docs.getdbt.com/#
Problem
Overlooking docs changes related to dbt-core changes
Solution
Automate the creation of docs issues by adding the user docs a label to merged PRs
Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

Before merging this, talk with the docs team about what they want in the issue as well as any labels they require.

@mikealfare because this is triggered off of a PR it needs to live in the repo that the PR lives.  That's why I wrote the workflow with the heavier logic in https://github.com/dbt-labs/actions/pull/128 in /actions so that adapters can call it as well.  So to get the functionality adapters would have to copy this workflow.


Copied self-casting from alt to space script, but doesn't work

Since I can't use tilde to ping, I have to use alt and put the self-casting in the space script instead as I use it far more often than normal-casting. However before I modified the space script, the space never worked to do normal-casting, so I tried with copy-pasting the self-casting and the space still doesn't function like it should. When casting spells it uses the default quick-casting and when using items with space it does nothing. Any idea of what might cause this?

it probably has something not tied correctly after modifications


Added the remaining CRUD methods to session service

Separate data model for session service, now decoupled from bootstrap data.
Implemented the remaining CRUD methods.
Added docs on how to launch the service, including address & port configuration options.

Please hold this a while. I'll need to add a another quick fix before we merge

ok, from my point of view it's good to go

good to go


RUMM-1804: Use Compose ViewPager instead of Fragment ViewPager

What does this PR do?
This change replaces Fragment-based ViewPager in the Jetpack Compose sample screen to the Compose-based ViewPager to be more closer to the Jetpack Compose.
Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[x] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #778 (9cb1410) into master (449e5ac) will increase coverage by 0.11%.
The diff coverage is n/a.


optimizing scorer -> isMatch

Changes outside of isMatch() got introduced as part of 'grunt prepublish '.

Superceeded by #22 , closing this one


[CT-1039] Snowflake: Configuration of virtuell warehouses for tests

Describe the feature
Extend the configuration of virtuell warehouses to tests.
https://docs.getdbt.com/reference/resource-configs/snowflake-configs#configuring-virtual-warehouses
ATM it's only possible to configure an explicit warehouse to explicit models.
In a scenario with many small models + 2-3 big models (data volume/ runtime) you would probably go to environmental config like this:
environment: PRODUCTION
warehouse: SMALL

virtuell warehouse ( for big models ): XL

Problem: It's not possible to influence the runtime of tests on your big model without changing the standard warehouse, which would potentially lead to a massive cost increase.
Describe alternatives you've considered
It would be possible to write tests as models and to test against these.
Problem: unnecessary models in repo and lineage
Additional context
Please include any other relevant context here.
Who will this benefit?
Cost sensitive customers.
Are you interested in contributing this feature?
Let us know if you want to write some code, and how we can help.

Hi @Kratzbaum92, this sounds similar to issue #23 .
Can you confirm this is indeed the case? Thanks!

Hi @Kratzbaum92, this sounds similar to issue #23 .
Can you confirm this is indeed the case? Thanks!

Yes, It's the same topic.

Thanks for the confirmation! Let's close in favor of that other issue. I just left a comment laying out what I see as the next steps there.


Modify cardLabel for Subscriptions

Needed to shorten the card label for subscriptions as a result of the UI review. There is not much space for that in Quickstart Cards.

/retest

/retest