Create STYLEGUIDE.md

We keep talking about wanting a style guide so here's a start!
The intention of these style choices is to be small additions to what we already get from using black/darker. Over time we should also be looking to add the linked flake8 plugins to further enforce the desired style.
This initial guide proposes the following policies:

Policy
Tooling

0
Use black (or darker for preexisting projects).
black darker

1
Prefer f-strings over all other string formatting styles (e.g. str.format, c-style).
flake8-use-fstring

2
Prefer pathlib over os.path.
flake8-use-pathlib

3
Define typing aliases at the top of files.

4
Define loggers at the top of files.

5
Only use assert in tests.
flake8-useless-assert

6
Use descriptive variable names: no one-char variablesavoid abbreviationsavoid overloading builtins
flake8-variables-names

7
When indicating a change that requires a future version of Python (or dropping a currently supported version) use the following format:# FUTURE: <minimum Python requirement>, <details>e.g.:# FUTURE: Python 3.9+, replace with ...

A number of thoughts:

I think having isort in here somewhere would be great
should we encourage type hinting as a requirement for new code and optional for existing code? Or suggest updating code while we work on it?
let's use flake8-spellcheck to make sure the code we add is readable and doesn't use abbreviations

the more this ideas are materialised via a .pre-commit-config.yaml in each repo (pyupgrade, isort, black, flake8*) the easier to role out the ideas consistently.
Regarding variable/function names: no mater how many rules are covering those, only a review by a person will tell if a name makes sense or not.
Though style is not everything... the right abstraction / architecture for a given problem can be much more important as style can be fixed easily by styling / renaming.. if the abstraction / architecture is wrong, the code requires much more attention...

Sounds good.

Use of black should provide output only, yeah?  We're not going to try to have it provide commits to automatically fix things?  (That would make the commit history harder to process and harder to verify.)
I recommend folks first try out flake8-spellcheck and flake8-variables-names.

@awwad We are currently employing darker (https://github.com/akaihola/darker) to gradually apply black's styling (i.e. black is only applied to the PR changes).
The problem with any flake8 check is they run on the entire code base. We've already attempted to apply flake8-variables-names to conda (https://github.com/conda/conda/pull/11444) and had to back out for now given the scope of the necessary changes. But yes, eventually we'd want to turn on all of these automated checks as things get cleaned up.

We currently use pre-commit + darker to auto-format PRs. Eventually, this would be modified to use pre-commit + black. Either way, no historical blame info will be lost since it's still part of the PR and the formatting will be attributed to whoever contributes the code changes.


docs(general): Update README and CONTRIBUTING

Update README.md to streamline the getting started information and move build and test instructions into CONTRIBUTING.md.

Merged 26-Apr-2017.


Endpoint diga.apimisc.de requires the contentType to be set for the encrypted data in the Multipart requests

Problem
Since last week no DiGA codes can be validated or bills be sent to the endpoint diga.apimisc.de as they installed an update and now the Multipart requests require the MediaType to be set for the encrypted xml content (nutzdaten).
Solution
Set the MediaType of the encrypted content of the multipart request to "application/octet-stream".
I tested this and it fixes the issue for the endpoint diga.apimisc.de while the other also accept it for the test code validation.

released in 2.0.1, thanks for the contribution!

thank you for the fast release 🚀


Need to fix twist lock vulnerabilities for Google Cloud Kubernetes Engine

Druid Version : 0.18.1 version.
There 5 high priority vulnerabilities which have poped up. They are give below:
Jackson
Guava
netty
log4j
logging-log4j.
I am trying to build druid after replacing the version in the pom.xml and trying to build the druid from source code.
I need urgent help how to build druid from source code . I mean exact commands and steps.
Any help is appretiated.
Thanks,
Ashish

Druid 0.18.1 build is also getting failed with below error
[ERROR] Failed to execute goal org.codehaus.mojo:exec-maven-plugin:1.6.0:exec (generate-binary-license) on project distribution: Command execution failed.: Process exited with an error: 1 (Exit value: 1) -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :distribution
root@debian:/home/ashish/druid_build/druid#


[Bug]: When using a Google Bucket for WAL storage the spec.backup.retentionPolicy setting has not effect

Is there an existing issue already for this bug?

[X] I have searched for an existing issue, and could not find anything. I believe this is a new bug.

I have read the troubleshooting guide

[X] I have read the troubleshooting guide and I think this is a new bug.

I am running a supported version of CloudNativePG

[X] I have read the troubleshooting guide and I think this is a new bug.

Contact Details
<EMAIL_ADDRESS>Version
1.22.1
What version of Kubernetes are you using?
1.27
What is your Kubernetes environment?
Cloud: Google GKE
How did you install the operator?
YAML manifest
What happened?
When attempting to use spec.backup.retentionPolicy to limit the amount of days WALs we keep, this setting appears to be ignored by the operator. We are using a standard Google Bucket which works in all other respects for WALs and basebackups.
Cluster resource
# dev16s-post.yaml
#
#
apiVersion: postgresql.cnpg.io/v1
kind: Cluster
metadata:
  name: dev16s
  annotations:
    cnpg.io/coredumpFilter: "0xFF"
spec:
  # Postgres 16 image
  imageName: eu.gcr.io/XXX/postgres:16.1.4
  imagePullPolicy: Always
  enableSuperuserAccess: true
  instances: 3
  primaryUpdateStrategy: unsupervised
  primaryUpdateMethod: restart
  startDelay: 30
  stopDelay: 30

  ephemeralVolumeSource:
    volumeClaimTemplate:
      spec:
        accessModes: ["ReadWriteOnce"]
        # example storageClassName, replace with one existing in your Kubernetes cluster
        storageClassName: "standard-rwo"
        resources:
          requests:
            storage: 200Gi

  affinity:
    tolerations:
    - effect: NoSchedule
      key: dedicated
      operator: Equal
      value: database

  # env variable set to get R working with this image
  env:
  - name: R_HOME
    value: /usr/lib/R
  - name: TMPDIR
    value: /var/lib/postgresql/wal/
  - name: LD_LIBRARY_PATH
    value: "/usr/lib/R/lib:/usr/lib/x86_64-linux-gnu:/usr/lib/jvm/default-java/lib/server:/usr/local/lib"
  - name: R_ENABLE_JIT
    value: "0"

  monitoring:
    enablePodMonitor: true

  postgresql:
    pg_hba:
      - hostssl all all all scram-sha-256
    parameters:
      shared_buffers: "8GB"
      max_connections: "300"
      work_mem: "32MB"
      maintenance_work_mem: "4GB"
      effective_io_concurrency: "400"
      max_parallel_workers_per_gather: "8"
      max_parallel_maintenance_workers: "8"
      effective_cache_size: "32GB"
      max_parallel_workers: "24"
      wal_buffers: "16MB"
      min_wal_size: "512MB"
      max_wal_size: "4GB"
      checkpoint_completion_target: "0.9"
      random_page_cost: "1.1"
      default_statistics_target: "200"
      autovacuum_max_workers: "10"
      autovacuum_naptime: "10"
      timescaledb.max_background_workers: "114"
      cron.max_running_jobs: "64"
      cron.use_background_workers: "on"

      dynamic_shared_memory_type: sysv
      shared_memory_type: sysv

      max_slot_wal_keep_size: "50GB"

    shared_preload_libraries:
      - 'plr'
      - 'citext'
      - 'quantile'
      - 'pg_trgm'

  storage:
    storageClass: premium-rwo
    size: 450Gi
  walStorage:
    storageClass: premium-rwo
    size: 300Gi

  resources:
    requests:
      memory: "32Gi"
      cpu: "4000m"
    limits:
      memory: "32Gi"
      cpu: "4000m"

  managed:
    roles:
      - name: stolon
        ensure: present
        superuser: true
        createdb: true
        login: true
        passwordSecret:
          name: stolon

  backup:
    volumeSnapshot:
      className: gke-vsc-pd
    barmanObjectStore:
      destinationPath: 'gs://XXX/CNPGdev16s'
      googleCredentials:
        gkeEnvironment: true
      wal:
        compression: snappy
        maxParallel: 2

  serviceAccountTemplate:
    metadata:
      annotations:
        iam.gke.io/gcp-service-account<EMAIL_ADDRESS>
Relevant log output
n/a

Code of Conduct

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

Hello @chrisalavoinehmm
This is due to using Volume Snapshot backups,
As per the documentation:

[retention policy is] currently only applicable when using the BarmanObjectStore method.
CloudNativePG API documentation regarding retention policy

My team use lifecycle policies on the bucket to remove old WAL instead.
The same issue will arise with the backups themselves: Old volumesnapshot backups are also not garbage-collected despite the retention policy.
You can find more details on a related GitHub issue I opened: https://github.com/cloudnative-pg/cloudnative-pg/issues/3657

I tried to set Lifecycle on my bucket and it completely broke the WAL shipping. How many days did you set your Lifecycle to?

I tried to set Lifecycle on my bucket and it completely broke the WAL shipping. How many days did you set your Lifecycle to?

We set it to the retention policy we want + 1 day (just in case), so it is deleted afterward.
I do not understand why the WAL shipping would be broken by setting a lifecycle rule on the bucket storing only the WAL

Me neither, but the fact remains that the spec.backup.retentionPolicy does not work as expected when using Google Buckets.

Me neither, but the fact remains that the spec.backup.retentionPolicy does not work as expected when using Google Buckets.

I think it is unrelated to using Google buckets, it is because you are using volume snapshots : my understanding is that in that situation barman will never do backups on the bucket so it will never compute the backups and wal archives that can be garbage collected with regard to the retention policy, and will never delete them.
So the same issue would arise with S3 for example (or a Google bucket using the S3 API)
The fact that the retention policy does not work with Volume snapshots is however only present in the API's documentation, not the rest of the documentation

@BaudouinH is right. We can only delete WAL files when we delete a backup, and we delegate the operation to Barman Cloud for that (so there's little that can be done in CloudNativePG - that is why we are working on the CloudNativePG Interface - CNPG-I).
When we delete a backup, we request Barman Cloud to remove the WAL files prior to the first available backup.


Update default VERSION in makefile to 0.0.1-latest

The 0.0.1 tag does not exist and the make deploy will fail if no other VERSION is supplied.

Description

Which issue(s) does this PR fix or relate to

Fixes #issue_number

PR acceptance criteria

[ ] Tests
[ ] Documentation

How to test changes / Special notes to the reviewer

This PR is replaced by https://github.com/janus-idp/operator/pull/207


*: update Cortex (and Prometheus) dependency

Update Cortex dependency (and Prometheus together) to include
https://github.com/cortexproject/cortex/commit/70dddb6b70b87f899ab115e79426a3ba522ae6ad.
storage.Querier finally has support for label.Matchers so now we can
fully pass them down. This is covered by tests. Plus, I've played around a bit
with these changes locally.
Signed-off-by: Giedrius Statkevičius<EMAIL_ADDRESS>
[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Nice! Is this pr ready for review now?

Nice! Is this pr ready for review now?

No, still need to fix the tests hence it is a draft

@yeya24 fixed the tests, PTAL :beers:

@yeya24 thanks for the very quick review! :heart:


setting up build environment on mac

Hello
Am trying to set up a build environment on my mac with xcode - I have installed QT and managed to link the libraries, but I still can't get the app to build, I am now getting QtGui/QApplication not found - any ideas what I need to do in my build path etc to get this working!?
Thanks!
Richard

Hey Richard,
Alas...  getting the build environment is almost as difficult as writing the app itself.  I will try to post some step-by-step instructions on the later.
I built this with Qt 4.8.1, XCode 7.2, OSX 10.11
https://download.qt.io/archive/qt/4.8/4.8.1/
~Chris

Thanks Chris - and thanks for fixing the broadcast thing. I think half my problem was that I had installed the newer version of QT. It looks like a lot of the QT’s locations are hard-coded, so I think I need to create a /Developer/tools folder and put some stuff in there, otherwise my Xcode project won’t play nicely.
I’m about to board a plane but submitted a pull request for something I did last night (you are correct in that I found it easier to follow your code and add a feature (maybe badly!) than to get QT to work!)
Great app by the way! Literally perfect timing for a project I’m working on
Richard
--
Richard Williamson
www.richard-williamson.com
<EMAIL_ADDRESS>07957 457 362

On 23 Mar 2016, at 17:52, Chris Mizerak<EMAIL_ADDRESS>wrote:
Hey Richard,
Alas... getting the build environment is almost as difficult as writing the app itself. I will try to post some step-by-step instructions on the later.
I built this with Qt 4.8.1, XCode 7.2, OSX 10.11
https://download.qt.io/archive/qt/4.8/4.8.1/ https://download.qt.io/archive/qt/4.8/4.8.1/
~Chris
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/ElectronicTheatreControlsLabs/OSCWidgets/issues/3#issuecomment-200465063

Managed to get my computer to build finally! It's a little buggy but..

installed homebrew
installed QT (brew install QT)
Edited the build phases to point to the moc in the location brew puts it /usr/local/Cellar/qt/4.8.7_2/bin/moc)
Copied the QT libraries from the brew folder to /Libraries/Frameworks

I get some errors but the build seems to work!


Font names are different when using getTextContent()

Attach (recommended) or Link to PDF file here:

anguish-languish.pdf
anguish-languish-2.pdf

Configuration:

Web browser and its version: N/A
Operating system and its version: Ubuntu 22.04
PDF.js version: 2.16.105
Is a browser extension: no

Steps to reproduce the problem:

Open a PDF using pdf.js
Get a page and call page.getTextContent()

What is the expected behavior? (add screenshot)
items contains a list of text items with a fontName attribute. I would expect fontName to be the same as the font name used in the PDF, e.g. in the attached file it should be Helvetica-Bold and Times-Roman.
What went wrong? (add screenshot)
For anguish-languish.pdf (above) instead of the original font names (Helvetica-Bold and Times-Roman), the font names appear to be truncated to Helvetica and Times
For anguish-languish-2.pdf, instead of AAAAAA+LiberationSerif-Bold and BAAAAA+LiberationSerif I see completely different font names: g_d0_f1 and g_d0_f2
According to this pull request it seems that a different font name is used. Is there a way to get the original font name?
That PR has a couple thumbs up (excluding my own) so it seems there's some interest in this.
Thanks!

PDF.js version: 2.16.105

Please note that that version is no longer supported, please find the latest releases at https://mozilla.github.io/pdf.js/getting_started/#download

According to this pull request it seems that a different font name is used. Is there a way to get the original font name?

No such functionality exists, unfortunately (for the reasons outlined there). Please keep in mind that PDF documents, from a specification perspective, uses slightly different code-paths for rendering and text-extraction.
Hence it's possible that the text-content may contain glyphs not actually present in the font-files themselves, with ligatures being one potential example, which is (part of) the reason that we use different fonts for rendering respectively text-selection.

@Snuffleupagus What about adding the original font to TextStyle? That would use much less memory since it appears only one TextStyle item is created for each font. Would such a PR be accepted?
It seems like it'd be pretty trivial to add: https://github.com/bmaupin/pdf.js/commit/a4fc11d186d6a3148ec46799629459e6d7b36116
Thanks!

For anguish-languish.pdf (above) instead of the original font names (Helvetica-Bold and Times-Roman), the font names appear to be truncated to Helvetica and Times
For anguish-languish-2.pdf, instead of AAAAAA+LiberationSerif-Bold and BAAAAA+LiberationSerif I see completely different font names: g_d0_f1 and g_d0_f2

The difference is non-embedded respectively embedded fonts (this apparently didn't make it into my previous response).

What about adding the original font to TextStyle? That would use much less memory since it appears only one TextStyle item is created for each font. Would such a PR be accepted?

That wouldn't actually work correctly for the non-embedded font case, since e.g. "Arial", "Arial,Bold", "Arial,Italic", and "Arial,BoldItalic" all map to "Helvetica" in the font.loadedName; hence the answer is unfortunately no.

@Snuffleupagus I probably wasn't very clear in my original request either.
What I labeled as the "original font names" (perhaps mistakenly) were actually the values I got from font.name (as per the commit I linked above).
The first file I attached (anguish-languish.pdf) doesn't have embedded fonts. However, the font names from font.name are Helvetica-Bold and Times-Roman, which is still better than Helvetica and Times.
For the other file, AAAAAA+LiberationSerif-Bold is also better than g_d0_f1.
Is the issue that "originalFontName" isn't an accurate descriptor? Would another property name be better? It would be great to get this information as the current fontName property loses some helpful information.
Thanks!

I wasn't talking about your specific example, but rather about the general case where your patch wouldn't do the right thing for a PDF document that contains (all of the) non-embedded fonts named "Arial", "Arial,Bold", "Arial,Italic", respectively "Arial,BoldItalic". As already mentioned they all map to "Helvetica" in the font.loadedName property, hence the only generally correct solution would be PR #10753 but as mentioned we're not at all keen on doing that.

However, the font names from font.name are Helvetica-Bold and Times-Roman, which is still better than Helvetica and Times.

We can't very well add a feature that only kind of works, as explained above, since that'll only lead to user confusion and support requests. Sorry, but there's unfortunately no great solution here that does the "right thing" without overhead for the default case.

That makes sense, thanks for your patience in explaining it.
I do have another proposal: getTextContent can take parameters; would it be acceptable to keep the current behaviour as-is, but to add a new parameter to add the original font name to the TextItems as per the referenced PR? That would allow those of us that would like this information to get it without increasing memory usage for those that don't.
Thanks!

I wasn't talking about your specific example, but rather about the general case where your patch wouldn't do the right thing for a PDF document that contains (all of the) non-embedded fonts named "Arial", "Arial,Bold", "Arial,Italic", respectively "Arial,BoldItalic". As already mentioned they all map to "Helvetica" in the font.loadedName property, hence the only generally correct solution would be PR #10753 but as mentioned we're not at all keen on doing that.

@Snuffleupagus Now that #15659 has made it so that this is no longer the case, would a PR to add the original font name to the TextStyle be considered?
Thanks!

@Snuffleupagus Thanks for your patience in explaining things.

I wasn't talking about your specific example, but rather about the general case where your patch wouldn't do the right thing for a PDF document that contains (all of the) non-embedded fonts named "Arial", "Arial,Bold", "Arial,Italic", respectively "Arial,BoldItalic". As already mentioned they all map to "Helvetica" in the font.loadedName property, hence the only generally correct solution would be PR #10753 but as mentioned we're not at all keen on doing that.

Now that #15659 has made it so that this is no longer the case, would a PR to add the original font name to the TextStyle be considered?
Thanks!

Now that #15659 has made it so that this is no longer the case, would a PR to add the original font name to the TextStyle be considered?

No, since all information should already be (indirectly) available; see e.g. https://github.com/mozilla/pdf.js/pull/15659/files#diff-bc84ed767f5905bdf8c544c715c08e9b751e7bb1db16c8182d9de515b56bc4b6R2269-R2272

@Snuffleupagus It works! Thanks so much for your help!
const { items, styles } = await pdfPage.getTextContent();

await pdfPage.getOperatorList();

for (const fontName in styles) {
  console.log('fontName=', fontName);
  console.log('originalFontName=', page.commonObjs.get(fontName).name);
  console.log();
}

fontName= g_d0_f6R
originalFontName= Helvetica-Bold

fontName= g_d0_f3R
originalFontName= Times-Roman

fontName= g_d0_f5R
originalFontName= Helvetica


KAFKA-3933: close deepIterator during log recovery

Avoids leaking native memory and hence crashing brokers on bootup due to
running out of memory.
Introduces kafka.common.ClosableIterator, which is an iterator that
can be closed, and changes the signature of
ByteBufferMessageSet.deepIterator to return it, then changes the
caller LogSegment to always close the iterator.
https://issues.apache.org/jira/browse/KAFKA-3933

Thanks for the PR. As discussed in the JIRA, we should also fix RecordsIterator (assuming that it suffers from the same issue as it looked to do when I checked). Also, we should call close in every usage, not just LogSegment IMO.

@ijuma if it's ok with you, I'd rather do the change to the client code in another PR, to keep changes relatively small and reviewable. Thoughts?

It's OK to do that part in a separate PR. Can you please file a JIRA for that then?

No problem. I'll finish up the code on this PR and then write that JIRA.

@ijuma I think I've fixed everything here. Note that some of the callers of this code rely on the iterator being a scala Iterator, so the code got a bit more complex, but nothing unreasonable I don't think.

@ijuma I found another issue that's related to brokers leaking GZIP memory, and I'd like to work it in here. Going to close until I have that piece in.


Better results screen

The current result screen is so dull. It doesn't contain any visual elements. It would be great to have some visual touch on the result screen, like a graph or something.
I think chart.js is a good idea for doing it, but I'm open to discussing other possible solutions.

How about using D3.js to show analytics of user after test.
D3.js is one of the best when it comes to showcasing data driven analytics.

@salmannotkhan
How about using D3.js to show analytics of user after test. D3.js is one of the best when it comes to showcasing data driven analytics.

d3 is a paid software, can you suggest some other open source library?

@salmannotkhan

How about using D3.js to show analytics of user after test. D3.js is one of the best when it comes to showcasing data driven analytics.

d3 is a paid software, can you suggest some other open source library?

d3 is open source and free.
https://github.com/d3


Gitea cannot be installed on posgresql schema different to public

Gitea version (or commit ref): 1.8.3 and 1.9.0-rc2
Git version: 2.22.0
Operating system: CentOS 7
Database (use [x]):

[x] PostgreSQL (11)
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] Not relevant
[ ] No

Log gist:

Description
If default schema for user is different of public (the best way), schema  is created on gitea on the user default schema.
On boot, the check for existing column id on table version is executed on schema public , then gitea try to add column but column already exist on user's default schema and crash.
Log :
SELECT column_name FROM INFORMATION_SCHEMA.COLUMNS WHERE table_schema = $1 AND table_name = $2 AND column_name = $3 [public version id]

I think there was issue for this already, there was even draft PR to fix this but it never got finished

duplicated with #5152


Crash when spawning a blueprint object on interaction with object

Hi Bit of a noob when it comes to working with unreal hoping someone can point me in the right direction with this.
I've set up a player ineteractive object, which includes the following python code:
def interact(self):
        self._debug("Interacting with python Object: %s" % self.Name)
        #chip_class = ue.find_class("BP_Isolinear_Chip_Command")
        a_iso_chip = ue.load_object(Blueprint, "/Game/Trek_Assets/Interior/Items/Isolinear_chips/BP_Isolinear_Chip_Command.BP_Isolinear_Chip_Command")
        a_iso_chip.set_actor_location(-103.0, -145.0, 96.0)
        self._debug("Actor Test: %s" % a_iso_chip)
        self.uobject.get_owner().Python_Reverse_Call("This Variable was returned by Python")

Note: _debug is a substitute for pythons print function.
My goal is to get the blueprint object to spawn in at a certain position, the rest of the code works fine until i try to use set_actor_location or
self.uobject.actor_spawn(a_iso_chip, FVector(-103.0, -145.0, 96.0), FRotator(0, 0, 0))
In both instances i'm getting hard crashes with no error logs or information to help me debug, im hoping im just missing something obvious which someone can point out or if i need to rethink my approach
Cheers

Hi, first of all actors must be spawned in the world. Otherwise you cannot set their properties. In addition to this, a Blueprint object, represent the "serialized asset", while instead you want the class represented by it:
a_iso_chip = ue.load_object(Blueprint, "/Game/Trek_Assets/Interior/Items/Isolinear_chips/BP_Isolinear_Chip_Command.BP_Isolinear_Chip_Command")
self.uobject.actor_spawn(a_iso_chip.GeneratedClass, FVector(-103.0, -145.0, 96.0), FRotator(0, 0, 0))

So, the key here is using the GeneratedClass property of the Blueprint class

I have added special notes for blueprint spawning: https://github.com/20tab/UnrealEnginePython/blob/master/README.md#spawning-notes
i will try to make the actor_spawn function smarter so it will recognize Blueprint class and will throw an exception

Thank you, thats worked perfectly now just what i needed


v1.8 backports 2021-08-10

#17000 -- config: Fix incorrect packet path with IPsec and endpoint routes (@pchaigno)
#17035 -- node-neigh: Wait instead of sleeping in unit tests (@aanm)

Once this PR is merged, you can update the PR labels via:
$ for pr in 17000 17035; do contrib/backporting/set-labels.py $pr done 1.8; done

test-backport-1.8

We've reverted https://github.com/cilium/cilium/pull/17000 because it was breaking IPsec connectivity tests on AKS, so we probably shouldn't backport it. I've just removed the labels from that PR. Sorry I wasn't to anticipate that :-(

We've reverted #17000 because it was breaking IPsec connectivity tests on AKS, so we probably shouldn't backport it. I've just removed the labels from that PR. Sorry I wasn't faster to anticipate that :-(

Dropped!

test-runtime


🚀 - Support Angular v18

Which ngx-ui-tour-* package is relevant to the feature request?
md-menu
Description
Add support for Angular v18

+1

+1

Support for Angular v18 is coming after ng-bootstrap releases their new version with Angular 18 support.

+1

Really appreciate this project and all the work by the maintainers. I'm sure you know but just in case not, https://github.com/ng-bootstrap/ng-bootstrap/releases/tag/17.0.0 is out =)

+1

v13 has been released. Check out the release announcement.


Need example of how to return > 500/2000 records

We're left dangling a bit at the end of this doc with:
If the table contains more than 500 records, the formula won't return record 501 or higher, even if it matches the criteria.

Please provide an example for how to return more records than the delegable limit. There are ways to do this, we need to see them here.
Rationale:

When a Sharepoint data source grows beyond 2000 records, the NEWEST records (2001th, 2002nd, etc.) are the ones dropped from the query, NOT the oldest ones.

Please provide some strategies at the end of this doc rather than a limitation.
Thank you

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

ID: 4a9e0bc1-6e4b-28ce-ba2f-00de8fcc6a86
Version Independent ID: c670fff1-17d4-abf1-b2df-1da9250e552c
Content: Understand delegation in a canvas app - Power Apps
Content Source: powerapps-docs/maker/canvas-apps/delegation-overview.md
Service: powerapps
GitHub Login: @lancedMicrosoft
Microsoft Alias: lanced

Please make use of the connectors and the respective functions when working with delegation.
For example, a scenario in this example can have the following formula in a gallery for Items property that only shows records created today:
Sort(Filter(Fruit,Created>Today()),Created,Descending)
The example elaborates on the concept and the effects of delegation. One formula for one connection such as SharePoint for a specific function in a delegation situation may not help in all possible scenarios of connector/function combination. However, an understanding of the delegation concept helps decide the best course of action for the app maker.
Additional help:

Power Apps Ideas forum
Power Apps Community forum
Help and support


Fix minor FTL formatting issues

About the PR

fixed some minor FTL formatting issues where comments were written as #stuff rather than # stuff
Why / Balance

It shows up as errors if you have the FTL extension, and that's kind of annoying.
Technical details

Media

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

no cl no fun

Checkfail is obviously completely unrelated

No files changed because i did this in the last upstream merge quietly... :')


Compile 5 times?

Can't I just compile VTK and VMTK once for the Python version I'll be using?

Yes, of course. The instructions in the README are for build and distribute the package.
But you can avoid to compile it, simply create a conda environment with specific python distribution and then install vmtk with
conda install -c vmtk/label/dev vmtk


Publish releases and branch tags to quay.io too

address  #1995

tested: https://ci.woodpecker-ci.org/repos/5586/pipeline/308/10


fixing missing declaration

fails to compile without this fix - seems like a simple omission during the recent refactor :)

Ah thank you!


WavPack support?

Is there any chance of supporting WavPack (.wv) playback? Would be useful for the.. soundboard lets call it... im making so its overall size is 2-3 times less.

Sorry, no plans at the moment. You can play AAC, MP3 and even FLAC on later versions of Windows so there are plenty of options for smaller file sizes


Zap created an on-chain transaction with a fee of less than 1 sat/byte

I chose the option for 1sat/byte, but the final transaction had a feerate of ~0.9sats/byte. This makes it very hard to propagate and now my funds are basically stuck. Good thing that it's a very small payment. I removed the transaction from my mempool, but LND still remembers it so I can't resend it. If you could also help me with the "abandoning" of the transaction on the LND side it would be great!
Here is the transaction in question:<PHONE_NUMBER>010199356c522d1af3eb0b7be3e7a9b96ea109198e76907447fb657e22c2397525ff00000000000000000001650700000000000022002063f29dbaeae5340566f8dede6ab48c619833c6e4da2b591b40cea041294d072e0247304402201c3d9e7d6facd74dab8720e9f3b972e8a5f2aed8de4a07d2930f731649f3a9920220387b5577e53006d378242d8aa2689f7d51d590fcbe1a5a99c16fe92d07e2c5ae01210336fb4944c5ff51957885d2796185ad51fa00d9eee932e980cd8f3d230facde730d210900

Zap version: Zap v0.5.2-beta
Operating System and version: Windows 10,

Fee rate is configurable. If you choose a low fee you can expect the transaction to take time to confirm, or even to never confirm depending on the state of the blockchain at the time.
Funds never get permanently "stuck". Transactions either confirm at some point in the future, or get expelled from the mempool.
This is by design and is unrelated to zap desktop. It's just how blockchains work.


filling up the custom button background

I am wanting to change the floating buttons to a custom 1 with a blue background. the image is round and is 1024 x 1024. When I place it into the code and run, the image is small and not filling up the default white background. How could I over come this?

You can use the KCFloatingActionButtonItem.buttonColor.
Below is simple example.
let item = KCFloatingActionButtonItem()
item.buttonColor = UIColor.blueColor()
item.title = "Custom item"
fab.addItem(item: item)

ok @kciter  the problem is if I change the background color, the icon itself is still small unlike the icons that come with the KFAB example project. Would this have to do with the icon not being in an asset set? I noticed the default ones are in a set of three inside an asset set called icon (1, @2x, and @3x).

@SirSamShaw
Can you give me your icons? I need re-produce.

yes thank you

KCFABItem's icon image size is 25x25 point. So, remove the your icons background color and shadow. And you should resize image to 25x25 point.

ah ok Thank you!, do I need to place them in an asset set or can I just
place them in by name directly in code after I resize them??
On Tue, Aug 16, 2016 at 9:38 PM, Lee Sun-Hyoup<EMAIL_ADDRESS>wrote:

KCFABItem's icon image size is 25x25 point. So, remove the your icons
background color and shadow. And you should resize image to 25x25 point.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kciter/KCFloatingActionButton/issues/61#issuecomment-240290235,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOLMafjs6qwprTjyER78LoOvwiEtdCOJks5qgmYHgaJpZM4JiTlq
.

@SamuelShaw
Once again,
KCFAB's item is already applied shadow and background color.
You can make using the design tool a transparent background and resize them.
Then, you place them in an asset set and add item in code.
var fab = KCFloatingActionButton()
fab.itemButtonColor = UIColor.blueColor() // Choose your color.
fab.addItem("title", icon: UIImage(named: "icName"))
// ...

ok, thank you!

Hi @kciter , @SirSamShaw  how can I change the main fab icon image ?


Fixed Export All selector

The export was not working anymore because of the new "Feedback" button in Roam Research.
This new selector fixes the problem.

The export has been working for me since the fix from https://github.com/everruler12/roam2github/commit/2009fecb67d186d1bdf16eeec4a577af8bb5c9d8

Oh I just realized the "Feedback" button only appears in the "help" database (https://roamresearch.com/#/app/help).
So this should fix the export of the "help" database only.

Yes, that's exactly what the https://github.com/everruler12/roam2github/commit/2009fecb67d186d1bdf16eeec4a577af8bb5c9d8 fix was for. I haven't had issues with exporting the Help graph since then. I'm wondering if maybe you had a fork that didn't get that update?

You're right, I wasn't properly updated. Sorry for that!

No worries! I appreciate you willing to help out


Is there way to delete last x neutral data?

Is there way to delete last x neutral data?

No sorry, there is no way to do that. You can delete all the trainings for a given action, but you cannot delete the last X trainings.


Fix Github action LUT visualization

The action seems to have problems when the PR is created from a fork.

Should be fixed now.


Installing Addons

The following link for Weave Net is no longer viable, and should be updated (e.g. pointing to the GitHub repo where it is now maintained, https://github.com/weaveworks/weave, which admittedly needs to be updated, but puts the onus on the project itself):

Weave Net provides networking and network policy, will carry on working on both sides of a network partition, and does not require an external database.

Google link-to-highlight: https://kubernetes.io/docs/concepts/cluster-administration/addons/#:~:text=public cloud environments.-,Weave Net,-provides networking and

Page related to the issue: https://kubernetes.io/docs/concepts/cluster-administration/addons
/language en

/retitle Broken link to "Weave Net" documentation in "Installing Addons" page
/kind bug

The suggestion to update the link from the outdated documentation website to the Weave Net GitHub repository (https://github.com/weaveworks/weave) seems like a sensible solution for now.

/triage accepted
/priority backlog

Additionally, upon reviewing the Weave Net project on GitHub, it appears there hasn't been any activity for quite some time (2+ years!). In the longer term, it might be worth considering removing the link if this continues to be the case.

I'd suggest to follow the maintained fork instead, now that there is one:
https://github.com/kubernetes/website/pull/45202#issuecomment-1976819974

I suggest we remove the line
https://github.com/weaveworks/weave hasn't had a commit in 2 years and Weave Works is shutting down

Closing this issue as it is resolved by PR #45445

/close


Add .env comment support and test

This adds support for
#TEST3=NOLOADENV
TEST2=LOADENV #TEST4=NOLOADENV
TEST=OVERLOADENV

Does dotenv support
TEST2=LOADENV#TEST4=NOLOADENV

I can work on this, but I'd have to recommend that this is only allowed if the value (LOADENV here) is quoted.
I also have another PR open; if you're ok with both, let me know and I can merge them together into a single one if it's easier.

should be linked to https://github.com/zztkm/vdotenv/issues/8

@nyx-litenite
Thanks for your PR!

I can work on this, but I'd have to recommend that this is only allowed if the value (LOADENV here) is quoted.

I also agree with this opinion.
I confirmed that this PR and # 9 are okay 👍
Can I merge at this time?
I am waiting for your contact!

You can merge them whenever you'd like. I don't you have any merge conflicts, but let me know if I can help with anything.

Actually, go ahead and merge the other PR but leave this one open. I'd like to leave this one open for the rest of the comment work. I'll change it to draft mode.

Merged into https://github.com/zztkm/vdotenv/pull/9 for ease of merge


Possibility to specify the getImageSource width/height?

Reason: accordingly with material design guidelines, toolbar icons should be 24dp (examples include transparent padding to became square).
I am using MaterialIcons.getImageSource(name, iconSize, iconColor) to generate image sources to create toolbar buttons.  The resulting toolbar buttons do not have the same width.
reference: https://material.io/guidelines/style/icons.html#icons-system-icons
Amazing project, btw.

Sounds like a solid feature, would you care doing a PR with this functionality?

I would be glad to do this PR as soon as I get back to a project using rnvi.

Closing, we'll look at a PR if one is created


test running oracle test cases on arm

Won't work because GHA Mac machines doesn't support nested virtualisation.


[T6A3][T13-C2] Aloisius Stephen

@aloisiusStephen  Some comments added. Please ack & close the PR after reading comments.

thanks for the comments!


Use SchemaV2, allow composite pk + multiple watermarks

Summary
Use SchemaV2 in the dbt adapter, which allows us to define composite primary keys and multiple watermarks. This code is not backward compatible to handle config input as a single string for both primary_key and watermark, but we can handle that if desired.
Test Plan
This could use a careful look and a recommended test strategy, but figured I'd get it out for review in the meantime.
For now I tested the composite pk via dbt run --select=events_count for:

fresh install, no existing resources
rerun model with the following scenarios:

no change

succeeds

no change except column renames

resources are destroyed + recreated

no schema change, no pk change, logic change (e.g., count(1) --> 2 * count(1)

resources are destroyed + recreated

no schema change, pk change

resources are destroyed + recreated

schema change

fails, same as main

I also tested watermarks (creating, modifying) and that worked as expected. Multiple watermarks are not supported by the Decodable API / Flink even though the schema v2 supports it (for forward-compatibility). If someone were to add multiple watermarks, you'll see this in the CLI

Decodable: InvalidRequest: {'timestamp': '2024-12-13T17:51:36.457+00:00', 'message': 'Invalid stream. Reason: Invalid schema: Multiple watermark definitions are not supported yet.'}

Hey @rob-apella, just wanted to check in on this and float an idea for next steps:
I think you were still planning to add some unit tests, and maybe make a few other tweaks, is that right? No rush from our side here, just want to make sure you're not blocked on me.
As for next steps: I think we'll be making some more changes to the config stuff soon to support some of your other use cases, and generally just bring the DBT adapter up to current WRT feature support. Would it make sense to  go ahead with what we have here as a pre-release for v2? That way you could have a published updated version to depend on but we could still reserve some time to make any other necessary breaking changes (if any) before calling v2 ready.
To your question about testing, I think if we can get CICD passing on this PR we'll be in decent shape for a pre-release. If there's anything we need to add in addition to any unit tests, we can figure that out before cutting a proper v2.
FWIW I don't think we need to maintain the backward compatibility for the changes you have here. I think it's just a major version bump and we can document the migration guide.
Let me know if this all works for you, or if you have an alternative you'd prefer.

I think you were still planning to add some unit tests, and maybe make a few other tweaks, is that right? No rush from our side here, just want to make sure you're not blocked on me.

yes - just need to carve out time. I don't have any other changes planned ... although I'll dig back through Slack and see if there was something I mentioned and forgot about. let me know if anything comes to mind on your end

Would it make sense to go ahead with what we have here as a pre-release for v2? That way you could have a published updated version to depend on but we could still reserve some time to make any other necessary breaking changes (if any) before calling v2 ready.

yes that sounds good. what's the mechanism for doing that? should I merge this into a different branch? if it's complicated, we can wait for the full V2 release with everything altogether, depending on your timeline for the other features

FWIW I don't think we need to maintain the backward compatibility for the changes you have here. I think it's just a major version bump and we can document the migration guide.

I think if we can get CICD passing on this PR we'll be in decent shape for a pre-release.

👍 to both


Using activate hook with promise

I've tried returning a promise object in activate hook, or assigning a promise object to activate hook directly, but neither worked.
Vue.component('example', {
  activate: function () {
    return ajax();
  }
})

Vue.component('example', {
  activate: ajax()
})

As I know promises are supported only in vue-routers hooks.
Please use forum and chat for questions.

As per https://github.com/vuejs/vue/issues/1169 ("The component will not be inserted until the user calls the passed-in callback, or return a promise."), I think it's a unimplemented feature, so I reported here.

Ah ok, didn't know :)

Ah, I forgot that issue mentioned a promise... it is not implemented (and not mentioned in the actual doc).  There are no other places that assume Promises in Vue.js core so introducing Promise handling for this particular case is probably not worth it, since you can just do activate (done) { ajax().then(done) }.


Failed to fetch https://packages.microsoft.com/ubuntu/14.04/prod/dists/trusty/main/binary-amd64/Packages  Hash Sum mismatch

Failed to fetch https://packages.microsoft.com/ubuntu/14.04/prod/dists/trusty/main/binary-amd64/Packages  Hash Sum mismatch
Occurs after installing this package and running apt-get update.
https://packages.microsoft.com/config/ubuntu/14.04/packages-microsoft-prod.deb
Started getting that error this morning while running chef. Did the same runs yesterday without issue. Not sure if something in the repo changed or what.

Seeing the same issue since yesterday

@dagood is it known problem? cc @leecow
Which version are you installing? .NET Core 2.1 / 2.2 / 3.0 Preview 4?

same here.
trying to install .net core 2.2

It certainly looks like the 14.04 Microsoft repo is broken. People are unable to get Powershell with the same error: https://github.com/PowerShell/PowerShell/issues/9424. It's also easy to repro in Docker.
Doc link for reference: https://dotnet.microsoft.com/download/linux-package-manager/ubuntu14-04/sdk-current. You don't even get as far as picking which SDK to install so 2.1 vs. 2.2 etc. doesn't matter.
Ubuntu 14.04 hit end of standard support just a couple days before this issue was filed--got to wonder if that's related.
@leecow can you reach out to Microsoft repo owners?

Also seeing the same issue, installing .NET Core 2.2 during apt-get update

I've reached out to the Microsoft repository admins via email about this issue, will update here with more info.

same here.

I also thought it was very coincidental that is started a few days before 14.04 goes EOL

The Microsoft repo admins are looking into it, no status yet.
ubuntu-distro-info tells me it's EOL 5 days ago, FWIW (but I wouldn't expect it to affect this):
#> ubuntu-distro-info --all -yeol -f | grep 14.04
Ubuntu 14.04 LTS "Trusty Tahr" -5
#> date -d 'now - 5 days'
Wed Apr 17 22:16:07 UTC 2019

They fixed it! I confirmed in an ubuntu:14.04 container. Thanks for filing the issue.

I've started seeing this issue again today on a Dockerfile that worked yesterday.
W: Failed to fetch https://packages.microsoft.com/ubuntu/14.04/prod/dists/trusty/main/binary-amd64/Packages  Hash Sum mismatch

Same here, issue happens again.

We're also experiencing this issue since approximately 5pm UTC yesterday although we're trying to install msodbcsql17 and ms-tools, the symptoms are the same. I have also tried the suggested trick of clearing /var/lib/apt/lists/* but there was no change to the situation.

Yeah... repros with a clean Docker container as well. Sending another mail to the Microsoft repo admins.

same here

same...

They fixed it, and I confirmed it works for me now. Thanks for the reports!

Instead of creating a new issue, I thought I should mention here that I started receiving the following error this morning when I attempted to reinstall dotnet-sdk-2.1 after sudo apt upgrade caused the same exact issue (which is why I uninstalled dotnet and reinstalled dotnet-sdk2.2 and dotnet-sdk2.1).
$ sudo apt install dotnet-sdk-2.1 -y
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following additional packages will be installed:
  aspnetcore-runtime-2.1 dotnet-hostfxr-2.1 dotnet-runtime-2.1 dotnet-runtime-deps-2.1
The following NEW packages will be installed:
  aspnetcore-runtime-2.1 dotnet-hostfxr-2.1 dotnet-runtime-2.1 dotnet-runtime-deps-2.1 dotnet-sdk-2.1
0 upgraded, 5 newly installed, 0 to remove and 0 not upgraded.
Need to get 134 MB of archives.
After this operation, 387 MB of additional disk space will be used.
Err:1 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 dotnet-runtime-deps-2.1 amd64 2.1.12-1
  404  Not Found [IP: <IP_ADDRESS> 443]
Err:2 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 dotnet-hostfxr-2.1 amd64 2.1.12-1
  404  Not Found [IP: <IP_ADDRESS> 443]
Err:3 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 dotnet-runtime-2.1 amd64 2.1.12-1
  404  Not Found [IP: <IP_ADDRESS> 443]
Err:4 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 aspnetcore-runtime-2.1 amd64 2.1.12-1
  404  Not Found [IP: <IP_ADDRESS> 443]
Err:5 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 dotnet-sdk-2.1 amd64 2.1.701-1
  404  Not Found [IP: <IP_ADDRESS> 443]
E: Failed to fetch https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/d/dotnet-runtime-deps-2.1/dotnet-runtime-deps-2.1.12-x64.deb  404  Not Found [IP: <IP_ADDRESS> 443]
E: Failed to fetch https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/d/dotnet-hostfxr-2.1/dotnet-hostfxr-2.1.12-x64.deb  404  Not Found [IP: <IP_ADDRESS> 443]
E: Failed to fetch https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/d/dotnet-runtime-2.1/dotnet-runtime-2.1.12-x64.deb  404  Not Found [IP: <IP_ADDRESS> 443]
E: Failed to fetch https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/a/aspnetcore-runtime-2.1/aspnetcore-runtime-2.1.12-x64.deb  404  Not Found [IP: <IP_ADDRESS> 443]
E: Failed to fetch https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/d/dotnet-sdk-2.1/dotnet-sdk-2.1.701-x64.deb  404  Not Found [IP: <IP_ADDRESS> 443]
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?

@kimbaudi The error you're receiving is also different, 404  Not Found rather than Hash Sum mismatch. Can you open a new issue with the rest of the commands and output too? (sudo apt-get update in particular.)

@dagood yes, I just submitted issue 3010

I'm also experiencing this issue, but with Ubuntu 18.04 running dotnet 2.2.401.
Here's the output of a sudo apt-get update
rui@localhost:~$ sudo apt-get update
[sudo] password for rui:
Hit:1 http://pt.archive.ubuntu.com/ubuntu bionic InRelease
Hit:2 http://pt.archive.ubuntu.com/ubuntu bionic-updates InRelease
Hit:3 http://pt.archive.ubuntu.com/ubuntu bionic-backports InRelease
Hit:4 http://ppa.launchpad.net/ansible/ansible/ubuntu bionic InRelease
Hit:5 http://security.ubuntu.com/ubuntu bionic-security InRelease
Hit:7 http://packages.microsoft.com/repos/vscode stable InRelease
Hit:8 https://download.docker.com/linux/ubuntu bionic InRelease
Get:9 https://packages.microsoft.com/ubuntu/18.04/prod bionic InRelease [3226 B]
Hit:6 https://packages.cloud.google.com/apt kubernetes-xenial InRelease
Hit:10 https://packages.gitlab.com/runner/gitlab-runner/ubuntu bionic InRelease
Get:11 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 Packages [66,6 kB]
Err:11 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 Packages
  File has unexpected size (66537 != 66619). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
  Hashes of expected file:
   - Filesize:66619 [weak]
   - SHA512:4754fea65f57f35fc52e332a6d5cddaf65ea00bbd9e9f428d3a4325c1ac26856190ae82022cdb7a41bf1337e5c7fb1a429933bae9817bacaa8a5186c2aae9f1b
   - SHA256:f971467de90380c9920956be80642d7f6eb0e75042822ea4537d9f5190e13d9f
   - SHA1:0f8e0e8ac649a20ef2029c9f4ffc6bc1ef559257 [weak]
   - MD5Sum:8aa7d0d41b74ea8b84a434a726ea92fa [weak]
  Release file created at: Tue, 20 Aug 2019 22:43:21 +0000
Fetched 3226 B in 2s (1475 B/s)
Reading package lists... Done
E: Failed to fetch https://packages.microsoft.com/ubuntu/18.04/prod/dists/bionic/main/binary-amd64/Packages.bz2  File has unexpected size (66537 != 66619). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
   Hashes of expected file:
    - Filesize:66619 [weak]
    - SHA512:4754fea65f57f35fc52e332a6d5cddaf65ea00bbd9e9f428d3a4325c1ac26856190ae82022cdb7a41bf1337e5c7fb1a429933bae9817bacaa8a5186c2aae9f1b
    - SHA256:f971467de90380c9920956be80642d7f6eb0e75042822ea4537d9f5190e13d9f
    - SHA1:0f8e0e8ac649a20ef2029c9f4ffc6bc1ef559257 [weak]
    - MD5Sum:8aa7d0d41b74ea8b84a434a726ea92fa [weak]
   Release file created at: Tue, 20 Aug 2019 22:43:21 +0000
E: Some index files failed to download. They have been ignored, or old ones used instead.

I'm also experiencing this issue, but with Ubuntu 18.04 running dotnet 2.2.401.
Here's the output of a sudo apt-get update
rui@localhost:~$ sudo apt-get update
[sudo] password for rui:
Hit:1 http://pt.archive.ubuntu.com/ubuntu bionic InRelease
Hit:2 http://pt.archive.ubuntu.com/ubuntu bionic-updates InRelease
Hit:3 http://pt.archive.ubuntu.com/ubuntu bionic-backports InRelease
Hit:4 http://ppa.launchpad.net/ansible/ansible/ubuntu bionic InRelease
Hit:5 http://security.ubuntu.com/ubuntu bionic-security InRelease
Hit:7 http://packages.microsoft.com/repos/vscode stable InRelease
Hit:8 https://download.docker.com/linux/ubuntu bionic InRelease
Get:9 https://packages.microsoft.com/ubuntu/18.04/prod bionic InRelease [3226 B]
Hit:6 https://packages.cloud.google.com/apt kubernetes-xenial InRelease
Hit:10 https://packages.gitlab.com/runner/gitlab-runner/ubuntu bionic InRelease
Get:11 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 Packages [66,6 kB]
Err:11 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 Packages
  File has unexpected size (66537 != 66619). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
  Hashes of expected file:
   - Filesize:66619 [weak]
   - SHA512:4754fea65f57f35fc52e332a6d5cddaf65ea00bbd9e9f428d3a4325c1ac26856190ae82022cdb7a41bf1337e5c7fb1a429933bae9817bacaa8a5186c2aae9f1b
   - SHA256:f971467de90380c9920956be80642d7f6eb0e75042822ea4537d9f5190e13d9f
   - SHA1:0f8e0e8ac649a20ef2029c9f4ffc6bc1ef559257 [weak]
   - MD5Sum:8aa7d0d41b74ea8b84a434a726ea92fa [weak]
  Release file created at: Tue, 20 Aug 2019 22:43:21 +0000
Fetched 3226 B in 2s (1475 B/s)
Reading package lists... Done
E: Failed to fetch https://packages.microsoft.com/ubuntu/18.04/prod/dists/bionic/main/binary-amd64/Packages.bz2  File has unexpected size (66537 != 66619). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
   Hashes of expected file:
    - Filesize:66619 [weak]
    - SHA512:4754fea65f57f35fc52e332a6d5cddaf65ea00bbd9e9f428d3a4325c1ac26856190ae82022cdb7a41bf1337e5c7fb1a429933bae9817bacaa8a5186c2aae9f1b
    - SHA256:f971467de90380c9920956be80642d7f6eb0e75042822ea4537d9f5190e13d9f
    - SHA1:0f8e0e8ac649a20ef2029c9f4ffc6bc1ef559257 [weak]
    - MD5Sum:8aa7d0d41b74ea8b84a434a726ea92fa [weak]
   Release file created at: Tue, 20 Aug 2019 22:43:21 +0000
E: Some index files failed to download. They have been ignored, or old ones used instead.

same for me

Yep happening again.

Being affected by the same issue.

Being affected by the same issue here

Being affected by the same issue here

can confirm dealing with the same here.

I have the same issue trying update repositories on Ubuntu 18.04.

#3221

Ubuntu 18.04 cannot update
Hit:1 http://mirrors.cloud.aliyuncs.com/ubuntu bionic InRelease
Hit:2 http://mirrors.cloud.aliyuncs.com/ubuntu bionic-updates InRelease
Hit:3 http://mirrors.cloud.aliyuncs.com/ubuntu bionic-security InRelease
Get:4 https://packages.microsoft.com/ubuntu/18.04/prod bionic InRelease [3,226 B]
Hit:5 https://download.docker.com/linux/ubuntu bionic InRelease
Get:6 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 Packages [66.6 kB]
Err:6 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 Packages
File has unexpected size (66537 != 66619). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
Hashes of expected file:

Filesize:66619 [weak]
SHA512:4754fea65f57f35fc52e332a6d5cddaf65ea00bbd9e9f428d3a4325c1ac26856190ae82022cdb7a41bf1337e5c7fb1a429933bae9817bacaa8a5186c2aae9f1b
SHA256:f971467de90380c9920956be80642d7f6eb0e75042822ea4537d9f5190e13d9f
SHA1:0f8e0e8ac649a20ef2029c9f4ffc6bc1ef559257 [weak]
MD5Sum:8aa7d0d41b74ea8b84a434a726ea92fa [weak]
Release file created at: Tue, 20 Aug 2019 22:43:21 +0000
Hit:7 http://archive.ubuntu.com/ubuntu bionic InRelease
Hit:8 http://ppa.launchpad.net/certbot/certbot/ubuntu bionic InRelease
Reading package lists... Done
E: Failed to fetch https://packages.microsoft.com/ubuntu/18.04/prod/dists/bionic/main/binary-amd64/Packages.bz2  File has unexpected size (66537 != 66619). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
Hashes of expected file:
- Filesize:66619 [weak]
- SHA512:4754fea65f57f35fc52e332a6d5cddaf65ea00bbd9e9f428d3a4325c1ac26856190ae82022cdb7a41bf1337e5c7fb1a429933bae9817bacaa8a5186c2aae9f1b
- SHA256:f971467de90380c9920956be80642d7f6eb0e75042822ea4537d9f5190e13d9f
- SHA1:0f8e0e8ac649a20ef2029c9f4ffc6bc1ef559257 [weak]
- MD5Sum:8aa7d0d41b74ea8b84a434a726ea92fa [weak]

I have same problem
Hit:1 http://kg.archive.ubuntu.com/ubuntu bionic InRelease
Ign:2 http://dl.google.com/linux/chrome/deb stable InRelease
Hit:3 http://kg.archive.ubuntu.com/ubuntu bionic-updates InRelease
Hit:4 http://dl.google.com/linux/chrome/deb stable Release
Hit:5 http://kg.archive.ubuntu.com/ubuntu bionic-backports InRelease
Hit:6 http://security.ubuntu.com/ubuntu bionic-security InRelease
Hit:7 https://cloud.r-project.org/bin/linux/ubuntu bionic-cran35/ InRelease
Get:8 https://packages.microsoft.com/ubuntu/18.04/prod bionic InRelease [3,226 B]
Get:10 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 Packages [66.6 kB]
Err:10 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 Packages
File has unexpected size (66537 != 66619). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
Hashes of expected file:

Filesize:66619 [weak]
SHA512:4754fea65f57f35fc52e332a6d5cddaf65ea00bbd9e9f428d3a4325c1ac26856190ae82022cdb7a41bf1337e5c7fb1a429933bae9817bacaa8a5186c2aae9f1b
SHA256:f971467de90380c9920956be80642d7f6eb0e75042822ea4537d9f5190e13d9f
SHA1:0f8e0e8ac649a20ef2029c9f4ffc6bc1ef559257 [weak]
MD5Sum:8aa7d0d41b74ea8b84a434a726ea92fa [weak]
Release file created at: Tue, 20 Aug 2019 22:43:21 +0000
Reading package lists... Done
E: Failed to fetch https://packages.microsoft.com/ubuntu/18.04/prod/dists/bionic/main/binary-amd64/Packages.bz2  File has unexpected size (66537 != 66619). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
Hashes of expected file:
- Filesize:66619 [weak]
- SHA512:4754fea65f57f35fc52e332a6d5cddaf65ea00bbd9e9f428d3a4325c1ac26856190ae82022cdb7a41bf1337e5c7fb1a429933bae9817bacaa8a5186c2aae9f1b
- SHA256:f971467de90380c9920956be80642d7f6eb0e75042822ea4537d9f5190e13d9f
- SHA1:0f8e0e8ac649a20ef2029c9f4ffc6bc1ef559257 [weak]
- MD5Sum:8aa7d0d41b74ea8b84a434a726ea92fa [weak]
Release file created at: Tue, 20 Aug 2019 22:43:21 +0000
E: Some index files failed to download. They have been ignored, or old ones used instead.

This (or something very similar) is definitely happening again! The current issue is tracked by https://github.com/dotnet/core/issues/3221.

Looks like this is happening again on our 16.04 builds on Ubuntu
lists...
4763
E: Failed to fetch https://packages.microsoft.com/ubuntu/16.04/prod/dists/xenial/main/binary-amd64/Packages.gz  Hash Sum mismatch
4764
E: Some index files failed to download. They have been ignored, or old ones used instead.
4765
The command '/bin/sh -c apt-get --yes update && apt-get --yes

@harshavardhana Yep! The current outage is tracked over at https://github.com/dotnet/core/issues/3988.
I'm going to lock this issue to encourage everyone to file new issues rather than add comments to this one, since it's happened a couple times now. Because the thread is closed, new comments might not be seen by the right people in the future. A new issue is more likely to get noticed. There's nothing wrong with posting relevant comments to old issues, but I want to make sure outages are seen as soon as possible. Thanks!


rasa_core.exceptions.UnsupportedDialogueModelError: Either retrain the model, or run withan older version. Model version: 0.0.0 Instance version: 0.12.3 Minimal compatible version: 0.12.0

I wrote the following command for running Rasa Core.
>python -m rasa_core.run -d framework/models/dialogue -u framework/models/nlu/default/model_20190121-131223 --endpoints framework/endpoints.yml
Running this command is generating the following error.
rasa_core.exceptions.UnsupportedDialogueModelError: The model version is to old to be loaded by this Rasa Core instance. Either retrain the model, or run withan older version. Model version: 0.0.0 Instance version: 0.12.3 Minimal compatible version: 0.12.0
Content of endpoints.yml file
action_endpoint:
    url: "http://localhost:5055/webhook/"

Rasa Core Version: 0.12.3
Rasa NLU Version: 0.13.8
Python 3.6.5
Since my model version is being showed 0.0.0, do I need to retrain it? If so, then how to retrain it with a higher version?

Traceback (most recent call last):
File "run.py", line 6, in
interpreter = NaturalLanguageInterpreter.create('/home/anil/Documents/slackbot/models/dialogue')
File "/home/anil/.local/lib/python3.6/site-packages/rasa_core/interpreter.py", line 34, in create
return RasaNLUInterpreter(model_directory=obj)
File "/home/anil/.local/lib/python3.6/site-packages/rasa_core/interpreter.py", line 234, in init
self._load_interpreter()
File "/home/anil/.local/lib/python3.6/site-packages/rasa_core/interpreter.py", line 257, in _load_interpreter
self.interpreter = Interpreter.load(self.model_directory)
File "/home/anil/.local/lib/python3.6/site-packages/rasa_nlu/model.py", line 301, in load
Interpreter.ensure_model_compatibility(model_metadata)
File "/home/anil/.local/lib/python3.6/site-packages/rasa_nlu/model.py", line 280, in ensure_model_compatibility
"".format(model_version, rasa_nlu.version))
rasa_nlu.model.UnsupportedModelError: The model version is to old to be loaded by this Rasa NLU instance. Either retrain the model, or run withan older version. Model version: 0.0.0 Instance version: 0.15.1

I'm having the same issue too!


Rider show page version .9

This is the initial HTML for a new rider page. It will likely need some edits and updates and changes once populated with live data.

Hey pals! I made a few updates:

Keep the ability to click a rider's name to see their location on the map (without immediately navigating away)
Update the message button to navigate to the rider's conversation
Populate the rider's campaign info with some data (haven't touched the campaign history/schedule stuff yet, and also can likely be a lot cleaner 💅)


document how to use startproject/app templates

fixes #147

Thanks. Rather than having two sections ("manual way" and "template way"), I tried to integrate the two (while also adding several subsections). Please take a look.

LGTM! Thanks


add-next-steps-resources-and-guides-for-imagine-cup

Added detailed resources and step-by-step guides for the Next Steps section, including Imagine Cup registration, Microsoft Startups Founders Hub, Discord, and the Student Innovator Series.

Hi @alischumacher
Would you please take a look at the changes made so far? Thanks!


Revert this back to the hardcoded version from change in SAK-40634

Relates to #6003

Whoops wrong repo! ;)

No worries, tbh I'm happy to see you again in GitHub ;)


chore: Format welcome message with emojis

Related Issues/PRs

None
What changes are proposed in this pull request?
Editing the welcome message posted on new PRs/Issues
How is this patch tested?
Used markdown to preview the text
Does this PR change any dependencies?

[x] No. You can skip this section.
[ ] Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

[x] No. You can skip this section.
[ ] Yes. Make sure you have added samples following below steps.

Find the corresponding markdown file for your new feature in website/docs/documentation folder.
Make sure you choose the correct class estimators/transformers and namespace.
Follow the pattern in markdown file and add another section for your new API, including pyspark, scala (and .NET potentially) samples.
Make sure the DocTable points to correct API link.
Navigate to website folder, and run yarn run start to make sure the website renders correctly.
Don't forget to add <!--pytest-codeblocks:cont--> before each python code blocks to enable auto-tests for python samples.
Make sure the WebsiteSamplesTests job pass in the pipeline.

AB#1893152

/azp run


Added Fiori Content

SAP Fiori free book added.
What does this PR do?
Add resource
For resources
Description
Why is this valuable (or not)?
   Abap to fiori resources.Further career development in sap fiori.

How do we know it's really free?
   The SapPss git hub link is added.The publisher of the book.

For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

I don't hink FIORI is a subject - does it go with ABAP?
Be sure to check the free-programming-books-lint output for linter errors. Alphabetical order and spacing are the most common problems.

this repo is for the source code examples, not for the book itself, which is not free


nodemon changes retriggers client webpack recompilation even though only server files updated

Hi,
I see package.json does nodemon exec babel-node bin/server.js
Following the code, that will trigger server/main.js which retriggers the webpack compiler for the client.
When updating a server file is it necessary to retrigger the webpack compiler for the client? I would think no since if I am updating the server folder, that has no effect on the client code and that could take a while if the client code is large.
Do you have this problem as well? (I am asking because my app is modelled after yours, but not exactly identical.). From the code I mentioned above however it does look like you would have the issue I mentioned above...
Any suggestions on fixing?

It's because nodemon.json this file I sepcify
{   "verbose": false,   "ignore": ["dist", "coverage", "tests", "src"] }

I think you misunderstand. The folder server is not ignored in your nodemon.json. Therefore any changes to server will trigger nodemon to restart the app. That triggers the webpack configs to be recompiled. However since webpack was not touched on your changes in the server folder, it doesn't make sense to recompile them. You can skip that step and only have nodemon restart your server. This is relevant for example if you are adding POST routes or updating middleware in your server folder, etc. Basically anything you are doing that doesn't touch webpack configs shouldn't have them recompile

Now I understand. It's a bug. I'll fix it

Hey @bodyno, still interested in fixing this? Otherwise let's close this 🙂 Thanks!

Please don't close if not fixed. It is a pretty crucial aspect of a universal rendering app.

@terencechow then fix it 🙂


base sepolia staging

Done in https://github.com/smartcontractkit/functions-hardhat-starter-kit/pull/209


poly-to-standard: ensure modular coefficient arthimetic in polymul

Ensures that the coefficent arithmetic is done mod the coefficient
modulus during accumulation in polynomial multiplication
Allows for the output of polymul to retain its original element
bitsize, and further, reduce the number of bits used in the divMod
operation after

Resolves #201

After applying the current changes I unfortunately fail all of the tests in polynomial/runner/* except lower_4_mul.mlir. This is because there is still a bufferization.clone call in the lowered llvm code.
From inspection, the difference (shown below) in these test cases is that lower_4_mul is the only one that requires a truncation at the end of the respective divMul. For all other testcases, this appears to be folded away when invoking --polynomial-to-standard because it is truncating from and to the same type.
In the previous implementation, the truncate is never folded away because we are still operating on the naivePolymulTensorType. So I suspect the error is a result of this folding behaviour, but I don't understand why this might be the case. Possibly, we clone the memory to do the truncate operation and then the truncate operation is folded away and so the cloned memory remains as dead code, then we don't run an applicable canonicalization that removes it as dead code? But that would be strange.
Would appreciate some pointers to or reasons as to why this might be the case.
Output logs: https://gist.github.com/inbelic/3a522181be6021409cb65a71958ebaf8

Excellent start! What strikes me as most curious is the remaining unrealized_conversion_cast op, which suggests some type did not lower properly.
%109 = builtin.unrealized_conversion_cast %108 : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)> to memref<12xi32, strided<[?], offset: ?>>

This first occurs after the --lower-to-llvm pass, and it suggests something that should have remained 32-bit was lowered to 64-bit, and hence it is incompatible with the subsequent lowering.
I suspect this is a bug upstream, but I want to take some more time to read through your code to try to figure out why this is happening. I may not have time to do that for a few days because I'm in the middle of a conference.
A (non-minimal) reproduction mlir: https://gist.github.com/j2kun/8d7082604ba93b8988ee0181e8d76f84
Run with
bazel run //tools:heir-opt -- --expand-strided-metadata --convert-to-llvm $PWD/test.mlir

Output:
   %239 = builtin.unrealized_conversion_cast %238 : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)> to memref<12xi32, strided<[?], offset: ?>>
    %241 = builtin.unrealized_conversion_cast %240 : memref<12xi32, strided<[?], offset: ?>> to !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>

Hm interesting. Such a lowering error could definitely be a consequence of the changes. I will take a closer look with that in mind.
Of course, no rush. Thanks for the quick response :)

I still don't have an answer, but I have a more minimal reproduction:
func.func @__heir_poly_mod_4294967296_1_x12(%arg0: tensor<23xi32>) -> tensor<12xi32> {
  %0 = polynomial.from_tensor %arg0 : tensor<23xi32> -> !polynomial.polynomial<<cmod=4294967296, ideal=#polynomial.polynomial<-1 + x**23>>>
  %1 = polynomial.add (%0, %0) : !polynomial.polynomial<<cmod=4294967296, ideal=#polynomial.polynomial<-1 + x**23>>>
  %2 = polynomial.to_tensor %1 : !polynomial.polynomial<<cmod=4294967296, ideal=#polynomial.polynomial<-1 + x**23>>> -> tensor<23xi32>
  %extracted_slice = tensor.extract_slice %2[0] [12] [1] : tensor<23xi32> to tensor<12xi32>
  return %extracted_slice : tensor<12xi32>
}

Which lowers via polynomial-to-standard to
module {
  func.func @__heir_poly_mod_4294967296_1_x12(%arg0: tensor<23xi32>) -> tensor<12xi32> {
    %0 = arith.addi %arg0, %arg0 : tensor<23xi32>
    %extracted_slice = tensor.extract_slice %0[0] [12] [1] : tensor<23xi32> to tensor<12xi32>
    return %extracted_slice : tensor<12xi32>
  }
}

Which then lowers via the rest of the pipeline to
module {
  llvm.func @__heir_poly_mod_4294967296_1_x12(%arg0: !llvm.ptr, %arg1: !llvm.ptr, %arg2: i64, %arg3: i64, %arg4: i64) -> !llvm.struc
t<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)> {
    %0 = llvm.mlir.constant(12 : index) : i64
    %1 = llvm.mlir.constant(0 : index) : i64
    %2 = llvm.mlir.constant(23 : index) : i64
    %3 = llvm.mlir.constant(1 : index) : i64
    llvm.br ^bb1(%1 : i64)
  ^bb1(%4: i64):  // 2 preds: ^bb0, ^bb2
    %5 = llvm.icmp "slt" %4, %2 : i64
    llvm.cond_br %5, ^bb2, ^bb3
  ^bb2:  // pred: ^bb1
    %6 = llvm.getelementptr %arg1[%arg2] : (!llvm.ptr, i64) -> !llvm.ptr, i32
    %7 = llvm.mul %4, %arg4  : i64
    %8 = llvm.getelementptr %6[%7] : (!llvm.ptr, i64) -> !llvm.ptr, i32
    %9 = llvm.load %8 : !llvm.ptr -> i32
    %10 = llvm.add %9, %9  : i32
    llvm.store %10, %8 : i32, !llvm.ptr
    %11 = llvm.add %4, %3  : i64
    llvm.br ^bb1(%11 : i64)
  ^bb3:  // pred: ^bb1
    %12 = llvm.mlir.undef : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
    %13 = llvm.insertvalue %arg0, %12[0] : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
    %14 = llvm.insertvalue %arg1, %13[1] : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
    %15 = llvm.insertvalue %arg2, %14[2] : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
    %16 = llvm.insertvalue %0, %15[3, 0] : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
    %17 = llvm.insertvalue %arg4, %16[4, 0] : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
    %18 = builtin.unrealized_conversion_cast %17 : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)> to memref<12xi32, s
trided<[?], offset: ?>>
    %19 = bufferization.clone %18 : memref<12xi32, strided<[?], offset: ?>> to memref<12xi32, strided<[?], offset: ?>>
    %20 = builtin.unrealized_conversion_cast %19 : memref<12xi32, strided<[?], offset: ?>> to !llvm.struct<(ptr, ptr, i64, array<1 x
 i64>, array<1 x i64>)>
    llvm.return %20 : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
  }
}

Please try applying this patch to your PR (git apply < file.patch should work).
I think it will be sufficient for this PR, though I'm not sure if this is exactly the right place to put this pass in the pipeline. lower_mul_0.mlir passes with this applied on top of your changes.
From<PHONE_NUMBER>110df2c61b1c571cc5798b48daf1bb Mon Sep 17 00:00:00 2001
From: Jeremy Kun<EMAIL_ADDRESS>Date: Sat, 23 Mar 2024 06:56:33 -0700
Subject: [PATCH] Add bufferization to memref pass to poly pipeline

---
 tools/BUILD        | 1 +
 tools/heir-opt.cpp | 2 ++
 2 files changed, 3 insertions(+)

diff --git a/tools/BUILD b/tools/BUILD
index fd5b8e52..df1e9b49 100644
--- a/tools/BUILD
+++ b/tools/BUILD
@@ -72,6 +72,7 @@ cc_binary(
         "@llvm-project//mlir:ArithDialect",
         "@llvm-project//mlir:ArithToLLVM",
         "@llvm-project//mlir:ArithTransforms",
+        "@llvm-project//mlir:BufferizationToMemRef",
         "@llvm-project//mlir:BufferizationTransforms",
         "@llvm-project//mlir:ControlFlowToLLVM",
         "@llvm-project//mlir:ConvertToLLVM",
diff --git a/tools/heir-opt.cpp b/tools/heir-opt.cpp
index a7e0ec0b..5ecb6989 100644
--- a/tools/heir-opt.cpp
+++ b/tools/heir-opt.cpp
@@ -34,6 +34,7 @@
 #include "llvm/include/llvm/Support/raw_ostream.h"  // from @llvm-project
 #include "mlir/include/mlir/Conversion/AffineToStandard/AffineToStandard.h"  // from @llvm-project
 #include "mlir/include/mlir/Conversion/ArithToLLVM/ArithToLLVM.h"  // from @llvm-project
+#include "mlir/include/mlir/Conversion/BufferizationToMemRef/BufferizationToMemRef.h"  // from @llvm-project
 #include "mlir/include/mlir/Conversion/ConvertToLLVM/ToLLVMPass.h"  // from @llvm-project
 #include "mlir/include/mlir/Conversion/IndexToLLVM/IndexToLLVM.h"  // from @llvm-project
 #include "mlir/include/mlir/Conversion/MemRefToLLVM/MemRefToLLVM.h"  // from @llvm-project
@@ -152,6 +153,7 @@ void polynomialToLLVMPipelineBuilder(OpPassManager &manager) {
   // But lowering to loops also re-introduces affine.apply, so re-lower that
   manager.addNestedPass<FuncOp>(createConvertLinalgToLoopsPass());
   manager.addPass(createLowerAffinePass());
+  manager.addPass(createBufferizationToMemRefPass());

   // Cleanup
   manager.addPass(createCanonicalizerPass());
--
<IP_ADDRESS>6.g6e790dbe36-goog

BTW, I think it's fine to leave the bufferization patch in this PR, so that the change is atomic with the change that caused its need.

Awesome. The patch did the trick and there is the expected behaviour. Okay sounds good I will reword the commit so you are the author and with your subject.

I will reword the commit so you are the author and with your subject.

No need to bother with that. It will all get squashed anyway before merging (see https://heir.dev/docs/contributing/#pull-request-review-flow)


Enhancement: Refactor code to use a single class

Current script implemented an anonymous class, resulting in 2 binary files when compiling. We want to try to get this down to one class only, so that compiling results in just one file.

Spent some time researching this. There isn't a way that I can see.


issue with JEI Compat

Issue description
when i try to see what certain recipes from the new blocks that the mod adds, it doesnt show the recipe so idk how to make things for the other machines
Steps to reproduce
No response
Minecraft version
1.21.1 (Latest)
Fabric loader version
0.16.7
Fabric API version
0.107.0+1.21.1
Oritech version
0.10.3
Other relevant versions
No response
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
No response

Oritech only supports EMI and REI for now. So for older version, you have to use them.
Hmm, I remember I heard of a universal third-party api that can support EMI, JEI and REI at the same time. Wait a minute.

That's it! Found it: TlaApi

Hi, JEI support is currently not available, as mentioned by @OakPlatinum. IIRC EMI already has JEI support, so just use REI or EMI for now.


Change .perl to .raku in examples

The problem
As part of the #3046 , .raku has superseded .perl. The latter still works perfectly (although it will be deprecated in the next vesion), but it would probably be better to stop using it in examples.
Suggestions
List below examples that use it and start changing it.

[x] Shell environment variables used by Raku
[x] class Set
[x] class List
[x] class Code
[x] role Iterable
[x] class Metamodel::ClassHOW
[x] role Metamodel::MethodContainer
[x] role Metamodel::MultipleInheritance
[x] class Method
[x] class Routine
[x] class Range
[x] class Signature
[x] class Any
[x] class Scalar
[x] class Attribute
[x] class Complex
[x] class Junction
[x] class Date
[x] class Int
[ ] class Mu
[ ] class Rat
[x] class Slip
[x] class Str
[x] role Buf
[x] class Proc
[x] class Backtrace
[ ] class Array
[ ] role Numeric
[x] class IO::Spec::Unix
[x] class IO::Spec::Win32
[x] class IO::CatHandle
[x] class IO::Path
[x] class IO::Path::Cygwin
[x] class IO::Path::Unix
[ ] class IO::Path::Win32
[x] class IO::Path::QNX
[x] class IO::Handle
[ ] class Pair
[x] class FatRat
[ ] class Seq
[x] class BagHash
[ ] class Nil
[x] class Bag
[x] role Rational
[ ] class CallFrame
[x] class SetHash
[x] class Cool
[ ] module Test
[x] role Iterator
[x] Lists, sequences, and arrays
[x] Functions
[x] Community
[x] Modules
[x] Type system
[x] Pod6
[x] Experimental features
[x] Control flow
[x] Syntax
[x] Glossary
[x] Raku native types
[x] Perl to Raku guide - functions
[x] Subscripts
[x] FAQ
[x] Iterating
[x] Ruby to Raku - nutshell
[x] Hashes and maps
[x] Perl to Raku guide - in a nutshell
[x] Containers
[x] Classes and objects
[x] Input/Output the definitive guide
[x] Numerics
[x] Grammar tutorial
[x] Regexes

FYI there is no occurrence of perl method in Shell environment variables used by Raku

Marking 'Experimental features' as complete since there's nothing to do.

If you have a unix-y shell, this should work to find any remaining failing pod6 files:
TEST_FILES=$(grep -lr '\.perl' doc) prove -e raku xt/examples-compilation.t

What we don't have a test for is that we have a 'method raku' section of a document (at least for those that have a perl method)

I think there is no occurrence of perl method in Shell environment variables used by Raku

Probably hit the ".perl6", safe to ignore.

False positive for class Int.

In class Mu the description of the raku method applied to type objects mentions the perl method. I was considering changing perl to raku in this description but it accuretly corresponds to rakudo's current implementation of the raku method. So I kept these occurences of the method name perl while submitting #3385. Should it be otherwise?

@stoned I'm seeing this after commenting the same in the PR...


[Revise subclass] SIDS ! MONDO:0010086  subclass of Mendelian disorder

Mondo term (ID and Label)
MONDO:0010086, sudden infant death syndrome
Suggested revision and reasons
https://www.ncbi.nlm.nih.gov/books/NBK513378/
"Evidence suggests likely genomic complexity and a degree of overlap amongst sudden cardiac death, SIDS, and SUDEP. The advent of next-generation sequencing will improve our understanding of the genetics of SIDS, and will result in more answers for families with a family history of sudden or unexplained deaths. However, a Mendelian approach is not likely to explain all cases."

I'm not sure if I can add a superclass assertion unless SIDS was always Mendelian.
cc @pnrobinson

To clarify, I was wondering if the subclass of assertion should be removed

Yes, I agree with you that it should. thanks for clarifying.

In general, SIDS is not a Mendelian disease. This entry (https://omim.org/entry/272120) implies that there are genetic variants that confer increased susceptibility to SIDS in some cases, and thatbut it does not imply that all or even most cases of SIDS are Mendelian.
The entry states that some Mendelian arrhythmia genes can cause long QT (and other analogous examples), which can cause a potentially deadly arrhtyhmia and lead to SIDS. In this case, I would say that the infant had the disease long QT with the phenotypic feature of sudden infant death (the latter is an HPO term). The disease SID syndrome implies that the cause is unknown.
@ahamosh thoughts?

SIDS is in OMIM because of the way people use OMIM.  I agree with Peter.  SIDS should not be in Mondo as a Mendelian phenotype.

I excluded the superclass of Mendelian disease, but I wonder if we should create a subclass of SIDS that is equiv to the OMIM term, and has the syn 'sudden infant death syndrome, susceptibility to' and conforms to the susceptibility pattern?
https://omim.org/entry/272120
Right now, the current class has the subclass axiom
'disease has basis in dysfunction of' some SCN5A
which is probably not true in every case.

There are lots of genes to which this can be linked and mostly to no genes at all.  Can we talk?  This is too complicated to type out.

Absolutely! Are you by chance free right now?
Or we have our Mondo call tomorrow at 12pm ET

Yes, happy to talk now:<PHONE_NUMBER>
Ada
Get Outlook for iOShttps://aka.ms/o0ukef

From: Nicole Vasilevsky @.>
Sent: Thursday, March 11, 2021 4:29:28 PM
To: monarch-initiative/mondo @.>
Cc: Ada Hamosh @.>; Mention @.>
Subject: Re: [monarch-initiative/mondo] [Revise subclass] SIDS ! MONDO:0010086 subclass of Mendelian disorder (#2759)
Absolutely! Are you by chance free right now?
Or we have our Mondo call tomorrow at 12pm ET
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmonarch-initiative%2Fmondo%2Fissues%2F2759%23issuecomment-797061669&data=04|01|ahamosh%40jhmi.edu|edc9673e430142be21f108d8e4d4c1c0|9fa4f438b1e6473b803f86f8aedf0dec|0|0|637510949720950912|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=SXgdMFSpQsfvkuGbvaR%2B%2F6n%2BE89xpjDSoA2fPvuyStE%3D&reserved=0, or unsubscribehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKKZWRG4Y5PMJP4CC3EQBHLTDER2RANCNFSM4YQH73AQ&data=04|01|ahamosh%40jhmi.edu|edc9673e430142be21f108d8e4d4c1c0|9fa4f438b1e6473b803f86f8aedf0dec|0|0|637510949720950912|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=FJ37%2Buq659gKtr%2B2jd453Sf7Ao519pHvRRWEGAerrSk%3D&reserved=0.

Per discussion with Ada, the recommendation is to obsolete this term, as it is not really a disease, and it is really an outcome of a variety of potential reasons, some which could be genetic, or environmental or maybe entirely unknown.


Treat empty string as valid translations (at least conditionally)

I'm submitting a ...
[x] bug report

Current behavior
If translation yields empty string, the translation is considered missing.
Expected/desired behavior
Empty strings, should either be considered valid translations always, or with a conditional switch, like
translateService.tolerateEmptyStrings();

Reproduction of the problem
See this plunker
What is the motivation / use case for changing the behavior?
Use case is pretty much described in #223 . When localizing links, this is essential.
** Comments **
One could add a space there:
{
  "SOME_KEY": " "
}

but this is error-prone as such kinda redundant whitespace can be lost when localization team does its job.
Please tell us about your environment:

ng2-translate version: 2.4.3
Angular version: 2.0.0-rc.6
Browser: all
Language: all

You're right, empty string shouldn't be an error ! Sorry about that, I think it's a regression

would you be willing to accept a PR that adds the suggested conditional switch?
I'd rather have empty values displayed with something like "[MISSING]: KEY" then just empty strings. At least this is the way I'm used to it with the old angular-gettext


Missing $ at Beginning of Command

In the provided example to deploy a template, the command is missing a $ (dollar sign) in beginning. This template file is being referenced in the deployment without being set as a variable.
Current example provided in documentation:
templateFile="{provide-the-path-to-the-template-file}"
az deployment group create
--name blanktemplate
--resource-group myResourceGroup
--template-file $templateFile
What the command should look like:
$templateFile="{provide-the-path-to-the-template-file}"
az deployment group create
--name blanktemplate
--resource-group myResourceGroup
--template-file $templateFile

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

ID: 7bf283cd-65c2-09ad-0af2-37bf95fe4d90
Version Independent ID: 46451ccf-174a-e4da-433f-aa8c5570557d
Content: Tutorial - Create and deploy template - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/template-tutorial-create-first-template.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@donovan-armstrong - thank you for your comment.  The commands are different depending on your execution environments.  The samples in the articles are created for shell.azure.com.
#please-close

@mumian Thanks for the clarification provided.
@donovan-armstrong Hope the response from @mumian has helped. Let us know if you have any further questions with regards to this topic. Thanks.


[SPARK-21335][SQL] support un-aliased subquery

What changes were proposed in this pull request?
un-aliased subquery is supported by Spark SQL for a long time. Its semantic was not well defined and had confusing behaviors, and it's not a standard SQL syntax, so we disallowed it in https://issues.apache.org/jira/browse/SPARK-20690 .
However, this is a breaking change, and we do have existing queries using un-aliased subquery. We should add the support back and fix its semantic.
This PR fixes the un-aliased subquery by assigning a default alias name.
How was this patch tested?
new regression test

cc @rxin @viirya

Test build #79311 has started for PR 18559 at commit 4d99c11.

LGTM

Test build #79312 has started for PR 18559 at commit 7279262.

Test build #79311 has finished for PR 18559 at commit 4d99c11.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79311/
Test FAILed.

Test build #79312 has finished for PR 18559 at commit 7279262.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79312/
Test FAILed.

Merged build finished. Test FAILed.

Test build #79316 has started for PR 18559 at commit 479e53c.

It'd be important to document what syntaxes are no longer allowed in the JIRA ticket (and PR description), and we also highlight that in release notes.

updated, I also added a release-note label to the JIRA ticket.

Test build #79316 has finished for PR 18559 at commit 479e53c.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79316/
Test FAILed.

Retest this please.

Test build #79321 has started for PR 18559 at commit 479e53c.

LGTM

Test build #79328 has started for PR 18559 at commit 822932c.

Test build #79321 has finished for PR 18559 at commit 479e53c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79321/
Test PASSed.

Test build #79328 has finished for PR 18559 at commit 822932c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79328/
Test PASSed.

Merged build finished. Test PASSed.

thanks for the review, merging to master!

Have we highlighted this in release notes?


[Low] Refactoring Cucumber/JS for better readability

I don't usually write Cucumber tests in JavaScript so I'm working on my style. I'm looking for feedback on better JavaScript style and whether this is an improvement in the readability of the tests.
One thing I'm finding difficult is that everything is a JavaScript promise which has to be resolved with .then before I can test the values, so if there's an easier syntax I'd be keen to learn it.

yeah promises... one thought might be to use async await which is definitely a more readable syntax.. . might require setting /eslintrc parserOptions "ecmaVersion": 8

Server needs to be running before cucumber can be run properly/pass.


(Question) Will crouton allow dual monitors?

I have a presentation that I need to do for my class, but their software requires W/M/L. My tiny Chromebook screen can not show to an entire class. So if i connect it through HDMI, will it show crouton and not CrOS?

It Will Be On Your Chroot If You start the chroot with it plugged in.


[ignition-msgs*] Fix flaky CI tests for ignition-msgs* ports (take 2)

Pass ADD_BIN_TO_PATH to vcpkg_install_cmake and remove custom PATH hacks. Less invasive version of https://github.com/microsoft/vcpkg/pull/14464 thanks to @Neumann-A feedback.
Describe the pull request

What does your PR fix? Fixes the problem mentioned in https://github.com/microsoft/vcpkg/pull/14324#discussion_r517128792, not sure if there is a corresponding issue.

Which triplets are supported/not supported? Have you updated the CI baseline? The PR should not change the supported triplets.

Does your PR follow the maintainer guide? Yes

cc @BillyONeal

LGTM.

Thanks @traversaro :)


logging_util

There are various portions of the code which are importing logging_util but I can't seem to find that in the codebase.

File was missing. Fixed in https://github.com/sstober/deepthought/commit/3494a5d9951dd413a1f501aa765898fb35412bd8
Thanks for reporting!
Sebastian


fix(platform-http): doesn't call inject.get when provider is instance…

Summary by CodeRabbit

New Features

Refined provider management within the injector by excluding specific providers from processing.

Bug Fixes

Improved control flow for retrieving providers, ensuring more accurate instance management.

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

v8.3.3
GitHub release

Your semantic-release bot :package::rocket:


this pakage only support sunmi printer POS device or working for all type of POS devices?

Yes, i testes in
K2
K2 mini
V2
V2s
@vijayvaghela72

okay thanks for reply
but my Question is working for all type of POS device or only for sunmi POS devices .

Ah ok!
This package works only in sunmi, because its using the sdk inside the devices.

Ohhh Okay got it .
Thanks @brasizza


Add Scratch.ArgumentType.VARIABLE and Scratch.ArgumentType.TARGET

Relatively self explanatory and I have mentioned it before but just in case,
As done with costumes and sound, add 2 more arguments, one for the variables in a project and another for the dropdown seen inside the "create clone of" block.
I'm hoping to use this in Clones Plus, as while the dynamic menus that are currently being used work as a placeholder, they are extremely buggy.

Bump, except I think it should go further than this since now more than 1 of my extensions could benefit from dedicated target/variable getter arguments.
If there was a way to have a variable field be determined by the target field in a block, Clones Plus 2 could be thing by now


Allow 256 colors for terminal vim on Windows

I am not sure if the following makes sense, so feel free to discard if not wanted.
Enable 256 colors when run on recent enough windows in terminal mode.
Since there is a glitch with default 256 colors, also define Normal highlighting color as a black one.
Also fixes vim/vim#9498

What is exactly the problem? A description and some steps to reproduce it would be great.

hm, i thought this is clear from the linked issues. Basically, this will make vim use 256 colors  in terminal mode on modern Windows (e.g. on cmd.exe)

What is the change at the end of termcapinit() needed for?  A comment would help.

hm, i thought this is clear from the linked issues. Basically, this will make vim use 256 colors in terminal mode on modern Windows (e.g. on cmd.exe)

I was referring to this bit

Since there is a glitch with default 256 colors, also define Normal highlighting color as a black one.

What's the problem? The code in termcapinit looks like a workaround for some sort of problem as calling the highlight stuff is definitely out of place in that function.
From what I can see the virtual terminal processing is unconditionally enabled at startup, we could add a new builtin terminfo entry called win32-vt with all the VT escape sequences and t_Co set to 256 and use that whenever vtp is detected at startup (similarly to what already happens for eg. 256color terminals on Linux). The old win32 using the legacy Win32 API can be then left alone and a lot of code can be simplified (eg. getting rid of all the tcap swapping machinery).

hm, i thought this is clear from the linked issues. Basically, this
will make vim use 256 colors in terminal mode on modern Windows
(e.g. on cmd.exe)

I was referring to this bit

Since there is a glitch with default 256 colors, also define Normal
highlighting color as a black one.

What's the problem? The code in termcapinit looks like a workaround
for some sort of problem as calling the highlight stuff is definitely
out of place in that function.
From what I can see the virtual terminal processing is unconditionally
enabled at startup, we could add a new builtin terminfo entry called
win32-vt with all the VT escape sequences and t_Co set to 256 and
use that whenever vtp is detected at startup (similarly to what
already happens for eg. 256color terminals on Linux). The old win32
using the legacy Win32 API can be then left alone and a lot of code
can be simplified (eg. getting rid of all the tcap swapping
machinery).

This solution possibly also helps with colorschemes adjusting to the
terminal.  We should at least give this a try and evaluate the resulting
situation.
--
"A mouse can be just as dangerous as a bullet or a bomb."
(US Representative Lamar Smith, R-Texas)
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I'll rework it completly. But I'll need some time

Still reworking this?

yes, did not have much time the past week, since I was very busy with my job


Fix webpack

https://webpack.js.org/configuration/
We need to be able to load see the bundle.js file. Right now it's not showing.

Server was included in bundle.js that is why we are building very slow. Need to improve.


Dubbo 2.6.x may not support multi group consumer #7985

ref to #7985
Just add some default keys to avoid such special cases

Codecov Report

Merging #7997 (fb25c72) into 2.6.x (077da34) will decrease coverage by 0.10%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##              2.6.x    #7997      +/-   ##
============================================
- Coverage     47.10%   46.99%   -0.11%
+ Complexity     4463     4461       -2
============================================
  Files           573      573
  Lines         25575    25554      -21
  Branches       4537     4515      -22
============================================
- Hits          12048    12010      -38
- Misses        11671    11686      +15
- Partials       1856     1858       +2

Impacted Files
Coverage Δ

...libaba/dubbo/rpc/cluster/support/ClusterUtils.java
62.26% <33.33%> (-3.70%)
:arrow_down:

...aba/dubbo/remoting/transport/mina/MinaChannel.java
42.25% <0.00%> (-11.27%)
:arrow_down:

...ba/dubbo/remoting/transport/netty/NettyClient.java
72.88% <0.00%> (-10.17%)
:arrow_down:

...mmon/threadpool/support/fixed/FixedThreadPool.java
80.00% <0.00%> (-7.50%)
:arrow_down:

.../threadpool/support/limited/LimitedThreadPool.java
83.33% <0.00%> (-5.56%)
:arrow_down:

...on/threadpool/support/cached/CachedThreadPool.java
85.71% <0.00%> (-4.29%)
:arrow_down:

...ubbo/rpc/protocol/dubbo/ChannelWrappedInvoker.java
37.50% <0.00%> (-4.17%)
:arrow_down:

...bo/rpc/protocol/dubbo/telnet/LogTelnetHandler.java
68.57% <0.00%> (-2.86%)
:arrow_down:

...a/com/alibaba/dubbo/remoting/exchange/Request.java
80.95% <0.00%> (-2.39%)
:arrow_down:

...bo/rpc/cluster/support/FailoverClusterInvoker.java
82.22% <0.00%> (-2.23%)
:arrow_down:

... and 16 more

Continue to review full report at Codecov.

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