set timeout

how can I set a timeout?

Hi. At this time, the library does not have any decorator to access UnityWebRequest.timeout property. But you can simulate it for your request with something like this:
var request = new As<string>(new Get(new Text("https://google.com")));
var timeout = Task.Delay(TimeSpan.FromSeconds(4.0f));

await Task.WhenAny(AsTask(request), timeout);

Given that any request is awaitable you can write your AsTask method like this:
    private static async Task<T> AsTask<T>(IAwaitable<T> awaitable) {
      var tcs = new TaskCompletionSource<T>();

      try {
        var result = await awaitable;
        tcs.SetResult(result);
      } catch (Exception e) {
        tcs.SetException(e);
        return default;
      }

      return await tcs.Task;
    }


ostest: sem need to be destroyed

Summary
ostest: sem need to be destroyed
Impact
none
Testing
ostest

LGTM.


Attempt to fix ATs by doing MEGA PURGE

Motivation and Context
ATs are unreliable/flaky.

[ ] Context Index has been kept up to date

What has changed
Attempt to fix by doing loads of purging/sleeping, in the hope that it makes things more reliable.
How to test?
Merge it; try it.
Links
Trello: https://trello.com/c/NhkaP8wN

 Fix broken ATs/pipeline


[PostgreSQL] bytea being written as UTF-8

With PostgreSQL, when writing a field of type Data to a column in psql of type bytea, it is coerced into a varchar UTF-8 stream, which fails when encountering invalid UTF-8 bytes.
Example model:
struct Thing {
  var info: Data
}

extension Thing : ModelProtocol {
  enum Field: String, ModelField {
    case info
    static let tableName = "things"
    static let primaryKey = Field.info
  }
  typealias PrimaryKey = Data
  func serialize() -> [Field: ValueConvertible?] {
    return [.info: info]
  }
  convenience init<Row: RowProtocol>(row: TableRow<Thing, Row>) throws {
    self.init(info: try row.value(.info))
  }
}

There are 2 bugs,
The first is a crash with the SQL library here where Int8($0) crashes due to a UInt8 being forced and value being out of bounds.  This should be fixed by replacing with Int8(bitPattern: $0)
The second is an exception raised from libpq when writing data to the table
invalid byte sequence for encoding "UTF8": 0xba

I'm not entirely sure how to resolve this, but the 4th argument in PQexecParams() from libpq takes types for params passed to it, it infers when its nil (which is what's set here), and tries varchar and fails.  It might be necessary to properly qualify the types sent.

Update:
There appears to be problems reading from bytea as well.
Per the above example:
try row.value(.info) as Data

The following succeeds, however the Data read is actually a hex string, rather than byte data.
Confirmed this by checking size in psql (44 bytes for my test case), and size of data read, 90 bytes (its prefixed with \x)
For my specific use case I've worked around both of these issues by:
When writing: base64 encode data before writing
When reading: Read as String -> create Data from hex string -> create Data from base64 encoded data


Fix itemFromNotch : fix list and equip in inventory.js

The other commands (toss, craft, use, stop) in inventory.js are still broken, build is still broken in digger.js.

Ok, ready to merge.


MRG: Fix picking

Closes #2842.

lgtm
@jaeilepp merge when travis is happy and you are too.

@jaeilepp do you want to take over and adjust whatever needs to be changed in mne/viz/raw.py?

@jaeilepp constant already removed :) But I'm not sure about the correct changes to make to viz/raw.py.

I think viz.raw is fine if it was already in 0.11. I'll merge if travis is happy.

Can't we get rid of the "workaround for the FIFF mess" that's in there?

Here:
https://github.com/mne-tools/mne-python/blob/master/mne/viz/raw.py#L243

Yeah, you're right. I made a PR to your branch.

That was simple :) Thanks.

travis is not happy for some reason.
otherwise LGTM

Failures are unrelated. Those are happening across al PRs


Table mapped structs do not support embedded structs

If I attempt to map:
type CreatedUpdated struct {
	Created sql.NullInt64 //time.Time
	Updated sql.NullInt64 //time.Time
}

type ExternalSource struct {
	Uuid string
	Name string
	CreatedUpdated
}

With:
dbmap.AddTableWithName(ExternalSource{}, "external_sources")

It will fail with  sql: converting Exec argument #2's type: unsupported type models.CreatedUpdated, a struct
This is because the embedded struct walking logic in dbmap.AddTable was removed with the intention to let sqlx handle the scanning.
Somehow we need to extract the logic in sqlx.StructScan and use it here, or if it does not generalise well add back in support for struct scanning in modl.
It also appears much of the bindPlan functionality from gorp has been abandoned, it would be nice to use prepared statements in the bind plan and also use the table mapping to remove as much reflection as possible when running a query. Since we have mapped the table up front I feel we should be able to use something leaner than sqlx.StructScan in Get or other planned queries...

I'm also very interested in this feature.
Is there any way we can help, or is it dependent on new reflectx features?

Any news? Thanks!

I'm interested in this as well.  Any updates?

Any update here? We are also struggling with this.

Also just ran into this issue. Am currently using @sqs fork as a workaround.
Any chance of getting #17 merged?

It seams that this repo is not maintained any more. We also moved to @sqs 10 days ago.

FYI, we (@sourcegraph) just moved to go-gorp/gorp as we are not using embedded structs anymore. The @sqs fork has worked well for us, but I don't plan on maintaining it.

On Mar 11, 2016, at 01:39, Robert Zaremba<EMAIL_ADDRESS>wrote:
It seams that this repo is not maintained any more. We also moved to @sqs 10 days ago.
—
Reply to this email directly or view it on GitHub.


wrong results and valgrind reports errors

#include <numpy/arrayobject.h>
#include "pybind11/pybind11.h"
#include "pybind11/stl.h"
#include "xtensor/xarray.hpp"
#include "xtensor/xtensor.hpp"
#include "xtensor/xcontainer.hpp"
#include "xtensor/xbroadcast.hpp"
//#include "xtensor/xbuilder.hpp"
#include "xtensor/xview.hpp"
#include "xtensor/xeval.hpp"
#include "xtensor-python/pyarray.hpp"
#include "xtensor-python/pytensor.hpp"
#include <algorithm>            // ?

namespace py = pybind11;

template<class E1>
auto logsumexp1 (E1 const& e1) {
  using value_type = typename std::decay_t<E1>::value_type;
  auto max = xt::amax (e1)();
  return max + xt::log (xt::sum (xt::exp (e1-max)));
}

template<class E1, class X>
auto logsumexp2 (E1 const& e1, X const& axes) {
  auto max = xt::amax (e1, axes)();
  return max + xt::log (xt::sum (xt::exp (e1-max), axes ));
}

template<class E1>
auto normalize (E1 const& e1) {
  auto shape = std::vector<size_t>{1};
  auto ls = logsumexp2 (e1, shape);
  //auto ret = e1 - xt::eval (xt::view(ls, xt::all(), xt::newaxis()));
  return ls;
}

PYBIND11_PLUGIN (logsumexp) {
  if (_import_array() < 0) {
    PyErr_SetString(PyExc_ImportError, "numpy.core.multiarray failed to import");
    return nullptr;
  }
  py::module m("logsumexp", "pybind11 example plugin");

  m.def("logsumexp", [](xt::pyarray<double>const& x) {
      return xt::pyarray<double> (xt::eval (logsumexp1 (x)));
    });

  m.def("logsumexp", [](xt::pyarray<double>const& x, std::vector<size_t>const& ax) {
      return xt::pyarray<double> (xt::eval (logsumexp2 (x, ax)));
    });

  m.def("normalize", [](xt::pyarray<double>const& x) {
      return xt::pyarray<double> (xt::eval (normalize (x)));
    });

  return m.ptr();
}

The commented out ret is the line I really want, but for debugging I just returned the intermediate value ls;
The python test:
print ('norm:', normalize (np.ones ((2,4))))

produces the wrong result:
norm: [ 1.69314718  2.38629436]
correct result is:
ls: [ 2.38629436  2.38629436]
when run under valgrind I get a number of use of uninitialized values, and a variety of other scary errors, and (interestingly), I get the correct answer!

Hey,
so for the memory issues the problem is the following: When you construct an xfunction (e.g. by using the operands +/- ... ) the xfunction takes a reference to the variable. But if you return the xfunction, and the referred variable goes out of scope you'll end up with a dangling reference. There are 2 ways to prevent this:
(a) move the variable into the xfunction by using std::move (e.g. return std::move(a) + 5;)
(b) xt::eval the xfunction to evaluate the function and store the results in memory
A second problem is that indeed, you're having broadcasting issues. But as far as I can tell, you'd have them in NumPy too :)
>>> a = np.zeros((2, 4))
array([[ 0.,  0.,  0.,  0.],
       [ 0.,  0.,  0.,  0.]])
>>> c = np.amax(a, 1)
array([ 0.,  0.])
>>> np.exp(a - c)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: operands could not be broadcast together with shapes (2,4) (2,)

Btw. also on the second amax in logsumexp2 you use () to get a value. However, this will (as far as I can tell) give you only one value, the (0) value. The correct way to do it would be to use eval again.

Any chance you have the functions working in NumPy?

Actually, the advice on using eval was slightly wrong in the case of pyarrays.
It's better to use: xt::pyarray<double>(max + xt::log (xt::sum (xt::exp (e1-max)))); and return that.

#include <numpy/arrayobject.h>
#include "pybind11/pybind11.h"
#include "pybind11/stl.h"
#include "xtensor/xarray.hpp"
#include "xtensor/xtensor.hpp"
#include "xtensor/xcontainer.hpp"
#include "xtensor/xio.hpp"
#include "xtensor/xbroadcast.hpp"
//#include "xtensor/xbuilder.hpp"
#include "xtensor/xview.hpp"
#include "xtensor/xeval.hpp"
#include "xtensor-python/pyarray.hpp"
#include "xtensor-python/pytensor.hpp"
#include <algorithm>            // ?

namespace py = pybind11;

template<class E1>
auto logsumexp1 (const E1& e1) {
  auto max = xt::amax (e1)();
  return xt::pyarray<double>(max + xt::log (xt::sum (xt::exp (e1-max))));
}

template<class E1>
auto logsumexp2 (const E1& e1, const std::vector<std::size_t>& axes) {
  auto max = xt::eval(xt::amax(e1, axes));
  max.reshape({1, e1.shape()[1]});
  return xt::pyarray<double>(max + xt::log(xt::sum(xt::exp(e1 - max), axes)));
}

template<class E1>
auto normalize (E1 const& e1) {
  auto axis = std::vector<size_t>{1};
  xt::pyarray<double> ls = logsumexp2(e1, axis);
  return ls;
}

PYBIND11_PLUGIN (logsumexp) {
  if (_import_array() < 0) {
    PyErr_SetString(PyExc_ImportError, "numpy.core.multiarray failed to import");
    return nullptr;
  }
  py::module m("logsumexp", "pybind11 example plugin");

  m.def("logsumexp", [](xt::pyarray<double>const& x) {
      auto res = logsumexp1(x);
      return res();
    });

  m.def("logsumexp2", [](const xt::pyarray<double>& x, const std::vector<std::size_t>& ax) {
      auto res = logsumexp2(x, ax);
      return res;
    });

  m.def("normalize", [](xt::pyarray<double>const& x) {
      return normalize(x);
    });

  return m.ptr();
}

This code seems to work more or less, but you'll need to check the broadcasting (maybe in numpy). I don't know how it's intended.

I tried this version
template<class E1, class X>
auto logsumexp2 (const E1& e1, X const& axes) {
  auto max = xt::eval(xt::amax(e1, axes));
  auto max2 = xt::broadcast (max, e1.shape());
  return std::move (max2 + xt::log(xt::sum(xt::exp(e1 - max2), axes)));
}

but this gives me:
  File "./test_logsumexp.py", line 6, in <module>
    print (logsumexp (u, (0,)))
MemoryError: std::bad_alloc

Thanks, but I didn't return pyarray from logsumexp2 because logsumexp2 is
also called from normalize - I assumed prematurely converting to pyarray
might have overhead
On Fri, Apr 14, 2017 at 1:24 PM Wolf Vollprecht<EMAIL_ADDRESS>wrote:

#include <numpy/arrayobject.h>
#include "pybind11/pybind11.h"
#include "pybind11/stl.h"
#include "xtensor/xarray.hpp"
#include "xtensor/xtensor.hpp"
#include "xtensor/xcontainer.hpp"
#include "xtensor/xio.hpp"
#include "xtensor/xbroadcast.hpp"
//#include "xtensor/xbuilder.hpp"
#include "xtensor/xview.hpp"
#include "xtensor/xeval.hpp"
#include "xtensor-python/pyarray.hpp"
#include "xtensor-python/pytensor.hpp"
#include             // ?
namespace py = pybind11;
template
auto logsumexp1 (const E1& e1) {
auto max = xt::amax (e1)();
return xt::pyarray(max + xt::log (xt::sum (xt::exp (e1-max))));
}
template
auto logsumexp2 (const E1& e1, const std::vectorstd::size_t& axes) {
auto max = xt::eval(xt::amax(e1, axes));
max.reshape({1, e1.shape()[1]});
return xt::pyarray(max + xt::log(xt::sum(xt::exp(e1 - max), axes)));
}
template
auto normalize (E1 const& e1) {
auto axis = std::vector<size_t>{1};
xt::pyarray ls = logsumexp2(e1, axis);
return ls;
}
PYBIND11_PLUGIN (logsumexp) {
if (_import_array() < 0) {
PyErr_SetString(PyExc_ImportError, "numpy.core.multiarray failed to import");
return nullptr;
}
py::module m("logsumexp", "pybind11 example plugin");
m.def("logsumexp", [](xt::pyarrayconst& x) {
auto res = logsumexp1(x);
return res();
});
m.def("logsumexp2", [](const xt::pyarray& x, const std::vectorstd::size_t& ax) {
auto res = logsumexp2(x, ax);
return res;
});
m.def("normalize", [](xt::pyarrayconst& x) {
return normalize(x);
});
return m.ptr();
}
This code seems to work more or less, but you'll need to check the
broadcasting (maybe in numpy). I don't know how it's intended.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/QuantStack/xtensor/issues/270#issuecomment-294196043,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHK0FLrrF_dr4ccUx-9fxfAACtHxElPks5rv6u1gaJpZM4M9tl4
.

Sorry if I wasn't concise enough :)
The problem is the following:
template<class E1, class X>
auto logsumexp2 (const E1& e1, X const& axes) {
  auto max = xt::eval(xt::amax(e1, axes));
  auto max2 = xt::broadcast (max, e1.shape());
  return std::move (max2 + xt::log(xt::sum(xt::exp(e1 - max2), axes)));
}
// NOW max and max2 go out of scope! and are deallocated.
// But still in the xfunction! -> that's a dangling reference and a memory error.

so you either have to do: (Note, I am not sure if moving the same variable twice will work. Actually i think it shouldn't. Not sure how we should deal with this case @SylvainCorlay / @JohanMabille ?)
template<class E1, class X>
auto logsumexp2 (const E1& e1, X const& axes) {
  auto max = xt::eval(xt::amax(e1, axes));
  auto max2 = xt::broadcast (max, e1.shape());
  return std::move(max2) + xt::log(xt::sum(xt::exp(e1 - std::move(max2)), axes)));
}

Or the from my point of view preferred solution:
template<class E1, class X>
auto logsumexp2 (const E1& e1, X const& axes) {
  auto max = xt::eval(xt::amax(e1, axes));
  auto max2 = xt::broadcast (max, e1.shape());
  return pyarray<double>(max2 + xt::log(xt::sum(xt::exp(e1 - max2)), axes)));
}

It is a good time to point to the relevant section of the docs, if you guys think that we should improve it:
http://xtensor.readthedocs.io/en/latest/closure-semantics.html#writing-functions-that-return-expressions
:)

Here is some python code that I think does what I want with the broadcasting.  Surprisingly, it was more difficult than I expected
def logsumexp_py2 (u, axes):
    m = np.max (u, axes)
    slices = [slice(m.shape[i]) if i not in axes else np.newaxis for i in range (len (u.shape))]
    m2 = m[slices]
    #print (m.shape, u.shape, m2)
    return m2 + np.log (np.sum (np.exp (u - m2), axes))

So basically, every dimension that was reduced over gets a newaxis

well, call yourself lucky because for this feature you'll need the dynamic view I just implemented ;)
However, the dynamic view currently only works on containers, so you'd need to eval your max.
The dynamic view currently works as follows:
auto sv = slice_vector(e);
for (int i = 0; i < e.dimension(); i++)
{
    if (i in axes)
        sv.push_back(xt::newaxis());
    else
        sv.push_back(e.shape()[i]);
}
auto view = dynamic_view(e, sv);

Btw. the code I referenced is not in master, yet. It's in #269.

I thought np.broadcast_to (max, e1.shape) should have worked, but this gave an error

maybe if you reshape + broadcast?

Any hints on this?  After changing logsumexp2 to return xt::pyarray, as suggested above, now
template<class value_type>
auto normalize (xt::pytensor<value_type,2> const& e1) {
  auto shape = std::vector<size_t>{1};
  auto ls = logsumexp2 (e1, shape);
  return xt::pyarray<value_type> (xt::eval (e1 - xt::view (ls, xt::all(), xt::newaxis())));
}

This doesn't compile, it seems xt::view won't work on xt::pyarray:
/home/nbecker/.local/include/xtensor/xview_utils.hpp:85:76: error: 'layout_type' is not a member of 'xt::pyarray<double>'

I would have hoped that xt::pyarray was a valid expression and this should work.

pyarray is a valid expression.
It seems that you are using the dev version of xtensor master where we add this layout type, and xtensor-python master does not yet reflect this (as of now, xtensor-python master requires xtensor 0.8.x)
https://github.com/QuantStack/xtensor-python#dependences-on-xtensor-and-pybind11

guess I just have to wait for xtensor-python update

Sorry :/ this is the downside of the bleeding edge but we are moving pretty quickly, so we should get xtensor 0.9 very quickly with the latest addition and then have a new xtensor-python release.

OK, so since issue has been going in multiple unrelated directions. To summarize

the dangling references issues are solved with the code review by wolf.
I open a separate issue on updating xtensor-python to the future xtensor 0.9 which has the layout template parameter.

So closing this one. We can continue the conversation on usual channels.


Disabling automatic file extension affix to avoid '.scss.scss' when explicit imports are used

Related to an issue i'm looking into here
I notice on your playground this issue is not present. However, if we try to use explicit extensions in the @import like this :
@import './b';

@import './c.scss';

@import './d';

inside a file a.scss. sass.js attempts to resolve c.scss.scss. I'm able to capture this in the importer function and replace double extensions. But wondering if perhaps you have a better way to avoid this problem?

Sorry i just read the issue guideline so i've probably noted something that isn't your problem! Going to libsass

The importer callback is very much the issue of sass.js. The extension duplication is caused by _libsassPathVariations

Thanks - i actually went and did my due diligence and double checked using node-sass and indeed. It didn't have any trouble parsing the same file. Reviewing the 'included files'
includedFiles:
      [ '/Users/KevinJones/work/gits/jspmSassPluginBug/src/a.scss',
        '/Users/KevinJones/work/gits/jspmSassPluginBug/src/_b.scss',
        '/Users/KevinJones/work/gits/jspmSassPluginBug/src/_c.scss',
        '/Users/KevinJones/work/gits/jspmSassPluginBug/src/_d.scss' ],

So the immediate question is how to resolve the issue? I could look into if no one else has time.

The extension duplication is caused by _libsassPathVariations

Maybe I shouldn't comment on issues before the first coffee. My statement was wrong, see explanation below.

I notice on your playground this issue is not present.

The playground does not use an Importer Callback, so file resolution is left to libsass.
You're using an importer. With that you are in complete control of how files are resolved. This line should probably call done({ error: 'your message here' }) instead?
You prefix files with an underscore which is not universally true. It also seems like your resolve-path.js is adding .scss to the filename.

@import "hello";

is resolved by libsass in the following order:
[
  'hello',
  '_hello',
  '_hello.scss',
  '_hello.sass',
  '_hello.css',
  'hello.scss',
  'hello.sass',
  'hello.css'
]

It seems you have hardcoded the third variant. And that's why the first does not resolve anymore.

Maybe I shouldn't comment on issues before the first coffee. My statement was wrong, see explanation below.

Hah, yeah can't say you're alone there. In a similar vein, i should hold back on patching my own library until i fully understand the problem. Thanks for the breakdown, its triggered a Eurika moment, and i think i know where we've borked the path...  If its right i'll close this issue as its 99% looking like its an internal issue.

Let me know if you require the filename list generated by libsassPathVariations() passed to the importer callback. Otherwise good luck!

Thanks - I don't think we'll need that for the moment but. In the end it was such an obvious error i felt bad i didn't see it a day ago - clearly not enough coffee was used.  Pull Request Here - thanks for the explanations though, its been educational.


removed useless reference and copy to the  variable

well, I think those references and the variable copied are useless, I can't find an explanation :P
/cc @liuggio @giorrrgio

up :)


More docs/guidance re: deployment.

We should document how to deploy obmd in common configurations. It's easy to just run the executable, but:

We should still write a blurb about it.
If we endorse that method in production, we should first implement TLS suppor (#14).
Many users may want to put it behind another web server anyway, for a number of reasons including having other web/http services running on the same box -- so we should document how to do this.

While we're talking about guiding newcomers, do we want to make an examples folder to save an example config file that people can copy?

Since the configuration is pretty small at the moment, I am fine either ways (having it in the readme or having a sample in the examples directory)

Yeah, I don't think we're at the point where there's enough information there for splitting it out to make sense.

Ok, sounds good then.  I'll work on the documentation after #22.

#26 is merged, closing.


Add notifications section to My Dashboard

Wireframe: http://h90c3m.axshare.com/#g=1&p=my_dashboard
Add the notifications section to the top of the My Dashboard page. This will require us to begin recording when notifications have been sent out. Once the plan associated with a notification has been viewed by the user we should stop displaying that notification.

Likely related to #760

global notifications feature added. closing issue.


Add end-to-end automated test workflow

Once this library is being consumed by instructlab/instructlab, it would be helpful to automatically test changes to this library in an end-to-end run of the ilab workflow. You can copy this workflow that I added to the sdg repo to do this. It should work with no modifications, though it does say "instructlab-sdg" in one comment.
https://github.com/instructlab/sdg/pull/33

This was reverted in #76. My comment there explains why this is relevant:

Testing the user end-to-end instructlab workflow is the point of this workflow. It will give you an early indication if your changes to the library break the CLI. Sometimes that might be on purpose, but in case it's not, it will let you know before you publish a release.
This is not intended to replace other functional testing aimed at the library more directly, but the full end-to-end workflow is important.
I'm OK if you want to disable it for the moment, but it needs to go back on at some point. A better way to do that is to just edit the workflow to turn it off instead of removing it from the repo.


Object has no member 'requireGuestOnly'

According to the docs there exists a
// requires the request client to be unauthenticated, aka. guest
$apis.requireGuestOnly()

but using this throws
TypeError: Object has no member 'requireGuestOnly'
and I can't find it here either

Hm, it seems that I've forgotten to add the binding. I'll update it with the next release.
If you want to emulate the same behavior as the middleware, you can try checking for the authRecord and admin context keys:
if (c.get("admin") || c.get("authRecord")) {
    throw new BadRequestError("The request can be accessed only by guests.")
}


Formatting get not exported to other formats

When exporting a complete formatted file the given values do not get exported to .doc, .md, .odt, .pdf, etc.
Check issue https://github.com/ether/etherpad-lite/issues/5211 as it is not clear if it is a problem with the core import/export functionality or a problem with the plugins that provide that information. The document inside etherpad gets actually properly formatted by those plugins!
That is the page that got exported! You can upload the NEW.ETHERPAD.txt file - remove (txt) before uploading.

Other export formats look as follow - on the same page:
Github doesn't like all formats so .txt got added and needs to get removed first.

DOC - New.doc.txt
ETHERPAD - New.etherpad.txt
HTML - New.html.txt
MD - New.md
ODT - New.odt
PDF - New.pdf
TXT - New.txt

Please provide exact steps to replicate!


Update polish translation

Thanks! Added text changes via this commit: https://github.com/brevven/silicon/commit/e27f86df760a54891a3c1dad222289d8e11eff79
Note I did not merge the __ITEM__ changes specifically


[Bug]WinDbgPreview crash on windows dev build 25295

Running WinDbgPreview on windows 25295.1000 crashes, tried uninstalling and reinstalling, but it doesn't work.
Here is the crash dump
CrashDumps.zip

This looks like an issue with Windows Insiders builds that affects other applications and is not related to WinDbg.
It looks like a fix is in progress and hopefully available in a newer Insiders build soon.


Add Sublist exercise

This PR takes a PR from @leetwinski and refactors it some, also adds a test generator template for the sublist exercise.
Should resolve #161.

LGTM thanks!


Need Updated Readme how using Docker

Expected Behavior

Training on server using Lean build on top of Docker
Actual Behavior

Got same issue #1173 , but no current solution
COPY failed: stat /var/lib/docker/tmp/docker-builder532641228/Launcher/bin/Release: no such file or directory
Potential Solution

Provide Step by step development config Lean using Docker
Reproducing the Problem

Following Usage in Dockerfile

docker build -t quantconnect/lean:foundation -f DockerfileLeanFoundation .
docker build -t quantconnect/lean:algorithm -f Dockerfile .
Got error COPY failed: stat /var/lib/docker/tmp/docker-builder532641228/Launcher/bin/Release: no such file or directory

System Information

AWS server Ubuntu 16.04.3 LTS
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] No current solution, I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

This is a duplicate, as you stated of #1173. As Jared stated in that issue, it is not a bug.
Please bring technical questions to Lean Group or Slack chat.

@AlexCatarino and @jaredbroad yeah i know, but it is necessary right to provide step by step and updated to Readme? so anyone who got started developing using Lean not confused anymore because of lack of documentation here

Lean execution/deployment in a docker container is not exactly Lean development, right?
This is a technical issue and those are addressed in Lean Group or Slack chat. Please use those channels.

Expected Behavior
Training on server using Lean build on top of Docker
Actual Behavior
Got same issue #1173 , but no current solution
COPY failed: stat /var/lib/docker/tmp/docker-builder532641228/Launcher/bin/Release: no such file or directory
Potential Solution
Provide Step by step development config Lean using Docker
Reproducing the Problem
Following Usage in Dockerfile

docker build -t quantconnect/lean:foundation -f DockerfileLeanFoundation .
docker build -t quantconnect/lean:algorithm -f Dockerfile .
Got error COPY failed: stat /var/lib/docker/tmp/docker-builder532641228/Launcher/bin/Release: no such file or directory

System Information
AWS server Ubuntu 16.04.3 LTS
Checklist

[x]  I have completely filled out this template

[x]  I have confirmed that this issue exists on the current master branch

[x]  No current solution, I have confirmed that this is not a duplicate issue by searching issues

[x]  I have provided detailed steps to reproduce the issue

Did you got solution to this issue I am trying to build docker image using docker build -t lean . command but its giving same error which you got in 2018, I used your commands as well but still same error any guide me helpful


Update components/update-service to update secret parameters in place

Fixes bugzilla 1564979

Updates components/update-service to update a secret in place rather than creating a new secret.

Attempted to break this massive function down into something a bit easier to maintain. Typescript is interesting, not sure I'm sold yet (but working on it), struggled with a type string[][] that didn't work (see note).  Made some other minor changes like using let instead of var, etc.
Attempted to update types, but object only works on a very simple object, TypeScript seems to think a multidimensional object should have a custom type def.  That would be a good bit of work to update throughout the codebase I think.
Compiler complains about this a lot:

For some reason I prefer the first:
// :)
let foo:object = { };
// :/
let foo: object = {};

But I can update this && add the space to make it happy.

Cleaned up the typescript lint warnings.


Add API authentication to cloud

Acceptance criteria

[ ] API authentication added for cozyauth-cloud (ee).
[ ] API token is JWT signed by Ed25119.
[ ] Public key for verification is provided as environment variable.
[ ] All requests to /passkeys require authentication.
[ ] Passkeys API is added to cozyauth-cloud.

See https://github.com/cozystack/cozyauth/issues/209#issuecomment-2299145500


aws[patch]:  Fix fails when calling multiple tools simultaneously

Fixes #6173

I've made a dirty fix that works... any ideas on how to do this more cleanly?

I've completed the test and refactored it slightly. Is this okay with you?


Unit Tests for Hyperedge are not running

Unit tests for the hyperedge class are not running.
Going back in the commit history, it seems that the issue arose when the unit test module was removed in favor of pytest. However, pytest requires the class names to start with Test, which is not the official name.
I will raise a PR with the changes.

Fixed by #198


Indentation

I'm pretty sure this is what's borking. Still don't know why its only borking for you and not me but 🤷♂

This branch works and is tidier YAML. I'm going to go ahead and merge 🎉


run 'flutter doctor' report 'Android SDK file not found: adb.'

Steps to Reproduce

1.Install Android Studio
2.run flutter doctor
Logs
➜  ~ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.14.6 18G3020,
    locale zh-Hans-CN)
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    ✗ Android SDK file not found: adb.
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
[✓] Android Studio (version 3.6)
[✓] VS Code (version 1.43.2)
[!] Connected device
    ! No devices available

I don't know how to solve this error, has anyone encountered such a situation?My flutter version is v1.12.13+hotfix.8,Android Studio‘s version is 3.6.1

Hi @leo193
can you please provide your flutter doctor -v ,
Thank you
possibly related to https://github.com/flutter/flutter/issues/51712

I also encountered the same problem.
Logs:
` ~ flutter doctor -v
[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.14.5 18F2058, locale zh-Hans-CN)
• Flutter version 1.12.13+hotfix.8 at /Users/songxuejiao/Documents/flutter
• Framework revision 0b8abb4724 (6 weeks ago), 2020-02-11 11:44:36 -0800
• Engine revision e1e6ced81d
• Dart version 2.7.0
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/songxuejiao/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.3
• ANDROID_HOME = /Users/songxuejiao/Library/Android/sdk
✗ Android SDK file not found: adb.
• Try re-installing or updating your Android SDK,
visit https://flutter.dev/setup/#android-setup for detailed instructions.
[✗] Xcode - develop for iOS and macOS
✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
Download at: https://developer.apple.com/xcode/download/
Or install Xcode via the App Store.
Once installed, run:
sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
sudo xcodebuild -runFirstLaunch
✗ CocoaPods not installed.
CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
Without CocoaPods, plugins will not work on iOS or macOS.
For more info, see https://flutter.dev/platform-plugins
To install:
sudo gem install cocoapods
[✓] Android Studio (version 3.6)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 44.0.2
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
[!] Connected device
! No devices available
! Doctor found issues in 3 categories.`

I also encountered the same problem.
Logs:
`
➜  ~ flutter doctor -v
[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.14.5 18F2058, locale zh-Hans-CN)
• Flutter version 1.12.13+hotfix.8 at /Users/songxuejiao/Documents/flutter
• Framework revision 0b8abb4724 (6 weeks ago), 2020-02-11 11:44:36 -0800
• Engine revision e1e6ced81d
• Dart version 2.7.0
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/songxuejiao/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.3
• ANDROID_HOME = /Users/songxuejiao/Library/Android/sdk
✗ Android SDK file not found: adb.
• Try re-installing or updating your Android SDK,
visit https://flutter.dev/setup/#android-setup for detailed instructions.
[✗] Xcode - develop for iOS and macOS
✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
Download at: https://developer.apple.com/xcode/download/
Or install Xcode via the App Store.
Once installed, run:
sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
sudo xcodebuild -runFirstLaunch
✗ CocoaPods not installed.
CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
Without CocoaPods, plugins will not work on iOS or macOS.
For more info, see https://flutter.dev/platform-plugins
To install:
sudo gem install cocoapods
[✓] Android Studio (version 3.6)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 44.0.2
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
[!] Connected device
! No devices available
! Doctor found issues in 3 categories.
`

Hi @leo193

Go to platform-tools folder in your Android SDK, make sure adb files aren't nested into another folder.  If nested,  move all files to top platform-tools folder. Platform-tool folder shouldn't nested and it should adb inside it

Install SDK tools from your SDK manager (if hidden, uncheck Hide Obsolete Packages)

Execute flutter doctor    --android-licenses and accept the licenses

Execute flutter doctor -v, it should show [✓] in Android ToolChain

Thank you

@ TahaTesser
Thank you. The problem is solved.The problem is that there are subdirectories under the platform tools directory, which is also called platform tools.Put all the contents of subdirectories at the top level to solve the error


Incorrect buffer size

This code is incorrectly giving back a slice to invalid memory. It allocates 12 u8s (12 bytes) and creates a slice from it of 12 f32s (48 bytes).
https://github.com/gwihlidal/meshopt-rs/blob/16a30462f38d67ea2cef355fbb10ef6c6c4daa32/src/utilities.rs#L181-L186
https://doc.rust-lang.org/std/slice/fn.from_raw_parts.html
The len argument is the number of elements, not the number of bytes.

Also if I'm not mistaken, it's returning a slice to the buffer that is on the stack.

Nice catch!

it's returning a slice to the buffer that is on the stack.

Oh absolutely, this is so so UB.
if vertex <= self.vertex_count {
            Err(Error::memory_dynamic(format!(
                "vertex index ({}) must be less than total vertex count ({})",
                vertex, self.vertex_count
            )))
        }

and this if is reversed! I wonder if anyone actually uses this method

Fixed for real in 94dfd72


unable to install extention advance rest client

Originally reported on Google Code with ID 313
What steps will reproduce the problem?
1. trying to install on google chrome with ubuntu
2.
3.

What is the expected output? What do you see instead?
it should be installed

On what operating system, browser and browser version?

ubuntu 12.04

Please provide any additional information below.
Could not install package: 'UTILITY_PROCESS_CRASHED_WHILE_TRYING_TO_INSTALL'. Could
not install package because a utility process crashed. Try restarting Chrome and trying
again.

Reported by vishal.igex on 2015-06-08 06:56:43

This looks more like Chrome problem. What channel did you used during the installation?


--continue-on-success is nor working correct

Describe the bug
--continue-on-success option isn't working.
If guest session is enabled, brutforce is spopping on first password
To Reproduce
Steps to reproduce the behavior i.e.:
Command: nxc smb ip.txt -u User -p brut/top20admin.txt  --continue-on-success
Resulted in:
└─$ nxc smb ip.txt -u User -p brut/top20admin.txt  **--continue-on-success**
SMB         <IP_ADDRESS>    445    BUG              [*] Windows 10 Pro 19045 x64 (name:BUG) (domain:Buh) (signing:False) (SMBv1:True)
SMB         <IP_ADDRESS>    445    BUG1             [*] Windows 10 Pro 19045 x64 (name:BUG1) (domain:Buh1) (signing:False) (SMBv1:True)
SMB         <IP_ADDRESS>    445    BUG              [+] Bug\User:admin (Guest)
SMB         <IP_ADDRESS>    445    BUG1             [+] Bug1\User:admin (Guest)
Running nxc against 2 targets ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00

**NetExec info**
 - OS: Kali 2024.3
 - Version of nxc: v1.5.2
 - Installed from: apt

If guest session is allowed, if you authenticate using any unknown domain user, you will get a valid authentication tho. Can you try the same thing but for a valid domain user ?
Otherwise it doesn't make sense to bruteforce unknown users

this case is not from my network)))
if i use cme, its look like this:
└─$ cme smb ip.txt -u user -p brut/top20admin.txt  --continue-on-success
SMB         <IP_ADDRESS>    445    BUH1             [] Windows 10 Pro 19045 x64 (name:BUH1) (domain:Buh1) (signing:False) (SMBv1:True)
SMB         <IP_ADDRESS>    445    BUH              [] Windows 10 Pro 19045 x64 (name:BUH) (domain:Buh) (signing:False) (SMBv1:True)
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:admin
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:123456
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:12345678
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:1234
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:Password
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:123
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:12345
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:admin123
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:123456789
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:adminisp
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:demo
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:root
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:123123
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:admin@123
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:123456aA@
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:01031974
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:Admin@123
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:111111
SMB         <IP_ADDRESS>    445    BUH1             [+] Buh1\user:admin1234

Ah yeah I got your point! Taking a look at it :)

@mpgn looks like it's related to this https://github.com/byt3bl33d3r/CrackMapExec/issues/321
I have got the same issue without the guest thing:

if you have only one user and a password is found, why you want to continue to bf ?
the feature is useful if you have multiple users

Yeah I agree xD
So going back to what I said before. There is no issue, if guest is enabled, authentication will always be valid so nxc stops.

Adding to the discussion:
Nxc removes users with a correct password from the queue, even if --continue-on-success is enabled because:

Bruteforcing the guest account doesn't make any sense
There is no reason to hammer passwords against an account where you already found a valid password

Fyi this feature was not implemented back in CME (at least the kali version), that is why you see the different behaviour.
Add another user to the queue and you will see the result:

the point is that the user "User" is valid, but because of the guest session enabled, I can't brute force it. How should I be in such a case?
That is why I cited the case from CME as an example

Are you sure you are authenticating against the correct domain? If the user exists in the domain you should get a failed login (as you can see in my screenshot).
In your screenshot it looks like you found the user for the domain "Buh" but in the cme output you are authenticating against the "Buh1" domain.

These look like local user. If you want to bruteforce local users, add the --local-auth. Otherwise disable the guest domain user and guest local accounts

If there still is something unclear feel free to reopen or hit us up on discord


Communication fault

After the adapter has been started, communication between the SMA Tripower and Homemanager breaks off after a few minutes.
Communication works again as soon as the adapter is stopped.
Versions:

Adapter version: 0.7.0
SUNNY TRIPOWER 10.0 SE, software 1.21.2.R, firmware 3.1.9.R
Sunny Home Manager 2.0
SL-Bridge 0.20.0 with 32,5 kWh battery

All software was updated to latest versions, bus is undependend SL-Bridge.
Without communication, the display of the inverter is faulty/incorrect and no control can take place because there are no reference values from the Home Manager

@HJGratz
This looks much like a network problem. Please check if your network fulfills the SMA requirements regarding multicast capabilities of the network:
All network components used must support at least version 3 (IGMPv3) of the IGMP protocol. If Ethernet switches are used in the system, it must be ensured that they do not carry out "IGMP snooping". This is the case with most simple switches, with higher-quality "managed" switches this function must be disabled.
I suggest that you post your question also in the ioBroker Forum, maybe there are users who have experience with configuring your kind of network for SMA multicast.
https://forum.iobroker.net/topic/43011/test-adapter-sma-em-v0-7-x-latest/242
https://forum.iobroker.net/topic/1100/sma-wechselrichter/575

@pdbjjens
Thanks for the hint! After replacing a switch with IGMPv3 (~30€) the problem was solved. The system has now been running for more than 8 hours without this failure (before it was only a few minutes). It looks solved :-)


Dont' use compose project name with dots, add dot test

The Problem/Issue/Bug:
With docker-compose 2.5.1...

https://github.com/drud/ddev/pull/3851#issuecomment-1132745998 (discussion)
https://github.com/docker/compose/issues/8881
https://github.com/compose-spec/compose-go/pull/197

Bottom line is that ddev ssh and ddev exec (possibly other things) don't work if COMPOSE_PROJECT_NAME has a dot in it. You get:
$ ddev ssh
service "web" is not running container #1
Failed to ddev ssh web: exit status 1

I imagine since ddev was using COMPOSE_PROJECT_NAME initially with docker-compose config, which inserts name into the .ddev-docker-compose-full.yaml (without dots) and then uses that .ddev-docker-compose-full again with COMPOSE_PROJECT_NAME set (and with a dot in it), that it triggers an obscure edge case.  It's easier to remove the dot here than to chase it in compose.
How this PR Solves The Problem:

Remove the dot
Add a test that uses project names with dots in them.

Manual Testing Instructions:

https://github.com/drud/ddev/pull/3851#issuecomment-1132745998

Automated Testing Overview:

Added TestCmdSSH

I tested and this works on docker. Are the colima failures real?
$ ddev version
 ITEM             VALUE
 DDEV version     v1.19.2-25-g6bee2020
 architecture     amd64
 db               drud/ddev-dbserver-mariadb-10.3:v1.19.2
 dba              phpmyadmin:5
 ddev-ssh-agent   drud/ddev-ssh-agent:v1.19.0
 docker           20.10.15
 docker-compose   v2.5.1
 docker-platform  e480
 mutagen          0.14.0
 os               linux
 router           drud/ddev-router:v1.19.0
 web              drud/ddev-webserver:v1.19.2

Thank you so much for the added tests!

Thanks so much as always.
Colima has a perpetual problem with port bindings. Doesn't affect ordinary users very much, but breaks the tests at least half of the time and I have to restart.

https://github.com/abiosoft/colima/issues/189


Activate pushing of images

Signed-off-by: Florian Froese<EMAIL_ADDRESS>

How come Build / Push images (push) is actually running in this PR?

How come Build / Push images (push) is actually running in this PR?

I pushed the branch to origin and not to my fork. So it ran the job from the commit (push) action.
That way I could test if a push would actually work. (And currently it does not because we need to remove some artifacts first)


[Question] Microphone in use?

This isn't really an issue, I just had no clue where to post such a question. Recently, my Windows updated, and I noticed a new feature in my tray that was telling me how many applications were currently using my microphone. Naturally, I was curious, so I clicked into it. I noticed the expected game I was playing for voice chat, but I also noticed Aurora was using my microphone.
Why does Aurora need to use my microphone? The only audio-based capability I've ever seen on the application is the ability to react to audio playing on the computer, but that has nothing to do with the microphone?

Likewise, same thing, updated windows recently and an icon on the task bar status area states "Aurora is using your microphone"

This is used for the game state variable microphone volume. You'll be able to disable it in the next version

Also, our discord is a pretty good place for questions like this: https://discord.gg/YAuBmg9


[joltphysics] Properly patching build so that it will install properly

I don't know if this issue was ever logged as an issue but the joltphysics package was not working properly. Unfortunately Jolt's build system is quite divergent from vcpkg's. I had to make some invasive removals to remove extra configurations for Distribution / Sanitized targets and made up for it by making sure that the Release build matches the behavior of the Distribution build.
Ideally I can get some of these changes made upstream but until then I wanted to provide this PR as an option for getting a port that's been in vcpkg for a while to actually work. Also I wanted to depend on it and was sad to find that it was non-functional.

[X] Changes comply with the maintainer guide
[X] SHA512s are updated for each updated download
[X] The "supports" clause reflects platforms that may be fixed by this new version
[X] Any fixed CI baseline entries are removed from that file.
[X] Any patches that are no longer applied are deleted from the port's directory.
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

Pulled this back to a draft because I just now figured out that I screwed up and trying to link debug will result in an iterator mismatch 😩

Oh strange. Only joltphysics-targets-release.cmake is generated. No -debug version? Weird. If anyone has any ideas I'd love a hand :)

When not provided upstream, exported CMake config must be named unofficial-<PORT> and use namespace unofficial::<PORT>::. https://learn.microsoft.com/en-us/vcpkg/contributing/maintainer-guide#add-cmake-exports-in-an-unofficial--namespace

Oh thanks! I missed this but it should be corrected now

Huh, now I'm getting vcpkg\buildtrees\joltphysics\src\v3.0.1-300ee9d871.clean\Jolt\Core\Memory.cpp.obj: is not a native COFF file. Recompile without /GL? when I try to link against this :/

Had to disable IPO

Still getting an issue with the debug target not getting generated though

Should have looked at the unofficial-brotli port sooner. Config fixup solved the debug build problems


Update .spi.yml

Fixes https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/issues/2807 - failures with the xcodebuild-based builds. The problem was that our package scheme heuristics failed and tried to build the scheme 'swift-test-Package'. This adds a hint for us which scheme to pick.
It also removes the swift_version: 5.9 attributes which pinned doc builds to use Swift 5.9. This is now the defaut, so this setting is superfluous (and would keep doc gen using 5.9 even after 5.10+ come out).

@swift-ci please test


Adjustable size of columns in tables (e.g. overview pages)

User story

As a {HR-employee
I want / need to be able to adjust the width of each column
So that {I can see the full length of information about the DI

Acceptance criteria

Create responsive table, so that all (DI) information is fully displayed in each row and column
put a maximum width for each column, so that big email adresses don't crash the whole table
if an email adress is larger than the maximum width, show it by hovering over the email adress
optionally: Moving the cursor to the head column should allow the user to adjust the width of each column manually

Definition of done (DoD)

code peer-reviewed (by at least one team member)
acceptance criteria met
integrated into a clean build
documentation updated
code is well-formatted (according to our norms) and structure
Code supports all functions/cases and all responses defined in the API
no severe bugs during main functionalities (program should not stop executing due to these bugs, e.g. runtime exception in Docker)
Bill of Materials updated if necessary
70% of the tests pass
important functions shall be tested

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

https://medium.com/@imdebasispanda/resize-table-column-angular-5cb58b67367


karma preprocessors result is not replected to spec

Bug Report
Affected Package
@angular-devkit maybe
Is this a regression?
Yes, the previous version in which this bug was working properly.
Description
I created new workspace with ng new
I changed template file from app.component.html to app.component.pug
and installed pug, karma-pug-preprocessor packages.
I added some codes in karma.conf.js:
...
plugins: [
...
require('karma-pug-preprocessor')
],
...
files: [
  'src/**/*.pug'
],
preprocessors: {
  'src/**/*.pug': ['pug']
},
...

and execute default specs using command yarn test.
With some log in karma-pug-preprocessor, I found this result.
Logs are added in createPugPreprocessor function in node_modules/karma-pug-preprocessor/index.js.
karma-pug-preprocessor is working properly, and result is good.
content, processed are as expected.
λ yarn test
yarn run v1.22.4
$ ng test
- Generating browser application bundles (phase: setup)...29 06 2021 00:19:34.067:FATAL [preprocessor.pug]: ### peprocessor.pug
\ Generating browser application bundles (phase: building)...29 06 2021 00:19:36.493:FATAL [preprocessor.pug]: ### peprocessor.pug function
29 06 2021 00:19:36.508:FATAL [preprocessor.pug]: ###  {
  file: File {
    path: 'D:/Workspace/cli12-pug/src/app/app.component.html',
    originalPath: 'D:/Workspace/cli12-pug/src/app/app.component.pug',
    contentPath: 'D:/Workspace/cli12-pug/src/app/app.component.pug',
    encodings: [Object: null prototype] {},
    mtime: 2021-06-28T14:44:56.676Z,
    isUrl: false,
    doNotCache: false,
    type: undefined,
    isBinary: null
  },
  content: '.content\r\n\tspan {{ title }} app is running!',
  processed: '\n
{{ title }} app is running!
'
}

But the component template in spec still remains a string of pug.
In app.component.spec.ts file, I added some log:
it('should render title', () => {
    const fixture = TestBed.createComponent(AppComponent);
    fixture.detectChanges();
    const compiled = fixture.nativeElement;
    console.log(compiled);
    console.log(compiled.querySelector('.content span'));
    expect(compiled.querySelector('.content span').textContent).toContain('cli12-pug app is running!');
  });

The value of compiled is still a string of pug:

.content span cli12-pug app is running!

So, compiled.querySelector('.content span') is null and spec failed.
Why the result of Karma preprocessor is not connected with component template in Karma?

I LOVE Angular ❤

This is expected.
The Karma builders uses Webpack under the hood which by passes preprocessors.
To use pug, you will need to use a 3rd party builder like https://github.com/manfredsteyer/ngx-build-plus although by doing so you are leaving the supported path.


Removing entry.client & entry.server causes remix dev to hang

What version of Remix are you using?
1.14.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
Just testing the ability to use default client/server entry files. When I remove them, Remix installs isbot to my dependencies (would be nice to know why, otherwise it may get removed accidentally in the future) and hangs on yarn dev. Here's the console log from when it ran normally, then started failing after removing the client/server files:

My entry.client.tsx:
import { RemixBrowser } from '@remix-run/react'
import { hydrate } from 'react-dom'

hydrate(<RemixBrowser />, document)

My entry.server.tsx:
import type { EntryContext } from '@remix-run/node'
import { RemixServer } from '@remix-run/react'
import { renderToString } from 'react-dom/server'

const handleRequest = (
  request: Request,
  responseStatusCode: number,
  responseHeaders: Headers,
  remixContext: EntryContext
) => {
  const markup = renderToString(
    <RemixServer context={remixContext} url={request.url} />
  )

  responseHeaders.set('Content-Type', 'text/html; charset=utf-8')
  //responseHeaders.set('Cross-Origin-Embedder-Policy', 'require-corp')
  //responseHeaders.set('Cross-Origin-Opener-Policy', 'same-origin')

  return new Response(`<!DOCTYPE html>${markup}`, {
    status: responseStatusCode,
    headers: responseHeaders,
  })
}

export default handleRequest

Expected Behavior
Remix should run properly without client/server entry files.
Actual Behavior
See above

To answer the isbot question, the default Remix entry.server.tsx file now includes isbot to determine whether to stream the response or immediately render it. This is new to recent versions of the templates, which use React Streaming SSR. If you created your project on an older version of the template you wouldn’t have that setup.

To answer the isbot question, the default Remix entry.server.tsx file now includes isbot to determine whether to stream the response or immediately render it. This is new to recent versions of the templates, which use React Streaming SSR.

bingo!

based on your entry files, it appears you're using react 17, is that correct? if so, that's unfortunately unsupported with optional entries currently, but #5681 will bring that to you soon

Got it, sorry I missed the React 18 requirement - I'll look at updating or try with the upcoming fix - thanks!


use namazu to simulate slow disk condition

Now we meet some panics when etcd does fysnc too slow and pd requests etcd failed, we should simulate this conditions and guarantee that even the disk is slow, tikv can work well too.
I think https://github.com/osrg/namazu is a good choice to do this, or any other tools?
/cc @iamxy @v01dstar

+1
Currently namazu does not support hooking fsync, but we can implement it and other calls easily.

Cool @AkihiroSuda
We will try it later.

Ping @v01dstar
Can we use namazu in docker now?

Hi @AkihiroSuda
I use namazu to test 3 PD + 3 TiKV + 1 PD in container mode, see https://gist.github.com/siddontang/d1903ef08bb7029039e23e7b38debefe
The config is:
# Policy for observing events and yielding actions
# You can also implement your own policy.
# Default: "random"
explorePolicy = "random"

[explorePolicyParam]
  # for Ethernet/Filesystem/Java inspectors, event are non-deterministically delayed.
  # minInterval and maxInterval are bounds for the non-deterministic delays
  # Default: 0 and 0
  minInterval = "0ms"
  maxInterval = "5ms"

  # for Ethernet/Filesystem inspectors, you can specify fault-injection probability (0.0-1.0).
  # Default: 0.0
  faultActionProbability = 0.0

  # for Process inspector, you can specify how to schedule processes
  # "mild": execute processes with randomly prioritized SCHED_NORMAL/SCHED_BATCH scheduler.
  # "extreme": pick up some processes and execute them with SCHED_RR scheduler. others are executed with SCHED_BATCH scheduler.
  # "dirichlet": execute processes with SCHED_DEADLINE scheduler. Dirichlet-distribution is used for deciding runtime values.
  # Default: "mild"
  procPolicy = "extreme"

[container]
  # Default: false
  enableEthernetInspector = true
  ethernetNFQNumber = 42
  # Default: true
  enableProcInspector = true
  procWatchInterval = "1s"
  # Default: true (for volumes (`-v /foo:/bar`))
  enableFSInspector = true

Is the config OK?
In my test, if I set maxInterval to about 30ms, I will wait a very long time for TiDB initialization, seem that the delay affects cluster seriously. @v01dstar
I just enableFSInspector, does it work for container? or should I need to do more things?
Thank you.

@siddontang
Thank you for trying 😃
The config seems basically OK.
Note that enableFSInspector works for container volumes.
So you need -v /foo:/tidata/tikv1.

Hi @AkihiroSuda
/foo is const and can't be changed?
We have multi volumns want to inspector, following is ok?
-v /var/tikv1:/tidata/tikv1
-v /var/tikv2:/tidata/tikv2
-v /var/tikv3:/tidata/tikv3

Not const, it's OK

It works now.
I will run cluster with namazu for a long time, then I will give you some report.
Thank you @AkihiroSuda


Add open container items as variations of their closed versions

In recent updates, Jagex added the ability to open the Seed box, Herb sack, Coal bag, and Gem bag much like the ability the Looting bag has had for some time. These items should be added as variations of their closed versions so that inventory tags, bank tags, etc. are maintained for both versions.

Though I haven't seen a linked pull request for this, it appears to be done.  Item IDs for the open versions of the items you mention are in the master branch.

It still seems to be missing the herb sack and seed box, but does contain the coal bag and gem bag.

Fixed in 9c90c0931c65abd6fc63a0a09c38b77a9e7c54dc / https://github.com/abextm/runelite-cache-code-updater/commit/a35c71c49e5277e037c7c9979d38e2fb7ed3c65f


Remove auth and db snippets

These snippets have been moved to firebase-admin-python
cc @hiranya911 @avishalom

LGTM 👍


added documentation for getcwd

I'm currently reading through Advanced Programming in the UNIX Environment which goes through most of the libc functions, explains them and does some example code. I'm trying to port the code into Rust.
I thought about adding my findings as documentation string into libc. Would that be a thing that would benefit this repo? I've added the documentation for getcwd in this PR, would that be good enough? I would add those documentation bits by bits. Is that ok or should I put as much documentation as possible in one huge PR?

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks for the PR! Right now though this crate is basically just a massive header file, and we probably don't want to go down the road of adding an example for all the functions bound here.
This does seem like some good documentation, though! @steveklabnik do you have any idea perhaps on where documentation like this might go?

Odd timing, I've recently been thinking about docs and was just wrapping up a PR starting to improve the situation for nix (https://github.com/nix-rust/nix/pull/385).  As can be seen in the getcwd example, writing code that uses the libc APIs correctly can be a task.
@philippkeller It looks like we don't currently have support for getcwd in nix.  Would you be interested in adding support (with docs) there?  The goal of nix is to wrap the underlying libc APIs (mostly 1-to-1 but varying where required to safely capture semantics).
In general, I think it would be preferable for people to be encourage to use either std, nix, or some other more specialized crate for interacting with libc functionality rather than using libc directly.  That being said, having better documentation on libc and the aforementioned advice are not mutually exclusive.

@posborne: sure! I am new to Rust though and if I would contribute code it needs some attentive pair of eyes on my PRs. If that is ok then I'd be happy to contribute. I was not really sure if the documentation really belongs here (as @alexcrichton says it's just a header file and I was not sure if that was intentional or not).
That said, @posborne if you could tell where getcwd should be living within nix, I'll retract my PR here and transform the PR into a nix PR with code and documentation.

That said, @posborne if you could tell where getcwd should be living within nix, I'll retract my PR here and transform the PR into a nix PR with code and documentation.

Currently, we generally map the libc header files (in this case unistd.h) to modules in the library, so unistd.rs.

This does seem like some good documentation, though! @steveklabnik do you have any idea perhaps on where documentation like this might go?

This is a good question. I'm not sure; I'm not totally opposed to adding docs here, but can also see why we might not want to.

@posborne one question before I move this to nix: getcwd is already implemented in std which is exposed via env::current_dir. Does it make sense to have it also implemented in nix?

@posborne one question before I move this to nix: getcwd is already implemented in std which is exposed via env::current_dir. Does it make sense to have it also implemented in nix?

The std implementation is probably what users should use but I think it would be OK to add to nix as well for completeness/convenience (if code was using other code from Nix extensively or working in a no_std environment it might make sense).
CC @fiveop @kamalmarhubi

I agree with @posborne in that we (nix) should aim for completeness, even in cases were we don't see the necessity.
And, even though I have no stake in this, I would think documentation for libc would be a bit redundant to the corresponding man pages. However, nix does change the interface sufficiently - even if only slightly - and has at least somewhat organized code, so that having its own documentation would be useful in my opinion.


💄 make sure the push messages are readable

Motivation

There is a "Register Device" menu item
The push messages are not readable because it's the same color as the backgroud

Description

Remove the "Register Device" menu
Change the color of the push message to make sure it is readable.

Progress

[x] Done Task

See https://docs.google.com/presentation/d/1FZL-Wy5D_wUo5ADrfhpRBh-Ny2rGauI3_C8SBrlk6rw/edit#slide=id.g3c52d834c7_0_205

@cianclarke @StephenCoady @aidenkeating mind taking a look?


feat(menu-item): fix toggle accessibility issue

Description
Legacy Menu Items were preventDefaulting events when a toggle is provided as children causing keyboard accessibility on the toggle to break. This PR makes sure the events for menu-item are only handled when the event target is the menu-item itself.

Missing unittest but I am not sure how keen you are to write unittests for legacy component.
Tested this with Cantina and toggle works but after toggle the Menu loses focus and next tab or arrow down closes the menu.


apply_ufunc erroneously operating on an empty array when dask used

Problem description
apply_ufunc with dask='parallelized' appears to be trying to act on an empty numpy array when the computation is specified, but before .compute() is called. In other words, a ufunc which just prints the shape of its argument will print (0,0) then print the correct shape once .compute() is called.
Minimum working example
import numpy as np
import xarray as xr

def example_ufunc(x):
    print(x.shape)
    return np.mean(x, axis=-1)

def new_mean(da, dim):
    result = xr.apply_ufunc(example_ufunc, da,
                            input_core_dims=[[dim]], dask='parallelized',
                            output_dtypes=[da.dtype])
    return result

shape = {'t': 2, 'x':3}
data = xr.DataArray(data=np.random.rand(*shape.values()), dims=shape.keys())
unchunked = data
chunked = data.chunk(shape)

actual = new_mean(chunked, dim='x')  # raises the warning
print(actual)

print(actual.compute())  # does the computation correctly

Result
(0, 0)
/home/tnichol/anaconda3/envs/py36/lib/python3.6/site-packages/numpy/core/fromnumeric.py:3118: RuntimeWarning: Mean of empty slice.
  out=out, **kwargs)
<xarray.DataArray (t: 2)>
dask.array<shape=(2,), dtype=float64, chunksize=(2,)>
Dimensions without coordinates: t
(2, 3)
<xarray.DataArray (t: 2)>
array([0.147205, 0.402913])
Dimensions without coordinates: t

Expected result
Same thing without the (0,0) or the numpy warning.
Output of xr.show_versions()
(my xarray is up-to-date with master)

INSTALLED VERSIONS
------------------
commit: None
python: 3.6.6 |Anaconda, Inc.| (default, Oct  9 2018, 12:34:16)
[GCC 7.3.0]
python-bits: 64
OS: Linux
OS-release: 3.10.0-862.14.4.el7.x86_64
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_GB.UTF-8
LOCALE: en_GB.UTF-8
libhdf5: 1.10.2
libnetcdf: 4.6.1
xarray: 0.12.3+23.g1d7bcbd
pandas: 0.24.2
numpy: 1.16.4
scipy: 1.3.0
netCDF4: 1.4.2
pydap: None
h5netcdf: None
h5py: 2.8.0
Nio: None
zarr: None
cftime: <IP_ADDRESS>
nc_time_axis: None
PseudoNetCDF: None
rasterio: None
cfgrib: None
iris: None
bottleneck: 1.2.1
dask: 2.1.0
distributed: 2.1.0
matplotlib: 3.1.0
cartopy: None
seaborn: 0.9.0
numbagg: None
setuptools: 40.6.2
pip: 18.1
conda: None
pytest: 4.0.0
IPython: 7.1.1
sphinx: 1.8.2

The warning may have been fixed in the development version of dask by https://github.com/dask/dask/pull/5103
The computing on empty arrays thing is due to the new meta tracking in dask 2.0, used for tracking the underlying type of arrays in a dask array. Dask supports setting meta explicitly inside blockwise. We might consider exposing this from xarray as well.

I am not sure if this is related or not, but my dask array has a different shape before and after computing.
result
<xarray.DataArray 'reflectance' (y: 1082, x: 1084)>
dask.array<shape=(1082, 1084), dtype=uint16, chunksize=(1082, 1084)>
Coordinates:
    band     int64 1
  * y        (y) float64 9.705e+05 9.705e+05 9.705e+05 ... 9.673e+05 9.672e+05
  * x        (x) float64 4.889e+05 4.889e+05 4.889e+05 ... 4.922e+05 4.922e+05

[87]
# the shape of the xarray and numpy array do not match after conversion to numpy array, the x and y dimensions reappear
np.array(result).shape
(1082, 1084, 44)

See: https://stackoverflow.com/questions/57419541/how-to-use-apply-ufunc-with-numpy-digitize-for-each-image-along-time-dimension-o

Closed by #3660


Why does rationals_le use the concrete type nat?

I was surprised when unfolding rationals_le to see that it specifies that le x y if there is a fraction p/q made of nats such that y=x+p/q:
Instance rationals_le `{Rationals Q} : Le Q | 10 := λ x y,
  ∃ num, ∃ den, y = x + naturals_to_semiring nat Q num / naturals_to_semiring nat Q den.

Why not use any type that is an instance of Naturals?

I agree. It seems better to generalize this to general instances.


log(ddl(ticdc)): add more related logs for ddl on the critical path

What problem does this PR solve?

Issue Number: close #4858
What is changed and how it works?
add more logs
Release note
`None`.

/run-dm-compatibility-test
/run-dm-integration-test
/run-kafka-integration-test
/run-integration-test

/merge

/run-all-tests

/run-verify-ci

/run-all-tests

/run-dm-integration-test

/merge

/run-verify
/run-kafka-integration-test
/run-integration-test

/run-kafka-integration-test
/run-verify

/run-verify

Codecov Report

Merging #4900 (d60e9e4) into master (687e248) will increase coverage by 0.1792%.
The diff coverage is 60.1960%.

Flag
Coverage Δ

cdc
60.9569% <60.7883%> (+0.3583%)
:arrow_up:

dm
52.4735% <50.0000%> (+0.0039%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4900        +/-   ##
================================================
+ Coverage   56.1240%   56.3032%   +0.1791%
================================================
  Files           522        524         +2
  Lines         65325      65554       +229
================================================
+ Hits          36663      36909       +246
+ Misses        25094      25047        -47
- Partials       3568       3598        +30

/run-dm-compatibility-test
/run-dm-integration-test
/run-kafka-integration-test
/run-integration-test

/run-dm-compatibility-test
/run-dm-integration-test
/run-kafka-integration-test
/run-integration-test

/run-kafka-integration-test
/run-verify

/run-leak-test


Karpenter 1.0.x Helm Chart Update

Description
What problem are you trying to solve?
I am trying upgrade to karpenter 1.0.x, but in the latest release (1.0.1) in this repository,  it is using older helm chart i.e. 0.37.x. Changes are present in main branch but not in the latest release.
How important is this feature to you?
It is essential to upgrade to karpenter 1.0.x version.

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Closing as a duplicate of https://github.com/aws/karpenter-provider-aws/issues/5415.


Add tooltips

Codecov Report

Merging #153 (7d32edd) into main (0851be5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #153   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           43        43
  Lines          848       848
  Branches       348       348
=========================================
  Hits           848       848

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


Windows machine file copy readme update

Update the readme for the Windows machine file copy task to state that environment variables are no longer supported since agent version 1.91
Fixes #912

Looks fine.

Fixed with 8ffdfc3efdac52aeafde43a11e7cdce015ee530f


[Question] Make preview window layout configurable

Hi,
this line opens a new 12-line height window at the top. I'd prefer if it were created next to the current window instead. Basically like in gv. ;-)
I looked through the code but couldn't find a way to hook into it.
I'd prefer any wild hack before suggesting to introduce a new option, but I don't see any other way. According to g:plug_window maybe g:plug_preview_window? But just setting it to vertical pedit doesn't allow to set the window size..
Maybe it would be best to emit a User event at the end of s:preview_commit()?
if exists('#User#PlugPreview')
  doautocmd User PlugPreview
endif

Then, in my case, I could hook into it with:
autocmd User PlugPreview wincmd p | wincmd L | wincmd p

What do you think?

autocmd User PlugPreview wincmd p | wincmd L | wincmd p

Yeah we could do that except that it's a little more complicated with the default layout (split to the left). I suppose you use non-default g:plug_window? But anyway, if we don't have a choice but to add something to vim-plug, wouldn't it be simpler for the users that we just provide g:plug_preview_window (or g:plug_pwindow)?
let g:plug_window = '-tabnew'
let g:plug_pwindow = 'vertical rightbelow new'

" vs.

let g:plug_window = '-tabnew'
autocmd User PlugPreview wincmd P | wincmd L | wincmd p

I wish there was a native option for configuring the preview window, but it looks like &previewheight is all we have. Given the situation, I think it makes sense to add a way to configure the layout. I'll look into it.

Yea, User events are more general and for people with very specific wishes. From the point of view of a user who just wants to influence the preview window layout, using a simple variable is probably much more comfortable.
And yes, &previewoptions is definitely missing. :-)
Thanks for considering.

User events are more general and for people with very specific wishes

Do you think it's worth it? Anyway for g:plug_pwindow, the patch is pretty simple.
diff --git a/plug.vim b/plug.vim
index ff1827b..d769553 100644
--- a/plug.vim
+++ b/plug.vim
@@ -2140,9 +2140,14 @@ function! s:preview_commit()
     return
   endif

-  execute 'pedit' sha
-  wincmd P
-  setlocal filetype=git buftype=nofile nobuflisted modifiable
+  if exists('g:plug_pwindow') && !s:is_preview_window_open()
+    execute g:plug_pwindow
+    execute 'e' sha
+  else
+    execute 'pedit' sha
+    wincmd P
+  endif
+  setlocal previewwindow filetype=git buftype=nofile nobuflisted modifiable
   execute 'silent read !cd' s:shellesc(g:plugs[name].dir) '&& git show --no-color --pretty=medium' sha
   normal! gg"_dd
   setlocal nomodifiable

Do you think it's worth it?

No, either solution is fine. The patch looks good! :shipit:


Feature Request: Configurable Upchecks

Allow the configuration of "up checks" that will be called to confirm if a container is up during a deploy. The upcheck should be in the form of an http action and url and optional response substring.
deis upcheck:set get /heartbeat "lubdub"
deis upcheck:unset

This sounds related to #1829 and #3674. @jwaldrip what do you think of the discussions in those issues?

1829 is canary deploys, I am not talking about doing that here. This is just true zero downtime deploys. Simple and straight forward.

So to define this in terms of code/scope/work: you're proposing a per-app healthcheck that the controller would use to confirm a container is running beyond checking the controller state, correct?

@carmstrong the thing we see at brandfolder is an old version gets destroyed before a new one is up and running ..
so publisher needs a check at least for status 200 ( and possible a regex body matcher configureable via etcd ofc ) before mapping that stuff to etcd ,, that prevents the issues seen in #3674
the other problem is that the controller tells the scheduler to delete the old version as soon as the new version is there even if it did not boot up sucessfull .. we could write a state with publisher to etcd .. and clean them up with a job in the controller instead of destroying them directly ..
that way we might have 2 versions online for a few seconds but archice a true 0 downtime ..
id like to hear thougths on that

This is being tracked in #3813, which is a solution to address a number of issues for application health checks during deploy and publishing.


Add classes for simple measurements

There are several techniques that are addressed in MOVPE that I am copy-pasting in each user plugin, I would like to place it here.
This would allow also highr visibility of the techniques we started working on.
They contain so far very few metadata. We can indeed keep espanding them more&more
these are the ones:

AFM
LightMicroscopy
Reflectance Spectoscopy
Hall

For reflectance and hall I would provide a general class, pretty much like the other two, and also the Lakeshore and Laytec specific classes and parsers (in separate plugin but inheriting from these classes)
let me know what's your opinion @hampusnasstrom
@ka-sarthak are you working on some of these already?

@aalbino2 I do have a use case for PFM, which is a specialization of AFM. I will be working on it in the coming weeks. Btw, it might be a good opportunity to check if your AFM class can be used as a base section for PFM.
In general, I do agree that we should move/develop more measurement-related schemas in this repo while making sure they are generic enough.


FFmpeg not installed | Request to be able to contribute to the project

Good day,
Thank you for the amazing work done on this project
I had an issue that was obvious after some debugging, and which should have been obviosue to me but I was running on a fresh os and did not think of installing FFmpeg
The error was suppressed/ignored on line 297 only after altering the code to the following did I get this error.
videoProbe, videoProbeError := ffmpeg.Probe(abs)

if videoProbeError != nil {
   log.Printf("Error: %v", err)
   return Video{}, fmt.Errorf("fatal error: %w", err)
}

I also added GetDuration as a method as it was/is useful for me to know the video's duration during runtime.
func (V Video) GetDuration() float64 {
	return V.duration
}

I am fairly new to go and have about 7 years of experience with Python and I am building a small project to cut my teeth on the Golang way of doing things. Would love to be able to contribute to the project as i am also looking for OSS projects to contribute towards.

AndreaNicola also has this in the pull request https://github.com/AndreaNicola


feat: Add ClickHouse exporter

Proposed Change
Adds ClickHouse exporter
Checklist

[ ] Changes are tested
[ ] CI has passed

Re-running the build-windows workflow, not sure it'll fix the issue though:
2023/10/30 16:25:27 failed to evaluate licenses: failed to fetch: fork/exec C:\hostedtoolcache\windows\go\1.20.10\x64\bin\go.exe: The filename or extension is too long. (output: )

PR looks good, but we'll want to figure out what to do about this check that's failing now (assuming it wasn't a fluke)

Ended up swapping out the windows image for a ubuntu image for build_windows, windows can't handle how long the command line is for license scanning anymore.


Upgrade Ruby from 2.6.6 to 2.7.x

Unfortunately, I can't do the upgrade myself within my Dockerfile because I'm using it on a Synology NAS and can't really access the core from the outside.
But I also think in general that it makes sense to close the known security gaps in this way

Synology ships with this?

Doesn't look like an upgraded Ruby 2.6.9 package has been released yet. https://launchpad.net/~brightbox/+archive/ubuntu/ruby-ng/+index?batch=75&memo=75&start=75
However according to Ruby, the maintenance for 2.6 is ending next month, so probably better to move to Ruby 2.7.2+ since thats the latest version that redmine 4.2 supports.
https://www.redmine.org/projects/redmine/wiki/RedmineInstall

Pushed release 4.2.3-1
@sameersbn Build needs to be kicked off

done!

Hi there,
yes it works on Synology. But not directly shipped from them. You can install Docker and then get the dockerfile from the docker sourcepool.
Actually there are known problems in Redmine with custom fields under ruby 2.7.x! It will be fixed in 4.1.6 and 4.2.4
Thanks and greetings

Actually there are known problems in Redmine with custom fields under ruby 2.7.x! It will be fixed in 4.1.6 and 4.2.4

Well thats annoying, I checked https://www.redmine.org/projects/redmine/wiki/RedmineInstall which only mentions avoiding 2.7.0/2.7.1
Found your issue. https://www.redmine.org/issues/36584
Seems like we should revert this change until 4.2.4 is released...

Posted 4.2.3-2 that reverts back to Ruby 2.6.6.  Hopefully redmine 4.2.4 will fix the issue with Ruby 2.7

I hope also ;)

Should leave this open since its not actually resolved yet

Redmine 4.2.4 has been released. Will update docker image
https://www.redmine.org/projects/redmine/wiki/Changelog_4_2

Well, i guess not quite yet, they posted the changelog but haven't uploaded the 4.2.4 tarball.

So not sure whats happening over there but who ever did the release attached it to the Downloads wiki page instead of uploading it to the normal releases url. https://www.redmine.org/releases/.
So our standard download link doesn't work.  Posted an issue to redmine.org, perhaps they are having some server issues and this is a temporary situation.
https://www.redmine.org/issues/36703

Now, it should be online...

Yup, sorry was busy last week.  Looking at it now
On Tue, Mar 8, 2022 at 10:33 AM CSCSxx @.***> wrote:

Now, it should be online...
—
Reply to this email directly, view it on GitHub
https://github.com/sameersbn/docker-redmine/issues/488#issuecomment-1061905826,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABK2RGKHASPAWEMVPGM2RIDU65XLXANCNFSM5N42RIAQ
.
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 modified the open/close state.Message
ID: @.***>

--
Jonathan Cormier
Software Engineer
Voice:<PHONE_NUMBER> x222
http://www.CriticalLink.com http://www.criticallink.com/
6712 Brooklawn Parkway, Syracuse, NY 13211
https://www.linkedin.com/company/critical-link-llc
https://twitter.com/Critical_Link

Just pushed 4.2.4 and 4.1.6.  However, looks like only "lastest" build got triggered.

Tags 4.2, 4.2.4, and 4.1.6-r2 have been pushed


[Feature Request] Allow specific script type in pipeline.js

Sails version: master
Node version: 4.2.6
NPM version: 3.5.2
Operating system: ubuntu lts 16.04

When using ReactJS with sails, I came across the issue, that I was not able to make use of sails native pipeline.js functionality when working with clientside javascript files, because ReactJS requires a custom script tag like  <script type="text/babel"></script>
I would like to here your opinion how this can be handled?
An approach might be, allowing to customize the script tag type in pipeline.js.
var jsFilesToInject = [
  'js/dependencies/sails.io.js',
  'js/dependencies/**/react.js|script/babel',
  'js/dependencies/**/*.js',
  'js/**/*.js',
];

Please let me know, how it can handled best.
I'm also willing to create a PR for this

ok, fixed!

@xhallix open to doing something to improve this in grunt-sails-linker -- although I'd prefer to solve this without introducing any new string notation that requires regexp parsing.
So what do you think of this:
var jsFilesToInject = [
  {
    src: 'js/dependencies/sails.io.js',
    environment: 'production',
    headers: {
      foo: 'bar'
    }
  },
  {
    src: 'js/dependencies/**/react.js',
    type: 'script/babel'
  },
  'js/dependencies/**/*.js',
  'js/**/*.js',
];

^^ doing it that way also allows us to more easily support the newer, easier way (Sails 0.12 and up) of configuring the sails.io.js client.  See http://sailsjs.org/documentation/reference/web-sockets/socket-client#?basic-configuration-using-html-attributes for more info. Technically still leaves a gap as far as the suggested strategy for sending through the CSRF token, but that's ok (and you can always do that part programmatically).
details

if possible, all of this implementation should live in grunt-sails-linker (if not, let me know and we'll figure it out-- may just need to make breaking changes to the way the task is configured.  Now is a good time to do it.)
other than src (which needs the standard glob support), provided they are strings, the other properties in the dictionaries can just be dumbly converted to HTML attributes, with normal escaping via _.escape() from Lodash 3.10.1 (note that normal HTML escaping works inside html attrs in all cases UNLESS attrs are not double-quoted.  In this case, we know they'll always be properly double-quoted since we're injecting the script tags ourselves.  So yeah, anyway, we can just _.escape() whatever's configured on the RHS.)
if a non-string is provided on the RHS, then it should be JSON.stringify()ed before being _.escape()d and injected  (the only way this should ever throw is in the case of circular objects, which really shouldn't be hanging out in your pipeline.js file anyway.   Also worth noting that it'll cause undefined array items and dictionary keys to get stripped, but that's ok too.)
if the glob expression in src matches >1 file, then each script tag injected should include all of the other HTML attributes as well
For consistency, we should do the same thing for <link> tags.

plan
@xhallix definitely open to a PR on this!  If you can take a pass at doing the necessary work in grunt-sails-linker, I'd be happy to review (and I can take care of making the necessary changes to the generators and docs-- I've been in there a bunch anyway)

@mikermcneil
At first, I really appreciate that most of the issues and features suggested here, are reviewed and answered in thoughtfully.
I like the approach you make and I will go ahead an make a PR as a proposal. Please do not expect the PR within the next week ;)

landed a pr as a first draft https://github.com/sailsjs/grunt-sails-linker/pull/20

@xhallix 🙇  https://github.com/sailsjs/grunt-sails-linker/pull/20#issuecomment-261687528

@mikermcneil https://github.com/sailsjs/grunt-sails-linker/pull/20#issuecomment-265066093 :+1:


redirect IP address to domain name

What would you like to have changed?

redirect requests to an IP address to the domain name.
Right now, with a site powered by certmagic, a request like

curl -vvIL http://<My-IP-address>

results in;
* Connected to <My-IP-address> (<My-IP-address>) port 80 (#0)
HEAD / HTTP/1.1
Host: <My-IP-address>
User-Agent: curl/7.58.0
Accept: application/json, */*

HTTP/1.1 301 Moved Permanently
Connection: close
Content-Type: text/html; charset=utf-8
Location: https://<My-IP-address>/
Date: Sat, 30 Nov 2019 20:24:48 GMT

* Closing connection 0
* Issue another request to this URL: 'https://<My-IP-address>/'
*   Trying <My-IP-address>...
* TCP_NODELAY set
* Connected to <My-IP-address> (<My-IP-address>) port 443 (#1)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*   CAfile: /etc/ssl/certs/ca-certificates.crt
*   CApath: /etc/ssl/certs

* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.3 (IN), TLS alert, Server hello (2):
* error:14094438:SSL routines:ssl3_read_bytes:tlsv1 alert internal error
* stopped the pause stream!
* Closing connection 1
curl: (35) error:14094438:SSL routines:ssl3_read_bytes:tlsv1 alert internal error

This proposal is that such a request should succed by been redirected to https://my_domain.com
Why is this feature a useful, necessary, and/or important addition to this project?

there maybe clients that perform dns lookup of a domain name and then issue requests to the returned IP address directly
it sounds like the right thing to do
it is what other webservers do, here is similar request been issued to google;

dig @<IP_ADDRESS> google.com

;; ANSWER SECTION:
google.com.		140	IN	A	<IP_ADDRESS>

curl -vvIL http://<IP_ADDRESS>

 Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 80 (#0)
HEAD / HTTP/1.1
Host: <IP_ADDRESS>
User-Agent: curl/7.58.0
Accept: application/json, */*

HTTP/1.1 301 Moved Permanently
Location: http://www.google.com/

* Connection #0 to host <IP_ADDRESS> left intact
* Issue another request to this URL: 'http://www.google.com/'
* Connected to www.google.com (<IP_ADDRESS>) port 80 (#1)
HEAD / HTTP/1.1
Host: www.google.com
User-Agent: curl/7.58.0
Accept: application/json, */*

HTTP/1.1 200 OK
Date: Sat, 30 Nov 2019 20:30:02 GMT
Expires: -1
Cache-Control: private, max-age=0

What alternatives are there, or what are you doing in the meantime to work around the lack of this feature?

N/A

Please link to any relevant issues, pull requests, or other discussions.

Why not just implement this in your own HTTP handler?


remove gifs

Appears a tad blurry on my screen, but that might be a size issue. I'd say merge it and we can address blurriness later if this truly is the issue.

Appears a tad blurry on my screen, but that might be a size issue. I'd say merge it and we can address blurriness later if this truly is the issue.

i just took a screenshot of the gif lol, so that's why its poor quality. we should absolutely come back to this. cc/ @meagancojocar


example3 test_evaluation function is not testing against the whole validation data

Hello developers,
first of all thank you a lot for this amazing software.
I am not sure if the test_evaluation function in examples/03_mnist_training.py is evaluating the validation data as intended.
I downloaded the current version and changed the EPOCHS parameter to 3 for a faster training time. The test_validation function prints that the "Number Of Images Tested = 157" although the validation data includes 10000 images. In the current implementation the validation is only done for the last image of the batch. Is this intended?

Hi @tz-ac,
Thanks, we are really glad you are enjoying this toolkit!
Thanks for bringing this issue up!
The DataLoader function returns batches of the validation set of length len(dataset) / batch_size and in the validation_data loader function we have specified batch_size = BATCH_SIZE which is 64. So the 157 * 64 accounts for the 10k images of the dataset with the last batch size a bit smaller than the rest.
You are right, given the way the examples is structured the validation is called only once at the end of the training and so only validated for the 157 images. You can change the batch_size parameter in the validation_data to 1 and that would run the validation over the whole dataset but still only once after the training for the number of epochs.
Another option is to include the validation function in the epoch loop (leaving batch_size = 64) so you get an estimate of the network performance per epoch.

Hi @Fabio-83,
thanks for your reply.
Reducing the batch_size of the validation_data is a nice way to validate all images in the validation data set.
From your answer I understand that it is the intendent behavior to only validate one image per batch.
Can you please elaborate a bit on the reason for the for loop in line 169:
for i in range(len(labels)):
    image = images[i].view(1, INPUT_SIZE)
    with torch.no_grad():
        pred = model(image)

Why is it necessary if only the last label of the validation batch is compared in line 179?
validation_label = labels.cpu().numpy()[-1]
if validation_label == predicted_label:
    predicted_ok += 1

Hi @tz-ac,
ohh, I may have not completely grasped your point earlier on.
I see what you mean now, yes that for loop may have some issues.
I don't think it was intentional, thanks for pointing this out, we will update the test_evaluation function.
How about making it more similar to what we use in some of the other examples:
def test_evaluation(model, val_set):
    """Test trained network

    Args:
        model (nn.Model): Trained model to be evaluated
        val_set (DataLoader): Validation set to perform the evaluation
    """
    # Setup counter of images predicted to 0.
    predicted_ok = 0
    total_images = 0

    model.eval()

    for images, labels in val_set:
        # Predict image.
        images = images.to(DEVICE)
        labels = labels.to(DEVICE)

        images = images.view(images.shape[0], -1)
        pred = model(images)

        _, predicted = torch.max(pred.data, 1)
        total_images += labels.size(0)
        predicted_ok += (predicted == labels).sum().item()

    print('\nNumber Of Images Tested = {}'.format(total_images))
    print('Model Accuracy = {}'.format(predicted_ok/total_images))

This way it should validate over the whole validation set:
Number Of Images Tested = 10000
Model Accuracy = 0.9347

Hi @Fabio-83,
no problem. I think my question was not the clearest one.
I tested your suggested function and it seems to work for me.
I think it is a good solution especially if it is more similar to the other examples.
Thanks for your help and clearification.


Update to Application terminology

This will be followed by a rename of the repository :+1:
Signed-off-by: hasheddan<EMAIL_ADDRESS>

There are also a number of downstream consumers (scripts, manifests) that would need to be updated as well.  We can brainstorm about where those may be :)


Add NtQuerySystemInformation Hook

Some malware samples hide themselves from process lists by hooking the NtQuerySystemInformation function. I have not written this myself, but if any ideas are needed, this is one of them. I do not have my own code to contribute for this, so I will provide you with a link for reference.
Windows API Hooking - Hide Process from Task Manager tutorial

Good idea. We will review this at a later time.

This has been reviewed. This requires a DLL. This has been accepted and will be implemented in a later version.


Develop

Coverage decreased (-0.3%) to 81.711% when pulling 81eb16499461025278e13f4ba423af4d4b7f384a on develop into 1c519bc431624b0a3a70ea5ff403e7eb57c02d60 on qa.


Enhancement 394

This change is

nit: Please provide a descriptive PR title and description; even if all the relevant information is in the linked issue its better to repeat it.

Pull Request Test Coverage Report for Build 864

7 of 12 (58.33%) changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 31.798%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controller/controller.go
0
1
0.0%

pkg/apis/tensorflow/v1alpha1/zz_generated.defaults.go
4
6
66.67%

pkg/apis/tensorflow/v1alpha1/zz_generated.deepcopy.go
0
2
0.0%

Totals

Change from base Build 861:
0.05%

Covered Lines:
580

Relevant Lines:
1824

💛  - Coveralls

/ok-to-test

Travis CI returns some errors:
pkg/apis/tensorflow/v1alpha1/types.go:1::warning: file is not goimported (goimports)
pkg/trainer/tensorboard.go:1::warning: file is not goimported (goimports)
pkg/trainer/training.go:1::warning: file is not goimported (goimports)
pkg/util/k8sutil/k8sutil.go:1::warning: file is not goimported (goimports)

If you have no idea to fix them, please let me know. I'm glad to help you :)

thanks! ditto for @jlewi 's comment on using complete sentences.

Thanks @jlewi for suggesting the ways of proper commenting, I will change the accordingly.
@gaocegege I have no idea to fix them. I'll be highly obliged if you could help me in this.

@gaocegege Assigning this PR to you. Please merge after you've approved.

@AK-ayush Please address the comments above about the comments https://github.com/kubeflow/tf-operator/pull/396#discussion_r169656746 and I will take a look at Travis CI

/assign @gaocegege

@AK-ayush Will you be able to address the comments?

Sorry for late reply. I will address the above comments and working on it now.

@AK-ayush Any chance you'll be able to update this PR this week?

hi @jlewi  sure, I will finish it within this week only.

I have deleted this PR because there were too many merge conflicts. I have opened a new PR.


feat: Add Popover Close and minor type improvment

I needed to close the popover after the user clicks a button in the popover content.  That button would fire an event and does something.  However it would not close the popover.  By adding the existing popover close as a child its now possible to do this.
        <Popover.Root let:ids>
            <Popover.Trigger asChild let:builder>
                <Button builders={[builder]} variant="outline">
                    <Upload/>
                    Upload PDF
                </Button>
            </Popover.Trigger>
            <Popover.Content>
                <Popover.Close asChild let:builder>
                    <FileSelect builder={builder}/>
                </Popover.Close>
            </Popover.Content>
       </Popover.Root>

The inside my FileSelect component I can just use a button that does the file upload and it would close the popover.
	<Input type="file" on:change={handleFileChange} />
	<Button class="" on:click={handleFileUpload} disabled={isUploading} builders={[builder]}>
			Upload
	</Button>

Some minor improvement to input component useful to export FormInputEvent for other TS code to be properly typed.
const handleChange = (event: FormInputEvent<Event>) => {...}

@huntabyte Thank you too for maintaining this code base.   Just checked the warning on the build about prettier, I did a quick npx prettier on my local checkout.  It should now be ok, seems unrelated carousel components has some small issue as well.


Dependency

Hi, i got this error:
[Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException]
The service "egulias.security_listeners_debug" has a dependency on a non-existent service "sensio_framework_extra.security.listener".

Hi @clubdesarrolladores
Which version of Symfony are you using? It could be that latest Symfony had removed this service.

I had the same issue, but that was not the version of Symfony that was the culprit.
I had:
sensio_framework_extra:
    security:
        annotations: false

which obviously prevented sensio_framework_extra.security.listener to be registered.
Turning that config to true resolved the issue.


change location of custom sections

They are put before the 'dependency' section now instead of the end

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (a8bbfd5) 49.86% compared to head (d4be65f) 50.52%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
+ Coverage   49.86%   50.52%   +0.65%
==========================================
  Files           7        7
  Lines         375      384       +9
==========================================
+ Hits          187      194       +7
- Misses        188      190       +2

Files
Coverage Δ

github_changelog_md/changelog/changelog.py
29.31% <100.00%> (+0.86%)
:arrow_up:

github_changelog_md/helpers.py
95.83% <71.42%> (-4.17%)
:arrow_down:

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