Create Document table for storing original loaded documents

We should have a document store, in addition to storing chunked/embedded passages.

Relates to something I've been thinking about - have the agent write "articles" summarizing what it knows about certain topics, and being able to update.
Currently IIUC there's not really a memory consolidation function, where old or outdated memories can be discarded. Eg if today I tell the agent is green but tomorrow I tell it, sorry I made a mistake yesterday, the sky is blue, ideally that old memory would be pruned somehow

Now that #708 is merged, this would be a great issue to get a community contribution on. The issue can be broken down in the following todos:

[ ] Add a Document data type to data_types.py that stores the entire original document text and its embedding and metadata
[ ] Add a doc_id field to Passage to associate passages with documents
[ ] Add a TableType.DOCUMENTS to the agent_store/storage.py and support document storage for chroma and postgres
[ ] In cli_load.py, when ingesting passages, also store the associated document and doc_id
[ ] Add a new function that allows the agent to retrieve the document associated with a passage or search over documents


Disambiguate "Keybindings" reference; add a few example keys

Summary of the Pull Request
It took me some trial and error to figure out how to bind "ctrl+pgdn".  This patch adds a small amount of documentation for that key combination and a couple others (both of which I tested).
Since I was here anyway, I suspect the use of the word "keybinding" was unintentional, as the thing it refers to is later referred to as a "command".  So, I tidied that up.
Tested by visual inspection.

References
None

PR Checklist

[N/A] Closes #xxx
[CHECK] CLA signed. If not, go over here and sign the CLA
[N/A] Tests added/passed
[N/A] Requires documentation to be updated
[READY] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments
N/A

Validation Steps Performed
Validated by manual inspection.
Comments
The new terminal (combined with WSL) is a welcome improvement that makes developing on Windows about 5x easier.  I am also glad to learn that I can customize the keybindings, as I am an avid emacs user.  Thanks for all your efforts!

Thanks for the additions @zadjii-msft !  Via the github interface, I committed those changes and then visually inspected the result of those three commits (viewable at https://github.com/ragerdl/terminal/blob/dev/ragerdl/add-keybinding-examples/doc/cascadia/SettingsSchema.md ).
When the PR is accepted, it's probably worth squashing the commits.  I thought about creating a new PR (or updating this PR if possible) that has the commits squashed, but I didn't want to mess with the github flow.  Thanks again~

@bitcrazed

Approved with @zadjii-msft's suggested changes.

Which then were added, so I'm gonna take that as an "approved" and actually just merge this (esp. since it's a docs PR)


Allow for easy way to handle inout arguments

Problem
Unfortunately dear-imgui API uses inout arguments very often. Python does not allow for such calling convention. So it is very hard to wire together imgui windows and widgets in the same way like in C++.
General idea
We could introduce some abstract base class that could be used in place of existing inout argument handling:
import abc

class InOutABC(object):
    __metaclass__ = abc.ABCMeta

    @abc.abstractmethod
    def set(self, value):
        pass

    @abc.abstractmethod
    def get(self):
        pass

    @classmethod
    def __subclasshook__(cls, C):
        if cls is InOutABC:
            if (
                any("set" in B.__dict__ for B in C.__mro__) and
                any("get" in B.__dict__ for B in C.__mro__)
            ):
                return True

        return NotImplemented

Usage
Then user could create his own signal-like class that would handle his inout argument data and could be used in imgui function calls without inconvenient tuple unpacking and multiple variable assignments:
class InOutWire(object):
    def __init__(self, value):
        self._value = value

    def set(self, value):
        self._value = value

    def get(self):
        return self._value

Such InOutWire could be used to easily glue together different widgets:
checked = InOutWire(False)

while True:
    imgui_ctx.new_frame()

    if imgui.checkbox("c1", checked)[0]:
        print "c1.clicked", checked.get()
    if imgui.checkbox("c2", checked)[0]:
        print "c2.clicked", checked.get()
    # ...

User could even create general objects to provide auto-generated inout arguments without the need to:
from collections import defaultdict

wires = defaultdict(lambda: Wire(False))

while True:
    imgui_ctx.new_frame()

    if imgui.checkbox("c1", wires['checked'])[0]:
        print "c1.clicked", wires['checked'].get()
    if imgui.checkbox("c2", wires['checked'])[0]:
        print "c2.clicked", wires['checked'].get()

Implementation details
Having the abstract base class InOutABC we can check if provided possible inout arguments have the compatible interface (set()/get() methods). We need to do that test in every function that will allow for such feature. Example implementation:
def checkbox(str label, state):
    cdef cimgui.bool inout_mode = isinstance(state, InOutABC)
    cdef cimgui.bool inout_state = state.get() if inout_mode else state

    result = cimgui.Checkbox(_bytes(label), &inout_state)

    if inout_mode:
        state.set(inout_state)

    return result, inout_state

Sadly this will require a lot of boilerplate and code changes. The ideal solution would be to have kind of decorator that can translate between ordinary/inout arguments. Inside of wrapped functions we could then always expect either InOutABC-compatible objects or simple flat arguments.
The trouble is that right now we allow arguments that are "inout" in the C++ layer to be both positional and keyword. In such situation I don't know how do provide such decorator that would manipulate arguments without introspecting function signatures with inspect. This would be easier if we would assume that inout arguments are always keyword-only. Good news is that Cython allows to have keyword-only arguments even on Python 2 and even without default values.
Example implementation and usage:
from functools import wraps

def allow_inout(kwarg_name, result_pos=1):

    def wrapper(func):

        @wraps(func)
        def wrapped(*args, **kwargs):
            kwarg = kwargs.get(kwarg_name, None)

            inout_mode = isinstance(kwarg, InOutABC)

            if inout_mode:
                kwargs[kwarg_name]=kwarg.get()

            result_tuple = func(*args, **kwargs)

            if inout_mode:
                kwarg.set(result_tuple[result_pos])

            return result_tuple

        return wrapped

    return wrapper

@allow_inout('state')
def checkbox(str label, *, state):
    cdef cimgui.bool inout_state = state
    return cimgui.Checkbox(_bytes(label), &inout_state), inout_state

Further improvements
Common idiom in imgui is to always check result of widget call using if to process user interaction. For every function that defines a widget and uses inout arguments we return a tuple. This means that some widgets will require result value indexing and some will not e.g:
if imgui.button("some button"):
    print("user interaction")

if imgui.checkbox("some checkbox", False)[0]
    print("user interaction")

This makes the API usage inconsistent. We could improve this by replacing simple return tuple with object that will take into account only the value during bool test e.g:
class InteractionTuple(tuple):
    def __bool__(self):
        return bool(self[0])

The downside: changes user expectation on tuple truth value. Other way around is to inverse whole problem. We could replace every return value that means user interaction or widget visibility with an object that clearly describes what are expectations about "True value". Example:
class InteractionTuple(tuple):
    def __bool__(self):
        raise ValueError

    @property
    def clicked(self)
        return bool(self[0])

class VisibilityTuple(tuple):
    def __bool__(self):
        raise ValueError

    @property
    def visible(self)
        return bool(self[0])

    @property
    def expanded(self)
        return bool(self[0])

# usage

if begin("my window").visible:  # alternatively: .expanded
    ...
    if imgui.checkbox("my checkbox").clicked:
        ...

I have added this to 0.0.1 milestone to ensure that we make a decision about this feature before we will ship first official release.

I have realised that using keyword-only arguments here is not a good solution:

it will make force users to use excessively verbose function calls
it would have to either affect all arguments with default values or be very inconsistent

The best if we could keep the core Python API as close to C++ API as possible. This is already the first reason why I have proposed this InOutABC interface. Forcing users to use keyword-only arguments will make a lot harder to port C++ examples to Python.
Fortunately I have found a simple way to handle both inout args and kwargs in a form of suggested decorator. Here is the modified code:
def allow_inout(arg_pos, kwarg_name, result_pos=1):
    def wrapper(func):
        @wraps(func)
        def wrapped(*args, **kwargs):
            try:
                kwarg = args[arg_pos]
                args = args[:arg_pos] + args[arg_pos+1:]

            except IndexError:
                kwarg = kwargs.get(kwarg_name, None)

            inout_mode = isinstance(kwarg, InOutABC)

            if inout_mode:
                kwargs[kwarg_name]=kwarg.get()

            result_tuple = func(*args, **kwargs)

            if inout_mode:
                kwarg.set(result_tuple[result_pos])

            return result_tuple

        return wrapped

    return wrapper

I still need to reconsider if maybe it would be better to handle inout kwargs explicitly in every such function. Explicit inout handling will be less expensive in terms of performance but would require more code changes and maintenance. We need to decide what is more important: more expressive and shorter code or code that executes faster.
Anyway, such change does not have to be backwards incompatible so we don't have to make any decision before the first release. I move this feature to next milestone.

I did some experiments in feature/8-inout-arguments branch and it doesn't seem to be as straightforward as I initially thought. I'm moving this to next milestone because I want to finally release 1.0.0


Properly check for or automatically install the right version of stem

We now depend on a feature added in stem 1.6.0-dev. See my commit message.
I can't quickly figure out how to specify in setup.py that we need to download stem from git.tpo.
It would be great if we could convince @atagar to release 1.7.0 :)

This is how pip can be configured to install from git:
https://stackoverflow.com/questions/16584552/how-to-state-in-requirements-txt-a-direct-github-source#16584935
Apparently this is possible in setup.py as well:
https://stackoverflow.com/questions/3472430/how-can-i-make-setuptools-install-a-package-thats-not-on-pypi

While I don't like having to have pip git clone stem (or have the user git clone stem themself), it has been working and should continue to work until there's a new version of stem.


[Question] Best way to pass extra params to MailEvent

I'm looking at the code and can not find any method for setting MailEvent params. Most obvious is to extend MailService and then get it by $e->getMailService()->getEventParams() or override createMailEvent() ($e->getParams()) but how do You think? Perhaps option would be useful enough to integrate it into the code?

Can you give me a use case in which that makes sense?
I mean, the MailEvent is always triggered by the MailService, you don't have to do anything.
If you need your listener to get extra params, just inject them to it, there's no need to pass the params to the MailService, so that it passes them to the MailEvent, so that it passes them to the listener.

Lets say I have UserMailListener which extend AbstractMailListener. Inside onSendError I would like to do some logging and inlcude data from UserService (which is also responsible for sending emails). Data could be for example UserEntity.

And how would you implement it if it was possible to pass data to the
MailEvent?
2016-11-13 12:47 GMT+01:00 xorock<EMAIL_ADDRESS>
Lets say I have UserMailListener which extend AbstractMailListener. Inside
onSendError I would like to do some logging and inlcude data from
UserService (which is also responsible for sending emails). Data could be
for example UserEntity.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/acelaya/ZF-AcMailer/issues/137#issuecomment-260181266,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACl-ZBg0YpzzPPhDk10Uq4JgwQ4acHG5ks5q9vjagaJpZM4Kwqy1
.

--
Alejandro Celaya Alastrué
http://www.alejandrocelaya.com

For example:
protected function createMailEvent($name = MailEvent::EVENT_MAIL_PRE_SEND, ResultInterface $result = null)
    {
        $event = new MailEvent($this, $name);
        if (isset($result)) {
            $event->setResult($result);
        }
        if (!empty($this->mailEventParams)) {
            $event->setParams($this->mailEventParams);
        }
        return $event;
    }

UserService.php
$mailService is AcMailer\Service\MailService instance passed by factory.
UserService::someMethod
$this->mailService->setMailEventParams(['user' => $userEntiry]);
$this->mailService->send();

That's too specific and would couple the MailService with certain email, while it should be stateless.
Imagine you set the params for an event and send the email.
If later on you send another email, the same params will be injected in the MailEvent, producing errors hard to debug.
Instead, you should define a listener (the UserMailListener in your case) that's capable of finding out the user by itself, for example by getting and EntityManager injected (or some other service that allows database data fetching).
The MailEvent already gives you access to the MailService, which gives you access to the Message. You can find the user by its email.
Another approach would be to log something in the UserService just before calling the MailService ("Trying to send email to user with ID 5").
Then the UserMailListener could catch the result and log "Email properly sent to<EMAIL_ADDRESS>Both lines will be in a row.
And finally, if you want to handle errors only, a good approach would be to make the UserMailListener to throw an exception onError. Then you can catch the exception in the UserService, where you already know the user.

As always, there are many ways to resolve this problem. I forked Your repo https://github.com/xorock/ZF-AcMailer/commit/8a2f4f35b6f66b2ba2088aec62411eac80d3bec7 (sorry for 2 commits, I did in the browser).
Mail event parameters are always cleared after mail was send, so double execution will not trigger MailEvent with same parameters. onSendError was only an example. Parameters might be needed for all events. EntityManager is just another layer of abstraction and not everyone will use it. I can't fetch user from storage by email, because when there was error, entity was not saved to storage. In my code, UserService throws exceptions on error, Controller catches them and do some other actions (including general logging). Code depends also on db transactions so all or nothing. Through the code as I showed, everything has single responsibility.
Because You can use MailService as a service, as controller plugin and in many other ways, event parameters might be everything - starting from UserEntity, Request object, some other service and they are strictly bound to triggered event. I think it's a good way. If You wish I can create pull request. And if not, maybe someone consider such approach to be useful in the future.


update a few devtools things

Updates a few unrelated sections of the devtools docs. Separate PRs for each update seemed like a waste of time. See the commits for descriptions of the changes.

Epic PR, looks great! LGTM.


WIP Review: Minor edits to install-w-machine.

Fixed typos and minor formatting issues.

LGTM
ping @moxiegirl

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "new-branch" git@github.com:rolfedh/swarm.git somewhere
$ cd somewhere
$ git rebase -i HEAD~3
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@nishanttotla @chanwit Thanks for your contributions! I've updated the file as requested.

Thanks @rolfedh. LGTM. We'll have to wait for @moxiegirl to approve too.

Thanks @rolfedh
LGTM

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "new-branch" git@github.com:rolfedh/swarm.git somewhere
$ cd somewhere
$ git rebase -i HEAD~3
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@rolfedh can you please squash the 2 last commit, there is still a commit without signature.

@vieux do you know why the check for signed commits isn't showing up?

My @docker.com email address went offline recently. I wonder if that, or updating my signature to my @gmail.com address today had any impact.

@vieux Could you ask someone with Git-fu resolve the signature issue. I'm concerned about messing things up.

@rolfedh @moxiegirl will take care of it

@nishanttotla Not sure the motivation for the change but the DCO check was removed from Travis: https://github.com/docker/swarm/pull/746


Add clamping functionality directly in CZML and Entity API

The Clamp to 3D Tiles demo in https://github.com/AnalyticalGraphicsInc/cesium/pull/6934 manually clamps the entity's position to a tileset based on a scene.clampToHeight call. It would be convenient to offer this functionality from CZML and the entity API.

I imagine the most straightforward way to do this would be to extend the HeighReference enum to cover 3D Tiles (probably always terrain and 3d tiles at the same time)

Also requested in https://github.com/CesiumGS/cesium/issues/11305.

Hi from https://github.com/CesiumGS/cesium/issues/11305.
I ended up using a Raycast for everything, which looks like:
    // Get position from lat/lon
    const position = Cesium.Cartesian3.fromDegrees(lng, lat, 10000); // Assume initial altitude of 10,000m

    // Cast a ray from position to the center of the Earth
    const ray = new Cesium.Ray(position, Cesium.Cartesian3.negate(Cesium.Cartesian3.fromDegrees(lng, lat), new Cesium.Cartesian3()));

    // Intersect the ray with the tileset
    const intersection = viewer.scene.pickFromRay(ray, [tileset]);

However, yes, I would love using a direct implementation, and not a hacky workaround.

From Sean in https://github.com/CesiumGS/cesium/issues/8606:

I'm a little worried that scene.clampToGround won't scale super well. Terrain keeps triangle data in CPU memory, wonder if we need to do the same for 3D Tiles at some point. And then optimize ray/triangle intersection tests.

Hello! Any news on this implementation? @ggetz
Thanks!

We need to get the altitude of a point from 3dTiles

@ddomit Are you looking to clamp a point to the surface of 3D Tiles, or sample the height directly?

I am looking to do both actually :)

We need to get the altitude of a point from 3dTiles

@ddomit Are you looking to clamp a point to the surface of 3D Tiles, or sample the height directly?

Any new features to implement this?


TypeError: __wbindgen_malloc is not a function

Disclaimer: This is the first time ever for me to include WASM into my project, so please forgive me if I am making a beginner's mistake.
I managed to include wasm-brotli into my Webpack project, but get this error during runtime:
wasm_brotli_browser.js?305f:14 Uncaught (in promise) TypeError: _wasm_brotli_browser_bg_wasm__WEBPACK_IMPORTED_MODULE_0__.__wbindgen_malloc is not a function
    at passArray8ToWasm (wasm_brotli_browser.js?305f:14)
    at compress (wasm_brotli_browser.js?305f:37)
    at _callee3$ (Encoder.js?1c5a:61)
    at tryCatch (runtime.js?96cf:63)
    at Generator.invoke [as _invoke] (runtime.js?96cf:293)
    at Generator.eval [as next] (runtime.js?96cf:118)
    at asyncGeneratorStep (asyncToGenerator.js?1da1:3)
    at _next (asyncToGenerator.js?1da1:25)
    at eval (asyncToGenerator.js?1da1:32)
    at new Promise (<anonymous>)
passArray8ToWasm @ wasm_brotli_browser.js?305f:14
compress @ wasm_brotli_browser.js?305f:37

Is this a bug in wasm.brotli or am I doing something wrong?
wasm-brotlis is called with await compress(data), where data is an Uint8Array. The app runs on vanilla Chrome.

This was a Webpack issue. I solved it by installing this package:
https://github.com/zhouzhi3859/wasm-module-webpack-plugin


Add a note how to hide the wrap-guide

Ideally this would be a option in the settings, but maybe a note will already help too.

Refs: https://github.com/atom/settings-view/issues/790 + https://github.com/atom/atom/issues/11782. There are many more issues and discussions about this, but these are just the recent two.

There is a typo. "I" should be "If". Change it to "If you would like to"?

Yay!! :bow: :tada:

Made a little change to bold the note and make the wording more clear. 👍 from me if you want to merge this @simurai :grinning:

@lee-dohm Thanks!
Ok, the build fails, but (to me) looks like unrelated timeouts, so 🚢


cirrus customer-testing: error: unknown option :--:--'duringgit checkout`

seen on:

https://github.com/flutter/flutter/pull/86116
https://github.com/flutter/flutter/pull/85797

Cloning into '../../bin/cache/pkg/tests'...
Downloading Dart SDK from Flutter engine 1c24b5050df26a5ba80810b41d66dbcd1506e7f7...
Building flutter tool...
error: unknown option `:--:--'
usage: git checkout [<options>] <branch>
   or: git checkout [<options>] [<branch>] -- <file>...

This was likely caused by https://github.com/flutter/flutter/pull/83964/files
cc @Hixie

I'm guessing we're getting bad input from find_commit.dart when the shell script calls:
git -C ../../bin/cache/pkg/tests checkout dart --enable-asserts ../tools/bin/find_commit.dart ../../bin/cache/pkg/tests``
https://github.com/flutter/flutter/pull/83964/files#diff-5d38bd26e45c3c438248bdadabeef5f8fbc75bec86c2f94f295d8965485724d7R34

https://github.com/flutter/flutter/pull/86143 is adding more verbosity

This has been fixed a long time ago


Fix tests that relied on Lodash 4

I mistakenly upgraded our tests to use Lodash 4, not realizing some of the features in Lodash 4 did not work in IE9-10. (seamless-immutable itself does not depend on Lodash; this was for tests only.)
We subsequently added tests that relied on those features, causing IE9-10 tests to fail. I have reverted to Lodash 3.10 and disabled those tests, so that the other tests can complete cross-browser testing as normal. However, now we have a few pending tests that need to be rewritten to use Lodash 3.10 features instead of the Lodash 4 features they currently use.
The pending tests are marked with xit in the test suite.
Help on this would be greatly appreciated! ❤

I am not very familiar with the infrastructure here, any easy way to reproduce the issue?
I tried to open a pull request and let it run against travis, expecting that I would get relevant output from there (assuming travis had credentials to run cross-browser tests with some third party service). However I am always getting only:
Zuul tried to run tests in saucelabs, however no saucelabs credentials were provided.
See the zuul wiki (https://github.com/defunctzombie/zuul/wiki/Cloud-testing) for info on how to setup cloud testing.

This is by contrast to https://github.com/rtfeldman/seamless-immutable/pull/178 where there is one travis instance that reported tests that passed, with explicit mentions of different browsers. I am not sure how to get those tests to run while not being rtfeldman :) .

@saivann I pushed what theoretically seems like a fix for that credentials complaint.
That said, if you can get all the tests passing locally on Lodash 3.10, that should be good enough. 🙂

I'm still seeing the same error as above - the only test that seems to break locally is the omitBy test here https://github.com/rtfeldman/seamless-immutable/blob/master/test/ImmutableObject/test-without.js#L176 which I've raised a PR for #196

Hmm, lodash only dropped support for ie8 and below. So either you found a
bug in lodash or you need to load the es5 shim?
On Tue, Jan 31, 2017, 10:56 PM Ryan Temple<EMAIL_ADDRESS>wrote:

I'm still seeing the same error as above - the only test that seems to
break locally is the omitBy test here
https://github.com/rtfeldman/seamless-immutable/blob/master/test/ImmutableObject/test-without.js#L176
which I've raised a PR for #196
https://github.com/rtfeldman/seamless-immutable/pull/196
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rtfeldman/seamless-immutable/issues/179#issuecomment-276505662,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADWlrzUVyv2v-vCufqXdpexQpuKta3Kks5rX64mgaJpZM4LKEr4
.


Error on VirtualBox 7.0.2 r154219

When I typed patch9x and press enter

VirtualBox immediately popped up this dialog box:

Here is the log file:
VBox.log

It's a bug in VirtualBox. Report it to Oracle, not here:
https://www.virtualbox.org/wiki/Bugtracker
VirtualBox 7.x is currently considered beta quality software, so you may want to stick to 6.1.x instead.

It is a VB error and as birdie says, but I just want to note that patcher9x works perfectly on 7.0.4 if you're still having problems. Thanks for the patch! :)


Requesting to add 'chunksize' arguments for updating existing services

Requesting to add 'chunksize' arguments to the two calls (delete, add) within the publishingtools.updateFeatureService() function. Using this with portalautomation.py I was encountering a '404' error for certain input data. I was able to overcome the error by trying this modification.
publishingtools.updateFeatureService():
See this -- https://github.com/Esri/ArcREST/blob/master/src/arcresthelper/publishingtools.py#L2683
And this -- https://github.com/Esri/ArcREST/blob/master/src/arcresthelper/publishingtools.py#L2689
Example modified:
resItm['DeleteDetails'] = fst.DeleteFeaturesFromFeatureLayer(url=fURL, sql=ext_service['DeleteInfo']['DeleteSQL'], chunksize=cs)

The functions being called already have parameter for chunksize.  Look see ...
def DeleteFeaturesFromFeatureLayer(self,url,sql,chunksize=0)
def AddFeaturesToFeatureLayer(self,url,pathToFeatureClass,chunksize=0,lowerCaseFieldNames=False)
Will need to also add little code to updateFeatureService() in order to set a default chunksize if the JSON config file lacks a value for that property.  In other words, the "ChunkSize" property is optional.
cs = 0
try:
  if ext_service['ChunkSize']: cs = ext_service['ChunkSize']
print "Using chunk size: %s" % cs

Example JSON config:
{
  "PublishingDetails": {
    "PublishData":"True",
    "ExistingServices": [
      {
        "URL": "put your url to feature service layer here",
        "DeleteInfo": {
          "Delete": true,
          "DeleteSQL": "1=1"
        },
        "FeatureClass": "..\\Maps and GDBs\\forDevelopment.gdb\\newDataForUpdate",
        "ChunkSize":200
      }
    ]
  }
}

Error as seen without this modification:
Add Failed: {'addResults': [], 'errors': {'error': {'code': 404}}}
What a tough nut to decipher. That error message, by the way, was also a modification I made while troubleshooting. The messaging for both the adds & deletes is lacking the ability to print the error message. So I tweaked on that as well.

Added option, please test and let me know if this resolves the issue

Tested and it works as expected. Success!
Thanks @MikeMillerGIS


Session inside component

After upgrade from 0.7.3 to 0.8.0 I had to find out that session is no longer injected into components. I couldn't find anything related to this change, only merged issue(https://github.com/simplabs/ember-simple-auth/pull/372) mentioning that session is injected into components automatically. If this is desired behavior it should be mentioned somewhere or do I have migration bug in app?

The session should definitely be injected into components - if it's not that's a bug. Please provide more info though or ideally set up a JSBin that illustrates the problem.

Well that was pain to find but I finally got it.
Session is not injected into component when generated component.js file is deleted. I made repository showcasing issue. There are two components, one with default.js file and other without and some fake authorization process but thats not important.
https://github.com/Keeo/ember-simple-auth-bug-showcase/commit/e1a8e2d981e070231544a88efe9efdeedc0d47b2

That's more an Ember CLI issue then I'd say (if not intentional anyway).

Agree, especially that ember-cli is still at version for ember 1.12.


DrawingView

Docs Build status updates of commit 7a835cb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Docs Build status updates of commit efebab8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit fe6f777:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Docs Build status updates of commit 3149c81:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Docs Build status updates of commit 4036539:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Made a first very quick pass: this still needs an entry in the TOC file

Docs Build status updates of commit 48cf6ec:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

Yaml File(s)
This PR includes changes to .yml file(s) owned by another author.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Docs Build status updates of commit 09f4669:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

Yaml File(s)
This PR includes changes to .yml file(s) owned by another author.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Docs Build status updates of commit cabc56a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

Yaml File(s)
This PR includes changes to .yml file(s) owned by another author.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Docs Build status updates of commit 8dc66be:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

Yaml File(s)
This PR includes changes to .yml file(s) owned by another author.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Docs Build status updates of commit 8d57991:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

Yaml File(s)
This PR includes changes to .yml file(s) owned by another author.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Docs Build status updates of commit 0578de2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

Yaml File(s)
This PR includes changes to .yml file(s) owned by another author.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Docs Build status updates of commit 4bbcd2c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

Yaml File(s)
This PR includes changes to .yml file(s) owned by another author.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.

Docs Build status updates of commit e17dc8d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/views/DrawingView.md
:white_check_mark:Succeeded
View

docs/maui/views/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

Yaml File(s)
This PR includes changes to .yml file(s) owned by another author.

File Change Percent
This PR contains file(s) with more than 20% file change.

Index File
This PR contains a change to the index file.


When activities api is called without a limit, only 20 records are returned

When the API : /api/device-mgt/v1.0/activities is called without a limit, only 20 records are returned. We should either throw an error asking users to specify a limit or either maintain across api standard when it comes to No. of records returned when a limit is not specified.

This has been limited purposefully for safeguard against expensive operations to fetch all records from db. This can be configured via  in cdm-config.xml


Mis-matched dll affecting Gtk+ and gtkmm

When I run a program I compile that links against gtkmm and Gtk+, windows can't find the libicuuc56.dll file. I went looking in the /mingw64/bin directory and found the libicuuc57.dll file. So it looks like the DLL was updated to a newer version, but the gtkmm and Gtk+ packages were not recompiled to work with the new version.  I found the missing DLL's (also needed libicudt56.dll) in a different installation of MSYS2 that had not been updated, and copied them into the /mingw64/bin directory and that fixed the problem.
I'm on windows 10 using the mingw64 gcc compiler to make a 64 bit application that uses gtkmm 3.
Thanks.

After more digging, I found the mistake, and it was related to some other packages I installed. False alarm.


Eliminate the Final Acceptance in the Signature Pad

In some acceptance testing many times clients are surprised to confirm their signature twice.
Similar to entering a PIN at self serve at a merchants once signed then nothing more should need to be done.

Moved to https://github.com/RyanNerd/rxchart-pad/issues/2


String Comparison - Case Sensitivity

Excel is case insensitive, XLS-XML-Formula-Expander is not.
"Bob" = "BOB" evaluates as TRUE in Excel, but XXLS returns FALSE.

You don't even work at ECW anymore, right? And it looks like the site this was built for isn't even run by ECW anymore.

Correct, I haven't worked at ECW for a few years now.  I didn't even realize this issue was still open.

Haha, it's cool. How's it going?


Thanks!.

Hi @TaguchiModels
Just thank you for the work done. 😄
I work on a facial recognition application for nextcloud. We tried to use the original nextcloud models with very good results, but of course there are people who had these bias problems, who hopefully improve with your work.
So far I'm testing it with a set of test images and at least I'm not seeing regressions. Therefore I am going to recommend that those who have had problems try it.
See:

https://github.com/matiasdelellis/facerecognition/pull/750

Hi @matiasdelellis,
Thank you for your kind words and for trying the Taguchi model. 😄
I am very happy to hear that you are seeing promising results so far.
Your recommendation to those who have had issues with the original model is very important to me. Knowing that my efforts might help others is incredibly rewarding.
Thank you!
Taguchi


add missing dependency in make example

https://github.com/aaronpeikert/reproducible-research/blob/9107c1490c03e7d080fe68b8b3dd55213450cdc9/manuscript.Rmd#L160
analysis.Rmd is missing.

I added this. I also took the liberty of revising the example and surrounding text to increase clarity. Please check whether I introduced errors when renaming the files in the example, @aaronpeikert. Thanks!

97d6c1a

Two things:

should we include an all target in the example?
Is the naming of files (namely report and preprocessed) consistent:

within the paper
in the example
with WORCS
I will check that later...

Very good point! Let's make it WORCS-compatible and rename (again) to manuscript.Rmd.
I haven't touched the example, yet.

all target is added now
any other compatibility issues reamining, @aaronpeikert ?

I think we can close this. Feel free to reopen when you detect other issues.


Drag and drop demo not working

Hi, the drag and drop jsFiddle doesn't seem to work:
http://jsfiddle.net/kashjs/n2gxobf7/
It seems that the way you get the id of the dropElement gives an error in the console. I have found a workaround, to replace line 64:
"if (dropElement.attr('id') === 'my-basket') {"
with
"if (dropElement.el.context.id === 'my-basket') {"

Yeah! it looks like the documentation had not been updated.
to get the actual element you do dropElement.el instead of dropElement

Ah it looks like the jsfiddle has not been updateed.

Fixed!!


Feature Request: add another shape

Feature Requested
Additional to the square and circle shapes, is it possible to add another shape in between (square with rounded corners) ?
Relevant Screenshots

Checklist

[ ] The feature would be useful to more users than just me. (→ If it's only relevant to you, have you considered using a CSS Snippet?)
[x] I have checked that the feature is not already available via the Style Settings plugin.

now this is the proper label ;)
yeah, that's possible. I'll do it when I finished the current to do list (change of accent, hover, and text colors, etc.) 😊

added in the latest update :)

Thank you! been waiting for this. Could you also make the checkboxes the same shape, please ?

sure, fixed it just now.
The shape settings affects hundreds of shapes throughout the theme, so I am pretty sure that I have still overlooked some :P


Prints ${spaceForNamedImports} in import statement

When autoimporting, this is what I get:
import Hapi${spaceForNamedImports} from 'hapi';

I tried to investigate the source of the problem, but the code that seems to be related (ImportOrganiser.js around line 60) is way too complex imho.
Disabling the spaceForNamedImports setting does not make a difference.

+1

+1


upgrade patternfly to v5, storybook and react to latest versions

Upgrade Patternfly, storybook, react and typescript to latest versions

fixes #26


[UnitTest/Framework] - Refactoring Unit-Tests / Switching to .Net Core

⚠  Under Construction ⛑
TODOS:

[x] Change Algorithms project to .net core
[x] Change DataStructures projects to .net core
[x] Remove "MainProgram" test.

Algorithms Tests

[x] BinarySearchTreeSorterTest
[x] BinaryTreeRecursiveWalkerTests
[x] CatalanNumbersTest
[x] CountingSortTest
[ ] GraphsBellmanFordShortestPaths
[x] GraphsBipartiteColoringTest
[ ] GraphsBreadthFirstPathsTest
[ ] GraphsBreadthFirstSearchTest
[ ] GraphsConnectedComponents
[ ] GraphsCyclesDetectorTests
[ ] GraphsDepthFirstSearchTest
[ ] GraphsDijkstraShortestPathsTest
[ ] GraphsTopologicalSorterTest
[x] GreatestCommonDivisorTest
[x] HeapSorterTest
[ ] InsertionSortTest
[x] LSDRadixSorterTest
[x] MergeSorterTest
[x] QuickSortTest
[x] StringEditDistanceTest
[x] StringPermutationTests

Data Structures Tests

[x] ArrayListTest
[x] AVLTreeTest
[x] BinaryHeapsTest
[x] BinarySearchTreeMapTests
[x] BinarySearchTreeTest
[x] BinomialHeapsTest
[ ] CliqueGraphTest
[x] CuckooHashTableTest
[x] DLinkedListTest
[x] GraphsDirectedDenseGraphTest
[x] GraphsDirectedSparseGraphTest
[ ] GraphsDirectedWeightedDenseGraphTest
[ ] GraphsDirectedWeightedSparseGraphTest
[x] GraphsUndirectedDenseGraphTest
[x] GraphsUndirectedSparseGraphTest
[x] GraphsUndirectedWeightedDenseGraphTest
[x] GraphsUndirectedWeightedSparseGraphTest
[x] HashTableSeparateChainingTest
[x] PriorityQueuesTest
[x] QueueTest
[x] RedBlackTreeMapTests
[x] RedBlackTreeTest ⚠
[x] SkipListTest
[x] SLinkedListTest
[x] SortedDictionaryTests
[x] SortedListTests
[x] StackTest
[x] TrieMapTest
[x] TrieTest

Misc

[ ] Fix broken codes and cleanup
[ ] Ready for merge

Framework changes:
ApplicationException => InvalidOperationException
Assembly.GetExecutingAssembly().Location => typeof(PrimesList).GetTypeInfo().Assembly.Location

I have been very lately... next week I'll continue coding this patch 😉 /cc @aalhour

Hey @ivandrofly, sure thing, no worries 😉

Seems like we are having conflicts with this branch...

Couple more then we finish... :)
PS: The conflicts also need to be resolved

Hello @ivandrofly, I have just merged PR #49, it might cause some problems with your migration. Can you take a look at it when you're done?

I have rebased and fixed most of the problem... I think this branch is ready to be merged and the remaining tests can be send in separated PR in order to avoid more merge conflicts in future. /cc @aalhour
PS: I have some modified files (locally) which can cause some red squiggly in VS after merged with master branch, that will be fixed in next PR.

Hello @ivandrofly,
Sorry for the late reply, I was very busy lately. This sounds good to me. I will merge it now and will be waiting for your next PR.
Cheers,
Ahmad


Fix ort adaptor

Type of Change
bug fix
Description
detail description
JIRA ticket:
https://jira.devtools.intel.com/browse/ILITV-2767
https://jira.devtools.intel.com/browse/ILITV-2768
Expected Behavior & Potential Risk
pre-ci and extension test pass
How has this PR been tested?
extension test
Dependency Change?
no

https://inteltf-jenk.sh.intel.com/job/intel-lpot-validation-top-mr-extension/4675/

https://inteltf-jenk.sh.intel.com/job/intel-lpot-validation-top-mr-extension/4675/

https://inteltf-jenk.sh.intel.com/job/intel-lpot-validation-top-mr-extension/4679/


Allow utf-8 character in config value

If i declare Chinese character become a value, when i run my app in xcode, it will fail and show the follow error message:
./ReactNativeConfig/BuildDotenvConfig.ruby:22:in `split': invalid byte sequence in US-ASCII (ArgumentError)
	from ./ReactNativeConfig/BuildDotenvConfig.ruby:22:in `<main>'

So i add this two lines code to avoid this error.
Encoding.default_external = Encoding::UTF_8
Encoding.default_internal = Encoding::UTF_8

now it works fine.

Oops, 2nd time this comes in! Thank you :+1:

released in 0.9.1


Null Forgiveness Operator not Explained

Under Examine Survey Results, the null forgiveness operator is used but not explained.  It is easy to miss the existence of the null forgiving operator if you use this document alone as a guide for using NRT, leading to issues where you have to use it (such as entity framework) and don't know about it.

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

ID: b991cba9-efd4-68ae-283a-b575054b4907
Version Independent ID: 793f790b-e2aa-e292-b12b-f4ba9763f172
Content: Design with nullable reference types
Content Source: docs/csharp/tutorials/nullable-reference-types.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

THanks for writing this @kateract
We agree! This issue is a duplicate of #13420
Closing as a duplicate. (When fixing #13420, this article should link to that update).


chore: update axum-test dependency to 15.5

Update axum test to the latest version.

Hey, I thought I would go through popular projects using Axum Test and update them to the most recent version. I hope this is useful, and let me know if it's an issue.

Thanks!


AssetManager not work correctly with Relative URL in getAssetUrl method by assetMap.

In base by assetMap we can change asset url to Relative URL.
But AssetManager use only 2 rule for that changing

Base rule. (BaseUrl + AssetUrl)
A relative URL that begins with // (two slashes) always replaces everything from the hostname onwards.

and forget about the third rule
3. A relative URL that begins with / (a slash) always replaces the entire pathname of the base URL.
I propose to change
public function getAssetUrl($bundle, $asset)
    {
        if (($actualAsset = $this->resolveAsset($bundle, $asset)) !== false) {
            return Url::isRelative($actualAsset) ? $this->baseUrl . '/' . $actualAsset : $actualAsset;
        } else {
            return Url::isRelative($asset) ? $bundle->baseUrl . '/' . $asset : $asset;
        }
    }

to
public function getAssetUrl($bundle, $asset)
    {
        if (($actualAsset = $this->resolveAsset($bundle, $asset)) !== false) {
            return Url::isRelative($actualAsset) && strncmp($actualAsset, '/', 1) ? $this->baseUrl . '/' . $actualAsset : $actualAsset;
        } else {
            return Url::isRelative($asset) ? $bundle->baseUrl . '/' . $asset : $asset;
        }
    }

This will be used for such substitution by assetMap like

assetMap value
genaretad url
result url

js/jquery.js
$baseUrl/js/jquery.js
http://sitehost/path/to/baseUrl/js/jquery.js

/js/jquery.js
/js/jquery.js
http://sitehost/js/jquery.js

//ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js
//ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js
http://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js

I am unsure.
Although such change can be made it will encourage bad practice. Using domain relative paths (starting from '/') will produce invalid results in case project is located not under domain root, e.g. 'http://domain.com/myproject'.
Also feature actually breaks a BC. If asset is placed in assetMap with leading '/' it produces URL with double '/', e.g. 'http://domain.com/assets//css/overrride.css', which is resolved by browser correctly as 'http://domain.com/assets/css/overrride.css'. If someone is making mistake placing leading '/' in asset of assetMap he may not realize this.
Current walkaround is using folder relative URL with '..'. For example, asset specified as '../js/jquery_overridde.js' will produce URL '/assets/../js/jquery_overridde.js', which will be resolved as 'http://domain.com/js/jquery_overridde.js'.

in case project is located not under domain root, e.g. 'http://domain.com/myproject'

in this case, those who will use "/" will be able to avoid this error, because they already know they need URL (or you can use even so sites baseUrl, instead assets baseUrl)

If someone is making mistake placing leading '/' in asset of assetMap he may not realize this.

double slashes repeatedly discussed on the Internet, and this is not normal behavior of any system or browser. if the user makes a mistake, it should lead to error, and do not pretend that a minor error.

Current walkaround is using folder relative URL with '..'. For example, asset specified as '../js
/jquery_overridde.js' will produce URL '/assets/../js/jquery_overridde.js', which will be resolved as 'http://domain.com/js/jquery_overridde.js'.

I agree that my version is not the most elegant and not the fact that the best, but the use of "..", for me it does not look better.

in this case, those who will use "/" will be able to avoid this error, because they already know they need URL

In this case result varies by environment. Following your logic you may just specify absolute URL for your 'jquery.js' just as well.
Relying on domain relative path (starting from '/') is not a good practice. And it does note metter where it is used: for assetMap or background-url specifications inside CSS files.
Someone may as well expect framework will automatically prepend such URL with UrlManager::baseUrl.

double slashes repeatedly discussed on the Internet, and this is not normal behavior of any system or browser. if the user makes a mistake, it should lead to error, and do not pretend that a minor error.

I have never told it is a valid usage. But if the fix is going to be made it should be reflected at UPGRADE.md just in case. Or postponed to 2.1.

I agree that my version is not the most elegant and not the fact that the best, but the use of "..", for me it does not look better.

Still, until solution is not provided you may use it. At least it does not depend on the environment and will work the same no mattter if project located under sub directory or not.
Or you can specify absolute URL for your asset.

Following your logic you may just specify absolute URL for your 'jquery.js' just as well.

Regarding me, I am already using the proposed solution.  And forgive but not for 'jquery.js', it was listed as an example.
I just decided to bring the idea up for discussion, because I believe it can be useful to others.

Someone may as well expect framework will automatically prepend such URL with UrlManager::baseUrl

If need this, can use another code (already adapted to the latest release):
public function getAssetUrl($bundle, $asset)
    {
        if (($actualAsset = $this->resolveAsset($bundle, $asset)) !== false) {
            $asset = $actualAsset;
			if (strncmp($asset, '/', 1)) {
				$basePath = $this->basePath;
				$baseUrl = $this->baseUrl;
			}
			else {
				$asset = ltrim($asset, '/');
				$basePath = "@webroot";
				$baseUrl = "@web";
			}
        } else {
            $basePath = $bundle->basePath;
            $baseUrl = $bundle->baseUrl;
        }

        if (!Url::isRelative($asset)) {
            return $asset;
        }

        if ($this->appendTimestamp && ($timestamp = @filemtime("$basePath/$asset")) > 0) {
            return "$baseUrl/$asset?v=$timestamp";
        } else {
            return "$baseUrl/$asset";
        }
    }

Issue resolved by commit 95ba06c


add css.js to challenges

Grid CSS sample from MDN Web Docs.

I'll close this PR and replace it with a less messy one.


User email change reauthentication

Hi, I need to reauthenticate user to change his/her email address.
I'm trying to use FirebaseAuth.getInstance().getCurrentUser().reauthenticate() which requires AuthCredential.
How can I get it using
startActivityForResult(                     AuthUI.getInstance()                             .createSignInIntentBuilder()                             .setProviders(AuthUI.GOOGLE_PROVIDER,                                     AuthUI.EMAIL_PROVIDER,                                     AuthUI.FACEBOOK_PROVIDER)                             .build(),                     RC_SIGN_IN);
?
Thanks

I resolved my issue.
I simply updated email in onActivityResult and it worked.


Permission error

I'm always getting a permission error in a derived Dockerfile when jenkins.sh is being called. It happens here:
echo "$f" >> $COPY_REFERENCE_FILE_LOG
My solution was to move the VOLUME directive at the end in the original Dockerfile because otherwise the chown permissions won't stick and everything will still be set to root:root instead of jenkins:root.

this has been fixed in c8c95d
just set COPY_REFERENCE_FILE_LOG to somewhere in the JENKINS_HOME

Same happens in plugins.sh which tries to write to /usr/share/jenkins/ref/plugins.

are you switching to root user as done in https://github.com/jenkinsci/docker#installing-more-tools ?

`FROM jenkins
COPY plugins.txt /usr/share/jenkins/ref/
RUN /usr/local/bin/plugins.sh /usr/share/jenkins/ref/plugins.txt`
That's what I'm doing, but for some reason the file plugins.txt does not exist in the target directory once the container is up. Looks like a user error but I just can't figure out why something that simple goes wrong.

works fine here

Is it closed? It also happen to me. How to solve this problem?

In my case, I used a script I wrote in which I used the wrong base image (not the derived).


Can't find DlibDotNetNative.dll and DlibDotNetDnn.dll files.

Hi,
First of all great wrapper, it seems like it is a very powerful library.
However when I try to train Helen I can't find the DlibDotNetNative.dll and DlibDotNetDnn.dll files.
I followed the instructions:

download models
entered "dotnet build -c Release"

I was wondering I someone could help me locate them. Thanks!

@SamuelvVelzen
DlibDotNetNative and DlibDotNetNativeDnn dll is not included in repository.
You can build it from DlibDotNet project or download DlibDotNet nuget packge and extract it.
And I'm very sorry to trouble you.
Current DlibDotNet project in solution has gap for FaceRecognitonDotNet.
Next release, update dlibdotnet and some issue, helen training program.
Please refer this issue about the above information,
https://github.com/takuya-takeuchi/FaceRecognitionDotNet/issues/77

Hi Takuya,
Thanks for reacting so fast and trying to help everybody.
After your answers I still have some questions. I was wondering if you could be bother to answer them.
First of all I want to include that I am trying this from a Mac using .NET Core 3.1.101. I am telling you this because I was wondering if this is supported because after read your dependencies libraries and products I came across OpenCVSharp that only supports windows and ubuntu. And I thought that dependencies had to be included in the project. Also when I downloaded your DlibDotNet project and when I tried to run it it gave the error that it didn't find assemblies for .NETFramework, version 4.6.1.
example error:
/usr/local/share/dotnet/sdk/3.1.101/Microsoft.Common.CurrentVersion.targets(1175,5): error MSB3644: The reference assemblies for .NETFramework,Version=v4.6.1 were not found. To resolve this, install the Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET Framework Developer Packs at https://aka.ms/msbuild/developerpacks [/Users/Samuel/Downloads/DlibDotNet-master/examples/WPF/FaceLandmarkDetection/FaceLandmarkDetection.csproj]
I was wondering if you could explain more about how I could extract the files from a nugget package. Because when I try to add a nugget package it can't find a project.
As last question I was wondering you meant with you comment about the current DlibDontNet project. Could you explain this a bit more:
Current DlibDotNet project in solution has gap for FaceRecognitonDotNet.
I am looking forward for your next release!

@SamuelvVelzen
I assume there is 3 questions.

Also when I downloaded your DlibDotNet project and when I tried to run it it gave the error that it didn't find assemblies for .NETFramework, version 4.6.1.

This program is not targeted to be working on not windows.
Because target framework is .NET Framework rather than .NET Core.

I was wondering if you could explain more about how I could extract the files from a nugget package. Because when I try to add a nugget package it can't find a project.

There is 2 methods.

change extension from *.nupkg to *.zip and extract it.
access to ~/.nuget/packages/dlibdotnet/<IP_ADDRESS>00301/runtimes/osx-x64/native

this has *.so. *.dylib or *.dll

As last question I was wondering you meant with you comment about the current DlibDontNet project. Could you explain this a bit more

There is no deep reason.
The last release of FaceRecognitionDotNet is June 2019. On the other hand, the last release of DlibDotNet is Dec 2019.
Especially, DlibDotNet has minor bug and age/gender classification function depends on latest DlibDotNet.
So I must update it.
BTW, you can play HelenTraining by modify reference setting of DlibDotNet.
Current setting referes ../src/DlibDotNet and ../src/FaceRecognitionDotNet project.
But you can modify from it to nuget package. then build system will copy and reference proper binary automatically.

Thanks for your lengthy explanation, it helped me so much. I could find DlibDotNetNative.dll and DlibDotNetNativeDnn.dll files. However I couldn't find the DlibDotNetDnn.dll file and I was wondering if this was a type error?
Also when I try to train Helen I follow these steps:
https://github.com/takuya-takeuchi/FaceRecognitionDotNet/tree/develop/examples/HelenTraining
However because I couldn't find the DlibDotNetDnn.dll file so I added the DlibDotNetNativeDnn.dll file. But when I try to generate the model using the following command:
dotnet run -c Release -- generate -m "models" -p 50
It throws this error:
`
Model: models
Padding: 50
Unhandled Exception: System.TypeInitializationException: The type initializer for 'DlibDotNet.NativeMethods' threw an exception. ---> System.DllNotFoundException: Unable to load DLL 'DlibDotNetNativeDnn': The specified module or one of its dependencies could not be found.
(Exception from HRESULT: 0x8007007E)
at DlibDotNet.NativeMethods.LossMetric_anet_type_create()
at DlibDotNet.NativeMethods..cctor() in /Users/Samuel/Downloads/FaceRecognitionDotNet-master/src/DlibDotNet/src/DlibDotNet/PInvoke/NativeMethods.cs:line 40
--- End of inner exception stack trace ---
at DlibDotNet.NativeMethods.get_frontal_face_detector()
at FaceRecognitionDotNet.FaceRecognition..ctor(String directory) in /Users/Samuel/Downloads/FaceRecognitionDotNet-master/src/FaceRecognitionDotNet/FaceRecognition.cs:line 70
at HelenTraining.Program.<>c__DisplayClass1_1.b__4() in /Users/Samuel/Downloads/FaceRecognitionDotNet-master/examples/HelenTraining/Program.cs:line 76
at Microsoft.Extensions.CommandLineUtils.CommandLineApplication.Execute(String[] args)
at HelenTraining.Program.Main(String[] args) in /Users/Samuel/Downloads/FaceRecognitionDotNet-master/examples/HelenTraining/Program.cs:line 411
`
I was wondering if you knew what this error is and how to fix it.

you should copy libDlibDotNetNative.dylib and lib DlibDotNetNativeDnn.dyllub rather than *.dll for OSX.

Thank you, this fixed the problem I could build and train the dataset without almost any problem.
I still have a few questions about the usability of the dataset but will open a new issue post about it so this question will remain clean.


DenseNetBC100

I have finished DenseNetBC100 for cifar10,please use python train.py to verify the code,details could be seen in readme.md

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

@huicui1996 Please sign the CLA firstly

@huicui1996 Please sign the CLA firstly

Ok.I have signed the CLA.

@huicui1996 您好 很高兴地宣布您跑通DenseNet100模型流程的时间（2021-04-01-16:51）在所有参赛选手中位居第二位，满足二等奖的条件（TOP3）

活动已结束，关闭


Fix the clairctl pull for schema 1 and 2

will test this one tomorrow, thanks @jgsqware

I've checked with an insecured registry configured to the docker daemon and it's working without the flag.
I don't think the flag --insecureshould be used. As I use the docker libs itself, I think it's better to use the same behavior as docker cli/daemon and use the configured insecure registry

@murat1985 When you have time you can test it on the master branch

closes #3


BH not pulling in username or name

Description
If the "Name" field in your GitHub profile is blank, BH will pull in a number string (need to investigate what the string is). Default behavior should be changed (e.g. to pull in username if name field is blank).
Screenshots

Files
To Reproduce
Set name field as blank in your GitHub profile.
Tasks
Remember to use helpful labels and milestones. If you use the "help wanted" label, Code for America will promote it widely.

One note: When I create a blog post, the "Author" field auto-populates with my GH username. Do we want to sync up the behavior of these items (the blog post author field and the username shown at the top of the page)?

fix #272


Would like a code sample on GZip/Deflate

Customers wish to see code samples on how to send compressed messages from the device, both single messages and compressed batches of messages. Ideally, in all Device SDK languages that can support this. Thanks

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

ID: bddf4c6b-8241-80b2-717b-ce3fe72e09c2
Version Independent ID: 89d31103-4bbf-d16b-b485-a826af56c81a
Content: Stream data as input into Azure Stream Analytics
Content Source: articles/stream-analytics/stream-analytics-define-inputs.md
Service: stream-analytics
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

@andrepodnozov Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Hi @andrepodnozov - thanks for your suggestion! We have some samples available in this blog: https://blogs.msdn.microsoft.com/streamanalytics/2018/08/07/processing-compressed-event-streams-with-azure-stream-analytics/. I will see what we can do about incorprating more examples into our docs.

#please-close

Sorry about delayed response. Unfortunately this code sample doesn't work, because GetSerializedEvents method always returns empty array. Also this code doesn't compile due to a superfluous generic type parameter on this method. Can you please double-check the sample code.

Hi @andrepodnozov - we have created a work item to create additional samples and to update the existing sample.

#please-close

I'll look forward to that. Thanks.


it 'uses username and access key if ENV variables are defined'

@titusfortner This spec is not clear, which user name and access key? The user will set it?
I would rather throw an exception if the Environment variables aren't set and not allow the user to set them directly through the code

I have in the past used a secret file that was in .gitignore and couldn't be checked in that stored sensitive info...
I guess we could have them set ENV instead of passing it into class... Java would have to also check system properties since it can't change env in code.

@titusfortner so are we good with forcing users to set environment variables? Otherwise, Runtime exception. It's a best practice, I see no reason with not forcing users into it.

I think what we agreed to in the PRs above is good to close this out.


Add svg converter

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features).
[ ] (If applicable) Tests have been added.
[ ] CHANGELOG.rst has been updated (with summary of main changes).

[ ] Link to issue (:issue:number) and pull request (:pull:number) has been added.

What kind of change does this PR introduce?

Adds an SVG converter to the PDF generating LaTeX dependencies

Does this PR introduce a breaking change?
It should not.
Other information:

coverage: 81.343%. remained the same
when pulling 404c5d7d21e78752bf595720851c366a9ddeb7b5 on fix-docs
into 86d63062c83e490b4fadc8cd684b3c0a629f33e9 on master.


DRAFT

Description
Describe the purpose of and changes within this Pull Request.
Testing
Do any existing tests cover this change? Are new tests needed?
Documentation
Is there any documentation impact for this change?
Installation
Is there any installer impact for this change?

Successful test of the fix to hyper-v adapter vSwitch connection on the 'bad' runner host.
Uploaded 140MB artifact in under 2 min.
Not blazingly fast, but it works and is reasonable for our expected file sizes: KM dump 600MB, perf ETLs 1GB.
I'll follow up with a change to compress those files.


Camtrap DP resource from an archive -> view button throws a 404 error

I am not sure it is supposed to work at this stage, but the view button in the overview page for the archived upload throws a 404 error when you click it

If resource is not published yet it'll display a 404 indeed. We should probably consider hiding the button

I think it would be best to remove, so publisher don't think something is wrong with the resource


change var name

Changed the bad named variable.

Codecov Report

Merging #194 into master will increase coverage by 1.55%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #194      +/-   ##
==========================================
+ Coverage   84.43%   85.99%   +1.55%
==========================================
  Files          11       11
  Lines         257      257
==========================================
+ Hits          217      221       +4
+ Misses         40       36       -4

Impacted Files
Coverage Δ

src/transaction.js
97.5% <100%> (ø)
:arrow_up:

src/baseRequest.js
79.31% <0%> (+6.89%)
:arrow_up:

src/request.js
92.85% <0%> (+14.28%)
:arrow_up:

Continue to review full report at Codecov.

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

why did u open a new PR?


Download counter

I don't know if we can store the counter for each fixture without too much extra effort, but it would allow us to sort by popularity.

how do you want to store the download numbers across different deployments?

That's exactly the issue why this is labeled as difficult. Maybe an external provider could be used, but I'm not quite sure if I like the idea.


Switched from java 11 to 17 and now get `initialize': wrong number of arguments (given 1, expected 0)

The import is...
      <dependency>
        <groupId>org.asciidoctor</groupId>
        <artifactId>asciidoctorj</artifactId>
        <version>3.0.0</version>
      </dependency>

...but I see the same 2.5.7.
The code is...
    try (Asciidoctor asciidoctor = Asciidoctor.Factory.create()) {

      return asciidoctor.convert(<string to convert>, Options.builder().build());
    }

...and I now get...
Caused by: org.smallmind.web.json.scaffold.fault.NativeObjectException: Exception in thread org.jruby.exceptions.ArgumentError: (ArgumentError) asciidoctor: FAILED: <stdin>: Failed to load AsciiDoc document - `initialize': wrong number of arguments (given 1, expected 0)
   at RUBY.load(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/load.rb:104)
   at RUBY.convert(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/convert.rb:78)
Caused by: org.jruby.exceptions.ArgumentError: (ArgumentError) `initialize': wrong number of arguments (given 1, expected 0)
   at org.jruby.RubyClass.new(org/jruby/RubyClass.java:922)
   at RUBY.initialize(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/attribute_list.rb:54)
   at org.jruby.RubyClass.new(org/jruby/RubyClass.java:935)
   at RUBY.parse_attributes(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/substitutors.rb:1337)
   at uri_3a_classloader_3a_.gems.asciidoctor_minus_2_dot_0_dot_23.lib.asciidoctor.parser.parse_block_metadata_line(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:2059)
   at uri_3a_classloader_3a_.gems.asciidoctor_minus_2_dot_0_dot_23.lib.asciidoctor.parser.parse_block_metadata_lines(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:2015)
   at uri_3a_classloader_3a_.gems.asciidoctor_minus_2_dot_0_dot_23.lib.asciidoctor.parser.next_section(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:373)
   at uri_3a_classloader_3a_.gems.asciidoctor_minus_2_dot_0_dot_23.lib.asciidoctor.parser.next_section(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:388)
   at RUBY.next_section(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:388)
   at RUBY.parse(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:103)
   at RUBY.parse(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/document.rb:538)
   at RUBY.load(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/load.rb:84)
   ... 1 more

I get this error in ubuntu 22.04, open jdk 17, but not in windows with graalvm 17, if that helps...

...but I see the same 2.5.7.

Can you clarify this comment? mvn dependency:tree should show if there's some other version in runtime...even though I thought Maven would pull the latest is set in the pom.
btw, AsciidoctorJ 3.0.0 uses asciidoctor v2.0.23, at least we know that version is in the classpath

What I mean is that I get that error under openjdk17 on ubuntu 22.04 with only 2.5.7 on the classpath, or only 3.0.0 on the classpath, but not both at the same time. I can guarantee the classpath because all of our jars are pulled and collected into a lib directory that serves as the classpath for tanukisoft scripts that are hosting the jvm, and all versions are fully sepcified in maven poms, and I have checked both the jars available to my local IDE (intelij), and the jars packed into the tanukisoft distribution, and they are both clean, either 2.5.7 or 3.0.0.
Here's the relevant bit from the lib directory...
apiguardian-api-1.1.2.jar
artifact-maven-6.0.0-SNAPSHOT.jar
asciidoctorj-3.0.0.jar
asciidoctorj-api-3.0.0.jar
asm-9.7.jar
asm-analysis-9.7.jar
..and the only asciidoctor jars are both 3.0.0. I would obviously prefer to resolve this issue with 3.0.0.
What I'm really asking is what the mechanism is that would cause that error. My ability to debug easily ends with...
org.jruby.RubyClass.new(org/jruby/RubyClass.java:922
...and the code there is a dynamic reference into some ruby code, I think... I have no clue how to go further. Is there a file in the asciidoctor jars I can double check? A line of code that should have some signature? Or some piece of java being loaded as a service?

I also checked the maven repository on the tanukisoft-based install where I get the error, and it's empty except for a few of our jars containing classpath available text templates. Nothing else in it, so the only asciidocj code on the machine is the 3.0.0 jars listed above. Just as proof maven isn't downloading other versions somehow and confusing the classpath.

It would be great if you could provide a reproducer.
It might make sense checking what JRuby jars are used.

I'd like to hand you a reproducer, but I develop on windows, where I can't a reproduction. Everything works fine. In ubuntu 24.04 I can reproduce in 2.5.7 0r 3.0.0. If I change the asciidoctorj version in my maven pom to 2.5.2 then the problem goes away in linux as well. With 3.0.0 it pulls jruby <IP_ADDRESS>, which is not surprising as that's the listed dependency. With 2.5.2 it's jruby <IP_ADDRESS>.
I'll try to put together a reproducer in linux and see. The document we feed to asciidoctorj is dynamically constructed, so i's a bit difficult, but I'll see what I can do, maybe capture it and create something small and runnable.

Been a while, sorry. I tried to put together a reproducer and was unsuccessful. The project works in ubuntu, with version 3.0.0, but not in our production setup. Production is run under the Tanukisoft wrapper, so I have to guess it may be the culprit. I'll attempt a reproducer. Even if I can reproduce I know that makes it not so much your problem, but if I can get a reproducer, maybe you could help me understand why it fails.

I can not get a reproducer that fails. If I reduce the Tanukisoft wrapper, Grizzly http daemon, Jersey endpoint, plus the several aspectj byte code injected joinpoints which make up our production applicaton, the actual AsciiDoctorJ conversion works perfectly. But with all the production paraphernalia in place, I get...
org.jruby.exceptions.ArgumentError: (ArgumentError) asciidoctor: FAILED: <stdin>: Failed to load AsciiDoc document - initialize: wrong number of arguments (given 1, expected 0)
And if I drop a breakpoint in JRubyAsciiDoctor,java:256 and poke around n the exception, I find this backtrace for that error...
RUBY.load(uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/load.rb:104)

((RubyArgumentError)e.getException().getCause()).backtrace.backtraceObject
0: org/jruby/RubyClass.java:922:in `new'
1: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/attribute_list.rb:54:in `initialize'
2: org/jruby/RubyClass.java:935:in `new'
3: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/substitutors.rb:1337:in `parse_attributes'
4: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:2059:in `parse_block_metadata_line'
5: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:2015:in `parse_block_metadata_lines'
6: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:373:in `next_section'
7: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:388:in `next_section'
8: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:388:in `next_section'
9: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/parser.rb:103:in `parse'
10: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/document.rb:538:in `parse'
11: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/load.rb:84:in `load'
12: uri:classloader:/gems/asciidoctor-2.0.23/lib/asciidoctor/convert.rb:78:in `convert'

Any notions? Some way I can capture any more specifics? This errors does not occur with asciidoctorj version 2.5.2 and does with 3.0.0,


Init a CONTRIBUTING.md file

This Pull Request adds a CONTRIBUTING.md file to guide others to contribute to this project.
Move some contents from README.md to CONTRIBUTING.md, and elaborate a bit on howto run specs.
--
Preview updated README.md
Preview CONTRIBUTING.md

The Development status text I'd still to have in the README.
The BUNDLER_TEST_VERSION was a recent addition to get CI setup, but it does cause the one test to break, which I only added to help troubleshoot CI, so I do need to get that cleaned up. I'm not sure if specifying it matters when running individual tests locally ... ? May require the ci rake task? But I'm just typing from memory after a few days off work.

The BUNDLER_TEST_VERSION was a recent addition to get CI setup, but it does cause the one test to break, which I only added to help troubleshoot CI, so I do need to get that cleaned up.

Do you want me to remove that spec? :)

no, i think i like the spec because it's a crucial part of the CI process, and only the CI process is going to regularly double-check the various combinations i'd like to support.
that said, we still need to make this simple for local development.


feat: disable thor features for smart contract wallets

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #5716   👈

This stack of pull requests is managed by Graphite.


to add additional labels

@natanielruiz  i thanks for sharing the implementation ,wonderful work . I have couple of questions

Where can add/delete the labels
Can i use weights trained on darknet having .weight file format

Thank you @abhigoku10 for your comment.
Please check the disclaimer on the project Readme.
Cheers


Zoom plugin improvements

This makes changes to ZoomPlugin, which was added in #3276. Each change is on its own commit on this branch, to make reversions/modifications easier if requested before approval.
Fix scroll when using trackpad
The zoom is currently triggered by the wheel event. On a trackpad, horizontal scrolling fires a wheel event with both an X- and Y- delta, unless the user scrolls perfectly horizontally on the trackpad, which is not typical. This creates a poor UX when the user is attempting to scroll because the waveform errantly zooms simultaneously. This PR changes the event listener to perform a zoom only when the Y-delta is greater than the X-delta, which is the case for the pinch gesture.
Account for wheel speed when zooming (breaking change)
Currently the zoom level is adjusted by the same amount for each wheel event that is fired. This PR changes the event listener to adjust the zoom level in proportion to the scroll/pinch distance. This offers a more consistent UX than relying on the quantity of event instances. As a result, scale changes from a magnification factor to a coefficient. A scale of .2 felt too unresponsive with the new formula so this changes the default to .5.
Allow using zoom plugin when minPxPerSec is 0 or default
The zoom formula that this change introduces swapped scale for minPxPerSec as the coefficient. Therefore, it is now possible to use this plugin without first having set minPxPerSec to a non-zero value.
Add maxZoom option
Optional. Specifies a maximum minPxPerSec value while zooming.
Checklist

[ ] This PR is covered by e2e tests
[ ] It introduces no breaking API changes

Quick question:
I noticed (using a mouse wheel), that holding shift doesn't have the expected result of scrolling/panning through the waveform horizontally. Would you want to include that improvement in this PR, or open a new one?

@ffxsam that's a good idea! Please feel free to create a separate issue for it.

@ffxsam I've added your request in #3389. 👍
@cmorbitzer thanks again for the amazing contribution! It's now released in 7.5.1. 🚀

@cmorbitzer @katspaugh I took this for a spin on my MacBook Air, and it works great! It's also pretty standard when compared against pro audio tools (Audition, iZotope RX, etc), in that down = zoom in, up = zoom out (or the opposite, depending on your Mac trackpad settings). 😆
The scrolling and panning action is perfect. Love this!

 This is pretty intense. 🤯
https://github.com/katspaugh/wavesurfer.js/assets/12532733/e8daee7a-e4ff-4914-8377-97304258e592

Adobe Audition's zoom, for reference:
https://github.com/katspaugh/wavesurfer.js/assets/12532733/44682b21-2287-4138-bbb1-dc2d3c066afa

I suggest we continue this discussion in the forum.


Fix addeventlistener

Same as #9601 but for v5.3.x

pushed to the wrong branch in my fork, #9603 is the right one


No data coming through

Hi there,
First I want to say a big thank you for putting this together.
I am reaching out with a small but essential problem... as I am not getting anything coming through from Airtable.
I was using the Prefab you have as an example (updating my App & Api keys) and all I get is: Airtable Unity - Environment prepared successfully
I did a test using just AirtableRequestTest.cs and I get the same Log message as above.
Looking in the class I see that there is a Debug.Log which should return a count... but this response by what ever reasons is not triggered.
I am using Unity 2019.4 (LTS)
Any advice is welcome
Thank you,
PS: happy to contribute in updating the documentation once getting this to work.

Sorry for the delay in the answer.
The message Airtable Unity - Environment prepared successfully means that you are all set to request data. For the test I recommend use the ListRecordsExample.cs, you can find this script at Assets\AirtableUnity\Scripts\Examples.
After playing your project with this on any gameobject you can force a test clicking on 3 dots above the script at the editor and on Get Table Records. There is a ContextMenu attribute to enable this call.
I hope you can made this work after this. Let me know if not.
Best regards

That's great, thank you very much for getting back.
I also found the TestCall  ContextMenu on AirtableRequestTest, this is also helpful.
Just one last question. When using the GetRecordExample I can point it to a Record ID, but how do you further point it to get a specific variable from that Record ID, based on the Field Name?
All works fine otherwise, thank you once again.

I'm glad the requests worked @olgv . You pumped me to add some other functionalities that were on my roadmap for this plugin. Thank you a lot for that.
BaseField
To get fields from your table you need to do this step:
Create a new class that has your airtable columns, an important thing is to avoid collum with space characters, if you are sure how this class is going to be you can get a JSON sample of them using the documentation link to your table.
https://airtable.com/<**YOUR APP KEY HERE**>/api/docs#curl/table:links%20importantes:list
On the right side of the screen, you will have a JSON example just like this

This is a list of your current records, let's focus on the first one

Now, let's focus on the fields object

This is going to be our class. Because on the Airtable Unity Package we have a class BaseField who holds the id, fields, and createdTime. This BaseField class receives a T parameter who is going to be your new class. In this case, I just showed our new class would have two properties:

public class MyClass
{
public string Nome;
public string Link;
}

If your table has a lot of fields. I recommend you to use this website to create your c# class.
After having your class you can call the following method <public void GetAirtableTableRecords> inside ListRecordsExample.

Maybe you will need to change it to public
You should call this using BaseRecord. The call should be something like this:

I hope you can make it work with these instructions. Ping me on any doubt

@olgv I hope you could make this work, I'm gonna close this issue. Just let me know if you have any other problem.
Best regards.

Thank you @lipemon1 ,we got it running in the end.
Struggling with getting it running in the build, but runs just fine in editor.

Good to know that you get it running. If there is anything I can help you to run on the build, please let me know. I already had serious problems for loading local files on android mobile devices

Great, thank you. It is a straight macOS build, troubleshooting at the moment at a slow pace.

Yes, seems to be a problem here... running a Development Build (Unity 2019.4.7f1) on macOS (10.15.6) and getting:

[Message: Operation is not supported on this platform.] - [Request https://api.airtable.com/v0/.....]
The error points to Proxy.cs line 136, WebRequestExtension.cs line 40 and last Coroutines.cs line 17

Same error seems to happen in a Windows build.
Thoughts appreciated.
Thank you

I will take a look on this script. I don't imagine what I'm using that is not supported. As soon as I check, I will let you know here


Provide lifecycle servicing

The mode manager usually knows the target state/mode and the current state/mode (via mode_inference) of a system or node. Currently we assume, that there is a trivial transition to the target state, e.g, activate to go to active, deactivate to go to inactive.
However, this is not always true, e.g., if actual state is already the target state, or if target state can only be reached with >1 transitions. In order to allow that, mode manager needs to implement some kind of lifecycle service that knows how to get a node from any state to any other.

The system modes's extended lifecycle model could be node specific.
For example, a node might be able to transition directly from active status in mode A to active status in mode B, but a different node might need to transition first to unconfigured status.
Actually, maybe this 2nd case should be the default one for node safety.

Actually, the 2nd case is the only one currently intended by the ROS 2 Managed Nodes design document. I.e., reconfiguration, which is what we do when switching mode, is only allowed in the unconfigured state.
Currently we ignore this constraint (for now) and re-configure (switching modes) while staying in the active state.

Currently we ignore this constraint (for now) and re-configure (switching modes) while staying in the active state.

I think MROS test cases could help analyse whether this (going through unconfigured) is something to enforce, or not, and promote guidelines for handling dynamic node configuration and a reference implementation applying them in navigation2.


ARROW-3826: [C++] Do not cache ccache in Travis CI

Our cache has grown quite large. I am interested to see if turning this off changes build times

Codecov Report

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

@@            Coverage Diff             @@
##           master    #2984      +/-   ##
==========================================
- Coverage   86.63%   86.61%   -0.02%
==========================================
  Files         491      491
  Lines       69438    69438
==========================================
- Hits        60157    60145      -12
- Misses       9189     9197       +8
- Partials       92       96       +4

Impacted Files
Coverage Δ

go/arrow/math/int64_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/memory/memory_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/float64_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/uint64_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/memory/memory_amd64.go
28.57% <0%> (-14.29%)
:arrow_down:

go/arrow/math/math_amd64.go
31.57% <0%> (-5.27%)
:arrow_down:

cpp/src/plasma/thirdparty/ae/ae.c
72.03% <0%> (-0.95%)
:arrow_down:

go/arrow/math/float64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/int64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/uint64_amd64.go
33.33% <0%> (ø)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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

Well based on comparing

Cache on: C++ library build takes ~208 seconds https://travis-ci.org/apache/arrow/jobs/456156382
Cache off: C++ library build takes ~616 seconds https://travis-ci.org/apache/arrow/jobs/456156382

Can't 100% attribute that to ccache but it seems that fetching the cache takes less than 20 seconds, so I'm going to close as won't fix

@wesm those links are both for cache off

Copy-paste failure. fixed


support ANSI colors in stdout

 ```{julia}
 printstyled(1/7, bold=true, color=:green, reverse=true)
 ```

doesn't work as expected. It is not enough to set color=true in IOCapture.capture(), see
https://github.com/JuliaDocs/IOCapture.jl/blob/874d47adccbf49708e43f2fcc9329695203846cb/src/IOCapture.jl#L112
This patch  https://github.com/MHellmund/QuartoNotebookRunner.jl/commit/9609377303cbc8061e8737012d48fd1c81801f50  works but is probably not the best way to go. Perhaps a modified version of IOCapture, also providing separate cells for stdout and stderr?

Hi @MHellmund, worker processes that run the notebooks don't get passed the color arg by default. We can consider changing that default if that's what users will prefer. What does the output look like when setting
---
julia:
    exeflags: ["--color=yes"]
---

```{julia}
printstyled(1/7, bold=true, color=:green, reverse=true)
```

Can you provide the output .ipynb result of that cell? We don't of course handle the quarto render part of rendering in this repo.

Ah, you are absolutely right!
Coloring works with  exeflags: ["--color=yes"]  and that is fine.
I did look into the documentation of Malt.jl but not into the source code :-)
Please close this issue.

seems reasonable to make --color=yes a default flag I would say

seems reasonable to make --color=yes a default flag I would say

Initially, I thought about including a fix for this issue in #73. But I started to question whether --color=yes is a good default setting when I remembered how cumbersome it was to parse log messages with colorized output in another project. Probably that's not a common thing to do in a qmd file but it made me wonder if there are any other negative implications.

For other formats like PDF/LaTeX, they are, alas, filtered out at some stage by the Quarto/Pandoc system.

Yeah, a bit unfortunate that that happens.

Probably that's not a common thing to do in a qmd file but it made me wonder if there are any other negative implications.

Worth taking into account. Users should still be able to do --color=no if they want no colour for any particular reason.


Generated HTML page looks nothing like example

Hi,
I'm trying to generate HTML for my swagger.json, but the page generated looks nothing like what I see in example. I also tried generating HTML from example json, but it still looks like styles are missing.
$ bootprint openapi http://petstore.swagger.io/v2/swagger.json target
Loading bootprint-openapi 1.1.0
Loading bootprint-json-schema 1.1.0
Loading bootprint-base 1.1.0
[ 'target/index.html',
'target/main.css',
'target/main.css.map',
'target/bundle.js',
'target/bundle.js.map' ]
this is what I see in console log
But result looks like this:

Which example do you mean? I mean: What is the example that looks different from your output? As far as I can see, the output looks good.

I was expecting styles and functionality that is shown here : http://petstore.swagger.io/ , but maybe I just misunderstood what the module does

Bootprint was meant more for a printable documentation, although I think no spec ever got printed.
But the goal was to generate docs that can be watched in the browser and send it to someone.
What you are looking for is swagger-ui, I believe.


Added support for setting subtitle files to MediaPlayer

Can you make it accept streams? It would be much more usefull in some projects.

Do you mean subtitle stream??.. What I added was just a way you can add a subtitle file supported by vlc to the player. The vlc library still does all the work.

I was just looking at it, but it seems that libvlc_video_set_subtitle_file is now deprecated :
https://github.com/videolan/vlc-3.0/blob/115c4a55a728ac504f8285f70d19c56ed9d873bb/include/vlc/deprecated.h#L128

@eadwintoochos : Please reimplement your PR using libvlc_media_player_add_slave
@Traderain : It looks like it is not possible in libvlc itself, so what you are asking for can't be implemented that way.

I'm now closing this PR since there are conflicts and because libvlc_media_player_add_slave should be used instead.
Please create another PR with the asked changes if you feel this is important.