Level 200: Automated Deployment of VPC

VPC security features
VPC layered subnet architecture
Automated deployments

please read the aws documentation everything will be there

Pl. Share  list  Doc's
ASAP

Pl. Share list Doc's
ASAP

security groups:
https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ec2-security-groups.html
NACL:
https://docs.aws.amazon.com/vpc/latest/userguide/vpc-network-acls.html

Mohan I expect this is something which you can search for your self. This issues page is for practical problems. Refrain from design discussion. For that you have to schedule meeting with me


Not receiving default params on larger projects(race condition maybe?)

Describe the bug
So i noticed that in larger projects a lot of the OG-image fails to receive default params from the meta fields on the project becomes larger.   I imagine this is some sort of race condition where it isnt actaully done writing those files yet as every file after that initial breakage seems to work.   This is just a guess,   i tried reproducing this on smaller projects,  but i think this only fails on large projects.

i guess my question is,  is there a way we can write the images only after the pre-render is done?  or somehow trigger it slightly later on the build?
also as a side note,  this is a project that is currently behaving as expected on nuxt seokit 1.3.13 this seems to only be a bug when doing the upgrade
Reproduction
No response
System / Nuxt Info
------------------------------
- Operating System: Darwin
- Node Version:     v18.10.0
- Nuxt Version:     3.9.0
- CLI Version:      3.10.0
- Nitro Version:    2.8.1
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      modules, site, linkChecker, css, turnstile, robots, ogImage, head, routeRules, typescript, sourcemap, devServer, devtools, htmlValidator, image, runtimeConfig, schemaOrg, nitro, appConfig, ignore, hooks, vite
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -
------------------------------

Hi, thanks for the issue.
Would you mind elaborating on what you mean by this exactly

fails to receive default params from the meta fields

Are you saying the title/description isn't coming through from the default title and meta description?
Can you share some code either way just as a sanity check?
I can't imagine that this would be a race condition issue as the the og image routes are only registered when the route using it is prerendered. The logic for prerendering works a bit differently though, as we cache all of the payloads where possible to avoid extra prerender time. Might be related to this.

@harlan-zw yea so basically the by default params the title and description is empty even though they are set with useHead().
out of the 300+ routes,  the only ones that suffer from this is that first batch of OG images, every single of them have no parms and after that first batch every single one works.  If i move those exact files with those exact same file structure to smaller projects to replicate this,  it works all of a sudden.  So the fact that this only happens on larger projects and the fact that it only happens to the first one,  makes it seem that something is starting something a tiny bit to early.
As for "I can't imagine that this would be a race condition issue as the the og image routes are only registered when the route using it is prerendered. " this is true,  if i look at the logs i can confirm the routes return that is has been pre-rendered successfully before OG image starts.
As for the reproduction i will continue to try get one for you that i can share,  but at the moment iv been struggling to replicated it on smaller projects as i said i think size and complexity might be adding that extra lag that is causing the first batch to break

Thanks for the extra details. Will attempt to make a reproduction myself, are you using any special nitro configuration?

na its all vanilla base settings.


WIP: Move travis CI to GitHub actions

[x] Move lint
[x] Move test
[ ] Move deploy

Possible extensions, use: https://docs.github.com/en/free-pro-team@latest/actions/reference/workflow-syntax-for-github-actions#jobsjob_idcontinue-on-error to include experimental/outdated versions in the matrix.


Fix MangoPay\PayInWebExtendedView

This PR is to fix the current broken MangoPay\PayInWebExtendedView class as explained in issue #411
I extended PayInWebExtendedView to Libraries\Dto
In order to match the current API, I made a few changes in the class variable:

Removed ExecutionDate (it's should not be there).
Added ExecutionType
Added Fingerprint

This allow to call the mangoPayApi->PayIns->GetExtendedCardView($payIn)) call again.

I've got it all green from Travis on my forked repository, can you please relaunch the build that failed on Travis as this is not related to the code itself.

Closing this PR in favor to PR #413


[mesheryctl] v0.7.0: Conventions on use of MeshKit Errors

See mesheryctl Command Tracker for current status of commands.
Current Behavior
mesheryctl uses portions of MeshKit today. MeshKit Errors are is functionality that mesheryctl needs to inherit from MeshKit.
Alongside the use of MeshKit errors will be the benefit of use of the utility for error code generation and documentation.
Desired Behavior
As an objective of the v0.7.0 release, plan out use of MeshKit errors, providing these instructions within the mesheryctl coding conventions documentation - https://docs.meshery.io/project/contributing-cli

Contributor Resources

mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation
Layer5 Community Slack

Keep this open for the time being (an assignment that flew right past me :rocket: )

Hey @mgfeller would you mind taking me with you if you are importing the meshkit error in mesheryctl ?

@sayantan1413 of course not, that would be nice :smile: It might take a while, though - it is free for anyone to pick.

Let's keep this open for now.
There is work done in mesheryctl to convert the errors (in other issues), and there is e.g. some discussion here  https://github.com/meshery/meshery/issues/3986.
Eventually, amend https://docs.meshery.io/project/contributing-cli with a section about errors - including how Cobra handles them too, and how the stringified version of meshkit error is used.
Also add a reference to https://docs.meshery.io/project/contributing-error
// @piyushsingariya @DelusionalOptimist

@sayantan1413 @meshery/mesheryctl-contributors, any progress to report here?

@sayantan1413 @meshery/mesheryctl-contributors, any progress to report here?

I am mentioning packages where I can't see an error.go file

app
config
pattern


first commit

Related Issue

Fixes #
Checklist:

[ ] I have read the contributor's guide.
[ ] I linked an issue in the previous section
[ ] I have commented on the linked issue
[ ] I was assigned the linked issue (not required)
[ ] I have tested the change to the best of my ability against the sandbox or a local build.

Optional items:

[ ] My change adds new text and requires a change to translations.
[ ] My change requires a change to the documentation.
[ ] I have submitted a PR to the documentation repo.
[ ] I was not able to test... (explain below, e.g. you did not have permissions to test a specific feature)
[ ] This change depends O-FISH Realm repository changes (explain below)

Optional: Add any explanations here

Optional: Add any relevant screenshots here

Hi @mfhan!
We noticed you haven't claimed your digital MongoDB swag - you're eligible even though we did not merge this PR. To claim your O-FISH badge, head on over to the MongoDB Community forums, login, and reply - making sure to link to this PR.


🐞 [Bug Report] - deterministic urls update

Describe the bug
With the release of Deterministic URLs, Cloud Run services now have two default urls:
SERVICE-PROJECTNUM.REGION.run.app
SERVICE-HASH-REGIONCODE.a.run.app

Avocano uses self-csrf to build the hash-regioncode variation of it's own URL, but not the new version.
This causes issues when trying to use the web-interface for the server, specifically when using the default presented URL in the Google Cloud Console or Service URL in the output of a gcloud run deploy (both report the projectnum variation)
Additional context
internal b/367711357

A possible way to do this, replacing most of the logic in _service_url
# pip install google-api-python-client

from googleapiclient.discovery import build

run = build("run", "v1")
fqname = f"projects/{project}/locations/{region}/services/{service}"
service = run.projects().locations().services().get(name=fqname).execute()
urls = service['metadata']['annotations']['run.googleapis.com/urls']

May no longer need google-cloud-run, instead google-api-python-client (or both).
Also need to update the ALLOWED_HOSTS to support a list, possibly similar to the Codelabs update https://codelabs.developers.google.com/codelabs/cloud-run-django#5:
# If defined, add service URLs to Django security settings
CLOUDRUN_SERVICE_URLS = env("CLOUDRUN_SERVICE_URLS", default=None)
if CLOUDRUN_SERVICE_URLS:
    CSRF_TRUSTED_ORIGINS = env("CLOUDRUN_SERVICE_URLS").split(",")
    # Remove the scheme from URLs for ALLOWED_HOSTS
    ALLOWED_HOSTS = [urlparse(url).netloc for url in CSRF_TRUSTED_ORIGINS]
else:
    ALLOWED_HOSTS = ["*"]


Removed http requests/unnecessary string operations

have you even tested this?

actually, this does seem to be better. you win this


test_sharded.py::test_sharded_to_from_l1 fails

test_sharded.py::test_sharded_to_from_l1 is a simple test that copies tensors to/from device using the ttl.tensor.to API using various memory configs and shard specs.
Note that the test is written under my assumptions for how the API is programmed, there might be bugs in the test itself.  The bug was found during writing a convolution test that tried to copy a block sharded input to device L1 using this API + COL_MAJOR shard orientation.

Might be related to https://github.com/tenstorrent-metal/tt-metal/issues/4811


[BUG]: Some typings return any eventhough they are explicitly set

Describe the bug
Basically the title is the description of the bug. I found that error using the createUserProvider
const provider = createUserProvider<Customer>({
  dataKey: 'customer',
  mode: 'fetch',
  loginUrl: {
    local: API_URL + '/api/v1/public/auth',
    social: API_URL + '/api/v1/public/auth/social',
  },
  logoutUrl: API_URL + '/api/v1/public/auth/logout',
  localOnly: false,
  httpClient: http.client,
  useProfile: () => {
    const { data } = useProfile();
    return data || null;
  },
});

provider.meta // shows any, because provider is interpreted as any

To Reproduce

install package in a new project
create a user provider
extract meta
see any type

Expected behavior
I expect it to be typed as { interceptors: Interceptors }

It might be related to react-request since it appeared in version 2.0.4, where the package was first integrated.


proposal for a uribeacon flag

I don't know what "hidden" flag in the uribeacon really is for. But I have an issue with physical web that I think can be resolved with such a flag. Or perhaps this is what that "hidden" flag is for.
It has to do with the "Edit URL's" feature that every user sees on their smart device's physical web tool. Assume 1000 bored passengers waiting in an Aiport terminal. Many of whom will experiment with the feature and try to select and change found Uri Beacons.
The casual millions of users will never have a need for the feature and should not be confused by having the feature presented to them. Perhaps the feature should only appear once a beacon as been manually (at the beacon) reset into the programming mode.

Our suggestion of implementation is to have a physical trigger to enable the Edit URL feature on the device.

It can be a button that enables the edit mode for 10 seconds
Or it can be a reset button and the device URL can only be edited when the URL is blank

Yes, I agree, a physical trigger on the beacon device is essential to drop it into Edit mode. What I am suggesting is to then set a bit in the advertizement flags to let a physcial web or other app know the beacon is in the updatable mode. I feel that the app should not have a Edit URL function exposed until there is a beacon willing to be re-programmed. Then a connection and authentication procedure can commence. The millions of people have no need to even know there is a Edit URL function available on the phone until they own a beacon.

Oh I see. Thanks for the feedback.

The 'hidden' flag is intended for devices that broadcast a URL but are
controlled by a master device. Imagine a room of lightbulbs but there is
one one master lighting system. You don't want all of those lightbulbs
showing up in your list when they clearly don't need to be. The "hidden"
flag is there to the client to optionally hide that item from the user. Of
course, it could, for example  group all hidden items under a hidden folder
if it really wanted to.
We've discovered that nearly anything can be done with meta data on the web
page that the URL points to but having a flag like hidden makes it much
much easier that making the full round trip each time.
Scott
On Fri, Jan 2, 2015 at 11:37 AM, tolson2000<EMAIL_ADDRESS>wrote:

I don't know what "hidden" flag in the uribeacon really is for. But I have
an issue with physical web that I think can be resolved with such a flag.
Or perhaps this is what that "hidden" flag is for.
It has to do with the "Edit URL's" feature that every user sees on their
smart device's physical web tool. Assume 1000 bored passengers waiting in
an Aiport terminal. Many of whom will experiment with the feature and try
to select and change found Uri Beacons.
The casual millions of users will never have a need for the feature and
should not be confused by having the feature presented to them. Perhaps the
feature should only appear once a beacon as been manually (at the beacon)
reset into the programming mode.
—
Reply to this email directly or view it on GitHub
https://github.com/google/uribeacon/issues/130.

Thanks Scott for explaining that hidden flag. What do you think about the idea of not having the "Edit URL" menu item showing up on millions of phones that have no business knowing about editing beacons. I think that feature can normally never show up in a users phone except when there really is a uribeacon preparing to be modified. I think to make that work, we need a flag for that purpose.

The "Edit URL" is entirely a utility function for makers using this first
round. Once we have this in a mainstream form, it will no longer be
offered. That ability will be put off to the URI Beacon app (or something
that the manufacturer ships with their beacons. In fact, in the longer run,
we expect most beacons to be configured at the factory and not be editable
at all. So in other words, this will change drastically as this product
space matures.
Scott
On Fri, Jan 2, 2015 at 6:14 PM, tolson2000<EMAIL_ADDRESS>wrote:

Thanks Scott for explaining that hidden flag. What do you think about the
idea of not having the "Edit URL" menu item showing up on millions of
phones that have no business knowing about editing beacons. I think that
feature can normally never show up in a users phone except when there
really is a uribeacon preparing to be modified. I think to make that work,
we need a flag for that purpose.
—
Reply to this email directly or view it on GitHub
https://github.com/google/uribeacon/issues/130#issuecomment-68579802.

OK, that sounds perfect then. Thanks a bunch.


Attestation for GARUDA

In the Objective section of GARUDA, there’s discussion of various ways to add trust to a server and it looks like there might be a technical way to add trust to one of the options.  This is a quote:

It's open source! That is great, and whatever Garuda produces will be open source, but just because something is open source doesn't prove that the system running it is actually running that open source implementation as is — it's pretty easy to cheat.

Remote attestation is a feature of Trusted Execution Environments that can guarantee to a caller that a certain version of the software is running, we’ve explored it a bit here.  It could work very nicely with Open Source software by giving browsers, or anyone, a way to guarantee that there’s no cheating by running other software so it might be worth including in the spec as a possibility.

Yes, that's very much in line with the kind of thing I would like to see happen. If we can rely on technical means to eliminate overhead and weak points (eg. the need to have actual on-site audits) then the governance structure can be kept to a minimum.


fix: headers with equals

Solution when you have equal sign in headers key, eg:
--headers 'Authorization=Basic dXNlcjpwYXNzd29yZA=='

You will have an error:
[error] invalid tag 'Authorization' should be key=pair

This fix solves this problem.

@kvaps JIRA and tests are missing. Also the commit message needs updating. Please see our general guidelines https://github.com/keycloak/keycloak/blob/master/CONTRIBUTING.md for details.

It has been a while since we requested changes to this PR. While the submission is appreciated, we are not able to merge it without these changes. Because it has been so long, we are going to close the issue. Feel free to reopen with the changes included, if your time permits.


[Feature] Add Cocktail Lamp Solver

Rule: https://puzz.link/rules.html?cocktail

Resolved in ef4a3cc.


Support module exceptions in named rule?

It would be nice to have the ability to exclude certain modules for the named rule.
Modules like react-router and immutable do not use ES6 exports, for example.
Alternatively, searching for modification of exports may allow it to work appropriately.

This is related to #28. I have gone back and forth on supporting parsing of CommonJS and other non-ES6 modules, from doing some simple analysis of exports, like you suggest, to just silently ignoring non-ES6 modules.
I assume given your comment that you use ES6 imports for all your modules? I have considered something like a detect-common setting that would allow for some interoperability with CommonJS. Feels like a pretty likely use case.
Do you think this proposal would support your use case(s)?

Also, FWIW, you can //eslint-disable-line import/named to squelch the report on a per-line basis. I think a pair of //eslint-disable import/named + //eslint-enable import/named around a block of CommonJS module imports would also work.
Though it's super lame to have to modify the source to deal with linting, IMO. So I am not suggesting this as a long-term solution, just a point of information.

I do use ES6 imports for all modules. I'm using Webpack with Babel to compile things down to CommonJS and ES5, so the behavior is as expected.
I think that detect-common would be a more than decent solution for the long term, but if it would be easier to silently ignore non-ES6 modules in the short term then that works just as well.
For now I've simply disabled the name rule since as I use Immutable just about everywhere.
Thanks for taking time to help me out :smiley:

2 new features:

import.ignore setting: a list of regex strings that will be ignored by all rules
all/relative-only per-rule setting: most rules now ignore absolutely-pathed modules. (no-unresolved is the exception as this is useful whether the module is ES6 or CommonJS)

I believe one or both of these solve the problem. I figure ATM it's more likely than not that only in-project files are actually references to their ES6 representations; most node_modules will be CommonJS for the short- and mid-term, probably. (we can cross that bridge when it comes :sweat_smile:)

Awesome! Thanks for putting time into this :+1:

Let me know if you run into issues. I bumped up to 0.5.1 since the relative-only default is essentially a breaking change. I'm going to start using it again today (had to disable it for similar reasons to yours) and I'll knock out anything I run into.

Another thought as I reflect: would it make more sense to ignore modules with node_modules in the complete path vs. absolute-ish module source paths?
i.e. I have a NODE_PATH=./src on my build so I can do things like
import { a, b } from 'jsx/some/module'

from anywhere in my code, but it's tough to distinguish at face value from something like
import { findDOMNode } from 'react'

...but the resolved filesystem path should contain node_modules. I may play around with that as a default.

Hm, that does seem like a better approach to base it off of whether the resolved path contains node_modules. It looks like you already addressed core modules of Node.js like crypto.
I haven't gotten time to adjust our lint file to use the new version just yet, but when I get to a point where I can test it thoroughly, I'll let you know if it works out.

For some reason I just can't get this to ignore things in node_modules:
Here is a simple example with Immutable:
https://gist.github.com/chase/341f060e62bea8888955
I'm probably doing something incorrectly, but I honestly can't figure out what.

Everything looks good with your settings, it should be disabling the parse by default.
I have been using it IRL with React/Immutable and it doesn't report for import/default...but I don't have many named imports from node modules, so there could be a difference there.
Maybe I flubbed something with the ESLint tests I wrote? I'll investigate when I get a chance.
--Ben

On Jul 19, 2015, at 11:51, Chase Colman<EMAIL_ADDRESS>wrote:
For some reason I just can't get this to ignore things in node_modules:
Here is a simple example with Immutable:
https://gist.github.com/chase/341f060e62bea8888955
I'm probably doing something incorrectly, but I honestly can't figure out what.
—
Reply to this email directly or view it on GitHub.

I downloaded your gist and I am unable to reproduce. It works as expected (no output from ESLint), but then I do get that output when I explicitly set import/ignore to []. (also expected)
I'm not sure what could be going on. One tenuous idea: could you have an old copy of the plugin installed globally / in an ancestor folder and it's somehow using that instead? (I think if ESLint is installed globally it might only use globally-installed plugins?) Not sure if that's possible, though. But I'm unable to reproduce on my machine. :confused:

Good call on the globally-installed plugin inconsistency, that's exactly what was happening. I forgot I had it installed globally. I really wish that ./node_modules/.bin/eslint test.js wouldn't use global plugins by default.


Add effect method

Motivation
In spite of having ability to use atom's subscribe/notify methods for subscribing on changes, it might be a bit tricky to track changes at multiple atoms at once. effect is designed to solve this problem! And it is similar to the effect method from other state managers (like Solid's signals, or state manager in Svelte)
How it works
effect runs its callback on the start, with initial values, as well as on any store(s) change. If callback returns cleanup function it will be performed before next callback run. Besides that, effect returns own cleanup function, which allows cancelling the whole effect.
Usages
effect can be used as analogue of subscribe method on one store:
import { atom, effect } from 'nanostores'

const $num = atom(10)

effect($atom, num => {
  console.log(`Number is ${num}`)
})

effect allows easily handle complex cases. This example shows how we can create resize observer atom, which will create ResizeObserver instance based on passed element and options (which can be changes outside, since they are both atoms). Whenever element or options are changed it re-runs effect and cleans up ResizeObserver instance for the previous effect run.
import { atom, effect, onMount, type ReadableAtom } from 'nanostores'

type ResizeObserverAtomOptions<E extends HTMLElement, Mapped> = {
  $element: ReadableAtom<E | null>
  $observerOptions?: ReadableAtom<ResizeObserverOptions>
  initialValue: Mapped
  mapper: (entry: ResizeObserverEntry) => Mapped
}

export function createResizeObserverAtom<E extends HTMLElement, Mapped>({
  $element,
  $observerOptions,
  initialValue,
  mapper
}: ResizeObserverAtomOptions<E, Mapped>) {
  const $result = atom(initialValue)

  onMount($result, () => {
    // Notice how we use returned cleanup function as returned function from onMount
    // It will automatically stop effect when $result no longer has any subscribers,
    // and cleanup everything
    return effect([$element, $observerOptions], (element, observerOptions) => {
      if (!element) {
        return
      }

      const resizeObserver = new ResizeObserver(entries => {
        const entry = entries[0]

        if (entry) {
          $result.set(mapper(entry))
        }
      })
      resizeObserver.observe(element, observerOptions)

      return () => resizeObserver.disconnect()
    })
  })

  return $result
}

Thanks. Great idea.


List Server not refreshing

If you request a list, it (the normal server) crashes and the list server doesn't receive anything. Thus client list server and server crash.

Should now be resolved


Add CI for checking minimal versions

https://users.rust-lang.org/t/psa-please-specify-precise-dependency-versions-in-cargo-toml/71277

Codecov Report

Merging #21 (d89a14a) into main (697792c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   89.24%   89.24%
=======================================
  Files           8        8
  Lines         679      679
=======================================
  Hits          606      606
  Misses         73       73

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 697792c...d89a14a. Read the comment docs.


How to Connect to ODB with USB? (help wanted)

Hello
I wanted to transform the ios code to mOS. So that there is also a simple example for mOS
I saw now that the application connects via Bluetooth. Is it also possible to connect via USB and how would it be done?

First: Thank you for your reply :D
I will try it once i find the time and then write my results here. I also tried this python lib (https://github.com/brendan-w/python-OBD) to connect over odb2 and it worked fine. After it worked i got excited about odb2 and i wanted to learn more about it and enlarge the lib with more commands. But idk where to find documentations and information on the topic. Most pages look shady or have just basic information. Maybe you can know where to find some good information? I am just curious and want to see what things you can do on the car. By the way i am mech engineering student and have access to norms at Perinorm. Maybe you can tell me which norms i have to look up? The car of my dad is a opel astra caravan H and i read that it uses CAN_500KBPS. I would be thankful if you can tell me where to find information on the topic :D

OBD2 is quite well standardized. Everything interesting for one who is developing OBD2 software can be found in SAE1979 and SAE1979_DA. If you have these standard documents, you're quite well equipped. The Wikipedia pages are a good source as well, but they contain only excerpts of the official standard documentations and are a bit outdated. Unfortunately SAE charges money for these documents, that's why they're not on the internet. They're not too expensive though (ISO documents are often way more expensive) and worth the money IMO.

I have bought the OBDLINK SX USB/OBD2 adapter. As expected, this mounts as a usbserial device node:
% ls -l /dev/*usbserial* crw-rw-rw-  1 root  wheel   21,   7 13 Aug 14:08 /dev/cu.usbserial-113010893810 crw-rw-rw-  1 root  wheel   21,   6 13 Aug 13:40 /dev/tty.usbserial-113010893810
With a tiny bit of configuration glue (done in ref 81c6453f2ae89b518f120497589465a9f3eee89c), I could talk with this USB adapter.


Couple of Issues on First Try

Greetings 👋🏻
Thank you for this extension.
I have noticed a couple of issues with this, after using it for a minute.

It's not clear, that you have to press Regenerate, when Directories are de-selected in the Select Directories view.
It's really hard to find the Regenerate button, because you have to scroll really far down, since every .git file is included & shown by default. You have to scroll a lot, but not too much, because else you see the visualisation, instead of the Select Directories section. The Regenerate button should be on top.
The Select Directories section should be left/right next to the visualisation, so you can compare easily.
Ideally, the Regenerate button should not be necessary & the view should automatically refresh on change.
Directories like .git & maybe even directories with tons of spammy files on a single level should be de-selected by default. Most of the time, you don't wanna see those in your tree, I would expect.
In a later stage of development, it would be nice to see the possibility of writing rules, filters, etc.
The naming in the Command Palette should have the word "Directory" in it, rather than just "Generate Tree Map", because it's easiest to find, this way.

Otherwise, good start. 😄

Hello there and thank you for reaching out!

Any suggestions on how to make it more clear? There were two options: Option 1 - Have the tree map update as you are checking and unchecking directories, but that would be inefficient for large workspaces (constantly generating the tree map each time a directory is checked or unchecked), Option 2 - Do your checking and unchecking and then click Regenerate when you are ready (which is the option I went with).
I debated between placing at the top of the checkbox tree map or the bottom. Both locations pose issues. If you're checking and unchecking things near the bottom, then having the button there is convenient. If you're checking and unchecking things near the top, then having the button there is convenient.  Perhaps in a new version, I will add a floating button that remains present in a corner at all times or some sort of placement that is even more efficient than the top or bottom of the checkbox tree (as both of those placements have their drawbacks, especially with large workspaces).
The problem with having them left/right is screen size. If viewed on a monitor, it is fine to do so. If viewed on a monitor snapped to 50% width of display, it's not so fine. Also, on a smaller laptop screen it is not ideal either to have left and right. Some directories are heavily nested and then you are just migrating to horizontal versus vertical scrolling. Open up a heavily nested workspace and you'll see how left and right could become ugly. But I know what you mean about wanting to see them side by side for comparisons.
I addressed this concept in the first bullet
While developing the extension, I hard-coded the exclusion of the .git and node_modules directories for the exact reason you stated, they are rarely needed in the visualization (tree map). However, the Select Exclusions tab is where you should set those exclusions. Some people may want it, most probably won't, but they have the ability to set them. Also, you can see in the README.md file that a later version will have persistence, and the ability to create global exclusions (which will apply to all workspaces) and workspace specific exclusions (which will apply to specific workspaces). Effectively allowing you to exclude .git or node_modules globally, but at least you get to set that versus me forcing it.
Great Ideas! Rules, filters, and many other ways to customize the tree map will be arriving in future releases. Some details are in the README.md file.
You are right on. I will change that. Possibly just the extension name "Directory Visualizer" or "Open Directory Visualizer", something like that?

Just wanted to thank you again for using the extension and reaching out with some great suggestions. Will consider them in future updates.


Merge release-2.15 branch to master: Cut release 2.15.1

Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

:+1:


adds multi-OS support to travis CI

Adds multi-OS Support to address #109
This requires a few changes to the current approach:

removing linux/macOS specific environment setup
changing the "language" feature because "Python" is currently not supported for Windows instances on Travis, using environment variables instead
separating the installation of conda, and the setup of the conda environment to keep the CI file easy to read

Codecov Report

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

@@            Coverage Diff            @@
##           master    #124      +/-   ##
=========================================
- Coverage   95.98%   95.9%   -0.08%
=========================================
  Files           9       8       -1
  Lines         448     440       -8
=========================================
- Hits          430     422       -8
  Misses         18      18

Impacted Files
Coverage Δ

pandera/__init__.py

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 5b29d9a...b09ed0c. Read the comment docs.

not sure why this build is failing:
https://travis-ci.org/pandera-dev/pandera/jobs/616075868#L418-L419

Please consider this PR WIP.

@cosmicBboy please re-review, I've had help from @bensaxby to resolve the test failures.
This PR extends CI support from Linux to also include Windows and osX/macOS by:

handling the way Python and Conda work on Travis for different OS versions
includes code like conda list because it's easier to debug the CI if it's included
contains a specific safety net for if the terminal window is sized too small. The setting of the doctest_global_setup variable is included because when Travis is running a Python environment on macOS it misreports the size of the terminal window as 0 cols, 0 rows. This can be replicated by running make doctest locally with a small terminal window. When the window is too small Pandas trims the dataframe output with ellipsis. I've been unable to replicate this bug locally, i.e. terminal always reports the correct size, so I think it's a bug in Travis-CI.
the intermittent Python 3.5 error was caused by a cache issue.

I'm not sure how to fix the codecov issue.

@mastersplinter looking good to me!
looks like flake8 is complaining:
tests/test_docs_setting_column_widths.py:4:1: E302 expected 2 blank lines, found 1
tests/test_docs_setting_column_widths.py:9:37: E711 comparison to None should be 'if cond is None:'
tests/test_docs_setting_column_widths.py:10:37: E711 comparison to None should be 'if cond is None:'
tests/test_docs_setting_column_widths.py:10:44: W292 no newline at end of file

will ignore code coverage drop, looks like a non-issue

sorry about that - flake8 compliance was a silly miss on my part.
FYI I've opened a bug report on the TravisCI Community documenting the specific issue with the way Python on macOS images on Travis report their terminal width.


.Net 7 AES error on Windows 7

When trying to build CDLC (test or release) on ver 2.0.0+ under win7 the following error appears:
AggregateException: One or more errors occurred. (The current platform does not support the specified feedback size.)

Inner exception:
CryptographicException: The current platform does not support the specified feedback size.
   at <StartupCode$Rocksmith2014-PSARC>.$PSARC.Create@309-3.MoveNext()

Version 1.6.2 works, So it must be connected to .Net 7 update.
It seems there was (is) a problem in .Net 7 on win7.
Found the only source with the same error (its in chinese, yet google translates okey-ish) - https://www.cnblogs.com/lindexi/p/17680647.html
It seems there is a workaround, yet it looks bulky and not really trustable.
Can you look into this issue? Is there a way to still support win7?

Windows 7 has never really been supported.
In the 2.0 version I changed the PSARC encryption to use an implementation provided by .NET (added in .NET 5) and it seems that W7 does not provide that algorithm. Reverting that should be relatively simple, but I don't have a Windows 7 machine to test it (I tried to setup a virtual machine some time ago, but couldn't get the program to even run).

Thank you for the answer.
I am eager to help with testing, if you'll find it sensible.
Though even steam stops support of win 7 on jan 1st. So I am not sure if it's worth it.
It's been three months since update, and it feels like I am first to stumble across the issue.


wip - show test results in ci

Closes #2798

Test suite run success
187 tests passing in 115 suites.

Huh, I never even realised that this GH comment was missing! Definitely better with it back.

@appleseed-iii this would be reeeeal handy

before

after


Update npm package

Could you update the npm package as it's currently on version 1.3.0 instead of 2.0 like this repo is?

+1

Done.. :)


Open question: Have dedicated literals been considered?

It will be a cat-and-mouse game trying to get hold of original intrinsics/getIntrinstic before someone else tampers them. So in my opinion this makes more sense as a syntax level thing, having %xyz% literals . i.e.,
const SomeIntrinsic = %IntrinsicName%;. This rules out those shenanigans entirely.
As a bonus/headache prevention, any invalid/unsupported IntrinsicName is a syntax error at parse time. Babel/transpilers will take care of providing intrinsics to legacy environments of course (as is tradition with all new syntax). Another bonus, transpilers are in the unique position to put in authentic references to those intrinsics.
getIntrinsic('%xyz%') is prone to typos if it fails silently on invalid names. We should strive for the language to be writeable without IDEs, TypeScript etc.. In good ol' text editors.
P.S: I have not checked if these dual percent sign literals will be in conflict with some other ES proposal. V8 uses the % prefix for internal method names, but that probably/hopefully won't conflict with percent sign on both ends.

Also, dedicated literals will likely have massive performance advantage over getIntrinsic function calls. Secondly, transpilers will have a hard time polyfilling if getIntrinsic takes dynamic values... (getIntrinsic(someVariable)).

Well, unless of course the transpiler is allowed to replace globalThis.getIntrinsic with a polyfilled version. Then back to square one above lol.

They won’t be ever increasing - as supported browsers fall off the list, the number of intrinsics that need polyfilling will diminish.
An intrinsic is anything that exists before your code runs - including polyfills.


onClick does not work for first time on Series, PieSeries..

After loading chart when I click on pie slice, it does not execute the function for first time but if I click it again then it works and for each click it works.
Issues to be persist for first click only.

Hi @sumitkothariqp, I'm not able to reproduce the issue mentioned (see here). Would you be able to post a broken example?

Hi thanks for the update.
I tried my code in your example and it worked. But it is still not working with my dashboard.
I am using react grid layout and redux. I am still identifying the issue at my end.
Thanks for creating react-jsx-highcharts.. really helpful.


MM-24086:Disable changing picture if LDAP sync'ed

Summary
When LDAP sync is turned on and a Picture attribute is selected, the user will not be able to change the profile picture. Instead they will be displayed a message that the picture is controlled by LDAP settings. This is the same way other LDAP Attributes work.
Ticket Link
https://mattermost.atlassian.net/browse/MM-24086
Screenshots

/update-branch

/update-branch

/cherry-pick release-5.24


add doc summary for getUniqueID

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
no code change

Fixed issues

This way it won't be empty in the table of methods.

Thank you


unpipe() is not implemented

Per convo with @mafintosh:
We're missing a Stream.prototype.unpipe() method for completeness and interop with other modules.

Ya let's add it

We really need this for gulp and the underlying stream utilities. This specifically came up when trying to implement streamx in the lead test suite


Merge initial create-t3-app prototype

Initial working prototype of the viewer concept. Adapted from webvtt-player and converted to functional, Typescript-format. Generates explorable metadata and transcriptions from supplied VTT files.

This is extremely gross right now! Only merging with master so that Vercel can deploy it smoothly. Massive audit + type fixing spree is necessary before more serious feature work.


Feature authorization

In this Milestone we did the the bellow project  requirements:

Install CanCanCan in your project.
Add a role column to the users table. Remember to use a migration for this.
A user can delete a post if it is theirs or if they have an admin role (column role has value "admin"). Use CanCanCan for this authorization.
For that you need to implement the post deleting functionality. Add the "Delete" button to the view and make sure that only authorized users can see it.
A user can delete a comment if it is theirs or if they have an admin role (column role has value "admin"). Use CanCanCan for this authorization.
For that you need to implement the comment deleting functionality. Add the "Delete" button to the view and make sure that only authorized users can see it.
Fix Linrer issues

Hi @Graciano1997 and @sheezu900,  it would be nice to not make commits when you submit your project for review as your recent commit broke your linter which is now failing

@shella12  thank you for recognizing all our efforts ... 🙏 🙏 🙏 🙏 🙏


egui_plot: looking for maintainer

egui_plot was originally written by me, then added to by a lot of people, including @Bromeon and @EmbersArc.
However, there is too many PR to this repository for me too keep up with, so I'd like to move out egui_plot to its own repository, with a new maintainer.

Wrong repository 🤦  Moved to https://github.com/emilk/egui/issues/4705


Decrease the verbosity of wget output in build logs

Addresses a new hadolint lint DL3047.
https://github.com/hadolint/hadolint/wiki/DL3047

Filed https://github.com/postmodern/ruby-install/issues/397 to see if this change can be made in ruby-install internals too.


python: ../nptl/pthread_mutex_lock.c:81: __pthread_mutex_lock: Assertion `mutex->__data.__owner == 0' failed.

when i ran my cli:
python wenet/bin/train.py --gpu 1 --config ./wenetspeech_train_conformer_bidecoder.yaml --data_type raw --symbol_table ./med_unit.txt --train_data ./data.list --cv_data ./data.list --model_dir ./model_saved --cmvn ./global_cmvn --num_workers 8
and i got issues:
python: ../nptl/pthread_mutex_lock.c:81: __pthread_mutex_lock: Assertion `mutex->__data.__owner == 0' failed.
Aborted (core dumped)

my cuda version is cuda11.3, torch version is 1.12.1
here is my cli history,
firstly, i ran in 1 3080ti:
python wenet/bin/train_wenetspeech.py \
--gpu 1 \
--config ./wenetspeech_train_conformer_bidecoder.yaml \
--data_type raw \
--symbol_table ./med_unit.txt \
--train_data ./data.list \
--cv_data ./data.list \
--model_dir ./model_saved \
--cmvn ./global_cmvn \
--num_workers 8
it can calculate loss, but OOM immediately

then i want to use 2 3080ti with cli below, and i had no idea what is ddp.init_method, so i just copied "file:///export/nfs/ddp_init " here:
python wenet/bin/train_wenetspeech.py \
--gpu 1 \
--config ./wenetspeech_train_conformer_bidecoder.yaml \
--data_type raw \
--symbol_table ./med_unit.txt \
--train_data ./data.list \
--cv_data ./data.list \
--model_dir ./model_saved \
--cmvn ./global_cmvn \
--num_workers 8 \
--checkpoint ./pretrained_model_list/wenetspeech-cn-conformer-checkpoint/20220506/final.pt \
--ddp.init_method file:///export/nfs/ddp_init \
--ddp.world_size 2 \
--ddp.rank 0 \
--ddp.dist_backend nccl

then i found it took too long, so i forced to stop it, and ran again, no matter what parameters are set,
python: ../nptl/pthread_mutex_lock.c:81: __pthread_mutex_lock: Assertion `mutex->__data.__owner == 0' failed.
Aborted (core dumped)

still raised
what s that, any help?

I encountered the same problem with K2. Maybe it is an environment problem.

I encountered the same problem with K2. Maybe it is an environment problem.

thanks for reply.


forwarded key events vs. blocked key events

Hello,
I've following situation.
I currently use JIntellitype which doesn't forward captured key events.
With system-hook I recognized, that key events are forwarded to following possible listeners.
Both has pros and cons. Is there a possibility to make it configurable?
Means, if I set the HotKey I could also set the param if the HotKey will be captured only by my application or stays alive and can be captured by other applications.
Could be a nice new feature.

Hello @luecking, I think this is a good idea. I just checked the documentation to confirm calling CallNextHookEx is indeed optional. But please be aware: Whether you get or not get the events, depends on the order of hooks registered. It could be that another application registered the hook, which will lead to your hook NOT beeing triggered and vice versa. I will see what I can do.

Done with #fc7691603677c28cc0bcf7a8f6c8307058bc9985 and release 3.5


Refactor SMT, MPT decoder

Refactor SMT, MPT decoder parts for better distinction, Instruction enum separation etc.

Is this issue still meaningful? Given the frontend + backend refactoring of @0xaatif and other SMT-related tracking issues?

https://github.com/0xPolygonZero/zk_evm/issues/275 covers this


[WIP] Expand profile-launcher configs to support the generic docker-run script

Add more configuration into profile-launcher (pl) so that it can support the generic docker-run script that Brian is working on.
PR Checklist

[ ] Added label to the Pull Request for easier discoverability and search
[ ] Commit Message meets guidelines as indicated in the URL https://github.com/intel-retail/automated-self-checkout/blob/main/CONTRIBUTING.md
[ ] Every commit is a single defect fix and does not mix feature addition or changes
[ ] Unit Tests have been added for new changes
[ ] Updated Documentation as relevant to the changes
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with repository license and clearly outlined the added dependency.
[ ] PR change contains code related to security
[ ] PR introduces changes that breaks compatibility with other modules (If YES, please provide details below)

What are you changing?

Issue this PR will close
close: #issue_number
Anything the reviewer should know when reviewing this PR?
Test Instructions if applicable

If the there are associated PRs in other repositories, please link them here (i.e. intel-retail/automated-self-checkout )

rebased to resolve the conflicts from main


Multi-signature Solution

Issue To Be Solved
Problem
Account transactions require a key weight requirements be met. Generally one key will have enough weight to execute a TX but for "multi-sig" transactions a tool is needed to coordinate signatures within the applicable time window. This window is currently estimated at 10 minutes or so.
Suggested Solution
Current Solution
E-mailing/messaging signed messages around. Presumably this is the same method the ancient Romans used during their campaigns in gaul.
Proposed Solution
A SPA with realtime message based backend that will coordinate the above. A proposer will create a voucher that will be stored server side and be given a deep link to pass to others to allow them to sign.

I think it would be nice to have this on chain. https://github.com/onflow/service-account/issues/66

Issue moved to onflow/flow-multisig #19 via ZenHub


⚠ Bofrost.at has degraded performance

In d1114e0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2203 ms

Resolved: Bofrost.at performance has improved in c2ec7e2.


feat(logging): add workspace name in the logging payload

Description
This PR adds workspace name in the logging payload.
(https://konghq.atlassian.net/browse/FTI-4728)
Testing instructions
Netlify link:
Checklist

[x] Review label added
[x] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

Leaving this PR open until the linked Kong PR is merged, otherwise it's good to go.


Use EventToCommand on Drawable

I'm using a MVVM pattern in my application and I want to use command on the event "Clicked" on drawable object in Mapsui.UI.Forms.
Did someone alerady did this or at least, it is possible ?
Thank's !

What exactly are you asking for? Clicked is not an ICommand but you ask how it could be used as a ICommand? Should it be wrapped in a command?

Yes that's it. I want to use it as a Command.

You could do this via a Behavior. See here.


Add Macro Compatibility Check to the main workflow

This pull request introduces the Swift Macro Compatibility Check GitHub Action into the CI workflow. This action verifies that the Swift package remains compatible with multiple versions of swift-syntax, which is critical for macro-based functionality.
Motivation for Adding This Check:

swift-syntax Versioning Complexity: As detailed by Point-Free in their article, "Being a good citizen in the land of SwiftSyntax", swift-syntax employs a versioning scheme where minor versions of Swift correspond to major versions of swift-syntax (e.g., SwiftSyntax 509.0 maps to Swift 5.9). This scheme introduces complexity in maintaining compatibility across versions.

Frequent Breaking Changes: Minor releases of swift-syntax have introduced breaking changes, which can cause issues with existing macros and dependencies. This action helps ensure that our macros remain functional across these versions.

Dependency Graph Resolution: By testing against multiple versions, we can prevent dependency conflicts in the broader Swift ecosystem. This is essential, especially as more libraries are adopting macros and using swift-syntax, increasing the likelihood of conflicting dependencies.

Proactive Development: Catching compatibility issues early in the development cycle allows us to resolve them before they become critical, reducing the risk of blocked releases or difficult debugging sessions later.

[!IMPORTANT]
To ensure the library can be used alongside others in the Swift packages, it need to support all versions of swift-syntax. Without this compatibility, users may face issues where they can't use several libraries together if they depend on different versions of swift-syntax.

Hi Mateusz, thanks for creating that useful reusable action, and for adding it to the toolkit’s CI!


ui: v0.9.0 images and drop envoy proxy container

Previously we used envoy proxy container to convert grpc-web traffic to
true grpc traffic, so ui backend accepts real grpc traffic. There is
an alternative approach to use special wrapper
for grpc service on backend:
https://github.com/improbable-eng/grpc-web/tree/master/go/grpcweb
Related code changes on hubble-ui itself was implemented in this PR:
https://github.com/cilium/hubble-ui/pull/226
This also bumps UI images to v0.9.0 with actual implementation.
Signed-off-by: Dmitry Kharitonov<EMAIL_ADDRESS>

@geakstr @sayboras are all changes good to merge or is there anything missing?


Draggable element is not inside of the node passed into cdkDragBoundary.

Hi everyone,
Is there a specific reason for this check drag.ts#L363-L366?
    if ((typeof ngDevMode === 'undefined' || ngDevMode) &&
      !element.contains(this.element.nativeElement)) {
      throw Error('Draggable element is not inside of the node passed into cdkDragBoundary.');
    }

The thing is it throws the error only in devMode, but it works perfectly fine in production. I think those lines can be removed.
Reproduction
Steps to reproduce:

Use the following html markup

...

run application in dev mode (ng serve)
Drag element: error appears in console log, draggable's position is not constrained
Build app for production (ng build --configuration production)
Drag element: no errors in console log, draggable's position is constrained

Expected Behavior
Same (production) behavior for dev environment
Actual Behavior
The following exception is thrown (in ngDevMode only)
Uncaught Error: Draggable element is not inside of the node passed into cdkDragBoundary.
    at CdkDrag._getBoundaryElement (drag-drop.js:3418)
    at SafeSubscriber._next (drag-drop.js:3446)
    at SafeSubscriber.__tryOrUnsub (Subscriber.js:183)
    at SafeSubscriber.next (Subscriber.js:122)
    at Subscriber._next (Subscriber.js:72)
    at Subscriber.next (Subscriber.js:49)
    at Subject.next (Subject.js:39)
    at HTMLDivElement.DragRef._pointerDown (drag-drop.js:411)
    at ZoneDelegate.push../node_modules/zone.js/dist/zone.js.ZoneDelegate.invokeTask (zone.js:434)
    at Zone.push../node_modules/zone.js/dist/zone.js.Zone.runTask (zone.js:205)
_getBoundaryElement @ drag-drop.js:3418

Environment

Angular: 12.2.10
CDK/Material:  12.2.9
Browser(s): any
Operating System (e.g. Windows, macOS, Ubuntu): any

The whole point of a boundary is to constrain the position of an element. It's pointless for it to not be an ancestor.

Hi crisbeto, I can't agree at this point.
In my case, an cdk-overlay element is constrained but boundary element and it is not an ancestor. Moreover everything works fine (but not in devMode), because only client rect is used for that containing, so element doesn't have to be an ancestor.
Thanks


bug: error when reindexing after upgrading to 0.2

When calling REINDEX INDEX on an HNSW index after upgrading and restarting, it throws the error internal error: entered unreachable code. Restarting Postgres again doesn't seem to have an effect. Brute-force search still works.
Steps leading up to this:
START TRANSACTION;

ALTER DATABASE immich SET search_path TO "$user", public, vectors;
CREATE SCHEMA IF NOT EXISTS vectors; -- this step is required for the ALTER EXTENSION line to work
UPDATE pg_catalog.pg_extension SET extversion = '0.1.11' WHERE extname = 'vectors';
UPDATE pg_catalog.pg_extension SET extrelocatable = true WHERE extname = 'vectors';
ALTER EXTENSION vectors SET SCHEMA vectors;
UPDATE pg_catalog.pg_extension SET extrelocatable = false WHERE extname = 'vectors';
ALTER EXTENSION vectors UPDATE TO '0.2.0-alpha.1';
SELECT pgvectors_upgrade();

COMMIT;

-- throws an error "The extension is upgraded so all index files are outdated." - not sure if intended behavior
SELECT idx_status FROM pg_vector_index_stat WHERE indexname = 'clip_index';

After restarting:
ALTER DATABASE immich SET search_path TO "$user", public, vectors;

-- throws a different error "The index is not existing in the background worker."
-- not sure if intended behavior, but we handle this error and continue
SELECT idx_status FROM pg_vector_index_stat WHERE indexname = 'clip_index';

REINDEX INDEX clip_index; -- throws 'internal error: entered unreachable code'

I also see instances of this in the logs:
ERROR:  pgvecto.rs: The index is not existing in the background worker.
ADVICE: Drop or rebuild the index.
CONTEXT:  while vacuuming index "face_index" of relation "public.asset_faces"`

This log doesn't show up for clip_index, only for face_index. The app doesn't call REINDEX INDEX face_index since doing this for clip_index throws an error first, so my hunch was that it was related to this. But these logs continue to appear after manually calling REINDEX INDEX face_index.
Calling SELECT idx_status FROM vectors.pg_vector_index_stat throws ERROR: called `Result::unwrap()` on an `Err` value: NoSuchType("vector_index_stat")
I can do some more testing to see if I can find the root cause. I wonder if it's because I ran the first part in a transaction?
Edit: Not running in a transaction produces the same result.
Edit 2: Dropping the two indices makes the SELECT idx_status FROM vectors.pg_vector_index_stat query return 0 rows successfully. But attempting to create the indices leads to internal error: entered unreachable code as well.
Edit 3: I noticed these logs after running SELECT pgvectors_upgrade():
WARNING:  pgvecto.rs is upgraded. Restart PostgreSQL to take effects.
immich_server            | [Nest] 8  - 01/30/2024, 5:32:25 AM    WARN [DatabaseService]
immich_server            |           The pgvecto.rs extension has been updated to 0.2.0.
immich_server            |           Please restart the Postgres instance to complete the update.
immich_postgres          | 2024-01-30 05:32:25.673 UTC [41] ERROR:  pgvecto.rs: The extension is upgraded so all index files are outdated.
immich_postgres          |      ADVICE: Delete all index files. Please read `https://docs.pgvecto.rs/admin/upgrading.html`.
immich_postgres          | 2024-01-30 05:32:25.673 UTC [41] STATEMENT:  COMMIT
immich_postgres          | 2024-01-30 05:32:25.675 UTC [41] ERROR:  pgvecto.rs: The extension is upgraded so all index files are outdated.
immich_postgres          |      ADVICE: Delete all index files. Please read `https://docs.pgvecto.rs/admin/upgrading.html`.
immich_postgres          | 2024-01-30 05:32:25.675 UTC [41] WARNING:  AbortTransaction while in ABORT state
immich_postgres          | 2024-01-30 05:32:25.676 UTC [41] ERROR:  pgvecto.rs: The extension is upgraded so all index files are outdated.
immich_postgres          |      ADVICE: Delete all index files. Please read `https://docs.pgvecto.rs/admin/upgrading.html`.
immich_postgres          | 2024-01-30 05:32:25.676 UTC [41] WARNING:  AbortTransaction while in ABORT state
immich_postgres          | 2024-01-30 05:32:25.677 UTC [41] LOG:  could not send data to client: Broken pipe
immich_postgres          | 2024-01-30 05:32:25.677 UTC [41] ERROR:  pgvecto.rs: The extension is upgraded so all index files are outdated.
immich_postgres          |      ADVICE: Delete all index files. Please read `https://docs.pgvecto.rs/admin/upgrading.html`.
immich_postgres          | 2024-01-30 05:32:25.677 UTC [41] WARNING:  AbortTransaction while in ABORT state
immich_postgres          | 2024-01-30 05:32:25.678 UTC [41] ERROR:  pgvecto.rs: The extension is upgraded so all index files are outdated.
immich_postgres          |      ADVICE: Delete all index files. Please read `https://docs.pgvecto.rs/admin/upgrading.html`.
immich_postgres          | 2024-01-30 05:32:25.678 UTC [41] PANIC:  ERRORDATA_STACK_SIZE exceeded
immich_postgres          | 2024-01-30 05:32:25.679 UTC [1] LOG:  server process (PID 41) was terminated by signal 6: Aborted
immich_postgres          | 2024-01-30 05:32:25.679 UTC [1] DETAIL:  Failed process was running: COMMIT
immich_postgres          | 2024-01-30 05:32:25.679 UTC [1] LOG:  terminating any other active server processes
immich_postgres          | 2024-01-30 05:32:25.681 UTC [1] LOG:  all server processes terminated; reinitializing
immich_postgres          | 2024-01-30 05:32:25.690 UTC [47] LOG:  database system was interrupted; last known up at 2024-01-30 05:32:23 UTC
immich_postgres          | 2024-01-30 05:32:25.786 UTC [48] FATAL:  the database system is in recovery mode
immich_postgres          | 2024-01-30 05:32:28.195 UTC [47] LOG:  database system was not properly shut down; automatic recovery in progress
immich_postgres          | 2024-01-30 05:32:28.204 UTC [47] LOG:  redo starts at 0/7A00118
immich_postgres          | 2024-01-30 05:32:28.209 UTC [47] LOG:  invalid record length at 0/7ADC768: wanted 24, got 0
immich_postgres          | 2024-01-30 05:32:28.209 UTC [47] LOG:  redo done at 0/7ADC708 system usage: CPU: user: 0.00 s, system: 0.00 s, elapsed: 0.00 s
immich_postgres          | 2024-01-30 05:32:28.354 UTC [49] FATAL:  the database system is in recovery mode
immich_postgres          | 2024-01-30 05:32:28.498 UTC [1] LOG:  database system is ready to accept connections

This is with running each command as a separate transaction so I think the STATEMENT:  COMMIT is just referring to SELECT pgvectors_upgrade()? I'm not sure.
There are some interesting logs there, especially PANIC:  ERRORDATA_STACK_SIZE exceeded, FATAL:  the database system is in recovery mode and invalid record length at 0/7ADC768: wanted 24, got 0. I think this is probably the root cause.
Altering columns to real[], dropping the indices and extension, re-creating the extension and re-applying the vector type and indices does work.

ALTER DATABASE take effects in new sessions so you may need:
ALTER DATABASE immich SET search_path TO "$user", public, vectors;
SET search_path TO "$user", public, vectors;

The index is not existing in the background worker. is intended behavior and you may need to reindex them first.
The custom type in PostgreSQL (vector) requires no uninitialized bytes during construction. We previously overlooked this and have corrected it in version 0.1.12, also adding a check for any uninitialized bytes in the data. This check has caused an issue: older data is now considered corrupt by 0.2.0. I'm not sure if we should provide a patch to fix it automantically.
This does not affect altering the column to real[] and reinstalling the extension (setting search_path and executing SELECT pgvectors_upgrade() are still required). So we suggest this way now. We apologize for any inconvenience caused.
cc @VoVAllen

We provide a patch to fix it. Now the upgrade process should be:

CREATE SCHEMA IF NOT EXISTS vectors;
ALTER DATABASE immich SET search_path TO "$user", public, vectors;
SET search_path TO "$user", public, vectors;

UPDATE pg_catalog.pg_extension SET extversion = '0.1.11' WHERE extname = 'vectors';
UPDATE pg_catalog.pg_extension SET extrelocatable = true WHERE extname = 'vectors';
ALTER EXTENSION vectors SET SCHEMA vectors;
UPDATE pg_catalog.pg_extension SET extrelocatable = false WHERE extname = 'vectors';
ALTER EXTENSION vectors UPDATE TO '0.2.0-alpha.2';

SELECT pgvectors_upgrade();

restart PostgreSQL

Get all vector indexes.

SELECT
    C.relname AS tablename,
    I.relname AS indexname
FROM pg_class C JOIN
    pg_index X ON C.oid = X.indrelid JOIN
    pg_class I ON I.oid = X.indexrelid JOIN
    pg_am A ON A.oid = I.relam
WHERE A.amname = 'vectors';

 tablename | indexname
-----------+-----------
t          | t_vector_idx

For every row, if the table name is t, the column name is vector and the indexname is t_vector_idx, execute:
ALTER TABLE t ALTER COLUMN vector SET DATA TYPE real[];
ALTER TABLE t ALTER COLUMN vector SET DATA TYPE vector;
REINDEX t_vector_idx;

Everything should works now.


Add training page to whitehall

The training page for whitehall is being temporarily taken down to be edited.
Publishers will still need to complete the training in this time so we have created this page in whitehall to hold the training materials needed.
https://github.com/alphagov/whitehall/assets/17481621/9359bcbb-dbd9-41b8-8235-8545188e996a

Screenshot of page

Trello card: https://trello.com/c/UvYJCVnm/2288-create-page-in-whitehall-for-training-videos

Looks good 🌞 Just a couple of comments which I've left inline.
Also the format of "Video: rest of the title" seems a bit awkward but if that's what content want to do we can do it.


[Error] Logstash 8.11.3 pipeline.conf (logstash.conf) issue

When trying to run the logstash portion of the compose it throws an error about the formatting of the .conf file. I am using the file that is provided in this repo and haven't made any changes to it.
Logstash error:
elastic-stack-logstash01-1    | [2023-12-23T05:23:21,079][INFO ][logstash.agent           ] Successfully started Logstash API endpoint {:port=>9600, :ssl_enabled=>false}
elastic-stack-es01-1          | {"@timestamp":"2023-12-23T05:22:36.930Z", "log.level": "INFO", "message":"loaded module [x-pack-aggregate-metric]", "ecs.version": "1.2.0","service.name":"ES_ECS","event.dataset":"elasticsearch.server","process.thread.name":"main","log.logger":"org.elasticsearch.plugins.PluginsService","elasticsearch.node.name":"es01","elasticsearch.cluster.name":"docker-eck"}
elastic-stack-logstash01-1    | [2023-12-23T05:23:21,097][ERROR][logstash.agent           ] Failed to execute action {:action=>LogStash::PipelineAction::Create/pipeline_id:main, :exception=>"LogStash::ConfigurationError", :message=>"Expected one of [ \\t\\r\\n], \"#\", \"input\", \"filter\", \"output\" at line 1, column 1 (byte 1)", :backtrace=>["/usr/share/logstash/logstash-core/lib/logstash/compiler.rb:32:in `compile_imperative'", "org/logstash/execution/AbstractPipelineExt.java:239:in `initialize'", "org/logstash/execution/AbstractPipelineExt.java:173:in `initialize'", "/usr/share/logstash/logstash-core/lib/logstash/java_pipeline.rb:48:in `initialize'", "org/jruby/RubyClass.java:931:in `new'", "/usr/share/logstash/logstash-core/lib/logstash/pipeline_action/create.rb:49:in `execute'", "/usr/share/logstash/logstash-core/lib/logstash/agent.rb:386:in `block in converge_state'"]}
elastic-stack-logstash01-1    | [2023-12-23T05:23:21,104][INFO ][logstash.runner          ] Logstash shut down.
elastic-stack-logstash01-1    | [2023-12-23T05:23:21,107][FATAL][org.logstash.Logstash    ] Logstash stopped processing because of an error: (SystemExit) exit
elastic-stack-logstash01-1    | org.jruby.exceptions.SystemExit: (SystemExit) exit
elastic-stack-logstash01-1    | 	at org.jruby.RubyKernel.exit(org/jruby/RubyKernel.java:808) ~[jruby.jar:?]
elastic-stack-logstash01-1    | 	at org.jruby.RubyKernel.exit(org/jruby/RubyKernel.java:767) ~[jruby.jar:?]
elastic-stack-logstash01-1    | 	at usr.share.logstash.lib.bootstrap.environment.<main>(/usr/share/logstash/lib/bootstrap/environment.rb:90) ~[?:?]

Any recommendations for trying to fix this issue?

I've figured it out, logstash needs to know where is the pipeline .conf file is, relevant snipet from docker-compose.yml:
  logstash01:
    depends_on:
      es01:
        condition: service_healthy
      kibana:
        condition: service_healthy
    image: docker.elastic.co/logstash/logstash:${STACK_VERSION}
    labels:
      co.elastic.logs/module: logstash
    user: root
    volumes:
      - certs:/usr/share/logstash/certs
      - logstashdata01:/usr/share/logstash/data
      - "./logstash_ingest_data/:/usr/share/logstash/ingest_data/"
      - "./logstash.conf:/usr/share/logstash/pipeline/logstash.conf:ro"
    environment:
      - xpack.monitoring.enabled=false
      - ELASTIC_USER=elastic
      - ELASTIC_PASSWORD=${ELASTIC_PASSWORD}
      - ELASTIC_HOSTS=https://es01:9200
    **command:
      - -f
      - /usr/share/logstash/pipeline/**

thanks for the heads up.  i'll test the one out.

i am unable to recreate this error and do not need to specify the -f command as this is taken care of by passing in the logstash.conf to the default pipeline directory.

@elkninja probably because this issue is related to version 8.11.3 and above. I had the same error message with 8.12.0.


Don't require all minted assets to be spent or burned

Issue Number
ADP-1113
Summary
This PR removes the restriction that all minted assets must be spent or burned.
Minted assets that are not spent or burned are now returned in the change outputs.

Yay! This is great :) Very cathartic.

bors r+

bors r+


Add 'PastelWheel' palette

Purpose:

Creates a new palette with light colors as per #2266

#f8c5c7 #fadec3 #fbf6c4 #e1ecc8 #d7e8cb #daebd7 #d9eef3 #cadbed #c7d2e6 #d4d1e5 #e8d3e6 #f8c7de

Whoops, it looks like there's an extra character in one of those hex codes... I'll fix that up and merge

Whoops, it looks like there's an extra character in one of those hex codes... I'll fix that up and merge

Thank you! 👍


Less Accurate Location Code

Is there a way to broaden/obscure a given location?

Codes with fewer digits represent a larger area. For example, 87G7MXQ4+M5 is the full code for the Statue of Liberty in New York. Shortening it to 87G7MXQ4 will represent a large rectangle that is centered at the same point (doesn't look like the plus.codes UI will let you do this directly though).
What's your use case? If you want to obscure the location a bit, you could theoretically pick a random neighbor.

To add to what @zongweil said, shortening is possible even beyond eight digits, by replacing pairs of digits with '0':
87G7MX00+
87G70000+
87000000+

Hey, my use case is I want to be able to get user location and obscure it before encrypting it and storing it onto a database

@zongweil Hey, my use case is I want to be able to get user location and obscure it before encrypting it and storing it onto a database

@bocops That's really helpful! Do they have documentation for doing so?

@alextechtai This is mentioned as "padding" in

https://github.com/google/open-location-code/blob/master/docs/olc_definition.adoc and
https://github.com/google/open-location-code/blob/master/docs/specification.md


Unable to execute script at line 40. Expected: }

Hi I received this error - I am using Adobe AE 24.2.1 (Build 2). all the best

Hi I received this error - I am using Adobe AE 24.2.1 (Build 2). all the best

Jakartaguy, did you get around this?  I got it once, but was fine with a different KML


Windows - Link libraries fixed

Reference: #9
Required here: mingw-w64-expresscpp

👍, thank you for the contribution

+1, thank you for the contribution

https://packages.msys2.org/base/mingw-w64-expresscpp


WIP: Make classes customizable

Declared some functions as protected and added also some optional parameter.
Also made the WPF Image clickable. So if you click an image the associated command will be executed. In my case to zoom into it.

I implemented your suggestions and added the "Markdig.Wpf.SampleAppCustomized" project as u wished.
I also changed the copy property of the benchmark image. Now it gets always copied.

Thanks. I'll try to review this week but I'm quite busy and I'm not available this weekend. So it might take a while.

Thanks. I'll try to review this week but I'm quite busy and I'm not available this weekend. So it might take a while.

Would be awesome, if u can publish it on NuGet till the end of the week. If this is possible :)


Test with new data/matches

I made a new version of the test data with a perfect matches file, here: https://github.com/yale-datachemist/entity-resolution/tree/main/data/test (benchmark_data_matches.csv and benchmark_data_records.jsonl).
Could we test this with the VectorLink classifier algorithm?

[ ] The record string is preformatted, so could we skip the handlebars stage for it? The other fields could still be formatted using handlebars.
[ ] We need a strategy for dealing with null fields, so that they don't skew the weights. Name and title are the only strictly required fields. The preformatted record string will never include null fields.

I'm also wondering, should there be a categorical variable based on the person field? I'm thinking of the case when there are multiple names derived from the same source record--the extracted data will typically be identical, except for the name, which is the key differentiating value.

Or many the person name should just be the blocking field, rather than the whole record string.

The null field thing is tricky - to be honest I'm not 100% sure what to do with this - possibly just placing some small random vector in is the right thing, but I'm not completely sure.
Skipping the templating phase is possible but would require a code change. Probably easiest is to create a template which is just a passthrough - i.e. just prints the string.
We can try to get this working with the new scalable code we're using now, or we can have a meeting with you and try to help you kick off the process with the old method.

@GavinMendelGleason, would some kind of simple imputation be sufficient (e.g., see https://chatgpt.com/share/67572fb2-2234-800d-84f5-d8101da01943)?
Passthrough makes sense--I'll try that.
What is the new scalable code? Is it different from the example tutorial on vectorlink.ai?


Question, more than issue. Every scan results says "Website crawl (100 pages)". Is Purple A11Y limited to 100?

As the title mentions, all of my scan results say "Website crawl (100 pages)" on the left side of the HTML report. Is Purple A11Y limited to 100 pages? What concerns me is that it hasn't varied at all. I would expect some sites to have 98, others 130, etc... Is there a JS/Config file I can check or edit how many pages are scanned? Thanks in advance! I appreciate all you do for this app!

Thanks,
Jack

It is limited to 100 pages when running from npm run start . You can increase this limit by using npm run cli ... as documented at https://github.com/GovTechSG/purple-a11y?tab=readme-ov-file#cli-mode

@younglim Okay, thank you! We never realized that.


clusterserviceversion.yaml.tpl still contains reference to ServiceID in code-generator

Describe the bug
A concise description of what the bug is.
Commit 15c7f5c730d71f54af0f42499d7f9196248a375f in code-generator removed the ServiceID metadata variable which is still being used in once in clusterserviceversion.yaml.tpl. This breaks the OLM build-controller process.
Steps to reproduce
Attempt to ACK_GENERATE_OLM=true make build-controller SERVICE=$SERVICE with latest main commits.
Expected outcome
A concise description of what you expected to happen.
Build should work without issues.
Environment

Kubernetes version
Using EKS (yes/no), if so version?
AWS service targeted (S3, RDS, etc.)
Should be an issue on any service.

Hi @trgeiger , there is already a PR open which brings ServiceID back
https://github.com/aws-controllers-k8s/code-generator/pull/213 so this OLM build will fix as soon as that PR merges.

For future changes to catch these errors before PR gets merged, I will add OLM tests that will run on every code-generator PR submission. This will make sure OLM functionality is not broken by the PR changes.

Thanks @vijtrip2 let us know if you want us to review the tests you create.

Hi @acornett21 , does following qualify as a successful test ? If yes, then I can add success of this command as validation in PR tests.
➜  code-generator git:(main) ACK_GENERATE_OLM=true make build-controller SERVICE=$SERVICE
building ack-generate ... ok.
==== building s3-controller ====
Copying common custom resource definitions into s3
Building Kubernetes API objects for s3
Generating deepcopy code for s3
Generating custom resource definitions for s3
Building service controller for s3
Generating RBAC manifests for s3
Running gofmt against generated code for s3
Updating additional GitHub repository maintenance files
==== building s3-controller release artifacts ====
Building release artifacts for s3-v0.0.5
Generating common custom resource definitions
Generating custom resource definitions for s3
Generating RBAC manifests for s3
Generating operator lifecycle manager bundle assets for s3
Generating bundle version 0.0.5
Generating bundle manifests
WARN[0000] ClusterServiceVersion validation: [OperationFailed] provided API should have an example annotation
Bundle manifests generated successfully in bundle
Generating bundle metadata
INFO[0000] Creating bundle.Dockerfile
INFO[0000] Creating bundle/metadata/annotations.yaml
INFO[0000] Bundle metadata generated suceessfully
INFO[0000] Found annotations file                        bundle-dir=../s3-controller/olm/bundle container-tool=docker
INFO[0000] Could not find optional dependencies file     bundle-dir=../s3-controller/olm/bundle container-tool=docker
WARN[0000] Warning: Value services.k8s.aws/v1alpha1, Kind=AdoptedResource: provided API should have an example annotation
INFO[0000] All validation tests have completed successfully

I used the s3 controller, but is there a candidate service-controller which you would like me to add for OLM functionality verification?
I don't think any of the controller currently have "/olm/olmconfig.yaml". Do you add that config file first and then generate the controller?

@vijtrip2 we have been testing with s3 and elasticache I think testing with either is fine. From an OLM prospective as long as the last line is INFO[0000] All validation tests have completed successfully this indicates a success, the WARNS in the middle we tend to ignore since the service API's (ie the actually s3 api's) move faster then the yamls everyone is pre-staging in the various controllers.
For the second question, we have /olm/olmconfig.yaml prestaged in a few repos, so that is a static file that needs to be present inside a repo before we generate the OLM bundle.
Please let us know if anything needs further clarification! Thanks again for the help.

Hi @trgeiger , there is already a PR open which brings ServiceID back aws-controllers-k8s/code-generator#213 so this OLM build will fix as soon as that PR merges.
For future changes to catch these errors before PR gets merged, I will add OLM tests that will run on every code-generator PR submission. This will make sure OLM functionality is not broken by the PR changes.

That's great news. Does #214 conflict with that or pose any issues? Or is it just complementary?

Whoops, mis-clicked!

Thanks @acornett21

we have been testing with s3 and elasticache

Great, I will add the test to generate s3-controller with ACK_GENERATE_OLM = true

we have /olm/olmconfig.yaml prestaged in a few repos, so that is a static file that needs to be present inside a repo before we generate the OLM bundle.

Is this necessary step? If yes, Can you please add this file in s3-controller and raise a PR for it? @jaypipes , will that be alright?
If adding olmconfig.yaml files in main branch of s3-controller is not an option, I can add it in a separate GH branch and test can curl the file and add that file in repo during test.
Can you please paste the content of olm/olmconfig.yaml for s3-controller?

@vijtrip2 The file is in main in the s3-controller controller already. We might be talking about different things, the file I am talking about is below.
https://github.com/aws-controllers-k8s/s3-controller/blob/main/olm/olmconfig.yaml


Replace Gin with standard lib HTTP handler

This change addresses the issues raised by @sonyafenge in #378.
The observations were correct and the data collected really helped me grok the issue at hand. The problem however turned out to be caused by the usage of multiple response writers which caused the writer to not be identified as a reader from during runtime. This in turn meant that Go would fall back to using a in memory buffer when copying blobs to the response writer instead of using a zero  copy system call. While #385 did increase the performance it also did not solve the core problem source. I do however appreciate the effort as it helped understand the problem.
We could have fixed this in Gin, but considering that a PR to do this has been open for over a year an attempt from me would probably have the same effect. Instead I have decided to completely remove Gin and instead implement our own response writer wrapper which does implement the reader from interface. This allows us to decrease the amount of dependencies and remove some complexity. As the registry component is pretty simple I see little use for complex routing logic.

@sonyafenge do you have the time to verify that this change fixes your issues?

I have determined that this change will not fix the issues, it will however bring us closer to the problem. The PR will no longer fix the issue. Hopefully I will be able to solve the read method in Containerd. Either way there is no use delaying merging this.


[WiP] Updates for in-cluster RR

Description

Adjust the BGP syncer to (configurably) watch all block affinities.  This is needed for Typha since Typha needs to service all clients from one cache.

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Replaced by / included in #919


Update to latest manifestival to apply strategic merge patches to its resources.

Fixes #226 #202 #160
Makes things a little more straightforward to implement whatever direction comes from #163
Proposed Changes

The latest manifestival performs the same strategic merging used by kubectl apply
Except for ConfigMap -- any change to the KnativeServing config entries will overwrite the corresponding ConfigMaps
Superfluous updates of Deployment and Image resources should be fixed
Diffs are now logged so it's easier to see why a resource is being updated

Release Note

The serving operator will now apply [strategic merging](https://kubernetes.io/docs/tasks/manage-kubernetes-objects/declarative-config/#merge-patch-calculation) of its resources to any changes made that aren't a part of its embedded manifest should not be overwritten.

How extensive is the testing that has been done here?

I feel good about the coverage. All of the testcases but one in there are carried over from the previous code in which I was manually calculating the patches. All of those cases ultimately run this line. And that library of course runs its own extensive tests. And even though only ConfigMaps in the k-s manifest actually follow this config path, I still carried all those tests over just to make sure I didn't break anything.
The last testcase is a verbatim ripoff of this example. Because its kind is a registered GVK (as are all the resources in the k-s manifest), it ultimately runs this line. And that's the same function kubectl runs, and its library also has extensive tests.

/lgtm


hp2p: 3.3 -> unstable-2023-10-25

Description of changes
Rewrite hp2p in C
Plotly visualisation is directly generated by hp2p. Python is not required anymore
CUDA version to test bandwidth between NVIDIA GPU
hp2p can generate binary output again
Signals can be use during an hp2p run for generating output
Print a warning during the run if communication is slow

EDIT: Description of changes above are from upstream hp2p
Upgraded to master branch because 4.0 causes a a core dump
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

The pr title and the commit should be: hp2p: 3.3 -> unstable-2023-10-25 and squashed into one commit :)

The pr title and the commit should be: hp2p: 3.3 -> unstable-2023-10-25 and squashed into one commit :)

Done. Thank you!

Did you add yourself as a maintainer and add yourself to the maintainer list under the maintainer folder?

Did you add yourself as a maintainer and add yourself to the maintainer list under the maintainer folder?

I upgraded locally to use the new version and pushed the changes so other people don't have to do the same, but I don't really plan to maintain the package.

It's ok for me, but maybe ask upstream to publish a 4.1 ?

They probably have their own release schedule and the amount of commits since 4.0 is pretty short.

Need to squash this into one commit

Need to squash this into one commit

done

@gesperon You only updated the PR title and not the commit message, I squash merged and corrected the commit message but in the future make sure both match.


ucentral-schema: Fix ucentral_apply.sh

Pass the config to ucentral.uc
Fixes: 0b9087bd440c ("ucentral: development update")
Signed-off-by: Brett Mastbergen<EMAIL_ADDRESS>

this is an old script ... please use following call signature " /usr/share/ucentral/ucentral.uc /etc/ucentral/examples/default.json"
I'll remove the old libexec script tomorrow, sorry forgot it even existed


Update selenium-browserbot.js

e.g. self.assertEqual("", driver.find_element_by_id("site-my-manag").get_attribute("id=site-my-manage"))
id=site-my-manag
id=site-my-manage

Thank you for the PR. Could you please provide a more detailed description of the changes?

Resolve element not found when using assertattribute

Thank you for the PR. Could you please provide a more detailed description of the changes?

Resolve element not found when using assertattribute


fix #2189 - remove current class from JsonSubTypes

The current class was being considered when resolving the subtypes.

Thanks a lot! included and replaced by  #3298


filter only for poly-X but nothing else

What would be the command-line to run AfterQC in order to filter only for poly-X reads but nothing else?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenGene/AfterQC/issues/31#issuecomment-351243801,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJpN-HRLOHZEgzVveKWmrf8YN8GMNsJks5s_x3egaJpZM4Q_IG5
.

Just to clarify, what I intend to do is to filter out the read pairs that look like polyA/T/C/G, but not trim or get rid of the reads that don't look like polyA/T/C/G but have low Q-scores or other issues.

use following option to disable other trimmer / filters
-f0 -t0 -q0 -n1000 -s1000

Brilliant, thanks!
On Wed, Dec 13, 2017 at 10:08 AM, Shifu Chen<EMAIL_ADDRESS>wrote:

use following option to disable other trimmer / filters
-f0 -t0 -q0 -n1000 -s1000
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenGene/AfterQC/issues/31#issuecomment-351344359,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJpN_TrHfAkdGGA9shdCfM0IubIYF-2ks5s_6ICgaJpZM4Q_IG5
.

I tried it with the command-line below, but it results in this:

/home/avilella/AfterQC/after.py --no_overlap -f0 -t0 -q0 -n1000 -s1000 --read1_file=/home/avilella/CEGX_Run499/CEGX_Run499_enigma-12345678/CEG67_125_3-77840783/CEG67-125-3_S3_L001_R1_001.fastq.gz  --read2_file=/home/avilella/CEGX_Run499/CEGX_Run499_enigma-12345678/CEG67_125_3-77840783/CEG67-125-3_S3_L001_R2_001.fastq.gz --good_output_folder=/home/avilella/CEGX_Run499/CEGX_Run499_enigma-12345678/CEG67_125_3-77840783//afqc
Traceback (most recent call last):
File "/home/avilella/AfterQC/after.py", line 228, in
main()
File "/home/avilella/AfterQC/after.py", line 222, in main
processOptions(options)
File "/home/avilella/AfterQC/after.py", line 175, in processOptions
filter.run()
File "/home/avilella/AfterQC/preprocesser.py", line 773, in run
self.addFiguresToReport(reporter)
File "/home/avilella/AfterQC/preprocesser.py", line 788, in addFiguresToReport
reporter.addFigure('Read1 per base discontinuity after filtering', self.r1qc_postfilter.discontinuityPlotly("r1_post_discontinuity", 'Read1 discontinuity curve after filtering'), 'r1_post_discontinuity', "")
File "/home/avilella/AfterQC/qualitycontrol.py", line 234, in discontinuityPlotly
json_str += "var layout={title:'" + title + "', xaxis:{title:'cycles'}, yaxis:{title:'discontinuity', range:" + makeRange(0.0, max(self.meanDiscontinuity)*1.5) + "}};\n"
ValueError: max() arg is an empty sequence

I tried it with the command below, but it results in this:

# /home/avilella/AfterQC/after.py --no_overlap -f0 -t0 -q0 -n1000 -s1000 --read1_file=/home/avilella/CEGX_Run499/CEGX_Run499_enigma-12345678/CEG67_125_3-77840783/CEG67-125-3_S3_L001_R1_001.fastq.gz  --read2_file=/home/avilella/CEGX_Run499/CEGX_Run499_enigma-12345678/CEG67_125_3-77840783/CEG67-125-3_S3_L001_R2_001.fastq.gz --good_output_folder=/home/avilella/CEGX_Run499/CEGX_Run499_enigma-12345678/CEG67_125_3-77840783//afqc
Traceback (most recent call last):
  File "/home/avilella/AfterQC/after.py", line 228, in <module>
    main()
  File "/home/avilella/AfterQC/after.py", line 222, in main
    processOptions(options)
  File "/home/avilella/AfterQC/after.py", line 175, in processOptions
    filter.run()
  File "/home/avilella/AfterQC/preprocesser.py", line 773, in run
    self.addFiguresToReport(reporter)
  File "/home/avilella/AfterQC/preprocesser.py", line 788, in addFiguresToReport
    reporter.addFigure('Read1 per base discontinuity after filtering', self.r1qc_postfilter.discontinuityPlotly("r1_post_discontinuity", 'Read1 discontinuity curve after filtering'), 'r1_post_discontinuity', "")
  File "/home/avilella/AfterQC/qualitycontrol.py", line 234, in discontinuityPlotly
    json_str += "var layout={title:'" + title + "', xaxis:{title:'cycles'}, yaxis:{title:'discontinuity', range:" + makeRange(0.0, max(self.meanDiscontinuity)*1.5) + "}};\n"
ValueError: max() arg is an empty sequence

This is with 2x150bp pairs of fastq.gz files.

Sorry, I made a mistake, it should be:
-f0 -t0 -q0 -n1000 -s0

I tested it and it works now. Thanks again for the prompt reply!


Add "use latest features" flag

During minor release we introduce new features in various ways:

data driven (currently mainly CMS driven)
configuration feature flag
CSS theme
Start features in an incubator library (new, not yet implemented)

We need this fine-grained control in order to not introduce breaking changes for existing customer. Other developer activities for training, POCs, innovative projects, new customers, demo's, etc. are however bothered unnecessary with this fine-grained control.
It would therefor be nice to introduce one * (astrix) flag to opt-in of the latest features. This flag is most likely required for both typescript and CSS.
We should also introduce this flag during the creation of a new Spartacus app with the CLI schematics.

QA steps:
For latest flag in feature level:

change feature level to latest ('*')

Expected result:

app should works just fine, all features should be available

For negation flag:

set feature level to 1.3
find component feature with 1.3 and change notation from '1.3' to '!1.3'

Expected result:

component or feature shouldn't be visible anymore

tested successfully


Improve the documentation

Many parts of our documentation are outdated

New documentation:
https://sphericart--58.org.readthedocs.build/en/58/


Handled runtime_error, and added a descructor.

Runtime_error handling is for NULL pointer assignment in dist and gen variables for distributor and generator of random password respectively.
Destructor is added to free dynamically allocated memory for dist and gen.

Closes #14


Spellforce 3 severe framerate drop

The game works very well but on some cases and always when the player has a base, the FPS drops drastically after a certain amount of time. Closing and starting the game solves the problem temporarily but after ~10 minutes the game starts to lag and after ~15 minutes it gets nearly unplayable. Lowering the video settings to their minimum doesn't help. I tried with
"dxgi.customVendorId = 1c06
dxgi.customDeviceId = 10de"
in the dxvk.conf to no avail.
In such moments GPU utilization is around 5-7% and the CPU around 10%. When there isn't frame drop and the game is working normaly, the utilization is ~30-35%.

I can't get an apitrace because the frame drop is after 10 minutes and the file will be enormous. @doitsujin since you mentioned previously you have access to the game, I can give you a savegame.
Software information
Name of the game, settings used etc.
System information

GPU: NVIDIA GTX 1080 Ti
Driver: 430.14.0
Wine version: 4.8
DXVK version: 1.2.1

Apitrace file(s)

Put a link here

Log files

d3d11.log:
SF3ClientFinal_d3d11.log

dxgi.log:
SF3ClientFinal_dxgi.log

What about memory utilization? Does this happen on Windows?

"severe framedrop" isn't something he can debug just by looking at log files

As is seen at the screenshot the memory is just around 2GB. My video card is with 11GB.
Here you are the save:
https://my.pcloud.com/publink/show?code=XZIKHA7Zxte4Yi7l98FBC2T5LW7Mnk1Nd9Lk
I don't have Windows but I searched on the Web and I can't find similar problem on Windows. Now I'll try with WineD3D and will report ASAP.

I meant RAM, not VRAM. Anyway, thanks for the save, will look probably tomorrow.
Anything special I need to do to reproduce the problem?

Nope, just wait ~10 minutes :)
Unfortunately I tried with WineD3D and the problem is here again.
My RAM is 32GB and when the problem occurs the amount of RAM taken is no more than 5GB for the whole system.

okay, so I had this running for ~20 minutes now and performance has always been between 30 and 50 FPS, using highest settings. Can't reproduce the problem. The game also doesn't spend much time on rendering at all.
If this happens for you with wined3d as well, the issue is most likely elsewhere. Note that I used Proton 4.2-5 for testing, not standalone wine.

Hm, very strange. For WineD3D I tested the game with clean prefix with vanilla Wine and the problem was there. May be I must try with Proton. It'll be interesting. Thank you :)

Confirming - with Proton there isn't such problem. Obviously Valve was implemented something but I don't know what :)


Always Unauthorized when testing the sample

I use the sample and follow the steps, but I cannot login to the app whenever I use normal account or admin
I track the steps and find the statuscode of response is "Unauthorized" though I can login to AD successfully.
I am sure that I have followed the document. So what is the issue?

Hey @woodwind... Is that the status code from the request to the To-Do list service?  If so, it is most likely something to do with your configuration values on the To-Do list service.

I guess that too because the request is sent to the service. However I just follow the steps to configure values. I don't understand why there is such permission issue.

If you post the code from your TodoListService Startup.Auth.cs file, as well as the token sent in the Authorization header of the request, I can have a look for you.

Ah...I find an issue in ida:audience, while after I change it, the authentication page is missing now and it prompts error for Access Denied. I think I need to check the steps and the project again. I will close the issue, thx!

Cool, no problem.


Get Multipart Upload File Etag

For multipart uploads using TransferManager, is there a way to get the final ETag of file without making an extra HeadObject request ?

Transfer Handle contains the metadata.

https://sdk.amazonaws.com/cpp/api/LATEST/class_aws_1_1_transfer_1_1_transfer_handle.html#a712dbb13626b689281a11ea805ee22a4 mentions that this specifically returns the metadata placed during upload which doesn't includes the etag. I tried ETag, etag and Etag in the map returned from GetMetadata and no such key was found in the metadata map.

How do I get off this mailing list?
Tim Yost - VP Business Development
p ::<PHONE_NUMBER> x8317
m::<PHONE_NUMBER>
a :: 3191 Red Hill Avenue, Suite 100 - Costa Mesa, California 92626
From: "Kunal Baweja"<EMAIL_ADDRESS>To: "aws/aws-sdk-cpp"<EMAIL_ADDRESS>Cc: "Subscribed"<EMAIL_ADDRESS>Sent: Monday, July 31, 2017 12:54:02 PM
Subject: Re: [aws/aws-sdk-cpp] Get Multipart Upload File Etag (#623)
https://sdk.amazonaws.com/cpp/api/LATEST/class_aws_1_1_transfer_1_1_transfer_handle.html#a712dbb13626b689281a11ea805ee22a4 mentions that this specifically returns the metadata placed during upload which doesn't includes the etag . I tried ETag , etag and Etag in the map returned from GetMetadata and no such key was found in the metadata map.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub , or mute the thread .
This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. If verification is required please request a hard-copy version. Altametrics, LLC. 3191 Red Hill Ave. Costa Mesa, CA 92626 altametrics.com

You can either Unsubscribe this conversation from left side bar or Unwatch the entire conversations from top menu.

We'll add it. Thanks for letting us know.

Now available, will be pushed out later today


Update fs2-core to 3.0.5

Updates co.fs2:fs2-core from 3.0.1 to 3.0.5.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2", artifactId = "fs2-core" } ]

labels: library-update, semver-patch

Superseded by #263.


Improve hexadecimal example

Updates the hexadecimal example to support uppercase letters and strings started with "0x" and "#"

Looks good, thank you! :)


Fixes issue with navbar search query string

Searching through the input field in the navbar is not sending the complete query string, and returns nothing on the first try. This PR fixes it by making it trigger SET_QUERY mutation in navsection component.
Fixes #188

@SingularReza please add tests to cover this change.


dns-security

Checklist for This Pull Request
🚨Please adhere to the guidelines for contributing to this repository.

[x] Ensure you are submitting your pull request to a branch dedicated to a specific topic/feature/bugfix. Avoid using the master branch for pull requests.
[x] Target your pull request to the main development branch in this repository.

[x] Check that your code additions do not fail any linting checks or unit tests.

Pull Request Description
Add support for DNS Security Profiles
What does this pull request accomplish?

Feature addition
Documentation update

Are there any breaking changes included?

[ ] Yes
[x] No

Is there anything the reviewers should know?
Thank you for your contributions!

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
:information_source: You can also turn on project coverage checks and project coverage reporting on Pull Request comment
Thanks for integrating Codecov - We've got you covered :open_umbrella: