Summation of tree changes

I've added a summation of all the changes to attributes, to the tooltip that appears on hovering over a node, when clicking that node would change multiple nodes in the tree (adding or removing).
It's a bit clunky, but I think it would be a nice feature to have, as you can quickly compare clusters of nodes.
It also doesn't have any form of localization support atm, but so does the tooltip line directly above.

I like the idea, I think it came up before already. However, I don't think it should always be on. It would need to be a toggle to not clutter the tooltip.

Yeah, this is something I had done before, but I never really finished the
idea. It definitely needs a menu option.
On Nov 22, 2016 6:27 AM, "brather1ng"<EMAIL_ADDRESS>wrote:

I like the idea, I think it came up before already. However, I don't think
it should always be on. It would need to be a toggle to not clutter the
tooltip.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/EmmittJ/PoESkillTree/pull/402#issuecomment-262254109,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHIbbbuAX8xacx_2Igd4lsEEJoX-HRM2ks5rAvu6gaJpZM4K48Ml
.

I've changed it to only show while pressing Ctrl.

So, I had some talk with @brather1ng, and we aren't sure ctrl is the best option (since ctrl + click is for zooming out). What about adding it to the menu as well as giving it a hotkey? This way you can toggle it on/off pretty fast without having to hold down a key.

Sure, I'll change it that way, it'll just take a bit of time for me to get into the code some more.
Is there any key you would prefer, or should I just pick one (or is there a system for rebindable keys)?

...you bound it to what most programs use as "Search" hotkey?

It's an easily reached hotkey and not yet bound.
And that's why I asked for they key I should bind it to,  I'm shit at choosing

What about Ctrl + g as in Group? I agree that Ctrl + f is not the best. I think I will bind that to the search-box actually :)

Seems to work fine 👍
I think the menu entry would fit better under View, but that's just my opinion.

Okay, I've added both those suggestions in.
I've also restructured the code a bit to make githubs diff spazz out less and to make it easier to see what I changed.

Sweet! Looks good. I am going to merge this now!


Remove all direct imports of css in favor of injected prop

Summary
Remove all direct imports of css in favor of the css injected prop. This is the preferred usage and is a pre-req to using contextual react-with-styles.
Reviewers
@majapw @TaeKimJR @ahuth

Coverage remained the same at 84.895% when pulling 8a6d2207c0007649a7d91bfd605a2379af3285b5 on nora--remove-direct-imports-of-css into 8c0ba9886a9f8410f65e65153e5a02e8c987ba08 on master.

Failing test seemed due to flakiness at UTC midnight; i reran.


Cookiecutter

When you all have time, try it out and lmk what you think! The command in the docs won't work until this is merged, but for now you can try
cookiecutter https://github.com/jmcarp/councilmatic-starter-template

[Resolves #13]

Ping @reginafcompton @evz: no rush on this, but looking forward to your thoughts when you have time. I would also like to add https://github.com/datamade/nyc-councilmatic/pull/214 to this if you all are interested.

@reginafcompton @evz @fgregg: anybody have time to review?

Ping @reginafcompton @evz @fgregg.


Improve performance and accuracy by baking in targets structure knowledge?

targets has a basic structure.  Once you have the directory for a target, a lot of dependencies are kept in hashed folders. I suspect those hashes include the version and feature flags.

This means for each of those folders, we can bail out at the first file that has too new of an atime.
We probably want to keep these entire folders rather than delete only part of them

The problem is this becomes more brittle because it becomes dependent on how cargo works.

This would be a nice change, but it would perhaps also involve some restructuring since we would have to defer the actual deletion until we are certain that all files are old enough and then delete the entire directory.
I also do get the issue of being sensitive to changes of how cargo/rustc stores the files in the target directory, although I would guess that it is pretty stable by this point.

I would guess that it is pretty stable by this point.

It is pretty stable, in that It has not changed in a very long time. It is an implementation detail, in that the Cargo team would be comfortable changing it. There has been some buzz about adding a folder to make it clear what is from crates.io (and unlikely to change) vs other sources (more likely to change), or a folder for each compiler version. If there are changes to the structure that would make it easier for cargo-sweep to work well, please let us know at the Cargo repo.

While investigating https://github.com/holmgr/cargo-sweep/issues/2#issuecomment-447607139, I learned some things about how the folder is structured. As I understand it, the folders come in sets. For example, if there is a folder atty-5ea146758e503e24 in /target/debug/.fingerprint then there may be a corresponding folder in /build, /incremental, /native and files with that name in /deps and /. I think it would be most reliable to keep/remove all of these files as a set. I believe (but need to verify) that cargo looks at the files in /.fingerprint for every transitive dependency on every bild. So the atime from there may be the most reliable. I also think that it will not be a big PR to have cargo add a timestamp file while it is looking at the fingerprint.
Meanwhile, the fingerprint already includes the version of rust needed for the files, sow next time I have a sec I will work on a version that lets you cargo +stable build && cargo +nightly build && rustup toolchain remove nightly && cargo sweep --installed and it will remove all the files related to nightly leaving you with the same files as if you had only run cargo +stable build. (Witch a lot of peepal need to clean the cache folder on CI when nightly updates. like https://github.com/rust-lang/crates.io/pull/1578)

Hmm, looks interesting. However, are we sure that .fingerprint is actually touched when doing a compile every time, i.e even if we compile twice in a row?
If you have time on work on that feature that cleans the cache folder on CI on nightly updates I would really appreciate it!

Move to close?

👍


DEV: pin sphinx in environment.yml

Reference issue
n/a
What does this implement/fix?
gh-21324 for environment.yml too. @agriyakhetarpal how are we looking for removing these pins after https://github.com/jupyterlite/jupyterlite-sphinx/pull/201 ?
Additional information
@gertingold bumped into this at the EuroSciPy 2024 sprint!

Hi @lucascolley, thanks for the ping! This should be resolved by now, shouldn't it? jupyterlite/jupyterlite-sphinx#201 was released in https://github.com/jupyterlite/jupyterlite-sphinx/releases/tag/v0.16.5, and https://anaconda.org/conda-forge/jupyterlite-sphinx/files has version 0.16.5 available, too. Please let me know if if you think something was/is misconfigured!

would you like to open a PR to remove the pins, and see if CI agrees? I can't remember the exact message we saw, but it was some deprecation warning coming from Sphinx.

which had the same issue until yesterday

Right, that might be it!

The error I was encountering today with Sphinx 8.0.2 was
Exception occurred:
  File "/home/gert/.conda/envs/scipy-dev-test/lib/python3.12/site-packages/sphinx/util/_pathlib.py", line 96, in __getattr__
    warnings.warn(_MSG, RemovedInSphinx90Warning, stacklevel=2)
sphinx.deprecation.RemovedInSphinx90Warning: Sphinx 9 will drop support for representing paths as strings. Use "pathlib.Path" or "os.fspath" instead.

That's iteresting, Sphinx seems to report a warning from Sphinx itself...

thanks @agriyakhetarpal ! I'll merge this to fix the dev setup right now, and hopefully we can remove the pin in gh-21482 soon.

Yes, sounds great, @lucascolley! I hope your talk at EuroSciPy went well and that you have a great time at the NF summit!


Fetch shop data after user is authenticated

Porting from #1427

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://master.staging.saleor.cloud/graphql/

Task linked: SALEOR-4442 Permissions list is not showed on non full access group


Release

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

:exclamation: No coverage uploaded for pull request base (main@e220481). Click here to learn what that means.

:exclamation: Current head 135b11e differs from pull request most recent head f00bdb1. Consider uploading reports for the commit f00bdb1 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage        ?   89.10%
  Complexity      ?      128
=======================================
  Files           ?        4
  Lines           ?      413
  Branches        ?       37
=======================================
  Hits            ?      368
  Misses          ?       26
  Partials        ?       19

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


Upgrade mysql-connector to 8.0.16

Did this fix your issue?

yep, I haven't seen the issue since we upgraded to this version


Add Adoptium Logo Styleguide Link to the "Branding" Documentation

Hello team,
Can we add the https://adoptium.net/es/docs/logo-styleguide/ link to the "Branding" block of information in the Documentation page: https://adoptium.net/docs/?  Thank you!

This is live for a moment.


Cannot run wpr / perfview inside docker container

Please fill out all the sections below for bug issues, otherwise it'll be closed as it won't be actionable for us to address.
Describe the bug
Performance tools like WPR and Perfview are unable to start inside microsoft container
(mcr.microsoft.com/windows/servercore:ltsc2022)
To Reproduce
Run the following commands:
(from host)
docker pull mcr.microsoft.com/windows/servercore:ltsc2022
docker run -dit mcr.microsoft.com/windows/servercore:ltsc2022
docker exec --user ContainerAdministrator -it  cmd
(from container)
wpr.exe -start CPU
Expected behavior
Performance tool starts collecting metrics
Configuration:

Edition: Windows 10.0.20348 Microsoft Windows Server 2022 Datacenter
Base Image being used: mcr.microsoft.com/windows/servercore:ltsc2022
Container engine: docker
Container Engine version 22.06.0

Additional context
error message received:
A required privilege is not held by the client.
Profile Id: CPU.Verbose.Memory
Error code: 0x80070522

Looking into this

Hi! It looks like we cannot start a trace for the CPU profile in a container at the moment. I'm trying to confirm that this is true. If we can in fact get CPU data in a container using WPR then I'll provide you with those steps.

@dindo-dindo
While we continue to work on providing more support for WPR in containers, there are other ways to gather CPU data in a container. You can use a command-line tool called logman that's already available in Windows Server Core 2022.
I don't know exactly what CPU data you'd like to measure but here's an example of how to capture the percentage of CPU time used by all processes on the system every 5 seconds.
Instructions

Start up your container in interactive mode (exactly how you did in your repro steps).

docker run -dit mcr.microsoft.com/windows/servercore:ltsc2022
docker exec --user ContainerAdministrator -it cmd

Inside the container, create a new counter log named "CPU Usage".

logman create counter -n "CPU Usage" -c "\Processor(*)\% Processor Time" -si 5 -f csv -o C:\logs\cpu_usage.csv

To check that you successfully created a counter log you can run logman query and you should see:

Data Collector Set              Type                  Status
----------------------------------------------------------------------
CPU Usage                       Counter               Stopped

Now we can start to collect data. You can confirm it is running by using logman query again.

logman start "CPU Usage"

When you're done collecting data:

logman stop "CPU Usage"

To see the output of the data, navigate to the folder C:\logs\ and you should see a new csv file. You can read the contents using a command like:

more cpu_usage.csv

More Details

-n defines the name of the counter as "CPU Usage"
-c specifies the counter path to the monitor which is all processes here
si specifies the interval of capture as 5 seconds
-f defines the log file format as CSV
-o is the output file location as C:\logs\ and file name cpu_usage.csv

Resources

Logman documentation
Collecting Performance Data into a CSV File Using Logman
Logman Forum

Has this been fixed in some way? I am hitting the same issue trying to profile inside a windows container. Originally trying to use:
PS C:\Program Files\Microsoft Visual Studio 17.0\Team Tools\DiagnosticsHub\Collector> .\VSDiagnostics.exe start 1 /attach:6672 /loadConfig:AgentConfigs\CPUUsageLow.json
However, which also hits permission issues. Profiling inside a windows docker container appears to be impossible?


Account for headless servers

Adds xvfb for headless server runs to both installation, and run scripts

I assume the script will still run the same with xvfb on GUIs?


Stop ignoring namespaces

Plumb the namespace through to output. Spec:

Each feature name still needs to be globally unique, i.e. this is illegal:

[feature.foo]
  namespace="A"
[feature.foo]
  namespace="B"

The feature name needs to be included in the output formation dictionary (this only affects dense PSV output), e.g.

attributes":[{"name":"A:foo","encoding{"primitive":"double"},"index":1}]

Nothing changes for the actual snapshot output, e.g. we would still have

ID000|foo|0

@markhibberd

💯

this is done. Example:
title = "Dense input example dictionary"

version = 1

chapter = []

namespace = """default"""

tombstone = """NA"""

[feed.person]
  columns = ["misery", "state", "injury"]

[fact.person]
  [fact.person.encoding]
    misery="int"
    state="string"
    injury="(location:string,severity:double)"

[feature.newest_misery]
  namespace  = "miserables"
  expression = "feature person ~> newest misery"

[feature.mean_misery_by_state]
  expression = "feature person ~> group state ~> mean misery"

[feature.mean_misery_by_injury_location]
  namespace  = "warzone"
  expression = "feature person ~> group injury.location ~> mean misery"

output formation dictionary:
{"global_properties":{"missing_value":"NA"},"entity_id":{"encoding":"string","index":0},"attributes":[{"name":"warzone:mean_misery_by_injury_location","encoding":{"listof":{"pairof":[{"primitive":"string"},{"primitive":"double"}]}},"index":1},{"name":"default:mean_misery_by_state","encoding":{"listof":{"pairof":[{"primitive":"string"},{"primitive":"double"}]}},"index":2},{"name":"miserables:newest_misery","encoding":{"primitive":"int"},"index":3}],"version":"1","encoding_version":"1"}

💯 Thanks.


Add autoOpenDisabled property

When datePicker.autoOpenDisabled = true:

Focusing and typing in field doesn’t open the overlay
Clicking on icon opens the overlay
Up / Down keys opens the overlay

If you type the value to the field and blur, it's always marked as invalid:
This should be fixed now

esc and enter buttons behave strangely with the new feature:
If you haven't yet opened the dropdown and type a value without opening the dropdown:

ESC does nothing (should revert the changes to the input as previously)
Enter does nothing (should apply the input's value as previously)

If you have opened and closed the dropdown earlier and type a value without opening the dropdown:

ESC applies the input's value as the date picker's value (should revert the changes to the input as previously)
Enter seems to work as expected

If you input a value to a date picker and focus another one, you enter some kind of an eternal loop on IE11:

Please test the feature on all supported browsers

If I select a value and press esc, the input value is set as the ISO value of the component....and then blurring the field will make it invalid

@yuriy-fix @tomivirkki updated validation logic and added tests. PTAL when you have time.


Automated cherry pick of #8889: bugfix: wait put host offline success

Cherry pick of #8889 on release/3.5.
#8889: bugfix: wait put host offline success

Codecov Report

Merging #8890 (189170a) into release/3.5 (e3fe6c4) will increase coverage by 11.96%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           release/3.5    #8890       +/-   ##
================================================
+ Coverage         6.03%   18.00%   +11.96%
================================================
  Files              795       84      -711
  Lines           124282     8850   -115432
================================================
- Hits              7503     1593     -5910
+ Misses          116114     7088   -109026
+ Partials           665      169      -496

Flag
Coverage Δ

aFlag
18.00% <0.00%> (+11.96%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/misc.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/proxy.go
0.00% <0.00%> (ø)

pkg/keystone/driver/cas/cas.go
11.11% <0.00%> (+0.52%)
:arrow_up:

pkg/keystone/tokens/authmethods.go

pkg/compute/models/storageschedtags.go

pkg/compute/models/capabilities.go

pkg/multicloud/esxi/vdisk.go

pkg/appsrv/response.go

pkg/mcclient/options/dnszone_recordsets.go

... and 599 more

Continue to review full report at Codecov.

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


Cannot read property '0' of null

Hello, when trying to use this library inside meteor I get the following error:
platform.js:734 Uncaught TypeError: Cannot read property '0' of null as reported here: https://github.com/meteor/meteor/issues/7839
Would the solution at the end of the ticket be accepted, or do you guys have another suggestion?
Thanks for your efforts,
regards,
lukas

Hi @sakulstra!
Sure, a PR would be great!

Moved to #111.


Improve haddock coverage of the project

I'm lost from time to time in the source, trying to figure out how the parts belong together.  I guess adding more extensive documentation to the functions would help with this in a big way.

More documentation would be great. However, bear in mind that the codebase is still unstable and large refactorings, such as #18, are coming.

I've been continuously improving Haddock coverage in Hadrian, and now I think we are in a good shape. Let me therefore close this issue, but if there are any specific places that require better documentation, please feel free to reopen, or better raise a new specific issue.


[bfgroup-lyra] Update to 1.6

Describe the pull request

What does your PR fix?
Updates bfgroup-lyra to 1.6

Which triplets are supported/not supported? Have you updated the CI baseline?
As before, Yes

Does your PR follow the maintainer guide?

Yes

Thanks!


[Airbrake] [Staging] No route matches {:action=>"gas_load", :controller=>"gas_asset_lists", :id=>nil, :testing_ground_id=>"495"} missing required keys: [:id]

Airbrake error: #2102
Airbrake project: ETMoses
Error type: ActionController::UrlGenerationError
Error message: No route matches {:action=>"gas_load", :controller=>"gas_asset_lists", :id=>nil, :testing_ground_id=>"495"} missing required keys: [:id]
Where: testing_grounds#gas_load
Occurred at: Apr 20, 2016 14:22:54 UTC
First seen at: Apr 20, 2016 14:22:53 UTC
Occurrences: 2 (2 since last deploy on Apr 20, 2016 14:19:26 UTC)
URL: https://beta-moses.energytransitionmodel.com/testing_grounds/495/gas_load
File: [GEM_ROOT]/gems/actionpack-4.1.8/lib/action_dispatch/routing/route_set.rb
Backtrace:
[GEM_ROOT]/gems/actionpack-4.1.8/lib/action_dispatch/routing/route_set.rb:219:in raise_generation_error
[GEM_ROOT]/gems/actionpack-4.1.8/lib/action_dispatch/routing/route_set.rb:192:in optimized_helper
[GEM_ROOT]/gems/actionpack-4.1.8/lib/action_dispatch/routing/route_set.rb:178:in call
[GEM_ROOT]/gems/actionpack-4.1.8/lib/action_dispatch/routing/route_set.rb:270:in block (2 levels) in define_url_helper

Closed by 4c44942a.


OSX 10.11 Beta (15A244d) 6 - unable to load kext

I am getting a message at boot about the kext not loading then this in console...
"8/4/15 8:44:47.236 AM Karabiner[534]: [INFO] IOService is not found.
8/4/15 8:44:48.621 AM Karabiner[534]: [INFO] BRIDGE_USERCLIENT_SYNCHRONIZED_COMMUNICATION connection is null"
I loaded the kext... without any issues...
sudo /Applications/Karabiner.app/Contents/Library/bin/kextload load <- Worked
Thoughts?  If you need anything to sort this out please let me know.

:+1:  manually loading the kext made it work for me.

:+1: manually loading the kext made it work for me.

FYI,
kextload /Applications/Karabiner.app/Contents/Library/Karabiner.signed.kext

Thanks for the manual fix. Same issue here, also worked around by manually loading the kext.

Thanks guys! It's amazing how much you release you use something when it's gone. Amazing utility!

I've fixed this issue at v10.6.92. (The latest beta version)
How to upgrade to the latest beta version:
https://pqrs.org/osx/karabiner/document.html#check-for-updates


Make Django dependencies optional

This branch splits all Django and database dependencies into an optional extra and allows tests to be run on code that doesn't require them without needing to have them installed.

:+1: from me because I think it makes it easier to only run the non-django tests easier.
Do we want to update the matrix to run the django tests and the non-django tests separately? Would that speed things up as they potentially can then happen concurrently?
The longer I've been disconnected from this code base the more I'm starting to think that the maintenance burden of two separate ways of connecting to Redis would have been a cheaper trade off than the kind of testing setup we're finding ourselves in now.
@JayH5 what are your thoughts on that? Considering you're freshest to this code :smile:

Do we want to update the matrix to run the django tests and the non-django tests separately? Would that speed things up as they potentially can then happen concurrently?

About half our test run time is JavaScript stuff and installing dependencies, so splitting the build like that would cost us time rather than saving it. Also, we still need build jobs with complete test coverage so we can report that accurately.

The longer I've been disconnected from this code base the more I'm starting to think that the maintenance burden of two separate ways of connecting to Redis would have been a cheaper trade off than the kind of testing setup we're finding ourselves in now.

This covers both Riak and Redis, and duplicating everything that uses those would be significantly worse than what we have now -- for starters, everything that operates on conversation, contact, or message objects would have to be duplicated.
A better way forward would be to move all the meaningful operations on those things into async-land services and replace the sync stuff with thin wrappers around calls to that. We've been making excruciatingly slow progress in that direction, but the new message store is an example of how we can do it.

cool, thanks, I'm :+1: anyway.

Out of interest, how did the Docker work lead to this work? Do the two relate? Did Docker need this?

Out of interest, how did the Docker work lead to this work? Do the two relate? Did Docker need this?

If our docker container is only running async processes, it has no need for the Django dependencies or access to Django's database, etc. If we can leave these out, we can have smaller container images that are faster to build and deploy.
Furthermore, psycopg2 can only be used with cpython and there are some unresolved difficulties with psycopg2cffi (it randomly segfaults when running tests with pypy) which currently make it impossible to use pypy if that's a hard requirement for vumi-go. We already have everything we need to build pypy images, except we can't install vumi-go in them.
Unrelated to the docker stuff, Django is full of magic initialisation that happens at import time and our Django apps perform blocking I/O all over the place. In order to guarantee that we're not accidentally (or deliberately, as I discovered in a couple of places) pulling this into code that needs to be async and non-blocking, we need a way to run tests without Django installed.

:+1:
650dfb2 is pretty cool :stuck_out_tongue:
@smn to be honest, I haven't really seen/done that much with Django so far so the implications of this code are somewhat lost on me. I learnt some interesting things about Python, though...


Team B08 Evaluation added

Please use the template committed by Shamik

updated the format.

pull the latest main and re commit


add github actions CI

Add github actions CI, when there is a push or pull_request event, the actions CI will be triggered:

Hi Andrey @asomov , actions CI will be triggered by PRs, so that anyone could konw if all the tests are passed, please consider it, thanks!


Uri() allows invalid path element

main() {
  var uri = new Uri(path: "//");
  print(uri.toString()); // prints ////
}

According to RFC 3986 (https://www.ietf.org/rfc/rfc3986.txt)

If a URI   does not contain an authority component, then the path cannot begin  with two slash characters ("//")

May be it makes sense to throw an exception in constructor in this case?

The URI Reference // is parsed as an empty authority and no path. If you check RFC 3986, it says:

relative-ref  = relative-part [ "?" query ] [ "#" fragment ]

relative-part = "//" authority path-abempty
              / path-absolute
              / path-noscheme
              / path-empty

so we match the "//" authority path-abempty production because authority can be empty (through host and reg-name) and so can path-abempty.

It seems for me that there is still some issue here.
  var uri = new Uri(path: "", scheme: "", host: "", userInfo: "");
  print(uri.toString()); // prints //

  uri = new Uri(path: "//", scheme: "", host: "", userInfo: "");
  print(uri.toString()); // prints ////
  print(uri.authority); // prints an empty string
  print(uri.path); // prints //

In the second case we have // then an empty authority  and path started with  //

Those cases look correct to me.
The empty userInfo is omitted in the toString (because an empty user-info is considered equivalent to no user-info, and not showing it is a kind of normalization).
An empty scheme is considered equivalent to no scheme (because schemes can't be empty anyway)
That leaves us with an authority (because you specify host, you get an authority, so the URI string will start with a // to initiate the authority).
That is followed by an empty authority and then the // of the path.
That is correct behavior - the URI does contain an authority (an empty one) which is different from not containing an authority, so the RFC rule about URIs without an authority does not apply.
The leading "//" is not part of the authority, it is a delimiter that signifies that there is one. That's why it is in the toString but not in uri.authority (like ? delimits the query, but isn't part of it).

Ok, thank you. But let's go back to the initial example:
  var uri = new Uri(path: "//");
  print(uri.toString()); // prints ////
  print(uri.hasAuthority); // prints false
  print(uri.path); // prints //

Here we have no autority (uri.hasAutority == false). Why URI is ////? I believe that here we have //, then empty autority and then  path // which is wrong

Oooh, yes that is a bug. It should say that it has an authority.
Seems some logic only existed in the toString method, and when I added hasAuthority, that was no longer a safe approach.

Fixed in the above CL.


perf: Simplify NetworkTransform Messaging

NetworkTransform serialized all data in a byte[],
but since compression and options are gone,  we can just
pass the data in Cmd and Rpc parameters

please test both in server auth and in client auth mode :)

Clients can't see each other move

This needs work still... bad jitter.

closing as discussed in discord.
NetworkTransform is too fragile. better make a custom one and add it to an experimental folder.


Edited notes are not updating in service details of Asset Details Page

Describe the Bug

When editing the service detail notes, the updated notes are not reflected in the "Service Details" section immediately. The changes are only visible after the page is reloaded.

To Reproduce

Steps to reproduce the behavior:

Open Assets tab.
Open any asset and edit the already existed notes of service history.
The edited notes are not updated in the service details notes immediately.

Expected Behavior

The updated notes should appear in the "Service Details" section immediately after the changes are saved, without requiring a page reload.

I want to work on this


Exercise 6 level 2

I followed prof's walk through but my level 2 is not printing. I suspect the problem lies in calling describeService but my code is the same as prof's but still unable. Anyone can help?

hey, the issue might actually be in your Main.java file. Mind if you can send ur Main class? I think we can trace the problem from there

hey, the issue might actually be in your Main.java file. Mind if you can send ur Main class? I think we can trace the problem from there

Hello, i havent reach level 3 though so i dont think the problem is in Main?
but here's my main!

If im not wrong i think your describeService should take in a CompletableFuture<List>. dk that solves ur problem tho

hi, services.get(service) returns a CompletableFuture<List<BusStop>> and hence you should not be combining to the resulting string. You should be passing the it to the describeService method instead and combining that result instead.
Just retain the original skeleton as much as possible. You just need to replace result += describeService(service, stops) using thenCombine since result is now a CompletableFuture<String>

hi, services.get(service) returns a CompletableFuture<List<BusStop>> and hence you should not be combining to the resulting string. You should be passing the it to the describeService method instead and combining that result instead.
You will also have to convert describeService to return a CompletableFuture as well
Just retain the original skeleton as much as possible. You just need to replace result += describeService(service, stops) using thenCombine since result is now a CompletableFuture<String>
For level3, you can only use join once so combine all the thenAccept method calls into ONE CompletableFuture<Void> then call join from there.

Hello, for the part on using thenCombine, i think i did that already? or did i do it wrongly?

Wait, im so sorry i think it should be correct, because my implementation is different from yours so I thought it was wrong
I'm not exactly sure where the problem is... would just recommend to do whatever is said above^^

Sorry where is the walkthrough posted exactly?

Wait, im so sorry i think it should be correct, because my implementation is different from yours so I thought it was wrong
I'm not exactly sure where the problem is... would just recommend to do whatever is said above^^

Okay thank you!

Sorry where is the walkthrough posted exactly?
@ArkAzrael
its posted in videos > web lectures > scroll all the way down!


GSE proposal for updating Get school experience page

Trello card
N/A
Context
As part of our work in GSE, we've been working on a new draft for the GSE page. This is an initial tightening up of the language on the page. This makes the GSE service itself a bit more visible.
We think there's probably more ambitious work that we can do with the Website team, but this is a start.
Guidance to review
@Tomcowan1 to review and provide feedback in the first instance (if possible!)

Couple of suggestions from me too @MylesJarvis - mostly re how we refer to events elsewhere
We'll be changing the phrase 'Teaching events' at some point very soon but will come back to that one!

Sorry @MylesJarvis just one more thing. The maths internship page you're linking to might be a bit of a dead end (internships haven't opened for this academic year yet - will most likely open in the NY). We'll update the page once it's all been agreed, but for now you might want to take that bit out and add it back in once the internships are live?

@Tomcowan1 can I get a final review on this once the new review app has deployed?


Old-Mongo connector: SQL2 AVG function seems to not work?

Environment:
Slamdata: 4.2.0 | Mongo: local on my machine -> MongoDB shell version v3.4.2
Issue:
AVG(column-name) in SQL-2 returns nothing.
Anecdote:

SELECT c.name as hotelName, AVG(c.rating) AS avgRating, count(*) AS ratingCount FROM /testDB/myMongo/hotelDB/hotelReviews AS c WHERE c.name like "Hyatt%" GROUP BY c.name
Returns attached image (Note that AVG returns nothing) - sounds like a BUG, since the documentation advertises AVG as functional in SQL-2 Documentation.

Other Explanations:

Don't hold water
Rating does have values in it, since one can see it in Preview (numbers) and Charts (avgRating by hotel) etc.

Dataset:
is this imported into Mongo.
Yes, I should test via REPL as well, but didn't

@zeeiyerWork Is rating a string value or a numeric value?

@jdegoes: numeric, imho.

db.hotelReviews.findOne();
{
"_id" : ObjectId("592ccf745f12b7696652f77a"),
"address" : "Riviera San Nicol 11/a",
"categories" : "Hotels",
"city" : "Mableton",
"country" : "US",
"latitude" : 45.421611,
"longitude" : 12.376187,
"name" : "Hotel Russo Palace",
"postalCode" : 30126,
"province" : "GA",
"reviews" : {
"date" : "2013-09-22T00:00:00Z",
"dateAdded" : "2016-10-24T00:00:25Z",
"doRecommend" : "",
"id" : "",
"rating" : 4,
"text" : "Pleasant 10 min walk along the sea front to the Water Bus. restaurants etc. Hotel was comfortable breakfast was good - quite a variety. Room aircon didn't work very well. Take mosquito repelant!",
"title" : "Good location away from the crouds",
"userCity" : "",
"username" : "Russ (kent)",
"userProvince" : ""
}
}

@zeeiyerWork the field that you want to aggregate on is reviews.rating and not rating as you have in your query. Once you make that change you should get the results you're expecting.

@dugdun, @jdegoes: got it - i just realized that after some more querying.
I guess a warning would have helped that this column does not exist, wrt usability (i suppose we could do more once SST is in play).

db.hotelReviews.find ( {name: /^Hyatt/}, {"name":1, "reviews.rating":1}).pretty()
{
"_id" : ObjectId("592ccf745f12b7696652fd4c"),
"name" : "Hyatt Regency Reston",
"reviews" : {
"rating" : "2"
}
}


Release v3.0.0

TODO

[x] New utils needs tests
[x] Tests broken
[ ] Cleanup
[ ] Can we now say that docs support IE11 as well? (this is different from what the Vue app itself supports, that really depends on how you configure it, only talking about the living docs here)
[ ] What else?

heads up: This release will be dropping support for Webpack 3.x.

Tests fixed and new ones added


Simpler way to validate enums

Rather than creating extra code for checking if status string is a valid TaskStatus enum, you can use Object.values(TaskStatus). It will also make it easier to maintain the code in the future if you add new enums.

Agree, I already did it too.
It always make sense to reference the enum rather than typing it manually. You'll only need to change it once in the task-status.enum.ts

But for the task status validation pipe you need to declare it manually. There may be the case that some status cannot be set by the user itself like 'INITIAL' or anything based on your app


Bug : Required enabled for all Customer Type

Not saving need to ill in all the details
needs to be solved

@iamshubh22 - moved the REGISTER button from the Login Page.
However, it is not visible to after the admin logged in And it is still accessible by the URL to the normal user.

When user is not logged in -
When Admin is loggedIn -


reimplement process using native

[@gavinking] We have done system, operatingSystem, and now runtime. I need a volunteer to do process.
[Migrated from ceylon/ceylon.language#754]

Now that we have scoped imports I resurrected this issue in the hope of finally implementing it fully, but I've encountered another problem I just can't figure out a nice solution for.
So the process toplevel object has a property called arguments which holds the arguments that were passed on the command line at startup. But to make that work in Java we have the main() methods pass their arguments to the process object using an @Ignored method called setupArguments().
The problem is that if we want to rewrite process using Ceylon-only code then we can't do this anymore because there's no such thing as a "hidden shared" method. And even if there were such a thing we have the problem that it gets passed a j.l.String[] which is not a type that the JS backend knows about.
We could just make the main() method pass the arguments to some private class in the language module's private com.redhat.ceylon.compiler.java package and make process get its arguments from that but that would break binary backward compatibility.
So the only crappy solution I can come up with is that the main() method would use introspection to find out on what Ceylon version it's running and either call the current/old/hidden process.setupArguments() or this new someHelperClass.setupArguments().
Are there any other ideas you people can come up with? @gavinking @FroMage @tombentley @lucaswerkmeister  @jvasileff

@quintesse is the issue that the native("jvm") can't define a shared method? Could you make it private and use setAccessible()?

I don't think using reflection to find out if there is a setupArguments() method is bad, but... is the question backwards? A 1.3.1 compiled program has:
    public static void main(.java.lang.String[] args) {
        .ceylon.language.process_.get_().setupArguments(args);
        .simple.run_.run();
    }

As long as we want to support that program, the shipping version of process will need to have setupArguments().

@gavinking we could yes, but unfortunately @jvasileff is right: it would still break BB compatibility. So unless someone can come up with a good idea I'm afraid this still has to be put in the freezer until such a time we break BB compat anyway. :/

Ah, right, sure.

For what it’s worth – I currently use a function to re-set process.arguments in ceylond, so I’d prefer it if that was still possible in some way.

I think @gavinking and I discussed at some point allowing additional non-native shared declarations in native classes. (Meaning you could add shared declarations that only exist in that particular backend implementation. We only allow that for non-shared declarations right now).
Perhaps combining that with an ignore annotation could work? (I never tried to use ignore in Ceylon as we do with @Ignore for Java. It should work, right?)

I think @gavinking and I discussed at some point allowing additional non-native shared member declarations in native classes.

Well the problem here is that I think we actually do want to hide the setupArguments() function from clients written in Ceylon, don't we?

Well the problem here is that I think we actually do want to hide the setupArguments() function from clients written in Ceylon, don't we?

Yeah, that's why I hoped putting an ignore annotation on it would still work, but that might be wishful thinking ;)

OK, finally done, after all this time, and seems to be working fine, though I don't have a good way to test it in a browser.

This change has resulted in the Jenkins build failing for the last 4 days.

This change has resulted in the Jenkins build failing for the last 4 days.

Grrr. Call me crazy, but I actually don't think that ceylon doc should just give up and stop doing work just because it encounters one error while typechecking the code.

@quintesse it should be fixed now, just waiting for the Jenkins build to run overnight.
Thanks.

Apparently you didn't fix the problem hard enough :)


Update the text for extended transaction statuses

Atomic Swap:

The swap is expected to complete in 10 minutes at most
If nobody accepts the order in 10 minutes at most, the offer will be automatically canceled
Swap failed, the money is being released back to your wallet

Regular transactions:

If the receiver doesn't get online in 10 minutes, the transaction will be canceled.
It is taking longer than usual. In case the transaction could not be completed it will be canceled automatically in 10 minutes.

checked


Footer is not sticking to the bottom of the viewport if the content isn't long enough

If the content isn't long enough, the footer isn't sticked to the bottom of the window.

Tried a list of things and none produced viable results:

https://chrisbracco.com/css-sticky-footer-effect/
https://stackoverflow.com/questions/18739937/how-to-keep-footer-at-bottom-of-screen
https://cssnewbie.com/creating-a-footer-that-sticks-to-the-bottom-of-the-viewport/
https://stackoverflow.com/questions/31179944/sticky-footer-till-document-height-greater-than-viewport
https://matthewjamestaylor.com/bottom-footer
https://stackoverflow.com/questions/16164736/css3-height-calc100-not-working

None of them worked. Even though we can find similar code in /layouts/Layout.jsx.
It's not that huge of an issue, but it's annoying af.


Flow: unused suppression comment warnings

I'm getting very easily unused suppression comment warnings when using genType with the existing flow setup
The warnings point to generated files to these lines:
// flowlint-next-line nonstrict-import:off

I'm wondering would it help if there would be a flag in genType config that allows disabling lint comments in output.
Or what are the reference lint settings that genType assumes? Are they available somewhere?
We're using in our .flowconfig
[lints]
untyped-type-import=error
nonstrict-import=error

If I remove nonstrict-import=error line, or set any other value to it, flow actually errors with genTyped code.

@TomiS this looks like an in-fight between the type checker, that generally wants you to add things, and the linter, which in this case seems to want you to remove thing based on insider information.
Not sure. Do you have any suggestions?

To be concrete: disabling lint comments will get the type checker to complain in a lot of cases, so it's a non-option.
Trying to couple tightly with the type checker, as the linter does, also sounds like a recipe for trouble, any time a new version of the type checker changes something.

@cristianoc I checked that if I manually remove the lint comments from the .gen.js files, the warnings disappear. So in my case it would be optimal if I could just add a flag to genType config in bsconfig.js that instructs genType to just skip writing the line:
// flowlint-next-line nonstrict-import:off

The flag could be 'off' by default.

@TomiS for that file, yes. For other files, no. It would mean when you change your code, you might get flow errors, not lint warnings, that you can't fix.

If a solution exists, it needs to apply to all files. Not just the result of compilaition for that specific file.
At the moment I can't think of one.

Unless one could add lint suppression annotations for liner warnings about type checker warning suppression annotations. Tough that is getting paradoxical quickly.

A config stfu=*.bs.js would solve all the issues in one go, if it existed.

@cristianoc Hmm, Sorry if I'm just not "getting it", but these are imports from other gentyped files. So .bs.js are not related, right?. The linting error is on a line like this:
// flowlint-next-line nonstrict-import:off
import type {foo as My_Types_foo} from '../../.././client-re/src/My_Types.gen';

and on pretty much all other similar imports in other .gen.js files. So my thought is that removing all of those lint comments from all of my gentyped files with a global flag would solve the problem. I don't quite understand how removing the lint comment would cause errors if the error only happens in imports between gentyped files that are all strict.

@TomiS suppressing only those cases should work. Thanks for the suggestion!
Is that enough to handle your issues?

@cristianoc Possibly/likely. Naturally need to test more (and I'm happy to test if there is a patch), but it would seem promising enough to at least try, imho.

Checked on a certain large project, and looks fine.

Now in master.
If if does not solve all the issues, feel free to re-open.
Otherwise, lmk if a new release would be useful.

Great. Thanks. A release would of course be handly if it's not too much trouble.

Release 3.19.0 being created.

Confirming the new release solved my problem. Thanks again!


Move FastUuidGenerator to Camel

As the FastUuidGenerator is not camel specific, we can bring it to Camel

https://github.com/apache/camel/pull/3990

requires camel 3.5.0


fix(osv): upgrade net pkg for osv scan

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes https://github.com/envoyproxy/gateway/issues/4951

Release Notes: Yes/No

@Xunzhuo can you make CI happy?

make go.testdata.complete will make CI happy.

fixed by https://github.com/envoyproxy/gateway/pull/4970


BackdropFilter doesn't constrain blurred area

Steps to Reproduce
Here is a build() method that results in a fullscreen blur when a blur area of 125.0 units wide was expected.
@override
Widget build(BuildContext context) {
return new BackdropFilter(
  filter: new ImageFilter.blur(sigmaX: 3.0, sigmaY:3.0),
  child: new Container(
    width: 125.0,
    height: double.infinity,
    color: const Color(0xAA234060),
    child: new Padding(
      padding: const EdgeInsets.only(top: 16.0),
      child: new Column(
        children: [
          new Expanded(
            child: new Icon(
              Icons.refresh,
              color: Colors.white,
              size: 40.0,
            ),
          )]
          ..addAll(_createDayButtons()),
      ),
    ),
  ),
);

}
Also, this behavior was working fine for me a week or so ago.  I'm guessing that I upgraded Flutter since then and that's why it's now broken.
This issue occurs in both the iOS simulator and the Android emulator. I haven't tried on a physical device.
This is a possible dup of
https://github.com/flutter/flutter/issues/16144
Flutter Doctor
[✓] Flutter (Channel dev, v0.2.11, on Mac OS X 10.12.6 16G29, locale en-US)
• Flutter version 0.2.11 at [redacted]/Libs/flutter
• Framework revision f408bb06f9 (5 days ago), 2018-04-10 14:27:53 -0700
• Engine revision 6474647f52
• Dart version 2.0.0-dev.46.0.flutter-ef926f98f5
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.1)
• Android SDK at [redacted]/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-27, build-tools 27.0.1
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)
• All Android licenses accepted.
[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 9.2, Build version 9C40b
• ios-deploy 1.9.2
• CocoaPods version 1.3.1
[✓] Android Studio
• Android Studio at /Applications/Android Studio 3.0 Preview.app/Contents
✗ Flutter plugin not installed; this adds Flutter specific functionality.
✗ Dart plugin not installed; this adds Dart specific functionality.
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
[✓] Android Studio
• Android Studio at /Applications/Android Studio 3.1 Preview.app/Contents
✗ Flutter plugin not installed; this adds Flutter specific functionality.
✗ Dart plugin not installed; this adds Dart specific functionality.
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1012-b01)
[✓] Android Studio (version 2.3)
• Android Studio at /Applications/Android Studio.app/Contents
✗ Flutter plugin not installed; this adds Flutter specific functionality.
✗ Dart plugin not installed; this adds Dart specific functionality.
• Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)
[✓] IntelliJ IDEA Community Edition (version 2017.3.4)
• IntelliJ at /Applications/IntelliJ IDEA CE.app
• Flutter plugin version 22.2.2
• Dart plugin version 173.4548.30
[✓] Connected devices (1 available)
• Android SDK built for x86 • emulator-5554 • android-x86 • Android 7.0 (API 24) (emulator)
• No issues found!

I've tried downgrading Flutter as far as 1.9 and I'm still seeing the issue.  Maybe I'm downgrading Flutter incorrectly?  I'm checking out older versions of the git repo.

I have same issue on dev and master channels, but beta channels is just fine

I just upgraded to Flutter beta 1.3.8 and this started happening. Seems like the Blur effect is applied to the whole parent and any bounds are ignored. Flutter (Channel beta, v1.3.8, on Mac OS X 10.14.3 18D109, locale en-RO)

I switched to stable and the problem is gone, but then found this https://github.com/flutter/flutter/issues/29070 which is most likely the cause. Anyone stumbling into the same problem should try to wrap the BackdropFilter in a ClipRect, as described in the issue.

The problem wasn't there in 1.2.1 but appeared after upgrading to Flutter (Channel stable, v1.5.4-hotfix.2, on Mac OS X 10.14.4 18E226, locale en-UZ)

I solve it by wrap BackdropFilter with ClipRect.
ClipRect(
    child: BackdropFilter(
        ...
    )
)

This is mentioned in the BackdropFilter class document.

I solved it by wrap BackdropFilter with ClipRect.
ClipRect(
    child: BackdropFilter(
        ...
    )
)

This is mentioned in the BackdropFilter class document.

I can confirm, it works, but there is also one problem though. If I do it inside a PageView, it looks good until I try to swipe the page. While it moves the whole page gets blurred

Hi @caseyryan Im experiencing the same issue with the PageView, did you managed to solve it?

Hi @matthew-carroll
Are you still experiencing this issue?   ClipRect does solve the issue when it comes to BackdropFilter

It's been a while since I filed this, so I'm not sure what my goal was :)
It looks like @caseyryan mentioned an issue with clipping, too. Should that be resolved before closing this ticket?

Since the issue isn't reproducible withoClipReact and you don't remember your gaol. I feel safe to close the issue
@caseyryan please file a new issue and provide all the details
If you disagree, please write in the comments and I will reopen it.
Thank you


Add the possibility to set the default disk location where to save SDK temporary files

The Amplitude SDK creates an "amplitude" folder inside the app Documents directory and this is ok.
However, because our app needs to expose the Documents folder in the native iOS "Files" app, we would like to avoid exposing the "amplitude" folder too, but we there seems to be no way to change its location from the SDK.
Could it be possible to add such feature in the SDK config?

Hi @luky88, you can customize your own storage provider to persist events on initialization. Amplitude Swift SDK follows the same architecture in this doc.
// Define YourOwnStorage() which implements Storage
// https://github.com/amplitude/Amplitude-Swift/blob/211d0c05830fab47e74fa9a053615cf422618a02/Sources/Amplitude/Types.swift#L62

Amplitude(
    configuration: Configuration(
        apiKey: "8c0fefcbd0effe2d799eb51be70053f8",
        storageProvider: YourOwnStorage()
        )
    )

We will add an example to the dev doc center soon❤


feat: adding contributing section to the community heading (#405)

Description
Following conversations over on #405, I thought this might be a nice addition to the website.
Somewhere people can click and start to explore how they can contribute to AsyncAPI projects.
At the moment this PR points to the Contributing GitHub Readme file that @alequetzalli has done.
I think the GitHub README file explains pretty nicely how to contribute, so maybe we can keep the context there but just point there from this link.
Anyway, let me know what you think @alequetzalli to the change 🙇
Sceenshot

I choose answer number 2? did I win? 😄

Think so 🤣


issues with --bind

Was unsure how to title this.. but here's the issue I have:
apt-cache search '' | fzf --bind "Ctrl-X:execute: apt-cache show \$(awk '{print $1}' <<< {}) | less"

works as expected. And so does
apt-cache search '' | fzf | awk '{print $1}'

However, combining the two:
apt-cache search '' | fzf --bind "Ctrl-X:execute: apt-cache show \$(awk '{print $1}' <<< {}) | less" | awk '{print $1}'

breaks the output, and prints the first column from apt-cache show rather than the selection from fzf. I'm honestly not sure if this is an issue with the shell or fzf, but is there a fix for this? I can do for example
fzf --multi --bind "ctrl-x:execute:apt-cache show \$(awk '{print \$1}' <<< {}) | less" --bind "return:execute: sudo apt install \$(awk '{print \$1}' <<< {})" < <(apt-cache search '')

but this then breaks reading input to apt install when asked Do you want to continue? [Y/n].

Hmm, this is an interesting problem with execute that I have overlooked.
seq 10 | fzf --bind 'x:execute:echo {}' | awk '{print "got:" $1}'

Pressing xxxxupenter will give:
got:1
got:1
got:1
got:1
got:2

Because the output of each execute invocation is also passed to awk. If you only want the selection from fzf, you'll need to filter the output yourself. I can imagine a few ways to do it, but since you're already using awk, END block will do the trick.
seq 10 | fzf --bind 'x:execute:echo {}' | awk 'END {print "got:" $1}'

Does this answer your question?

Does this answer your question?

To be honest, not really :disappointed:
I have a script that's basically a fancier version of apt-cache search '' | fzf --multi | awk '{print $1}' | xargs -I% sudo apt install % [1] and I would really love to add the --bind..apt-cache show.. functionality to it, but as far as I can tell it seems like I can't..
Unless you can think of a way to have awk distinguish the output from fzf --multi rather than less..
[1] full script is here, if you like. It just tries to work with Arch and the AUR and colorize the package list..

I don't think there's something fzf can do in that case. You'll have to find out a way not to use stdout. I didn't read your script, but maybe you can use stderr instead?
seq 10 | fzf --bind 'x:execute:echo {} 1>&2' | awk '{print "got:" $1}'

@DanielFGray You could do the following :
seq 10 | fzf --bind 'x:execute:echo {} | tee /tmp/file | less' ; lastline=$(tail -n 1 </tmp/file) && echo "$lastline"

The real problem is that you can't both have interactive commands inside the --bind, and redirect the output of fzf (because the interactive commands are then also redirected to the output of fzf).
You could do the following
seq 10 | fzf --bind 'x:execute@echo {} | less@,enter:execute@echo {} > /tmp/file && killall fzf@' ; lastline=$(tail -n 1 </tmp/file) && echo "$lastline"

The killall part is a big hack, I don't know exactly how I could close fzf in that case.
An other option is to use tmux, and to run the interactive commands in an other pane

I think we can use alternate screen here:
seq 10 | fzf --bind 'x:execute:tput smcup; echo {}; sleep 0.1; tput rmcup' | awk '{print "got:" $1}'

(I wasn't familiar with the concept, it was suggested by @D630 in #495)

apt-cache search '' | sort | fzf  --bind "Ctrl-X:execute: tput smcup; apt-cache show \$(awk '{print $1}' <<< {}) | less; tput rmcup"  --multi | awk '{print $1}'
doesn't seem to help much, unless I'm doing that wrong.. the other approach I had was
fzf --multi --bind "ctrl-x:execute:apt-cache show \$(awk '{print \$1}' <<< {}) | less" --bind "return:execute: sudo apt-get install \$(awk '{print \$1}' <<< {})" < <(apt-cache search '')
but then apt-get aborts as soon as it prompts Do you want to continue? [Y/n].. I could force the install with apt-get -y install but that's not ideal at all
I suppose I could fiddle with the --expect switch in fzf, but I was hoping to avoid re-building the list each time.. though I guess at this point it might be the only solution

@DanielFGray Can you also try the following one?
--bind='Ctrl-X:execute[p={} ; less < <(apt-cache show "${p%% *}")]'
--bind='return:execute[p={} ; </dev/tty sudo apt-get install "${p%% *}" >/dev/tty]'

What is this sorcery???

The sorcery is </dev/tty :)
By the way, you can put your bash code in a function atttributed as an export:
__apt ()
case $1 in
c1)
        apt-cache show "${2%% *}" \
        | less -R;
;;
c2)
         </dev/tty \
         sudo apt-get install "${2%% *}" \
         >/dev/tty;
esac

export -f __apt

And then:
apt-cache search '' \
| {
    fzf \
        --bind='ctrl-x:execute(__apt c1 {} )'  \
        --bind='enter:execute(__apt c2 {} )';
}


Portability

Increase the portability by not using some Unix specific utilities and using the libuv provided counterparts.

/cc @ibc

Wow, I think it's great.

Works nice, merged. Thanks a lot!


Uninstallation on Mac OS X

@dnobori How do I uninstall SoftEther on Mac if I downloaded the .pkg file?

where did u download .pkg file?


parent 2d4aaf35735397a1d89f7f5df28f8a0a080d569c

compiled version with unmovable models

@yiyix I think this PR is able to be closed, but let me know if I am mistaken and we can revisit.


[NO QA] Prevent HybridApp crash when action without actionName is copied to clipboard

Details
Prevents this Firebase crash when certain actions are long-pressed and copied to clipboard. It occurs in HybridApp because the action doesn't contain an actionName.
Fatal Exception: com.facebook.react.common.JavascriptException: TypeError: Cannot read property 'actionName' of null, js engine: hermes, stack:
anonymous@1:3390451
isOldDotReportAction@1:3390437
onPress@1:5899905
anonymous@1:9829268
interceptAnonymousUser@1:9827989
onPress@1:9829140
t14@1:7103573
onPressAction@1:4601029

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/439382
Tests

I'm unable to hardcode action/actionname to undefined due to type errors. Somthing causes the action to be null in HybridApp. I can't hardcode it to be null for local testing:

Offline tests

N/A
QA Steps

N/A
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos
N/A

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


[mode/simple addon] Allow capturing with single tokens

Closes #4970. Even if you beat me to it I still like my commit better :P And I have arguments: It also works with token: null and fixes the documentation.

Oh, docs are always nice, but why the extra check? typeof definitely won't return "string" for null or undefined.

Exactly. typeof won't return string, so null or undefined will be considered an array.

Hah, yeah, I should stay away from my laptop after 20:00. Thanks for the fix, merged!


@conda-forge-admin, please add bot automerge

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
Same as title.
Installed packages
NA

Environment info
NA

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/deduplipy-feedstock#4.


Can i send json on websocket

i need to send a json string over websocket to get correct response from server. now i created the json and when i try to send it with  "socket.write(string: json)" and "socket.write(data: json)" it shows JSON has no member data/string. I am using SwiftyJson to create json.

Yes you can send JSON, binary or any encoding you want over a WebSocket. I assume this is an issue with your implementation, but without a code example of your issue, there isn't much help we can provide.

You can use JSONSerialization to serialize an object into binary JSON data.
`func send(_ value: Any, onSuccess: @escaping ()-> Void) {
	guard JSONSerialization.isValidJSONObject(value) else {
		print("[WEBSOCKET] Value is not a valid JSON object.\n \(value)")
		return
	}

	do {
		let data = try JSONSerialization.data(withJSONObject: value, options: [])
		socket.write(data: data) {
			onSuccess()
		}
	} catch let error {
		print("[WEBSOCKET] Error serializing JSON:\n\(error)")
	}
}

`

Is there possible to send "complete" json objects over websocket?
For instance: {"userID"<EMAIL_ADDRESS>"device" : "01"}
My issue is that json objects are sent through websockets with ' ' inside the json object instead of " "   :/
my code snippet:
    func connectUserToMachine(){
        let user = userData.init()
           print(user)

           let jsonEncoder = JSONEncoder()
           let jsonData = try! jsonEncoder.encode(user)
           let json = String(data: jsonData, encoding: .utf8)

           print(json)  //prints: {"userID"<EMAIL_ADDRESS>"device" : "01"}
           self.socket?.write(string: json!)  // sends it like: {'userID'<EMAIL_ADDRESS>'device' : '01'}

    }


Update servicenow.md

The supported version info has not been updated, it currently missing New York (and maybe Orlando too(?))

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

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

@azooinmyluggage Could you review this pull request? If you don't want to list the versions that are not supported, maybe we could change the text string to "The integration is compatible with Kingston and later versions of ServiceNow" for clarity. Please check. Thanks.

#sign-off
I see support for New York mentioned on the marketplace documentation. There are two more updates since this PR. I'll follow-up to see if those are also valid.

@juliakm: I'm sorry - only the author of this article, @azooinmyluggage, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Missing permissions to Google Cloud Storage object after enabling uniform_bucket_level_access

File: themes/default/content/docs/get-started/gcp/deploy-changes.md
After performing the change, altering of index.html properties failed with error
  gcp:storage:BucketObject (index.html):
    error: 1 error occurred:
    	* Error when reading or editing Storage Bucket Object "index.html-b9e45f3": googleapi: Error 403<EMAIL_ADDRESS>does not have storage.objects.get access to the Google Cloud Storage object., forbidden

Looks like it's correlated with enabling uniform_bucket_level_access on the bucket as docs on https://cloud.google.com/storage/docs/uniform-bucket-level-access say

Caution: If you enable uniform bucket-level access, you revoke access from users who gain their access solely through object ACLs and prevent users from administrating permissions using bucket ACLs. Be sure that you read considerations when migrating an existing bucket prior to enabling uniform bucket-level access.

Looks like the script should be updated to cover for it (add permissions for account owner through IAM binding)

I wasn't able to reproduce this. In many cases, it's a replace (due to the way the fields are marked). Even with an update, I was able to update the object. Could you please describe more about your auth setup?


Support envfile task level and global option

Adds support for a new envfile option at global or task level. #22
Includes an original lightweight parser for envfiles closely following bash syntax.
Still to do:

feature tests
document the feature

Thanks for taking a look.

Nice. This is tested to work, right?
EDIT: Also, why did you "UI" so much?

I tried it with a few simple scenarios, I intend it test it more and also add a few scenarios to the feature test suite before merging and making a pre-release.
The "UI" line changes are all about being able to print a warning at context.py:90 when the UI is in verbose mode.

The current envfile parsing is very fault tolerant, which is good, but doesn't emit warnings for syntax errors.
With this envfile:
TEST=yeetus  # Epic

 e = A
 A = b
a = b
a = c

Only TEST is set. It doesn't send an error! (Removing spaces around the = will make it set all of these). Poe doesn't warn on duplicate keys either.

The current envfile parsing is very fault tolerant, which is good, but doesn't emit warnings for syntax errors.

Good catch, regexes are tricky beasts. I added some tests with bad input to give more confidence.

Good, it's erroring now. Except the error message is very cryptic... You might wanna use something like Lark which has very epic error messages. (Lark can generate a standalone so you don't need to worry about an extra dependency)

Good, it's erroring now. Except the error message is very cryptic... You might wanna use something like Lark which has very epic error message support. (Lark can generate a standalone so you don't need to worry about an extra dependency)

Good point. I tweaked the error handling so it's a bit more specific and helpful I think.
I think it's good to go now, but I'll leave this open a short while longer in case you'd like to give more feedback before I merge and create another pre-release.

Looks good. It would be nice to have suggestions but at least the errors are correct.


Issue with JWT bearer token grant in case the initial JWT was issued by the UAA

What version of UAA are you running?
75.0.0
How are you deploying the UAA?
I am deploying the UAA using cf-deployment
What did you do?

cf login
cf oauth-token
POST to /oauth/token with JWT bearer token grant

What did you expect to see? What goal are you trying to achieve with the UAA?
I expect to get a token of the user from which I initially retrieved the oauth token via "cf oauth-token".
What did you see instead?
A new shadow user was created based on the "sub" claim of the JWT retrieved via "cf oauth-token". And the retrieved JWT was issued for that newly created shadow user.
Details
During JWT Bearer with an UAA-issued token, the UAA constructs an internal Identity Provider config that is used in the authentication of JWT Bearer. This internal config does not define any attributeMappings.
The issue seems to appear after the merge of PR#1330 which caused the user to be updated when the externalID changes, including the username. And as the attribute mapping is empty, the default is used, which is the "sub" claim of the token. But it should be "username" to "username".
We will provide a PR that fixes the issue.

Thanks @torsten-sap, we started looking at this. May I ask you to clarify your description of the reproduction steps? Since these steps involve multiple tokens, it's not clear which tokens you are referring to. Let me rephrase the description & please correct me if that's not what you meant:

Hi @peterhaochen47,
In general the flow as you describe it looks good.
However this is not restricted to UAA internal users, but also happens for users of external IdPs (in our use case we always have such an external IdP user - I have not tested UAA internal users explicitly, but as there is no mechanism to identify those at that part of the code, it probably holds also for them).
The important part is, that the access-token-foo that is used for the JWT Bearer was issued by the UAA itself.

Thanks @tack-sap, I have corrected the description in my comment. Now I understand the issue, I will add comments + questions in the PR #1530 now.

Closing as PR was merged.


ccmlib/scylla_node.py: nodetool(): prepend api-port instead of append

Nodetool commands can contain a -- sequence, after which all subsequent command line parameters are considered positional. This means that the option we are trying to add, -Dcom.scylladb.apiPort=10000, will be parsed as a positional args and will likely trigger failure in parsing the command-line arguments.
Instead of appending it to the command string, prepend it. At this point, the cmd starts with the nodetool command and node.py::nodetool() will prepend host and port args. Do the same and prepend the rest api port.
Tested locally with a test which fails with the previous code.

Follow-up to https://github.com/scylladb/scylla-ccm/pull/560.

Example failure with the previous code: https://jenkins.scylladb.com/job/scylla-master/job/scylla-ci/6803/testReport/junit/alternator_tests/TesterAlternator/Tests___dtest___test_create_snapshot_and_refresh/
ccmlib.node.ToolError: Subprocess /jenkins/workspace/scylla-master/gating-dtest-release/scylla/.ccm/scylla-repository/17168/share/cassandra/bin/nodetool -h <IP_ADDRESS> -p 7199 -Dcom.sun.jndi.rmiURLParsing=legacy refresh -- alternator_user_table user_table -Dcom.scylladb.apiPort=10000 exited with non-zero status; exit status: 2;
stdout: error: too many positional options have been specified on the command line

Try --help.


scala3 tracing failing to detect name of method

Mostly everything is fine in scala3 but I'm struggling to make trace for me. instead of printing the function name where the error occurred it always prints something like ParserGrammar$$$Lambda$122/0x0000000800d4b920@cad498c):1:3 where ParserGrammar is an object and the parser is a standard def within the object.
It does seem to work on built in parsers okay (eg end-of-input)
Possibly the problem is related to CharPred and the function (Char => Boolean) I am passing it always being traced as an unhelpful anonymous lambda. It may not even be a scala 3 thing.

@caoilte please provide a code sample that reproduces the issue


undefined symbol: NxClient_GetDefaultJoinSettings

libnxclient was not linked, below changes are needed for this "undefined symbol" failure.

Duplicate of this PR https://github.com/rdkcentral/rdkservices/commit/075170fbe397dba24828bd1ad252f45f9fec2ceb


7.2_Importance_of_License

Description
Importance of software licensing and reasons to use software licensing
Domain
Open Source
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

/assign

@Aryamanz29 pls assign me this issue i have to make  a PR for other files as well.


subprocess OverflowError: long too big to convert

Hi, I'm trying to use the find_coefficients method from xfoil_module. But I faced this error :
<type 'float'>

Traceback (most recent call last):
  File "C:/Users/PINTO ASU PETHUK/Desktop/TugasAkhir/pinto.py", line 2, in <module>
    find_pressure_coefficients(airfoil='naca0012', Reynolds = 1e6, alpha=12.,NACA=True)
  File "C:/Users/PINTO ASU PETHUK/Desktop/TugasAkhir\xfoil_module.py", line 853, in find_pressure_coefficients
    NACA=NACA, PANE=PANE)
  File "C:/Users/PINTO ASU PETHUK/Desktop/TugasAkhir\xfoil_module.py", line 207, in call
    startupinfo = startupinfo)
  File "C:\Python27\lib\subprocess.py", line 386, in __init__
    errread, errwrite), to_close = self._get_handles(stdin, stdout, stderr)
  File "C:\Python27\lib\subprocess.py", line 536, in _get_handles
    c2pwrite = self._make_inheritable(c2pwrite)
  File "C:\Python27\lib\subprocess.py", line 570, in _make_inheritable
    _subprocess.DUPLICATE_SAME_ACCESS)
OverflowError: long too big to convert

Here's my source code :
from xfoil_module import find_coefficients
find_coefficients(airfoil='naca0012',alpha=12.)

I need it to work for my thesis :(
Thanks!

Hello,
I do not have this error on my computer. Could you provide more information regarding your install, OS, and where you are running this? Just from the Error message, I am not able to figure out what the problem is. I am using Python 3.7 on Windows OS.
Thanks,
Pedro

Hi,
I'm sorry, but i think that you use Python 2.7 in your programs. And now I have new problem with your print syntax without using "()" after print.
Thanks!
Pinto


Sawfish: 1.11.90 -> 1.12.0

Motivation for this change
Sawfish Upates:
[X] librep: 0.92.5 -> 0.92.6
[X] rep-gtk: <IP_ADDRESS> -> <IP_ADDRESS>
[X] Sawfish: 1.11.90 -> 1.12.0
Things done

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

[X ] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X ] Tested execution of all binary files (usually in ./result/bin/)
[X ] Fits CONTRIBUTING.md.

Thanks!


Extend interfaces that contain RestAction close() from AutoCloseable

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the [latest JDA version][download].
[x] I have checked the branches or the maintainers' PRs for upcoming features.

Feature Request
Now close() method in interface such as PrivateChannel return RestAction. What about rename method to RestAction<Void> closeNative() and add default realization for AutoCloseable.close()
For example:
RestAction<Void> closeNative();

default void close(){
    closeNative().queue();
}

You don't need to close private channels, and promoting this through an AutoClosable implementation would be counter-productive since it would imply incorrect usage of the interface.


Side nav getting closed when clicked outside.

Description
I am using side-nav but when clicked on overlay it gets closed. I want to restrict it , something similar is provided in overlay for modal with property dismissible=true

You could overlay the overlay so it doesn't send the click event, even thought that's quite hacky...

@deprilula28  yes I am using the same as an alternate, but I am looking for a prominent solution

http://materializecss.com/side-nav.html#options

@Supremex That's not what he meant I'm pretty sure... That closes the sidenav when you click an anchor button, he wants to stop closing when you click the overlay

As this framework is geared towards providing good user experience through material design. I don't think this is a necessary addition as it goes against the metal model users have built up using other material side navs


Restrict to TF 0.13 or above

so https://www.hashicorp.com/blog/custom-variable-validation-in-terraform-0-13 can be used

Fixed by https://github.com/terraform-google-modules/terraform-google-group/pull/17


test/test Add function stubs for native and applicable (ignored) tests.

A significant refactor of the code was merged recently that implemented the cuda
back end. Native and OpenCL were left behind and commented out. Further,
building with cargo test --no-default-features --feature native failed because
cudann was still being pulled in by the tests. This commit:

fixes the cfg guards so dropping cuda support is possible for some people.

Adds the function stubs for the Native backend. These are left w/
notimplemented!();

The same tests for the cuda backend are copy/pasted and refactored to apply
to the Native backend.

This puts us in a good position for someone to just implement the Native backend
functions now.
NB: There is still a failing test for people building without Cuda: the lib.rs doc test. However, adding a
cfg guard to it uglifies it too much for it to remain a useful example, so I left it alone.
r? @MichaelHirn

Big thanks @ehiggs.
I made two tiny changes and uploaded them into PR #5

I added three flags in the native file, which suppress the long list of warnings, that come up at compilation.

I made the lib.rs doc tests work for native. I think the trick is, that you can use #  in front of a line of code, which will then not show in the final docs. Now the docs look just like before, but includes feature support.


Feature proposal: Allow keyword "never" as schedule

Hi,
how about allowing keyword "never" as schedule which never is called. this would allow flexible configurations.
I am using Quarkus with stage dependent configurations. for e.g. %test i would like to turn off some schedules explicitly. currently I am using "0 0 0 1 1 ? 2099" which in fact means "never" but is hard to see briefly in config file. having a key word here would make configuration more explicit.
One could think of other keywords, too.
Could be this feature is already there and I did not find it. Sorry also if the community already discussed this and denied it. Just wanted to share this idea.
Kind regards and thanks for your efforts.

Umm the "never" is already implemented as you say "0 0 0 1 1 ? 2099". In my opinion seems quite clear that it will never take place using the already implemented  cron expressions


Make Load Combinations available in ISM's Analysis Results

Changes needed for ISM API to make Load Combinations available in Analysis & Design Results (in addition to Load Cases).
Structural Analysis Schema changes: Added a common MixIn for Load Combinations and Load Cases,
Integrated Structural Model Schema changes: Deprecated old Analysis & Design Results relationships that reference Load Cases. Instead, added new relationship counterparts that reference the newly added MixIn.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending?  Please create a branch (not fork) from master and then raise a PR to merge the changes.


fire autocompletion by matching the actual key content typed in

Discussed in #353. There is a bug in GraphiQL with non-QWERTY keyboards that, because the key mapping is different from the English keyboard, the autocompletion pop-up is shown for incorrect set of characters (in QWERTZ keyboard for instance, Alt-8 types { character, which the code recognizes within the whitelisted boundary).
Fixed by matching the exact key character.

cc @ejoebstl - I think this is closer to what we want the behavior to be. Will merge soon!

Exciting, thank you!


[Regression] TextBox text selection is not set to End while SelectAll and click on white space after text

Describe the bug
TextBox text selection is not set to End while SelectAll and click on white space after text
https://github.com/AvaloniaUI/Avalonia/assets/4997065/10a557e4-f785-4d57-8a6d-644c248e2f57
On Any OS.

Actually it doesn't deselect at all even if you click on the selected area.


New function and some made External

New function that will return the mime mapping for a file name.  Path Helper functions made External.

Something is not quite right. I cannot see, which changes you are proposing. Under "Files Changed", it appears everything has been added, which isn't the case. COD419 is in the repo and there should hence be a limited number of changes. Could you take a look once more? Also, your changes include translations, which should be removed.
Once done, I'll take another look! Thanks!

Well, the COD419.TXT is not in your master branch.  What will happen if you put it there?  Will you then see the changes I suggest?

It's not? I added it earlier today! It should be in master :-)

It is in now.  Decline this pull request, I will create a new one.

I will ask you to add objects to master if I need them before the next pull request.


vendoring django completion

Description

Some vendored libraries like the bash completion of django resides in the django source-code.
To not have to include the entire django project in bash-it just to get django completion we have to rewrite the commits git vendor creates.

But git vendor update django-completion wont work. You would have to go through the steps above again to clean out unwanted files.
Motivation and Context

#1818 and potentially #1819
However as mentioned earlier this is not really a method I would recommend.
Some more discussion of this problem is needed.
How Has This Been Tested?
Enabled the django completion and checked with the complete command.
Ran local tests
Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] If I have added a new file, I also added it to clean_files.txt and formatted it using lint_clean_files.sh.

I kind if want to make my own version of git vendor that doesn't merge a subtree but just creates a commit on the branch you can easily amend and do rebasing on.
I like that git vendor doesn't have a config file but I really don't like how troublesome it is to work with regarding cleaning the commit and rebasing.

Idle hands...
I have made a fork of git vendor https://github.com/buhl/git-vendor
It now puts a nice little squashed commit on the top of the branch instead of the horrible dangling merge commits.
You can test it like so.
./repos/ $ mkdir test && cd test && git init .
./repos/test $ git vendor add apm https://github.com/vigo/apm-bash-completion 1.0.0 --debug
./repos/test $ git log
./repos/test $ git vendor list
./repos/test $ git vendor remove apm
./repos/test $ git log
./repos/test $ git vendor list
./repos/test $ git vendor add apm https://github.com/vigo/apm-bash-completion 1.0.0 --debug
./repos/test $ git log
./repos/test $ git vendor list
./repos/test $ git vendor update apm --debug
./repos/test $ git log
./repos/test $ git vendor list

There might be some edge cases especially when updating. In worst case git will throw you into a cherry-picking mode you can just abort.
I create a temporary branch where the subtree will be added and discarded from.
Using this method would greatly aid working with PR's on vendored libs.
I am aware of the irony of being the one to suggest git vendor in the first place to now talking about using something else :smiley:

Thanks @NoahGorny
Before you merge this I have to rebase the branch otherwise it won't merge. Which means I have to do alle the steps again 🤐
Which is also why I have started a fork of git-vendor 😇

Thanks @NoahGorny
Before you merge this I have to rebase the branch otherwise it won't merge. Which means I have to do alle the steps again
Which is also why I have started a fork of git-vendor

It seems mergeable to me, but okay!
I also think you should contact @Tyrben about merging your fork to his repo, as I kinda think its better and more sustainable than another fork

I think I tried a merge locally when I started using vendor and it failed because git vendors squashed commit doesn't share the same root  commit or something. You are welcome to give it a try and merge it :smile:
I think I will try to suggest my changes to @Tyrben but they are not ready yet. There are some issues (I think) when updating, files that where removed from the vendored repo will not be removed with my current implementation.
Let me know if you want to try a merge or I should do the rebase

nice @NoahGorny

@buhl, is your fork of git-vendor OK to use?
I'm asking b/c the patch has something about lender that I don't follow. (No need to explain so long as it's OK to just use in place of the regular one.)

the fork should be okay to use, but seems like @buhl can even try to merge it upstream- the owner of the original repo (@brettlangdon) seems to be active again!

Oh man, it has been a long time since i looked at git-vendor. I will look into if they want to accept my changes upstream :smiley_cat:


"SyntaxError: can use starred expression only as assignment target" on Debian Jessie

Hi, this is my system
wget --version && python3 --version && google-chrome --version
GNU Wget 1.16 built on linux-gnu. [...]
Python 3.4.2
Google Chrome 59.0.3071.36 beta

uname -a Linux Debian-87-jessie-64-minimal 4.9.0-0.bpo.2-amd64 #1 SMP Debian 4.9.18-1~bpo8+1 (2017-04-10) x86_64 GNU/Linux
and this is the error I get. Its probably related to the python version, however I have no idea really.
python3 ./archive.py example_ril_export.html pocket
  File "./archive.py", line 91
    *'wget --no-clobber --page-requisites --adjust-extension --convert-links --no-parent'.split(' '),
               ^
SyntaxError: can use starred expression only as assignment target

Today (8th of May) its working fine with your instructions on my Mac machine.

Installing Python 3.6 and using that version instead fixes the issue.


.err file appears after a task completes

Right now when a task is completed, it absorbs .err file into the .task file so that the error messages can be embedded (and be retrieved from sos status). However, on the cluster system, the .err files are written some time after the jobs are completed, so the task file can contain partial .err file.

This make is slower in retrieving status of task files because sos will have to check if there are new .err and .out file after tasks are completed.


[DOC] Add a new setting index.check_pending_flush.enabled to index settings OS doc page

What do you want to do?
Add a new setting index.check_pending_flush.enabled to  index settings OS doc page

[x] Request a change to existing documentation
[ ] ~Add new documentation~
[ ] ~Report a technical problem with the documentation~
[ ] ~Other~

Tell us about your request. Provide a summary of the request and all versions that are affected.
index.check_pending_flush.enabled (Boolean): This setting controls Apache Lucene's Index writer setting checkPendingFlushOnUpdate which controls if an indexing thread should check for pending flushes on update in order to help out on a full flush
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
PR: https://github.com/opensearch-project/OpenSearch/pull/12710/
POC Issue: https://github.com/opensearch-project/OpenSearch/issues/11628

THanks, @CaptainDredge! Will this be released in 2.13?

@hdhalter not sure yet, if the PR gets merged today then yes it will get released in 2.13


fix: removed dependency while using domain name instead of hosted zone

This PR is included in version 1.39.1 :tada:


Language Translator using GUI

Script Title - Language Translator
Brief - A GUI based language translator.
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed

Programming Language

[x] Python

Happy Coding

Please assign this issue to me.

Add the GUI to current version #39


Promise.map(promises, callback, { concurrency: 1 }) does not call callback with promises in same order they were passed.

When using this function and an array of 3 promises, you might expect it to call the map function on each promise in order. Instead the mapper is called on the first item, then the last item then the second item (the general pattern seems to be [0, (n-1), (n -2), ..., 1].
In some cases it would be nice if you could rely on the order being the same, e.g. if the queue of promises represents work to do in order, and the concurrency limit can then be used to limit CPU load.

Right, if you want to do it in order call .each instead of .map.

Thanks!

Also aliased as .mapSeries in 3.0 btw.

Ah .each doesn't use the return value which I need, Promise.map with concurrency of 1 does exactly what I need but processes the values out of order. Guess I can do what I need with Promise.reduce.

@ohjames  for the record - in 3.0 .each returns the results, you can do .each in 2.9 and accumulate the results to a closure variable as a workaround (or .reduce that also works).


AutoModelForZeroShotObjectDetection does not work

Hello,
First of all, I would like to thank you for your excellent research.
I am currently testing the "grounded_sam2_tracking_demo_custom_video_input_gd1.0_hf_model.py"
And it seems that the issue arises from the line
grounding_model = AutoModelForZeroShotObjectDetection.from_pretrained(model_id).to(device)
as I am unable to load the model from Hugging Face.
Could you please let me know if the model is currently unavailable?
Additionally, is there a way to set up the grounding_model in the same manner as in the local demo within this Python file?
Thank you for your assistance!

Hello @stupidyoh , the model is available in huggingface now, would you like to check if it is network issue in your local env or not. If there're some network issue, you can use another mirror source:
export HF_ENDPOINT=https://hf-mirror.com

Additionally, is there a way to set up the grounding_model in the same manner as in the local demo within this Python file?

Yes it is the same, and we will try to provide this code this week as soon as possible. Stay tuned, you can also try to implement it yourself referring to this code: https://github.com/IDEA-Research/Grounded-SAM-2/blob/main/grounded_sam2_local_demo.py

You can try this demo:
python grounded_sam2_tracking_demo_custom_video_input_gd1.0_local_model.py

I was modifying the local_demo because I couldn't resolve the network issue.
Thank you for uploading the new code.
I have confirmed that there are no issues with the running the code.

Hi @stupidyoh , you're welcome, you can try to implement your idea based on our code, if there is any other problem you can raise another issue.

We've implemented local model for custom video tracking in our repo, you can try this demo as follows:
python grounded_sam2_tracking_demo_custom_video_input_gd1.0_local_model.py

Could you also implement local model for grounded_sam2_tracking_demo_with_continuous_id.py?


Change the way of using env var

The PR is to fix the issue of missing comments:

Expect the template pipeline information:

The following pipelines have been queued for testing:python - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:python - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Upgrade to 3.11 Broke Mictrack 510/600 Devices (tlt2h port 5030)

Hello I just upgraded to 3.11 and the devices on port 5030 (tlt2h port) the Mictrack 510's and Mictrack600's all stopped updating.  All my OSM and TAIP devices are updating correctly.   Soon as I downgraded to 3.10 Mictrack devices worked just fine.
Watching the log file shows the device sending the same data as before. But on version 3.11 it ether doesn't like it or thinks it's invalid as I get no INFO update.. Just the debug line.
I have checked the HEX debug between the versions and there is no difference in the debug info expect the timestamp of a few minutes it takes to change versions..  But 3.10 devices update fine 3.11 don't..

 Here is a build to try:
https://www.dropbox.com/s/cgmlansv32j3ii0/tracker-server.jar?dl=0