run middleware before and after

would be great if we can choose to run the middleware functions either before or after .run
Instead of use:
{
  before: [  'midd1'  ],
  after: [ 'midd2' ]
}

Thanks for suggestion, will implement in future releases.

Another recommendation would be to add an api to easily save data from the hooks in the global scope:
read-package.js hook
function(next) {
 // Open package.json if it exists and store its contents
  var pkg = ...
  this.state({
    version: pkg.version,
    dependencies: pkg.dependencies
  })
  next()
}

In my commands I can use the read-package middleware and retrieve the state:
use: [ 'read-package'],
run: function() {
  var pkg = this.hook('read-package').state()
  // pkg.version === '1.0.0'
  // pkg.dependencies === { ... }
}

This way you can very easily add common middleware that is used in different commands and have a clear convention of how the global state is accessed, instead of adding random properties to this.global

@agonbina I think this does not fit with the middleware/hooks concept. I imagine it as a chain of functions, that can affect the path to the main command or alter its behavior. For this use case I would rather make a separate script in lib/read-package.js and use it as a helper function.

I think in frameworks like express you can attach data to the middleware chain,  like the req object thats passed in as an arg to each middleware function. But yeah for that use case I could do as you said, but still I don't see a clean  way to share state between "hooks" as they are running, besides this.global[prop] = ...?

@agonbina Currently no, this.global is the only way to share data that Ronin provides.

:+1: I'll just use this.global[middleware-id] when I need it


Rename tests to examples

Rename tests to examples
Use poetry to run cookiecutter

Motivation
This is part of the efforts to make reviewing changes to templates easier. See #35 for the full list.

Current dependencies on/for this PR:

main

PR #38

PR #39   👈

This comment was auto-generated by Graphite.

@nick-mobilecoin Bump for restack


AttributeError:

Description

On start, the mod_rightclick module cannot load

2023-08-01 20:47:00.286 | INFO     | __main__:<module>:25 - Successfully found config/config.yml!
2023-08-01 20:47:00.292 | SUCCESS  | libs.utility:verify_config:162 - Config did not need repaired!
2023-08-01 20:47:00.294 | INFO     | __main__:<module>:38 - Successfully loaded config/config.yml: {'cog_dontload': ['bad_example.py'], 'debug': True, 'discord': {'co_owners': [883031597291569193], 'owner':<PHONE_NUMBER>41064293, 'roles': {'staff': 0}, 'testing_guild':<PHONE_NUMBER>638638110, 'token': 'REDACTED'}, 'fizzdev=catboy': True, 'messages': {'error': '⚠ **An Error Occured!** ``` [[error]] ``` Please report this to the devs.', 'noperm': '**🚫 No permission**', 'noperm_log': '[[user]] (ID: [[user_id]]) tried to run "[[command]]" but doesnt have permission!'}, 'mysql': {'db': 'tiramisu', 'host': 'localhost', 'pass': 'ImACatb0y!', 'port': '3306', 'user': 'tiramisu'}, 'storage': 'sqlite'}
2023-08-01 20:47:00.294 | DEBUG    | asyncio.selector_events:__init__:54 - Using selector: EpollSelector
2023-08-01 20:47:00.295 | WARNING  | nextcord.client:__init__:336 - PyNaCl is not installed, voice will NOT be supported
2023-08-01 20:47:00.295 | DEBUG    | nextcord.client:event:1275 - on_ready has successfully been registered as an event
2023-08-01 20:47:00.298 | DEBUG    | cogs.admin:setup:125 - Setup cog "admin"
2023-08-01 20:47:00.301 | DEBUG    | cogs.announce:setup:68 - Setup cog "announce"
2023-08-01 20:47:00.304 | DEBUG    | cogs.utilities:setup:100 - Setup cog "utilities"
2023-08-01 20:47:00.320 | DEBUG    | cogs.help:setup:53 - Setup cog "help"
Traceback (most recent call last):
  File "/home/tiramisu/.local/lib/python3.11/site-packages/nextcord/ext/commands/bot.py", line 741, in _load_from_module_spec
    spec.loader.exec_module(lib)  # type: ignore
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/tiramisu/Tiramisu/cogs/mod_rightclick.py", line 13, in <module>
    from libs import moderation, utility, modals
  File "/home/tiramisu/Tiramisu/libs/modals.py", line 100, in <module>
    class BugReportModal(nextcord.ui.Modal):
  File "/home/tiramisu/Tiramisu/libs/modals.py", line 101, in BugReportModal
    def __init__(self, channel: nextcord.Channel, questions):
                                ^^^^^^^^^^^^^^^^
AttributeError: module 'nextcord' has no attribute 'Channel'. Did you mean: 'channel'?

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/tiramisu/Tiramisu/bot.py", line 143, in <module>
    bot.load_extension(f'cogs.{filename[:-3]}')
  File "/home/tiramisu/.local/lib/python3.11/site-packages/nextcord/ext/commands/bot.py", line 863, in load_extension
    self._load_from_module_spec(spec, name, extras=extras)
  File "/home/tiramisu/.local/lib/python3.11/site-packages/nextcord/ext/commands/bot.py", line 744, in _load_from_module_spec
    raise errors.ExtensionFailed(key, e) from e
nextcord.ext.commands.errors.ExtensionFailed: Extension 'cogs.mod_rightclick' raised an error: AttributeError: module 'nextcord' has no attribute 'Channel'

Steps to reproduce

Run
AttributeError

Platform and Version
Operating System:

Linux
CPU Archetecture:

x86_64
Python Version:

Python 3.11.4
Tiramisu Version:

Check only those that apply

[x] I am running straight from git

Branch: dev-report_modals

[ ] I am running a release downloaded from the releases page

Release number:

When disabling the cog, another cog throws the same error. Here is a list of other cogs that throw this error after the previous ones do the same:

debug
applications
tickets
reporting

This does not happen in the main branch


Configurable GoCQL Timeout Period

On the slow inefficient queries (for example, with allow filtering) we run into GoCQL timeout period.
Suggested solutions:

Just raise the timeout
Make timeout configurable

Possible improvement:

Warn a user about an inefficient query

If you are using protocol version prior to 4, you can try to set Timeout paramter of cluster object in gocql to higher value.
https://stackoverflow.com/questions/42922757/getting-timeout-error-with-gocql
Not sure why it relates to the protocol version

The important point, it can relate not to GoCQL settings, but to the C* cluster settings. If it is, we cannot do anything, only describe it in the "Known Issues" wiki page.

GoCQL settings example:
  var c *gocql.ClusterConfig
    if env.Production {
        c = gocql.NewCluster(
            "<IP_ADDRESS>",
            "<IP_ADDRESS>",
            "<IP_ADDRESS>",
            "<IP_ADDRESS>",
            "<IP_ADDRESS>",
        )
    } else {
        c = gocql.NewCluster("<IP_ADDRESS>")
    }
    c.Consistency = gocql.One
    c.Keyspace = CassandraKeyspace
    c.Timeout = 1 * time.Minute.               <-------- HERE
    c.NumConns = 1
    c.Compressor = gocql.SnappyCompressor{}
    return c.CreateSession()
    ```

To be released as 1.1.0


[CodeCompletion] Suggest freestanding after @

Explanation: We were not suggesting freestanding after @
Scope: Code completion
Risk: Low since it only affects attribute completions
Testing: Added regression test
Issue: rdar://108163121
Reviewer: @rintaro on https://github.com/apple/swift/pull/65428

@swift-ci Please test


Definition List with multiple DD's displays incorrectly

A Definition List that has a definition with multiple descriptions will display incorrectly.
Specifically, a DT element followed by more than one DD elements will place the second DD element in the wrong position in the grid. This will cause the list to display incorrectly
To Reproduce
Attempt to display this Definition List using Just The Docs:
Phone
: First definition for a phone
: Second definition for a phone

You will see the following. The second description is too far to the left.

This is because it is being placed in the wrong grid position:

Desktop (please complete the following information):

OS: macOS Mojave
Browser: Chrome
Version: 80.0.3987.132

Additional Info
This causes a knock on effect for later definitions. If you look at this example:
Phone
: First definition for a phone
: Second definition for a phone

Car
: First definition for a car

You will see that the Car entry is too far to the right

@pmarsceill the "next minor release" label for this issue was added 3 months ago; the same goes for #277, which also involves styling for description lists. Could these issues be addressed in time for 0.3.2?
I'm not sure what styling would be best for dls in their full generality. The current grid styling works well for single-dt, single-dd lists where the dts are all rather short, but it doesn't look so good when there is a long dt:

Would it be possible to display all the dds on separate (indented) lines when any of the dts is wider than some fraction of the page width? Grid styling looks pretty complicated, and I've never tried using it myself…
Something looking like these examples might be better for users who want to exploit the full generality of the dl element  in semantic markup.
BTW, the JtD test example for description lists uses HTML syntax, but I think it's much nicer to use Kramdown's own markup for them, e.g.:
Name
: Godzilla

Born
: 1952

Birthplace
: Japan

Color
: Green

I suggest to replace the HTML in the test example by the above.

I wrote:

I'm not sure what styling would be best for dls in their full generality.  The current grid styling works well for single-dt, single-dd lists where the dts are all rather short, but it doesn't look so good when there is a long dt [...]

For typical uses of description lists, it seems reasonable to assume that dts are all quite short, whereas dds can be more than one line. Perhaps the following layout would be suitable?
           Term: Brief description of Term

    Longer Term: Longer description of Term,
                 possibly more than one line

           Term: First description of Term,
                 possibly more than one line

                 Second description of Term,
                 possibly more than one line

          Term1:
          Term2: Single description of Term1 and Term2,
                 possibly more than one line

          Term1:
          Term2: First description of Term1 and Term2,
                 possibly more than one line

                 Second description of Term1 and Term2,
                 possibly more than one line

PR #401 provides the layout sketched above.


new folder disappears

I create a new folder and rename, and after seconds I reload the webpage , the folder disappears.
This happen almost constantly.
Sorry I don't know how to reproduce it here.

I have not been able to see this happening on my end. Which version are you using? This is shown in the about dialog (click the 'i' icon on top-right).

Version: 0.5.20151125
This happened many times today morning, but now works fine.
It is very strange indeed.


Migrate from lazy_static to once_cell

lazy_static involves macros, takes a bit longer to compile, and once_cell on the way to being added to std.
This PR uses const_vec3a from glam where possible and use once_cell::sync::Lazy instead where const is not available.

Thanks for this!
It looks good on my end, I think I might undo some of the formatting, just for legibility reasons, but otherwise it's good.
I'll merge this and add the reformatting + version bump in a separate commit.


Is there a roadmap for FreeSurfer.jl?

Congratulations on this fantastic idea! I was really hoping something for Freesurfer in Julia was going on.
Is there a roadmap? Should Freesurfer.jl implement the recon-all streams as well? I would definitely like to help with that :)

Hi! It started as a bit of a pet project, so it's been focused on dMRI so far, but the sky is the limit! Would love to hear what others would be interested in implementing. Any chance you'll be at OHBM?

I'll be there, yes! Establishing a roadmap could be a nice project for one of the Brainhacks as well

I'd definitely be up for doing a julia project at Brainhack if there's interest!

I'm on the OHBM BrainHack organizing team this year, so I don't know if I can commit this time, but I'm definitely interested.

No worries, it can be December. But it'd be good to poll people for interest at OHBM brainhack so we can have at least some discussion ahead of time.

This might be interesting for a prospect of porting the recon-all routines to Julia:
https://surfer.nmr.mgh.harvard.edu/fswiki/ReconAllFilesVsSteps


MessageBus: Only have one logical implementation, refactoring TCP out into swappable Network

If we fix #70 then we need to do the same in Network.deliver_message to match this logic.
But this is a sign that we actually need to refactor MessageBus a little so that it accepts a swappable Network which does the TCP connection stuff that MessageBus.Connection currently does.
The goal of this refactor would be to get rid of test/message_bus.zig completely and make sure that we’re testing the exact same message bus logic.
@ifreund  what are your thoughts on this?

Is this still relevant, since there is no header-only message pool anymore, and the message pool isn't owned by the MessageBus? I don't see much (if any) shared logic between the primary & test MessageBus implementations.

If there's no longer any shared logic at all between primary and test implementations, then let's close!


[DOC] remove references to mtype/machine type scitype/stype

the terms mtype/machine type and scitype/stype were introduced with the datatype module. These are non standard terms that mean nothing to most people and are sure to cause confusion.
For example, the header for the _convert module is this
"""Machine type converters for scitypes."""
and comments like this
"obj : object to convert - any type, should comply with mtype spec for as_scitype
there is no reference here to what these mean nor a link to an explanation. As far as I can tell it is a reinvention of the concept of abstract data types with some weird inclusion of objects. So a scitype would be an abstract data type such as a list, an mtype would be an implementation of a list such as a linked list or an array. I see absolutely no need for this distinction and think all is does is confuse things

it basically gives a form out automation to convert, but I find it convoluted and confusing.
Exports
-------
convert_to(obj, to_type: str, as_scitype: str, store=None)
    converts object "obj" to type "to_type", considerd as "as_scitype"

convert(obj, from_type: str, to_type: str, as_scitype: str, store=None)
    same as convert_to, without automatic identification of "from_type"

mtype(obj, as_scitype: str)
    returns "from_type" of obj, considered as "as_scitype"
---

I would like to remove these terms completely, and replace them with, for example
input type/internal type/output type
then have simple converters that convert from one to another and remove the scitype
just looking at usages of this, to_type is a required parameter, so all this does
def convert_to(
    obj,
    to_type: str,
    as_scitype: str = None,
    store=None,
    store_behaviour: str = None,
):

is some convoluted checking, then calls convert! This adds no value I can see. In first instance I would just change the terminology, but really I doubt the below adds any value at all.
   # input checks on to_type, as_scitype; coerce to_type, as_scitype to lists
    to_type = _check_str_or_list_of_str(to_type, obj_name="to_type")

    # sub-set a preliminary set of as_scitype from to_type, as_scitype
    if as_scitype is not None:
        # if not None, subset to types compatible between to_type and as_scitype
        as_scitype = _check_str_or_list_of_str(as_scitype, obj_name="as_scitype")
        potential_scitypes = mtype_to_scitype(to_type)
        as_scitype = list(set(potential_scitypes).intersection(as_scitype))
    else:
        # if None, infer from to_type
        as_scitype = mtype_to_scitype(to_type)

    # now further narrow down as_scitype by inference from the obj
    from_type = infer_mtype(obj=obj, as_scitype=as_scitype)
    as_scitype = mtype_to_scitype(from_type)

    # if to_type is a list, we do the following:
    # if on the list, then don't do a conversion (convert to from_type)
    # if not on the list, we find and convert to first mtype that has same scitype
    if isinstance(to_type, list):
        # no conversion of from_type is in the list
        if from_type in to_type:
            to_type = from_type
        # otherwise convert to first element of same scitype
        else:
            same_scitype_mtypes = [
                mtype for mtype in to_type if mtype_to_scitype(mtype) == as_scitype
            ]
            if len(same_scitype_mtypes) == 0:
                raise TypeError(
                    "to_type contains no mtype compatible with the scitype of obj,"
                    f"which is {as_scitype}"
                )
            to_type = same_scitype_mtypes[0]

So, after 1117 we are left with transformations and forecast base class dependencies
from aeon.datatypes import (
    VectorizedDF,
    check_is_mtype,
    check_is_scitype,
    convert_to,
    mtype_to_scitype,
)

and 16 calls in forecasting to
from aeon.datatypes import (
check_is_mtype,
check_is_scitype,
convert,
convert_to,
mtype_to_scitype,
scitype_to_mtype
)
Easiest now to address the conversions, leaving the VectorizedDF to last

Maybe some, but will raise a new issue: KILL WITH FIRE


404 during installation

The installer 404s when installing ArcDPS + ArcDPS Mechanics + d912pxy. The file it references is ArcDPS' build templates, which apparently has been removed as of 10-17-2019, https://www.deltaconnected.com/arcdps/ - It won't let me install ArcDPS, or any of the other addons because of this.

https://www.reddit.com/r/Guildwars2/comments/dhspu0/arcdps_build_templates_to_be_disabled_tomorrow/ :(

Hotfixed in v0.4.5


We should use buffered chans.

          We should use buffered chans.

Originally posted by @adrielp in https://github.com/liatrio/liatrio-otel-collector/pull/123#discussion_r1335243292

I think this is already done https://github.com/search?q=repo%3Aliatrio%2Fliatrio-otel-collector+"chan+"&type=code

yup, this was a duplicate and the other one didn't look to get closed. thanks for catching this!


Feature Request: Hide Columns by Name along with Index Numbers

Nice idea

Hi @asthinasthi,
I'm pleased to announce you that this new feature is available in version 1.5.0, just released!
You can get it here:
https://github.com/fbaligand/kibana-enhanced-table/releases/tag/v1.5.0


Feature:Supporting picking year

Can we separately pick day, month, year ?

Ok, I will add year, but it will be an option ;)
by default it will not be visible, you can enable it

But, the today / tomorrow will not have any sense ?

I understand. You can have 4 modes.

The current view (default)
Year, Month, Day View
Hour, Minute View
Year, Month, Day View, Hour, Minute View.

why not ;)
2017-06-29 9:30 GMT+02:00 Debjit<EMAIL_ADDRESS>
I understand. You can have 4 modes.

The current view (default)
Year, Month, Day View
Hour, Minute View
Year, Month, Day View, Hour, Minute View.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/florent37/SingleDateAndTimePicker/issues/60#issuecomment-311885743,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFfQXIqZtEqPY8xStVwjlzl6_AdxacWIks5sI1J4gaJpZM4OHpyc
.

:)

how to enable Year, Month, Day View, Hour, Minute View. ???

Year, Month, Day View in progress?

I love this widget! Any progress on year picking? :)

Getting year from selected date doesn't work. Regardless of which year I choose it shows 2018:
Date date = enddatetimePicker.getDate();
SimpleDateFormat simpleDateFormat = new SimpleDateFormat("E, dd MMM yyyy @ HH:mm");
tvValueEndDateTime.setText(simpleDateFormat.format(date));
dialog.dismiss();


I have a question.

Modify some code in 'eval_yolo.py'.
But that result FPS is lower than 'README.md' FPS. ( about 1~5FPS)
So I don't know what is problem.
Is there something wrong?

I measure FPS with "trt_yolo.py" directly.
You would see a lower FPS with "eval_yolo.py" because:

Detection confidence threshold (1e-2) is lower.  So the postprocessing (NMS) would take longer.
Reading various jpg files causes additional loading on the CPU and I/O.  It would slow down inference code a little bit.

ok thank u


"saveLinks: true" removes existing links in putSync

Steps to Reproduce
I have 2 levels of IsarLink.
For class A,B and C ... A contains Link to B, B contains Link to C
First create object for B and save the link value (C).
Now, when I create object A and save the link value (B), the link between B and C breaks.
Code sample
class A {
  Id id = Isar.autoIncrement;
  final bLink = IsarLink<B>();
}

class B {
  Id id = Isar.autoIncrement;
  final cLink = IsarLink<C>();
}

class C {
  Id id = Isar.autoIncrement;
  final String name;
  C(this.name);
}

void test() {
  C cObject = C("Simon");
  B bObject = B();
  bObject.cLink.value = cObject;

  isar.writeTxnSync(() {
    isar.collection<B>().putSync(bObject, saveLinks: true);
  });
  // fetching B gives: {id: "idB", cLink: {id: "idC", name: "Simon"}} -> correct behaviour

  // next part takes place in a different place in my code
  A aObject = A();
  aObject.bLink.value = bObject;
  isar.writeTxnSync(() {
    isar.collection<A>().putSync(aObject, saveLinks: true);
  });
  // fetching A gives: {id: "idA", bLink: {id: "idB", cLink: null}} -> cLink value becomes null
}

// An example that does not result in this issue.
void working_test() {
  C cObject = C("Simon");
  B bObject = B();
  bObject.cLink.value = cObject;
  A aObject = A();
  aObject.bLink.value = bObject;
  isar.writeTxnSync(() {
    isar.collection<A>().putSync(aObject, saveLinks: true);
  });
  // fetching A gives: {id: "idA", bLink: {id: "idB", cLink: {id: "idC", name: "Simon"}}} -> correct behaviour
}

Version

android-arm64-emulator
Flutter version:3.3.0
Isar version: 3.0.0-dev.8
Isar Generator version: 3.0.0-dev.8

I cannot seem to be able to reproduce this. Could you try the latest version, 3.0.0-dev.14?

@Jtplouffe I tried with 3.0.0-dev.14 as well.
Here's a working example of the issue https://github.com/shivamstaq/isar_test

@Jtplouffe I tried with 3.0.0-dev.14 as well. Here's a working example of the issue shivamstaq/isar_test

I was able to replicate it.
It seems that when saving an object with saveLinks: true, and that object contains links to object that are already in the database, that linked object will have their links cleared.
E.g.
Let's say we have the following schema: A -> B -> C
If we save an instance of C into the database, then we save an instance of B, which has a link to the saved C, and after that we save an instance of A, which has a link to the saved B, the B will have it's links cleared.
I will try to look into this

To add to this,
For some reason, putSync(obj, saveLinks:true) only saves IsarLink and does not save IsarLinks (specially for second order links)
Example:
class Parent {
  final Id id = Isar.autoIncrement;
  final aLink = IsarLink<A>();
}

class A {
  final Id id = Isar.autoIncrement;
  final bLink = IsarLink<B>();
  final bLinks = IsarLinks<B>();
  final cLinks = IsarLinks<C>();
}

parentObj = Parent();
obj = A();
obj.bLink.value = B('test');
obj.bLinks.addAll([B('test), B('test1')]);
obj.cLinks.addAll([C(1), C(2)]);

parentObj.aLink.value = obj;

isar.writeTxnSync(() {
  isar.collection<A>().putSync(parentObj, saveLinks: true);
})

this results in obj only being populated with bLink value, bLinks and cLinks remain empty

After trying out a lot of stuff, I think I have found the issue.
The issue comes down to the fact the the id in your collections are final.
That means that when creating and storing object C, the id on the instance cannot be updated by isar.
So removing the final on ids and re-running the build runner seems to fix this.
@leisim The only thing that I'm wondering is why didn't isar create new instances, since the id was still Isar.autoIncrement after storing the object. Maybe it's silently failing because of the unique String id index?

To add to this, For some reason, putSync(obj, saveLinks:true) only saves IsarLink and does not save IsarLinks (including for second order links)
Example:
class Parent {
  final Id id = Isar.autoIncrement;
  final aLink = IsarLink<A>();
}

class A {
  final Id id = Isar.autoIncrement;
  final bLink = IsarLink<B>();
  final bLinks = IsarLinks<B>();
  final cLinks = IsarLinks<C>();
}

parentObj = Parent();
obj = A();
obj.bLink.value = B('test');
obj.bLinks.addAll([B('test), B('test1')]);
obj.cLinks.addAll([C(1), C(2)]);

parentObj.aLink.value = obj;

isar.writeTxnSync(() {
  isar.collection<A>().putSync(parentObj, saveLinks: true);
})

this results in obj only being populated with bLink value, bLinks and cLinks remain empty

Regarding this issue, I was also able to reproduce it.
Like stated in my previous comment, making the ids not final makes it so that the objects will be saved properly.
There is still another issue, which is that we cannot retrieve the added object from the links if the object is not saved (attached) or if the id of the object is Isar.autoIncrement.
This is more of a limitation, since links objects are stored in a map of <Id, OBJ>, so multiple objects could have Isar.autoIncrement.
After saving the object, we can query it and the links will be correct.
In file isar_links_common.dart:
bool add(OBJ value) {
  if (isAttached) {
    final id = getId(value);
    if (id != Isar.autoIncrement) {
      if (_objects.containsKey(id)) {
        return false;
      }
      _objects[id] = value;
    }
  }

  removedObjects.remove(value);
  return addedObjects.add(value);
}

This means that OBJ with Isar.autoIncrement will be saved, but won't be "loaded".
I will check for the possibility of adding objects that are only in addedObjects in _objects on save.

@leisim Regarding the first issue (and the previous comment), do you think we should prevent ids from being final? It could prevent user errors and confusion.
The downside is that collection could no longer be const, since all properties won't be final.


Add file_limit support for RedHat platforms

After fixing the exec issue, this can be merged.

@DavidS thanks for the review, fixed the exec title and rebased !

Excellent, thanks!


Create link for phone and web wrong

There are some wrong when display phone and weblink
Example: https://big-up.style/tokyobigup only https://big-up.sty highlight other black.

@mdtuyen https://github.com/molon/MLLabel/pull/58  thanks for suggestion. :)


v1.2.2 breaks publishing when using semantic-release-monorepo

Hello,
We use lerna, semantic-release and semantic-release-monorepo to version and publish our projects' shared modules.
The change that triggered<EMAIL_ADDRESS>has broken our pipeline.
This is because in order to use semantic-release-monorepo you need to put your analyzeCommits and generateNotes configurations inside a monorepo Object of your release configuration.
See the semantic-release-monorepo: Example of use with non-default set of plugins for an example of this.
If we include an release.analyzeCommits configuration, then semantic-release-monorepo does not execute at all. If we put it inside a monorepo Object, as outlined above, then it is now ignored because of the change in<EMAIL_ADDRESS>and as such, neither out preset: eslint, nor our custom releaseRules: Object get picked up by semantic-release-monorepo, which then prevents semantic-release from versioning and publishing, as it can't find any valid changes.
I'm unsure if this will also affect other plugins which use semantic-release-plugin-decorators, however, I will also file this issue in the semantic-release-monorepo repository.
For now, we've hard versioned semantic-release-plugin-decorators to v1.2.1, at the same time, it would be nice to not be locked into this version to be able to receive future updates. 🙂
Thanks,
Christos.

Thanks for the report @constantology! This should be fixed with 1.2.3


Error fetching Visitor using SDK

Summary
Trying to fetch using the SDK results in an error. This is only affecting the SDK, not the API endpoint.
Able to replicate it trying to:

fetch the visitor object.
fetching dropped assets

Expected Behavior
An object is returned, with the visitor information.

Current Behavior
Getting an error { message: 'User matching the API key not found.' } with status 404

Possible Solution
Not sure

Steps to Reproduce
1, After creating a visitor using the SDK
const topia = await new Topia({
     apiDomain: "api.topia.io",
     apiKey: process.env.API_KEY || "",
     interactiveKey: process.env.PUBLIC_KEY || "",
     interactiveSecret: process.env.PRIVATE_KEY || "",
 });

const visitor = new VisitorFactory(topia).create(req.body.visitorId, urlSlug);

Trying to fetch the visitor:

console.log(await visitor.fetchVisitor())
or
const world =  await new WorldFactory(topia).create(process.env.URL_SLUG);
await world.fetchDroppedAssets();

Results in Error: { message: 'User matching the API key not found.' }

Context (Environment)
Might be only an issue with my account. I have changed my API_Key and tested with the updated key.

@saqibmk This is working for me both locally and in production. Can you please verify you're using the correct API key for the environment and let me know if you still run into issues? Also try updating your code to the following:
const Visitor = new VisitorFactory(topia)

const visitor = Visitor.create(req.body.visitorId, urlSlug)
// or
const visitor = Visitor.get(req.body.visitorId, urlSlug)

@LinaBell I updated the API Key and also tried generated a new one. Still getting the same issue.

Invalid credentials, no fix needed


How to Update Chart Data

Hi, I'd suggest you take a look at the sets example. The chart will automatically update and redraw whenever the props are updated.

Gonna take the thumbs up as a sign that that's what you were looking for :)


fix: remove additional API call and fetch data from props

IDE-520
Description:
1: On confirm-change popup open, the getPlan API was again being called which is unnecessary as plan can simply be passed from the parent component which calls plans list API. Instead we passed the plan as search param to the navigate function.
2: This effect hook was unnecessarily called. Need to confirm if removing this can have any side effects.
useEffect(() => {
    fetchActiveSubsciption();
}, [fetchActiveSubsciption]);

Testing
Ongoing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 9.651%

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
10836

Relevant Lines:
112283

💛  - Coveralls


add configuration on a per-subreddit basis through the sheevbot wiki page

currently it pulls all settings from the sheevbot wiki page on /r/PrequelMemes

I think this is resolved


⚠ Bofrost.at has degraded performance

In 6e819a7, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1266 ms

Resolved: Bofrost.at performance has improved in 1bb57b1.


Support Usage Tokens Output in Claude API

Is your feature request related to a problem? Please describe.
Currently, reporting usage dictionary from OpenAI API is supported as seen in this document and usage dictionary. https://python.useinstructor.com/concepts/usage/?h=token+usage
However, Claude API patch does not have this functionality, even though usage is available from a successful 200 response from Anthropic's server:
200 Response from https://docs.anthropic.com/en/api/messages
{
  "content": [
    {
      "text": "Hi! My name is Claude.",
      "type": "text"
    }
  ],
  "id": "msg_013Zva2CMHLNnXjNJJKqJ2EF",
  "model": "claude-3-opus-20240229",
  "role": "assistant",
  "stop_reason": "end_turn",
  "stop_sequence": null,
  "type": "message",
  "usage": {
    "input_tokens": 10,
    "output_tokens": 25
  }
}

Describe the solution you'd like
Instructor should patch Claude's API and surface the usage dictionary as part of the output in the second tuple like so:
structure_output, completion = client.chat.completions.create_with_completion(...)
completion.usage # should returns usage, consists of input and output tokens

Use Anthropic Claude through LiteLLM, the usage and cost gets reported
import instructor
from litellm import completion
from litellm import completion, completion_cost, cost_per_token
from pydantic import BaseModel

class User(BaseModel):
    name: str
    age: int

client = instructor.from_litellm(completion)

resp, completion = client.chat.completions.create_with_completion(
    model="claude-3-opus-20240229",
    max_tokens=1024,
    messages=[
        {
            "role": "user",
            "content": "Extract Jason is 25 years old.",
        }
    ],
    response_model=User,
)

assert isinstance(resp, User)
assert resp.name == "Jason"
assert resp.age == 25

usage = completion.usage
input_tokens = usage.prompt_tokens
output_tokens = usage.completion_tokens
total_tokens = usage.total_tokens
    input_cost_usd, output_cost_usd = cost_per_token(model, prompt_tokens=input_tokens, completion_tokens=output_tokens)
    completion_cost_usd = completion_cost(completion_response=raw_result)

Describe the solution you'd like
Instructor should patch Claude's API and surface the usage dictionary as part of the output in the second tuple like so:

structured_output._raw_response.usage works but doesn't take retries into account.
@jxnl maybe we attach cumulative usage data here? It's currently getting lost while processing response. https://github.com/jxnl/instructor/blob/081418d59a397b38a1b66fe58a64ef94f9124a6b/instructor/process_response.py#L97-L100

Describe the solution you'd like
Instructor should patch Claude's API and surface the usage dictionary as part of the output in the second tuple like so:

structured_output._raw_response.usage works but doesn't take retries into account.
@jxnl maybe we attach cumulative usage data here? It's currently getting lost while processing response.
https://github.com/jxnl/instructor/blob/081418d59a397b38a1b66fe58a64ef94f9124a6b/instructor/process_response.py#L97-L100

related https://github.com/jxnl/instructor/issues/715

usage or other completion param doesn't work for Iterables

'list' object has no attribute '_raw_response'


Add box background from Acernity UI

You have to use acernity UI and add the box background.

I would like to work on it

assigned.

Current Implementation
https://github.com/Sayak-Bhunia/mystory/assets/116720762/41d6fee1-00fa-4f9b-8a7b-aeff0cdb31c3

Looks good please adjust the width of some components as they are not responsive.

Is this fine?
https://github.com/Sayak-Bhunia/mystory/assets/116720762/35e16eab-39fb-4aef-aa77-9468fb0ef3c4

U see the components from acernity ui are slightly on the right side

Oh okay okay

I have fixed the spacing

(maybe box layout may give a feel that the components are not aligned)

https://github.com/Sayak-Bhunia/mystory/assets/116720762/f3062fe7-2ae9-4fc2-965b-be30024a08ca

merged.


Example: Uart send at_commands

Contribution description
The best way to get a communication between an esp32 and a samr21, is using the UART' s from both devices. So there are a older example of Uart tah will be worked to integrated a series of commands at, to set the wifi configuration, uart protocol in this PR only give support to all commands AT that was worked and integrated in wifi-subsys

Testing procedure

Issues/PRs references

this PR it's being developed into #155 so this will be closed


fix(autolayout): When SpaceBetween no spacing

GitHub Issue (If applicable): #1128

PR Type
What kind of change does this PR introduce?

Bugfix

What is the current behavior?

when in spacebetween the spacing is still use, setting a minimal space between autolayout children

What is the new behavior?

when spacebetween the spacing will be ignore

PR Checklist
Please check if your PR fulfills the following requirements:

[c] Tested code with current supported SDKs
[ ] Docs have been added/updated which fit documentation template. (for bug fixes / features)
[ ] Unit Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
[ ] Wasm UI Tests are not showing unexpected any differences. Validate PR Screenshots Compare Test Run results.
[c] Contains NO breaking changes
[ ] Updated the Release Notes
[c] Associated with an issue (GitHub or internal)

Other information

Internal Issue (If applicable):

@Robert-Louis Are you able to add a test ?

@jeromelaban I was planning adding test during the day for various autolayout scenario. I'm not 100% at ease with the runtime so I wanted to take time for this.

@jeromelaban I added the test


Adding support of Scheduling a notification

Describe the solution you'd like
As a business user, i would like to have an ability to schedule a campaign/notification using App Center portal. The notification/campaign should be sent only as per scheduled date/time.
Describe alternatives you've considered
Currently, we are exploring the option of implementing the scheduling of notifications at our end. We would handle the scheduling functionality portion and then create/send the notification from the code using the App Center SDK.
Additional context
Scenarios could be as below:

Send a notification after a new build is released as per schedule.
Send a notification on new feature release as per schedule.

@mehul19851 thanks for the suggestion.

@mehul19851 that feature is in Notification Hubs today, will likely make it into App Center push later.

Hey @jwargo, just wondering if you have any updates here.  I'd use scheduled push notifications today if it were available.  Since it's not, I am building the scheduling on my end instead.
Thanks!

@jacob-israel-turner No update yet, I've added it to our requested feature list and I'm working on prioritizing the list now.

Great - thanks so much @jwargo!

Thank you very much for your feedback on App Center Push. We announced yesterday in a blog post that we’re retiring the Push service later this year and recommending customers transition to Azure Notification Hubs instead. The good news is that the same team operates both App Center Push and Azure Notification Hubs, so all customer feedback and feature requests have been transferred to the Azure Notification Hubs roadmap.
The good news is that Azure Notification Hubs already supports scheduled notifications.


Password reset is missing client-side data validation

Steps

Go to osf.io
Go to your Account Settings
Without filling in any of the boxes, click "Update Password"

Expected behavior
I expected red text to appear above the button warning me that the fields are blank.
Actual behavior
There is an unnecessary server side call for validation as the page reloads.

@lbanner Can I work on this?

OK


Adding support for the BMA400 accelerometer

Hi,
Adding support for the BMA400 accelerometer.

Hi,
Since Chocolatey is currently not available it is not possible to test on Windows. Kindly bear with the ping-pong between the CI test and related fixes.

Devicetree bits look fine. A few comments.

No chance we can make this multi-instance like we already did for BME280? https://github.com/zephyrproject-rtos/zephyr/pull/23741

Devicetree bits look fine. A few comments.

No chance we can make this multi-instance like we already did for BME280? #23741

Do you mean SPI and I2C support?

No chance we can make this multi-instance like we already did for BME280? #23741

Do you mean SPI and I2C support?

That would be multiple bus support. "Multi-instance" means automatic allocation of device instances based on the contents of the devicetree using DT_INST_FOREACH as done in #23741.

See also https://docs.zephyrproject.org/latest/guides/dts/howtos.html#create-struct-devices-in-a-driver

See also https://docs.zephyrproject.org/latest/guides/dts/howtos.html#create-struct-devices-in-a-driver

Probably in an upcoming update. This PR was meant only for basic functionality. The upcoming update will also include interrupts and features like the Step counter.

This PR was meant only for basic functionality.

OK, understood. No objections.

@BoschSensortec any plans to update this PR?


Refactored notifications & extensions.

Summary/Changes Made

Refactored notifications to support passing the sender name,
Added support for JSON based views using 'DynamicUI',
Moved the broadcaster handler to its own function/extension.
Created 'ExtensionDoc_fromAE' to document actions which can be received by an extension
Created 'ExtensionDoc_toAE' to document which actions can be sent by an extension"

TODO
None
Motivation
Extension system
Testing
Nope.
Screenshots/GIFs

How to replicate:
create ~/Library/Application Support/com.auroraeditor/Extensions/test.JSext/extension.js
with those contents:
AuroraEditor.respond("openWindow", {
  view: '[{"type": "Text","title": "Wait, am i generating views from JSON?"}]',
});

Checklist
Please ensure all of the following are completed before submitting the PR:

[x] Code has been tested and verified.
[x] Documentation has been updated to reflect the changes.
[x] All tests pass successfully.
[x] Code follows the project's coding guidelines and style.
[x] The branch is up-to-date with the latest changes from the main branch.
[x] Reviewed by at least one other contributor.

Related Issues
This PR addresses the following issue(s): None.
Additional Notes
None.

Thanks for submitting a pull request!
We will review it as soon as possible.

Hey @0xWDG,
Thanks for your contribution.
Your PR has been merged!
Your name will appear on https://auroraeditor.com/#contributors in a few moments, if you have contributed to the editor.
Thanks again for your contribution!
Kind Regards,
Aurora Editor Team.


feat(client): support opening masks by setting "_url_getter"

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 18 (22.22%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 83.947%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tensorbay/client/segment.py
4
18
22.22%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
5517

Relevant Lines:
6572

💛  - Coveralls


Immplement compiler support for leg groups

add support for a command to define groups of legs that can be used in move and other commands to specify that the action should be applied to each of the legs in the group. Group names will be single letters, case insensitive, and the following groups will be defined by default:
a - all - 1,2,3,4,5,6
b - back - 3,6
f - front - 1,4
l - left - 4,5,6
r - right - 1,2,3
o - odds - 1,3,5
e - evens - 2,4,6
m - middles - 2,5
Also add support in move commands for specification of a leg group in place of a single leg number

added support for script command MoveRelLastLocal which is a leg move relative to the last (i.e. current) leg position

next git merge for this branch doesn't deliver working leg groups, but does have the test script required for issue 14.
I'm going to try to keep this branch open, for further work on leg groups.


Fix site header to be more generic

Lets fix it to say "mmo marketplaces" instead of ffxiv

It nearly needs it's own domain imo.
Keen to make it a monorepo but it feels weird having a split focus.
@oldben87 what are your thoughts?
I've not made a monorepo in the last couple of years so not sure how much work they are.
Otherwise we'll just start splitting the UI to make it obviously different.

Is the saddlebags name very specific to final fantasy?

Kinda, but I dont have any paid features in mind for wow yet.  Part of the reason for the wow searches is to find a promoter willing to make videos on us and share the word because no one in ffxiv will do that atm. Theres definitely some overlap between the two.
2 domains means no promotion for the ffxiv stuff.

Organic search is going to be more valuable than marketed concepts.
2 domains enables organic search considerably easier unless someone is gonna start writing a shit ton of content in the form of blog posts or the like.

Side note, on a route, add a meta function and create a description tag and/or opengraph tags.
See: https://remix.run/docs/en/v1/api/conventions#meta


[DSR] - Azure Linux 3.0 ARM64

Name of the Distribution
Azure Linux
Version of the Distribution
3.0
Package Types

[ ] Deb
[ ] RPM
[ ] Tar.gz
[ ] Snap - Stop! Please file your issue in PowerShell-Snap instead.

Processor Architecture
ARM64
.NET Core Support

[X] The version and architecture of the Distribution is supported by .NET Core.

Distribution Requirements

[X] The version of the Distribution is supported for at least one year.
[X] The version of the Distribution is not an interim release or equivalent.

Exemption Justification
No response
Progress

[ ] An issue has been filed to create a Docker image in PowerShell-Docker

For PowerShell Team ONLY

[ ] Docker image created
[ ] Docker image published
[ ] Distribution tested
[ ] Update packages.microsoft.com deployment
[ ] Lifecycle updated
[ ] Documentation Updated

What is the difference between Azure Linux and Mariner? PowerShell already does releases for Mariner, it is the only distribution that it already produces ARM packages for.
What's the difference between Azure Linux and Mariner?

Azure Linux and Mariner are the same image with different branding. Please use the Azure Linux OS SKU when referring to the image on AKS.

Hi @rhubarb-geek-nz, this is a build request for an update to Azure Linux that the PowerShell team will be working on in the near future. When complete, we will announce the release details in our lifecycle docs here: https://learn.microsoft.com/en-us/powershell/scripting/install/installing-powershell-on-linux?view=powershell-7.4

https://github.com/PowerShell/PowerShell-Docker/pull/829 is merged. does it mean this issue is completed?


Problem with parsing int64 values?

When I'm trying to insert BigInt and Timestamp values, I get errors, both with prepared statements and appenders. I'm not sure if it's a bug or me doing something wrong, because there are no mentions of BigInts and Timestamps in the documentations for now.
How to Reproduce the Problem
Here is a minimal test case:
import { DuckDBInstance, DuckDBTimestampValue } from '@duckdb/node-api';

const db = await DuckDBInstance.create(':memory:');
const connection = await db.connect();

await connection.run(`
  CREATE TABLE IF NOT EXISTS test (
    id BIGINT PRIMARY KEY,
    value INTEGER NOT NULL,
    timestamp TIMESTAMPTZ NOT NULL
  );
`);

const query = await connection.prepare(
  'INSERT INTO test (id, value, timestamp) VALUES ($1, $2, $3);',
);
// throws "bigint out of int64 range"
query.bindBigInt(1, BigInt(1));
// Works as expected
query.bindInteger(2, 100);
// throws "micros out of int64 range"
query.bindTimestamp(3, new DuckDBTimestampValue(BigInt(Date.now()) * 1000n));

await query.run();

const appender = await connection.createAppender('main', 'test');

// throws "bigint out of int64 range"
appender.appendBigInt(BigInt(1));
// Works as expected
appender.appendInteger(100);
// throws "Unimplemented type for cast (INT64 -> INT32)""
appender.appendTimestamp(new DuckDBTimestampValue(BigInt(Date.now()) * 1000n));
appender.endRow();

appender.close();

Specifications

Version: 1.1.3-alpha.7
Platform: Bun 1.1.38
OS: Manjaro Linux, kernel 6.6.63-1-MANJARO

Interesting. I tested the above on Node v20.14.0 and it works fine. Perhaps this is something specific to Bun?

Sure enough, I ran the exact same script on the same machine using Bun, and it failed in the ways you describe. Could be a bug in Bun's implementation of the Node Addon API.

Yup, there's a TODO in the Bun code regarding the lossless parameter of napi_get_value_bigint_int64 (and similar functions). This could definitely lead to some of the errors mentioned.

Asked about this in the relevant Bun issue.

Should be fixed in Bun v1.1.40.

Yes, it seems to be working as expected now. Thank you!

Thanks for the quick fix @190n!


"Need type annotation for" for ManyToManyField

I want to migrate away from the django mypy plugin (it currently doesn't work for newer mypy versions).
I have taken this example from the Django website:
from django.db import models

class Publication(models.Model):
    title = models.CharField(max_length=30)

    class Meta:
        ordering = ['title']

    def __str__(self) -> str:
        return self.title

class Article(models.Model):
    headline = models.CharField(max_length=100)
    publications = models.ManyToManyField(Publication)

    class Meta:
        ordering = ['headline']

    def __str__(self) -> str:
        return self.headline

Running mypy version (0.910) on this file:
$ mypy --strict models.py
models.py:14: error: Need type annotation for "publications"
Found 1 error in 1 file (checked 1 source file)

I think the ManyToMany needs to be typed as:
publications = models.ManyToManyField[Publication, Any](Publication)

# or maybe
publications = models.ManyToManyField[Publication, object](Publication)

I think the second type argument is the through param which is optional.
I don't think there is a way to provide a default generic param though, maybe we could handle it with overloads?

Edit: it might be worth reviewing whether the extra generic param for the through table hurts more than it helps aka maybe we should remove it

This hurts a lot for people that uses through though, because it makes it impossible to have the intermediate table automatically typed when you do this. For example, most of my ManyToMany fields that I use in my projects usually has a through table, which contains intermediate information.
I don't know much about how mypy --strict works, but shouldn't it fall to this overload: https://github.com/sbdchd/django-types/blob/main/django-stubs/db/models/fields/related.pyi#L427 ?
Also, it seems that the problem is related to the fact that there's a __init__ together with __new__. The real proper way to fix this and any other issues, which should also work for both mypy and pyright, would be to remove all __init__ from all fields and keep just the __new__ in them. That would mean having to quadruplicate the __new__ signature in all of them. Maybe it is for the best to do that?

IMO this can probably be closed as it was already solved.

So how can we actually annotate a ManyToManyField? Is it worth adding this to the docs?

So how can we actually annotate a ManyToManyField? Is it worth adding this to the docs?

The same way as @sbdchd suggested, but you can pass the through model as the second argument if you have any.
e.g.:
publications = models.ManyToManyField[Publication, Any](Publication)

publications = models.ManyToManyField[Publication, object](Publication)

# to be at least more specific that it is a Model subclass
publications = models.ManyToManyField[Publication, models.Model](Publication)

# for ManyToManyField with through
publications = models.ManyToManyField[Publication, "ArticlePublication"](
    Publication,
    through="ArticlePublication",
)

And IMO probably worth mentioning in the docs

In what Python version are you able to subscript ManyToManyField like that?
TypeError: type 'ManyToManyField' is not subscriptable

Python 3.11, Django 4.2.4

In what Python version are you able to subscript ManyToManyField like that?
TypeError: type 'ManyToManyField' is not subscriptable

Python 3.11, Django 4.2.4

You need to install and use the django_stubs_ext package, or monkey patch it yourself like this:
for cls in [
    QuerySet,
    RawQuerySet,
    BaseManager,
    models.ForeignKey,
    models.JSONField,
    models.ManyToManyField,
]:
    cls.__class_getitem__ = classmethod(lambda cls, *args, **kwargs: cls)

I personally monkey patch it myself as it seems easier for me =P

Ah! I can't believe I missed that! Probably because I haven't had to install it so far, I haven't needed it for Manager[Model] etc. Perhaps these have been added to Django now. This is the first time I've needed it.


Style tags missing in  on IE11

When loading my application on IE11, I get a blank page with no console errors.
Since the  tag is missing the

I apologize, but this is as close to the issue as I can get. It is pretty vague for me and the codebase I am testing is huge.

This was apparently an error with one or more of the many dependencies in my project.
I fixed it by adding the BabelEnginePlugin to webpack.


Improve pipeline run performance on single-node Orchest deployments

Description
Improves the performance of pipeline runs by executing all containers (of every Step) in the same Kubernetes Pod. This way users don't have to wait for a Pod to be allocated for every Step (which can take a considerable amount of time, e.g. for the Orchest quickstart pipeline this takes 30s out of the total 60s to execute the pipeline on my machine).
Note: These changes only apply to the single-node case, for multi-node deployments the previous implementation is(/should be) left unchanged with the exception of updating the Step status based on Kubernetes Pod status changes.
How Argo containerSet works

A container set actually starts all containers, and the Emissary only starts the main container process when the containers it depends on have completed."

See: https://argoproj.github.io/argo-workflows/container-set-template/#resource-requests:

I don't think resource requests are an issue on a single-node deployment given that the node already has certain hardware capabilities.

-> status of all containers gets set to Running when the pipeline run starts. Which we fix with some custom logic.
How to test
When running using the setup of our development docs then you will be running on a single-node setup and thus you can test whether the single-node setup works.
For testing multi-node, you could update the SINGLE_NODE environment variable in the celery-worker deployment to make Orchest believe to run the pipelines in a multi-node way.
Example executing the Quickstart pipeline on single-node
https://user-images.githubusercontent.com/26223174/185914570-15b5a1df-6baf-4f34-b3d9-98b97ca41b61.mp4
Example executing the Quickstart pipeline on single-node
https://user-images.githubusercontent.com/26223174/185931824-11b32950-5a60-43d1-80eb-2cd90688d40c.mp4
Note: Steps are now left in Pending (instead of Running) until the underlying Kubernetes Pod (that runs the Step container) is started.
Checklist

[x] I have manually tested my changes and I am happy with the result.

[x] Job runs
[x] multi-node still works as expected?

[x] The documentation reflects the changes.

[x] Cover requirement of the Emissary runne in the install docs (--no-argo), because then the user needs to run Argo with Emissary (on single-node) as well.

[x] Can we get the step_uuid without having to use the displayName in the single-node setting to increase robustness? (as per comment) --> I've asked the question in their Slack channel. --> We decided the PR can be released whilst we wait for an answer as the solution works fine for now.
[x] Issues with image pull policy of argo init containers? (as per https://github.com/orchest/orchest/pull/1214#issuecomment-1223723360)
[ ] Small rewrite instead of the if branching logic?
[x] I haven't introduced breaking changes that would disrupt existing jobs, i.e. backwards compatibility is maintained.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.7 out of 8 committers have signed the CLA.:white_check_mark: fruttasecca:white_check_mark: jerdna-regeiz:white_check_mark: ricklamers:white_check_mark: iannbing:white_check_mark: yannickperrenet:white_check_mark: nhaghighat:white_check_mark: astrojuanlu:x: dependabot[bot]You have signed the CLA already but the status is still pending? Let us recheck it.

Might be worth it to slightly refactor the functions that create the pipeline workflow to avoid branching on SINGLE_NODE within their scope, i.e. splitting them into different functions

I see that there is a rather long start time of the pod/container when running in single node mode. There are some suspcious logs worth looking into

These pulls seem to always be happening

These pulls seem to always be happening; this doesn't seem to occur on dev

Fixed in https://github.com/orchest/orchest/pull/1214/commits/6b1e9775042270f30ba0ab5e5dac0dc88130c9f0 (as per https://github.com/argoproj/argo-helm/blob/5112c7782423349707c1c13d3da973009c53d830/charts/argo-workflows/templates/controller/workflow-controller-config-map.yaml#L48). However, it does sound strange to me that this isn't happening on dev (but didn't feel the need to verify since this change will make sure it doesn't happen).

Fixed in https://github.com/orchest/orchest/commit/6b1e9775042270f30ba0ab5e5dac0dc88130c9f0 (as per https://github.com/argoproj/argo-helm/blob/5112c7782423349707c1c13d3da973009c53d830/charts/argo-workflows/templates/controller/workflow-controller-config-map.yaml#L48). However, it does sound strange to me that this isn't happening on dev (but didn't feel the need to verify since this change will make sure it doesn't happen).

Great to hear. Indeed it not happening on dev is quite strange to me as well, I verified on a cloud instance and wasn't happening there either

Indeed it not happening on dev is quite strange to me as well, I verified on a cloud instance and wasn't happening there either

I think I found why. We do a podSpecPatch that sets the imagePullPolicy of the "main" container to IfNotPresent. This was only done in the multi-node case and thus explains why it didn't happen on dev.


Remove unnecessary files from npm package

I've noticed a bunch of unnecessary files in the published npm package (.history directory, source files, travis/prettier/jest configs, ...) (see<EMAIL_ADDRESS>/ Code).
Instead of using .npmignore to ignore a bunch of files, I replaced it with files in package.json.
The build/src/ directory should be everything that is needed for this package (Licence, readme and package.json are always included).
This should drop the unpacked size from 137 kB down to 48.1 kB. This could be further improved by minifying/bundling.

I've noticed a bunch of unnecessary files in the published npm package (.history directory, source files, travis/prettier/jest configs, ...) (see<EMAIL_ADDRESS>/ Code).

While the .history directory should not be published (I added it to .gitignore but forgot to add it to .npmignore) I always publish the source itself and I hate when packages don't do that. (This is to make debugging easier, you can just go through the files in node_modules than having to pull up the repository.)

This should drop the unpacked size from 137 kB down to 48.1 kB. This could be further improved by minifying/bundling.

IMHO download size of unpackaged packages is basically irrelevant at this size level, and minification/bundling should happen on the consumer-level. Minified + gzipped this package turns into 2.9kB which is fair I feel. 🙂

While the .history directory should not be published (I added it to .gitignore but forgot to add it to .npmignore) I always publish the source itself and I hate when packages don't do that. (This is to make debugging easier, you can just go through the files in node_modules than having to pull up the repository.)

Thats fair. I still prefer files over .npmignore, because who knows what directories IDEs will add in the future that might get published without anyone noticing.


Added documentation for NetBSD.

Added a new file that documents the image preparation and running instructions for NetBSD.

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

Please also add yourself to CONTRIBUTORS file and we are ready to go.

@dvyukov Done! :smile:

Sorry, I meant both AUTHORS and CONTRIBUTORS.

At first I didn't read the initial description for who should be included in AUTHORS. After you pointed out CONTRIBUTORS, I read it and thought, how could I hold the copyright!
Anyways, I've added it!

Well, you not hold the copyright for docs/netbsd.md.
Thanks


TM syntax updates

Various cleanups/improvements to the .tmLanguage file. Also adds the LABEL instruction. :baby_chick:

LGTM

thanks :)


Missing link to dataset

"Please download processed dataset from here". Please put the link the dataset.

I have added the link.


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20220128043444982 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20220128043444982 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 48.282%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14766

Relevant Lines:
29039

💛  - Coveralls


Possibility of Unity (Android, iOS, Mac OS X, Linux) Support?

I'm curious if the library's authors have any information about the possibility of porting / extending / supporting this library for additional platforms under Unity / Mono.
In my case I have the sqlite3 library built for each platform (Android, iOS, Mac OS X, Linux) and I'd like to use SQLNado over the library on those OS.
Is this even remotely possible? I am interested in doing the port / contributing the bridge code if it is possible -- I would be using it in a game project, compliant with the licenses.
Any guidance, suggestions, or rough ideas / concepts are very welcome. Thanks for your time and thank you for the wonderful SQLite library!

Hi Cory,
I developed SQLNado using .NET standard (2+) with that exact idea in mind :-)
Basically, any platform that support .NET standard (which is .NET Framework, UWP, .NET Core, Xamarin/Mono and associated OSes) should be able to use SQLite provided the binary is available (which is the case for SQLite).
Every interop code is located in SQLiteDatabase.cs. I think it shouldn't be a lot of work, probably tweak DllImport values, and maybe some signature, arguments, argument types, etc...
If you're used to .NET Core on Linux, that would be the easiest platform to start with. If only I had some spare time I could try it, but my technical knowledge beyond Windows is currently weak, I must admit :-)

This is wonderful news, thank you very much for the response.
I have very little cross-platform .NET experience -- I am a C++ developer and when I have used C# it has been on Win32. What I do have is a reasonable level of expertise with building SQLite3 libraries for every platform.
I will start with .NET Core on Linux and see what I can come up with. It sounds like you've already done all the hard work!

Ok, great, don't hesitate to come back to me if I can do anything.

Note I've modified SQLNado so it's more cross-platform ready. It currently support Windows and Linux, with more to come.

Let me know if you need any help, I have a modified version that works on Android/iOS/Mac OS X/Linux/Win64/Nintendo Switch with Unity IL2CPP.

Just an update: SQLNado now supports Android fine (using SQLite official Android builds).

About performance in general (couldn't say in your particular context), it should be close to what SQLite provides.
Some things to make sure:

when you do lots of updates, is much better (order of magnitude) to create a surrounding transaction when doing multiple updates. I always found that counterintuitive in comparison with other databases "transactions", but it's a game changer. This one is on SQLite.
depending on context, you might want to change how SQLite works, with various settings, for example:

db = new SQLiteDatabase(...)

// https://stackoverflow.com/a/47745836/403671
db.MemoryMapSize = -1;

...

db.JournalMode = SQLiteJournalMode.Off,
db.SynchronousMode = SQLiteSynchronousMode.Off,
db.LockingMode = SQLiteLockingMode.Exclusive

...

when you do updates and the schema (table/indices) doesn't change it's much better to "synchronize" tables and indices once per "session" (whatever that means in the context, can be once per process in Windows), and use an option in save calls. If you don't do that, SQLNado will try to synchronize on each save call. This one is on SQLNado. So, something like this in an hypothetical "sales" database :

public sealed class SalesDatabase : SQLiteDatabase
{
    private readonly SQLiteSaveOptions _saveOptions;

    public SalesDatabase(ProductsDatabase products, ILogger<SalesDatabase>? logger)
        : base(Path.Combine(FileSystem.Current.AppDataDirectory, "sales.db"))
    {
        foreach (var type in SalesTableTypes)
        {
            // so we don't have to do at each call
            SynchronizeSchema(type);
            SynchronizeIndices(type);
        }

        // we synchronize at startup, so we don't need to sync at each save
        _saveOptions = CreateSaveOptions();
        _saveOptions.SynchronizeSchema = false;
        _saveOptions.SynchronizeIndices = false;

        ...
    }

    public override bool Save(object obj, SQLiteSaveOptions? options = null)
    {
        // we synchronize at startup, so we don't need to sync at each save
        options ??= _saveOptions;
        return base.Save(obj, options);
    }

    // list of tables known to this database, that we want to sync only once
    private static IEnumerable<Type> SalesTableTypes
    {
        get
        {
            yield return typeof(Order);
            yield return typeof(BasketProduct);
            yield return typeof(FavoriteProduct);
            yield return typeof(Setting);
        }
    }
}

Great SQLite performance tips. Android gamedevs will know the performance issues I'm talking about.
Unity gamedevs (who are compiling C# and then converting the IL bytecode to C++ and recompiling it with native plugin source mixed in) will recognize the advantages of __Internal in the IL2CPP toolchain, especially on mono runtimes.


Rename two ABI exports

Based on issue: https://github.com/massalabs/massa-sc-runtime/issues/196

Leaving as draft while the PR on massa-sc-runtime is not merged with main.

The PR on runtime as been merged. What's the next steps to merge here @Thykof ?


Docs: Add "requiring ports" to common problems

Also differentiate between troubleshooting and setup

Thanks for the effort! You want to add constraints to Waiting problem?

Hey Matthias, that would make sense and indeed I started with that. Right now, I approach the topic from the "Recipes" side: How to partition your cluster statically?
I am also unclear about what to write about constraints in the Troubleshooting section. Since what would you write for constraints? That they are a likely candidate for deployment problems? That you have to make sure the spelling is correct?
Another topic I would like to add eventually is interference with Mesos roles.
Do you think that the documentation is in a bad state if we first merge as is? We can add more "problems" as we go.
BTW I renamed the page title since I didn't want it to start with Marathon. I wanted the prefix be significant. If you have nice suggestions, shoot ahead.

Like it. I think there are a lot points to add to that topic - let's do them one by one.

2 questions - other than that it looks good to me.

I changed the title. I still don't like it but can't think of something better.

Thanks.


Search neighbouring tiles while dragging

Download one extra row of tiles to each side by default.
Start downloading from center

If tile placement manually becomes a problem, have a look at draw_node and apply_transform https://dearpygui.readthedocs.io/en/latest/documentation/drawing-api.html#scene-graph


[error] 5890#0: 4 failed to run log_by_lua: ./kong/init.lua:203: attempt to index local 'plugins_iterator' (a nil value)

Summary
kong.conf (everthing else at default)
role = data_plane
database = off
declarative_config = /var/www/html/middleware/kong/sample-kong.yml
yaml file is as follows:
_format_version: "1.1"
services:
-   name: httpbin
url: "https://httpbin.org/" #for easy testing, make sure you have internet.
routes:
-   name: httpbin
hosts:
- "httpbin.org"
-   name: middleware-backend
url: "http://mine.local/" #replace with your actual url.
routes:
-   name: mine-backend
hosts:
- "mine.local"
Steps To Reproduce

any curl command
curl --request GET --url http://localhost:8000/get --header "host: httpbin.org"
centos7 environment
build via DEVELOPER.md
no error during kong start (all OK)

Additional Details & Logs
2020/03/15 16:14:26 [error] 5890#0: 4 failed to run log_by_lua: ./kong/init.lua:203: attempt to index local 'plugins_iterator' (a nil value)
stack traceback:
./kong/init.lua:203: in function 'execute_plugins_iterator'
./kong/init.lua:1068: in function 'log'
log_by_lua(nginx-kong.conf:104):2: in main chunk while logging request, client: <IP_ADDRESS>, server: kong, request: "GET /get HTTP/1.1", host: "httpbin.org"
attempt to index local
What does this really mean, do i need a database in place?

does it work if you don’t set role?

or alternatively set it to traditional.

same error? why attempt to index?
2020/03/31 12:18:26 [error] 5890#0: 4 failed to run log_by_lua: ./kong/init.lua:203: attempt to index local


Update "setting-the-foundation" page

Automatically generated by Netlify CMS

@sonnakim, @schaferjh: Looks like the slides for the 18F presentation on equity-centered design have disappeared from the URL they were at, and I couldn't find where they've moved to. I changed the link in our guidelines to point to the presentation video itself on YouTube. What do you think of that change?

@niqjohnson that's too bad. pointing to the YouTube video is definitely a good alternative!


Error building with Api Compatibility Level ".NET 2.0 Subset" for WebGL

I have been running into problems with building my project into webgl using the project currently up on the git. Unity sets the Api Compatibility Level to ".NET 2.0 Subset" by default and I was able to resolve the issue temporarily by switching the Api Compatibility Level to ".NET 2.0" but this makes the build times longer and build sizes larger.
These are the errors I get after it fails to build:

Failed running D:\unity20173\Editor\Data\il2cpp/build/UnityLinker.exe --api=NET_2_0_Subset -out="C:\Users\ndrexl\Downloads\UnityGLTF-master (1)\UnityGLTF-master\UnityGLTF\Temp\StagingArea\Data\Managed\tempStrip" -l=none -c=link --link-symbols -x="D:\unity20173\Editor\Data\PlaybackEngines\WebGLSupport\Whitelists\Core.xml" -f="D:\unity20173\Editor\Data\il2cpp\LinkerDescriptors" -x "C:\Users\ndrexl\Downloads\UnityGLTF-master (1)\UnityGLTF-master\UnityGLTF/Temp/StagingArea/Data/methods_pointedto_by_uievents.xml" -x "C:\Users\ndrexl\Downloads\UnityGLTF-master (1)\UnityGLTF-master\UnityGLTF/Temp/StagingArea/Data/UnityEngine.xml" -x "C:\Users\ndrexl\Downloads\UnityGLTF-master (1)\UnityGLTF-master\UnityGLTF\Temp\StagingArea\Data\preserved_derived_types.xml" -d "C:\Users\ndrexl\Downloads\UnityGLTF-master (1)\UnityGLTF-master\UnityGLTF\Temp\StagingArea\Data\Managed" -a  "C:\Users\ndrexl\Downloads\UnityGLTF-master (1)\UnityGLTF-master\UnityGLTF\Temp\StagingArea\Data\Managed\Assembly-CSharp.dll"
stdout:
Fatal error in Unity CIL Linker
System.NullReferenceException: Object reference not set to an instance of an object.
at Mono.Linker.TypeReferenceExtensions.c__Iterator1.MoveNext()
at Mono.Linker.Steps.TypeMapStep.TryMatchMethod(TypeReference type, MethodDefinition method)
at Mono.Linker.Steps.TypeMapStep.c__Iterator0.MoveNext()
at Mono.Linker.Steps.TypeMapStep.MapVirtualInterfaceMethod(MethodDefinition method)
at Mono.Linker.Steps.TypeMapStep.MapVirtualMethods(TypeDefinition type)
at Mono.Linker.Steps.TypeMapStep.MapType(TypeDefinition type)
at Mono.Linker.Steps.TypeMapStep.ProcessAssembly(AssemblyDefinition assembly)
at Mono.Linker.Steps.BaseStep.Process(LinkContext context)
at Mono.Linker.Pipeline.Process(LinkContext context)
at UnityLinker.UnityDriver.Run()
at UnityLinker.UnityDriver.RunDriver()
stderr:
UnityEngine.Debug:LogError(Object)
UnityEditorInternal.Runner:RunProgram(Program, String, String, String, CompilerOutputParserBase) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/BuildUtils.cs:128)
UnityEditorInternal.Runner:RunManagedProgram(String, String, String, CompilerOutputParserBase, Action1) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/BuildUtils.cs:73) UnityEditorInternal.AssemblyStripper:RunAssemblyLinker(IEnumerable1, String&, String&, String, String) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/AssemblyStripper.cs:89)
UnityEditorInternal.AssemblyStripper:StripAssembliesTo(String[], String[], String, String, String&, String&, String, IIl2CppPlatformProvider, IEnumerable1) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/AssemblyStripper.cs:82) UnityEditorInternal.AssemblyStripper:RunAssemblyStripper(String, IEnumerable, String, String[], String[], String, IIl2CppPlatformProvider, RuntimeClassRegistry) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/AssemblyStripper.cs:204) UnityEditorInternal.AssemblyStripper:StripAssemblies(String, IIl2CppPlatformProvider, RuntimeClassRegistry) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/AssemblyStripper.cs:114) UnityEditorInternal.IL2CPPBuilder:Run() (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/Il2Cpp/IL2CPPUtils.cs:152) UnityEditorInternal.IL2CPPUtils:RunIl2Cpp(String, IIl2CppPlatformProvider, Action1, RuntimeClassRegistry, Boolean) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/Il2Cpp/IL2CPPUtils.cs:41)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)

And I will get this one too:

Exception: D:\unity20173\Editor\Data\il2cpp/build/UnityLinker.exe did not run properly!
UnityEditorInternal.Runner.RunProgram (UnityEditor.Utils.Program p, System.String exe, System.String args, System.String workingDirectory, UnityEditor.Scripting.Compilers.CompilerOutputParserBase parser) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/BuildUtils.cs:130)
UnityEditorInternal.Runner.RunManagedProgram (System.String exe, System.String args, System.String workingDirectory, UnityEditor.Scripting.Compilers.CompilerOutputParserBase parser, System.Action1 setupStartInfo) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/BuildUtils.cs:73) UnityEditorInternal.AssemblyStripper.RunAssemblyLinker (IEnumerable1 args, System.String& out, System.String& err, System.String linkerPath, System.String workingDirectory) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/AssemblyStripper.cs:89)
UnityEditorInternal.AssemblyStripper.StripAssembliesTo (System.String[] assemblies, System.String[] searchDirs, System.String outputFolder, System.String workingDirectory, System.String& output, System.String& error, System.String linkerPath, IIl2CppPlatformProvider platformProvider, IEnumerable1 additionalBlacklist) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/AssemblyStripper.cs:82) UnityEditorInternal.AssemblyStripper.RunAssemblyStripper (System.String stagingAreaData, IEnumerable assemblies, System.String managedAssemblyFolderPath, System.String[] assembliesToStrip, System.String[] searchDirs, System.String monoLinkerPath, IIl2CppPlatformProvider platformProvider, UnityEditor.RuntimeClassRegistry rcr) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/AssemblyStripper.cs:204) UnityEditorInternal.AssemblyStripper.StripAssemblies (System.String stagingAreaData, IIl2CppPlatformProvider platformProvider, UnityEditor.RuntimeClassRegistry rcr) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/AssemblyStripper.cs:114) UnityEditorInternal.IL2CPPBuilder.Run () (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/Il2Cpp/IL2CPPUtils.cs:152) UnityEditorInternal.IL2CPPUtils.RunIl2Cpp (System.String stagingAreaData, IIl2CppPlatformProvider platformProvider, System.Action1 modifyOutputBeforeCompile, UnityEditor.RuntimeClassRegistry runtimeClassRegistry, Boolean debugBuild) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/Il2Cpp/IL2CPPUtils.cs:41)
UnityEditor.WebGL.WebGlBuildPostprocessor.CompileBuild (BuildPostProcessArgs args) (at /Users/builduser/buildslave/unity/build/PlatformDependent/WebGL/Extensions/Unity.WebGL.extensions/BuildPostprocessor.cs:350)
UnityEditor.WebGL.WebGlBuildPostprocessor.PostProcess (BuildPostProcessArgs args) (at /Users/builduser/buildslave/unity/build/PlatformDependent/WebGL/Extensions/Unity.WebGL.extensions/BuildPostprocessor.cs:915)
UnityEditor.PostprocessBuildPlayer.Postprocess (BuildTargetGroup targetGroup, BuildTarget target, System.String installPath, System.String companyName, System.String productName, Int32 width, Int32 height, BuildOptions options, UnityEditor.RuntimeClassRegistry usedClassRegistry, UnityEditor.BuildReporting.BuildReport report) (at C:/buildslave/unity/build/Editor/Mono/BuildPipeline/PostprocessBuildPlayer.cs:272)

I am using Unity 2017.3.1 for this, but I have tested with 2017.3.0 as well.

Are you positive this is a problem with UnityGLTF? I don't see any mention of any of our source files in any of that.

Yes I confirmed it by downloading the up to date recent batch of files uploaded here and just tried to build a Unity WebGL build from the project included (and after building the VS project included to make sure the .dll's are in the correct spot) and it gave me the error above.
In my other project I am using it for, I removed the GLTF plugin altogether and was able to build just fine. Added it back in and it will error with the same error as above.

Looks like there's a workaround for this in #138


[bug] Shift-clicking on an item in the lab block (e.g. syring) causes the game to crash.

Shift-clicking on an item in the lab block (e.g. syring) causes the game to crash.

fixed


Refactor handle-resource-offer! into smaller functions and add unit tests

Handle-resource-offer is the main function for matching tasks to offers. It is also very large and can be split into smaller functions that make testing easier.

Also add unit tests for handle-resource-offers!

Associated PR: https://github.com/twosigma/Cook/pull/259


V2 Api no Tweet POST interactions

I cant seem to find anywhere in your docs a create_tweet() option.
No way to post a reply although Ive heard its create_tweet(), I cant see any info on this in the docs.

https://tweepy.readthedocs.io/en/v4.14.0/search.html?q=create_tweet
https://tweepy.readthedocs.io/en/v4.14.0/client.html#tweepy.Client.create_tweet
https://tweepy.readthedocs.io/en/v4.14.0/asyncclient.html#tweepy.asynchronous.AsyncClient.create_tweet


Expose ncclimo --parallel command line option

In some cases, I found it necessary to change the --parallel option in ncclimo to successfully generate time series or climo files with zppy. But zppy currently does not expose that option to the user and so one is forced to manually edit the bash script.

@golaz ncclimo is called in three places -- twice in climo.bash and once in ts.bash. The 2 climo calls had --parallel=mpi, so in #452 I changed the parameter value to be user-defined. The ts call doesn't use parallel though. Should it be conditionally called depending if the user has set a value for it?

Options:

Define parallel = string(default="") in [default], parallel = string(default="mpi") in [climo]. As mentioned in #212, this means that the [climo] default would override a user-provided value in [default]. By default, climo would run with mpi and ts would run without the parallel option at all.
Define parallel = string(default="mpi") in [default]. Have ts run with mpi by default, just like climo.

In #452, I go with the first of the above two options, to maintain current functionality by default.


Broken U-SQL Tutorial link

The URL for the U-SQLTutotial is broken.

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

ID: ca34cf09-1532-3ddd-5984-1e8c481b92b7
Version Independent ID: 9705533d-12db-4ff3-53a3-ab0a91594633
Content: Get started with U-SQL language in Azure Data Lake Analytics
Content Source: articles/data-lake-analytics/data-lake-analytics-u-sql-get-started.md
Service: data-lake-analytics
GitHub Login: @xujxu
Microsoft Alias: xujiang1

@jaluecht  Thanks for the question! We are investigating and will update you shortly.

@jaluecht  I have assigned the issue to the content author to evaluate and update as appropriate.

@lingluo0531 would you please take a look at this issue to see if any update needed? Thanks.

fixed

Merged #182634 into master.
https://github.com/MicrosoftDocs/azure-docs-pr/pull/182634

@jaluecht , could you kindly advise how to close the issue? thanks!

You may close ticket
From: lingluo0531 @.>
Sent: Monday, December 13, 2021 7:26 PM
To: MicrosoftDocs/azure-docs @.>
Cc: jaluecht @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Broken U-SQL Tutorial link (Issue #84529)
@jaluecht https://github.com/jaluecht  , could you kindly advise how to close the issue? thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/azure-docs/issues/84529#issuecomment-993035687 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AC2HFV27GJ2WXBJNF3F6ZL3UQ2FLJANCNFSM5I6A7UIQ .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://github.com/notifications/beacon/AC2HFV6BOFPX5XH6U36OK7DUQ2FLJA5CNFSM5I6A7UI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHMYILJY.gif

@jaluecht   I don’t know how to close, and  I cannot find any place having a 'close' button . Could you advise or help direct close? Thanks!


how to use Destination Starrocks - Airbyte Connector

did anyone tried the airbyte starrocks connector, I tried https://github.com/StarRocks/starrocks/tree/main/contrib/destination-starrocks on master airbyte repo, but said Cannot locate tasks that match ':airbyte-integrations:connectors:destination-starrocks:airbyteDocker' as project 'destination-starrocks' not found in project ':airbyte-integrations:connectors'.

You want to follow this.  https://github.com/StarRocks/starrocks/discussions/23713.   I wouldn't compile the container and just use the one that StarRock's docker hub account gives you.


feat: emit helper packet events for indexing pending packets

Closes: #63
Description

To derive the string representation of the key bytes, I opted for base64 encoding due to its perceived safety, and a more human-readable output.
The add_event method for publishing IBC events has been superseded by the newly exposed emit_event method.

PR author checklist

[X] Linked to GitHub issue.
[X] Added tests.
[X] Updated code comments and documentation (e.g., docs/).
[X] Tagged one reviewer who will be the one responsible for shepherding this PR.

Merging this PR so we can have the typed events along with auxiliaries in our next sov-rollup-starter update. In case the logic of constructing aux events should be improved, we will consolidate it later.


Databroker Prometheus endpoint

To visualize the VSS data, it would be nice to have an integration with Prometheus.
Since databroker already is a gRPC server, it may be easy to add this feature, as a Prometheus exporter is just an HTTP endpoint ("/metrics") returning a flat multi-line text response with metrics.
The VSS tree could be flattened and provided as Prometheus Metrics, together with the current vehicle signal value and its last-updated timestamp:
# Format: <metric as vss path in prometheus notation[labels as {key=value}]> <current value> <timestamp in nanoseconds>
vehicle_cabin_comfort_desiredtemperatur 21<PHONE_NUMBER>000
vehicle_cabin_hvac_station_row2_right_fanspeed 60<PHONE_NUMBER>000
vehicle_powertrain_tractionbattery_charging_chargevoltage.phase1<PHONE_NUMBER>158000
vehicle_cabin_hvac_station_row2_left_fanspeed{unit="percent", min="0", "max="100", type="actuator"} 35<PHONE_NUMBER>000
vehicle_chassis_axle_row1_wheel_left_speed{unit="km/h", uuid="47897f20b2745b6aa2d0f76f1ecf824a", datatype="float"} 80<PHONE_NUMBER>000
...

Notes / Open Questions

Prometheus documentation on how to implement an exporter: https://prometheus.io/docs/instrumenting/writing_exporters/
The syntax of prometheus metrics is different than the VSS spec, so we need to convert the VSS Path notation (lower case, replacing dots with underscore etc). That is the simplest case.
Prometheus labels could be used for metadata, such as signal type, signal unit, min and max values etc.
Not sure about which values to export by default, so it would make sense to make this configurable later, eg only numeric and boolean values, but no strings, or another filter based on type (only sensores, but no attributes). As for some values, it makes sense to have the attributes as well ("Max") to be able to determine the relative usage ("Min" < "Current" < "Max") and visualize that in percentage.
Prometheus always expects the full list of the metrics afaik, so not only on change. The metrics exporter is always doing a "full dump" of all values

Hi @mikehaller , sounds quite "doable",
I'd assume that "should be" like a 10 min job, using the python lib.
(not sure about getting all the metadata details, but again, that could be hjacked via config first, and would just indicate a point, where need to extend/broaden the GRPC/Py-lib Api path)
@eriksven @sophokles73 @LuLeRoemer I have in the back of my mind you have played with it? Or do I mix it up with OpenTelemetry? is that something different. Please help me out here cloud natives :)
Another option might be using the existing hono-influxdb connector, and then trying feeding Prometheus from influx, but I am not sure it is easier: You suddenly need Hono in backend, and you would still need to write a "pyapi-to-hono-mqtt" adapter, that is likely not any easier than just serving /metrics yourself. But maybe @eriksven  has an opinion here

@SebastianSchildt You might have misheard or misinterpreted something here. There is no such thing as a hono-influxdb connector. Eclipse Hono in fact reports its metrics to an OpenTelemetry collector which can then be used to forward the metrics to other components. It also provides an endpoint for scraping by Prometheus. That said, we have been experimenting with doing something similar for values in the Databroker. For that purpose we created a client that polls the Databroker for a set of (current) values periodically and then uses the OpenTelemetry API within the client to report the values. However, when the metrics get exported to the OpenTelemetry Collector, they are getting aggregated already, e.g. being turned into average values over the last polling interval. So that didn't work out for us. We now explicitly POST the polled values to an InfluxDB in a set of measurements.
IMHO we should be careful to not confuse data export with reporting metrics here. For the former, exposing an HTTP endpoint that dumps all data might be a valid option. However, I guess this might become more inefficient with the number of VSS paths increasing. For the latter, I would suggest to use the OpenTelemetry SDK for instrumenting the Databroker code to keep track of some meaningful metrics (e.g. number of requests/s etc) and then also use the flexibility of the OpenTelemetry Collector to make these metrics available to whoever is interested.

Thank you for the clarification,  understood (I hope) So I think what Mike is talking about then is data export , so that is completely unrelated to any metrics/telemetry from databroker.
Wrt to Hono-Influx connector I was referring to https://github.com/eclipse/kuksa.cloud/tree/master/utils/hono-influxdb-connector , that could be used to get VSS data out of Hono, but as I said above, likely similar to just implement a promtheus-compatible /metrics http endpoint.

Some example code to get you started @mikehaller No metadata yet, and since I do not know how to deal prometheus side with values that don't exist (currently I use "nan"), you might need to change that, to leave those metrics out. Anyway, it is only two files
requirements.txt
web.py
kuksa-client

kuksa-prometheus.py
import web
from kuksa_client.grpc import VSSClient
from datetime import datetime

vsspaths = [ "Vehicle.Speed",
             "Vehicle.Body.Trunk.Rear.IsOpen",
             "Vehicle.Cabin.HVAC.Station.Row2.Left.FanSpeed",
             "Vehicle.Cabin.HVAC.Station.Row2.Right.FanSpeed",
             "Vehicle.Chassis.Axle.Row1.Wheel.Left.Speed",
             "Vehicle.Powertrain.TractionBattery.Charging.ChargeVoltage.Phase1",
           ]

urls = (
    '/', 'mainpage',
    '/metrics', 'metrics'
)
app = web.application(urls, globals())

class mainpage:
    def GET(self):
        return """
                    <title>KUKSA metrics</title>

KUKSA metrics

Find awesome metrics here.
                """

class metrics:
    def GET(self):
        prometheus_out=""
        with VSSClient('<IP_ADDRESS>', 55556) as client:
            current_values = client.get_current_values(vsspaths)
            for path in vsspaths:
                convertedpath=str(path).lower().replace(".","_")
                value="nan"
                timestamp=0
                if current_values[path] is not None:
                    value=current_values[path].value
                    timestamp=int(datetime.timestamp(current_values[path].timestamp)*1e9)
                prometheus_out+=f"{convertedpath} {value} {timestamp}\n"
        print(current_values)
        return prometheus_out

if __name__ == "__main__":
    app.run()

Example output (I only provided Speed)
vehicle_speed 0.0<PHONE_NUMBER>136367104
vehicle_body_trunk_rear_isopen nan 0
vehicle_cabin_hvac_station_row2_left_fanspeed nan 0
vehicle_cabin_hvac_station_row2_right_fanspeed nan 0
vehicle_chassis_axle_row1_wheel_left_speed nan 0
vehicle_powertrain_tractionbattery_charging_chargevoltage_phase1 nan 0

Feel free to play. If you intend to make it a somewhat more useful example, it might fit well in https://github.com/eclipse/kuksa.val/tree/master/kuksa_apps  🔧PRs always welcome 😁

indeed, i was misusing the term metrics here. what @sophokles73 is saying that Databroker might expose its own metrics like
how many datapoints are registered
how many clients are connected
how many subscriptions are active
etc
i was more looking for a data export and "accidentally" misusing prometheus format here, so i can scrape them easily into grafana.
two separate things i guess.
i will have a look at implementing it as a separate client, although that means i need to subscribe to all datapoints and keep them in memory. was hoping for an implementation inside of databroker to minimize that overhead.

separate client, although that means i need to subscribe to all datapoints and keep them in memory.

I don't think that's necessary.
If I understand correctly, Prometheus would periodically ask the endpoint: "give me all your metrics". Or does it only want changes?
If Prometheus doesn't ask for this to frequently (i.e. multiple times per second) I think it would be perfectly reasonable for the client to fetch everything from databroker on every request. This isn't a lot of data, and I don't think the latency would be much of an issue either.
If implementing an endpoint for "actual" metrics in databroker is in the cards (i.e. exporting requests / second counters or number of subscribers etc), it would probably make sense to also add the values of datapoints as another possible metric to export. I don't know if this is considered a misuse or not..

If I understand correctly, Prometheus would periodically ask the endpoint: "give me all your metrics". Or does it only want changes?

It takes whatever it gets :-) If the service that is being scraped by Prometheus does return values for only certain metrics, then that is what Prometheus will record in its time series DB (and interpolate missing values).

If Prometheus doesn't ask for this to frequently (i.e. multiple times per second) I think it would be perfectly reasonable for the client to fetch everything from databroker on every request. This isn't a lot of data, and I don't think the latency would be much of an issue either.

That is configurable but most Prometheus setups will scrape the targets every 10-15 seconds only. While I tend to agree that (currently) we are not talking about much data, that might change with the number of data entries.
In general, though, having a way to export all current data seems feasible, I would simply try to not get it confused with the Databroker's (real) metrics of operation. That said, you might also wonder if the Prometheus metrics data format is appropriate for this data export functionality or if a more general format wouldn't be a better fit. You will probably not tie this functionality to the assumption that the client is always a Prometheus server, right?

We are about to archive this repo soon. If you consider this issue as important please file a new issue at one of the new Kuksa repos at https://github.com/eclipse-kuksa


added link of home page in footer logo and my data

Please merge it for hacktoberfest.

This will be all counted as one PR. Is it ok for you? Or you can make two separate PR for data add and Adding link to Footer one. @saugat-rimal

i will make two seperate pr

okey

i have requested 2 pr. please merge it.


Port VPC Support to v2 Installer

v1 Installer used to support VPC (i.e. make the forseti VM use the network/subnetwork of another project).  #841
However, this is not ported to v2 Installer and should be.

@johnrevans6 will work on this.

Fixed in #1874


Add support series and map binary clusters plots

These were partially supported before, but this implements a few improvements like:

Allow passing a BitVector of clusters
Move some common functions to theme.jl
When plotting a BitVector cluster, use "Target" and "Non-Target" as legend labels

Close #494

@ConnectedSystems done.


Display a user friendly validation message for rate

Display a user-friendly validation message for the rate input field indicating the correct format instead of showing the "NAN" on amount

Restrict this field to numerical input.


DOMVM Doesn't Play Nice with Concatenation

Just noticed this while investigating a (severe) memory leak in my app. DOMVM includes a "use strict"; outside of it's enclosure, and so makes all following scripts strict, even though they may not be written for it.
/* domvm.micro.min.js */
/*
 https://github.com/leeoniya/domvm (3.x-dev, micro)
*/
'use strict';(function(C,x){...

It should be:
/* domvm.micro.min.js */
/*
 https://github.com/leeoniya/domvm (3.x-dev, micro)
*/
(function(C,x){'use strict';...})(this,function(){'use strict';...});

huh, gotta see if rollup's umd wrapper or closure compiler puts it there. my guess is the latter.

okay, so this is a known closure compiler issue [1]. Rollup does in fact put "use strict" at the top of its umd wrapper and Closure removes it. Closure will then add it to the entire file if language_out=ECMASCRIPT5_STRICT (which it has been).
i guess we'll need to set the language_out to ECMASCRIPT5 and add a hack into the build script to shove it back in after minification. pretty sad to need a workaround for this, but not difficult to do.
[1] https://github.com/google/closure-compiler/issues/997

Looks like this will do it:
--output_wrapper="(function(){'use strict';%OUTPUT%}).call(this)"

no, that'll re-wrap rollup's umd wrapper. which is not what we want. i already have a hack in the build script to undo Closure's aggressive inlining [1], so it's just a matter of adding an extra replace call there.
245f048fd25c739675768e0fc76a36159ef77492 should fix this,
[1] https://github.com/leeoniya/domvm/blob/3.x-dev/build.js#L150


Is this extension unmaintained?

I would like to contribute a solution, however it looks like no issues have been addressed and only the pull request for a license has been approved. It would probably be better to maintain this than to maintain a fork or build something else.
Is anyone actively maintaining this? If not, might I offer my help?
@jjallaire I see you're pretty busy, however I would like to directly solicit your attention here. Please let us know if there's any way we can contribute to the effort to add features (e.g. snippets).

Please feel free to contribute. Team is limited, and time that goes with it. We have a lot of project in Quarto ecosytem to focus and even if we are watching this repo, we are not that active short term on adding new feature.
We can definitely review PR. So please do contribute if you want to.
Thank you for your help !

Gotcha! I'm sorry my other comment came off as harsh. I just really think quarto is amazing and want help make it better. I'll see what I can cook up.

Thank you !


Fix paths for ivy-style repos in grepJar

dunno, looks ok i guess

You can check that this makes sense by doing something like ls ~/.ivy2/cache/org.scala-lang/scala-library/ and seeing the jars and srcs folder.


Update main.css

Hey. Thanks for taking your time out and contributing by opening this PR.
I've reviewed your code and concluded with these comments.

I suggest you to squash all the five commits into one single commit so that it's easy to look into.

Everything else LGTM! ❤
I'll merge the code once you fix and test this accordingly.
Thanks @enweazudaniel

You deleted the css file? It is need for styling the application. Kindly fix that, please.


Auto resizing textarea

auto expanding textarea up to max-height 600px as a user increase number of lines
#9e39d63 shows graph names and node/edge label names correctly with some metadata improvement of marodins and instead of retreiving the whole metadata, I decided to pass graph name and only the metadata under that graph name will
be retreived. but I have some issues with refreshing. It gives me average of 1.4s every time refreshing regardless of its size.

will work on it to fix it.

Please refer to https://github.com/apache/age-viewer/issues/52#issuecomment-1306319389 prior to merging this as it includes changes to already working back-end and front-end code.


Decide once and for all what the heck to call everything

There's been a lot of discussion about what to name things. We need to have one final discussion about this while everyone is co-present and decide on this.

[ ] What is Diaperbase (Inventory) called
[ ] What is Partnerbase (CRM) called
[ ] Should we use subdomains for each application
[ ] What subdomain should we use for Diaperbase
[ ] What subdomain should we use for Partnerbase

Other questions may arise, but I think those seem to be the recurring ones that have not yet found final resolution.

@marcheiligers is also invited but I can't get GH to add him to the Assignees list
Anyone else is welcome to join if they'd like to!

@armahillo Just name the time/place.