Cannot add property 0, object is not extensible

In my project, I'm trying to draw multiple polyline path by pushing an empty array every time user press the button.
Basically, I'm working with 2D array and trying to push latest GPS data into the last child of the array
constructor(props) {
    super(props);
    this.moundted = false;
    this.state = {
      latestRegion: new MapView.AnimatedRegion({
        latitude: 0,
        longitude: 0,
        latitudeDelta: LAT_DELTA,
        longitudeDelta: LONG_DELTA
      }),
      routeCoordinates: [],
      routeCount: -1,
      markerArray: [],
      prevLatLang: {},
      distanceTravelled: 0,
      running: false
    }
  }

onButtonPressed = () => {
    const { running, routeCount } = this.state
    const latestLatLong = pick(this.state.latestRegion, ['latitude', 'longitude'])

    let marker = {
      title: (running ? 'End Point' : 'Starting Point'),
      description: (running ? 'You finished here' : "You Started here"),
      coordinate: latestLatLong
    }

    this.setState({
      running: !this.state.running,
      markerArray: [...this.state.markerArray, marker]
    })

    if (!running) {
      this.setState({
        routeCoordinates: this.state.routeCoordinates.concat([[]]), //add array
        routeCount: this.state.routeCount + 1
      })

      this.followWatchCoords()
    } else {
      this.stopFollowing()
    }
  }

Then I would call the watchPosition polyfills and calculateDistance to find out the distance between each marker
calculateDistance = (coords) => {
    const { latitude, longitude } = coords
    const { routeCoordinates, routeCount, distanceTravelled } = this.state

    const prevLatLang = {
      latitude: parseFloat(latitude),
      longitude: parseFloat(longitude)
    }
    const positionLatLong = pick(coords, ['latitude', 'longitude'])

    let newArr = [...routeCoordinates]
    newArr[routeCount].push(positionLatLong)    // error occurs here

    this.setState({
      routeCoordinates: newArr,
      distanceTravelled: distanceTravelled + this.calcHaversineDistance(prevLatLang),
      prevLatLang
    })
  }

This is how I render my polylines
{this.state.routeCoordinates.map((coordinates, index) => (
            <MapView.Polyline
              key={index}
              coordinates={coordinates}
              strokeColor="#19B5FE"
              strokeWidth={5} />
          ))}

I checked my state before rendering and its all matched with how its suppose to be

I'm not sure if I'm reading this right, but you shouldn't need to map over the <MapView.Polyline/> Component
Simply:
<MapView.Polyline
    key={index}
    coordinates={this.state.routeCoordinates}
    strokeColor="#19B5FE"
     strokeWidth={5}
/>

Unless you want multiple, separate Polylines on the screen?

Yes you are right, I'm trying to draw multiple lines

Okay, but I'm assuming this is a running application? So you're not showing different paths, just different segments of one path, right?
What is the reason for doing that? Is it for an undo feature? (My app has similar functionality, but I've gone about it a different way)

Ah I see. For my app I'm trying to show a different path. For instance, user press start button and the app would start collecting GPS data until the button is pressed again (to stop). All data will be pushed into one array to show the polyline lo. Please note that the array is a child of a parent. When the button is pressed again (to start), then a child array would be created and all incoming data will be stored in that child.

Okay. The structure you're using makes sense then. Could you console.log a snippet of this.state.routeCoordinates?

With the approach that I'm using, I can always get
[
[
{ lat: ?num, long: ?num } ,
{ lat: ?num, long: ?num } ,
],
[
{ lat: ?num, long: ?num } ,
{ lat: ?num, long: ?num } ,
]
]
But I believe is the problem is with the mutation of the array. I've tried concat and other methods and no luck :(

Okay, well the object you need to give Polyline is:
{
    latitude: Number,
    longitude: Number,
}

Sure. I have confirmed that the object I'm passing in is correct.
Now I try to do it this way and the error is gone but it is mutating my array the second time start button is pressed.
this.setState({
      routeCoordinates: [routeCoordinates[routeCount].concat(positionLatLong)],
      // routeCoordinates: [...this.state.routeCoordinates, routeCoordinates[routeCount].concat(positionLatLong)], //this wont wory
      distanceTravelled: distanceTravelled + this.calcHaversineDistance(prevLatLang),
      prevLatLang
    })


Metrics App

hello reviewer I hope you are having a great day, any suggestion will be welcome, thank you very much for the support 😄 🌴

For this part of the project create an app with a single page with Redux and React.

Project requirements
API First you need to choose an API to base the development of the webapp on. The API should allow you to get numeric values based on a parameter. We recommend that you choose an API that is open or needs an API key. If you choose an API that require authentication, you should implement it on your own.
Some example APIs are:

Financial modeling prep API: stock data.
Air pollution API: air quality data.
You can find many APIs in this GitHub repo or in ProgrammableWeb. Some of the APIs requires a token for authentication and some others are just open.

Design

You should follow these design guidelines, including:

Colors (select one main color for your website).
Typography: font face, size and weight.
Layout: composition and space between elements.
Original design idea by Nelson Sakwa on Behance.

The Creative Commons license of the design requires that you give appropriate credit to the author. Therefore, you must do it in the README of your project.

Interactions
Home page

When the page loads, the webapp shows the list of categories that could be filtered out by some parameter, for example by the category name.
Along with the category name, you will display some numeric values per category that come from the API.
When the user clicks (or taps) on a category item, the application navigates to the details page.

Details page

In the details page, the webapp retrieves data from the API to show detailed data in the category.
When the user clicks on the "Back" button (<), the user navigates to the home page.

##Testing requirements

Create unit tests for pure functions in your Redux code.
Create integration tests for your application using the React Testing Library.
You may need to mock the access to the API, so that your tests don't send actual requests.
You may need to mock the connection to the Redux Store.

Technical requirements

The project is a single-page application (SPA) built with React and Redux.
The data retrieved from the API should be stored in the Redux store.
You should filter the data that you retrieve from the API using a Filter stateless component.
Every page (the main page and the pages for each item) should have a unique route within the SPA.
The project should be deployed and accessible online (add a link to the online version of your app to your README file).

Workflow

Create the application
Create the folders
Create the files
Create store
Create actions and reducers
Deploy the app
Record the video (the video is here down 👇🏽 )
Fix liters

Video link
➡  https://www.loom.com/share/3cffa3951dda45bba3d9bab1ff03cf8b

Hi @PierreBarba, the link to  MIT.md file is still broken can you quicky update it?


More icon colours

From Martin, via email.
So the key question is: how do I know if everything is working. So I
would differentiate the following states:

currently syncing: blue & spinning
each folder has at least one node it is connected to (i.e. if
required each folder is able to sync to at least one remote node) =>
blue (normal)
at least one folder has at least one node it is connected to (i.e.
physical Internet connection seems to work, but it cannot be
guaranteed, that all changes on the local node reach remote nodes)
=> some other color/icon (dark blue?), or an exclamation mark over
the icon, or ...
no single folder has at least one node it is connected to (i.e.
there is no connection that is useful for syncing) => gray
"disconnected" (logically, not necessarily physically)
Of course it would be great to see the Out-Of-Sync state, but if
there are problems, this could be postponed to a later release (or
better: set as option if this should be displayed) => some other
color/icon, or an exclamation mark over the icon, or ..

Don't know if this is meant for discussion, but I will through my opinion anyway :innocent::

This is current behaviour, right? And it's also spinning when scanning and exchanging indexes, I think.
(and also 3.) I think this is useless. Different look of an icon should tell you that:
Something is wrong (currently grey when Syncthing is not running/connected)
Something is happening (currently spinning, see 1.)
(still about 2.+3. - don't know how to format it) It shouldn't have another (semi-OK) state causing that some users (and maybe only on some devices) will never achieve full-OK state and when they do, it will cause more confusion than it would be its benefit. Syncing multiple folders to multiple devices simply isn't something what could be fitted to one icon without need to open GUI from time to time. Another reason against this: Syncthing itself also doesn't differentiate between those 2 states of folder. When no other device sharing that folder is connected, folder is "Up to date". (I know that Syncthing GTK does differentiate those, but it does it in the main window and it has its own GUI. Syncthing itself could have this "Offline" folder state too, but that is for another debate.) This also means that folders in GUI looks exactly the same for one folder having one node connected (one icon colour) or all folders having one node connected (other icon colour), causing confusion which most users won't be able to figure out (I think me included, so for me this is the strongest argument against).
Sounds good, but if this would be grey, then "Syncthing not running/disconnected" would have to be some other colour (and red/orange are "error"/"warning" colours and that is not a good idea because Syncthing could be down for a reason, i.e. "I don't want to sync now but leaving SyncTrayzor running"), so I would suggest dark blue as a transition between blue "OK" and grey " Syncthing down".
I either didn't understand this (because when we're "Out of state", we are "Syncing" and spinning blue apply here) or it is "Master folder only" case of use (rare, but legit, with "Propagate changes" in right click menu even more legit?) or it is the same as 2.+3., meaning: Keep it reasonable and simple, its Windows only (any plans to go multi-platform?) app after all with main purpose to bring Syncthing to those Dropbox/Google Drive masses (I think, because to bring BTSync people Syncthing itself seem good enough for me) and they will have enough troubles to set it right and understand purpose of multiple folders.

This took me more than an hour to create so I hope it's not worthless and I didn't forget anything important. Will come back if I did (and if this post will still be here :wink:).
P.S.: Apologies to Windows users (I'm one of them), if you're reading this then "masses" not applies to you.
P.P.S.: I'm overusing brackets, I know :stuck_out_tongue_winking_eye:

This issue is half a reminder to myself, and half an opportunity for discussion.
Thanks - I'll go over your comments in detail later, when I have some time. The main thing I'm worried about is remaining obvious: there's nothing worse than a cryptic icon, where you can never remember what state each of the colours mean.

Right!

Is almost correct - it actually spins when any data is being transferred, up or down (which should cover index transfers, uploading, downloading, etc).

I'm inclined to agree with you on pretty much all points. The only remaining suggestion, I think, is "no single folder has at least one node it is connected to", which I'm interpreting as "at least one folder will not sync, because it's not connected to anything". An icon which is half blue and half grey might be a good option here - but I'm not sure how useful it is. One of my folders is almost never connected to another device.

I think this has been overtaken by the new icons - paused, warning, etc.
Following these, I don't really want to change the icon unless it's the grab the user's attention (because there's something that they should know about, that isn't "normal operation").


[Automation] Generate Fluent Lite from Swagger consumption#package-2021-10

[Automation] Generate Fluent Lite from Swagger consumption#package-2021-10

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure.resourcemanager:azure-resourcemanager-consumption


Init the backend

First install the intelij.
You should have permium version of intelij idea for workiing  with springboot.
https://education.github.com/discount_requests/student_application
Request for github student pack from here with  uni email address. Then we can get intelij license.
Then clone the repo and open the directory with intelij idea

Installed and clone the repo. thanks in advance


fix: make base URL for relative redirects consistent

Will fix #9880. Only commit for now is adding to the tests from #8842; does this look like the behavior we want?
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[ ] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Ideally yeah, I think this would be the behaviour. I looked into the implementation though and I think it's going to be quite tricky — at present, the information for each 'node' of a route (which basically consists of the page options + the result of calling load) is bundled together, meaning we can't determine the correct trailingSlash value without also running the load function... which throws an error, preventing us from reading the value.
So we'd need to separate those two things in order to make it possible to normalize pathnames prior to redirection. If that causes client.js to increase in size then we'll have to consider how important we think it is to cover this edge case. Ideally, redirects should probably be root-relative so that this situation doesn't arise.

Thank you for the reminder.
I have been rather stumped by this — managed to get it working for .js but not for .server.js routes; I'll push those commits now. I'd be okay with it if anyone else wants to pick up this PR from here.
Some miscellaneous notes:

setting server.hmr.overlay to false resolves the flakiness in /test/apps/basics/
current behavior before this PR depends on  get_navigation_result_from_branch mutating its url argument
function call graph in src/runtime/client/client.js is a bit complex:


Report exception cause to backend

Fixes #705
Implement implement "chained exceptions" reporting to backend server (part of https://github.com/elastic/apm/issues/40)
In Java, each exception has at most one "cause" exception, thus the 'cause' array has at most one element.
Checklist

[x] Implement code
[x] Add tests
[ ] Update documentation
[ ] Update CHANGELOG.md
[ ] Update supported-technologies.asciidoc
[ ] Added an API method or config option? Document in which version this will be introduced.
[x] Manual test with latest master branch of backend that supports this feature.

Codecov Report

Merging #878 into master will increase coverage by 0.48%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #878      +/-   ##
============================================
+ Coverage      64.4%   64.88%   +0.48%
  Complexity       85       85
============================================
  Files           231      231
  Lines          9436     9407      -29
  Branches       1230     1224       -6
============================================
+ Hits           6077     6104      +27
+ Misses         2970     2917      -53
+ Partials        389      386       -3

Impacted Files
Coverage Δ
Complexity Δ

.../apm/agent/report/serialize/DslJsonSerializer.java
91.67% <100%> (+5.41%)
0 <0> (ø)
:arrow_down:

...ic/apm/agent/servlet/ServletTransactionHelper.java
74.55% <0%> (-0.93%)
0% <0%> (ø)

...ava/co/elastic/apm/agent/impl/context/Request.java
71.87% <0%> (+5.2%)
0% <0%> (ø)
:arrow_down:

.../co/elastic/apm/agent/impl/error/ErrorPayload.java
50% <0%> (+50%)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

jenkins run the tests


gimpPlugins.gmic: 3.3.5 -> 3.3.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gimpPlugins.gmic is: GIMP plugin for the G'MIC image processing framework
meta.homepage for gimpPlugins.gmic is: http://gmic.eu/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.3.6 with grep in /nix/store/xlc00gs6k7jgyq6x5mlz9fr08qksnlyb-gmic-qt-gimp-3.3.6
found 3.3.6 in filename of file in /nix/store/xlc00gs6k7jgyq6x5mlz9fr08qksnlyb-gmic-qt-gimp-3.3.6

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

gimp-with-plugins
gimpPlugins.gmic
gmic-qt

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xlc00gs6k7jgyq6x5mlz9fr08qksnlyb-gmic-qt-gimp-3.3.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gimpPlugins.gmic https://github.com/r-ryantm/nixpkgs/archive/962ec471bac8d85e001450ff9d9ec860f7a7bbdb.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/962ec471bac8d85e001450ff9d9ec860f7a7bbdb#gimpPlugins.gmic

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xlc00gs6k7jgyq6x5mlz9fr08qksnlyb-gmic-qt-gimp-3.3.6
ls -la /nix/store/xlc00gs6k7jgyq6x5mlz9fr08qksnlyb-gmic-qt-gimp-3.3.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

3 packages built:

gimp-with-plugins
gimpPlugins.gmic
gmic-qt

Maintainer pings
cc @AndersonTorres @lilyinstarlight for testing.

@NixOS/nixpkgs-merge-bot merge


mass app build fails on a new template

when the _params.auto.tfvars.json file is blank, mass app build fails.
INF Building application...
ERR an error occurred while generating provisioner files error="error compiling dev params: error getting dev params: error getting existing params: unexpected end of JSON input" provisioner=terraform
Error: error compiling dev params: error getting dev params: error getting existing params: unexpected end of JSON input
Usage:

Resolved


Use propagate! instead of Libtask.consume

This PR removes Libtask in the propagation of ParticleContainer. Instead it introduces a function propagate! that propagates all particles one step and returns true if all particles are propagated to the final time step and false otherwise (maybe one would rather want to use more explicit symbols instead of booleans?).
I always had the same impression as @mohamed82008 (see https://github.com/TuringLang/Turing.jl/issues/1233) that in particular for the propagation of the particle container one does not have to use Libtask. However, since collect(::ParticleContainer) returns the particles IMO it would be surprising if iterating over a particle container would propagate it. Moreover, since it seems there are no states that have to be tracked by the container (I even removed n_consume and logE in the particle container), it felt more natural to add a method for propagation, similar to resample! for the resampling step, than using a dedicated iterator for ParticleContainer.
I didn't fully understand the existing implementation of the likelihood estimation (isn't the currently reported log evidence incorrect since logZ neglects the constant term -log(num_particles)?), so I used the algorithm in http://www.it.uu.se/research/systems_and_control/education/2019/smc/schedule/lecture9.pdf (pages 5, 6, and 7) as a template - basically, now the implementation for SMC and PG should perform exactly the same sequence of propagation and resampling steps as noted there using propagate! and resample!.
Aside from that, I'm slightly unhappy with the current implementation of SMC since it somehow misuses spl.state.average_logevidence to save the log likelihood estimate of all particles that is computed in sample_init!, then uses it in every iteration of step! in which the ParticleTransitions for every particle are created, and finally computes the "overall" log evidence in bundle_samples! by setting spl.state.average_logevidence to the mean of the log evidence values of these particles (i.e., the value that spl.state.average_logevidence was set to in sample_init!). It's also a bit annoying that for SMC an iteration keyword argument is needed (IMO that should be part of the state, i.e., transition and not be part of the generic algorithm in AbstractMCMC since it seems only/mostly SMC uses it).

Thanks, @devmotion. It's a good idea to remove the dependency on Libtask.comsume since it does not add any extra functionality except for recycling a function name. I think we can further make Particle's dependency on Libtask optional, by implementing an iterator interface mentioned by @mohamed82008 for CTask types. This way, the SMC implementation can support alternative approaches for resampling particles other than using Libtask.CTask, e.g. for less generic models that can be represented as Markov process. In such models, we can either manually handle particle duplication during the resampling step, or consider something like ResumableFunctions.jl.

I didn't fully understand the existing implementation of the likelihood estimation (isn't the currently reported log evidence incorrect since logZ neglects the constant term -log(num_particles)?)

I think we had a step for subtracting the constant term in previous versions of the code, but I can't recall where that was, and when that was removed.

I think we can further make Particle's dependency on Libtask optional, by implementing an iterator interface mentioned by @mohamed82008 for CTask types. This way, the SMC implementation can support alternative approaches for resampling particles other than using Libtask.CTask, e.g. for less generic models that can be represented as Markov processes. In such models, we can either manually handle particle duplication during the resampling step, or consider something like ResumableFunctions.jl.

I completely agree, probably it would be useful to implement and use an iterator interface for particles, so that one could use alternative particle implementations such as ordinary stateful iterators. IMO it's just not useful and unintuitive (due to the behaviour of collect) to implement an iterator interface for propagating a ParticleContainer. I thought the changes for Particles should better go into a separate PR (and as you said, one should first implement the interface for CTask).

I'll update the test in a separate PR.


Create Spark job to load pages into Fluo

While init.sh initializes an empty Fluo and Accumulo, load.sh will add pages to Fluo and combine with previous data.  This should be written in Spark to parallelize the loading across several nodes.

Completed in #20


contrib\cmake: how properly depend on generated tensorflow .lib

Used contrib\cmake ( https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/cmake ) on Windows 10 64bit and VS 2015 64bit to build tensorflow  for C++.
Opened generated tf_tutorials_example_trainer.vcxproj  in VS Release x64 - runs VERY GOOD, but rebuilds whole tensorflow every time i hit play! Tried Build->Configuration Manager->Uncheck projects that i don't want to always build - errors (what projects i need to keep checked to build only tf_tutorials_example_trainer.vcxproj?).
How to depend on tensorflow statically (or at least prevent long build)?
Failed attempts:
Created new VS console project Release x64 (to prevent full rebuilds).
Used step-by-step build https://gist.github.com/derofim/b4f150da1269b81af8d12744df730708
Added tensorflow include dirs.
Added .lib files:
Release\tf_protos_cc.lib
protobuf\src\protobuf\Release\libprotobuf.lib
Used same flags as in example project.

Error: No session factory registered for the given session options

Tried solutions:
/WHOLEARCHIVE as in https://msdn.microsoft.com/en-us/library/mt732963.aspx - LNK1000: Internal error during IncrBuildImage
/OPT:NOREF - no effect
/FORCE:MULTIPLE  - no effect
 /INCLUDE - what to place here for force inclusion of unused symbols? May it help?

@vit-stepanovs I feel like your answer at #6396 might be also applicable here?

Yes, it looks like the same issue.

Then closing this issue.
later this month, we will move all windows support onto bazel.
I think it will look slightly different, and we will be able to use bazel's incremental build then.
Until bazel arrives, I suggest tring suggestions at #6396.
Closing this issue.


F-Droid

Would you consider submitting this app to F-Droid?

OH, very good suggestion.

Do you have any ideas how to do that? I mean via the git repository of F-Droid.

You will have to ensure that you meet the inclusion policy located here. Once you've ensured you have met the crietria you place this app in the submission queue located here. More details are available on this page.

Right

Ok , put in the submission queue, I will close this after it is on F-Droid.

I have put in the submission queue, however this project uses the Constraint Layout, which is in the alpha status, and they said this can be only available on F-Droid if the Constraint Layout becomes stable.

Just resubmitted to F droid, wait and see

The issue has been closed with an "added" label. However it's not available through fdroid yet. According to the Fdroid issue, there's a build failure: https://gitlab.com/fdroid/rfp/issues/58

@offa  I have update the ConstraintLayout to the stable version, will that be ok now.

Thanks, I can't wait to get OpenLauncher through Fdroid (instead of building from source every time)! 👍

OpenLauncher is now available on F-Droid!? But the Link on the F-Droid website is dead.
Is this release intended and ready to use or by accident?

While the related issue does not have any news, the wiki states it's available now – the latest version.

I dont know what is happening on f droid, but previously, they said google
hasnt added the constraint layout to the repo that used by f droid, so the
lib want not found and a build error happened, but now I still dont know, I
will need to check it up when i back home on my pc.

Ok now, lets close this old 'issue' 😆


Getting an error message from LSP although there is no error

Description
I have the following script:
CREATE TABLE IF NOT EXISTS dwh.dim_employee AS (
	SELECT
		customerid
		, initcap(firstname) AS "firstname"
		, initcap(lastname) AS "lastname"
		, company
		, address
		, city
		, state
		, country
		, postalcode
		, phone
		, fax
		, email
		,
	split_part(
			email
			, '@'
			, 2
		) AS "domain"
		, supportrepid
		, last_update
	FROM
		stg.customer AS c
);

it shows an error message of column not found at:
initcap(firstname) AS "firstname"
If I remove the initcap() function then the error disappear.
DBeaver Version
24.3.0
Operating System
windows 10
Database and driver

PostgresSQL version: 17.2-2
PostgreSQL JDBC Driver (can't find the version, if it helps its the default one)

Steps to reproduce
No response
Additional context
I tried the following:
disconnect and reconnect,
reopen the application

Thanks for the bug report

I think there is overall some bugs with the LSP.
Found out that he doesn't recognize CTE at all.
For example:
WITH miliseconds_to_seconds AS (
	SELECT
		t.trackid
		, t.milliseconds / 1000 AS "seconds"
	FROM
		track AS t
)
SELECT
	*
	, mts.seconds AS "seconds"
	, mts.seconds / 60 || ':' || MOD(mts.seconds, 60) AS "length"
FROM
	track AS t
LEFT JOIN album AS a ON
	t.albumid = a.albumid
LEFT JOIN artist AS art ON
	a.artistid = art.artistid
LEFT JOIN mediatype AS m ON
	t.mediatypeid = m.mediatypeid
LEFT JOIN genre AS g ON
	t.genreid = g.genreid
LEFT JOIN miliseconds_to_seconds AS mts ON t.trackid = mts.trackid

The LSP will generate error :
⚠Table or subquery mts not found
at the lines in the SELECT statement:
, mts.seconds AS "seconds"
, mts.seconds / 60 || ':' || MOD(mts.seconds, 60) AS "length"

It works with CTE; it just doesn't handle some expressions (we're working on it). For example, in this case, those following expressions break it mts.seconds / 60 || ':' || MOD(mts.seconds, 60) AS "length".
Thank you for providing these cases!

Thank you so much for the fast responses!

Activating Hippe Engine seems to help with the problem above.
I think it should be on by default (unless its experimental)
For anyone who wants to enable it:
File -> Preferences -> Editors -> SQL Editor -> Code Completion -> Activate Hippe Engine for autocompletion

[!note]
Make sure to enable it on the global settings


Add dockerfile to support FPGA comilation for PAC

Added dockerfile to support PAC FPGA compilation, based on Dockerfile.ubuntu-18.04.
Tested with vector_add sample design

The ubuntu 18.04 image has been deprecated, if this is something you still want to propose, please let me know.


Handle Vue ref parameters

Allows to use refs which is really useful
Consider such example
const { data: todos, isFetched: areTodosFetched } = useGetTodosQuery();

const firstTodoId = computed(() => (areTodosFetched.value && todos.value?.[0]
  ? todos.value[0].id
  : undefined));

const { data: firstTodoNotes } = useGetTodoNotesQuery({ id: firstTodoId }, { enabled: areTodosFetched });

The second request will be triggered only after the first one is finished. So, we will know todo's id later only and using ref it becomes reactive

Guess need to update something else to make types to accept ref as well
Should be like
import { Ref } from 'vue';

export type GetTodoNotesQueryParameters = {
  id: number | Ref<number>;
};

But I'm not sure how to do that yet

Thanks for the suggestion! Couple of questions though (I'm not familiar with Vue, so sorry if it's something obvious :)):

Does it make sense to make spread parameters of a Ref type as well?
Currently we have 2 overloads:export function useGetPetByIdQuery(dto: GetPetByIdQueryParameters, ...)
export function useGetPetByIdQuery(petId: number,...)

Should the second one be changed as well to
export function useGetPetByIdQuery(petId: Ref<number>,...)?
Your solution with completely 'overriding' ReactQuery.liquid file works, but I hope you don't mind if I'd try to come up with something regex-based so that it's easier in support (if/when ReactQuery.liquid changes)

So, version 15.2.9 includes these Ref changes that you did, hopefully it works as expected


Master branch broken?

I've tried installing from the master branch (id'd as 0.5.2-dev at the time of writing this) and from the normal stable (0.5.1), and  while it will install just fine and even creates projects OK, when I do a first run (not having changed the new projects files at all) it fails with: Error: 404 Not Found: org.apache.cordova.engine.crosswalk.
After some digging, it seems this is because the repo for crosswalk has been renamed (to org.crosswalk.engine), with the cordova registry entry removed(?); and due to the org.apache.cordova.engine.crosswalk plugin getting marked as deprecated (in: cca-manifest-logic/src/plugin-map.js), it has removed all the current versions of my crosswalk files and cannot find the new ones to add.
It's been like this nearly the whole month, any ETA on when this will get sorted out? (for now I can manually fix things, but its a pain)

Yeah, it's in an annoying state right now in that it depends on an
unreleased version of cordova-lib. Should work out if you npm link the
master version of cordova-lib and cordova-cli. Also don't forget to npm
link the cca-manifest-logic module that exists within the repo.
On Mon, Feb 23, 2015 at 10:19 PM, Shane Gadsby<EMAIL_ADDRESS>wrote:

I've tried installing from the master branch (id'd as 0.5.2-dev at the
time of writing this) and from the normal stable (0.5.1), and while it will
install just fine and even creates projects OK, when I do a first run (not
having changed the new projects files at all) it fails with: Error: 404
Not Found: org.apache.cordova.engine.crosswalk.
After some digging, it seems this is because the repo for crosswalk has
been renamed (to org.crosswalk.engine), with the cordova registry entry
removed(?); and due to the org.apache.cordova.engine.crosswalk plugin
getting marked as deprecated (in: cca-manifest-logic/src/plugin-map.js
https://github.com/MobileChromeApps/mobile-chrome-apps/blob/master/cca-manifest-logic/src/plugin-maps.js#L51),
it has removed all the current versions of my crosswalk files and cannot
find the new ones to add.
It's been like this nearly the whole month, any ETA on when this will get
sorted out? (for now I can manually fix things, but its a pain)
—
Reply to this email directly or view it on GitHub
https://github.com/MobileChromeApps/mobile-chrome-apps/issues/524.

Thanks for the heads up @agrieve, I npm linked to the latset head of each oth the libs you mentioned , and sure enough it worked. I assume this will all be sorted out in 0.5.2 (?)

Great! Yes. It will.

FYI - this is now resolved, and there's a release-candidate for the next version of cca:
npm install -g<EMAIL_ADDRESS>


Update free-programming-books-pl.md

Removed dead links and added Python and Django books.

Thanks for these improvements, and welcome to free-programming-books. You're the 994th contributor to the project!
One of the items you've added is a course. Are there any other courses in the polish list that we would want to split off into a free-courses-pl list?

also see #2358


Fixed some package importing

I checked the code files and fixed some package importing.

LGTM!. The thing about this PR's is that are super hard to downmerge. I'll set it as a 3.0 milestone if you agree @sazzad16

@marcosnils agreed!


Compile error at the latest one

Failed to build latest.
capability.c: In function ‘capability_read_public’:
capability.c:53:9: error: ‘TPM2_PT_TPM2_HR_PERSISTENT’ undeclared (first use in this function); did you mean ‘TPM2_PT_HR_PERSISTENT’?
53 |         TPM2_PT_TPM2_HR_PERSISTENT, &more_data,
|         ^~~~~~~~~~~~~~~~~~~~~~~~~~
|         TPM2_PT_HR_PERSISTENT
capability.c:53:9: note: each undeclared identifier is reported only once for each function it appears in
make[2]: *** [: capability.o] Error 1
make[2]: Leaving directory '/home/linux/cryptfs-tpm2/src/lib'
make[1]: *** [Makefile:9: install] Error 2
make[1]: Leaving directory '/home/linux/cryptfs-tpm2/src'
make: *** [Makefile:12: install] Error 2

Fixed by https://github.com/jiazhang0/cryptfs-tpm2/commit/684b868e68afef9bcbfecd20ea4da2f220445d03


[FORMATTING] BETWEEN is not expected after CASE WHEN

Input data
Which SQL and options did you provide as input?
SELECT
    CASE
        WHEN t1.type = 'Firm' THEN
            CASE
                WHEN some_table.start_dt <= getdate() THEN 100 / 100
                WHEN some_table.start_dt BETWEEN getdate() AND (DATEADD(year, 1, getdate())) THEN ROUND((t1.pct + 15) / 100, 2)
                ELSE ROUND(t1.pct / 100, 2)
            END
        ELSE 0
    END AS probability_pct
FROM some_table;

Expected Output
SELECT
    CASE
        WHEN t1.type = 'Firm' THEN
            CASE
                WHEN some_table.start_dt <= getdate() THEN 100 / 100
                WHEN some_table.start_dt BETWEEN getdate() AND (DATEADD(year, 1, getdate()))
                    THEN ROUND((t1.pct + 15) / 100, 2)
                ELSE ROUND(t1.pct / 100, 2)
            END
        ELSE 0
    END AS probability_pct
FROM some_table;

Actual Output
Error: Parse error at token: BETWEEN at line 6 column 42
Unexpected BETWEEN token: {"type":"BETWEEN","raw":"BETWEEN","text":"BETWEEN","start":182,"precedingWhitespace":" "}. Instead, I was expecting to see one of the following:

So the error is on this line:
WHEN some_table.start_dt <here it's not expecting the next token with the BETWEEN> BETWEEN getdate()

Usage

How are you calling / using the library?

sqlFormatter.format(data, {
  language: "tsql",
  tabWidth: 4,
  keywordCase: "upper",
  expressionWidth: 100,
  tabulateAlias: false,
})

What SQL language(s) does this apply to? - TSQL
Which SQL Formatter version are you using? 12.0.0

Thanks for reporting. Though it's a duplicate of #500, so I'll close this issue.
As a workaround for now you can wrap the between expression inside parenthesis like:
WHEN (x BETWEEN 1 AND 10) THEN ...

Hopefully I'll get to fixing this... This issue has been open for a while now.


Porting changes for telemetry from master for Remote PowerShell Task

Adding a basic layered-telemetry to RemotePowerShell Task. PREDEP= pre-deployment error; DEP= error during deployment; UNKNOWNPREDEP = Unknown pre-deployment error; PREREQ = Error in prerequisite. This is followed by a numbered code distinguishing error within a category.

Looks good.

Looks fine.


[TFLite, Converter] Converting error on unusual dense layer

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): GIT_VERSION: 'v1.12.1-15611-g025365a736', VERSION: '2.0.0''
Python version: 3.6.8
Bazel version (if compiling from source): 0.26.1
GCC/Compiler version (if compiling from source): GCC 8.3.0
CUDA/cuDNN version: No
GPU model and memory: No

Describe the current behavior
When run the code below, conversion errors appear that are in the logs below.
Code to reproduce the issue
import tensorflow as tf

class MyDense(tf.keras.layers.Layer):
  def __init__(self, num_units, **kwargs):
    super(MyDense, self).__init__(**kwargs)
    self.num_units = num_units

  def build(self, input_shape):
    kernel_shape = [input_shape[-1], self.num_units * 2, self.num_units]
    bias_shape = [self.num_units]

    self.kernel = self.add_weight("kernel", shape=kernel_shape, trainable=True)
    self.bias = self.add_weight("bias", shape=bias_shape, trainable=True)
    super(MyDense, self).build(input_shape)

  def call(self, inputs):
    return tf.einsum("ac,cde->ade", inputs, self.kernel) + self.bias

inputs = tf.keras.Input(shape=(10,), dtype=tf.float32)
outputs = MyDense(15)(inputs)
model = tf.keras.Model(inputs=inputs, outputs=outputs)
model.summary()

converter = tf.lite.TFLiteConverter.from_keras_model(model)
tflite_model = converter.convert()
print("SUCCESS!")

Logs
2019-10-15 14:43:59.548878: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 AVX512F FMA
2019-10-15 14:43:59.566412: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2019-10-15 14:43:59.567598: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x3502840 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2019-10-15 14:43:59.567616: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
Model: "model"
_________________________________________________________________
Layer (type)                 Output Shape              Param #
=================================================================
input_1 (InputLayer)         [(None, 10)]              0
_________________________________________________________________
my_dense (MyDense)           (None, 30, 15)            4515
=================================================================
Total params: 4,515
Trainable params: 4,515
Non-trainable params: 0
_________________________________________________________________
2019-10-15 14:43:59.640959: I tensorflow/core/grappler/devices.cc:60] Number of eligible GPUs (core count >= 8, compute capability >= 0.0): 0 (Note: TensorFlow was not compiled with CUDA support)
2019-10-15 14:43:59.641037: I tensorflow/core/grappler/clusters/single_machine.cc:356] Starting new session
2019-10-15 14:43:59.642744: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:829] Optimization results for grappler item: graph_to_optimize
2019-10-15 14:43:59.642759: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:831]   function_optimizer: function_optimizer did nothing. time = 0.003ms.
2019-10-15 14:43:59.642763: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:831]   function_optimizer: function_optimizer did nothing. time = 0ms.
2019-10-15 14:43:59.657830: I tensorflow/core/grappler/devices.cc:60] Number of eligible GPUs (core count >= 8, compute capability >= 0.0): 0 (Note: TensorFlow was not compiled with CUDA support)
2019-10-15 14:43:59.657896: I tensorflow/core/grappler/clusters/single_machine.cc:356] Starting new session
2019-10-15 14:43:59.660184: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:829] Optimization results for grappler item: graph_to_optimize
2019-10-15 14:43:59.660198: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:831]   constant_folding: Graph size after: 21 nodes (-3), 23 edges (-4), time = 0.76ms.
2019-10-15 14:43:59.660203: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:831]   constant_folding: Graph size after: 21 nodes (0), 23 edges (0), time = 0.248ms.
Traceback (most recent call last):
 File "convert_error.py", line 25, in <module>
   tflite_model = converter.convert()
 File "/workspace/.local/lib/python3.6/site-packages/tensorflow_core/lite/python/lite.py", line 447, in convert
   **converter_kwargs)
 File "/workspace/.local/lib/python3.6/site-packages/tensorflow_core/lite/python/convert.py", line 449, in toco_convert_impl
   enable_mlir_converter=enable_mlir_converter)
 File "/workspace/.local/lib/python3.6/site-packages/tensorflow_core/lite/python/convert.py", line 200, in toco_convert_protos
   raise ConverterError("See console for info.\n%s\n%s\n" % (stdout, stderr))
tensorflow.lite.python.convert.ConverterError: See console for info.
2019-10-15 14:44:00.554744: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 10 operators, 19 arrays (0 quantized)
2019-10-15 14:44:00.554825: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 10 operators, 19 arrays (0 quantized)
2019-10-15 14:44:00.554930: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] After general graph transformations pass 1: 3 operators, 8 arrays (0 quantized)
2019-10-15 14:44:00.554945: F tensorflow/lite/toco/graph_transformations/propagate_fixed_sizes.cc:118] Check failed: dim_x == dim_y (450 vs. 15)Dimensions must match
Fatal Python error: Aborted
Current thread 0x00007f4cae2e4740 (most recent call first):
 File "/workspace/.local/lib/python3.6/site-packages/tensorflow_core/lite/toco/python/toco_from_protos.py", line 52 in execute
 File "/workspace/.local/lib/python3.6/site-packages/absl/app.py", line 250 in _run_main
 File "/workspace/.local/lib/python3.6/site-packages/absl/app.py", line 299 in run
 File "/workspace/.local/lib/python3.6/site-packages/tensorflow_core/python/platform/app.py", line 40 in run
 File "/workspace/.local/lib/python3.6/site-packages/tensorflow_core/lite/toco/python/toco_from_protos.py", line 89 in main
 File "/workspace/.local/bin//toco_from_protos", line 10 in <module>
Aborted (core dumped)

Issue replicating for TF-2.0,kindly find the gist of colab.

@Vooblin,
Could you please add the below two lines of code before converting the model and check if it works.
converter.experimental_new_converter = True
converter.target_spec.supported_ops =[tf.lite.OpsSet.TFLITE_BUILTINS, tf.lite.OpsSet.SELECT_TF_OPS]

I was able to run the code without any issues on TensorFlow v2.2.0-rc4. Please find the gist of it here. Thanks!

@Vooblin,
Is this still an issue? Please feel free to close the issue if resolved. Thanks!

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


/etc/os-release folder does not exist

brew doctor output
Your system is ready to brew.

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 4.1.5-35-g020508b
ORIGIN: https://github.com/Homebrew/brew
HEAD: 020508b7b337b7aa6b8ae6cd527a0ad3277c340e
Last commit: 3 hours ago
Core tap JSON: 16 Aug 10:26 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 12
Homebrew Ruby: 2.6.10 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: dodeca-core 64-bit arm_blizzard_avalanche
Clang: 14.0.3 build 1403
Git: 2.39.2 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 8.1.2 => /usr/bin/curl
macOS: 13.5-arm64
CLT: <IP_ADDRESS>.1.1683849156
Xcode: N/A
Rosetta 2: false

What were you trying to do (and why)?
brew style --fix
What happened (include all command output)?
Inspecting 1178 files
..........................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................
1178 files inspected, no offenses detected
Issues were detected, but none were auto-fixable. Use another format to see them.
In /opt/homebrew/Library/Homebrew/brew.sh line 596:
[[ -n "${HOMEBREW_LINUX}" ]] && HOMEBREW_OS_VERSION="$(source /etc/os-release && echo "${PRETTY_NAME}")"
^-------------^ SC1091 (info): Not following: /etc/os-release: openBinaryFile: does not exist (No such file or directory)
^------------^ SC2154 (warning): PRETTY_NAME is referenced but not assigned.
For more information:
https://www.shellcheck.net/wiki/SC2154 -- PRETTY_NAME is referenced but not...
https://www.shellcheck.net/wiki/SC1091 -- Not following: /etc/os-release: o...
What did you expect to happen?
I expect not to have these errors:
^-------------^ SC1091 (info): Not following: /etc/os-release: openBinaryFile: does not exist (No such file or directory)
^------------^ SC2154 (warning): PRETTY_NAME is referenced but not assigned.
Step-by-step reproduction instructions (by running brew commands)
just run the command:
brew style --fix

@MikeMcQuaid Do we want to also disable SC2154 for this line? #15877 only disabled SC1091, so I still see the SC2154 warning here:
In /opt/homebrew/Library/Homebrew/brew.sh line 598:
  [[ -n "${HOMEBREW_LINUX}" ]] && HOMEBREW_OS_VERSION="$(source /etc/os-release && echo "${PRETTY_NAME}")"
                                                                                ^------------^ SC2154 (warning): PRETTY_NAME is referenced but not assigned.

For more information:
  https://www.shellcheck.net/wiki/SC2154 -- PRETTY_NAME is referenced but not...


Premature end of file

Hi,
I want to download attachment from email message,but sometimes it can be throw exception like
microsoft.exchange.webservices.data.core.exception.service.remote.ServiceRequestException: The request failed. ParseError at [row,col]:[1,868391] Message: Premature end of file. 	at microsoft.exchange.webservices.data.core.request.SimpleServiceRequestBase.internalExecute(SimpleServiceRequestBase.java:74) 	at microsoft.exchange.webservices.data.core.request.MultiResponseServiceRequest.execute(MultiResponseServiceRequest.java:158) 	at microsoft.exchange.webservices.data.core.ExchangeService.internalGetAttachments(ExchangeService.java:1503) 	at microsoft.exchange.webservices.data.core.ExchangeService.getAttachment(ExchangeService.java:1538) 	at microsoft.exchange.webservices.data.property.complex.Attachment.internalLoad(Attachment.java:411) 	at microsoft.exchange.webservices.data.property.complex.Attachment.load(Attachment.java:430) 	at microsoft.exchange.webservices.data.property.complex.FileAttachment.load(FileAttachment.java:234)

Same problem. Anyone fix this?

same issue. get this error intermittently


prevent users from installing this package

along the same lines as https://github.com/pinojs/express-pino-logger/issues/41 and https://github.com/pinojs/express-pino-logger/issues/59, the existence of this lib just causes confusion, imo.  as noted in https://github.com/pinojs/express-pino-logger/pull/60, this is intended to be identical to pino-http, only more discoverable for people working with express.  however, it isn't kept up to date -- pino-http has been on major version 8 for 4 months, meanwhile this is still on version 7, which in fact erroneously depends on pino-http version 6.  (master depends on version 7, but that change hasn't been published).
this can be fixed forever if we just explicitly prevent people from installing this package and point them to the maintained one instead.

I think we should just ship a npm deprecate warning on this module instead of the postinstall script.

Yeah. Just archive this repo and put the warning up on npm.

Yeah sounds great.

Done


Vim extension insert mode prevents panes from updating

Describe the bug
This seems to be some sort of conflict with the Vim extension and entering into insert mode.
Splitting test and implementation left and right, when you update a value inside a test case using insert mode, the gutter coverage in the implementation doesn't update. Switching to another document and back again in the implementation pane causes it to update.
To Reproduce
Steps to reproduce the behaviour:
Pre: install Vim and Code Gutter

Go to javascript example
Split test-coverage.js to right, test.js on left
In test.js, enter into insert mode and change testFunc(1) to testFunc(21)
run npm run test
Observe no change to gutter coverage on line 2 in test-coverage.js
Open a new file in right pane (with test-coverage.js)
Click back to test-coverage.js
Observe the gutter has now updated
Use 'x' to delete the 2 you added.
Run npm run test
Observe that gutter coverage updated

Changes that do not enter into insert mode (e.g. 'x' to delete a char, 'dd' to delete line, cmd+z, cmd+x) do not seem to trigger the problem.
Expected behaviour
When using insert mode, the gutter coverage should still update in test-coverage.js immediately.
Screenshots

Desktop (please complete the following information):

OS: macos
Extension Version: 2.4.0
VSCode Version: 1.37.1

Additional context
Add any other context about the problem here.

@lukebarton Most likely this is due to how the vim extension edit mode triggers events that the extension listens too (or in your case doesn't listen too). If you exit edit mode does the coverage update then? What about if you save the file, after exiting edit mode, does the coverage update?

@lukebarton going to close this due to age but if you get back to me with more information I will gladly open it again.


DuplicateOptionAnnotationsException thrown on mixinStandardHelpOptions by picocli-codegen

DuplicateOptionAnnotationsException is thrown by picocli-codegen when mixinStandardHelpOptions is set to true on both a command with scope INHERIT and a subcommand.
Example:
@CommandLine.Command(
        name = "example",
        mixinStandardHelpOptions = true,
        scope = CommandLine.ScopeType.INHERIT,
        subcommands = AutoComplete.GenerateCompletion.class
)
public class Example {
}

Stacktrace:
[ERROR] FATAL ERROR: picocli.CommandLine$DuplicateOptionAnnotationsException: Option name '-h' is used by both field boolean picocli.CommandLine$AutoHelpMixin.helpRequested and field boolean picocli.CommandLine$AutoHelpMixin.helpRequested
        at picocli.CommandLine$DuplicateOptionAnnotationsException.create(CommandLine.java:17963)
        at picocli.CommandLine$DuplicateOptionAnnotationsException.access$2500(CommandLine.java:17957)
        at picocli.CommandLine$Model$CommandSpec.addOption(CommandLine.java:6572)
        at picocli.CommandLine$Model$CommandSpec.addMixin(CommandLine.java:6818)
        at picocli.CommandLine$Model$CommandSpec.mixinStandardHelpOptions(CommandLine.java:7157)
        at picocli.CommandLine$Model$CommandSpec.mixinStandardHelpOptions(CommandLine.java:7172)
        at picocli.CommandLine$Model$CommandSpec.mixinStandardHelpOptions(CommandLine.java:7172)
        at picocli.codegen.annotation.processing.AbstractCommandSpecProcessor$Context.connectModel(AbstractCommandSpecProcessor.java:922)
        at picocli.codegen.annotation.processing.AbstractCommandSpecProcessor$Context.access$000(AbstractCommandSpecProcessor.java:807)
        at picocli.codegen.annotation.processing.AbstractCommandSpecProcessor.tryProcess(AbstractCommandSpecProcessor.java:194)
        at picocli.codegen.annotation.processing.AbstractCommandSpecProcessor.process(AbstractCommandSpecProcessor.java:165)

Hi @charphi , this is by design:
putting scope=INHERIT on a command means that you don't need to put mixinStandardHelpOptions = true on any subcommands any more.
This is documented in the user manual: https://picocli.info/#_inherited_command_attributes
Other options are not inherited.

Hi @charphi , this is by design:
putting scope=INHERIT on a command means that you don't need to put mixinStandardHelpOptions = true on any subcommands any more.
This is documented in the user manual: https://picocli.info/#_inherited_command_attributes
Other options are not inherited.

Oh, I see what you mean now; since GenerateCompletion is a built-in command, people may expect to be able to combine it with scope = INHERIT.
Perhaps we should add a WARN admonition to the Inherited Command Attributes section that this is a known limitation.

Oh, I see what you mean now; since GenerateCompletion is a built-in command, people may expect to be able to combine it with scope = INHERIT.
Perhaps we should add a WARN admonition to the Inherited Command Attributes section that this is a known limitation.

Ah wait, I think I understand what you mean (this time for real :rofl: - at least I hope...)
There is an inconsistency: at runtime, the above Example class does not throw a DuplicateOptionAnnotationsException; it has some special logic to prevent this error.
However, this special logic is missing from the annotation processor, so at compile time, when the annotation processor is enabled, we are seeing this error.
So, we should fix the annotation processor to not fail on the above Example class.

Ah wait, I think I understand what you mean (this time for real :rofl: - at least I hope...)
There is an inconsistency: at runtime, the above Example class does not throw a DuplicateOptionAnnotationsException; it has some special logic to prevent this error.
However, this special logic is missing from the annotation processor, so at compile time, when the annotation processor is enabled, we are seeing this error.
So, we should fix the annotation processor to not fail on the above Example class.

I pushed a fix to master. Please verify when you have a chance.
Thinking to release 4.6.2 soon, that includes this fix.

I've just tested and it works. Thanks!


allow privileged into core network

In our dev setup, we don't use a 'core network'; didn't catch this earlier.

Codecov Report

Merging #470 (4507e69) into master (0bf6e4f) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #470      +/-   ##
==========================================
- Coverage   39.46%   39.43%   -0.04%
==========================================
  Files          34       34
  Lines        5263     5265       +2
==========================================
- Hits         2077     2076       -1
- Misses       3186     3189       +3

Impacted Files
Coverage Δ

assemblyline_core/scaler/controllers/docker_ctl.py
12.11% <0.00%> (-0.08%)
:arrow_down:

assemblyline_core/dispatching/dispatcher.py
76.42% <0.00%> (-0.13%)
:arrow_down:

Continue to review full report at Codecov.

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


Add e2e test for cors annotations #171

Ⅰ. Describe what this PR did
Add e2e test for cors annotations #171
Ⅱ. Does this pull request fix one issue?
NO
Ⅲ. Why don't you add test cases (unit test/integration test)?
this is a test PR
Ⅳ. Describe how to verify it
verify by make e2e test
Ⅴ. Special notes for reviews

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

Codecov Report

Merging #179 (65136ee) into main (9400f7b) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #179   +/-   ##
=======================================
  Coverage   42.33%   42.33%
=======================================
  Files          29       29
  Lines        4866     4866
=======================================
  Hits         2060     2060
  Misses       2645     2645
  Partials      161      161

Thanks @Hinsteny, ping me when you are ready for review.

@Xunzhuo Please help to Review it again


Webpack plugins should be separate npm's

So that they can be used in i.E. https://github.com/shama/gulp-webpack without adding webpack itself as a dependency.

Which plugin are you trying to use? I think they are all exposed, e.g.
var gulpWebpack = require('gulp-webpack');
var plugin = new gulpWebpack.webpack.DefinePlugin()

@binarykitchen Is this relevant still?

nope


RFC: Radio Groups as card layouts

Which scope/s are relevant/related to the feature request?
radio-group
Information
Allow Radio Groups to be styled in any layout such as these cards.

label is part of BrnRadioComponent how to style the label for custom layouts? Should the label be extracted from BrnRadioComponent to be styled by the user and follow a similar API as shadcn?
Shadcn API
<RadioGroup defaultValue="option-one">

            <RadioGroupItem
              value="card"
              id="card"
              className="peer sr-only"
              aria-label="Card"
            />

              <svg
                xmlns="http://www.w3.org/2000/svg"
                viewBox="0 0 24 24"
                fill="none"
                stroke="currentColor"
                strokeLinecap="round"
                strokeLinejoin="round"
                strokeWidth="2"
                className="mb-3 h-6 w-6"
              >
                <rect width="20" height="14" x="2" y="5" rx="2" />
                <path d="M2 10h20" />
              </svg>
              Card

   ...
</RadioGroup>

Describe any alternatives/workarounds you're currently using
No response
I would be willing to submit a PR to fix this issue

[X] Yes
[ ] No

How about we make styles and classes for the label, indicator, input inside the brn-radio inputs that can be overwritten?
That way we have more flexibility on how to style the actual button. The anatomy of the brn-radio is inspired by Material, and I think in the case you are describing the label would be marked with sr-only and the indicator/touchTarget is where you would style the Card

Input component design
API design when we would add class inputs for label, indicator and input for brn-radio could look like
<brn-radio-group hlm class="grid grid-cols-3 gap-4" [(ngModel)]="card">
  <brn-radio hlm value="card" labelClass="...">
    <svg>...</svg>
    Card
  </brn-radio>
   ...
</brn-radio-group>

What about refactoring the hlm directives to components to use them instead
<hlm-radio-group class="grid grid-cols-3 gap-4" [(ngModel)]="card">
  <hlm-radio value="card" labelClass="...">
    <svg>...</svg>
    Card
  </hlm-radio>
   ...
</hlm-radio-group>

Extract label component design
The API could look like as follow when we pull out the label from brn-radio
<brn-radio-group hlm class="grid grid-cols-3 gap-4" [(ngModel)]="version">

    <brn-radio hlm value="card" />

      <svg>...</svg>
      Card

  ...
</brn-radio-group>

Or with hlm components
<hlm-radio-group class="grid grid-cols-3 gap-4" [(ngModel)]="version">

    <hlm-radio value="card" />

      <svg>...</svg>
      Card

  ...
</hlm-radio-group>

You think it would be better to move the card layout into the indicator selector, something like this:
<brn-radio-group hlm class="grid grid-cols-3 gap-4" [(ngModel)]="card">
  <brn-radio hlm value="card" labelClass="sr-only">

      <svg>...</svg>
      Card

    Card
  </brn-radio>
</brn-radio-group>

Interesting. What are you thinking?

After reviewing this I think my absolute favorite would be this. And inside hlm-radio we allow you access to the label classes and styles.
<hlm-radio-group class="grid grid-cols-3 gap-4" [(ngModel)]="version">
  <hlm-radio value="card">
      Card
  </hlm-radio>
  ...
</hlm-radio-group>

And then for a custom radio we could do something like this:
<brn-radio-group class="grid grid-cols-3 gap-4" [(ngModel)]="card">
  <brn-radio value="card" labelClass="sr-only">

      <svg>...</svg>
      Card

    Card
  </brn-radio>
</brn-radio-group>

This sounds good to me.
<hlm-radio-group class="grid grid-cols-3 gap-4" [(ngModel)]="version">
  <hlm-radio value="card">
      Card
  </hlm-radio>
  ...
</hlm-radio-group>

But the for the custom radio, I don't like that we need to repeat the label. What is there reason not to project the custom radio into the label and apply the needed styles?

@goetzrobin I started with converting the hlm radiogroup directives to components. However, it is not working yet.
For example the hlm-radio wraps brn-radio, but brn-radio injects the BrnRadioGroupComponent which is undefined. Should I create a draft PR and we can discuss a solution there? Is it possible for HlmRadio to extend BrnRadio?
@Component({
	selector: 'hlm-radio',
	template: `
		<brn-radio [class]="_computedClass()" [value]="value()" [disabled]="disabled()">
			<ng-content></ng-content>
		</brn-radio>
	`,
	imports: [BrnRadioComponent],
	standalone: true,
	host: {
		class: 'block',
	},
})
export class HlmRadioComponent {

@marcjulian that works for me. I was also going to take a crack at it, but if you already started we can combine efforts!


BAT Handling in Checkpoint Disks Issues

Purpose or Intent
Address two problems handling the BAT for Checkpoint disks:
Depending upon the disk an attempt may be made to
access BAT information prior to initializing it, so initialize when appropriate.
Based on the VHDX spec the size of the BAT for checkpoint disks is:
Total BAT Entries = Sector Bitmap Blocks Count *  (Chunk Ratio + 1)
but there are checkpoint disks where the Sector Bitmap Blocks count is 0.
This is because the calculation of Sector Bitmap Blocks count (as well as
Data Blocks count) was using Integer division and ".ceil", resulting in an
incorrect value.
Links
The above formulas are found in the VHDX spec here:
https://docs.com/grigory-polischuk-1/4673/vhdx-format-specification-v1-00
@roliveri @Fryguy please review when able.  Thanks.

So based on investigation and discovery of the true culprit here the bug was in calculation of the sector_bitmap_blocks_count and data_blocks_count - using integer division and ".ceil".  The integers were changed to floats and this problem has been addressed.  The discussion of memo-ization of the bat access is moot.
@roliveri @Fryguy please review and merge.  Thanks.

LGTM :+1:  @roliveri ?

Darga backport details:
commit df5a641951fe21505346fe8e7d57d7832a771e7a
Author: Richard Oliveri<EMAIL_ADDRESS>Date:   Wed Jul 27 23:21:00 2016 -0400

    Merge pull request #10063 from jerryk55/vhdx_checkpoint_handling_bug

    BAT Handling in Checkpoint Disks Issues
    (cherry picked from commit f4f9d1043471ffe62b04beb703a7994ee0dbc865)

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


Regex Issue with beginning of line in feedlist/articlelist

Hey,
So I tried using this line in my config file: highlight feedlist "^[0-9 ]+ N " white color8 bold underline
And it won't match the new/unread feeds. Same thing happens with highlight articlelist. However this line: highlight feedlist "[0-9 ]{4,} N " white color8 bold underline
matches perfectly fine. So what's the issue here? I've tested it with https://www.regexpal.com/, so try the first one there with your article/feed line. It should work!
This problem has existed in newsbeuter before, but finally got around to reporting it.
Newsboat version (copy from newsboat -v):
newsboat 2.12.0
System: Linux 4.17.0-1-amd64 (x86_64)
Compiler: g++ 7.3.0
ncurses: ncurses 6.1.20180714 (compiled with 6.1)
libcurl: libcurl/7.60.0 GnuTLS/3.5.19 zlib/1.2.11 libidn2/2.0.4 libpsl/0.20.2 (+libidn2/2.0.4) libssh2/1.8.0 nghttp2/1.32.0 librtmp/2.3 (compiled with 7.60.0)
SQLite: 3.24.0 (compiled with 3.24.0)
libxml2: compiled with 2.9.4

Hi! There are many more regex syntaxes than Javascript and PCRE presented on regexpal.com. In this instance, Newsboat uses POSIX Extended ones (like grep --extended).
Both your regular expressions are valid, but the first one doesn't match because Newsboat puts a special tag at the beginning of lines that contain unread feeds. Looks like this:
    1   (0/23)  Some feed
    2   (0/100) Another feed
<unread>    3 N (1/13) Unread feed

Naturally, the first regex doesn't match the last line.
This is more of an architectural error, and I don't know offhand how best to fix it. So there won't be an immediate patch. But we'll get to it eventually.
Thank you for the report! If you have any other long-standing issues that you never came around to report, please do so—we're currently in the process of reworking Newsboat's internals, and it would be nice to know of as many bugs as possible so we can do things right^Wbetter this time around.


No ChefSpec or InSpec/ServerSpec tests for this cookbook.

As the Issue Subject says, there are no ChefSpec, or InSpec or ServerSpec tests for the cookbook.
I'm not sure if that's because its not possible to test because of the availability of the McAfee product for public testing or not.

Hi there,
You are correct - the main reason being that you need to obtain the software and at the time of the library writing it wasn't something that could be freely installed. I left McAfee about a year+ ago but I can check and see if something has changed there. I'm assuming you currently must use or have McAfee in house?


Second empty page on the app tray.

To rep: (assumes a single page of apps)
On the home page Swipe up to open the app tray
Then swipe up again.
You get a second empty page.
(had reported this in TJC and forgot to put it here since it seems to be an issue with out port)

Could it be that we have .desktop files that not appear on the launcher that cause this???

Can confirm this. On XZ2C, if there is an icon in the last row, it results in an extra blank page, no additional page is created if the last row is kept empty. Wonder if it is because lipstick thinks not all icons are accommodated in one page. I tried changing the UI scaling to 1.75, didn't help though.

XZ2C can accommodate 7 rows in its screen. Compared to the original J1 which has 6 and the issue doesn't happen there i kind of suspect a hard coded value somewhere. No idea what happens to the other officially supported phones.


[BUG]App not reflecting edited or deleted messages

Description
To demonstrate that it is possible to edit messages, I did a test chat with a user via DM. The other user deleted his messages on his android app which is correctly shown on the web interface, however, all messages, edited, before the edit and deleted still appear on my Android client
Devices and Versions

Your Rocket.Chat.Android version: 3.2.0
Your Rocket.Chat Server version:  0.72.0
--

Mobile device model and OS version: Huawei P20 Pro, Android 8.1.0
Steps to reproduce

User 1 delete or edit message on DM.

User 2 sees message as deleted and edited on Web Interface.

Android App shows addition messages that is edited and deleted message still present.

The problem could be identical to issue #1754


Update hash-rules.md

Hash 256 updates.
Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions and updating sections
[ x] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

Disabling Auto merge. Waiting on Maise

Team decided to wait until Monday.


Refactor Lambdas to use queue prefix and reuse logic

https://github.com/cds-snc/notification-terraform/pull/188 introduced a new variable to hold the SQS queue prefix. We should use this in SES and SNS Lambdas.
On top of this, it would be good to refactor code and put methods in a common lib because we often need to get a queue and push Celery envelopes into it in all these Lambdas.
We could have a small Python file holding this logic that would be included in all 3 Lambdas.

Ping @jimleroyer, as requested https://github.com/cds-snc/notification-terraform/pull/188#discussion_r573951345


Docstring changes for docs API

I've edited ephys_acute.py and ephys_chronic.py to test whether the docs API renders correctly for InsertionLocation, ClusteringParamSet, Unit, and Electrode tables. In local deployment, the API rendered without issues even before the changes were made, making it difficult to test the changes locally.
If this results in rendering the API successfully, I will make these changes for all modules and tables that are not covered in this PR.
Other changes to ephys_acute.py and ephys_chronic.py were performed by the Black formatter in VSCode.

@CBroz1, the merge conflicts in CHANGELOG are resolved now. I tried to address them using GitHub's web interface but I wasn't sure if that would've resulted in a self-merge when resolved so I've resolved conflicts by pulling from upstream and resolving locally.

Thanks @kushalbakshi.
In the future, we should separate any blackifying changes into a single commit so that the reviewer can inspect intentional changes separately. I set up a vscode shortcut to toggle format on save - lmk if you'd like help setting that up.

Also, this makes sense. I will keep that in mind for future commits. And I'll reach out to you separately to set up the shortcut. That would make things a lot easier.

@kushalbakshi Did you mirror the changes to the other ephys_* modules?  I may have missed those updates if it was in a separate pull request.

@kushalbakshi Did you mirror the changes to the other ephys_* modules? I may have missed those updates if it was in a separate pull request.

I just went back and looked through the site to double check. Everything seems to be rendering correctly on all ephys modules.


Build warnings: Assembly has non matching file name

Steps to reproduce

Import SharpZipLib in Unity package Manager
Switch to windows build target
Build

Expected behavior
No warnings when building
Actual behavior
Assembly 'ICSharpCode.SharpZipLib' has non matching file name: 'SharpZipLib.dll'. This can cause build issues on some platforms.

Version of SharpZipLib
1.2.2-preview.1
Obtained from (only keep the relevant lines)

Package installed using UPM

I'm not sure what that means - the official builds should have the file name ICSharpCode.SharpZipLib.dll already.
Also not sure where version 1.2.2 comes from  - the official releases are just 1.2.0 and 1.3.0.

Ah I see. It seems Unity write their own wrapper
https://github.com/needle-mirror/com.unity.sharp-zip-lib

Sorry I thought the upm package came directly from this repository.

@StephenHodgson I were looking into providing a Unity package, but since there doesn't seem to be any "default" package source (like nuget.org) I dropped it.
I'm also not too thrilled about them using different versioning. Perhaps there is a good reason?
But yeah, their package seem to build from our source but then rename the output assembly for whatever reason.

Check out OpenUPM. It's a good up and coming default package source. Also you can host directly on npm for upm packages.
OpenUPM uses GitHub directly as the source and just caches as needed.

I'm also not too thrilled about them using different versioning. Perhaps there is a good reason?
Yeah doesn't surprise me that Unity is doing their own thing.

I tried created a branch for UPM, and adding it works in Unity using this URL: https://github.com/icsharpcode/SharpZipLib.git#upm
I also managed to add v1.3.1 to OpenUPM.
Not sure if this is a good way to do it, but it does seem to work. 🤷♀


Extract dask.tf into individual modules with separate state files for different tiers

First pass at reorganising Terraform config. This new organisation creates identical infrastructure as before, but is more neatly organised. Next steps would be:

Set up env-prod
Apply changes to other terraform configs
Rationalise all terraform config into two repos:

jade-terraform-modules
jade-terraform-environments
This allows for modules to be tagged on github and for production environments to use pinned versions, e.g.
module "dask-worker" {
source = "github.com/met-office-lab/jade-terraform-modules/blob/0.10.2/example"
}
Jacob + Niall, would be good to get your thoughts here as it's a bit different from what we discussed.
Alex.

Another example of splitting module / provider: https://github.com/hashicorp/best-practices/tree/master/terraform


@IBInspectable Error

I keep getting this Error:

Property cannot be marked @IBInspectable because its type cannot be represented in Objective-C

on the following line of code in ColorMatchTabsViewControllerDataSource:

@IBInspectable open weak var dataSource: ColorMatchTabsViewControllerDataSource? {
}

@rnkyr How to fix it when i installed this pod ?

Hi, @dieptran00888
Thanks for pointing out this issue one more time. I've fixed it in feature/swift-4
Cocoapods trunk will be updated in a few days. Meanwhile, you can check the fixe directly from master branch


Default argument error with C API: Waves.h

When using load library in Matlab to load the C API, the following error message is produced:
------Error in the Model, Error-Causes/Messsages are :
Building libmoordyn_thunk_maci64 failed.  Compiler output is:
/usr/bin/xcrun -sdk macosx14.0 clang -I"/Applications/MATLAB_R2023a.app/extern/include" -fno-common -arch x86_64 -mmacosx-version-min=10.15 -fexceptions -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX14.0.sdk -I"/Users/rdavies/work/wecsim_libs/WEC-Sim_Applications/Mooring/MoorDyn" -I"/Users/rdavies/work/wecsim_libs/WEC-Sim/source/functions/moorDyn" "libmoordyn_thunk_maci64.c" -o "libmoordyn_thunk_maci64.dylib" -bundle
In file included from libmoordyn_thunk_maci64.c:27:
In file included from /Users/rdavies/work/wecsim_libs/WEC-Sim/source/functions/moorDyn/MoorDyn2.h:39:
/Users/rdavies/work/wecsim_libs/WEC-Sim/source/functions/moorDyn/Waves.h:77:59: error: C does not support default arguments
                                        MoorDynSeafloor seafloor = 0);

I worked around it by removing the seafloor input at line 76 in Waves.h and at line 899 in Waves.cpp. At line 909  in Waves.cpp I replaced the seafloor variable with 0. This a temporary fix because it removes the capability, but necessary for use of the C API.
Note: we are not using the Matlab wrapper with WECSim because of compatibility issues with older versions of Matlab

Yeah, @AlexWKinley we should be careful of not adding C++ stuff on the .h files.
We are fixing this for 2.2.2

Since this require changing the ABI, I decided to move this to the next minor release, 2.3.0


Apidoc reformatting

Following up to the comments on my first API docs fix PR, here is a replacement of the language keywords null and true (false didn't occur anywhere in the API docs, it seems) with the <see langword="..."/> tag.

Thank you!


WIP: adds flake8-builtins

I mark this as a WIP, because I don't have rust on my machine right now because of the SECURITY reasons 😒
So, I've never actually executed the code I wrote.
Plus, I need to write tests.
Original source: https://github.com/gforcada/flake8-builtins/blob/22566d388b9ac0a87ec2efc8d7be95c3303ed651/flake8_builtins.py

Looking good! Do you plan to run + test + get it out of draft state? Or want a hand given the above comment RE installing Rust?

I actually had to find a way to run this to find proper column numbers 🙈
It was fun writting code without ever running it!

Hahaha I don't understand how you did this

Code looks great, merging shortly...


Add Sepolia deployment

I assume tests are probably failing because it's trying to access secrets from @pclaesen's repo and the env vars are not set there.

Tests run successfully after the merge.


about test.py

when I run test.py, it raises some error, I don't know what the exactly problem it is, can you please help me?
Model restored.
0 / 68 ...
Loading train file /home/ASIS/data/stanford_indoor3d_ins.sem/Area_5_conferenceRoom_1.npy
Processsing: Shape [0] Block[0]
2019-09-14 18:39:28.811094: E tensorflow/stream_executor/cuda/cuda_blas.cc:636] failed to run cuBLAS routine cublasSgemm_v2: CUBLAS_STATUS_EXECUTION_FAILED
Traceback (most recent call last):
File "test.py", line 262, in
test()
File "test.py", line 183, in test
feed_dict=feed_dict)
File "/home/anaconda3/envs/asis/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 895, in run
run_metadata_ptr)
File "/home/anaconda3/envs/asis/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1128, in _run
feed_dict_tensor, options, run_metadata)
File "/home/anaconda3/envs/asis/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1344, in _do_run
options, run_metadata)
File "/home/anaconda3/envs/asis/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1363, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InternalError: Blas SGEMM launch failed : m=32768, n=32, k=9
[[Node: layer1/conv0/Conv2D = Conv2D[T=DT_FLOAT, data_format="NHWC", dilations=[1, 1, 1, 1], padding="VALID", strides=[1, 1, 1, 1], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/device:GPU:0"](layer1/concat, layer1/conv0/weights/read/_293)]]
[[Node: ins_fa_layer3/Squeeze/_585 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_2283_ins_fa_layer3/Squeeze", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]]
Caused by op u'layer1/conv0/Conv2D', defined at:
File "test.py", line 262, in
test()
File "test.py", line 79, in test
pred_sem, pred_ins = get_model(pointclouds_pl, is_training_pl, NUM_CLASSES)
File "/home/ASIS/models/ASIS/model.py", line 29, in get_model
l1_xyz, l1_points, l1_indices = pointnet_sa_module(l0_xyz, l0_points, npoint=1024, radius=0.1, nsample=32, mlp=[32,32,64], mlp2=None, group_all=False, is_training=is_training, bn_decay=bn_decay, scope='layer1')
File "/home/ASIS/utils/pointnet_util.py", line 187, in pointnet_sa_module
data_format=data_format)
File "/home/ASIS/utils/tf_util.py", line 165, in conv2d
data_format=data_format)
File "/home/anaconda3/envs/asis/lib/python2.7/site-packages/tensorflow/python/ops/gen_nn_ops.py", line 639, in conv2d
data_format=data_format, dilations=dilations, name=name)
File "/home/anaconda3/envs/asis/lib/python2.7/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/home/anaconda3/envs/asis/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 3160, in create_op
op_def=op_def)
File "/home/anaconda3/envs/asis/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1625, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
InternalError (see above for traceback): Blas SGEMM launch failed : m=32768, n=32, k=9
[[Node: layer1/conv0/Conv2D = Conv2D[T=DT_FLOAT, data_format="NHWC", dilations=[1, 1, 1, 1], padding="VALID", strides=[1, 1, 1, 1], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/device:GPU:0"](layer1/concat, layer1/conv0/weights/read/_293)]]
[[Node: ins_fa_layer3/Squeeze/_585 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_2283_ins_fa_layer3/Squeeze", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]]
Annother question: I can run train.py and estimate_mean_ins_size.py  successfully, but I was wondering,that why train.py just takes about several minutes to train.
waiting for your kindly response! thanks very much

Please ask how your problem was resolved, I have encountered the same situation and look forward to getting your reply.


ROS 2 Tutorials-Teleoperation keyboard

I set up all configs and the related parts. Also, I've not gotten any errors but the cf is not getting the keyboard command for take-off. I added the terminal outputs associated with the related parts below. What could be the reason, thanks.
first command:
ros2 launch crazyflie_examples keyboard_velmux_launch.py
terminal output:
ntukenmez3@ae-icps-407120:~/Documents/ros2_ws/src$ ros2 launch crazyflie_examples keyboard_velmux_launch.py
[INFO] [launch]: All log files can be found below /home/ntukenmez3/.ros/log/2023-10-17-17-40-13-213907-ae-icps-407120-3879157
[INFO] [launch]: Default logging verbosity is set to INFO
[INFO] [crazyflie_server.py-1]: process started with pid [3879158]
[INFO] [vel_mux.py-2]: process started with pid [3879160]
[crazyflie_server.py-1] [INFO] [1697578815.432539570] [crazyflie_server]:  radio://0/80/2M/E7E7E7E7E7 is fully connected!
[crazyflie_server.py-1] [INFO] [1697578815.432841106] [crazyflie_server]: All Crazyflies are fully connected!
[crazyflie_server.py-1] [INFO] [1697578815.437555995] [crazyflie_server]:  radio://0/80/2M/E7E7E7E7E7: commander.enHighLevel is set to 1
[crazyflie_server.py-1] [INFO] [1697578815.459489481] [crazyflie_server]:  radio://0/80/2M/E7E7E7E7E7: locSrv.extPosStdDev is set to 0.001
[crazyflie_server.py-1] [INFO] [1697578815.459974424] [crazyflie_server]:  radio://0/80/2M/E7E7E7E7E7: locSrv.extQuatStdDev is set to 0.05
[crazyflie_server.py-1] [INFO] [1697578815.476950645] [crazyflie_server]:  radio://0/80/2M/E7E7E7E7E7: stabilizer.controller is set to 1
[crazyflie_server.py-1] [INFO] [1697578815.477449967] [crazyflie_server]:  radio://0/80/2M/E7E7E7E7E7: stabilizer.estimator is set to 2
[crazyflie_server.py-1] [INFO] [1697578815.483605848] [crazyflie_server]: All Crazyflies parameters are initialized
[crazyflie_server.py-1] [INFO] [1697578815.485278574] [crazyflie_server]: All Crazyflies loggging are initialized

second command (on the second terminal):
ros2 run teleop_twist_keyboard telop_twist_keyboard
terminal output:
ntukenmez3@ae-icps-407120:~/Documents/ros2_ws/src$ ros2 run teleop_twist_keyboard teleop_twist_keyboard

This node takes keypresses from the keyboard and publishes them
as Twist messages. It works best with a US keyboard layout.

For Holonomic mode (strafing), hold down the shift key:
---------------------------
   U    I    O
   J    K    L
   M    <    >

t : up (+z)
b : down (-z)

anything else : stop

q/z : increase/decrease max speeds by 10%
w/x : increase/decrease only linear speed by 10%
e/c : increase/decrease only angular speed by 10%

CTRL-C to quit

currently:	speed 0.5	turn 1.0

Hi! I can confirm this! it seems that the vel_mux script is not starting up from the launch file so I'll need to check why that is.

aha found it pretty quickly. You've got to change the crazyflie id here.
https://github.com/IMRCLab/crazyswarm2/blob/a83684e31f7a164f3729f98fadc89e775293be19/crazyflie_examples/launch/keyboard_velmux_launch.py#L34
I did find, that if you take off and land, you can't take off again. So that is a different issue. For now you've got to restart the crazyflie.

aha found it pretty quickly. You've got to change the crazyflie id here.
https://github.com/IMRCLab/crazyswarm2/blob/a83684e31f7a164f3729f98fadc89e775293be19/crazyflie_examples/launch/keyboard_velmux_launch.py#L34
I did find, that if you take off and land, you can't take off again. So that is a different issue. For now you've got to restart the crazyflie.

What could be the reason for this issue, I mean after the first take-off and land motion, why cannot send a take-off command?

After the crazyflie id was changed as "cf1-->c231", it worked pretty well.

The not taking off after a land, it's probably due to the chances in the latest crazyflie firmware so if you have an older release of the firmware it should work fine.

I tried it, but it did not work.

ahh oke then you have the latest release. No worries, I'm working on it :) https://github.com/IMRCLab/crazyswarm2/pull/309
with the current situation, you just have to restart the crazyflie everytime you want it to take off again...

I'm converting this to a dicussion as this wasn't really an error but a support question. For the later issue there is already an issue made https://github.com/IMRCLab/crazyswarm2/issues/308


Remove link to deprecated cluster monitoring

Cluster monitoring user heapster has been removed as part of the merge request https://github.com/kubernetes/kubernetes/pull/85512. Therefore the section about cluster monitoring as well as the link to the now missing page was removed.

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Remember to delete this note before submitting your pull request.
For pull requests on 1.17 Features: set Milestone to 1.17 and Base Branch to dev-1.17
For pull requests on Chinese localization, set Base Branch to release-1.16
Feel free to ask questions in #kubernetes-docs-zh
For pull requests on Korean Localization: set Base Branch to dev-1.16-ko.<latest team milestone>
If you need Help on editing and submitting pull requests, visit:
https://kubernetes.io/docs/contribute/start/#improve-existing-content.
If you need Help on choosing which branch to use, visit:
https://kubernetes.io/docs/contribute/start#choose-which-git-branch-to-use.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

/assign @daminisatya

/remove-lgtm
@tom1299 I think this PR should target the dev-1.18 branch rather than master. What do you think?

/lgtm cancel

Page preview:
https://deploy-preview-18193--kubernetes-io-master-staging.netlify.com/docs/concepts/configuration/manage-compute-resources-container/

/cc @VineethReddy02

Actually, @tom1299, would you be willing to rebase these commits against branch dev-1.18?

Actually, @tom1299, would you be willing to rebase these commits against branch dev-1.18?

Well, I did my very best :wink:

👋 Please rebase this PR on dev-1.18 in order to avoid merge conflicts.
Feel free to /hold cancel after you've rebased.
/hold

/hold cancel

@sftim Can we get this merged? I see some changes after your lgtm

@VineethReddy02 the force-push looks like a rebase to me and the preview looks good too.
It's a small change and you sound happy to have it in the release, so:
/lgtm
/approve


Add challenge authentication to azure-keyvault-keys

This implements HTTP challenge-based authentication for azure-keyvault-keys (another PR will copy this code to azure-keyvault-secrets). Challenge parsing and caching code is largely from the track 1 implementation.
Here's the gist of how the new auth policy works. On each request,

Retrieve a cached challenge for the URL

If cache misses, send an unauthorized, bodiless request to elicit a challenge; parse and cache that

Authenticate as instructed by the challenge
Add authorization to, and send, the original request
If the response is 401, it's possible we have an outdated challenge cached. Try to parse and cache a challenge from the response.

If this fails, something else is wrong -> return the response
If this succeeds, authenticate the request according to the new challenge and send it again (if the response is again 401 that response will be returned because this policy can do no more)

I made one interesting change to the challenge cache (which maps URLs to challenges): https URLs specifying port 443 are considered equivalent to https URLs which do not, i.e. https://myvault.vault.azure.net:443 == https://myvault.vault.azure.net so far as the cache is concerned. I did this to prevent redundant challenges when iterating paged results. Our client doesn't include the port in URLs it forms, so the initial cache entry won't have a port in its key. Next page links formed by the Key Vault API, however, do include the port. Thus when the client requests a page using one of these links, exact URL matching causes erroneous cache misses and unnecessary auth flows.
FYI @Azure/adp-keyvault

Can one of the admins verify this patch?


Information

Hello , will be possible to delete on your site all my leaked information ? .

yes it would! Could you shoot me that info via email at<EMAIL_ADDRESS>i'll have it removed ASAP


When trying to install in wsl2 using ubuntu as Distro, the script for shell doesn't work

When trying to run the next script in bash or zsh "curl -fsSL https://deno.land/x/install/install.sh | sh "
print this message: "sh 39: unzip: not found"
Is this a known issue. Does it have a workaround?
thanks

This means that the unzip utility is required. The work-around is installing unzip, for example, with sudo apt-get install unzip.

@MarkTiedemann  Thank you so much, it works! I'm going to close the issue.


[solo] Fix skydns - two wrongs make a right

@gimaker @rohansingh @mattnworb @negz

:+1:

We should fix the state of skydns. Right now skydns is rebuilt from master here https://github.com/skynetservices/skydns whenever someone manually builds helios solo base image. We have no way of knowing what kind of skydns we've built later.

Current coverage is 48.96%

Merging #899 into master will increase coverage by +0.05% as of 0d0e5c9

@@            master    #899   diff @@
======================================
  Files          271     271
  Stmts        12706   12706
  Branches      1634    1634
  Methods          0       0
======================================
+ Hit           6215    6221     +6
- Partial        470     473     +3
+ Missed        6021    6012     -9

Review entire Coverage Diff as of 0d0e5c9

Powered by Codecov. Updated on successful CI builds.

Nice explanation in the commit message 👍🏻
Is it meant to commit the -verbose flag to skydns in here? I wonder if the output would become large if solo was running for a while and sending lots of DNS requests (I'm not sure how docker logs stores things).
For the unpredictable nature of building SkyDNS, is it available as a package to install with apt-get?

We admit this is super funky, but there's no way to force skydns to
speak only TCP right now.

And it shouldn't. What it should do is properly forward the response with the truncation flag set to the requester (who would then retry using TCP). Instead it just blows up and never responds to the requester at all :(

This doesn't work on my mac now. See #900 for an alternative approach


[RFC]  Integration with cuDNN via IREE compiler/runtime plugins

One of the initial goals of openxla-nvgpu plugin is to show how to integrate NVIDIA libraries with the IREE compiler/runtime. The work is already in progress, and few PRs are merged. The design of this integration is outlined in this document: https://docs.google.com/document/d/1WzSH7LdQdL1CQmlIOUyy6auDiX6d3cl5LAzZU_I4KCY/edit#

Thanks Eugene for sharing the doc, it looks like a solid plan.
I will try to cover some things that are not discussed in the doc here.
Input dialect
We initially started with lowering from mhlo, but switched to stablehlo now. The two dialects are mostly the same, so this shouldn't be difficult, but we will likely need to update the resnet50 model where we want to show the performance advantage of using libraries compared to the code that IREE is currently able to generate.
Do I understand correctly that the various IREE importers are targeting StableHLO and the standard IREE pipeline is able to consume this?
Layout assignment
This is currently being worked out on the IREE side. My very high level thinking is that it will provide an external layout interface that can be injected to StableHLO ops to communicate preferred layouts. We could then inject the same interfaces to cuDNN ops to constraint the layouts to what cuDNN expects.
Cost model
We will implement a model that determines the cost of StableHLO ops and their cuDNN graph equivalent. This will determine which subgraphs are outlined to cuDNN ops. This needs to take downstream fusion opportunities into account, because the performance profiles of a fused vs unfused op are vastly different.
Compilation pipeline
I need to leave this as a placeholder, because I haven't looked into it yet. But we should document our requirements for hooking into the IREE compilation pipeline. The main open issue here is when/how we perform the outlining of StableHLO ops and lowering to cuDNN ops. As far as I know, the downstream path from cuDNN ops is already working.


Support for batch import of cross referenced objects

Consider this scenario
In single batch I have referenced object and the object that references him.
Author {name: "John Doe", id: "1" }
Title {name: "foo", id: "99", author[{ beacon:"1" }] }
This would lead to error because due to multiprocessing on server title is processed before Author exists and weaviate checks the integrity.
Preferred way of resolving:
Etienne (Weaviate)   (https://weaviate.slack.com/archives/C017EG2SL3H/p1652791978632029?thread_ts=1652454634.332879&cid=C017EG2SL3H)
A potentially different solution that offloads a bit of risk to the user would be the following: We could add a flag that ignores the validation and accepts any reference whether the object exists or not. If the id is part of the same batch, it wouldn’t matter. If the id is not part of the batch and doesn’t exist elsewhere it would be a reference that can never be resolved which essentially the same state as if you delete a referenced object, so not a big problem.
I like this proposal since I know that I am inserting objects that don't exist yet and I am sure that I have them in the batch.
Additionally I'd suggest also extending functionality of client.data_object.exists (https://weaviate.io/developers/weaviate/current/restful-api-references/objects.html#check-if-a-data-object-exists-without-retrieving-it) to option of sending multiple ids (and retrieving the ids that don't exists) to allow managing reference integrity on client site in a more efficient way.

I think that this should be still open. For example mongodb has a simple parameter that enforces data to be procesed in order… (disabling pararelism I guess)
more people asked for this in slack

Agreed @ju-bezdek, with keeping this issue open.
@etiennedi could you re-open this one? I think this is still relevant:

A potentially different solution that offloads a bit of risk to the user would be the following: We could add a flag that ignores the validation and accepts any reference whether the object exists or not.

Agreed and reopened.

I've added the backlog label, so this appears on the feature roadmap page for voting and will not be subject to the stale bot anymore :-)