[SPARK-28187][build] Add support for hadoop-cloud to the PR builder.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12176/
Test PASSed.

Test build #106974 has started for PR 24987 at commit 2012782.

Test build #106974 has finished for PR 24987 at commit 2012782.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106974/
Test PASSed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12178/
Test PASSed.

Test build #106975 has started for PR 24987 at commit 2012782.

Merging to master.

Test build #106975 has finished for PR 24987 at commit 2012782.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106975/
Test PASSed.


Support smooth scrolling

Adds a stub to allow directly pulling smooth scroll events from reason-sdl2

Awesome! Great to see this making its way in - thanks @szbergeron !

Depends on changes in https://github.com/revery-ui/esy-sdl2/pull/6, https://github.com/revery-ui/reason-sdl2/pull/25

Awesome! Great to see this making its way in - thanks @szbergeron !
Thanks, happy to help :smiley:

/azp run

@bryphe is running CI intended to be a privileged action?

/azp eun

/azp run

Bunch of funky stuff seems to be going on with CI for this

Hygiene check fails because of formatting. Run esy format locally, commit and push to fix. Windows is still working and the others succeeded, but with "issues" that are normal as far as I'm aware.

Running esy format locally doesn't seem to change anything, not sure why the check failed

/azp run

/azp run

Running esy format locally doesn't seem to change anything, not sure why the check failed

Might be different version

This includes a lot of unrelated changes, which makes it really hard to review. Do you think you could try a rebase?

Oh whoops, figured it would reconcile. I can try rebasing but not sure if that would help.

Might make more sense to just revert the top two commits and remake them

An interactive rebase allows you to do pretty much that.

I seem to have gotten myself in a bit of a pickle trying to do it with rebasing, lots of conflicts and none of them make much sense :woman_shrugging:

That happens sometimes. Just git rebase --abort and try again.
The way I think about a rebase is that it replays a set of commits onto the HEAD of the branch or ref you specify. Typically the set set of commits is every commit in your branch from the most recent common ancestor, but you can select the commits yourself by doing an interactive rebase with git rebase -i.
When you run git rebase -i  you'll get a list of commits in your editor. Delete every line with a commit  that should already be in the new base, ie. that isn't yours and aren't merge commits. Save and quit. You'll most likely still get conflicts, but they should hopefully be relatively small and straight forward to fix since it's just your own changes.
Hope that helps! If not, let me know and I'll try to guide you as best I can.

/azp run

Looks much better! Didn't realize the change was this tiny though. It really would have been much easier to just copy the changes out and back in after resetting. But hopefully you learned something new!

Something's up with the Linux CI:
Failed to fetch https://packages.microsoft.com/ubuntu/16.04/prod/dists/xenial/main/binary-amd64/Packages.bz2  Hash Sum mismatch

@bryphe?

I think this is obsolete now and is anyway superseded by libscroll integration, so closing while I can get libscroll patch at https://github.com/revery-ui/revery/pull/766 working.


Explore further performance improvements in S3 datatools

I haven't tested the latest code. Will do that tomorrow but wanted to push it out to get eyes on it.

I haven't tested the latest code. Will do that tomorrow but wanted to push it out to get eyes on it.


Add logging for normal Bind errors

This simply differentiates a broker failure response as outlined in the
spec versus some other unknown error.

Needs test.

It looks like at present an unbind request should also be sent in the unexpected error case, but I know that orphan handling is something that is in review now and won't be forgotten.

This now depends on PR https://github.com/kubernetes-incubator/service-catalog/pull/1000


Entity Manager does not configure persists operation after create it

Hello people.
I have the follow issue:

First I check that the entity manager is open
if it is not open I log the error and create a new one
after this I thry to persist a entuty with cascade persists operations but the new entity manager have not configured this operations.

the code:
/**
     * @return EntityManager
     */
    public function getManager($errorContext = [])
    {
        /** @var EntityManager $manager */
        $manager = $this->doctrine->getManager();

        if (!$manager->isOpen()) {

            $errorContext = array_merge(['stack_trace' => debug_backtrace()], $errorContext);

            $this->logger->error('Manager is not connected to database - Reopening connection', $errorContext);

            $manager = $manager->create(
                $manager->getConnection(),
                $manager->getConfiguration()
            );
        }

        return $manager;
    }

The exception:
A new entity was found through the relationship 'Pet\Food#food' that was not configured to cascade persist operations for entity: Food@000000004aacae7600007fdbf3fa2867. To solve this issue: Either explicitly call EntityManager#persist() on this unknown entity or configure cascade persist this association in the mapping for example @ManyToOne(..,cascade={"persist"}). If you cannot find out which entity causes the problem implement 'Food#__toString()' to get a clue.

Some idea?
Thanks!

That's because you probably have an entity managed by a different entity manager instance in the mix. You should really just clear everything and retry the entire business interaction, instead of relying on existing in-memory entities.

Closing, since this is not an ORM issue, but a question.

@Ocramius right, there is the issue, thanks for the support :+1:


Add nanmean_along_batch to BatchedTensor

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (56a1063) 100.00% compared to head (ee576fb) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #226   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          913       915    +2
  Branches        81        81
=========================================
+ Hits           913       915    +2

Impacted Files
Coverage Δ

src/redcat/tensor.py
100.00% <100.00%> (ø)

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


calling sync_prerecorded fails with ssl.SSLEOFError: EOF occurred in violation of protocol (_ssl.c:2396)

What is the current behavior?
    dg_client = Deepgram(DEEPGRAM_API_KEY)

    with open(PATH_TO_FILE, 'rb') as audio:
        source = {'buffer': audio, 'mimetype': MIMETYPE}
        options = { "punctuate": True, "model": "base"}
        response = dg_client.transcription.sync_prerecorded(source, options)

What's happening that seems wrong?

fails with
Traceback (most recent call last):
  File "/opt/homebrew/lib/python3.10/site-packages/deepgram/_utils.py", line 139, in attempt
    with urllib.request.urlopen(req) as resp:
  File<EMAIL_ADDRESS>line 216, in urlopen
    return opener.open(url, data, timeout)
  File<EMAIL_ADDRESS>line 519, in open
    response = self._open(req, data)
  File<EMAIL_ADDRESS>line 536, in _open
    result = self._call_chain(self.handle_open, protocol, protocol +
  File<EMAIL_ADDRESS>line 496, in _call_chain
    result = func(*args)
  File<EMAIL_ADDRESS>line 1391, in https_open
    return self.do_open(http.client.HTTPSConnection, req,
  File<EMAIL_ADDRESS>line 1351, in do_open
    raise URLError(err)
urllib.error.URLError: <urlopen error EOF occurred in violation of protocol (_ssl.c:2396)>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/mikhailkuznetcov/Developer/speech-rec-python/deepgram_version.py", line 36, in <module>
    main()
  File "/Users/mikhailkuznetcov/Developer/speech-rec-python/deepgram_version.py", line 28, in main
    response = dg_client.transcription.sync_prerecorded(source, options)
  File "/opt/homebrew/lib/python3.10/site-packages/deepgram/transcription.py", line 355, in sync_prerecorded
    return SyncPrerecordedTranscription(
  File "/opt/homebrew/lib/python3.10/site-packages/deepgram/transcription.py", line 111, in __call__
    return _sync_request(
  File "/opt/homebrew/lib/python3.10/site-packages/deepgram/_utils.py", line 153, in _sync_request
    return attempt()
  File "/opt/homebrew/lib/python3.10/site-packages/deepgram/_utils.py", line 148, in attempt
    raise (Exception(f'DG: {exc}') if exc.status < 500 else exc)
AttributeError: 'URLError' object has no attribute 'status'

Steps to reproduce

To make it faster to diagnose the root problem. Tell us how can we reproduce the bug.

Expected behavior

transcribe the file

What would you expect to happen when following the steps above?

Please tell us about your environment
Python:

We want to make sure the problem isn't specific to your operating system or programming language.

Operating System/Version: Mac M1
Language: Python

Other information

Anything else we should know? (e.g. detailed explanation, stack-traces, related issues, suggestions how to fix, links for us to have context, eg. stack overflow, codepen, etc)

Were you using Docker? If so, which image were you using?

This is happening to me too in Macos Ventura 13.4.1 (c) (22F770820d)
The weird thing is the first time I ran the example, everything went well. Now that I wanted to re run it, I'm receiving this error.

It happens here sometimes too. Linux mint 20.

I got the same error (Ubuntu, Python 3.11).
A quick way to reproduce this is to send a prerecorded request to a whisper model with the keywords=Whisper parameter. Because the whisper models (veeeeeeeheeeery unfortunately ;)) do not support the keywords feature, the API will return a 400:
{
	"err_code": "INVALID_QUERY_PARAMETER",
	"err_msg": "When using Whisper, keywords are not available.",
	"request_id": "..."
}

When you do this async with response = await dg_client.transcription.prerecorded(source, options), an exception is raised that at least hints you into the right direction:
Traceback (most recent call last):
...
aiohttp.client_exceptions.ClientResponseError: 400, message='Bad Request', url=URL('https://api.deepgram.com/v1/listen?model=whisper-medium&keywords=Whisper')

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
...
Exception: DG: 400, message='Bad Request', url=URL('https://api.deepgram.com/v1/listen?model=whisper-medium&keywords=Whisper')

However, when you do this sync with response = dg_client.transcription.sync_prerecorded(source, options), the above mentioned and misleading exception is raised:
Traceback (most recent call last):
...
ssl.SSLEOFError: EOF occurred in violation of protocol (_ssl.c:2423)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
...
urllib.error.URLError: <urlopen error EOF occurred in violation of protocol (_ssl.c:2423)>

The expected result would be that the same thing happens for the client user when hitting a 400, whether the call was made async or sync.
Also I'd like to add two personal preferences:

I would prefer it if there was an option to return the error response instead of raising an exception, so that I can check the status code and a potential error message of the response on my own.
The client library doesn't implement their own exception classes here, but instead raises the most basic Exception class with a more or less cryptic message:  raise (Exception(f'DG: {exc}') if exc.status < 500 else exc). For convenient error handling in downstream applications, most client libraries implement custom exceptions, with well differentiated speaking class names. The current implementation forces the client user to except Exception, which will (like a bare catch) subsume every possible exception that could occur.

@dvonthenen  👋

Hi @shershen08
The Python SDK is going through a major overhaul right now. If you had a moment to check against the code in main branch, that would be helpful. I'm hoping at least minimally, the exception message is significantly improved. We can start with that and go from there.

Hi @shershen08. The alpha.5 release is out. Can you please retry with this release and see if you run into the same problem?

Closing due to inactivity. If there is still a problem with the latest release, please reopen or file a new issue. thanks!


Peek - can't view PDF/HTML soft links

Microsoft PowerToys version
0.72.0
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
Peek
Steps to reproduce

Creat a PDF. (For example, save this web page as PDF)

Create a soft link targeting it.
One way to do so is PowerShell:
New-Item -Type SymbolicLink -Target original.pdf -Path link.pdf

Open the Explorer, Peek at the link.

✔ Expected Behavior
The PDF is shown.
❌ Actual Behavior
The peek window appears, but the file does not.

Unable to open the file
Something is wrong.
[Refresh the page]

Other Software
MS Edge 115.0.1901.203
Additional information

It's a PDF-only issue: texts (*.txt), images (*.jpg) and markdown (*.md) will work.
If you peek at *.html, an empty window will be shown, but there will be no error pop-up (as for *.pdf).
Hard links work.
Pasting file URIs into MS Edge behaves similarly. (Note: If you double click the soft link (open with MS Edge), the file URI would be the target.)

Relevant but not identical: #2527

Must be the "browser" control (using WebView2, iinm)

Thanks, I didn't know it relates to WebView2 before. So I make a few more experiments (based on the source code): soft/hard links × pdf/md/html × peek / double click / paste file URI into MS Edge.

All hard links work.
Pasting file URIs into MS Edge behaves similarly.

(I've updated Additional information above)

Are you still experiencing this issue? /needinfo

Yes, I have just confirmed in v0.85.1. (✅ = confirmed, ❓ = not tested)

✅ The main issue still occurs with exactly same behaviour.
✅ It's a PDF-only issue: texts (*.txt), images (*.jpg) and markdown (*.md) will work.
✅ If you peek at *.html, an empty window will be shown, but there will be no error pop-up (as for *.pdf).
✅ Hard links work.
❓ Pasting file URIs into MS Edge behaves similarly. (Note: If you double click the soft link (open with MS Edge), the file URI would be the target.)


Wrong translations in CzechRepublic holidays

Important specifications

Czech Republic
http://svatky.najdise.cz/statni-svatky-2018

Description
You have wrong naming holidays in czech language. Concrete in 8. May and, there you have only "Den vítězství or Den osvobození", but should be "Den vítězství - Den osvobození od fašismu".
Next wrong translation is from "Easter Sunday", there you have only "Velikonoční", but should be "Velikonoční neděle".

Thanks for the info have you a better source for the informations? wikipedia or goverment page?

I am from Czech Republic. There is more sources for example you can look into https://publicholidays.cz/cs/2018-dates/ .


Fix Error serving documents with Django Cache Middleware enabled [#983]

Changes to fix #990:

changed the import to from wsgiref.util import FileWrapper
updated the view to return a StreamingHttpResponse

There appears to not be any tests for the docs serve view. I'll quickly add some

@kaedroho good catch. On a related note, I spoke with @gasman who was not sure there was a good way to test this particular issue.

It's probably possible by running a test with the cache middleware enabled. But I think just checking that the response is streaming is enough as Django would never cache those.

Should I close this in favor of #991?

@kaedroho Yep, that's a fair point... as a unit test, the thing we really care about is "the response correctly identifies itself as a streaming one to any middlewares that might care about that fact", not "the cache middleware doesn't crash" - so adding AssertTrue(response.streaming) (which is the flag that cache middleware actually checks for) should do it.

@zerolab
If you merge #991, this ticket should automatically mark itself as merged as well.


Account Transaction : Init Contract

Purpose
Added support for Account Transaction - Init Contract
Changes
Added support for Account Transaction - Init Contract
Checklist

[x] My code follows the style of this project.
[x] The code compiles without warnings.
[x] I have performed a self-review of the changes.
[x] I have documented my code, in particular the intent of the
hard-to-understand areas.
[x] (If necessary) I have updated the CHANGELOG.

CLA acceptance
By submitting the contribution I accept the terms and conditions of the
Contributor License Agreement v1.0

link: https://developers.concordium.com/CLAs/Contributor-License-Agreement-v1.0.pdf

[x] I accept the above linked CLA.

Closing as this is part of https://github.com/Concordium/concordium-java-sdk/pull/146


Merge 3.43

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Patch coverage has no change and project coverage change: -19.78 :warning:

Comparison is base (4ea46f5) 64.35% compared to head (cd27fa4) 44.57%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #653       +/-   ##
===========================================
- Coverage   64.35%   44.57%   -19.78%
===========================================
  Files         226      152       -74
  Lines       21781    15614     -6167
===========================================
- Hits        14017     6960     -7057
- Misses       6998     8334     +1336
+ Partials      766      320      -446

Flag
Coverage Δ

1.17.x
44.20% <ø> (-19.81%)
:arrow_down:

1.19.x
44.64% <ø> (-19.46%)
:arrow_down:

1.20.x
44.59% <ø> (-19.44%)
:arrow_down:

insecure
?

integration
?

macOS-latest
44.57% <ø> (+0.03%)
:arrow_up:

secure
?

ubuntu-latest
44.57% <ø> (-19.75%)
:arrow_down:

unit
44.57% <ø> (-0.10%)
:arrow_down:

windows-latest
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 172 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Update session7-report-template.md

san

Is this really needed? If it is needed why it is finglish?

I couldn't find your ID in the class. Have you audited? I wanted to give you extra points because of your contributions but apparently you haven't audited.


how to use with bottom navigation

i want to use compose destinations with bottom navigation. Help :(

It’s really not that different than using official library.
Check sample project, for BottomBar file and check this:
https://composedestinations.rafaelcosta.xyz/common-use-cases/bottom-bar-navigation

Let me know below how that goes 🙂


Please change the text from "機能" to "Functions".

Describe the bug
Please change the text from "機能" to "Functions" on the left navigation.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://docs.microsoft.com/ja-jp/learn/browse/'
Click on 'Products' on the left navigation.
Scroll down to '機能'
See error. Please do not translate it because It is a service name of Azure.

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [Windows 10 Enterprise Edition]
Browser [chrome]
Version [69.0.3497.100]

Additional context
Add any other context about the problem here.

@SonjaSaltzman  - Can you take a look at this?

@SunnyDeng could you please address & close?

@ishasahni1 assigning to you

Hi @ishasahni1 The issue does not seem to repro any longer, as you can see in SS below:

Also, please remember to assign the issues to me so we can import them and work on the accordingly.
@SonjaSaltzman @Sara-Nicolini FYI


TEN-129 amended percentage to read correctly

https://github.com/hmrc/app-config-staging/pull/10508
https://github.com/hmrc/app-config-qa/pull/15111
https://github.com/hmrc/app-config-production/pull/14653

amended


Refactor obstacle framework in Open-RMF

Before proceeding, is there an existing issue or discussion for this?

[X] I have done a search for similar issues and discussions.

Description
This is a meta ticket that broadly summarizes improvements for the obstacle detection pipeline in Open-RMF.
Goal: Report and react to obstacles that impede mobility within the Open-RMF framework.
Background

Sources for reporting obstacles

2D vision systems: CCTVs, RPi Cameras, thermal cameras, etc
3D vision systems: stereo cameras, depth sensors, etc
2D/3D lidars: stationary, on robots, etc

Types of reactions

Replan robot routes
Close down lanes
Alert users
Update speed limits for robots

Checklist

[ ] rmf_obstacle_msgs

[ ] Replace our BoundingBox3D with an BoundingBox3D message from vision_msgs
[ ] Refactor Obstacle.msg to support obstacles represented in various forms. It currently has fields for BoundingBox3d bbox and serialized data. We could ideate other data ways to communicate obstacle presence (OccupancyGrid, octomap, bonxai, etc.) It might be better to also add a field and enum constants for users to indicate the type of obstacle populated in the msg.
[ ] A new message that the rmf_obstacle_server could publish with collated information on all the obstacles per level in the building. An OccupancyGrid field within the message may be helpful.

[ ] rmf_fleet_adapter

[ ] Add API to FleetUpdateHandle which enables/disables ability for an external node to close/reopen lanes over the lane closure topic. Similarly for speed limits.
[ ] Add an API to RobotUpdateHandle to report the presence of obstacles. This information can be published to the rmf_obstacle_server which can then publish response commands to other fleet adapters.

[ ] rmf_obstacle_ros2

[ ] Add pure abstract base classes for

[ ]  ObstacleBehavior with methods

init: To initialize the plugin with a ROS 2 node reference to create any pub/subs needed
obstacle_cb: To reach to obstacle infrormation

[ ] ObstacleReporter with methods

init: To initialize the plugin with a ROS 2 node reference to create any pub/subs needed
report: To report obstacle.

[ ] rmf_obstacle_reporter node that

initializes a publisher to publisher over /rmf_obstacles.
loads and initializes base classes of ObstacleReporter type using pluginlib.

[ ] Convert all existing rmf_X_detector packages in this repo into plugins for rmf_obstacle_reporter.
[ ]  rmf_obstacle_server

[ ] Create a new node that subscribes to /rmf_obstacles published by various "reporting" nodes, collate the information per level, and publish the unified data.
[ ] Have the node rely on pluginlib to load plugins of ObstacleBehavior base class.
[ ] Convert existing lane_blocker_node to a plugin that can be loaded by rmf_obstacle_server.

[ ] rmf_visualization

[ ] Switch to subscribing to collated obstacle information published by rmf_obstacle_server and visualize obstacles depending on which level is active in the view.
[ ] Visualize other types of obstacles (not just BoundingBox3D)

[ ] rmf_demos

[ ] Showcase rmf_obstacle capabilities in a demo world.

[ ] Bonus:

[ ] A navigation2 costmap plugin that subscribes to this msg and updates the global costmap of the robot.
[ ] Have mapf receive occupancy information as well.

To discuss any of the implementations further, please use this Discussion ticket


video memory .png file sprite -- Convert to xxxx2D error

Godot version:
3.2.4 beta3
OS/device including version:
win10/rx5700
Issue description:

when i choose Sprite -- click Sprite menu -- Create xxxxxx2D -- , the following error occur:

Steps to reproduce:
when i set .png -- Import -- Compress -- Lossless, everything is ok, but Video RAM not.

@denghe Please upload a minimal reproduction project to make this easier to troubleshoot.


Question: How to create sub sitemaps?

I want to create a sitemaps that has sub sitemaps. Is it possible to create sub sitemaps?

We have 2 examples in README:

https://github.com/ekalinin/sitemap.js#example-of-sitemap-index-as-string
https://github.com/ekalinin/sitemap.js#example-of-sitemap-index

These examples no longer exist. Anywhere else?

@MarcGodard https://github.com/ekalinin/sitemap.js#building-just-the-sitemap-index-file or right below it is the example that auto generates the sitemaps and index.

@derduher thanks, I did find that after. I was more looking for a way to do both and couldn't find it (a sitemap that has urls and other sitemaps). After I found what you mentioned, I changed my strategy and used what you mentioned and separated the other urls.


Docx into SQL Server table

Would it be possible with this tool to parse an existing Docx document and grab its data to send to a SQL Server database?

@brightworks Assuming "this tool" means the Open XML SDK, you can absolutely do this. This obviously assumes that you know what "its data" is and how you want to map that to the database.
For example, if you wanted to create one record per paragraph and have its contents (e.g., inner text without formatting) written to one column, you can do that. Your definition of "its data" obviously determines the amount of processing required to extract and generate what you want.

Thank you for the quick response. This is really helpful.

As a "warning": The OpenXML element "graph" and the different parts can be overwhelming at the beginning. For such "extraction" scenarios we often use the FlatOpc-Format. The flat opc is basically a "flat" xml representations of all parts.
Now you could just use a XPath to select your desired content.

@robertmuehsig made a good point. There is no single approach to extracting data from an Open XML document. Depending on what exactly you want to extract, you would be looking at a single Open XML part, e.g., the MainDocumentPart, or multiple such parts, e.g., additionally multiple header and footer parts, style definitions part, numbering definitions part, settings part, etc. The Flat OPC format makes it easier to process those parts, e.g., using XSLT or Linq to XML. However, if all you are interested in is contained in the MainDocumentPart, you don't need the Flat OPC format.
Regardless of the approach you choose, you would always use the Open XML SDK to create a WordprocessingDocument, e.g., through:
using (WordprocessingDocument wordDocument = WordprocessingDocument.Open(path, true))
{
  [...]
}

Based on that, you can then create a Flat OPC XDocument or string, using
XDocument flatOpcDocument = wordDocument.ToFlatOpcDocument();

or
string flatOpcString = wordDocument.ToFlatOpcString();

The XDocument and string both represent the whole WordprocessingDocument with all its parts, some or many of which might not be relevant in your use case, though.
Alternatively, you can work with individual Open XML parts and their contents. If you want to do that, you can use two approaches. Firstly, you can work with the strongly typed classes (e.g., Document, Body, Paragraph, Run, Text) provided by the Open XML SDK. Secondly, you can use the Open XML PowerTools and Linq to XML. In both scenarios, however, you would use the Open XML SDK to get hold of the Open XML Parts, e.g., the MainDocumentPart:
MainDocumentPart part = wordDocument.MainDocumentPart;

Let's assume the following simplified XML in the MainDocumentPart. In Microsoft Word, that is a single "Hello World!" paragraph formatted using the built-in "Normal" style (which is implicit).
<w:document xmlns:w="...">
  <w:body>
    <w:p>
      <w:r>
        <w:t>Hello World!</w:t>
      </w:r>
    </w:p>
  </w:body>
</w:document>

Using the strongly typed classes, you could do something like this:
Document document = part.Document;  // Get root element
Text text = document.Descendants<Text>().First();  // Get text element;
string value = text.Text;  // Get content, i.e., "Hello World!"

You might have noticed the W.t. This is an XName defined representing the w:t XML element. Thus, where the Open XML SDK provides classes (e.g., Text) for those XML elements (e.g., w:t), the Open XML Power Tools define XNames (e.g., W.t).
Note that all of the above approaches operate on a very low level of abstraction and you will need to understand the Open XML format very well in all cases. Some things that are very easy and straightforward when using Word automation can be surprisingly hard and complex on the Open XML level. However, working on the Open XML level is often one or two orders of magnitude faster than using Word automation.
I am using all of the above approaches in my development. I would definitely recommend you have a look the Open XML PowerTools, because they contain some handy tools, e.g., the MarkupSimplifier, which let you preprocess the XML before doing your own extraction. Not all of the XML contained in those parts will be relevant for you but can get in the way of your extraction logic.

@brightworks Since this question appears to have been answered, I'm going to close this issue. Please reopen if you feel like there is an issue with OpenXml that needs to be addressed.


DisplayResolution name localization is cached and will always display in the first loaded locale

Describe the bug
When I change UI language in the CMS the display resolution name remains in the language it was first loaded in. The value is cached inside the DisplayResolutionBase.Name property.
To Reproduce
Steps to reproduce the behavior:

Add a translation for the resolutions inside the src/Foundation/lang/Display_EN.xml localization file.
Start foundation, log in.
In the CMS page editor, see the resolution name are in language A (probably english)
Change the UI language to language B.
Go back to see the resolution name did not change to language B and remains language A.

Expected behavior
Resolution names should be localizable
Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: 101

Merged, thanks for the PR!


Accept and prioritize params passed in the body of post requests for …

Following a conversation related to this issue regarding passing parameters in the body, I have created this pull request.
https://discourse.drone.io/t/allow-passing-custom-parameters-in-request-body-instead-of-as-query-params/7486
Please forgive me if I have made any silly mistakes, It is my first time using golang and my testing was done in a sandbox project with a simple server and postman because I wasn't exactly sure how to set up the environment and test it inside the drone environment.

 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.

I was thinking something like this:
-for key, value := range r.URL.Query() {
+r.ParseForm()
+for key, value := range append(r.URL.Query(), r.Form...) {

I was thinking something like this:
+r.ParseForm()
+for key, value := range append(r.URL.Query(), r.Form...) {
-for key, value := range r.URL.Query() {

I tried using append the way you suggested but it didn't work because the returned value is a map for r.URL.Query() and r.Form. I also changed the condition to handle both Form data and JSON.
Let me know if this is OK.

This PR has been automatically closed due to inactivity.


Blade Rush doesn't attack when using Gather Power

When using Gather Power (move action) and Blade Rush (standard action) the result is the character does the 30' move but then doesn't attack at the end of it. If Gather Power is turned off the ability works as expected.

Addendum: Blade Rush now simply does not trigger a KB attack under any circumstances that I can recreate.


Az login - JavaScript is required to sign in

Describe the bug
az login causes command to hang in bash shell

To Reproduce
Run az login from bash on Windows.
Expected behavior
az login to prompt for devicelogin
Environment summary
uday@DESKTOP-RGMO1N7:~$ az --version
azure-cli (2.0.41)
acr (2.0.28)
acs (2.1.3)
advisor (0.5.1)
ams (0.1.1)
appservice (0.1.36)
backup (1.1.2)
batch (3.2.4)
batchai (0.3.1)
billing (0.1.8)
cdn (0.0.14)
cloud (2.0.15)
cognitiveservices (0.1.13)
command-modules-nspkg (2.0.1)
configure (2.0.17)
consumption (0.3.1)
container (0.2.1)
core (2.0.41)
cosmosdb (0.1.21)
dla (0.1.0)
dls (0.0.22)
dms (0.0.1)
eventgrid (0.1.12)
eventhubs (0.1.3)
extension (0.1.0)
feedback (2.1.3)
find (0.2.11)
interactive (0.3.25)
iot (0.1.21)
keyvault (2.0.23)
lab (0.0.22)
maps (0.2.0)
monitor (0.1.8)
network (2.1.5)
nspkg (3.0.2)
policyinsights (0.1.0)
profile (2.0.27)
rdbms (0.2.5)
redis (0.2.14)
reservations (0.2.1)
resource (2.0.32)
role (2.0.26)
servicebus (0.1.2)
servicefabric (0.0.12)
sql (2.0.27)
storage (2.0.35)
vm (2.0.35)
Extensions:
dev-spaces-preview (0.1.6)
storage-preview (0.1.3)
Python location '/opt/az/bin/python3'
Extensions directory '/home/uday/.azure/cliextensions'
Python (Linux) 3.6.1 (default, Jul  2 2018, 18:43:15)
[GCC 4.8.4]
Additional context
Add any other context about the problem here.

By default az login opens up a browser for you to authenticate yourself. If in your environment the condition is not fit you can use the command az login --use-device-code to fall back to a text-based interactive login.

Many uses bash on Windows and this new default breaks authentication flow.

@uday31in, az login should work with bash on window, which CLI has particular logics for that. Can you run uname -r and share with me the output?

please reactivate with more info


New builders suffixed by OrNull and OrThrow for NotBlankString

Description
Like discussed in #104, we would like to introduce experimental builders for the NotBlankString type that should:

return null in case of a failure if the builder is suffixed by OrNull
throw an IllegalArgumentException in case of a failure if the builder is suffixed by OrThrow.

These builders should work on all platforms and should be declared in the NotBlankString.kt file after the toNotBlankString function.
Their tests should be declared in the NotBlankStringTest class after the string_toNotBlankString_should_fail_with_a_blank_String function.
Checklist

[x] Follow the Issue implementation section in the contributing guidelines until the Checking the API binaries section.
[x] Copy the Kotlin code below and paste it in the NotBlankString.kt file.

Function signatures
/**
 * Returns this string as a [NotBlankString], or returns `null` if this string
 * is [blank][String.isBlank].
 *
 * Here's some usage examples:
 *
 * ```kotlin
 * var result: NotBlankString? = "hello world".toNotBlankStringOrNull()
 * println(result) // hello world
 *
 * result = "  ".toNotBlankStringOrNull()
 * println(null) // null
 * ```
 *
 * You can use the [toNotBlankStringOrThrow] function for throwing an
 * [IllegalArgumentException] instead of returning `null` when this string is
 * [blank][String.isBlank].
 */
@ExperimentalTextApi
@ExperimentalSinceKotoolsTypes("4.4")
public fun String.toNotBlankStringOrNull(): NotBlankString? {
    TODO()
}

/**
 * Returns this string as a [NotBlankString], or throws an
 * [IllegalArgumentException] if this string is [blank][String.isBlank].
 *
 * Here's some usage examples:
 *
 * ```kotlin
 * var result: NotBlankString = "hello world".toNotBlankStringOrThrow()
 * println(result) // hello world
 *
 * "  ".toNotBlankStringOrThrow() // IllegalArgumentException
 * ```
 *
 * You can use the [toNotBlankStringOrNull] function for returning `null`
 * instead of throwing an [IllegalArgumentException] when this string is
 * [blank][String.isBlank].
 */
@ExperimentalTextApi
@ExperimentalSinceKotoolsTypes("4.4")
public fun String.toNotBlankStringOrThrow(): NotBlankString {
    TODO()
}

[x] Check your changes in the generated documentation.
[x] Check your changes in the public API binaries and update them.
[x] Copy the following Markdown code block and paste it in the Unreleased section in the CHANGELOG, replacing the YourGitHubProfile placeholder by your GitHub profile (see an example here).

Entry in changelog
### Added

**Experimental** builders suffixed by `OrNull` and `OrThrow` for the following
types:
- `NotBlankString` (issue [#174] implemented by [@YourGitHubProfile](#)).

[#174]: https://github.com/kotools/types/issues/174

[x] Commit your local changes by running the git commit -a command in your terminal with the message below.

Commit message
feat(#174): add experimental builders for 'NotBlankString'

New builders: 'toNotBlankStringOrNull' and 'toNotBlankStringOrThrow'.
These are not implemented yet.

[x] Implement the toNotBlankStringOrThrow function by resolving the following tasks:

[x] Add unit tests for this new function.
[x] Run tests on the JVM platform. These should fail at this stage.
[x] Implement this new function without changing its signature.
[x] Run tests on the JVM platform. These should pass at this stage.
[x] Commit your local changes by running the git commit -a command in your terminal with the message below.

Commit message
feat(#174): implement the 'toNotBlankStringOrThrow' function

[x] Implement the toNotBlankStringOrNull function by resolving the following tasks:

[x] Add unit tests for this new function.
[x] Run tests on the JVM platform. These should fail at this stage.
[x] Implement this new function without changing its signature.
[x] Run tests on the JVM platform. These should pass at this stage.
[x] Commit your local changes by running the git commit -a command in your terminal with the message below.

Commit message
feat(#174): implement the 'toNotBlankStringOrNull' function

[x] Create a Pull Request.

Maintainers only

[x] Refactor the construction of the type by using functional programming tools provided by Kotlin for improving its readability, composability and performance.

We should fix the version specified in the ExperimentalSinceKotoolsTypes annotation:
// Before
@ExperimentalSinceKotoolsTypes(version = "4.4")
public fun String.toNotBlankStringOrNull(): NotBlankString?

@ExperimentalSinceKotoolsTypes(version = "4.4")
public fun String.toNotBlankStringOrThrow(): NotBlankString

// After
@ExperimentalSinceKotoolsTypes(version = "4.3.1")
public fun String.toNotBlankStringOrNull(): NotBlankString?

@ExperimentalSinceKotoolsTypes(version = "4.3.1")
public fun String.toNotBlankStringOrThrow(): NotBlankString


Allow for thread interruption

This PR addresses #354, but I would like feedback before I consider merging.  Especially from @benjchristensen / @abersnaze.
All of the unit tests that are added (Thanks @aadeon!) now pass, but I would love to see a better way to accomplish this goal.
It's hacky in a few ways:

I believe that the Observable.subscribeOn method has a bug where unsubscription is not propagated to the work which is submitted to the new Scheduler.  A Subscription is generated in OperatorSubscribeOn in the schedule() method and then forgotten about.  I believe that's the precise Subscription we need to unsubscribe from.
Upon unsubscription, we need to optionally interrupt the thread.  At the moment, Subscriptions.from(Future<?> f) wraps a Future and upon unsubscription, cancels this future.  It hardcodes the parameter to cancel, however.  This is the value we need to have drawn from the HystrixCommandProperties.  So I believe that we should consider adding this to RxJava proper, rather than the hack I've done here. That hack is to not call the createWorker() in the Scheduler interface, but cast the Scheduler to a HystrixContextScheduler and call its (new) createWorker(boolean shouldInterrupt) method.  Since this is really only valid for thread pools, I'm not sure how much of this belongs in the Scheduler interface.

NetflixOSS » Hystrix » Hystrix-pull-requests #15 FAILURE
Looks like there's a problem with this pull request

Hystrix-pull-requests #254 FAILURE
Looks like there's a problem with this pull request

Also note that Javanica tests are failing as a result.  It wasn't obvious to me what went wrong there, so I might need help from Javanica experts after we sort out the code in hystrix-core

Hystrix-pull-requests #255 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » Hystrix » Hystrix-pull-requests #16 FAILURE
Looks like there's a problem with this pull request

@abersnaze already got 1 of these merged into RxJava master: https://github.com/ReactiveX/RxJava/pull/2575.  This addresses the issue where unsubscription was not propagated properly in OperatorSubscribeOn

@mattrjacobs I had to revert that PR because it was a no-op in RxJava at best because RxJava's workers track all scheduled tasks and thus unsubscribing the worker while having a task is equivalent of unsubscribing the task. See this test which schedules a bunch of tasks and cancels them all at once with a single unsubscription; the test then shows that no task was retained.
Looking at HystrixCommandScheduler the task-scheduling logic is quite old and has several races (is it even used?). The HystrixContextSchedulerWorker.unsubscribe doesn't unsubscribe the actual worker but only a boolean subscription, thus the RxJava practice of cancelling the entire worker won't affect any scheduled tasks.

Thanks for taking a look @akarnokd , I appreciate it.  Yes, the Scheduler being used is the HystrixContextScheduler.
I am now looking through the examples of concrete Schedulers in RxJava for inspiration.  I hope you don't mind if I ask for a review once I get my head wrapped around those and apply those learnings to the HystrixContextScheduler.

Closing this in favor of #647


Periodic segfault/malloc issues in macOS repl when using tag unions

Sometimes when running repl tests, we see errors like this in tests that involve tag unions:
roc(41971,0x118087dc0) malloc: *** error for object 0x7fcad9d1d8a0: pointer being freed was not allocated
roc(41971,0x118087dc0) malloc: *** set a breakpoint in malloc_error_break to debug

roc(42941,0x11cff6dc0) malloc: Incorrect checksum for freed object 0x7fb790f599a0: probably modified after being freed.
Corrupt value: 0x7ffee8411c88
roc(42941,0x11cff6dc0) malloc: *** set a breakpoint in malloc_error_break to debug

This can manifest as a segfault:
---- repl_eval::num_add_checked stdout ----
thread 'repl_eval::num_add_checked' panicked at 'repl segfaulted during the test. Stderr was ""', cli/tests/helpers.rs:343:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

We can also sometimes see these segfaults when running the repl manually, but not always - try running List.first [ 1, 2, 3 ] a few times on macOS to reproduce it.
It seems some pointer is pointing to the wrong memory, because in some rare cases it's even overwriting memory outside the normal program - in this case, overwriting some of the memory in the ena data structure:
thread 'main' panicked at 'index out of bounds: the len is 120 but the index is<PHONE_NUMBER>', /Users/rtfeldman/code/roc/vendor/ena/src/snapshot_vec.rs:151:10

thread 'main' panicked at 'Unexpected FlatType FlexVar(None) in num_to_ast', cli/src/repl/eval.rs:846:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

One potential clue is that mono IR consistently prints this out when running these tests in isolation:
emitted runtime error VoidValue

Hard to say if this still relevant or not because it is intermittent, but I ran List.first [ 1, 2, 3 ] in the repl on my Mac about 30 times and didn't have any issues so I'm going to close the issue.


[$500] LHN-Paid and 1:1 report missing in LHN

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.3.91-6
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
Action Performed:

Go to https://staging.new.expensify.com/
Tap on a report eg:user B
Tap plus icon to select Request money
Enter an amount
Request money
Navigate back to LHN - Note 1:1 and IOU report in LHN
Open User B account and note only IOU report is displayed in LHN
Open the bold IOU report in LHN and pay elsewhere
When user B pay elsewhere, note the 1:1report in user A LHN disappears.
When user B pay elsewhere and navigate to LHN, now both 1:1 and IOU report displayed but no paid transaction report shown
As user A, tap on IOU report and note request amount paid.
Navigate back to LHN and note the reports displayed

Expected Result:
When a IOU is created, both 1:1 and IOU report must be displayed for user A and user B and after pay elsewhere, paid IOU report must be displayed in LHN for both users
Actual Result:
For user A, the 1:1 report disappeared in LHN when user B pay elsewhere and paid IOU report not displayed in LHN.
For user B, after user A request money only IOU displayed in LHN and only after pay elsewhere and revisiting LHN 1:1 report is shown but paid IOU report not shown in LHN
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos

Android: Native

Android: mWeb Chrome
https://github.com/Expensify/App/assets/115492554/cd65377d-ef45-4989-b662-3bb167976b19
https://github.com/Expensify/App/assets/115492554/2b45813a-2f74-4279-9a5c-36082ca812e4

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018dfeb9cdb5e3ad75
Upwork Job ID:<PHONE_NUMBER>739455488
Last Price Increase: 2023-10-26

There is an issue in the data coming from Pusher on the RequestMoney API call:
{
    "onyxMethod": "merge",
    "key": "report_1776447671437860",
    "value": {
        "iouReportID": "6625022465297579",
        "participantAccountIDs": {
            "1": 15945552
        }
    }
}

It returns the participantAccountIDs as an object, while it is supposed to be an array:
{
  "onyxMethod": "merge",
  "key": "report_1776447671437860",
  "value": {
    "iouReportID": "6625022465297579",
    "participantAccountIDs": [
      15945556
    ]
  }
}

Hi @paultsimura (not an engineer) are you saying this should be handled from the back-end?

Hi @paultsimura (not an engineer) are you saying this should be handled from the back-end?

@jliexpensify  I think so, yes. The Pusher now behaves weirdly with these requests.
Somehow, the "Request Money" flow got resolved for me, but the "Pay elsewhere" by user B still removes the 1:1 chat for user A.
These are the updates this user receives from Pusher:
{
    "onyxMethod": "merge",
    "key": "report_6248612838888226",
    "value": {
        "iouReportID": null,
        "participantAccountIDs": []
    }
}

The participantAccountIDs is set to an empty array, so this report disappears from LHN because of this check:
https://github.com/Expensify/App/blob/6c2e08741475635713b71aa2ba47c2cc6e354d9e/src/libs/ReportUtils.js#L3258-L3276

I couldn't reproduce this on Mac Chrome. @jliexpensify Are you able to reproduce this?

Just reproduced on latest main:
https://github.com/Expensify/App/assets/12595293/3f32e275-8dca-4c3f-ba62-e9b75ff6bad0
@mananjadhav

Hmm I can't repro on Staging v1.3.92-4: when I click on Pay Elsewhere everything seems to work as intended?

@izarutskaya can you repro on the newest version of Staging?

Bumo @izarutskaya !

Hi @jliexpensify, I cannot reproduce this on Web/mWeb. Build - v1.3.95-0
https://github.com/Expensify/App/assets/115492554/c66e0783-808c-4e14-a446-ba4bb76b14f1

Great, thanks! Lets close this one out.


[WIP] feat: Recover dead public gateway links

This PR closes #640, enabling dead public gateways to be redirected to the default public gateway. Added as an experimental option, defaulting to false.

Hi @colinfruit is it still wip, or ok to review?

Hi @lidel, the basic structure is fine to review. I'm working on tests, finding a clean way to stub promises (browser.tabs.query/create), and reviewing the error codes that were added, to make sure I have all that we want, and none that we don't want. Glad to hear your comments!

To elaborate a little more on the testing part of my work, I've been going through a couple of different libraries to see if any of them with promisify sinon-chrome. The only two that could theoretically work are then-chrome and chrome-promise. chrome-promise is no longer maintained, just bug fixes, and then-chrome is outdated and seemingly abandoned. The only other option afaik is stubbing the needed APIs manually. Has this been discussed before for this project, and is there a better way to accomplish this?

@colinfruit I stumbled upon similar problem with testing WebExtension APIs before. On top of the problem with abandoned libraries, sometimes Chromium and Firefox have slight changes in some APIs, making a generic library not feasible. In redirect tests I ended up writing bare minimim input-output tests for modifyRequest.* methods with hardcoded input specific to Chromium and Firefox, to ensure we have regression tests for each vendor  (example).
I think we should be fine with adding test/functional/lib/ipfs-request-gateway-recover.test.js with similar tests for modifyRequest.onCompleted and modifyRequest.onErrorOccurred (modelled after ipfs-request-gateway-redirect.test.js). If there are no vendor-specific APIs in onCompleted and onErrorOccurred, a single set of tests for both Firefox and Chrome will be enough.
Let me know if you got any questions.

@lidel, that sounds good, except for the fact that I will have to rework the way the onErrorOccurred originally was written. The current setup doesn't return anything, so in my current test setup I am checking that browser.create was called with the expected arguments, it('should redirect broken non-default public gateway IPFS request to public gateway', async function () {       const request = url2request('https://nondefaultipfs.io/ipfs/QmYbZgeWE7y8HXkH8zbb8J9ddHQvp8LTqm6isL791eo14h', 500)       await requestHandler.onCompleted(request)       assert.ok(browser.tabs.create.withArgs({ url: 'https://ipfs.io/ipfs/QmYbZgeWE7y8HXkH8zbb8J9ddHQvp8LTqm6isL791eo14h', active: true, openerTabId: 12345 }).calledOnce, 'tabs.create should be called with IPFS default public gateway URL')     }) I was under the impression that it would be bad practice to add a return value solely for the purposes of testing, but I'm happy to rework them if you're OK with this?

@lidel just added my tests as a working example so it's easier to see what I am doing, will remove them if your previous comment is what you want. :)

thanks for the review, @lidel, resolved most of your suggestions. It would be cool to recover broken sub-domains. Maybe a separate issue to track this would be good? I'm glad to take a look at it if I have some time.


trigger twice or thrice

In scheduling some job often repeated execution
Test expression：
0/2 * * * * ?
0 */2 * * * *

Please clarify your issue. Are you finding the job is triggered too often, or not often enough?

I am seeing this as well on Windows Server 2003.  My program simply prints the schedule and the elapsed number of seconds since it was last fired.
I am using the same three schedules on each system:

*/5 * * * * *
*/20 * * * * *
@every 10s

Windows Server 2003 / go version go1.6.3 windows/386
The first six entries or so were all blurted out at once as soon as the program started; so the Job implementation is being fired multiple times.  The @every 10s fires a little too soon but does work and you can see all the others that are every 5 or every 20 seconds came out multiple times before the single @every 10s.
cron-exec [*/5 * * * * *]: Time since last: 2
cron-exec [*/5 * * * * *]: Time since last: 1
cron-exec [*/5 * * * * *]: Time since last: 4
cron-exec [*/20 * * * * *]: Time since last: 7
cron-exec [*/5 * * * * *]: Time since last: 0
cron-exec [*/20 * * * * *]: Time since last: 0
cron-exec [*/5 * * * * *]: Time since last: 0
cron-exec [*/20 * * * * *]: Time since last: 0
cron-exec [*/5 * * * * *]: Time since last: 0
cron-exec [*/20 * * * * *]: Time since last: 0
cron-exec [*/5 * * * * *]: Time since last: 0
cron-exec [*/20 * * * * *]: Time since last: 0
cron-exec [*/5 * * * * *]: Time since last: 0
cron-exec [*/20 * * * * *]: Time since last: 0
cron-exec [*/5 * * * * *]: Time since last: 0
cron-exec [*/20 * * * * *]: Time since last: 0
cron-exec [*/5 * * * * *]: Time since last: 1
cron-exec [*/20 * * * * *]: Time since last: 1
cron-exec [@every 10s]: Time since last: 9

debian / go version go1.7.1 linux/386
This is working correctly.
cron-exec [*/5 * * * * *]: Time since last: 3
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [@every 10s]: Time since last: 10
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [*/20 * * * * *]: Time since last: 18
cron-exec [@every 10s]: Time since last: 10
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [@every 10s]: Time since last: 10
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [*/20 * * * * *]: Time since last: 20

Windows 10 / go version go1.6.2 windows/386
This also works correctly.
cron-exec [*/5 * * * * *]: Time since last: 1
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [@every 10s]: Time since last: 10
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [*/20 * * * * *]: Time since last: 11
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [@every 10s]: Time since last: 10
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [@every 10s]: Time since last: 10
cron-exec [*/5 * * * * *]: Time since last: 5
cron-exec [*/20 * * * * *]: Time since last: 20
cron-exec [*/5 * * * * *]: Time since last: 5

As far as I'm aware go 1.6.3 is the latest supported on Win2003.  The older go 1.6.2 works but on a newer version of Windows.  I suspect this is some sort of problem related specifically to the older Windows OS.  I wouldn't imagine you'd want to support a 13 year old dead OS but you might want to make a mention on the project's homepage.
I haven't tested any other schedule (such as * */5 * * * *) on Win2003.  My end goal is a daily schedule which might work on Win2003 - or the machine I deploy to might be a newer Windows anyways.

Here is the git diff output that fixes this on Windows 2003.
diff --git a/cron.go b/cron.go
index f8d325d..1f0772d 100644
--- a/cron.go
+++ b/cron.go
@@ -183,6 +183,9 @@ func (c *Cron) run() {
                timer := time.NewTimer(effective.Sub(now))
                select {
                case now = <-timer.C:
+                       if now.Sub(effective) < 0 {
+                               continue
+                       }
                        now = now.In(c.location)
                        // Run every entry whose next time was this effective time.
                        for _, e := range c.entries {

The problem is that if the sleep duration is very small, around 1ms, then the sleep may return immediately.  The same sleep time may then be calculated and return immediately again.  And this can happen a few times.  All I did is make sure that the duration that passed between now and effective was >= 0; i.e. that some form of sleep actually occurred.

Sorry for the trouble - the updated waiting mechanism on the v3 branch works for this by verifying the time has arrived for the job:
		select {
		case now = <-timer.C:
			now = now.In(c.location)
			// Run every entry whose next time was less than now
			for _, e := range c.entries {
				if e.Next.After(now) || e.Next.IsZero() {
					break
				}
				go c.runWithRecovery(e.Job)
				e.Prev = e.Next
				e.Next = e.Schedule.Next(now)
			}

Please let me know if you see any remaining problems on the v3 branch. Thank you


Remove the O Series

What happened:
There is low value n the O Series. Let's remove it.
KubeVirt gained memory overcommit, an admin sided feature, that's easier and more consistent to use.
What you expected to happen:
A clear and concise description of what you expected to happen.
How to reproduce it (as minimally and precisely as possible):
Steps to reproduce the behavior.
Additional context:
Add any other context about the problem here.
Environment:

KubeVirt version (use virtctl version): N/A
Kubernetes version (use kubectl version): N/A
VM or VMI specifications: N/A
Cloud provider or hardware configuration: N/A
OS (e.g. from /etc/os-release): N/A
Kernel (e.g. uname -a): N/A
Install tools: N/A
Others: N/A

@fabiand I believe that support pre-dates the work @vladikr did in https://github.com/kubevirt/kubevirt/pull/9799 that is used by this instance type class. I'm not against removing it I just wanted to check with you both that we still want to deprecate this as I'll also remove support from the instancetype.kubevirt.io API.

I would remove the O Series.
But I do not understand remove support from the instancetype.kubevirt.io API. could you rephrase this for me?
Also: We might want to add the "graduation level" to instanceTypes

I would remove the O Series.
But I do not understand remove support from the instancetype.kubevirt.io API. could you rephrase this for me?

https://github.com/kubevirt/kubevirt/pull/9799 specifically introduced the spec.memory.overcommitPercent attribute used by this class. If it's no longer used here I was going to deprecate it in v1beta1 to potentially drop ahead of v1 in the future.

Also: We might want to add the "graduation level" to instanceTypes

Possibly, we could definitely use it for preferences for non-GA'd OS's etc.


Refactor packages

Coverage decreased (-22.2%) to 59.14% when pulling ba3dda5155af566476b61dd3f7a8c0d7961a3d67 on package-refactor into e629969f497e44bbb625f1a68eeebb3bf292da00 on master.

Coverage decreased (-22.1%) to 59.198% when pulling 381d808770804fbf4155450a294ddbffce56f3fd on package-refactor into e629969f497e44bbb625f1a68eeebb3bf292da00 on master.

Coverage decreased (-3.2%) to 78.133% when pulling ad45538c52bd4dd7fb8b700b1d85c73d4619f67b on package-refactor into e629969f497e44bbb625f1a68eeebb3bf292da00 on master.


Disable PR creation

This is now handled by the hacbs-contract/infra-deployments-ci repo.
https://issues.redhat.com/browse/HACBS-1773

Codecov Report

Merging #336 (64a3027) into main (689e4e6) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #336   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           61        61
  Lines         1743      1743
=========================================
  Hits          1743      1743


Add conda package for kni and KNITransfer

In LearningTestRunner, the new python api (kht_test) runs systematically the tests for Khiops, Khiops_coclustering and KNI. It fails if KNITransfer is missing. Therefore I added 2 new conda packages (as sub-packages): kni and kni-transfer.
A second commit concerns kht_test.py. The origin of the problem is that linux packages use openmpi and conda package use mpich. kht_test is now compliant with both: the flags specific to openmpi are translate in environment variables.

LGTM.


Showing Old Code in JSON validator

I pushed the original page to domain.com/.well-known/apple-app-site-association then a day later pushed another update. I visited the URL and confirmed that on my side of things I see the latest update, but this tool shows the original version of the JSON. How long until the page updates? Should it update automatically?

Hi Lump, could you tell me where you're testing this? As far as I know, the server I hosted this one is no longer around (Heroku shut me down because I didn't do anything).

It was a branch.io page that had this repo documented as the source. I can get you the exact URL tomorrow.

Oh - they forked my repository after I stopped working there and took theirs private, I believe. This repo is only in line with the very, very initial pass of the Apple Universal Link guidelines back in 2015 maybe? I should really just archive this project at this point. I'm not sure if he'll response, but @aaustin or his GH profile is the Branch CEO and can probably speak to whether they are maintaining this and if they can help.


CTA Localization / Text Accessibility

Description
Allows text of various lengths and sizes to display appropriately in the notification box / call to action display.
Other changes
Test Snapshots updated

iOS Before
iOS After
Android Before
Android After

CTA
Before
After
Before
After

Tested
Tested locally on iOS and android devices.
How others should test

Launch the app with default text sizing and insure the CTA displays correctly.
Chance the text size and display size (android only) relaunch the app and insure that the CTA displays correctly.

Related issues

Fixes #149 #579

Backwards compatibility
As this impacts styling only it is backwards compatible.
Known Issues
Currently device width and height is only calculated on app load from App.tsx. This means that launching the app, changing system settings and bring the app to the foreground does not recalculate these variables. One solution for this would be move to use https://reactnative.dev/docs/usewindowdimensions and setup an event listener for window resizing; however, that is outside of the scope of this PR.  Currently the workaround for resizing is to close and relaunch the app.

@etuleu the CTA should expand to contain all the text and the largest CTA will determine the size of the others.


Proposal 1

I see two different ways the HTTPRequest could work.
import S4

class MyCoolRequest: S4.HTTPRequest {

}

``swift
import S4
extension MyCoolRequest: S4.HTTPRequestConvertible {
var s4HTTPRequest: S4.HTTPRequest {
return S4.HTTPRequest(self.method, self.....)
}
}

Out of curiosity, why use a class for the request?

First off, as I read this proposal I'm reminded that enums are impossible (difficult?) to represent as a protocol, so I think it make sense that anything that must be represented as an enum be a concrete type as defined in the proposal. Based on our discussion I think that HTTPMethod and HTTPStatus are well defined.

About the enums. Protocols can't be used as keys to dictionaries and could be desirable for HTTPMethod specially.

I guess I should mention that String or Int could be used as keys without the enums...

@paulofaria Vapor uses a class for Request, and I could see other frameworks wanting to do that. Vapor uses a lot of callbacks and copying the Request struct for each of those seems like a lot of extra CPU work.

@paulofaria So are you saying Requests in a project conforming to S4 should be sub-structs of S4.HTTPRequest?

We should comment on the diff, not on the PR itself. this way we can have a logical sequence of messages. So I'm gonna reply in the HTTPRequest section.

@paulofaria Good idea, I'll stop spamming the general PR comments. >_<

I would like to propose one final time, and then I will resist, but I think we will be significantly more productive if we break this up.
The contents included are far to comprehensive to appropriately discuss in the message board style. Each of the components would benefit from a unique pull request that has a good focus.
Once we agree on these, one by one, they can be merged into the repo. Right now, it's way too much to follow and we can't merge any of it until we agree on all of it.
@tannernelson @paulofaria @bradhilton

@LoganWright I agree 100%. We might as well split them up now. It's probably not going to get easier.


Bad Links

Both links in step one and two in the Create Az Rm template link to the general DevTestLabs doc page, not helpful documentation on how to create the template.

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

ID: 22c509e4-dac8-9d73-1087-87aad0017012
Version Independent ID: 98070d8d-aab5-60f0-73e0-fe5d7137293c
Content: Manage a VM in DevTest Labs - Azure Pipelines
Content Source: docs/pipelines/apps/cd/azure/deploy-provision-devtest-lab.md
Product: devops
Technology: devops-cicd
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

Thanks for your feedback
Looks like the doc above is not created or maintained by the DevTest Labs team.
Here is a doc from our product docs: https://docs.microsoft.com/en-us/azure/lab-services/devtest-lab-integrate-ci-cd-vsts
Can we please replace this doc with the link I shared above?
Thanks,
Tanmayee

@RoopeshNair, can you please address my comment above?
Tanmayee

@steved0x  - can you please do the needful?


ProgressRing does not jump backward to a lower value when progress decreases

Describe the bug
When the ProgressRing needs to reduce the amount of progress made, the ring does not move backwards it only moves forwards.
Steps to reproduce the bug

Steps to reproduce the behavior:

Set ProgressRing to a value of 60.
Set the value to 50.
ProgressRing completely fills in and animates forward to the value of 50.

Expected behavior
ProgressRing will jump back to the value of 50.
Version Info
WinUI 2.5
NuGet package version:
Microsoft.UI.Xaml v2.5.0-prerelease.201027002

I think this is a duplicate of #3173, @MarissaMatt can you confirm?

You're correct! @ranjeshj could we assign someone to the issue that @chingucoding mentioned above? I'll close this one after the other bug is assigned. Good catch @chingucoding!

@chingucoding I saw you said you would like to look into the duplicate issue. Did you run into an issue with AVP that prevented the negative playback rate from working?

@StephenLPeters The issue is that the animation does in fact not play in reverse. When setting the playback rate to -1, instead of animating to that value, the progressring immediately jumped to that value. It would probably require some digging into the animation to figure out why a negative playback rate does not work as expected.

@StephenLPeters The issue is that the animation does in fact not play in reverse. When setting the playback rate to -1, instead of animating to that value, the progressring immediately jumped to that value. It would probably require some digging into the animation to figure out why a negative playback rate does not work as expected.
Edit: Created a PR to fix the behavior as best as we/I can. Animating backwards doesn't work, it jumps to the value, however that is far better than the current behavior.

Jump when going backwards is what the spec calls for.

Jump when going backwards is what the spec calls for.

Oh I did not know that. In that case, I updated the PR description to close both this and #3173, I am sorry about the confusion and delay caused.


Missing images

The api works (based on examples.php). But the json doesn't contain the links to the image files.
The  json shoild include the image files too.

Hi @bschelst,
I don't see the URLs in the JSON response returned from GloriaFood's API, and it's not documented in their repo.
Are you requesting that the URLs be included in the API's response, or that this package should be able to parse the data received from the API?
Please note that this isn't an official GloriaFood package.
Thanks

Closing issue due to lack of response.


*: Add lifecycle events to events API

Resolves #4322
Signed-off-by: Thomas Hipp<EMAIL_ADDRESS>

jenkins: test this please

Some comments in-line and as discussed we should limit the context right now to be empty in all cases except for:

Rename should have the new name
All snapshot events should have the snapshot name

Should be fine now.


Allow scientific notation in written numbers

Update to #30: attempts to allow writing out scientific notation where legal (i.e. not onset or duration fields).
motivated by encountering
ErrorException(\"failed to fit number into EDF's 8 ASCII character limit: 4.180821e-7\")

when writing out an EDF. We don't know why or which field this was in though.
This PR changes the semantics:

if allow_scientific=false, which I have set by default, then underflow to 0 is allowed. This was formerly disallowed in #30 (although I did not see a discussion why). The former semantics were a bit weird in that rounding/truncation was OK except when the result was truncated to 0, in which case it errored.
if allow_scientific=true, then we use %G in sprintf to allow either of scientific notation or decimal representation. This shouldn't really underflow nor with any kind of Float64 representable number, but could if somehow one got a BigFloat or such in there.

I also set the code to allow scientific notation with all floating-point fields of the headers (but with onset/durations of annotations).
Would appreciate thoughts on this & careful review

Is this too slow?

the allow_scientific=true path is only used for headers, so should not be a perf concern
the allow_scientific=false path is used for annotations, so could be a perf concern, if it is slow to call for several million numbers

Let us benchmark.
Benchmark
Try every Float16 50 times, which is 1.5 million numbers
function all_floats(; n=50)
    out = Char[]
    sizehint!(out, 30720 * n)
    for _ = 1:n
        x = floatmin(Float16)
        while x <= floatmax(Float16)
            str = EDF._edf_repr(Float64(x))
            push!(out, str[1])
            x = nextfloat(x)
        end
    end
    return out
end

PR
julia> @time length(all_floats())
  0.731282 seconds (9.98 M allocations: 1.442 GiB, 7.50% gc time)
1536000

julia> @time length(all_floats())
  0.727198 seconds (9.98 M allocations: 1.442 GiB, 7.72% gc time)
1536000

julia> @time length(all_floats())
  0.736517 seconds (9.98 M allocations: 1.442 GiB, 7.93% gc time)
1536000

Master
julia> @time length(all_floats())
  0.344242 seconds (12.52 M allocations: 889.876 MiB, 6.08% gc time)
1536000

julia> @time length(all_floats())
  0.345693 seconds (12.52 M allocations: 889.876 MiB, 5.95% gc time)
1536000

julia> @time length(all_floats())
  0.347998 seconds (12.52 M allocations: 889.876 MiB, 6.80% gc time)
1536000

Conclusion
So it is ~2x slower, but still fast enough that it doesn't seem like a big issue.

Is allowing underflow to 0 dangerous?

We allow this now when allow_scientific=false. This is used for onsets & annotations, and for non-floating-point types.

for onsets and annotations, a sufficeintly small time like 1e-7 does seem OK to round down to 0. If the onset is right at the start of the recording, marking that as time 0 seems OK. If an annotation has very small duration, since EDF does allow 0 or even missing duration, rounding that down to 0 does seem OK too.

when allow_scientific=true, we can represent even floatmin(Float64) with a few digits of precision. This is used for floating point fields in the headers.

Therefore I think it is probably OK, given the we can see all the usages and check them individually.

Is scientific notation even allowed in the EDF+ spec?
from https://www.edfplus.info/specs/guidelines.html#:~:text=Numbers in EDF(%2B) headers,3 (Okt 2004) they say

2a, E-notation (May 2009). Numbers in EDF(+) headers may have the scientific E notation as in 1E2345, +012E+34, -1.34E09 and +1.234E-5. Note, though, that the 8 characters of some EDF(+) numberfields are used more efficiently by -123.456 uV than by -1.23E-4 V.

So it seems it is. We do not use it for non-header fields such as annotation durations and onsets.

Is scientific notation even allowed in the EDF+ spec?
from https://www.edfplus.info/specs/guidelines.html#:~:text=Numbers in EDF(%2B) headers,3 (Okt 2004) they say

2a, E-notation (May 2009). Numbers in EDF(+) headers may have the scientific E notation as in 1E2345, +012E+34, -1.34E09 and +1.234E-5. Note, though, that the 8 characters of some EDF(+) numberfields are used more efficiently by -123.456 uV than by -1.23E-4 V.

So it seems it is. We do not use it for non-header fields such as annotation durations and onsets.

Whether it's mentioend in the spec and whether it's supported by other readers is a different thing.  Maybe we don't care about compatibility with say MNE and friends but we should at least try scientific notation exports and see if our preferred viewers can view the generated files

One thing I realized is that in TAL, there isn't the same 8 character limits (as far as I can tell in the spec). And that is the only place we currently force the decimal printing with floating-point numbers. So we could try raising the limit there (or just do a regular @sprintf("%f", x)).

This actually does fix my internal bug, which now has an MWE at https://github.com/beacon-biosignals/EDF.jl/issues/74, and results in EDFs that are loadable in EDFBrowser at least (and which pass it's validation check) but I don't think this approach is actually necessary because we should just allow rounding to 0 for fields like physical_min. So I think a different fix would be more direct here as well as less concerning about portability.


Added subscription support

This is my contribution to IoT-Inspector python client, as the old one did not support subscribing.

Sorry, but this is terrible. We can implement it together next week.


Some way to generate multiple unique ids from one id

Caused by  #421
GetUniqueId(base_id, number) - return value should not be changed between calls for same pairs (base_id, number).

There should be better way to do it, but for a while the simple implementation is enough.


How to compute the scale of omnidata depth?

Thanks for your work, but I do not find the code to compute the scale of omnidata depth in the paper.

This part isn't ready yet since we found it not quite stable. A simplified version is here by simply computing the median of the reprojected depth ratio between SfM and omnidata. I will revisit this part when I have some bandwidth. Thank you!

Thanks dor your reply, I have another question, when I have D435 depth which is noisy, Can I compute the scale between D435 depth and omnidata depth in this way ?  Or by simply computing the median of thed depth ratio between D435 and omnidata ?

Or by simply computing the median of thed depth ratio between D435 and omnidata ?

I think this simplest practice would work fairly well, just be aware of the outliers from D435 (e.g. zero-depth pixels; pixels with depth > 5m) and compute the median.

Or by simply computing the median of thed depth ratio between D435 and omnidata ?

I think this simplest practice would work fairly well, just be aware of the outliers from D435 (e.g. zero-depth pixels; pixels with depth > 5m) and compute the median.

Thank you, I will try it

I have another question， Is this scale normal?

I have another question， Is this scale normal?

This looks incorrect. Realsense pixel depth by default uses mm (meaning that you need to divide pixel value by 1000). Some of the sensors have a depth factor of 2500 or 5000.

Thank you , I have divide 1000, but the ratio is still not unstable

OK, then would you please collect a set of images and compute the median scale per frame, and show the histogram of such medians?

The histogram is the ratio distribution of one depth image, but the variance is so large,  I cannot know what is wrong

---- Replied Message ----

From
Wei<EMAIL_ADDRESS>

Date
08/30/2023 01:23

To
<EMAIL_ADDRESS>

Cc
Jing ***<EMAIL_ADDRESS>

Subject
Re: [theNded/torch-ash] How to compute the scale of omnidata depth? (Issue #23)

OK, then would you please collect a set of images and compute the median scale per frame, and show the histogram of such medians?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

At a second glance I think it is not necessarily wrong. I can see the median is around 75 and the variance is ~50. This seems a reasonable distribution for monocular depth.

Hello，when I use sensor depth for scannet scene0050_00, the mesh is worse than learned depth，why is it?

When I load my data , I always occur in fusing dataset:
unordered_base::try_insert : Associated bucket and excess list full stdgpu::vector::size : Size out of bounds: -17 not in [0, 12182]. Clamping to 0 stdgpu::vector::size : Size out of bounds: -17 not in [0, 12182]. Clamping to 0 stdgpu::vector::size : Size out of bounds: -17 not in [0, 12182]. Clamping to 0 stdgpu::vector::size : Size out of bounds: -17 not in [0, 12182]. Clamping to 0 stdgpu::vector::pop_back : Object empty unordered_base::try_insert : Associated bucket and excess list full stdgpu::vector::size : Size out of bounds: -17 not in [0, 12182]. Clamping to 0 Fuse frame 31:  13%|████████████▊                                                                                | 31/239 [00:01<00:12, 16.03it/s] Traceback (most recent call last):   File "demo/train_scene_recon.py", line 266, in <module>     model.fuse_dataset(dataset, dilation)   File "demo/train_scene_recon.py", line 63, in fuse_dataset     fuser.fuse_dataset(dataset)   File "/opt/anaconda3/envs/sdfstudio/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context     return func(*args, **kwargs)   File "/mnt/dataset/zengjing/torch-ash/demo/rgbd_fusion.py", line 62, in fuse_dataset     self.fuse_frame(datum)   File "/opt/anaconda3/envs/sdfstudio/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context     return func(*args, **kwargs)   File "/mnt/dataset/zengjing/torch-ash/demo/rgbd_fusion.py", line 227, in fuse_frame     ) = self.grid.spatial_init_(points, dilation=self.dilation, bidirectional=True)   File "/opt/anaconda3/envs/sdfstudio/lib/python3.8/site-packages/ash/grid.py", line 152, in spatial_init_     assert masks.all()

It looks like the hash map capacity is not enough. Would you please try to increase the capacity here https://github.com/theNded/torch-ash/blob/master/demo/train_scene_recon.py#L40?
I fixed this number for easier loading/saving, but perhaps I can put it in a metadata field for more flexibility. That will make the hash map size adaptive.
Another way to debug this is to visualize the mesh before the error (say extract the mesh at frame 25 and see how it looks like). This may help identify the issue.

Hello，when I use sensor depth for scannet scene0050_00, the mesh is worse than learned depth，why is it?

If you simply use the rgbd_fusion part without rescaling (which is the right way), the result should look better than this. If you apply the finetuning (i.e. the training part), then it is possible that the results look worse. You may want to add losses designed for RGB-D from this paper https://dazinovic.github.io/neural-rgbd-surface-reconstruction/.


Horizontal Distortion (during any kind of visual movement)

Describe the bug
Weird Horizontal Distortions during any kind of movement, i.e; either when i move or a mob/block is moved.
Modpack version
v3.8.1
Launcher
MultiMC
Other things

[ ] I updated FO to a newer version (mention, what version to what version)
[ ] I removed or added some mods (mention, which ones)

Additional context
Cannot record video as proof because it isn't visible in recording output file. Just on my Display, here's my system info just in case;
CPU: Core i5-9600k
GPU GTX 1650 4Gb
Display: 4K@60Hz

Turning off Borderless Mining Mod fixed it.

That is weird though, because the mod is disabled by default on FO.

Yea, i usually enable borderless windowed mode to identify game windows using OBS.

Okay. Check their issue tracker out, maybe they have an issue relevant to you: https://github.com/comp500/BorderlessMining/issues


[WORKSPACES] Chrome Profile arguments, not opened in the desired location

Microsoft PowerToys version
0.85.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Workspaces
Steps to reproduce
i am opening two chrome profiles using CLI arguments.
the issue is that the default profile is always opened in the  first ( left ) chrome window , even if i swap the argument.
Chrome window 1 are expected to be "Profile 2"  ( yellow)
Chrome window 2 are expected to be "Default"
No matter what  ,it always opens Default on the "left" Chrome window.
Previous PowerToys version i was able to "hack" it by swapping the arguments, but now it no longer works.
Setting same profile on both opens the same profile on both. but having 2 different ones opening on the desired location does not.

✔ Expected Behavior
Expected behaviour is that "Profile 2" ( Yellow) is opened on the Left and "Default" on the right, as image below

❌ Actual Behavior
In image below you see that the Args fore thie frist (left chrome) is "Profile 2"   this is the "yellow chrome window. AS you see in the image below it is on the RIght and not on the left.

Swapping the arguments gives the same result ,

Other Software
No response

Thank you for the feedback. Currently Workspaces has issues with multiple instances of the same app. We are working on the fix. /dup #34730


Failed to execute 'getImageData' on 'CanvasRenderingContext2D'

http://scratch.mit.edu/projects/182276507
http://phosphorus.github.io/#182276507
Error: Failed to execute 'getImageData' on 'CanvasRenderingContext2D': The source height is 0.
    at Sprite.touching (https://phosphorus.github.io/phosphorus.js:1643:37)
    at eval (eval at scopedEval (https://phosphorus.github.io/phosphorus.js:4014:14), <anonymous>:2:8)
    at Stage.P.runtime.P.Stage.step (https://phosphorus.github.io/phosphorus.js:3938:15)

An error that just occurred when I died in my game.

same error


move example app into repo

is it possible to move the example app into the repo? this will ensure that the example always stays aligned with the current API

Previously that was the case: the app was part of the repo. However, at some point the dependencies for the example app broke, and it completely broke compilation of the project (due to the virtual workspace/monorepo).
I've decided to take it out so that I could continue work on the main library, but I wouldn't mind bringing back in here. I did (and do) not have much time to find out why it broke and update the dependencies though...

I did (and do) not have much time to find out why it broke and update the dependencies though

i could perhaps take a look? I might get some time in the next few days. Some of the dependencies in that project are pretty old, old enough that you could be running into a Future compat issue.

the issue is with an outdated dependency in eventually-postgres
specifically, refinery-core relies on a module in serde which used to be public, but is now private. I guess that should have been marked as a breaking change, but wasn't (or the dependency resolution would pick it up instead of the compiler)
error[E0603]: module `export` is private
   --> ...../.cargo/registry/src/github.com-1ecc6299db9ec823/refinery-core-0.3.2/src/runner.rs:11:12
    |
11  | use serde::export::Formatter;
    |            ^^^^^^ private module
    |
note: the module `export` is defined here
   --> ...../.cargo/registry/src/github.com-1ecc6299db9ec823/serde-1.0.130/src/lib.rs:275:5
    |
275 | use self::__private as export;
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^

refinery-core v0.3.2
├── refinery v0.3.0
│   └── eventually-postgres v0.2.0
│       └── eventually v0.4.0
│           └── eventually-app-example v0.1.0 (/home/dan/Code/rust/eventually-app-example)
└── refinery-macros v0.3.0 (proc-macro)
    └── refinery v0.3.0 (*)

I think the solution is to release eventually 0.4.1 with eventually-postgres 0.3.0 (which bumps the refinery dependency from 0.3.0 to 0.6.0)

might be worth starting a discussion what topology this repo should have (with respect to the various packages in the workspace)

I think the solution is to release eventually 0.4.1 with eventually-postgres 0.3.0 (which bumps the refinery dependency from 0.3.0 to 0.6.0)

Now that we've merged some breaking changes on master, it becomes a bit more difficult to release anything. At this point I'd rather try to work as much as possible towards 0.5.0 and release that one instead.
As for the app, I think it'd be nice to move it back inside the repo if https://github.com/get-eventually/eventually-app-example/pull/2 allows us to. Wdyt?

Closing this one as the app example has been moved in the repo already 👍🏻


Fix: #137

Hello, just a naive attempt at trying to refactor this for clarity as per issue #137
Feel free to use or not, hope I didn't forget anything.

This change is

 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.:x: SilverGhostBS sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

android/build.gradle, line 15 at r2 (raw file):

        classpath "com.android.tools.build:gradle:3.6.0"
    }
}

Do we need jcenter as a repo? jcenter seems to function as a proxy for multiple repos so I've seen the same artifiact (based on maven coordinates) have different sha values depending on whichever repo is hit.

I think if you don't put it you'll have problem finding trove4j, but I'm
not sure, there is a reason I added it though.
Le jeu. 26 mars 2020 à 13:26, Shane Isbell<EMAIL_ADDRESS>a
écrit :

android/build.gradle, line 15 at r2
https://reviewable.io/reviews/thaliproject/tor_onion_proxy_library/144#-M3Ms8PL6TYWCK3EnnEs:-M3Ms8PL6TYWCK3EnnEt:bdd7hxm
(raw file
https://github.com/thaliproject/tor_onion_proxy_library/blob/c5064eb0c24aaddff697e91804d340de34126c86/android/build.gradle#L15):
    classpath "com.android.tools.build:gradle:3.6.0"
}

}
Do we need jcenter as a repo? jcenter seems to function as a proxy for
multiple repos so I've seen the same artifiact (based on maven coordinates)
have different sha values depending on whichever repo is hit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thaliproject/Tor_Onion_Proxy_Library/pull/144#issuecomment-604563850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AETGQGYD3Z6WB5BKO5BYSZLRJOF4DANCNFSM4LTRGA3Q
.

Reviewed 3 of 3 files at r1, 4 of 6 files at r2.
Reviewable status: 7 of 9 files reviewed, 2 unresolved discussions (waiting on @SilverGhostBS)

android/build.gradle, line 13 at r2 (raw file):

    }
    dependencies {
        classpath "com.android.tools.build:gradle:3.6.0"

Shouldn't this keep using the variable rather than hard coding?
android_tor_installer/build.gradle, line 13 at r2 (raw file):

    }
    dependencies {
        classpath "com.android.tools.build:gradle:3.6.0"

Same comment from other gradle file

Yes, I agree having it as a variable makes sense but it prevents Android Studio from suggesting you to update when a new version comes out.

Plus it adds an exta "obstacle" when you want to build the android module separately from the rest

gradle.properties, line 2 at r2 (raw file):

# Configure Android Gradle Plugin Version
androidplugin=3.6.0

We would need at least gradle 5.6.4 for this version of the android plugin. The current configuration in the gradle wrapper is for 4.6. This would break the command line build.

@SilverGhostBS Can you fix the conflict. This PR will help us a lot for the android lib. Thanks.


Custom cell implementation in docs

Been waiting for the explanation for custom cell implementation process here. When is the doc going to be published? We have to sort of hunt for clues right now and takes unnecessary time

Lets do it together. You ask me your questions here, I will answer them and at the end we will collect it into a doc.

Hey @jassmith I have been trying to use the glide data grid and I enjoy the performance and user experience. It just feels natural.
My only challenge right now is building/drawing custom cells because I do not understand the API and can't find much documentation.
This is a sample table I am trying to migrate to glide

I would appreciate some guidance on drawing these kinds of cells. There are more custom cells not shown in this image. I guess just getting to better understand the API would help me build all the others. Thanks.
cc @mertdeveci5


Regarding the replacement of tshark with python port sniffing

Hello @Ali-Razmjoo, I came across todo in the core/network.py regarding the replacement of tshark with python port sniffing (e.g. https://www.binarytides.com/python-packet-sniffer-code-linux), I would like to work on it. Please let me know if I can work on it.

@tulja @Ali-Razmjoo : i also saw that the link in the todo explicitly says that if we just follow this ,it could break things for Windows as this doesn't support Windows OS.
and there are also other options available like https://github.com/dugsong/pypcap
what do you'll think?

The Socket creation is different for Windows and Linux, but we can add appropriate check for the platform using (https://stackoverflow.com/questions/110362/how-can-i-find-the-current-os-in-python) and perform neccessary packet filtering. I tested it on Windows 8.1 and Ubuntu 16.04, with few checks I'm able to run the same code for both Windows and Linux. If we use external packages (which internally have few issues like https://github.com/dugsong/pypcap/issues) things may break. I think it is better option to go with raw_socket with proper checks, @Ali-Razmjoo  Please let me know what you think

The Socket creation is different for Windows and Linux, but we can add appropriate check for the platform using (https://stackoverflow.com/questions/110362/how-can-i-find-the-current-os-in-python) and perform neccessary packet filtering. I tested it on Windows 8.1 and Ubuntu 16.04, with few checks I'm able to run the same code for both Windows and Linux. If we use external packages (which internally have few issues like https://github.com/dugsong/pypcap/issues) things may break. I think it is better option to go with raw_socket with proper checks, @Ali-Razmjoo Please let me know what you think

if the performance is better, I'm in.


fix(api): wrong window.setIcon argument type

What kind of change does this PR introduce? (check at least one)

[x ] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[ x] No

The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
Just a quick one, as I was trying the new beta rc !

Nice catch, thanks!


Select GPS unit

GT-U7(NEO6M)
GNSS MAX Click
M100_MINI GPS

The GNSS Unit was selected.
"GNSS 7 Click"


Support two-factor auth

Should work the same as https://github.com/primitybio/cellengine-r-toolkit/blob/master/R/authenticate.R.

@gegnew this one is a priority since it blocks using the entire toolkit if you have 2FA enabled. Have time soon to work on it?


NoProviderError after RC6 upgrade

Had ng2-idle implemented on a RC5/Webpack installation after following the GETTING STARTED part of the readme. Everything working fine. Upgraded to RC6 yesterday and got an error + app crash.
Image with the chrome console error
NoProviderError {_nativeError: Error: No provider for KeepaliveSvc! at NoProviderError.Error (native) at NoProviderError.Ba…, keys: Array[1], injectors: Array[1]}
_nativeError
:
Error: No provider for KeepaliveSvc! at NoProviderError.Error (native) at NoProviderError.BaseError [as constructor] (http://localhost/struktur/assets/d0a4e284/vendor.bundle.js:1618:39) at NoProviderError.AbstractProviderError [as constructor] (http://localhost/struktur/assets/d0a4e284/vendor.bundle.js:2102:21) at new NoProviderError (http://localhost/struktur/assets/d0a4e284/vendor.bundle.js:2133:21) at ReflectiveInjector_._throwOrNull (http://localhost/struktur/assets/d0a4e284/vendor.bundle.js:3735:24) at ReflectiveInjector_._getByKeyDefault (http://localhost/struktur/assets/d0a4e284/vendor.bundle.js:3763:30) at ReflectiveInjector_._getByKey (http://localhost/struktur/assets/d0a4e284/vendor.bundle.js:3726:30) at ReflectiveInjector_.get (http://localhost/struktur/assets/d0a4e284/vendor.bundle.js:3535:26) at NgModuleInjector.AppModuleInjector.createInternal (AppModule.ngfactory.js:451:67) at NgModuleInjector.create (http://localhost/struktur/assets/d0a4e284/vendor.bundle.js:9911:81)
constructResolvingMessage

I have not tried to install the ng2-keepalive library in my app.
What I did try was to declare the KeepaliveSvc class as a provider in app.module.ts and in the component where the Idle functions run in different ways with no success.
Setup:
"dependencies": {
    "@angular/common": "2.0.0-rc.6",
    "@angular/compiler": "2.0.0-rc.6",
    "@angular/compiler-cli": "0.6.0",
    "@angular/core": "2.0.0-rc.6",
    "@angular/forms": "2.0.0-rc.6",
    "@angular/http": "2.0.0-rc.6",
    "@angular/platform-browser": "2.0.0-rc.6",
    "@angular/platform-browser-dynamic": "2.0.0-rc.6",
    "@angular/router": "3.0.0-rc.2",
    "@angular/upgrade": "2.0.0-rc.6",
    "@ng-bootstrap/ng-bootstrap": "^1.0.0-alpha.3",
    "angular2-apollo": "^0.4.3",
    "angular2-modal": "^2.0.0-beta.12",
    "apollo-client": "^0.4.11",
    "core-js": "^2.2.0",
    "es6-promise": "^3.2.1",
    "ie-shim": "^0.1.0",
    "isomorphic-fetch": "^2.2.1",
    "ng2-bootstrap": "^1.0.24",
    "ng2-idle": "^1.0.0-alpha.15",
    "ng2-redux": "^3.3.5",
    "redux": "^3.5.2",
    "rxjs": "5.0.0-beta.6",
    "typescript": "^1.9.0-dev",
    "zone.js": "~0.6.12"
  }

app.module.ts:
import { IDLE_PROVIDERS } from 'ng2-idle/core';

providers   : [
  ...
  IDLE_PROVIDERS
]

auth-timer.component.ts:
import { Idle, DEFAULT_INTERRUPTSOURCES } from 'ng2-idle/core';
constructor(
  private idle: Idle
)
{
  idle.setIdle(600);
  idle.setTimeout(180);
  idle.setInterrupts(DEFAULT_INTERRUPTSOURCES);
  idle.onIdleStart.subscribe(() => {
      console.log('IdleStart');
    });
  idle.onIdleEnd.subscribe(() => {
      console.log('IdleEnd');
    });
  idle.onTimeoutWarning.subscribe((countdown:number) => {
      console.log('TimeoutWarning: ' + countdown);
    });
  idle.onTimeout.subscribe(() => {
      console.log('Timeout');
    });
  // start watching for idleness right away.
    idle.watch();
}

I have the same issue

Here is an workaround
extend the Idle Class and provide it instead of Idle
eg
import { Optional, Injectable } from '@angular/core';
import { Idle, SimpleExpiry } from 'ng2-idle/core';

@Injectable()
export class PalIdle extends Idle {
    constructor(expiry: SimpleExpiry) {
        super(expiry);
    }
}

then in your component, module whatever: providers: [PalIdle, SimpleExpiry]

Thanks @elvirdolic ,this worked great for me as well!

I'm using Ionic RC0, i'm trying to use the workaround form @elvirdolic, but I keep getting an error:
node_modules/ng2-idle/core.js does not export SimpleExpiry
It works fine using ionic serve, but when I try and make a prod bundle (ionic build ios) it fails as above - I assume its the AOT compiler that is complaining for some reason.
Can you provide a working repo with the workaround in it?
My code snippet:
import { Idle} from 'ng2-idle/core';
import { SimpleExpiry} from 'ng2-idle/core';
@Injectable()
export class PalIdle extends Idle {
constructor(expiry: SimpleExpiry) {
super(expiry);
}
}
@Component({
templateUrl: 'app.html',
providers: [PalIdle, SimpleExpiry],
})
export class MyApp {
...


No unnecessary path compression

Start path compression if parent is not root, since initial p is already under the root, It will avoid compression for already compressed paths

@williamfiset what do you think?

Does anybody disagree?
On Thu, 20 May 2021 at 01:40, Vyacheslav Bespalov @.***>
wrote:

@williamfiset https://github.com/williamfiset what do you think?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/williamfiset/data-structures/pull/49#issuecomment-844714225,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI26I3POEDYQ57C2BCD5LF3TOSODHANCNFSM42K6QU5Q
.

--
Thanks in credit,
Kanan

@williamfiset

@williamfiset

Can you detail the concerns?

This repo is no longer being maintained: https://github.com/williamfiset/data-structures#project-status
Happy to review it for the algorithms repo. Looks good at a high level


Revert "[common][server][dvc-client] Make a true non-blocking end-offset fetcher

This reverts commit 262101c6968abbd47b5d2ea28e4eb9e0c4784778.

Thanks. Task 1000 passed in 8m, no internal retries needed.

At least now we know this did cause flakiness. Need to find the RCA now.