[LIBCLOUD-905] Added list images method for OnApp

Added list images method for OnApp
Description

Implements methods for listing available images for OnApp driver
Includes tests and documentation

JIRA ticket: https://issues.apache.org/jira/browse/LIBCLOUD-905
Provider API doc: https://docs.onapp.com/display/52API/Get+List+of+All+Templates
Status

done, ready for review

Checklist (tick everything that applies)

[x] Code linting
[x] Documentation
[x] Tests
[ ] ICLA

👍

@tonybaloney quick question - do I still need to attach a patch to the issue at JIRA as explained in the dev guide? Looks like these changes are already in trunk.


Update README.md, removing all v3.0 features

These features aren't in Turbolinks Classic, which is currently at v2.5.3, so they shouldn't be in the Readme

@rafaelfranca Any reason this can't be merged? I know the tests are failing, but they must be failing on master too - this is a pure documentation change.


Package (vendor) routes.php gets loaded before main routes.php

I had a hard time overriding the routes of a vendor package installed with composer.
After a hour of struggling with routes I found that routes.php in a packages folder gets loaded before the main routes.php.
Is this a bug?
I think it is (or at least a problem), because if a package has a route like this:
Route::get('{all}', //etc
it will "catch" all the routes and everything in the main routes.php will be ignored.
So basically we can't override parametered routes of a vendor package, but vendor routes can override (prevent) the routes in the main routing file.

@GrahamCampbell, @CurosMJ
Any thoughts on this?

@GrahamCampbell, @CurosMJ
Any more comments?

Official package support is removed in L5.


Add support for time entries

Thanks for building this tap! Getting time entries out of ClickUp is really useful for data warehousing, e.g. building project reports. We've done so using google apps scripts for a while, but I want to migrate this to a more mature stack and looking into meltano/singer.
Here's what we're currently doing:
  // we need to manually fetch assignees first...
  const team = getClickUp_Team_();
  const userIds = team.teams[0].members.map(x => x.user.id).join(",");

  // The ClickUp TimeEntry API is... crap. By putting in start_date=0 we tell it to give us all data from the beginning of time without paging
  const url = "https://api.clickup.com/api/v2/team/2488205/time_entries?start_date=0&assignee=" + userIds;

I understand that this could be modeled as a dependent stream in a way? (time_entries stream depends on team?)

Yes a TimeEntry stream, with a parent of Team would (seemingly) work well here. How long does it take (roughly) for all of your time entries to come over? We could just pull them all, or implement an incremental model to save the state (so we don't pull them all every time, if timing is an issue)

We're currently not loading incremental, takes about 15s for 5000 entries which I think is fine for now. Incremental mode thus wouldn't have to be on the list I'd say.
Incremental might have the problem that in a certain "active" time window of a couple weeks records might get changed around (clickup allows deleting/modifying entries and that's also often used in real world settings ;-) ).


Card Swipe Sign In/Out in Kiosk Mode

The CELTS team requested we add a sign in and sign out feature to keep track of hours more accurately.  The first swipe in functionality should still put the user into the eventParticipant table, but the hours should be added based on the time between the swipe in and swipe out.

 CELTS #196: Card Swipe Sign In/Out in Kiosk Mode


Make the parser much more robust to enormous symbols.

This teaches the parser to resize its buffer up to a reasonably maximum limit, and then if a line
slams into that maximum, it just enters a PANIC RECOVERY mode where it discards bytes until it
finds a newline or EOF, then resumes normal behaviour. This basically just ends up discarding horrible
enormous (80kb+) symbols instead of discarding the entire file.
Also this adds a bunch of "symbol:" TRACE logs to make figuring this stuff out easier in the future. In particular if we do end up discarding a bunch of your lines, that will be noted in the TRACE at least.

this also gives better diagnostics to the issues seen in #520

merging as a hotfix, will review/test later


use american english for pipeline definition

Users expect to write american english in pipeline definitions.
Therefore rename "finalise" to "finalize"

I assume the pipeline is already adapted?

@RaphaelVogel: Can u please rebase? Then we can merge this PR?


Use lastPoll time as cutoff on subsequent polls

This is additional work towards #85
Using the last poll time as a cutoff marker on subsequent polls should help with robustness against missed events, for instance if a previously scheduled poll event was missed or delayed. In reverse an unscheduled poll event could be triggered in an interval, losing the assumed synchronisation of the static cutoff mechanism.  If the configuration surface is extended to allow variable polling rates (per feed, for example) then this could simply be extended to a map, or similar.

As mentioned in the body of the PR, this does introduce a potential change to the default behaviour. As such it could be expanded to be enabled behind a config option.

I've updated the PR to change cutoff_delta to poll_rate to better reflect how its used with this change


clarifying what groups can scan

Add a note:
"When trying to complete a scan of a file share. Only Agents in the default group are utilized."

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

ID: 9a94b401-5a3e-7b47-f264-1b1bea8cbccb
Version Independent ID: a7f30ee0-4a71-a37d-50ec-2b0d44cbdf72
Content: Agent groups in Migration Manager' - Migrate to Microsoft 365
Content Source: migration/mm-agent-targeting.md
Service: sharepoint-online
Sub-service: sharepoint-migration
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Updating -- thanks Jarai!

@jaraizer
This has been added to the Troubleshooting document and to the Step 2 topic in the File Share solution.


Add Event Feedbacks

Describe the feature you'd like
Add Event Feedback which shows a list of all the events feedback in a recycler view once the event/session has been concluded
Would you like to work on the issue?

[x] Yes

@iamareebjamal https://api.eventyay.com/v1/events/1/feedbacks?sort=rating&filter=[] the above is allowed for logged in user I guess it shouldn't be right?

@iamareebjamal https://api.eventyay.com/v1/events/1/feedbacks?sort=rating&filter=[] the request is allowed for logged in user I guess it shouldn't be right?

Why it shouldn't be?

@iamareebjamal submitting a feedback should be allowed from logged in user but showing the list of feedbacks I guess not .i think other users should logged in or not should be able to the feedback's rating for the event

Your language is wrong. It should be the request is not allowed for unauthenticated user I guess it should be right? or the request is only allowed for logged in user I guess it shouldn't be right?

Open an issue on the server

I didn't mention, but I was already working on this, Can I continue @aggarwalpulkit596 if you don't mind?

I already have completed some part of this PR @angmas1

Okay, you can continue.


build gmsh-compatible glibc

gmsh built on Debian stretch may be not consistent with glibc library on other systems.
We may need to build SOLVCON-specific glibc so it could run on different systems smoothly.
Please refer to this comment of #145.
https://github.com/solvcon/solvcon/issues/145#issuecomment-192033672

@tai271828, we'd need to do something with this and #145, #146.

We have not touched this issue for a while and Debian stretch has officially released on June 17th 2017. I tried to reproduce the issue today by python setup.py build_ext --inplace, and the result looks good now[1]. The issue could not be reproduced anymore.
The reason that we could not reproduce this issue may be:

Debian stretch bumps its glibc
gmsh version bumps

I didn't look into the details. However I suppose that it should be Debian stretch changed before officially releasing.
[1]
(root) ubuntu@debian:~/solvcon$ nosetests --with-doctest
.................................................................................................................................................................................................................................................................................................................................................................SSS
----------------------------------------------------------------------
Ran 356 tests in 4.569s

OK (SKIP=3)
(root) ubuntu@debian:~/solvcon$ lsb_release -a
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 9.2 (stretch)
Release:	9.2
Codename:	stretch
(root) ubuntu@debian:~/solvcon$ uname -a
Linux debian 4.9.0-4-amd64 #1 SMP Debian 4.9.51-1 (2017-09-28) x86_64 GNU/Linux

Sounds like this issue may be closed.  Please reopen if otherwise.

Checked the stdc++ lib and stretch did bump its version[1]. We could close this issue.
[1] It is now GLIBCXX_3.4.22 instead of GLIBCXX_3.4.20 reported in https://github.com/solvcon/solvcon/issues/145#issuecomment-192033672

(root) ubuntu@debian:~/solvcon$ lsb_release -a
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 9.2 (stretch)
Release:	9.2
Codename:	stretch
(root) ubuntu@debian:~/solvcon$ uname -a
Linux debian 4.9.0-4-amd64 #1 SMP Debian 4.9.51-1 (2017-09-28) x86_64 GNU/Linux
(root) ubuntu@debian:~/solvcon$ man strings
(root) ubuntu@debian:~/solvcon$ strings /usr/lib/x86_64-linux-gnu/libstdc++.so.6 | grep GLIBCXX
GLIBCXX_3.4
GLIBCXX_3.4.1
GLIBCXX_3.4.2
GLIBCXX_3.4.3
GLIBCXX_3.4.4
GLIBCXX_3.4.5
GLIBCXX_3.4.6
GLIBCXX_3.4.7
GLIBCXX_3.4.8
GLIBCXX_3.4.9
GLIBCXX_3.4.10
GLIBCXX_3.4.11
GLIBCXX_3.4.12
GLIBCXX_3.4.13
GLIBCXX_3.4.14
GLIBCXX_3.4.15
GLIBCXX_3.4.16
GLIBCXX_3.4.17
GLIBCXX_3.4.18
GLIBCXX_3.4.19
GLIBCXX_3.4.20
GLIBCXX_3.4.21
GLIBCXX_3.4.22
GLIBCXX_DEBUG_MESSAGE_LENGTH
(root) ubuntu@debian:~/solvcon$

This is already closed.

Yes it is closed. I want to add more information for reference in the future to make it more completed.


cd failed at Python Semantic Release

Looks like the recent PR failed when merging to main. See https://github.com/UBC-MDS/covizpy/runs/4962891724?check_suite_focus=true
Not sure if this is due to the previous step to disable admin branch protection. Any ideas?

Seems the command of semantic uses a role that is not an administrator. Thus I have just added to use an admin token (generated by Rong) in the latest PR #63 . Similar discussion at #58

Is this due to the branch protection option being active?

We can try removing the below rule. My guess is that it disallows the removing branch protection without an approval.
At least 1 approving review is required by reviewers with write access.

Worse case is to just remove it and let the original cicd workflow run. I've been testing it whole morning and hopefully I found a way to work =D

It works now in my own repo~
https://github.com/thomassiu/pycounts/blob/main/.github/workflows/ci-cd.yml

fixed as part of #72


Feature request: Support closed source licenses

Cargo bitbake won't generate a recipe when there is no license info in Cargo.toml
However for internal projects there might not be a license.
LICENSE = "CLOSED"

in a recipe is a valid way to signal yocto that there is no valid license. However if I use that I still get
# FIXME: update generateme with the real MD5 of the license file
LIC_FILES_CHKSUM=" \
file://CLOSED;md5=generateme \
"

in the generated file.
It would be nice if that could be left out

Thank you for the ticket. For reference the documentation behind this is available at http://www.yoctoproject.org/docs/2.3.2/mega-manual/mega-manual.html#sdk-license-detection

Please see https://github.com/cardoe/cargo-bitbake/tree/license-closed and let me know if you think that'll work. You can test it by cloning the repo and checking out that branch and running cargo install -f in the top level directory.

/cc @jcreekmore Since you've contributed to the project.

This seems to work as expected. It still leaves a superfluous
# FIXME: update generateme with the real MD5 of the license file
LIC_FILES_CHKSUM=" \
"

But that does not leave a warning during the build.
Thanks!

As far as removing it out entirely it would be a bunch more work. I would certainly accept a patch to clean that up.


No longer producing Location HTTP Response header

I've noticed that since switching to 2.1-SNAPSHOT, PUT/POST requests are no longer producing the Location header (the Content-Location header is still there, though, with the same value). Is this intended?

Oops! Early in the DSTU3 development process we decided to get rid of one of the two headers since they were redundant.
Possibly I'm remembering this part wrong but I think we picked for Content-Location to be the one that survived. Either way, it's definitely Location now in the spec so HAPI's doing the wrong thing.
I'll update.


Log Capture for Unit Tests

Use https://github.com/Hakky54/log-captor for unit tests

Unable to use for our cases


That logo font....

Try and spice it up a bit by not using Roboto for everything. http://ufonts.com/fonts/hybrea.html

Switched to Hybrea. Looks WAY better now.


[new page] external resources

We need to put a page in the docs that links to the white papers, videos, and relevant blog posts (CP vs AP, TSM, Clustering closed source, etc.) so they are easier for users to discover from the docs.
URL for relevant page?
something under the Guides or Concepts section, maybe?

A short list of links to mention, there are no doubt others:
https://influxdata.com/technical-papers/
https://influxdata.com/videos/
https://influxdata.com/blog/update-on-influxdb-clustering-high-availability-and-monetization/
https://influxdata.com/blog/graphite-whisper-influxdb-migration-utility-beta/
https://influxdata.com/blog/category/tech-tips/
https://influxdata.com/blog/how-to-create-iot-influxdb-google-cloud-platform-part-1/
https://influxdata.com/blog/getting-started-with-sending-statsd-metrics-to-telegraf-influxdb/
https://influxdata.com/blog/testing-influxdb-storage-engines/
https://influxdata.com/blog/new-storage-engine-time-structured-merge-tree/
https://influxdata.com/blog/influxdb-clustering-design-neither-strictly-cp-or-ap/
https://influxdata.com/blog/combining-kapacitor-and-continous-queries/
https://influxdata.com/blog/influxdb-markedly-elasticsearch-in-time-series-data-metrics-benchmark/
@pauldix @toddboom @jwilder @nathanielc @sparrc @jginfluxdata @ShubhraKar Thoughts on the above links? Are some too old now? Did we miss some other good content?

remove

https://influxdata.com/blog/testing-influxdb-storage-engines/
https://influxdata.com/blog/influxdb-clustering-design-neither-strictly-cp-or-ap/

Somewhat fixed in the 1.0 branch - a new External Resources section links to the videos and blogs page.


Fix VS Code formatting to match mvn fmt:format

Dropped fmt:format since I couldn't make it behave the same as the VS Code extension.


Adding Hover Effect to All Buttons

Currently, none of the buttons across the site have a hover effect, which affects the overall user experience. The task is to add a hover effect to all buttons, enhancing interactivity and visual feedback when users interact with them. Ensure that the hover effect is consistent across the website, with a smooth transition that complements the existing design.
Requirements:
Apply a subtle hover effect (e.g., color change, shadow, or background shift) to all buttons.
Ensure that the effect is responsive and works well across all devices.
Implement the hover effect using CSS.

@Tejashri-Taral please add gssoc-ext  and hacktober tag

Please assign this task to me . Hover all the button as per the template related changes in UI also.

HI,
I see that the underline below "About" is not working and to maintain uniformity u want to remove it from feature, contact us and product as well... In my opinion , if we add the underline to all the tags, it would look better, but if u wish it the other way , it's absolutely fine.
During the hover we can change the color as well as the SIZE of the text which will make the overall UI look even better. I would like to contribute and solve this issue .

Hello, please assign me to this task. Applying hover effects, along with other enhancements such as shadows and bold styles, will uplift the overall look of the website. These effects include changes in color, font size, borders, and more in the buttons, with additional animations for a short interval of time. I am excited to contribute and take the user experience to the next level!

@Tejashri-Taral  plz assign me


Deprecate noRender

The noRender prop should be deprecated in favor of wrapped components. This will allow greater flexibility in parent-child relationships and specifically with passing context into a component.

hmm this seems like an unnecessary move, why remove the option completely? I find the "component floating in the void" aspect of noRender really confusing, and error prone, and extremely limiting.
users working with the component in the editor have no idea they can't add code outside the component, and will get very unhelpful errors when they do. It also means we cannot demonstrate any logic, can't attach handlers, can't show state, etc. This decision really only works for simple display only or completely contained components.

I understand the benefits of using wrapped components but I don't grasp how to permit creation of new components from the editor without disabling noRender.
Can you explain it?

So, I am trying to implement a higher order component similar to http://khan.github.io/react-components/#sortable.  How can I now do this without noRender?  I've been banging my head on a wall for a few hours trying to get this to work.  Maybe you could include an example of how to do this?

Sorry @ianvs , will give this repo some love this week, been tied up getting our other repos released. Can you post an example of what you're trying to do?

I haven't forgotten about this, just need to take the time to create a gist.    The link to the Kahn Academy component should pretty clearly show what I'm trying to do (i.e. show an example of using a higher-order-component).

https://github.com/FormidableLabs/component-playground/pull/49

I'd rather it just wasn't deprecated at all.  I don't see the reason this functionality would need to be removed.

Basically the reason it is deprecated is that this is not supposed to be a REPL or a code-example plugin - it was built for showcasing built components in something of a living style guide. For the times when you  do need to do something more than showcasing a built component, you can use es6Preview.
The two main problems are this:

noRender === false turns the component playground into stunted REPL. We already have es6Preview which means there are two code paths to accomplish mostly the same thing.
No support for context - This is the big one. By allowing the no-render to be false we lose the ability to control the root and therefore the ability to control context or any other thing like i18n or top level data. Using noRender causes inconsistent API behavior which is why it got deprecated.

@IanVS - Is there a reason you can't use the ES6Preview for what you want?

I will try out es6Preview, thanks.  Is it the same thing as es6Console, mentioned in the readme and examples?  If so, it doesn't seem like quite what I need.  I am indeed showcasing built components in a living styleguide, except that some of my components will be higher-order components.  Using console.log to render the examples seems a bit clunky, since the example code I want to provide is essentially an example of how to implement the library's components, but I shouldn't judge it before I give it a try...

Since 1.0 has been released and this isn't removed yet (which I'm grateful for), I wanted to come back and look at this again.
I agree that removing noRender without some way of providing functionally equivalent behavior would be a bummer. There are a lot of cases where the ability to provide a complete react component and render it in an example is helpful. The Khan Academy examples provided previously are a good example of a higher order component use case.
Unless I'm missing something, es6Console doesn't provide even remotely the same functionality as noRender. I wasn't able to render a component through a console.log with es6Console, and even if that was theoretically possible it wouldn't make any sense and would be pretty jarring and weird as documentation.
All that is to say that I feel strongly that this option shouldn't be removed without first providing another way to handle higher order components.

@alexlande and @coopy - since i am merging #74, could we also officially table and close (for now) this issue? I'd really like to do is get rid of the scary README warning:

noRender - (Deprecated, Remove at 1.x)

I should note that Formidable uses noRender all over our docs. e.g. http://formidable.com/open-source/victory/docs/

Yeah, definitely 👍


Design the API for get list of book categories

Specification of the get list of book categories API.

API Specification is completed


Adding recurse option to gitrm

Adding recurse option to gitrm with relevant docs/tests

Excellent, merging.

Published to NPM<EMAIL_ADDRESS>


Feature/dm 361 summary report user creation failure

Heya @sven1103 and @KochTobi as discussed i cherry-picked the relevant commits from this PR to #35 to avoid branching confusion. Therefore I'd close this PR


-like completion not working in vim

That's just it. Even though nvim-completion-manager does work on vim (with vim-hug-neovim-rpc), the -like completion source doesn't get activated, neither by <Plug>(cm_force_refresh) nor after the third repeated word in a row.

Have you updated vim-hug-neovim-rpc to the latest version?
This feature requires the new version updated a day ago

Oops, sorry. Should have updated before filing the issue. It does work with the latest commit


Rancher Server not accessible with K3s version v1.17.7+k3s1

Environmental Info:
K3s Version:
k3s version v1.17.7+k3s1 (b0260b33)
Node(s) CPU architecture, OS, and Version:
Linux ip-172-31-12-253 5.3.0-1023-aws #25~18.04.1-Ubuntu SMP Fri Jun 5 15:18:30 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Cluster Configuration:
Single node cluster
Describe the bug:
Rancher does not work on the instance (having the k3s version v1.17.7+k3s1 installed). The UI gives "404 page not found"
Steps To Reproduce:

Installed K3s:
k3sup install --ip=<IP_ADDRESS> user=root --k3s-version=v1.17.7+k3s1
-Installed Docker:
Docker version 19.03.12, build 48a66213fe
-Installed Rancher:
docker run -d --restart=unless-stopped -p 80:80 -p 443:443 -v /opt/rancher:/var/lib/rancher rancher/rancher:v2.4.5

Expected behavior:
Rancher should work, UI should should be accessible to add the cluster
Actual behavior:
On accessing the Rancher Server, it gives "404 page not found"

Are you trying to run both k3s with the containerd backend, and Docker, with Rancher in Docker, on the same host? Why aren't you installing Rancher into k3s?


Fix old cast style use in generated file and support lib.

Fixes LLVM warning: use of old-style cast [-Wold-style-cast]

Can you please re-generate the code for the example and test cases to show the impact of the generator changes?  Also please ensure the tests build and pass.  You can run make all from the root which will do everything, if you're on a Mac).

There you go! Test are passing now and generated code looks ok.

Thanks for your effort! This repo is no longer supported by Dropbox and about to be archived. I'm not going to try to merge this PR at the last minute, but I'll leave it open for reference by anyone who encounters the same problem.


New option that allow setting the url of the proxy

Thanks for your work! Will merge after these few minor fixes!

LGTM, thanks!


Subsystems and lifetimes

Feel free to sit on this. I'd like a chance for some feedback before this all gets merged. Documentation.
Closes #431
There are new subsystem types for each subsystem: VideoSubsystem, AudioSubsystem, JoystickSubsystem, etc.
let sdl_context = sdl2::init().unwrap();
let video_subsystem = sdl_context.video().unwrap();

let window: Window = video_subsystem.window("Test", 800, 600).build().unwrap();

Types that cannot outlive SDL, such as Window, Joystick, GameController, and AudioDevice, all contain an Rc to their respective subsystems. They each include something like Rc<VideoSubsystem>. In turn, each subsystem includes Rc<SdlDrop>.
This means that SDL and the subsystems will only be quit once nothing depends on them.
I initially started with a statically-checked lifetime system, i.e. having Window<'a>, where 'a is the Video subsystem. This turned out to be too cumbersome and frustrating to use, so I abandoned it in favor of loose Rcs.
The only downsides I can think of for the Rc-based system is a) it's theoretically slower due to ref-counting (but likely not noticable), and b) if any of the types are leaked, then SDL_Quit will never be called. Not calling SDL_Quit is still safe, but it's undesirable (X11 may not restore from fullscreen).
Most module functions have been moved into the subsystem types. These functions require their respective subsystems to be initialized before calling, as I could gather from the SDL2 source code.
let sdl_context = sdl2::init().unwrap();
let video_subsystem = sdl_context.video().unwrap();

println!("{}", video_subsystem.get_current_video_driver());
// Instead of: sdl2::video::get_current_video_driver()

The tests are failing because of the line: subsystem!(TimerSubsystem, ll::SDL_INIT_VIDEO, sync);
I'll fix it in a jiffy.

The RC stuff only happens when dealing with dropping the subsystems, right? I'm pretty sure that isn't a critical code path that people are going to notice slowdown in ;)
I like this PR, but as with all big changes that happen to the API, it's probably best to see if anyone else has comments before I go a'mergin.

Things to think about: how does this affect libraries like sdl2_image?

The RC stuff only happens when dealing with dropping the subsystems, right? I'm pretty sure that isn't a critical code path that people are going to notice slowdown in ;)

Yup, exactly. The Rc stuff only happens on initialization and dropping, and they're not invoked that often. SDL itself also does a bunch of ref-counting magic behind the scenes, so we're only adding a minuscule amount of overhead when all things are considered. :)

I like this PR, but as with all big changes that happen to the API, it's probably best to see if anyone else has comments before I go a'mergin.

Totally. I hate inadvertently breaking reasonable use cases, so I want to give this a while to see if any potential issues suddenly pop up.

Things to think about: how does this affect libraries like sdl2_image?

I don't think this will affect rust-sdl2_image. The from_ll methods from Surface and Texture haven't changed, and they don't require any initialized subsystems to work.
As far as I can tell, the other SDL subprojects I know of shouldn't be affected, either.

+1 this PR, definitely needed for sharing around the Sdl context and its subsystems between crates. The current setup makes it very difficult to initialize subsystems after Sdl itself has been created, especially if it was made in another crate.

This should be good to merge now. I got glium_sdl2 and some of my other projects ported to this and #443, and there weren't any problems.

Ooy, I just noticed that SDL_GL_GetAttribute and SDL_GL_SetAttribute require the video subsystem to be initialized. Maybe I can hack something together real quick.

sdl2::Sdl::new is not pub, I assume it's supposed to be.

I'm going to merge this over lunch :)

@AngryLawyer Any word on the merge?

Sorry about the delay, this weekend was rather mad with it being Pride in Brighton.
In we go!


Add go mod tidy to install script

It might be worthwhile adding go mod tidy to the install script here since it kept failing for me and the error message was rather cryptic.
What fixed it for me was running go mod tidy.
Might have been because I installed go via apt-get before running the script, not realizing that it automatically installs go via snap.

Hello @breakfast0x ,
I just tried go mod tidy and the content of file go.mod  and go.sum didn't change. So basically that meant the modules had been tidy already.
Meanwhile this script install.sh installs go only if it's not been included in your PATH, which usually means it's not installed yet.

Can't repro.


Figure out the performance implications of using (#.) or (.#) vs. (.)

This seems to be the main selling point of the operators.
We should add some benchmarks and summarize the results in the docs.

Generally speaking, they're helpful in in higher-order contexts when there might not be enough inlining to avoid problems. f . coerce or coerce . f can potentially build a closure that just passes an argument to f, while coerce #. f and f .# coerce are really just f. The reason is that if GHC can't see that f has arity at least 1, then f and the dotted version could actually be different. This tends only to be a problem when f is totally unknown and the HOF doesn't inline. So it usually won't matter, but sometimes can.


Duel Pass Rewards Content

Hello, this isn't an issue itself but just a question. There have been some Duel Passes with unique rewards like Pets, Icons, and other accessories like the current Pass. My question is if its possible that they are added as purchable content for this offline version. Thank you for you time.

Duel pass isn't implemented. I don't have any plans of implementing it. You can enable all items in Settings.json https://github.com/pixeltris/YgoMaster/blob/master/Docs/Settings.md#datasettingsjson

Got it, thank you pixeltris


Feat 621 eventing

Change or Enhancement
Changelog
New

New TokenAttribute custom argument to retrieve the token from the request.
Update .gitignore file.

Changed

Replace getToken helper method with the custom TokenAttribute argument.
springfox-swagger2 migrated to version 3.0.0. New URLs:

API docs: http://localhost:7720/api-docs/
Swagger UI: http://localhost:7720/swagger-ui/

Refactor and formatted existing classes to conform to Google Style Guide for Eclipse Java.

Style Guide: https://github.com/google/styleguide
Profile: https://github.com/google/styleguide/blob/gh-pages/eclipse-java-google-style.xml

Removed

Unused Maven dependencies (probably a few more unused dependencies left but need to check this).

@iuliancorcoja I checked with @marcusdroy the project is currently using the Google Style for Java with the default 2 spacing for tabs.
Looking at some of the styling changes, this is showing 4 spaces per tab.

Fixed! Yeah, so Google Style is independent on tab size and type (spaces or tabs). I had this configured in my IDE and changed it to 2 spaces.

@iuliancorcoja I checked with @marcusdroy the project is currently using the Google Style for Java with the default 2 spacing for tabs.
Looking at some of the styling changes, this is showing 4 spaces per tab.

Fixed! Yeah, so Google Style is independent on tab size and type (spaces or tabs). I had this configured in my IDE and changed it to 2 spaces.

Great clean up by @iuliancorcoja with a great addition of the token resolver.


Fix null pointer access in error reporting

Codecov Report

Merging #1983 into canary will decrease coverage by 0.29%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           canary   #1983     +/-   ##
========================================
- Coverage    6.52%   6.23%   -0.3%
========================================
  Files          90      90
  Lines        4120    4120
========================================
- Hits          269     257     -12
- Misses       3851    3863     +12

Impacted Files
Coverage Δ

src/util/report-error.js
0% <0%> (ø)
:arrow_up:

src/util/get-project-name.js
90.9% <0%> (-9.1%)
:arrow_down:

src/util/get-files.js
91.53% <0%> (-8.47%)
:arrow_down:

Continue to review full report at Codecov.

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


Move to next.js

Move from Gatsby to Next. Why?
In this project, right now, the gatsby configuration it is just overwhelming and graphql it is just not useful at all (here).
the tricky point will be done using exportPathMap

Moved to next!


fetchbranches: Fix typo

Fix a typo found by the go static checkers.
Fixes: #100
Signed-off-by: Graham Whaley<EMAIL_ADDRESS>

lgtm

lgtm, but travis is failing, can you please check?

Odd Travis failure - failed to do some git magic. Here is the output. @jodh-intel any idea here before we just nudge Travis for a retry?
go test .

ok  	github.com/clearcontainers/tests/cmd/checkcommits	0.008s

go install -ldflags "-X main.appCommit="058ede1c99511d42a9d8be0851ac9c5b199daeab" -X main.appVersion=0.0.1" .

Running checkcommits version 0.0.1 (commit 058ede1c99511d42a9d8be0851ac9c5b199daeab)

Detected TravisCI Environment

Failed to run command /usr/bin/git rev-list --no-merges --reverse master..d9b082b2f63495f2f78519803ff51ca5ef002bfd: exit status 128 (stdout: , stderr: fatal: Invalid revision range master..d9b082b2f63495f2f78519803ff51ca5ef002bfd

)

@grahamwhaley - Did you maybe force push multiple times relatively quickly?  The only solution I'm aware of is to close the PR, and re-raise it (maybe add a reference between the two PRs for traceability).

@jodh-intel I don't remember doing a 'fast push', but I am due to do another push to do some updates - so, I'll try that and see what happens, and if we still fail with the SHA oddity then I'll do the PR dance. Thanks!

@grahamwhaley - if you did force push multiple times, it will never work until you re-open the PR I'm afraid.
See: https://github.com/travis-ci/travis-ci/issues/7830

(Force pushing does work, but not if you do it "too quickly" for Travis it seems).

Thinking back, no, I probably didn't force push - this was single word change in a single commit. So, I've rebased and pushed again and we'll see what happens. If Travis fails again (which I think I just saw it pass....) then I'll re-PR...


Unable to find platform speedtest 0.88.0

after upgrade to 0.88.0 the speedtest is not working.

The platform was split into component and platform and renamed. See https://www.home-assistant.io/blog/2019/02/20/release-88/#breaking-changes and docs.

Error while setting up platform speedtestdotnet
my config in sensors.yaml
sensor:

platform: speedtestdotnet
scan_interval:
minutes: 30
monitored_conditions:
- ping
- download
- upload
in configuration.yaml
speedtestdotnet:

The config section was moved from sensor to its own component. Please read the documentation.

Where is the documentation that explains how to change the scan interval?  Is this done using the category labeled "other" in the customize section?


Can't download torrents on Android N Beta 1

A toast message will sometimes pop up saying "Failed to move torrent to /storage/emulated/0/..." I've also tried setting different directories.
Tried clearing data/cache, I have Storage permissions enabled.
WhatAndroid 2.1.7 Beta on Nexus 6P running Android N Beta 1/Dev Preview 3.

I can second this. I have a nexus 5x running Android 7. The torrent downloads but is not moved into the correct folder. This stops transmission starting the torrent as it cannot find the file.


Multi Prism support

< Main issue
Add the ability to create multiple prisms from within a single application.

MXSCOMP-40


model weights and inference

Hello - thanks for this very interesting paper and work!
I was just wondering if you had model weights and inference code we can try. Thank you!

loading_ckpt
https://mail.google.com/mail/u/0?ui=2&ik=96449053f0&attid=0.1&permmsgid=msg-a:r-7239896480587092773&view=att&disp=safe&realattid=f_m1anbuxg0
(17 K)
https://mail.google.com/mail/u/0?ui=2&ik=96449053f0&attid=0.1&permmsgid=msg-a:r-7239896480587092773&view=att&disp=safe&realattid=f_m1anbuxg0Hi
Kevin,
Sorry for the late reply.
I attached the checkpoints for our model. You can load the parameters under
Pytorch Lightning's mode.
However, there are strict limitations on the input format of the model, you
can refer to the attached file for the create_batch_class function, this is
used for classification finetuning.
Best
Xing
Checkpoints_C.ckpt
https://drive.google.com/open?id=1qdcXb9CRJZgniNVaViiZaQXNiWxpiTnQ
Kevin Song @.***> 于2024年9月12日周四 20:22写道：

Hello - thanks for this very interesting paper and work!
I was just wondering if you had model weights and inference code we can
try. Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/Mr81wx/HoopTransformer/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBSM4RGFNFSIN75LKI4ESRDZWHLU5AVCNFSM6AAAAABOD2O2GWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUZDGMBWGMYTQMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you for the access! I don't think I have the file with create_batch_class function. Would you be able to add that to the github repo? If not, could you send it to my email<EMAIL_ADDRESS>

try this link
https://colab.research.google.com/drive/1YCGmMn5CBC0j8d5sj45Y53FcAvxvoJWE?usp=drive_link
Kevin Song @.***> 于2024年9月20日周五 15:13写道：

Thank you for the access! I don't think I have the file with
create_batch_class function. Would you be able to add that to the github
repo? If not, could you send it to my email? @.***
—
Reply to this email directly, view it on GitHub
https://github.com/Mr81wx/HoopTransformer/issues/1#issuecomment-2363710210,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBSM4RCAQHWOVIX56NNLNN3ZXQNPXAVCNFSM6AAAAABOD2O2GWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRTG4YTAMRRGA
.
You are receiving this because you commented.Message ID:
@.***>

That works. Thank you and I'll go ahead and close the issue.

Great!
We have updated the new model extensions for defense players. Please let me know if you need them.
We're also looking for some soccer and football data to see this framework's performance. If you have similar resources and ideas, we can work together!

2024年9月20日 下午3:17，Kevin Song @.***> 写道：
That works. Thank you and I'll go ahead and close the issue!
—
Reply to this email directly, view it on GitHub https://github.com/Mr81wx/HoopTransformer/issues/1#issuecomment-2363718978, or unsubscribe https://github.com/notifications/unsubscribe-auth/BBSM4RHMXBZD4Q2C6AQNRNDZXQOALAVCNFSM6AAAAABOD2O2GWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRTG4YTQOJXHA.
You are receiving this because you commented.

Sounds good. thank you!

Another quick question on the play embeddings... I noticed the output from the encoder is in the shape of [6,121,256]. When you create play embeddings, do you just flatten this tensor? I couldn't find the method in the paper

average pooling
Kevin Song @.***>于2024年9月20日 周五16:27写道：

Another quick question on the play embeddings... I noticed the output from
the encoder is in the shape of [6,121,256]. When you create play
embeddings, do you just flatten this tensor? I couldn't find the method in
the paper
—
Reply to this email directly, view it on GitHub
https://github.com/Mr81wx/HoopTransformer/issues/1#issuecomment-2363866801,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBSM4RASYRCACZ5KX3IKGILZXQWEZAVCNFSM6AAAAABOD2O2GWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRTHA3DMOBQGE
.
You are receiving this because you commented.Message ID:
@.***>

Gotcha. Do you mind sharing the parameters you used for average pooling as well? kernel_size, stride?

we just take mean() at the 1st and 2nd dimensions，kernel size 5*121.  There
may be better parameters here.
Kevin Song @.***>于2024年9月20日 周五16:42写道：

Gotcha. Do you mind sharing the parameters you used for average pooling as
well? kernel_size, stride?
—
Reply to this email directly, view it on GitHub
https://github.com/Mr81wx/HoopTransformer/issues/1#issuecomment-2363898700,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBSM4RF4U523C7DVC44TR3DZXQX47AVCNFSM6AAAAABOD2O2GWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRTHA4TQNZQGA
.
You are receiving this because you commented.Message ID:
@.***>

Got it. Thank you!
On Fri, Sep 20, 2024 at 10:50 AM Mr81wx @.***> wrote:

we just take mean() at the 1st and 2nd dimensions，kernel size 5*121. There
may be better parameters here.
Kevin Song @.***>于2024年9月20日 周五16:42写道：

Gotcha. Do you mind sharing the parameters you used for average pooling
as
well? kernel_size, stride?
—
Reply to this email directly, view it on GitHub
<
https://github.com/Mr81wx/HoopTransformer/issues/1#issuecomment-2363898700>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/BBSM4RF4U523C7DVC44TR3DZXQX47AVCNFSM6AAAAABOD2O2GWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRTHA4TQNZQGA>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/Mr81wx/HoopTransformer/issues/1#issuecomment-2363914572,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFZB6ZG27YN4QDZRKRSQPXLZXQY2NAVCNFSM6AAAAABOD2O2GWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRTHEYTINJXGI
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Create a Twilio account

Done; invited @ssrihari and @govindkrjoshi. Let me know if anyone else needs an invite.


Add a Jenkinsfile to trigger the website build when something changes in docs

We can create it and make it each 3 hours.

Done with #40


Am I wrong or gpt-turbo-3.5 is just limited?

I have a list of report files about agriculture (pdf files and each file is about one month (Jan, Feb, ..., Dec)) located in docs, now I just get chatgpt work with those files. However, when I have a question comparing statistics between November and May, the gpt says it does not find information. Is this a normal thing?

It's a little normal for gpt-3.5 -> it seems to struggle with complex tasks
But there are ways to get around it. What does your query/index setup look like?

max_input_size = 4096
num_outputs = 1
max_chunk_overlap = 20
chunk_size_limit = 256
prompt_helper = PromptHelper(max_input_size, num_outputs, max_chunk_overlap, chunk_size_limit=chunk_size_limit)
prepend_messages = [
SystemMessagePromptTemplate.from_template("You are a helpful assistant that use Vietnamese in every response, response with the max tokens being 1")
]
llm_predictor = ChatGPTLLMPredictor(llm=ChatOpenAI(temperature=0, model_name="gpt-3.5-turbo", streaming=True), prepend_messages = prepend_messages)
service_context = ServiceContext.from_defaults(
llm_predictor = llm_predictor,
prompt_helper = prompt_helper)
def construct_index(directory_path):
documents = SimpleDirectoryReader(directory_path).load_data()

index = GPTVectorStoreIndex.from_documents(documents, service_context=service_context)

index.storage_context.persist(persist_dir='./storage')

return index

@app.route('/api/chatbot', methods=['GET'])
def chatbot():
args = request.args
input_text = args.get('input')
# rebuild storage context
storage_context = StorageContext.from_defaults(persist_dir='./storage')
index = load_index_from_storage(storage_context = storage_context,

                                service_context = service_context)

query_engine = index.as_query_engine(
    streaming=True,
)

response = stream_with_context(query_engine.query(input_text).response_gen)

return Response(response)

I do something like this, is there any thing I can do more to do the complex tasks?

I think, you need to change num_outputs to bigger value. This parameters means number of output tokens. It is not like 'n' in chatGPT.

@dinhan92 yea, @trover97 is correct, you'll want to change that (although it's not having a huge impact right now, it can cause problems later)
For complex queries like comparing statistics, you will be interested in the sub-query engine, which breaks a query down into multiple questions
https://gpt-index.readthedocs.io/en/latest/use_cases/queries.html#multi-document-queries

Sorry, I put the num_outputs to 512 before I posted this but the paragraph will get truncated, my data is in Vietnamese

@logan-markewich am I right that this Multi-Document Queries can not be used with streaming = true? Because I get the error in:
for text in response.response_gen:
AttributeError: 'Response' object has no attribute 'response_gen'
It says there is no response_gen which is used with streaming=true here

@dinhan92 I think you can still stream, but you'll need to set streaming=true in two places
See the guide here:
https://gpt-index.readthedocs.io/en/latest/how_to/customization/streaming.html

@logan-markewich I already do as that guides, In fact, I am able to do streaming with Python and Flutter, however when using stream with Multi-Document Queries, I get that error

Ah, do you mean I have to run with this:
synth = ResponseSynthesizer.from_args(streaming=True, ...)
query_engine = RetrieverQueryEngine(response_synthesizer=synth, ...)

Yes! And also be sure to set streaming=True in the OpenAI/ChatOpenAI definition

@logan-markewich
I have tried this: https://gpt-index.readthedocs.io/en/latest/examples/composable_indices/city_analysis/City_Analysis-Decompose.html
with Complex Query 1 and add streaming = true:
graph = ComposableGraph.from_indices(
GPTSimpleKeywordTableIndex,
[index for _, index in report_indices.items()],
[summary for _, summary in index_summaries.items()],
max_keywords_per_chunk=50
)
decompose_transform = DecomposeQueryTransform(
llm_predictor, verbose=True
)
custom_query_engines = {}
for index in report_indices.values():
    query_engine = index.as_query_engine(service_context=service_context)
    transform_extra_info = {'index_summary': index.index_struct.summary}
    tranformed_query_engine = TransformQueryEngine(query_engine, decompose_transform, transform_extra_info=transform_extra_info)
    custom_query_engines[index.index_id] = tranformed_query_engine

custom_query_engines[graph.root_index.index_id] = graph.root_index.as_query_engine(
    retriever_mode='simple',
    response_mode='tree_summarize',
    service_context=service_context,
    # streaming=True
)

query_engine_decompose = graph.as_query_engine(
    custom_query_engines=custom_query_engines,
    **streaming=True**
)

But it seems streaming is not in ComposableGraph,
I have tried synth = ResponseSynthesizer.from_args(streaming=True, ...)
query_engine = RetrieverQueryEngine(response_synthesizer=synth, ...) but do not know where to set custom_query_engines.
Can youi guide me?

It's a little annoying, but here's a minimum example for me
This is possible currently, but it's a little tricky. We can improve this in the future though!
>>> from llama_index import Document, GPTListIndex, LLMPredictor, ServiceContext
>>> documents = [Document("hello world")]
>>>
>>> from langchain.llms import OpenAI
>>> llm = OpenAI(model_name='text-davinci-003', temperature=0, streaming=True)
>>> service_context = ServiceContext.from_defaults(llm_predictor=LLMPredictor(llm=llm))
>>>
>>> index1 = GPTListIndex.from_documents(documents, service_context=service_context)
>>> index2 = GPTListIndex.from_documents(documents, service_context=service_context)
>>>
>>> from llama_index.indices.composability import ComposableGraph
>>> graph = ComposableGraph.from_indices(GPTListIndex, [index1, index2], index_summaries=['1', '2'], service_context=service_context)
>>> query_engines = {
...     index1.index_id: index1.as_query_engine(streaming=True),
...     index2.index_id: index2.as_query_engine(streaming=True),
...     graph.root_index.index_id: graph.root_index.as_query_engine(streaming=True)
... }
>>> query_engine = graph.as_query_engine(custom_query_engines=query_engines)
>>> response = query_engine .query("Hello world")
>>> type(response)
<class 'llama_index.response.schema.StreamingResponse'>
>>> for token in response.print_response_stream()

I am trying to do the compare contrast between 2 documents here:
storage_context_1 = StorageContext.from_defaults(persist_dir=f"./storage_Baocao_T05_2022")
index1 = load_index_from_storage(storage_context = storage_context_1,
# embed_model=embed_model,
service_context = service_context)
storage_context_2 = StorageContext.from_defaults(persist_dir=f"./storage_Baocao_T11_2022")
index2 = load_index_from_storage(storage_context = storage_context_2,
                                # embed_model=embed_model,
                                service_context = service_context)

graph = ComposableGraph.from_indices(GPTListIndex, [index1, index2], index_summaries=["Tài liệu này về Baocao_T05_2022", "Tài liệu này về Baocao_T11_2022"], service_context=service_context)
query_engines = {
    index1.index_id: index1.as_query_engine(streaming=True),
    index2.index_id: index2.as_query_engine(streaming=True),
    graph.root_index.index_id: graph.root_index.as_query_engine(streaming=True)
}

query_engine = graph.as_query_engine(custom_query_engines=query_engines)

I see that the answer is streaming, but there is no information at all in the answer, the question is: "Cho tôi hỏi tình hình trồng trọt tháng 5 so với tháng 11" and the answer is: "Xin lỗi, không có thông tin về tình hình trồng trọt tháng 5 và tháng 11 trong đoạn văn trên. Vui lòng cung cấp thêm thông tin để tôi có thể trợ giúp bạn."

When I only ask: "Cho tôi hỏi tình hình trồng trọt tháng 11", I get this error: Retrying langchain.chat_models.openai.ChatOpenAI.completion_with_retry.._completion_with_retry in 4.0 seconds as it raised RateLimitError: Rate limit reached for default-gpt-3.5-turbo in organization org-PH8zdIn1RI0V7H7Aak3Lzy73 on requests per min. Limit: 3 / min. Please try again in 20s. Contact us through our help center at help.openai.com if you continue to have issues. Please add a payment method to your account to increase your rate limit.

Yea on the trial accounts, it's heavily rate limited 😅

Ah sorry, forgot you were doing compare/contrast. You'll have the best luck with that using the newer sub question query engine (but you may hit rate limits unless you add payment info to openai lol)
https://github.com/jerryjliu/llama_index/blob/main/docs/examples/query_engine/sub_question_query_engine.ipynb

I do like this with the link you send and get this error: ValueError: LLM must support streaming and set streaming=True.
storage_context_1 = StorageContext.from_defaults(persist_dir=f"./storage_Baocao_T05_2022")
index1 = load_index_from_storage(storage_context = storage_context_1,
# embed_model=embed_model,
service_context = service_context)
storage_context_2 = StorageContext.from_defaults(persist_dir=f"./storage_Baocao_T11_2022")
index2 = load_index_from_storage(storage_context = storage_context_2,
                                # embed_model=embed_model,
                                service_context = service_context)

query_engine_1 = index1.as_query_engine(streaming=True)
query_engine_2 = index2.as_query_engine(streaming=True)

query_engine_tools = [
    QueryEngineTool(
        query_engine = query_engine_1,
        metadata = ToolMetadata(name='bao_cao_thang_5', description='Cung cấp thông tin về nông nghiệp tháng 5')
    ),
    QueryEngineTool(
        query_engine = query_engine_2,
        metadata = ToolMetadata(name='bao_cao_thang_11', description='Cung cấp thông tin về nông nghiệp tháng 11')
    )
]

synth = ResponseSynthesizer.from_args(streaming = True)

query_engine = SubQuestionQueryEngine.from_defaults(query_engine_tools=query_engine_tools, response_synthesizer=synth)

response = await query_engine.aquery(input_text)

I ve upgrade llama index, it does not meet that error yet, but a RateLimitError, it seems that I have to pay for that

Never mind, I know how to fix this, thanks @logan-markewich


Release 0.14.0

@hugovk Can you check the current code (and HISTORY) and tell me if you think we could release 0.14.0 now? I'd just like to add that the 0.14.x series will be the last supporting Python 2. Would you like to lead the release process?

If I may, I'd like to suggest to calling it 1.0 so tablib can stop using ZeroVer 😬

My plan was to use 1.0 on the Python 3-only version (that should follow 0.14.0 rather soon), but I'm open to discussion!

Oh right, that works, too. I don't mind either way.

Yep, will release this weekend. Here's a start: https://github.com/jazzband/tablib/pull/402.
As discussed in https://github.com/jazzband/tablib/issues/390, let's go for 0.14.0 for this one, and then onwards and upwards with 1.0 when dropping 2.7!

Updated release notes merged (https://github.com/jazzband/tablib/pull/402). Will now add and push a tag, and we can check the release on Jazzband:

When you’re ready to do a release to PyPI simply make sure to prepare all the code you’d like just as before (e.g. update AUTHORS, CHANGELOG, documentation, etc.), commit the changes, tag them with git tag and push the code to GitHub with git push --tags.
If all goes according to plan, Travis CI will run the test suite for the pushed tag, create release files, uploads it to the Jazzband site, for the lead members or roadies to review. They will be able to confirm the uploads and release them to PyPI individually.

https://jazzband.co/about/releases

The CI successfully built and uploaded the two distributions:

https://travis-ci.org/jazzband/tablib/jobs/600006208#L777

There are currently 2 uploads for tablib, with 2 unreleased.

Multiple uploads for version 0.14.0 found!
Please make sure you release all uploads individually below.

File name tablib-0.14.0-py3-none-any.whl

File name tablib-0.14.0.tar.gz

https://jazzband.co/projects/tablib
I downloaded the files, and can install the wheel fine with pip. Contents of the tar.gz looks good.
Will click the release buttons for each.

Tablib 0.14.0 is released!
https://pypi.org/project/tablib/0.14.0/
$ pip install tablib
Collecting tablib
  Downloading https://files.pythonhosted.org/packages/7f/8d/665f895e4355f1ee95665e003b994512c8b670148a921aa9acec9d1607fb/tablib-0.14.0-py3-none-any.whl (65kB)
     |████████████████████████████████| 71kB 993kB/s
Requirement already satisfied: openpyxl>=2.4.0 in /usr/local/lib/python3.7/site-packages (from tablib) (3.0.0)
Requirement already satisfied: markuppy in /usr/local/lib/python3.7/site-packages (from tablib) (1.14)
Requirement already satisfied: pyyaml in /usr/local/lib/python3.7/site-packages (from tablib) (3.13)
Requirement already satisfied: xlwt in /usr/local/lib/python3.7/site-packages (from tablib) (1.3.0)
Requirement already satisfied: xlrd in /usr/local/lib/python3.7/site-packages (from tablib) (1.2.0)
Requirement already satisfied: odfpy in /usr/local/lib/python3.7/site-packages (from tablib) (1.4.0)
Requirement already satisfied: jdcal in /usr/local/lib/python3.7/site-packages (from openpyxl>=2.4.0->tablib) (1.4.1)
Requirement already satisfied: et-xmlfile in /usr/local/lib/python3.7/site-packages (from openpyxl>=2.4.0->tablib) (1.0.1)
Requirement already satisfied: defusedxml in /usr/local/lib/python3.7/site-packages (from odfpy->tablib) (0.6.0)
Installing collected packages: tablib
Successfully installed tablib-0.14.0

Thanks everyone who contributed!

hurray \o/


Show progress/info screen during code sideloading

Feedback from several sources complained about perceived hangups during code sideloading and unsatisfying behavior with connection issues.
I assume this can only be implemented natively.

Related: #1695

One possible mitigation would be to use the CLI to display some kind of progress indicator. The CLI (when used for sideloading) is aware of the beginning of a load operation (by delivering boot.min.js) and the approximate end of it, due to the device connecting via WebSocket.
However this does not cover the common case that there is no connection possible to begin with. This can only by solved by the native clients. While the developer app does a pre check if a package.json is found, this is not in place if the developer console is used to enter the url.
So I propose to align the sideloading process of both clients to this:
When the developer console is enabled (i.e. we are the developer app or a debug enabled app), an application is already loaded (e.g. the developer app) and a REMOTE URL is to be loaded either by API or by the developer console, the native client performs the following steps:

If there are spaces anywhere in the given URL, remove them. (Common user error)
Perform the following checks:

Is the URL in a valid format?
Is a network connection available?
Can a package.json be found at the given url within ~3 seconds max.
Can a boot.min.js be found at the given url within ~3 seconds max.

If the answer to any of these is no, keep the current app runing and show a popup stating the reason the app could not be loaded.
If all checks pass, start loading the new app giving SOME kind of feedback the loading started, be it a toast, a splash screen, a spinning wheel, ANYTHING but a blank white screen.
If any error occurs while initializing tabris, OR no tabris.Tabris object is created within 10 seconds, open the developer console, stop the whole process, open the developer console and print an error stating the reason the load operation failed.

With the addition of the new developer tools ui and accompanying tabris.devtools object, the is has been resolved.


[News page loading] Incorrect message for user

Environment: Chrome Version 86.0.4240.111 (64-bit)/Windows 10 Home 1903
Reproducible: always
Steps to reproduce

Go to GreenCity https://ita-social-projects.github.io/GreenCityClient/#/welcome
Log into the system.
Click on 'Eco news' link in the header.
Click 'Create news'
Input valid values to all mandatory fields
Click 'Publish'

Actual result

Expected result
delete 'the'.

Not a bug.


Bazel for gRPC Python development

Bazel has been around with gRPC for a while, and the gRPC Python has adopted Bazel last summer. It's time to see what is missing to use Bazel for gRPC Python development builds.
Let's recap how to test with Bazel:
# For all available tests
$ bazel test --test_output=errors //src/python/grpcio_tests/tests/...

# For specific test
$ bazel test --test_output=errors //src/python/grpcio_tests/tests/channelz:channelz_page_test
$ bazel test --test_output=errors //src/python/grpcio_tests/tests/unit:_api_test
...

Currently, here are the pending features for Bazel:

Test with Python 3 (Bazel & Python 3 integration is really hard to get it right)
Enable all the test cases
REQUESTED Merge test BUILD files into one

If you want other features, please comment below this issue ;)

Can we merge the Bazel BUILD files for tests into a single file? I believe that's how we did it internally as well. The way it's done now makes it hard to use for people wanting to import it to monorepos (with things like copybara, etc).

@mehrdada Sounds good! Added to the description.

Any updates here? Are we solved all the pending features?

Python 3 support is added, and I’m looking for sometime to integrate the
unit tests.
On Sat, Mar 23, 2019 at 12:07 jiayu<EMAIL_ADDRESS>wrote:

Any updates here? Are we solved all the pending features?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/issues/17622#issuecomment-475896246, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHDWcPJoCS2360R55Pfngq36Zby9gToEks5vZntegaJpZM4Zo1td
.


Switch tracing on/off via automation apps

To make the app more battery friendly (#148, …) it might be helpful to be able to declare safe zones where tracing is off and the app consumes no extra power.
There may be several ways to detect what a safe zone is:

location
known WiFI
known BT devices
audio/notifications profile

Supporting all of these seems unnecessarily complicated so perhaps it's best to leave it to existing automation tools (Tasker, Automate, Locale, Bixby Routines, …). Integration with the first three is achieved via plugins (more info). Bixby Routines doesn't seem to be as extensible, but it can invoke App Shortcuts:

I think it's quite likely that Tasker/Automate/Locale can also invoke App Shortcuts so I'd start by adding these: one to disable tracking and another to enable it. Then we can test this in the wild and see if it does the job and see what interesting use cases the users come up with.
Oh and we'll get this for free: the possibility of putting on/off buttons on home screen. :-)

We are considering the simplest way which we could implement this. We would be happy to accept PR for App Shortcuts.

This looks great. +1 for app shortcuts, that could be fairly simple.

I did some research and unfortunately, app shortcuts are designed for slightly different use-case - to open the app in some state. In other words, I'm able to pause/resume the tracking but the app would be open in the same moment which is kind of unwanted. I'm now working on receiving broadcaster URL to pause/resume from other apps, I have to check compatibility with Tasker, Automate etc.

@jendakol I guess you can open the app and call finish() when it's done :)

@davidvavra You are theoretically right, but I've faced a weird issue:

When I used intent opening main Activity, the intent in onStart was different than was the real one opening the app (so I couldn't recognize what's going on).
When I used new Activity (created just for this purpose), it worked nicely but the main activity was still open after that, like if it was added to the stack.

If you have an idea how to hack this... :smile:

@jendakol Pass some intent extras or intent action to the activity. Detect if you are launching from the shortcut based on it.

@davidvavra I've tried to pass both custom action as well as ACTION_VIEW, both with extras - IIRC custom action didn't work at all, ACTION_VIEW cause opening the app with ACTION_MAIN like if the app was opened normally, no extras included - just completely different Intent than was started. I'll probably  try once again and write down all possibilities I'm trying :-(

@jendakol Let me copy code from Settle Up :)
    <shortcut
        android:shortcutId="add_transfer"
        android:icon="@drawable/ic_shortcut_add_transfer"
        android:shortcutShortLabel="@string/add_transfer">
        <intent
            android:action="SHORTCUT_ADD_TRANSFER"
            android:targetPackage="cz.destil.settleup"
            android:targetClass="cz.destil.settleup.gui.MainActivity" />
    </shortcut>

override fun onCreate(savedInstanceState: Bundle?) {
  if (isLaunchedFromShortcut() && isAddTransferShortcut()) {
   // navigate to add transfer screen
  }
}

private fun isLaunchedFromShortcut(): Boolean = intent.action?.startsWith("SHORTCUT")
            ?: false // if there is no action, it is started from widget

private fun isAddTransferShortcut(): Boolean = intent.action == "SHORTCUT_ADD_TRANSFER"

@davidvavra Yeah, but these are static shortcuts - I think they doesn't make sense in eRouška, we need to have dynamic shortcuts to be able to reflect current state (scanning/paused).

@jendakol I would add two static shortcuts - for enabling and disabling. People who automate this will call the one they want.

@davidvavra
onNewIntent FTW :tada:
Shortcuts will work :wink:

Backend guy discovering Android lifecycle 😱

Solved in #175 .

This is no longer available with current version (from play store), is it possible to get the automation back ?

Yes, we removed it due to time constraints and because the new version consumes much less battery, so the need of turning it off is minimal. Feel free to send us a pull request with the automation.

Hi @ceresek,
from our findings, this function was almost not used at all and now the need for it is even smaller. If you insist on having it, either implement it by yourself or convince enough people to request such function here or through the support :smile:

As the original poster of this idea, I must say that since the wakelock issue was fixed (which was, iirc, fairly shortly after this workaround was implemented), battery usage has not been as issue at all, thus rendering this workaround unnecessary.

The bad news is, the exposure notification API will display a user prompt every time it is start()ed, which means bye bye automation unless Google changes this behavior. So no PR.
As a small consolation, the service does pause gracefully when bluetooth is turned off, so if the user does not need bluetooth except for exposure broadcasts and scanning, the automation can simply manipulate bluetooth settings directly.


Modify lakefs image to use major

@N-o-Z Is there is reason to update docker-compose to use pull_policy: always? We use these samples during demos & webinars and pull always will slow down the container creation during the demo (even if we already have the image locally). Can you please remove pull_policy: always?


Adding window to confirm changes to softwares on the system

When a user uninstall a package software-station should show the list of packages that depends on it
and to prompt if user wants to proceed.
This is needed to avoid accidental uninstall of important packages.

basically related to https://issues.ghostbsd.org/issues/102

This window is to confirm before running the installation or deletion of packages with Software Station.


Not indented correctly

https://github.com/rook/kubectl-rook-ceph/blob/ef787842a5e73ecac7687cb7706254e574b44d9e/.github/workflows/ci-for-diff-ns.yaml#L56
This should be kubectl rook_ceph ...

ALso need to investigate in the CI why it was passing even with this bug.

Also somewhere there should be mentioned that flags should be only used before the arguments and commands.
For example:           kubectl rook_ceph -o test-operator -n test-cluster ceph status

ALso need to investigate in the CI why it was passing even with this bug.

this is not a bug, kubectl-rook_ceph or kubectl rook_ceph behaves the same since we install kubectl-rook_ceph in the $PATH
and kubectl treat it as plugin

ALso need to investigate in the CI why it was passing even with this bug.

this is not a bug, kubectl-rook_ceph or kubectl rook_ceph behaves the same since we install kubectl-rook_ceph in the $PATH and kubectl treat it as plugin

Any specific reason to install this too kubectl-rook_ceph

ALso need to investigate in the CI why it was passing even with this bug.

this is not a bug, kubectl-rook_ceph or kubectl rook_ceph behaves the same since we install kubectl-rook_ceph in the $PATH and kubectl treat it as plugin

Any specific reason to install this too kubectl-rook_ceph

yes, to accept the tool as kubectl plugin it should have prefix of kubectl-

@parth-gr can we close this?


Stuck on Checking for updates for Rust language server

Check for existing issues

[X] Completed

Describe the feature
Since 0.103.1 zed update, i get this showing all the time

It stays on app restarts. It stayed there for 2 days already. I have good network. So no idea why it's stuck. Or what to do to unblock it.

Hi Nikita,
Would it be possible to get the logs? (Either from ~/Library/Logs/Zed/Zed.log or zed: open log palette action)

https://gist.github.com/nikitavoloboev/b22a453435aae69715393b1fd2688880

zed-industries-issue-2050-stuck-rust-lsp.zip
Mirroring your upload; we've had a few issues like that which we're trying to track down


Incompatible trait for user model

The trait included in the user model should be as follows (on Laravel 5.3 with Authentication):
class User extends Authenticatable {     use Messagable; }

Can you try adding use Cmgmyr\Messenger\Traits\Messagable; at the start of your User.php file

@TomHowardUK are you referring to what is in the readme file?

Yeah @cmgmyr


production enviroment problem

Hello,
I have a little problem in production. I call sample methods in local, there is no problem. But when I call sample methods in production, I can not recieve data. Always get errors. "{
"status": 400,
"error": "RouteNotFoundException",
"message": "There is no route for ''."
}"
http://yeni.hayatatatkat.com/restApi/test
Thank you for helping me.
Sincerely.

Looks like your apache/nginx configuration is different to your local.

Hello,
I found the problem. In client.php __construct methods has a $_SERVER['PATH_INFO'] variable. And in my server this variable is empty. But in local, we get value.
Anyway I'm gonna fix this problem via workaround. But I do not understand why path_info variable is empty.
        $_SERVER['PATH_INFO'] = !empty($_SERVER['PATH_INFO']) ? $_SERVER['PATH_INFO'] : (!empty($_SERVER['ORIG_PATH_INFO']) ? $_SERVER['ORIG_PATH_INFO'] : '');
        if (isset($_SERVER['PATH_INFO']))
            $this->setUrl($_SERVER['PATH_INFO']);

Thank you.


DatasetGenerator missing

Hi @koriavinash1 ,
In Trainer.py,  there exist from DatasetGenerator import DatasetGenerator. However, there is no code named DatasetGenerator, can you provided it.
Thx!

Hi @SwagJzzZ ,
Thanks for spotting the missing file, I've made a push with DatasetGenerator ...
Thankyou


wrap callbacks in $apply

Thanks for putting this together. It was really helpful to get the slider integrated into my project.
I just want to contribute back a change that made my life easier. Most other angular components wrap callback in $apply as soon as possible, and I wanted to do the same here.

@johntseng I faced on problem at my project, that param 'to' is not working. Your solution seems good. It works well on mine too.

This project is pretty much useless without this PR. Please merge!


Block access to private and public pages for locked users

Now if a user is locked because his/her trial has ended we:

Return 404 for their public resources like visualizations
Redirect to an upgrade page when he/she try to access a private part

So @javitonino it's ok to move it to acceptance?

Can you review Hound complaints first? It seems like you may have some indentation problems. Also, a NEWS entry would be nice (this is a new feature after all!). Otherwise, LGTM, to acceptance after those changes.

I fixed the hound keeps complaining about it, looks like it has some kind of problem. I've executed rubocop in my machine and it says everything is ok.
I haven't add nothing to the NEWS.md because I don't know if we want to public this as a whole or ot

@ceritium talking about the use of skip_before_filter in the affected endpoints:

I'm not a big fan of using the dreaded filters (dreaded for me of course) but in this particular case makes sense
I want to have this behavior encapsulated in the application controller and I'm not sure if spread skip_* through the controllers is a good idea

In any case, I really appreciate the suggestions, especially the skip_before_filter one and I'm going to make some fix using your advises :)

Everything seems 👍 . Moving to deploy


prjtrellis: update SRC_URI

The main development now happens on https://github.com/YosysHQ/ instead of the SymbiFlow organization.

Thanks


[Feat]: Add Jenkins

What feature?
The leading open source automation server, Jenkins provides hundreds of plugins to support building, deploying and automating any project.
Add screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

The template you used is incorrect. Please use the correct template and create a new issue.


Nasty bug with SelectableTextBlock & TextAlignment = Right

Describe the bug
The first thing to say is that I can't expect a solution because I can't repeat the problem in a minimal code sample. In my application, I have Grids within Grids containing Controls that are built automatically from JSON. The problem does not seem to want to show itself in a simple example.
However, perhaps the resolution might be apparent to the Avalonia team, or perhaps there might be a suggestion. I will post again if I find a way to create a minimal sample.
THE PROBLEM
The problem is this: when SelectableTextBlock is used within TextAlignment=Right, the contents are hidden in some cases. I cannot fathom, so far at least, what property values cause it to be hidden in some, and not in others. Screenshots below.
Here are the characteristics:

The problem does NOT manifest with TextBlock, but only with SelectableTextBlock
It does not manifest with SelectableTextBlock.TextAlignment=Left, but only with Right
If the SelectableTextBlock is forced to resize once on screen, the contents magically appear
The problem has an intermittent component. Approx. 1 in 10 executions result in the contents displaying normally with no change to the code.

MAYBE IMPORTANT: I am using TextWrapping=Wrap.  FlowControl is Left (not right). Despite what the screenshot below may suggest, I am only setting TextAlignment.
Here are 3 screenshots:
Below, SelectableTextBlock with alignment Right. This is the problem.

Below, TextBlock with alignment Right. This is OK.

Below, SelectableTextBlock with alignment Left. This is OK.

To Reproduce
Unable to produce minimal sample that displays the problem.
Expected behavior
No response
Avalonia version
11.1.4
OS
Linux
Additional context
No response

To add, I am using a subclass of SelectableTextBlock. It has the following override:
protected override Type StyleKeyOverride => typeof(SelectableTextBlock);

If you use plain control and not your own, does it reproduce?
Do you load items async by any chance? Is so, use the Dispatcher maybe
There are some text related PRs currently which you may want to test

Hi, thanks for the replying

If you use plain control and not your own, does it reproduce?

I have literally just tried that now, so that I am not subclassing. Yes, the problem remains.

Do you load items async by any chance? Is so, use the Dispatcher maybe

No. No async in my code.

There are some text related PRs currently which you may want to test

What's a PR?

Problem report?

#17271

Sure, I'll try it. But I'm having no luck referencing it.
<PackageReference Include="Avalonia" Version="11.2.999-cibuild0047982-alpha" />
<PackageReference Include="Avalonia.Desktop" Version="11.2.999-cibuild0047982-alpha" />
<PackageReference Include="Avalonia.Themes.Fluent" Version="11.2.999-cibuild0047982-alpha" />

error NU1102: Unable to find package Avalonia with version (>= 11.2.999-cibuild0047982-alpha)
What do I need to do?

Do you have the feed uri from the bot comment in your nuget feed sources? Preview checkmark may be needed checked as well.

OK, I figured it and added the feed. Thank you.
The problem still remains, however.

Found if I double click on the missing area, the text appears.

Mmmm. Strange. I manually deleted the bin/obj directories and find now that I can't repeat the problem any longer with 11.1.4.

Scrub my last remark. The problem remains, and with 11.2.999-cibuild0047982-alpha.
I'd simply left in the use TextBlock rather than SelectableTextBlock.

I have a sample, and some thoughts.
protected override void OnOpened(EventArgs e)
{
    Width = 500;

    var container = new Grid();
    container.Background = new SolidColorBrush(Colors.AliceBlue);
    container.ShowGridLines = true;

    container.HorizontalAlignment = Avalonia.Layout.HorizontalAlignment.Right;
    //  Comment out above, and uncomment below, and Shows!
    // container.HorizontalAlignment = Avalonia.Layout.HorizontalAlignment.Stretch;

    container.RowDefinitions.Add( new(new(20, GridUnitType.Pixel)) );
    container.RowDefinitions.Add( new() );
    container.RowDefinitions.Add( new() );
    container.ColumnDefinitions.Add( new(100, GridUnitType.Pixel) );

    container.ColumnDefinitions.Add( new(1.0, GridUnitType.Star) );
    //  Comment out above, and uncomment below, and Shows!
    //container.ColumnDefinitions.Add( new(GridLength.Auto) );

    container.ColumnDefinitions.Add( new(100, GridUnitType.Pixel) );

    var block = GetBlock<SelectableTextBlock>("CommandButton1");
    //  Comment out above, and uncomment below, and Shows!
    // var block = GetBlock<TextBlock>("CommandButton1");

    block.Foreground = new SolidColorBrush(Colors.Red);
    container.Children.Add(block);
    Grid.SetColumn(block, 1);
    Grid.SetRow(block, 1);

    Content = container;
}

private static TextBlock GetBlock<T>(string text)
    where T : TextBlock, new()
{
    var b = new T();
    b.TextWrapping = TextWrapping.Wrap;
    b.Text = text;
    b.TextAlignment = TextAlignment.Right;
    b.VerticalAlignment = Avalonia.Layout.VerticalAlignment.Top;
    b.HorizontalAlignment = Avalonia.Layout.HorizontalAlignment.Stretch;
    return b;
}

The text, in the above code, does not show most of the time. But uncomment/comment-out any one of the highlighted lines, and it does.
I think at the root of the problem is that we have a Grid with a GridUnitType.Star column, where the Grid itself is not Stretch. I can see that this scenario is fundamentally hard to resolve in terms of sizing, because I had to do it myself in my own code(*).
HOWEVER, the code above behaves very enigmatically. Do not be surprised if the problem does not show itself. Bizarrely, the same code when run again will behave differently. I include a screenie below to prove that I am not imagining this.
In the above, the behavior is different with a TextBlock vs SelectableTextBlock. With TextBlock, the text content ALWAYS appears, with SelectableTextBlock mostly it doesn't, but sometimes it does.
(*) I tried to avoid it, and set HorizontalAlignment to Stretch when alignment is Left. However, what might be happening is when something is Right, the code says: "Oh that should be Right!" and overrides HorizontalAlignment as well. I'll check this out.

This is the build you should be using 11.3.999-cibuild0052710-alpha

Same problem, only that double clicking on SelectableTextBlock no longer causes it show.

What I think I have is this:

If SelectableTextBlock.TextAlignment=Left, there's no problem. Only with Right. We can see the "3" text shows, although this will also be set to Right. I don't think I can avoid this scenario.

Please try the Dispatcher. It solves the issue for me looks like. If this is by design? I think not.
/cc @Gillibald
protected override void OnOpened(EventArgs e)
{
    Width = 500;
    Dispatcher.UIThread.Post(PrepareContent, DispatcherPriority.Loaded);
}

private void PrepareContent()
{
    var container = new Grid();
    container.Background = new SolidColorBrush(Colors.AliceBlue);
    container.ShowGridLines = true;

    container.HorizontalAlignment = Avalonia.Layout.HorizontalAlignment.Right;
    //  Comment out above, and uncomment below, and Shows!
    // container.HorizontalAlignment = Avalonia.Layout.HorizontalAlignment.Stretch;

    container.RowDefinitions.Add( new(new(20, GridUnitType.Pixel)) );
    container.RowDefinitions.Add( new() );
    container.RowDefinitions.Add( new() );
    container.ColumnDefinitions.Add( new(100, GridUnitType.Pixel) );

    container.ColumnDefinitions.Add( new(1.0, GridUnitType.Star) );
    //  Comment out above, and uncomment below, and Shows!
    //container.ColumnDefinitions.Add( new(GridLength.Auto) );

    container.ColumnDefinitions.Add( new(100, GridUnitType.Pixel) );

    var block = GetBlock<SelectableTextBlock>("Test");
    //  Comment out above, and uncomment below, and Shows!
    // var block = GetBlock<TextBlock>("CommandButton1");

    block.Foreground = new SolidColorBrush(Colors.Red);
    container.Children.Add(block);
    Grid.SetColumn(block, 1);
    Grid.SetRow(block, 1);

    Content = container;
}

private static TextBlock GetBlock<T>(string text)
    where T : TextBlock, new()
{
    var b = new T();
    b.TextWrapping = TextWrapping.Wrap;
    b.Text = text;
    b.TextAlignment = TextAlignment.Right;
    b.VerticalAlignment = Avalonia.Layout.VerticalAlignment.Top;
    b.HorizontalAlignment = Avalonia.Layout.HorizontalAlignment.Stretch;
    return b;
}

Thank you. I tried it but it doesn't work with my application. There is Grid within Grid in that.

We need a minimum sample to reproduce, otherwise no idea tbh.

I'd thought I'd given one.

@timunie , Actually the work around you suggested does appear to work with 11.1.4, but not with 11.3.999-cibuild0052710-alpha.
I was linking against   11.3.999-cibuild0052710-alpha.
Thank you for the suggestion.

I have a request. Can we make the following method protected scope in the next release?
InlineUIContainer.AppendText()
This refers to #17239, which I could extend myself if it was protected, rather than internal.
I will try to implement the SelectableTextBlock class myself simply by copying your own code initially. This may allow me to look at the following:
SelectableTextBlock does not highlight selection when Inline has background color #17244
SelectableTextBlock does not work properly when TextAlignment is not Left #17243
SelectableTextBlock does not highlight selection when Inline has background color #17244
I'm not sure I can do this because I may not access scope to the required methods without building Avalonia itself. Looking at the SelectableTextBlock code itself, I think I might be able to excect for one issue:
This is in the static constructor:
    CopyingToClipboard = RoutedEvent.Register<SelectableTextBlock, RoutedEventArgs>("CopyingToClipboard", RoutingStrategies.Bubble);

However, CopyingToClipboard is a non-static member. I'm not clear on how you managing to compile this.
Anyway, I need to do other things in life for a little while, but will come back to it.
Thanks again

Please try the Dispatcher. It solves the issue for me looks like. If this is by design? I think not. /cc @Gillibald
protected override void OnOpened(EventArgs e)
{
    Width = 500;
    Dispatcher.UIThread.Post(PrepareContent, DispatcherPriority.Loaded);
}

private void PrepareContent()
{
    var container = new Grid();
    container.Background = new SolidColorBrush(Colors.AliceBlue);
    container.ShowGridLines = true;

    container.HorizontalAlignment = Avalonia.Layout.HorizontalAlignment.Right;
    //  Comment out above, and uncomment below, and Shows!
    // container.HorizontalAlignment = Avalonia.Layout.HorizontalAlignment.Stretch;

    container.RowDefinitions.Add( new(new(20, GridUnitType.Pixel)) );
    container.RowDefinitions.Add( new() );
    container.RowDefinitions.Add( new() );
    container.ColumnDefinitions.Add( new(100, GridUnitType.Pixel) );

    container.ColumnDefinitions.Add( new(1.0, GridUnitType.Star) );
    //  Comment out above, and uncomment below, and Shows!
    //container.ColumnDefinitions.Add( new(GridLength.Auto) );

    container.ColumnDefinitions.Add( new(100, GridUnitType.Pixel) );

    var block = GetBlock<SelectableTextBlock>("Test");
    //  Comment out above, and uncomment below, and Shows!
    // var block = GetBlock<TextBlock>("CommandButton1");

    block.Foreground = new SolidColorBrush(Colors.Red);
    container.Children.Add(block);
    Grid.SetColumn(block, 1);
    Grid.SetRow(block, 1);

    Content = container;
}

private static TextBlock GetBlock<T>(string text)
    where T : TextBlock, new()
{
    var b = new T();
    b.TextWrapping = TextWrapping.Wrap;
    b.Text = text;
    b.TextAlignment = TextAlignment.Right;
    b.VerticalAlignment = Avalonia.Layout.VerticalAlignment.Top;
    b.HorizontalAlignment = Avalonia.Layout.HorizontalAlignment.Stretch;
    return b;
}

Hi Tim, @Gillibald
The work around you suggested has stopped working in 11.2.2.
This is a breaking issue for me. Any ideas?
Cheers

Update, adding a forced change of size seems to be a work around:
public MainWindow()
{
    // Bug #17305 work around
    Width = 0;
}

protected override void OnOpened(EventArgs e)
{
    Width = 1400;
    Height = 1000;
    Dispatcher.UIThread.Post(PrepareContent, DispatcherPriority.Loaded);
}


feat: Use docker compose instead of a single docker container

Is your feature request related to a problem? Please describe.
A lot of our developers work on Windows and running start-database.sh requires a bunch of extra steps
Describe the solution you'd like to see
start-database.sh suggest running itself in WSL, but I think it would be way easier to supply a docker-compose.yml. This also allows for random password generation to be used in the docker-compose.yml and in the .env file.
Describe alternate solutions
N/A
Additional information
If people are interested in this solution, I can make a pull request with the suggested functionality.

Hi, to clarify, do you see the server and client parts of the nextjs to be separate docker containers? or something else?

@datasalaryman just the database (so the stuff that's currently being started into docker when you run the start-database.sh command)

@TheDevilOnLine so the docker-compose file will be structured like this?
services:
  web:
    ...
  postgres:
    ...

okay, I'll see if I can take that on


Use the travis-ci cache to speed up builds

Coverage remained the same at 61.698% when pulling 118090ae7aa03724b79a5395cda48c4e0a5d99b5 on travis-cache into 2d28fecd4aab5f6a25a7826b2abf9f1a787eb4c7 on master.


[...] /path/to/project/root/package.json can currently [..] have [...] no other fields

Hey there!
I have a project directory, where both cargo and npm reside. The package.json has, among others, a scripts section to centralize custom commands related to the project.
When I add
#[wasm_bindgen(module = "/src/foo.ts")]
[...]

to a Rust file, I get the following error when compiling (in this case by invoking wasm-pack, but it should be unrelated):
error: NPM manifest found at `/path/to/project/root/package.json` can currently only have one key, `dependencies`, and no other fields
Error: Running the wasm-bindgen CLI
Caused by: failed to execute `wasm-bindgen`: exited with exit code: 1

Can you help me understand why this limitation exists? Why is e.g. ignoring the other keys not an option?

I am also having the same issue (I am also in a ts project)
I changed my import to point to some JS incase it was that.
#[wasm_bindgen(module = "lib-jess/src/bindings/ts/helpers/import.js")]
extern "C" {
    fn import_jess_sync(path: &str) -> Result<String, JsValue>;
}

I was using JS and had the same error. Had to move all the JS code to a separate folder and it worked :grimacing:

Interesting il try that. I think the function I wanted to export was in the same folder as some ts sources. This looks like an edge case and some unexpected behaviour, certainly unhelpful error either way.

Thanks for the report! This was initially done to be as conservative as possible, but I think that this has tripped up enough users that it's fine to ignore all non-dependencies keys and only use the ones that are relevant. In that sense I'd be happy to have a PR to change this!

@alexcrichton its this code at crates/cli-support/src/js/mod.rs:2776
if key != "dependencies" || iter.next().is_some() {
            bail!(
                "NPM manifest found at `{}` can currently only have one key, \
                 `dependencies`, and no other fields",
                path.display()
            );
}

My Wasm module is cohabiting with a Typescript package, so I did what most would do, npm init, this means I share this package with several other libraries and test tools. I understand the philosophy of pay for only what you use and minimalism but, the package.json should be out of scope, it is not in Rusts domain. I would trust npm do its validating checks. Unless I am missing something critical I don't understand this check at all, what happens if I just delete it?
Happy to PR this, just not contributed to anything before.


Circular dependency prevents startup for MongoDB-based Event Publication Registry & AfterSaveCallback callback

It seems similar to GH-181
When using @ApplicationModuleListener and AfterSaveCallback on the same spring bean it results in an Circular dependency notification that cannot been prevented by using pring.main.allow-circular-references = true.
Error:
The dependencies of some of the beans in the application context form a cycle:

┌─────┐
|  documentManagementServiceImpl defined in file [/Path/to/spring-modulith-test-event-error/target/classes/test/document/DocumentManagementServiceImpl.class]
↑     ↓
|  documentRepository defined in test.document.domain.DocumentRepository defined in @EnableMongoRepositories declared on MongoRepositoriesRegistrar.EnableMongoRepositoriesConfiguration
↑     ↓
|  mongoTemplate defined in class path resource [org/springframework/boot/autoconfigure/data/mongo/MongoDatabaseFactoryDependentConfiguration.class]
↑     ↓
|  mappingMongoConverter defined in class path resource [org/springframework/boot/autoconfigure/data/mongo/MongoDataConfiguration.class]
↑     ↓
|  extractionEventListener defined in file [/Path/to/spring-modulith-test-event-error/target/classes/test/extract/application/ExtractionEventListener.class]
↑     ↓
|  testUseCase defined in file [/Path/to/spring-modulith-test-event-error/target/classes/test/extract/application/TestUseCase.class]
└─────┘

Listener
@Component
public class ExtractionEventListener implements AfterSaveCallback<Extraction> {

    private TestUseCase testUseCase;

    public ExtractionEventListener(TestUseCase testUseCase) {
        this.testUseCase = testUseCase;
    }

    @ApplicationModuleListener
    public void handle(DocumentReadyToRun event) {
        System.out.println(event.documentId());
    }

    @Override
    public Extraction onAfterSave(Extraction entity, Document document, String collection) {
        return entity;
    }
}

A full test setup can be found here: https://github.com/arne-kroeger/spring-modulith-test-event-error
Moving  @ApplicationModuleListener method to a separate component is my current workaround.

Even listeners are obtained very early in the lifecycle of the ApplicationContext and required by the MongoMappingContext. In combination with implementing the AfterSaveCallback it's easy to create these cycles as they're looked up very early as well. I guess there's nothing we can do about that on the Spring Modulith side.


small doc changes from GitHub issues

change order of genes x cells in create_profile_matrix documentation.
add updateGeoMxSet() to vignette

Don't forget version bump prior to push to Bioc


toLocaleString() not working

When specifying toLocaleString() and using MockDate it always returns the US format:
MockDate.set('2022-01-31');

it('should format to UK time', () => {
   expect(new Date().toLocaleString('en-GB')).toBe('31/1/2022, 12:00:00 AM');
});

Expected: "31/1/2022, 12:00:00 AM"
Received: "1/31/2022, 12:00:00 AM"

Odd, I am not able to reproduce:
  it('should format to other locales', function() {
    MockDate.set('1/31/2020 1:00 PM');
    should.equal(new Date().toLocaleString('en-GB'), '31/01/2020, 13:00:00')
  })

Is en-GB your default locale on your computer?


[requires.io] dependency update on master branch

This change is

Current coverage is 100.00%

Merging #33 into master will not affect coverage as of 15308c7

@@            master     #33   diff @@
======================================
  Files            7       7
  Stmts          263     263
  Branches        23      23
  Methods
======================================
  Hit            263     263
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 15308c7

Powered by Codecov. Updated on successful CI builds.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Expression Evaluation

Hi,
Is there a way to evaluate/convert the IEnumerable<WhereExpressionInfo<T>> WhereExpressions  to Expression<Func<T, bool>>
Thanks,

Not sure what are your requirements, but if you're trying to apply a given specification against in-memory data, that is already available out of the box. Here is a sample
var spec = new CustomerSpec();
var customers = new List<Customer>();
var evaluatedCustomers = spec.Evaluate(customers);

Thanks for your quick reply. I am asking if I can  evaluate/convert to Expression<Func<T, bool>> meaning reducing all the IEnumerable<WhereExpressionInfo<T>> WhereExpressions to a single Expression
The reason I want to do this I wanna use the Specification against AzureTableStorage but their API does not support IQuerable they only accept Expression<Func<T, bool>> So I want to convert IEnumerable<WhereExpressionInfo<T>> WhereExpressions  to Expression<Func<T, bool>> and pass it as a parameter to their API

Hey @Jimmy-Ahmed,
Yea, it can be done. It's not directly related to this library, you just need to merge the expressions.
Here is an extension for you
public static class SpecExtensions
{
    public static Expression<Func<T, bool>>? MergeWhereExpressions<T>(this ISpecification<T> spec)
    {
        Expression? result = null;
        var parameter = Expression.Parameter(typeof(T), "x");

        foreach (var expression in spec.WhereExpressions.Select(x => x.Filter))
        {
            var expr = ParameterReplacerVisitor.Replace(expression, expression.Parameters[0], parameter) as LambdaExpression;
            _ = expr ?? throw new InvalidExpressionException();

            result = result is null
                ? expr.Body
                : Expression.AndAlso(result, expr.Body);
        }

        return result is null
            ? null // If the input is empty collection, what do you want to return? null?
            : Expression.Lambda<Func<T, bool>>(result, parameter);
    }
}
public class ParameterReplacerVisitor : ExpressionVisitor
{
    private readonly Expression _newExpression;
    private readonly ParameterExpression _oldParameter;

    private ParameterReplacerVisitor(ParameterExpression oldParameter, Expression newExpression)
    {
        _oldParameter = oldParameter;
        _newExpression = newExpression;
    }

    public static Expression Replace(Expression expression, ParameterExpression oldParameter, Expression newExpression)
      => new ParameterReplacerVisitor(oldParameter, newExpression).Visit(expression);

    protected override Expression VisitParameter(ParameterExpression p)
      => p == _oldParameter ? _newExpression : p;
}

And the usage
var spec = new CustomerSpec();
var expr = spec.MergeWhereExpressions();

Thanks a lot! This works like a charm :) I have a final question is there a way to convert/evaluate Expression<Func<T, TResult>> Selector to a list of strings meaning if you have a selector to get Prop1, prop2, prop3 for example and you called something like
Selector.ToStringSelector() this would return a list of strings containing "Prop1", "Prop2", "Prop3"
Thanks in advance...

It failed on the edge cases like Score = p.Scores[0] however it was a good start. Thanks for sharing :)

Hey, @Jimmy-Ahmed.
I assume Steve's suggestion gave you some hints on how to move forward. I'm closing this issue. If you have further questions regarding the library, let us know.


fix(1384): SmartLDAP fixes and improvements

Address small bugs in SmartLDAP code, add recursive getParents() method, and make child key modification optional.
Resolves #1384.

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines
[x] documentation is changed or added

Description of change

@jgribonvald This PR extracts the child key modification into an external bean that will be autowired when added to a Spring Framework context. This should allow for your customization as well as support uPortal installations that need this to be unmodified.


Remove support for CONFIG_FS_WRITABLE

And also CONFIG_FS_READABLE.
This resolves issue #484 and is a companion to the PR 131 in incubator_nuttx_apps.  Per Issue 484:
This is kind of a useless configuration variable that occurs in many places and adds a lot of complexity and makes debug a lot more difficult.
It makes debug more difficult because it is not consistently set and requires a lot of debug time before you finally figure out "I need to set CONFIG_FS_WRITABLE.
In most places affected by CONFIG_FS_WRITABLE, the configuration option is stupid: Who needs read-only MMC or read-only SmartFS. Just not a useful thing. And many places that do need CONFIG_FS_WRITABLE do not automatically set it. Like drivers/mtd/ftl.c. In that case, you can debug for most of the day before you finally figure out why the you can't access flash via character driver (which is a really common configuration for resource limited platforms).
It is hard to imagine a platform that supports a file system but has no writing enabled. I could imagine a system with only ROMFS or only CROMFS. But that is really about it and, in those cased, CONFIG_FS_WRITABLE has no effect. I am not aware of any such case, but that would be the only kind of platform that would suffer from the change.
But my opinion is that in the bigger picture, NuttX would be cleaner, less complex, and more usable with CONFIG_FS_WRITABLE removed.
CONFIG_FS_REABABLE was also becaue it does nothing.

@xiaoxiang781216 This is currently failing the same sim checks that is blocking your PR.
However, if we get past that, I don't think it will be able to build the checks.  I will verify that any failed configurations work on my PC, but I do not expected the nutx/ changes to build correctly without the apps/ changes or vice versa.
What do you recommend?

@xiaoxiang781216 yes, I verfied on apps/ PR 131 that the failing builds workd perfectly will when both the apps/ and nuttx/ changes are in place.  How, do you propose that we verify this?

@xiaoxiang781216 yes, I verfied on apps/ PR 131 that the failing builds workd perfectly will when both the apps/ and nuttx/ changes are in place. How, do you propose that we verify this?

Let's bypass the precheck system before it can handle the PR relationship in apps/nuttx.