Can you configure authentication (-auth-file) with an environment variable?

https://github.com/mailhog/MailHog/blob/master/docs/Auth.md says you can pass " -auth-file=docs/example-auth" with a password file.
https://github.com/mailhog/MailHog/blob/master/docs/CONFIG.md talks about configuration with  command line options or environment variables but makes no mention of -auth-file
Can you configure authentication (-auth-file) with an environment variable?
Thanks,
James

Yes you can, its defined here https://github.com/mailhog/MailHog/blob/d6585029d60980c67bd988a18532f2b91d10f385/config/config.go#L34

IMHO it would be even better if you could configure basic auth directly via env variables, so you don't need to provide a password file. E.g. MH_AUTH_USERNAME and MH_AUTH_PASSWORD :)


Add the last missing range operators

This commit adds the last missing range operators.
Additionally it fixes the #[auto_type] support for the existing range functions. It also goes over the documentation again and fixes minor inconsistencies there. Finally a change log entry is added.
Fixes #4092

Reviews are always welcome even if you are not part of the review team. If you want it's also possible to join that team. In the end it's just a group of people that get pinged for such reviews.
ASCII art refers to this little diagrams added to all operators:
https://github.com/diesel-rs/diesel/pull/4151/files#diff-bf88b4900340a435caafd7c947e7b9a95e61ff158ef37596ab3262b3c754c45fR82-R84


When will parent.Xrm start NOT working on free standing Web Resources?

Hi, this page says:
parent.Xrm. will work if the HTML web resource is loaded in a form container. However, for other places, such as loading an HTML web resource as part of the SiteMap, parent.Xrm. also won’t work.
When exactly won't parent.Xrm work in free standing web resources?
Thanks

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

ID: 31f2e23f-87e2-62be-f107-08c2e99b97e5
Version Independent ID: 03aa8ec3-a827-95cf-d7cf-f0a014c3484b
Content: GetGlobalContext function and ClientGlobalContext.js.aspx in model-driven apps - Power Apps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/reference/GetGlobalContext-ClientGlobalContext.js.aspx.md
Service: powerapps
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

@potsiemj Thanks for contacting us. Please contact support or ideas forum.  To contact Support, visit https://powerapps.microsoft.com/support/ and for ideas forum, visit https://ideas.powerapps.com. Closing this issue.


[WATCH] Asteroid City FULLMOVIE (Free) Online streaming 4k

Universal Pictures! Here’s options for downloading or watching Asteroid City streaming the full movie online for free on 123movies & Reddit including where to watch Universal Pictures’ movie at home. Is Asteroid City 2023 available to stream? Is watching Asteroid City on Disney Plus, HBO Max, Netflix or Amazon Prime? Yes we have found an authentic streaming option / service. Details on how you can watch Asteroid City 3 for free throughout the year are described below.
➤►🌍📺📱👉Watch 🔴✅👉 Asteroid City Movie HD
➤►🌍📺📱👉Download 🔴✅👉 Asteroid City Movie HD
Watch Asteroid City (2023) Fullmovie (Online Free)
Now Is Asteroid City available to stream? Is watching Asteroid City on Disney Plus, HBO Max, Netflix, or Amazon Prime? Yes, we have found an authentic streaming option/service. A 1950s housewife living with her husband in a utopian experimental community begins to worry that his glamorous company could be hiding disturbing secrets.
Showcase Cinema Warwick you'll want to make sure you're one of the first people to see it! So mark your calendars and get ready for a Asteroid City movie experience like never before. of our other Marvel movies available to watch online. We're sure you'll find something to your liking. Thanks for reading, and we'll see you soon! Asteroid City is available on our website for free streaming. Details on how you can watch Asteroid City for free throughout the year are described
If you're a fan of the comics, you won't want to miss this one! The storyline follows Asteroid City as he tries to find his way home after being stranded on an alien Asteroid Cityt. Asteroid City is definitely a Asteroid City movie you don't want to miss with stunning visuals and an action-packed plot! Plus, Asteroid City online streaming is available on our website. Asteroid City online is free, which includes streaming options such as 123movies, Reddit, or TV shows from HBO Max or Netflix!
Asteroid City Release in the US
Asteroid City hits theaters on January 14, 2022. Tickets to see the film at your local movie theater are available online here. The film is being released in a wide release so you can watch it in person.
How to Watch Asteroid City for Free?release on a platform that offers a free trial. Our readers to always pay for the content they wish to consume online and refrain from using illegal means.
Where to Watch Asteroid City?
There are currently no platforms that have the rights to Watch Asteroid City Movie Online.MAPPA has decided to Asteroid City the movie only in theaters because it has been a huge success.The studio , on the other hand, does not wish to divert revenue Streaming the movie would only slash the profits, not increase them.
As a result, no streaming services are authorized to offer Asteroid City Movie for free. The film would, however, very definitely be acquired by services like Funimation , Netflix, and Crunchyroll. As a last consideration, which of these outlets will likely distribute the film worldwide?
Is Asteroid City on Netflix?
The streaming giant has a massive catalog of television shows and movies, but it does not include 'Asteroid City.' We recommend our readers watch other dark fantasy films like 'The Witcher: Nightmare of the Wolf.'
Is Asteroid City on Crunchyroll?
Crunchyroll, along with Funimation, has acquired the rights to the film and will be responsible for its distribution in North America.Therefore, we recommend our readers to look for the movie on the streamer in the coming months. subscribers can also watch dark fantasy shows like 'Jujutsu Kaisen.'
Is Asteroid City on Hulu?
No, 'Asteroid City' is unavailable on Hulu. People who have a subscription to the platform can enjoy 'Afro Samurai Resurrection' or 'Ninja Scroll.'
Is Asteroid City on Amazon Prime?
Amazon Prime's current catalog does not include 'Asteroid City.' However, the film may eventually release on the platform as video-on-demand in the coming months.fantasy movies on Amazon Prime's official website. Viewers who are looking for something similar can watch the original show 'Dororo.'
When Will Asteroid City Be on Disney+?
Asteroid City, the latest installment in the Asteroid City franchise, is coming to Disney+ on July 8th! This new movie promises to be just as exciting as the previous ones, with plenty of action and adventure to keep viewers entertained. you're looking forward to watching it, you may be wondering when it will be available for your Disney+ subscription. Here's an answer to that question!
Is Asteroid City on Funimation?
Crunchyroll, its official website may include the movie in its catalog in the near future. Meanwhile, people who wish to watch something similar can stream 'Demon Slayer: Kimetsu no Yaiba – The Movie: Mugen Train.'
Asteroid City Online In The US?
Most Viewed, Most Favorite, Top Rating, Top IMDb movies online. Here we can download and watch 123movies movies offline. 123Movies website is the best alternative to Asteroid City's (2021) free online. We will recommend 123Movies as the best Solarmovie alternative There are a
few ways to watch Asteroid City online in the US You can use a streaming service such as Netflix, Hulu, or Amazon Prime Video. You can also rent or buy the movie on iTunes or Google Play. watch it on-demand or on a streaming app available on your TV or streaming device if you have cable.
What is Asteroid City About?
It features an ensemble cast that includes Florence Pugh, Harry Styles, Wilde, Gemma Chan, KiKi Layne, Nick Kroll, and Chris Pine. In the film, a young wife living in a 2250s company town begins to believe there is a sinister secret being kept from her by the man who runs it.
What is the story of Asteroid City?
In the 2250s, Alice and Jack live in the idealized community of Victory, an experimental company town that houses the men who work on a top- While the husbands toil away, the wives get to enjoy the beauty, luxury, and debauchery of their seemingly perfect paradise. However, when cracks in her idyllic life begin to appear, exposing flashes of something sinister lurking below the surface, Alice can't help but question exactly what she's doing in Victory.
In ancient Kahndaq, Teth Adam bestowed the almighty powers of the gods. After using these powers for vengeance, he was imprisoned, becoming Asteroid City. Nearly 5,000 years have passed, and Asteroid City has gone from man to myth to legend. Now free, his unique form of justice, born out of rage, is challenged by modern-day heroes who form the Justice Society: Hawkman, Dr. Fate, Atom Smasher, and Cyclone.
Production companies : Warner Bros. Pictures.
At San Diego Comic-Con in July, Dwayne “The Rock” Johnson had other people raising eyebrows when he said that his long-awaited superhero debut in Asteroid City would be the beginning of “a new era” for the DC Extended Universe naturally followed: What did he mean? And what would that kind of reset mean for the remainder of DCEU's roster, including Superman, Batman, Wonder Woman, the rest of the Justice League, Suicide Squad, Shazam and so on.As
Asteroid City neared theaters, though, Johnson clarified that statement in a recent sit-down with Yahoo Entertainment (watch above).
“I feel like this is our opportunity now to expand the DC Universe and what we have in Asteroid City, which I think is really cool just as a fan, is we introduce five new superheroes to the world,” Johnson tells us. Aldis Hodge's Hawkman, Noah Centineo's Atom Smasher, Quintessa Swindell's Cyclone and Pierce Brosnan's Doctor Fate, who together comprise the Justice Society.) “One anti-hero.” (That would be DJ's Asteroid City.)
“And what an opportunity. The Justice Society pre-dated the Justice League. So opportunity, expand out the universe, in my mind… all these characters interact. That's why you see in Asteroid City, we acknowledge everyone: Batman , Superman , Wonder Woman, Flash, we acknowledge everybody.There's also some Easter eggs in there, too.So that's what I meant by the resetting. Maybe resetting' wasn't a good term.only
In addition to being Johnson's DC Universe debut, “Asteroid City” is also notable for marking the return of Henry Cavill's Superman. The cameo is likely to set up future showdowns between the two characters, but Hodge was completely unaware of it until he saw the film.
“They kept that all the way under wraps, and I didn't know until maybe a day or two before the premiere,” he recently said Asteroid City (2022) FULLMOVIE ONLINE
Is Asteroid City Available On Hulu?Viewers are saying that they want to view the new TV show Asteroid City on Hulu. Unfortunately, this is not possible since Hulu currently does not offer any of the free episodes of this series streaming at this time. the MTV channel, which you get by subscribing to cable or satellite TV services. You will not be able to watch it on Hulu or any other free streaming service.
Is Asteroid City Streaming on Disney Plus?
Unfortunately, Asteroid City is not currently available to stream on Disney Plus and it's not expected that the film will release on Disney Plus until late December at the absolute earliest.
While Disney eventually releases its various studios' films on Disney Plus for subscribers to watch via its streaming platform, most major releases don't arrive on Disney Plus until at least 45-60 days after the film's theatrical release.
The sequel opened to $150 million internationally, which Disney reports is 4% ahead of the first film when comparing like for likes at current exchange rates. Overall, the global cume comes to $330 million. Can it become the year's third film to make it past $1 billion worldwide despite China and Russia, which made up around $124 million of the first film's $682 million international box office, being out of play? It may be tough, but it's not impossible. Legging out past $500 million is plausible on the domestic front (that would be a multiplier of at least 2.7), and another $500 million abroad would be a drop of around $58 million from the original after excluding the two MIA markets. It'd be another story if audiences didn't love the film,but the positive reception suggests that Asteroid City will outperform the legs on this year's earlier MCU titles (Multiverse of Madness and Love and Thunder had multipliers of 2.2 and 2.3 respectively).
As for the rest of the box office, there's little to get excited about, with nothing else grossing above $10 million as Hollywood shied away from releasing anything significant not just this weekend but also over the previous two weekends. When Black Panther opened in 2018, there was no counterprogramming that opened the same weekend, but Peter Rabbit and Fifty Shades Freed were in their second weekends and took second and third with $17.5 million and $17.3 million respectively. That weekend had an overall cume of $287 million compared to $208 million this weekend Take away the $22 million gap between the two Black Panther films and there's still a $57 million gap between the two weekends. The difference may not feel that large when a mega blockbuster is propping up the grosses,but the contrast is harsher when the mid-level films are the entire box office as we saw in recent months.
Asteroid City, which is the biggest grosser of the rough post-summer, pre-Asteroid City season, came in second with just $8.6 million. Despite the blockbuster competition that arrived in its fourth weekend, the numbers didn't totally collapse, dropping 53 % for a cume of $151 million. Worldwide it is at $352 million, which isn't a great cume as the grosses start to wind down considering its $200 million budget. Still, it's the biggest of any film since Asteroid City, though Asteroid City will overtake it any day now.
Asteroid City came in third place in its fourth weekend, down 29% with $6.1 million, emerging as one of the season's most durable grossers and one of the year's few bright spots when it comes to films for adults. The domestic cume is $56.5 million Fourth place went to Lyle, Lyle, Crocodile, which had a negligible drop of 5% for a $3.2 million sixth weekend and $40.8 million cume., in fact )
, which isn't surprising considering it's the only family film on the market, and it's Asteroid City to grossing four times its $11.4 million opening. Still, the $72.6 million worldwide cume is soft given the $50 million budget , though a number of international markets have yet to open.
Finishing up the top five is Asteroid City, which had its biggest weekend drop yet, falling 42% for a
210 million from a budget of just $20 million.
Asteroid City Online In The U.S?
Most Viewed, Most Favorite, Top Rating, Top IMDb 𝗠ovies online. Here we can download and watch 123𝗠ovies 𝗠ovies offline. 123𝗠ovies website is the best alternative to Asteroid City (2023) 𝗙ree online. We will recommend 123𝗠ovies is the best Solar𝗠ovie alternatives.
There are a few ways to watch Asteroid City online in the U.S. You can use a streaming service such as Netflix, Hulu, or Amazon Prime Video. You can also rent or buy the 𝗠ovie on iTunes or Google Play. You can also watch it on-demand or on a streaming app available on your TV or streaming device if you have cable.
Asteroid City Full 𝗠ovie
Asteroid City Full 𝗠ovie download
Asteroid City Full 𝗠ovie 𝗙ree online
Asteroid City Full 𝗠ovie 123 𝗠ovies
Asteroid City Full 𝗠ovie online
Asteroid City Full 𝗠ovie youtube
123 𝗠ovies Watch Asteroid City (2023) 𝗠ovie Online Full Streaming at Home
123 𝗠ovies Asteroid City 2023 MP4/720p 1080p HD 4K Hindi Tamil dubbed filmywap
Watch Asteroid City (2023) 𝗙ree Online Streaming at Home
YTS Watch Asteroid City 𝗙ree Online Streaming
123𝗠ovies Where Watch Asteroid City 2023 𝗙ree Online Streaming At home
123𝗠ovies Asteroid City (2023) 𝗙ree watch online streaming at Home

sdfweeq


Tabs, code snippet copy button, edit link broken

Couple of defects have been reported for the CS website:

The tabs on the platform setup guide for apollo and starbuck instructions are not working anymore.
The copy icons have vanished on code blocks.
The edit button for pages points to the wrong URL (links use main instead of master)

MkDocs 8 came with breaking changes (release notes). We should pin the version in the build process if possible or limit to minor upgrades to make sure that they are no surprises when deploying.


No data copied when string contains %

Hi, great work and a very useful application!!
I've an issue with % characters as they have not being copied to the clipboard. I've tested it on demo page and after that the swf movie seems to be gone.
I there any issue about it?
Kind regards.

I'm going to consider this a duplicate of #565.


Update firewall-faq.yml to add support force tunneling in Azure Firew…

…all Basic
Update firewall-faq.yml to add support force tunneling in Azure Firewall Basic

@vhorne
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@vhorne,
I committed your suggestions.
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
Thanks!

#sign-off


Feedback edits

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/Create-Packages/project-json-Impact.md
:warning:Warning

Details

docs/Consume-Packages/Dependency-Resolution.md
:white_check_mark:Succeeded

docs/Consume-Packages/Package-Restore.md
:white_check_mark:Succeeded

docs/Consume-Packages/Reinstalling-and-Updating-Packages.md
:white_check_mark:Succeeded

docs/Create-Packages/Dependency-Versions.md
:white_check_mark:Succeeded

docs/Guides/Create-NET-Standard-Packages-VS2017.md
:white_check_mark:Succeeded

docs/Guides/Install-NuGet.md
:white_check_mark:Succeeded

docs/Policies/NuGet-FAQ.md
:white_check_mark:Succeeded

docs/Schema/packages-config.md
:white_check_mark:Succeeded

docs/Create-Packages/project-json-Impact.md

Line 41: [Warning] Illegal link: [Dependency resolution](../consume-packages/dependency-resolution.md#dependency-resolution-in-nuget-3x) -- missing bookmark. The file Consume-Packages/Dependency-Resolution.md doesn't contain a bookmark named dependency-resolution-in-nuget-3x.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/Create-Packages/project-json-Impact.md
:warning:Warning

Details

docs/Consume-Packages/Dependency-Resolution.md
:white_check_mark:Succeeded

docs/Consume-Packages/Package-Restore.md
:white_check_mark:Succeeded

docs/Consume-Packages/Reinstalling-and-Updating-Packages.md
:white_check_mark:Succeeded

docs/Create-Packages/Dependency-Versions.md
:white_check_mark:Succeeded

docs/Guides/Create-NET-Standard-Packages-VS2017.md
:white_check_mark:Succeeded

docs/Guides/Install-NuGet.md
:white_check_mark:Succeeded

docs/Policies/NuGet-FAQ.md
:white_check_mark:Succeeded

docs/Schema/packages-config.md
:white_check_mark:Succeeded

docs/Create-Packages/project-json-Impact.md

Line 41: [Warning] Illegal link: [Dependency resolution](../consume-packages/dependency-resolution.md#dependency-resolution-with-project-json-and-packag- references) -- missing bookmark. The file Consume-Packages/Dependency-Resolution.md doesn't contain a bookmark named dependency-resolution-with-project-json-and-packag- references.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/Create-Packages/project-json-Impact.md
:warning:Warning

Details

docs/Consume-Packages/Dependency-Resolution.md
:white_check_mark:Succeeded

docs/Consume-Packages/Package-Restore.md
:white_check_mark:Succeeded

docs/Consume-Packages/Reinstalling-and-Updating-Packages.md
:white_check_mark:Succeeded

docs/Create-Packages/Dependency-Versions.md
:white_check_mark:Succeeded

docs/Guides/Create-NET-Standard-Packages-VS2017.md
:white_check_mark:Succeeded

docs/Guides/Install-NuGet.md
:white_check_mark:Succeeded

docs/Policies/NuGet-FAQ.md
:white_check_mark:Succeeded

docs/Schema/packages-config.md
:white_check_mark:Succeeded

docs/Create-Packages/project-json-Impact.md

Line 41: [Warning] Illegal link: [Dependency resolution](../consume-packages/dependency-resolution.md#dependency-resolution-with-project-json-and-package-references) -- missing bookmark. The file Consume-Packages/Dependency-Resolution.md doesn't contain a bookmark named dependency-resolution-with-project-json-and-package-references.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Consume-Packages/Dependency-Resolution.md
:white_check_mark:Succeeded

docs/Consume-Packages/Package-Restore.md
:white_check_mark:Succeeded

docs/Consume-Packages/Reinstalling-and-Updating-Packages.md
:white_check_mark:Succeeded

docs/Create-Packages/Dependency-Versions.md
:white_check_mark:Succeeded

docs/Create-Packages/project-json-Impact.md
:white_check_mark:Succeeded

docs/Guides/Create-NET-Standard-Packages-VS2017.md
:white_check_mark:Succeeded

docs/Guides/Install-NuGet.md
:white_check_mark:Succeeded

docs/Policies/NuGet-FAQ.md
:white_check_mark:Succeeded

docs/Schema/packages-config.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add switch for graphiql

Checklist

[x] npm test passes
[x] tests and/or benchmarks are included
[x] documentation is changed or added
[x] commit message follows commit guidelines

Affected core subsystem(s)

Description of change

Graphiql should be opened for developers, not everyone.

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #6   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           8      8
  Lines          80     81    +1
=====================================
+ Hits           80     81    +1

Impacted Files
Coverage Δ

app/middleware/graphql.js
100% <100%> (ø)
:arrow_up:

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 f21b36a...14b6b01. Read the comment docs.

1


Embedded views not compatible with sites that restrict iframes

Some blogging sites restrict the use of iframes. It would be great to find a solution for embedding that doesn't require iframes. e.g. Wordpress

I don't know that this is achievable. Sorry Wordpress users :(


Fix BigQueryNodeVisitor to use tableName() method

The LogicalRelationVisitor was matching BigQueryRelations, causing nodes to be reported as DirectBigQueryRelation_struct<...>. The UnknownEntryFacet captures information on unvisited nodes, including unknown LogicalRelations, so this code is a) redundant and b) forces the node visiting to happen in a specific order to be correct. Rather than impose an order, which won't scale to extensions that might be authored outside of this library, I removed that code.
I also noticed that the existing Spark integration tests still rely on the bytebuddy/javaagent approach rather than using the Spark ListenerBus. I modified the extension to use the listener approach and updated affected tests accordingly.

Codecov Report

Merging #266 (2ccef45) into main (5f189a9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #266   +/-   ##
=======================================
  Coverage   87.58%   87.58%
=======================================
  Files          31       31
  Lines        1675     1675
=======================================
  Hits         1467     1467
  Misses        208      208

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 5f189a9...2ccef45. Read the comment docs.


vxGridRowSelectionChange event is not fired when column filter values are checked/unchecked

Steps:

Grid is loaded with data
Apply a filter for a column, this makes grid to show only filtered records
select a record from the grid
Now reset the filter
This shows all the records back with previously selected records got unselected
But vxGridRowSelectionChange event is not fired.

since, in this case, there is a scope for multiple rows to be unselected, thus 'vxGridRowMultiSelectionChange' event is fired.


Remove spinning wheel while waiting for experimenter in baseline tasks

The program should freeze/block using select library by python for efficiency, so it is best not to change unless this is an issue

The program should freeze/block using select library by python for efficiency, so it is best not to change unless this is an issue


[Python] test_cuda_numba_interop fails locally

Describe the bug, including details regarding any error messages, version, and platform.
I get some failures locally when running test_cuda_numba_interop. This is probably because I have some rather old CUDA hardware, but I wonder why some of those tests (and not all of them) are requiring 8.4 while my device has compute capability 8.0. This used to work some (long?) time ago.
Example traceback:
_________________________________________________________________ test_numba_memalloc[uint8-pyarrow.cuda] _________________________________________________________________
Traceback (most recent call last):
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/cudadrv/driver.py", line 2824, in add_ptx
    driver.cuLinkAddData(self.handle, enums.CU_JIT_INPUT_PTX,
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/cudadrv/driver.py", line 327, in safe_cuda_api_call
    self._check_ctypes_error(fname, retcode)
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/cudadrv/driver.py", line 395, in _check_ctypes_error
    raise CudaAPIError(retcode, msg)
numba.cuda.cudadrv.driver.CudaAPIError: [222] Call to cuLinkAddData results in CUDA_ERROR_UNSUPPORTED_PTX_VERSION

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  [...]
  File "/home/antoine/arrow/dev/python/pyarrow/tests/test_cuda_numba_interop.py", line 169, in test_numba_memalloc
    darr[:5] = 99
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/cudadrv/devices.py", line 232, in _require_cuda_context
    return fn(*args, **kws)
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/cudadrv/devicearray.py", line 667, in __setitem__
    return self._do_setitem(key, value)
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/cudadrv/devicearray.py", line 726, in _do_setitem
    _assign_kernel(lhs.ndim).forall(n_elements, stream=stream)(lhs, rhs)
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/dispatcher.py", line 486, in __call__
    specialized = self.dispatcher.specialize(*args)
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/dispatcher.py", line 715, in specialize
    specialization.compile(argtypes)
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/dispatcher.py", line 926, in compile
    kernel.bind()
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/dispatcher.py", line 197, in bind
    self._codelibrary.get_cufunc()
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/codegen.py", line 195, in get_cufunc
    cubin = self.get_cubin(cc=device.compute_capability)
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/codegen.py", line 170, in get_cubin
    linker.add_ptx(ptx.encode())
  File "/home/antoine/mambaforge/envs/pyarrow/lib/python3.10/site-packages/numba/cuda/cudadrv/driver.py", line 2827, in add_ptx
    raise LinkerError("%s\n%s" % (e, self.error_log))
numba.cuda.cudadrv.driver.LinkerError: [222] Call to cuLinkAddData results in CUDA_ERROR_UNSUPPORTED_PTX_VERSION
ptxas application ptx input, line 9; fatal   : Unsupported .version 8.4; current version is '8.0'
---------------------------------------------------------------------------- Captured log call ----------------------------------------------------------------------------
ERROR    numba.cuda.cudadrv.driver:driver.py:392 Call to cuLinkAddData results in CUDA_ERROR_UNSUPPORTED_PTX_VERSION

I have a Pascal GPU (GeForce GT 1030).
$ nvidia-smi
Tue Mar 19 17:19:19 2024
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 525.147.05   Driver Version: 525.147.05   CUDA Version: 12.0     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  NVIDIA GeForce ...  Off  | 00000000:04:00.0  On |                  N/A |
| 35%   36C    P0    N/A /  19W |    579MiB /  2048MiB |      5%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+

Component(s)
GPU, Python

@kkraus14 Any ideas here?

The PTX ISA version is 8.4 but your driver only supports 8.0. I'm not sure why Numba would.be generating a newer PTX version.
@gmarkall any chance you could help here?


Questions about batch inference and evaluation for coco

Hi! Thanks for sharing your excellent work. I have some questions abount batch inference  and evaluation for coco.
First question is abount batch inference. I have try to evaluate refcoco and coco by per sample inference using your model, but it is too slow, abount 2 hours for refcoco and 10 hours for coco in A100. In order to inference quickly, I try to batch inference, and question is here, when padding left for short sample, the inference result is different from no padding, It's very strange. Have you encountered this promblems?
Second question is abount evaluation for coco. I have written an evaluation script based on the calculation method of metrics given in your paper, and choosed a prompt template from your papers. Finally, the metrics I get is higher then your paper, I want to know if there is a problem with my evaluation script. Can you release your evaluation script? Thanks！

Hi sexan,
Thank you for your interest in our work.
As for the inference speed, if you use a single node (8 A100s) for inference, the total time is about 1h-1.5h. If you use a single gpu, 10 hours may be right. The padding approach should be in consistent with the original LLaVA. If you try to use batch inference, the stopping criteria should also be modified. We recommend to increase the total batch size while keep the batch size of single gpu to 1, which is also utilized in projects like Detectron2. As for the evaluation for coco, under different inference settings, the results will have some difference. If you have checked the visualization results, you can use your present script for the higher result, which we think is acceptable and is encouraged to share with the community with a pull request. Our evaluation script will be released with the entire project soon.
Thanks again for your attention. Hope to get your continual focus and suggestions to improve Griffon.

Hi sexan,
Thank you for your interest in our work.
As for the inference speed, if you use a single node (8 A100s) for inference, the total time is about 1h-1.5h. If you use a single gpu, 10 hours may be right. The padding approach should be in consistent with the original LLaVA. If you try to use batch inference, the stopping criteria should also be modified. As for the evaluation for coco, under different inference settings, the results will have some difference. If you have checked the visualization results, you can use your present script for the higher result, which we think is acceptable and is encouraged to share with the community with a pull request. Our evaluation script will be released with the entire project soon.
Thanks again for your attention. Hope to get your continual focus and suggestions to improve Griffon.

Hi  @sexan,
Thank you for your interest in our work.
As for the inference speed, if you use a single node (8 A100s) for inference, the total time is about 1h-1.5h. If you use a single gpu, 10 hours may be right. The padding approach should be in consistent with the original LLaVA and the generation and stopping function. As for the evaluation for coco, under different inference settings, the results will have some difference. If you have checked the visualization results, you can use your present script for the higher result, which we think is acceptable and is encouraged to share with the community with a pull request. Our evaluation script will be released with the entire project soon.
Thanks again for your attention. Hope to get your continual focus and suggestions to improve Griffon.

Thank you for your response.
I've noticed that in your model inference, you've customized a stopping criterion which adds the  symbol as a stop token. However, the original llava model defaults to using  as the stop token. In my actual inference process, it seems that encountering  typically signals the end of the process. It appears that the new stop token  you've added doesn't affect the inference outcome. However, this newly added stopping strategy is suitable only for single-sample inference. Could you please clarify if I can safely remove this parameter during batch inference? Looking forward to your response. Thank you!"

Hi, @sexan
I’m currently trying to reproduce the mAP metrics on COCO and have encountered some issues. Could you please share your evaluation code and scripts? I would greatly appreciate the opportunity to reference your work.
Thank you very much for your help!

Hi @sexan
I think it will only ends when meeting the , and the  does not work. You can also customize it in the initialization of stopping_criteria.


vSphere: Add error exit in configure script to prevent silent failure

Now if any commands in the configure script fail, the install hangs since the script doesn't return any error:
https://github.com/kubernetes/kubernetes-anywhere/issues/299
This PR adds error exit handling in configure script for vSphere provider in order to prevent silent failures.

@abrarshivani @kerneltime

@abrarshivani @kerneltime
All the comments are addressed. Please review again thanks :)


If I want to train on another dataset

Hello, thank you for your work. Which part of the code do I need to change if I need to train on another dataset

Hi, @Xiejiahao233 ! The format of the dataset needs to follow that of MORPH II we provided, and the configuration files need to be tweaked accordingly. If everything goes correctly, no code modification requires.


Implementing selective FTP deploy to deploy only changed files

Currently, our FTP deploy process uploads all files to the server every time we deploy, even if only a few files have been modified. This can be time-consuming and inefficient, especially when dealing with large projects with many files.
To improve the deploy process, we would like to request a feature that allows us to selectively upload only the files that have been modified since the last deployment. This feature would greatly reduce the deployment time and save bandwidth usage on both our end and the server's end.
We understand that there may be some challenges in implementing this feature, such as tracking the changes made to files since the last deployment and identifying which files need to be uploaded or by simply comparing the last write time of the two files. However, we believe that this feature would greatly improve the overall efficiency of our deploy process and make it easier for our team to manage deployments.
We welcome any feedback or suggestions on how this feature can be implemented, and we are happy to collaborate with the community to make this feature a reality. Thank you for considering our feature request.

v4+ uploads only changed files.
Uploaded file state is synced to the .ftp-deploy-sync-state.json file. But some FTP account cannot read this file (maybe that's why it's not working?)
At the end of the upload you should see a log with the following:
🎉 Sync complete. Saving current server state to "./.ftp-deploy-sync-state.json"

Does it work in in case of DANGEROUS_CLEAN_SLATE set to true ?

No. Look at your deployment logs, you'll see the following warning:
🗑 Removing all files on the server because 'dangerous-clean-slate' was set, this will make the deployment very slow...
Also look at the readme, you'll see the following description
"Deletes ALL contents of server-dir, even items marked as --exclude argument"


Changes to make int-safe

 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.Henry Robinson seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


doc: make events in the document clearer

Fixes #412
Hi @seratch, how does this look?
Let me know if this needs any changes.
Thanks
Category (place an x in each of the [ ])

[ ] slack_bolt.App and/or its core components
[ ] slack_bolt.async_app.AsyncApp and/or its core components
[ ] Adapters in slack_bolt.adapter
[x] Document pages under /docs
[ ] Others

Requirements (place an x in each [ ])
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run ./scripts/install_all_and_run_tests.sh after making the changes.

 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? Let us recheck it.

Hi @RhnSharma, thanks for taking the time to make this pull request! Perhaps, there may be more to change in the documents. I would like other maintainers to check this first.

@stevengill I have made the changes you suggested. Please take a look at it and let me know if this needs any changes.
Thanks

@stevengill This pull request now looks good to me. Do you have any comments on it?

Will merge this pull request tomorrow if there is no other comment

@RhnSharma Thanks a lot for making this pull request 👍


Discussion(meta): while configuring, only wait for the barriers related to the specific parts of the graph to be collected

In RisingWave, configuration changes such as DDL statements or modifications to parallelism are bound to a specific epoch, with the operation information spreading through the graph along with the corresponding barriers.
Currently, the meta node waits until all actors have received the barrier before completing the operation. I am considering whether it might be possible to only wait for the completion of barriers related to specific parts of the graph.
There are two main motivations for this consideration:

We previously discussed decoupling checkpoints, using a log store to decouple upstream and downstream checkpoints. One issue is that the log store does not handle configuration changes well. Even if it can store them, this means the configuration change has to wait a long time before it is actually consumed on the graph. Now, we are preparing to discuss the use of the log store during backfill, and I believe there will be similar issues. This method still does not support changing the parallelism of a materialized view (MV) that is being created, or creating an MV on a MV that is being created, but it solves the problem of changing other streaming jobs.

During on-call support, I often encounter users complaining about the slow completion of DDL operations, and upon investigation, I find that the barrier latency is too high. I intuitively feel that some users are not really concerned about the end-to-end latency but are affected by the fact that we have forcibly bound the control plane's latency to the barrier latency. By only waiting for the barriers of the relevant actors, we can reduce unnecessary waiting for other slow actors.

To be honest, I'm not very familiar with the parts related to the meta node, so I can't assess the engineering difficulty and workload. Furthermore, after writing this issue, I realized that it only alleviates some problems, but many key issues remain unresolved. Therefore, it might just be a feature that would be nice to have, but not critical.
cc @fuyufjh @yezizp2012 @shanicky

Link to #15490 and #13396, sharing quite similar motivations and proposals.


plan

Plan how to get from current state to release of a useable resource for new adopters at RabbitEars or BUNS

I realized these were just process ideas, spinoffs for public development tasks:

[ ] document prompt for skel
[ ] doc the app you used for ui training
[ ] doc the icon creation
[ ] changelog
[ ] changeblog


Missing Azure Service Bus message properties in Python SDK message, only message body available

Package Name: azure-servicebus
Package Version: 7.11.1
Operating System: MacOS Ventura 13.4.1
Python Version: 3.10.12

Describe the bug
Only the message body is available when receiving them from an Azure Service Bus Topic Subscription. The message properties (both broker properties and custom properties) are missing.
To Reproduce
Steps to reproduce the behavior:

Create a service bus client from connection string
Create a receiver using servicebus_client.get_subscription_receiver
Enumerate the messages in the receiver
Inspect message contents, e.g. with print(str(msg))

Expected behavior
I was expecting properties such as message_id, content_type, custom_properties and body to be there. They are not, and the contents of the body are the only available information on the message (and there is no property named body either, they are all in the root of the object).

Thanks for your feedback @lurifaxel . We will investigate and get back to you asap.

Hi @lurifaxel - In order to print all properties, you can call print(repr(message)). print(str(message)) will only print the body. Thanks!


Flurry is not receiving any events

Hey,
just installed your package and executed this in my Root.js
import FlurryAnalytics from 'react-native-flurry-analytics';

FlurryAnalytics.setAppVersion('32');
FlurryAnalytics.setDebugLogEnabled(true);
FlurryAnalytics.setCrashReportingEnabled(true);
FlurryAnalytics.startSession('My key here');
FlurryAnalytics.logEvent('appOpen');
FlurryAnalytics.logPageView();

but my flurry dashboad shows no activity whatsoever

Same issue for me on Android using react native 0.49.5 and flurry analytics 3.0.1. iOS is working just fine.

@IvRRimum  what was the issue, can you please tell ?


Add option to kill tmux session if not connected to within a certain time period

Use case: I've set up my workflow to automatically run the upterm action if the job fails. This works really well, except for one thing - it causes the action not to complete for hours if there's a failure, because the upterm server just waits there. I could set timeout-minutes on the entire step, but that will kill the terminal even if a user connects to it. Instead, I'd like it if there were a timeout-minutes field specific to this action that terminates the action if no one SSH's to the server within that number of minutes, but keeps the terminal open if someone connects.

This would be amazing! When can we expect this to be available?

@bsgrigorov waiting on approval of #18

Shouldn't there be a timeout even if someone connects to the server? I've had instances where I connected to the server and forgot to terminate the session and github actions kept running for 3-4 hours. It would be great if we can set a session TTL. Default could be like 20 mins.

@telotortium Shouldn't there be a timeout even if someone connects to the server? I've had instances where I connected to the server and forgot to terminate the session and github actions kept running for 3-4 hours. It would be great if we can set a session TTL. Default could be like 20 mins.

@bsgrigorov I should be possible to implement that using bash's existing feature where the timeout is configured with the TMOUT environment variable. Would you want to give it a try?

I realized you can set timeout-minutes parameter on either the GitHub Actions step or job.


Optimize lines_lf

Many of the same optimization we have applied to the other methods. Big speedups here as well.

Ah, thanks a bunch for this!  I'll try to review this soon.

Thanks a bunch!


Extent for data

Tampere city region is the main area of interest, but there are some routes that are part of bigger routing system, so they should be taken into account.
Possible solutions:

Mask for showing Tampere city region area more "clearly" and other regions under the mask layer slightly translucent
Overlay with Tampere city region + those routes that are inside AND intersect the border
Manually picking the routes that are wanted (list of routes from municipalities)

To do:

[x] Mockup
[ ] Muncipality cookie cutter?

@anniinakovalainen add mockup to this

@Rikuoja added the mockup for this, but do you need a cookie cutter (layer with all the municipalities)? Or can you do this with this info? Problem might be that the routes that should be continuing over the border are not "intact" so that if we create a GIS overlay it just takes parts of the routes, but not all. If the Lipas-data has municipality info in all routes, then there could be some query for this (e.g. take all routes into account that are within these municipalities). One option is to create a municipality layer in the map view that "covers" the areas that are not wanted... So maybe at least 3 ways to do this.

@anniinakovalainen ideas on how to do this?

@SannaJokela Hmm, I was wondering if the data extent could be determined in the database? I would do it like this: when the data is brought to the kooste database, it is clipped with the Tampere region. If this is done in the frontend, I have no solution other than a buffer (that has been done already). @Rikuoja might have some idea if the map of Tampere region could mask the data in the frontend?

Joona comments: 3 options:

this could be done in front side (mask) if we do not want to hard code it to the database. Inverted polygon type solution....
More simple way to do in the database: materialized view with extent.
OR: Pgtile serv supports functions, that takes certain parameters into account. This might be the best solution. This needs the extent polygon. https://github.com/CrunchyData/pg_tileserv#function-layers

Won't be doing any of those. All of them require

importing all kaupunkiseudut data to the frontend or backend, and
having the huge polygon filter the data in the frontend or backend, and
creating either separate duplicate views for each table, or separate functions that cannot be implemented since we want to be able to use CQL for further filtering of the data.

Instead, use a simple CQL filter on the cityName field. The field is already in lipas and some other tables. So we just need to fill in this field in all tables, and everything will be fine and dandy. There are some data (notably OSM) that is only visible when we zoom in, and it doesn't really matter if we show that outside the cities in question.
The main thing is that the lipas points aren't shown beyond the Tampere region, and that's easy with no backend changes just by adding the CQL cityName filter.


Support rational <-> floats?

Take a closer look at the rationals library to see if this is well supported.

This is supported by the rationals library. However, the real question is how should this be enabled? Via opts/flags? By default? (probably not by default).


memory hotplug doesn't work, guestCurrent not changed.

What happened:
I try to test memory hotplug according to the https://kubevirt.io/user-guide/compute/memory_hotplug steps, It did trigger a update migration, but after the update migrate, the guestCurrent not changed and the memory not hotplugged.
What you expected to happen:
After the hotplug request is processed and the Virtual Machine is live migrated, the new amount of memory should be available to the guest and visible in the VMI object.
How to reproduce it (as minimally and precisely as possible):

Enable feature-gate

kubectl --namespace kubevirt patch kv kubevirt -p='[{"op": "add", "path": "/spec/configuration/developerConfiguration/featureGates", "value": ["VMLiveUpdateFeatures"]}]' --type='json'
kubectl --namespace kubevirt patch kv kubevirt -p='[{"op": "add", "path": "/spec/configuration/vmRolloutStrategy", "value": "LiveUpdate"}]' --type='json'
kubectl --namespace kubevirt patch kv kubevirt -p='[{"op": "add", "path": "/spec/workloadUpdateStrategy/workloadUpdateMethods", "value": ["LiveMigrate"]}]' --type='json'

create a vm with maxGuest:

      domain:
        cpu:
          maxSockets: 64
          cores: 1
          sockets: 1
          threads: 1
        memory:
          maxGuest: 64Gi
          guest: 1Gi

get status.memory

[root@hff-k8s-1 ~]# kubectl get vmi hffvm-1 -o json | jq .status.memory
{
  "guestAtBoot": "1Gi",
  "guestCurrent": "1Gi",
  "guestRequested": "1Gi"
}

patch the VM

kubectl patch vm  hffvm-1  -p='[{"op": "replace", "path": "/spec/template/spec/domain/memory/guest", "value": "2Gi"}]' --type='json'

get status.memory after hotplug

[root@hff-k8s-1 ~]# kubectl get vmi hffvm-1 -o json | jq .status.memory
{
  "guestAtBoot": "1Gi",
  "guestCurrent": "1Gi",   # not changed
  "guestRequested": "2Gi"
}

Additional context:
cpu hotplug works well, but memory hotplug doesn't work.
Is there any dependency or configuration I am missing?
Environment:

KubeVirt version (use virtctl version): 1.2.2
Kubernetes version (use kubectl version): v1.27.13
VM or VMI specifications: N/A
Cloud provider or hardware configuration: N/A
OS (e.g. from /etc/os-release): CentOS Linux 7 (Core)
Kernel (e.g. uname -a): 5.4.230-1.el7.elrepo.x86_64
Install tools: N/A
Others: N/A

/cc @acardace
Hoping you can help as I see you authored the relevant doc. Thanks!

Hi @fangfenghuang, unfortunately the guest OS you're using doesn't support memory hotplug.
To successfully use memory hotplug you need a guest using at least Linux 5.16.
If that still doesn't work feel free to re-open this issue, thank you!

@aburdenthehand I see that I did not mention it in the user-guide, I definitely need to. I'll push a PR to update the user-guide soon.

Hi @fangfenghuang, unfortunately the guest OS you're using doesn't support memory hotplug.
To successfully use memory hotplug you need a guest using at least Linux 5.16.
If that still doesn't work feel free to re-open this issue, thank you!

After upgrading the guest kernel version to 6.9.4-1.el7.elrepo.x86_64, memory hotplug effective.


configFile broken in v2?

Looks like eslint dependency is no longer accepting configFile. eslint still hasn't fixed it in v2.2

Looks like eslint is ignoring .eslintrc.* files
https://github.com/eslint/eslint/issues/5314

That's not an issue with this plugin.


404 - Page not found

404 - Page not found for TryGetRangeShardMap (Java), TryGetListShardMap (Java) and GetShardMap (Java)

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

ID: d410fa11-0b87-5900-8240-51bcd12131ce
Version Independent ID: d331cade-e7ac-9acc-57bc-3596f989c234
Content: Query sharded Azure SQL databases
Content Source: articles/sql-database/sql-database-elastic-scale-multishard-querying.md
Service: sql-database
GitHub Login: @stevestein
Microsoft Alias: sstein

@kazshi Thank you for the valuable feedback,we are investigating the issue.

@kazshi I have assigned the issue to the content author to evaluate and update as appropriate.

@kazshi - thanks for reporting this. Links should all work now.

#please-close

@stevestein Confirmed the links work. Thanks for the update.


Optimize endpoint index getter and also rename some methods to be mor…

…e readable
What type of PR is this?

/kind cleanup
/kind enhancement
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #775
Previously we iterate all the endpoint map when delete a workload. Now the index are cached in userspace
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

cc @LiZhenCheng9527

/lgtm
/approve


[jts] Foreign Key to self

According to the spec...

Thus, to use Foreign Keys you must be referencing Data Packages.

That's cool, but self-referential foreign keys are quite common and IMHO can be supported directly in JTS without requiring that the data be in a data package.
Thoughts?

@pwalsh do you have an explanation of how this would work? the idea was that an "external" FK would require thing that external "thing" was a data package. Agree that internal keys would not require this - and we may need to clarify language on this.

I think the MVP for this would be that foreignKey.reference hash can describe a self-referencing relation by:

datapackage: ""
resource: "self" (or some similar string that can reasonably be expected not to be the name of a file in the same directory)

Other fields all remain.

Ah, i get this. Is this a FK within the same table (that's a bit odd but not impossible e.g. tree / parent setups).
Or do you want this for multiple tables in same directory? If so how do you reference without the DataPackage.json resources setup?

FK within same table, yes. I think it is common enough.
For multiple tables in same directory, I think it is of to require a data package (as a good practice).

Here is an example implementation of a foreign key to self:
https://github.com/pwalsh/israeli-municipal-budget-tree/blob/master/datapackage.json#L120
The example is in a data package, but that is not relevant to this particular implementation.

@pwalsh i get this one though i think in discussion we established this was really a very unusual set up for parent (related to a taxonomy essentially) and that best practice for this taxonomic setup was not to have parent but to record explicitly the multiple levels of the taxonomy on each row.

@rgrp: I'm really not convinced it is unusual: as you say, self-referencing taxonomies are a thing, so if we separate the principal from the fact that my example is a budget structure, I really do think it is relevant for JTS. I know there are more efficient tree implementations, and these would likely be much harder to represent in JTS, but this still seems to me a very useful pattern.
For the specific issue of multiple levels per row in my actual data, I do that on a different implementation of this data with path tuples that represent the levels/nodes above and below each node, but not sure how to represent it well in CSV.

I want to move to implement or WONTFIX. what do we think
/cc @paulfitz @jpmckinney

I agree on implement or WONTFIX. If I get any third party support, I'll implement :).

Self-reference seems likely in machine-generated tree structures. As @rgrp describes, a different structure is more likely in a human-edited tree structure. If no one implements, and if we're trying to close issues, then I suppose it will be wontfix.

On further reflection, it seems I just can't let go of this one, so here's a pull request: https://github.com/dataprotocols/dataprotocols/pull/201
@jpmckinney your comment about machine-generated vs. human-generated is at the core of the issue, and in my experience exactly the opposite is true.
So, I'm going to potentially beat the dead horse and make the case I made above, again.
Here is an example schema that would implement support for self-referencing FK in JSON Table Schema:

https://github.com/pwalsh/il-municipal-budget-tree/blob/master/datapackage.json#L125

Here is the data it describes:

https://github.com/pwalsh/il-municipal-budget-tree/blob/master/data.csv

Note the code and parent columns. It is fairly self-explanatory.
I don't have a depth or level column in CSV, or any other representation of the path from self -> root (that, to me, seems like work for a machine, not a human, and in an app that uses this data, I do calculate the path and depth/level on load to a database).
Now, the CSV sources here are a mixture of some things I've scripted and some manual data entry.
In my experience working people doing this data entry, it is much, much easier to understand how to enter "parent" than it is to work with depth, for example (depth implies holding the whole model of the tree in mind, and this specific line's position in it, whereas parent is a more manageable affair of "this is under that").

@pwalsh can we close this one now that you've merged the fix? I'd leave you to add the reference to the PR / commit that closed it.

Added in https://github.com/dataprotocols/dataprotocols/pull/201


App Search input box flushed with the menu panel on left.

Input is flushed on the left menu. The entire field can be inside a mat-menu-item with left/right margins.

Merged


Work station lock/unlock triggers

How can we create a OnWorkStationLock or OnWorkstationUnlock task with code using this wrapper?
In Windows station in the trigger drop down , triggers for this are available, but i cant find the corresponding trigger in the wrapper .
Help please.....
Originally posted: 2012-04-23T03:07:33

Add a SessionStateChangeTrigger to your task with the appropriate value assigned to the StateChange property.
new SessionStateChangeTrigger(TaskSessionStateChangeType.SessionLock) { UserId = "joe" };
new SessionStateChangeTrigger(TaskSessionStateChangeType.SessionUnlock);
Originally posted: 2012-04-23T16:47:31

Dahall,
I tried that last time also, but its creating a task as a one time task, unlike onworkstation task as created when i create it from the UI. Also when i lock the screen, this task is not getting fired.
Here's my code snippet:
using (TaskService ts = new TaskService(pc, userId, domain, pwd, forceV1))
{
                    TaskDefinition td = ts.NewTask();
td.RegistrationInfo.Description = "On Workstation Lock";
td.Principal.LogonType = TaskLogonType.ServiceAccount;
td.Settings.Priority = System.Diagnostics.ProcessPriorityClass.Idle;
td.Settings.RunOnlyIfLoggedOn = false;
                   td.Settings.ExecutionTimeLimit = new TimeSpan(45, 0, 0, 0);
                    td.Triggers.Add(new SessionStateChangeTrigger() { StateChange = TaskSessionStateChangeType.SessionLock, UserId = "apt\koushikch" });                    td.Actions.Add(new ExecAction(targetTask, "", null));
                    ts.RootFolder.RegisterTaskDefinition(taskName, td);
}
Originally posted: 2012-04-23T22:45:01

When I use that trigger, it works as long as my action can be run by the SYSTEM account or if I specify an account and password when I register the task. The history will show failures and successes.
Originally posted: 2012-04-24T15:01:53

I already have a OnIdle task and that is executing the same action under the System account.
I need the same action to happen when the workstation is locked.
As you can see, as my requirements of actions are same, so i have created the 2 tasks in the same way, but with different trigger conditions.
The OnIdle task is working perfectly fine, but  this is not working....
Originally posted: 2012-04-25T00:54:30

Dahall,
When i export the tasks from task scheduler UI in xml format,
I see difference in the cases:

Case when its created by the wrapper :

          2012-04-25T13:24:00      true

Case when its manually created from the UI:

          true      SessionLock
As you can see the problem is with the trigger type. Its not getting created as a StateChange trigger from the wrapper, as expected
Originally posted: 2012-04-25T01:55:30

I sorted it out...my mistake... i was having the forceV1 flag on. It should ve for v2
Originally posted: 2012-04-25T04:36:57


working with Big Data

we are using Bayesian optimisation with big data params tuning . the bayesian parameters running function breaks if data is more than 100 kb. can you guide which libraries can accelerate the training and bayesian optimisation function to run on big data like 100mb - 1 gb of data .

This is not a memory problem, your data has missing values (read the trace).

hi fmfn how to speed the running function fast or bayesian_opt fast to process big data ?


namespaced leader elector RBAC for cluster scope federation

A new role for leader elector is created in federation controller namespace for bindings.
secret and configmap has minimum privilege
Fixes #687

@marun the name is better. Fixed.

Nice, we're now minimally privileged.  Thank you!
/approve
/lgtm


Is there a way to express negation

General Information

[ ] Bug
[x] Improvement
[ ] Feature
[ ] Other

Description
Is there a way to express negation  --> not end with js or similar?
(Add images if possible)
Steps to reproduce
(Add link to a demo on https://jsfiddle.net or similar if possible)
Versions

natural-regex:
Browser:

Hi @haikyuu,
at the moment this syntax is not supported. I'm not sure if we can implement negation as a general operator with regular expressions, I have to study if it is feasible.
A regex that match that could be this one:
const regex = NaturalRegex.from(`
    start not followed by
    group
        anything, ends with js
    end group.
    anything, end.
`);

// regex = /^(?!(?:.*js$)).*$/

regex.test('file.js'); // false
regex.test('file.json'); // true


⚠ Add go modules to IPAM API

What this PR does / why we need it:
This PR adds go modules to IPAM API. The reason to add the go modules in API is to uplift IPAM for CAPM3. List of modules updated:

sigs.k8s.io/controller runtime from v0.7.0 to v0.9.0-beta.0
sigs.k8s.io/cluster-api from v0.3.12 to v0.3.11-0.20210429135050-9d825629d89c v1alpha4
k8s.io/client-go from v1.5.1 to v0.21.0
k8s.io/apimachinery from v0.20.1 to v0.21.0
k8s.io/apiextensions-apiserver from v0.20.1 to v0.21.0
k8s.io/api from v0.20.1 to v0.21.0

Important:
Once this PR goes in, those who need to uplift IPAM, and they are only using IPAM APIs , you need to change your go modules to use
github.com/metal3-io/ip-address-manager/api/v1alpha1

instead of:
github.com/metal3-io/ip-address-manager

If you are using/referring to other parts of code in IPAM not residing in API/v1alpha1, you need to use both the modules:
github.com/metal3-io/ip-address-manager/api/v1alpha1
github.com/metal3-io/ip-address-manager

/hold
/assign @maelk
/cc @kashifest

This looks good to me. Do we need to run integrations test?

/assign @maelk
/cc @kashifest

/approve
I believe this wont pass test-integration with the current changes right ?

/hold

/close
go modules in this patch located under /api/v1alpha1 which is not ideal as there might be a /api/v1alpha2 in future and we need to consolidate modules outside of the v1alphaX folder anyways. Moving them out to /api in the current patch created some issues. I opened another PR to have go.mod under /api.


Add support for Cache Assignment Groups to DB and TO API

What does this PR (Pull Request) do?
Cache Assignment Groups simplify assigning overlapping subsets of caches to Delivery Services. Cache Assignment Groups also provide a way to override which mid caches are used on a per-delivery service basis.
This commit does not yet include changes to CRConfig or remap.config generation. Those changes will follow in a future commit.
The DB schema is changed to:

Add cacheassignmentgroups, deliveryservice_cacheassignmentgroup, and cacheassignmentgroup_server tables
Add deliveryservice_assignedservers view

This change affects the following endpoints:

Add /api/1.4/cacheassignmentgroupsserver API endpoint (and added docs)
Add Cache Assignment Group support to /api/1.4/deliveryservice_server and /api/1.4/deliveryservice/servers and /unassigned_servers
Added cacheAssignmentGroup to /api/1.4/deliveryservices endpoint

Also includes additional TO API tests for cache assignment groups

[x] Relates to #3557

Which Traffic Control components are affected by this PR?

Documentation
Traffic Control Go Client
Traffic Ops

What is the best way to verify this PR?
Pull in the PR branch, migrate DB schema on to_test and run the API test on TestCacheAssignmentGroups
The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3775/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3859/
Test PASSed.


Duration/current playhead time display?

Is there any way to display duration and/or current time?

Added timeDisplay prperty

Works fine, thank you.


chore: update component owner for AWS to carolabadeer

Short description of the changes
I am transferring ownership of AWS X-Ray Components to @carolabadeer, who will help maintain them going forward per CONTRIBUTING.md.

Welcome @carolabadeer :slightly_smiling_face:


Dependency ordering with internal modules (namespaces)

TypeScript Version:
1.8.0
Code
I'm porting my ASP.NET MVC app platform (Serenity) which were using Saltaralle C# -> JS transpiler to TypeScript. Currently TS is an alternate option but hoping to make it primary soon, after resolving some issues.
Saltaralle and its predecessor Script# uses some code similar to TypeScript internal modules or namespaces. So it was natural for me to choose this model over more complicated AMD/CommonJS etc. modules, which is a bit overkill for web apps in my humble opinion. Also it is much easier to explain to novice users.
I'm using tsconfig.json with outFile option, and no "files" list, so all .ts files in all subfolders are included by default.
If you have A.ts:
namespace Some {
        export class A {
        }
}

and B.ts:
namespace Some {
        export class B extends A {
        }
}

I'd expect combined output.js file to have A defined before B, otherwise it would cause a javascript error.
But, sometimes this is true, sometimes opposite. I know that you'd suggest me to put /// references or put files in correct order in tsconfig.json files property, or use external modules, but this exactly what i'm trying to avoid. I've read among several issues reported here, and workarounds offered like these, but dependency order resolving shouldn't be the job of developer, compilers are specially made for this.
I'd handle these workarounds myself, for my own projects, but my targeted users (devs) usually don't have the skills to understand what this ordering is all about. They'll simply report a bug with details: "not working", and without looking at their output file, i can't even guess what it is really about.
I understand that in some edge cases this might be difficult, but please offer an option to perform topological sort on input files for basic cases like this, e.g. a class extends another, references it in a decorator argument etc.
Expected behavior:
I'd expect internal modules to be topologically sorted based on dependencies.
Actual behavior:
Sometimes it is not. Derived class outputted before base class.

I know that you'd suggest me to put /// references or put files in correct order in tsconfig.json files property, or use external modules, but this exactly what i'm trying to avoid.

Why?
What happens if the user writes code like this?
A.ts
console.log('A init');
class A extends C { }
console.log('B init');
class B { }

B.ts
console.log('C init');
class C { }
console.log('D init');
class D extends B { }

There's no real solution here other than understanding that code ordering matters.

I clearly understand code ordering matters for TypeScript and Javascript. As i commented above, there may be some edge cases. Let's say it should at least work with one class per file.
Anyway Saltaralle is also a transpiler and it can handle the case you wrote above without any problems at all. It will order classes correctly before outputting.

You can re-order C# code arbitrarily because there's no code outside of classes in C#. But in the above example, where should the console.log statements go? What if those statements have important side effects that change the behavior of class initialization?

I understand, but my request wasn't this. I didn't ask TS to order classes, just input files. If something could be resolved with // reference statements, it should be resolved with topological sort too. If not, it's perfectly ok. Your sample can't be resolved with /// references too...

Maybe it was my mistake to state Saltaralle can do this, was just informational. Also this should be an "option" that goes into tsconfig.json, not default.

+1 on @volkanceylan see for an instance:
In the same file:
===================================================
class Utilizer {
public static iUseIt= Proxy.objectProxy
}
class Proxy {
public static objectProxy = Proxy.createTheObject()
private static createTheObject () : any {
return "test the dependency"
}
}
console.log(Utilizer.iUseIt)
Console shows the error:
Uncaught TypeError: Cannot read property 'objectProxy' of undefined
If you invert Proxy and Utilizer the run gives the correct result.
And code is class-encapsulated.

Or worse:
class Utilizer {
public static iUseIt= Proxy.objectProxy
public static tellme() {
console.log(Utilizer.iUseIt)
}
}
class Proxy {
public static objectProxy = Proxy.createTheObject()
private static createTheObject () : any {
	return "test the dependency"
}

}
Utilizer.tellme()

After working on a project for over a year, I am surprise that we just ran into this issue as well.  I'm surprised the compiler does not throw an error on something like:
namespace x{
	import test = y.test;

	export class Instance{
		public user:any = new test();
	}
}

namespace y{
	export class test{
		public anything:string = 'anything';
	}
}

var instance = new x.Instance();

At the time the import runs (in an IIFE), the y namespace has not yet been defined and you get an undefined error.

Worst part is it is really hard to understand what is the problem from the error message you get. This is especially true in large scale projects, where TypeScript is supposed to "scale" better compared to Javascript.

I Agree:
If being tied to the class definition order is the intended behavior of TS the compiler could at least emit a warning, based on the info available to it in tsconfig.json.

In my use case we are either wiredep'ing individual files which just come in in alphabetic order or we concat/uglify them into one application code file.   Coming back from ngConf, it is apparent that external modules (ES6 imports) and using rollup.js or WebPack v2 (to concatenate as needed) are the preferred solution.

@rcollette more than on file compilation order it's dependent
on the transpilation order and the dynamic nature of JS.
The small example I posted before works (breaks) even as a single file.
That's why I think that tsc should at least file a dependency warning.
Or change definition semantics and make another step away of JS.

Users already started to report issues related to this ordering problem. I think it should at least be deterministic in ordering. If input files are not specified, and located by searching in file system, compiler shouldn't enumerate them in random order. It should order them with some some deterministic sort, e.g. full path name. Otherwise, it produces different output in each compile, and it works on one compile, while it doesn't in another.

I've written about the hazards of outFile https://basarat.gitbooks.io/typescript/content/docs/tips/outFile.html :rose:

It should order them with some some deterministic sort, e.g. full path name

@volkanceylan You can order the files in tsconfig.json (or if not using that use _references.ts) :rose: . Best to just use modules :heart:

Adding files to tsconfig.json manually and in the position it should is like something from ancient times. And i cant imagine a compiler that cant do such a simple thing. Using outFile is bad just because compiler cant handle it properly, not because its a bad idea. If a build tool can order files correctly, compiler should do it much easier.

if a build tool can order files correctly, compiler should do it much easier.

The build tool can only do it if you provide the hints needed aka module exports / imports. And then the TypeScript compiler can also do it (e.g. --outFile + --module requirejs) :rose:

As an analogy, we don't re-order statements in a function. It's up to you to determine what the correct order of statements in a function is, and it's up to you determine what the correct order of your files is (if matters for your program). In any other mode than folder enumeration, that ordering follows a deterministic algorithm.
I agree that when enumerating files in a folder, that ordering should be deterministic. Logged #8776

Ok, thanks, i can live with deterministic order and references statements.


Add option to disable ELLPACK format/index arrays for GPU training

Currently, XGBoost has a GHistIndexMatrix which stores data in CSR format; however, if the data is completely dense (no missing data such as NaNs), it optimizes this away by not reserving the extra space for the index arrays. This is not true of the EllpackPageImpl, which allocates the space and creates the indices, regardless of whether is_dense is set or not.
We currently use XGBoost to build trees on large amounts of almost completely dense data, which can be quantized to 1 byte per float. This means that the sparsity optimizations actually balloon the size of the memory required, with the GPU memory being used being almost exactly 3 bytes per float, as one would expect when storing dense data in Ellpack. One can observe with the just GHistIndexMatrix that the memory consumption increases by 3-4x when a single NaN is added to the data, making the is_dense false.
The feature request is to simply allow the user to turn off CSR/Ellpack format and store densely with a manual config option in QuantileDMatrix.
I've tried reading the code, and I don't understand it fully, so I can't be of much help with coming up with a roadmap.

For GPU ellpack, it's already done in the master branch. Dense data will use smaller amount of money.

Oh, that's great! Just to confirm, #10870 is the PR that implements this functionality?

Yes. For relatively dense data (as defined in the PR description), XGBoost should be much faster and use smaller memory.


Handle missing logfetch tail data

This PR handles the case in which a logfetch tail response has no 'data' field.

LGTM


DBI::dbRemoveTable not working

Hi
It's not possible to delete tables in R.
Here's a MWE
Step 1: Create a DB
library(dplyr)
library(DBI)
library(duckdb)

con <- DBI::dbConnect(
      duckdb::duckdb(),
      "mtcars.duckdb"
)

copy_to(
    con,
    mtcars,
    "mtcars",
    temporary = FALSE
)

Step 2: Delete the table (doesn't work, the file size remains the same but then the table is unaccessible)
dbRemoveTable(con, "mtcars")

This should be cleaned up on checkpointing, at the moment however, checkpointing only occurs on startup. So if you restart your R shell and reconnect it should free up that storage space. See also #49

This should be cleaned up on checkpointing, at the moment however, checkpointing only occurs on startup. So if you restart your R shell and reconnect it should free up that storage space. See also #49

I tested that and it doesn't work either

Indeed I can confirm the issue of space not being freed. Here is a script that reproduces the issue.
library(DBI)

db_file <- tempfile("duckdb_")

con <- dbConnect(duckdb::duckdb(), db_file)
dbWriteTable(con, "test", data.frame(a=integer(10000000)))
dbDisconnect(con, shutdown=TRUE)

system(sprintf("du -hs %s*", db_file))

con <- dbConnect(duckdb::duckdb(), db_file)
dbRemoveTable(con, "test")
dbDisconnect(con, shutdown=TRUE)

system(sprintf("du -hs %s*", db_file))

con <- dbConnect(duckdb::duckdb(), db_file)
dbListTables(con)
dbDisconnect(con, shutdown=TRUE)

system(sprintf("du -hs %s*", db_file))

Please note though that the storage space is not lost, it can be used by another table.


Anyone have a good client for MultiOgar?

Would be very useful and would help me understand more about clients.

I'm working on the client for MultiOgar. Will be published soon.
Here is screenshot how it looks now:

Thank you very much. This will help me create my own client, with my own features and once I'm done, I will release it. Cheers, and tell me when you're done with it,

@Barbosik can you make it for public servers?

@nuclearc: no, vanilla servers doesn't allow connections from non-vanilla clients

then how i connected once?

Can i get a client please?

I'm working on the client for MultiOgar. Will be published soon.

ok fire! thanks, where will you post it? in the issues? otherwise send it to my email here -<EMAIL_ADDRESS>

^^ Please read the comment above.

i did, but he doesnt say where.

^^ https://github.com/Barbosik/MultiOgar/issues/294#issuecomment-239910902

Yeah i know, cool. but doesnt specify where.

lol

@Barbosik what font are you using for your client?

i dont think its gonna be published :/

then i think i'm gonna make this thingy public:

Nuclear xD I'm gonna a publish the client you and I made and pls answer on
Skype!
On 2 Oct 2016 8:29 a.m., "nuclearc"<EMAIL_ADDRESS>wrote:

then i think i'm gonna make this thingy public:
[image: image]
https://cloud.githubusercontent.com/assets/8341270/19019112/05ef7d50-888b-11e6-9dfa-b327f381b51e.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Barbosik/MultiOgar/issues/294#issuecomment-250956392,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASEASX-hojPcAO9oDNQ8k-VIvvaCk1Iiks5qv082gaJpZM4JkXiP
.


Feature Request: Ability to toggle old text only view

The new graphical view looks nice and i'm certain a lot of work was put into it but it is harder for me to quickly look at the information. Even more so if I try to bring up the site on my phone. Previously, I would be able to see almost every merchant on the screen but now, because of the size of the images, each merchant and the map location takes up the entire screen and i'd have to scroll down one merchant at a time.

I second this. The new cards take so much of the screen I have to look at the cards one at a time to see anything. Before you could only take a glance at the list and see. Really makes me wonder how this was thought as an improvement.

Or enable the cursor hover map view when we choose collapse card view.

The new view is a huge stepback.

It is not possible to see all items on my screen setup now and I have to alt+tab and scroll.
I have to focus too much to find the relevant information. The previous tabular form is much more suitable, because the information is always in the same place.

Unfortunately, since it is a blazor app, there is no easy way to tweak the UI myself.
I would very much welcome if there was a switch that could turn it back to the old form.

Issues with new view:

Takes way more space on screen
Doesn't allow to quickly see not yet submitted locations (opacity/dimming would help)
Visual noise "steals" attention from finding what card/item exactly are available.

Let's be honest, real lostmerchants value is quick and reliable information. Visual appeal is way less essential.
Please consider reddit thread https://www.reddit.com/r/lostarkgame/comments/xz5p34/looks_like_lostmerchanrts_changed_their_layout/ as part of this issue, because not every lostmerchants' user has github account to participate in discussion.
Suggestion:
I would suggest to implement radical UI/UX changes gradually and give users option to switch between old/new layout to see the percentage and how new layout is accepted by audience.

After a couple of hours using it, I have discovered more points that bother me.

The card color is completely meaningless (to me) as I want to buy all items anyway. The color should really represent what I have already cleared. The item inside the card has a color already so there is no point in having it duplicated in the whole card.
I have previously used the "thumb up" button to mark what I have already cleared. This information is now not visible at first glance. As mentioned in point 1, the color of the card should change with my upvote.
Although I already remember most of the spawn positions by heart, I still occasionally need a reminder. When the small cards view is used, this information is completely unavailable, which forces me to use the big cards all the time, just for the 1 case in 20 that I don't remember.

And a side note: My secondary monitor is in vertical orientation, and I used just a half of the screen for lost merchants. Now I have to use the whole screen to see it all, even in collapsed mode.

I think the most annoying part is still information being placed here and there not able to have a quick glance. I liked the previous layout way better.

Same here, don't like the new one. It's eye candy but not very clear to get the information quick. Would love to get an "opt-out" to the old layout.

Unfortunately I have to agree with what others have stated.  I am sure a lot of work went into this, but the old layout was more useful/easier to use.

Please change it back to the previous format, I use fancy zones and it get split vertically to 1/2 of my screen, previously I was able to see everything when it was 1/2 my screen now it is 4 cards and I have to scroll to see and the cards are hard to quickly glance and determine what status of the merchants are.

I just wanted to add, I really love and appreciate lostmerchants.com and use it every day. The new version is honestly so much harder to use for me. Could you please add a toggle to get the old version of the website back for those of us who want it? Pretty please? Thank you!!

Thank you so much for the time/energy you put into this site. A toggle to view the old option would be incredible, it just takes up way to much real estate on my screen to view everything. Thank you again,
appreciate everything you do!!

I agree this request, I wanted to highlight the following comment from the pull request introducing this change.

Yeah, I'll try to sketch something on the weekend for a compact view, as I assume that's the most contested point :)

So seems like it's already on their radar.

Could you give us the choice to go back to the previous view (text only) ? The new view is nice, but I find it less efficient...
Maybe you could at least add a filter to remove uninteresting combinaisons in order to lighten the view ?

Hey folks! Compact mode is almost here, so you can check out the details in
https://github.com/Xeio/WanderLost/pull/162

PS: if there's any designers or other FE folks out there, we could really use your help :)

Furthermore, we added a dedicated issue to tracking the feedback for the new UI and some additional information on what is already addressed and still needs to be looked at -> https://github.com/Xeio/WanderLost/issues/163
Feel free to add any comments / suggestions you have to the linked issue. It helps us to keep it all in one place for better tracking + we will try to keep its description up-to-date whenever a new change / issue is introduced

Hey folks! Compact mode is almost here, so you can check out the details in #162

PS: if there's any designers or other FE folks out there, we could really use your help :)

Thanks for the good news. I personally think spreadsheet/table view is the most efficient way of representing info.
The previous view served this purpose really well; it's just so easy to look at the corresponding column for location after spotting the item I've set to track at a quick glance.
The only thing I didn't like about the old view is when ppl submit fake info(or misclick) and even with tons downvote the wrong info still stays there occupying extra roles. Maybe like the location map we can make the fake info to display only when cursor  hover over for detail?


[2440] Reduce field width of subject fields

Context
https://trello.com/c/GX9OzgHH/2440-s-general-snagging-field-width-for-course-details-subject
Changes proposed in this pull request

Change width of surrounding container using a exisiting ulility class

Before

After

 [S] General snagging - field width for course details => subject


Incorrect height of selecting nodes dialog

What are the steps to reproduce?

Place complete node to the workspace.
Open the node property UI.
Click the "Select nodes..." button.

What happens?
Notification dialog has incorrect height. Therefore, entire buttons are not visible.

What do you expect to happen?
Notification dialog has correct height. The following screenshot is flow editor in Node-RED v1.3.4.

This problem seems to come from the commit between v1.3.4 and v1.3.5.
Please tell us about your environment:

[x] Node-RED version: v2.0.0-beta.1
[x] Node.js version: v16.1.0
[x] npm version: v7.11.2
[x] Platform/OS: Ubuntu 21.04 (WSL2 on Windows 10)
[x] Browser: Google Chrome  91.0.4472.124(Official Build)(64 bit)

As this is broken in 1.3.5, I've cherry-picked the commit into the master branch as well for the next 1.x maintenance release.

Yes, it is also needed for Node-RED v1.3.6. Thank you for your caring.


Unable to export project to macos

Steps to reproduce:

Download https://github.com/rayxuln/Spine-Runtime-For-Godot-Example
Export above project to MacOS using default settings
Try to open exported .dmg and execute app inside.
Project opens and then immediately exits

The above steps work fine on a non-spine project of mine, so it's definitely the custom spine stuff.

i can't get it to return any errors or anything either so unsure of how to provide additional troubleshooting information.

Can u attach a log? Usually created by running in command line: godot > log.txt
Thomas

On Jul 19, 2020, at 8:58 AM, Jonathan Chun<EMAIL_ADDRESS>wrote:

Steps to reproduce:
Download https://github.com/rayxuln/Spine-Runtime-For-Godot-Example
Export above project to MacOS using default settings
Try to open exported .dmg and execute app inside.
Project opens and then immediately exits
The above steps work fine on a non-spine project of mine, so it's definitely the custom spine stuff.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I'm a little confused as to how I'm supposed to be generating a log. Are you talking about the godot editor's log itself? The Godot engine/editor itself exports fine and I see no errors.
Here's the output of ./godot.osx.tools.x86_64 > log.txt
arguments
0: ./godot.osx.tools.x86_64
Current path: /Users/jchun/git/godot/bin
Godot Engine v3.2.2.stable.custom_build.6a9fbafcb - https://godotengine.org
OpenGL ES 3.0 Renderer: AMD Radeon Pro 555 OpenGL Engine

Registered camera FaceTime HD Camera with id 1 position 0 at index 0
Editing project: /Users/jchun/git/Spine-Run-Time-For-Godot-Example (::Users::jchun::git::Spine-Run-Time-For-Godot-Example)
arguments
0: /Users/jchun/git/godot/bin/godot.osx.tools.x86_64
1: --path
2: /Users/jchun/git/Spine-Run-Time-For-Godot-Example
3: --editor
Current path: /Users/jchun/git/godot/bin
Godot Engine v3.2.2.stable.custom_build.6a9fbafcb - https://godotengine.org
OpenGL ES 3.0 Renderer: AMD Radeon Pro 555 OpenGL Engine

Registered camera FaceTime HD Camera with id 1 position 0 at index 0
Exporting to /Users/jchun/Library/Caches/Godot/Spine Run Time Test Project.app
Creating /Users/jchun/Library/Caches/Godot/Spine Run Time Test Project.app/Contents/MacOS
Creating /Users/jchun/Library/Caches/Godot/Spine Run Time Test Project.app/Contents/Frameworks
Creating /Users/jchun/Library/Caches/Godot/Spine Run Time Test Project.app/Contents/Resources
ADDING: Contents/Resources/icon.icns size: 489675
ADDING: Contents/Info.plist size: 1251
ADDING: Contents/PkgInfo size: 9
ADDING: Contents/MacOS/Spine Run Time Test Project size: 43059220
hdiutil returned: created: /Users/jchun/git/Spine-Run-Time-For-Godot-Example/export/Spine Run Time Test Project.dmg

However, when I try to execute the .app file

it just doesn't open. No logs, no errors. I tried opening the .app file itself from the command line using the open command as documented here
https://osxdaily.com/2007/02/01/how-to-launch-gui-applications-from-the-terminal/
and no logs or errors in the terminal are shown either

Did a little bit more testing. Looks like it will launch/run fine even with a SpineSprite node added to a scene. The problem occurs the second I try to create a SpineAnimationStateDataResource
This will crash:

This is fine:

Did you load the spine json? You need load your spine export first.
Sorry, don’t have access to a pc atm so I’m following from phone.
Thomas

On Jul 19, 2020, at 10:31 AM, Jonathan Chun<EMAIL_ADDRESS>wrote:

Did a little bit more testing. Looks like it will launch/run fine even with a SpineSprite node added to a scene. The problem occurs the second I try to create a SpineAnimationStateDataResource
This will crash:
This is fine:
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

to clarify, it works when the spine json is NOT loaded. I'm simply proving that there's an issue with SpineSprite loading. If I have anything loaded in there, it crashes on start.

Thank you. Unfortunately I don’t have access to a Mac. I’ll try this on windows to see if I can reproduce there.
Thomas

On Jul 19, 2020, at 11:24 AM, Jonathan Chun<EMAIL_ADDRESS>wrote:

to clarify, it works when the spine json is NOT loaded. I'm simply proving that there's an issue with SpineSprite loading. If I have anything loaded in there, it crashes on start.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

great, thanks. I have 2 macbooks and have only tested it on one, so i'll see if I can test it on my other one later.
I did try exporting to HTML5 as well to see what happens and get the following:

This makes me think perhaps my problem is that I'm exporting using a default template and I need to be exporting using a custom one? At least in the WASM export, i see that it thinks it's running "Godot Engine v3.2.2.stable.official" even though I definitely built it using the custom build with the spine module.

I've never actually done this before so very possible I'm just exporting incorrectly and need some guidance on how to do this. Because the project is so new right now it's pretty lacking and might be a good chance for us to come up with something.

Wait, you’re using the default template?
You have to build custom templates with the spine runtime to make spine work and use those. Otherwise the exported godot does not know what to do with the spinesprite.
It should say stable.custom_build or something like that.
Thomas

On Jul 19, 2020, at 12:10 PM, Jonathan Chun<EMAIL_ADDRESS>wrote:

great, thanks. I have 2 macbooks and have only tested it on one, so i'll see if I can test it on my other one later.
I did try exporting to HTML5 as well to see what happens and get the following:
This makes me think perhaps my problem is that I'm exporting using a default template and I need to be exporting using a custom one? At least in the WASM export, i see that it thinks it's running "Godot Engine v3.2.2.stable.official" even though I definitely built it using the custom build with the spine module.
I've never actually done this before so very possible I'm just exporting incorrectly and need some guidance on how to do this. Because the project is so new right now it's pretty lacking and might be a good chance for us to come up with something.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks. That must be what I'm missing. I'll do some googling to figure out how to do that. Yes, I was using the default template and didn't realize there was a missing step.

If you built the editor, you can also easily build the template for macOS. Just execute the same build command but add
tools=no target=release_debug
for debug template and
tools=no target=release
For prod template
This way you can test quickly
HTML template is more involved and required installation of nodejs etc. More info on the godot compile documentation.
Thomas

On Jul 19, 2020, at 1:03 PM, Jonathan Chun<EMAIL_ADDRESS>wrote:

Thanks. That must be what I'm missing. I'll do some googling to figure out how to do that. Yes, I was using the default template and didn't realize there was a missing step.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks -- I'm following this guide here: https://godotforums.org/discussion/20550/how-to-create-export-templates right now. I'll play around with the HTML stuff perhaps later, but just want to try getting a normal export out first.
I'll close this issue out early because everything is pointing towards just my inexperience with the exporting process here. I'll re-open if I still have issues.


[13pt] FIX: PROJ: proj_create_from_database: Cannot find proj.db

Earlier the error PROJ: proj_create_from_database: Cannot find proj.db showed up in just a few places. We put in a partial fix in shared_functions.py -> fix_proj_path_error.
After upgrading some packages, the error is showing up everywhere.  Investigate and fix or at least see if we can propgate the fix_proj_path_error.
Note: That function does assume a specific pathing that might be the same on all machines. It will need an upgrade.
Maybe we can even surpress the error ???

Tried a ton of experiments and this is now showing up quite a bit and needs to be solved. Especially with linting and upgrading packages.
I put some notes in the shared_functions.fix_proj_path_error


Post Install Cloud Firestore Rules Note Error

The rules in the post install instructions give the following error:
Error saving rules - Line 4: Forward slash '/' found where identifier or binding expected.; Line 4: Missing 'match' keyword before path.; Line 4: Unexpected '$'.; Line 4: Unexpected '$'.; Line 4: mismatched input 'param' expecting {'{', '/', PATH_SEGMENT}; Line 4: Missing 'match' keyword before path.; Line 5: mismatched input 'allow' expecting {'{', '/', PATH_SEGMENT}; Line 18: Unexpected 'match'.
I'm not sure if it is an error with the rules or if they need to be updated for a newer version of Firestore.

Same from the Firebase Console > Firestore Database > Rules

...following my previous answer, I think by default these should be "customers" and "products":

...would become:

Hi @rutherfordlabs
We're these rules copied from Github?
This document will appear updated on the extensions console:

Is this still an issue, perhaps a note explaining where these are updated would help?


NilLiteral.nil? return false

Hi,
As the title says:
{% puts nil.nil? %}

false

I am not sure this is a bug, NilLiteral is not nil, but it feels weird.
Tested with 0.36.1, as 1.0.0 is not yet available on playgrounds.

The parser transforms nil.nil? to nil.is_a?(Nil). In regular code that is correct and works as expected. But in the macro language the type of nil is NilLiteral and .is_a?(Nil) returns false.
I don't think .nil? was ever intended for the macro language. It just happens to be accepted by the compiler because of the implicit transform to IsA.
We could fix the behaviour or let .nil? be undefined in the macro language.
It's not really necessary there. The reason for .nil? in the regular language is to provide a reliably way for the compiler to identify nil types without the equality operator which can be overriden in user code.
But in the macro language that's not an issue anyways, so every x.nil? could just be written as x == nil.
It doesn't seem that .nil? is used that much in macro expressions anyways, otherwise it should have long been noticed that it's completely broken.