WIP:  hamilton + quokka

Working on how to make Hamilton handle Quokka better.

Thoughts to think about:

quokka, like spark, relies on a central object to be manipulated so that it can perform query optimizations.
adding support for hamilton means that we need to under the hood manipulate this object correctly.
I believe with a default hamilton function we have enough information to know what to do, it's just a matter of figuring out how to traverse the DAG in a way to construct the right order of operations.

Sketch of some code
# Data Loading
# Filtering is part of data loading -- do we also expose columns like this?
@extract_columns(*["l_quantity", "l_extendedprice", "l_discount", "l_tax", "l_returnflag", "l_linestatus"])
def lineitem(qc: QuokkaContext, path: str,
             filter: str = "l_shipdate <= date '1998-12-01' - interval '90' day") -> DataStream:
    """Loads and filters data from the lineitem table"""
    ds: DataStream = qc.read_csv(path, sep="|", has_header=True)
    if filter:
        ds = ds.filter(filter)
    return ds

# transforms we want people to write
def disc_price(l_extendedprice: pl.Series, l_discount: pl.Series) -> pl.Series:
    """Computes the discounted price"""
    return l_extendedprice * (1 - l_discount)

def charge(l_extendedprice: pl.Series, l_discount: pl.Series, l_tax: pl.Series) -> pl.Series:
    """Computes the charge"""
    return l_extendedprice * (1 - l_discount) * (1 + l_tax)

@groupby("l_returnflag", "l_linestatus")
def grouped_lineitem(l_quantity: pl.Series, l_extendedprice: pl.Series,
                        disc_price: pl.Series, charge: pl.Series, l_discount: pl.Series,
                        l_returnflag: pl.Series, l_linestatus: pl.Series) -> GroupedDataStream:
    pass

# maybe more subtly
def groupby_lineitem(l_returnflag: pl.Series, l_linestatus: pl.Series, *, l_quantity: pl.Series, l_extendedprice: pl.Series,
                        disc_price: pl.Series, charge: pl.Series, l_discount: pl.Series,
                        ) -> GroupedDataStream:
    pass

Parking a thought -- what about just enabling hamilton type functions instead of with_column?
def disc_price(l_extendedprice: pl.Series, l_discount: pl.Series) -> pl.Series:
    """Computes the discounted price"""
    return l_extendedprice * (1 - l_discount)

def charge(l_extendedprice: pl.Series, l_discount: pl.Series, l_tax: pl.Series) -> pl.Series:
    """Computes the charge"""
    return l_extendedprice * (1 - l_discount) * (1 + l_tax)

def main(qc, path):
    temp_module = ad_hoc_utils.create_temporary_module(disc_price, charge)
    adapter = QuokkaGraphAdapter_V2(base.DictResult())

    lineitem = qc.read_csv(path, sep="|", has_header=True)
    d = lineitem.filter("l_shipdate <= date '1998-12-01' - interval '90' day")

    dr = hamilton.Driver({}, temp_module, adapter=adapter)
    d = dr.execute(["disc_price", "charge"])
    f = d.groupby(["l_returnflag", "l_linestatus"], orderby=["l_returnflag", "l_linestatus"]).agg(
        {
            "l_quantity": ["sum", "avg"],
            "l_extendedprice": ["sum", "avg"],
            "disc_price": "sum",
            "charge": "sum",
            "l_discount": "avg",
            "*": "count",
        }
    )
    return f.collect()

Tweaking the above slightly:
def disc_price(l_extendedprice: pl.Series, l_discount: pl.Series) -> pl.Series:
    """Computes the discounted price"""
    return l_extendedprice * (1 - l_discount)

def charge(l_extendedprice: pl.Series, l_discount: pl.Series, l_tax: pl.Series) -> pl.Series:
    """Computes the charge"""
    return l_extendedprice * (1 - l_discount) * (1 + l_tax)

def main(qc, path):
    temp_module = ad_hoc_utils.create_temporary_module(disc_price, charge)
    adapter = QuokkaGraphAdapter_V2()

    lineitem = qc.read_csv(path, sep="|", has_header=True)
    d = lineitem.filter("l_shipdate <= date '1998-12-01' - interval '90' day")

    dr = hamilton.Driver({}, temp_module, adapter=adapter)
    d = dr.execute(["disc_price", "charge"], inputs={c: d for c in d.schema}) # default is to append columns to passed in dataframe
    f = d.groupby(["l_returnflag", "l_linestatus"], orderby=["l_returnflag", "l_linestatus"]).agg(
        {
            "l_quantity": ["sum", "avg"],
            "l_extendedprice": ["sum", "avg"],
            "disc_price": "sum",
            "charge": "sum",
            "l_discount": "avg",
            "*": "count",
        }
    )
    return f.collect()

The QuokkaGraphAdapter_V2 adapter then intercepts


Bug:

Description
The application encounters a Stream connection closed. Resubscribing TypeError: Failed to fetch error after a period of idle activity. This error seems to be related to CORS policy, specifically the absence of the 'Access-Control-Allow-Origin' header on a requested resource from 'https://dev.xmtp.network/message/v1/subscribe'. The error causes an unwanted log output that appears intermittently after some minutes of idle activity.
The full error message is as follows:
 javascript Copy code Stream connection closed. Resubscribing TypeError: Failed to fetch inbox:1 Access to fetch at 'https://dev.xmtp.network/message/v1/subscribe' from origin 'https://dev.xmtp.chat' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled. _app-a3d1fbcef7da14c9.js:25 POST https://dev.xmtp.network/message/v1/subscribe net::ERR_FAILED 502 ... TypeError: Failed to fetch
Expected Behavior
In the idle state, the application should not generate any log output related to network changes or errors. As such, this CORS-related error message should not appear.
Steps to Reproduce
This error occurs after some minutes of idle activity in both development and production environments. The error log appears periodically at seemingly arbitrary intervals.
Relevant conversation threads regarding this issue can be found here:
Slack Conversation: https://xmtp-labs.slack.com/archives/C02BABHFZG9/p1685465049777349
Discord Conversation: https://discord.com/channels/831836269558235136/1113936882095620176
Additional Information
The occurrence of this error does not seem to depend on any specific user action or scenario but rather appears during idle activity.

@fabriguespe this is with xmtp-js, correct?


Possibility of customizing ion-content scrollbar in Chrome Browser

Feature Request
Ionic Info
Run ionic info from a terminal/cmd prompt and paste the output below.
Ionic:

   ionic (Ionic CLI)             : 4.3.1
   Ionic Framework               : @ionic/angular 4.0.0-beta.15
   @angular-devkit/build-angular : 0.10.3
   @angular-devkit/schematics    : 7.0.3
   @angular/cli                  : 7.0.3
   @ionic/angular-toolkit        : not installed

System:

   NodeJS : v10.11.0 (C:\Program Files\nodejs\node.exe)
   npm    : 6.0.1
   OS     : Windows 10

Describe the Feature Request
Possibility to customize scrollbar in chrome
Describe Preferred Solution
::-webkit-scrollbar {
width: inherit
height: inherit;
}
::-webkit-scrollbar-track {
background: inherit;
}
::-webkit-scrollbar-thumb {
background: inherit;
border-radius: inherit;
border: inherit;
}

I can not change because of the SHADOW DOM

@GuilhermeBCC that Shadow DOM is related to [ViewEncapsulation](https://blog.thoughtram.io/angular/2015/06/29/shadow-dom-strategies-in-angular2.html) of Angular, not Ionic issue, but I got that style to work inside my application, because actually -webkit-scrollbar related to browser styling, not the component itself, it is style directive for how browser should render the scrollbar.
Anyway, you maybe need to provide an example, because I do totally understand your issue, I got that before, but it was an issue with my template and got things fixed when I fixed my template.

@GuilhermeBCC  I think you're right. I'm facing the same issue, can not change the .inner-scroll class which is inside the SHADOW DOM of the ion-content component


Allow isReserved() to handle non-string inputs

Carried over from #772 - change made on dev branch as requested.

Thanks!


General: Legacy io not used in global plugins

Brief description
Session data from legacy_io are not used in most of global plugins.
Description
Usage of legacy_io.Session was replaced with using data stored on pyblish context. Some plugins are still using legacy io but those plugins already have opened PR changing them or replacement in them would need more overall changes.
Additional information
Replacement usage of session keys stored in context.data:

"AVALON_APP" -> "hostName"
"AVALON_PROJECT" -> "projectName"
"AVALON_ASSET" -> "asset"
"AVALON_TASK" -> "task"

Testing notes:

[ ] Farm cleanup plugin is working
[ ] Collect anatomy instance data has right project locally and on farm
[ ] Collect scene loaded versions  has right project locally and on farm
[ ] Integrate is integrating to right project locally and on farm
[ ] Validate editorial asset is validating correctly

In AE it fails locally with

Should be fixed now.


singularity: 2.4 -> 2.4.2

Semi-automatic update. These checks were done:

built on NixOS
ran /nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/bin/singularity -h got 0 exit code
ran /nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/bin/singularity --help got 0 exit code
ran /nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/bin/singularity help got 0 exit code
ran /nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/bin/singularity -v and found version 2.4.2
ran /nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/bin/singularity --version and found version 2.4.2
found 2.4.2 with grep in /nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2
found 2.4.2 in filename of file in /nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2

cc @jbedo

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘singularity-2.4.2’ in /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/applications/virtualization/singularity/default.nix:53 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Partial log (click to expand)
/nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/libexec/singularity/cli/instance.stop.exec: interpreter directive changed from "/bin/bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/libexec/singularity/cli/image.expand.exec: interpreter directive changed from "/bin/bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/libexec/singularity/cli/help.exec: interpreter directive changed from "/bin/bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/libexec/singularity/cli/check.exec: interpreter directive changed from "/bin/bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/libexec/singularity/helpers/check.sh: interpreter directive changed from "/bin/bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/libexec/singularity/helpers/help.sh: interpreter directive changed from "/bin/bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/libexec/singularity/helpers/inspect.sh: interpreter directive changed from "/bin/bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/libexec/singularity/helpers/image.sh: interpreter directive changed from "/bin/bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/libexec/singularity/helpers/apps/list.sh: interpreter directive changed from "/bin/bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2

Success on aarch64-linux (full log)
Partial log (click to expand)
/nix/store/s1qadgbscd3hn9ywnz6c15rc50m35ksr-singularity-2.4.2/libexec/singularity/bootstrap-scripts/deffile-driver-localimage.sh: interpreter directive changed from "/bin/bash" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/bash"
/nix/store/s1qadgbscd3hn9ywnz6c15rc50m35ksr-singularity-2.4.2/libexec/singularity/bootstrap-scripts/deffile-driver-docker.sh: interpreter directive changed from "/bin/bash" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/bash"
/nix/store/s1qadgbscd3hn9ywnz6c15rc50m35ksr-singularity-2.4.2/libexec/singularity/bootstrap-scripts/deffile-driver-debootstrap.sh: interpreter directive changed from "/bin/bash" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/bash"
/nix/store/s1qadgbscd3hn9ywnz6c15rc50m35ksr-singularity-2.4.2/libexec/singularity/bootstrap-scripts/deffile-driver-busybox.sh: interpreter directive changed from "/bin/bash" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/bash"
/nix/store/s1qadgbscd3hn9ywnz6c15rc50m35ksr-singularity-2.4.2/libexec/singularity/bootstrap-scripts/deffile-driver-arch.sh: interpreter directive changed from "/bin/bash" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/bash"
/nix/store/s1qadgbscd3hn9ywnz6c15rc50m35ksr-singularity-2.4.2/libexec/singularity/bootstrap-scripts/checks.sh: interpreter directive changed from "/bin/bash" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/bash"
/nix/store/s1qadgbscd3hn9ywnz6c15rc50m35ksr-singularity-2.4.2/libexec/singularity/bootstrap-scripts/functions: interpreter directive changed from "/bin/bash" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/bash"
/nix/store/s1qadgbscd3hn9ywnz6c15rc50m35ksr-singularity-2.4.2/bin/run-singularity: interpreter directive changed from "/bin/sh" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/sh"
/nix/store/s1qadgbscd3hn9ywnz6c15rc50m35ksr-singularity-2.4.2/bin/singularity: interpreter directive changed from "/bin/bash" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/bash"
/nix/store/s1qadgbscd3hn9ywnz6c15rc50m35ksr-singularity-2.4.2

This breaks image building as mnt/container is not owned by root:
`Singularity: mount (U=0,P=3105)> Root must own container mount directory: /nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/var/singularity/mnt/container
ERROR  : Root must own container mount directory: /nix/store/64n6qi39xvh6jvi7qzpkfxpq1d57xw45-singularity-2.4.2/var/singularity/mnt/container
Singularity: mount (U=0,P=3105)> Retval = 255
ABORT  : Retval = 255
`


contact page

#89
@nisha331 merge it first please or else conflicts are gonna arise again n again.

resolve conflict plz

Like really again!!!


How to specify target in generated href

Is it possible to specify target="_blank" in generated href ?

It would be possible to add that to generated links, but I'm loathe to add too many options or departures from what's directly in the Google Doc content. This would be easy enough to add after conversion via a Perl or Python script.

Closing for now -- this is a non-obvious side-effect of conversion, and not everyone wants that behavior.


Django 1.10 support for FieldTracker

Fixes #232
So two things going on here.
First, as of Django 1.8, we should be using get_deferred_fields rather than manually probing for isinstance(field_obj, DeferredAttribute).
Second, as of Django 1.10, it seems inherited models don't pull through the deferred fields in __class__.__dict__. Have to consult the base class to get them. I have a feeling this might be a regression and will probs raise a ticket with django to make sure, but this bandaid should work for the time being. There's a slight niggle in that dj110 now supports overriding fields from the base class. I think I'm building the dict in the correct order so that stuff is overridden correctly, but there might need to be additional tests added to check for this (can open a new issue). Also, perhaps there's a way to get a hold of the deferred field that avoids the use of __class__.__dict__ altogether?
Tests all pass now so removed dj110 from allowed failures.

After some django digging, a handle to DeferredAttribute is now obtainable by accessing the field through the instance's class. I've updated the PR to use this implementation, as it's considerably neater.


Add Navbar in second page

I want to work on this Issue.Please assign me to this.

there isnt a need for nav bar in second page, the second page is just there so that u have idea about how to multiply pages

Ok thanks!


Update interested-technical-writers.md

Hi! Thanks for your interest in Season of Docs! Would you please add your information to the end of the file as specified in the instructions?


QMAPS-2159 fix error messages height on mobile

Description
error panels on mobile were too tall
now they're as tall as the text
NB: this fix shouldn't impact the display of these panels when there's no error
Screenshots

@xem

thanks for the explanations, I hadn't use this mechanism until today and it wasn't super clear for me how to use it properly.

You're right, this feature is powerful but confuse and hard to discover. But it's great when our existing components can be used as-is to bring enhancements like this :+1:


ModuleNotFoundError: No module named 'compose.cli'; 'compose' is not a package

$ make
Downloading and patching SQLite
--2022-02-27 00:45:54--  https://sqlite.org/2020/sqlite-amalgamation-3330000.zip
Resolving sqlite.org (sqlite.org)... 2600:3c00::f03c:91ff:fe96:b959, <IP_ADDRESS>
Connecting to sqlite.org (sqlite.org)|2600:3c00::f03c:91ff:fe96:b959|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 2417079 (2.3M) [application/zip]
Saving to: ‘sqlite-amalgamation-3330000.zip’

sqlite-amalgamation-3330000.zip                100%[===================================================================================================>]   2.30M  1.26MB/s    in 1.8s

2022-02-27 00:45:56 (1.26 MB/s) - ‘sqlite-amalgamation-3330000.zip’ saved [2417079/2417079]

Traceback (most recent call last):
  File "/usr/bin/docker-compose", line 33, in <module>
    sys.exit(load_entry_point('docker-compose==1.27.4', 'console_scripts', 'docker-compose')())
  File "/usr/bin/docker-compose", line 25, in importlib_load_entry_point
    return next(matches).load()
  File "/usr/lib/python3.9/importlib/metadata.py", line 77, in load
    module = import_module(match.group('module'))
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 972, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 981, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'compose.cli'; 'compose' is not a package
make: *** [Makefile:44: third_party/sqlite3.33-src/sqlite3.c] Error 1

Ok, I had to reinstall docker-compose by:
$ sudo apt-get remove docker-compose
$ pip3 install docker-compose


Adding datetime format decimal and no decimal

fixes https://github.com/Azure/azure-sdk-for-cpp/issues/776

Thank you Victor! I think, "With" and "Without" should not be part of the DateTime format. but should be another enum class TimeFractionFormat { DropTrailingZeros = 0, AllDigits = 1, Truncate = -1 }, And ToString() should take a parameter, with default value of TimeFractionFormat::DropTrailingZeros

Truncate

It looks like Rfc1123 won't parse/generate any Decimals, so, adding an extra parameter to ToString() for the Decimal part would be only applied for the ISO format.  In that case, it becomes weird.

@Jinming-Hu, @vhvb1989 - why did we need this truncation behavior at all? What scenario motivated the request?

What would the user have to do to get full round-trippability without loss of data?
For example, what code what I write to round-trip exactly:
"2020-10-13T21:06:15.99999Z" to DateTime, and back to string - and we get back exactly "2020-10-13T21:06:15.99999Z".
Having it turn to "2020-10-13T21:06:15Z" or ""2020-10-13T21:06:15.0000000Z" would be data loss.
We should have tests for such round-tripping.


Add custom variables such as URL, description, etc

Is there a way to pass URL and other variables?

Can you provide a use case? What's the reason you want to do this?

You can already do this for your translation strings, for what it's worth.

For our messages we use the title, description and url variables.
A workaround was to package the variables inside a json string then reference them inside the notification blade.

On Sep 9, 2024, at 2:22 AM, Muhammed Sari @.***> wrote:

Can you provide a use case? What's the reason you want to do this?
—
Reply to this email directly, view it on GitHub https://github.com/masmerise/livewire-toaster/issues/51#issuecomment-2337224043 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACBHOBU2Y7YHLQUKRJMKQ7TZVU5BNAVCNFSM6AAAAABNHEUV7OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZXGIZDIMBUGM .
You are receiving this because you authored the thread. https://github.com/notifications/beacon/ACBHOBRZFTEOPRGLICENDPLZVU5BNA5CNFSM6AAAAABNHEUV7OWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTULJ42WW.gif Message ID: @.***>

Can you provide a code example? It is still not entirely clear to me.

Here is our hub.blade.php
 $alignment->is('bottom'),
 'top-1/2 -translate-y-1/2' => $alignment->is('middle'),
 'top-0' => $alignment->is('top'),
 'items-start rtl:items-end' => $position->is('left'),
 'items-center' => $position->is('center'),
 'items-end rtl:items-start' => $position->is('right'),
 ])>

Alexander
On Sep 10, 2024, at 7:49 AM, Muhammed Sari @.***> wrote:
Can you provide a code example? It is still not entirely clear to me.
—
Reply to this email directly, view it on GitHub https://github.com/masmerise/livewire-toaster/issues/51#issuecomment-2340446181 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACBHOBXE3DGQGZALGCXX75TZV3MEVAVCNFSM6AAAAABNHEUV7OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBQGQ2DMMJYGE .
You are receiving this because you authored the thread. https://github.com/notifications/beacon/ACBHOBR2H7TGS6EVK33H2DTZV3MEVA5CNFSM6AAAAABNHEUV7OWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTULQBP6K.gif Message ID: @.***>

Sorry, having a difficult time posting code in the comments.  Here is a PDF of the code.
hub.blade.php.pdf

That didn't do it either 😅.
Anyway, I think I know what you're getting at now: you want to be able to trigger some user actions from within the toast.
Unfortunately, this is out of scope for this package. The sheer small size of this package allows it to be lightning fast and that was the main goal from the get-go. You can use something like filament/notifications (it can be used standalone) for the functionality that you desire.

I will look into.
Thank you

Alexander
On Sep 12, 2024, at 4:50 AM, Muhammed Sari @.***> wrote:

Closed #51 as not planned.
—
Reply to this email directly, view it on GitHub https://github.com/masmerise/livewire-toaster/issues/51#event-14229067595 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACBHOBSEBLHYM6O66JO2SSLZWFIS3AVCNFSM6AAAAABNHEUV7OVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGIZDSMBWG42TSNI .
You are receiving this because you authored the thread. https://github.com/notifications/beacon/ACBHOBSFCN6OZ26BXLU6YYDZWFIS3A5CNFSM6AAAAABNHEUV7OWGG33NNVSW45C7OR4XAZNWJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XKUY3PNVWWK3TUL5UWJTYAAAAAGUA6K5FQ.gif Message ID: @.***>


Make an autoscaling plugin

I plan to write an autoscaling plugin system.
It will be invoked like so:
python manage.py autoscaler
And it will then look at the plugin configuration and work somewhat like my comments mentioned in this thread:
https://talk.vespene.io/t/slurm-isolation/
It should be usable to execute anything, but a trivial shell executor that can easily pass arguments to something like terraform or a custom shell script would be the easiest thing to write.
I'm not positive exactly what configuration will look like, but building this will also involve a new docs chapter.
The common use case this is intended to cover is the classic "AWS autoscaling" worker case.
To do this, we can also add a new flag to the worker, that allows it to run one and only one job and then exit, which could make some of this easier.
(We could also add a flag to the worker so it doesn't try to process the organization).
Configuration of autoscaling workers would be somewhat of an advanced topic, requiring construction of a good worker image preconfigured to speak to the DB, but in all this seems like it would be a very easy system to maintain.  By suggesting tools like terraform and making it pluggable, we can avoid maintaining any cloud specific plugins.

This is done - https://talk.vespene.io/t/autoscaling-features-just-added/51
Comment on the thread if you have ideas or questions!


cp release files to the release dir

Currently copied to the src dir

I think there's an extra space in the code that would mess up the cp target:
cp $^ $RELEASEDIR && cp imports/* $RELEASEDIR imports
It should be:
cp $^ $RELEASEDIR && cp imports/* $RELEASEDIR/imports


412 (Precondition Failed)

Hey guys,
I am uploading a file to s3 using the plugin, however, I received a 412 (Precondition Failed) error once the upload is done.
Any idea what might be causing this?
Thank you!

It is a server side issue

@danialfarid I know it is a closed issue. But, do you know what might be the issue on server side?


Documentation for external volumes

supersedes #3607
/cc @sascala

lgtm
@branden any final comments? @kolloch @sttts ?

OK, @jdef, I've made those final edits. I believe etiquette dictates you perform the merge? (I'm pushing to publish this ASAP because I believe some tooltips reference the page).


Regarding GET db-session cookie from the SSO request

To get db-session cookie, just disable redirects in this request call https://github.com/informagico/FusionSolar-API-RE/blob/3383fee7e9ad6bb0e4f2a822f2f7e1ef15c5682e/src/app.js#L103-L103 , not sure how to do in axios, probably setting maxRedirects to zero might work

Uh, that's cool thank you! Will take a look in the next days 🤘

@Luke265 it is indeed working! Thanks for the suggestion 👍
Hope to find time to implement the rest in the near future, but feel free to fork and contribute ❤


add function to get transferred size

Get transferred sizes for HAR files generated with chrome-har-capturer.

Coverage decreased (-1.7%) to 97.264% when pulling 737de8be783e6da18d34b9b2d11f5d421573dd83 on max0xff:feature-transferred-size into fcfd7b226ea970d889cbaa50e7abedb6c7cd60e4 on mrname:master.

Thanks for contributing @max0xff! I'm working today but will try finish reviewing by EOD. In the meantime, would you mind adding the new functionality to the README?

Coverage decreased (-1.7%) to 97.273% when pulling 41ffc7988ab5e39afe082fbf454a1da08b90b6cf on max0xff:feature-transferred-size into fcfd7b226ea970d889cbaa50e7abedb6c7cd60e4 on mrname:master.

Coverage decreased (-1.7%) to 97.273% when pulling 68e500933f0e677f12444d5003bb6d10aab40207 on max0xff:feature-transferred-size into fcfd7b226ea970d889cbaa50e7abedb6c7cd60e4 on mrname:master.

Coverage decreased (-1.7%) to 97.273% when pulling f63aff26ef5e18747591e492d411cdbaeb1b23ec on max0xff:feature-transferred-size into fcfd7b226ea970d889cbaa50e7abedb6c7cd60e4 on mrname:master.

Coverage decreased (-0.2%) to 98.765% when pulling c8d6c9dd9650350821c7483bebe4b40e7b5102ef on max0xff:feature-transferred-size into fcfd7b226ea970d889cbaa50e7abedb6c7cd60e4 on mrname:master.

Coverage decreased (-0.2%) to 98.765% when pulling c8d6c9dd9650350821c7483bebe4b40e7b5102ef on max0xff:feature-transferred-size into fcfd7b226ea970d889cbaa50e7abedb6c7cd60e4 on mrname:master.

Coverage decreased (-0.2%) to 98.765% when pulling c8d6c9dd9650350821c7483bebe4b40e7b5102ef on max0xff:feature-transferred-size into fcfd7b226ea970d889cbaa50e7abedb6c7cd60e4 on mrname:master.

Thanks for contributing @max0xff! I hope this is the first of many open source contributions for you!


AVRO-2405 Make Travis green again

Would be great to get Travis green again, so we can work on 1.10.0 and 1.9.1 👍
https://jira.apache.org/jira/browse/AVRO-2405
Update. This is very sad, it looks like that the previous Java11 jobs were just running 8: https://travis-ci.org/apache/avro/jobs/525676162#L880 Apache Forrest does not support Java11.

I've removed Forrest from the build since it is not Java11 compatible. In the end, we don't need this for the CI, but only for generating the docs while releasing. We can do this with Java8.

I've removed the commented Apache Forrest install from the Dockerfile. Installing Forrest is part of the How to release guide.


Added debug logs

Can you include https://github.com/FreeRDP/FreeRDP/commit/ae49581673e55680251c0050787fbffca38d95ca here too?

Thanks! Did another test, works fine.


Fix 'idz' is not defined

Fix 'idz' is not defined

@Negashev thanks for contributing this! Looks good to me.
We are in the process of figuring out our CLAs/commit process before we merge this. Let me circle back soon with the next steps.

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


Audio recorded in mimeType audio/x-matroska despite specifying blobPropertyBag as audio/mpeg

Here is an example of the file generated using the media recorder-> https://botspace-inbox.s3.ap-south-1.amazonaws.com/assets/5f64256823133d00d13d7b19_1650675975443.mp3
On checking the metadata from -> https://exifmeta.com/ the results are as follows -> https://cln.sh/j6bBwz
The file mimeType is set to audio/x-matroska, I want the file mimeType to be audio/mpeg.
const {         status,         startRecording,         stopRecording,         mediaBlobUrl,     } = useReactMediaRecorder({         audio: true,         blobPropertyBag: {             type: "audio/mpeg"         }     });

Any update on this? Trying to set audio codec to AAC but neither mediaRecorderOptions nor options works.
<ReactMediaRecorder
               mediaRecorderOptions={{CODEC specified here}}
               options={{CODEC specified here}}
                video
                ...
                render={({
                ....}) =>
/>

It would be great to change the default audio codec from OPUS to other protocols.


Weird mic and refresh icon on the bottom right

Windows Terminal version (or Windows build number)
1.8.1521.0
Other Software
No response
Steps to reproduce
It just shows up sometimes.
Expected Behavior
No response
Actual Behavior

Is there any other software you're using? I'm pretty sure these icons do not belong to the terminal.

@skyline75489 I see. Currently running softwares are:

PowerToyz
VS Code
Notion
Edge

Will close the issue. Thanks!

This was driving me crazy! I tracked this problem down to the NVIDIA GForce Experience.
Seems that GForce thinks the WinTerm Preview is a video game and you might want to record your session and comment on it.
I just stopped the NVIDIA services and set them to manual.
MS, you might want to investigate why GForce Experience thinks this is a game or what you have enabled in the Preview build that looks like a game.

@ThomasMConnors We've actually had this discussion a few places over the repo, and already managed to get in contact with NVidia about this. See https://github.com/microsoft/terminal/issues/11230#issuecomment-921279806


Investigate term.js alternatives

term.js is showing its limitations, the two options are to either fork it ourselves to suit our use case, or go with one of its more prominent forks:

Community fork, converted to TypeScript https://github.com/sedwards2009/extraterm
Built for commercial product https://github.com/sourcelair/xterm.js
https://github.com/IonicaBizau/web-term
https://github.com/shockone/black-screen

sedwards2009/extraterm looks very active https://github.com/sedwards2009/extraterm/graphs/contributors
Here are some of the problems that have sparked this issue:

Integrated terminal cursor is in the wrong position after resize #6784
Autoscrolling does not work in integrated terminal #6755
selection problem in integrated terminal #6750
Integrated terminal wraps incorrectly #6740
Using cls in integrated terminal sometimes freezes it #6735
The integrated terminal appears to be bottom-aligned, which makes it wobbly #6729
Terminal: I am not getting scrollbars for scrolling? #6602

Might I suggest:
https://github.com/shockone/black-screen
It's an electron/react app that seems to do a really nice job of enhancing the traditional TUI. I haven't had a chance to try it, but never have I seen anything coming close to bridging CLI and GUI. This is really fkn cool, Imagine that intergrated with VS Code!
That project appears to be pretty mature, and it's even written in typescript! :wink: TBH, I don't know anything about the Electron api, but I would assume(/hope) that it would be possible to run that console in an isolated process, which renders its gui to a container docked within the VS Code editor . Then create some mechanism to enable communication between the editor/terminal via some new IPC interface. But then again, it might just be easier to merge the two projects completely, I don't know enough about either backend to offer any meaningful conjecture on the implementation....
However, I recently started using ConEmu, which has this render to pane feature and it's awesome. Yet, the two apps are oblivious of each other. There is no out-of-the-box-way to facilitate communication between the two. Just to launch a task I would have to write some contrived daemon that runs in the shell waiting for vsc to run some other script that pokes that daemon into performing some action within the active shell... yuck. :hear_no_evil: Imagine a unified workspace / intellisense bridging that gap. damn.
Few cool features (but definitly checkout the repo itself!!)
Intellisense-esque completion when typing into the shell (and theres is  a custom completion provider api in black-screen)

JSON explorer like in chrome devtools:

Still works with your favorite tools:

Obviously this would be a wayyyyy larger investment (in terms of manhours) than term.js but the possibilities are mind boggling when you consider some of the possible workflows you could set up with this...
My friends, just.......

Thanks for the suggestion @rozzzly, my main concern with something like that is that it's not leveraging people's terminal setups. For example, I would probably be losing some information I find most useful as I have a custom prompt to display git status for me. Will check it out though.

I have actually tried black-screen before, it seemed kind of cool but it was different to what I was used and didn't particularly want to change :P

@Tyriar totally agree with your concerns.
I wrote an atom package for embedding a terminal (github.com/romgrk/termrk) (using term.js, but definitely not the best solution), and a recurrent point of attraction for users was to have their usual terminal setup available from Atom, be it zsh, bash, or other.

We can mix black-screen and xtrem feature into out vscode terminal

Spend a while bashing at xterm.js and created a bunch of issues https://github.com/sourcelair/xterm.js/issues. It certainly seems to be in a better state than term.js imo, I haven't played much with it actually within vscode yet though.

@Tyriar I can confirm that term.js cannot handle DBCS characters well.

@be5invis we're likely going to go with xterm.js, I created a bunch of i18n issues over there after playing with it a bit:

Pasting in 日本語 adds an unexpected line break after 本 which puts the cursor off screen https://github.com/sourcelair/xterm.js/issues/61
CJK characters are not treated as double width chars https://github.com/sourcelair/xterm.js/issues/62
Moving over CJK text jumps 2 characters at a time https://github.com/sourcelair/xterm.js/issues/70

I think they're all related to the width and byte size of the characters.

I am seeing my integrated terminal does not honor line breaks... example in notepad:
mkdir directory 1[enter]
mkdir directory2
copy paste the above into terminal, it will run as one command with line breaks removed: mkdir directory1mkdir directory2
This happens on windows term with both cmd.exe and powershell so I assume it relates to the integrated terminal functionality itself?

@danekan can you open a new issue for that and mention your on Windows? It seems to be working find on Linux.

sure

Oh there's another problem: term.js treats 〈 and 〉 as full-width, while Windows measures the real character width and layout them (at least under CP936),

Put out a big PR that fixes the major scrolling issues with xterm.js https://github.com/sourcelair/xterm.js/pull/101
Since I've already merged in xterm, I'm going to close this issue off.


net/http - HTTP Response body not filling POST data values

What version of Go are you using (go version)?

$ go version
go version go1.17.8 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ uname -a
Linux pwnbox-base 5.14.0-2parrot1-amd64 #1 SMP Debian 5.14.6-2parrot1 (2021-09-25) x86_64 GNU/Linux

What did you do?

Noted that for some websites (for example Wordpress Login pages), when net/http is used for performing a HTTP POST request (not tested for GET requests), some variables inside the HTTP response body don't store POST data provided by the request For example, by running the following GO script for performing a HTTP POST request to a public Wordpress-based website:
package main

import (
    "bytes"
    "encoding/json"
    "fmt"
    "io/ioutil"
    "net/http"
)

func main() {
    reqBody, err := json.Marshal(map[string]string{
        "log": "admin",
        "pwd": "tedsdad",
        "wp-submit": "Log In",
        "redirect_to": "http://www.rangelagos.com/wp-admin/",
        "testcookie": "1",
    })
    if err != nil {
        print(err)
    }
    resp, err := http.Post("http://www.rangelagos.com/wp-login.php",
        "application/json", bytes.NewBuffer(reqBody))
    if err != nil {
        print(err)
    }
    defer resp.Body.Close()
    body, err := ioutil.ReadAll(resp.Body)
    if err != nil {
        print(err)
    }
    fmt.Println(string(body))
}

inside the HTTP response body, on the login form section, we get this html code:

                Username

                <input type="text" name="log" id="user_login" class="input" value="" size="20" tabindex="10" />

Focus on the value variable that is empty.
If we execute the same HTTP Post request by the following cURL command:
curl 'http://www.rangelagos.com/wp-login.php' -H 'User-Agent: Mozilla/5.0 (Windows NT 10.0; rv:78.0) Gecko/20100101 Firefox/78.0' -H 'Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8' -H 'Accept-Language: en-US,en;q=0.5' --compressed -H 'Content-Type: application/x-www-form-urlencoded' -H 'Origin: http://www.rangelagos.com' -H 'DNT: 1' -H 'Connection: keep-alive' -H 'Referer: http://www.rangelagos.com/wp-login.php' -H 'Cookie: xLanguage_=en; wordpress_test_cookie=WP+Cookie+check' -H 'Upgrade-Insecure-Requests: 1' -H 'Sec-GPC: 1' --data-raw 'log=admin&pwd=dsads&wp-submit=Log+In&redirect_to=http%3A%2F%2Fwww.rangelagos.com%2Fwp-admin%2F&testcookie=1'

inside the HTTP response body, on the login form section, we get this html code with the right value of value variable:

                Username

                <input type="text" name="log" id="user_login" class="input" value="admin" size="20" tabindex="10" />

in this last case, we have the value variable correctly filled by the data admin.
It means that the value variable, in some specific cases, cannot be written with the right value but stay empty.
Tried also by other go scripts of other mates or found online but with the same result.

i think it is something about server side response.

If we execute the same HTTP Post request by the following cURL command

Just a note that the curl request is not identical to your go code's request. It's different in several ways, including using application/x-www-form-urlencoded rather than application/json.

This is a server side issue, not one with Go.
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Enable traces batching by default

Requirements for Contributing to this repository

Fill out the template below. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
The pull request must only fix one issue at the time.
The pull request must update the test suite to demonstrate the changed functionality.
After you create the pull request, all status checks must be pass before a maintainer reviews your contribution. For more details, please see CONTRIBUTING.

What does this PR do?

Enables sending trace data in batches by default.
Prior to this change trace data batching could be enabled by setting the DD_JENKINS_ENABLE_TRACES_BATCHING env variable to true.
This was done as a precaution in case batched requests did not work as expected.
Now that they are known to work (there is a customer who has enabled batching), we can enable them by default.
Description of the Change

Alternate Designs

Possible Drawbacks

Verification Process

Additional Notes

Release Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bug fix MUST have appropriate tests (unit, integration, etc...)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have one changelog/ label attached. If applicable it should have the backward-incompatible label attached.
[ ] PR should not have do-not-merge/ label attached.
[ ] If Applicable, issue must have kind/ and severity/ labels attached at least.

Change LGTM, but I'm wondering if we just could keep the env var with default true

I guess I'm cutting corners a bit here, but I feel confident enough about batching - there's a customer that already verified that it works. And if we do happen to have a problem, there's the option of downgrading the plugin (which is not ideal, but not much worse than having to change an env var and do a restart).


Rename TokenCache.cacheHasChanged to TokenCache.hasChanged

Node TokenCache used to be CacheManager. At that point, CacheManager.cacheHasChanged made sense. Now that it's been updated to TokenCache, TokenCache.cacheHasChanged is a bit redundant. Proposing to update this to TokenCache.hasChanged

Coverage remained the same at 83.339% when pulling bdaac1999551a37c510c68423ae93922a02a2348 on msal-node/token-cache into dd24960579fcb7d4e13026a8674e4ada1287cb95 on dev.


[Color Picker- Editor]: Color Contrast ratio of 'Color Format'(RGB) text is 2.964:1 which is less than 4.5:1.

Microsoft PowerToys version
"0.41.1"
Running as admin

[ ] Yes

Area(s) with issue?
ColorPicker
Steps to reproduce
Test Environment:
OS: Windows 10 Version Dev
App version: 0.41.1
App name: PowerToys
Screen Reader: Narrator
Tool: Accessibility Insight for Windows
Repro Steps:

Launch 'PowerToys' application.
Navigate to 'Color Picker' tab.
Then Open 'Color Picker' using defined Shortcut key.
Pick any color.
Inside Color Picker Editor, Check 'Color format' text like RGB Color Contrast and Observe the issue.

User Impact: -> Priority 1
Low vision user are impacted here as they will not able to see text properly as luminosity ratio is not good.
WCAG Reference:
https://www.w3.org/TR/WCAG21/#contrast-minimum
Attachment

✔ Expected Behavior
Color Contrast ratio of 'Color Format'(RGB) text should be Greater than or equal to 4.5:1.
❌ Actual Behavior
Color Contrast ratio of 'Color Format'(RGB) text is 2.964:1 which is less than 4.5:1. Issue repro with all color format text inside color picker editor.
Other Software
No response

@niels9001 @jaimecbernardo, is this something we can address quickly? This is going to appear on our next accessibility review at the end of the month as out of SLA.

Assuming this is missing .47 so assigning to .49

@crutkas When do we wrap-up 0.47? PR was just submitted for this issue so it might be something that is still on time for 0.47

We started test builds on Friday, but we can run a new one to get this in for 0.47. Thanks @niels9001 !

Resolved with the release of v0.47.0.


Added no_border option to console.print_exception

Type of changes

[ ] Bug fix
[x] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[x] I've run the latest black with default args on new code.
[x] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[x] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
I added no_border option to console.print_exception.
This PR closes the issue #2936

There may be a need for a simpler output format, but no_border is too descriptive.
A future release may have a switch that does this, and other changes.
Thanks for the PR.

The boxes around the traces make it very noisy communicating with AI. It looks create for humans but gets messed up when pasting it. A flag to turn it on/off makes it easier when actively developing the code.

@willmcgugan this was a great pr in the lack of "better" solution - would have been great to include it while you look for other long-term options

This is pretty much the only thing holding us back from using rich tracebacks right now - it just makes it horrendous to try copy&paste a traceback with all the border characters.


Update React Native document picker to v8

Details

We've submitted a PR to fix an issue with RN document picker: https://github.com/rnmods/react-native-document-picker/pull/527 and it was merged
Fixed Issues

$ https://github.com/Expensify/App/issues/7267
Tests

[ ] Verify adding document attachments works and uploads the selected document

QA Steps

[ ] Verify adding document attachments works and uploads the selected document

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop

iOS

https://user-images.githubusercontent.com/12156624/150413370-d3f27957-50c8-472b-9ea2-e6250932ec6c.mov
Android

https://user-images.githubusercontent.com/12156624/150405574-7bd51d62-4184-43cc-b6d3-49ea0fb4565c.mp4

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @mountiny in version: 1.1.31-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @AndrewGable in version: 1.1.32-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[+] check whether the handshake done is sent before write mp_ack

 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.1 out of 2 committers have signed the CLA.:white_check_mark: Yanmei-Liu:x: cherylsyYou have signed the CLA already but the status is still pending? Let us recheck it.


[help] How to use Angular umd reference like ng.core ?

In angular 5, i Can use ng.core,ng.animations... and any other umd reference after ng build. in browser
for example

But after Angular-cli 7 .
I cant do this any more.

I checked the angular source code and i found they have provided Umd module. but why after cli building i missed the Umd reference ?

The issue tracker in this repository is reserved for feature requests and bug reports. For support requests, I'd recommend you to check the correct channels:

StackOverflow with tag Angular and Angular CLI
Angular's gitter channel

Looks like that when the webpack runtime bootstraps the module, it passes an empty object as the value of exports.
Although this seems intentional and interacting with global properties is strongly discouraged, let's reopen the issue in case @hansl, @filipesilva, @alan-agius4 have something to add.

If i  separate import the core.umd.js , it could work. So I can't judge whether this is a cli's Bug or some webpack configuration lead to this .

I check the building result. and found this.
@mgechev Thanks for your patient again.

The whole concept behind webpack is to allow more modular development. This means writing isolated modules that are well contained and do not rely on hidden dependencies (eg. globals).
While there are ways to access globals in a webpack build, you’d need to customize your config which will take you off the supported route. See: https://webpack.js.org/guides/shimming if you still want to pursue this.
As @mgechev mentioned, it is highly discouraged interacting with global references.

@alan-agius4 Thanks a lot ! i will think about it.


docs: update example usages by removing deprecated ioutil

The PR updates example usages in comments for the ParseHook, ParseSystemhook, and ParseWebhook functions by replacing the deprecated ioutil package with io.
Follows #1521

Thanks @alexandear , merging! 🎉


Chore Rescheduling issue

Hi MariusWirtz,
This to inform  facing issue with rescheduling chore. While rescheduling active chores  its deactivating the chore and updated the time then reactivating this steps are happening (Checked my log) and chore timestamp also updated in data folder. but some how the chore is not executing after successfully updated the chore details.
Doubt: is Chore required restart?
regards,
Ram Kumar

Hi @Rnvksm,
what version of the TM1 Server are you using?
Use this code to find out:
from TM1py import TM1Service

with TM1Service(address='localhost', port=50501, user='Admin', password='apple', ssl=True) as tm1:
    print(tm1.version)

Hi Marius,
Ran the above query got this info 11.8.00000.33 ( Planning Analytics <IP_ADDRESS>)
Today restarted my instance after that my chore got executed as per reschedule.
As per my knowledge chore doesn't require restart right  but some how reschedule functionality(TM1py) is working after restart.

Hi @Rnvksm,
thank you.
This is interesting. If it only works after a restart, it sounds like an issue in the TM1 Server to me.
Perhaps to work around the issue, you can delete the chore and recreate it instead of updating it.

Thanks @MariusWirtz  for quick reply.
I tried with save data after rescheduling chore. still that's not working. i made a utility called Chore automation Dashboard there i added all functionalities related with chores except reschedule everything is working fine.

Interesting.
If you reschedule the chore like this, do you still have to restart the server?
from TM1py import TM1Service

with TM1Service(address="", port=12354, ssl=True, user="admin", password="apple") as tm1:
    chore = tm1.chores.get(chore_name="1400_DailyProcessing")

    chore.reschedule(hours=-1)

    tm1.chores.delete(chore_name=chore.name)
    tm1.chores.create(chore=chore)

Thanks @MariusWirtz
Sure. i will try and will let you know the result.
Regards,
Ram Kumar

Hi @MariusWirtz ,
I tried the above code still chore is not  executing by chore scheduler. i think it need restart.
Thats the only option working for reschedule  the chore.

Thank you for the confirmation.
I will reproduce the issue on my site and report it with IBM.

Thanks Marius,
If you find any solution please let me know


New showcase example on CodeSandbox

Feature description
I think it would be interesting to have an editable example in README, like the ones from Blueprint and Material (however the Material one may be too simple). I tried to run the example by @tajo on codesandbox, but had no luck so far. Apparently, the dependencies, as they are today, are too complex for codesandbox, as can be seen here. Does anyone else thinks that it would be interesting to have a slightly bigger example on README as some sort of showcase? If so, should we aim at making the example by @tajo run on codesandbox or incorporating it as a separate page inside the documentation? Or perhaps creating something new but more simple, just for codesandbox?  Thanks!

The more examples, the merrier! CodeSandbox should work; however, without Fusion.js. Do you want to give it a shot?

Nice! Yes, I would like to. I will let you know how it goes :)

Hi @tajo ! Sorry for coming back to you so late, I had a tough week. I tried to follow the instructions to run your example, but got the following error:

When I tried to open https://unpm.uberinternal.com/baseui/-/baseui-5.4.0.tgz#b4606513b264b610e1cabc0347c6a4831d663faf on the browser, I was asked for a username and password, so I guess we would need to replace uberinternal.com dependencies for something public. Besides this, since we will need to remove fusion.js and you made a tutorial for your app on README, should I start from scratch, maybe using create-react-app?

try pulling the latest master. it should resolve the unpm issues

@IgorWilbert Oops, my bad. I committed wrong yarn.lock. It's fixed now.

No problem @tajo, now I was able to run the project, thank you for fixing it. Do you have any suggestion about where should I start working to remove fusion? Thanks!

I think you can keep Fusion? CodeSandbox has a Next.js template. Fusion is very similar to Next.js so I assume it should work in that environment as well. If not, I would replace Fusion with Next.js. Or there is create-react-app template as well. So this would be my preference:

Preserve Fusion
Use Next.js template
Use CRA template

Hey @tajo ! Just to keep you updated, I started a sandbox with Next.js and uploaded files manually. It automatically shifted to Node.js template, and seems to be working fine, as can be seen here. I'll review it more carefully soon, as there seem to be some details missing. Thanks!

@tajo I reviewed it and it seems to be fine https://codesandbox.io/s/elegant-dhawan-b7kgx .The console errors are the same that can be seen if you run the project. What would you like me to do with it? Transfer the sandbox to another account, add the link above to README, or something else?


Otherstone Trader Recipe doesn't respect output quantity

Describe the bug
Otherstone Trader Recipe doesn't respect output quantity
To Reproduce
Trade stone with Otherstone trader. Four stone gets one otherstone.
Expected behavior
Recipe JSON suggests it should give 2 otherstone.
Screenshots
If applicable, add screenshots to help explain your problem.
System (please complete the following information):

Occultism Version: occultism-1.16.5-1.7.1
OS: Windows
Minecraft Version: 1.16.5
Modpack Link and Version, or list of mods:
Small instance with only Occultism, JEI, Curios, and Patchouli,

Additional context
  "type": "occultism:spirit_trade",
  "ingredients": [
    {
      "item": "minecraft:stone"
    },
    {
      "item": "minecraft:stone"
    },
    {
      "item": "minecraft:stone"
    },
    {
      "item": "minecraft:stone"
    }
  ],
  "result": {
    "item": "occultism:otherstone",
    "count": 2
  }
}```

And, if not a bug, then consider this a feature request as I'd like to be able to increase his output for e6e as a considerably cheaper way of getting otherstone compared to the fire.

Definitely a bug! The change was made to enable making the trader more useful than the fire .. will look into that!


Don't save widget default titles

Summary of the pull request
When the user saves a widget with not title, a default title is displayed for them. For repository widgets, it is the name of the repository. For user widgets, it is the username. This title was being saved in the widget data, but resulted is some QoL issues. This PR makes these default titles to not be saved anymore, and just rendered in the content is loaded to be presented.
References and relevant issues
Detailed description of the pull request / Additional comments
As the issue above describes, when the user saves a repository widget, it sets the title to be its repository's full name. When the user customizes this widget and changes to another repository, the previous title keeps saved, requiring the user to actively change or erase the title.
This default title does not need to be saved as it is a information that we get on time from the widget's information. So it is the best choice to not save it in the widget's data string and just generate the string when it is needed.
Validation steps performed
PR checklist

[ ] Closes #430
[ ] Tests added/passed
[ ] Documentation updated

I renamed the PR since the original wording made it sound like a bug, instead of a fix to the bug.


[stable/postgresql] Fix data permission error on pod restart (#14538)

The command 'mkdir -p' has no effect if the directory already exists. To ensure the init container sets the mode for an existing data directory, we call 'chmod 700' separately.
Signed-off-by: Tim Guenthner<EMAIL_ADDRESS>
What this PR does / why we need it:
The Postgres pods fail when restarted due to the data directory mode not being set by the init container. This change ensures the correct mode is set.
Which issue this PR fixes

fixes #14538

Special notes for your reviewer:
This issue was introduced recently in v5.2.2.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @carrodher

/lgtm
/ok-to-test


Update DragonSparkMax to support closed loops

Update to eliminate TODOs and make sure the closed loop works

void DragonSparkMax::Set(double value) comments out the control modes and just does open loop processing.


bugfix: "You deleted flag undefined"

Description
A small fix for having correct deleted flag id instead of undefined.
Motivation and Context
Before

After

How Has This Been Tested?
Manually, locally.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #273 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #273   +/-   ##
=======================================
  Coverage   81.76%   81.76%
=======================================
  Files          26       26
  Lines        1530     1530
=======================================
  Hits         1251     1251
  Misses        210      210
  Partials       69       69

Continue to review full report at Codecov.

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


Failed to download NLTK data: HTTP ERROR 405 / 403

>>> nltk.download("all")
[nltk_data] Error loading all: HTTP Error 405: Not allowed.

>>> nltk.version_info
sys.version_info(major=3, minor=5, micro=2, releaselevel='final', serial=0)

Also, I tried to visit https://raw.githubusercontent.com/nltk/nltk_data/gh-pages/packages/corpora/cmudict.zip. Got the same HTTP 405 ERROR.
Find the same problem on stackoverflow: https://stackoverflow.com/questions/45318066/getting-405-while-trying-to-download-nltk-dta
Any comments would be appreciated.

It seems like the Github is down/blocking access to the raw content on the repo.
Meanwhile the temporary solution is something like this:
PATH_TO_NLTK_DATA=/home/username/nltk_data/
wget https://github.com/nltk/nltk_data/archive/gh-pages.zip
unzip gh-pages.zip
mv nltk_data-gh-pages/ $PATH_TO_NLTK_DATA

@alvations Thank you very much!

Is there any alternative for the command line downloads such as this?
python -m nltk.downloader -d ./nltk_data punkt

@plaihonen you should be able to use this alternative index by doing something like python -m nltk.downloader -u https://pastebin.com/raw/D3TBY4Mj

@rvause Works perfectly. Thank you!

+1. this was a several hour surprise this morning. Went with completely bypassing nltk download for now

GitHub is currently blocking access because "a user is consuming a very large amount of bandwidth requesting files".  They have also suggested that we should look at a different way of distributing data packages, e.g. S3.

Even with alternate index, anyone finding that some packages still don't work?
Specifically, for me, the stopwords package gives me a 405, the others (brown, wordnet, punkt, etc) do not.

yes, Im not able to download the nltk stopwords either. I get the 405 error when I do >python -m nltk.downloader -u http://nltk.github.com/nltk_data/

Hey, I am trying to run python -m nltk.downloader stopwords, but getting 405 error. Can anyone point me in the right direction?

@dfridman1 @prakruthi-karuna read the issue above. Work around is:
python -m nltk.downloader -u https://pastebin.com/raw/D3TBY4Mj all

We have a few projects using this in our ci system. Rather than having to update all of them with the -u parameter is there another way we can specify that data. Maybe an environment variable or config file?

Yeah, I've just received this too:

Hi Liling,
I work on the Support team at GitHub, and I wanted to notify you that we've had to temporarily block access to files being served from raw.githubusercontent.comURLs for the alvations/nltk_data repo. Currently, a user is consuming a very large amount of bandwidth requesting files from that repo, and our only option at the moment is to block all requests. We've actively working on ways to mitigate the problem, and we'll follow up with you when we have an update.Please let us know if you have any questions.
Cheers,Shawna

@zxiiro See https://stackoverflow.com/questions/3522372/how-to-config-nltk-data-directory-from-code

@ewan-klein @stevenbird I think we need a new way to distribute data but that'll require some rework of the nltk.downloader.py.

Seemingly, we have no choice but to change the data distribution channel:

Hi Liling,
Wanted to follow up on this with some additional information. We've been discussing the issue internally, and it's highly likely that we will not be restoring raw access to repos in the nltk/nltk_data fork network for the foreseeable future. The issue is that there are a number of machines that are calling nltk.download() at a very high frequency. We cannot restore raw access until that activity stops.Feel free to share this message with the nltk community. We're hoping that whoever is doing this will be alerted to the problem, and stop whatever process is doing this.
Cheers,Jamie

One would think they could just block those IPs specifically. But maybe there is more to it than that.

I do have a docker image that downloads nltk_data, but I wasn't rebuilding it frequently. I hope I wasn't one of those high traffic users.

Is there a process we can follow at this time?

Maybe someone configured their scripts on AWS wrongly. @everyone please help to check your instances while we find an alternative to data distributing

Hi Liling,
We can't share specific numbers, however the requests are coming from a large number of AWS instances. We suspect it could be a script or build process gone awry. We don't know much beyond that.
Cheers,Jamie

Well, I don't use AWS.
:relieved:

Code-wise, maybe we've to change how frequent the same package is being updated too. Otherwise no matter which distribution channel we migrate to, the same service disruption will happen.

Maybe something torrent-based would work?

Not sure what the license is like, but you could make them public on s3: https://aws.amazon.com/datasets/

@alvations seems only the gzip download works for now. And packages needed to be moved under /home/username/nltk_data/ folder.
export PATH_TO_NLTK_DATA=/home/username/nltk_data/
wget https://github.com/nltk/nltk_data/archive/gh-pages.zip
unzip gh-pages.zip
mv nltk_data-gh-pages $PATH_TO_NLTK_DATA
# add below code
mv $PATH_TO_NLTK_DATA/nltk_data-gh-pages/packages/* $PATH_TO_NLTK_DATA/

Do we have a temporary work-around yet?

@darshanlol @alvations mentioned a solution. If you are trying to build a docker, the following worked for me:
ENV PATH_TO_NLTK_DATA $HOME/nltk_data/
RUN apt-get -qq update
RUN apt-get -qq -y install wget
RUN wget https://github.com/nltk/nltk_data/archive/gh-pages.zip
RUN apt-get -y install unzip
RUN unzip gh-pages.zip -d $PATH_TO_NLTK_DATA
# add below code
RUN mv $PATH_TO_NLTK_DATA/nltk_data-gh-pages/packages/* $PATH_TO_NLTK_DATA/

I try to change the default url  in 'nltk.downloader.py'.But the issue still exist.

Workaround suggested is no longer working:
python -m nltk.downloader -u https://pastebin.com/raw/D3TBY4Mj all

Currently this is the only working solution:
PATH_TO_NLTK_DATA=/home/username/nltk_data/
wget https://github.com/nltk/nltk_data/archive/gh-pages.zip
unzip gh-pages.zip
mv nltk_data-gh-pages/ $PATH_TO_NLTK_DATA

As @alvations said this is the only working solution.

PATH_TO_NLTK_DATA=/home/username/nltk_data/
wget https://github.com/nltk/nltk_data/archive/gh-pages.zip
unzip gh-pages.zip
mv nltk_data-gh-pages/ $PATH_TO_NLTK_DATA

But even after downloading all the pages i was facing issues as my NLTK downloader was not able to detect all the packages downloaded for that you may have to manually change the download directory value through command.
This page has proper command that i used to configure NLTK data packages
Click on the above link for answer.

Here're a couple of proposals to resolve this problem after reading around and finding alternatives.
Make corpora pipable

First, we'll change it such that all nltk_data are pip-able. (So every new environment will require a new pip install and we no longer rely of the physical directory)

We will need to keep track of some sort of index too for the download to fetch and track versions.

Then we also need to have some sort of overhaul on the code, the downloader.py and all the related corpus reader interface

Possibly pip limitations (from PyPI side) can stop the rogue users/machines with high frequency requests

Hosting the data on S3 / Zenodo or some private host
This would require us to simply relink the links in the index.xml to the appropriate links.
But if the traffic remains high due to some installation / automation script gone wrong, we end up bugging one service provider to another and if it's S3 incur cost.

Any other suggestions?
Any brave soul who wants to take this on?

@harigovind511, yeah you have to either put the downloaded nltk_data folder in one of the standard locations where the nltk knows to look for it, or append to nltk.data.path to tell it where to look. The automatic downloader just looks for a standard location.

 My vote would be for pip unless there's some problem (or taboo) with large packages on pip?
Using pip would also solve the manual nltk.download() and in-code package management.

Files seem back up? Seems wise though to continue to seek alternative distribution mechanisms. In my own organization though we plan to move to hosting internally and check in quarterly

I'd like to understand what $PATH_TO_NLTK_DATA does. Is it configuring an alternate local download URL for where NLTK get's it's data?
I'd like to setup a local cache of NLTK data so I was wondering if setting this tells NLTK to work offline?

Since the root of the problem is bandwidth abuse, it seems a poor idea to recommend manual fetching of the entire nltk_data tree as a work-around. How about you show us how resource ids map to URLs, @alvations, so I can wget just the punkt bundle, for example?
The long-term solution, I believe, is to make it less trivial for beginning users to fetch the entire data bundle (638MB compressed, when I checked). Instead of arranging (and paying for) more bandwidth to waste on pointless downloads, stop providing "all" as a download option; the documentation should instead show the inattentive scripter how to download the specific resource(s) they need. And in the meantime, get out of the habit of writing nltk.download("all") (or equivalent) as sample or recommended usage, on stackoverflow (I'm looking at you, @alvations) and in the downloader docstrings. (For exploring the nltk, nltk.dowload("book"), not "all", is just as useful and much smaller.)
At present it is difficult to figure out which resource needs to be downloaded; if I install the nltk and try out nltk.pos_tag(["hello", "friend"]), there's no way to map the error message to a resource ID that I can pass to nltk.download(<resource id>). Downloading everything is the obvious work-aroundin such cases. If nltk.data.load() or nltk.data.find() can be patched to look up the resource id in such cases, I think you'll see your usage on nltk_data go down significantly over the long term.

@zxiiro PATH_TO_NLTK_DATA has no meaning to the nltk, it's just a variable in the sample script. NLTK_DATA does have special meaning. See http://www.nltk.org/data.html, all the options are explained.

@alexisdimi agreed on the nltk.download('all'). Sorry that was such an old answer from my early days. I should advise against it.
One of the problem with wget directly to a package is that it's still relies on the raw content on github. During the down time, the https://github.com/nltk/nltk_data/blob/gh-pages/packages/tokenizers/punkt.zip  link was also leading to the 403/405 error.

Looks like the lockout has been lifted, that's great! Now I hope there are some tickets that will work toward preventing similar problems in the future (maybe along the lines I suggested, maybe not).

@alexisdimi putting warnings that suggest for users to download the appropriate models is a good idea.

For those running NLTK in a CI environment. I'd like to propose GH-1795 which allows use to specify an alternative URL for downloading. The idea here is one can setup a local copy of nltk_data on a webserver (or even python -m http.server) and then have a global variable that can override the download URL.
This is so that we can override without modifying projects local command calls to include -u from a CI system like Jenkins.

Question to Github regarding pip data distribution using releases and pip installation:

We're looking for alternatives to host the nltk_data and one of it is to host them as repository releases as how SpaCy does it https://github.com/explosion/spacy-models/releases
Could we just check with you whether the same block will be executed if similar high frequency requests were made to the repository releases? Or is the repository releases treated differently from the raw content on Github?
Regards,
Liling

Some updates on Github side:

Hi Liling,
Using Releases just moves the requests to a different part of our infrastructure. If that volume of bandwidth were to start up again, we would still have to block those requests, even if they were to Releases.
We've tried to think of some ways that the data packages could remain on GitHub, but there's honestly not a good solution. We're just not set up to be a high volume CDN.
Cheers,
Jamie

@owaaa / @zxiiro +1 on hosting internally for CI. We're doing this now, and the advantage for EC2/S3 users is that you get to put the data (or the subset of it you need) close to where you want to build the machines. If you're across availability zones, you can just replicate buckets where you need and be more robust to what's going on outside AWS.
@alvations I quite like the data/model as package idea in spaCy, but one of the consequences is that if you use virtualenv, your environment directories can balloon in size as your packages lives there. Of course, this buys you completely isolated and auditable data/model versions, which is valuable for a project with frequent model updates like spaCy, but not a free lunch 😕

Gitignore has venv in it anyways. Imo using pip for corpus would make it much more easier to distribute programs that require a specific corpus too. Different packages for each corpus would help in not bloating stuff too. If someone then wants to install all corpuses the pip install nltk-corpus-all can easily install them all as dependencies. Similarly for popular.

While trying to download NLTK data I'm facing this error.
Error loading punkt: <urlopen error [WinError 10061] No connection could be made because the target machine actively refused it>
I just run these 2 commands:
import nltk
nltk.download('punkt')

@sthakur369
Perhaps some of the solutions from https://github.com/nltk/nltk/issues/3092#issuecomment-1366526151 will help. Alternatively, others have reported success with a VPN or mobile hotspot. I believe the problem is caused by a DNS issue somewhere, and it cannot be reproduced by everyone.
For reference, the nltk.download will try to load this page: https://raw.githubusercontent.com/nltk/nltk_data/gh-pages/index.xml
Which is hosted by GitHub. For some users, this page will simply not load.

@tomaarsen thanks for the reference. The issue is with Jio's DNS, I switched my wifi to another network, and now it's working fine. But it's really interesting how come only Jio's DNS is blocked

That that it works now! I'm afraid there's not much I can do to help, nor am I sure who I could contact about the issue. There's some more discussion on it here: https://github.com/community/community/discussions/32889

I am facing the same issue. How to download?

@adsk2050 if you are using JIO network then switch it to some other network and try again.


fix transformers_env

The bug is triggered when the TRANSFORMERS_OFFLINE and TRANSFORMERS_CACHE environment variables are set in the system.
The logic needs to be handled carefully.

tested  train sdxl model successfully.

windows tested？


My test suite thinking

Test Suite
The intention of the Test Suite is to write tests that can be used to verify that a Solid Server implementation is in compliance with the Solid Specifications. As such, it is not intended to cover uses of Solid entirely, as it does not test front-end implementations.
For now, this document represents @kjetilk's personal view, but it may evolve into an introduction to what will be "The Solid Test Suite".
The intention is further that implementors can set up a test server, populate it with some initialization data, and set up a public URL for it, submit that to the Solid Test Server, and then launch the test suite against it to get a test report. It should also be possible to
integrate into continuous integration systems for testing there.
There exists several test suites that are made by other parts to test important components of Solid, notably the Linked Data Platform 1.0 Test Suite. It is therefore not the intention to implement all tests that are needed, rather provide a comprehensive overview.
Integration Points
To provide a comprehensive overview of tests, we need some points of integration. To bring the existing test suites together, we have decided to package them in a Docker container. Each test suite will create an EARL report, and then the EARL Reports can be presented in a consistent manner.
RDF-based fixtures
There was some desire within the project to have at least the test manifest formulated with RDF. Since we bring along a large legacy, it would take some significant effort to bring all that into RDF. However, @konobi had a generic framework for formulating test fixtures in a given data structure, and it was straightforward to adapt that to be used with RDF.
These test fixtures formulations can link to test implementations, e.g. Web::Solid::Test::Basic. The framework could allow these to be implemented in several languages, for now, it is natively Perl.
Initial seed
The POD server under test needs some initial data, at the very least, it needs a root ACL to authorize the test client to start writing data.
It is therefore attractive to start with such a minimum and write tests that test quite extensive workflows, consisting of both reads and various write operations with and without authentication and authorization. This would result in rather large test scripts.
Another possibility is to have relatively small and reusable test scripts that can be parameterized to test many different situations by changing parameters in the fixtures. This is attractive because it can test smaller units and so more easily pinpoint problems. It would,
however, require many resources to seed the tests.
We already have tests that resemble the latter, and arguable, it is desireable to formulate the tests mostly in terms of the fixture tables, and on balance, my plan is therefore to use the latter approach.

textual:

"set up a test server," ... "submit that to the Solid Test Server" confusing because it calls both the tester and the thing being tested a "test server".
"It is therefore not the intention to implement all tests that are needed, rather provide a comprehensive overview." -> I think you mean we will reuse existing test suites? "provide an overview" could also be interpreted as just referring developers to other test suites instead of including them in this one.

content:

re earl, good idea! so far I had just used a trivial text parser, but let's switch to EARL.
"There was some desire within the project to have at least the test manifest formulated with RDF" - that's definitely not my desire, I prefer to write tests using Jest, which is what we already use for the tests of inrupt's pod-server, or Mocha which is the one used by NSS. I imagine @acoburn will prefer to write tests he contributes in Java, and luckily we can, thanks to the Docker setup. This means you can write yours in Perl+RDF, but let's leave the setup like it is, with multiple independent testers. Unless you feel like converting all the existing ones? ;)
"fixtures" - I think I agree, as long as these fixtures are uploaded through LDP and not through a backdoor?

Regarding the root ACL, it can just be world-writable, and then the tester can set stricter ACLs on sub-containers (probably using webid's like http://tester:1234/profile/card#me) for the various ACL-related tests.

@michielbdejong from my perspective, any language is fine for writing tests so long as there is a decent RDF parser/serializer. If there's standardization around JavaScript, that works for me. If it's some other language (Python, Ruby, Java, etc), that's fine, too.

We now already have a mix of Perl, JavaScript and Java in https://github.com/solid/test-suite/tree/master/testers.

I think we can close this now, as we have advanced beyond it.


Conditional deployment not working as expected.

Bicep version
Bicep CLI version 0.19.5 (87ca110fc0)
Describe the bug
Conditional deployment deploys into UK South, but does not deploy into UK West. Expected behavior is rg-storage-test-uks containing satest123uks in UK South, and rg-storage-test-ukw containing satest123ukw in UK West. Actual behavior is rg-storage-test-uks containing satest123uks in UK South, and empty rg-storage-test-ukw in UK West.
To Reproduce
module.main.bicep
param region string
param location string
resource storageAcccountUks 'Microsoft.Storage/storageAccounts@2022-09-01' = if (region == 'uks') {
name: 'satest123${region}'
location: location
sku: {
name: 'Standard_ZRS'
}
kind: 'StorageV2'
}
resource storageAcccountUkw 'Microsoft.Storage/storageAccounts@2022-09-01' = if (region == 'ukw') {
name: 'satest123${region}'
location: location
sku: {
name: 'Standard_LRS'
}
kind: 'StorageV2'
}`
main.bicep
targetScope = 'subscription'
param dateTime string = utcNow('yyyyMMddHHmmss')
param region string
param location string
resource resourceGroup 'Microsoft.Resources/resourceGroups@2022-09-01' = {
name: 'rg-storage-test-${region}'
location: location
}
module storageAccount 'module.main.bicep' = {
scope: resourceGroup
name: 'deployStorageAccount-${dateTime}'
params: {
region: region
location: location
}
}
uks.main.bicepparam
using 'main.bicep'
param location = 'uksouth'
param region = 'uks'
ukw.main.bicepparam
using 'main.bicep'
param location = 'ukwest'
param region = 'ukw'
uks.deploy.ps1
$dateTime = (Get-Date).ToString("yyyyMMddHHmmss")
$location = "uksouth"
$region = "uks"
az deployment sub create --name "mainDeployment-$dateTime" --location $location --template-file main.bicep --parameters "$region.main.bicepparam"
ukw.deploy.ps1
$dateTime = (Get-Date).ToString("yyyyMMddHHmmss")
$location = "ukwest"
$region = "ukw"
az deployment sub create --name "mainDeployment-$dateTime" --location $location ` --template-file main.bicep --parameters "$region.main.bicepparam"
Additional context
In module.main.bicep, if I change the name: 'satest123${region}' to name: 'satest456${region}' of resource storageAcccountUkw, the deployment works without issue. However, this messes my resource naming convention up. I can work around this issue using a single resource definition with the ternary operator (which is arguably a more elegant solution), but in some cases I will need to supply a value for the deployment in UK South (e.g. zones = [ '1' '2' '3' ]) and a null value in UK West (e.g. Azure Firewall), but not every resource provider necessarily supports the provision of a null value.

Possibly linked to #11290 ?

Closing as a duplicate of #11290.
The ARM engine is expected to raise a validation error when two resource declarations target the same resource ID due to the behavior you have observed. A bugfix that will restore the validation behavior is currently rolling out to all ARM regions.


NEW IDEA

Your project seems really impressive. However, I find the resource content displayed over the resources in the editor a bit visually cluttered and not very user-friendly. I have a small suggestion, and I'd be really satisfied if you, being competent as you are, could update it accordingly.
before

after

append idea is font size change and font color
is best!!!!

Your project seems really impressive. However, I find the resource content displayed over the resources in the editor a bit visually cluttered and not very user-friendly. I have a small suggestion, and I'd be really satisfied if you, being competent as you are, could update it accordingly.
before
after

What should happen here if the line contained multiple references to string resources?
They couldn't all be aligned to the left edge.
If immediately next to each other, how would they best be visually distinguished?
How should truncation work if some were very long?
How could it be made clear which reference in code represents which string?

append idea is font size change and font color is best!!!!

It currently uses the size and color defined within the editor for displaying strings.
I thought it best to try and make it look as much like it would if you typed the string directly in code.
What's the reason for wanting to change this?
How would different be better?
Do you want to make it bigger or smaller?

Let me share my thoughts.
What should we do if there are multiple references to string resources on a line?
I hadn't considered the case of multiple references, but if there's only one, left-aligning would make the source editor look beautiful.
If there is more than one resource, left alignment should be ignored, or there should be another expression method.
It would also look good to use multiple lines with left-aligned resources in the order of resources. Your way of expression is more beautiful than comments.
Not everything can be aligned to the left edge.
This condition is included in item 1.
When there is only one resource on the line, left alignment is used.
How can they be visually distinguished best if they are right next to each other?
Similar to the question when there is more than one resource in item 1.
Options like TextTrimming with "..." and tooltip expressions could be considered. However, it may differ slightly from the developer's intention.
It doesn't force the expression to be above the resource. Once you get used to it, you can proficiently use it in any way.
How should truncation work if some are very long?
Similar question to when there is more than one resource in item 1.
It would be similar to the TextTrimming option in item 3.
Using multiple lines would also solve this problem to some extent.
How can we clearly know which reference in the code represents which string?
Similar question to when there is more than one resource in item 1.
P.S.: It works in comments too, which I really, really like.
I'm using attributes for multilingual support. It's perfect for me as it's expressed in comments."
Feel free to ask if you need further clarification or assistance!

The size of your expression and the actual size seem a bit different.
I am attaching the image below.
This was captured at 100% zoom.


[FEATURE] Offline mode for launcher

It should be possible to use this launcher while being offline.
There should be a button to download all the patches and use the local storage

Technical it should be possible to launch patches which are already downloaded.

@LitzeYT Any idea where we could put the button to download all the patches?
This is required for the offline mode.


Too early validation when using MatAutocomplete and “updateOn: 'blur'”

Bug, feature request, or proposal:
Bug
What is the expected behavior?
When using autocompletion on a field together with updateOn: 'blur' on the FormControl / FormGroup, selecting an option from the autocompletion panel should neither trigger an update of the form nor a field validation. The update should only be triggered when the user moves on to the next field or submits the form.
What is the current behavior?
When clicking on an option in the autocompletion panel, the FormControl is updated with the old, incomplete value and the validation triggered. Then the new value is inserted into the field and the autocompletion panel is closed.
Depending on the validation, the field is now marked to be in error even though the shown value is valid. This is because the validation was run on the old, incomplete value. No update or validation should have occurred as the field was not left.

What are the steps to reproduce?
A simple StackBlitz example can be used to reproduce it. Enter "23" in the first text field and then select "ABC23" by clicking with the mouse.
What is the use-case or motivation for changing an existing behavior?
Technically, the behavior is ok because clicking in the autocompletion panel triggers a blur event. From a UI perspective however, it's complete nonsense: the user selects a valid value and is shown an error.
The text field and the associated autocompletion panel should be treated as a single element. Therefore, changing the focus between them should not be treated as a blur event and should not trigger a FormControl update or validation.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?

Angular 6.0.4
Angular Material 6..2.1
MacOS X 10.13.5
Google Chrome 67

Is there anything else we should know?

Still occurring in version 8.2.3, any chance of this PR making 8.2.4?

+1


ASIC footprint is wrong

The footprint of the ASIC needs to be changed. It is mirrored according to Tim Edwards.

Fixed!


Invitation link expired after credential was issued

For my<EMAIL_ADDRESS>invitation the link is expired even though I have my credential.
This may be a hold-over from the fact that my invite was made early in the development cycle when features were being coded and deplyed.

The people I invited last week in Nova Scotia have expired links. When you try to use them they get prompted to Request Token "your token has expired. To continue using please request a new one." and a field to enter an email address into.
1-  It's not clear what email address to enter
2 - there is no way to generate a refeshed token or make a new invitation for the user who's invitation has expired.

Changes that I will implement:

The API will return the assigned e-mail address and pre-populate that e-mail input with said email address.
Two notes here:
a. You should be able to refresh the token if they haven't consumed it. I'll rerun the tests for that today.
b. If the user has issued themselves a token it will currently expire and the behavior describe above will be accurate. I will adjust it so in the case the user has issued themselves a credential their link will never expire.

Corrected in: https://github.com/bcgov/identity-kit-poc/pull/85. Will update when live on pathfinder & verified.

@swcurran @watkinspd Verified that these are resolved in the pathfinder version. There is one bug still which is related to the other ticket. I'll reference that there.


V3 add key features and tags

Don't squash, in case we need to revert either features or tags.
Merge or merge with fast forward instead.
Ref: #1083

@kotp I've incorporated these into #1124


SEO - Updates to non secure url references and redirected links

Summary
Many references to Urls using http:// where https:// is available have been updated. Additionally, older links that now redirect to new pages have also been updated to their correct final destinations.
Ticket Link
https://mattermost.atlassian.net/browse/MAR-183

Hello @lawrencejohnson,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@justinegeffen Is there something specific here that you wanted me to review?

@amyblais, Lawrence mentioned pinging you about the changelog as part of this PR. Just wanted to make sure that wasn't affected by any changes.


Irrelevant TAS topic

The TAS topic is for tool-assisted speedrunning, not transient absorption spectroscopy. This tag should be removed from the repository.

Thanks for your comment. Each community uses their own acronyms and overlap is impossible to avoid. But TAS is not an essential one here, so the tag has been removed.


Add initial tools for managing derivations and React compatibility

Initial tools for using dataverse derivations in more places to enable fewer re-renders.
Foundation for #197

Very cool PR
+1 to a rename of derivedChild. deriveEl seems okay (although, does it work with any element or only Styled ones? Maybe it should be called deriveStyledEl? Or styledElFromDerivedProps?)

I like the changes to dataverse too

Re-requested Aria for the review of dataverse changes.

@colelawrence couple of unresolved comments remain. Feel free to merge after addressing them.


Debugging python code inside Carla in VS Code?

I'm using VSCode (in Ubuntu) to debug python code in this plugin. In some cases, I would like to debug the carla's python code (not in scenario runner's) by setting breakpoints etc. I'm trying but cannot find best way yet.
It would be helpful that someone to provide the example configuration of launch.json, settings.json etc..

Hi Kino, any chance you can share your exact changes to the json file, I suspect I'm missing some important step.


32-bit support is busted

A few guards against overflow in 32-bit numbers inadvertently broke 32-bit support, which is pretty embarrassing.

I believe this to be fixed in commit be2fa6 but I'll hold off on closing this until it's released in v0.0.9 .

Version <IP_ADDRESS> builds and its tests pass on i686.


Enhancements:Add a show/hide password symbol on the password fields

From tenfour-archive created by Eve-wanderi: ushahidi/tenfour-archive#1074
Expected behaviour
Can we also add a show/hide password symbol on the password fields so that its easier for users to confirm what passwords they typed in.
Actual behaviour
Currently its very hard to know if my passwords match or if am typing the correct password.

Design decision.


Adaptation to the new parameters returned by the MAPS API

Description
The tile format is returned in the formatTiles URL  tile param instead of format
Type of change

Feature

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/react-api/src/hooks/useTileFeatures.js
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1166

Relevant Lines:
1564

💛  - Coveralls


KAFKA-3194: Validate security.inter.broker.protocol against the adver…

…tised.listeners protocols

Should we also be validating consistency between listeners and advertised.listeners security protocols?

@ijuma Good point, we should probably make sure advertised.listeners is a subset of listeners protocols.

Thanks for these improvements, it's really helpful for users to have informative fail-fast behaviour if things are misconfigured.

@ijuma Updated the PR. Thanks for the review.

One minor comment, the rest looks good to me.

@ijuma My bad. Fixed now.

All tests pass locally.

@junrao what do you think of this?

@junrao @ewencp @gwenshap @guozhangwang
Would any of you have a chance to review this today?

LGTM.


Pdf/Txt/Csv download toolbar buttons - add features so that these don't get skipped

Since #10942, all the toolbar buttons need an RBAC feature with the same name, or an explicit skip, otherwise they never get shown. Related to https://github.com/ManageIQ/manageiq/issues/11872 (but doesn't close, it's more than just the download buttons).
Drift:
Added the features under "Display Instances Drift", and updated pdf button id for consistency.
Compare:
Updated button ids to match the existing features.
Timelines:
Added features under "Timelines"

Euwe Backport details:
$ git log
commit 4692f78a9878a731a5b6e67ec276b9921c55ed68
Author: Milan Zázrivec<EMAIL_ADDRESS>Date:   Thu Oct 13 13:44:41 2016 +0200

    Merge pull request #11873 from himdel/toolbar-rbac-fix

    Pdf/Txt/Csv download toolbar buttons - add features so that these don't get skipped
    (cherry picked from commit 647ea6a80aeef2690c97775b5d6491544550855f)


WW-5385 Decompose StrutsVelocityContext for better re-usability

WW-5385
Decomposes StrutsVelocityContext into:

CompositeContext - combines multiple contexts into 1
ChainedVelocityContext - corrects #getKeys to also return keys of chained context
DirectiveVelocityContext - is additionally capable of rendering Struts directives
StrutsVelocityContext - is additionally capable of looking up items in the ValueStack

The DirectiveVelocityContext specifically, enables applications greater flexibility in which capabilities they wish to leverage. ChainedVelocityContext and CompositeContext are useful utility contexts that can be used for general Velocity purposes.

SonarCloud note: There are indeed 6 levels of inheritance here but 3 of them are within the Velocity library - plus each of our context classes have a distinct purpose/use


Click on the label to toggle (parent) or select the element

Is there a way to toggle a parent element or select an element by clicking on the title instead of "X" or ">" to toggle a parent element ?
Thanks

By toggle you mean toggle the open/closed state or toggle the
checked/unchecked state? In any case, it's not possible with the current
code as-is, but the patch to allow that is a single line modification. Let
me see if I can do it this afternoon.
El El jue, 24 mar 2016 a las 3:34, JihadMotii-REISys <
<EMAIL_ADDRESS>escribió:

Is there a way to toggle a parent element or select an element by clicking
on the title instead of "X" or ">" to toggle a parent element ?
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/alexandernst/angular-multi-select/issues/84

Done, please have a look at http://alexandernst.github.io/angular-multi-select/#/mods

@alexandernst This works great ! but I noticed that I can't use them both when I include them ?

One of the 2 available mods makes a click on the item's label open the
item, while the other one makes the item being clicked to toggle the
un/checked state. How would you expect them both to work at the same time?
El El jue, 24 mar 2016 a las 16:43, JihadMotii-REISys <
<EMAIL_ADDRESS>escribió:

@alexandernst https://github.com/alexandernst This works great ! but I
noticed that I can't use them both when I include them ?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/alexandernst/angular-multi-select/issues/84#issuecomment-200893562

I was thinking for nodes, the toggle state should be applied and for leaf, the check/uncheck state should be applied. What do you think ?

Ah, that indeed sounds reasonable. Let me write another mod.
El El jue, 24 mar 2016 a las 16:53, JihadMotii-REISys <
<EMAIL_ADDRESS>escribió:

I was thinking for nodes, the toggle state should be applied and for leaf,
the check/uncheck state should be applied. What do you think ?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/alexandernst/angular-multi-select/issues/84#issuecomment-200898670

Now you have a new mod in 6.3.1 which should do exactly what you want.

:+1: