runtime dependency is erroneously also added as a testCompile dependency

Given the following build file:
apply plugin: 'java'
apply plugin: 'idea'

repositories {
    jcenter()
}

dependencies {
    runtime 'org.slf4j:slf4j-api:1.7.21'
}

When I run:
gradle idea

The resulting project should contain a scope="RUNTIME" dependency for slf4j-api.
What I actually get is:
    <orderEntry type="module-library" scope="RUNTIME">
      <library>
        <CLASSES>
          <root url="jar:///Users/daniel/.gradle/caches/modules-2/files-2.1/org.slf4j/slf4j-api/1.7.21/139535a69a4239db087de9bab0bee568bf8e0b70/slf4j-api-1.7.21.jar!/"/>
        </CLASSES>
        <JAVADOC/>
        <SOURCES>
          <root url="jar:///Users/daniel/.gradle/caches/modules-2/files-2.1/org.slf4j/slf4j-api/1.7.21/f285ac123f201fb4b028bac556928d7cf527ef48/slf4j-api-1.7.21-sources.jar!/"/>
        </SOURCES>
      </library>
    </orderEntry>
    <orderEntry type="module-library" scope="TEST">
      <library>
        <CLASSES>
          <root url="jar:///Users/daniel/.gradle/caches/modules-2/files-2.1/org.slf4j/slf4j-api/1.7.21/139535a69a4239db087de9bab0bee568bf8e0b70/slf4j-api-1.7.21.jar!/"/>
        </CLASSES>
        <JAVADOC/>
        <SOURCES>
          <root url="jar:///Users/daniel/.gradle/caches/modules-2/files-2.1/org.slf4j/slf4j-api/1.7.21/f285ac123f201fb4b028bac556928d7cf527ef48/slf4j-api-1.7.21-sources.jar!/"/>
        </SOURCES>
      </library>
    </orderEntry>

The second of these, the scope="TEST", should not be here, as I did not ask for it as a compile dependency.
I can reproduce this both under Gradle 3.4 (where I found it) and 3.4.1.

TEST contains both the test compile and test runtime dependencies in Idea, so there is bo way for us to distinguish the two. Any runtime dependency is transitively also a testRuntime dependency.
I'm closing this as working as intended, as this doesn't change idea's behavior. It's a purely cosmetic concern in the .iml file.

Not actually true, because it does change IDEA's behaviour.
In particular, transient dependencies which you only had in runtime are now allowed in compile-time for tests, allowing people to call classes they're not supposed to be calling. If you remove the extraneous "TEST" element, IDEA goes back to not allowing these classes to be used in the code.

Demonstration:

Create a file which uses a class which should not be on the compile classpath:

$ cat src/test/java/TestClass.java
import org.slf4j.Logger;

public class TestClass {
    Logger logger;
}

Run gradle idea (in case you didn't have the src/test/java dir before doing this)

Go to IDEA and hit compile - notice that compilation succeeds, when it should not.

Go back to the terminal. Hand-edit the .iml file to remove the "TEST" dependency.

Re-run gradle idea

Go to IDEA and hit compile - notice that compilation now (correctly) fails.

I encourage you to use IDEA's importer with the "Module per SourceSet" option instead of running gradle idea. Additionally, @vladsoroka is planning to bring this feature to Gradle's idea plugin, making you able to further customize how a single project is mapped to multiple IDEA modules.
We have no plans to make the mapping to a single IDEA module any more complex than it is now. The fact that runtime and compile are not separated for tests is not going to change in IDEA. Instead, the way forward is generating separate modules.

The problem at the moment doesn't seem to be entirely IDEA lacking a place to put "testRuntime" stuff, but rather Gradle putting dependencies into "test" which weren't in "test" or "testRuntime" in the first place. In the example build, I only put it into "compile", so really in the IDEA project, it should only be in "compile"...

I only put it into "compile", so really in the IDEA project, it should only be in "compile"...

That would require complex hardcoded addition/subtraction rules, which place a large maintenance burden on us. We've been there and it significantly slowed down any improvement to our IDEA plugin.
Please give the IDEA importer a spin, it should work just fine unless your build has some fancy .iml-postprocessing rules.


Errors and warnings when running make serve for the Polars book

When I run make serve locally for the Polars, I have the errors below.
They are mostly related to the lazy API and time-series section.
Is it 'normal' for these errors to happen?
If not, I would be happy to make a PR for it
2023-03-13 23:10:42 [INFO] (mdbook::book): Book building has started
2023-03-13 23:10:42 [ERROR] (mdbook::preprocess::links): Error updating "{{#include ../outputs/lazy_api/output4.txt}}", Could not read file for link {{#include ../outputs/lazy_api/output4.txt}} (/home/lc/projects/polars-book/user_guide/src/lazy-api/../outputs/lazy_api/output4.txt)
2023-03-13 23:10:42 [WARN] (mdbook::preprocess::links): Caused By: No such file or directory (os error 2)
2023-03-13 23:10:42 [ERROR] (mdbook::preprocess::links): Error updating "{{#include ../outputs/lazy_api/lazy_eager_query.txt}}", Could not read file for link {{#include ../outputs/lazy_api/lazy_eager_query.txt}} (/home/lc/projects/polars-book/user_guide/src/lazy-api/../outputs/lazy_api/lazy_eager_query.txt)
2023-03-13 23:10:42 [WARN] (mdbook::preprocess::links): Caused By: No such file or directory (os error 2)
2023-03-13 23:10:42 [ERROR] (mdbook::preprocess::links): Error updating "{{#include ../outputs/lazy_api/q1_plan.txt}}", Could not read file for link {{#include ../outputs/lazy_api/q1_plan.txt}} (/home/lc/projects/polars-book/user_guide/src/lazy-api/../outputs/lazy_api/q1_plan.txt)
2023-03-13 23:10:42 [WARN] (mdbook::preprocess::links): Caused By: No such file or directory (os error 2)
2023-03-13 23:10:42 [ERROR] (mdbook::preprocess::links): Error updating "{{#include ../outputs/lazy_api/q1_opt_plan.txt}}", Could not read file for link {{#include ../outputs/lazy_api/q1_opt_plan.txt}} (/home/lc/projects/polars-book/user_guide/src/lazy-api/../outputs/lazy_api/q1_opt_plan.txt)
2023-03-13 23:10:42 [WARN] (mdbook::preprocess::links): Caused By: No such file or directory (os error 2)
2023-03-13 23:10:42 [ERROR] (mdbook::preprocess::links): Error updating "{{#include ../outputs/lazy_api/df4.txt}}", Could not read file for link {{#include ../outputs/lazy_api/df4.txt}} (/home/lc/projects/polars-book/user_guide/src/lazy-api/../outputs/lazy_api/df4.txt)
2023-03-13 23:10:42 [WARN] (mdbook::preprocess::links): Caused By: No such file or directory (os error 2)
2023-03-13 23:10:42 [ERROR] (mdbook::preprocess::links): Error updating "{{#include ../outputs/lazy_api/q9.txt}}", Could not read file for link {{#include ../outputs/lazy_api/q9.txt}} (/home/lc/projects/polars-book/user_guide/src/lazy-api/../outputs/lazy_api/q9.txt)
2023-03-13 23:10:42 [WARN] (mdbook::preprocess::links): Caused By: No such file or directory (os error 2)
2023-03-13 23:10:42 [ERROR] (mdbook::preprocess::links): Error updating "{{#include ../outputs/lazy_api/dataset.txt}}", Could not read file for link {{#include ../outputs/lazy_api/dataset.txt}} (/home/lc/projects/polars-book/user_guide/src/lazy-api/../outputs/lazy_api/dataset.txt)
2023-03-13 23:10:42 [WARN] (mdbook::preprocess::links): Caused By: No such file or directory (os error 2)
2023-03-13 23:10:42 [ERROR] (mdbook::preprocess::links): Error updating "{{#include ../../examples/time_series/time_zones/mixed_offsets.py:03:09}}", Could not read file for link {{#include ../../examples/time_series/time_zones/mixed_offsets.py:03:09}} (/home/lc/projects/polars-book/user_guide/src/howcani/timeseries/../../examples/time_series/time_zones/mixed_offsets.py)
2023-03-13 23:10:42 [WARN] (mdbook::preprocess::links): Caused By: No such file or directory (os error 2)
2023-03-13 23:10:42 [ERROR] (mdbook::preprocess::links): Error updating "{{#include ../../outputs/time_series/time_zones/mixed_parsed.txt}}", Could not read file for link {{#include ../../outputs/time_series/time_zones/mixed_parsed.txt}} (/home/lc/projects/polars-book/user_guide/src/howcani/timeseries/../../outputs/time_series/time_zones/mixed_parsed.txt)

The "outputs" folder is apparently missing, and the output.txt files do not appear in the book any more.
Example here:
https://pola-rs.github.io/polars-book/user-guide/lazy-api/lazy-schema.html

Seems to be related to this issue https://github.com/pola-rs/polars-book/issues/277. Not sure if @braaannigan is working on a fix.

@r-brink  @bmaz I'm looking into this

the lazy API part is now fixed, however there are some warnings for NbConvert.
Not sure if anything should be done about this?
[NbConvertApp] Converting notebook ./user_guide/src/notebooks/introduction_polars-py.ipynb to python
/home/luca/projects/polars-book/.venv/lib/python3.10/site-packages/nbformat/__init__.py:93: MissingIDFieldWarning: Code cell is missing an id field, this will become a hard error in future nbformat versions. You may want to use `normalize()` on your notebooks before validations (available since nbformat 5.1.4). Previous versions of nbformat are fixing this issue transparently, and will stop doing so in the future.
  validate(nb)
[NbConvertApp] Writing 5914 bytes to user_guide/src/notebooks/introduction_polars-py.py
[NbConvertApp] Converting notebook ./user_guide/src/notebooks/introduction_polars-py.ipynb to markdown
/home/luca/projects/polars-book/.venv/lib/python3.10/site-packages/nbformat/__init__.py:93: MissingIDFieldWarning: Code cell is missing an id field, this will become a hard error in future nbformat versions. You may want to use `normalize()` on your notebooks before validations (available since nbformat 5.1.4). Previous versions of nbformat are fixing this issue transparently, and will stop doing so in the future.


Changed default deprecation reason

Currently GraphiQL console doesn't support fields with "" (empty string) deprecation reason. I'd like to have an ability to annotate a field as @Deprecated to achieve a deprecation in GraphiQL.

Thanks for the contribution!
Sorry I didn't react sooner!

I extended this in #133 to easily allow a custom reason.


Speed up scripted tests

Recycle the compiler bridge and the scala instance (with its parent
classloader) for every scripted test that has the same options (the
scala version). This way, we avoid resolving the bridge and classloading
it every time per scripted test.

https://github.com/sbt/zinc/pull/315/commits/c2cbf0da36eaf2f622269a5cdc0b5a664f862367 alone makes the build more than twice faster.
Before: [error] Total time: 533 s, completed Jun 13, 2017 2:21:06 PM.
After: [error] Total time: 533 s, completed Jun 13, 2017 2:21:06 PM.

This is so frustrating. For some reason, Travis is not running the tests...

Summary of changes:
The following commits make the CI 7x faster than before. This is achieved
through two main improvements:

Scripted tests are not resolving the compiler and class-loading all the jars
per test (we only do it once per scripted invocation).
It uses Drone instead of Travis, which does not impose any constrain on
resources. Tests are run in a machine with 48 cores and 228GB of memory.

Parallelization at the build level has been removed because modules are too
intertwined with each other, and would require compilation of almost all the
modules before proceeding to test things. Therefore, .drone.yml runs everything
together.

Closing and opening again to update status of PR, that should be passing.

Merging, travis hook has been disabled.


Not adding items in inventory

Hello, great plugin, I am having some trouble could be my own fault but if not might be a bug.  Items are not being added to inventory, heres a copy of my config file.  I tried it with and without having all the slots listed.  i checked it in yaml validator and it says its ok on my side.
)
(           )   ) (       ( /(       )
( )\     ) ( /(( /( )\  (   )())(  ( /(
)(() ( /( )())()|()))|((_)\ )\ )()|
(()  )()|))())/ _ /(()_ ((|(|_))/)\
| _ )(()| || | | ()) | |/ / () |(()
| _ / _` |  _|  || / -)  ' <  | |  (-<
|/_,|_|_||_| |_\ ||__//

An clean port of BattleKits to PocketMine-MP
once-per-life: true
# Whether or not players can only use kits once per life
list-format: "%s: %s%s"
# Kit display format when listing kits
# If you don't understand it don't change it
show-kit-info-on-respawn: true
# Whether or not kit information is shown on respawn
# NOT implemented yet
econ-prefix: $
# Economy prefix for price display
sign-trigger: "[BattleKits]"
# Signs will be treated as kit signs if this is on the first line
# Changing it may cause some issues
preferred-economy: false
# If you have multiple economies set the one you want here, set to false to just auto-detect
# You can set an economy of your own provided that class loads before BattleKits
# battlekits\economy\Economy and battlekits\economy\PocketMoney are built in
kits:
leather:
commands:  # Executed on kit give
- 'tell {player} Enjoy your kit :)'
items:
helmet: '298'
chestplate: '299'
leggings: '300'
boots: '301'
'0': ‘272'
'1': ‘0’
‘2’: ‘0’
'3': ‘0’
'4': ‘0’
'5': ‘0’
'6': ‘0’
'7': ‘0’
'8': ‘0’
'9': '0'
'10': '0'
'11': '0'
'12': '0'
'13': '0'
'14': '0'
'15': '0'
'16': ‘0’
'17': ‘0’
'18': '0'
'19': '0'
'20': '0'
'21': '0'
'22': '0'
'23': '0'
'24': '0'
'25': '0'
'26': '0'
'27': ‘0’
'28': ‘0’
'29': ‘0’
'30': ‘0’
'31': ‘0’
'32': ‘0’
'33': ‘0’
'34': ‘0’
'35': ‘0’
chainmail:
commands:  # Executed on kit give
- 'tell {player} Enjoy your kit :)'
items:
helmet: '302'
chestplate: '303'
leggings: '304'
boots: '305'
'0': ‘272’
'1': ‘278’
‘2’: ‘0’
'3': ‘0’
'4': ‘0’
'5': ‘0’
'6': ‘0’
'7': ‘0’
'8': ‘0’
'9': '0'
'10': '0'
'11': '0'
'12': '0'
'13': '0'
'14': '0'
'15': '0'
'16': ‘0’
'17': ‘0’
'18': '0'
'19': '0'
'20': '0'
'21': '0'
'22': '0'
'23': '0'
'24': '0'
'25': '0'
'26': '0'
'27': ‘0’
'28': ‘0’
'29': ‘0’
'30': ‘0’
'31': ‘0’
'32': ‘0’
'33': ‘0’
'34': ‘0’
'35': ‘0’

That copy and paste showed up weird, everything was spaced right before i submitted the post.

Maybe try pastebin?

http://pastebin.com/raw.php?i=zF7pr4ZH

http://pastebin.com/raw.php?i=fRD7JcJ6

I also forgot to add i am using the bleeding build 48

Im having the same problem
Cazador:
active-in: #Remove this setting for all worlds
- 'PvP'
cost: 50
commands:  # Executed on kit give
- 'effect {player} 5 240 0 0'
on-give-message: "§cCompraste el kit §6Cazador" #Message displayed when kit is delivered
items:
  helmet: '298'
  chestplate: '299'
  leggings: '300'
  boots: '301'
  '0': '261'
  '1': '297:2'
  '2': '262:16'


http::Uri instead of url::Url

Crate version: 0.5
OS: macOS Mojave (10.14.5)
Output of running docker version on the command line:

Client: Docker Engine - Community
 Version:           18.09.2
 API version:       1.39
 Go version:        go1.10.8
 Git commit:        6247962
 Built:             Sun Feb 10 04:12:39 2019
 OS/Arch:           darwin/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          18.09.2
  API version:      1.39 (minimum version 1.12)
  Go version:       go1.10.6
  Git commit:       6247962
  Built:            Sun Feb 10 04:13:06 2019
  OS/Arch:          linux/amd64
  Experimental:     false

Instead of
use url::Url;

let docker = Docker::host(Url::parse("http://yourhost").unwrap());

I think one should do
use http::Uri;

let docker = Docker::host("http://yourhost".parse::<Uri>().unwrap());

I think that the url is a much smaller library and should be a perfect match here because it is only required to parse an URL. The http provides much more type types that are not required for the use cases of shiplift.

I'm confused now. I opened the issue because for me Url wasn't working and I had errors saying I should provide a Uri. Am I missing something?

Ah, okay, I messed up the issue description and I thought you suggested to use http::Uri instead url::Url. I agree that url::Url is the much better choice.

@schrieveslaach http is a transitive dependency of hyper anyway, no?

Yes, hyper relies on http

What I mean is, it doesn't matter whether it has extra types or functionality you don't need, since it's statically linked already, so no matter how lightweight Url is, it's duplicated functionality. Is it more ergonomic for parsing url strings in some way than the http types?

I think the issue here is that the http crate does not include any URL type. It include only an URI type. URIs are the super type of URL (see here) and in the desribed use case only URLs are valid input parameters. Therefore, IMHO shiplift should express that restriction through good parameter types.

My original issue is basically that if I try and do what the README says
use url::Url;

let docker = Docker::host(Url::parse("http://yourhost").unwrap());

I get expected struct http::uri::Uri, found struct url::Url

My original issue is basically that if I try and do what the README says
use url::Url;

let docker = Docker::host(Url::parse("http://yourhost").unwrap());

I get expected struct http::uri::Uri, found struct url::Url

I've filed a PR that fixes this documentation error: https://github.com/softprops/shiplift/pull/207


How to all-in?

I'm trying to use the library to build a game tree. However, I do not know how to apply all-in action to a state. Do I need to code this function my self?

I wrote my own function as follow:

def apply_action(self, action):
        match action:
            case "fold":
                self.state.fold()
            case "all_in":
                self.state.complete_bet_or_raise_to(self.state.stacks[self.state.actor_index] + self.state.bets[self.state.actor_index])

And run it with:
a = EnhancedState()
a.state.deal_hole('AsAd')
a.state.deal_hole('KsKd')
a.state.complete_bet_or_raise_to(200)
a.state.complete_bet_or_raise_to(500)
a.apply_action('all_in')
a.apply_action('all_in')
print(a.state.all_in_status)

I got this error message:
Traceback (most recent call last):
  File "/home/huy/my_poker/enhanced_state.py", line 90, in <module>
    a.apply_action('all_in')
  File "/home/huy/my_poker/enhanced_state.py", line 45, in apply_action
    self.state.complete_bet_or_raise_to(self.state.stacks[self.state.actor_index] + self.state.bets[self.state.actor_index])
  File "/home/huy/miniconda3/envs/poker_kit/lib/python3.11/site-packages/pokerkit/state.py", line 4246, in complete_bet_or_raise_to
    amount = self.verify_completion_betting_or_raising_to(amount)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/huy/miniconda3/envs/poker_kit/lib/python3.11/site-packages/pokerkit/state.py", line 4094, in verify_completion_betting_or_raising_to
    self._verify_completion_betting_or_raising()
  File "/home/huy/miniconda3/envs/poker_kit/lib/python3.11/site-packages/pokerkit/state.py", line 4070, in _verify_completion_betting_or_raising
    raise ValueError('There are not enough chips in stack.')
ValueError: There are not enough chips in stack.

So I can not make two people both all-in using the code above.

Your way of handling all-in in 'apply_action' is correct. A more elegant way is by doing self.state.complete_bet_or_raise_to(self.state.max_completion_betting_or_raising_to_amount), but your method is correct too.
def apply_action(self, action):
        match action:
            case "fold":
                self.state.fold()
            case "all_in":
                self.state.complete_bet_or_raise_to(self.state.max_completion_betting_or_raising_to_amount)

The problem, however, is in your second code snippet: the latter call of a.apply_action('all_in'). What happened just before this call is that the former player raised all-in. Since the latter actor's stack is covered by the former actor, the latter actor cannot bet or raise since he/she is already covered by the former's all-in. Therefore the latter actor can only fold or call. Part of the problem is that the PokerKit's error message was not clear enough, that is my mistake.
There are two ways you can fix this:

Instead of all-inning, simply call (or fold, but this will not all-in).

a = EnhancedState()
a.state.deal_hole('AsAd')
a.state.deal_hole('KsKd')
a.state.complete_bet_or_raise_to(200)
a.state.complete_bet_or_raise_to(500)
a.apply_action('all_in')
# a.apply_action('all_in')
a.state.check_or_call()  # ``a.state.fold()`` is possible too, but this will fold, not all-in
print(a.state.all_in_status)

Make all-in also mean call "all-in", not just betting/raising.

def apply_action(self, action):
        match action:
            case "fold":
                self.state.fold()
            case "all_in":
                if self.state.can_complete_bet_or_raise_to():
                    self.state.complete_bet_or_raise_to(self.state.max_completion_betting_or_raising_to_amount)
                else:
                    self.state.check_or_call()


Feature/db schema-diagram

Medical tables
We @ClarenceAtim AND @RileyManda  have completed the implementation of  database schema according to the stipulated requirements : 🌱

[x] Created patient table
[x] Created medical history table
[x] Created indexes and foreign key medicalhistories and  history table

Invoices|Invoice-Items|Treatments

[x] Created invoices table
[x] Created invoice_items table
[x] Created invoice_items and invoice indexes
[x] Created indexes and foreign keys

Please kindly review our implementation and suggest any improvements and changes.
Thank you 🙏🏻

requested changes:

[x] Implemented index on invoices
[x] Implemented(optional


Define cluster/node quota per user/group

What kind of request is this (question/bug/enhancement/feature request):
Enhancement / Feature request
Feature description
Allow rancher administrators to template and restrict the amount of super-resources allocated per group/user.
Set cluster count quotas per user/group. Set node type quota (per cloud provisioner) per user/group.
Allow only a subset of cloud provisioners (when creating clusters) to a user/group.
User stories

A school administrator wants every student to allow 1 personal cluster, and 3 clusters per group (for test/staging/prod arrangements), in specific resource quotas due to budget constraints.

A department manager has budget leftover enough to allow every user a personal (restricted) cluster on-prem to allow for personal projects/experiments.

A communal group (that allocates learning resources upon request) wants to not make a repeat of abuse of it's system, by locking users to only 1 cluster with 3 nodes of a specific type, but only if those users belong to a specific group that's allowed to access these resources.

@SheilaghM


Proposal: Perun Channels - Integration with go-perun

Project Abstract
This proposal is a follow-up on our first proposal, which resulted in a Substrate Pallet that implements the Perun smart contract logic for Polkadot.
We now propose to integrate the Perun-Polkadot on-chain logic with the go-perun library.
The go-perun library implements the Perun client protocols and provides an abstract, simple, and secure interface for working with Perun Channels on any supported blockchain backend.
Through the integration, Perun Channels on Polkadot will become accessible.
For which grant level are you applying?

[ ] Level 1:  Up to $10,000, 2 approvals
[x] Level 2:  Up to $50,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for > $100k Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied, renamed ( project_name.md) and updated.
[x] A BTC or Ethereum (DAI/USDT) address for the payment of the milestones is provided inside the application.
[x] I have read and acknowledged the terms and conditions.
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted.

 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.

@Noc2 Thanks for the feedback. We included two additional milestones based on it. I have added them in separate commits and will squash once approved. Note that adding the CLI Demo bumped the grant from level 2 to level 3.


Allow hooks to have knowledge of internal functions + prevent infinite loop

As of the general discussion of 2023-01-27, we decided the following:

Force all hooks executions inside internal functions
Expose all those functions to our plugins

As such, as an example, our dependency keys plugin would directly call deleteDataCacheRecord instead of calling the cache adapter directly.

Resolved in v1.5.0


Using sign_psbt causes the Jade to crash

Hi,
I'm using the sign_psbt endpoint but it's causing the Jade to crash. We're running 2of3 multisig with the config registered successfully on the Jades. Here is the function call for the sign_psbt message. The experience is that the Jade immediately turns off and reboots, landing in a locked state.
export const signPsbt = async (port: SerialPort, worker: Worker, network: string, psbt: string): Promise<string | void> => {
  const sign_psbt: Message = {
    id: '6979',
    method: 'sign_psbt',
    params: {
      network,
      psbt: bytesFromBase64(psbt),
    },
  }

  console.log("Message:", psbt, sign_psbt)

  try {
    const response = await worker.handleMessage(encode(sign_psbt), port)
    if (isResponse(response) && typeof response.result === 'string') {
      const psbt: string = response.result
      return psbt
    }
  } catch (error) {
    // removed for brevity
  }
}

Here is what is logged
Message: cHNidP8BAFIBAAAAAS3QiWM1nkpB44dFLIRsOF9jIKPIqxW9Ab+u+7Vwf5LjAAAAAAD9////AVUDAAAAAAAAFgAUh3B3EwIHRWJoDaDd6nYt5Rnxh40ewyUAAAEA6gIAAAAAAQEgN48SVsx6Ln8Xqe7yTBFPV3o1AHUPqIXa8Gh0dQmohQAAAAAA/f///wLoAwAAAAAAACIAIHT7AGgVUhB15sA2hfE+6afcRjqf2Rp3XantE2ssT6eP84iIAAAAAAAWABTieTDf+jlafFb3+5h31A/8xsSZiAJHMEQCIHwhjjxZY4658Y1Wr5+BxAUA2VjHtcqUjBzad7UTcleLAiBXbF/0RGvm9wbvCe7gGHarq6OZaNsMBwcjFSVTFwFv6AEhAjmJb1ndX4fkN8169T77XRvwzBu2Fx7LQ6Z4IXunoGYoDsMlAAEBK+gDAAAAAAAAIgAgdPsAaBVSEHXmwDaF8T7pp9xGOp/ZGnddqe0TayxPp48BBWlSIQM3l5f8CleBlOSmEX6hPYWcJ6pPI4TlM9ECaIYGc8bAOCEDYDnMnL7dp/3Hfsx+cdsugdqGvoTmYfNfGrTPiXj1d38hAxAM9uz362wLVsGyUyqwPxKzLMk1AF96ClKOz2AIRMAAU64iBgMQDPbs9+tsC1bBslMqsD8SsyzJNQBfegpSjs9gCETAABwQTkzqMAAAgAEAAIAAAACAAgAAgAAAAAAAAAAAIgYDN5eX/ApXgZTkphF+oT2FnCeqTyOE5TPRAmiGBnPGwDgMdzesuAAAAAAAAAAAIgYDYDnMnL7dp/3Hfsx+cdsugdqGvoTmYfNfGrTPiXj1d38cJMRrejAAAIABAACAAAAAgAIAAIAAAAAAAAAAAAAA
{
    "id": "6979",
    "method": "sign_psbt",
    "params": {
        "network": "testnet",
        "psbt":  ArrayBuffer(660)
    }
}

Here are the two attempted translations from base64 to bytes for the message
const bytesFromBase64 = (base64String: string) => {
   return Uint8Array.from(atob(base64String), c => c.charCodeAt(0))
 }

function bytesFromBase64(base64: string) {
  var binary_string = window.atob(base64);
  var len = binary_string.length;
  var bytes = new Uint8Array(len);
  for (var i = 0; i < len; i++) {
      bytes[i] = binary_string.charCodeAt(i);
  }
  return bytes.buffer;
}

Here is the Jade mnemonic
1. heart
2. wink
3. unable
4. cinnamon
5. fetch
6. crystal
7. hedgehog
8. misery
9. labor
10. youth
11. gift
12. dad

Here is the message used to register the multisig
{
   "id":"0",
   "method":"register_multisig",
   "params":{
      "network":"testnet",
      "multisig_name":"name_goes_here",
      "descriptor":{
         "variant":"wsh(multi(k))",
         "sorted":false,
         "threshold":2,
         "signers":[
            {
               "fingerprint":{
                  "0":119,
                  "1":55,
                  "2":172,
                  "3":184
               },
               "derivation":[

               ],
               "xpub":"tpubD6NzVbkrYhZ4X3vCdevm3aFUdeSAWsbR4UhhdApbXnVUnvuwtq5NYqQ7zCkNhYG4JcPdGW4Wg4ZrVSYGpdoWsn337qmkP6CFgs7pZid12Sa",
               "path":[

               ]
            },
            {
               "fingerprint":{
                  "0":36,
                  "1":196,
                  "2":107,
                  "3":122
               },
               "derivation":[
<PHONE_NUMBER>,
<PHONE_NUMBER>,
<PHONE_NUMBER>,
<PHONE_NUMBER>
               ],
               "xpub":"tpubDFPPqz7V4y4fujT46Bq1iSgN4auFmJ38aepkC1ajdtjmm9LFGqwGGpN4uFvJ1vRhzGw3qUtnww7VJc5iGSBhuq4zkhUQVytdaeBrS49bN6h",
               "path":[

               ]
            },
            {
               "fingerprint":{
                  "0":16,
                  "1":78,
                  "2":76,
                  "3":234
               },
               "derivation":[
<PHONE_NUMBER>,
<PHONE_NUMBER>,
<PHONE_NUMBER>,
<PHONE_NUMBER>
               ],
               "xpub":"tpubDFbQzqXwjoWLZztv3B9oTAsXHzKwtkhUHGpGmUgwo58bzzF1pCCgUZsAjAb3ULAmGXXCEjhrSAZYV6mZrMDEDqpC9zcAeFxAV4KaN1wDgNL",
               "path":[

               ]
            }
         ]
      }
   }
}

Here is the Multisig registration logged from get_registered_multisigs
{
   "name_goes_here":{
      "variant":"wsh(multi(k))",
      "sorted":false,
      "threshold":2,
      "num_signers":3,
      "master_blinding_key":{

      }
   }
}

Hoping this is due to a failed conversion from base64, any help is much appreciated.

Thanks for all this info - super helpful.
Can reproduce - looks like an issue our side.  ;-(
FYI/notes:

multisig registration looks good/correct - standard unsorted-multi wsh 2of3.
psbt looks good/standard, nothing unusual here - one input needing signing by this multisig, one output.
if I set jade wallet to some other unrelated seed, the call works and returns the PSBT unchanged  (there was nothing for my unrelated signer to do).  But you can see it 'works' on jade etc, so all b64/psbt encoding should be fine.
if I DO NOT register the multisig wallet (or delete it) on jade, the sign psbt call works perfectly with your signer, and returns the psbt updated with that signer's signature.
[ NOTE: jade does not need the multisig registered to sign atm, just to produce/validate receive and change addresses ]

So I think we (jade) have an issue when trying to match input/output utxos to potentially relevant multisig records.
But def our side - I will investigate as a top priority.

Thanks also for the detailed response! Appreciate you investigating it for us and marking it as top priority. I'll roll with unregistered signing for the moment and update when resolved.

@JamieDriver I am wondering whether the empty array for the derivation path of one of the signers is the culprit
        "derivation":[]

I believe I have this fixed locally - just need to get a second set of eyes (PR, review etc.) and it'll get merged asap.

Merged to master.  Will be included in next fw version.  Will leave item open until tested/agreed resolved.

Should be resolved in fw 1.0.23 - please close if you agree.
Thanks.

@JamieDriver can be closed, thanks!


[FEAT] Force cache refresh for a given method

Information
Is your feature request related to a problem? Please describe.
The platform cache let developer to cache method from a Service or Controller. But sometimes we need to refresh the cached data because the consumed data from the backend has changed (the backend have been called a notification endpoint to tell our service that a change has occurred).
Describe the solution you'd like
A new method on PlatformCache will allow to force the refresh cache data:
import {Injectable} from "@tsed/di";
import {PlatformCache} from "@tsed/platform-cache";

@Injectable()
export class NotificationService {
  @Inject()
  protected cache: PlatformCache;

  @Inject()
  protected service: CachedMethods<MyService>;

  refresh(id: string) {
    return this.cache.refresh(() => this.service.get(id))
  }
}

Acceptance criteria

[ ] PlatformCache.refresh refresh the cache data if the data exists!

:tada: This issue has been resolved in version 7.9.0 :tada:
The release is available on:

v7.9.0
GitHub release

Your semantic-release bot :package::rocket:


Auto attach affects whether the debug console shows up when I start debugging

From https://github.com/microsoft/vscode/issues/108202

Auto attach -> smart
Run "Launch VS Code"
Debug console does not appear
Disable auto attach
Start debugging again
The debug console does appear

I'm guessing that auto attach is attaching to the prelaunch script (which is as designed in 'smart' mode, since it's a normal .js script in the workspace.) In auto attach we use internalConsoleOptions: 'neverOpen' to not reveal the console, since that would be quite annoying when using the terminal. There may be some overlap between the attach in the prelaunch script exiting and the debug session starting, and I'm guessing that the new session doesn't then also open the terminal.
I don't think this is something tweakable in js-debug, so deassigning myself, but let me know if there's anything for me to do here

In auto attach we use internalConsoleOptions: 'neverOpen' to not reveal the console

Oh, that makes perfect sense then.


Adding Swift Example Project

This PR adds an example Swift project using VCSimpleSession.  This project attempts to match the Objective-C sample project as closely as possible, importing VCSimpleSession.h through a Bridging Header.  This was built using Xcode Version 6.3 (6D570) targeting the iOS 8.0 SDK.
I'm still studying Swift style best practices, so let me know if you want to make any modifications to the code style or formatting.

@jgh- Please let me know if you would like me to update the .swift files with the VideoCore license you use in the Objective-C sample project.

Ah yeah, I noticed that after - I'll update them, no worries :)

Thank you!


Any example for flash-decoding example to show?

Hi friends:
I am wondering whether there is some flash-decoding end2end example to show? Like llama demo?

Hi, have you made it to test this on llama?


Bottom navigation example for Gallery app

From https://www.google.com/design/spec/components/bottom-navigation.html#
aka "bottom bar"

i think we have this now.


svg support

please can you add svg support to the library

As there's no native support for SVGs in Android I can't add support for it. If you are talking about vector drawables please look at https://github.com/hdodenhof/CircleImageView/issues/167.


Create button ease of use

Will make the create button a bit easier to use.

When the Create button is clicked, the naming field gets auto-focus.
If the Create button is used in a reference field, the PrefixLabel will no longer clog the inspector.
The confirm create and deny create buttons also get triggered when return (enter) or escape is pressed respectively.

Sorry, I was browsing issues and I didn't realize linking them left behind a trail of breadcrumbs. Won't happen again!

Sorry, I was browsing issues and I didn't realize linking them left behind a trail of breadcrumbs. Won't happen again!

I actually found a better way, but it is so different that we might have to save it for V5.
https://user-images.githubusercontent.com/27729987/104252748-12e67f00-5473-11eb-8b8c-dd9fc10f4746.mp4
We can skimp the whole drawer down to 60 lines and remove a lot of complexity from it.

I actually found a better way, but it is so different that we might have to save it for V5.
https://user-images.githubusercontent.com/27729987/104252748-12e67f00-5473-11eb-8b8c-dd9fc10f4746.mp4
We can skimp the whole drawer down to 60 lines and remove a lot of complexity from it.


Inky Impression Image

Image of the inky Impression running.
Colors need to be reversed, but that's on me.
If you only need icons this should work fine

(cc: @Hothomir)

That came out much better than I expected, great!

Awesome! I'm trying to experiment with the correct colors so it looks good as well. I'll keep you posted!

Please see #17 for further discussion.


scripts/motdgen: Remove outdated LTS motd information

The LTS is now a regular public channel and does not need special docs
for credential configuration. The only thing left that needs attention
is the possibility to freeze the LTS stream to opt-out of automatic
major version updates.
How to use
Backport to all channels, which may need to use a backport branch in this repo: flatcar-3033-backport, flatcar-3139 and the ones for 3227 needs to be created.
Testing done
none

[ ] Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
↑ todo in coreos-overlay

Thanks, yes


Added Laravel 6.x compatibility

added Laravel 6.x compatibility
added .gitignore
set minimum Laravel version to 5.5
set minimum PHP version to 7.0
set PHPUnit version to 6.*
require ext-json
fixed PSR2 errors
updated install doc

Is this package no longer maintained?

@Holiehandgrenade ?


Change - Remove deprecated force parameter on concat resources

The $force parameter to concat is deprecated and has no effect.  This commit silences warning messages in the puppet server logs.  For example:

2017-02-13 08:32:21,798 WARN  [qtp1532802229-36820] [puppetserver] Scope(Concat[/var/lib/pgsql/data/pg_ident.conf]) The $force parameter to concat is deprecated and has no effect.

Doesn't change current functionality since the param has no affect within the concat module. All this will affect are logs. LGTM 👍


Publish ReactMDL to CDN?

Would you consider removing the out folder from the .gitignore file and publishing

ReactMDL.js
ReactMDL.js.map
ReactMDL.min.js
ReactMDL.min.js.map

to Git?
I know it is somewhat against best practices to do so, but it makes it very easy to load React-MDL from RawGit CDN, like this:
https://cdn.rawgit.com/download/react-mdl/v1.4.1-bin/out/ReactMDL.min.js
I made a fork of your repo and did so myself. I am willing to maintain it myself, but I wonder what your opinion is on this.
Thank you for your awesome work on this library!

You can use npmcdn to get the same result. Here's the latest version for example<EMAIL_ADDRESS>

Ah! That's great! Thanks!

Here is a proof of concept
There is more than just adding the npm cdn ;-)
you will also have to add the custom material.js and material.css
http://codepen.io/darkyen/pen/VaxPjg?editors=0010

Each demo on the website has a link to codepen if needed. (But I just noticed it's currently broken, I have to investigate why)

@tleunen it works with @1.4.2 but not the latest.


Remove boost-locale

It turns out that we've not actually used it for years, and it's been referenced by our CMake for no reason.
Redo of https://github.com/OpenMW/openmw-deps-build/pull/14, but now it's coming from my fork so shouldn't push a bunch of stuff to GitLab.

It's still showing some of the doomed runs, but the important pull_request ones should succeed this time.

The important jobs did indeed succeed.


Various issues arised from #111 related to creating an account on signup

It looks like the createAccount was designed to work without a registrar.  https://github.com/cryptonomex/graphene-ui/blob/1e5139c5ffe9d70b14f77cc56924c41abda130d4/web/app/components/Account/CreateAccount.jsx#L57-L62 .. But I am getting an error when I try to create my first account: ERROR AccountActions.createAccount Error: value required for registrar_id: null .. and the GUI reports "Failed to create account: account - unknown error"  Should this say something about using a faucet instead?

yes, the first account is supposed to be created via faucet. I think "unknown error” means it couldn’t connect to the faucet, I’ll take a look if I can improve errors reporting.

On Sep 10, 2015, at 10:23 AM, James Calfee<EMAIL_ADDRESS>wrote:
It looks like the createAccount was designed to work without a registrar. https://github.com/cryptonomex/graphene-ui/blob/1e5139c5ffe9d70b14f77cc56924c41abda130d4/web/app/components/Account/CreateAccount.jsx#L57-L62 https://github.com/cryptonomex/graphene-ui/blob/1e5139c5ffe9d70b14f77cc56924c41abda130d4/web/app/components/Account/CreateAccount.jsx#L57-L62 .. But I am getting an error when I try to create my first account: ERROR AccountActions.createAccount Error: value required for registrar_id: null .. and the GUI reports "Failed to create account: account - unknown error" Should this say something about using a faucet instead?
—
Reply to this email directly or view it on GitHub https://github.com/cryptonomex/graphene-ui/issues/192#issuecomment-139252326.

--
Regards,
Val


Refactor Mixtral to reuse code from MegaBlocks

Hello! A portion of the MoE implementation for Mixtral is copied directly from MegaBlocks. It's somewhat error prone code and I've been meaning to factor out helpers for it, which we could reuse to avoid having this duplicated in vLLM. If this is interesting to you I'll send a PR :)

Hi @tgale96, sounds wonderful! We'd appreciate your contribution!

Great :) I'll get to it this week, I think!
No, we won't recompile when the batch size changes. We will only re-compile when the constexpr parameters of the kernels change, which are implementation details like the tile dimensions and threadblock-swizzle config.

Hi @tgale96, we've just merged #2090, which doesn't use Megablocks but seems to solve the performance (and ROCm compatibility) issue at the moment. However, we'd love to see your PR since the current method might not be efficient for other future MoE models.

Hi! The PR is pretty simple in vLLM - those functions I highlighted we'll exposed as public APIs, we'll delete them from vLLM and then we'll call into megablocks for them. If you'd like to see it I'm happy to make the changes, but if vLLM won't use the APIs we will probably avoid exposing them so we are free to change them in the future :)
I'm very curious to understand that PR better! I dropped a question over there :)


Cow: Watch give-engineering.com

Cow requests the watch of the watch_keyword give-engineering\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

give-engineering\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Many reconciled transfers in DB with next_execution_time = 0

Describe the bug
Somewhere this is getting reset to 0 when it shouldn't be...
To Reproduce
Steps to reproduce the behavior:

Go to testnet-prod database.
See transfers in Reconciled status.
Observe next_execution_time

Additional context
Add any other context about the problem here.

This looks like it's because in some cases the sequencer isn't responding to the POST request. Potentially could get fixed by #4326

No longer seeing this anymore!


Upload: DioErrorType.DEFAULT on audio files

New Issue Checklist

[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
flutter

Platform Version
Channel stable, 1.22.6

Dio Version
3.0.10

Android Studio / Xcode Version
Android Studio 4.1.2

Repro rate
all the time

Issue Description and Steps
When trying to upload an audio file (.aac), the following error happens:
I/flutter ( 5187): DioError [DioErrorType.DEFAULT]: type '_StreamHandlerTransformer<List<int>, Uint8List>' is not a subtype of type 'StreamTransformer<Uint8List, Uint8List>' of 'streamTransformer'
I/flutter ( 5187): #0      Stream.transform (dart:async/stream.dart)
I/flutter ( 5187): #1      DioMixin._transformData (package:dio/src/dio.dart:1039:18)
I/flutter ( 5187): #2      DioMixin._dispatchRequest (package:dio/src/dio.dart:921:26)
I/flutter ( 5187): #3      DioMixin._request._interceptorWrapper.<anonymous closure>.<anonymous closure>.<anonymous closure> (package:dio/src/dio.dart:849:37)
I/flutter ( 5187): #4      DioMixin.checkIfNeedEnqueue (package:dio/src/dio.dart:1121:22)
I/flutter ( 5187): #5      DioMixin._request._interceptorWrapper.<anonymous closure>.<anonymous closure> (package:dio/src/dio.dart:846:22)
I/flutter ( 5187): #6      new Future.<anonymous closure> (dart:async/future.dart:175:37)
I/flutter ( 5187): #7      _rootRun (dart:async/zone.dart:1182:47)
I/flutter ( 5187): #8      _CustomZone.run (dart:async/zone.dart:1093:19)
I/flutter ( 5187): #9      _CustomZone.runGuarded (dart:async/zone.dart:997:7)
I/flutter ( 5187): #10     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1037:23)
I/flutter ( 5187): #11

Here the interested piece of code.
        await Dio().put(
          presignedUrl,
          data: file.openRead(),
          options: Options(
            contentType: lookupMimeType(file.path),
            headers: {
              "Content-Length": file.lengthSync(),
              'x-amz-acl': 'public-read',
            },
          ),
        );

I checked the runtime type of file.openRead() when file is a aac format File.
It actually is _BoundSinkStream<dynamic, Uint8List> and I think this is the problem.
How can I solve it?


Unable to remove validation for some specific field conditionally

Hi @llauderesv
Unable to remove validation for some specific field conditionally.

Hi @llauderesv
I have shared some here. Please check changeDesignationVal method.
// form related code start
// Define your state schema
const stateSchema = {
designation: { value: "", error: "" },
email: { value: "", error: "" }
};
// Create your own validationStateSchema
// stateSchema property should be the same in validationStateSchema
// in-order a validation to works in your input.
const stateValidatorSchema = {
designation: {
required: true
},
email: {
required: true,
validator: {
func: value => /^[a-zA-Z0-9._-]+@[a-zA-Z0-9.-]+.[a-zA-Z]{2,4}$/.test(value),
error: "Invalid email format."
}
}
};
const {
values,
errors,
dirty,
handleOnChange,
handleOnSubmit,
setStateSchema,
disable,
setDisable
} = useForm(stateSchema, stateValidatorSchema, callAPI);
const { designation, email } = values;
// form related code ends
/* Method is responsible for  API call */
const callAPI = (state) => {
console.log(state);
}
/* Method is responsible for changing Designation Value */
const changeDesignationVal = (item) => {
if (item.value === 'introducer') {
// here I want to remove required: true validation for email field
} else {
// here I want to again  add required: true validation for email field
}
}
return (
<>

<Form.Row>
<Form.Group>

{errors.designation && dirty.designation && (
<Form.Text className="text-danger text-left">
{errors.designation}
</Form.Text>
)}
</Form.Group>
                     {/* email */}
                     <Form.Group>
                          <Form.Control type="email" placeholder="Email" size="lg" name="email" value={email} onChange={handleOnChange} />
                          {errors.email && dirty.email && (
                               <Form.Text className="text-danger text-left">
                                       {errors.email}
                                </Form.Text>
                           )}
                     </Form.Group>
      </Form.Row>
   </>

Hi @akash-int,
You should do the validation inside your designation stateSchemaValidator like below, func returns a boolean, if true error will shown and if not nothing happens. And you can now remove the requried true prop in your email stateValidatorSchema.
const stateValidatorSchema = {
  designation: {
    required: true,
    validator: {
        func: (value) => value !== 'introducer',
        error: "Email is required"
      }
  },

Hi @llauderesv
This is not working for me in my scenario because if I choose introducer

I have to hide all other fields of the form
Remove validation from all other fields
Reset all other fields data if anything is written to that fields

If again I change the value from the dropdown I have to add validation to all other fields :-(
After implementing your example form is not validating. It remains still disabled as I have not provided all other fields data.
This is my actual state schema
const stateValidatorSchema = {
designation: {
required: true,
validator: {
func: (value) => value !== 'introducer',
error: "Email is required"
}
},
prefix: {
required: true
},
firstName: {
// required: true,
validator: {
func: value => /^[a-zA-Z]{2,20}$/.test(value),
error: "Invalid firstName format."
// Minimum 2 characters are required for firstName
}
},
lastName: {
// required: true,
validator: {
func: value => /^[a-zA-Z]{2,20}$/.test(value),
error: "Invalid lastName format."
}
},
email: {
// required: true,
validator: {
func: value => /^[a-zA-Z0-9._-]+@[a-zA-Z0-9.-]+.[a-zA-Z]{2,4}$/.test(value),
error: "Invalid email format."
}
}
};

Hi @llauderesv
Please check this sandbox link
https://codesandbox.io/s/xenodochial-bas-kr9hj?file=/src/App.js

I have to hide & remove validation from all other fields while introducer is selected
Again have to show & add validation to all other fields while director is selected.

Thanks in advance.

Hi @akash-int
I updated your sandbox to accomodate your concern, as an Intermitent solution I added a new function setValidatorSchema to set a brand new validation schema for your state depending on your condition.

Hi @llauderesv
I have created that sandbox without sign in so somehow your changes are not showing there.Can you please create a new sandbox & share that link here :-)


IPv6

Returning tuple from socket.getaddrinfo() too long using IPv6.
Working for IPv4 / IPv6:
        self.address_family, _, _, _, addrinfo = socket.getaddrinfo(
                host=args.host,
                port=None,
                flags=socket.AI_ADDRCONFIG
            )[0]

        self.ip_address = addrinfo[0]

Sorry for the delay, and thank you for the report and the solution :) It's fixed now via https://github.com/grongor/knock/pull/12


Add option to make a command not run in a shell

One of my formatters gets passed off to sed, which was broken by #38.
Additionally, running commands in a shell causes it to be affected by what &shell is set to.
I'll admit I don't entirely understand why #38 was needed, but this should retain the behaviour
that it added. It adds a no_shell option, which if set to true, passes the command as a table to jobstart.
If it's false (or nil since that's falsey) then the current behaviour is maintained.

For reference, this is the sed formatter I use:
function()
  return {
    exe = 'sed',
    args = {'-e', [[s/[ \t]*$//]], '-e', [[:a; /^$/ { $d; N; ba; }]]},
    stdin = true,
  }
end

It removes trailing whitespaces and lines.

Running the commands inside the shell was the original behavior.
#38 just reverted it back to how it was before #18
At first, I thought running them directly would make more sense, but this broke a bunch of things.
Most users expect the commands to work the same as they do in their shell.
To fix your command, I think you only have to wrap the regex in quotes.
'-e', [['s/[ \t]*$//']], '-e', [[':a; /^$/ { $d; N; ba; }']]
Not really sure if we need an option for this :thinking:
But I'm open for a discussion.

Yeah, I'm not sure this is something I want to open up. Most formatting tools assume it's being called by a shell process vs being called directly. I think having to wrap your regex in quotes makes more sense. Plus lua makes this much more approachable.

I had mostly figured that commands running directly was the intended behavior because arguments are exposed as an array, but if that wasn't intended, this probably isn't necessary.


Needs delay between socket creation and connect operation

Hi,
I just started using Starscream and had some weird issue:
When I tried to follow your example with an object that performs:
var socket = WebSocket(url: NSURL(scheme: "ws", host: "localhost:8080", path: "/")!)
socket.delegate = self
socket.connect()

The connection never happened.
It didn't arrive the server nor did it fire the delegate method I've set.
What I had to do is to separate the socket creation and connect() method to two different functions which are called one after another:
func create() {
  self.socket = WebSocket(url: NSURL(scheme: "ws", host: "localhost:8080", path: "/")!)
  socket.delegate = self
}
func connect(){
  socket.connect()
}

This way everything worked fine, the connection succeeded and I the delegate method got fired as expected.
From what I can conclude there has to be some time interval between the socket creation and connection.
I am using XCode 7 and swift 2
I integrated Starscream vie cocoapods with the following line in the Podfile:
pod 'Starscream', '~> 1.0.0'
Did anyone else get that weird issue? Am I misunderstanding something very basic here?
Thanks for the help in advance!

This is expected behavior. The example in the README just demonstrate the basics of the API. You need to keep a reference to the socket object or it will get deallocated. If you just do var socket in viewDidLoad or something like that it is a local variable and gets deallocated almost immediately and consequently doesn't do anything. See the SimpleTest project in the examples folder on basic setup/usage.

@daltoniam Thanks for the explanation.
I got two more questions regarding this:

How could I know that the socket will get deallocated if there's no reference to it? It is written or documented somewhere?
Do you think it is worth mentioning in the README somewhere or at least in the specific example which actually does not work in reality?

Thanks for the help again,
I really appreciate it.

@erez-rabih It's in the Swift language. If there's no reference to an object it will be deallocated. https://developer.apple.com/library/ios/documentation/Swift/Conceptual/Swift_Programming_Language/AutomaticReferenceCounting.html#//apple_ref/doc/uid/TP40014097-CH20-ID48

@erez-rabih. Yeah @nuclearace summed it up nicely and found the docs. I could mention that in the README, but I feel it should be implied on how Swift works. Kinda of a "how far the rabbit hole do you want to go" for an example. We also have the example projects, so I think that should clear up any remaining ambiguity.


[Bug] mlc-ai-nightly-rocm not found in pip.

🐛 Bug
mlc-ai-nightly-rocm not found in pip.
To Reproduce
Steps to reproduce the behavior:

Follow the documentation for installing the library: python3 -m pip install --pre -U -f https://mlc.ai/wheels mlc-chat-nightly-rocm56 mlc-ai-nightly-rocm56.

But I get:
#27 [20/40] RUN python3 -m pip install --pre --force-reinstall mlc-ai-nightly-rocm mlc-chat-nightly-rocm -f https://mlc.ai/wheels
#27 0.718 Looking in links: https://mlc.ai/wheels
#27 1.228 ERROR: Could not find a version that satisfies the requirement mlc-ai-nightly-rocm (from versions: none)
#27 1.229 ERROR: No matching distribution found for mlc-ai-nightly-rocm
#27 ERROR: executor failed running [/bin/bash -o pipefail -c python3 -m pip install --pre --force-reinstall mlc-ai-nightly-rocm mlc-chat-nightly-rocm -f https://mlc.ai/wheels]: exit code: 1
------
 > [20/40] RUN python3 -m pip install --pre --force-reinstall mlc-ai-nightly-rocm mlc-chat-nightly-rocm -f https://mlc.ai/wheels:
0.718 Looking in links: https://mlc.ai/wheels
1.228 ERROR: Could not find a version that satisfies the requirement mlc-ai-nightly-rocm (from versions: none)
1.229 ERROR: No matching distribution found for mlc-ai-nightly-rocm

Expected behavior
I expected: that the package installation would always succeed.
A suggestion: I've also noticed that there's a single tag: v0.1.dev0 release for this project. I know this is a "nightly" supported only, but maybe we could have some versioning still to avoid breaking things? Then people could use the bleeding-edge nightly if they wanted to, but folks could instead opt for nightly-2023-11-08-v0.1.dev0 for example.
Environment

Platform (e.g. WebGPU/Vulkan/IOS/Android/CUDA): ROCM.
Operating system (e.g. Ubuntu/Windows/MacOS/...): ubuntu.
Device (e.g. iPhone 12 Pro, PC+RTX 3090, ...): independent of device.
How you installed MLC-LLM (conda, source): the problem is installing it.
How you installed TVM-Unity (pip, source): no.
Python version (e.g. 3.10): 3.10
GPU driver version (if applicable): not applicable.
CUDA/cuDNN version (if applicable): not applicable.
TVM Unity Hash Tag (python -c "import tvm; print('\n'.join(f'{k}: {v}' for k, v in tvm.support.libinfo().items()))", applicable if you compile models): not applicable.

Please follow the documentation here: https://llm.mlc.ai/docs/install/tvm.html#option-1-prebuilt-package

There should be a suffix like rocm57, i suppose?

There should be a suffix like rocm57, i suppose?

Thanks for the fast response. I'll try that, but in any case, I think this could be affecting others.

oh i think it’s just a typo on your end. as you mentioned in step 1, the documentation says:
python3 -m pip install --pre -U -f https://mlc.ai/wheels mlc-chat-nightly-rocm56 mlc-ai-nightly-rocm56

but the instructions you’ve got are using:
#27 [20/40] RUN python3 -m pip install --pre --force-reinstall mlc-ai-nightly-rocm mlc-chat-nightly-rocm -f https://mlc.ai/wheels

which lacks the suffix 56/57

The suffix 56/57 corresponds to ROCm versions 5.6 and 5.7, which are not related to MLC LLM’s version, so even if with date-based naming, we still need 56/57 as the suffix.

I assume this issue is gone after fixing the typo, but please feel free to reopen this issue if there’s still anything wrong

Thanks for your input. That worked for our case but I don't want to miss out on the opportunity of saying others could be affected if they install through the CPU tab. I still think that labeling your releases, even if they are nightly could help the user base.

Fair enough. Thanks for the input! What do you think we should do to avoid users mistakenly installing a CPU-only pip package in the CPU tab?

Thanks for giving attention to this!

Fair enough. Thanks for the input! What do you think we should do to avoid users mistakenly installing a CPU-only pip package in the CPU tab?

The documentation is clear enough on that part. I think it was I that didn't pay enough attention.
My perspective here is more on the release part. Ideally I would like to pin to a version, even if a dev one:
python3 -m pip install 'mlc-ai-nightly-rocm56==1.2.3'

So that I'm sure to be using the same code over different days.
Irrespective of specifying the CPU/GPU architecture, it looks like using:
python3 -m pip install --pre -U -f https://mlc.ai/wheels mlc-chat-nightly-rocm56 mlc-ai-nightly-rocm56

For instance, will give me a slightly different package every now and then. Correct me if I'm wrong...

@fmv1992 Yes, the latter command will always give you the up-to-date wheel package. Actually, for now we apply an auto-cleaning routine and do not keep the old versions. That is to say, we will automatically remove old wheels once there are newer ones. This is because we do nightly build everyday, and the total wheel list will be very large if we keep multiple history versions for different combination of OS, Python versions, CUDA/ROCm versions.


chore: add rust-toolchain.toml

Matches our toolchain version in the vm repo. Fixes build issues with #20

closing in favour of #22


Http 200 ok for non entitled addresses

This is based on PR #39. Only the last commit is relevant.
Return '200 OK' for non entitled addresses
Previously we returned a 404 error when addresses where not contained
in the merkle tree.
This is indistinguishable from trying to load from a wrong server/URL
and may hide those errors from the user.
Instead we now return a normal response with originalTokenBalance set
to zero and an empty proof.

Codecov Report

Merging #40 into master will decrease coverage by 3.61%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   78.71%   75.09%   -3.62%
==========================================
  Files           6        6
  Lines         249      261      +12
==========================================
  Hits          196      196
- Misses         53       65      +12

Impacted Files
Coverage Δ

merkle_drop/server.py
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Create a Contribute.md file

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

Contribute.md file is not present in github code repository of hyper. All related matter is present in readme file. Kindly make a separate Contribute.md

There isn't a lot of thing about contributing and I think it is better to have it directly in README.
Btw, your contriubtions will be very welcome do not hesitate to search Good for beginners issues.


Changed so all values from the forms are escaped with htmlentities()

XSS was possible here. Issue solved where value is used (instead of escaping every input when you create the forms). Unsure if the same solution has to be implemented on other calls.

If you want to save the value in the database, unescaped?

I'd say send a flag to the function to save unescaped and default to always being escaped. Since escaping on every input field is a lot more work than not escaping the few times you need it.


Upgrade to rxjs 7 issue on this.schedulerActionCtor undefined

Error in using Subject - this.schedulerActionCtor undefined
Environment

Runtime: [e.g. Node v12.16.0, Chrome Version 89.0.4389.90 (Official Build) (32-bit)]
RxJS version: 7

Steps to reproduce:
import React, { useEffect } from 'react';
import { Subject } from 'rxjs';
import { debounceTime, buffer } from 'rxjs/operators';

const SearchSubject = new Subject();
const debounceSearch$ = SearchSubject.pipe(debounceTime(1000));
const Search$ = SearchSubject.pipe(buffer(debounceSearch$));

 useEffect(() => {
    const SearchSubscriber$ = Search$.subscribe((value) => {
      const search = value[value.length - 1];

      dispatch(action(search));
    });

    // cleanup
    return () => {
      SearchSubscriber$.unsubscribe();
    };
  }, [dispatch]);

This really needs a minimal reproduction to be actionable.

@cartant Apology for the missing steps.  Added the steps above.  schedulerActionCtor is undefined on initialize. Might need to check whether this method is defined before initializing it.

Thanks. However, this is such a fundamental failure, I think you'd have to provide a reproduction that involves your build process. Or at least explain how you are building this. The only place that QueueScheduler is created is here:
https://github.com/ReactiveX/rxjs/blob/e19e104d011233d83bc10c37f1ee0b3ac6e15612/src/internal/scheduler/queue.ts#L67
And you can see that the ctor is passed. This applies to all of the schedulers.
Also, your repro doesn't involve the queueScheduler, so I cannot see how it can effect the error message you've included in the issue.
FWIW, I cannot reproduce the problem with:
import { Subject } from "rxjs";
import { debounceTime, buffer } from "rxjs/operators";

const searchSubject = new Subject();
const debounceSearch$ = searchSubject.pipe(debounceTime(1000));
const search$ = searchSubject.pipe(buffer(debounceSearch$));
const subscription = search$.subscribe();

searchSubject.next("f");
searchSubject.next("fo");
searchSubject.next("foo");

setTimeout(() => {
  subscription.unsubscribe();
}, 5e3);

Also, it's worth noting that the Scheduler ctor has not changed (other than an internal property being renamed) from v6:
https://github.com/ReactiveX/rxjs/blob/881cacdc99a7ebae219b595004c632f7358f730d/src/internal/Scheduler.ts#L33-L36
to v7:
https://github.com/ReactiveX/rxjs/blob/f9d29f7e75ff9394e3f4f79cfd4688593eb757c9/src/internal/Scheduler.ts#L28-L30
and the queueScheduler declaration:
https://github.com/ReactiveX/rxjs/blob/881cacdc99a7ebae219b595004c632f7358f730d/src/internal/scheduler/queue.ts#L66
has not changed either:
https://github.com/ReactiveX/rxjs/blob/f9d29f7e75ff9394e3f4f79cfd4688593eb757c9/src/internal/scheduler/queue.ts#L67


Amazing work

Amazing work!!!! Looking forward to more features supported!

@yafei-liu Thanks!
We're searching for applications to which cuda-bundle-adjustment is applicable.
Any suggestions are welcome!


Allow to disable draggable

Hi @gltiennd!
It wasn't possible but now it is. Update to 1.16.0 and change your code to:
Docking(layout: _layout , draggable: false)

Thanks!


Failed to spawn process in lazyvim

Describe the bug
when i run the following command coursier bootstrap .... scalameta .. on cmd on windows, it seems working, but when i run install metals inside nvim, i get this error.
...ta/Local/nvim-data/lazy/plenary.nvim/lua/plenary/job.lua:404: Failed to spawn process: {
  _additional_on_exit_callbacks = {},
  _shutdown_check = <userdata 1>,
  _stderr_results = {},
  _stdout_results = {},
  _user_on_exit = <function 1>,
  _user_on_stderr = <function 2>,
  _user_on_stdout = <function 3>,
  args = { "bootstrap", "--java-opt", "-Xss4m", "--java-opt", "-Xms100m", "org.scalameta:metals_2.13:latest.release", "-r", "sonatype:snapshots", "-o", "C:\\Users\\dtwj6af\\AppData\\Local\\Temp\\nvim\\nvim-metals\\metals", "-f" },
  command = "coursier",
  enable_handlers = true,
  enable_recording = true,
  interactive = true,
  pid = "ENOENT: no such file or directory",
  stderr = <userdata 2>,
  stdin = <userdata 3>,
  stdout = <userdata 4>,
  user_data = {},
  <metatable> = <1>{
    __index = <table 1>,
    _create_uv_options = <function 4>,
    _execute = <function 5>,
    _pipes_are_closed = <function 6>,
    _prepare_pipes = <function 7>,
    _reset = <function 8>,
    _shutdown = <function 9>,
    _stop = <function 10>,
    add_on_exit_callback = <function 11>,
    after = <function 12>,
    after_failure = <function 13>,
    after_success = <function 14>,
    and_then = <function 15>,
    and_then_on_failure = <function 16>,
    and_then_on_failure_wrap = <function 17>,
    and_then_on_success = <function 18>,
    and_then_on_success_wrap = <function 19>,
    and_then_wrap = <function 20>,
    chain = <function 21>,
    chain_status = <function 22>,
    co_wait = <function 23>,
    is_job = <function 24>,
    join = <function 25>,
    new = <function 26>,
    pid = <function 27>,
    result = <function 28>,
    send = <function 29>,
    shutdown = <function 30>,
    start = <function 31>,
    stderr_result = <function 32>,
    sync = <function 33>,
    wait = <function 34>
  }
}
stack traceback:
	...ta/Local/nvim-data/lazy/plenary.nvim/lua/plenary/job.lua:404: in function '_execute'
	...ta/Local/nvim-data/lazy/plenary.nvim/lua/plenary/job.lua:447: in function 'start'
	.../Local/nvim-data/lazy/nvim-metals/lua/metals/install.lua:50: in function 'do_install'
	.../Local/nvim-data/lazy/nvim-metals/lua/metals/install.lua:152: in function <.../Local/nvim-data/lazy/nvim-metals/lua/metals/install.lua:99>
	[C]: in function 'pcall'
	...ta/lazy/nvim-metals/lua/telescope/_extensions/metals.lua:27: in function 'key_func'
	...nvim-data/lazy/telescope.nvim/lua/telescope/mappings.lua:257: in function <...nvim-data/lazy/telescope.nvim/lua/telescope/mappings.lua:256>

Expected behavior
Metals should be installed without problems
Operating system
Windows
Version of Metals
1.0.1
Commit of nvim-metals
57cff9a

when i run the following command coursier bootstrap .... scalameta .. on cmd on windows, it seems working, but when i run install metals inside nvim, i get this error.

Could you give some more details on how you're running this? Is this what you're seeing when you run :MetalsInstall?

sorry for short info,
yes i run :MetalsInstall

sorry for short info, yes i run :MetalsInstall

Hey, sorry I haven't gotten back to this. I'm not really sure how to reproduce as I don't have a windows machine and also don't use lazyvim.

If you installed coursier with scoop, a quick way to fix this is create a link named cs.bat to coursier.cmd.
to achieve this, following these steps:

enter cmd environment with cmd
cd %USERPROFILE%\scoop\shims
mklink cs.bat coursier.cmd

After these steps, relaunch your terminal and enter nvim and run MetalsInstall


eliminate  np_session and npc_sessions from metadatamapper

We just had a meeting with Ben and Arielle. We decided to keep npc_mvr and npc_ephys, but try to eliminate npc_sessions and np_session from the dependencies.
Two things need to be done.

np_session object and properties should be factored out of the Camstim class.
npc_sessions.get_newscale_coordinates should be relocated to another place (npc_ephys I think?). This will be tracked here for Ben: https://github.com/AllenInstitute/npc_sessions/issues/122. And then the function call should be changed to accomodate the new namespace

Figured it out. We are able to get a session uuid from the pkl file and then can use that as input to a call to mtrain to get the regimen script url and name.
The rest of the refactoring was simple enough.
Done in PR #174


False error on CRLdp does not contain url

I believe there may be a bug in Zlint. If you look at this certificate:
https://censys.io/certificates/6ef0debdfc2d9af68c5174c2b88ccfabde9540a1b6b96e2526b59e6a9ab09b10
You will see it has a zlint error, specifically, it is:
sub cert crl distribution points does not contain url

And if you look at the CRLdp in the CenSys user interface it shows a single ldap URI:

But if you look at the same certificate under OpenSSL you see:

Upon inspection of the lint code, it seems it is doing the right things:
https://github.com/zmap/zlint/blob/master/lints/lint_sub_cert_crl_distribution_points_does_not_contain_url.go#L43
So I guess the X.509 decoder is only returning a single CRLdp?

From https://www.ietf.org/rfc/rfc5280.txt:
   If the DistributionPointName contains multiple values, each name
   describes a different mechanism to obtain the same CRL.  For example,
   the same CRL could be available for retrieval through both LDAP and
   HTTP.

So unless the CABR says otherwise, it seems like this error is indeed erroneous.

@justinbastress Thanks for the quick response!
The CABR has nothing on this, beyond a HTTP link must be present.

It seems to me that the RFC - as in many cases - leaves room for interpretation and that both options seem valid, i.e. the formulation as quoted above from the RFC allows both.

In such cases what clients support is a good litmus test. Windows, OSX, OpenSSL, and probably others seem to like the nested approach.

Just for asking for clarification, the nested Approach refers to which coding option mentioned above?
CRLDistributionPoints: [ { fullname: [ "url1", "url2", "url3" ] } ] or the secondOption. E.g. Windows displays this first option correctly.

Yes, if I understand your question correctly I mean the encoding.

Thanks for quickly solving this issue! Kim

Thank you for resolving this!


Text highlighted after choosing a select option

Describe the bug
When choosing a select option the cursor behaves as though it is being held and begins to highlight text around the select.root
Functions normally in Chrome
To Reproduce

Click a select field where a root has some siblings or parent elements with Text
choose an option, upon the list-box closing move the cursor around and the text will be higlighted

A quick sandbox showing the behavior
https://codesandbox.io/p/sandbox/sad-water-z2rkg4?file=/src/App.tsx:21,25

Expected behavior
Focus should remain on the select trigger and the cursor should not highlight other text

OS: MX Linux
Browser Firefox
Version 116.0.3

You can add user-select: none; to the select item, such as in https://kobalte.dev/docs/core/components/select#controlled-value.

Works! thanks!


Doesn't work on android 12L

The feature unified signal icons doesn't works on android 12L.

For me, the custom colors no longer work since the march update.

same for custom colors here

Custom colors don't seem to work too on lineage 19.1 (12L)

Same here on Arrow OS 12.1 (12L)

This project has been discontinued. So the dev won't fix it.
If someone could make a fork of it, it would be great.


PropsTable is not working with components encapsulated by React.forwardRef()

<PropsTable of={Input} /> only works when I remove the React.forwardRef(). The following code is Typescript, and I also have typescript: true on my config.
import * as React from "react"
import styled from "styled-components"

// Estilos ---------------------------------------------------------------------

const StyledInput = styled.input.attrs({
  className: "form-control",
})`
  height: 40px;
  margin-bottom: 0;
  font-size: 0.8125rem;
`

const FormRow = styled.div.attrs({
  className: "form-group",
})`
  margin-bottom: 1.25rem;
`

// <Input /> -------------------------------------------------------------------

interface IPInput {
  id: string
  label?: string | false
  placeholder?: string
}

const Input = React.forwardRef(
  (
    { id, label, placeholder, ...extraProps }: IPInput,
    ref: React.RefObject<HTMLInputElement>,
  ) => (
    <FormRow>
      <label
        htmlFor={id}
        className={[false === label ? "sr-only" : ""].join(" ")}
      >
        {false === label && placeholder ? placeholder : label}
      </label>
      <StyledInput
        innerRef={ref}
        id={id}
        placeholder={placeholder}
        {...extraProps}
      />
    </FormRow>
  ),
)

export default Input

Same here! 😢

I found a solution but it looks ugly.
const WrappedComponent = React.forwardRef(
  function MyComponent(props, ref) {
    return <StyledComponent innerRef={ref} {...props} />;
  }
);

<PropsTable of={WrappedComponent.render} />

@sandiiarov innerRef has been deprecated in styled-components. forwardRef is what will need to be used in the future.

@Jared-Dev Yeah I know. It doesn't matter in that example.

It looks like as soon as your component is wrapped inside a HOC, it won't render the prop types. Quickest solution for now is to also export your component as a named export without the HOCs. That way your prop types will work again.

@jnaudin Thanks for posting such detailed solutions. The first one works great, but the second one I got MyComponent.render is not a function error. Any ideas?

I think that this will be resolved in the v1 :v:

@jnaudin Thanks for posting such detailed solutions. The first one works great, but the second one I got MyComponent.render is not a function error. Any ideas?

I think you imported the wrapped component and not the HOC:
import {MyComponent} from './MyComponent'
instead of
import MyComponent from './MyComponent'
Can you show me what your code ?

Closed in favor of the newest release.


[SPARK-44340][SPARK-44341][SQL][PYTHON][FOLLOWUP] Set partition index correctly for WindowGroupLimitExec,WindowExec and WindowInPandasExec

What changes were proposed in this pull request?
This is a followup of https://github.com/apache/spark/pull/41899 and https://github.com/apache/spark/pull/41939, to set the partition index correctly even if it's not used for now. It also contains a few code cleanup.
Why are the changes needed?
future-proof
Does this PR introduce any user-facing change?
'No'.
How was this patch tested?
existing tests

ping @vinodkc cc @cloud-fan @viirya

thanks, merging to master/3.5!

@cloud-fan @vinodkc Thank you!

@beliefer can you open a 3.5 backport PR? one of the main PR is in 3.5 but the other is not.

@beliefer can you open a 3.5 backport PR? one of the main PR is in 3.5 but the other is not.

OK


Update Docs around AzureConverter

Preprocessing components page:

Adds Azure + Markdown file converters
Adjusts description of formats

Table QA Guide:

Link Azure Converter

@brandenchan I somehow could not access the preprocessing page in the vercel preview. Do you see why this might be broken?


Bug fix wrong result on map_agg with streaming agg

Proposed changes
fix wrong result on map_agg with streaming agg

run buildall


Escape {{string}}

Expected Behavior
Hi, I try to send request similar to:
{
"a": "a",
"b": "{{1}}"
}
Literally I need send {{1}} and not convert {{1}} into var 1.
I try to scape { and }
I try to create var 1 with value {{1}}
Nothing works.
It's posible send literally, into quotes string with {}?
Thanks
Actual Behavior
replace literally {{whatever}} by whatever variable
Reproduction Steps
No response
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
Version: Insomnia 2022.7.0-beta.1 Build date: 10/6/2022 OS: Darwin x64 22.1.0 Electron: 19.0.3 Node: 16.14.2 V8: <IP_ADDRESS>-electron.0 Architecture: x64
What operating system are you using?
macOS
Operating System Version
Ventura
Installation method
brew
Last Known Working Insomnia version
No response

@fernandomacho the closest you can get to this is by opening the request settings and enabling the option to Skip rendering of request body:

Keep in mind this will disable the ability to use environment variables, but on the other hand you'll be able to send requests in that way.

Thanks!

On 14 Oct 2022, at 11:06, Filipe Freire @.***> wrote:
Closed #5280 https://github.com/Kong/insomnia/issues/5280 as completed.
—
Reply to this email directly, view it on GitHub https://github.com/Kong/insomnia/issues/5280#event-7588485970, or unsubscribe https://github.com/notifications/unsubscribe-auth/AO2NF7UKBXTEXJJ4ONDCCJTWDEPBNANCNFSM6AAAAAARFAQD44.
You are receiving this because you were mentioned.


Marvell AWS IoT SDK

Marvell's Starter SDK for AWS IoT Service available here,
https://github.com/marvell-iot/aws_starter_sdk
More information,
http://www.marvell.com/microcontrollers/aws-iot-starter-kit/

Hi @mahavirj would you like give us a  PR ?


Update CR owner references and infrastructure references to use the latest CAPI CRs

Update CR owner references and infrastructure references to use the latest CAPI CRs.
TODOs:

[ ] which customers use gitops? ( @alex-dabija )
[ ] which customers mange CRs on their own? ( @alex-dabija )
[ ] ...;

Which CRs?

on AWS: Cluster, MachineDeployment;
on Azure: Cluster, AzureCluster, AzureMachine, MachinePool, AzureMachinePool;

This is handled in the cluster-app, so it will be replaced when the app is upgraded or version removed. No need for separate ticket as this will come out 'naturally'


Adding shortcut to check transaction history on Etherscan

Is your feature request related to a problem? Please describe.
After submitting a transaction, user need to open metamask and click to the transaction in order to check it's status on Etherscan. If people is using hardware wallet, then they won't able to check it since they're not connecting via Metamask
Describe the solution you'd like
Adding a shortcut on the transaction confirmation screen so people can check the TX on Etherscan after submitting TX. It should diligently support Mainnet/Ropsten/Tomochain block explorer

Resolved in 36eac0184f08b7884c1c3d4b80fb0bab752ab1b3, but in this version, only support for Ropsten network

Close in favor of #36


"Encoding" dropdown is backwards when viewing messages in queue

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.26.0
Regression From
N/A (fresh install)
Architecture
x64
Storage Explorer Build Number
20221001.2
Platform
Windows
OS Version
Windows 11
Bug Description
When viewing messages in an Azure Queue, the "Encoding" dropdown has the opposite expected behavior when selecting "Base64" versus "UTF8".
Steps to Reproduce

Launch storage explorer
Create a new queue and add a message
Double click on the message to open the "View Message" dialogue.
Toggle through the "Encoding" dropdown options.

Actual Experience
Here's what it looks like when Base64 is selected:

And here's what it looks like when UTF8 is selected:

As you can see, the opposite encoding is used.
Expected Experience
The selected encoding option should match the encoding used in the "message text" field.
Additional Context
No response

@jakraft the dropdown is correct, though albeit confusing. When you select a value in the dropdown, you are basically saying "this message is stored as X, so decode it". So for example, for base64, you would select that if the message is encoded as base64 and you want it decoded so you can read it. We will try to update these labels to reduce confusion. Does that all make sense?

@jakraft what do you think about these new labels/values?
For adding a message:

For viewing a message:

@MRayermannMSFT thanks for the clarification! Agreed, I think those labels are much clearer.

New labels merged for 1.27


Migrate IDENTITY and NOT NULL properties

I noticed that the exasol to exasol migration does not apply IDENTITY and NOT NULL properties, which can does result in unwanted behaviour (i.e. the NULL checks are not applied / the IDENTITY column isn't filled automatically).
This change adds the migration of those properties.

Indeed, migrating NOT NULL and IDENTITY can be useful. Thanks for your contribution!


Roll over command not working on android

I know there are issues with plug-in support on android am not sure what the cause is. Seems like either obsidian doesnt want to preform some action or the plugin needs to be forked for android.

For the newest version, there are no known issues.
Can you give me insights on what exactly is not working?


docs: update usage

With the 0.2.0 release, the addition of a second getClient import triggers an error:
Syntax error: Identifier 'getClient' has already been declared

Remove from the example to help the unsuspecting.

@rexxars et al - any desire for this improvement?

Oh. This was separately reported in #27 and then fixed in #28 without me being aware of this PR. Sorry!


No "View Error..." action displays in the context menu for one broken QA storage account

Storage Explorer Version: 1.24.0-dev
Build Number: 20220421.6
Branch: rel/1.24.0
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.3 (Apple M1 Pro)
Architecture: ia32\x64
How Found: Ad-hoc testing
Regression From: Not a regression
Steps to Reproduce

Select one storage account -> Add it to Quick Access.
Go to 'ACCOUNT MANAGEMENT' panel -> Remove the account.
Go to 'EXPLORER' panel -> The QA storage account shows as broken.
Right click the broken storage account -> The context menu displays.
Check whether the action 'View Error...' displays.

Expected Experience
The action 'View Error...' displays.
Actual Experience
No action 'View Error...' displays.

Additional Context

Related issue: 3101.
This issue also reproduces for one QA attached storage account after detaching its attachment.
This issue also reproduces if deleting the storage account in portal.

Update: the action 'View Error' displays in the context menu for one broken QA storage account, but the error details is not helpful.

Verify this issue on the build main/20220609.2. Fixed.


Add contributors

Add contributors with bot

@all-contributors please add @Tomme for core and maintenance

@all-contributors please add @Tomme for code and maintenance

@all-contributors please add @mattiamatrix for maintenance and

@all-contributors please add @mattiamatrix for maintenance

https://github.com/all-contributors please add @Jrmyy for code

@all-contributors please add @Jrmyy for code

https://github.com/all-contributors please add @nicor88  for bug

@all-contributors please add @nicor88 for bug

@all-contributors please add @Jrmyy for bug

https://github.com/all-contributors please add @maiarareinaldo for bug

@all-contributors please add @maiarareinaldo for bug

@all-contributors please add @henriblancke for code and bug

@all-contributors please add @chrischin478 for code and bug

@all-contributors please add @sanromeo for code and bug

@all-contributors please add @Danya-Fpnk for code and bug

@all-contributors please add @Avinash-1394 for code

@all-contributors please add @sashakcc for code


Global cleanup

Codecov Report

Merging #23 (d1f0d60) into main (48e8dfa) will decrease coverage by 48.90%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #23       +/-   ##
==========================================
- Coverage   48.89%   0.00%   -48.90%
==========================================
  Files          21      21
  Lines         953     952        -1
==========================================
- Hits          466       0      -466
- Misses        487     952      +465

Impacted Files
Coverage Δ

src/StochasticVehicleScheduling.jl
0.00% <ø> (-100.00%)
:arrow_down:

src/learning/easy_problem.jl
0.00% <0.00%> (ø)

src/learning/trainer.jl
0.00% <0.00%> (ø)

src/solution/exact_algorithms/plne.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/solution/model_builders.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 9 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Move PublicSuffixList to amphp/uri

I think that's a better place for it.

Has been moved to an Internal namespace now, so we can always do that once we have a second use case.