AttributeError

I installed bloodhound-python, using (the now obsolete) Python 2.7. It seems to work fast and with a nice low memory footprint, but during the group retrieval it crashes (possibly confidential information redacted):
$ uname -a
FreeBSD xxxxxxxx 12.1-RELEASE-p1 FreeBSD 12.1-RELEASE-p1 r356271 GENERIC  amd64
$ bloodhound-python -u xxxxxxxx -p xxxxxxxx -d xxxxxxxx -gc gc._msdcs.xxxxxxxx
INFO: Found AD domain: xxxxxxxx
INFO: Connecting to LDAP server: xxxxxxxx
INFO: Found 1 domains
INFO: Found 2 domains in the forest
INFO: Found 100000 computers
INFO: Found 100000 users
INFO: Connecting to LDAP server: xxxxxxxx
INFO: Connecting to GC LDAP server: gc._msdcs.xxxxxxxx
INFO: Found 100000 groups
Traceback (most recent call last):
File "/usr/local/bin/bloodhound-python", line 8, in
sys.exit(main())
File "/usr/local/lib/python2.7/site-packages/bloodhound/init.py", line 286, in main
disable_pooling=args.disable_pooling)
File "/usr/local/lib/python2.7/site-packages/bloodhound/init.py", line 85, in run
trusts_enum.dump_domain(collect)
File "/usr/local/lib/python2.7/site-packages/bloodhound/enumeration/domains.py", line 119, in dump_domain
_, aces = parse_binary_acl(domain, 'domain', ADUtils.get_entry_property(domain_object, 'nTSecurityDescriptor'), self
.addc.objecttype_guid_map)
AttributeError: 'ADDC' object has no attribute 'objecttype_guid_map'
leaving me with a users.json and groups.json, but an empty domains.json. Is this a know problem? Or maybe the domain I used it on has something unusual in it?

I think the issue is resolved but the repositories have not been updated. If you install the script by cloning the repository git clone https://github.com/fox-it/BloodHound.py.git and installing the tool with cd BloodHound.py && python setup.py install it should work.

That's a quick reaction! :-)
I should have mentioned I used a repository install as you guessed. I've built the latest version from source. It then gave me a different error:
[..]
INFO: Found 100000 groups
Traceback (most recent call last):
File "/usr/local/bin/bloodhound-python", line 11, in
load_entry_point('bloodhound==1.0.0', 'console_scripts', 'bloodhound-python')()
File "build/bdist.freebsd-12.1-RELEASE-p1-amd64/egg/bloodhound/init.py", line 286, in main
File "build/bdist.freebsd-12.1-RELEASE-p1-amd64/egg/bloodhound/init.py", line 85, in run
File "build/bdist.freebsd-12.1-RELEASE-p1-amd64/egg/bloodhound/enumeration/domains.py", line 126, in dump_domain
File "build/bdist.freebsd-12.1-RELEASE-p1-amd64/egg/bloodhound/ad/trusts.py", line 73, in init
File "/usr/local/lib/python2.7/site-packages/impacket/structure.py", line 84, in init
self.fromString(data)
File "/usr/local/lib/python2.7/site-packages/impacket/structure.py", line 149, in fromString
self[field[0]] = self.unpack(field[1], data[:size], dataClassOrCode = dataClassOrCode, field = field[0])
File "/usr/local/lib/python2.7/site-packages/impacket/structure.py", line 380, in unpack
return unpack(format, data)[0]
struct.error: ('unpack requires a string argument of length 1', "When unpacking field 'Revision | <B | [][:1]'")
I then switched to Python 3.7.6 and installed the latest impacket from github. Also again ldns3 version 2.5.1. A similar error occures:
Traceback (most recent call last):
File "/usr/local/bin/bloodhound-python", line 11, in
load_entry_point('bloodhound==1.0.0', 'console_scripts', 'bloodhound-python')()
File "/usr/local/lib/python3.7/site-packages/bloodhound-1.0.0-py3.7.egg/bloodhound/init.py", line 286, in main
File "/usr/local/lib/python3.7/site-packages/bloodhound-1.0.0-py3.7.egg/bloodhound/init.py", line 85, in run
File "/usr/local/lib/python3.7/site-packages/bloodhound-1.0.0-py3.7.egg/bloodhound/enumeration/domains.py", line 126,
in dump_domain
File "/usr/local/lib/python3.7/site-packages/bloodhound-1.0.0-py3.7.egg/bloodhound/ad/trusts.py", line 73, in init
File "/usr/local/lib/python3.7/site-packages/impacket-0.9.21.dev1+20200214.111603.5259ed0f-py3.7.egg/impacket/structure.py", line 84, in init
self.fromString(data)
File "/usr/local/lib/python3.7/site-packages/impacket-0.9.21.dev1+20200214.111603.5259ed0f-py3.7.egg/impacket/structure.py", line 149, in fromString
self[field[0]] = self.unpack(field[1], data[:size], dataClassOrCode = dataClassOrCode, field = field[0])
File "/usr/local/lib/python3.7/site-packages/impacket-0.9.21.dev1+20200214.111603.5259ed0f-py3.7.egg/impacket/structure.py", line 382, in unpack
return unpack(format, data)[0]
TypeError: ("a bytes-like object is required, not 'list'", "When unpacking field 'Revision | <B | [][:1]'")
When I add --collectionmethod Trusts, the error occurs at the start of the data collection.
I guess this is an impacket issue. Unfortunately, I am not a programmer. I have hacked impacket/structure.py to show some debugging info. It seems it breaks when data equals an empty list. I hacked it to rewrite data if it is an empty list but then it fails a bit later. But I guess this is an issue then for impacket, not for BloodHound.py.

Used the pip version this morning and got the same initial error. Cloning from the repo worked for me.

@gigawalt that seems like a different issue. I'm wondering if you somehow have a trust with a domain without an SID. I just pushed a potential fix for this, could you try updating from the latest commit on master?

Did so, which yielded this error:
AttributeError: 'ADDomainTrust' object has no attribute 'domainsid'

Whoops, that was a typo there, though it confirms my suspicion. I fixed the typo, if you could pull again and test, would be much appreciated.

\o/ works!
Thanks again for the help (and for making this tool available!)


Note about your absence

Please add a note about your absence.
If you have no time to develop, please allow more maintainers or at least add a note. Just answering to one of the many issues would help too.
Since you seem to be pushing in other repositories it seems like you simply ignore this project which is very sad.
Yo spent a lot of time on esdoc and it would be nice to know what's going on or if anybody can be of any help.
Right now I'm not sure if it's safe to assume that this project is dead. If so please tell us so we can start looking for other options.
With the highest respect THANK YOU

This is why I created esdoc2 -> https://github.com/esdoc2/esdoc2 - because, if you see my post with the owner here: https://github.com/esdoc/esdoc/issues/480

He refuses to add contributors, and in my opinion, is breaking the TOS with Github, because he created an organization called "esdoc" but it is his "personal project" -

THIS PROJECT IS DEAD - until he decides to commit to it, which given his history I would be surprised

He doesn't speak english, which makes things very hard to communicate with him

ESDOC2 - USE THAT PLEASE!  ESDOC2 is the contributor version, and is looking for contributors.  I know you have an issue on there, but I can't maintain esdoc2 myself.  I need help, and so if people could send me their npm logins, I will add them as contributors so they can publish to npm as well.

Thanks!!

@MattMcFarland IMHO esdoc2 is not production ready. I opened several issues.
https://github.com/esdoc2/esdoc2-plugins/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc
I want to move away from esdoc ASAP but right now I don't see an option without doing a lot of adjustments in my project.


No titles or previews

Hi!
On my mopidy instance the extension seems unable to fetch or parse titles and thumbnails.

(here using iris, same issue with moped)
This makes organizing videos pretty much impossible. Audio does play as expected. Is this a misconfiguration on my behalf, a bug or an API restriction? I'm not using API key or Music API.

On second thought this seems intentional, but is it possible to see the title of a video anyhow?

Thank you, I tried reinstalling the extension with the link you gave and now everything works! I did exactly nothing else.

Even playback history works now, and it didn't before. In fact, the extension is now even able to play videos unavailable in my country, albeit title and thumbnail are missing as before (as I imagine they can't be fetched from the public API in such scenario).
Question: will I be able to upgrade to a newer version once it's on pypi?

@LeafyLappa - not sure... I've just pushed a new version; give it a try

Everything seems to work, I'm closing this issue. Thank you!

Good to hear! Thanks for the feedback


Update spirv-tools

Update spirv-tools
Update spirv-tools installs to latest-ish

Updates spirv-tools to 2022.4 (and 2022.3 and 2022.2 because I have been slacking).

NB: the validation messages are busted: https://github.com/KhronosGroup/SPIRV-Tools/pull/4831#discussion_r997310010

But the rules are real, and we are breaking them, I would suggest just reading the spec:

VUID-StandaloneSpirv-Flat-04744: Flat mandatory on Fragment Inputs other than f32

they say "integer or double-precision floating-point" but presumably in reality the non-Flat default is an interpolation that is only supported for f32 (and vectors with f32 elements, right?)

VUID-StandaloneSpirv-Flat-06201: Flat/NoPerspective/Sample/Centroid illegal on Fragment Outputs
VUID-StandaloneSpirv-Flat-06202: Flat/NoPerspective/Sample/Centroid illegal on Vertex Inputs

Now that the whole deal in https://github.com/EmbarkStudios/rust-gpu/pull/928#issuecomment-1281199133 has been fixed, I've bumped the spirv-tools version further so we should get better errors on CI now.

Also moving https://github.com/EmbarkStudios/rust-gpu/pull/928#discussion_r997334264 here to be able to hide it:

Hah, you bumped all of these by 116 CI builds, so they are in lock-step, just with an offset from their initial setup at the very start? We could use that to our advantage and write a script that finds a set of 3 for a specific date - once we have e.g. the macOS CI build ID, we can derive the others from it and query the bucket for e.g. ".../${macos_id - 12}/${date}-*/install.tgz".
It's a bit messy but as long as we can do 3-4 curl requests in CI we can avoid hardcoding anything more than that one CI build ID (and even that we might be able to derive from the date if it's a reliably-daily build...).

(i.e. I thought maybe we could automate it - however it is annoying to do this on 3 different platforms when I only trust one of them to have the right tools for quickly script this, without e.g. reaching for Python)

Blocking (and marking as draft) this PR on:

#933

Which I opened separately to show the effects on tests in isolation, before updating spirv-val.

I talked to @Jake-Shadle and there's some good ideas around moving away from this model to something that both avoids so many builds of the C++ code and avoids dealing with potentially-wildly-different system-installed tooling. (At the very least the system-installed mode could move to linking against the .a libraries, which could then be switched from "weird GCP CI artifacts bucket" to our own builds)
For now I want to merge this for the test fixes (when using system-installed SPIRV-Tools on other branches).


Missing version <IP_ADDRESS>

Hi,
Nothing about <IP_ADDRESS> that is deployed actually when setting typeHandlerVersion to 1.0 on Windows

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

ID: 8201f331-0fa7-a801-9f4d-75d65cd77625
Version Independent ID: 86a66791-11bb-4f20-f77d-42fe58f5dcf4
Content: Install the Azure Monitor agent - Azure Monitor
Content Source: articles/azure-monitor/agents/azure-monitor-agent-install.md
Service: azure-monitor
Sub-service: agents
GitHub Login: @bwren
Microsoft Alias: bwren

@bd-clara Thanks for your comment. We will review the issue and get back to you shortly.

Thanks! Looks like this has been updated.
#please-close


UnicodeDecodeError: 'utf-8' codec can't decode byte 0xa0 in position 56: invalid start byte

Traceback (most recent call last):
  File "/usr/local/bin/adbsync", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/usr/local/lib/python3.11/site-packages/BetterADBSync/__init__.py", line 379, in main
    files_tree_destination = fs_destination.get_files_tree(path_destination, follow_links = args.copy_links)
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/BetterADBSync/FileSystems/Base.py", line 45, in get_files_tree
    return self._get_files_tree(tree_path, statObject, follow_links = follow_links)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/BetterADBSync/FileSystems/Base.py", line 33, in _get_files_tree
    tree[filename] = self._get_files_tree(
                     ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/BetterADBSync/FileSystems/Base.py", line 30, in _get_files_tree
    for filename, stat_object_child, in self.lstat_in_dir(tree_path):
  File "/usr/local/lib/python3.11/site-packages/BetterADBSync/FileSystems/Android.py", line 176, in lstat_in_dir
    for line in self.adb_shell(["ls", "-la", path]):
  File "/usr/local/lib/python3.11/site-packages/BetterADBSync/FileSystems/Android.py", line 87, in adb_shell
    adb_line = adb_line.decode(self.adb_encoding).rstrip("\r\n")
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xa0 in position 56: invalid start byte

After adbsync push failed. rerun adbsync push get this error.

I'm also getting this error

This might be to do with python version, I had this error, and after upgrading python and pip it went away

Still happens with Python 3.12.2. Using on MacOS via pipx
  File "/Users/x/.local/pipx/venvs/betteradbsync/lib/python3.12/site-packages/BetterADBSync/FileSystems/Android.py", line 176, in lstat_in_dir
    for line in self.adb_shell(["ls", "-la", path]):
  File "/Users/x/.local/pipx/venvs/betteradbsync/lib/python3.12/site-packages/BetterADBSync/FileSystems/Android.py", line 87, in adb_shell
    adb_line = adb_line.decode(self.adb_encoding).rstrip("\r\n")

I have non-English names on my device

This looks like a duplicate of #44 which has a test input to work with


sudobackend: line 10: /proc/19148/fd/1: No such device or address

E:\work\git-bash-shell>ver

Microsoft Windows [Version 10.0.16299.248]

E:\work\git-bash-shell>bash

cdyf@qil  /e/work/git-bash-shell (lxss)
$ git --version
git version 2.16.2.windows.1

cdyf@qil  /e/work/git-bash-shell (lxss)
$ arch
x86_64

cdyf@qil  /e/work/git-bash-shell (lxss)
$ ./bin/wrap/sudo echo test

/e/work/git-bash-shell/bin/wrap/sudobackend: line 10: /proc/3892/fd/1: No such device or address

Can you post the result of this?
$ echo $$
$ ./bin/wrap/sudo echo test
$ ./bin/wrap/sudo bash

I was try to add which bash to sudobackend
I got a result like /c/WINDOWS/system32/bash
I think is is Bash for Windows 10

cdyf@qil  /e/work/win-sudo (master)
$ echo $$
4272

cdyf@qil  /e/work/win-sudo (master)
$ echo $$
4272

cdyf@qil  /e/work/win-sudo (master)
$ ./sudo echo test

cdyf@qil  /e/work/win-sudo (master)
$ ./sudo bash

/e/work/win-sudo/sudobackend: line 11: /proc/516/fd/1: No such device or address

/e/work/win-sudo/sudobackend: line 11: /proc/16016/fd/1: No such device or address

Hm... What happens if you run ./sudo cmd which will work even if WSL is enabeld?

What happens if you run ./sudo cmd

cdyf@qil  /e/work/win-sudo (master)
$ echo $$
4272

cdyf@qil  /e/work/win-sudo (master)
$ ./sudo cmd

/e/work/win-sudo/sudobackend: line 11: /proc/19264/fd/1: No such device or address

Hm... 19264 and 4272 should be equal, but they aren't somewhy.

If you add echo $$ line to bash?

Does it work for you now?

I was add echo $$ line to sudo
cdyf@qil  /e/work/win-sudo (WSL)
$ echo $$
4272

cdyf@qil  /e/work/win-sudo (WSL)
$ ./sudo echo ok
5688

/e/work/win-sudo/sudobackend: line 11: /proc/5688/fd/1: No such device or address

$SHELL -c "echo $$ && ./sudo echo ok"
4272
9152

I'm now getting another error even on my machine, so I'll just try to figure out what's going on before fixing this.

Made some fixes. Try now please.

Finally found the cause of this bug, which I used in vs code and Cmder for this project, not git-bash.exe.


Update userguide to follow bash_app string formatting change in PR #1034

PR #1034 removed implicit formatting of bash_app strings

This PR has been superceded by already-merged #1468


Melee uptime calculation

In quite few logs next issue was observed:

Melee uptime % is quite low, even tho the character in question seems to be attacking for the whole duration of the fight.

Example log (saw multiple logs with similar behavior in the last week):
https://classic.warcraftlogs.com/reports/yXfk6nLzr47j1TqW#fight=43&source=39
Char: Takadk

This may be due to the number of misses/parries.
See the miss/parry/dodge calculation in WCL

I don't think its due to miss/parry. Few of my own logs have miss/parry (just due to boss mechanic), but the analyzer still states that melee uptime is 100%

the max swing speed was previously set to 2.5s. Ideally we detect the weapon type (1h vs 2h) and use a swing speed of 4s or something if it's a 2h -- I can probably get to this this week if no-one else does

Oh yeah, I had totally forgotten about this - whichever of us gets to it first
I need to get some basic Frost functionality in but I don't want that to delay getting the extension up and running


Added hostnames to tls certificates (#7231)

Backport of https://github.com/knative/eventing/pull/7231

/cc @pierDipi @matzew @creydr


Delete all exsisting hulls for current character when you Xup

Delete all exsisting hulls for current character when you Xup. So you cannot Xup as alt and as main.

According to @testkil he rolled this change into his PRs


Make a generic pass-through for all flickr API functions that happen post-oauth.

Probably not worth doing.


Prevent static linking segfaults

libunwind prevents END_OF_STACK errors, but the backtrace is empty.
Relates to https://github.com/crystal-lang/crystal/issues/4276#issuecomment-313678321
Fixes https://github.com/crystal-lang/crystal/issues/6934

Done as you said @ysbaddaden (the failed CI is unrelated)

Note that when compiling with libstdc++ (e.g. the compiler eventually does, because we link LLVM) there are no unwind issues. I'd like to know if a backtrace is still available in that case, even if we linked libunwind?

@ysbaddaden here is a quick with a a file containing raise "test"
/ # crystal build --static --link-flags "-lstdc++" a.cr
/ # ./a
Invalid memory access (signal 11) at address 0x0
Segmentation fault (core dumped)
/ #
/ # crystal build --static --link-flags "-lstdc++ -lunwind" a.cr
/ # ./a
Failed to raise an exception: END_OF_STACK
[0x0] ???

The linker won't link libstdc++ just like that, sadly, the built binary must depend on symbols from libstdc++ for it to be linked. Building the crystal's compiler will link LLVM, which will in turn link libstdc++ that LLVM depends on, and this is know to fix the unwind bug.
We use this trick to build shards statically against musl in the distribution tricks:
https://github.com/crystal-lang/distribution-scripts/blob/master/linux/Dockerfile#L118
What I'm wondering, is whether this patch, always linking libunwind, has any effect on this trick: will libunwind always take precedence over libstdc++ or is it the opposite?

We can use -Wl,--no-as-needed -lstdc++ -Wl,--as-needed to link the library regardless

The require "llvm/lib_llvm"; require "llvm/enums" seems to be better for the end user because we'll still have the stacktrace when we link statically.

The backtrace is still empty, the only difference I see is we don't have to install libunwind-dev.


Sourcemap corrections

clarify --sourceroot option meaning.
Correct relativePath to work for dist/js/filename case.
Add test.
Check for valid location when construction a ParseTree.
Don't mark the placeholder parser source as input source
Fixes #1685

Just ping me when this is ready for review

PTAL

clarify --sourceroot option meaning.
sourcemap sourceRoot value

So the --source-root value will now simply be passed through as the sourceRoot output in the source map and not treated as a local path?

So the --source-root value will now simply be passed through as the sourceRoot output in the source map and not treated as a local path?

The sourceroot code is not changed in this Pull Request.  I change the description emitted by the command line help.  The --source-root value can be true for basepath of the output file name, false to omit, or a string to be loaded in to the sourcemap.

Ok, thanks. I just wanted to double check that documentation will be accurate once published or else it would cause more confusion.

LGTM minus the location assertions.


Products syncing on update when the private app is not selected.

When updating products I have 2 private apps that should feed 2 different projects. When the app that is configured with the hull sanity studio is selected in available sales channels and apps for the product the sync works, but the problem is that products with the other app selected also sync to sanity.

Hey there @highsandlows! Check out issue #19 where this exact situation is addressed :)

Thanks @ndimatteo I'm Really enjoying Messing about with HULL you have done a really great job. Trying to keep myself busy while I wait for Shopifys hydrogen to play nice with next.js.


Create Pages “1-1”

Automatically generated by Netlify CMS

👷 Deploy Preview for thriving-duckanoo-a2ea0a processing.

Name
Link

🔨 Latest commit
202f5c126ae70d08f3eece1c9bd3da9c819c70ce

🔍 Latest deploy log
https://app.netlify.com/sites/thriving-duckanoo-a2ea0a/deploys/653789b3b282c20008f09329


Rails tutorails with gem "gcloud"

Is there any source where i can learn rails with gem "gcloud"

Hi @sajjadmurtaza-ror, thanks for the question. The Google Cloud's Getting started with Ruby has an example app that uses gcloud in a rails app. The tutorial uses named branches, so the datastore service is added in the 2-cloud-datastore branch, and storage service is added in the 3-cloud-storage branch. We'd love to know if this what you are looking for.

@blowmage you are great. Its awesome.
But i have lot of question about Gcloud::Datastore. Because am new for it.
an i ask i you don't mind.

The best place for general questions like this might be StackOverflow.com
-- feel free to use that for high-level questions! :)

Actually the Gcloud::Datastore community is not in huge number. Anyway thanks.

Feel free to ask on StackOverflow or email me directly. Thanks!


Lodash not defined as dependency

Within src/lib/GaugeComponent/hooks/arc.ts lodash is used as dependency. However, lodash is neither defined as dependency nor as peer dependency.

Thanks for reporting that


Some minor spelling fixes and suggested changes

I'll base my suggestions on this PR then when I review it fully :)


Speaker diarization

Hello,
I really impressed with this library.
And I have a question, how to achieve accurate speaker diarization using this library?

This currently needs to be supported. I might add support for this later, but unfortunately, I don't have the capacity for it yet.
Closing this Issue for now -> I'll re-open if and when I'll add support for it.


Autogen package docs

@rasienko
I think this is at a good place. I haven't found an easy way to pull the release notes, and I think we could continue to refine the presentation of some of the content, but looking at the history of this PR is giving me anxiety lol.
Even if we don't announce it yet, I think this is probably a good place for a merge and we can regroup and prioritize the next set of work into a new PR.

Closed in favor of https://github.com/madewithkoji/docs/pull/153


Improve the UI according to modern standards

Current UI is simple to a fault and makes the site look static and old, it should be a little more animated.

@masoodtariq which UI is being referred.


AttributeError case

class F:
    __slots__ = ["a"]
    b = 1

f = F()
f.b = 2

Was originally recorded in https://github.com/aroberge/friendly/issues/146

Should also take care of TypeError: vars() argument must have __dict__ attribute at the same time.

Also take care of ValueError: 'b' in __slots__ conflicts with class variable when an attempt is made to define a class variable with the same name as a variable included in __slots__.


Getting a 429 when calling https://google.github.io/osv.dev/post-v1-querybatch/

Describe the bug
The calls have been rate-limited, despite what https://google.github.io/osv.dev/faq/#is-the-api-rate-limited says.
I was just trying to finish the unit tests, and after that they won't be run very often.
To Reproduce
Steps to reproduce the behaviour:

Call the API with a couple hundred dependencies, 10 or 15 times closely together.

Expected behavior
The calls shouldn't be rate-limited, or the FAQ needs to be updated (our preference of course would be no rate-limiting).

Our FAQ is still accurate with respect to rate limiting, something else is afoot here. Investigating.

I'm kicking myself for not taking a screenshot, but I thought it was going to keep happening and I'd get another chance. But it never happened again.

@jimshowalter so it was a transient issue and you can no longer reproduce it?

Correct.

We did further investigation on our end, and could still see evidence of intermittent 429's being served. In the process of debugging further, we resolved the problem (unfortunately without getting to the root cause).
We have made adjustments to our SLO monitoring so 429's are considered problematic, so if the problem reoccurs, our alerting should bring it to our attention.


Thread Admin Password for OTBR

I have followed this codelab (https://openthread.io/codelabs/openthread-border-router#0) to setup an OpenThread Border router and an end device.
On an Android phone, I have installed Thread application and I would like to access to my OTBR. It requires a Thread Admin Password. Is there a default password or where can I find it?
Can I find this info in Openthread Border Router Web GUI?
Thanks

The Thread Admin Password is also referred to as the Commissioning Passphrase.
When configuring the Operational Dataset, the Commissioning Passphrase may be set using the dataset pskc -p command. See src/cli/README_DATASET.md#pskc for more details.

Closing stale issue.


Support token request API

Support manually mounted tokens, similar to OpenUnison, for better security.

by upgrading to k8s api client v0.22.4 this is already handled for us.  just need to mount, done in the helm charts for openunison


Get the updated files from here.

https://github.com/sidd4698/IOS-App---Treasure-Hunt

@rushabh55 @sapna4g

Ill merge it this time, but next time you commit on the main repo.
Irrespective of conflicts. If you want make a new branch. But no new repos.
On Sat, Feb 7, 2015 at 1:37 PM, Siddesh Pillai<EMAIL_ADDRESS>wrote:

@rushabh55 https://github.com/rushabh55 @sapna4g
https://github.com/sapna4g
—
Reply to this email directly or view it on GitHub
https://github.com/rushabh55/TreasureHunt-iOS/issues/6#issuecomment-73376673
.

alright


joinmarket v0.9.11 in joininbox v0.8.3, jam 0.2.0, build improvements

@openoms so you want to merge this for v1.11.0rc4?


Kim 167580601 remove texting option

Description
Removes the text -- Coming soon!  from the SM UI, preferred contact info.  Also removes all instances of it in the app.  To support zero-downtime migration, the PR to remove the field from the SM table will come after this one gets deployed.
Setup
Go to the service member app.
As user<EMAIL_ADDRESS>go to Edit Move and Contact Info, Edit.  See that text is removed.
Create a new milmove service member user and and move through the wizard until you get to the preferred contact method section.
Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Request review from a member of a different team.
[ ] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

Screenshots

Thanks for the thorough review (and catch).  What a weird set of fixture data, with text, email, and phone all the preferred contact methods!  I wonder if we even use that fixture.


sources: some small tweaks to the curl tests

Small simplifications while working on this file.

Had to rebase to fix conflicts


[libra framework] remove Testnet module

This module was a temporary measure to allow different behavior in testnet and mainnnet. We no longer need it and will implement behavior differences via sending transactions or using a different genesis rather than using this module. This will allow us to ensure that all deployments of Libra are running the same Move code paths.
This has a few consequences:
1. All P2P transactions on testnet are now subject to dual attestation for transfers over the threshold (1000 LBR, or 1000 * 10^6 microLibra)
2. Because of (1), I discovered that the dual attestation threshold is not set correctly. I changed the code to use 1000 * Libra::scaling_factor
3. The account creation tests in account universe rely on the existence of an account type T that can create other accounts of type T. After this commit, no such account exists in our system. I disabled these tests, but hopefully we can resurrect them once we add unhosted wallets to the system.

@bors-libra r=tzakian

:pushpin: Commit e4e261c has been approved by tzakian

:hourglass: Testing commit e4e261c0c04832cee389c59db28610d318e401ce with merge ae9b316b3e1940b4b10668d613dcced0e2e6163f...

:broken_heart: Test failed - checks-actions_land_blocking_test

@bors-libra retry

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: tzakian
Pushing ae9b316b3e1940b4b10668d613dcced0e2e6163f to master...

:eyes: Test was successful, but fast-forwarding failed: 422 Update is not a fast forward

@bors-libra retry

@bors-libra r=tzakian

:pushpin: Commit 464bca3 has been approved by tzakian

:umbrella: The latest upstream changes (presumably a991deee2d06fde02351a7e04f4814fd07f12c88) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r=tzakian

:pushpin: Commit c055eed has been approved by tzakian

:hourglass: Testing commit c055eedffd0fd83610e884822bf84ca589b59c61 with merge 73e18e7c6e2cf0ed01bb28683f337afcd77242cf...

:broken_heart: Test failed - checks-actions_land_blocking_test

@bors-libra retry
Not sure what happened there, see some discussion in #network

:hourglass: Testing commit c055eedffd0fd83610e884822bf84ca589b59c61 with merge 1df2c5f1a2379194942e1d11680159d028854ff7...

:broken_heart: Test failed - checks-actions_land_blocking_test

@bors-libra r=tzakian

:pushpin: Commit e2dbc8a has been approved by tzakian

:hourglass: Testing commit e2dbc8a9959a24a91069559fb3fb4d79815ebfcd with merge fc3aff93563743e97b4a214732687a98c0a434f0...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: tzakian
Pushing fc3aff93563743e97b4a214732687a98c0a434f0 to master...


WLED adapter gets confused when WLED-firmware contains usermods/extended json info

my WLEDs do have the usermod "usermod_Fix_unreachable_netservices" enabled, to keep them connected (without, the web ui stops responding after some minutes being idle). This mod does extend the json info object (as it is intended), with the little 'speciality' that this extension contains some (seemingly properly quoted) HTML.
I think the json result is valid, at least common parsers do not complain, and results do look proper; however, WLED-adapter throws warnings which look like as if the request is not properly parsed.
The json request result (/json/info):
{"ver":"0.11.1","vid":2101130,"leds":{"count":150,"rgbw":false,"wv":false,"pin":[2],"pwr":250,"maxpwr":4000,"maxseg":12,"seglock":false},"str":false,"name":"WLED-2","udpport":21324,"live":false,"lm":"","lip":"","ws":0,"fxcount":118,"palcount":56,"wifi":{"bssid":"78:8A:20:B7:0D:74","rssi":-64,"signal":72,"channel":1},"fs":{"u":49,"t":1024,"pmt":1615056140},"arch":"esp8266","core":"2_7_4_7","lwip":1,"freeheap":21048,"uptime":1883032,"u":{"⚡ Ping fix pingsDelay <input type=\"number\" min=\"5\" max=\"300\" value=\"30\" onchange=\"requestJson({PingDelay:parseInt(this.value)});\">sec":[62762],"⚡ Reconnects":[1]},"opt":127,"brand":"WLED","product":"FOSS","mac":"483fda7eae39"}
WLED adapter 0.5.6 in iobroker log:
2021-03-26 21:21:56.792  - ESC[33mwarnESC[39m: wled.0 (12313) Used invalid characters: wled.0.483fda7eae39._info.u.⚡ Ping fix pingsDelay <input type="number" min="5" max="300" value="30" onchange="requestJson({PingDelay:parseInt(this.value)});">sec changed to wled.0.483fda7eae39._info.u.&#x26A1_ Ping fix pings_span style=_display:block_padding-left:25px_Delay _input type=_number_ min=_5_ max=_300_ value=_30_ onchange=_requestJson({PingDelay:parseInt(this.value)})_sec_/span_ 2021-03-26 21:21:56.805  - ESC[33mwarnESC[39m: wled.0 (12313) State "wled.0.483fda7eae39._info.u.⚡ Ping fix pingsDelay <input type="number" min="5" max="300" value="30" onchange="requestJson({PingDelay:parseInt(this.value)});">sec" has no existing object, this might lead to an error in future versions 2021-03-26 21:21:56.806  - ESC[33mwarnESC[39m: wled.0 (12313) Used invalid characters: wled.0.483fda7eae39._info.u.⚡ Reconnects changed to wled.0.483fda7eae39._info.u.&#x26A1_ Reconnects
Not sure if this is related to the fact that usermods are also using the "u:" key, though on root level; the existing "u:" is in "fs:" scope... Perhaps this could somehow be fixed in WLED adapter? Adapter version is 0.5.6, JS controller 3.2.16

can you re-test with 0.5.7 please ?

this should ben fixed in WLED as the JSON makes no sense at all with an attribute like this :
"u": {
"⚡ Ping fix pingsDelay <input type="number" min="5" max="300" value="30" onchange="requestJson({PingDelay:parseInt(this.value)});">sec": [
62762
],
as u see the key value contains HTML formatted values

sorry, not sure what you mean; as i already mentioned - the original JSON (as posted above) is valid according to RFC 8259. Can be easily checked with any JSON vaildator, e.g. https://jsonformatter.curiousconcept.com/. Screenshot would be a bit too big for posting here, i think... :)
Unfortunately, testing WLED adapter 0.5.7 will take a while, since i have to re-activate that installation for this; has been a while since i used that... ;)

sorry, not sure what you mean; as i already mentioned - the original JSON (as posted above) is valid according to RFC 8259. Can be easily checked with any JSON vaildator, e.g. https://jsonformatter.curiousconcept.com/. Screenshot would be a bit too big for posting here, i think... :)
Unfortunately, testing WLED adapter 0.5.7 will take a while, since i have to re-activate that installation for this; has been a while since i used that... ;)

the JSON itself is a valid format,, but unusable... in the logic we transfer JSON keys to ioBroker states meaning
{key:value}
as the key here is an HTML format, it can't be translated to a proper state format.
We expect a key and value and not value combined in a key so in stead of one HTML sting each value should be a separate set like :
{"u":
{ "Key_1":
"attr1": value,
"attr2": value,
}
}

Ok, got it, thanks; so perhaps this should be noted somewhere, that usermods with GUI extensions are not supported because of this...

or I need to know how this key are generated to just filter them out here in the adapter :)
Already did something similar here for a value PID, maybe I need to do that more generic
https://github.com/DrozmotiX/ioBroker.wled/commit/42a948b6864b681e01a867056e1d29b808614c2b#diff-58417e0f781b6656949d37258c8b9052ed266e2eb7a5163cad7b0863e6b2916aR712
but need examples how it looks like :D

fixed in 0.6.x


[move] Unify source maps to operate upon Loc locations #9022 - (107)

Motivation
Many types in the Move codebase are defined to operate upon a generic "source location" type, in order to operate on both OwnedLoc (a source location type that stores its file field as a String), and on Loc (a source location type that used to store its file field as a &str reference, and now stores it as a globally interned string Symbol).
Remove OwnedLoc, since it no longer provides functionality that differs from Loc. This allows the generic source map types to be transformed to simpler, concrete types.
In addition, helper functions implemented to map between types that use OwnedLoc and those that use Loc can also be deleted.
Test Plan

CI/CD tests are covered

Closing this PR due to technical error, raised same PR https://github.com/diem/diem/pull/10652

Closing this PR due to technical error, raised same PR https://github.com/diem/diem/pull/10652


Rename content/issues/tracking-your-work-with-issues/assigning-issues…

…-and-pull-requests-to-other-github-users.md to content/issues/tracking-your-work-with-issues/jirayupiwonz/docs, .

Why:
Closes [issue link]

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[x] This pull request impacts the contribution experience

[x] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Not only are you spamming the Magisk git you're spamming the GitHub friggin git too? Wow.


Review Mortality 2020 data

Hi @camerondow35, this is for you to review..let me know if you have questions..
Problems founds in Mortality_Survey_2020_csv

[ ] More than 1800 records have no date or surveyor
[x] Tag 30001: was DC in 2019, now is DS. It has same comment as last year. Should be PD. Tag 12271, was DS last year. Should be PD
[x] Why some NOT Fraxinus trees that are Alive have ‘crown.position’ information? Multiples species have that info, in total 22 trees, why?.
[x] Frpe 123412 and fram 133706 are missing crown.position
[ ] For some comments you wrote XX% whitening? What does it mean “whitening”? getting white? I have no idea what it means.

Others:
After all coring is done you need to indicate in the datasheet if ‘cored’=yes or no (column AE).
About data folder:
All files produced with the r script 1-Creating_data_files_from_raw_data.R, have same length (13370 observations) but mortality_2020 has 72555 obs. Please check the code, there must be something wrong there..

More than 1800 records have no date or surveyor

The only records w/o dates are PD, am I supposed to put a date for those too?

Tag 30001: was DC in 2019, now is DS. It has same comment as last year. Should be PD. Tag 12271, was DS last year. Should be PD

Mistakes - fixed.

Why some NOT Fraxinus trees that are Alive have ‘crown.position’ information? Multiples species have that info, in total 22 trees, why?.

Ah, they were AU last year and I forgot to delete the crown position. Fixed.

Frpe 123412 and fram 133706 are missing crown.position

Judging by dbh 2018 and quadrats they were in, I assigned them both S

For some comments you wrote XX% whitening? What does it mean “whitening”? getting white? I have no idea what it means.

% whiteneing refers to EAB whitening. It's the same comments Alyssa used last year. Remove?

About data folder:
All files produced with the r script 1-Creating_data_files_from_raw_data.R, have same length (13370 observations) but mortality_2020 has 72555 obs. Please check the code, there must be something wrong there..

For some reason all of mine had 75526...I'll try rerunning the script but there were parts of it I don't understand

Ran the script and pushed the result. Now has 13380...

answers to your questions @camerondow35

More than 1800 records have no date or surveyor
The only records w/o dates are PD, am I supposed to put a date for those too?

Yes, they should have the same date when you visit that quadrat.

For some comments you wrote XX% whitening? What does it mean “whitening”? getting white? I have no idea what it means.
% whiteneing refers to EAB whitening. It's the same comments Alyssa used last year. Remove?

That term "EAB whitening" is not in the literature. What does it means to you? I think Alyssa meant  "20% bark flaking" . When the bark is removed by woodpeckers on a damaged tree (while searching for EAB larvae) the inner bark will get lighter. I think that's the reason why she came with that term. But we don't have to repeat it.  Tell me what did you mean with that and then we can make a decision to change the term in 2020.

OK would you like me to add dates based on the quadrat survey date or shall I just add one date for all PD?
Yes, whitening refers to the EAB code W: bark blonding. I see in 2018 % was not noted in cases of W. I feel like 'whitening' is intuitive to me, since the code is a W. It might be confusing to use whitening for 2019 then change it in 2020, no?

OK would you like me to add dates based on the quadrat survey date or shall I just add one date for all PD?

they should have the same date when you visit that quadrat (based on the quadrat survey date)

Yes, whitening refers to the EAB code W: bark blonding. I see in 2018 % was not noted in cases of W. I feel like 'whitening' is intuitive to me, since the code is a W. It might be confusing to use whitening for 2019 then change it in 2020, no?

Nope. Whitening is not a term related to EAB infection, and the 'W" was establish to refere to "Woodpecker" not "whitening" so let's don't repeat the mistake as 2019. Please replace by bark blonding in your comments.

Nope. Whitening is not a term related to EAB infection, and the 'W" was establish to refere to "Woodpecker" not "whitening" so let's don't repeat the mistake as 2019. Please replace by bark blonding in your comments.

Should I replace it in 2019 as well, while im at it?

For the dates I just used this code for(i in 1:nrow(Mortality_Survey_2020)){ Mortality_Survey_2020[i,32] <- ifelse(is.na(Mortality_Survey_2020[i,32]), Mortality_Survey_2020[(i-1),32], Mortality_Survey_2020[i,32]) }
i think this is the most efficient way of doing it.

sure for both

ok, pushed to github.


Implement has_sorted_indices, has_canonical_format, sort(ed)_indices() for sparse matrices

This PR brings the current implementation to full compliance with SciPy, but I don't know what's the best way to measure performance impacts...I measured SpMV and do not see slowdown with this PR. Not sure what else I should look into, but I think for compatibility this PR is good to go.
Notes:

The property code is largely borrowed from SciPy so that it's easier to maintain and stay compliant.
The two has_* properties are mainly for compressed matrices (i.e. CSC/CSR); for COO, it doesn't have has_sorted_indices, and its has_canonical_format is set during creation or when sum_duplicate() is called.
Importantly, COO does not validate this property when it is accessed.
The internal flags _has_* are not directly accessed anywhere in the codebase; they are maintained by the corresponding properties.
Whenever an operation does not guarantee the status of these properties, they should not be overwritten manually. This is in line with SciPy.

TODO:

[x] Expand PR description
[x] Add tests?
[x] Clean up
[ ] Measure performance changes (how?)

Done. PTAL.

pfnCI, test this please.

Jenkins CI test (for commit 7e152e5b9a727522cc50047d6e7ac1ab4650ea88, target branch master) succeeded!

@cjnolet If you have time, could you evaluate if and how this PR would have performance impact?

@leofang,
Sure. I think I had already copied the implementation from Scipy into #3533 just to get something working but I can merge yours into that branch next week and do a little profiling.

pfnCI, test this please.

Jenkins CI test (for commit d025fdc564559fe5aceccf72c95ea6e8a35c76f0, target branch master) succeeded!

pfnCI, test this please.

Jenkins CI test (for commit a85331df313786de26f52873aa1842176ae6b7e3, target branch master) succeeded!

LGTM! I'm going to merge this PR, and please post a new issue if it has a significant performance impact.

Thanks!


Feature Request: Chat names in log files

Hello,
your original application saved chat logs with the players names in the chat.

(10:38 PM) REEGZ (REEGZ): game stuff, not real life stuff
(10:38 PM) Viles (Viles): yeah i dont care
(10:38 PM) Nogaxeh (Nogaxeh): lol

But the newer version i am not getting the names, see below:

(02:49 AM)  nah all good, catch ya bro
(02:50 AM)  welp, I'm hopping off for the night, I'll probably be on tomorrow
(02:50 AM)  see ya man
(02:50 AM)  see ya bro

the main reason for this i log the chat so i can keep track of any abuse that may occur.
any chance we can get this put in, or am i missing a setting somewhere?

Thanks for pointing this out.
I'll have it fixed and up tomorrow :)

any update on this?

Just added two updates. The last update should fix this :)

issue fixed, thanks i can see the names in the chat logs


Dropdown seems to be missing 'active' property

Issue description

components: Dropdown
reactstrap version 5.0.0-beta
import method es
react version 16.2.0
bootstrap version 4.0.0

What is happening?
I'm using <Dropdown> component within <Nav> component (property 'nav' set to true).
I can successfully set <DropdownItem> property 'active', but <Dropdown> itself has no such property.
Therefore the Dropdown entry in the navbar isn't highlighted although its child (<DropdownItem>) is active/highlighted.
What should be happening?
In pure Bootstrap, I would set the class of <li> representing the dropdown to "nav-item dropdown active".
Doing this would correctly highlight the Dropdown button in the navbar.
I would expect <Dropdown> to also expose an 'active' property.
Steps to reproduce issue

...
...

Error message in console
paste error message with stacktrack here

Code

For now the workaround is manually adding the className or using NavItem with the tag prop set to Dropdown
<NavItem tag={Dropdown} nav ...>

Shall I Close this issue? Your answer helped me a lot! I still expect, that "Dropdown" component gets an "active" property in a later release ...

We can leave it open to track adding the active prop. I think that something like nav && active && 'active' needs to be added to the classnames.


Screenshots are not displayed correctly in Safari browser

The screenshots on instructions.html are not displayed correctly as they appear stretched vertically. On other browsers however this bug won't occur. Fixing this bug would obviously make for a better experience for all Safari users.

the issue occurs because of following lines in instruction.css:
https://github.com/lksjgr/uni-wien/blob/e9beb09ba7e8554ba55650980ab4636d94525f7d/css/instructions.css#L21-L35


Thank you for this complete emoji list

I thank you for your hard work in giving us the list and making it easy for people like me to be familiar with emojis on github.

Great Job tks :🖖

Thank you so much

ir putu vertimas is rusiu i lietuviu

Such a rich information. Many thanks to all that made it possible.


portDiscovery finding no printers - android

I have a TSP100 LAN on the same network as well as a TSP100III connected via bluetooth and neither device is being seen by portDiscovery. I am getting a 'cannot find printer' error. In addition some output before that is:
03-14 14:55:22.784: W/CordovaPlugin(28931): Attempted to send a second callback for ID: StarIOPlugin1594234404
03-14 14:55:22.784: W/CordovaPlugin(28931): Result was: []
03-14 14:55:22.784: W/PluginManager(28931): THREAD WARNING: exec() call to StarIOPlugin.portDiscovery blocked the main thread for 6180ms. Plugin should use CordovaInterface.getThreadPool().
The app does hang for a little bit (6 seconds i guess) so portDiscovery must be searching.

@tapndine have a look a this version of the plugin https://github.com/auctifera-josed/starprnt, (Disclaimer: I am a contributor of that project)


Please review osd and eng traineddata PKGBUILD for msys2

I have updated the pkgbuild for OSD and ENG traineddata for msys2 - please see https://github.com/Alexpux/MINGW-packages/pull/673
and make any required changes

Please write correct issue report


Namespaces Support

We'got used to the fact that all Angular directives start with 'ng-' and all third-party libraries follow it by using their own prefix. The problem is that such approach is neither enforced nor it is stipulated in any way, except for the fact it may break things, if not followed.
If there was a way for a library to register it's prefix as predefined, this would solve many issues if incompatibility and erroneous overrides.
Example:
var myLib = app.library('ml');

myLib.directive('DirName', [function () {
    return {
    }
}]);

Such library would simply redirect registration to module app, while automatically extending directive name as if it were 'mlDirName'. Such approach would guarantee that directives from different libraries do not get into naming conflicts. Only library name/prefix would have to be unique in this case.

@VitalyTomilov this is duplicate of existing issues, ex.:

https://github.com/angular/angular.js/issues/2767
https://github.com/angular/angular.js/issues/2447

and those issues got some discussion going already. Please, let's centralize this discussion - could you add your comments in there?
Closing as a duplicate.

Oh, man, almost a 2-year long discussion without a resolution still...
Yeah, I was late to the party :)


Mhartid illegal instruction exception

Hi,
Just a quick question about the mhartid csr in the Ibex core. The RISC-V Privileged Spec says that mhartid should be a read-only CSR, and I noticed that Ibex will throw an illegal instruction exception if any write to mhartid is encountered. However, with the misa csr, which is hardwired to its reset value, Ibex will just ignore any writes to that address if it encounters any, instead of throwing any exceptions.
As a result, I was wondering if this difference in behavior was intended, since both CSRs are intended to be read-only?

Hi @udinator ,
thanks for opening this issue.
Yes, the difference between writing mhartid and misa is intended.
mhartid is a read-only CSR. This can be seen right from looking at bits 11 and 10 of its address (0xF14). 2'b11 means read-only and the spec defines that write accesses to read-only CSRs lead to an illegal instruction exception.
In contrast, the misa CSR is theoretically writable as bits 11 and 10 of its address (0x301) are 2'b00. However, the spec defines this register as WARL (write-any, read-legal). This means the implementation defines the subset of values defined in the spec which the CSR can actually hold. For misa, the write capability would theoretically allow to enable/disable certain ISA extensions at run time. But as Ibex is a fairly simple core, this feature is not supported. Or in other words, the only misa value supported in Ibex is the reset value. Ibex supports enabling/disabling the E and M extensions at design time which impacts the reset value of misa.
Best regards,
Pirmin


Terminal starts at the top of the window after leaving a "screen"

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

OS version and name:  macOS 10.14.2 (Mojave)
Hyper.app version:  3.0.2
Link of a Gist with the contents of your .hyper.js:  https://gist.github.com/LeSeulArtichaut/324351ba22b27b7dbe9e50c7a1fe5359
Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere):  N/A (No errors in console, and didn't have time to dive into the sources)
The issue is reproducible in vanilla Hyper.app:  Is Vanilla

Issue

After leaving a screen (as a secure ssh on a distant machine) using Ctrl-A d or Ctrl-D, Hyper resumes the previous shell (normal behaviour) but writes from the top of the canvas without clearing the window, making it really difficult to read the terminal. Hyper should either draw from the bottom of the "used space", or clear the canvas.
Thanks a lot for considering the issue!

I do have a similar issue as well. I use ssh to connect to my primary environment, and sometimes if left inactive for too long, my connection will drop out, this will often result in hyper resuming the original shell but from the top of the canvas with an uncleared window.
I usually do a quick "Ctrl + C" which clears it and I can continue on my merry way but it would be appreicated if I could keep the contents of what I was doing on the screen i.e. hyper starts from the bottom of the "used space".

@MacAhmed This is my workaround, setting custom css in ~/.hyper.js
...
css: `
      .term_fit.term_wrapper { overflow: inherit; }
    `
...


Create Jenkins project to publish to marketplace

Create project on internal Jenkins instance that would build extension package, upload it to internal server for verification and then ask confirmation before publishing. This way we would verify exactly the same bits we are about to publish to marketplace.

Jenkins project is created, nightly builds are available from https://download.jboss.org/jbosstools/adapters/snapshots/vscode-tekton/


Updated arkworks

Just to make sure I don't merge this in too soon, can we add details of what's remaining here or is this ready to merge?

This is ready, we just had to update the version, no other changes


Kirki_Styles_Frontend not found

Issue description:
Kirki_Styles_Frontend is referenced in core/dynamic-css.php but the class doesn't seem to be available. I think Kirki_Modules_CSS should be used there instead.
Version used:
3.0.15
Using theme_mods or options?
theme_mods
Code to reproduce the issue (config + field(s))
General issue

@FrankM1 wow, I don't know how this one was missed...
Thank you! fixed in the develop branch, will be included in 3.0.16


What Type Of Ide i Can Use Whit OBEngine?

title

I personally use Visual Studio Code to edit the Lua files but you could use any text editor.
Try to choose an editor with Lua Syntax Highlighting and maybe a linter / autocompletion features.
You could go with VS Code, Atom, Notepad++, Sublime Text etc...

have you use this ide? [https://studio.zerobrane.com/](ZeroBrane Studio)
Feature highlights

Small, portable, open-source, and cross-platform (Windows, macOS, and Linux);

Auto-completion for functions, keywords, and custom APIs;

Syntax highlighting and folding for 110+ languages and file formats;

Project view with auto-refresh and ability to hide files and directories from the list;

Interactive console to test Lua code snippets with local and remote execution;

Integrated debugger with support for local and remote debugging, including cross-platform and on-device debugging;

Live coding with Lua, LÖVE, Gideros, Moai, Corona, GSL-Shell and other Lua engines;

Multi-language support with translations to French, Spanish, Russian, German, Chinese, Esperanto, Italian, Bulgarian, and Brazilian Portuguese languages and documentation to add your own;

50+ extension packages in the package repository;

Fuzzy search for files, symbols, and library functions, function outline, and more.

also is open source so you can make a fork of that ide for OBenine :)

Nice ! I'll take a look, thanks :)

no problem :)


[Environment] Can't create an Environment with any templates besides master.

For replay this bug you need:
Test case

Go to SS and login under any account.
Go to 'Environment' and try to create an environment with 'Master' template. (Result: you can create it.)
Try to create an Environment with 'Mongo' or 'Casandra' template.

Result: it fails on preparing templates or cloning containers.

karaflogs2016-04-07 18_13_12.txt

2016-04-07 12:12:33,255 | INFO  | tp1017977050-598 | CommandProcessor                 | 191 - io.subutai.core.executor.impl - 4.0.0.RC7-SNAPSHOT | Received:
@nk-the-crazy, @kylycht  seems problems on Kurjun side
{
"type": "EXECUTE_RESPONSE",
"id": "9AA1C3F1F9F1506E2B4CFFEC81BD097A0561FB11",
"commandId": "7de457e1-0058-4192-ac53-aa29a5f43e8d",
"pid": 0,
"responseNumber": 1,
"stdOut": "time\u003d"2016-04-07 12:12:32" level\u003dinfo msg\u003d"[Importing apache]" \ntime\u003d"2016-04-07 12:12:32" level\u003dinfo msg\u003d"[Version: 4.0.0, branch: dev]" \n\r 0 B / 25 B    0.00%time\u003d"2016-04-07 12:12:33" level\u003derror msg\u003d"[Failed to check MD5 after download. Please check your connection and try again.]" \n",
"exitCode": 1
}


concepts: auto-link to Basic Concept page form Glossary and tooltips

Would be nice that when there's actual md content under a basic concepts file frontmatter (tooltip text)

The entry's title in the Glossary links to the full concept page (see https://github.com/iterative/dvc.org/pull/2453#pullrequestreview-711436406)
The tooltip text is auto-appended with a "More info" link in a new line (see https://github.com/iterative/dvc.org/pull/2755#pullrequestreview-737852600)

Cc @rogermparent


fix(text): Render list items correctly

Fixes #1135
Fixes #1138
See changes made in https://github.com/ietf-tools/xml2rfc/commit/3712523cb325bb43b1b164e833dd436c1c62a295 for context.
This change can affect if ol is used inside a li as the first element and the li element is inside another ol element.
I could not find any published RFCs with XML that match the above condition.
These are the RFCs that I found that use ol as the first element in li:

RFC8679
RFC8687

I didn't notice any changes in text output due to this change. But there are a few minor changes (with spacing) due to changes over the years.

I think it's worth trying to change things such that generatiing
   4.  4.1  Sublist with '%p' parent counter element in the list
            counter, item one

       4.2  Item two.

       4.3  Item three.

we generate
   4.

       4.1  Sublist with '%p' parent counter element in the list
            counter, item one

       4.2  Item two.

       4.3  Item three.

Unless that breaks the intended fix for the rfc-to-be.

What is better about that superfluous empty line?
(You can probably have one if you want with <br>)

Look at it in context with the rest of the nested <ol> above it in the test document:  https://github.com/ietf-tools/xml2rfc/blob/fdb6fb8ad2024b3483f7480a07f8370e673c27e2/tests/valid/elements.bom.text#L366-L458 and consider what would happen if there were any text between the  and  at https://github.com/ietf-tools/xml2rfc/blob/fdb6fb8ad2024b3483f7480a07f8370e673c27e2/tests/input/elements.xml#L472-L473

I have converted PR to a draft.
The changes from https://github.com/ietf-tools/xml2rfc/pull/1137/commits/e889f930b982c9bc9045f8a6e653f25a6464ec58 add a new line when with the list style to both ol and ul.
This was skipped previously.
Specifically, this adds list style + new line when li contains an artwork, figure, or sourcecode as the first child.
See how this changed test outputs in https://github.com/ietf-tools/xml2rfc/pull/1137/commits/ca8286a4126bc28bba095eb7e708c5a7424aadef
Note that the test output has date changes and multiple content changes propagated because of the addition of new lines.

@rjsparks
Here's list of affected^1 published RFC diffs:

https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc8742.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc8811.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc8793.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc8668.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc8687.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc9599.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc8679.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc8927.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc8677.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc8667.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc8698.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc9195.txt
https://author-tools.ietf.org/diff?url_1=https://github.com/kesara/t4/raw/refs/heads/main/xml2rfc-1137/rfc9687.txt

The above list is a bit confusing? I only see impact from this PRs change on rfc9599.

The above list is a bit confusing? I only see impact from this PRs change on rfc9599.

In most case there's empty="true" attribute set on the element or the parent level, hence no changes.


AttributeTO with null values handling

From chicchiricco on August 20, 2010 17:08:53
When creating, ignore.
When transforming UserTO in UserMod (for example), put it as attributeToBeRemoved only.
Original issue: http://code.google.com/p/syncope/issues/detail?id=36

From chicchiricco on August 31, 2010 03:53:08
Status: Started

From chicchiricco on August 31, 2010 07:45:43
Status: Fixed


DotDot operator with char operands in pipeline throws error

Steps to reproduce
'A'..'B' | ForEach-Object { $_ }```

Expected behavior
-----------------

```A
B
A
B```

Actual behavior
---------------

```A
B
Cannot convert value "A" to type "System.Int32". Error: "Input string was not in a correct format."
At line:1 char:1
+ 'A'..'B' | ForEach-Object { $_ }
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : InvalidArgument: (:) [], RuntimeException
+ FullyQualifiedErrorId : InvalidCastFromStringToInteger```

Environment data
----------------

```Name                           Value
----                           -----
PSVersion                      6.0.0-rc
PSEdition                      Core
GitCommitId                    v6.0.0-rc
OS                             Microsoft Windows 10.0.15063
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0```

repros for me.

Interesting; I didn't even know support for character ranges was implemented. Can I suggest changing "dotdot operator" to "range operator" in the title?

I went with "dotdot operator" because that is what was used in the PR that added the character range implementation. https://github.com/PowerShell/PowerShell/pull/5026

I see  - that very nonstandard name is the reason that I wasn't able to find this new feature, despite then already knowing that it exists (based on this issue), so thank you for adding the link.
I guess my mentioning the construct's official name here - range operator - now makes this issue more discoverable and, by extension, now also the linked PR, but my preference is always to have the important keywords in the title, as an in:title search then allows for a more focused search.

Another datapoint (and workaround): Forcing generation of the entire array up front avoids the problem:
('A'..'B') | ForEach-Object { $_ }  # OK, due to (...)

The output from Trace-Command strongly suggests that this code path is reached before the operator is explicitly invoked when the .. appears as the very first operation in a pipeline:
PS C:\> Trace-Command -Expression {'A'..'B' |ForEach-Object { $_ }} -Name * -Option All -FilePath ($t = [System.IO.Path]::GetTempFileName()) 2>$null; Get-Content $t -TotalCount 2
TypeConversion Information: 0 : Converting to integer.
TypeConversion Information: 0 : Exception converting to integer: "Input string was not in a correct format.".

Why, I'm not entirely certain yet, will keep digging.
Re: the "nonstandard" name, I took it from the token that represents the underlying range operator :-)

@SteveL-MSFT: Your title edit just made my day, thanks (I wish I were kidding).
@IISResetMe: Fair enough, and perhaps "DotDot operator" is reasonably well-known; my term for it came from Get-Help about_Operators, and I would argue that it's the more sensible name:
Operators should be named for their purpose (only), not their syntactic form (too).
I won't repeat my rant about about the "dot sourcing operator" here.
Although, who's to say? Perhaps it's time for the "asterisk multiplication operator", or the "hyphen-minus subtraction operator", and, last but not least, the "dot-dot range operator".

@IISResetMe - the idea was to name tokens after the syntactic form because it might not make sense if a token was allowed in some new context, e.g. if .. was allowed where it is no longer the range operator.


Bot go away from pokestop instead of walking toward them

Expected Behavior
walk to pokestop, spin it then reach the next one
Actual Behavior

Steps to Reproduce
launch the bot normally with configuration done
setting that can help
location : "48.892407,2.389109"
step : 4
walk speed : default (4.16)

Other Information
OS: Debian x86_64
Git Commit: 18f232bffb8df487654e6c52ff3eea859a120925

Use latest dev.


CI is failing

...since I don't think we have an issue tracking this already.
I tried to fix this in #423, but it looks like not finding the shell isn't the issue.
It appears that there are no files matching /etc/*release, which the checkout action is looking for.
Where is the code that actually builds the docker image?

The code that builds the docker image is in https://github.com/fare/gerbil-utils/blob/master/scripts/make-docker-image.ss
and that's what I'm attempting to fix today.

Working on it in https://github.com/Glow-Lang/glow/pull/428


[🐞] inconsistent rendering between SSR and CSR

Which component is affected?
Qwik Runtime
Describe the bug
SSR interprets <button disabled={''}> as disabled but CSR interprets it as NOT disabled.
Reproduction
https://stackblitz.com/edit/qwik-starter-u3vczw?file=README.md,src%2Froutes%2Findex.tsx
Steps to reproduce
Click on the toggle button and observe that the target button disabled does not correctly follow from SSR.
System Info
n/a

Additional Information
No response

what behaviour is expected?
disabled="" is a valid attribute, so CSR should add it or SSR should ignore attributes with empty values?

I would say that "" should render as <button disabled/>. So I think CSR is correct, and SSR behavior is wrong.

If so, then SSR is correct and CSR is not. I will create a PR, please verify :D

disabled is a boolean, not a string. JSX is not attributes.

Imo, it should not be fixed, but emit a warning in dev mode.

I've  tag uses,

Here the output works as expected when it SSR rendered, but the option's value attribute does get disappeared when it is CSR rendered.
It is causing serious bugs!

This is when SSR rendered.

This is when CSR rendered.

@iamriajul can you share that piece of code or something like it?

<option value={''}>Select</option>

@iamriajul Can you explain why value="" is needed? I agree that CSR and SSR should do the same thing but I don't know what the correct thing is.
Here's a playground. SSR will render "" as value and true as value="". CSR will render "" as nothing and true as value="true". Which is correct?
https://qwik.builder.io/playground/#v=1.2.19&f=Q0o0pkbm2BKNDqL2pTj9QAIDWk3YIgzWMNBESl1QV0KqGIXs1ErbaogWPWCeK02thfsJGC2geAWL2ioZKtkZ2uhDxNCVwKorRA0I1WSkZGekoKuQnJGfD6or8kBRDdSAyxyoLmMlO2MCSkyU7EwIKFGyAzZ9klPxq6oGVSC1diASVaENMEOAPAXlJRVB49oG0j5DNEuqNTRt7ZADUFu7FsiHxgJQJbDABsYTvGEHNAWUQkZz9NDM0QA

Can you explain why value="" is needed

It's the initial value of my signal.
eg:
const method = useSignal<'' | 'VIDEO' | 'AUDIO'>();
// Where `` empty string means no filtering is applied.
// And `VIDEO`or `AUDIO` means their filtering is applied respectively.
<select bind:value={method}>
<option value="">Select a Filter</option>
<option value="VIDEO">Video Filter</option>
<option value="AUDIO">Audio Filter</option>
</select>

When user doesn't select any option or select the first item which is Select a Filter with value of empty string  .  the signal then updated accordingly with empty string. but when in CSR mode the the HTML doesn't even have the value attribute that results in my Signal value to be Select a Filter when user select the first item. But I don't expect my signal to have Select a Filter as instead of   empty string.

Ok we discussed with Misko and both SSR and CSR should do the following:

anything falsy (not string, not number) => attribute removed
true => <div attr/>
"" => <div attr=""/>

So both need fixing


Matching multiple types

Currently only the first matching data type is returned, however it is possible that dropped content may contain data of multiple types. For example when dropping a draggable image from another page, the url and html types may both be present.
Is there scope to rework the plugin to return all matched data types, rather than just the first match?

The problem is that React DnD (and DnD Core underneath it) don't support the notion of "multiple types" for simplicity. Maybe you should use HTML5 API directly instead if you want this. Another option is to make an umbrella type for all HTML5 types, and instead make the specific “subtypes” available on the item object so you can read them in canDrag. That would be a breaking change for this backend, but we can do this if it covers all previous use cases and adds new ones.

Closing, as we don't have plans to implement this.
Feel free to come back with your experiments, if you make any!

Thanks Dan, I'll post if so.


101 Add Like button

Resolves #101
REQUIRED:

[x] Write the issue number associated with this pull request after "Resolves #"
[x] Add the correct label for this pull request (you can just use the same label as the associated issue)
[ ] Be sure to share a link to this pull request in Slack/Discord and ask for a review! <3

Do not review until https://github.com/OriOri-CCP7/oriori/pull/107 has been approved & merged


Please share the versions of GPU, PyTorch, CUDA, Python, and Linux.

Please share the versions of GPU, PyTorch, CUDA, Python, and Linux that allow you to run Genesis properly, so I can use them as a reference.

Hi, I have just resolved my case.
My environment:

Mac Studio
Python 3.10.16 (Anaconda/conda 24.7.1)
Sequoia 15.2
M2 Ultra
torch 2.5.1

Commands:
conda create --name genesis python=3.10 -y
conda activate genesis
python -m pip install genesis-world
python -m pip install torch torchvision torchaudio
python render_on_macos.py -v
Hope if it helps your case. Thanks.

Intel 14700, NVIDIA 4070,
Ubuntu 22.04 , NVIDIA535, CUDA 12.2
python 3.10
torch 2.2
It is easy to install genesis. The default version of PyOpenGL is 3.1.7, but it will encounter errors during runtime. I found that version 3.1.4 of PyOpenGL can be used. As the user guider reporting, if you followed 2.A and see “GLIBCXX_3.4.30 not found”.
cd ~/anaconda3/envs/genesis/lib
mv libstdc++.so.6 libstdc++.so.6.old
ln -s /usr/lib/x86_64-linux-gnu/libstdc++.so.6 libstdc++.so.6
However, LuisaRender has strict requirements for Ubuntu and CUDA versions.

Hi, I have just resolved my case.
My environment:

Mac Studio
Python 3.10.16 (Anaconda/conda 24.7.1)
Sequoia 15.2
M2 Ultra
torch 2.5.1

Commands: conda create --name genesis python=3.10 -y conda activate genesis python -m pip install genesis-world python -m pip install torch torchvision torchaudio python render_on_macos.py -v
Hope if it helps your case. Thanks.

@Habatakurikei Hi, just curious about genesis on mac. Do you meet OpenGL.error.GLError when executing python render_on_macos.py -v?
OpenGL.error.GLError: GLError(
	err = 1282,
	description = b'invalid operation',
	baseOperation = glBindFramebuffer,
	cArguments = (GL_DRAW_FRAMEBUFFER, 0)

Hi, I have just resolved my case.
My environment:

Mac Studio
Python 3.10.16 (Anaconda/conda 24.7.1)
Sequoia 15.2
M2 Ultra
torch 2.5.1

Commands: conda create --name genesis python=3.10 -y conda activate genesis python -m pip install genesis-world python -m pip install torch torchvision torchaudio python render_on_macos.py -v
Hope if it helps your case. Thanks.

@Habatakurikei Hi, just curious about genesis on mac. Do you meet OpenGL.error.GLError (#165) when executing python render_on_macos.py -v?
OpenGL.error.GLError: GLError(
	err = 1282,
	description = b'invalid operation',
	baseOperation = glBindFramebuffer,
	cArguments = (GL_DRAW_FRAMEBUFFER, 0)

No, I did not face any error including GLError through that setup.
I attach a video how python render_on_macos.py -v worked on my Mac studio.
https://github.com/user-attachments/assets/0f762550-09a6-434b-9f02-823385aa0fb2

GPU: Quadro RTX 4000
PyTorch version: 2.5.1+cu121
CUDA Version: 12.2
Python version: 3.10.9 (via pyenv)
Linux: Ubuntu 20.04

Hi, I have just resolved my case.
My environment:

Mac Studio
Python 3.10.16 (Anaconda/conda 24.7.1)
Sequoia 15.2
M2 Ultra
torch 2.5.1

Commands: conda create --name genesis python=3.10 -y conda activate genesis python -m pip install genesis-world python -m pip install torch torchvision torchaudio python render_on_macos.py -v
Hope if it helps your case. Thanks.

@Habatakurikei , thank you for posting this! I have been struggling to make any progress with a Python virtual environment (tried both Python 12 and 11). I did not try Python 10 in the venv so I am not sure if that was the reason..... but I did use your recipe with Miniconda on my Mac M3 and it finally worked! :-)

Environment Details
Operating System

Ubuntu 22.04

Python Environment (inside a Conda environment)

Python 3.9
PyTorch 2.5.1

GPU Setup

CUDA 12.2
NVIDIA Driver 535
Hardware: RTX 3080Ti

Installation Steps
CUDA Installation

Used CUDA 12.2.2 runfile installer from NVIDIA Developer site: https://developer.nvidia.com/cuda-12-2-2-download-archive?target_os=Linux&target_arch=x86_64&Distribution=Ubuntu&target_version=22.04&target_type=runfile_local
Target: Ubuntu 22.04, x86_64

NVIDIA Driver Installation

Installed via ubuntu-drivers: https://ubuntu.com/server/docs/nvidia-drivers-installation
Command: sudo ubuntu-drivers install nvidia:535

✅ Confirmed working with LuisaRender


style: added resize feature to preview plugin; editor wrap configurat…

…ed to auto mode
@ala-n, @PolinaVolchetskaya , @Sisha3342 , @nattallius

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


"Shut down by issue" executors are not cleaned up, is this some kind of a missing feature?

I have been debugging a fairly rare issue in our production system which is based on spring boot reactor and uses amongst others, Lettuce and Netty. Sometimes we see that we start to get a fairly small rate, but stable error rate increases on certain Lettuce calls.
Where I got my investigation is the following: Some usual error comes on a Mono and a rather unusual situation happens because as it seems, that the next handling might have been already called on that. So MonoNext draws an onErrorDropped because it sees a 'done' has already happened. This will roll up to SingleThreadEventExecutor only to shut down that executor for sure. However, it turns out, that by design MultithreadEventExecutorGroup is not capable of handling this situation gracefully in the wider picture.  In case of Lettuce, DefaultEventExectorGroup is used (here). So what is the consequence is that the group still has this item and all usages of that executor becomes "rejected", adding a stable failure rate into a running Netty server instance. I've been thinking for a while, which part is the issue here and didn't able to decide if Lettuce should use a different event executor group or reinitialize this or what, but came on the conclusion, that running instances should not be harmed, so maybe this or a yet-to-be-existing executor group should rather heal itself. Overall this is seen just as a "harmless" timeout if the redis was somekinda cache, but the logs showed rejection on executors.
While I'm sure we still need to trace down why happens sometimes this 'done' collision, I strongly belive, that a better executor pool handling is needed here. There are numerous reasons this could happen later on with other bugs, plus the whole code seem to "support" that an executor can go into a shutdown without shutting down the server itself. Question is, how can any client user of these codes make sure that the executor group is able to clean up failing executors of such reason and replacing them with new ones?
Expected behavior
When an executor fails in a group, the group should heal itself by removing failed and shut down executors and replacing them with new ones.
Actual behavior
An executor which was shut down by an error and not by server shutdown remains in the executor group and it causes a stable failure rate in server calls
Steps to reproduce
Breakpoints were set into RedisPublisher.OnComplete.run and MonoNext.onError
An artifhttps://github.com/reactor/reactor-core/blob/master/reactor-core/src/main/java/reactor/core/publisher/MonoNext.java#L88
Because the issue was rare, I needed to artifically create the situation that happened by overriding values. First of all, in the RedisPublisher.OnComplete, I set 'signal' to an exception only to go into MonoNext's onError handler after a while. At that point, what is the fishy story is that this.done was already true. I set that values as well to true with the debugger and in this way, the exception will roll out to that particulare shutdown code.
Minimal yet complete reproducer code (or URL to code)
https://github.com/martin-tarjanyi/lettuce-executor-reject-repro
Netty version
4.1.51
JVM version (e.g. java -version)
OpenJDK Runtime Environment Corretto-<IP_ADDRESS> (build 1.8.0_222-b10)
OS version (e.g. uname -a)
Windows 10

It came to my knowledge, that this certain onErrorDropped induced exception though is fixed from a certain reactor version:
https://github.com/reactor/reactor-core/releases/tag/v3.4.0?fbclid=IwAR0t73ArcwxX3MpIDlhB4Q-rXpd0ROHRo7VGhrHo37izBSmP82WzoeCyavg

It came to my knowledge, that this certain onErrorDropped induced exception though is fixed from a certain reactor version:
https://github.com/reactor/reactor-core/releases/tag/v3.4.0?fbclid=IwAR0t73ArcwxX3MpIDlhB4Q-rXpd0ROHRo7VGhrHo37izBSmP82WzoeCyavg

Very nice of y'all that you don't even consider checking this, I mean I don't care anymore, but there should have been an answer at least.

I suggest to open this up to the reactor core team or who is responsible to use Netty as a library. I don't see yet (we need a full Netty only reproducer) to consider this problem as something which Netty has to solve 🙏

If you don't care it anymore @ujhazib please close it 🙏


Minicpm enabling

What does this PR do?
Enables MiniCPM3 model for Causal LM. Follows #1133 in optimizing remote code. The following changes were added:

htcore.mark_step() between each decoder layer to reduce graph size
Add token_idx support passthrough to Attention implementations. (Currently incomplete, missing reset of cache)

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

Performance benchmark
Command:
python run_generation.py --model_name_or_path openbmb/MiniCPM3-4B  --max_new_tokens 512 --prompt "推荐5个北京的景点。" --batch_size 1 --temperature 0.7 --do_sample --top_p 0.7 --use_kv_cache --use_chat_template [--bf16] [--use_hpu_graphs]

| throughput (tokens/s) | Gaudi 2 Lazy mode | Gaudi 2 Graph mode |
| BF 16 | 8.77 | 65.16 |
| FP 32 | 9.89 | 44.46 |

@pi314ever , you need to add jsonschema and datamodel_code_generator into requirements.

@yao-matrix I have removed the tokenizer and validated it to run with default requirements.txt.

@pi314ever  It's not clear what configurations that you have tested to enable this model.
Could you provide all the configuration that you tested (Example below) - You can open HS ticket and log details there.

batch_size
input len
output len
compile time
first token
throughput
ref throughput
memory

1
128
128

max_batch or best batch
128
128

1024
1024

2048
2048

128
2048

2048
128

4k?   16?

large   sequence length?

@jiminha  Tested configurations (with bf16):

batch_size
input len
output len
compile time (s)
first token (ms)
throughput (token/s)
memory (GB)

1
8
512
25.43
36.6
65.317
9.87

1
128
512
25.817
38.5
64.26
10.2

1
128
2048
107.52
166.8
58.296
14.62

1
512
2048
109.828
201.5
56.996
15.35

2
8
512
26.207
58
128.89
11.29

2
128
512
26.453
62
126.96
11.97

2
128
2048
121.166
306
103.69
20.12

2
512
2048
127.39
379
98.587
22.19

8
8
512
41.216
183.8
316.57
19.82

8
128
512
42.15
186.8
308.17
22.35

8
128
2048
240.903
1187
208.23
55.22

8
512
2048
263.52
1483
190.47
63.49

Testing command:
python run_generation.py --model_name_or_path openbmb/MiniCPM3-4B  --prompt "推荐5个北京的景点。" --temperature 0.7 --do_sample --top_p 0.7 --use_kv_cache --use_hpu_graphs --bf16 --max_input_tokens $INPUT_LEN --max_new_tokens $OUTPUT_LEN --batch_size $BATCH_SIZE

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Fixed log rotation for Mesos master/agent logs.

High Level Description
Given a logrotate config with neither a time-based rotation criteria, nor a size option, logrotate will default to rotating files once they hit 1 MB.  The maxsize option apparently only takes effect if there is a time-based rotation option (i.e. daily or weekly).
Given the current logrotate config, log retention is 256 times smaller than intended.
Related Issues

None at the moment.

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:

N/A

@mesosphere-mergebot label Ready For Review

@kaysoky Who else would be good to review this PR?

needs one more review before ship it

@kaysoky Is there a JIRA for this? Might we want to backport this to 1.9.1/1.8.9?

There's a private JIRA: https://jira.mesosphere.com/browse/DCOS-15653
In terms of backporting, we can do so easily enough (earliest version this affects is the 1.8.x branch).

Great, thanks! Now that this is approved, could you open the backport PRs for 1.8.9 and 1.9.1 branches, and we'll get them promptly approved.

@mesosphere-mergebot bump-ee

Cherry-picked into 1.8.9 in https://github.com/dcos/dcos/pull/1569 but we still need a 1.9.1 cherry-pick/PR


Achievement: Refactor (Part 2)

Description
This PR focuses on refactoring Dashboard and the editor(right) panels of Control.
For more details: Achievement Goal Rework
Changes

Update Achievement Types & add Expression Types based on latest changes on Achievement Goal Rework
Remove unnecessary props, subcomponents and tests in Control
Implement useContext for inferencer to be shared across subcomponents in Dashboard, Control
use NaN  instead of -1 as focusId, newId default state
Add <Prompt /> for confirmation before leaving Control with unpublished changes
Add showSimpleConfirmDialog for confirmation before deleting achievement/goal
Rename: cardTileUrl -> cardBackground, canvasUrl -> coverImage (fixes #1448)
Use cloneDeep for editable achievements & goals to prevent overwriting inferencer before admin clicks 'save'
Add Tooltip for buttons in Control
Major style updates for Dashboard & Control

How to Test
The Staging Backend is not updated yet. Set REACT_APP_USE_BACKEND=FALSE in the .env file to test with mock data.

Pull Request Test Coverage Report for Build 7446

45 of 341 (13.2%)  changed or added relevant lines in 35 files are covered.
18 unchanged lines in 9 files lost coverage.
Overall coverage decreased (-2.0%) to 31.126%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/achievement/view/AchievementViewCompletion.tsx
0
1
0.0%

src/commons/achievement/view/AchievementViewGoal.tsx
0
1
0.0%

src/features/achievement/AchievementActions.ts
2
3
66.67%

src/features/achievement/ExpressionTypes.ts
1
2
50.0%

src/commons/achievement/AchievementOverview.tsx
0
3
0.0%

src/commons/achievement/AchievementTask.tsx
0
3
0.0%

src/commons/achievement/card/AchievementXp.tsx
1
4
25.0%

src/commons/achievement/control/AchievementPreview.tsx
0
3
0.0%

src/commons/achievement/control/achievementEditor/AchievementAdder.tsx
0
4
0.0%

src/commons/achievement/control/goalEditor/GoalAdder.tsx
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/commons/achievement/AchievementView.tsx
1
0%

src/commons/achievement/control/AchievementPreview.tsx
1
0%

src/commons/achievement/control/GoalEditor.tsx
1
0%

src/commons/achievement/view/AchievementViewCompletion.tsx
1
0%

src/pages/achievement/control/AchievementControl.tsx
1
0%

src/pages/achievement/subcomponents/AchievementDashboard.tsx
1
5.26%

src/commons/achievement/control/AchievementEditor.tsx
2
0%

src/commons/achievement/view/AchievementViewGoal.tsx
2
20.0%

src/commons/achievement/utils/AchievementInferencer.ts
8
58.29%

Totals

Change from base Build 7431:
-2.0%

Covered Lines:
2109

Relevant Lines:
6350

💛  - Coveralls


Add examples to search for @userhandle mentions

In the readme and examples, we have search for text, hashtags and listing posts from a user.
However, maybe include an example to search for @userhandle mentions as well?
That would complete the program

The program is already capable of that, you can try the "advanced search" of twitter. You can set-up an advanced query there and copy the the resulting string query and give it to the program:
Example:

For tweets mentioning @elonmusk:

python scraper --query="(@elonmusk)"

For tweets that mentions @elonmusk with atleast 1000 replies from January 01, 2020 - August 31, 2023:

python scraper --query="(@elonmusk) min_replies:1000 until:2023-08-31 since:2020-01-01"

Great, advanced use, thank you


Develop vighneshwar

Removing if else that we don't need anymore.

Opened this PR to show how git works. It adds no value here, sorry.

Hey thanks for teaching people about git and OSS contribution. Feel free to use this repo anytime.


Prepare release v1.2.0

Codecov Report
Patch and project coverage have no change.

Comparison is base (cd71e56) 93.02% compared to head (dc6160a) 93.02%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #125   +/-   ##
=======================================
  Coverage   93.02%   93.02%
=======================================
  Files          52       52
  Lines         788      788
  Branches      121      121
=======================================
  Hits          733      733
  Misses         24       24
  Partials       31       31

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Component split

I would like to get your opinions about splitting zend-cache into different components.
The reason for that is simply a testing and dependency hell.

It's not possible to define a required extension in composer.json so all adapters have to manually check if the current environment is matching the requirements
Testing against all adapters is a nightmare and takes very long.

The travis.yml file al already very complex but it still doesn't test all common cases.
Some extensions are missing or not 100% compatible for PHP-7 which makes things here more complicated.
Doesn't test against different (most common) extension versions
Nearly impossible to run tests for ZendServer adapters

another reason is that normally you only need 1 or 2 adapters but that's not very important as zend-cache doesn't install so many files per adapter.

In my opinion it makes sense to split parts into it's own repository as long as the part requires a non standard extension or another currently optional dependency.
Thoughts ?
This is a structure how this could look like:
zendframework/zend-cache

Exceptions
Patterns
Service
Factories
Storage Interfaces
Storage Plugins

all except the Serializer plugin

Storage Adapters

BlackHole
Filesystem
Memory

zendframework/zend-cache-serializer

adds the serializer storage plugin which requires zend-serializer

zendframework/zend-cache-apc

adds the Apc storage adapter requires the apc extension (or compatible extension)

zendframework/zend-cache-apcu

adds the Apcu storage adapter requires the apcu extension

zendframework/zend-cache-dba

adds the Dba storage adapter requires the dba extension

zendframework/zend-cache-memcache

adds the Memcache storage adapter requires the memcache extension

zendframework/zend-cache-memcached

adds the Memcached storage adapter requires the memcached extension

zendframework/zend-cache-mongo

adds the Mongo storage adapter requires the mongo extension
It's currently the MongoDb adapter but there is also another extension called mongodb

zendframework/zend-cache-redis

adds the Redis storage adapter requires the redis extension

zendframework/zend-cache-session

adds the Session storage adapter requires the zend-session

zendframework/zend-cache-wincache

adds the WinCache storage adapter requires the wincache extension

zendframework/zend-cache-xcache

adds the XCache storage adapter requires the xcache extension

zendframework/zend-cache-zendserver

adds the ZendDataCacheDisk and ZendDataCacheShm storage adapter requires the zend data cache extension
This is currently called ZendServerDisk / ZendServerShm because the extension is part of Zend Server

This makes a ton of sense as far as I'm concerned; it's exactly what we're doing in zend-mvc currently.
I'd recommend:

Start creating the new packages. IIRC, you have the ability to create repos, @marc-mabe, so you can either create them directly here, or create them first under your username, and then move them to this organization. When each has a stable tag, ping me so I can add them to packagist.
On the develop branch of this repo, as we add the new packages to packagist, remove the adapter and related files via a pull request, and add the new package as a suggestion.

With regards to the AdapterPluginManager, the new components should be made compatible with zend-component-installer. In doing so, you can have the Module class return configuration for the adapter plugin being exposed. (You should likely also do a ConfigProvider so folks can use it with Expressive!) I can provide more guidance on this aspect as needed.

:+1: It's the logical continuation of the hard work done by @weierophinney.

@weierophinney sorry for the late response. Nice to hear your ACK.
There are some details I like to discuss before:

Should we introduce a test repo zend-cache-test which includes common tests like https://github.com/zendframework/zend-cache/blob/master/test/Storage/Adapter/CommonAdapterTest.php that can be re-used by all other sub-components?
If I understand correctly the component installer is used to install ZF components within ZF applications but using zend-cache as a component in another context this approach doesn't work.

Instead it should be possible for sub-components to inject itself into the *PluginManager by using composer autoload -> files. This way a sub-component would be available out-of-the-box just by adding it by composer.

I have a lot of open PRs ready which I would like to merge before but reviewing takes ages. I don't think it would be a good idea to merge my own stuff but currently my speed of operation goes down a lot just for waiting for someone else that is able to review my own stuff and pinging and pinging again.

Cheers

@weierophinney @Ocramius Today I had some time and moved already a lot of repos from zend-cache into it's own.
Could you please take a look if that's on a good way? Thanks!

https://github.com/marc-mabe/zend-cache
https://github.com/marc-mabe/zend-cache-test   (Reusable testing utilities / common test classes ...)
https://github.com/marc-mabe/zend-cache-serializer
https://github.com/marc-mabe/zend-cache-apc
https://github.com/marc-mabe/zend-cache-apcu
https://github.com/marc-mabe/zend-cache-memcache
https://github.com/marc-mabe/zend-cache-memcached
https://github.com/marc-mabe/zend-cache-dba
https://github.com/marc-mabe/zend-cache-xcache

Cheers

Sorry, I didn't join the discussion before.
Yes, this looks good, especially if we can manage to make the extension dependencies required when picking single packages (that would require requesting the single packages via composer, manually).
Unsure if the serializer needs to be extracted: probably worth keeping test, cache and serializer in a single package, while the adapters move each to a single component if they require particular extensions.
I'm also not sure if it's worth splitting out cache components that don't require extensions...

@Ocramius Thanks for your comment!
I have moved the serializer plugin into another package as it requires the zend-serializer package.
Same (will) happen to the session adapter which requires zend-session.

I'm also not sure if it's worth splitting out cache components that don't require extensions...

Yes that's basically all about. I have one exception only, the zend-cache-test package. This package contains some reusable / common tests and will be used by all other zend-cache* packages in development. The reason for that is that I want to have this dependency only for development require-dev and not on normal installations.
The following packages also needs to be moved:

the redis storage adapter
the mongodb storage adapter

Also I like to rename this to just mongo storage adapter as it's using ext-mongo and not ext-mongodb

The session storage adapter

Everything else should be kept in the main repo as it doesn't have special dependencies.
Right now I have one problem that I don't know how to solve:
The component contains a factory StorageFactory used to instantiate storage adapters and plugins like
StorageFactory::factory([
    'adapter' => [
        'name'    => 'apc',
        'options' => ['ttl' => 3600],
    ],
    'plugins' => [
        'exception_handler' => ['throw_exceptions' => false],
    ],
]);

But how can I register the services on installation using the zend-component-installer?

@Ocramius There are actually the following problems:

These test files are located in tests/ and so will not be auto-loadable by composer as it's defined in autoload-dev.
If I move them into src/ and add to .gitattributes

Files are not available on downloading compressed package (Or am I wrong?)
We get test files in src/ not usable as part of package as it depends on PHPUnit but this is defined in require-dev

@Ocramius Also moved the zend-server adapters to an own repo and after several hours I could make the tests run on travis by installing Zend-Server, rewriting phpunit start script and testing over HTTP call :)
@weierophinney Please could you take a look at https://github.com/marc-mabe/zend-cache-zendserver/blob/master/.travis.yml and validate if that is ok and allowed how I run the tests.
Thanks.

I love the idea of separating out the various storage adapters to their own repositories. That way users are only getting code related to the adapter(s) they use, and we can have much easier, more targeted instructions on contributing (typically via vagrant and/or docker).
I've looked at the zend-cache-zendserver Travis setup, and it looks reasonable.
Would this split target a v3 release, then? And I assume we'd want each adapter brought into the zendframework organization, and tagged with a stable 1.0.0 release before removing them here, correct?

I've looked at the zend-cache-zendserver Travis setup, and it looks reasonable.

Thanks for review :smiley:

Would this split target a v3 release, then?

Yea, also I have one or two other things I want to archive for 3.0 because small BC break.

And I assume we'd want each adapter brought into the zendframework organization, and tagged with a stable 1.0.0 release before removing them here, correct?

Yes, of course before tagging a stable version of zend-cache with adapters removed the adapters have to have a stable version already and everything has to be in zendframework organization.
I was planning to start with adapter versions with version 3.0 as the adapters had stable versions before within zend-cache.

Does anyone have an idea ow knows someone how would be able to test the WinCache adapter.
I don't have a windows system at hand and even if so I would not like to fight setting up a microsoft webserver an windows in my free time.

@marc-mabe The only thing I can think of is to use a Windows vagrant image as a base, and then have the Vagrantfile download and install PHP to it. IIRC, Travis has windows images, too, which might make this possible there.

As the support for APC dropped years ago and there is no compatible PHP 7+ version, shouldn't we drop support for that extension at all?

@boesing We can when we bump to PHP 7.1.

This repository has been closed and moved to laminas/laminas-cache; a new issue has been opened at https://github.com/laminas/laminas-cache/issues/7.


Add recommended policy ARN for KMS controller

Issue #, if available:
Description of changes:

Add recommended policy ARN for KMS controller

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/retest

/lgtm

The release test is failing because kms controller is still using old runtime version.
Merging this PR since the failure is unrelated to this change.


Fully conform to Semantic Versioning 2.0.0 spec

This pull request makes the Version class conform fully to the semver spec, fixing https://bugs.swift.org/browse/SR-113.
I was unclear about the function of Specifier in this project, so I did not update that class. I would be happy to do so if need be!

Specifier doesn't want support there since it is intended to represent part of the 3 number tuple. Correct me if wrong @mattt

Will pull when we have tests. If you haven't the time, I can write them.

Great work!

@mxcl Correct. The Specifier type is defined to be a sparse counterpart to a Version, where trailing components can be omitted. That said, adding SemVer extensions components to Version with 4a7621 should probably have a corresponding change to Specifier.

@jakeheis Added some line comments with suggested refactoring. Great work, and thanks for your contribution!

@mattt! You're alive! :)

@mattt Thanks for the code review! I've refactored those parts you found confusing, hopefully should be good to go now
@mxcl I'd be happy to take a swing at writing the tests, I'll see what I can do

Will merge, if you can squash the history. Thanks.

👍🏻🎆


Documentation - Single Line Closure Updates

This PR adds a public docstring for the new error API and updates the public docstring formatting to be consistent. It also updates the README to be more tailored to the autoclosure APIs.

Looks good.