sandboxes: Add protobuf package to grpc-bridge Python dependencies

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description:
I encountered this error on the grpc_bridge sandbox:
$ docker-compose exec python /client/client.py set foo bar
Traceback (most recent call last):
  File "/client/client.py", line 4, in <module>
    import kv_pb2 as kv
  File "/client/kv_pb2.py", line 6, in <module>
    from google.protobuf import descriptor as _descriptor

Adding the Python protobuf package to the Dockerfile fixes it.
$ docker-compose exec python /client/client.py set foo bar
setf foo to bar

Risk Level: Low
Testing: Manual
Docs Changes:  N/A

@nzoschke LGTM. Can you fix DCO please? (See contributing guide). Sorry for the trouble.


How do I open a new or existing file for random rw access without truncating it?

Mode:

"r+" won't create new files
"w" truncates
"wb" truncates
"w+" truncates
"a+" no random writes

"a+" no random writes

I really doubt that :o
Might just need to reset to the start of the file.

The issue tracker is not a good place for questions. Please check out some other options at https://crystal-lang.org/community/

"a+" no random writes

I really doubt that :o
Might just need to reset to the start of the file.

It's a function of Posix when a file is opened with O_APPEND.  From the linux man page:
Before each write(2), the file offset is positioned at the end of the file, as if with lseek(2).  The modification of the file offset and the write operation are performed as a single atomic step.
Test code:
File.open("test.txt", "a+") do |io|
  3.times do |i|
    io.seek 0
    io.puts "append #{i}"
  end
end

Expected output:
append 2

Actual output:
append 0
append 1
append 2

Run it again and the file length doubles rather than overwriting.

The issue tracker is not a good place for questions. Please check out some other options at https://crystal-lang.org/community/

I'm not really asking how to do something with crystal as it stands but how to adjust the API to support common file operations.  Consider this a feature request or API bug report.

You must manually check whether the file exists or not, then choose to open with w (create file) or r+ (read/write no truncate). There is no abstraction for what you want to achieve.
Please use the forum to ask questions. The diacussion may eventually lead to open an issue if there isn't a solution to the question.

Checking for a file first creates a race condition.  I need O_EXCL semantics.  There isn't a solution.
Maybe I wasn't specific enough.  My intention was to open a discussion on the File API by showing a security related deficiency in the current API and avoiding what happened when I submitted #7822 which while simple and seemingly necessary (how does Process.chroot function if you can't drop privileges?  It doesn't) is sitting in limbo for another PR #7829 with no code to show for it while API design is discussed.  As #7829 shows there were numerous PR's submitting code to solve the process problem that mostly stagnated until they were closed.
So instead of submitting a PR allowing O_EXCL that may or may not be accepted, how do I start the discussion on what will be accepted as in PR#7829?

You identified a problem, good! Don't phrase the issue title as a question then, "Missing API to...". If you know a good lower level solution, describe it, in medium detail :) If you want to discuss a new high level API, actually propose  one :) "Let's add a new mode, "a-"", "Let's allow seeking back for "a+"", ...

Should this issue be closed in favour of #7857 then?


Dataset

Hi ！
Your work is great, When will the dataset and CodaLab be open？
Thanks.

Hi @ztx313, sorry for the late reply. I have updated the private test set and the CodaLab . Thanks for your interest in our work!
Best,
Yilun


Pods: Network name is missing on the review screen

Please refer to the ticket https://jira.mesosphere.com/browse/DCOS_OSS-1370
Needs to be added to: https://github.com/dcos/dcos-ui/blob/master/plugins/services/src/js/service-configuration/PodNetworkConfigSection.js

@nLight can I pick this too!

@adityatandon007 it's yours :) Thanks!

@nLight Does this require the cluster

@adityatandon007 I'd say no. You can hit the same screen by trying to launch a multicontainer service:

@nLight I am unable to launch a multi container service

@adityatandon007 what is the issue? You don't need to launch it just Proceed until the Review screen

@nLight I couldn't launch a multi container service. Here is a screen shot and I figured that I have to add some like this.
<ConfigurationMapRow>               <ConfigurationMapLabel>Network Name</ConfigurationMapLabel>               <ConfigurationMapValueWithDefault                 value={getNetworkTypes(appConfig.networks)}               />               {action}             </ConfigurationMapRow> but I couldn't start a new multicontainer service. How can I start that.

@nLight Here is how my services section look. I think there is something wrong with the setup I am not able to see the + icon to add a new service.

@nLight any help would be great

@adityatandon007 hey, I think those are quirks of running the UI of fixtures. You can't really start anything without a cluster. But clicking into /some/group-with-pods/podEFGH should let you see the screen. As I can see from the fixture there is a network with a name.

Hey @adityatandon007 is there anything I could do to unblock you?

@nLight didn't get what you mean by

is there anything I could do to unblock you?
By the way I didn't get the time to look into this. I will try to get to that review screen without starting the cluster. Because I don't know how to set up a cluster.

@nLight nope couldn't get to the review screen for a multicontainer pod.

@nLight I think I would need to give the cluster url in my config file. But for that I would have to setup an aws cluster. I don't want to go throught that setup. Is there any dummy cluster url that I can add to config file to start the cluster. If there is no other way to set up a cluster url then, some else is free to pick this up!

@adityatandon007 you don't need a cluster for that. Just click into this /some/group-with-pods/podEFGH then Configuration tab - that's the same screen

@nLight I got that but I want to ask that I am not able to see the network type and I know because the appConfig.networks  might be null but then how can I get the name of the network.

@nLight I have added the Network Name  under Networking but it still needs to call the function to get the name of the network like it is doing in case of Network Type but I can't get to output since I don't know what key it is to used to get the network name since my appConfig.networks is null. I know I'm almost there.

Let me see...

Could you please create a work in progress PR?

It's in! Thank you for the effort! 🍁


Display exceptions in watch mode

In the current version, uncaught exceptions are not showed in watch mode.
This is a 1 line fix to print it and continue the watch.

Cherry-picked into master

coooool

Published in<EMAIL_ADDRESS>

how romantic: http://cl.ly/C4pS

Interesting: https://github.com/Yipit/clay.js/issues/4


Move skipped builds to local worker

@Mic92 While watching builds being processed it looks like the skipped builds aren't parallelized?

https://github.com/buildbot/buildbot/blob/c6dc292f38d0353d33215f8809c617afa47e11a6/master/buildbot/worker/base.py#L645 according to here the worker should be able to attach to multiple builders. We don't set max_builds our selves. I'll try to experiment with this and see if i can get it parallel.

Thanks.
I remembered that the lix fork did something with the skipped builds, they actually removed it, not sure if that is relevant here or not.
https://git.lix.systems/lix-project/buildbot-nix/commit/156e6e3deae3edd06de7de1dc1dd20de0b6a4c74

The idea of this was to avoid having to do network I/O or take a lock for skipped builds. But I never properly tested how much of a performance difference this makes.


Refactor AUSF app, sbi, processor, consumer

Hi @TYuan0816
Please revise your PR based on the review comments.
Thanks.

Hi everyone. Thanks for your comments. I have made the corrections you suggested. Please help me check it. Thank you.


test: add more tests for golang filter extension.

Commit Message:
add more tests for golang filter extension.
Additional Description:
already covered the existing APIs, it's a good start for the new coming APIs.
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/review @wangfakang

@wangfakang Thanks, all fixed.


!game_stats not handling special characters

!game_stats doesn't seem to handle special characters like ℍ
The following command returns all of the game_stats data when it should default to just the first 10 rows.
!game_stats ℍ

This command with respect the limit given but will match to any game name
!game_stats ℍ 10

The full cause of this issue was not tested for but it can be assumed to lie somewhere in these lines from game_stats.js
if (isNaN(args[1]) && typeof args[1] !== 'undefined') {
        data = await database.getStatisticsForGame(
          args[1].replace(/["']/g, ''),
          args[2]
        );
      } else {
        if (typeof args[1] !== 'undefined') {
          data = await database.getGameStatistics(parseInt(args[1]));
        } else {
          data = await database.getGameStatistics(10);
        }
      }

It would also be worth having a discussion of enforcing a 50 game stat limit in any none DM channel.

After further testing it seems to be an issue with the name look up. ℍ appears to match to a large number of game names in the same way a single character like b does. Looking at the code I can't see how this is an issue with our implementation but perhaps it's something we have to tell postgres to handle a bit better?

c214ae4 fixes this issue.
The command assumed that if the parameter given is not a number, as is the case for the emoji, that a second parameter for a limit is supplied. As it isn't, no limit is passed to the database. The command now assumes a default limit of 20 in the case of providing a name, or enforces a limit up to 20 if another number is provided as a limit.

The emoji still shouldn't be matching to game names though?

It should, but as I don't store game names in a unicode field long enough
for the H emoji to fit, it just matches everything.
On Sun, 18 Nov 2018 at 14:52, Isabel Jenkins<EMAIL_ADDRESS>wrote:

The emoji still shouldn't be matching to game names though?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ciarancrocker/omnium/issues/19#issuecomment-439698648,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI_9uEhDp_7mDdpKlBbrvdFCHybkurEks5uwXQ1gaJpZM4YnvLH
.


Remove space at the start of response

You wouldn't know how to remove the space from in front of response that gets added by default would you? I'm trying to pass true and false to a boolean to use as an automated type of switch, but I need to remove the space from in front of the response output. It works without the space, but with the output automatically adding it, it messes it up lol.

This is all that's holding me up lol.

I apologize that I haven't got back to you to fix this easy issue. I was working on a new version of the node and making sure this was fix in various places but I couldn't push it due to an unforseen issue on how comfy works.
Just to double check is possible the model itself has a empty space on the model file template in ollama this is an error they made when creating the template file and almost everyone copied it so it could be that too. Other than that I will inspect the current node to find where the problem is I am already applying some regex and strip to the responses so is weird it still comes like this

I apologize that I haven't got back to you to fix this easy issue. I was working on a new version of the node and making sure this was fix in various places but I couldn't push it due to an unforseen issue on how comfy works. Just to double check is possible the model itself has a empty space on the model file template in ollama this is an error they made when creating the template file and almost everyone copied it so it could be that too. Other than that I will inspect the current node to find where the problem is I am already applying some regex and strip to the responses so is weird it still comes like this

No worries, completely understand. I got a bit caught up in my spritesheet maker which is way less intensive lol. I may have to add a node to format the text worst case scenario because it may just baked into the model like you said.


fix : specify index type in searchBuilder

In gitlab by @MohamedHamouGisaia on May 23, 2017, 14:47

In gitlab by @MohamedHamouGisaia on Jul 6, 2017, 16:56
added ~1918909 label

In gitlab by @MohamedHamouGisaia on Jul 6, 2017, 16:56
added ~1918367 and removed ~1918909 labels

In gitlab by @MohamedHamouGisaia on Jul 6, 2017, 16:56
closed

In gitlab by @MohamedHamouGisaia on Jul 6, 2017, 16:56
added ~1918368 and removed ~1918367 labels

In gitlab by @MohamedHamouGisaia on Jul 6, 2017, 16:56
removed ~1918368 label


HA 0.115.0b0 causes a semver failure

Describe the bug
Installing Home Assistant 0.115.0b0 results in the following error message during startup:
2020-09-07 15:54:26 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up alexa_media platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 193, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/alexa_media/sensor.py", line 120, in async_setup_entry
    return await async_setup_platform(
  File "/config/custom_components/alexa_media/sensor.py", line 79, in async_setup_platform
    alexa_client = class_(
  File "/config/custom_components/alexa_media/sensor.py", line 433, in __init__
    if semver.compare(HA_VERSION, "0.113.0") >= 0
  File "/usr/local/lib/python3.8/site-packages/semver.py", line 108, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/semver.py", line 786, in compare
    v1 = VersionInfo.parse(ver1)
  File "/usr/local/lib/python3.8/site-packages/semver.py", line 656, in parse
    raise ValueError("%s is not valid SemVer string" % version)
ValueError: 0.115.0b0 is not valid SemVer string

Looks like it doesn't like the string.
To Reproduce
Install the latest version of the Alexa Media Player from HACS (v2.11.0), restart Home Assistant, monitor logs.
Expected behavior
No errors during startup.
System details

Home-assistant (version): 0.115.0b0
Hassio (Yes/No): Yes
alexa_media  (version from const.py or HA startup): 2.11.0
alexapy (version from pip show alexapy or HA startup): 1.14.1
Amazon 2FA is enabled (y/n). We will not debug login issues if unanswered: Yes

Logs
Please provide logs. We'll be most likely asking for them anyway.
2020-09-07 15:54:26 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up alexa_media platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 193, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/alexa_media/sensor.py", line 120, in async_setup_entry
    return await async_setup_platform(
  File "/config/custom_components/alexa_media/sensor.py", line 79, in async_setup_platform
    alexa_client = class_(
  File "/config/custom_components/alexa_media/sensor.py", line 433, in __init__
    if semver.compare(HA_VERSION, "0.113.0") >= 0
  File "/usr/local/lib/python3.8/site-packages/semver.py", line 108, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/semver.py", line 786, in compare
    v1 = VersionInfo.parse(ver1)
  File "/usr/local/lib/python3.8/site-packages/semver.py", line 656, in parse
    raise ValueError("%s is not valid SemVer string" % version)
ValueError: 0.115.0b0 is not valid SemVer string

Additional context
Add any other context about the problem here.

Thanks. Apparently HA doesn't actually follow semver syntax.

Ah, yeah.  It looks like all the betas are tagged that way on purpose.

2.11.1 should solve this. Please let me know if it doesn't.

Looks good, thanks!


Can't read old models saved with saveRDS.lgb.Booster on version 4.3.0 of lightgbm

Description
I have a long term project that uses lightgbm for the modeling.  I made a bunch of models throughout the years which I access again from time to time to predict on new data. I used to save my models with lightgbm::saveRDS.lg## Description I have a long term project that uses lightgbmfor the modeling.  I made a bunch of models throughout the years which I access again from time to time to predict on new data. I used to save my models withlightgbm::saveRDS.lgb.Booster()and read them withlightgbm::readRDS.lgb.Booster().  I’m trying to run the code with lightgbmversion 4.3.0, and now my code fails telling me thelightgbm::readRDS.lgb.Booster()` doesn’t exist anymore:

I’ve found in this issue (https://github.com/rstudio/bundle/issues/55) that the function was remove and that we should use readRDS instead. When I do so, the model is loaded with no error:

However, the object created is unusable:

Is there a way to load old model created with lightgbm::saveRDS.lgb.Booster() back into the new version of lightgbm?  I guess either by making readRDS works with files created with saveRDS.lgb.Booster() or add back the lightgbm::readRDS.lgb.Booster() into the package for this kind of use cases?
Rerunning all my old models would be difficult and downgrading my version of lightgbm do not sound ideal either.
Reproducible example
See print screen above.  Sorry I work in a close environment and can't export objects or code.
Environment info
LightGBM version 4.3.0 install from CRAN,
working in R 4.4.0 on a windows machine

Thanks for using LightGBM.

what version of R was the model file created with?
what version of LightGBM?

The answer to this question will depend on your answers to those questions.
And can you please edit the description as follows?

replace those screenshots with the exact text of code + logs

see https://meta.stackoverflow.com/questions/285551/why-should-i-not-upload-images-of-code-data-errors

remove duplicated information + fix formatting

see https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax

Thanks for your response.

what version of R was the model file created with?
what version of LightGBM?

Both those questions are hard to answer as I don't keep track of the version of neither R or my packages when I run a code.  The objects date from before 2023, therefore I can expect R to have been around version 4.2.* and lightgbm at version 3.3.*.  It's the best I can do, sorry.

About the formatting, sorry about the doubling of the first sentence, it's a copy-paste error that happened after I reread my post.  As for the screenshots, I don't want to get into an argument here.  I know my question wasn't up to standard.  Print screens are to avoid and reproducible example that are easily copy-pasted and machine readable is what to aim for.  However, I work in a corporate environment that as no other way out but screenshots.  It was pretty much that or nothing.  Making a reproducible example would imply building two different environments, one old to save a model and one new to read it, which is time consuming.  I (wrongly?) expected my question to be relatively strait forward and simple and a direct consequence of deprecating readRDS.lgb.Booster().  Feel free to close this issue if you disagree with me or if the description and answers I provided you are not sufficient to understand the problem.

Both those questions are hard to answer as I don't keep track of the version of neither R or my packages when I run a code.

Ok thanks. When I have time to look at this again, I'll try to create a reproducible example for you using R 4.3.0 and {lightgbm} 3.3.1.
If you plan on using .rds for long-term storage of any R objects, you should keep track of this information. It is possible that a future version of R will not be able to read .rds objects produced by an older version.
See:

the version arg at https://www.rdocumentation.org/packages/base/versions/3.6.2/topics/readRDS
https://stackoverflow.com/a/56704704/3986677

I work in a corporate environment that as no other way out but screenshots

It's fine. I manually retyped the code from your screenshots. It took me just 30 seconds, and now this issue will be discoverable by others facing the same problem and searching for those error messages.


RSA_Cipher

Submission Title
RSA_Cipher
Submission Type
Algorithm
Submission Description
RSA is an asymetric algorithm for public key cryptography created by Ron Rivest, Adi Shamir and Len Adleman.
Submission Directory
Algorithm => Cryptography_Algorithm => RSA_Cipher => RSA_Cipher.extension
Contributors

[ ]  C++ -
[ ]  C -
[ ]  Python -
[ ] Java -
[ ] JavaScript -
[ ] TypeScript -
[ ] Golang -
[ ] Ruby -
[ ] php -
[ ] Swift -
[ ] C# -

Can I implement in python?

Allow me to implement this in java.

Can you please assign this to me in C/c++

can I implement in python

Your code already contains a python file for RSA cipher ,do you still want
me to contribute for this?
I can contribute using some other language like C++
On Sun, 11 Oct 2020, 01:13 Priyanshu Tiwari<EMAIL_ADDRESS>wrote:

Assigned #665 https://github.com/ahampriyanshu/algo_ds_101/issues/665
to @maityankan10 https://github.com/maityankan10.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ahampriyanshu/algo_ds_101/issues/665#event-3863058247,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANM7E5K32MYGPEEPQSAZSMLSKC2NXANCNFSM4SD43O7A
.

please respond asap can I contribute in c++

please respond asap can I contribute in c++

go ahead :+1:

thanks everyone :v:


Any plan to support multiple servers?

Would be nice to give pheanstalk an array of server ip's and ports and have it just randomly (or even not so randomly) choose which one to send the job to.  This would eliminate all of the developers who use pheanstalk from having to roll their own sharding solution.

Any movement on this? I'm investigating writing my own round-robin implementation outside of it.
+1 on interest.

Hi there,
Maybe I have not understand the first message,
But I don't think that Pheanstalk have to be able to send to many server.
Pheanstalk have to be able to send to one server only. If you want to add some round-robin or "load-balancing", you have to add a Load Balancer such as Keepalived, Cisco ACE or any IP Load Balancing..
What do you think about it ?
Thomas.

+1 for this as well.  Really need to have beanstalk setup with some sort of failover on AWS.

Pinging all the people who subscribed to this topic, still interested on this? I would like to discuss some ideas, I'm implemeting a HA version of this client (not the same from #110), before making a pull request i would like to discuss the approach I'm taking.
Here there is a piece from a docstring explaining how my approach works, totally open to criticism and new ideas.
/**
 * HighAvailability Pheanstalk client.
 *
 * This client allows using multiple beanstakd servers. The behavior is experimental and subject
 * to change. There is a different behavior for producers and consumers:
 *
 *  - Producers will put all jobs on the first working connection the found, when an error is found
 *    they will start using the next connection available and keep using that connection until
 *    another error happens.
 *
 *  - Consumers will start reserving jobs from the first working server, until an error happens or a
 *    timeout is reached (a timeout is forced if not specified, explained later) in any of both
 *    scenarios the consumer will open a connection to the next server and try to reserve from it.
 *
 * Timeouts are forced on every reserve command to force consumers to jump from one server to
 * another when they cannot find a job, this way we can be sure that at any time all server queues
 * are going to be consumed and no tasks are going to be left forgotten. If your queues are never
 * emptied, then you should try adding more consumers or this implementation will not work at all
 * for you.
*/

@hugochinchilla I feel that this is a niche that, while nice, should not be part of base classes of this library.
If you want to add new class that use composition to implement it I'll review a PR.
Based on the age of this issue i'm closing it for now, if you want to start working on a PR just open one with a reference to this issue. Feel free to open it as soon as you start (just add WIP in front of the PR name).


VSCode 1.92 YAML grammar breaks MDX syntax highlighting

Does this issue occur when all extensions are disabled?: No, VSCode 1.92 YAML grammar updates cause a regression when embedded in the MDX syntax highlighting.

VS Code Version: 1.92
OS Version: Pop!_OS 22.04

Steps to Reproduce:

Launch VSCode with code --disable-extensions
Install the unifiedjs.vscode-mdx extension
Create a file named whatever.mdx with the following content:---
foo: 1
---

This is highlighted as a YAML string

# This is highlighted as a YAML comment

This is highlighted as a YAML error

Notice how the syntax highlighting is broken. This was not the case in VSCode < 1.92.

Removing all lines from the frontmatter fixes it:
---
---

This is highlighted as a YAML string

# This is highlighted as a YAML comment

This is highlighted as a YAML error

Proper MDX syntax highlighting

A workaround is to install the redhat.vscode-yaml extension. This uses a different YAML grammar.
Related issues:

https://github.com/wooorm/markdown-tm-language/issues/10
https://github.com/mdx-js/mdx-analyzer/issues/462
https://github.com/mdx-js/mdx-analyzer/issues/464
https://github.com/orgs/mdx-js/discussions/2520
https://github.com/orgs/mdx-js/discussions/2521

Seems like this came in with the new yaml grammar https://github.com/microsoft/vscode/pull/219833
I believe that grammars that use source.yaml need to switch to use a begin/while pattern for the front matter like in this change: https://github.com/microsoft/vscode-markdown-tm-grammar/pull/162
@RedCMD Is there anything that can be done on the yaml grammar side to handle this more elegantly?

@mjbvz I'll see what I can do
its caused by this line being too greedy
https://github.com/RedCMD/YAML-Syntax-Highlighter/blob/8de6455b5306c4963eeaef692ee3375d80a23f0e/syntaxes/yaml-1.2.tmLanguage.json#L15
however I can't just restrict it, otherwise the cross-document VERSION feature won't work
there definitely is quite a few extensions using the old FrontMatter code https://github.com/search?q=lang%3Ajson+A-{3}&type=code 200+

I think I've got something working
found a few VSCode TextMate bugs at the same time

you can test out the latest changes by installing my pre-release YAML extension

For the recovery release, I'm going to roll back to the old grammar. In insiders, I'll fix forward and grab your fix @RedCMD.

is there a way to roll back to the previous grammar while we await a release?
I can tell our users to install the redhat extension, but a fix that required no action of them would be better

The release went out yesterday.

Thank you, I upgraded and this seems fixed

I missed the upstream fix when I pulled in the YAML grammar for 1.93: https://github.com/microsoft/vscode/issues/224978#issuecomment-2336966993

Verified in
1.93.1
Commit: 38c31bc77e0dd6ae88a4e9cc93428cc27a56ba40
Date: 2024-09-11T17:20:05.685Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.15.0-119-generic

In 1.94.1 vs code corrupt YAML again (

Thanks for reporting @FragsterAt. I'm reverting the YAML grammar in the main branch, so the next VS Code insiders will not have this issue. We've missed the chance to get the fix into 1.94.2, but if we have a 1.94.3, then I'll make the change there too.

Actually, @FragsterAt I'm not seeing the same issue:

Ah, I had an extension installed that was providing a different yaml grammar. I still see the bug.

Actually, @FragsterAt I'm not seeing the same issue:

Please look in https://github.com/vuejs/language-tools/issues/4831
In  1.93.1 All was ok, but in 1.94.1 it broken again.
.vue files can have "sections" in different languages, yaml section broke highlighting (now a little different in comparison to 1.93.0, but still corrupted) in next blocks
with yaml block in 1.94.1 ()

without yaml block in 1.94.1

https://gist.github.com/FragsterAt/213f76e1e6a4592ac071236849af6392

Moving to the normal October milestone.

Grammar revert is already done in main.


Containerize iscsid

In GitLab by @paul.laffitte on Oct 26, 2020, 16:59

In GitLab by @paul.laffitte on Oct 27, 2020, 23:38
mentioned in commit 779df6814e2b8495def85103eb0b1c8d2d8b543a

In GitLab by @paul.laffitte on Oct 28, 2020, 22:19
mentioned in commit a90d306b239e26838bf957e6eb72b9e0ca6e8372

In GitLab by @paul.laffitte on Oct 28, 2020, 22:33
mentioned in commit e0033e61a47ba7b9b16d81cb590c7dc7106394f1

In GitLab by @arcln on Oct 28, 2020, 22:36
mentioned in commit e5ee1309fb5f971cedd1a409c30926bb6c38ff1c


TimeSeries doesn't work on python 3.6.4

ts.get_daily('VFINX')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.6/site-packages/alpha_vantage/alphavantage.py", line 173, in _format_wrapper
    self, *args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/alpha_vantage/alphavantage.py", line 158, in _call_wrapper
    return self._handle_api_call(url), data_key, meta_data_key
  File "/usr/local/lib/python3.6/site-packages/alpha_vantage/alphavantage.py", line 74, in _retry_wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/alpha_vantage/alphavantage.py", line 259, in _handle_api_call
    response = urllib.request.urlopen(url)
AttributeError: module 'urllib' has no attribute 'request'
I think that you need to go back to importing urlopen directly. 1.3.6 works correctly.

Hey @rkhwaja I am hating that urlib package. In some python versions is urlib.request.urlopen and in the other one is urlib.urlopen. I'll see if I can fix it later.
BR Romel

I think I will have to ditch urllib for python requests, some people had some issues with the library and I reckon requests should be better. I will try to do it over the weekend.
BR
Romel

With a rewrite using Requests library, you should allow to pass a session object like we did in https://pandas-datareader.readthedocs.io/en/latest/cache.html
so it will be possible to cache queries for a better network usage.
see https://github.com/pydata/pandas-datareader/blob/f3d754c484bcecb6e1d28859feeb446c73db283b/pandas_datareader/_utils.py#L42
and https://github.com/pydata/pandas-datareader/blob/f3d754c484bcecb6e1d28859feeb446c73db283b/pandas_datareader/base.py#L56

Hi @rkhwaja , Can you test with the newest version?
Hi @femtotrader, I still haven't added the session handler., but will later.
BR
Romel

Please be clear and careful about caching! If I'm using this for real-time data, I definitely don't want to hit a cached endpoint!


Test suite cannot add bundle from version catalog

Expected Behavior
Test suites should be able to use bundles from version catalogs. See #18632
Current Behavior
Build script fails to compile when attempting to add a dependency to a bundle.
Context
I'm working on a Kotlin project that has a bundle of related testing libraries defined.  I want to add a dependency on that bundle to the default test suite, or to every test suite if more than one are declared.
I was able to workaround the issue, and have included the workaround in the example below.
libs.versions.toml
[versions]
kotest = "5.5.4"

[libraries]
kotest-framework = {module = "io.kotest:kotest-runner-junit5", version.ref = "kotest"}
kotest-datatest = {module = "io.kotest:kotest-framework-datatest", version.ref = "kotest"}
kotest-assertions = {module = "io.kotest:kotest-assertions-core", version.ref = "kotest"}
kotest-property = {module = "io.kotest:kotest-property", version.ref = "kotest"}

[bundles]
kotest = ["kotest-framework", "kotest-datatest", "kotest-assertions", "kotest-property"]

build.gradle.kts
testing {
    suites {
        configureEach {
            if (this is JvmTestSuite) {
                useJUnitJupiter()

                dependencies {
                    // This does not work.
                    implementation(libs.bundles.kotest)

                    // Workaround
                    libs.bundles.kotest.get().forEach {
                        implementation(it)
                    }
                }
            }
        }
    }
}

Output
e: /Users/victorcardona/test-suite-version-catalog/build.gradle.kts:14:21: None of the following functions can be called with the arguments supplied:
public operator fun DependencyAdder.invoke(dependencyNotation: CharSequence): Unit defined in org.gradle.kotlin.dsl
public operator fun DependencyAdder.invoke(dependency: Dependency): Unit defined in org.gradle.kotlin.dsl
public operator fun DependencyAdder.invoke(files: FileCollection): Unit defined in org.gradle.kotlin.dsl
public operator fun DependencyAdder.invoke(dependency: Provider<out Dependency>): Unit defined in org.gradle.kotlin.dsl
public operator fun DependencyAdder.invoke(externalModule: ProviderConvertible<out MinimalExternalModuleDependency>): Unit defined in org.gradle.kotlin.dsl

Steps to Reproduce

Create a version catalog that contains a bundle.
Configure a test suite in build.gradle.kts and attempt to add a dependency to the bundle.
See example at https://github.com/VictorCardona75/test-suite-version-catalog

Your Environment

Build scan URL:
JDK: Azul Zulu 18.0.2
Gradle: 7.6
Kotlin: either 1.7.10 or 1.8.0

Thank you for providing a valid reproducer.
The issue is in the backlog of the relevant team and is prioritized by them.

Confirmed working with 7.5 and broken with 8.0.

This was an intentional change as part of improving the type-safety of the dependencies block. You should use implementation.bundle(...) instead. See https://docs.gradle.org/current/userguide/upgrading_version_7.html#changes_to_dependency_declarations_in_test_suites.


Icon, button, appbar rendering issues in internal web app

See: b/197480079

Everything is pointing at https://github.com/flutter/flutter/issues/86809. I'm going to deduplicate and increase the priority of https://github.com/flutter/flutter/issues/86809.


logging php 500 Internal server error

I'm trying to follow https://cloud.google.com/logging/docs/setup/php
I installed composer, created ~/composer.json
  {
          "require": {
                  "google/cloud-logging": "^1.29.1",
                  "google/cloud-core": "^1.52.7",
                  "google/gax": "^1.26.0"
          }
  }

Ran "composer install"
Followed all the steps in https://cloud.google.com/logging/docs/setup/php
Created http://sidesfamilyeuchre.net/phpEuchre/helloPHP.php
  <?php
    use Google\Cloud\Logging\LoggingClient;
    $logging = new LoggingClient([
        'projectId' => 'useful-memory-360712'
    ]);
    // $logger = $logging->psrLogger('phpEuchre');
    // $logger->error('some mysql exception');
  ?>

When I hit the page I get 500 with no error message.

Hi @ssides , thanks for raising this issue.
Error: 500 usually depicts something is wrong with the server. Can you try again and see if you're consistently / seldom getting the error, or you've stopped getting the error?

Closing this due to inactivity.


Add FEDAC Schools

All FEDAC schools in Catalonia use the http://fedac.cat domain.
Some schools offer the scientific/technological baccalaureate. (For example: http://www.fedac.cat/horta.php)
Email adresses are of style<EMAIL_ADDRESS>or<EMAIL_ADDRESS>

@rdvdev2 Pull request merged. Thank you!


WIP - v1.0.0

Work in progress..
Changes:

changed markup syntax from [...] to {...}
new layout syntax to align/stack renderables
@with_repr and install_term_repr
improved threading in progress bar display
improved string highlighting and added to tprint
refactored/improved errors and logging code
custom Theme support

In progress

grid: https://github.com/FedeClaudi/Term.jl/pull/101
[x] merge formatting from grid PR
[x] Table - finish code
[x] non-destructive layout workflow

To do:

Add tests for new functionality
Update docs + new functionality

Codecov Report

Merging #102 (9c98c35) into master (613502b) will decrease coverage by 5.13%.
The diff coverage is 50.55%.

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
- Coverage   86.91%   81.77%   -5.14%
==========================================
  Files          24       27       +3
  Lines        1421     1619     +198
==========================================
+ Hits         1235     1324      +89
- Misses        186      295     +109

Impacted Files
Coverage Δ

src/Term.jl
77.77% <ø> (ø)

src/macros.jl
100.00% <ø> (ø)

src/theme.jl
0.00% <0.00%> (ø)

src/_repr.jl
6.81% <6.81%> (ø)

src/repr.jl
23.33% <23.33%> (ø)

src/__text_utils.jl
82.73% <33.33%> (-4.09%)
:arrow_down:

src/renderables.jl
71.05% <37.50%> (-13.33%)
:arrow_down:

src/inspect.jl
93.82% <66.66%> (ø)

src/layout.jl
77.64% <67.02%> (-15.54%)
:arrow_down:

src/_inspect.jl
91.17% <83.33%> (ø)

... and 19 more

Continue to review full report at Codecov.

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

As a side note, considering the number of breaking changes introduced in https://github.com/FedeClaudi/Term.jl/pull/101, I think it should be released as 1.0.0 instead of 0.4.0 according to semver :wink:

@johnnychen94  an update on the multi-renderable layout discussed here: https://github.com/FedeClaudi/Term.jl/pull/101#issuecomment-1140968184
There's now a Compositor that accepts an Expr specifying a layout. The Layout expression uses symbols for names of different elements and (w, h) for their size:
layout = :(A(50, 10) * B(50, 10))
compositor = Compositor(layout; hpad=3, vpad=1)

the Compositor parses this and creates place holders of the appropriate shape and lays them out as described in the layout expression (note the *):

you can then go and change individual elements of the composition:
compositor.elements[:B].renderable = Panel(; width=50, height=10)  # the size needs to match what we specified for :B

This is particularly powerful since you can use the full layout "syntax" (vstack, lvstack..) and use padding too to create complex layouts:
layout = :(
    rvstack(
        (A(25, 5)/B(25, 5)), C(50, 5); pad=2) * D(25, 19) * lvstack(
            E(25, 5), F(25, 5), G(50, 5); pad=1
    )
)

The fact that it creates these placeholders for you means you can rapidly prototype the layout without having to mess around with its contents.  The other main advantage is that you can interactively change the "content" without having to worry about the layout any-longer. So if you iteratively clear out the console and print the compositor with updated renderables you can make something with a live update. This (a bit contrived) shows it:
using Term
import Term.Compositors: Compositor
import Term.Repr: termshow
import Term.Layout: vstack
import Term.Console: clear
import Term.Tables: Table
using Term.Boxes

using UnicodePlots
import MyterialColors: Palette, blue, pink

boxes = [
    :SQUARE, :SQUARE_DOUBLE_HEAD, :MINIMAL, :MINIMAL_HEAVY_HEAD,
    :HEAVY_EDGE, :HEAVY_HEAD, :DOUBLE, :DOUBLE_EDGE,
    :SIMPLE, :SIMPLE_HEAD
]
colors = getfield.(Palette(blue, pink; N=10).colors, :string)

layout = :((A(50, 15) * B(50, 15)) / C(103, 10))
compositor = Compositor(layout; hpad=3, vpad=1)

for i in 1:10
    X = rand(5, 2)

    _plot = Panel(
        string(
            scatterplot(X[:, 1], X[:, 2], title="{red}My Scatterplot{/red}", width=30, height=8)
        ); width=50, height=15, box=:NONE
        )

    _lineplot = string(
            lineplot(rand(100), title="{blue}My lineplot{/blue}", width=95, height=8)
        )

    data_table = Table(round.(X, digits=3);
                header = ["X", "Y"],
                style=colors[i],
                header_style="bold green",
                box=boxes[i])

    _data = Panel(data_table; width=50, height=10, justify=:center, padding=(2, 2, 0, 0),
                style="red dim", box=:HEAVY, title="Data table", title_style="default red bold")

    compositor.elements[:A].renderable = _plot
    compositor.elements[:B].renderable = _data
    compositor.elements[:C].renderable = _lineplot
    clear()
    print(compositor)
    sleep(.25)
end

https://user-images.githubusercontent.com/17436313/171287714-75b3c5b6-45c3-475b-80a0-e745fddcc2c5.mov
Still some bugs to fix and a few quality of life improvements, but I like where this is going!

Wow, I'm very impressed with the recent improvements :clap: .
Here are a few general comments:
Are you familiar with the julian syntax:
a = if something
    [1]
else
    nothing
end

instead of :
if something
    a = [1]
else
    a = nothing
end

? It saves a few chars and avoids repetition.
Regarding the new compositor, using Expr is very powerful. Should we implement auto layout (with a layout keyword) as I tried when defining grid or should we use Table as  you suggested in https://github.com/FedeClaudi/Term.jl/pull/101#issuecomment-1141425872 ?
I'd like to be able to use display(compositor) (high level) instead of println(compositor) (low level, should be debug or repr ?).
Shouldn't we use an OrderedDict for the compositor elements ? I think it should retain insertion order (julia Dict are not ordered), so that update! can be called using the same Expr sequence.
I guess Compositor (and maybe update! ?) should be exported by Term.jl.
Some minimalist example I'd wish for:
using UnicodePlots, Term

panel(p; kw...) = Panel(p isa String ? p : string(p, color=true); fit=true, style="hidden", kw...)

plots = collect(
  panel(lineplot(1:2, width=20, height=10, color=c))
  for c in (:red, :green, :blue)
)

compositor = Compositor(plots; hpad=3, vpad=1, layout=(1, 5))  # auto-layout, should insert hidden placeholders auto-magically

display(compositor)

I didn't know that syntax! I've been using a ? b : c for simple expressions but not for multi-lines expression. I like that!
Thanks for the comments, let me try to address them:

Regarding the new compositor, using Expr is very powerful.

Indeed, though I would eventually like to support matplotlib's string layout syntax as it's more intuitive, the expression syntax is more powerful.

I'd like to be able to use display(compositor) (high level) instead of println(compositor) (low level, should be debug or repr ?).

Okay, it can have the same behavior as the renderables.

Shouldn't we use an OrderedDict for the compositor elements ? I think it should retain insertion order (julia Dict are not ordered), so that update! can be called using the same Expr sequence.

I'm not sure what you mean at the end?
update!(compositor, :A, Panel(; width=50, height=15))

update! specifically names the compositor element to update, it doesn't need the layout Expr? update! updates the content of the Compositor, it doesn't trigger its rendering. At render time the compositor uses the stored layout expression to render whatever content is available.  We can use OrderedDict but I'm not sure what the benefit would be here?

Should we implement auto layout (with a layout keyword) as I tried when defining grid or should we use Table as you suggested in https://github.com/FedeClaudi/Term.jl/pull/101#issuecomment-1141425872 ?

Compositor is different from the AbstractRenderables like Panel in Term - indeed it's not a renderable. That means it can't be used for layout with other renderables (compositor / panel won't work). There's pros and cons for this choice, but the main reason is that it works differently since you can update it's content at will and in a modular way, so I think it should be a different kind of object (though a Renderable(compositor) method sould probably be available.
So I think it would still be good to have a Grid renderable which builds upon Table to create a standard renderable. At the same time it would be nice to have Compositor(plots; hpad=3, vpad=1, layout=(1, 5))  as an option too. There might be a bit of overlap there but I think it's best to have both things as separte options:

if you want a renderable, use Grid - can't do fancy layouts like compositor
if you want a fancy layout + renderable: Renderable(compositor)
if you want something that eats renderables and you can update piece meal use compositor.

Perhaps the Grid layout could be built through Renderable(compositor), but at that point I think Table makes more sense (also it lets you add headings).
Sorry for the wordy reply, but I guess my thoughts on the matter are not 100% clear so I just wanted to lay them out and get your feedback since it's been so helpful!

I won't have time to work on it before the end of the week, so you can go ahead by taking the code that was removed from https://github.com/FedeClaudi/Term.jl/pull/101.
I'd be happy to review when ready, and add test scenarios though before the final merge !

Got distracted with the markdown parser, but adding the grid functionality now. Adding some edge cases where the behavior is not what I'd expect.
import Term.Layout: grid, PlaceHolder

rens = repeat([PlaceHolder(10, 5)], 9)
grid(rens; pad=(2, 1), aspect= 1.5, placeholder=PlaceHolder(10, 5; style="red"))

Really looking forward to these updates!
I've been playing around with iterator wrappers that create nested progress bars and just updated to this branch, but am now not getting any progress bars printed, e.g. in the following example, no progress bar is shown:

pbar = ProgressBar()
job = addjob!(pbar; N = 100)
start!(pbar)
for i in 1:100
    update!(job)
    sleep(0.03)
    i % 25 == 0 && println("We can print from here too")
end
stop!(pbar)

If you're already aware of this, please ignore my comment 😅

HI Lorenzo,
Thanks for trying out the newer version. The way progress bars work has changed a bit so that a sleep statement is not necessary any longer (previously you needed one in the loop to ensure that the progress bar would be rendered, now it happens in a separate thread).
As a consequence you need to call render on the progress bar:
using Term.Progress

pbar = ProgressBar()
job = addjob!(pbar; N = 100)
start!(pbar)
for i in 1:100
    update!(job)
    sleep(0.03)
    i % 25 == 0 && println("We can print from here too")
    render(pbar)
end
stop!(pbar)

Note that if you use @track or with you don't need start!, stop! and render. These are the recomended ways to use the progress bars since they're more robusts to errors during the progress bar display (also they're more convenient):

pbar = ProgressBar()
job = addjob!(pbar; N = 100)
with(pbar) do
    for i in 1:100
        sleep(0.03)  # only needed to slow down the display
        i % 25 == 0 && println("We can print from here too")
        update!(job)
    end
end

Let me know if it works and if you have any other question or problem!

@t-bltg I think this is pretty much ready to merge in master. The tests could be made a bit more thorough, but otherwise the functionality I wanted to have for the next release is there - when we merge I can start working on the docs update. Let me know what you think!

Hey Federico, thanks for the quick response!
Unfortunately, I can't use with as I'm writing a wrapper for iterators (e.g. for x in withprogress([1, 2, 3]); ...; end) and I need to keep the progress bar alive while the loop body is running and only stop it once the loop has finished. I'll look at the source code of with to see how to do that safely 👍

I see. That's a use case that I would like for Term to be able to handle, if you come up with a good solution perhaps you can contribute a PR?

@t-bltg I think this is pretty much ready to merge in master.

Am trying to review this, but the test don't pass locally:

details
PANEL - FIT - measure: Test Failed at [...]/Term.jl/test/__test_utils.jl:69
  Expression: length(unique(widths)) == 1
   Evaluated: 2 == 1
Stacktrace:
 [1] macro expansion
   @ [...]/share/julia/stdlib/v1.7/Test/src/Test.jl:445 [inlined]
 [2] testpanel(p::Panel, w::Int64, h::Nothing)
   @ Main [...]/Term.jl/test/__test_utils.jl:69
PANEL - FIT - measure: Test Failed at [...]/Term.jl/test/__test_utils.jl:69
  Expression: length(unique(widths)) == 1
   Evaluated: 2 == 1
Stacktrace:
 [1] macro expansion
   @ [...]/share/julia/stdlib/v1.7/Test/src/Test.jl:445 [inlined]
 [2] testpanel(p::Panel, w::Int64, h::Nothing)
   @ Main[...]/Term.jl/test/__test_utils.jl:69
PANEL - FIT - measure: Test Failed at [...]/Term.jl/test/__test_utils.jl:69
  Expression: length(unique(widths)) == 1
   Evaluated: 2 == 1
Stacktrace:
 [1] macro expansion
   @ [...]/share/julia/stdlib/v1.7/Test/src/Test.jl:445 [inlined]
 [2] testpanel(p::Panel, w::Int64, h::Nothing)
   @ Main [...]/Term.jl/test/__test_utils.jl:69
Test Summary:              | Pass  Fail  Total
PANEL - FIT - measure  |  171     3    174
ERROR: LoadError: Some tests did not pass: 171 passed, 3 failed, 0 errored, 0 broken.
in expression starting at [...]/Term.jl/test/07_test_panel.jl:182
in expression starting at [...]/Term.jl/test/runtests.jl:43
ERROR: Package Term errored during testing

Hey thanks @johnnychen94 !
I've added a method to tprint to parse MD objects so that they get printed out nicely. When using Term's repr functionality docstrings are parsed with the new functionality.
So I'm not sure what you mean by this:

default docstring print with Term's tprint

Also, regarding:

, and would probably be troublesome when REPL docstring priting manner is changed when people indirectly load

The new repr functionality is optional, you need to run install_term_repr() like for the logging and stacktraces.

@t-bltg
That's weird, the tests are passing on both of my machines and on the GH actions. Are you using a particularly narrow terminal window?

$ echo $LINES $COLUMNS
22 80
$ julia -e 'using Pkg; Pkg.test("Term")'  # OK
[...]

$ echo $LINES $COLUMNS
71 195
$ julia -e 'using Pkg; Pkg.test("Term")'  # KO
[...]

The tests should be independent on the size of the terminal.

They should already be. I've tried with different windows sizes and the emulated terminal for GH actions should be 80 columns wide. Let me see if I can replicate it.
Wait from what you say, is it failing for the wider terminal?

Wait from what you say, is it failing for the wider terminal?

Seems so:
$ echo $LINES $COLUMNS
22 167  # OK
22 195  # KO

I see. For default docstring print I mean overring how help> map displays

Yeah no I won't be touching that.

but I think it's better to leave it as as an optional (opt-in) feature. Indeed, I wouldn't suggest it in the documentation

I'd say it's the dev's responsability to avoid doing that. I'd be for letting people know that this exists but adding a warning that if your code is going to be used by others you might want to avoid doing this, what do you think?
@t-bltg  thanks, I will look into this.

I'd be for letting people know that this exists but adding a warning that if your code is going to be used by others you might want to avoid doing this, what do you think?

Indeed, it's okay as long as it's an opt-in feature.
Furthermore, if our GSoC project progresses smoothly, then we'll eventually just move every overriding function provided in install_term_repr to the new package (say, TermDisplays) and just advertise using TermDisplays instead.

I see. That's a use case that I would like for Term to be able to handle, if you come up with a good solution perhaps you can contribute a PR?

For sure will open a PR if I get it working in a clean way 👍

It remains to be determined what's causing tests failures on wide displays before merging into master.

I'm onto that.

Panel("."^1500; fit=true) is a simple bug reproducer.

@t-bltg this: https://github.com/FedeClaudi/Term.jl/pull/102#issuecomment-1153999960 is fixed now correct?

@t-bltg this: https://github.com/FedeClaudi/Term.jl/pull/102#issuecomment-1153999960 is fixed now correct?

Yes, it's correctly rendered when displaysize(stdout) == (22, 195) which was the term size in the original bug. I've tested with alternative sizes (resizing my term window either very flat, very tall or squarish on a 1920x1080 display), and it seems to render correctly now.

@FedeClaudi, I forgot: would it be possible to export the symbol grid from the Term.jl module ?

sure I'll add it in Master

sure I'll add it in Master

FYI, I've a PR pending for grid fixes, so it'd be great to wait a few hours before releasing 1.0.0.

yeah I need to write the docs, it might take me a couple of days I'm a bit busy this week. But either way no rush, I can wait to release until you're ready too.

@t-bltg  docs for the new grid functionality you've contributed: https://fedeclaudi.github.io/Term.jl/dev/layout/grid/

Thanks for the hard work ! I'm also thinking of creating an exhaustive grid image (as what you've done with https://github.com/FedeClaudi/Term.jl/blob/master/readme.png) at the top of the UnicodePlots.jls README.md, when 1.0.0 is released: Term + UnicodePlots = :heart: .

I'm wondering if we should shamelessly steal / adapt the @layout macro from Plots.jl (defined in RecipesBase.jl) in order to fine tune grid placements as in https://docs.juliaplots.org/latest/gallery/gaston/generated/gaston-ref55/#gaston_demo_55.

I originally wanted to have something like this: https://matplotlib.org/3.5.0/tutorials/provisional/mosaic.html#string-short-hand
for Compositor but found it very hard to implement.
But yeah it  would be great to have!

Also, we could maybe hide placeholders by default unless the whole grid consists of placeholders ...

yes

Also for Compositor I will add that you can give the elements' size in fraction of total width:
layout = :(A(.7, 1.0) * B(.3, 1.0))

will create a layout where A takes up 70% of the width.

Ha, thanks for the hints on Compositor (I'm still learning the internals of Term.jl :)) and mpl. I'll have a look at this within the day, I'm sure we can come up with something efficient, simple and robust.


Who have the 1.13c D2GS source code ?

Hello as the title say it, i'm searching a working 1.13c D2GS source code that can compile on VS 2017.
It's impossible to get that damn D2GS Multi-threaded to work on windows.
I spent 4h to fix it, i re-used official PVPGN.Pro protocol for authentication and all, the Custom D2CS now connect fine with the official PVPGN i'm able to connect/create characters and all, the realm is fine as well.
I did recompile D2GS (MultiThreaded) and disabled the Patch_D2.mpq file size as i have custom patch, and this still doesn't do anything ! that trash error "main: Failed to startup net connector".
Also i don't really want the MultiThread i would like the original 1.13c D2GS that is used by MagicBuilder but i didn't find the source of this D2GS, is it public at least ?
I would like the source because in addition to my mod i would like to introduce server sided administrator command and a real WorldEvent trigger.
Any help would be appreciated !
Ps: I don't want to use linux ! wtf.

There is only 1.09d source code for D2GS.exe https://github.com/pvpgn/d2gs109
Source code for 1.10 was not published, and this D2GS.exe used for all later versions (for example, checksum is the same for 1.10 and 1.13c).
As I understand, @tesseract2048 takes 1.09d sources and made some modifications to make it working with his D2GE engine, that handles game logic https://github.com/pvpgn/d2gs/tree/master/D2GS
His D2GE uses LibD2Server that was written by Marsgod — it's a patch in assembler for d2server.dll and C++ source of d2warden.dll.
Source for LibD2Server you can find in @tesseract2048 repository or in every build from Marsgod https://pvpgn.pro/download/d2gs/
There is no source for d2server.dll, this is a gateway to communicate with Diablo 2 game.

Thanks for the reply, looking to fix the "failed to start net connector" error, i'm so close to have it working on windows 🥇

I was able to compile all the required binaries. This is really a quest how to do it and to understand how to setup. I spent the whole day, all programs are running, but I still could connect to a server with No realm are available in Diablo 2.
May be you can make it work or someone else. I just will describe below all that I know about it.
Complete binaries: pvpgn_d2gs_tesseract.zip
All the binaries are required to run together. So it's not enough only modified D2CS as I assumed before — modified PvPGN and D2DBSDotNet are also required.
Compile
D2GS.exe, D2GE.exe
https://github.com/pvpgn/d2gs (in this repository I removed Patch_D2.mpq size verification for D2GS.exe)
It can be compiled in Visual Studio 2015/17, if in a project property set Linker > Input > Ignore All Default Libraries from Yes (/NODEFAULTLIB) to No.
PvPGN.exe, d2cs.exe
https://github.com/tesseract2048/pvpgn
I take my old package with PvPGN 1.8.5 http://harpywar.com/?a=articles&b=2&c=1&d=30 and replace files from the repository.
Open PvPGN.sln and add new files: bnetd/bnetd_log.c, bnetd/bnetd_log.h, fdwatch_iocp.h, fdwatch_iocp.c. And remove three fprintf functions defininions, cause it was already defined somewhere else in my Visual Studio environment.
D2DBSDotNet.exe
https://github.com/tesseract2048/D2DBSDotNet
Download MySql.Data.dll from http://mysql.com and update the unresolved MySql.Data dependency for the Visual Studio project. Then build the project, nothing special.
How to setup
Mostly similar with https://pvpgn.pro/d2gs_installation.html
PvPGN

conf/bnetd.conf
(I don't know the purpose of these vars, I assume it's optional)

mirror_ip = <IP_ADDRESS>
vip_experience = 0
#eqn_hashstr =

Run PvPGN.exe or PvPGNConsole.exe

D2DBSDotNet

conf/d2dbs.conf

mysql_host=<IP_ADDRESS>
mysql_database=d2dbs
mysql_user=root
mysql_pass=

Create a new MySQL database d2dbs and execute script init.sql
Run D2DBSDotNet.exe, it has no any visible interface, you should see a running process in a task manager. It can be installed as a Windows service with -s or --service parameter.

D2CS
Run d2cs.exe or d2csConsole.exe
D2GS
Run d2gs/D2GS.exe, it also has no any visible interface, but a process should be running.
You should see in d2gs.log:
07/16 19:24:05.766 D2GSConnectToD2xS: Connected to D2DBS Successfully
07/16 19:24:05.767 D2GSSendClassToD2DBS: Send connection class packet to D2DBS
07/16 19:24:05.867 D2GSConnectToD2xS: Connected to D2CS Successfully
07/16 19:24:05.867 D2GSSendClassToD2CS: Send connection class packet to D2CS
07/16 19:24:05.967 D2GSNetRecvPacket: CS socket become writable
07/16 19:24:05.967 D2GSNetRecvPacket: DBS socket become writable

As I understand, a new process D2GS.exe should start for every created Diablo 2 game.
Note: D2GS.exe will exit immediately if D2CS or D2DBSDotNet is not running.

Hmm ok thanks, i will test this when i can and i will keep you updated.

What do you mean? You do not need to compile it yourself, I already shared pvpgn_d2gs_tesseract.zip in the previous post.

I've heard that Tesseract's D2GS has a built in firewall, so that we can block 0xFF01 and 0x1400 packets, without the false positives from iptables.
How can I use it?

@HarpyWar  why the pvpgn_d2gs_tesseract.zip cant run in windows xp for example? Send the error win32 not valid application.

@geckotdf you should really question why you would use Windows XP in 2018.

Likely because it's Windows XP. I'm guessing that @HarpyWar compiled the binaries without Windows XP support which occurs by default with newer versions of Visual Studio.

@RElesgoe I see, well I try in 32 bits Windows 7 and works, so its related to the Win XP support.-
By the way continue working on - And I believe its something related with "Mirror_IP" in the bnetd.conf
If you any IP different from the host or # in Mirror_IP you have two errors:
[error] sd_accept: s_addr: 301a8c0, mirror_addr: xxxxxx
[fatal] sd_tcpinput: d2cs sock: 1000
[info ] on_d2cs_authreply: d2cs <IP_ADDRESS> authed [debug] conn_set_realm: [1000] set to "D2CS" [info ] realm_active: realm D2CS actived
And the Realm its created, but in practice we know thats not true, I believe because we have a "fatal" error in the tcpinput.
But if you put the local IP where D2CS its running in the Mirror_IP you get:
[error] sd_accept: s_addr: 301a8c0, mirror_addr: 301a8c0
No fatal error, but the realm dont say its created.-
I believe the mirror IP its where the realm its working - In any of the two ways if you try to connect via "Multiplayer" options you can connect but when try to create a game you get
handle_bnet_packet: [572] (logged in) got error handling packet type 0x1cff, len 32
In my opinion its very clear the problem its the compilation of the D2CS.-

Ignore this message [error] sd_accept: s_addr: 301a8c0, mirror_addr: 301a8c0, it's not actually an error. Do you ever get a message saying could not create realm?

In regards to handle_bnet_packet: [572] (logged in) got error handling packet type 0x1cff, len 32, it seems that tesseract is blocking open game types.

[fatal] sd_tcpinput: d2cs sock: 1000 isn't an actual error either, but it does indicate that d2cs made a connection to pvpgn. What is not working when it says realm activated?

When say "realm activated" happend what @HarpyWar say:
No realms are available
You can register, login, create the character but after that say no realms available and logout - Then when you try to login again happend the same No realms are available

What do you have in realms.conf?

"D2CS"			"PvPGN Closed Realm"		<IP_ADDRESS>:6113
Use the same IP where my D2CS its running.-

@RElesgoe for example this is an output trying to login:
Apr 23 16:43:59 [error] sd_accept: s_addr: 1401a8c0, mirror_addr: 301a8c0 Apr 23 16:43:59 [info ] conn_create: [1052][1000] sessionkey=0x5ade1fa4 sessionnum=0x00000002 Apr 23 16:43:59 [debug] sd_accept: [1052] client connected to a bnet listening address Apr 23 16:43:59 [info ] handle_init_packet: [1052] client initiated file download connection Apr 23 16:43:59 [trace] conn_shutdown: [1052] connection allready closed Apr 23 16:43:59 [info ] file_send: [1052] sending file "bnserver-D2DV.ini" of length 364 Apr 23 16:43:59 [debug] sd_tcpinput: [1052] read returned -1 (closing connection) Apr 23 16:43:59 [info ] conn_destroy: [1052] closed file connection Apr 23 16:44:01 [debug] file_read_attrs: loading "var\users/gecko" Apr 23 16:44:01 [fatal] conn_login: [1028] "gecko" from <IP_ADDRESS> logged in (right password) Apr 23 16:44:01 [info ] _client_loginreq2: [1028] "gecko" logged in (correct password) Apr 23 16:44:07 [info ] _client_setemailreply: [1028] init account "gecko" email to<EMAIL_ADDRESS>Apr 23 16:44:07 [debug] sd_tcpinput: [1028] read returned -1 (closing connection) Apr 23 16:44:07 [info ] conn_destroy: [1028] "gecko" logged out Apr 23 16:44:07 [info ] conn_destroy: [1028] closed bnet connection

Last thing I try it was put the server in my job and connect me from outside, to avoid any LAN problem between the PVPGN and client.-
No working, the same error "realms not available"

There is only 1.09d source code for D2GS.exe https://github.com/pvpgn/d2gs109
Source code for 1.10 was not published, and this D2GS.exe used for all later versions (for example, checksum is the same for 1.10 and 1.13c).
As I understand, @tesseract2048 takes 1.09d sources and made some modifications to make it working with his D2GE engine, that handles game logic https://github.com/pvpgn/d2gs/tree/master/D2GS
His D2GE uses LibD2Server that was written by Marsgod — it's a patch in assembler for d2server.dll and C++ source of d2warden.dll.
Source for LibD2Server you can find in @tesseract2048 repository or in every build from Marsgod https://pvpgn.pro/download/d2gs/
There is no source for d2server.dll, this is a gateway to communicate with Diablo 2 game.

I've try compile 1.09 src from Your repo but i've got linker error LNK2001 __imp_QueryInterface any idea why ?

Rookie question.
Can someone tell me how to add mid Mod (several dll) to the D2gs to make it work? What should I change in the source code to read it? I appreciate the help.
Or what parameters should I modify?


Integrate Join Game functionality into the frontend

When navigating to the Live Duels tab there is a Join Game button. This needs to connect with the smart contract to call the join_game function in order for a player to join the game

Can I work on this

@web3technologies I am a smart contract developer, with experience in Frontend development as well.
I would like to be assigned this issue
What I will do is to integrate the "Join Game" functionality with the smart contract's join_game function in the Mancala game.
My knowledge in smart contract will help in reviewing code to locate the join_game function within the smart contract. Understanding its purpose, inputs, and outputs is crucial for successful integration.
Connection using a Frontend Framework and interacting with the contract will be the next thing.

Hi @kateberryd @estherbreath I will assign both of you this issue. You can both work on it and submit your PR's. We will review both of them prior to merge.

@web3technologies Please who do I contact on telegram?

You can contact me at @zachcook

I am going to close this issue now @estherbreath @kateberryd
we are trying to move fast internally so we can have better issues that aren't blockers for y'all at the OnlyDust hack coming up in 2 weeks.


fix: CONTRIBUTING.rst "Automatically execute examples" bug fix

Fixes tiny bug on the CONTRIBUTING.rst
Currently, just a line above https://docs.litestar.dev/latest/contribution-guide.html#creating-a-new-release, the docs are cut-off after "This is equivalent to:"

After the fix:

Thanks @FergusMok!

@FergusMok Can you change the PR to target main?

@all-contributors add @FergusMok to docs


Configurable GoogleService-info.plist name

Regarding the PR #33, it would be nice if the filename could be configurable as in our company we have different names for each production stage

Good idea! Feel free to submit a PR and we can take a look.

Just submitted a PR

Closing this issue as the PR has been merged in 0.2.2


Implement web support in lottie-react-native

Problem
We recently sponsored support for .lottie files in lottie-react-native. However, we currently use react-native-web-lottie to provide a unified interface for lottie across native and web.
However, react-native-web-lottie is unmaintained, and the patch we have for it is basically a complete rebuild of this simple lib.
Now we'd like to start using .lottie files, and there is an existing dotlottie player for web, but at this point I don't think it makes sense to continue with react-native-web-lottie.
Solution
Let's build web support into lottie-react-native for both lottie JSON and .lottie file formats.

cc @wojtus7

Hi!
I would like to take care of this task 🥰
Overall my idea is to:

Create web examples in lottie-react-native repo
Add web implementations to make web versions of the example app work the same as android/iOS/windows equivalents

Then create PR for lottie-react-native and after it is hopefully merged use it in Expensify (also removing react-native-web-lottie library)

Hey, @roryabraham as I am working with @wojtus7 on this task I have some questions about this issue:

I want to make sure, that we are on the same page: We want to add the dotlottie library to the lottie-react-native library (as a dependency), and then use it to support the web version.
Do we need plain .json support? The dotlottie library supports only .lottie files. I can use lottie-react library, to support JSON files as well.

I want to make sure, that we are on the same page: We want to add the dotlottie library to the lottie-react-native library (as a dependency), and then use it to support the web version.

Yes

Do we need plain .json support? The dotlottie library supports only .lottie files. I can use lottie-react library, to support JSON files as well.

Yes, we need plain .json support as well to upstream our efforts.

Thanks!

@wojtus7 @kosmydel what's the latest here?

Hey @roryabraham, short update:
I've prepared a draft of the PR for the upstream.
I think that we should first upgrade the lottie-react-native library in the App to the latest version (6.3.1) because it has some breaking changes. We have to do that anyway, so the sooner is better. In that way, I will be able to focus only on the web implementation, instead of upgrading the library.
FYI I'm going to be OOO next week, so @wojtus7 will handle this issue during my absence.

Still in development. @kosmydel was OOO

@roryabraham  Since I'm going OOO for two weeks and @kosmydel already kinda took lead in this task I think this is reasonable to change assigned person ☺

Hey @roryabraham,
I'm sorry for the delay. I was catching up with my other tasks.
During our internal SWM review, I got a suggestion, that I can use expo to create a webpack.config.js file for the web instead of writing a custom one. It would be shorter and cleaner like here, and it might support hot library reloading in the demo. If we have time, I can have a look at it tomorrow and see how much time it would take. Otherwise, if it is urgent, we can continue with our current approach.

@kosmydel I don't to see how that's relevant to this task at all. This ticket is focused on building a web implementation in lottie-react-native.

Upstream PR was merged three days ago

6.4.0 was released with web support yesterday, thanks @kosmydel and @matinzd!

https://github.com/Expensify/App/pull/30772 was merged, so once that goes to production this issue can be considered complete

@ishpaul777 thanks for noticing that! A PR with a fix.


[Settings][Find My Mouse] Improve UX for disabled animations

Summary of the Pull Request
Like the OP I also hit Find My Mouse having disabled animations but the message didn't provide accurate info to find the option and I ended up using the good old "Advanced System Settings".
Figured out that the option location differ on W10/W11.
This PR aim to tweak a little bit the message and provide a link to open the related Windows setting page based on OS version.

PR Checklist

[x] Closes: #34224
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Manually tested the button.
Verified that the URIs bring to the correct Windows settings page for both W10 and W11.

I wonder if making "system settings" part of the text here a hyperlink would end in a nicer UI

Wdyt?

"Windows settings" is used for theme related options.

WDYT about this?
Animations are disabled in your system settings. Open Windows settings

"Windows settings" is used for theme related options.

WDYT about this? Animations are disabled in your system settings. Open Windows settings

I meant just to remove the "Open settings" button and make "system settings" clickable. The phrasing is ok imho

Oh, my bad 😅
I think the current implementation (message + button) is in line with other infobars used in the settings app: see CNF and TextExtractor.
But can change this if needed: only concern is that localize that may be tricky.


[ci:component:github.com/gardener/etcd-backup-restore:0.5.2->0.6.0]

Release Notes:
Added the `embedded-etcd-quota-bytes` flag to allow configuring the backend quota size of the embedded etcd instance used during restoration of data.

The golang version has been upgraded to `v1.12.0`.

In case of storage provider is not configured, i.e. backup disabled, we skip the backup dependent sanity checks.

Unnecessary data validation will now be skipped, allowing for quicker etcd restarts.

swapnilgm [1:30 PM]
Hi Rafael,
FYI, I have triggered release pipeline for etcd-backup-restore which will create PR on gardener. I have supporting gardener PR ready on https://github.com/gardener/gardener/compare/master...swapnilgm:optimize-etcd-validaiton
Unfortuantely i have to leave now for a while. So, might not be able to raise PR on image chagne branch immediately. Will do it by EOD.


Support equality checks for ranges

Be able to equality check ranges, intervals and >= tests.

Closed with v0.32.0.


NPE when looking at nearby caches

Version b489b69ce090b64e6fd7059ee1e0d1fbb592ab69 does not work for me, it crashes when querying nearby caches:
E/AndroidRuntime( 8495): FATAL EXCEPTION: Thread-20
E/AndroidRuntime( 8495): java.lang.NullPointerException
E/AndroidRuntime( 8495): 	at cgeo.geocaching.cgData.saveCache(cgData.java:1215)
E/AndroidRuntime( 8495): 	at cgeo.geocaching.cgeoapplication.storeWithMerge(cgeoapplication.java:593)
E/AndroidRuntime( 8495): 	at cgeo.geocaching.cgeoapplication.addSearch(cgeoapplication.java:557)
E/AndroidRuntime( 8495): 	at cgeo.geocaching.cgBase.searchByAny(cgBase.java:2175)
E/AndroidRuntime( 8495): 	at cgeo.geocaching.cgBase.searchByCoords$5bd01a3a(cgBase.java:2182)
E/AndroidRuntime( 8495): 	at cgeo.geocaching.cgeocaches$geocachesLoadByCoords.run(cgeocaches.java:1874)
W/ActivityManager(  208):   Force finishing activity cgeo.geocaching/.cgeocaches

@rsudev, did the merge you committed work perfectly for you?
Edit: I confirm that it works fine for me with d46438381be1e4bf4186e0a426ebed66655266b9 (the version before the merge).

This should be fixed at multiple places:

The field should be initialized with CacheType.UNKNOWN instead of null.
CacheType getById() and getByPattern() should never return null, but CacheType.UNKNOWN instead.
The cache merge should handle CacheType.UNKNOWN in addition to null.

I've changed some places to avoid a null value in the cacheType property (because with the UNKNOWN type it should never be null again). I'm not sure whether this is complete but I can retrieve online cache lists again.

@samueltardieu perfectly? Surely not, it still cannot read my mind :)
I must admit I didn't even test it (as Bananeweizen did the review and I was just the 'merge-monkey'),
but went caching with it yesterday and did not experience problems (but I almost never use nearby in the first place)

Closing. Please reopen in case of more problems in this area.


Scan by Scan Filtering replaced by Baseline Correction in File Summary, batch queue and saving project

Basic information

My operating system and version: Windows 10
My MZmine version: 3.2.48

What happened

Please describe what happened and how it differs from what you expected to happen.
Using "Show File Summary", the Scan by Scan Filtering module used is replaced by the Baseline Correction one (with default parameters) and propagated to the batch mode using "Open in Batch Queue".
Furthermore, if the project is saved and opened again, a baseline correction is ran instead of a Scan by Scan Filtering.
To reproduce: open a cdf file (but I'm pretty sure it's independent from the format), run the Scan by Scan Filtering module then "Show File Summary".

Please upload screenshots of any errors and your log/mzmine.log file.

If I can help for tests or further information, feel free to ask.
Have a good day!

Perfect, thanks ;)


Seperated to variables.less

Seperated less configurations and variables.

Why not importing the variables directly in the rzslider.less file? I don't see why we need an app.less file...

Of course, that's a solution as well. It's a matter of opinion, if users should be able to import 1 default with configuration or 2.
I have made the changes, for me it's fine either way.
Thanks.

Thanks. Fast and efficient.


Feature: Full TUI with more options

"In the future" - options to build such a thing: https://thecustomizewindows.com/2016/12/build-colorful-command-line-tui-shell-scripts/

On reflection. This is better done by Wagyu Installer


Opening terminal on IOS Safari bug

Opening terminal in safari causes the website to redirect back to the homepage

Thanks for this report! I added WebGL recently with a fallback, but maybe it's broken or never worked. I'll look into it.

Thanks to the awesomeness of BrowserStack (https://www.browserstack.com/open-source) I've managed to get myself a license to test on Safari. I'll be getting to the bottom of this issue ASAP. Along with a bunch of other browser compat issues I am likely to find.

Fixed this. Safari seems to have some WebGL support, but not enough for xterm.


Prevent to deploy an already existing release

Prevent going on if release already exists
This could happen if you don't use a long date in ansistrano_release_version

Tests seem to be failing

Been thinking about this and it is a bit pointless
Yeah it might happen but preventing this would be your responsability. We use a long date by default so if you don't want to use this setup that's fine but again this only add some extra code for very little benefit


Strange bevahiour during Search and Replace

Hello
I'm using Tabular Editor 2.11.6 on a Windows Server 2016 machine.
I'm experiencing a strange behaviour when I'm using Search and Replace in the Expression Editor:
I have to replace a table name on several measures:

I click on a Measure
I open Search and Replace and replace all occurrences of a string
I click on the next measure, with the Search and Replace window still open
Click on the first line of the measure and click on Next
The first occurrence of the string is found
When I click on Replace the string will be replaced, but moved to the bottom of the code and a "Not Found" message  window opens
After Undo this step and restart the search and replace (With the Search and Replace window still open) it works

I was able to reproduce this effect on a Power BI Desktop file.
Can you please check this?
Thank you
Regards
Salvatore

Thanks for reporting this.
As a workaround, until this bug is fixed, consider using a script. For example, the following script will replace all occurences of the string "BEFORE" with the string "AFTER" in the DAX expression of the currently selected measure:
Selected.Measure.Expression = Selected.Measure.Expression.Replace("BEFORE", "AFTER");

If you select multiple measures, use a loop like so:
foreach(var m in Selected.Measures)
    m.Expression = m.Expression.Replace("BEFORE", "AFTER");

...or to simply loop through all measures across all tables use:
foreach(var m in Model.AllMeasures)
    m.Expression = m.Expression.Replace("BEFORE", "AFTER");

Fixed in 2.11.7.

Thank you for fixing this


Move to pip and requirements.txt for simplicity

• clean up the poetry and ci build
Slack Message

@gomagwu please specify the build similarly to how it used to be done...
- name: Set build version from git
  run: |
    poetry version `poetry version -s`.`git rev-list --count HEAD`

or maybe just the short hash from

git rev-parse --short HEAD

Hmm... I think option 3 in https://stackoverflow.com/questions/60430112/single-sourcing-package-version-for-setup-cfg-python-projects is pretty nice
Maybe instead of hard coding the version string there it could read from a file (__version.py perhaps) and that file could be appended to during the build process like:

echo -git rev-parse --short HEAD >> __version.py

The version number specified in at the point of creating the release on github is what is used when publishing on PyPI. That's why I didn't include version in the configuration files

@gomagwu let's go with this strategy:

.gitignore stablerandom/__build_version.py
during build echo git rev-list --count HEAD-git rev-parse --short HEAD > stablerandom/__build_version.py
stablerandom/init.py gets something like:

`version = `0.3`
if <__build_version.py exists>:
    build_version = read from file...
    version = f"{version}-{build_version}"
I'm not sure if - or . should be used as separator...

Went back to poetry


Add Observable Data Table Viewer OJS and OJS Inspector previews to docs

see #73 and #78
Pics:

docs updated. see this section: https://github.com/RandomFractals/vscode-data-table#observable-data-table--viewer-


Add check for valid index and value prop documentation

I added a check for invalid index values in toggleItem.
Sometimes when re-rendering the switch, I was getting an error because it would try to evaluate using invalid indexes that were being passed to toggleItem.

@jkdrangel I also updated the documentation to add the value prop. Let me know if you need me to change anything!

Also added some optional type options to allow RegisteredStyles


[dart2js/Dart 2.0] Fix behavior of type errors on partial instantiation in dart2js

It was decided in #31953 that the partial instantiation operator needs to check the bounds of type arguments at the point of partial instantiation, rather than waiting until the resulting closure is called. The VM is currently being updated with the correct behavior, but dart2js also has the incorrect behavior. For example, the following test should pass:
// Copyright (c) 2018, the Dart project authors.  Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
//
// This test checks that necessary type argument bounds checks are performed
// eagerly during partial instantiation, rather than being delayed until the
// partially instantiated closure is invoked.

import "package:expect/expect.dart";

class C<T> {
  void foo<S extends T>(S x) {}
}

void main() {
  C<Object> c = C<int>();
  void Function(String) fn;
  Expect.throwsTypeError(() {
    fn = c.foo;
  });
}

I've suggested P1 priority for this issue because fixing the semantics here is a breaking change.

Closing because this seems to be passing in dart2js on TOT


different email for last low credit notification

The email that is sent for the lowest notification should be different than the rest, as the last notification is the credit cutoff, and no more messages can be sent after this. The email should reflect this.

Ready for review.

One minor comment, otherwise looks good to me.

Changes made. Ready for re-review.

:+1: from me. I guess we should wait to see if @SteveBarnett has any comments though.

@SteveBarnett I've implemented your suggestions. Have a look and see if you're happy with it.

@rudigiesler Fixed my typos (sorry!). :+1:

:+1: once Travis is happy.


Wire up new data-plane-adoption-OSP-17-to-extracted-crc for zuul check

The new job was recently added with [1]
[1] https://review.rdoproject.org/r/c/rdo-jobs/+/50590
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running

recheck
should be able to run now, merged @ depends-on

recheck
the two failed at different points in the test
https://review.rdoproject.org/zuul/build/280fbaaf246b472da29ff4d36111b8fb
https://review.rdoproject.org/zuul/build/ebb5f1f9fe424509a1512b9081d8e9a8

/lgtm


fix(readme): simplify commander change warning

No need for betas anymore.

Oh, thanks for catching that!


⚠ SceneTime has degraded performance

In a4225f7, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1273 ms

Resolved: SceneTime performance has improved in 6b5df55 after 6 minutes.


Plugin Update & Roadmap Questions

Hi! It's been some time since there has been activity on this official plugin. Can you shed some light on the development plans? I think many users would be interested in updates is some of the following popular topics.

Local device control (#3 #165)
Proper handling of offline devices (#27 #65 #117 #232)
Adaptive Lighting (#28)
Many issues closed over "no activity" that are still outstanding (https://github.com/tuya/tuya-homebridge/issues?q=label%3Ano-issue-activity+is%3Aclosed).

Can you provide an update on what your priorities for this plugin are?

We were promised local device control a long time ago. What happened to that?


Lack of support for Organisations in Local Provider

Current Behavior

On logging in through the "Meshery" provider in the locally running instance, errors like these pop up:

In the terminal we can see requests made to http://localhost:9081/api/identity/orgs failing with 404 status code, with Status Code: 401.failed to get organizations as the response body.
Expected Behavior

Organisations to be fetched successfully
Screenshots/Logs

Environment

Host OS: Linux
Platform: Local

To Reproduce

Go to http://localhost:9081/provider
Click on Select your provider and select Meshery
Fill in login credentials and sign in
Get redirected to http://localhost:9081 and see the errors pop up in a while

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

For me its working fine with meshery provider

I checked for the issue again after pulling latest recent changes from master and found that it is working now. Thanks!


Poll the substream validation before polling Notifications

In tests, it can happen that Notifications doesn't produce any events which causes poll() to return Poll::Pending and the substream validation futures won't get polled.
Poll the futures before calling Notifications so results for substream validations are received even if Notifications is not producing any events.

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable-int
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2693911

bot merge


Minimum compiler version

When bumping from R4 to ^HEAD, Aegisub build will fail on Debian 11 and Ubuntu 20.04. We find out that BestSource introduced std::to_chars for FP numbers since https://github.com/vapoursynth/bestsource/commit/ddaac8dc7c280eea90a816416899a7f9de438764#diff-fe0470628acf368c8971852850ca82394169b0fd8a64c78426663f21ee0ef2a2R1663, and thus requires gcc 11 / clang 14 to build, which is relatively new.
See GCC Implementation Status - C++ 17 - Elementary string conversions, libc++ C++17 Status - Note - P0067
Yeah, it's one of the C++17 features implemented AFTER gcc/libstdc++ 9 claimed to have “complete support for C++17”
We suggest using the old code as fallback for older compilers. If not, then at least the required compiler version should be clarified in the README.

C++17 is C++17. Is there even a macro to detect this easily?

Is there even a define to detect this easily?

AFAIK, no. Literally I would read __clang_major__ and __GNUC__, but this would be cumbersome, admittedly…

Fixed I guess, feature test macros are your friend


How does the function get triggered?

Does this function read the NSG logs once, then stops?
Does it register some kind of hook to get triggered every blob update?
I have 4 independent functions setup, looking at 4 different storage accounts. They share an app service plan. But the seem to process once, then stop.
I don't know enough about how this is working to know if this is intended behaviour or i'm missing something.

Function is triggered by updates to the storage account.
On Sun, Feb 16, 2020, 7:32 AM Daniel Snelling<EMAIL_ADDRESS>wrote:

Does this function read the NSG logs once, then stops?
Does it register some kind of hook to get triggered every blob update?
I have 4 independent functions setup, looking at 4 different storage
accounts. They share an app service plan. But the seem to process once,
then stop.
I don't know enough about how this is working to know if this is intended
behaviour or i'm missing something.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AzureNetworkWatcherNSGFlowLogsConnector/issues/18?email_source=notifications&email_token=AAJ2XLG5TLOJ2N334QBQJRDRDDTZPA5CNFSM4KWAQ34KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IN2NIGQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ2XLH7ZLGKEK3PBFHX2CTRDDTZPANCNFSM4KWAQ34A
.


crystal deps remove

When you execute "remove" it will erase the current copy of the libraries but as soon as you run "install" again the dependencies will be re-downloaded unless you remove them from the Projectfile.
I'd rather expect instead that the directories are automatically deleted when a dependency is removed from the Projectfile and a "crystal deps install" is executed.
BTW, please rebase your changes in a separate branch before sending a PR so the merge gets more clean and concise. Notice you're including another change regarding the vagrant vm here.

Or something like "crystal deps clean" that would remove dependencies that are no longer defined in Projectile?

I implemented this in order to be able to switch between forked repos.  I.e. "weskinner/crystal-sdl", "farleyknight/crystal-sdl".  However, I see now that crystal deps update handles this already.  It does leave a weskinner-crystal-sdl dir in .deps/ but that's doing little more than taking up space.  It's another case that could be handled by a crystal deps clean or just as part of the install process like @waj suggests.
@waj: sorry about the dirty pull request.  I see what you mean about the separate branch / rebase strategy now.  I'll close this one out and create a new one when I rewrite the deps clean behavior.


remove SpotifyExampleContent from Package.swift

If you include SpotifyAPI using Swift Package Manager, it includes the SpotifyExampleContent in your resulting app, adding several megabytes of unused content into your app.

And what about people who use the example content?

Good point. Creating a new library without it is probably a better option. I'm curious who is actually using the example content in a shipping app though? Seems like it is more useful for unit tests, but perhaps I am misunderstanding its use.

SpotifyExampleContent is not intended to be used in shipping apps (although there may be limited use cases for this). It is primarily intended for previews and unit tests. In order for this target to be imported into another project to be used in previews, it must be included in a library. So, you can create another library that includes this target.

Implemented in version 3.0.0.


Cannot set cookies in the hooks.server.js for redirects

Describe the bug
I am unable to set the cookie. The cookie is always undefined. I believe that this is because the event is not returned, but a new response is sent instead. I wanted to to get rid of the extra cookie package. :)
 event.cookies.set("foo", "bar");
  return new Response(null, {
     status: 302,
     headers: new Headers({ Location: `/'})
  })

Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-frzqi9?file=src/hooks.server.js
Logs
No response
System Info
System:
    OS: macOS 12.3.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 791.16 MB / 64.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 18.3.0 - /opt/homebrew/bin/node
    npm: 8.11.0 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 105.0.5195.102
    Safari: 15.4

Severity
annoyance
Additional Information
No response

Hi leminsk8, you have to set the cookies directly in the set-cookies header.
If I'm not mistaken, you need to call resolve(event) for any effects of event to apply.

Thanks @s3812497... I thought so too...


Cant set padding from pie chart sections

I am trying to set some padding or saving for my pie chart sections/items. I did not like the idea of the clustered together. I tried doing this with the "sectionsSpace" property inside "PieChartData" bt it does not get effected. Please help.

Hi, as we mentioned here, you can set centerSpaceRadius to double.nan to let the library calculate the centerSpaceRadius, in that way padding will be applied.
Does it make sense?
Do you think we should use double.infinity instead?

BTW, you should use double.infinity instead or double.nan in the new version.
Cheers!

Check out the 0.10.1 version.
Cheers!


openstack_security group should not list Non-VPC RFC-1918 CIDR error

Openstack environment do not have a VPC so while listing securit group issues it should not give the following error,
"Non-VPC Security Group contains Private RFC-11918 CIDR" issue as this issue is applicable only to AWS environment. Attached screen shot of error type.

This is because the OpenStack security group auditor is subclassing the AWS Security Group auditor.
https://github.com/Netflix/security_monkey/blob/develop/security_monkey/auditors/openstack/openstack_security_group.py#L26
You'll need to either re-implement or override the AWS specific methods.
I think it would be worthwhile to make a generic auditor for network and have subclass the AWS, GCP, and OpenStack auditors against it with cloud-specific auditor functions.

@mikegrima  I am able to override the AWS specific function just by adding the below in openstack/openstack_security_group.py
def check_securitygroup_ec2_rfc1918(self,sg_item)

When I do this it is ignoring the RFC errors and fixies the issue.
Now thinking the 2nd way gneric auditor for network and subclass, I am not sure what exactly you are trying to say here. Can you please elobrate or specify a draft like how it can be implemented ?

@naggappan I added the rfc1918 override to #1066.

This has been fixed in #1066.


Standardize transaction success and failure messaging

There are currently four ways we show transaction status.

Home Page:

Failure:  An orange circle with an "i" followed by the tec code in darker orange.
Success: A green circle with a check mark

Ledger Page

Failure: An orange circle with an "i".
Success: Nothing

Account and Token Page

Failure: A darker orange pill with an "i" followed by "Fail - {tecCode}"
Success: A green pill with an checkmark followed by "Success"
Mobile: Just show icon for failure or success

Transaction Page

Failure: An orange circle with an "i" followed "Fail - {tecCode}" in darker orange.
Success: A green pill with an checkmark followed by "Success"

#Solution:
Modes

Default

Failure: A darker orange pill with an "i" followed by "Fail - {tecCode}". Will also link to documentation
Success: A green pill with an checkmark followed by "Success"

Shorthand

Failure: A darker orange pill with an "i" followed by "{tecCode}"
Success: A green pill with an checkmark

Pages

Home Page: Shorthand
Ledger Page: Default when Failure
Account and Token Pages: Default and just show icon on mobile and hide external link icon
Transaction Page: Default

Related to #190


Add React component for code blocks

https://github.com/rajinwonderland/react-code-blocks

We have this working.


what is the inference time or fps achieved on your hardware ?

could someone let me know how the model performs on your hardware, it would really help if you could let me know the inference time or fps achieved with the hardware, dataset and the backbone you have obtained the results from.

the inference speed is very similar to mask rcnn because our introduced bilayer mask head is lightweight.


Timeline content is missing headings which can allow screen reader user to quickly jump from one step to another

Clarity Version: 3.1.1
WCAG 2.1 Guideline: 2.4.6 Headings and Labels
Test Environment:

Test URL: https://clarity.design/documentation/timeline
OS: Windows 10
User Agents/Assistive Technologies: Firefox 76.0
Screen reader: NVDA 2020.1

Steps to reproduce:

Open the test URL.
Select any one of the tabs provided for horizontal, vertical and angular timelines.
Try to perform quick navigation by trying to jump to a particular step.

Expected result:
Screen reader user should be able to jump quickly from one step to another.
Actual result:
No provision provided to quickly jump through the steps. Note: Visual placement of content in timeline allows the user to segregate the steps, so sighted users and skim-through or jump through the steps quickly. For example, each step is spaced proportiontely under each status symbol and timestamp. Visually some content is marked bold to indicate some section.

Hi there, this is an automated message.
In an effort to dedupe and track accessibility reports consistently, this issue is being closed and tracked internally with our Accessibility team. It will still be triaged and addressed as part of ongoing work.


Outdated signature in packages/integration_test/lib/integration_test_driver_extended.dart documentation

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
The documentation for IntegrationDriver recommends this snippet in some places
await integrationDriver(
    onScreenshot: (String screenshotName, List<int> screenshotBytes) async {
      final File image = File('$screenshotName.png');
      image.writeAsBytesSync(screenshotBytes);
      // Return false if the screenshot is invalid.
      return true;
    },
  );

After moving to Flutter 3.10 the onScreenshot signature is caught by the dart analyzer. Changing it to this fixed the issue for me.
    onScreenshot: (String screenshotName, List<int> screenshotBytes, [Map<String, Object?>? args]) async {

Expected results
No error
Actual results
error: The argument type 'Future Function(String, List)' can't be assigned to the parameter type 'Future Function(String, List, [Map<String, Object?>?])?'. (argument_type_not_assignable at [xxx] test_driver/integration_test.dart:13)
Code sample
Code sample
See here: (https://api.flutter.dev/flutter/package-integration_test_integration_test_driver_extended/integrationDriver.html)

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

I can confirm this as well, and @githubmonkey's solution works.

I can confirm this as well, and @githubmonkey's solution works.