NPE in osmapi.Conversion

Occured around Quarter Jack Surgery in Wimborne.
This is probably because one of the element fields like username is not present.
Stacktrace:
java.lang.NullPointerException: null cannot be cast to non-null type kotlin.String
	at net.pfiers.osmfocus.service.osmapi.ConversionKt.jsonToElements(conversion.kt:71)
	at net.pfiers.osmfocus.service.osmapi.ElementsDownloadManager.processDownload(ElementsDownloadManager.kt:1)
	at net.pfiers.osmfocus.service.osmapi.EnvelopeDownloadManager$download$result$1$1.invokeSuspend(EnvelopeDownloadManager.kt:2)

Cheers for making the GitHub issue :)


Refactor Window(Left|Right)

This PR address #630
It depends on #655 that fix #652
It also do a part of #654

@atifaziz you can review #655 first


Improve exception handling in forked coroutines

(Spin-off from #893.)
In the long run, i think we need a better approach to handling exceptions in forked coroutines. Right now the following doesn't work as intended:
task = cocotb.fork(a_coroutine_that_fails_soon())
yield longer_than_soon()
try:
    yield task
except Exception as e:
    pass  # we never get as far as here right now, but I think we ought to
else:
    assert False

In the above example, the failure happens, and the outcome is stored. What happens if the forked coro is never yielded on? Where does the exception go?

In the above example, the failure happens, and the outcome is stored.

That's what should happen, but it's not what does happen

I don't agree. forked coroutines throwing exceptions should causing TestError. Many uses of fork are meant to start up 'threads' (yes, not great terminology but it mirrors fork) that run forever and are never meant to be yielded upon. This is used in many monitors and drivers. If exceptions are captured and the test never yields on the drive/monitor 'threads' (if they are meant to run forever they won't be yielded), those exceptions are never seen. I can see the how the behavior is useful, but I think it should exist in a new interface.

One potential way to support both behaviors is to make scheduler.add work as listed in the OP, and make fork wrap that with a function that yields on the coroutine and throws a TestError if an exception is caught. This will maintain the old behavior of fork while allowing the new behavior in a new interface that simply aliases scheduler.add as fork does now.
Also,there would have to be a special case for TestCompletes otherwise the test would continue longer than desired.

If exceptions are captured and the test never yields on the drive/monitor 'threads' (if they are meant to run forever they won't be yielded), those exceptions are never seen.

That doesn't have to be true. The test cleanup could collect exceptions from all the remaining "threads".


Sidecar container lifecycle changes since Kubernetes 1.18

Output of helm version:
version.BuildInfo{Version:"v3.12.1", GitCommit:"f32a527a060157990e2aa86bf45010dfb3cc8b8d", GitTreeState:"clean", GoVersion:"go1.20.4"}
Output of kubectl version  --output=yaml:
clientVersion:
buildDate: "2023-05-17T14:20:07Z"
compiler: gc
gitCommit: 7f6f68fdabc4df88cfea2dcf9a19b2b830f1e647
gitTreeState: clean
gitVersion: v1.27.2
goVersion: go1.20.4
major: "1"
minor: "27"
platform: linux/amd64
kustomizeVersion: v5.0.1
serverVersion:
buildDate: "2023-05-18T15:22:01Z"
compiler: gc
gitCommit: 890a139214b4de1f01543d15003b5bda71aae9c7
gitTreeState: clean
gitVersion: v1.26.5+rke2r1
goVersion: go1.19.9 X:boringcrypto
major: "1"
minor: "26"
platform: linux/amd64
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
RKE2
basically when installing a chart with a job which has a sidecar container lifecycle job won't finish.
helm install --namespace helm-test -f values.yaml helm-test1 ./testapp

apiVersion: batch/v1
kind: Job
metadata:
  name: {{ include "testapp.fullname" . }}
  annotations:
    helm.sh/hook: "pre-install, pre-upgrade"
spec:
  template:
    spec:
      containers:
        - image: nginx:latest
          lifecycle:
            type: Sidecar

Release "helm-test1" does not exist. Installing it now.
W0711 13:09:27.923456    4500 warnings.go:70] unknown field "spec.template.spec.containers[1].lifecycle.type"

nice to have, but also it's from 1.18, it'd be nice for helm to have it too

It seems that there is no field named type under Lifecycle , could you show me some document about it ?

This appears to be based on a mistaken belief that a proposed-but-not-actually-accepted feature back from 1.18 got implemented; even the name of the issue is very similar to the title of an article on the subject: Sidecar container lifecycle changes in Kubernetes 1.18


Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Minor Changes

ba02b18: Added tokens for the tabs component
Token diff
Tokens added (51):

accent-content-color-selected
neutral-subdued-content-color-selected
tab-item-bottom-to-text-compact-extra-large
tab-item-bottom-to-text-compact-large
tab-item-bottom-to-text-compact-medium
tab-item-bottom-to-text-compact-small
tab-item-bottom-to-text-extra-large
tab-item-bottom-to-text-large
tab-item-bottom-to-text-medium
tab-item-bottom-to-text-small
tab-item-compact-height-extra-large
tab-item-compact-height-large
tab-item-compact-height-medium
tab-item-compact-height-small
tab-item-focus-indicator-gap-extra-large
tab-item-focus-indicator-gap-large
tab-item-focus-indicator-gap-medium
tab-item-focus-indicator-gap-small
tab-item-height-extra-large
tab-item-height-large
tab-item-height-medium
tab-item-height-small
tab-item-start-to-edge-extra-large
tab-item-start-to-edge-large
tab-item-start-to-edge-medium
tab-item-start-to-edge-quiet
tab-item-start-to-edge-small
tab-item-to-tab-item-horizontal-extra-large
tab-item-to-tab-item-horizontal-large
tab-item-to-tab-item-horizontal-medium
tab-item-to-tab-item-horizontal-small
tab-item-to-tab-item-vertical-extra-large
tab-item-to-tab-item-vertical-large
tab-item-to-tab-item-vertical-medium
tab-item-to-tab-item-vertical-small
tab-item-top-to-text-compact-extra-large
tab-item-top-to-text-compact-large
tab-item-top-to-text-compact-medium
tab-item-top-to-text-compact-small
tab-item-top-to-text-extra-large
tab-item-top-to-text-large
tab-item-top-to-text-medium
tab-item-top-to-text-small
tab-item-top-to-workflow-icon-compact-extra-large
tab-item-top-to-workflow-icon-compact-large
tab-item-top-to-workflow-icon-compact-medium
tab-item-top-to-workflow-icon-compact-small
tab-item-top-to-workflow-icon-extra-large
tab-item-top-to-workflow-icon-large
tab-item-top-to-workflow-icon-medium
tab-item-top-to-workflow-icon-small

Run report for 1fafe09a
Total time: 34.7s | Comparison time: 32.9s | Estimated loss: 1.8s (5.3% slower)

Action
Time
Status
Info

🟩
SetupNodeTool(18.13.0)
4.2s
Passed

🟩
InstallNodeDeps(18.13.0)
11s
Passed

🟩
SyncNodeProject(tokens)
0.7ms
Passed

🟩
RunTarget(tokens:build)
18.2s
Passed

🟩
SyncNodeProject(visualizer)
0.2ms
Passed

🟩
SyncWorkspace
0ms
Passed

🟩
RunTarget(visualizer:noop)
17.6ms
Passed

🟩
RunTarget(tokens:generateDiffResult)
1.3s
Passed

🟩
RunTarget(tokens:noop)
1.3s
Passed

Touched files
.changeset/hip-insects-enjoy.md
packages/tokens/CHANGELOG.md
packages/tokens/package.json


(FACT-1423) Update Ubuntu fact test with new platform names

Ubuntu beaker platform names no longer use codenames, and
instead just use the version.
In addition, this commit adds test support for Ubuntu
15.10 and 16.04.

/cc @melissa from a RE point of view, would you prefer that we have a Facter test failure whenever we add a new Ubuntu, so we remember to test it?

👍

CLA signed by all contributors.


Fixed typo in Include conf.d statement

Collectd were including quite strange, non-default conf.d directory
"/opt/collectd/etc/collectd.conf.d".
I assume that it were typo, because default for that configuration
is "/opt/collectd/etc/conf.d". Fixed template to contain the
default path and ensure that directory exists.

Thank you!


Runtime error with classification models on data encrypted with Pyfhel

Hi,
I recently got my hands-on with this amazing library. I was able to encrypt/decrypt a simple image with help of Pyfhel's encrypt/decrypt operations. The image was converted to a 1D array first and then encryptBatch was applied. The resulted PyCtxt was decrypted with decryptBatch and converted back to image matrix while preserving its shape to get the original image.
I need to apply a classification model on this encrypted data now. The model should train with encrypted data and an unencrypted or encrypted label and predict the output.
However, I am getting following runtime error while training the model.

RuntimeError                              Traceback (most recent call last)
 in
----> 1 clf = LogisticRegression(random_state=42,verbose=10).fit(X_train, y_train)
~/anaconda3/lib/python3.7/site-packages/sklearn/linear_model/_logistic.py in fit(self, X, y, sample_weight)
1525
1526         X, y = check_X_y(X, y, accept_sparse='csr', dtype=dtype, order="C",
-> 1527                          accept_large_sparse=solver != 'liblinear')
1528         check_classification_targets(y)
1529         self.classes = np.unique(y)
~/anaconda3/lib/python3.7/site-packages/sklearn/utils/validation.py in check_X_y(X, y, accept_sparse, accept_large_sparse, dtype, order, copy, force_all_finite, ensure_2d, allow_nd, multi_output, ensure_min_samples, ensure_min_features, y_numeric, warn_on_dtype, estimator)
753                     ensure_min_features=ensure_min_features,
754                     warn_on_dtype=warn_on_dtype,
--> 755                     estimator=estimator)
756     if multi_output:
757         y = check_array(y, 'csr', force_all_finite=True, ensure_2d=False,
~/anaconda3/lib/python3.7/site-packages/sklearn/utils/validation.py in check_array(array, accept_sparse, accept_large_sparse, dtype, order, copy, force_all_finite, ensure_2d, allow_nd, ensure_min_samples, ensure_min_features, warn_on_dtype, estimator)
529                     array = array.astype(dtype, casting="unsafe", copy=False)
530                 else:
--> 531                     array = np.asarray(array, order=order, dtype=dtype)
532             except ComplexWarning:
533                 raise ValueError("Complex data not supported
~/anaconda3/lib/python3.7/site-packages/numpy/core/_asarray.py in asarray(a, dtype, order)
83
84     """
---> 85     return array(a, dtype, copy=False, order=order)
86
87
~/anaconda3/lib/python3.7/site-packages/pandas/core/series.py in array(self, dtype)
752               dtype='datetime64[ns]')
753         """
--> 754         return np.asarray(self.array, dtype)
755
756     # ----------------------------------------------------------------------
~/anaconda3/lib/python3.7/site-packages/numpy/core/_asarray.py in asarray(a, dtype, order)
83
84     """
---> 85     return array(a, dtype, copy=False, order=order)
86
87
~/anaconda3/lib/python3.7/site-packages/pandas/core/arrays/numpy_.py in array(self, dtype)
182
183     def array(self, dtype=None) -> np.ndarray:
--> 184         return np.asarray(self._ndarray, dtype=dtype)
185
186     _HANDLED_TYPES = (np.ndarray, numbers.Number)
~/anaconda3/lib/python3.7/site-packages/numpy/core/_asarray.py in asarray(a, dtype, order)
83
84     """
---> 85     return array(a, dtype, copy=False, order=order)
86
87
Pyfhel/PyCtxt.pyx in Pyfhel.PyCtxt.PyCtxt.float()
RuntimeError:  wrong PyCtxt encoding (not FRACTIONAL)

Can you share your code further? I'm afraid it is quite hard to replicate and investigate otherwise.

Maybe related to #42? If so, it was hopefully solved. Please try again with the latest version of Pyfhel. If the error persists, please provide code to reproduce the error.


shallowequal.js filepath errors when upgrading to 0.88.0

I'm upgrading from 0.81.1 to 0.88.0 and I'm getting this when I try to start my dev server:
./node_modules/semantic-ui-react/dist/commonjs/modules/Search/Search.js
Cannot find file: 'shallowequal.js' does not match the corresponding name on disk: './node_modules/semantic-ui-react/dist/commonjs/lib/shallowEqual.js'.

I don't see any mention of this issue online, so I imagine my dependencies are donked up somehow. Any tips? Modifying the file names to get rid of this error leads to a rabbithole of shallowequal related bugs, and of course is not a sustainable solution anyhow.

Try to remove your node_modules and install dependencies again, then please ensure that you're running 0.88.0 as there are no such lines:

<EMAIL_ADDRESS>


Encountering error with resource types

I have this .wit file
package testdemo:component;

interface clients {
  resource testdemo-client {
    hello-world: func() -> string;
  }
}

interface events {
  record foo {
    bar: string,
  }

  resource testdemo-event {
    parse: func(input: string) -> result<foo, string>;
  }

}

world testdemo {
  export clients;
  export events;
}

implemented by this Rust code
wit_bindgen::generate!({
    world: "testdemo",
    exports: {
        "testdemo:component/clients/testdemo-client": TestdemoClients,
        "testdemo:component/events/testdemo-event": TestdemoEvents,
     }});
use exports::testdemo::component::clients::GuestTestdemoClient;
use exports::testdemo::component::events::{Foo, GuestTestdemoEvent};

pub struct TestdemoClients;
pub struct TestdemoEvents;

impl GuestTestdemoClient for TestdemoClients {
    fn hello_world(&self) -> String {
        return "Hello World".to_string();
    }
}

impl GuestTestdemoEvent for TestdemoEvents {
    fn parse(&self, input: String) -> Result<Foo, String> {
        return Ok(Foo { bar: input });
    }
}

Generating this js code via jco with mostly default settings.
jco transpile ./dist/wasm/testdemo_component.wasm -o ./src/generated-js-bindings --no-namespaced-exports

Generated js code hidden for length

const base64Compile = (str) =>
  WebAssembly.compile(
    typeof Buffer !== "undefined"
      ? Buffer.from(str, "base64")
      : Uint8Array.from(atob(str), (b) => b.charCodeAt(0))
  );

class ComponentError extends Error {
  constructor(value) {
    const enumerable = typeof value !== "string";
    super(enumerable ? `${String(value)} (see error.payload)` : value);
    Object.defineProperty(this, "payload", { value, enumerable });
  }
}

let dv = new DataView(new ArrayBuffer());
const dataView = (mem) =>
  dv.buffer === mem.buffer ? dv : (dv = new DataView(mem.buffer));

const isNode =
  typeof process !== "undefined" && process.versions && process.versions.node;
let _fs;
async function fetchCompile(url) {
  if (isNode) {
    _fs = _fs || (await import("fs/promises"));
    return WebAssembly.compile(await _fs.readFile(url));
  }
  return fetch(url).then(WebAssembly.compileStreaming);
}

const instantiateCore = WebAssembly.instantiate;

const resourceHandleSymbol = Symbol("resource");

const symbolDispose = Symbol.dispose || Symbol.for("dispose");

const utf8Decoder = new TextDecoder();

const utf8Encoder = new TextEncoder();

let utf8EncodedLen = 0;
function utf8Encode(s, realloc, memory) {
  if (typeof s !== "string") throw new TypeError("expected a string");
  if (s.length === 0) {
    utf8EncodedLen = 0;
    return 1;
  }
  let allocLen = 0;
  let ptr = 0;
  let writtenTotal = 0;
  while (s.length > 0) {
    ptr = realloc(ptr, allocLen, 1, (allocLen += s.length * 2));
    const { read, written } = utf8Encoder.encodeInto(
      s,
      new Uint8Array(memory.buffer, ptr + writtenTotal, allocLen - writtenTotal)
    );
    writtenTotal += written;
    s = s.slice(read);
  }
  utf8EncodedLen = writtenTotal;
  return ptr;
}

let exports0;
let exports1;
let exports2;
let memory0;
let postReturn0;
let realloc0;
let postReturn1;

class TestdemoClient {}

TestdemoClient.prototype.helloWorld = function helloWorld() {
  var handle0 = this[resourceHandleSymbol];

  if (handle0 === null) {
    throw new Error('Resource error: "TestdemoClient" lifetime expired.');
  }
  if (handle0 === undefined) {
    throw new Error('Resource error: Not a valid "TestdemoClient" resource.');
  }

  const ret =
    exports1["testdemo:component/clients#[method]testdemo-client.hello-world"](
      handle0
    );
  if (handleTable0.get(handle0)) {
    throw new Error("Resource error: borrows were not dropped");
  }
  var ptr1 = dataView(memory0).getInt32(ret + 0, true);
  var len1 = dataView(memory0).getInt32(ret + 4, true);
  var result1 = utf8Decoder.decode(new Uint8Array(memory0.buffer, ptr1, len1));
  postReturn0(ret);
  return result1;
};

class TestdemoEvent {}

TestdemoEvent.prototype.parse = function parse(arg1) {
  var handle0 = this[resourceHandleSymbol];
  if (handle0 === null) {
    throw new Error('Resource error: "TestdemoEvent" lifetime expired.');
  }
  if (handle0 === undefined) {
    throw new Error('Resource error: Not a valid "TestdemoEvent" resource.');
  }

  var ptr1 = utf8Encode(arg1, realloc0, memory0);
  var len1 = utf8EncodedLen;
  const ret = exports1[
    "testdemo:component/events#[method]testdemo-event.parse"
  ](handle0, ptr1, len1);
  if (handleTable2.get(handle0)) {
    throw new Error("Resource error: borrows were not dropped");
  }
  let variant4;
  switch (dataView(memory0).getUint8(ret + 0, true)) {
    case 0: {
      var ptr2 = dataView(memory0).getInt32(ret + 4, true);
      var len2 = dataView(memory0).getInt32(ret + 8, true);
      var result2 = utf8Decoder.decode(
        new Uint8Array(memory0.buffer, ptr2, len2)
      );
      variant4 = {
        tag: "ok",
        val: {
          bar: result2,
        },
      };
      break;
    }
    case 1: {
      var ptr3 = dataView(memory0).getInt32(ret + 4, true);
      var len3 = dataView(memory0).getInt32(ret + 8, true);
      var result3 = utf8Decoder.decode(
        new Uint8Array(memory0.buffer, ptr3, len3)
      );
      variant4 = {
        tag: "err",
        val: result3,
      };
      break;
    }
    default: {
      throw new TypeError("invalid variant discriminant for expected");
    }
  }
  postReturn1(ret);
  if (variant4.tag === "err") {
    throw new ComponentError(variant4.val);
  }
  return variant4.val;
};
const handleTable0 = new Map();
let handleCnt0 = 0;
const finalizationRegistry0 = new FinalizationRegistry((handle) => {
  const handleEntry = handleTable0.get(handle);
  if (handleEntry) {
    handleTable0.delete(handle);

    if (handleEntry.own) {
      exports0["0"](handleEntry.rep);
    }
  }
});

const handleTable2 = new Map();
let handleCnt2 = 0;
const finalizationRegistry2 = new FinalizationRegistry((handle) => {
  const handleEntry = handleTable2.get(handle);
  if (handleEntry) {
    handleTable2.delete(handle);

    if (handleEntry.own) {
      exports0["1"](handleEntry.rep);
    }
  }
});

const $init = (async () => {
  const module0 = fetchCompile(
    new URL("./testdemo_component.core.wasm", import.meta.url)
  );
  const module1 = base64Compile(
    "AGFzbQEAAAABBQFgAX8AAwMCAAAEBQFwAQICBxQDATAAAAExAAEIJGltcG9ydHMBAAoVAgkAIABBABEAAAsJACAAQQERAAALAC4JcHJvZHVjZXJzAQxwcm9jZXNzZWQtYnkBDXdpdC1jb21wb25lbnQGMC4xOC4wAI0BBG5hbWUAExJ3aXQtY29tcG9uZW50OnNoaW0BcQIAN2R0b3ItW2V4cG9ydF10ZXN0ZGVtbzpjb21wb25lbnQvY2xpZW50cy10ZXN0ZGVtby1jbGllbnQBNWR0b3ItW2V4cG9ydF10ZXN0ZGVtbzpjb21wb25lbnQvZXZlbnRzLXRlc3RkZW1vLWV2ZW50"
  );
  const module2 = base64Compile(
    "AGFzbQEAAAABBQFgAX8AAhoDAAEwAAAAATEAAAAIJGltcG9ydHMBcAECAgkIAQBBAAsCAAEALglwcm9kdWNlcnMBDHByb2Nlc3NlZC1ieQENd2l0LWNvbXBvbmVudAYwLjE4LjAAHARuYW1lABUUd2l0LWNvbXBvbmVudDpmaXh1cHM"
  );
  ({ exports: exports0 } = await instantiateCore(await module1));
  ({ exports: exports1 } = await instantiateCore(await module0));
  ({ exports: exports2 } = await instantiateCore(await module2, {
    "": {
      $imports: exports0.$imports,
      0: exports1["testdemo:component/clients#[dtor]testdemo-client"],
      1: exports1["testdemo:component/clients#[dtor]testdemo-client"],
    },
  }));
  memory0 = exports1.memory;
  postReturn0 =
    exports1[
      "cabi_post_testdemo:component/clients#[method]testdemo-client.hello-world"
    ];
  realloc0 = exports1.cabi_realloc;
  postReturn1 =
    exports1[
      "cabi_post_testdemo:component/events#[method]testdemo-event.parse"
    ];
})();

await $init;
const clients = {
  TestdemoClient: TestdemoClient,
};
const events = {
  TestdemoEvent: TestdemoEvent,
};

export { clients, events };

Trying to run either of the code snippets below results in an error:
const testClient = new clients.TestdemoClient();
const output = testClient.helloWorld()

//or

const testEvent = new events.TestdemoEvent();
const parsedEvent = testEvent.parse("ATestString");

Throws:
Error: Resource error: Not a valid "TestdemoClient" resource.
 ❯ TestdemoClient.helloWorld src/generated-js-bindings/testdemo_component.js:83:11
     81|   }
     82|   if (handle0 === undefined) {
     83|     throw new Error('Resource error: Not a valid "TestdemoClient" resource.');
       |           ^
     84|   }
     85|

//and

Error: Resource error: Not a valid "TestdemoEvent" resource.
 ❯ TestdemoEvent.parse src/generated-js-bindings/testdemo_component.js:108:11
    106|   }
    107|   if (handle0 === undefined) {
    108|     throw new Error('Resource error: Not a valid "TestdemoEvent" resource.');
       |           ^
    109|   }
    110|

And looking through the generated js that makes sense because var handle0 = this[resourceHandleSymbol]; and I am not seeing anywhere that this[resourceHandleSymbol] is actually being set anywhere, so handle0 would be expected to be undefined.
I'll try to get up a repro of this on my github, but at the moment the code for it is tied up with some other things

Thanks for the quick reply, that workaround worked perfectly! The generated code now creates a constructor for the resource classes that properly sets this[resourceHandleSymbol] (see below). Seems like at least this chunk of the constructor should be generated whether the constructor exists or not.
class TestdemoClient{
  constructor() {
    const ret = exports1['testdemo:component/clients#[constructor]testdemo-client']();
    var handle1 = ret;
    var rsc0 = new.target === TestdemoClient ? this : Object.create(TestdemoClient.prototype);
    var rep2 = handleTable0.get(handle1).rep;
    Object.defineProperty(rsc0, resourceHandleSymbol, { writable: true, value: rep2});
    finalizationRegistry0.register(rsc0, handle1, rsc0);
    Object.defineProperty(rsc0, symbolDispose, { writable: true, value: function () {} });

    handleTable0.delete(handle1);
    return rsc0;
  }
}

Looking into this further, the bug is not that this doesn't work, but that it kind does.
That is - resources need to be defined as constructible within the component model itself - if there is no constructor then there is no construction!
So the bug fix here is actually to make the JS constructor error in the case where there is no constructor for a resource.
I will add a PR to this effect soon.

That is - resources need to be defined as constructible within the component model itself - if there is no constructor then there is no construction!

I'm not quite sure I am getting that from the wit docs I have been looking at. Specifically the section on resource types says:

Lastly, a resource statement can contain at most one constructor function, which is syntactic sugar for a function returning a handle of the containing resource type.

If a resource must have a constructor I would expect that to say something like "a resource statement must contain exactly one constructor function".
So if thats definitely correct I'll create an issue there requesting clarification in the wit docs. Additionally I would expect the Rust wit_bindgen::generate! macro to error on resources with no constructor method so I'll create an issue in that repo too.
Slight aside, is there a Zulip or anything for following jco progress? I don't see one mentioned in the docs, but would be interested in keeping up with things!

Resources don't need to have constructors defined, if the component that defines the resource will always itself be creating the resource instances and returning those handles out to other users.
Also WASI host resources don't have constructors for the same reason - the host is the only one who can create those objects to begin with.
But yes, if you want a component to define a resource, where that resource can ever be constructed outside of the component that defines it, then there must be a constructor.
Added a Zulip link to the readme, thanks for the bump on that.


New bidder adapter: pubgenius

Type of change

[x] New bidder adapter

Description of change
Add a new bidder adapter for pubGENIUS

test parameters for validating bids

{
  bidder: 'pubgenius',
  params: {
    adUnitId: '1000',
    test: true
  }
}

Tested using the Hello World sample page.

maintainer<EMAIL_ADDRESS>[x] official adapter submission

For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:

https://github.com/pubgenius/prebid.github.io/pull/1

Test passed. Close this to create a new PR for prebid.js team


Updating ML extension version to 0.11.0

This PR fixes #

Pull Request Test Coverage Report for Build 987667145

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 42.753%

Totals

Change from base Build 987295558:
-0.001%

Covered Lines:
26167

Relevant Lines:
55573

💛  - Coveralls


Subject headings browse

Explore implementation of a subject heading browse:
Utilizing top level LCCN categories, enable browse on a separate page and explore possibilities for integrating browse across hierarchy on both search and separate browse pages.

This may also cover hierarchy display or navigation around subject facets

I set up a separate subject heading index (BAM) that relied on a query against Dave's backup SPARQL endpoint for LC triples.  The query retrieved all LC Subject Headings that also had a LCCN.  There are 89068 entries in this index, where each entry has the title of the subject heading, subject heading URI,full classification numbers/codes and classification facet that includes the first and first two letters of the classifications for that subject heading.


Issue in IE9 with custom filter and dropdown.

Hi All,
I have an issue with custom angular filter and select element.
I created simplified version of my application in jsfiddle: http://jsfiddle.net/98cnbv1x/6/. Tested with angular 1.3.9.
Here is what it does:
there is a notion of items and hosts, items can be in two states: "Not started" and "In progress".
To bring item to "In progress" item needs to use one of the hosts. If the host is already in use it should not be available for items that are still in "Not started".
I update available hosts after each transition of Items between states.
Steps to reproduce the issue:

use IE9;
for ThirsItem select Host3 from dropdown;
press "To In Progress" button on ThirdItem, item should be moved to InProgress state;
expand dropdown of SecondItem (don't touch FirstItem), it contains 2 hosts now, because 1 is in use;
press "To Not Started" on ThirdItem.

expected behavior: all dropdowns should contain all 3 hosts;
bug: SecondItem has only 2 hosts, because it was expanded;
Cannot reproduce this issue in Chrome and Mozilla.

Something must be going down if I'm still seeing three items in the first select by step 4.

@aabenoja I can still reproduce it with 1.3.9.
@VasylVuy I've just tried the latest version from master and it seems to be working fine: http://jsfiddle.net/8j5er1yp/. Could you please confirm?

Hosts do not filter when you move items to "In Progress" in your version http://jsfiddle.net/8j5er1yp/

Actually you fixed that bug with this version: http://jsfiddle.net/8j5er1yp/
But introduced new:
after steps of the previous bug just move ThirdItem to "In Progress" again and it will contain all three hosts, but shoul contain only two.
It happens in IE9 only.

I think this is something to do with IE9 not repainting correctly. If you actually the bind the availableHosts property to the template then this doesn't happen... We might be able to find a place where we need to hit some DOM property to force IE9 to do this repaint.

This sounds possibly related to the prior IE bugs reported on this issue #4749, #7692.

Can't repro in 1.4.9 or 1.5.3


Add app:init command

This PR replaces #467, as that one is from a 3rd party repo so I couldn't push to it.
Adds an "app:init" command, which will add viable if generic .platform* files to a repository.
Current status: The command runs and produces files, enough to allow it to be pushed by Git.  The routing.yaml file is functional. The .platform.app.yaml file is not. The services.yaml file has some examples commented out, with instructions and a link to the documentation.

[ ] Allow init from examples, as a shortcut
[x] Routing
[ ] Services/relationships
[ ] Web/locations configuration
[ ] Mounts
[ ] Build flavors
[ ] Disk size(s)
[ ] More complicated things: hooks, dependencies, extensions, crons

@pjcdawkins Thought: Should we be treating each language/version as a target, or should we be treating each stack as a target?  Damien is right that a wizard to build up the full, say, Drupal 8 configuration is going to be way way too complicated.  But, instead of options "php 5.5", "php 5.6", "nodejs" etc. we could offer "Generic PHP", "Drupal 7', "Drupal 8", "Symfony 3", "Generic NodeJs", etc.  Those files would be FAR more consistent, and the customization rather trivial (just relationships, basically).
Some customization is still likely appropriate by the user, but far less than if we left out, say, the entire rules section.  It would also allow us to almost entirely eliminate the "generic $language" example repositories, or else make them simply an artifact of this script.
Does that seem reasonable?


build: switch to Go 1.21 and tidy deps

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind documentation

/kind tests

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area commands

/area pkg

/area events

What this PR does / why we need it:
Avoid importing old transitive dependencies.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
With this PR, unwanted hashicorp packages listed by https://github.com/falcosecurity/evolution/issues/305 will be removed.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Wait idle fix

Description
Added a fix for a crash in the Wait Idle sample: render_context is reset in WaitIdle::prepare and then used some lines below in the call to stats->request_stats, tested in Windows and Android.
Fixes (no issue submitted)
General Checklist:
Please ensure the following points are checked:

[v] My code follows the coding style
[v] I have reviewed file licenses
[v] I have commented any added functions (in line with Doxygen)
[v] I have commented any code that could be hard to understand
[v] My changes do not add any new compiler warnings
[v] My changes do not add any new validation layer errors or warnings
[v] I have used existing framework/helper functions where possible
[v] My changes build and run on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[v] I have tested the sample on at least one compliant Vulkan implementation
[v] If the sample is vendor-specific, I have tagged it appropriately
[v] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[v] Any dependent assets have been merged and published in downstream modules

I actually think the WaitIdle sample is wrong, It should be overriding prepare_render_context() and then this wouldn't be required. I'll take a look.


Other open questions (JOSS review 2)

Why do you use two different licenses (MIT for the code and reference CC BY 4.0 in the paper)? They do not differ much. For consistency I would use one for both.
Do you want to include the contributions page from the docs in the Github page?

Why do you use two different licenses (MIT for the code and reference CC BY 4.0 in the paper)? They do not differ much. For consistency I would use one for both.

So you're suggesting I can use MIT for both. Sure, will see how I can change that.

Do you want to include the contributions page from the docs in the Github page?

If by the Github page, you mean the README, then as far as I believe it is included under ✨ Implemented Methods

So you're suggesting I can use MIT for both. Sure, will see how I can change that.

At least, I don't see a reason why you would want to use two different licenses. I am no expert on this topic.

If by the Github page, you mean the README, then as far as I believe it is included under ✨ Implemented Methods

I must have missed this.

It may be the case that I didn't get to choose a license for the paper as I can't find that option in either the paper markdown or submission page. What confuses me is that MIT License seems to be defined as "a software license"; meanwhile, when I Googling Can CC-BY-4.0 be used for software seems to quickly suggest that the answer is no.
Do you have any thoughts on this? Or would you think that it implies that it's okay for them to be different as these are different types of material? Otherwise, let me know if you have idea of how to change it for the paper.
Thank you.

Due to the JOSS submission
https://github.com/openjournals/joss-reviews/issues/6310

I see. I didn't know they are both solely for different purposes.


Account suspension (2)

This is the second of (probably) three PRs related to account suspension. This one implements tests for the suspension services and performs checks at login time to ensure a user's account is not suspended. The next will implement the admin page for suspending users.

LGTM!


Resetting workflow with Searchable Attributes causes non-deterministic error

Modify searchable attribute sample to not pass in searchable attribute on start
Add an activity between 2 searchable attributes.  https://pastebin.com/AcBkWTDZ
Run the workflow
reset workflow to event id 11

This results in non-deterministic error.
Make sure to run the sample latest client release.

Looks like replay decision is getting corrupted when no searchable attributes are passed in when starting workflow.  For now pass in a dummy Searchable Attribute to workaround this issue.

This is fixed as part of https://github.com/uber-go/cadence-client/pull/889.
Will be released in v0.10.1


Clean Readme File

How would you improve the documentation? Please describe.
In the For Contributers section, there is no need to use git init command. I think just forking and cloning the repo will work.
Additional information
No response

Oh, yeah @kailashchoudhary11. So, do you want to work on this issue?

Yes I would like to work on it with your guidance!

Yes I would like to work on it with your guidance!

Ok @kailashchoudhary11 . Please link this issue to your PR.

HI @SaptarshiSarkar12 Can I work on this?

@c99SRS  I'm already working on it

@kailashchoudhary11 is working on this issue, so, @c99SRS you may modify the changes after his PR is merged or you may open an issue for a feature request or bug etc. and work on that.

@SaptarshiSarkar12 Will it be okay if I simply type the commands in a code block. Cause I don't really know how to make GIF. Maybe you could guide me a bit on it.

@SaptarshiSarkar12 Will it be okay if I simply type the commands in a code block. Cause I don't really know how to make GIF. Maybe you could guide me a bit on it.

You can make terminal gif here -https://www.terminalgif.com/ .

@SaptarshiSarkar12 Thank you! I have made the GIF. Now where should i host it?


tests for public dataset

Description
I added tests for public dataset + a small change in the public dataset when the dataset download fails. We officially reach 100% coverage with this 👯
Fixes issue #184
Implemented changes
Insert a description of the changes implemented in the pull request.

[x] implemented test cases for public dataset
[x] changed error message in public dataset if download fails

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change is or requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Codecov Report
Patch coverage: 100.00% and project coverage change: +2.10 :tada:

Comparison is base (2609840) 97.89% compared to head (3077ecd) 100.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #264      +/-   ##
===========================================
+ Coverage   97.89%   100.00%   +2.10%
===========================================
  Files          24        28       +4
  Lines         997      1080      +83
  Branches      233       248      +15
===========================================
+ Hits          976      1080     +104
+ Misses         19         0      -19
+ Partials        2         0       -2

Impacted Files
Coverage Δ

src/pymovements/datasets/public_dataset.py
100.00% <100.00%> (+40.38%)
:arrow_up:

src/pymovements/__init__.py
100.00% <0.00%> (ø)

src/pymovements/events/idt.py
100.00% <0.00%> (ø)

src/pymovements/events/ivt.py
100.00% <0.00%> (ø)

src/pymovements/exceptions.py
100.00% <0.00%> (ø)

src/pymovements/utils/checks.py
100.00% <0.00%> (ø)

src/pymovements/events/events.py
100.00% <0.00%> (ø)

src/pymovements/events/engbert.py
100.00% <0.00%> (ø)

src/pymovements/datasets/dataset.py
100.00% <0.00%> (ø)

src/pymovements/transforms.py

... and 8 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Explore page compression on dbo.performance_counters

Is your feature request related to a problem? Please describe.
On avg, for 15 days of data, the table dbo.performance_counters is consuming 50 gb of space.
Describe the solution you'd like
Explore page compression on this table as columns like path, object, host_name, counter, and instance are repeated values.
Describe alternatives you've considered
Already exploring another option of Normalizing the schema of dbo.performance_counts.

Page compression has been implemented successfully. Code changes are checkedin into Install-SQLMonitor.


Failed to create directory

Don't know why, but it fails to create the directory everytime. I used the example code from documentation.

Try this https://github.com/shlexware/Rayfield/blob/main/Documentation.md#secure-mode


Forward and rewind not working

Expected behaviour
When i pass the rewind and fastForward options to plyr initialization, i expect to get two buttons for each action:
    this.kplayer = new Plyr( pDivPlayer, {

        invertTime: false,

        // Localisation

        i18n: {

            rewind: 'Rewind 5s',

            fastForward: 'Forward 5s',

        },

    } );

Actual behaviour
But the player is not adding a button for Forward and other for Rewind
Environment

Browser: Chrome
Version:
Operating System: Ubuntu 18.04
Version:

**So what can i do to make those buttons appear to easily forward and rewind 5 seconds?  Something like this but with those buttons integrated with the original ones from plyr

Reading the docs would be a good start. Have a look at the options section 👍

OK thanks, now the buttons are appearing but they are using the default time that is 10 seconds and i want to be 5 seconds. I've changed the values accordingly to the docs but still going forward 10 seconds. Here is my code:
    //initialize plyr

    let player = new Plyr( pDivPlayer, {

    invertTime: false,
    i18n: {

    rewind: 'Rewind 5s',

    fastForward: 'Forward 5s',

    },

    controls: [

    'play-large', // The large play button in the center

    'restart', // Restart playback

    'rewind', // Rewind by the seek time (default 10 seconds)

    'play', // Play/pause playback

    'fast-forward', // Fast forward by the seek time (default 10 seconds)

    'progress', // The progress bar and scrubber for playback and buffering

    'current-time', // The current time of playback

    'mute', // Toggle mute

    'volume', // Volume control

    'captions', // Toggle captions

    'settings', // Settings menu

    'fullscreen', // Toggle fullscreen

    ]

    } );

Ok so read some more of the docs. seekTime Is what you’re looking for. It might be worth reading the docs again?


Feature Request: Use automatic authentication with Google Cloud Functions?

First of all thank you for this wonderful, well-documented tool. It's been a joy to use.
One little issue that I am having when trying to use this in a Google Cloud Function.
When running Google Cloud Functions it seems that the authentication can be taken care of automatically, as it says [https://cloud.google.com/functions/docs/securing/function-identity]:

During function execution, Cloud Functions uses the service account<EMAIL_ADDRESS>as its identity. For instance, when making requests to Google Cloud Platform services using the Google Cloud Client Libraries, Cloud Functions can automatically obtain and use tokens to authorize the services this identity has permissions to use.

So instead of putting the private key etc. in our function environment variables, it would be wonderful if we could just leave out the auth, or pass an option to use the default Cloud Functions auth.
I tried deploying it to see if it would automatically obtain and use the functions, but unfortunately that wouldn't work.
Error: You must initialize some kind of auth before making any requests
    at GoogleSpreadsheet._setAxiosRequestAuth (/srv/node_modules/google-spreadsheet/lib/GoogleSpreadsheet.js:117:13)
    at <anonymous>
    at process._tickDomainCallback (internal/process/next_tick.js:229:7)

That was without any call to doc.useServiceAccountAuth(), so I tried just passing the client_email
doc.useServiceAccount({ client_email<EMAIL_ADDRESS>});

But then I get this error:
Error: No key or keyFile set.
    at GoogleToken.getTokenAsync (/srv/node_modules/gtoken/build/src/index.js:125:19)
    at GoogleToken.getToken (/srv/node_modules/gtoken/build/src/index.js:77:21)
    at JWT.refreshTokenNoCache (/srv/node_modules/google-auth-library/build/src/auth/jwtclient.js:156:36)
    at JWT.refreshToken (/srv/node_modules/google-auth-library/build/src/auth/oauth2client.js:143:25)
    at JWT.authorizeAsync (/srv/node_modules/google-auth-library/build/src/auth/jwtclient.js:139:35)
    at JWT.authorize (/srv/node_modules/google-auth-library/build/src/auth/jwtclient.js:135:25)
    at GoogleSpreadsheet.renewJwtAuth (/srv/node_modules/google-spreadsheet/lib/GoogleSpreadsheet.js:86:26)
    at GoogleSpreadsheet.useServiceAccountAuth (/srv/node_modules/google-spreadsheet/lib/GoogleSpreadsheet.js:81:16)
    at init (/srv/lib/dictionary-maker.js:25:19)
    at Object.<anonymous> (/srv/lib/dictionary-maker.js:32:3)

This would be a great feature to be able to take advantage of the Cloud Functions automatic credential fetching and auth. (See Google docs on this issue)

Hi, I'm stumbling on the same issue…
what is the simplest way to authenticate with the package in a Google Cloud Function? Is it referencing a manually exported key for the given service account in an environment variable?
@adueck, What did you end up doing? Did you find a way to use the automatic authentication with the package in the end?

@juliendorra, I ended up using the environment variables in Google Cloud/Firebase functions. I put a service account email and key in my Firebase functions environment variable (basically the same as Google Cloud functions) and accessed in in the code like this:
await doc.useServiceAccountAuth({
    client_email: functions.config().serviceacct.email,
    private_key: functions.config().serviceacct.key,
});

I had same problem too.
My solution is to use useRawAccessToken with access token which is taken through auth library.
Document for useRawAccessToken
getAccessToken
const { GoogleSpreadsheet } = require('google-spreadsheet');
const { GoogleAuth } = require('google-auth-library');

const doc = new GoogleSpreadsheet('<the sheet ID from the url>');

const auth = new GoogleAuth({
  scopes: 'https://www.googleapis.com/auth/spreadsheets'
});
const token = await auth.getAccessToken();

if (token == null) {
  throw new Error("Failed to get access token")
}

await doc.useRawAccessToken(token);

I had same problem too. My solution is to use useRawAccessToken with access token which is taken through auth library. Document for useRawAccessToken getAccessToken
const { GoogleSpreadsheet } = require('google-spreadsheet');
const { GoogleAuth } = require('google-auth-library');

const doc = new GoogleSpreadsheet('<the sheet ID from the url>');

const auth = new GoogleAuth({
  scopes: 'https://www.googleapis.com/auth/spreadsheets'
});
const token = await auth.getAccessToken();

if (token == null) {
  throw new Error("Failed to get access token")
}

await doc.useRawAccessToken(token);

this is exactly what I was looking for except that I couldn't use default application credential from the firebase login.
you have to use the service account!
https://medium.com/google-cloud/google-oauth-credential-going-deeper-the-hard-way-f403cf3edf9d this medium post has perfect explanation.


No module named PyVista

Thank you so much for the demo example, I tried it and it worked on localhost.
However, when I try to upload it on cloud, I am getting error

Traceback (most recent call last):
File "/home/appuser/venv/lib/python3.7/site-packages/streamlit/scriptrunner/script_runner.py", line 475, in _run_script
exec(code, module.dict)
File "/app/streamlit/m1.py", line 4, in
import pyvista
ModuleNotFoundError: No module named 'pyvista'

Can you please suggest what I am doing wrong?

Hi @bhargav1195, thanks for your message!
Are you using the requirements.txt and packages.txt files in your repo? If not, please do it, or simply clone my repo. Whit that, I think it should work.
Let me know if this solves your problem 👍

HI @napoles-uach ,
Thank you so much for your response. It worked perfectly. Amazing.

i get some error , could you help me
File "/home/appuser/venv/lib/python3.9/site-packages/streamlit/scriptrunner/script_runner.py", line 557, in _run_script
exec(code, module.dict)
File "Klasifikasi_Sawit1.py", line 4, in
import cv2


Fix display of informative messages when estimating heterogeneous effects by cohort

The informative message for the SA estimation is too long and overflows to the right.

Summary: In this issue we fixed a bug when displaying heterogenous effects estimation results.
Thread continues in #160


Cambridge University Press styles

As discussed here: https://github.com/citation-style-language/styles/pull/2807

[x] Create numeric style: done on 13 Jul 2017 https://github.com/citation-style-language/styles/pull/2807
[x] Create author date style: done on 05 Sept 2017 https://github.com/citation-style-language/styles/pull/2934
[ ] Create dependents: emailed publisher on 14.07.2017 for data (followed up with editors again in Jun-2018 & Jul-2022. No answer)

List of journals: https://www.cambridge.org/core/what-we-publish/journals

I've never seen a standardized style for CUP journals, unfortunately: The "Cambridge UP" styles that we have are for their books.


chore: experimenting with a new figma plugin

resolves #2630

Should tsconfig not extend from our root config? Might get different typing behaviours with different settings.
"extends": "../../tsconfig.json"


Remove terraform as an install dependency

Side effect of this PR is that the user of qhub no longer controls the terraform version used. Instead a terraform binary is installed automatically with any terraform commands.
Closes https://github.com/Quansight/qhub-cloud/issues/289

You have removed config files from the asset, I don't recall where were they used?

Yeah they are no longer used or reffered to. qhub init made them not needed as they were examples of how to configure qhub. documentation also addressses the purpose of these files.

The curl backoff worked and has made the tests for resilient https://github.com/Quansight/qhub-cloud/pull/417/checks?check_run_id=2179731674#step:18:1.

Ready for merge

@aktech or @tylerpotts pinging for review


[enh] Add autoIncrement() in Migrations to allow fine grained data type in PK

What steps will reproduce the problem?
This stackoverflow question demonstrates the exact problem.
In a nutshell: primaryKey() is the only way to set an auto increment field that is db agnostic but it is ever an int data type. Sometimes whe need it to be either smallint or bigint. Having a autoIncrement() would make it possible (AUTO_INCREMENT for MySQL, IDENTITY(1,1) for MSSQL, and so on);

I just saw that there is already a bigPrimaryKey(). Just lacking a smallPrimaryKey().

I'm OK adding it.

I am working on it. :+1:

The syntax you mentioned is pretty good really. I just did not see how would it cover primary keys. Would it accept this?
$this->createTable('tableName', [
    'id' => $this->smallInt()->primaryKey(), // would it be this or something else?
    'name' => $this->string()->index('indexName'),
    'topic_id' => $this->integer()->foreignKey('indexName', 'referencedTable', 'referencedColumn'),
]);

Thanx.

Sorry, I just saw #11638 right now. I had not perceived that the issue and the comment were in separated issues. Thnx.


Fix State transition err when hdfs ccm feature is used

When we used dynamometer to test HDFS performance, the test encountered a error when generate DataNode Block info. The error stack is
Error: java.io.IOException: State transition not allowed; from DEFAULT to FILE_WITH_REPLICATION         at com.linkedin.dynamometer.blockgenerator.XMLParser.transitionTo(XMLParser.java:107)         at com.linkedin.dynamometer.blockgenerator.XMLParser.parseLine(XMLParser.java:77)         at com.linkedin.dynamometer.blockgenerator.XMLParserMapper.map(XMLParserMapper.java:53)         at com.linkedin.dynamometer.blockgenerator.XMLParserMapper.map(XMLParserMapper.java:26)         at org.apache.hadoop.mapreduce.Mapper.run(Mapper.java:151)         at org.apache.hadoop.mapred.MapTask.runNewMapper(MapTask.java:828)         at org.apache.hadoop.mapred.MapTask.run(MapTask.java:341)         at org.apache.hadoop.mapred.YarnChild$2.run(YarnChild.java:174)         at java.security.AccessController.doPrivileged(Native Method)         at javax.security.auth.Subject.doAs(Subject.java:415)         at org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1690)         at org.apache.hadoop.mapred.YarnChild.main(YarnChild.java:168)
After checking Fsimage xml and the source code, we find that XMLParser can not parse the lines correctly, these lines are like
<directive><id>8963</id><path>/user/somepath/path1</path><replication>3</replication><pool>cache_other_pool</pool><expiration><millis>1544454142310</millis><relatilve>false</relatilve></expiration> <directive><id>8964</id><path>/user/somepath/path2</path><replication>3</replication><pool>cache_hadoop-data_pool</pool><expiration><millis>1544497817686</millis><relatilve>false</relatilve></expiration> <directive><id>8965</id><path>/user/somepath/path3</path><replication>3</replication><pool>cache_hadoop-peisong_pool</pool><expiration><millis>1544451500312</millis><relatilve>false</relatilve></expiration> <directive><id>8967</id><path>/user/somepath/path4</path><replication>3</replication><pool>cache_other_pool</pool><expiration><millis>1544497602570</millis><relatilve>false</relatilve></expiration>
These fsimage xml lines are generated when HDFS Centralized Cache Management (CCM) feature is used.
So we add a patch to ignore the CCM fsimage xml lines when parsing xml.

@xkrogen hi Erik,  could you please have a look?

Hi @yuxintan , thanks for reporting this. We don't use the caching feature so we haven't encountered this issue.
I wonder if there's a better way to handle this by adding support for the CCM feature to Dynamometer? I'm not sure how much work this would be, since I don't know much about how it works.
For this patch in particular, I'm fine with excluding the directives, but not too sure about the current logic. First off, is there really no </directive> closing the XML tag? If not, this is a bug with the XML generator -- the XML you've pasted is broken. I took a look at the code, and at least on branch-2.7, I see code that should be generating the closing tag.
Second, is it guaranteed that a line containing <directive> will never also contain a real file tag, and that the directive line won't be split across multiple lines? I want to ensure that we don't accidentally skip files, and that we don't accidentally only skip part of the directive. If possible, I would rather skip the entire <CacheManagerSection>...</CacheManagerSection> to avoid any such issues and make it very clear what is being skipped.

@xkrogen Thanks very much for reviewing the patch.
CCM feature improve HDFS performance by storing some specific files' blocks in DataNode's memory, NameNode only need save these paths in its memory, there is little or no effect on NN. So we don't add support for the CCM feature to Dynamometer.
For the XML tag issue, it's the lack of clarity in my description led to confusion. There is </directive> closing the XML tag after <directive><id> line. The CCM XML lines are exactly like
<directive><id>8963</id><path>/user/somepath/path1</path><replication>3</replication><pool>cache_other_pool</pool><expiration><millis>1544454142310</millis><relatilve>false</relatilve></expiration>
</directive>
<directive><id>8964</id><path>/user/somepath/path2</path><replication>3</replication><pool>cache_hadoop-data_pool</pool><expiration><millis>1544497817686</millis><relatilve>false</relatilve></expiration>
</directive>
<directive><id>8965</id><path>/user/somepath/path3</path><replication>3</replication><pool>cache_hadoop-peisong_pool</pool><expiration><millis>1544451500312</millis><relatilve>false</relatilve></expiration>
</directive>
<directive><id>8967</id><path>/user/somepath/path4</path><replication>3</replication><pool>cache_other_pool</pool><expiration><millis>1544497602570</millis><relatilve>false</relatilve></expiration>
</directive>

and the directive line won't be split across multiple lines, which are similar to other XML <inode> lines.
The patch only ignores these lines and bypass the problem, but I'm quite of your opinion that it's more elegant to avoid any such issues by skipping the entire <CacheManagerSection>...</CacheManagerSection>.
Thank you very much for your attention and kindly advice.

Got it, thanks for the detailed description.
I'm re-visiting this and feel like the right way to solve this is actually to add a new {{State}}, {{INODE_SECTION}}. When the {{}} tag is seen, transition into {{INODE_SECTION}}, when {{}} is seen, transition out. Then, the rest of {{parseLine()}} can simply ignore text whenever the state is {{DEFAULT}}. Then any {{}} tag outside of the INode section is completely ignored.
Let me know what you think.

Hey @yuxintan , I put up a PR with my described idea at PR #82, can you check and see if this solves your issue?


K8SPXC-264 Operator version update

Test name
Status

recreate
passed

init-deploy
passed

storage
passed

scaling
failed

big-data
passed

scaling-proxysql
passed

self-healing
passed

limits
passed

upgrade
passed

restore-to-encrypted-cluster
passed

monitoring
passed

monitoring-2-0
passed

self-healing-advanced
passed

upgrade-consistency
passed

security-context
passed

affinity
passed

demand-backup
passed

operator-self-healing
passed

one-pod
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

scheduled-backup
passed

demand-backup-encrypted-with-tls
passed


Reject Notification Alert Text

When you reject the notification permission the alert text isnt visible, however it is on bluetooth so this is weird.
https://imgur.com/a/QuTkP3j

Trivial fix, somehow the text for the alert's action had been set to "". Pushed straight to dev.


Fix csv content-type error message

Fixes #1805
I was not sure if the application/csv here should also be changed? I'm thinking yes, but applicaiton/csv is a bad type.

bors merge


Running Cargo purely as a build tool

I'm trying to integrate Cargo into my company's internal build system. We already do dependency management via this system, and vendor packages by importing them individually and modeling their dependencies via our own manifest system. This has worked for a variety of languages with wrapper scripts to configure their build tool, but cargo's dependence on a particular registry-as-git-repo is making
Vendoring all of a packages dependencies in a package defeats the purpose of our internal build system, and leads to repo bloat. We could in theory build a fake registry per build, but that feels like overkill for this problem. We could require Cargo.toml's be rewritten so they define a conventional path (or rewrite them at the build stage to do that), but that's a stopgap measure and likely a brittle one. We also noticed that doing path overrides doesn't take version into account, and while we'd be modeling version dependencies in our dependency manager that doesn't mean we want to stop Cargo from doing it as well.
There've also been several issues from people wanting an offline mode for other reasons, so it's not like its just us. Is there any way/any plan to decouple the dependency resolution stage of cargo from build stage?

I think the best solution here would indeed be to rewrite Cargo.toml files to use path dependencies instead of registry dependencies. If you're already vendoring everything then you've got it all stored locally for this, and if there are no dependencies on the remote registry then Cargo won't attempt to talk to it.

I think you can now do this using a repo-local .cargo/config file to override the paths to each dependency [1], and [http] timeout = 0
[1] http://doc.crates.io/guide.html#overriding-dependencies
[2] http://doc.crates.io/config.html#configuration-keys

So, at the moment, if all crates are specified in Cargo.toml via a path, cargo doesn't attempt to fetch the registry.
However, if we override the paths through the use of a .cargo/config, it still attempts to fetch the registry, even if all of the dependencies are satisfied.
@alexcrichton Would it be possible to check the config paths before we decide if we need to go online?

@valenting unfortunately path overrides are implemented in such a way that working like that is impossible. This is what [replace] is intended for, however, to fix.

@alexcrichton I fiddled with the cargo source code a bit, and I managed to make this work.
This way, the paths are overrided before we actually update the registry, so it doesn't attempt to go online. I don't know the code well enough to know if is correct to do this, but it fixes my use case.
diff --git a/src/cargo/ops/cargo_compile.rs b/src/cargo/ops/cargo_compile.rs
index 12356ac..8117754 100644
--- a/src/cargo/ops/cargo_compile.rs
+++ b/src/cargo/ops/cargo_compile.rs
@@ -111,6 +111,8 @@ pub fn resolve_dependencies<'a>(root_package: &Package,
         registry.add_preloaded(root_package.package_id().source_id(), source);
     }

+    try!(add_overrides(&mut registry, root_package.root(), config));
+
     // First, resolve the root_package's *listed* dependencies, as well as
     // downloading and updating all remotes and such.
     let resolve = try!(ops::resolve_pkg(&mut registry, root_package, config));
@@ -120,8 +122,6 @@ pub fn resolve_dependencies<'a>(root_package: &Package,
     // overrides, etc.
     let _p = profile::start("resolving w/ overrides...");

-    try!(add_overrides(&mut registry, root_package.root(), config));
-
     let method = Method::Required{
         dev_deps: true, // TODO: remove this option?
         features: &features,

Actually, this doesn't seem to be related to my change :) It works if I build cargo master, but not with the latest rust-nightly.
Did something land in the past 1-2 days that might have fixed it?

@valenting unfortunately that actually changes the Cargo.lock that's emitted, whereas path overrides are intended to not change the Cargo.lock.
I'm not personally aware of anything that changed behavior recently, but path overrides do have some level of nondeterminism unfortunately which may not play nicely with everything else.

Yeah, my bad, I wasn't installing the binary properly.
Indeed, this does seem to be changing Cargo.lock.
Any idea on how and when the [replace] way is supposed to work? (it doesn't seem to work at the moment).

Ah [replace] may be beta+ for now (I forget if it's landed in stable), but it basically does what you did in the sense of it changes the lock file.

I believe the main request in this issue is addressed by https://github.com/alexcrichton/cargo-vendor and https://github.com/alexcrichton/cargo-local-registry or by the grander build system integration plan, so I'm going to close this in favor of those.


abbreviations not working

The other items in the examples work, but i can't seem to get keyboard.add_abbreviation('@@'<EMAIL_ADDRESS>to work.

Finally getting back to this.
@@ does not work. I tried ii and got an error message. Digging deeper.

The _os_keybord is missing a required method: type_unicode().

Also, the "backspace" key was not defined. Added it as an alias for "delete".

@@ does not work, but ii, II, and 22 all work.

There are a couple things going on here.
I just tested the latest version of keyboard on Windows and confirmed that it suffers the same problem: the word listener depends upon upper() to capitalize lowercase characters, which works for ii and II but not 22 and @@.
There was also an issue with the Mac fork where modifier keys (shift, alt, etc.) were not being detected by the key listener, which caused the word listener to ignore capitalized words. So while ii and 22 were working, II was actually not working on Mac.
I've fixed the issue with the key listener, so shift keys will now be correctly reported. I also added "right command" to the key list.

Submitted an issue on the main branch for the word listener problem, as it's not specific to the Mac port.


Release Google.Cloud.ApigeeConnect.V1 version 2.1.0

Changes in this release:
New features

Enable REST transport in C# (commit 496c8ab)

Created release for Google.Cloud.ApigeeConnect.V1-2.1.0


iPad: Last editor icon is misaligned

Last editor icon is misaligned

Is this fixed ?

Seems to be fixed !
Client version	v1.2.114
API version	v0.4.41
IMG version	v0.1.10


feature: BERTE-548 Split conflict resolution steps

Hello miniscruff,
My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.
Status report is not available.

Incorrect fix version
The Fix Version/s in issue BERTE-548 contains:

None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

3.6.2

Please check the Fix Version/s of BERTE-548, or the target
branch of this pull request.

/approve

Incorrect fix version
The Fix Version/s in issue BERTE-548 contains:

None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

3.6.2

Please check the Fix Version/s of BERTE-548, or the target
branch of this pull request.
The following options are set: approve

Queue out of order
The changeset has received all authorizations to enter the merge queue,
however the merge queue is currently in an incoherent state. In order
to protect this pull-request, I have not added the changeset to the
queue.
Please contact a member of release engineering.
The following options are set: approve

In the queue
The changeset has received all authorizations and has been added to the
relevant queue(s). The queue(s) will be merged in the target development
branch(es) as soon as builds have passed.
The changeset will be merged in:

:heavy_check_mark: development/3.6

The following branches will NOT be impacted:

development/1.0
development/2.0
development/3.0
development/3.1
development/3.2
development/3.3
development/3.4
development/3.5

There is no action required on your side. You will be notified here once
the changeset has been merged. In the unlikely event that the changeset
fails permanently on the queue, a member of the admin team will
contact you to help resolve the matter.
IMPORTANT
Please do not attempt to modify this pull request.

Any commit you add on the source branch will trigger a new cycle after the
current queue is merged.
Any commit you add on one of the integration branches will be lost.

If you need this pull request to be removed from the queue, please contact a
member of the admin team now.
The following options are set: approve

I have successfully merged the changeset of this pull request
into targetted development branches:

:heavy_check_mark: development/3.6

The following branches have NOT changed:

development/1.0
development/2.0
development/3.0
development/3.1
development/3.2
development/3.3
development/3.4
development/3.5

Please check the status of the associated issue BERTE-548.
Goodbye miniscruff.


[Bug] Nested dropdowns don't work on mobile

Bug report
What I did
I am 100% sure that this was working. Somehow it's not working anymore 🤡
In fact when I opened the issue https://github.com/Laravel-Backpack/demo/issues/574#issuecomment-1717406610 to test, it was working locally and failing in our online demo like @karandatwani92 pointed out. After some updates, it's gone.
At this moment it's not working anywhere, so I suspect we may have broken it in the meanwhile, but I've spend way more time than I would like here and still not able to find the root cause for it to fail now, so I am writing this down, maybe I have an ha-ha moment.
It works just fine on CoreUI themes (v2 and v4).
In tabler the mobile menu only open the first dropdown, the nested ones when clicked just close the parent without opening the child.
If anyone has a bright idea here, fine, otherwise I will revisit this later this week.

https://github.com/Laravel-Backpack/theme-tabler/blob/main/resources/assets/js/tabler.js#L15
Probably related to bs-auto-close being enabled by default, since we're matching on href that would mean there are multiple active dropdowns which wouldn't work with auto-close enabled? So I'd think it's not necessarily related to mobile, but inherently so because of duplicate href.

Hey @StefanPrintezis thanks for the feedback. 🙏
What's the purpose of using the same routes multiple times? I am genuinely asking, because I usually either create operations for specific scenarios, like user/1/activate, or users/blocked (instead of adding users?blocked=true to activate some filter).
Let me know if you think we should handle that scenario in a different way, or if there are use cases that I am missing.
Cheers

The mobile menu would be in your  and the desktop menu would be in your , both having the same route

Thanks @StefanPrintezis for the clarification. I will investigate it and get back to you 🙏

This should be fixed in 1.2.1 released earlier today.
Sorry it took a little bit to get to fix this, hope everything is right now.
If not, please re-open 🙏


isDebugEnabled() returning false when debug enabled through env var

I have several sections in my debug wrapped in the logger.isDebugEnabled()
What I noticed is that setting the log level to debug through the env var LOG_LEVEL causes these log lines not to appear. This seems like a bug to me.
Expected Behavior

isDebugEnabled should return true.
Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Configure logging via config to be INFO
Override to DEBUG via LOG_LEVEL env var

Environment

Powertools version used:
Packaging format (Layers, Maven/Gradle):
AWS Lambda function runtime:
Debugging logs

How to enable debug mode**

# paste logs here

Thanks @asmith-r7 For reporting the issue. I did some digging and Can see that the LOG_LEVEL env variable gets overridden by system property back to INFO.
Which means that either we will have to find another name for our environment variable to be something like POWERTOOLS_LOG_LEVEL instead.
or we will have to support it via java tool options via
JAVA_TOOL_OPTIONS: -DLOG_LEVEL=INFO.
In terms of user experience, I believe having a different environment variable names is better than the later.

Something like this https://github.com/awslabs/aws-lambda-powertools-java/compare/log-level?expand=1

Available in 1.3.0 now


Modularization

Modularization of the different functions called by the switch case results in all cases I have found so far in code-breaking bugs. While I would like to modularize at this point in time, it is not feasible with the given timeframe, I will look into this more later.

Workaround found. If the import is inside of a function node does not treat it as a circular import and the main menu can be reached after completing each function.


Fixes issue with table rendering on mobile

Fixed issue with columns overlapping without scrollbars.
Fixes #42

Tested with both viewports(mobile and desktop) from Chrome DevTools. Keeps bootstraps responsive features and is scrollable horizontally on mobile with no column overlapping.

Updated pull request to re-add table-responsive class to district_needs page table that was removed as part of PR #55 which aimed to close Bug #67 by removing the class.
The PR fixes both #67 and #42 by removing manual overwrite of table-layout in style.css .

@nevinvalsaraj The #42 seems to be fixed. Can you close this?
@biswaz @vigneshhari

cc @IEEEKeralaSection/code-reviewers

cc @IEEEKeralaSection/moderators

Seems this is the recommended way to add responsive tables in Bootstrap 3.
http://bootstrapdocs.com/v3.0.3/docs/css/#tables-responsive


inference accuracy is 0.002 while  training accuracy is 0.96

I am confused when I am training the model  that the training loss dropped really well and training accuracy   also to almost 0.96 , but when I freeze the model to pb file the inference accuracy is only 0.002. I dont know which place I ignored

Hey, I also meet this problem.
Have you solved this?


Closes #7: Add stopgap fix to parse PM timings

Yeah this should do it.
Issue is the date-fns library itself not parsing AM/PM. I guess this is proof that v2 is still in alpha  eh @randName.
I'll update the gh-pages after one of you merges
PS: Also updated the term in the production DB

Whoops just realized that if people upload their timetables right now, the DB gets incorrect schedule info. For this reason I'm just gonna merge the fix right now myself.


Support Kubernetes v1.22

Issue:
Currently, NGINX Service Mesh's charts use a deprecated version of the Mutating Webhook Configuration API.
The admissionregistration.k8s.io/v1beta1 API version of MutatingWebhookConfiguration and ValidatingWebhookConfiguration is no longer served as of v1.22.
Proposed Solution:
Update use of this API to admissionregistration.k8s.io/v1

Closing this, as we have added 1.22 support in 1.3.


Virtualbox boot failure

After running vagrant up, eventually Virtualbox was started, but the VM could not start. I tried running it in Virtualbox manually, and got this error:

I'm running Arch Linux x86_64 fully up-to-date.

Disabling audio in the vm appears to let me boot.


Adaswarm API not workig

As you mentioned optim.Adaswarm() is not working please tell me how to implement this in my code.

Have you taken a look at example.py for an example usage?

is adaswarm is not available in pytorch library ?
and i want to use adadwarm in resnet50 optimiser function for predicting
cat vs dog please help me ?
On Fri, Jan 28, 2022, 9:04 PM Zabil Cheriya Maliackal <
@.***> wrote:

Have you taken a look at example.py
https://github.com/AdaSwarm/AdaSwarm/blob/main/examples/main.py for an
example usage?
—
Reply to this email directly, view it on GitHub
https://github.com/AdaSwarm/AdaSwarm/issues/46#issuecomment-1024333399,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJWY4SEREKCOYBZFEW5FDT3UYKZPJANCNFSM5LYFUW2Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Have you taken a look at example.py for an example usage?

is adaswarm is not available in pytorch library ?
and i want to use adadwarm in resnet50 optimiser function for predicting cat vs dog please help me ?

Have you taken a look at example.py for an example usage?

I'm also finding it difficult to use this example to test AdaSwarm out in a Kaggle challenge. It appears that AdaSwarm is used as a loss function there as opposed to an optimiser, and it's hard to tell where the global variables (such as dataset) are imported from, what they represent and how they are set...

@Pyromonk thanks for the feedback.
At this moment AdaSwarm is a loss function that works with the Adam optimiser. We are still working on adding support for adding custom datasets etc. As of now it only works with MNIST, wine and Iris and these can be set as environment variables at
https://github.com/AdaSwarm/AdaSwarm/blob/7b9228098b623de0ae0c56b9944dc38917462e03/adaswarm/utils/options.py#L69-L74
As you can see Iris is the default.
Supported dataset are at
https://github.com/AdaSwarm/AdaSwarm/blob/7b9228098b623de0ae0c56b9944dc38917462e03/adaswarm/utils/options.py#L69-L74
Out of curiosity can you tell us how you want to use AdaSwarm?


[IMPROVEMENT] faulty documentation of iscsi installer helper

Describe the bug (🐛 if you encounter this issue)
The documentation describes a daemon set based installation of iscsi, but the file references an alpine image that is outdated.
To Reproduce
Steps to reproduce the behavior:

run kubectl apply -f https://raw.githubusercontent.com/longhorn/longhorn/v1.4.0/deploy/prerequisite/longhorn-iscsi-installation.yaml
observe the pods downloading a really old image

Expected behavior
A recent image should be downloaded and the code run.
Log or Support bundle
Environment

Longhorn version: documentation issue
Installation method (e.g. Rancher Catalog App/Helm/Kubectl):
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version:

Number of management node in the cluster:
Number of worker node in the cluster:

Node config

OS type and version:
CPU per node:
Memory per node:
Disk type(e.g. SSD/NVMe):
Network bandwidth between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):
Number of Longhorn volumes in the cluster:

Additional context
Add any other context about the problem here.

@anotherthomas @innobead I think we should probably move all the util / scripts / deployments to use the same suse image we now use for the other components. To make sure that these additional tooling would easily work with the offline/airgap setups.

Using a full SuSE image to run apt sounds slightly excessive ;)
But a single image does make offline setups easier. I'll update the PR. New ticket or do we recycle this one?


Limit of characters

Hello. I have placeholders with big colors code, but the plugin have a limit of characters.

simple:
titles: '&#ee8500&lS&#dd7b00&lu&#cc7000&lr&#bb6600&lv&#aa5b00&li&#995100&lv&#884600&lal'
scores:
13: '&r'
12: '  &6⛏ &fRank: &7%luckperms_prefix%'
11: '  &#DCE35B⛁ &fSaldo: &a%vault_eco_balance_formatted%'
10: ''
9: '  &#D00B65♥ &fStatus&7: &a%marriagemaster_IsMarried%'
8: '  &c⚔ &fClan&7: &7%simpleclans_clan_color_tag%'
7: ''
6: '  &d☆ &fPinhata&7: &5%pinataparty_current_votes%&8/&5%pinataparty_votes_needed%'
1: ''

See the rank on image
https://prnt.sc/_LWlwWq3VkvS

Hey, what version is your server running?

Minecraft Purpur 1.18.1
Last plugin version

By last plugin version I assume you mean 3.10.1 correct?
Do you have ProtocolLib on your server and are you running ViaVersion/ViaBackwards (or anything that adds multiple version support)?

Yes, 3.10.1.
Yes, I have ProtocolLib.
Yes, I have ViaBackwards and ViaVersion 4.1.1

And what version is your client running on?

My client is the native version?
Native version: 1.18.1

Interesting, and are you using the vanilla client or do you have any mods installed (e.g. Optifine, Fabric, Forge, Lunar Client, etc)?

Im using Optifine 1.18.1
Another players see scoreboard with the limit of characters too

Ok, can you send me the output of the variable %luckperms_prefix% so I can test this locally.

%luckperms_prefix% for me >>> &#a3025a[&l&#930264&lD&r&#83026do&#730177n&#630180o&#53018a]&r
Obs.:

Ok

The commit 21a1688 should alleviate this issue. I'll be releasing a new version on spigot probably by the end of the week.
If you are running a server with ViaBackwards or forceMultiVersion enabled your character limit will be the following:
On 1.12 and bellow: 32 characters
On 1.13 and above: 80 characters (players bellow 1.13 will still only be able to see 32 characters)
If you don't have ViaBackwards (or ignoreViaBackwards is enabled) and forceMultiVersion is disabled:
On 1.12 and bellow: 32 characters
On 1.13 and above: 128 characters
Keep in mind each hex color is the same as having 14 characters and can make you reach the maximum amount of characters very quickly.

Hey, after taking another look at this I realized I was applying an unnecessary limit of characters for servers on 1.13 and newer (that is if you have ProtocolLib). Commit faf1723 fixes that, I'll be releasing an update on Spigot tomorrow.
Just to be clear if you don't have ProtocolLib or are on 1.12 and older you will still be limited to the same limits I specified above.


IndexSelect doesnt handle non-contiguous tensors

IndexSelect doesnt handle non-contiguous tensors
eg, turns out, the last tensor in char-rnn.OneHot is non-contiguous.
Whilst I could modify char-rnn to not make the last tensor non-contiguous, it seems better to modify IndexSelect to handle non-contiguous, so I might do that, in cltorch, sometime, maybe, possibly.
(Edit: I mean, the function handles them, but the kernel doesnt; the function therefore creates and destroys a new tensor each time).

Fixed in https://github.com/torch/cutorch/pull/242


i915-perf: set a different default colors to GPU timeline events

By default we're currently using the same color for all GPU timeline
events. But some i915 driver versions don't have tracepoint support to
build correlation back to the process that submitted the work, so that
we can use the process' color.
So just assign a default color so that we can at least tell items
apart on the timeline.

Thanks Lionel!

Thanks Lionel!

Thanks to you :)

Thanks to you :)


feat(products): add API to add products

Related Issue
Closes: #728
Describe the changes you've made ✍
I have created an API to add products into the database , for this i have created a productSchema and Products routes
Screenshots (if any) 📸

so now i have to make another pr after these changes?

so now i have to make another pr after these changes?

Why would you make another PR ?
You just go to your code, check out the branch you made Nitish-hack  and work, and just commit and push them. Those will be auto-added to this PR.

okay okay great

Is this fine now  @IAmTamal

I've made all the requested changes , now please merge it if its correct

thanks


Error "Did you mean?  exist?"

3.2.0 :002 > CS.countries
/home/valery/.local/share/gem/ruby/3.2.0/gems/city-state-0.1.0/lib/city-state.rb:286:in `countries': undefined
method `exists?' for File:Class (NoMethodError)

    if ! File.exists? COUNTRIES_FN
             ^^^^^^^^
Did you mean?  exist?

Yes, I'm facing the same issue, an official release would be nice.

Fixed on 1.1.0.


changed error box messages on page one and page two. changed folder n…

…aming structures

error box was working in earlier iterations but stopped working as things were changed. You fixed that and we had a deployment issue with case sensitive naming of our files so you made changes to those, great job


feat: track best fight duration in kill count notifier

Allow kill count notification on personal best time (even if interval was not met)
Utilize separate message template for best time notification
Add latest fight duration to extra json data

Crash is resolved, general functionality of the notifier still works as expected, will think about what approach to take to test the actual personalBest changes.
{
   "type":"KILL_COUNT",
   "extra":{
      "boss":"Vorkath",
      "count":1375,
      "gameMessage":"Your Vorkath kill count is: 1375.",
      "time":"PT2M22S",
      "isPersonalBest":false
   },
   "content":"felanbird1 has defeated Vorkath with a completion count of 1375",
   "playerName":"felanbird1",
   "embeds":[

   ]
}

If easily done I'd prefer the milliseconds be cut

I agree the duration formatting could be nicer. As for the milliseconds, can you try testing with:

It claims tenth of a second, but I recall seeing examples with hundredths (but maybe they were all 0's in the hundredths place)... also we could skip milliseconds altogether if the precise varbit is not enabled

I think the millisecond option is fairly common, I see the .60's/.40's fairly often on streams

I think the millisecond option is fairly common, I see the .60's/.40's fairly often on streams

Yeah he was correct, toggling that setting gives me .xx kill times on Alch hydra

It's only the nearest tenth of a second, so maybe we can remove the 3rd decimal

I'm happy with these changes, good to merge


Allocate static IP addresses from the last /24 of cluster subnet

What this PR does / why we need it:
When VNET integration is enabled (pods are assigned private IP addresses from VNET address space), static (for masters) and dynamic (for agents and pods) addresses are being allocated from the same subnet. This results in race conditions where a dynamic address allocation can execute before a static address allocation for the same IP address. In that case, the static allocation will fail and result in deployment failures.
This PR fixes

When the cluster subnet prefix is /17 or longer, getFirstConsecutiveStaticIPAddress returns IP addresses from the same /24 as the pods. Update that function to return from the very last /24 subnet, clearly separating static addresses from dynamic addresses.
Update validation code to fail if the cluster subnet is too small (less than 9 bits for host IDs). The last 8 bits are used for dynamic addresses for all nodes and pods. The 9th bit is set for static addresses.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged):
fixes #882
Special notes for your reviewer:
Release note:
NONE

This change is

@acs-bot test this please

@acs-bot test this please


[operator] enable bashrc before container start

Why are these changes needed?
The ray cluster will not load the .bashrc before the container start. The reason is the ray process starting command is like this:
    Command:
      /bin/bash
      -c
      --
    Args:
      ulimit -n 65536; ray start --head  --system-config='{"gcs_rpc_server_reconnect_timeout_s": 20}'  --block  --num-cpus=4  --memory=8000000000  --dashboard-host=<IP_ADDRESS>  --node-ip-address=$MY_POD_IP  --port=6379  --metrics-export-port=8080

/bin/bash -c means non-Interactive mode to run bash.
reference to https://www.gnu.org/software/bash/manual/bash.html#Bash-Startup-Files
non-Interactive mode would not load /etc/profile, .bashrc, ...
we can use login mode to get similar effect with the interactive mode

Related issue number
https://github.com/ray-project/kuberay/issues/409
Checks

[*] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[*] Manual tests
[ ] This PR is not tested :(

can you please elaborate more on what you mean by "iterative" in the title?  your comment says "interactive"
   -i     If the -i option is present, the shell is interactive.
   -l     Make bash act as if it had been invoked as a login shell
          (see INVOCATION below).

https://www.man7.org/linux/man-pages/man1/bash.1.html

@akanso sorry for the unclear expression. I have updated the comment.
I want to use the login shell(-l) to enable interactive login shell, which would load .bashrc before process start

@akanso does the change looks good to you?

looks good to me.


Make it possible to work by having a global installation

Behavior right now:

app-root is returning the global node modules path, where it should look for where it's being run from

This is now available for testing 🎉
if anyone's interested in testing it, please run the following and let me know if it works for you or not:
Install it globally
npm install -g webdash@next
#run webdash serve in your project:
webdash serve

Install it locally
npm install --save-dev webdash@next
npx webdash serve

@jadjoubran What about the plugins, do we have to install them locally in our project?

the config file should always be in the project.. I don't think it'll ever be possible to move it outside as all the values will change from 1 project to another (for example manifestPath will be different from project to another)
For the plugins, for now yes they have to be installed locally..
does it make sense to allow them to work globally?
Especially that they differ from project to project & if we look at other tools (like grunt & webpack) I think they require you to have the plugins installed locally

Makes sense! 😄

it seems this is broken because is-global is returning true when webdash is installed with npm install webdash --save-dev which is breaking the paths
Will be available in 1.4 instead


Turn off BooleanOperatorAnalyzer by default

Currently the BooleanOperatorAnalyzer wants to enforce everyone to use sign style operators by default.
Thats just one style that fits some companies but not others.
After a meeting with Claes, we came to the conclusion that all analyzers about code style (natls.style.* settings) should be off by default.
So the new possible settings should be:
natls.style. comparisons=false # default
natls.style. comparisons=short
natls.style. comparisons=sign

@MarkusAmshove - working on this.

I misread the working on :)


[containerapp] az containerapp compose create: Fix bug where environment's resource group is not resolved from --environment when the input value is a resource id

Related command

az containerapp compose create
Description

az containerapp compose create: Fix bug where environment's resource group is not resolved from --environment when the input value is a resource id.

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

containerapp

Hi @wangzelin007 @yonzhan
Could you help to review this PR when you have chance? Thanks!


Add the possibility to validate only the needed files

fixes #135

@YHuard can you test this PR?

I've run some tests and it works like a charm, thanks! :+1:


Fix rsx rosetta element and attribute mapping

This PR changes rsx-rosetta to use the mapping of html->rsx names for elements and attributes defined in dioxus-html instead of converting the case of the attributes and elements.
This fixes the following cases:

translating h1-h6

old:
h_1 {}

new:
h1 {}

translating web components

old:
my_component {}

new:
my-component {}

translating unsupported attributes

old:
div { unrecognizedattribute: "asd", "hello world" }

new:
div { "unrecognizedattribute": "asd", "hello world" }

Sweet, thanks!


Update ui-components.md

The links are dead. Looks like the maintainer deleted the repos? :disappointed: I was excited when I stumbled upon the vuetify one.
Please Read
Link Submission
Please submit your link(s) to the official Vue.js Awesome List repository (read first their contributing guide).
If they merge your pull request there, it will also be updated in this repository.
But if you really want to add it here, I would still review it.
Bug Fix or New Feature
For bug fixes or new features to this repository, delete this template text and proceed to provide information about the changes introduced in this pull request.
Contribution Agreement
By submitting a pull request to this repository, you are accepting this contribution agreement:

I dedicate any and all copyright interest in this software to the public domain. I make this dedication for the benefit of the public at large and to the detriment of my heirs and successors. I intend this dedication to be an overt act of relinquishment in perpetuity of all present and future rights to this software under copyright law.

Thank you @kevinkhill 🎉


PGE Artifact Development

Epic for tracking tasks related to artifact, document or deliverables other than the core PGE software.

Closing, as the Epic-type ticket feature seems to have been removed from Github


Pipeline Step View messed up

Is there any activity here?
Just tried the theme and it kind of messes up the pipeline steps view:

Could manage to kind of fix it by adding those 2 lines at the end of the script:
#pipeline-box .table-box {padding: 0 0 0 18.3em !important;}
#pipeline-box .table-box .table-viewPort tr.totals .stage-start .cell-color { padding: .3em .1em  !important;}
First line fixes overlapping boxes
Second line avoids wrapping of text in header ->  "~37min 1s"


feat: New capabilities for node and edge

Added a separate select for edge
Added the label effect on node
The CSS file that imports reactflow
Added transparent transmission of various events

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

Comparison is base (c2b5669) 80.75% compared to head (c129503) 80.75%.
Report is 24 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   80.75%   80.75%
=======================================
  Files           3        3
  Lines         239      239
  Branches       32       32
=======================================
  Hits          193      193
  Misses         46       46

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


generator: support foreign dependencies

Fixes https://github.com/planetscale/vtprotobuf/issues/6
The fix suggested by @alexism (https://github.com/planetscale/vtprotobuf/issues/6#issuecomment-860878143) seems sensible, although it could be a tad expensive, so I'm skipping the special handling in all cases where we know for sure we've already generated a fast Marshal implementation for the code.

Thank you for the quick patch. You beat me to it.
It looks great.


Authentik SSO

I think this has been proven out to be a viable platform. I'm going to close this issue.
see https://github.com/crutonjohn/gitops/commit/a00731677ebafa56706d8aee196ac482d5622cda


Fix CI/CL

Some points:

one commit = one change, I'm not fond of "mixed bag" commits which change a whole bunch of things
commented code is a code smell: either we want the code or we don't (we have the git history).
I haven't looked into the detail but why comment out a failing test? It used to pass, if it doesn't it's a sign either the test needs redesigning or something has changed under our feet

@jlaine Your points are correct. I'm just trying to figure out why the tests are failing, then I'm clean up the PR.

I came here because I was curious to see the progress.
I learned that it is possible, if you want to have large style changes as this one, to ignore a specific commit in the git blame.

@jlaine Your points are correct. I'm just trying to figure out why the tests are failing, then I'm clean up the PR.

Sounds good! Again kudos for the energy you're putting into this, don't get me wrong I'm very grateful for your involvement :)

AFAICT the test that was "commented" is in fact test_doctests.py which is completely removed? Could we have the rationale for this and have it in a separate commit please?
What tool did you use to reformat the tests? Currently we run black, flake8 and isort (the two last ones might be replaced by ruff if it supports Cython code).

black is not being applied to the tests directory.


fix: check nil before save model to local cache

Codecov Report

Merging #194 (7ca898e) into master (28427b2) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #194   +/-   ##
=======================================
  Coverage   62.21%   62.21%
=======================================
  Files          41       41
  Lines        6269     6269
=======================================
  Hits         3900     3900
  Misses       1908     1908
  Partials      461      461

Impacted Files
Coverage Δ

master/recommend.go
29.41% <0.00%> (ø)

Continue to review full report at Codecov.

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


[Feature Request]: To see other user profile and their posts.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
When user click this they will redirect to that user profile page where he/she can see the all info regarding that user and all the post created by him/her.

Use Case
Public
Benefits
User enhancement
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Create a route for profile/userid which is similar to the current user profile. Just that it does not shows that user's email.

Okay, so i will hide the email and rest should be public right ?

Okay, so i will hide the email and rest should be public right ?

Yes


Allow document set options. Fixes #1332

Add options parameter to document set method and pass that to this.ref.set.
It looks like my editor removed extra whitespace on the file. What is the style guide rules for trailing whitespace?

Thanks for the PR. Don't worry about the whitespace ;)


Add support for include folders

It would be awesome if there was support to add other folders to search for !include files.
Something similar to support in vscode:  https://github.com/qjebbs/vscode-plantuml
plantuml.includepaths: Specifies the include paths besides source folder and the diagramsRoot.

Hi @yababer I am currently too busy to add functionality here.
Accepting Merge Requests.

sounds good, if I have a bit of time this weekend I will push something up.

@christo-ph I had a few minutes and added this feature.  Can you give me access so I can create a pr.

We ended up forking the plugin in our local repo and made changes there.  Let us know if you want to push it back here, I will close this issue for now.

Sorry, I missed you last message. You should simply make an merge request, this should work.

To work properly, you should fork from GitHub directly. Work on the code and then make the pull request. I suppose, you were working by cloning this repo?

Anyway, you should now be able to push

Thanks @yababer , I merged your pull request.
Would be nice if you also add the new behavior to the README.md in the root folder.


feat: add merge_build_and_host_envs section

Added the merge_build_and_host_envs section as named in schema.json.
It merges build into host env. But leaves the build env untouched.
Run dependencies are built from new host env.
Modified the test to use merge_build_and_host_envs

Fixes #518

It also still creates the build env ... We want to merge build and host, and therefore don't need to create the build env at all.