Handle bogus operation

to address https://github.com/jscep/jscep/issues/88

Thanks for the PR.
Failure was due to Travis CI communication with SonarQube.  Can you rebase to pick up the updated Travis configuration and rebuild please?

Done.

Whoops.  Sorry for the massive delay.  Tests passing now.

Can you add 5270d4888b6d1001d25b367033b155bd368dfc86 to the PR?

I think it's already in, no?  See https://github.com/jscep/jscep/pull/89/files#diff-600376dffeb79835ede4a0b285078036R331.

Thanks for merging!

Took me long enough!
On Wed, 19 Sep 2018 at 20:43, David Byron<EMAIL_ADDRESS>wrote:

Thanks for merging!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jscep/jscep/pull/89#issuecomment-422933341, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAnaOxYcY3FG6gGxUmiK-sgY5LRlLpyJks5ucp4_gaJpZM4Usj0s
.


fix: update to the latest API spec

What kind of change does this PR introduce?
Bug fix, feature, docs update, ...
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 6 (33.33%)  changed or added relevant lines in 2 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 57.741%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/projects.go
0
2
0.0%

internal/projects/create/create.go
2
4
50.0%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
11.76%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
6385

Relevant Lines:
11058

💛  - Coveralls


ufunc 'logical_or' did not contain a loop with signature matching types (None, <class 'numpy.dtypes.StrDType'>) -> None

Trying to load in the following co-efficients from CSV:
0.942154,1.06136
-1.88431,-1.88424
0.942154,0.884379

[ ERROR][00:42:40.211] [input_coeffs.py:853] While saving the filter coefficients, the following error occurred:
ufunc 'logical_or' did not contain a loop with signature matching types (None, <class 'numpy.dtypes.StrDType'>) -> None

I don't see anything in the GUI or any CLI args to get the version information, so I'm not sure what I have downloaded.

Thanks! Looks good o me so far.
By the way, I find the icons a little confusing on the clipboard import. Visually, it's not entirely clear to me which is import and which is export, and I generally have to hover over. Maybe moving the arrow a bit further to the side might make it clearer. I think also the way they point to the right kind of make it feel that the load one is taking things out of the application whilst the save is importing into the application. Perhaps making the load/import button appear to the left of the save/export might be more in-line with other applications, as typically load appears before save. But maybe it's just me.


Report an import_statistics issue to HA Core

As per discussion in #30

Create a code example describing the way we are using the import_statistics with sensor
Describe the problem and report it to HA Core

Could this discussion and current open PR be the solution we were looking for??
https://github.com/home-assistant/architecture/discussions/964
https://github.com/home-assistant/core/pull/100239

Is this fix in 23.10.1 related?
https://github.com/home-assistant/core/pull/101490

I will revert my version to a time when we were experiencing this issue and report back.

With the most recent version of Home Assistant I switched back to using the normal sensor(non statistic) as my water consumption on the energy dashboard, and it seems to be working great for the last 2 days

Amazing..I will try the same test!

I have tried this for the last few days as well, however, I still need to use the statistic sensor to see granular data.(I’m metric if that makes a difference)dOn Oct 21, 2023, at 3:48 PM, disforw @.***> wrote:
Amazing..I will try the same test!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

Released in #2.4.0


3rd grid trade bought above last buy price

Version
Version: v0.0.90 (48865a0)
Latest Version:
v0.0.90
Description
I have set up 6 grid trades for CHZUSDT that are only supposed to be executed when TV is Strong Buy.
Today a spike caused the grid trade to fire above the last buy price. Which caused the last buy price to rise as well.
{
  "orgLastBuyPrice": 0.19279549839228294,
  "orgQuantity": 311,
  "orgTotalAmount": 59.959399999999995,
  "newLastBuyPrice": 0.19326044905008632,
  "newQuantity": 579,
  "newTotalAmount":<PHONE_NUMBER>9999999
}

Expected Behaviours
Would it be possible not to order additional coins, once the current price is above last buy price?
I do not see any advantage in doing so, other than to wait for the coin to climb some additional points...

Hi @Rayn0r
Sorry for the late reply.
Can you provide the log when it's happening?


Average over the queries in the intersection of relevance judgements and results

Hello, I'm recently using the trectools and find that the evaluation is not the same as trec_eval. In trec_eval, the evaluation is always averaged over the queries in the intersection  of relevance judgements and results. However, in trectools, the evaluation is averaged over all queries in the results, and some of them may not be judged; they should be ignored rather than contribute to a value of 0.

Hi Yu-Shi,
Thank you for your message and for using our package.
That is a good point! Would you be able to provide a minimal example of the situation you are reporting?
Thanks,
Joao

Hello,
Thanks for your response!
You can use these runs: https://github.com/thunlp/ConversationQueryRewriter/tree/master/results and use the judgment here: https://trec.nist.gov/data/cast/2019qrels.txt. The evaluation metric is NDCG@3. If you run trec_eval and trectools, you will find that they give different results.
Please note that due to some errors, the evaluation results provided in the readme of ConversationQueryRewriter are slightly lower than the correct ones. I will fix them in the future.
Thanks,
Shi Yu

Hi Shi Yu,
Thanks for report this issue. I pushed a fix to a separate branch. Please see here.
It solves most of your problems, but there are still some pending things to do here.
Running a few tests locally, I noticed that:

The BPref results are not the same as trec_eval. The difference is for queries that have all the documents assessed as relevant (e.g., 31_3 of your qrels file).

num_ret differs from trectools and trec_eval, not sure why.

Would you be so kind to have a look at the remaining issues?
Thank you very much,
Joao

Hi Joao,
Thank you for the quick fixing. I haven't had a chance to check the things you said, but I check the NDCG@3 on the file bert_base_run_oracle.trec and find that the updated version still doesn't give the same result as trec_eval does: 0.5474 (trec_eval), 0.5488 (trectools).
I took some time to investigate it and found that the difference comes from query 69_2. The run is like that:
...
69_2 Q0 MARCO_38426 1 3.800645589828491 BERT-base
69_2 Q0 MARCO_3640412 2 3.6633083820343018 BERT-base
69_2 Q0 CAR_1e496da6dbe1e1c2d80b30970038b92a57799749 3 3.6544055938720703 BERT-base
69_2 Q0 MARCO_2900384 4 3.6544055938720703 BERT-base
69_2 Q0 MARCO_4056499 5 3.4888365268707275 BERT-base
...

The first three documents in that run (MARCO_38426, MARCO_3640412, and CAR_1e496da6dbe1e1c2d80b30970038b92a57799749) all have the relevance judgements of 4, thus trectools assign a NDCG@3 of 1. However, the result for this query from trec_eval is 0.7654; note that the 3rd and 4th document have exactly the same score, but MARCO_2900384 is an unjudged document (which should be treated as irrelevant) and I think that trec_eval does some kinds of sorting to place MARCO_2900384 in front of CAR_1e496da6dbe1e1c2d80b30970038b92a57799749.
It seems that trectools is aware of the document sorting problem and has such code to do the sorting (eg. in get_map() ):
        if trec_eval:
            trecformat = self.run.run_data.sort_values(["query", "score", "docid"], ascending=[True,False,False]).reset_index()
            topX = trecformat.groupby("query")[["query","docid","score"]].head(depth)
        else:
            topX = self.run.run_data.groupby("query")[["query","docid","score"]].head(depth)

But I can't find such a snippet in get_ndcg(). Is that a bug or for some special reasons?
Thank you again for your attention,
Shi Yu

For the problem of BPref, I set per_query=True and find that some queries disappear from the result list, such as 59_6, 61_3, 67_7. They are all judged, so they should appear in the per query list.

I'm sorry I may not have enough time to follow up on this recently. I find a note that may help you correct the implementation of BPref: https://github.com/usnistgov/trec_eval/blob/master/bpref_bug. Thank you a lot!

I check the NDCG@3 on the file bert_base_run_oracle.trec and find that the updated version still doesn't give the same result as trec_eval does: 0.5474

Hi Shi Yu,
You are right. There was an alternative tool that calculated NDCG without re-ranking ties, but it is better to use the same approach for all metrics within trec_tools. I will update it as well.

Regarding BPref, you are absolutely right. The line to blame is very likely this one here.
Let me know if you be available to help to fix it. Otherwise, I can work on that in May.
Thank you very much in advance!

I'm willing to help, but I'm busy with other things recently. Sorry to say that.
Looking forward to the updated version! 😊

Hi! I recently came across the same issue as OP. Any updates on the fix? I can provide help if you need @joaopalotti

Hi @snovaisg, thanks for your message. I will have some free time this weekend and should be able to work on this, but your help would be highly appreciated. I started a fix on this branch, but some metrics still need work, such as bpref.

Ok will take a look!

Hi @snovaisg, have you had any progress? Thanks!

Hi @snovaisg and @Yu-Shi. I finally had some available time and fixed this problem. Let me know if you have further comments and thanks for this fruitful discussion and suggestions for improvement.


Naming convention between Model and Api classes

Class names are the same for FAPI\Boilerplate\Api\Tweet and FAPI\Boilerplate\Model\Tweet\Tweet.
I understood same word should mean same thing:

Here the Tweet name can easily mean a Tweet in the model context. But what's about FAPI\Boilerplate\Api\Tweet? What is the role of this class? It's different and it has the same name.
How to avoid confusion?

Short answer: you can't always. In many cases you would have to use names with meta keywords in them (Collection, Model, Resource) which rarely fits into the models. Although in natural language you use them in parallel: "The List endpoint of Tweets [resource] returns [a collection of] Tweets" At least I haven't found a sane solution to this problem.
What I have in mind though is if the Api (Resource) classes should be plural or not (Tweet vs Tweets, like Stat vs Stats)

I think PHP classes and namespaces should always be in singular. But I have a hard time making singular of "stats". But to be consistent, we should do "stat".
However, ApiClient::stats() is still okey. It is a function.

Not sure. API Resources are usually plural (Organizations, Users, etc) and I see a bit of inconsistency here.

Yeah. Rest endpoints should be in plural. But it does not mean that we have to use plural in our API classes. Or.. we decide to use plural and it will solve the question in #46

Well, I don't insist, Github API is inconsistent about this as well: https://github.com/KnpLabs/php-github-api/tree/master/lib/Github/Api
No, it wouldn't. We have a Tweets collection object.


Added responsive layout for the jobs page

Changes:

Navbar as logo without text
Banner size is updated based on screen
About section is vertical first, then horizontal
jobs list on mobile and tablet, and table on desktop
Applications form full width inputs on mobile
Vertical footer items on mobile

updated


App crash when using specific Windows utilities

Hi,
I tried just_playback and it worked! But I have a problem integrating it with my app.
When there is an instruction where a specific Windows component is used, it crashes.
Can't open tkinter filedialog, application freezes.
Also, my module ported from c++ to use the Taskbar Thumbnail Toolbar causes the application to terminate when the ThumbBarSetImageList() function is called.
Working on Windows 10 with Python 3.10.
Example
from just_playback import Playback
from tkinter import filedialog
playback = Playback()
playback.load_file('music-files/sample.mp3')
playback.play()
filedialog.askdirectory(title="Select folder")

The sample code runs fine on my Linux machine.
This seems to be a problem with tkinter on Windows as discussed here  and here. Try issuing all calls on the Playback object in a separate thread.

It isn't an interface freeze error. The tkinter app works fine.
Crash when there is a call to a particular windows utility. Maybe a bad interaction with a dll.
Windows 10,11. In linux works fine.
from just_playback import Playback
from tkinter import filedialog
playback = Playback() #removing this initialization the problem disappears
filedialog.askdirectory(title="Select folder") #not shown, crash
print("not executed")

No problem with a new thread. I don't know why it works, but it's not the right way

No problem with a new thread. I don't know why it works, but it's not the right way

So it does work in a separate thread? And if it does why do you feel it's not right?

It worked with the pygame.mixer module without the need for a new thread.
The tkinter app works fine, it even plays songs with just_playback and everything works fine. Except when I use filedialog() or Taskbar Thumbnail Toolbar in Windows.
A non-blocking call such as Playback() should not crash specific functions.
I can't do ctrl+c to terminate the example program.


Upgrade packages in .tool-versions and gems

Context
Latest package update before launch. I won't be doing these as often once we go live.
bundle update --bundler

brew upgrade asdf
asdf plugin update --all

asdf install nodejs 19.7.0
asdf local nodejs 19.7.0
npm install --global yarn

asdf install redis 7.0.9
asdf local redis 7.0.9

Hasn't been approved quickly, so I'll close it for now.


feat: upload file by drag and drop

Screen:

Detail:
Upload multiple files:
There is a race condition bug in the old resource upload where consecutive calls to createResourceWithBlob can cause errors in the resource list. To address this issue, I have created a new function called createResourcesWithBlob that can process multiple files simultaneously.

LGTM overall.
And I find a style issue:

It has been resolved by adding "w-full".
now:

The height is flash yet. Because can't add the min-h-32(didn't present) to the className

LGTM


[Snyk] Upgrade yup from 1.2.0 to 1.3.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade yup from 1.2.0 to 1.3.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 3 versions ahead of your current version.
The recommended version was released 2 months ago, on 2023-09-29.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (fef9c9a) 99.67% compared to head (354166a) 99.67%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #108   +/-   ##
=======================================
  Coverage   99.67%   99.67%
=======================================
  Files         178      178
  Lines        3385     3385
  Branches      545      545
=======================================
  Hits         3374     3374
  Misses         11       11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Implement alignment for allocator

Support >8 Bytes alignment by allocating more space and storing the real pointer.

Thanks for the PR! I've checked it out and seems to work well.


Property aggregate query causes panic if the call to BatchGetAssetPropertyAggregatesPagesWithContext needs to be paged

What happened:
Run a property aggregate query with resolution set to auto and increase the time range until it causes the call to BatchGetAssetPropertyAggregatesPageAggregation needs to be paged.
This causes a panic because there is a mismatch of expectations when building the frames because we assume the success entries is flat with each item representing a single asset https://github.com/grafana/iot-sitewise-datasource/blob/9ea952893e9da5f8f719ecc1dd462311d34ea3ac/pkg/framer/property_aggregate.go#L109-L110
See original comment on this issue here https://github.com/grafana/iot-sitewise-datasource/pull/179#issuecomment-1535064145
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Screenshots

Anything else we need to know?:
Environment:

Grafana version:
Plugin version:
OS Grafana is installed on:
User OS & Browser:
Others:

Moving this to Next since I see a user is running into this issue

this should be fixed by #194 as well


glib: follow profile updates when searching apps

Motivation for this change
GNOME Shell monitors XDG_DATA_DIRS for changes using inotify for the
purpose of updating the list of applications displayed in the overview
as they are installed or removed. Because the relevant directories are
symlinks under NixOS and are swapped on change, Shell never notices it.
To remedy this, also monitor various profile directories to signal an
update whenever profiles are changed.
This uses a patch from Guix, slightly modified to also encompass user
environment profiles (/etc/profiles/per-user/$USER) in addition to the
system profile (/nix/var/nix/profiles/system) as well as regular user
profiles (/nix/var/nix/profiles/per-user/$USER/profile).
Fixes #12757
Known limitation: On installations to /etc/profiles/per-user, the list of applications gets updated, but Shell fails to find the icons for new applications. This is the case when installing them declaratively through home-manager's NixOS module. Other methods, such as ad-hoc via nix-env/nix profile install work fine.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 118100 at 5fa00518 run on aarch64-linux 1

830 packages marked as broken and skipped:

OVMF-CSM
adoptopenjdk-hotspot-bin-13
adoptopenjdk-hotspot-bin-14
adoptopenjdk-jre-hotspot-bin-13
adoptopenjdk-jre-hotspot-bin-14
agdaPackages.iowa-stdlib
aldor
alliance
ams-lv2
amule
...

16218 packages skipped due to time constraints:

AusweisApp2
DisnixWebService
EBTKS
EmptyEpsilon
Fabric (python38Packages.Fabric)
MIDIVisualizer
MMA
OVMF
OVMF-secureBoot
R
...

31 packages built successfully:

asciidoctor
atk (gnome2.atk ,gnome3.atk)
cairo (gnome2.cairo)
dejavu_fonts
desktop-file-utils
epoxy
fontconfig
fontforge (fontforge-fonttools ,python38Packages.fontforge)
gd
gdk-pixbuf
glib (gnome2.glib ,gnome3.glib)
libsoup (gnome2.libsoup)
pango (gnome2.pango)
gobject-introspection (gnome3.gobject-introspection)
vala (gnome3.vala ,vala_0_48)
gts
harfbuzz
json-glib
libnotify
libqrtr-glib
libva
libva-minimal
mesa
python38Packages.pycairo
python38Packages.pygobject3
shared-mime-info
spideroak
wayland
wayland-protocols
xlibsWrapper (xorg.xlibsWrapper)
xorg.libXft

/rebase staging

Seems to me that the patch is a little limited? It doesn't work with home-manager, would it not be better to special-case all links to /nix instead of special-casing system and user profiles?

maybe the solution is to check here if the directory it wants to watch is a symlink to /nix, and if so, pass FALSE instead of TRUE?
https://github.com/frida/glib/blob/ceb3ada29104dc85dbb116e7dca32ba565aad368/gio/gdesktopappinfo.c#L1433

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/required-nix-gnome-tweaks-hacks/20711/3

I'm still have the same exact issue on NixOS 22.11


Add json output for Labs and Projects

There is already a PR on this topic with a different approach which was unfortunately not completed https://github.com/okfde/codefor.de/pull/206
The front matter configuration on content/labs/_index.md and content/projekte/_index.md ensures that only for these two pages a json is generated.
The jsonify function takes an object and converts it to an RFC 8259 compliant JSON object.

Ping @johnjohndoe can you take a look since you're the author of #206?

@iamsilvio can you add some information about the endpoints to the README file?

Should I describe where the data is located after deployment or how to build this for other sites?

Should I describe where the data is located after deployment or how to build this for other sites?

I would suggest adding a section to the README which explains that there is an API which offers the site content as machine-readable data. The top endpoints can be written down but I won't get lost into details which might quickly change over time. Please also mention which websites parts are not available via the API.

When I worked on the API in #206 I run into the problem that the rendered JSON is not valid. I see the same in your branch:

Are you planning to resolve the errors?

I have validated the output!

Interesting! I trusted the JSONView browser extension (until now). What tool are you using?

https://jsonformatter.curiousconcept.com

But you are right, it's valid according to RFC 8259 but not to ES5 https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Trailing_commas#trailing_commas_in_json

Did you see the iscjklanguage property? Seems like a strange name - I cannot find it in the sources.

But you are right, it's valid according to RFC 8259 but not to ES5 https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Trailing_commas#trailing_commas_in_json

Maybe that's why JSONView is unhappy. I can't say.

Did you see the iscjklanguage property? Seems like a strange name - I cannot find it in the sources.

Seems like this is predefined by hugo https://gohugo.io/content-management/front-matter/#predefined among others.
IMO these predefined fiels are  implementation details that shouldn't be exposed through the files. We don't want people to rely on these fields in case we ever change the underlying generator

Ping @johnjohndoe can you take a look since you're the author of #206?

@ubergesundheit I like what @iamsilvio configured here. It is much less custom code and therefore much easier to maintain. - I guess what is left is to make the CI pipeline happy. I have no insides here.

Nice. JSONView is happy now :heart_decoration:

:rocket: Nice. Are you going to add endpoints for individual projects and labs?

🚀 Nice. Are you going to add endpoints for individual projects and labs?

Not today but if you can use it, I will do it next week.

Can you add one or two sentences about this API to the Readme.md file in this repos root?


Smart cli like youtube-dl instead of menus

Prerequisites

[X] Did you read FAQ section in readme.md?
[X] Did you test with the latest releases or commit ?
[X] Did you search for existing issues in Issues?

Description
Thanks for making PU2! (name too long)
I would be nice if PU2 acted more like youtube-dl, you give him a link and it guesses what you want, which is, most likely, to just download the image. You can then add more flags if you want more, like youtube-dl LINK --write-description
Expected behavior:
$ pixiv pixiv.net/artworks/123
Image at ./image.png

$ pixiv pixiv.net/users/111
Downloaded everything to ./

$ pixiv -h
pixiv
   -h help
   --by-rank download by rank
   --by-member download by member id

Versions
PixivDownloader2 version 20230105

I mean you can send it command line arguments to bypass the menus to get exactly what you want. PRs are welcome if you want to implement it, else gallery-dl does exactly what you're suggesting

Guess we have a bug then?
Doesnt matter which command I run, I get the menus, I suspect the git error at beginning is the cause, I just ran the pixiv command after installing from AUR
[@USER]$ pixiv --help
error: cannot pull with rebase: You have unstaged changes.
error: Please commit or stash them.
┌──────────────────────────────────────────────────────────┐
│ PixivDownloader2 version 20230105                        │
│ https://github.com/Nandaka/PixivUtil2/releases           │
│ Donate at https://bit.ly/PixivUtilDonation               │
└──────────────────────────────────────────────────────────┘
Reading /home/USER/.local/share/PixivUtil2/config.ini ...
Configuration loaded.
Using default DB Path: /home/USER/.local/share/PixivUtil2/db.sqlite
Creating database... done.
Only process members where the last update is >= 7 days ago
Username login is broken, use cookie to log in.
Trying to log in with saved cookie
Logging in, return url: https://www.pixiv.net/en/
Login successful.
My User Id: xxxxxxxxx.
Premium User: False.
┌──────────────────────────────────────────────────────────┐
│ PixivDownloader2 version 20230105                        │
│ https://github.com/Nandaka/PixivUtil2/releases           │
│ Donate at https://bit.ly/PixivUtilDonation               │
└──────────────────────────────────────────────────────────┘
── Pixiv ───────────────────────────────────────────────────
 1.  Download by member_id
 2.  Download by image_id
 3.  Download by tags
...

Indeed the source was changed
On branch master
Your branch is up to date with 'origin/master'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   PixivUtil2.py

Untracked files:
  (use "git add <file>..." to include in what will be committed)
        pixiv_venv/
        "\343\202\265\343\202\244\343\203\263\343\201\223\343\201\231 (11229342)/"

PixivUtil2.py was given execute permission, restoring the file to HEAD we have this
.rw-r--r--  77k sakura sakura 16 Dec 00:41 PixivUtil2.py

My guess is that the script is not executable by default, and the AUR script tries to make it executable, which causes the git error at the beginning, PixivUtil2 drops whatever command it has, and opens the menu

Even after commiting it still always opens the menu pixiv -h , maybe the aur script is broken

I'm on windows and It works. e.g. PixivUtil2.py -s 4 -f list.txt --is -x
To export followed artist PixivUtil2.py -s  e  -p  y -x --ef export.txt

Looking at the /usr/bin/pixiv helper script in the arch package that loads the venv, it looks like the command line arguments aren't being passed onto PixivUtil2.py
Adding $@ to the end of each $HOME/.local/share/PixivUtil2/PixivUtil2.py call will fix it.

@nicodoggie Would this not be an issue for the arch package maintainer?

Totally, thanks for telling me @nicodoggie

Would this not be an issue for the arch package maintainer?

Most definitely. I suppose I wanted it to be on the record here, since I saw this thread before I tried to investigate myself. I fully intend to report it to the AUR


Automatically link backend and gateway?

Should we simplify the workflow by defaulting or autodetecting the connection between the backend and gateway?

I'll look into this as we discussed.

will be addressed with #27

done.


Create Dataset for single label Text categorisation

I have provided a link to download a dataset for single label text classification

hi addy, it is possible to process and split it into train/dev/test set? currently after unzip the file, there is no clue how to use it.


Remove red text box from top of each datasets page

An accessibility audit recommended that the 'the exemptions' link in the red text box that is displayed at the top of each dataset page, i.e. the text box with the title 'Some of these data elements might not be available for research purposes', should have a more detailed contextual description for users of accessibility software.
However, as it's no longer intended that the F&E tool will be used directly in the NPD data application process, the red text box is no longer needed and should be removed from each dataset page.

NB - this ticket has been approved by the Product Owner, Gwen McGill

I have tested this change and confirm it has passed the tests.

Awaiting the result of the accessibility test on 14/09/2020.

The accessibility re-test picked up no issues, so this change is ready to push to production. However, I'll do a final check with Chris Douglass and Martin Johnson, to confirm that they're happy with the change.

Users have checked change. Moving to 'Ready to Release'.

Deployed to production and tested. Awaiting approval by PO, Gwen McGill, before closing the ticket.

PO has agreed to close the ticket.


What roles can NMDC DH play in the future?

Human + API repair enrichment and curation of EMSL, JGI or  KBase data?
Integration with NMDC runtime?
@hubin-keio and @scanon, do you have anything to contribute?

My main concerns are DH is going to play with the NMDC run-time API (or other APIs). Now unique sample identifiers are assigned by the NMDC run-time API but without associated sample metadata. Will there be a single authority of truth for sample information across NMDC? And do we have a clear path of linking sample metadata from JGI and EMSL?

I don't believe there is any actionable task here. Closing.


from __future__ import annotations not respected

on recompile of a function definition change, jurigged does not respect from __future__ import annotations
say you have a module like this:
from __future__ import annotations

def f(a: tuse) -> int:
    ...

and say you run jurigged my_module.py
and say you additionally change:
def f(a: tuse) -> int:
# to
def f(a: tuse) -> str:

jurigged raises an error in reevaluation (NameError: tuse is not defined or something like that).
the reason is that the appropriate future flags aren't passed into the compile function.
i'll push a fix for this shortly, just wanted to create an issue first.

created #37 for this a few days ago, @breuleux if you want to take a look when you have a chance

created PR #37 for this issue a few days ago. @breuleux if you can take a look when you have a chance.

👋 Thanks for the fix here. Since the PR was merged, can this issue be closed? Thanks!

Yeah, I had to publish a new version on PyPI first, so that's done.


Update examples to use wifi_info and connectivity package

Issue received in email:

HI,
Sir as I was going through the documentation and trying to figure it out to work I would like to inform you that the connectivity package has been updated and the breaking change is (The getWifiName, getWifiBSSID and getWifiIP are removed to wifi_info_flutter).
Also sir I would really be thankful if you could suggest to me that I should put the GET wifi code in the wifi_info.dart file in your example code.
Thanks for your help and Thankyou for working on getting esptouch to flutter
🙌

See changelog in the connectivity package:

2.0.0 [Breaking Change] The getWifiName, getWifiBSSID and getWifiIP are removed to wifi_info_flutter.

@vargavince91  Do we need update package for the compatibility with connectivity package ? or it just the example issue ?

@ducnguyenenterprise This is only about updating the example so that it uses the latest versions of its dependencies.

@ducnguyenenterprise This is only about updating the example so that it uses the latest versions of its dependencies.

Another deprecations
wifi_info_flutter --> network_info_plus

This plugin has been replaced by the Flutter Community Plus Plugins version, network_info_plus.

connectivity  --> connectivity_plus

This plugin has been replaced by the Flutter Community Plus Plugins version, connectivity_plus.

@vincevargadev I don't see any usage for them in the example, only comments talking about using
https://github.com/smaho-engineering/esptouch_flutter/blob/64832e7fc5bffa23e5d69d0a5c0268a614cd8f45/example/README.md?plain=1#L85it.
Did you mean that or have I missed something?.


fix import from MoM 1.9.3 on android

Changes to allow Valkyrie to find MoM 1.9.3 data on Android.
The newer versions of the official app moved the locations of the installed data around.
Tested as working on Android 8 - Android 10 (inclusive). It doesn't work on Android 11+ due to scoped storage.
It now has a solution for importing assets on Android 11+ as well.

These changes are on top of the fixes previous fixes for importing from official MoM 1.9.3.
The previous fix was enough for the import to work on mac and windows but on android the data locations also were moved around in the newer versions.
On Android 11+ cross app access data access is restricted so the import fails. The only way I can see around this is the "Hosted file import" idea or having the user copy the data manually using a file browser which has full access to the filesystem.

After looking at the Proof of Concept:
https://github.com/folderv/androidDataWithoutRootAPI33
I been able to get access to import the MoM official app data on Android 11 and Android 13.
By asking the user for permission to access specific folders, like so:

This is a build with this permission fix: Valkyrie-android-2.5.9d.apk

Awesome! It would be fantastic if this would really solve the problem.
Did you also ensure to catch the case that the user revokes the permission to the folder/cancels giving the permission initially?

The permission will be requested each time the user clicks import/reimport. Once the import has completed the permission isn't needed anymore.
Currently if the user jumps out of request window without accepting or changes the "use this folder" to a different folder the import won't work properly so I need to make a way to catch that.

Also I've just realised I only implemented the permission request for MoM and I need to implement it for Road to Legend on Android 11+
That shouldn't be too much extra work.
sigh Android Storage Access Framework and Permissions are kinda painful. Google will probably come up with some new changes that will break things in newer android versions.
I note Road to Legend official app isn't even available on android 13

I note Road to Legend official app isn't even available on android 13

Oh I did not know that. So it is not even possible to install it on an Android 13 device?

Some discussion regarding Road to Legend App on android 13 came up here. It is not working on Android 13 phones with 64Bit only but working on older phones with Android 13. Therefore your fix should work but somebody else will need to verify it once a build would be ready (I currently have not Android 13 device to test this).

Right. Agreed. I only have an old device that can run android 11 and device (which is too new) that runs android 13 for testing.
The mechanism for permission and import should work just as well for Road to Legend as for Mansions of Madness but yes it would be good to confirm once I get a chance to complete the fix.

I've added Road to Legend to the permission request code.

@seinsinnes Is anything code still missing in this PR? If not I will make a code review later today and then merge it to 2.6 release branch. Then we could set up a release package afterwards and ask the community to test it.
I think you already created release packages before (since getting the release bat script ready takes quite an amount of work)?

@Quantumrunner
I wish I could say yes it is ready.
I've just realised that the RtL official app stores its data differently that the current MoM offical app.
RtL puts it in Android/obb instead of Android/data. That I think is the older setup.
I'm just going to have to put in a bit more logic to choose the right locations to import from.

Ok no hurry then. I just wanted to ensure you don't get the feeling this PR will stay open forever. This PR and the other changes you made for Version 2.6 will definitely make some huge improvements for future prove of Valkyrie.

@Quantumrunner Thanks for following this up. I was beginning to think that these changes (and the general asset import changes) would never make it into the main Valkyrie repo.
I've made that fix and tested RtL import and MoM import on Android 11.
And MoM import on Android 13.
I note the import takes a pretty long time on the average android device also there maybe a memory leak in the import code. It works but maybe not optimally.

This is the binaries build from the branch:
Valkyrie-android-2.5.9d.apk
valkyrie-linux-2.5.9d.tar.gz
valkyrie-macos-2.5.9d.tar.gz
valkyrie-windows-2.5.9d.7z
valkyrie-windows-2.5.9d.exe
valkyrie-windows-2.5.9d.zip

I tried your build (I have an Android 13 tablet to test now). I can't get it to work because I can not navigate to the correct folders. When I click the import button the file chooser opens but it only shows folders like DCIM, Downloads, Videos  etc.

When you go to import. It should look like this:

You need to be able to just click "Use this folder" if you have to navigate around it probably won't.
If you don't see similar to the images then the file system structure on the tablet must be different than on my phone.

Are you trying to import MoM data?
I wonder where the data is if not Android/data/com.fantasyflightgames.mom

If you see DCIM, Downloads, Videos  etc. do you also see the Android folder?

Yes but the Android folder only shows Media subfolder instead of any other files.
I have installed Total Commander App to see if it has the same problem and it does. In total commander I can go to /storage/emulated/0/Android/obb but when I then click at com.fantasyflightgames.rtl a dialog opens with the message:
On Android >=13, the external SD card is write protected.

Location: Android/obb/com.fantasyflightgames.rtl

You need to choose the directory from a special Android dialog to give Total Commander write rights

Clicking Yes in the dialog opens the same file picker you implemented and then I'm stuck with the same problem. Instead of choosing a folder automatically it just shows the file picker dialog on basic level and a warning in the top
Can't use this folder

To protect your privacy, choose another folder

Furthermore clicking then back in the file chooser goes back to Valkyrie which is then stuck with a black screen.

We should definitely investigate if clicking the back button in the folder picker dialog can be catched so that the user is not stuck with a black screen in Valkyrie afterwards.

If you'd be willing I'd be curious to know if you get the same behaviour with MoM.
I'm not able to test RtL on my Android 13 device (only on Android 11) but MoM import works on my Android 13 device. It would be useful to know if the error is just RtL on Android 13 or a difference between our two devices.
Perhaps the access rules for Android/obb (where RtL stores its data) and Android/data (where MoM stores its data) are different.

Yes same behaviour when I try to import MoM. So not a problem of the file location. When I try any app in data or obb folder using total commander it is the same behaviour everywhere.

Ok, something different between the two devices.
But in terms of the file browsers yes this looks exactly the same on my device. The Android/data and Android/obb folders being hidden and/or restricted is what I'm hoping to work around.

Ok after some more tests I think this is really a device bug because Lenovo might not have correctly implemented this API interface. Seems that no app (I tried with another file explorer app) is able to reach the correct folder but instead always lands in the root folder of the device.

I've updated the apk so you can choose a different folder in the "Use this folder" dialog when importing.
If any file manager will allow you to copy Android/obb/com.fantasyflightgames.rtl to the custom folder and then choose the custom folder for import it should work.
It isn't great since I was trying to remove that kind of manual handling but maybe better than nothing.
Dealing with Android's SAF has been one of the most frustrating development experience I've ever had.
Valkyrie-android-2.5.9d.apk

Unfortunate if it is just down to vendors implementing the SAF differently.

The import works without access to all files on my phone but could be worth trying to give Valkyrie permission:

It isn't great since I was trying to remove that kind of manual handling but maybe better than nothing. Dealing with Android's SAF has been one of the most frustrating development experience I've ever had.

I would not remove the option where it auto selects the correct folder because I hope for most users this would still be the option that works. This would then heavily reduce amount of support questions because people don't need to understand where to find the files and copy them to other folder manually.
Instead we could split it into two options (e.g. Import from App, Import from other folder)? This could be done with two buttons or instead using a followup page where users can select an option.

The import works without access to all files on my phone but could be worth trying to give Valkyrie permission:
This did not change anything unfortunately. Still no folders showing inside Android folder. I still think it is my device being buggy. :(

I'm sorry to hear. Very disappointing.
No folders inside the Android folder is how it appears to me in the default file browser, but file managers like Total Commander or Cx File Explorer could see inside Android/data.
I set about reproducing what those file managers were doing in Valkyrie.

It is hard to know what the best workflow is here. If users are going to copy data around then they could also just copy in a pre-converted set of assets into Valkyrie's import folders, if they can get their hands on them from somewhere.

I could solve the import issue now. So at least under Android 13 there is a solution:

Install Total commander.
In Total Commander, go to the home folder and from there to "Installed Apps".
Search for the app "Files" and click on it.
In the properties of the app, click on the 3 dots in the upper right corner and select "Uninstall updates".
Repeat the two previous steps if you also have an app called Files by Google.

New post for Descent issues:
Current problem: As you mentioned before:

I note the import takes a pretty long time on the average android device also there maybe a memory leak in the import code. It works but maybe not optimally.

The import for Road to Legend is running know for more than 10 minutes and does not seem to finish. Does it write any log files anywhere that I could check? From what I see in Total commander the .obb file has been copied to Valkyrie folder

but afterwards it seems to get stuck during import (even though some content has been copied:

MoM import taking 20 seconds is far too fast it means it didn't grab all the assets.
Also the missing images confirms this.
If the .obb file has been copied then the import should work. Like I said the import is extremely slow partly due to number of files contained in the obb which need to be checked for assets.
Valkyrie on Android logs to the standard Android logging system. Viewable by anything that can do a logcat.

Also note that a large number of assets are downloaded by the official MoM as it starts up so if you have installed MoM but never opened it Valkyrie won't get a complete set of assets.
I've discovered that these are pulled from ffg's s3 bucket:
https://s3.amazonaws.com/ffg-asset-bundles/mansions-of-madness/AssetBundles/1.9.3/Android/scenarios
So theoretically the extra assets could be downloaded from ffg's bucket for MoM bypassing the whole android security issue but it won't work for RtL because all the extra assets for RtL are stuffed in the obb.

I've done some optimisations.
I was taking 5 hours to import Descent on an old phone! It now takes 12 minutes on the same device.
Valkyrie-android-2.5.9e.apk
valkyrie-linux-2.5.9e.tar.gz
valkyrie-macos-2.5.9e.tar.gz
valkyrie-windows-2.5.9e.7z
valkyrie-windows-2.5.9e.exe
valkyrie-windows-2.5.9e.zip

Since integration of FSAF has been moved to external project this PR is approved now.


Update professional-development.md

In consultation with Rachael, updated instructions for prodev sprints (removing ref to Jason Heaster), added link to Travel info for expenses and moved the historical prodev amounts to an accordion. Also applied sentence case to headings for consistency.

:books: Documentation preview :books:: https://civicactions-handbook--1036.org.readthedocs.build/en/1036/

 Update Prodev page

@grugnog since I pushed up a change here, can you review it?

@ContentBrewmaster @dmundra this looks good - I wonder if there is a possibility to improve some of the long/complex sentences identified by the checks though?

Agree @grugnog that a full rewrite is needed. This was an interim edit to remove inaccurate info (reference to Jason) and support more findability for travel-related expenses. We could pause this until I return and can work with Marc/Rachel on a more comprehensive edit, or release this version now.

@grugnog looks like remark lint ran but didn't report the suggestions like before. See warning here https://github.com/CivicActions/guidebook/actions/runs/4086895319/jobs/7046744378

Warning: Unexpected input(s) 'github_token', 'command', valid inputs are ['entryPoint', 'args']

Fixed the remark lint warning but not seeing the inline suggestions appear in the diff view (files tab).

@dmundra looks like this is fixed - we have complaints about hard to understand sentences again :)

Thanks @marctjones!


Fails run with "init() got an unexpected keyword argument 'session'" for every config file

Tried multiple configs, templates, recent versions etc.
Every run starts with:
"
| Connecting to TMDb...                                                                              |
| init() got an unexpected keyword argument 'session'    "
My TMDB api key is correct. Verified.
Took a spin through the code, and nothing jumped out.
User error or is there something that needs to be changed?

update your requirements your using an old version of the tmdbv3api module


Fix dependency injection problems

[ ] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[ ] Ensure that the pull request title represents the desired changelog entry
[ ] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

@kuisathaverat do you mind if I merge this change?


Add links to current User profiles

Is your feature request related to a problem? Please describe.
Dockstore has user profiles, but there's no easy way to navigate to "my profile" without entering into the address bar.
Describe the solution you'd like
Add links to user profiles in places that make sense. https://dockstore.org/users/david4096

Rename the Account & Profiles pulldown to just say "Account" and have a separate link for "Profile"
Link from the Account page

Describe alternatives you've considered
More in addition to the above

In places where the username shows up on Dockstore (e.g. a public entry) ensure a link to the public profile is there

Additional context
This came up when looking at the usage of our public profile API https://ucsc-cgl.atlassian.net/browse/SEAB-5631
GitHub does this

┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.15
┆Issue Number: DOCK-2439
┆Sprint: 119 - Rouge River
┆Issue Type: Story

Works, but I think the Organization update page isn't quite right. I'm going to log a separate ticket for that and close this one.
[Domain] - https://qa.dockstore.org
Webservice - 7da93c1
UI - 2.11.1-61-g6b02daed
Deploy - 1.14.1-38-g6c0f33c
cwlParsingLambdaVersion: n/a
wdlParsingLambdaVersion: n/a
nextflowParsingLambdaVersion: n/a
galaxyParsingLambdaVersion: 0.0.6
checkUrlLambdaVersion: n/a


Getting frequent warnings about inbound HTTP traffic rule

After a recent reboot of a server, I am seeing quite frequent warnings from the "Inbount network traffic to HTTP server on unexpected port" rule.
The warning just states the connection (fd.name) is <IP_ADDRESS>:0 -> <IP_ADDRESS>:0. When adding proc.cmdline and proc.pcmdline to the output, it states that both are apache2 -k start.
Is this something others are experiencing as well? Could it be a bug with Falco, or is it something weird going on with my Apache setup?

I enabled the rule HTTP server unexpected network inbound traffic from  /etc/falco/rules.d/application_rules.yaml (I'm pretty sure that's the one you're using?) and restarted apache a few times without noticing any events.
I wonder if you're experiencing dropped system calls, which can result in incomplete state within falco and as a result false positives. Can you try running falco with -v? At shutdown, you can look at the Driver Events: and Driver Drops: values. You can also try -s <stats file> to write periodic stats on events received and dropped to a file, and then check the file to see if you have a non-zero drops count.

I enabled the rule HTTP server unexpected network inbound traffic from /etc/falco/rules.d/application_rules.yaml (I'm pretty sure that's the one you're using?) and restarted apache a few times without noticing any events.

Yes, that is the one.
We have a few other servers used for testing etc. as well, running with the same setup, but this has not been reported from any of them. I would assume it is also connected to the load on the production servers.

I wonder if you're experiencing dropped system calls, which can result in incomplete state within falco and as a result false positives. Can you try running falco with -v? At shutdown, you can look at the Driver Events: and Driver Drops: values. You can also try -s <stats file> to write periodic stats on events received and dropped to a file, and then check the file to see if you have a non-zero drops count.

That sounds more like it. I can give it a try and fire up Falco with both -v and -s <file> though and see what it reports :slightly_smiling_face:

Gotten two messages of the kind so far today. The neither output from -v or the file set with -s reports anything unusual when this occurs.

After running falco with -v -s /tmp/falco-stats.log for about 5-6 days now I can report that the verbosity does not help much, in fact nothing at all.
The stats file (/tmp/falco-stats.log) that is written to I can not really make too much sense out of, but I have found that in the JSON, only the first sample property, the cur.events property and delte.events property really change. The cur.drops isn't 0 but also does not change (stable at 1174047).
cur.events seems to increment in general as if it was a timestamp, sample seems to increment per line and delte.events seem quite random. Not sure if any of that is of any help at all.
I will try and upgrade to 0.13.0 and see if that changes anything.

Thanks for that info. cur.drops is the most important one. It shows on an interval-by-interval basis how many dropped system calls there were. And if that value is mostly 0 and is 0 when you saw the falco alerts, then dropped system calls is probably not the source of the problem you're seeing.
From looking more closely at the inbound macro, I think a better version of it could be the following:
- macro: inbound
  condition: >
    (((evt.type in (accept,listen) and evt.dir=<) or
      (evt.type in (recvfrom,recvmsg) and evt.dir=< and
       fd.l4proto != tcp and fd.connected=false and fd.name_changed=true)) and
     (fd.typechar = 4 or fd.typechar = 6) and
     (fd.ip != "<IP_ADDRESS>" and fd.net != "<IP_ADDRESS>/8") and
     (evt.rawres >= 0 or evt.res = EINPROGRESS))

This handles a few additional ways messages could be received (recvfrom,recvmsg) but also explicitly ignores non-blocking accepts in a better way than the old macro.
Do you want to try that version of the inbound macro instead?

Applied this last night and so far so good. Usually there are a couple of messages during the morning "rush traffic" on the sites, but nothing so far.

Great, let us know how it looks over the weekend.


Add health endpoint

Currently the docu recommends to add this endpoint yourself. However I think this can be included in the telemetry plugin as it is required for telemetry (availability).
Kyma requires such an endpoint as colleagues pointed out.
BR, Marten

+1

I was the colleague giving the feedback :)
The health endpoint is not necessarily required for telemetry, it is more that the telemetry must be configured accordingly to exclude communication to the health endpoint. Otherwise trace data gets emitted for every single health check performed by kubernetes.
In general a CAP app should be deployable on kubernetes, even without the plugin. Kubernetes requires well-defined endpoints for readiness and liveness probes to operate the application well, so indeed I would expect to have the health endpoint covered by core components.
The helm chart generated by cds is configuring "/" as endpoint for readiness and liveness probe at the moment. That endpoint is not excluded from span instrumentation (and probably should not), and with that the plugin is indeed emitting spans for the probes, which is a lot of not usefull data.

will be added to @sap/cds (^7.8)


Normal attributes are removed with --transform

Need a way to override this behavior because three.js throws an error because it expects the normal attribute to be present
related https://github.com/mrdoob/three.js/pull/29462

Currently gltfjsx (via glTF Transform) will remove a normal attribute is if the geometry is either unlit (MeshBasicMaterial) or a point cloud. three.js does not in general require a normal attribute in these cases, but it sounds like you're hitting a problem with Decal geometry. That could be fixed upstream in three.js, but I think a PR here would be OK too.
If you'd like to disable that, it would be necessary to enable the keepAttributes flag here:
https://github.com/pmndrs/gltfjsx/blob/8da0e2656649f1b2611bfac013b32ed7210d7cf0/src/utils/transform.js#L74
This change would also cause unused UVs and tangents to be kept, relevant to https://github.com/pmndrs/gltfjsx/issues/235.


'Solution' object has no attribute 'strStr'

------------------------------ Find the Minimum Number of Fibonacci Numbers Whose Sum Is K/find.py
Traceback (most recent call last):
File "/Users/hughbrown/workspace/Solutions-To-Problems/LeetCode/Python Solutions/Find the Minimum Number of Fibonacci Numbers Whose Sum Is K/find.py", line 24, in
Solve = Sol.strStr(haystack = "sadbutsad", needle = "sad")
AttributeError: 'Solution' object has no attribute 'strStr'

Fixed it. Should work.


Disable 'Timestamps' in Docker logs to prevent double-timestamps

Original Issue: https://github.com/openshift/origin/issues/2309
Having this option set to 'true' you will see double timestamps in the output of container logs:
2015-05-18T08:51:59.918042576Z 2015-05-18 04:51:59,912 - dock.plugins.squash - DEBUG - Archive generated

We can either 1. disable timestamps here and leave timestamps for Docker daemon or 2. make it optional similar to 'follow' (if it makes sense).

@smarterclayton FYI

@smarterclayton @brendanburns is there something you can do with the CLA bot? ;-) I think RH has company CLA signed...

Fixed #8100.

@mfojtik you have to be added to the internal list

LGTM

I'll use #8100 to track adding this as an API option on logs.

@smarterclayton could I please be added to the internal list as well? Google bot doesn't like me...


Patch of documentation to address #267 (lack of statusInfo in SessionInfo)

What type of PR is this?
Add one of the following kinds:

correction
documentation

What this PR does / why we need it:
Clarifying that only the qosStatus can be polled, but not the statusInfo which is provided only within EventQosStatusChanged
Which issue(s) this PR fixes:
This documentation patch does not fix the issue #267 finally.
Special notes for reviewers:
While creating this PR, I'm not any more sure that we should fix the issue in this way, even short-term. Instead there is the option to create a patch release v0.10.1 which adds the statusInfo also to SessionInfo.
Left the version number open for discussion.
Changelog input
 release-note

Additional documentation
This section can be blank.
docs

@RandyLevensalor @eric-murray Would you do your final review ... I will prepare the patch release as soon this is merged.


Happy New Year 2024

Hi @ekzhang,
Wow just came across this amazing tool. Installed it and its amazing!!! The latency is very good 😊😊😊. I think its better to have the installer script in github releases or repo instead of s3. Also wondering what s holding you back from supporting windows?
Hiro

Awesome, glad it's working well for you! Yeah I agree the distribution platform could be improved a bit -- we should at least save old archived versions of binaries instead of overwriting a single file path. And the releases should be automated, tagged, etc. I'll work on this in the future.
We could support Windows, I just never got around to it since I don't have a Windows computer myself to develop or test on. I saw some users on Windows open sshx in WSL2, the Linux kernel compatibility layer

Yeah, i tried wsl2 and its working flawlessly so far

I'm now so used using sshx for debugging cd/ci-issues,  what a great tool,
but I just found out it does not support windows.
It's the first time on my current project, I have to troubleshoot remote pipelines running on windows.

Open to ports! I think Windows support would be awesome.


Add defaultName to "hyprctl workspacerules" output (and equivalent IPC response)

Description
Hi,
I'm currently playing around with Hyprland as a replacement for Sway, and I quickly ran into some issues replicating my Sway workflow in Hyprland while using Waybar.
Overall, Hyprland looks awesome, so I'm hoping I can shore up a few things and start using it as my daily driver soon.
Original problem
I use persistent workspaces exclusively, 10 on each monitor. So I set up my workspaces in Hyprland similarly to how I did on Sway:
workspace = 10, defaultName:1/1, monitor:DP-3, persistent:true, default:true
workspace = 11, defaultName:1/2, monitor:DP-3, persistent:true
workspace = 12, defaultName:1/3, monitor:DP-3, persistent:true
workspace = 13, defaultName:1/4, monitor:DP-3, persistent:true
workspace = 14, defaultName:1/5, monitor:DP-3, persistent:true
workspace = 15, defaultName:1/6, monitor:DP-3, persistent:true
workspace = 16, defaultName:1/7, monitor:DP-3, persistent:true
workspace = 17, defaultName:1/8, monitor:DP-3, persistent:true
workspace = 18, defaultName:1/9, monitor:DP-3, persistent:true
workspace = 19, defaultName:1/0, monitor:DP-3, persistent:true

workspace = 20, defaultName:2/1, monitor:DP-1, persistent:true, default:true
workspace = 21, defaultName:2/2, monitor:DP-1, persistent:true
workspace = 22, defaultName:2/3, monitor:DP-1, persistent:true
workspace = 23, defaultName:2/4, monitor:DP-1, persistent:true
workspace = 24, defaultName:2/5, monitor:DP-1, persistent:true
workspace = 25, defaultName:2/6, monitor:DP-1, persistent:true
workspace = 26, defaultName:2/7, monitor:DP-1, persistent:true
workspace = 27, defaultName:2/8, monitor:DP-1, persistent:true
workspace = 28, defaultName:2/9, monitor:DP-1, persistent:true
workspace = 29, defaultName:2/0, monitor:DP-1, persistent:true

Waybar immediately shows these workspaces on the correct monitor whenever I save hyprland.conf (which is an awesome feature, btw), using my old Sway styling for persistent + empty workspaces, but there are a few problems:

If I haven't moved to that workspace yet, Waybar just displays the workspace ID (eg. 10 or 25), which isn't what I want.
If I move to that workspace, Waybar now shows two different workspace buttons. Eg. if I move to 2/5 I'll have:

One workspace button for 24, always styled as an empty + persistent workspace.
One workspace button for 2/5, always styled as an active workspace containing windows. If I switch away from that workspace it sticks around and is still stylized as an active workspace containing windows, even if it's empty (which I assume I can fix later in Waybar somehow).

Digging into the code
In sitting down to try to understand why this is happening, I took a look at what Waybar is doing and how it interacts with Hyprland to create these buttons. Here's what I came up with:

When first launching Waybar, to get the list of persistent workspaces from Hyprland, Waybar reads hyprctl workspacerules here. Seeing that there are persistent workspaces that Waybar hasn't tracked yet, it starts tracking them and creates buttons for them. However, Waybar only gets the workspace ID and not the defaultName from hyprctl workspacerules, so that's what it has to use.

(It does this via IPC instead of the actual terminal commands, but I'll treat these two things as if they are the same throughout the rest of this FR.)

Whenever a workspace is created in Hyprland, Waybar reads hyprctl workspaces here. However, now it gets the workspace name, so Waybar uses that instead.
Since the workspace name and the workspace ID aren't the same, Waybar treats this as two different workspaces that it needs to track, seemingly causing my issue.

(Definitely not an expert, just started looking into these two codebases today, could be wrong, yada yada yada.)
Proposed feature
Assuming I'm correct in how these two projects interact, it seems to me like the best path forward towards getting my workflow to work properly is the following:

Waybar should be tracking Hyprland workspaces by IDs under the hood, not by name. This way Waybar can read the configs and listen for other events (eg. workspace renames, etc), without having to translate through workspace names as a proxy, and the ID/name mismatch I described above won't break stuff.

I can make a FR on Waybar if someone confirms I'm correct on the interaction, and I could probably even take a swing at fixing it myself if Waybar devs OK the direction.

Hyprland should include the defaultName in the hyprctl workspacerules response, so that Waybar can immediately show the persistent workspace's name rather than having to show its ID whenever the workspace hasn't been created yet.

I could maybe take a swing at this too if someone points me in the right direction. Does src/debug/HyprCtl.cpp actually feed into the IPC response as well? Or does this just control hyprctl?

Backmatter
I also played around with the name:foo format of Hyprland workspace identifiers, and it actually gets me closer to my intended workflow. However, two problems there:

Waybar seems to handle this much better at the moment: most of my workspaces will only ever have one button which is how it worked in Sway. However, for some reason (which I haven't stepped through the code to figure out quite yet), Waybar still indefinitely shows two buttons for my default persistent workspaces (eg. I'll have two 1/1 buttons on the one screen, and two 2/1 buttons on the other screen).
I have less control over the ordering of my workspaces in Waybar. Eg: 1/0 and 2/0 always show up on the left of the workspace list, but I want the workspaces to be in "keyboard order" (eg. 1/1, 1/2, [...], 1/9, 1/0) to match my workspace-related binds. From #665 and #4336, it looks like this is WAI and I should be using ID-based workspace identifiers if I want to be able to order "named" workspaces.

Either way:

It kinda just feels more-correct if Waybar tracks Hyprland workspaces by ID rather than name.
I feel like hyprctl workspacerules should probably just include the defaultName setting regardless, and it seems (?) like a small FR.

LMK if my workflow is actually supported via some other means and I'll go try it out :)

I aint readin allat but I can add defaultName to the output, sure.


Document how to add python datasource?

According to docs, python datasources are allowed. But I cannot see python option when setting up new datasource. Is there anything need to be configured but not yet documented?

In settings.py, lines 115-131define enabled query runners.  Recommend either adding redash.query_runner.python to QUERY_RUNNERS in settings.py or defining it in REDASH_ENABLED_QUERY_RUNNERS environment variable before launch.
https://github.com/EverythingMe/redash/blob/master/redash/settings.py#L115-L131

@Oneross Was it ignored on purpose or it's a bug? If it is a bug, I can submit a PR with redash.query_runner.python added; If it is not, I will document it instead.

I'm new to the community, I have a pull request ready for documentation (just a notes section) - my take is that since security of the sandboxed runner isn't 100% it's off by default.

@Oneross Cool. Could you please comment or reference this issue once your PR is merged?

@Oneross Oh hey I see the PR. Thanks for that :) I am going to close this issue.

@oneross is correct. It's off by default to avoid users enabling it without fully understanding what they're doing.
Btw, I'm not a hacker but I managed to bypass the sandbox given a library like requests is whitewashed.

It is somehow confusing to have three way to enable/disable a query runner, isn't it?

By returning False in enabled() class method of the query runner
By changing settings.py
By setting environment variables

The enabled() method makes sure that the query runner can be actually used (I.e. It has all dependencies). And it's a safe guard against people enabling query runners without installing the needed packages.
Setting the env variable is a way to control what data sources you want to support in your redash instance.
Changing settings.py isn't recommended as this is what the Env vars are for.
Coming to think of it, we can change enabled() into a runtime error, that you will get when trying to run a query. Might make things clearer.

@arikfr , I updated the pull req to reflect preference against changing settings.py.


Support AWS ECR

Why
There's no ECR support on dockertags
What
I implemented ECR support using aws-sdk-go.
To use aws-sdk-go package, this PR needs go mod for packaging (I PR-ed #14).

CI will be green if the #14 is merged.


stability issues

It seems this fix has many stability issues, tested on 7 PC's (DEP disabling: no effect), 6x - Win7 x64 and 1 - Win10Pro. It works only on 3 of them (except on one requires compatibility mode set for sh3.exe), on Win10 PC - intro video freeze, anything else in game seems working ok, another one - 0xc000001d error (no error if disabled Patch option in .ini, but process just dies then), others - process just dies (no errors reported). It works ok without sh3proxy on all of them.
PS: no issues with sh2proxy at all.

Win10 PC - intro video freeze, anything else in game seems working ok

So you can skip the video? Do other videos freeze (e.g. credits)? sh3proxy doesn't touch video playback at all.

another one - 0xc000001d error (no error if disabled Patch option in .ini, but process just dies then)

Which option exactly are you disabling? Patch is a name for a set of options.
Does the error message contain the current instruction pointer (eip/rip CPU register value)? Unfortunately, Windows error messages are not informative at all.

others - process just dies (no errors reported)

A good way to find out why would be to disable all sh3proxy options that can be disabled and see if the game still works. If it does, try enabling them by one until you find the one that breaks the game. I suggest you also use v0.7 for this, since UseCWD patch can be disabled there. Also try setting the video resolution in sh3proxy.ini equal to the native display resolution.
Unfortunately, I can't fix any of these issues unless I know exactly what patches break the game. I've also had the game die silently, but this only happened on a virtual machine, so I blame its file sharing. On every live machine I've tested, I had no issues at all.

yes, video can be skipped, credits not tested (and if don't skip it for a long time, game will crash)
Enable video patches (this one), and no instruction pointers..
on machines where process just silently dies it doesn't matter which options is disabled at all.

The game also was tested on vmware ws 12, OS: win10pro - same problem with video playback as on real hardware, other OS not tested.
v0.7 have the same problem, and yes.. i tested every option.

Does your sh3.exe have the same crc32 checksum (372ae792)? (You can check it using e.g. http://code.kliu.org/hashcheck/.)

yes, crc32 is 372ae792
MD5(sh3.exe)= cabab17c9db0b2e16f6281912ca48cf8
SHA256(sh3.exe)= a51f956bd5be21fd704c4d19cf0674a175d002081da43b1d377be83226c36e5e

Well, i found root's of all problems..
First, function WRAP_FN_IFACE, in some cases incorrect path is passed to LoadLibrary (causing process to silently die), changing code to look like this fixes this issue:
if (!origDLL) {
        char path[MAX_PATH], fullPath[MAX_PATH];
        char name[] = WRAP_DLL_NAME".dll";
        memset(fullPath, 0, sizeof(fullPath));
        memset(path, 0, sizeof(path));
        GetSystemDirectory(path, MAX_PATH);
        snprintf(fullPath, MAX_PATH, "%s\\%s", path, name);
        origDLL = LoadLibrary(fullPath);
        if (!origDLL)
            ExitProcess(1);
    }
Second problem comes from clampPow2() function, this cause 0xc000001d error on some systems.

Thanks for debugging.

changing code to look like this fixes this issue:

So I suppose memsetting path to 0 fixed it? Then it must be GetSystemDirectoryA not writing NUL to terminate the string. Wine uses strcpy, so now it makes sense why this never failed with wine.

Second problem comes from clampPow2() function, this cause 0xc000001d error on some systems.

Looks like gcc generated misaligned code, will look into this.

Second problem is this popcnt asm code is not portable in clampPow2() function.
Here is portable version of popcount:
static int
popcount(unsigned int x)
{
#define INIT1(X)                                \
     ((((X) & (1 << 0)) != 0) +                  \
      (((X) & (1 << 1)) != 0) +                  \
      (((X) & (1 << 2)) != 0) +                  \
      (((X) & (1 << 3)) != 0) +                  \
      (((X) & (1 << 4)) != 0) +                  \
      (((X) & (1 << 5)) != 0) +                  \
      (((X) & (1 << 6)) != 0) +                  \
      (((X) & (1 << 7)) != 0))
#define INIT2(X)   INIT1(X),  INIT1((X) +  1)
#define INIT4(X)   INIT2(X),  INIT2((X) +  2)
#define INIT8(X)   INIT4(X),  INIT4((X) +  4)
#define INIT16(X)  INIT8(X),  INIT8((X) +  8)
#define INIT32(X) INIT16(X), INIT16((X) + 16)
#define INIT64(X) INIT32(X), INIT32((X) + 32)

    static const uint8_t popcount[256] = {
        INIT64(0), INIT64(64), INIT64(128), INIT64(192)
    };

    return (popcount[x & 0xff] +
            popcount[(x >> 8) & 0xff] +
            popcount[(x >> 16) & 0xff] +
            popcount[x >> 24]);
}

static inline
int clampPow2(int val, int min, int max) {
    int cnt = 1;

    /* __asm__ ("popcnt %1, %0;" : "=r"(cnt) : "r"(val)); */
    cnt = popcount(val);

    if (cnt > 1) /* Must be pow2 */
        val &= (1 << (8 * sizeof(int) - __builtin_clz(val) - 1));

    /* Clamp */
    if (val > max)
        val = max;
    if (val < min)
        val = min;
    return val;
}

Second problem is this popcnt asm code is not portable in clampPow2() function.

Not portable to what? I was assuming generic x86, which is what users are limited to anyway. It hardly affects performance, so I might rewrite it as a simple loop then.

I mean that popcnt instruction is not supported by any CPU.

Oh, you're right, I missed the fact it's a part of SSE4.2.

Can you test whether this build works? It still fails to start for me on Windows, unless I run sh3.exe from command line.

why there is no source code?

Can you publish the patches so I can merge them?
I didn't publish the changes yet since the fixes might not work. I've replaced popcnt and made path in WRAP_FN static, but I am not sure if this fixes actually the issue, as I can't even get logging working on Windows. Using your fix for WRAP_FN seems to be helping, though, so I will merge that.

here is the diff from my version

Nice, thanks. I'll integrate this soon.


Fix JmeVehicleWheel: wrong Sheet.Set name

Fix https://github.com/jMonkeyEngine/sdk/issues/589

Thank you for your contribution!


Repo sync for protected CLA branch

The pull request is created from master637369980913484239 to master to fix git push error for protected CLA branch

@cmcclister : Thanks for your contribution! The author(s) have been notified to review your proposed change.


The appendix file of the paper

Hello! Thanks for your special work!
I want to download your paper's appendix file from IEEE, but I noticed that under the IEEE link to the paper there is no way to download the appendix file. How can I download the appendix file of the paper?
Thank you.

Hi,
You can find the appendix at the end of the pdf on arxiv.

That's helps me a lot, thanks.


Test and migrate Neo4J example to 1.9.

I deployed the DCOS 1.9 cluster on AWS using the cloudformation template with single-master (1-master and 5-slaves). The neo4j example works perfectly out-of-the-box. I have updated the README and added necessary images.

Reference issue #90


feat(net): interface discovery

Allows you to specify interface-type=wired/wireless instead of interface=ifname
Closes #339

@patrick96 how would i go about testing this?

Oh sorry. I meant I still need to test it.

I tested this patch by changing my module/wlan to use interface-type = wireless instead of interface = wlan0 and it didn't detect my wlan0 interface. Here's some info:
$ ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
3: enp0s25: <BROADCAST,MULTICAST,DYNAMIC,UP,LOWER_UP> mtu 1500 qdisc fq_codel state UP group default qlen 1000
    link/ether b8:ae:ed:76:b8:53 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global enp0s25
       valid_lft forever preferred_lft forever
    inet6 fdc9:8a32:cbfb:0:baae:edff:fe76:b853/64 scope global dynamic mngtmpaddr
       valid_lft 7006sec preferred_lft 3406sec
    inet6 fe80::baae:edff:fe76:b853/64 scope link
       valid_lft forever preferred_lft forever
4: tinc.doronbehar: <POINTOPOINT,MULTICAST,NOARP,DYNAMIC> mtu 1500 qdisc fq_codel state DOWN group default qlen 500
    link/none
5: wlan0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP group default qlen 1000
    link/ether 34:13:e8:35:48:e6 brd ff:ff:ff:ff:ff:ff
    inet6 fdc9:8a32:cbfb:0:3613:e8ff:fe35:48e6/64 scope global dynamic mngtmpaddr
       valid_lft 7006sec preferred_lft 3406sec
    inet6 fe80::3613:e8ff:fe35:48e6/64 scope link
       valid_lft forever preferred_lft forever

It didn't detect my enp0s25 wired interface either. I've applied this patch with a rebase to master.

EDIT: Sorry guys, this was a mistake, I'm testing several PRs open here :upside_down_face: . I haven't tested this PR.

@patrick96 did you ever get around to testing this?

@patrick96 bump

@patrick96 :eyes:

@patrick96 I would really love to see this go through, there's some way that I can help with testing it.

@patrick96 Really looking forward to this addition.^^

@joaquingx While we wait for this PR to go through, what a ended up doing was setting up my config file like
[module/network]
type = internal/network
interface = ${env:DEFAULT_NETWORK_INTERFACE}

and on .profile added
export DEFAULT_NETWORK_INTERFACE=`ip route | grep '^default' | awk '{print $5}' | head -n1`
So every time my .profile is sourced, the default interface is updated to the current one, If it changed.

@patrick96 ping again

@joaquingx While we wait for this PR to go through, what a ended up doing was setting up my config file like
[module/network]
type = internal/network
interface = ${env:DEFAULT_NETWORK_INTERFACE}

and on .profile added
export DEFAULT_NETWORK_INTERFACE=`ip route | grep '^default' | awk '{print $5}' | head -n1`
So every time my .profile is sourced, the default interface is updated to the current one, If it changed.

Hey guys :), I suppose to tagging me here is a typo :sweat_smile:, if possible please change it :pray:

I think it would be good to leave the no-options config to a separate PR. imo that should try to find whichever interface is "active" which seems out of scope of this.

I think it would be good to leave the no-options config to a separate PR. imo that should try to find whichever interface is "active" which seems out of scope of this.

I think it would be good to leave the no-options config to a separate PR. imo that should try to find whichever interface is "active" which seems out of scope of this.

You're right. I also didn't really think about the "active" part. I would have just taken the first available non-virtual interface. I am going to open a new issue for this.

I think it would be good to leave the no-options config to a separate PR. imo that should try to find whichever interface is "active" which seems out of scope of this.

You're right. I also didn't really think about the "active" part. I would have just taken the first available non-virtual interface. I am going to open a new issue for this.

I have decided against adding active interface detection. Most use cases for that can be covered with three modules:
[module/net-base]
format-disconnected =
...

[module/net-wired]
inherit = module/net-base
interface-type = wired

[module/net-wireless]
inherit = module/net-base
interface-type = wireless

This will only show active interfaces as well.

I have decided against adding active interface detection. Most use cases for that can be covered with three modules:
[module/net-base]
format-disconnected =
...

[module/net-wired]
inherit = module/net-base
interface-type = wired

[module/net-wireless]
inherit = module/net-base
interface-type = wireless

This will only show active interfaces as well.

When it will be released?

Don't  know yet


Update zh_CN.ts

corrected several obvious machine translation errors in Chinese

Looks good!
Next time make sure to merge into the future update branch
Thanks for contributing! 😄


After Upgrading Package Unable To Run PHPUnit

Testbench Version: 6.0.0
Laravel Version: 8.0.0
PHP Version: 7.4
Database Driver & Version: n/a

Description:
1) GeneaLabs\LaravelSignInWithApple\Tests\Browser\SignInWithAppleTest::testButtonIsRendered
Facebook\WebDriver\Exception\UnknownServerException: unknown error: Chrome failed to start: exited abnormally.
  (unknown error: DevToolsActivePort file doesn't exist)
  (The process started from chrome location /usr/bin/google-chrome is no longer running, so ChromeDriver is assuming that Chrome has crashed.)
  (Driver info: chromedriver=85.0.4183.87 (cd6713ebf92fa1cacc0f1a598df280093af0c5d7-refs/branch-heads/4183@{#1689}),platform=Linux 5.4.0-47-generic x86_64)

/home/forge/laravel-sign-in-with-apple/vendor/php-webdriver/webdriver/lib/Exception/WebDriverException.php:175
/home/forge/laravel-sign-in-with-apple/vendor/php-webdriver/webdriver/lib/Remote/HttpCommandExecutor.php:376
/home/forge/laravel-sign-in-with-apple/vendor/php-webdriver/webdriver/lib/Remote/RemoteWebDriver.php:136
/home/forge/laravel-sign-in-with-apple/vendor/orchestra/testbench-dusk/src/TestCase.php:106
/home/forge/laravel-sign-in-with-apple/vendor/laravel/dusk/src/Concerns/ProvidesBrowser.php:200
/home/forge/laravel-sign-in-with-apple/vendor/laravel/framework/src/Illuminate/Support/helpers.php:234
/home/forge/laravel-sign-in-with-apple/vendor/laravel/dusk/src/Concerns/ProvidesBrowser.php:201
/home/forge/laravel-sign-in-with-apple/vendor/laravel/dusk/src/Concerns/ProvidesBrowser.php:95
/home/forge/laravel-sign-in-with-apple/vendor/laravel/dusk/src/Concerns/ProvidesBrowser.php:65
/home/forge/laravel-sign-in-with-apple/tests/Browser/SignInWithAppleTest.php:16

Repo with issue
https://github.com/GeneaLabs/laravel-sign-in-with-apple
I have no idea if the issue is down to test bench-dusk, my package, or something in-between (maybe something in the upgrade process?). Any suggestions welcome :) thanks!
I have google-chrome installed:
❯ /usr/bin/google-chrome --version
Google Chrome 85.0.4183.102

https://github.com/orchestral/testbench-dusk/compare/5.x...6.x
There no significant changes on our end and I don't believe Laravel Dusk introduce anything significant as well.
Have you tried force kill any chrome process on the machine?

@crynobone Thanks for the follow-up. I don't show any chrome processes running, and have also rebooted, just to make sure. I googled the heck out of this error, but couldn't find any solutions that seemed to fit the bill. The google-chrome version matches the driver as well:
❯ ./vendor/bin/dusk-updater detect --auto-update
 ----------------- -----------
  Installed         Version
 ----------------- -----------
  Chrome/Chromium   85.0.4183
  ChromeDriver      85.0.4183
 ----------------- -----------

@mikebronner are you still having this issue?

@Keoghan Unfortuantely, yes. :) Do you have any ideas?

Looking around there are a couple of possibilities, which involve adding some extra options:
explicitly setting the debugging port with --remote-debugging-port=0  (0 will stop it trying to connect to one)
--no-sandbox and --disable-dev-shm-usage
the primary search for me was the DevToolsActivePort file doesn't exist part which I think is the primary cause of your issue there.
Have you tried any of those by any chance by customising the driver() method?

Just an update, I still unable to replicate the issue.

I had the same problem，and after restart computer, this problem is vanish. when you first install this package , you can recurrence this bug

I install this package in win10, and in ubuntu 20.04,The bug can replicate

Very sorry folks, I am also unable to replicate (testing with the package that @mikebronner shared). I have tried fresh installing on my mac and a VM with ubuntu 20.04

Can replicate it in github actions

Happy to review if you're able point me to where I can 👍

I and my rranslation software is  don't  understand what you mean.

I and my translation software is don't understand what you mean.

Sorry, do you have a github action set up that I can review?

I have created a simple test repo using testbench dusk and Github Actions.
https://github.com/Keoghan/testbench-dusk-test
Happy for you to PR something that shows the issue if you can replicate, @mikebronner @tu6ge

I have two commit solve the problem, Unsolved Problems. You can see their diff code

This is the commit info

I am not found vendor/laravel/dusk/bin/chromedriver-linux in your document. and in laravel dusk document is not found it.

Ok - so it looks like your issues are now resolved @tu6ge?

Yes

@mikebronner are you still experiencingthese issues?

Hi @Keoghan, sorry for being absent here. I have not had a chance to retest yet, been super busy on other projects and gave up on the original project I was using this in. I tried following the thread, but am not understanding what the solution ended up being.

No worries @mikebronner.
In tu6ge's case he needed to get chromedriver-linux running in the test container on Github Actions before running the tests. Not sure if yours will be the same.

Ah, no. I had that installed and running. Go ahead and close. I will report back if I do run into this in the future.


Enable requireSignOnPaper option for non-template envelopes

Fixes  #124
This copies the same approach from get_inline_signers. @jondkinney or @tcopeland I took the liberty to make the PR since it was rather small. I'm pretty sure it's legit to pass this option when creating a non-template envelope but my DocuSign experience is still pretty limited so I obviously defer to your insights.

Cool thanks @micahiriye I'll take a look.

Merged and v0.3.8 released, thanks again @micahiriye !


Disable ChromeOS jobs again

This has regressed again: https://gitlab.com/Linaro/tuxsuite/-/issues/162

Rather than delete everything, how about simply leaving the badges on the readme, leaving the config url stuff in the generator, then just commenting out the individual builds with an added comment which is a link to the tuxsuite gitlab issue?

Will commenting out all the builds work? We'll basically generate empty TuxSuite and GitHub Actions files, which I don't think will work (I suspect at the very least, TuxSuite will error that there are no builds in the build set).
I have to at least delete the GitHub Actions files and at that point, the badges become dead links, which is why I removed them. Probably should have called that out in the commit message...


Updated org percentage to 2.5%

Changed models.py so that Organization.default_application_fee_percent and Event.default_application_fee were now 2.5 instead of 1.5, and also updated about.html and pricing.html in templates/brambling accordingly.

This will need a migration as well.

Otherwise, looks good!


Custom markers

Issue description
I want to display dynamic data in markers, like gauges or pie charts. Cureently this is not possible as only predefined markers are available.
It would be great to have a SebmGoogleMapCustomMarker and its contents would be used as the marker
Current behavior
Only predefined markers can be shown
Expected/desired behavior
Custom components should be used as markers

I've also seen #557 but it's not quite the same


Media player entity not created

I have two devices using HASS.Agent:

1st one working fine, integration set up notification service and media player entity exist
2nd one: integration only created the notification service but not the media player entity.

During HA Core restart and autodiscovery of the HASS.Agent integration for 2nd device, there was no audio program running (other than for the 1st device).
So after the initial integration adding I tried several times:

start a program like Spotify and play music
restart HA
reload HASS.Agent integration for 2nd device

Nothing helped, there's no media player entity showing up for that device.
In MQTT browser I still only see one entry in hass.agent / media_player.
Is there a certain rain dance ceremony needed for adding the media player?

Is the Media Player enabled in the configuration for the second device?

  I was sure either I configured both machines completely identically or it was enabled by default.
HASS.Agent is now restarted. What's needed to make the entity appear in the HASS.Agent integration in HA? I try to avoid restarting HA Core if possible...

OK, that's interesting: the checkbox disappears.

Hit save and YES

Turn back to settings / media player:

Hello.

Actually I have the same issue on a recent installation. HA has been restarted several times since the HASS.Agent install, as has the Windows PC that is running the agent.
All the other sensors are there and fine, but the media player is not showing up - yet it has for previous Windows devices.
Are there any other local permissions we can look at providing?

@loryanstrant is your GUI setting keeping the tick? As stated my affected client does not.

@loryanstrant is your GUI setting keeping the tick? As stated my affected client does not.

My tick stays there, but it doesn't do anything.

it doesn't do anything

Like the progress in this issue 😄
Unfortunately there seems to be nothing from a user perspective to test or provide. Sit & wait mood activated...

Bump-i-di-bump

It retained the computer (different name) in the hass.agent/media_player topic, and after removing the entire hass.agent topic, removing the hass.agent integration in homeassistant, restarting homeassisant and adding it using the auto-discovery made it all work again.

Thank you, this helped me.

@bcutter maybe this helps you.
I had this issue for quite a while and was able to solve it by removing an old reference of my previous computer in the MQTT server. It retained the computer (different name) in the hass.agent/media_player topic, and after removing the entire hass.agent topic, removing the hass.agent integration in homeassistant, restarting homeassisant and adding it using the auto-discovery made it all work again.
Use https://github.com/thomasnordquist/MQTT-Explorer to connect to your MQTT server and check to see if there is any old/different PC names under the hass.agent/media_player topic

You are a life saver! Thank you for sharing this.

@bcutter maybe this helps you.
I had this issue for quite a while and was able to solve it by removing an old reference of my previous computer in the MQTT server. It retained the computer (different name) in the hass.agent/media_player topic, and after removing the entire hass.agent topic, removing the hass.agent integration in homeassistant, restarting homeassisant and adding it using the auto-discovery made it all work again.
Use https://github.com/thomasnordquist/MQTT-Explorer to connect to your MQTT server and check to see if there is any old/different PC names under the hass.agent/media_player topic

Unfortunately this did not work out for me: in the hass.agent topic there's only one media player topic - the one from the other machine working fine.
The issue is, that this media player topic is not created at all for the machine...

@bcutter maybe this helps you.
I had this issue for quite a while and was able to solve it by removing an old reference of my previous computer in the MQTT server. It retained the computer (different name) in the hass.agent/media_player topic, and after removing the entire hass.agent topic, removing the hass.agent integration in homeassistant, restarting homeassisant and adding it using the auto-discovery made it all work again.
Use https://github.com/thomasnordquist/MQTT-Explorer to connect to your MQTT server and check to see if there is any old/different PC names under the hass.agent/media_player topic

Unfortunately this did not work out for me: in the hass.agent topic there's only one media player topic - the one from the other machine working fine.
The issue is, that this media player topic is not created at all for the machine...

If it's not created, then you have to start over. But before you go that route, make sure that media player is enabled in the Hass Agent on the machine that's missing.  I'd start here:
Close HASS.Agent
Uninstall the addon from Home Assistant (HA)
Restart HA
Install the addon
Restart HA
Start HASS.Agent

That's a wild route. Next to killing my working machine this way, it also ignores the discovered behavior in HASS.Agent on the endpoint itself:
https://github.com/LAB02-Research/HASS.Agent-Integration/issues/23#issuecomment-1495060384
Before purging the whole (working) environment I would like to make sure it's not an endpoint thing - which it tastes like at the moment.

Tried all of solutions, mentioned here. Nothing worked. Media_player topic is not created. And the Media checkbox dissappears after hass.agent restart.
Don't know what to do.

Tried all of solutions, mentioned here. Nothing worked. Media_player topic is not created. And the Media checkbox dissappears after hass.agent restart.
Don't know what to do.

I know it's been a while but I would suggest making a github issue over on the Hass agent github(the app one). Because if no mqtt message is being added then it is nothing Todo with the integration and this repo.

@Bajkot @bcutter could you please enable the debug logging option and upload logs from the "scenario" where the checkbox dissapears? Basically:

enable extended logging
tick the media player checkbox
wait a few seconds
restart HASS.Agent

@DrR0X-glitch is right that if the MQTT mesage is not published this is most likely a HASS.Agent issue and not the integration's fault.
The provided logs will allow me to verify this and have some starting point for the investigation - I'll create a HASS.Agent issue then to cover this.

for me its still not working.. im out of ideas what to do.. everythiing is up to date, still no media players from windows 11 pc

Did you try the forked version?

Did you try the forked version?

what fork? Agent fork or the integration in HA fork?
Can you point me to that fork?
I used the HACS the agent integration that is 3months old (it says that in the HACS store page).
Then i got the client for windows from official https://github.com/hass-agent/HASS.Agent repo

Then i got the client for windows from official https://github.com/hass-agent/HASS.Agent repo

Yes that's the actively maintained fork. The original repo we're tracking this issue here seems to be currently unmaintained.

So I see the topics published in MQTT Explorer

But I don't see media player in HA, what am I missing? Please help I did try deleting the topics and reinstalling the integration and it didn't work.

So after a lot of searching, I stumbled upon this thread.
Uninstalling the existing HASS.Agent integration in HA (also removing all existing entities) and then reinstalling the one here (mentioned in the thread above) worked. I now have the media player entity in HA and can control music playing in a chrome browser on my windows machine from HA.


Send file by api v3

Issue: #53
@kytrinyx I follow your feedback from the issue, and implemented.
I have some concerns about encode base64 the image, usually it grows the size by 33%(Internet talking), we could Gzipped.
I don't know what you think about it.
@groyoh

I don't want to do the base64 encoding, since we're just serving an image as a file, not as part of a JSON response.

Great @kytrinyx .I will finish with this one today and keep working on the other ones.
Thanks.

@kytrinyx Are we removing the old readme endpoint ??

@kytrinyx Is there anything else you want me to do for this feature ?

I'm not sure which endpoint you mean. We'll eventually get rid of the entire api v1-v2, but the readme endpoint in v3 looks good now, and is ready to be used in the readme page on the site.

Sorry I don't know what I was thinking I think I was really tire :disappointed:

Oh, having read your code I now understand your question. No, we're not getting rid of the README endpoint, but I rearranged a couple things yesterday while planning how to refactor some things in the website codebase. Look in v3/routes/exercises.rb and test/v3/routes/exercises_test.rb once you've rebased onto master.

No worries :) Things move really quickly sometimes, and there are so many moving pieces it's hard to keep track.

Thanks for the patience I will clear the Pull Request

Thank you!

One thing before this is ready--would you make sure that we get a 404 rather than a 500 if the track doesn't exist or the docs directory doesn't exist?

Sure no problem
On 27 Nov 2015 20:11, "Katrina Owen"<EMAIL_ADDRESS>wrote:

One thing before this is ready--would you make sure that we get a 404
rather than a 500 if the track doesn't exist or the docs directory doesn't
exist?
—
Reply to this email directly or view it on GitHub
https://github.com/exercism/x-api/pull/78#issuecomment-160191600.

@kytrinyx Hope this time I got it right, and once agin thanks for the patience

There are  a couple of tiny style things, but they can be fixed in a later PR.
Thanks for taking the time to work on this.

Thanks @kytrinyx I haven't use MiniTest much


Changes to make it work in idea >=2016

Tested with ultimate 2017.2.2
Workaround to display the settings panel in idea >=2016. Some work still required to adapt to the new API (mainly settings persistence, but also other deprecated classes), but this worked with minimum impact.
IDEA >=2016 requires a new entry for config in the plugin.xml, so I simply created a provider that returns the component to mimic the behaviour of previous versions.
					<extensions defaultExtensionNs="com.intellij">
					<projectConfigurable provider="com.wrq.rearranger.RearrangerConfigurableProvider"/>
					</extensions>

is there anywhere a "compiled" version to test those changes -> i do not want to set up everything right now :D

Sure @aepfli , just download the jar from here: https://github.com/itajun/Rearranger
Cheers,
Itamar

Can you guys update the version from Intellij plugin repository?


chore: fix link in all-contributors bot

Type
enhancement

Description

Updated the badge link in the .all-contributorsrc to direct users to the acknowledgements section of the README.
Changed the commit type to better reflect the nature of contributions.

Changes walkthrough
Relevant filesConfiguration changes

.all-contributorsrcUpdate All Contributors Configuration

.all-contributorsrc
Updated the badge template link to point to a specific section in the GitHub repository.  Changed the commit type from "docs" to "contributors".

+2/-2

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

@all-contributors please add @annerademacher for financial


Bug_Form> Default/ Expanded> 'Children' control prop> Error message appears if user click the 'ref' value and click outside without changing the value.

Describe the bug
Error message appears if user click the 'ref' value and click outside without changing the value
To Reproduce
Steps to reproduce the behavior:

Go to https://storybook-preview-290--upbeat-sinoussi-f675aa.netlify.app/?path=/story/form--default
Click on 'Controls'
Navigate to 'Children' controls
Click on 'ref' value
Verify the Preview

Expected behavior
Error message should not appear.
Screencast
https://somup.com/c3nblPThxK
Desktop:
OS: [Win 10]
Browser [chrome]
Version [Version 98.0.4758.82 (Official Build) (64-bit)]

refs are references to another element. By updating the ref prop with no other element it errors out because there is nothing to reference. This is not a prop you can just change on the fly without having the element available to put in as a value.