Adapter-specific installations

Stop updating PyPi package dbt (i.e. pip install dbt)

Remove setup.py for package dbt
Ideally, start raising some kind of deprecation warning?

Remove Homebrew formula + Docker image for dbt
Add Homebrew formulae and Docker images for:

dbt-core
dbt-postgres
dbt-redshift
dbt-snowflake
dbt-bigquery
dbt-spark

Update: pip install dbt shouldn't just stay stuck on v0.21.0 forever. We need to decide what this should do starting with v1.0.0. I think we should raise some kind of error/warning encouraging folks to switch to pip install dbt-<adapter>. In addition to that warning/error, should we:

Install all the same packages as before
Install just dbt-core (likely to raise an Adapter <name> not found! error)
Install nothing (an error in the truest sense)

@jtcohen6 Whatever we do with pip install dbt should probably be reflected in brew install dbt as well yes?

@iknox-fa Thanks for those links!
pip
After spending some more time with this, I'm leaning toward deprecating rather than disabling. I don't think we gain a lot by bricking production pipelines that depend on (unpinned) pip install dbt, on the day that we release v1.0. However, we should include a warning message (during install) to the effect of:
"pip install dbt" is deprecated, and will stop supporting new versions on [date].
Please install your specific database adapter as `pip install dbt-<adapter> instead.
Link to docs: ...

Then, a few months/versions later, we should feel comfortable disabling the dbt package by raising an error. What do you think?
homebrew
Homebrew feels like a different beast. I don't think anyone's production pipelines should depend on brew install dbt (weird if they did!), but certainly many users have already tapped dbt-labs/dbt and installed the dbt formula. I don't have an amazing understanding of the way Homebrew works, but I'm imagining existing users could do something like:
brew unlink dbt
brew link dbt-bigquery

And new users could do:
brew tap dbt-labs/dbt # rename to dbt-core?
brew install dbt-bigquery
# win: no need to install snowflake dependencies

That said, there is definitely a small-but-mighty set of users (our colleagues!) who will want to install and use multiple plugins in the same environment. It would be a shame if they had to run brew unlink ... && brew link ... every time they wanted to switch between dbt-snowflake and dbt-bigquery. Is it possible to install multiple formulae that share the same symlink? Or enable each plugin as an "extra" in the same main formula?
If we need to, we can punt on Homebrew for the time being, and just rename the dbt formula to dbt-core. That kind of renaming appears straightforward and commonly done.

Sounds good to me @jtcohen6.  I'm not sure about homebrew either, but I'll do some experimenting and see what I can sort out.

@iknox-fa That's fine by me; Catalina (10.15) came out >2 years ago. I'm okay deferring to GHA on which MacOS versions we can reasonably support, for now. Homebrew isn't something that folks depend on in production, it's really a quality-of-life improvement to ease installation for Mac users brand new to local development.

There's been further confusion in slack vis-a-vis the actual pip install commands.  @jtcohen6 can you confirm that this is the desired outcome matrix?
Pip (pre-1.0 state)

Install method
codebases used
Databases supported

pip install dbt
dbt2 repo + all adapter repos
Postgres + all adapters

Pip (1.0 + state)

Install method
codebases used
Databases supported

pip install dbt-core
dbt-core3 repo
Postgres

pip install dbt-<adapter-name>1
dbt-core repo3 + one adapter repo
Postgres + <adapter>

pip install dbt-postgres2
dbt-core repo3
Postgres

pip install dbt2
dbt-core3 repo
Postgres

1: One of spark, bigquery, redshift, or snowflake.
2: Not officially blessed installation method, prefer pip install dbt-core.
3: dbt-core and dbt refer to the same repo, it was re-named and the new name used where appropriate.

@iknox-fa Amazing Markdown, thanks for the work laying this out in detail :)
To make a long story short: everything PyPi-related should work the exactly the same as it does today. Yes, we're renaming the repo to dbt-core, and we moved Redshift/Snowflake/BigQuery into separate repos, but we're not changing their PyPi packaging at all.
The big question is, what to do with the PyPi package named dbt, defined here. Note that this is just a cosmetic consideration—it shouldn't change anything about how we're packaging/distributing the underlying components—we just know that a lot of people are still using pip install dbt. Starting in v1, this could:

Raise a deprecation warning, then nstall dbt-core, dbt-postgres, dbt-redshift, dbt-bigquery (backwards compatible). This might be tricky if we've released a newer patch/prerelease of dbt-core than
Warn, then install dbt-core + dbt-postgres (the equivalent of installing -r editable-requirements.txt in the dbt-core repo, i.e. for local development/testing; knowing that most people will encounter errors and have to switch)
Warn, then install dbt-core only (knowing that people will definitely encounter errors)
If all of that is too tricky to figure out, we should keep it simple and just raise a good old fashioned error message when someone tries pip install dbt --pre or pip install dbt==1.0.0b1: "Going forward, you must install dbt-`.

Pip (pre-1.0 state)

Install method
GitHub repo used
Databases supported

pip install dbt-core
dbt repo1 (core/)
none! just the core library

pip install dbt-<adapter-name>
dbt repo1, external <adapter>2?
any <adapter> db

pip install dbt-postgres
dbt repo 1 (plugins/postgres)
Postgres (like above)

pip install dbt
dbt repo1 (core/ + plugins/)
Postgres, Redshift, Snowflake, BigQuery

Pip (1.0 + state)

Install method
GitHub repo used
Databases supported

pip install dbt-core
dbt-core repo (core/)
none! just the core library

pip install dbt-<adapter-name>
dbt-core repo + any one <adapter> repo
any <adapter> db

pip install dbt-postgres3
dbt-core repo (plugins/postgres)
Postgres (like above)

pip install dbt4
dbt-core+ ???
???

1: dbt-core and dbt refer to the same repo, it was re-named and the new name used where appropriate.
2: spark, bigquery, redshift, snowflake... but also sqlserver, athena, trino, etc.
3: This is good/preferred! It's exactly like pip install dbt-<adapter-name>. Postgres is only a special case in that its plugin code will continue living in the dbt-core repo (plugins/postgres), rather than a separate repo.
4: Will display deprecation warning. Prefer pip install dbt-<adapter-name>.

Update:

We've resolved the Homebrew question: formulae will be centralized in homebrew-dbt. There's more work to do there, but we feel 80% good about the progress so far.
Moved the pip install dbt conversation over to #4100
Docker images will be built from wheels, and we're leaning toward GHCR as the registry to host them

I'm going to close this issue for now. Let's pick up on the unfinished threads after we've cut v1.0.0-rc1.


jetson.utils.videoSource Hangs While Trying to Connect to an RTSP Camera

Hi Dusty @dusty-nv
I'm trying to connect to a Milesight camera via RTSP in Python.
Whenever jetson.utils.videoSource(url) is reached, the code gets stuck in a blocking mode manner and doesn't move forward from here and the following outputs are shown :
[gstreamer] initialized gstreamer, version <IP_ADDRESS>
[gstreamer] gstDecoder -- creating decoder for <IP_ADDRESS>
Opening in BLOCKING MODE
NvMMLiteOpen : Block : BlockType = 279
NVMEDIA: Reading vendor.tegra.display-size : status: 6
NvMMLiteBlockCreate : Block : BlockType = 279

This doesn't happen with my other cameras (a Grundig & CSI Cameras) but happens with this specific camera.
When trying the video-viewer app for the first time, then the above outputs are written and the application doesn't work, but if I stop the app with Ctrl+Z and then start it again, the application is able to connect to the camera and show it. Unfortunately this can't happen with the Python code since the jetson.utils.videoSource(url) part gets stuck...
I've already read problems associated with the NvMMLiteBlockCreate : Block : BlockType = 279 output but so far the only options are to recompile the latest version of the project or use a variety of different gstreamer plugins that could or could not solve the problem (which seems to also need another recompile of the project).
Is this a known issue and can it be fixed without having to recompile the project?
Thanks as always Dusty

I recompiled/reinstalled the project and the problem is still there...

Here's another related issue: https://github.com/dusty-nv/jetson-inference/issues/615

Can you try recompiling with ENABLE_NVMM=off?  Like this:
cd /path/to/your/jetson-inference/build
cmake -DENABLE_NVMM=off ../
make
sudo make install

Leaving this comment here so that it may help others: My problem lied in an old Camera Firmware that needed to be updated.


0.5.8 fails to start with panic: sync: negative WaitGroup counter

Describe the bug
The just released version v0.5.8 crashes on startup. I just deployed a fresh clean k3s cluster which pulled the latest version but the kube-vip-ds pods all just crash. If I revert to v0.5.7 everything works fine
To Reproduce
Steps to reproduce the behavior:

Deploy k3s with kube-vip daemonsets
Try to create a load balancer service and see that External IP is stuck
View the pods in kube-system and see all the kube-vip-ds pods crashing
Check the logs and see

panic: sync: negative WaitGroup counter

goroutine 55 [running]:
sync.(*WaitGroup).Add(0x1cf14c8?, 0xc0000c6008?)
        /usr/local/go/src/sync/waitgroup.go:83 +0xda
sync.(*WaitGroup).Done(...)
        /usr/local/go/src/sync/waitgroup.go:108
github.com/kube-vip/kube-vip/pkg/manager.(*Manager).servicesWatcher(0xc000226310, {0x1cf1490?, 0xc000272d00}, 0xc0000ade20)
        /src/pkg/manager/watch_services.go:161 +0x11ef
github.com/kube-vip/kube-vip/pkg/manager.(*Manager).startARP.func3({0x1cf1490, 0xc000272d00})
        /src/pkg/manager/manager_arp.go:156 +0x86
created by k8s.io/client-go/tools/leaderelection.(*LeaderElector).Run
        /go/pkg/mod/k8s.io/client-go@v0.25.2/tools/leaderelection/leaderelection.go:211 +0x11b

Expected behavior
The pods not to crash and load balancing to work
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS/Distro: Ubuntu 22.04
Kubernetes Version: v1.25.5+k3s2
Kube-vip Version: 0.5.8

Kube-vip.yaml:
---
apiVersion: apps/v1
kind: DaemonSet
metadata:
  labels:
    app.kubernetes.io/name: kube-vip-ds
    app.kubernetes.io/version: v0.5.8
  name: kube-vip-ds
  namespace: kube-system
spec:
  selector:
    matchLabels:
      app.kubernetes.io/name: kube-vip-ds
  template:
    metadata:
      labels:
        app.kubernetes.io/name: kube-vip-ds
        app.kubernetes.io/version: v0.5.8
    spec:
      affinity:
        nodeAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            nodeSelectorTerms:
              - matchExpressions:
                  - key: node-role.kubernetes.io/master
                    operator: Exists
              - matchExpressions:
                  - key: node-role.kubernetes.io/control-plane
                    operator: Exists
      containers:
        - args:
            - manager
          env:
            - name: vip_arp
              value: "true"
            - name: port
              value: "6443"
            - name: vip_interface
              value: ens192
            - name: vip_cidr
              value: "32"
            - name: cp_enable
              value: "true"
            - name: cp_namespace
              value: kube-system
            - name: vip_ddns
              value: "false"
            - name: svc_enable
              value: "true"
            - name: vip_leaderelection
              value: "true"
            - name: vip_leaseduration
              value: "5"
            - name: vip_renewdeadline
              value: "3"
            - name: vip_retryperiod
              value: "1"
            - name: address
              value: <IP_ADDRESS>
            - name: prometheus_server
              value: :2112
            - name: enableServicesElection
              value: "true"
          image: ghcr.io/kube-vip/kube-vip:v0.5.8
          imagePullPolicy: Always
          name: kube-vip
          resources: {}
          securityContext:
            capabilities:
              add:
                - NET_ADMIN
                - NET_RAW
      hostNetwork: true
      serviceAccountName: kube-vip
      tolerations:
        - effect: NoSchedule
          operator: Exists
        - effect: NoExecute
          operator: Exists
  updateStrategy: {}
status:
  currentNumberScheduled: 0
  desiredNumberScheduled: 0
  numberMisscheduled: 0
  numberReady: 0

---
apiVersion: v1
kind: ConfigMap
metadata:
  name: kubevip
  namespace: kube-system
data:
  range-global: <IP_ADDRESS>-<IP_ADDRESS>

Additional context
Add any other context about the problem here.

I also got caught out by this error.
What was slightly more problematic is the ports previously assigned were not released after the crash and so the ports remained  even though the VIP was up and broadcasting.
This meant the service was unreachable with a connection was reset by the peer error.  I was forced to manually restart the server that had picked up the VIP to clear the ports and free up the VIP to go to another server.
Although I can understand the VIP not being dependant on a port being free, the fact the port was able to be allocated was still effectively in a failed state, perhaps I'm missing something but if the kube-vip cannot fulfill both the port allocation and the VIP surely it should give up its claim and allow another kube-vip service to inherit the VIP?

Strange, I'll need to improve the release testing as some of the waiting group logic has broken the behaviour of running it as a daemonset :|

Found the bug, that was added in error.

This is fixed in 0.5.9, out of sync wait.Done()


How the Frozen model from tensorflow audio recognition is made and converted to tensorflow lite model?

After following the steps train.py and freeze.py from the tutorial ,the structure of my frozen model looks like(https://imgur.com/a/JtNVkHw) which is different from the official frozen model conv_actions_frozen.pb(https://imgur.com/a/KJXExbV).
When I Converted Frozen model to Tensorflow lite using the steps:
import tensorflow as tf
converter = tf.compat.v1.lite.TFLiteConverter.from_frozen_graph("./my_frozen_graph.pb", input_arrays=['decoded_sample_data', 'decoded_sample_data:1'], output_arrays=['labels_softmax'])
converter.allow_custom_ops=True
tflite_model = converter.convert()
open("output.tflite", "wb").write(tflite_model)

The structure of my tflite model (https://imgur.com/a/uceoHlo) is also different from the original speech commands tflite model(https://imgur.com/a/lWmxl9d). Also, When i am testing my tflite model on android studio, it is getting crashed.
I suspect something went wrong in the creation of frozen model from the tutorial. Could someone kindly help in generating the frozen model for speech commands.
Tensorflow version-2.1.0
python version - 3.7.3

@petewarden Kindly go through the question once

@vihari1729 Can you please provide a standalone code to reproduce the issue? Thanks!

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


checkworthy features processing added

Creates dataset where each row is topic-claim and has the following columns:

topic
value
claim
number of origins
avg. degree of origins
max degree of origins
avg centrality of origins
max centrality of origins
for every time step aggregation:
- number of nodes visited
- avg degree of nodes visited
- max degree of nodes visited
- avg centrality of nodes visited
- max centrality of nodes visited
- max distance traveled from origin node

Note: betweenness centrality calculation takes a long time. consider running the create_simulation_network separately and store the resulting network. Then you can run the simulation without having to calc the centrality of the network nodes each time.

i'm going to merge and try and run this


Why FundSiafunds requires a siacoin input of exactly 'amount'? I didn't see the necessity.

As told in the codebase:
// FundSiacoins will add a siacoin input of exactly 'amount' to the
// transaction. A parent transaction may be needed to achieve an input with the
// correct value. The siacoin input will not be signed until 'Sign' is called
// on the transaction builder.

I am working on build transactions locally, this part is not so intuitive for me.  Especially involving with building A parent transaction .Perhaps anyone can explain the philosophy under it.

Thanks.   I want to make it more precisely with the mining fee:
Input:     15.1 SC(from the 15.1 SC output)
Output1:     10 SC(sent to the destination address)
Output2:      5 SC(sent back to your own wallet)
MinerFee:   0.1 SC(sent to the miner)

Your illustration is very clear. But creating two transactions instead of one would probably cost more miner fees, right?  So Can I consider your implementation is optimizer other than necessity?

Indeed, creating a "parent" transaction will result in more miner fees. If you are constructing your transaction manually, and you don't care about putting all of the inputs and outputs in one transaction, then it is cheaper to do that.


Implement client.role.get()

Implement client.role.get() (see GatewayController.get() for an example)
GET /api/v1/role/1 HTTP/1.1
Host: <IP_ADDRESS>:8080
Accept-Encoding: gzip, deflate
...

HTTP/1.1 200 OK
Access-Control-Allow-Origin: *
Content-Length: 147
Content-Type: application/json
Date: Mon, 18 May 2020 17:53:49 GMT
Server: BlueData EPIC 5.0

{
  "_links": {
    "self": {
      "href": "/api/v1/role/1"
    },
    "all_roles": {
      "href": "/api/v1/role"
    }
  },
  "label": {
    "name": "Site Admin",
    "description": "Role for Site Admin"
  }
}

I've created a mock rest api server to test this.  You can start it with:
 /opt/SoapUI-5.5.0/bin/mockservicerunner.sh -m "REST MockService" tests/HPECP-REST-API-SOAPUI.xml

You can then install the CLI with:
pip3 install -e .

After that, configure the cli:
cat > ~/.hpecp.conf <<EOF
[default]
api_host = localhost
api_port = 8080
use_ssl = False
verify_ssl = False
warn_ssl = False
username = admin
password = admin123
EOF

and finally, run the cli:
hpecp role get /api/v1/role/1

This should make the call to the mock rest service.  It will fail initially because you need to implement the logic! :). The mock service will return json for role 1.
Likewise, you can simulate the api returning a non-existing role with:
hpecp role get /api/v1/role/99

The mock server will return a HTTP 404 error for this request.
Using the mock rest service is documented in the developer docs


OCR support for single articles

First of all, the script works very well, thanks for that.
Is it possible to read single articles per ocr from the receipt ?
This would be very nice!

Hey there,
parsing single articles might be tricky because from my tests OCR is quite error-prone. That's because the receipt scans might not be of the highest quality (old receipts, low resolution...) and the article names are usually not well-known words that can get recognized easily by OCR.
That said, it would be great to run some more tests and see if those limitations are still the case. I personally don't have time to do that, but if you or anybody wants to give it a shot then I'd be happy to accept PRs or discuss our options.

Thanks for your fast answer.
I tried a few things with tesseract and found someting intresting.
Using the --psm 4 flag, gives good results.
 O0
ertragsnummer<PHONE_NUMBER>
ao? AD000000031010

In my opinion, this could work well.
But there should be a possibility for a manual check of the data inside the app.

That looks quite promising!
Can you try -l deu as well? Wonder if we can also detect words like Transakticens-Nr or Kartenfo] denummer with that or some similar tweaking.

This is fantastic!. I did do similar work using python pytesseract on very noise images. Good on you mate.
Seyhan

So, I played around a bit and got a useful result:
Added a new function in parse.py:
    def parse_lineitems(self):
        item = namedtuple("item", ("article", "sum"))
        items = []

        for line in self.lines:

            reSearch = re.search(r"(...+)\s(-|)(\d,\d\d)\s", line)

            if hasattr(reSearch, 'group'):
                items.append(item(reSearch.group(1), reSearch.group(3).replace(",",".")))

        for i in items:
            print ("Artikel: " + i.article + " Summe: " + i.sum)

This works pretty well:
Artikel: 2155 thunfisch in wasse Summe: 1.19
Artikel: 2155 thunfisch in wasse Summe: 1.19
Artikel: 46002 bio rinderhackfl. Summe: 3.59
Artikel: 46002 bio rinderhackfl. Summe: 3.59
Artikel: 49036 bio-reibekäse Summe: 1.59
Artikel: 46932 bio-käsegenuss Summe: 1.49
Artikel: 44011 hähnshenbkrustfilet Summe: 2.99
Artikel: 49036 bio-reibekäse Summe: 1.59
Artikel: 49036 bio-reibekäse Summe: 1.59
Artikel: 4659 bio-semüse-sort.  Summe: 1.99

Unfortunately I'm not quite sure how the communication between server and parser works to build the function into the server. Can anyone help me ?

I have found out that the server does not talk to this parser yet, but uses a pypi package. Should this be changed ?

Great progress!
Can you try and test your code on the test receipts as well?
E.g. https://github.com/ReceiptManager/Parser/blob/master/data/img/IMG0003.jpg, which does not have numbers in front of the article names. Might need some tweaking of the regex maybe.

I have found out that the server does not talk to this parser yet, but uses a pypi package. Should this be changed ?

Yes, @monolidth and me talked about it and it makes sense to use the parser code in the server. Help welcome.
I'm thinking we could use the same pypi package name that the server is using at the moment and push the package from this Parser repo and then fix the client to make use of the new package. Help welcome.

Looks like @monolidth is already on it, improving the code over at dev. 😀

Output from your IMG0003 sample looks like this:
Artikel: saftorangen Summe: 1.49
Artikel: banane golden v. Summe: 1.19
Artikel: 0,410 kgx 1,99 eur/kg Summe: 0.82
Artikel: pflaume 7509 Summe: 0.99
Artikel: orto mio oliven Summe: 0.69
Artikel: hautklar 3ini Summe: 5.98
Artikel: 2stkx Summe: 2.99
Artikel: today nachfüllbe Summe: 0.65
Artikel: spül-/hh- tücher Summe: 0.75
Artikel: leergut einweg Summe: 2.00
Artikel: brstkkex: Summe: 0.20
Artikel: rückgeld bar eur Summe: 9.44
Artikel: a= 19,0% 4,52 0,86 Summe: 5.38
Artikel: b= 7,0% 4,84 0,34 Summe: 5.18
Artikel: gesantbetrag 9,36 Summe: 1.20

Looks good, but can be improved

Small improvements implemented, changed sharpnes to 0.8  and stopping at "rückgeld" or "summe" looks like this now:
Item: sap orange Total: 1.49
Item: golden banana v. Total: 1.19
Item: 0.410 kgx 1.99 eur/kg Total: 0.82
Item: plum 7509 Total: 0.99
Item: orto mio olives Total: 0.69
Item: clear 3ini Sum: 5.98
Item: 2pcsx Total: 2.99
Item: today refill Total: 0.65
Item: washing-up/high wipes Total: 0.75
Item: disposable empties Total: 2.00
Item: brstkkex: Total: 0.20

Unfortunately I am not a good python developer, but I am happy to help if I can.

@Dielee
I removed sensitive image information. The original image contained exif gps coordinates.

See commit: ad78ad03febef0447fd66f0e636e85720c7a47c1.

@monolidth yes, thank you very much.
Commit looks good! We should stop parsing if we find something like sum_keys in config.yaml.
Dont know how to do this...

Build something ugly
    def parse_items(self):
        item = namedtuple("item", ("article", "sum"))
        articleName = ""
        parseStop = False
        items = []

        config = open("config.yml")
        parsed_config = yaml.load(config, Loader=yaml.FullLoader)
        stopWords = parsed_config.get("sum_keys")

        for line in self.lines:

            match = re.search(r"(...+)\s(-|)(\d,\d\d)\s", line)
            if hasattr(match, 'group'):
                articleName = match.group(1)

            for stop in stopWords:
                parseStop = fnmatch.fnmatch(articleName, f"{stop}*")
                if parseStop == True:
                    return items

            if hasattr(match, 'group') and parseStop != True:
                items.append(item(match.group(1), match.group(3).replace(",", ".")))

        return items

This works pretty well

Yeah, agree. I was tired yesterday night, but this is a feature which is required.
Thanks for sharing, I will take a look at this as soon as possible.
Regards,
William

Very nice, thanks a lot!

Some friendly notes:
Use the sneak case naming convention. Additionally, you can simplify statements like
  if parse_stop == True:
                    return items

to:
  if parse_stop:
                    return items

Regards,
William

Thanks, such tips are always good. These are my first lines of python code.

Looks good, thank you 👍


Ad block no longer working

Select your issue type: (check at least one)

[x] Bug
[ ] Question
[ ] Suggestion
[ ] Other (please describe):

Describe your issue:
When using the ad blocking feature of this extension, i get a message saying that I need to disable ad blocking, but I don't want to if I don't have to.
https://imgur.com/a/3Z4IN link to the message I am talking about.

I got the same problem, I think it has to do with the 1.3.0 additions to adblocking. Using strictly uBlock Origin to block the ads doesn't trigger the anti-adblock from 9a-s side

oh wow. Thanks for the report. I will try to find out whats up.

@SimonMattila @SheepKid12 did this start today or 2-3 days back? Also for some reason I seem to be getting that message with uBlock Origin on (and 9anime Companion remove ads off)

Tested version 1.2.0, results are the same. The only way to remove the disable adblock warning is to completely disable 9anime Companion and then using ad blocker of your choice. Can anyone please confirm this? Previous versions of the extension can be found here: https://github.com/lap00zza/9anime-Companion/releases

I'm getting the same "Please disable AdBlock then refresh this page and try again." error message.
I have tried disabling "remove ads" in 9AC with ublock running but still receive the error message.
I have tried disabling ublock with 9AC running with "remove ads" enabled and still receive the message.
have tried disabling ublock with 9AC unning with "remove ads" disabled and videos will play.
I have tried disabling both ublock and 9AC completely and videos will play.
Hope this helps.

 message
But then if you have any of those extension on and you wait the video page to load until it finishes, when you press the play button, you will see the "Please disable AdBlock then refresh this page and try again."
I assume 9anime detect this until the page is fully loaded

Please try the builds here and let me know if the adblock message still shows up: https://ci.appveyor.com/project/lap00zza/9anime-companion/branch/master/artifacts
How to install: https://github.com/lap00zza/9anime-Companion/wiki/Running-in-Developement-Mode

Working in Firefox Quantum 57.
There is an ad still showing in the video player though.

Build worked for me in Waterfox 55.2.2 with Ublock Origin enabled.

@Wyn10 please try once with 9ac remove ads on and Ublock Origin disabled and see if the ads are still there.

Once build <IP_ADDRESS> is fully tested, I will release an emergency fix to Chrome Web Store and Addons.Mozilla.Org.

Version <IP_ADDRESS> tested and working perfectly.

well they seem to put up some fight , this is the reason why you are getting those errors https://www.reddit.com/r/9anime/comments/7el3p7/9anime_changes_adblock_blocker_banner_ads_etc/ , and I think are changing the adblock killer everyday...

Saw that post. It takes quite a bit of effort to find a way to disable the anti adblocker. If they do it everyday it will be really hard for me to keep up.

but a small trick helped me you could try that in your javascript , the trick is:

Use UBlock Origin ( That kills all ads from 9anime ) or any other adblocker in general ( yours)
The Ad Killer is in the autoplay feature , you just need to rebuild that autoplay
I created a small script to open the next episode url for autoplay

Manually
You just need to copy the next episode url and then paste it in the address bar and press enter.
Simple , right.. ?

hope this helps!

Yup that works. That was the first thing I tried today. I was planning to stick with that at first, but I really wanted to find out what the detection was 😄 (personal curiosity). If they every change it again I will just do what you mentioned.

Added the following banner on the settings page which will be removed in the next update.

This issue is fixed in release 1.3.1 (https://github.com/lap00zza/9anime-Companion/releases/tag/1.3.1). Changes will be made available in Chrome Web Store and Addons.Mozilla.Org in 1 hour.

Changes are live. Please update to 1.3.1.


Rework transactions to prevent double booking without paying

We currently have a gut, where when two transactions are requested nearly simultaneously, our balance checks fail, because they are not in the same transaction as the update of the balance.

I would suggest optimistic locking:
https://vladmihalcea.com/optimistic-vs-pessimistic-locking/
https://typeorm.io/#/select-query-builder/set-locking

Let's check if we have to use QueryBuilder for all queries if we want to use the TypeOrm version of occ.
We could also extend the repositories and add a version on our own to get the same result.
But lets talk about this in person :+1:


Additional Option in Exsting standard Option Field in NAV

Hi Team ,
Can we add additional option in any standard field in AL environment.
For Eg: in Sales Price table , can we add "Dim Code" as new option  in "Sales Type" and write program accordingly in new codeunit?

Hi @shalu712 ! Please see https://github.com/Microsoft/AL/issues/590 .


Added feature to display the articles for the trends shown below the carousel

Added an id for the 3 trending topics listed below the carousel and applied a function for all 3 queries. The function makes an api call that displays the articles that won't be shown in the carousel (articles that are not in the first 5 returned by the api call).

article trends showcases topics and bulma classes are implemented in the html. Looks good.


Cannot open this link https://oblador.github.io/react-native-vector-icons/

[ ] Review the documentation: https://github.com/oblador/react-native-vector-icons
[ ] Search for existing issues (including closed issues): https://github.com/oblador/react-native-vector-icons/issues

Environment

Description
Describe your issue in detail. Include screenshots if needed.
Not able to open the website i am getting this issue react-dom.production.min.js:216 RangeError: Invalid code point NaN

Reproducible Demo
Let us know how to reproduce the issue. Include a code sample, share a project, or share an app that reproduces the issue using https://snack.expo.io/. Please follow the guidelines for providing a MCVE: https://stackoverflow.com/help/mcve

Yes since yesterday

Yes still cant open link

For the meantime, the archive.org link from march 24 works fine.
https://web.archive.org/web/20230324180526/https://oblador.github.io/react-native-vector-icons/

Yes or this one from expo is also fine https://icons.expo.fyi/Index

related:
https://github.com/oblador/react-native-vector-icons/issues/1521

App.js:10 Uncaught RangeError: Invalid code point NaN
at Function.fromCodePoint ()

https://github.com/oblador/react-native-vector-icons/issues/1524#issuecomment-1640266134

Duplicate of #1521


REST API Plugin

Closes #344

Added a REST API plugin

Has an optional --host argument
Has an optional --port argument

Added a basic, first route that lists all of the files for definitions in the active context
Adjusted the parse function to allow overriding the source_uri of definitions
Added a test for the files endpoint

I've had to add some duct tape to address shortcomings how plugins provide info about themselves and how the parser is structured. I'll create a story to address those shortcomings before putting this PR into review.


Backport 2.7.4: Purify tooltip content with v-clean-tooltip directive

Summary
This replaces the use of the v-tooltip directive with a custom v-clean-tooltip directive that uses the DOMPurify library to sanitize HTML input before rendering. html usage is discouraged by v-tooltip^1, and should only be used on trusted content. Similar to #8466, we sanitize existing usages across the board throughout Rancher Dashboard.
Occurred changes and/or fixed issues

Adds new v-clean-tooltip directive that sanitizes tooltip content based on rules defined in shell/plugins/clean-html-directive.js
Replaces all existing instances of v-tooltip with v-clean-tooltip

Technical notes summary

In essence, we are monkey patching the bind method that v-tooltip utilizes to configure the directive. By taking this approach, we can ensure that we are able to sanitize tooltip content without altering the original behavior of the v-tooltip directive.
Areas or cases that should be tested

We need to investigate tooltips throughout Rancher Dashboard and ensure that there are no notable regressions after this change has been applied.
backports #8919 into release-2.7.4

Skipping e2e tests. Tests already passed against master but are failing with an error unrelated to this change
Opening Cypress...
[6998:0519/182247.510296:ERROR:sandbox_linux.cc(377)] InitializeSandbox() called with multiple threads in process gpu-process.
[6998:0519/182247.523652:ERROR:gpu_memory_buffer_support_x11.cc(44)] dri3 extension not supported.
We encountered an unexpected error talking to our servers.
We will retry 3 more times in 30 seconds...
The server's response was:
RequestError: Error: connect ECONNREFUSED <IP_ADDRESS>:1234
We encountered an unexpected error talking to our servers.
We will retry 2 more times in 1 minute...
The server's response was:
RequestError: Error: connect ECONNREFUSED <IP_ADDRESS>:1234
We encountered an unexpected error talking to our servers.
We will retry 1 more time in 2 minutes...
The server's response was:
RequestError: Error: connect ECONNREFUSED <IP_ADDRESS>:1234
We encountered an unexpected error talking to our servers.
We will retry 0 more times in ...
The server's response was:
RequestError: Error: connect ECONNREFUSED <IP_ADDRESS>:1234
We encountered an unexpected error talking to our servers.
Because you passed the --parallel flag, this run cannot proceed because it requires a valid response from our servers.
The --ciBuildId flag you passed was: backport-2.6:-purify-tooltip-content-with-`v-clean-tooltip`-directive_3989_id_5026918712
The server's response was:
RequestError: Error: connect ECONNREFUSED <IP_ADDRESS>:1234


Update swanky check

Checkbox 4 from the original issue #114 is not fully completed:

fetch and store the dependency table on init and check use

We should update node.supportedInk version in swanky config when performing the checkCliDependencies task in the init command. You can make and helper performing the semver.gt against our cargo-contract dependencies map like you did in "Verify ink version" from the check command. Then update the node.supportedInk version in swanky config accordingly.

Sorry, I don't understand properly what you mean by updating node.supportedInk. I thought that it should be set initially in nodeInfo for every SwankyNode version. Can you explain what factors affect the update of the node.supportedInk?

why was it closed instead of merged? @ipapandinas

I created a single PR #205 to address the overlap between features and to prepare for the upcoming release. I closed this particular one due to significant conflicts. @pmikolajczyk41

@ipapandinas okay, I was just confused that the other 3 PRs were merged, so I wanted to ensure - thanks!


not possible to import from a file?

I can see the function -import is not a part of this library. Is there any intention of adding that influx command?

I think it's easy to make a small wrap that read from file then insert them by calling existing function, and it's not necessary to add an addition function.


High CPU usage in iphone5

Thanks for your kindly sharing, it really saves me hours; however, I found that the library runs with very high CPU usage under iphone 5 models. The phone will become very hot after some minutes with 140% CPU usage. It there any way to improve this?  By the way, I can not get the webrtc codes compiled by myself. Is there any guides to correctly build webrtc in mac?

I'm in the same boat as you, i have tried several guides and they all seem to not build correctly.
Apparently the webrtc code runs hot on lower end devices, you should check out the google code forum about it.

Originally, I doubt the high fps (up to 44 or higher) can be the reason. Finally I feel disappointed when I try to lower the frame rate using the codes below. The CPU usage still goes to 120% or even higher!

(RTCMediaConstraints *)defaultMediaStreamConstraints {
NSArray mandatoryConstraints = @[
[[RTCPair alloc] initWithKey:@"maxHeight" value:[NSString stringWithFormat:@"%d",self.maxHeight]],
[[RTCPair alloc] initWithKey:@"maxWidth" value:[NSString stringWithFormat:@"%d",self.maxWidth]],
[[RTCPair alloc] initWithKey:@"maxFrameRate" value:[NSString stringWithFormat:@"%d",self.maxFrameRate]],
[[RTCPair alloc] initWithKey:@"minFrameRate" value:[NSString stringWithFormat:@"%d",self.minFrameRate]]
];
RTCMediaConstraints theConstraints =
[[RTCMediaConstraints alloc]
initWithMandatoryConstraints:mandatoryConstraints
optionalConstraints:@[@"frameRate":@"15"]];
return theConstraints;
}

However, there is some evidence that the renderer can be the cause due to the following site:
https://perch.co/blog/perchrtc-custom-webrtc-rendering/
I would give a try if there is any progress and share with you guys.

i have decided to use opentok, their library works perfect for webrtc, but you have to go through them.

pochuang,
You mean that the webRtc lib ignores initialization of  mandatoryConstraints ?

Finally, I found that the AVCaptureSession could be the reason why my iphone is burning! Before this adjustment, the cpu usage is always 120% or higher, but after this it drops down to 70%! This is a very huge change although it's not a very great solution. The magic is to set sessionPreset to low like this:

(RTCMediaConstraints *)defaultMediaStreamConstraints {
static RTCMediaConstraints *defaultMediaStreamConstraints;
static dispatch_once_t onceToken;
dispatch_once(&onceToken, ^{
NSArray *mandatoryConstraints = @[
[[RTCPair alloc] initWithKey:@"maxHeight" value:[NSString stringWithFormat:@"%d",self.maxHeight]],
[[RTCPair alloc] initWithKey:@"maxWidth" value:[NSString stringWithFormat:@"%d",self.maxWidth]],
[[RTCPair alloc] initWithKey:@"minWidth" value:[NSString stringWithFormat:@"%d",self.minWidth]],
[[RTCPair alloc] initWithKey:@"minHeight" value:[NSString stringWithFormat:@"%d",self.minWidth]],
[[RTCPair alloc] initWithKey:@"maxFrameRate" value:[NSString stringWithFormat:@"%d",self.maxFrameRate]],
[[RTCPair alloc] initWithKey:@"minFrameRate" value:[NSString stringWithFormat:@"%d",self.minFrameRate]]
];
  NSArray *optionalConstraints = @[];

  defaultMediaStreamConstraints =
  [[RTCMediaConstraints alloc]
   initWithMandatoryConstraints:mandatoryConstraints
   optionalConstraints:optionalConstraints];

});
return defaultMediaStreamConstraints;
}

#if !TARGET_IPHONE_SIMULATOR && TARGET_OS_IPHONE
RTCMediaConstraints *mediaConstraints = [self defaultMediaStreamConstraints];
RTCAVFoundationVideoSource *source =
[[RTCAVFoundationVideoSource alloc] initWithFactory:_factory
constraints:mediaConstraints];
if ([source.captureSession canSetSessionPreset:AVCaptureSessionPresetLow]) {
source.captureSession.sessionPreset = AVCaptureSessionPresetLow;
}
#endif
I am still working on this for a better solution. If you have any idea here, please let me know.

Please make a pull request for more details, what are values your self.maxHeight, self.minWidth etc..? The values is important and affect on a performance of CPU.
And does it make sense to use a "dispatch_once" ?  I would like to contribute towards it,  because I look into this problem.

The accepted values are listed below:
#define DEFAULT_MAX_HEIGHT 960
#define DEFAULT_MAX_WIDTH 720
#define DEFAULT_MIN_FRAME_RATE 5
#define DEFAULT_MAX_FRAME_RATE 12
#define ENABLE_BOOST_PERFORMANCE YES
The minWidth and minHeight do not work,  so I modified it like this:

(RTCMediaConstraints *)defaultMediaStreamConstraints {
NSArray *mandatoryConstraints = @[
[[RTCPair alloc] initWithKey:@"maxHeight" value:[NSString stringWithFormat:@"%d",self.maxHeight]],
[[RTCPair alloc] initWithKey:@"maxWidth" value:[NSString stringWithFormat:@"%d",self.maxWidth]],
[[RTCPair alloc] initWithKey:@"maxFrameRate" value:[NSString stringWithFormat:@"%d",self.maxFrameRate]],
[[RTCPair alloc] initWithKey:@"minFrameRate" value:[NSString stringWithFormat:@"%d",self.minFrameRate]]
];
NSArray *optionalConstraints = @[];
RTCMediaConstraints* theConstraints =
[[RTCMediaConstraints alloc]
initWithMandatoryConstraints:mandatoryConstraints
optionalConstraints:optionalConstraints];
return theConstraints;
}

And the sessionPreset part like this:
#if !TARGET_IPHONE_SIMULATOR && TARGET_OS_IPHONE
RTCMediaConstraints *mediaConstraints = [self defaultMediaStreamConstraints];
RTCAVFoundationVideoSource *source =
[[RTCAVFoundationVideoSource alloc] initWithFactory:_factory
constraints:mediaConstraints];
if(ENABLE_BOOST_PERFORMANCE){
    dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{
        if ([source.captureSession canSetSessionPreset:AVCaptureSessionPresetLow]) {
            [source.captureSession stopRunning];
            source.captureSession.sessionPreset = AVCaptureSessionPresetLow;
            [source.captureSession startRunning];
        }
    });
}
videoTrack =
[[RTCVideoTrack alloc] initWithFactory:_factory
                                source:source
                               trackId:@"ARDAMSv0"];

#endif
WebRTC will change sessionPreset due to your device's resolution at first. That will make your video more clear but become heavy to CPU. You may take a look at the source code downloaded from google.
The video resolution and frame rate you received from others will also affect CPU in iPhone. For me, I reduce it to 640x480.
Now the CPU usage is down to 58% or a little higher than 60%. Not pretty satisfying compared to Android's (the CPU usage is 30% only!) but is workable for now.
By the way, It will take a long time to change sessionPresent when your captureSession is already running. For my situation, I need to stop it temporarily and start it latter after making changes.

hi  all.
how can you fixup the video card problem?

is this problem fixed in the newest version? Or do we still need to make changes manually?


Fixed actorsystem termination timeout exception

+1 pending resolving @dmitri-carpov 's comment about runtime halt.


Simplify anchor metadata

Follow-up to https://github.com/godotengine/godot/pull/59185#pullrequestreview-951262422 with some extra improvements.

Thanks!


download stop at 32%

PS C:\Python27> python spotdl.py --song "adele hello"
Adele - Hello
Song with same name has already been downloaded. Re-download? (y/n): y
  1,908,966 Bytes [32.76%] received. Rate: [  21 KB/s].  ETA: [181 secs]
Converting Adele_-_Hello.m4a to mp3
Fixing meta-tags
PS C:\Python27> python spotdl.py --song "inna yalla"
inna yalla
Song with same name has already been downloaded. Re-download? (y/n): y
  1,698,814 Bytes [32.60%] received. Rate: [  22 KB/s].  ETA: [159 secs]
Converting inna_yalla.m4a to mp3
Fixing meta-tags
PS C:\Python27>

[x ] Using latest version as provided on the master branch
[ x] Searched for similar issues including closed ones

What is the purpose of your issue?

[ ] Script won't run
[ x] Encountered bug
[ ] Feature request
[ ] Question
[ ] Other

System information

Your python version: python 3.6.1
Your operating system: windows 10

Description

This is most likely a bug in pafy. Can you try running the below snippet:
import pafy

video = pafy.new('https://www.youtube.com/watch?v=ZW7HYGb_gG8')

best = video.getbestaudio(preftype='m4a')
best.download()

best = video.getbestaudio(preftype='webm')
best.download()

It should download 2 songs. Let me know if they download successfully or fail just like before.

downloading worked today!!! I suppose it was something with my internet connection Thank you,


Review/Improve Visual Editor Heading when Expanded

The "Visual Editor" heading text in the editor pane feels a little redundant now, and whilst it's only around 25px high, it's some space we can potentially re-claim.
Current thoughts:

Remove heading when expanded; keep it when collapsed
Move the 'collapse' button onto the command bar, next to the help (?) icon

Agreed. More space back would be useful.


Preview HTML of   R Markdown doesn't show embedded plot

Preview is part of Web Essentials, not RTVS. https://github.com/madskristensen/WebEssentials2015

I got it. Sorry for my misunderstanding...


Missing test contracts in 2_diamonds.js

In 2_diamonds.js, const diamondCut part, the test contracts seem to be missing. I would have expected to see the following two elements in the diamondCut array:
[Test1Facet.address, FacetCutAction.Add, getSelectors(Test1Facet)],
[Test2Facet.address, FacetCutAction.Add, getSelectors(Test2Facet)]

Yes, the test contracts could be added that way.   In this case the test contracts are added in this this test script: https://github.com/mudgen/diamond-1/blob/master/test/diamondTest.js#L121

Yes, the test contracts could be added that way.   In this case the test contracts are added in this this test script: https://github.com/mudgen/diamond-1/blob/master/test/diamondTest.js#L121


chore: bump version 0.4.1

Versioned docs, and correct version on readme for artifact hub and OLM

/approve

/merge


Ensure daemon file's parent directory exists

On systems where LaunchAgents directory does not exist in ~/Library/ (that's what I had on my macOS 12.6), the create_daemon (and thus maintain) command fails. This PR fixes that by making sure that the parent directory of the daemon_path file exist.

Very nice, thanks for the addition (and style upgrades)!


[Datastore] saving error: Illegal value for [object Object] of type double: string (not a number)

I updated the gloud up to 0.36.0 version (preveous 0.33.0) and currently I cannot save the data to the datastore using by gcloud.datastore.save(entities, callback). When I tried to save the entities
[
	{
		"key": {
			"name": "120",
			"kind": "CD_Campaign",
			"path": [
				"CD_Campaign",
				"120"
			]
		},
		"data": {
			"persons": 20,
			"attempted": 0,
			"contacted": 0,
			"bounds": {
				"min": {
					"lng": -122.50355,
					"lat": 37.70851
				},
				"max": {
					"lng": -122.43805,
					"lat": 37.7907
				}
			},
			"disp": {
				"NH": 0,
				"LD": 0,
				"VC": 0,
				"LB": 0,
				"D": 0,
				"NC": 0,
				"IN": 0,
				"DC": 0,
				"RF": 0,
				"MV": 0,
				"WN": 0,
				"AV": 0,
				"V": 0,
				"B": 0,
				"LM": 0,
				"CB": 0
			},
			"areaIds": [
				"0500000US06075"
			]
		}
	}
]

I got the error:
> Error: Illegal value for [object Object] of type double: string (not a number)
>     at Error (native)
>     at null.<anonymous> (C:\Data\Projects\dashboard\node_modules\protobufjs\dist\ProtoBuf.js:1769:23)
>     at ProtoBuf.Reflect.ElementPrototype.verifyValue (C:\Data\Projects\dashboard\node_modules\protobufjs\dist\ProtoBuf.js:1825:25)
>     at ProtoBuf.Reflect.FieldPrototype.verifyValue (C:\Data\Projects\dashboard\node_modules\protobufjs\dist\ProtoBuf.js:3436:33)
>     at MessagePrototype.set (C:\Data\Projects\dashboard\node_modules\protobufjs\dist\ProtoBuf.js:2443:59)
>     at MessagePrototype.set (C:\Data\Projects\dashboard\node_modules\protobufjs\dist\ProtoBuf.js:2434:38)
>     at Message (C:\Data\Projects\dashboard\node_modules\protobufjs\dist\ProtoBuf.js:2363:34)
>     at ProtoBuf.Reflect.ElementPrototype.verifyValue (C:\Data\Projects\dashboard\node_modules\protobufjs\dist\ProtoBuf.js:1877:28)
>     at new ProtoBuf.Map.Map (C:\Data\Projects\dashboard\node_modules\protobufjs\dist\ProtoBuf.js:4894:46)
>     at ProtoBuf.Reflect.FieldPrototype.verifyValue (C:\Data\Projects\dashboard\node_modules\protobufjs\dist\ProtoBuf.js:3427:28)

Maybe problem in ProtoBuf.js or in another methods which are sending float value as string into ProtoBuf.js. As a hot fix I changed the code in ProtoBuf.js (line 1825):
                // Float
                case ProtoBuf.TYPES["float"]:
                case ProtoBuf.TYPES["double"]:
/*                    if (typeof value !== 'number')
                       fail(typeof value, "not a number");
                    return value;*/

                    // hot fix
                    if (typeof value !== 'number') {
                        if (typeof value === 'string') {
                            var asFloat=parseFloat(value);
                            if (isNaN(asFloat)) {
                                fail(typeof value, "not a number");
                            } else {
                                return asFloat;
                            }
                        } else {
                            fail(typeof value, "not a number");
                        }
                    }
                    return value;

I wasn't able to reproduce this error. Would you create a bare-minimum repo that reproduces this bug consistently?

I reproduced this bug when I used the "stephenplusplus/gcloud-node#spp--1412" version. The bare-minimum repo is here:
$ git clone https://bitbucket.org/gromtech/gcloud-node-1421.git

Okay, that explains it. I'll try to find a fix.

Going to link to this issue in the PR, since this isn't actually broken on master.


FST Lisboa dataset submission

Greetings,
First off, we'd like to thank you for this great initiative.
The link to our shared album has been sent via email.
Thank you

Hi Francisco, I have received your e-mail and will be taking a look at the PR tonight and tomorrow morning.
Thank you for the initiated contribution.

Hi Francisco, everything's looking good with the labels, one of the best quality ones so far. Nice job!
Could you please add your information to the Documentation Markdown file? Basically updating the table and adding your entry to the contributors.
As soon as that's done we can merge.

Hello David, that's great to hear! We had a proper labeling party! I've added our entry to the table as per your request.
Once again, thank you for your work and it was our pleasure to contribute.
FST Lisboa


Consider implementing P2447

Please consider implementing P2447 (std::span and the missing constructor), possibly as an opt-in optional feature since it hasn't been accepted into the C++ standard yet. Thanks!

Thanks for your suggestion, Davide. Will look into it.

@martinmoene thanks!
I'm playing a bit with the new constructor (as of commit 052dad8b613aef2a14517f2899b1b438eb5decc7) and I'm getting a strange error with GCC 11.2 (-std=c++14)
.../detail/nonstd/span-lite.hpp: In instantiation of ‘constexpr nonstd::span_lite::span<T, Extent>::span(std::initializer_list<typename std::remove_cv<_Functor>::type>) [with long unsigned int U =<PHONE_NUMBER>3709551615; typename std::enable_if<(U == nonstd::span_lite::dynamic_extent), int>::type <anonymous> = 0; T = const unsigned char; long unsigned int Extent =<PHONE_NUMBER>3709551615; typename std::remove_cv<_Functor>::type = unsigned char]’:
.../detail/nonstd/span-lite.hpp:1088:26: error: initializing ‘nonstd::span_lite::span<const unsigned char>::data_’ from ‘std::initializer_list<unsigned char>::begin’ does not extend the lifetime of the underlying array [-Werror=init-list-lifetime]
 1088 |         : data_( il.begin() )
      |                  ~~~~~~~~^~

My code looks like this:
const Foo& getFoo()
{
  static const Foo foo({'b', 'a', 'r'});
  return foo;
}

And class Foo has a constructor that takes a span<const uint8_t> and makes a copy of its content.
Curiously, make_span doesn't trigger the error (but I have to specify the type):
const Foo& getFoo()
{
  static const Foo foo(make_span<uint8_t>({'b', 'a', 'r'}));
  return foo;
}

Thanks, my quick look is not yet enough to solve this :)

Here's the example in Compiler Explorer.
Clang does not issue the warning '...’ does not extend the lifetime of the underlying array [-Winit-list-lifetime].
I'm getting the impression that GCC is trying to be a bit too helpful here.
Do you think suppressing this warning in the relevant location would be a reasonable thing to do?

I'm getting the impression that GCC is trying to be a bit too helpful here.

Indeed, it's definitely a false positive (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102482)

Do you think suppressing this warning in the relevant location would be a reasonable thing to do?

Yeah, either suppress the warning (also suggested in comment 5 of the GCC bug) or move the assignment into the body of the constructor (suggested in comment 4)

@Pesa Thanks for your very specific help!

Thank you for quickly implementing this extension!
I'm testing commit d9a35a7ff0042049e47d535af7108045baae0be0 and it seems to work as expected. The only small problem is that the #pragma throws a warning on GCC 8 and older.
.../nonstd/span-lite.hpp:1073:33: warning: unknown option after ‘#pragma GCC diagnostic’ kind [-Wpragmas]
 # pragma GCC diagnostic ignored "-Winit-list-lifetime"
                                 ^~~~~~~~~~~~~~~~~~~~~~
.../nonstd/span-lite.hpp:1092:33: warning: unknown option after ‘#pragma GCC diagnostic’ kind [-Wpragmas]
 # pragma GCC diagnostic ignored "-Winit-list-lifetime"
                                 ^~~~~~~~~~~~~~~~~~~~~~

According to https://gcc.gnu.org/gcc-9/changes.html, the warning was introduced in GCC 9.

You're welcome.
I'll try and guard the #pragma :)

Thanks. The latest commit works flawlessly. Tested with GCC 7 through 11, clang 5 through 12 (on Linux with libstdc++), and various versions of Xcode (11.3+).

FYI the #pragma workaround for the spurious warning does not work when precompiled headers are used.

Thanks, it's a pity :)
"GCC #pragma does not work with precompiled header":

Bug 64117 - warning control #pragmas in precompiled headers are not obeyed for template code
Bug 56549 - #pragma once ineffective with BOM in include file

How about implementing the other workaround (move the assignment into the body of the constructor) instead of the #pragma?

Thanks, will look at that.

Looks like that will require span_constexpr (C++11) to become span_constexpr14:
    template< extent_t U = Extent
        span_REQUIRES_T((
            U != dynamic_extent
        ))
    >
#if span_COMPILER_GNUC_VERSION >= 900   // prevent GCC's "-Winit-list-lifetime"
    span_constexpr14 explicit span( std::initializer_list<value_type> il ) span_noexcept
    {
        data_ = il.begin();
        size_ = il.size();
    }
#else
    span_constexpr explicit span( std::initializer_list<value_type> il ) span_noexcept
        : data_( il.begin() )
        , size_( il.size()  )
    {}
#endif

Would this be a sane approach?

hmm true. I'm using C++14 so I didn't notice the constexpr problem. The above approach looks totally sensible to me, though I can also patch my local copy of span-lite (I would understand if you didn't want to "downgrade" the constexpr-ness to support PCHs).

I've no problem to implement above.
Might someone complain, next step is to make the solution configurable.

Thanks! commit 8f7935ff4e502ee023990d356d6578b8293eda74 works nicely!
I suppose this issue can be closed now.


Transport: Pool, Conn: Added keyspace to conn and refiller

Added "IO" prefix to errors got by sendRequest caused by connection/internal error to differentiate them from other errors
UseKeyspace can be called to try change keyspace of every present and future connection in connection pool

Fixes #153

In future try to put the additional stuff like "Added "IO" prefix to errors got by sendRequest " to separate commits.

I have a radical idea, add keyspace to NewConnPool.
If empty skip the setting.
If not call UseKeyspace on every new connection.
There are two things that we need to take into consideration.

the keyspace in never changed on the fly, an app operates in a keyspace period.
if you manage to change the keyspace in some but not all connections you are screwed

Overall I see coding skill improvements, keep it up @Neofine

I have removed IO errors from connection as those are not used now.

We need to remove more code in the pool, the validation could go to ConnConfig Validate function.


Error in Scenario 2 Step 2

The title of Step 2 says "require approved app". The detail says "require app protection".

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

ID: 14348ba7-48f0-1862-ebec-0fc3640e587d
Version Independent ID: 8188cee4-3f28-6014-f3be-1ee9e9b183b6
Content: App protection policies with Conditional Access - Azure Active Directory
Content Source: articles/active-directory/conditional-access/app-protection-based-conditional-access.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@Air-Git thanks! Fixing this.

I went through the flows for scenarios 1 and 2 and believe they should be the same for each. I changed the second to match the first and include this text:

Under Access controls > Grant, select the following options:

Require approved client app
Require app protection policy (preview)
Require one of the selected controls

@MicrosoftGuyJFlo

https://github.com/MicrosoftDocs/azure-docs-pr/pull/109351


Remove Netscaler plugin

Integration tests: https://beta-jenkins.mcc.schubergphilis.com/job/cosmic/job/0002-tracking-repo-branch-build/25/testReport/
Merge in combination with: https://github.com/MissionCriticalCloud/cosmic-client/pull/38

Ping @remibergsma @wilderrodrigues @miguelaferreira @bheuvel


Add support of ZIP with Zstd compression

Is your feature request related to a problem? Please describe.
Zstd has a high decompression speed and a high compression ratio. It's supported in .zip file format officially.
Describe the solution you'd like

Add Zstd decompressor to the extension.
Describe how to recompress the .zip archive with Zstd (using WinZip or 7-Zip-zstd)

At the moment it is not possible to do this.


Bump to the latest eks-d releases

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

Updated at https://github.com/aws/eks-anywhere-build-tooling/pull/2469


Possible bug in MPS.expectation_value_multi_sites

Describe the bug
I'm trying to retrieve the expectation value of a Wilson loop operator, so that I call tenpy.networks.mps.MPS.expectation_value_multi_sites(). However, it seems to run into a bug.
Minimal example or code extract
Setup:
from tenpy.models.spins import SpinModel
from tenpy.networks.mps import MPS

M = SpinModel({})
product_state = ["up"] * M.lat.N_sites
psi = MPS.from_product_state(M.lat.mps_sites(), product_state, bc=M.lat.bc_MPS)
Sz = psi.get_op(["Sz"], 0)

To calculate the two-operator (for simplicity, two Sz's) expectation value on two sites:
ops = [Sz, Sz]
psi.expectation_value_multi_sites(operators=ops, i0=0)

it fails with an AttributeError, though it passes in running
ops = ["Sz", "Sz"]
psi.expectation_value_multi_sites(operators=ops, i0=0)

and
ops = [Sz]
psi.expectation_value_multi_sites(operators=ops, i0=0)

About the configuration:
tenpy 0.7.2 (compiled),
git revision unknown using
python 3.8.0 | packaged by conda-forge | (default, Nov 22 2019, 19:11:38)
[GCC 7.3.0]
numpy 1.19.2, scipy 1.5.2

To do matrix exponential, I use for example:
import tenpy.linalg.np_conserved as npc
expSz = npc.expm(Sz)

which will substitute Sz in the above.
Error traceback
If applicable, copy-paste (the first and last part of) the error traceback
AttributeError                            Traceback (most recent call last)
<ipython-input-17-f75bc6f41a4a> in <module>
      1 ops = [Sz, Sz]
----> 2 psi.expectation_value_multi_sites(operators=ops, i0=0)

...

~/.conda/envs/tenpy/lib/python3.8/site-packages/tenpy/linalg/np_conserved.py in __eq__(self, other, eps)
   2285         if self is other:
   2286             return True
-> 2287         if other.chinfo != self.chinfo:
   2288             raise ValueError("other array has different charges!")
   2289         other = other._transpose_same_labels(self._labels)

AttributeError: 'str' object has no attribute 'chinfo'

Great catch, thanks for the very clear report!
Luckily, this was an easy fix.


Adopt a Drain Logo is too big in Outlook

The logo in the automated email shows up way too large, but only in Outlook. Photo now shows up fine. Reference email that Joel sent.

update- logo showed up ok in Outlook 365- so maybe this is only an issue in older versions of the program?

Below is what a 300px logo looks like on Web and Mobile.


CI/CD

@boggsey  - Do you know what this was ticket was on trello? It had no description.

Sorry, this was a ticket for figuring out the continuous integration/ continuous deployment strategy. It's technically in progress.


[Textfield] Multiline Textfield is larger than expected

Hi, I'm trying to achieve a dialog with simple textfields. The problem I'm facing is when I use the multiline props it 'brakes' the width of the component. Also it has not the correct padding. Deleting InputProps and InputLabelProps seems to fix the problem.

Thank you !
Expected Behavior
The multiline Textfield should be the same width as the Textfield multiline props = false.
Steps to Reproduce (for bugs)
Check this sandbox :
https://codesandbox.io/s/o4o59xp8k6
Your Environment

Tech
Version

Material-UI
v1.1.0

React
v15.4.0

browser
Chrome

etc

hi,
when using the new variant=outlined this looks really ugly in forms with multiline textfields.

Any update on this?

Any update ?


[BUG] Long loading times on actions that require user input

very slow loading times on any event
A clear and concise description of what the bug is.
any action on the website that requires new information or logging in is extremly slow and can take upwards of 3 minutes to complete any idea why this could be happening?
the website is running on amazon webserver 2 and currently does not have a https address

also i cannot register but im guessing this is all because of http

Hi,
Thank you for letting me know about the issue.
Could you attach your PHP error log? It would help me locate the bug easily.
Previously, the slow action problem only occurred once due to incorrect setup of Config.php and DiscordConfig.php files. Have you followed the tutorial to set them up correctly?

Please also note that the panel requires at least php version 8.0.

any action on the website that requires new information or logging in is extremly slow...

What exactly do you mean by "new information"?
Do you mean all features that require user input?

yes all actions that require user input are very slow
i am currently running php 8.2.3 maybe thats the problem and i should dial back the version?
i believe i have configured the php files correctly but i will redo it and see if it solves the issue

also i currently do not haver any php error logs to show

https://phlanax.cc/panel/auth/register.php
This page isn’t workingphlanax.cc is currently unable to handle this request.
HTTP ERROR 500
what could be the cause of this

The PHP version appears to be fine, but without a PHP error log,
i can`t helpy you as an error 500 can have various interpretations.

The location of the PHP error log can be found in either /var/log/httpd/error_log or /var/log/apache2 directory.
Another option is to modify the location of the PHP error log by using the .htaccess file.

Tried to access the logs and it said the following in Filezilla "directory /var/log/mariasb: permission denied"
"Failed to retrieve directory listing"

According to the error message you are trying on "mariadb".
However, the logs are located in /var/log/httpd/error_log or /var/log/apache2.
Is there any other way to view the logs on amazon?

my apologies i forgot to check this ticket i will be trying to go into the correct folders thank you

Until approximately when can we expect the server to be up and running again?

Until approximately when can we expect the server to be up and running again?

Not exactly sure I should be able to work on it this weekend sorry

Hello there and sorry for just dashing in here.
Did you try hosting your website on another hosting-platform?
Also I tried hosting my own Website on a free hosting platform and since amazon webservers (and correct me if I am wrong) are free, I would also try hosting your Website on a different platform.
Sorry if I couldn't help I'm not really the web-host kind of guy.

And even if its paid I would try another hosting website. You can use one thats free too just to see if the problem is consistent on any hosting platform or just the one hoster

To ensure optimal loading times, it is important to include webhooks in the DiscordConfig.php file.
Failure to do so may result in delays.

You can reopen this issue if your server is back online and you have access to your log file.


lista 1 prueba

Channel ID (required)
ComediaMadeinSpain.us
Channel Name
No response
Alternative Names
No response
Network
No response
Owners
No response
Country
No response
Subdivision
No response
City
No response
Broadcast Area
No response
Languages
spain
Categories
No response
NSFW
None
Launched
No response
Closed
No response
Replaced By
No response
Website
No response
Logo
No response
Notes
No response

?


[ja] add figure/description in content/ja/docs/concepts/architecture/_index.md

This is a Feature Request
What would you like to be added

add the figure in https://kubernetes.io/docs/concepts/architecture/
add the description in https://kubernetes.io/docs/concepts/architecture/

ref. #42272
Why is this needed
It has not yet been added
Comments
/language ja
/assign
/area localization

/triage accepted


Spanish translation

So I started the Spanish translation here, but wanted to be sure I'm doing it correctly before proceeding with the rest of it.
I noticed the convention is lang-2-letter-country-code, but there is also french named just fr and the Spanish translation also fits best just as es (is that ok?)

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @aturon (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks @alx741 ! It looks good to me.
Why is plain 'es' appropriate? I don't recall why 'fr' is used instead of something more specific. Maybe because there's overwhelmingly one type of French? @steveklabnik probably remembers.

I now nothing about the best practices about this i18n naming, but I thought es fits better because I'm using Spanish that complies with Real Academia Española and no linguistic regionalisms whatsoever, so making es-EC (for Ecuador) would imply the possibility of adding es-AR, es-ES, es-CO, es-MX, and a dozen more with no differences between each other.

Thanks @alx741 !


Git Version not working with Tags only

Problem Description:
The makefile does acquire the image version by using a git version container. This fails on old merged states if the commit you are using is only tagged and do not belong to a branch anymore.
How to Reproduce:
Checkout for example the tag v1.3.1. Try to build any of the images:
franzi@devserver2:~/yocto-images$ ./yocto.sh
+Target: image
Image: cpu01-devtools-image
make IMAGE_DIR=cpu01-devtools-image image
Generate image version
parse error: Invalid numeric literal at line 1, column 5
parse error: Invalid numeric literal at line 1, column 5
make: *** [Makefile:38: image] Error 4

This happens because the json generated by git version contains:

INFO [06/06/24 9:47:50:06] Working directory: /gitINFO [06/06/24 9:47:50:06] IsDynamicGitRepository: FalseINFO [06/06/24 9:47:50:07] Returning Project Root from DotGitDirectory: /git/.git/ - /git/INFO [06/06/24 9:47:50:07] Running on Unix.INFO [06/06/24 9:47:50:07] Project root is: /git/INFO [06/06/24 9:47:50:07] DotGit directory is: /git/.gitINFO [06/06/24 9:47:50:09] Begin: Loading version variables from disk cache  INFO [06/06/24 9:47:50:09] Cache file /git/.git/gitversion_cache/AB1E9EBF3D4A048565D74968977E59C100D52DD4.yml not found.  INFO [06/06/24 9:47:50:10] End: Loading version variables from disk cache (Took: 0.54ms)  INFO [06/06/24 9:47:50:17] Using latest commit on specified branch  INFO [06/06/24 9:47:50:18] Begin: Getting branches containing the commit '9182eb76548c2d3c864516e5fe369973f1f26e15'.    INFO [06/06/24 9:47:50:18] Trying to find direct branches.    INFO [06/06/24 9:47:50:18] No direct branches found, searching through tracked branches.    INFO [06/06/24 9:47:50:18] Searching for commits reachable from 'origin/HEAD'.    INFO [06/06/24 9:47:50:18] The branch 'origin/HEAD' has a matching commit.    INFO [06/06/24 9:47:50:18] Searching for commits reachable from 'origin/beta'.    INFO [06/06/24 9:47:50:18] The branch 'origin/beta' has no matching commits.    INFO [06/06/24 9:47:50:18] Searching for commits reachable from 'origin/bsp_5_7_2'.    INFO [06/06/24 9:47:50:18] The branch 'origin/bsp_5_7_2' has a matching commit.  INFO [06/06/24 9:47:50:18] End: Getting branches containing the commit '9182eb76548c2d3c864516e5fe369973f1f26e15'. (Took: 9.30ms)  INFO [06/06/24 9:47:50:21] Running against branch: (no branch) (9182eb76548c2d3c864516e5fe369973f1f26e15)  INFO [06/06/24 9:47:50:21] Current commit is tagged with version 1.3.1, version calculation is for metadata only.  WARN [06/06/24 9:47:50:21] An error occurred:
It looks like the branch being examined is a detached Head pointing to commit '9182eb7'. Without a proper branch name GitVersion cannot determine the build version.

The tag used is lightweight and not annotated. Maybe this could be a problem, because the git documentation says (https://git-scm.com/docs/git-tag):

Annotated tags are meant for release while lightweight tags are meant for private or temporary object labels. For this reason, some git commands for naming objects (like git describe) will ignore lightweight tags by default.

Update:
The docker container for generating the git version seems to be deleted.
franzi@devserver2:~/yocto-images$ docker pull elbb/bb-gitversion:0.7.0
Error response from daemon: pull access denied for elbb/bb-gitversion, repository does not exist or may require 'docker login': denied: requested access to the resource is denied


fix_cmp0075: set CMP0075 to NEW

Update policy: `Include file check macros honor ``CMAKE_REQUIRED_LIBRARIES```
This avoids warnings with recent versions of Cmake

Applied in v5.0.0


Fix links to "A Guide to Python's Magic Methods" and "Probability and Statistics Cookbook"

Thanks for your PR. A big incoming PR fixing these two links being already in the pipeline, I think it will be easier to close this one without merging it as soon as the other lands. Sorry about that. :(

No worries, last I looked the new links in the other PR were different.  Once that one is in, I'll check again and close or modify if I feel these links are better.

Great, thank you! The other PR has just been merged, please check and report. :)

The magic method link is still broken, I'll close this PR and open a new one with that change


Formatting of base in constructor

Describe the solution you would like
public class WhereFuncPathTests : UnitTestBase
{
    public WhereFuncPathTests(DatabaseFixture fixture)
: base(fixture)
{ }
}

Should be converted/formatted (CTRL+SHIFT+I) to:
public class WhereFuncPathTests : UnitTestBase
{
    public WhereFuncPathTests(DatabaseFixture fixture)
        : base(fixture)
    { }
}

A lot has changed with O# formatting since this issue was opened. If you are still having problems, please open a new issue at https://github.com/omnisharp/omnisharp-roslyn/issues


Spack on dom

Issue to track progress & status

Openend a ticket at CSCS about it:
https://jira.cscs.ch/plugins/servlet/desk/portal/1/SD-51639
(visible for g110 group members)


WIP: feat(volume): Add pulseaudio backend

Provides an option to use PulseAudio instead of ALSA for the volume module, which is determined at compile time. If both are available, it will be compiled with PulseAudio by default.
The only configuration option currently is sink, which provides the sink to be used as default (name can be found by running pacmd list-sinks). The backend subscribes to any changes in the PulseAudio sinks, so if the sinks change (e.g. headphones are unplugged) and the specified sink disappears, it will automatically revert to the default sink. Similarly, when the specified sink reappears, it will use it instead of the default. It seems like usually there is only one sink, so this option should be optional. The default configuration uses the default sink at all times.
Currently, there is no volume cap, so it can go over 100% to an extremely high value, as supported by PulseAudio (not sure whether to limit or leave it to the user).

I have not looked at the code yet, but I think it's great that polybar gets native pulse support. One thing right away: I think it would be better to have a dedicated pulseaudio module instead of cramming the functionality into the same module.
This way people can setup either module without having to recompile everything and also the valid syntax of the volume module would not depend on how polybar was compiled. Also it would break all existing configurations that use the built-in volume module on systems that also have pulse installed, once polybar is recompiled.
I will try to take another look at this on the weekend and give some more feedback.
Thanks!

Yeah, I think I made a mess of the build configuration there as well. Since the volume module is essentially the alsa module, I think it would be best to rename it to alsa and deprecate volume (but still link the option internal/volume to internal/alsa for now).

Yeah, I thought the same thing. And once we push the next major release we can remove the internal/volume module.

Should I rename the volume module in this PR or do it separately?

I think we leave it like it is in this PR and open a separate one, once this one is merged

I've added logging, a volume limit, and cleaned up some redundant volume retrieving.
If an abstract module were to be built, it would only have get_output(), get_format(), part of update(), and part of the constructor. Creating one would probably make things too complicated.

Sorry for the wait on this. This is quite a big PR and I haven't had the time to review it properly. I will try to get in some thourough testing in the next few weeks.

@NBonaparte Why do we check for the pulseaudio binary in addition to the libpulse library?
If we only checked for the library we could change the dependency in the AUR frompulseaudio to only libpulse.

You can have libpulse installed without having pulseaudio. For instance, chromium only requires the former, and I tried to run pulseaudio a long time ago before realizing I actually didn’t have the binary installed.

Ah, I see. But the binary is not necessary for polybar to run?

Yeah I suppose. It would be up to the user to ensure that they actually have the binary installed.


Bump pipeline and route version

With the addition of a new API parameter, let's bump the pipeline version, which implies
bumping the route version after make generate-api.

Ohh the test failure is probably because the version is hard coded in the test


Improve help for kubectl gs template cluster

kubectl-gs template cluster command should provide explanation and examples in the usage (--help). Has nothing currently.

Hello @giantswarm/team-honeybadger and @giantswarm/team-rocket would you have something to contribute, please ?

Flags are descriptive enough for me, but an example would be great. One I could almost copy-paste.

Hii @tfussell , would you have some example to add here please.

We will have public docs available once https://github.com/giantswarm/giantswarm/issues/20821 is merged. I can copy the example command into the --help output for now.
That said, there was a concern about having too much provider-specific information there. My best idea to deal with that was to move kubectl gs template cluster --provider <provider> into a subcommand like kubectl gs template cluster <provider> so that the help text can only show flags relevant to the given provider. What does everyone think about that?

closing as team rocket and customers are not using kubectl-gs, documentation also seems sufficient for other teams now.


ISPN-11437 Protostream Processor missing from infinispan-bom

https://issues.redhat.com/browse/ISPN-11437

Merged. Thanks @ryanemerson !


Missing dependencies for SOCKS support

File "twitterBot.py", line 39, in
twitter.tweet('Hello world')
File "twitterBot.py", line 15, in tweet
self.api.update_status(status=message)
File "/home/since95/pythonApi/pythonApi/local/lib/python2.7/site-packages/tweepy/api.py", line 194, in update_status
)(post_data=post_data, _args, kwargs)
File "/home/since95/pythonApi/pythonApi/local/lib/python2.7/site-packages/tweepy/binder.py", line 245, in call
return method.execute()
File "/home/since95/pythonApi/pythonApi/local/lib/python2.7/site-packages/tweepy/binder.py", line 189, in execute
raise TweepError('Failed to send request: %s' % e)
tweepy.error.TweepError: ***# Failed to send request: Missing dependencies for SOCKS support.**
Getting error , i am using it behind the proxy.. proxy variable is SET stills shows error

Closed in https://github.com/tweepy/tweepy/commit/254c63d5f59a16b89df398f91f2e6751647946bf.


Add missing dependency

Add missing assembly.
fixes #370

@twenzel your changes have been merged, thanks for your contribution 👍


Adds Github to signin page providers

Hey, I just made a Pull Request!
Added Github for regular OAuth2 flow as an experiment to understand the auth-backend.
I'm aware that a RFC for Github Apps is still in progress but in the meantime I've considered that it would be useful to help more folks onboard. We could drop this change in the future when Backstage supports Github App.
The option appears in the SignIn page

The ID is displayed with the username

Avatar and username

Avatar and display name

Scope change

Considerations:

The Id is Github's username.
Reduced the scope access to read-only. It might be a source of concern if Backstage asks write access to User's Profile.
Email is provided only if is available in the user's public profile. If having an email is important, we can use user:email to fetch emails from another endpoint.
Display name is provided only if is available in the user's public profile. Username is used as fallback.
Implementation was based on Gitlab provider.

Questions:
Am I supposed to use BackstageIdentityApi? Gitlab's implementation is not using but according to docs/auth/README.md gitlab and this PR should use it?
Remarks and topics for discussion (and to open another PR):

The packages/app/src/App.tsx lists all providers. So every time there is a new provider, it must be added there. However, i'm concern that folks will face conflicts constantly because their local repositories also have their changes. Are we ok on moving that listing to app_config.yaml?
The same issue occurs in packages/core/src/layout/SignInPage/providers.tsx, plugins/auth-backend/src/service/router.ts, etc. This seems to be a good point for conflicts any time a new provider is added. What's your take on having a self-registration for each new provider, in order to minimize changes to these files?
The collapse button of the sidebar is well on top of the identity providers and is a pain to try to sign-in/logout since is easy to missclick. Easy to notice after the constant testing 😅 Any issue to move the button to the top?

:heavy_check_mark: Checklist

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

Happy first contribution! Thanks a lot 🎉 💯 🥇


Logging: print out error messages when opening worker tabs

👍 you should feel free to just make these changes against master, and PRs if you want a second opinion.


[core] Compatibility with React 19 types

Summary
To be compatible with React 19, types must be compatible. Many changes are codemoddable with npx types-react-codemod@latest but a lot of changes need to be done manually because your relied on element.props being typed as any. There are also some 3rd party libraries that need to be be compatible first.
A full overview of the necessary changes can be found in https://github.com/eps1lon/material-ui/pull/619. You can also check out https://github.com/users/eps1lon/projects/3/views/9 to get an overview what other apps/libraries need to do for migration.
Tasks

[x] https://github.com/mui/material-ui/pull/42346
[x] https://github.com/mui/material-ui/issues/42380
[x] Upgrade to updated @types/ packages when published (DefinitelyTyped#69022) and fix remaining issues (see #42346)

Search keywords: types react 19

@DiegoAndai one more thing to be handled for React 19 compatibility.

@types/react and @types/react-dom bumped in https://github.com/mui/material-ui/pull/44672


build-digitalocean-centos69-x64-1: No space left on device

Build failed on build-digitalocean-centos69-x64-1 (https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk14u/job/jdk14u-linux-x64-openj9-linuxXL/5/consoleFull) with:
error: unable to write file test/functional/CacheManagement/src/tests/sharedclasses/options/junit/TestOptionsDefault.java
error: unable to write file test/functional/CacheManagement/src/tests/sharedclasses/options/junit/TestOptionsDefaultRealtime.java
error: unable to write file test/functional/CacheManagement/src/tests/sharedclasses/options/junit/TestOptionsNonpersistent.java
error: unable to write file test/functional/CacheManagement/src/tests/sharedclasses/options/junit/TestOptionsNonpersistentRealtime.java
fatal: cannot create directory at 'test/functional/CacheManagement/testcode': No space left on device
warning: Clone succeeded, but checkout failed.

Have removed my as test jobs and the jdk13u build jobs from the machine to resolve this.


Investigate FA failures on default path

Resolved in https://github.com/intel/intel-xpu-backend-for-triton/commit/984905a97dc537ac7988f5a0c7fbdf83ea85922b and https://github.com/intel/intel-xpu-backend-for-triton/commit/466cb15a3a24f3558db5802eb7b1fe6010ac00e0.


Timestamp in Transfer History is Optional

Made the transfer history timestamp an Option so it's not required in the schema.
This should make it so that clients based on the schema can accept responses from contracts that don't provide this info.

@assafmo It will be optional in the schema, see #12


1.0

Merge 1.0

Current coverage is 98.76% (diff: 100%)

Merging #1285 into master will increase coverage by 0.45%

@@             master      #1285   diff @@
==========================================
  Files            29         29
  Lines          6505       7831   +1326
  Methods           0          0
  Messages          0          0
  Branches       1088       1550    +462
==========================================
+ Hits           6395       7734   +1339
+ Misses           58         49      -9
+ Partials         52         48      -4

Powered by Codecov. Last update c4e20bd...b0bba62


Bugfix: File Response Problems

This was a fun one to debug. I've been using Response::file() to try to return a file. In this case, I've been using both application/zip and application/vnd.openxmlformats-officedocument.spreadsheetml.sheet Content-Type headers.
When calling this method, the response would hang until the connection timed out, even thought the correctly formatted file was present in the file system of the server. Eventually, I tracked the problem down to the setting of the Content-Length header.
To my surprise, changing the order of the readfile($path) and $this->send() calls in Response::file() fixed the issue. My theory here is that the problem arises when the Content-Length header is set to a non-zero value when the output buffer is empty.
I couldn't come up with an appropriate unit test for this fix, but I'm open to suggestions. I've thoroughly tested this with integration tests and various other debugging strategies.

Wait a minute! I have an idea!
Maybe you could try changing the Content-length header value setting to instead be additive with whatever may or may not be in the output buffer via ob_get_length(), something like this, maybe:
$this->header('Content-length', ob_get_length() + filesize($path));

Agh. So I tried the solution you thought of (and it was a good one), however it didn't work (kept timing out), so the problem may not actually be the one that I'm thinking of. This is really kinda weird. I can definitely track the problem to the setting of headers. I'm going to have to debug this one further.

Hmm, interesting. Ok, let me know what you find out! 😃

Really great job here, @mcos! Thanks for the sensible fix!


[FEATURE]  Add ability to name agents/servers

Problem
I would like to be able to have different names for my node to help with discovery in tools like k9s.
If I (a) create the cluster and then (b) create the nodes I can achieve this.
But if I need to create the cluster and nodes at the same time i.e. via -agents flag then this is not possible.
And some use cases e.g. per-node volumes are only possible using this approach
Solution
Either:

Add a "k3d node rename" command.
Add a new flag to k3d create cluster "--name KEY=VALUE[@NODE]"

Hi @nadenf , thanks for opening this issue!
So since you're talking about k9s, you're referring to the Kubernetes Node Name, right?
k3d only sets the "hostname" of the nodes (i.e. the container name), which is the default used by Kubernetes (K3s) as node name.
So you can use e.g. --k3s-arg "--node-name=mynode"@agent:0 to name the first agent (didn't test this).

@iwilltry42 .. That does indeed work.
Would be nice still to have a friendly config option.

Hi @iwilltry42,
I'm new to k3d. Can i try this issue? I'll inform if i couldn't finish it.

Hi @shankeerthan-kasilingam, welcome to the project 🙂
Sure, go for it! Let me know if you need any help or insights.


Convert all components to composition API

Depends on #308 (will need one additional change in controls.vue since this branch missing a commit that adds a watcher)
Closes #247
Converts the remaining options API over to composition. It didn't require too much work, but we could improve this by grouping together relevant code so it's not so spread out, especially in MultiMatrix.vue

Force pushed to fix the problem next to "depends on #308" in the description.


Fix for logic based on CallToUnresolvedCallee flag

The check for the dynamism of the return from unresolved callees incorrectly ended up applying to calls where the flag had propagated from another call site, including the entry point callable itself. This change introduces new infra to track the call expression that introduced the flag as opposed to all calls the flag had propagated to, which keeps the desired protection but avoids rejecting other call expressions.

FYI: I confirmed the new test case fails on main but passes with the fix, so we know it is testing the specific flag propagation the change is intended to side-step.


chore(Algebra): turn induction' into induction

This PR turns the (soon to be deprecated?) induction' into induction.

Thanks!
bors merge


Difference between editor preview and blot drawing when using Math.random()

It wasn't clear to me that I had to use rand() to prevent this because I hadn't scrolled to that part of the docs.
I didn't realize I was using Math.random() because I copied code from an example implementation of a K-mean algorithm online

I'd propose fixing by prototyping Math.random() to give a warning on first usage. That'd help...

warn people who don't know they need to use rand(),
people who accidentally used Math.random() even though they know to use rand()

if someone implements a warning system for different functions, can you also add a warning for print()? I once had a 100 long for loop and had to close the print page prompt 100 times.

@BrightTheBackpack Want to create a new ticket for that? I think that's a separate discussion because Math.random() is something that we're treating different on our site. Things like print() alert() and prompt() can be prototyped, but that's more browser specific, not site specific.

Just tested this and it's no longer erroring on submissions like this: https://blot.hackclub.com/editor?src=https://raw.githubusercontent.com/hackclub/blot/main/art/BeautifulSaturn-AlexB/index.js


Update the Wasmi fuzzing oracle to version 0.31.0

Currently Wasmtime uses the 2 years old Wasmi version 0.20.0.
Since then Wasmi has improved substantially and added support for new Wasm proposals such as bulk-memory, reference-types and tail-calls which we can now enable.
Wasmi v0.31.0 has recently been audited and is used by some large projects, thus is a lot more battle tested than the previous v0.20.0.
Besides that the most notable change are performance improvements which should make fuzzing with Wasmi a tiny bit faster.
Look into the future: Since roughly half a year I am working on the next major Wasmi version v0.32.0 which is a complete rewrite of the Wasmi executor featuring a much more powerful register-machine execution model and optional lazy compilation & validation. I hope that it becomes stable enough for use soon to provide it as fuzzing oracle to Wasmtime. Due to the changes we refer to the new Wasmi version as Wasmi (register) and the old Wasmi as Wasmi (stack). It might even make sense to have both versions as oracles at the same time because they have very different strengths.

With respect to cargo vet:
  wasmi:0.31.1 missing ["safe-to-run"]
  wasmi_arena:0.4.0 missing ["safe-to-run"]
  wasmi_core:0.13.0 missing ["safe-to-run"]
  wasmparser-nostd:0.100.1 missing ["safe-to-run"]

All those crates are maintained by Wasmi maintainers (me).

Thanks!
To avoid a new vet of wasmparser-nostd would it be possible to conditionally build wasmi with the official wasmparser crate instead? That way we could get that auto-vetted.

Wasmi v0.31.0 is no longer in development so I am not going to support the standard wasmparser for it, however, I am going to think about the implications doing so for future Wasmi versions starting from v0.32.0 once released. Since Wasmi already has a std crate feature it might be simple to just use wasmparser if it is enabled and use wasmparser-nostd otherwise.

Ah ok, in that case this is going to have to hold off until one of us gets a chance to vet the dependencies here. I'm stretched a bit thin at the moment so it may be a bit before I can personally get to this (but others can of course beat me to it)

Please tell me if and how I can help with vetting. :)

Reading over some code I think that this impl is not sound as it's supposed to be T: Sync. Additionally though I don't think you should need the unsafe impl at all. You can try changing the PhantomData to either fn() -> Idx or fn(Idx) I think as one of them might require the impl and one might not.
Otherwise the wasmparser-nostd diff is quite large or larger than I remember from the wasmparser diff, so diffing those directories is requiring a good deal of time to go through to verify it's the same. Additionally I haven't even gotten to wasmi yet which glancing at it is quite large and additionally contains a good deal of unsafe code to check.
Currently though wasmi is only used for fuzzing so it doesn't necessarily need a thorough review or strict vetting. Do others have thoughts on whether we should add exemptions for this dependency as opposed to vetting it?

Reading over some code I think that this impl is not sound as it's supposed to be T: Sync. Additionally though I don't think you should need the unsafe impl at all. You can try changing the PhantomData to either fn() -> Idx or fn(Idx) I think as one of them might require the impl and one might not.

Thanks for catching this bug! The code is quite old and hasn't seen a lot of love lately.

Otherwise the wasmparser-nostd diff is quite large or larger than I remember from the wasmparser diff, so diffing those directories is requiring a good deal of time to go through to verify it's the same. Additionally I haven't even gotten to wasmi yet which glancing at it is quite large and additionally contains a good deal of unsafe code to check.

The unfortunate truth is that especially with the component model a lot of very no_std unfriendly abstractions such as the IndexMap has been added which required me to come up with my own no_std friendly version of it. I never intended this to be a long lasting effort but here we are.

Currently though wasmi is only used for fuzzing so it doesn't necessarily need a thorough review or strict vetting. Do others have thoughts on whether we should add exemptions for this dependency as opposed to vetting it?

To provide you with a bit of transparency here about the state of Wasmi:

Cons:

I work on the project alone.
There are pretty much no code reviews due to above point.
There is a fair amount of unsafe code use, but mostly in the Wasmi bytecode executor.

Pros:

There are a good deal of unit, integration, e2e and fuzz tests.
We do have a very extensive CI (incl. miri)
Wasmi v0.31.0 is already used in production by different companies some many months.
Wasmi v0.31.0 has received a security audit by SRLabs.
Wasmi 0.31.0 is much less complex than the Wasmi v0.32.0-beta.n (on master).

I just made this use Wasmi v0.31.1 instead of v0.31.0 which is more explicit. Note that v0.31.0 got yanked some time ago.

Do others have thoughts on whether we should add exemptions for this dependency as opposed to vetting it?

I'm happy with adding a cargo vet exemption for wasmi 0.31.1 which notes that its only intended to be used in the context of testing. I don't think there is any real benefit to our project to have a higher bar than wasmi already meets for the sake of a fuzzing oracle.

I'm happy with adding a cargo vet exemption for wasmi 0.31.1 which notes that its only intended to be used in the context of testing. I don't think there is any real benefit to our project to have a higher bar than wasmi already meets for the sake of a fuzzing oracle.

Agreed.

Reading over some code I think that this impl is not sound as it's supposed to be T: Sync. Additionally though I don't think you should need the unsafe impl at all. You can try changing the PhantomData to either fn() -> Idx or fn(Idx) I think as one of them might require the impl and one might not.

btw. I just published a new wasmi_arena v0.4.1 and yanked v0.4.0 with the unsoundness fix. (https://crates.io/crates/wasmi_arena/0.4.1)

Ok I've posted https://github.com/bytecodealliance/wasmtime/pull/7810 with new vet metadata. If you rebase this PR on top of that once it lands should be good to go

Something went terribly wrong with the rebase ... let me investigate ...

Thanks for the patience while we figure out the vetting, and thanks again for helping us out with the update!

Happy to help and thanks for having Wasmi as fuzzing oracle! :)


Update getTimezoneOffset JSDoc for clarity

Acknowledgement

[X] I acknowledge that issues using this template may be closed without further explanation at the maintainer's discretion.

Comment
Update getTimezoneOffset JSDoc for clarity

What's unclear about it?