Fix character range grammar parsing

GBNF grammar for character ranges is not parsed correctly. For example, [a-z] is currently parsed as ("a" | "-" | "z").
This PR updates it to match upstream:
if (pos[0] == '-' && pos[1] != ']') {
    if (!pos[1]) {
        throw std::runtime_error("unexpected end of input");
    }
    auto endchar_pair = parse_char(pos + 1);
         pos          = endchar_pair.second;
    out_elements.push_back({LLAMA_GRETYPE_CHAR_RNG_UPPER, endchar_pair.first});
}

https://github.com/ggerganov/llama.cpp/blob/72272b83a3878e91251218c981b4c6ec16c33912/common/grammar-parser.cpp#L241-L248

Kudos to you for jumping into that side of the code. Looks good to me.


inception_ensemble example uses wrong preprocessor

The inception_ensemble example uses channel-wise mean/std normalization, whereas I believe the source inception model just rescales to [-1,1] see here

Hi @philnguyenresson,
I think there are many flavors of the data processing for the inception network. It very much depends on the model you have. For example Pytorch implementation uses channel-wise normalization.
Also, this is just an example. You should adjust it to the data processing applied during the training of your models.

Sure, I will make modifications as I need them for my own code.
I'm just talking about the model given in the example "inception_v3_2016_08_28_frozen", which I believe was trained with [-1,1] normalization


Document configuration file properties

Right now there is just an example file in the sample metadata.

Documentation site created at : https://sites.google.com/view/orgdocumentor


Implements sysctl vm.ps4dev.trcmem_avail

Required by CUSA13434.

Shouldn't the type be u32?

Oh, right.


Expose most of typescript API classes and fields

To perform analysis on the table fields used by SquashQL measures and criteria, in order to optimize sub-queries, it is useful for Mercio to access the internal data structure of the typescript SquashQL API. Currently, not all classes are exported, and some have private fields, which makes analysis more complex. (we are forced to use typescript hacks)
Ideally, to facilitate this analysis, the SquashQL API should expose most of its classes and make most of their fields public.
As an illustration, the following measure classes are currently not exposed:
class LongConstantMeasure implements Measure {
  readonly class: string = PACKAGE + "LongConstantMeasure"
  readonly alias: string

  constructor(private value: Number) {
  }

  toJSON() {
    return {
      "@class": this.class,
      "value": this.value,
    }
  }
}

class DoubleConstantMeasure implements Measure {
  readonly class: string = PACKAGE + "DoubleConstantMeasure"
  readonly alias: string

  constructor(private value: Number) {
  }

  toJSON() {
    return {
      "@class": this.class,
      "value": this.value,
    }
  }
}

Other classes are exposed but keep some of their fields private:
export class Criteria {

  constructor(public field: Field,
              public fieldOther: Field,
              private measure: BasicMeasure,
              private condition: Condition,
              public conditionType: ConditionType,
              public children: Criteria[]) {
  }
}

I will do it ASAP as I realize it when developing my own UI

I think everything is exposed now. If anything is missing, please open an issue.


Autocomplete only for EFGJ

Describe your specific features or fixes
Resolves: https://jira.corp.adobe.com/browse/MWPW-134006
Should have 0 impact on the Autocomplete for EFGJ languages
For all other languages, no network request should be sent for autocomplete
Test URLs:

Before: https://stage--express--adobecom.hlx.page/express/templates/?lighthouse=on
After: https://mwpw-134006-ac-only-to-efgj--express--adobecom.hlx.page/express/templates/?lighthouse=on

Close as people now want this to go live in PROD before GA. I'll do a new PR in the old repo and later synch it here


[diem-framework] Fix test caching in CI tests

This PR attempts to fix issues where the file system isn't seeing all of the files for the Move package and causing some tests to fail. To prevent this we compile to a tempdir to get around any possible race conditions that could be happening.
Observed this failure on my machine by running cargo nextest in language every 10 or so times.  After this change I observed no errors due to the diem framework racing in 50-60 runs of the tests.

/land


Revert "fix(clerk-js): Disable chunking for clerk headless build"

Reverts clerkinc/javascript#1645

Closing in favor of #1650

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Limit date input box options

Since looking at COVID, limit the range of possible date inputs to be the 1st observation in dataset till the last date in dataset (which is sys.date()-1).

https://user-images.githubusercontent.com/61358854/158384107-e52cb6f1-3eb9-4579-a736-6372d49771c2.mov


One unreliable result on the leaderboard

Hello, thanks to your great leaderboard! It really helps to know the latest work in FSL! However I found one unreliable from the leaderboard:

LGM-Net | ICML | 2019 | 6CONV | Inductive

I think this result is unreliable because it has a problematic label leak issue from their codes (https://github.com/likesiwell/LGM-Net/issues/5) and the authors of this paper haven't make any response on this issue. Now that this result ranks No. 1 in the inductive settings. I think it may be misleading to people who are not familiar with the paper and not very fair to other researchers. Before the authors of that paper make any clarifications, could you consider remove this result from the leaderboard? I think it would be great to the FSL community. Thanks!

Hi @hukkai,
Thanks for your interest in our project.
As I am also working on FSL, I noticed that issue several months ago. However, as I am not 100% sure that the results are unreliable, I still included that paper in the leaderboard as it is already published in ICML.
If the authors of that paper still fail to update the results or respond to the GitHub issue in the following months, we will remove those results from the leaderboard.
Thanks again for your suggestion.
Best,
Yaoyao

Hi @yaoyao-liu，is there any updates regarding this results? Thanks!

Hi @yaoyao-liu，is there any updates regarding this results? Thanks!

I didn't see any updates. I will temporarily remove these results from the leaderboard.


docs should give clearer signposting of how to update DDEV config when updating a CMS

Is there an existing issue for this?

[X] I have searched the existing issues

Output of ddev debug test
No response
Expected Behavior
It's not clear from the docs how to deal with DDEV when updating a project's core codebase. For example, when updating Drupal from 9 to 10, or 10 to 11, the PHP version will need to change.
Running ddev config apparently does this, but it's not obvious to go looking there since it's mostly described as a command for initial setup.
A section should be added to https://ddev.readthedocs.io/en/stable/users/usage/managing-projects perhaps.
Actual Behavior
n/a
Steps To Reproduce
No response
Anything else?
No response

how to deal with DDEV when updating a project's core codebase

When you change your CMS, you have to configure the PHP version you want, right? Does that have to do with DDEV?

I'd also agree that when updating any software component (CMS, DB, Solr, etc) it's on the developer to manage that. It might be too much (and maybe presumptive?) for DDEV to overwrite that. For example, how would DDEV manage changing Solr versions if you go from D9-11? Or other CMS?

I just don't think it's DDEV's job to manage these things. It would increase scope too much. Closing for now, but thanks for the suggestion.

I'm confused, because I'm sure someone on Slack told me that the ddev config command already supported this -- that is, you can re-run it on a existing project that is, say on D9, when you want to upgrade it to D10.
So this issue was just about documenting that more clearly.

ddev config --update will inspect your code (any time) and if the DDEV autoconfiguration understands what to do, will set the defaults. It's not specifically about updating a CMS. But for Drupal 11, for example, DDEV would detect Drupal 11, and update defaults for it.
DDEV would try to do these things for Drupal: https://github.com/ddev/ddev/blob/a96ab265933cc9c94c7533740fd40d78add0ccc5/pkg/ddevapp/drupal.go#L366-L377
But it's certainly nothing of the scope you suggested in the issue here. Just a bit of finesse.
The ddev config docs attempt to explain this situation. Click the pencil in the upper right if you have more to add to it. Or suggest where else you might want an edit.

Yup, I saw it's mentioned in the docs for the config command. My point is that this feature would be more discoverable if it were mentioned in users/usage/managing-projects. I can have a look at editing that, but I am not sure where you'd like a new section to go in that document.

This new feature was added to v1.23.0 specifically for the new "drupal" project type. Because the quickstarts are often ddev composer create recipes, they don't have any code at first ddev config time. Now that "drupal" is a thing instead of adding a new type every time there's a new version, DDEV can detect the various varieties after composer install using ddev config --update.
The quickstarts for drupal do have this.

I'm not sure whether you mean we shouldn't add new docs, or where we should add them :)

If you can see where to do it, please discuss or click the pencil.

Note that DDEV says this after a ddev composer create:
ddev composer create was successful.
Consider using `ddev config --update` to autodetect configuration for your project

Maybe you would want it to say that somewhere else?


Provide funtion for checking permissions on a token

Please do not report security vulnerabilities here. The Responsible Disclosure Program details the procedure for disclosing security issues.
Thank you in advance for helping us to improve this library! Your attention to detail here is greatly appreciated and will help us respond as quickly as possible. For general support or usage questions, use the Auth0 Community or Auth0 Support. Finally, to avoid duplicates, please search existing Issues before submitting one here.
By submitting an Issue to this repository, you agree to the terms within the Auth0 Code of Conduct.
Describe the problem you'd like to have solved
We are using the Auth0 Core RBAC feature, so we have set up some roles and permissions on the Auth0 dashboard that will be using to control access to some API endpoints over our application. The problem I'm having is I didn't found the way to check for "permissions" instead of "scope" using this library.
This is the function given to check "scopes" but no option for checking other field in the JWT token is given. I can't understand how this library is recommended by Auth0 and has no support over all the features that are given (sold) on the platform.
Describe the ideal solution
The ideal solution I imagine is to add some function like requiredPermissions that will make the same operations over "permissions" field instead of "scopes"
Alternatives and current work-arounds
Currently using the old stack including
express-jwt, and jwks-rsa - For JWT token validation.
express-jwt-permissions - For JWT token permissions validations. You can specify wether the token is on the request and which field to check (scope, permissiosn or whatever the token has)
Additional context
I was trying to use this library instead of the express-jwt, express-jwt-permissions, and jwks-rsa stack following this recent blog post where it recommends to use this library as the other ones would only be maintained and won't get new features or further develpment apart from maintenance.

Hi @Hugomm - thanks for raising this
Because permissions is a proprietary Auth0 claim and this SDK is a generic OAuth SDK, we don't provide a requiredPermissions feature. But you can use the claimIncludes method to achieve the desired result.
app.get('/api/admin/managers',
    claimIncludes('permissions', 'admin', 'manager'),
    (req, res, next) => {
      // ...
    }
);

// Or create your own `requiredPermissions` method
const requiredPermissions = claimIncludes.bind(null, 'permissions')

app.get('/api/admin/managers',
    requiredPermissions('admin', 'manager'),
    (req, res, next) => {
      // ...
    }
);

Thanks @adamjmcgrath for your clear and fast answer.
I have tested the function using the proposed wrapper to make things clearer and it's working as intended. Let me check it with my team and will close the issue if there are no further questions.


Feat: Make IAM Role Optional

what

Make IAM role conditional on feature flag.
Run make github/init.
Fix unintentional nested array in stream_arns output.
Fix usage snippet such that it conforms with other modules.

why

Not all CloudWatch Logs log group use cases require an IAM role meant for assumption by an EC2 instance. The IAM role should be conditional.
Running make github/init updates GHA workflow-related files.
Running make github/init updates context.tf to the latest distribution from null-label, which now has new features such as the tenant label.

references

https://github.com/cloudposse/terraform-null-label/releases/tag/0.25.0

/test all


dragData for bubble chart

Awesome plugin. Works really well with line charts, however, I can't seem to make it work with bubble charts. Whenever the data point bubble is dragged it disappears.

Glad to hear that you like the plugin!
For now, unfortunately only line charts are supported. With the bubble charts, I guess one would need to discuss the expected behavior while dragging first. Would you like to increase/decrease the Bubble’s size on mousemove up/down or rather shift it up/down/left/right. Probably a mix of left/right click mouse events?
Let me know what you think, then we can proceed.

I had in mind a bubble that's draggable around onClick left/right/up/down.
Something similar to the following
https://ecomfe.github.io/echarts-doc/public/en/tutorial.html#An Example: Implement Dragging
https://www.highcharts.com/products/plugin-registry/single/3/Draggable Points

Should be working with the newest release. See the bubble chart example.


Catch basic impossible dates (pre-1900) and assume a timestamp for da…

…te-only strings.

This PR resolves #603.


Possible to detect change in any column?

Hi Christian,
This may be a feature request,
Is there a workaround to allow detecting change of any column of a row,
and then obtain only the required columns of the changed row?
Thanks!

Would be very useful to also return the rowNumber

rowNumber is retured, is specified in the C# model: check integration test https://github.com/christiandelbianco/monitor-table-change-with-sqltabledependency/blob/master/Tests/TableDependency.SqlClient.IntegrationTest/RowVersionTypeTest.cs

Concerning your initial request "detecting change of any column of a row", simply create e C# model with all properties matching table columns. Then, specify includeOldValue parameter. In this way, the notification will contains new values as well as old values. At this point you can make a manual check to detect which table column has been updated

Nice to know rowNumber is returned.
However,
I have a badly designed table with about 30-40 columns.
With includeOldValues, it would be 60-80 message types.
When trying to update a row, the query actually timeout and failed.
Does that have to do with the fact that too large nunber of columns are specified, instead of a simple SELECT *?

mmm...never tested with a table containing so many columns.
Can you send me the SQL script to create it. So i can reproduce it locally on my PC?

Not really familiar with SQL Server, hopefully the file attached works
You may need to modify for it to work with your setup
arscusttable.zip

If it helps, this is the class:
TableClass.zip


WinUI 0.5 Treeview not yet functional.  Same UWP code causes exception in WinUI.

Describe the bug
The WinUI Treeview code is generating an exception in code that works correctly in UWP.
Steps to reproduce the bug

Steps to reproduce the behavior:

Goto https://github.com/Noemata/RosettaNavigation and create local copy of repo
Build and run TreeViewUwp1 solution.
Observe how it is supposed to run.
Build and run TreeViewWinUI1 solution.
Crashes on load.

Expected behavior
The WinUI TreeView should behave exactly as the UWP variant.
Screenshots
n/a
Version Info

NuGet package version:
[WinUI 3 - Project Reunion 0.5: 0.5.0]

Windows app type:

UWP
Win32

No
Yes

Windows 10 version
Saw the problem?

October 2020 Update (19042)
Yes

Device form factor
Saw the problem?

Desktop
Yes

Additional context
n/a

Do you have easy access to the callstack of the crash? :)

@StephenLPeters , just run the sample for yourself.  I've pinpointed where the crash happens for you in the code comments.  I wish I got this level of information from my clients ;-)  These samples were built specifically to ease the process of vetting the remaining issues with UWP and WinUI.  Run them!

UWP is no longer supported by WinUI 3.  If you can repro the issue on current builds without UWP, please reactivate or open a new issue.
Thanks!


Storing the wall

How should the wall be stored? Should it be an image?

Wall is currently stored in source as one large image. This can be moved to Firestorage eventually.


Controls Player Character

Bind the controls that allow the player to control the Player Character.

Resolved with following commit: https://github.com/BraisGlezArias/TheHeroNeverDies/commit/383ff4c620b0d33ad23dc6979f9e2934ab21bdb0


#1203 Breaks paths to component source

Change Type
Correction
Proposed Changes
After #1203 component source paths are broken.
For example in the Command Docs component source link (https://github.com/huntabyte/shadcn-svelte/tree/main/apps/www/src/lib/registry/default/ui/command) is not longer valid and returns 404 after the rename.

Corrected paths
# old
https://github.com/huntabyte/shadcn-svelte/tree/main/apps/www/src/lib/registry/default/ui/
                                                     ^^^^^^^^
# new
https://github.com/huntabyte/shadcn-svelte/tree/main/sites/docs/src/lib/registry/default/ui/
                                                     ^^^^^^^^^^

Maybe a deployment went wrong or something doing through the source it looks like the paths were corrected already

Hello;
Related: #1204

Not sure how I didn't see this I could've sworn I looked


CPS is displayed in the overview

Bug Report
Please answer these questions before submitting your issue. Thanks!
What did you do?
Open Overview page.
What did you expect to see?
See QPS.
What did you see instead?
See CPS (Command Per Second). The "command" is a concept from MySQL protocol.
What version of TiDB Dashboard are you using (./tidb-dashboard --version)?
master.

Fixed by https://github.com/pingcap/tidb-dashboard/pull/1241


Rolling up to multiple objects via text id field

Hi! Back again with an edge case issue...
I have an object (Charge__c) that has a fake lookup field which is a text field, and can contain the id of one of a few different objects. I started with rollups from that child to one of the parent objects and it worked fine. When I added in another rollup to another valid parent object, I started getting errors when running the rollups. I'm running them in flow using the CMDT action. Both rollups use the text "id" field for the Lookup Field On Calc Item.
Here's the error:
EXCEPTION_THROWN [800]|System.SObjectException: Parent_A__c.Total_Paid_rollup__c does not belong to SObject type Parent_B__c
Here's the stack trace:

Any ideas what could be going wrong?
Thanks!

Ah that’s an interesting one! I will have to take a look, should be an easy fix. Also, hats off to you - love the multiple parent lookup solution!

Nice, thanks! And it's so satisfying to trigger all those rollups via the one apex action, no matter where they are rolling up to!

@solo-1234 is this in the most recently published version (v1.3.23) that you are getting this error? I ask because a fix for this should be in place - it was introduced in v1.3.10  and the stacktrace you've provided doesn't align with the current versions of those methods.

Ah, good point! I'm on an older version, had been waiting to work out the other kinks before upgrading my prod org. I'll upgrade it now and see what happens!

That fixed it! Many thanks!!

No problem 👯


[MPLUGIN-394] Respect input encoding

PluginReport didn't specify an encoding while reading the
Plugin-Descriptor. This causes the fallback to the plattform encoding
and ignores the (user specified) input encoding.
This commits adds an integration test to reproduce the error and fixes
it by using the input encoding for the creation of the used Reader.
The integration tests forces the plattform encoding to CP1252 (Western
European charset used in Windows) while the source (input) and output
encoding is UTF-8.
The source file contains non-ASCII characters which have a different
representation in both encodings.

@SebastianKuehn please also look at #72

@hboutemy, @slawekjaranowski I added an commit to use XMLStreamReader to detect the encoding from the input xml.

great job, thank you @SebastianKuehn


v4.1: switch custom form control

This adds a neat styling feature for the custom form controls.
Markup stays the same as the checkbox but displays it as a toggle switch.

fixes #22090

Dope! Thank you for jumping in on this. I'll take a look at it for our v4.1 ship :).

@mdo @XhmikosR @andresgalante I messed up here :( and cant figure out how to revert it, any suggestions?
Sorry for this... don't understand what wen't wrong

@gijsbotje: you can just do git rebase -i upstream/v4-dev and clean up your local branch.

maybe i did something wrong, but it didn't work...

@gijsbotje: you had messed up your branch. Do not merge from upstream. Just rebase it, if needed.
Now, I cleaned it up and force pushed it. So, make sure you do git fetch origin && git reset --hard origin/custom-switch-form-control before anything else otherwise you will overwrite the cleanup.

@XhmikosR thanks dude, will do
i tried to make a pr for a different issue and my forked v4-dev branch is messed up i think
i'll see if i can fix it all

@gijsbotje / @mdo the solution to modifying the custom-checkbox with .custom-checkbox-switch is super handy.
http://wingman.mediumra.re/components-bootstrap.html#forms

"mdo removed this from Inbox in v4.2 13 days ago" :(

@mdo  @XhmikosR Any chance we could see the toggle-switch in this release?

Personally I'm not sure it's ready yet.

@gijsbotje: will you be able to address @MartijnCuppens's comments above?

sorry got pre-occupied with work. i'll try and fix it before the weekend

i'm struggling a little bit...
as the colour scheme changed, the unchecked state of the switch now doesn't have a gray background. which results into the indicator being not visible until the switch is checked.
to illustrate my issue:

@MartijnCuppens @XhmikosR any suggestions on what colors to use for what?

Yeah, that was the issue I noticed too when I said I didn't think this was ready along with transitioning costly properties.
I'll try to have a look tomorrow, but @MartijnCuppens will probably beat me to it.

would this be ok? i used the color of the border for the indicator

Can you include screenshot of other special control (eg. checkbox) bein on/off/disabled?
Also, is there "disabled + on" state?

@rafalp i've paired the switch with its checkbox equivalent

thanks for reminding me about the :checked:disabled state, forgot about that

@MartijnCuppens fixed your comments as best as I could think of.
main question to resolve is if the color for the circle when it's unchecked is ok

Those are looking very good, thanks 👍

@gijsbotje, the color of the circle looks ok.
IMO the switch looks a bit small, maybe we should try to increase the size a bit?
It would also be nice to have total control over the track width and height and the spacing between the switch and the label text. With 'the track', I mean this element:

Maybe it's possible to tweak this a bit to so that it's possible to set the following variables:

The track width
The track height
Margin between the switch and the label text
The size of the circle

If we have control over this in _variables.scss, it would be easier to control the look of the switch and it would also be possible to make them look like in this PR:
https://github.com/twbs/bootstrap/pull/26691

I would be careful with making switch too big - you'll potentially end with component being too big to use "inline", requiring special positioning or, worse, custom form layout altogether.

Good point, @rafalp. Forget about size increment than.

Yeah, I am thinking now how this component should be sized. Right now its sized same way checkbox is, meaning its interchangeable with it. Perhaps it should also support .form-control-lg and .form-control-sm for extra variety?
Just dropping ideas - it may be too late for this, or material for new PR bumping extra sizing to custom controls?

💡 Support for contextual classes could be interesting, too.

I would do that as a seperate PR as it also affects the other controls

Yeah, I am thinking now how this component should be sized. Right now its sized same way checkbox is, meaning its interchangeable with it. Perhaps it should also support .form-control-lg and .form-control-sm for extra variety?
Just dropping ideas - it may be too late for this, or material for new PR bumping extra sizing to custom controls?

Indeed as you said yourself material for new PR 😄

Maybe it's possible to tweak this a bit to so that it's possible to set the following variables:

The track width
The track height
Margin between the switch and the label text
The size of the circle
If we have control over this in _variables.scss, it would be easier to control the look of the switch and it would also be possible to make them look like in this PR:
#26691

I was fiddling around with the variable custom-control-gutter, but this isn't really the gutter, right? It doesn't define the space between the indicator and the label, but rather defines the space that the label is pushed over. Then the indicator is pulled back the same amount.
So by default the gutter is set to 1.5rem. This gives the checkbox and radio a breathing room of .5rem. Therefore shouldn't gutter be .5rem? And the left properties set to $custom-control-indicator-size + $custom-control-gutter which would result in 1.5rem?
If we implement this change, controlling the track width and space between switch and the label would be much easier. to get it to be accurate in the current setup i would need to do something like:
$custom-switch-width + ($custom-control-gutter - $custom-control-indicator-size)
So if you guys agree with the change to $custom-control-gutter I will make a separate PR with those changes. I can imagine this would have some impact on people that make use of the sass, so that would be your call.
What i'm really struggling with and get wrap my head around is making the height and circle size customizable. I'm currently using the calc css function to get the height and with like this:
calc(#{$custom-control-indicator-size} - #{$custom-control-indicator-border-width * 4})
First the 1rem indicator-size then subtracting the border width * 4 to create space between the circle and the track. This can not be done in sass math as $custom-control-indicator-border-width is a pixel value and $custom-control-indicator-size is a rem value, which are incompatible in sass.
i'm also struggling with the positioning of the circle when you increase or decrease the size of the circle. currently this calc function is taking care of the vertical positioning:
calc(#{(($font-size-base * $line-height-base - $custom-control-indicator-size) / 2)} + #{$custom-control-indicator-border-width * 2})
I'm far from a math guy so if so if someone could assist me in these issues that would be much appreciated.


upgrade hugo version

The rdf4j.org website currently uses Hugo 0.40.1. We should look at bumping version to 0.60 or newer since it has several improvements in markdown rendering.
See https://github.com/gohugoio/hugo/releases

There have been several backward-incompatible changes in Hugo releases. The most obvious is the fact that by default it no longer renders raw html that is included in markdown - something we use in several places. We'll need to adjust our config and/or content to make sure everything still works with the newer Hugo version.


Correct Vim settings

The current settings highlight the 93rd column of all files (it always runs). The proposed fix results in the column highlighting only for *.jl files.

Could there be something else going on here? As far as I can tell from a web search, ftplugin is the recommended way to do filetype-specific settings. Can you find a recommendation that says otherwise?

Looks like b:undo_ftplugin is used in the julia-vim version of ftplugin/julia.vim. Not sure, but maybe the after should be appending to that? Just a guess on a place to look.


Nested demo folders are weird on read the docs

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Execute '...'
See error

Observed behavior
Describe or paste the exact output that demonstrates the bug (for example, the stack trace if it is an exception).
Expected behavior
A clear and concise description of what you expected to happen.
Environment
Operating system: for example: macOS, Ubuntu
Python version: for example: 3.7.2
Package versions: for example: stellargraph==0.8.3, tensorflow==2.0.0
The following Python code can automatically compute the data for this section (for example: run the code in a Jupyter notebook cell, or, on macOS, copy the code and run pbpaste | python - in a terminal):
import subprocess, platform, sys
pkgs = subprocess.Popen(["pip", "freeze"], stdout=subprocess.PIPE, stderr=subprocess.STDOUT).communicate()[0].decode("utf-8")
print("Operating system: `{}`\nPython version:\n```\n{}\n```\nPackage versions:\n<details>\n\n```\n{}\n```\n\n</details>".format(platform.platform(), sys.version, pkgs))

Additional context
Add any other context about the problem here. For example:
Data used: if this occurs with a specific dataset, which dataset and how to acquire it
Screenshots: if applicable, add some screenshots to explain the behaviour

Maybe worth rearranging on ReadTheDocs before they start linking to them.
But divergence between ReadTheDocs and Git repo....
Note there are some links to Notebook CI blogpost..


[stable/drone] bump drone version - CVE-2019-9512 and CVE-2019-9514

Signed-off-by: Rauny Brandão<EMAIL_ADDRESS>What this PR does / why we need it:
Bump drone server and agent to the latest version
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@christian-roggia and @zakkg3

/ok-to-test

/lgtm


Add missing fallback themes config for list operation

Solves

https://github.com/Laravel-Backpack/theme-coreuiv4/issues/38
https://github.com/Laravel-Backpack/CRUD/issues/5184

How
by adding a fallback theme config to ui.php for list operation.

Hey @karandatwani92 thanks for the PR 🙏
How about the other keys ? I know some of them are exclusive of tabler, so they don't make sense to be in ui. But others like body also throw errors on CRUD pages, as reported in #5229
Is there any reason to left those keys out that I am missing ?
Cheers

Hey Master @pxpm . I considered them already and found only CRUD's list view requires classes.___ attribute. Nowhere else CRUD required any attributes like classes.body, classes.sidebar, classes.header, classes.footer.
Thus, no need to add them. 😬

Hey @karandatwani92 I found this in theme coreui-v2 https://github.com/search?q=repo%3ALaravel-Backpack%2Ftheme-coreuiv2 classes.body&type=code
So they may be needed ?

@pxpm let me explain
Why no need for other attributes?
Themes may ask for n number of attributes it could be body or anything. And theme config should only be responsible for those values which the theme requires. Those values must come up with the theme itself.
What fallback attributes should be covered in ui.php
Only attributes that are required by CRUD, PRO, Dev Tools or any Backpack package which has views, so they could match with the theme's classes by overriding default classes.
Till now I have found, only the CRUD's list operation needs attributes which we are adding here.


Get-Credential password confirmation

PR Summary

Get-Credential : Add parameter -ReEnterPassword  to resolve #10625
PR Context

By adding -ReEnterPassword parameter the user will be prompted to re-enter the password and performs a comparison. If comparison fails null is returned and user is notified about password mismatch.
PR Checklist

[ ] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[ ] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[x] Documentation needed

[x] Issue filed:  #6034

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

@iSazonov Thank you for the feedback. I have made the appropriate changes. Please review.
I see the below error message in Linux test. I can see the file is present. What could be casing this issue.
Executing all tests in 'test/powershell/Language/Operators/NullConditional.Tests.ps1', 'test/powershell/Language/Parser/Parsing.Tests.ps1', '/test/powershell/Host/TabCompletion/TabCompletion.Tests.ps1' with Tags RequireSudoOnUnix
Resolve-Path: /home/vsts/work/1/a/bins/publish/Modules/Pester/4.10.1/Pester.psm1:1326
Line |
1326 |  …             & $script:SafeCommands['Resolve-Path'] -Path $unresolvedP …
     |                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find path
     | '/test/powershell/Host/TabCompletion/TabCompletion.Tests.ps1'
     | because it does not exist.

Tests completed in 191ms
Tests Passed: 0, Failed: 0, Skipped: 0, Pending: 0, Inconclusive: 0
WARNING: Tests failed. See the issue below.
Exception: /home/vsts/work/1/s/tools/ci.psm1:672
Line |
 672 |  … ll -ne $_} | ForEach-Object { Test-PSPesterResults -ResultObject $_ }
     |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Parameter cannot be processed because the parameter name
     | 'Error' is ambiguous. Possible matches include: -ErrorAction
     | -ErrorVariable.

##[error]PowerShell exited with code '1'.

@Shriram0908 Please open new issue in PowerSehll-Docs repo and add the reference to the PR description.

Re-entering a password is normally done when creating a username/password, but not when using a password to access resources.  I am not sure how useful this change is, particularly since it is opt-in via a parameter switch.  I would like to get the committee's opinion before accepting the change.

@PaulHigin Have you any concerns about remoting scenarios?

@iSazonov  Good question.  I think this should work fine over remoting since the system will make remote host callbacks for prompting as before, but it would be good to check.

@PowerShell/powershell-committee reviewed this, we agree on adding this capability, however we suggest that the switch should be called -ConfirmPassword

@PaulHigin Could you approve the PR?

@Shriram0908 Please rebase to pass CI-static.

@Shriram0908 Please rebase to pass CI-static.

@iSazonov I rebased the branch but now mac os tests are failing. I don't think the failure has anything to do with my changes. I checked in Azure DevOps pipeline the error is related to test-connection and update-help

@Shriram0908 CI-MacOS failure is know issue.

@Shriram0908 Please rebase to pass CIs.

@Shriram0908 Please rebase to pass CIs.

@iSazonov Done... :)

@Shriram0908 Thanks for your contribution!

Thank you @iSazonov and @PaulHigin for your feedback.


Getting Started guide - deleting records does not require a fetch

The getting started guide states that "we must first fetch a record from the database and then call Friends.Repo.delete to delete that record", but the documentation for Ecto.Repo.delete specifies that a struct or changeset may be used, so there is no need to first fetch the record when a struct is used.

If you already have the struct, then you don't need to fetch it, but most usually you do need to fetch it, yes. We will update it accordingly, thank you!


Issue With create_tflm_tree.py

I am running an automated process that pulls TFLM into an existing VIsual Studio project. It was working before (a few weeks ago), but am seeing it break now.
The sequence is as follows:

The repo is cloned into a temp folder
I run python tensorflow/lite/micro/tools/project_generation/create_tflm_tree.py, [destination folder]
This eventually internally calls make -f "tensorflow\lite\micro\tools\make\Makefile" third_party_downloads, where an exception is thrown at line 225 of create_tflm_tree.py where make is run for this command.
The Makefile loads and crashes in tensorflow/lite/micro/tools/generate_cc_arrays.py", line 26: from PIL import Image throws an error, ModuleNotFoundError: No module named 'PIL'. This python file appears to be called in tensorflow\lite\micro\tools\make\helper_functions.inc line 499

There are several articles about the PIL module not being found, and how to fix it by installing pillow. Does pillow need to be manually installed to replace PIL for tflite-micro?
Note; I just created an empty python file and added import PIL and ran it by itself, and it generates the same error. So there seems to be an issue with package setup, but this also doesn't seem to be inherently resolved by the automated build process.

Yes, pillow needs to be manually installed. And you might need Wave as well.
Here is where we have updated the CI to install these two dependencies:
https://github.com/tensorflow/tflite-micro/blob/1ef3830fa693bc5ab745100e53d55da97b4f8c29/.github/workflows/ci.yml#L40-L41
See related issue: https://github.com/tensorflow/tflite-micro/issues/401

Thank you, @advaitjain! So if I understand correctly, that action will only run on a pull request when building with bazel on ubuntu, is that correct? I can add this to my automation process.

Correct, I only linked to the workflow yml as an example of what you will need to do.

Update:
I am using msys64 for make, and attempting to manually install Pillow in this global environment, using mintty with the command "python3 -m pip install Pillow". This command works fine when running in DOS which uses a different Python interpreter, but using the interpreter in msys64 seems to have issues caused by pip trying to build Pillow on the fly in a Windows environment as explained in this link:
https://github.com/python-pillow/Pillow/issues/2860
I am going to try to get Pillow installed without being built, just thought you might like to be aware of this potential issue building on Windows.


fixing slight issue with banner height

Kwenta Futures banner is expanding on the Markets page specifically.
Description
removing the hardcoded height fixes the odd behavior on the Markets Page
Related issue
N/A
Motivation and Context
consisitency across pages
How Has This Been Tested?
tested this locally on dev and against production (in elements tab)
Screenshots (if appropriate):
this:

to this:

just realized this is still in draft


Build errors on Mac in osvr_log_to_csv

Reported by a user, CMake in build log reported AppleClang 7.0.2 (so newer than Travis-CI).  Key pieces from the errors:
OSVR-Core/apps/osvr_log_to_csv/osvr_log_to_csv.cpp:57:25: error: no type named 'RowProxy' in 'osvr::util::CSV'
inline osvr::util::CSV::RowProxy &&
       ~~~~~~~~~~~~~~~~~^

OSVR-Core/apps/osvr_log_to_csv/osvr_log_to_csv.cpp:58:29: error: no type named 'RowProxy' in 'osvr::util::CSV'
operator<<(osvr::util::CSV::RowProxy &&row,
           ~~~~~~~~~~~~~~~~~^

OSVR-Core/apps/osvr_log_to_csv/osvr_log_to_csv.cpp:68:17: error: no member named 'row' in 'osvr::util::CSV'
 g_csvOutput.row()
 ~~~~~~~~~~~ ^

Bit puzzling, since it passes travis and other CI fine. The first two, OK, perhaps it doesn't like referring to nested classes with rvalue refs, could try to work around that and pull the class out, but I don't have any clue why it won't handle the last one, aside from the fact that it's a member function (of a global) that returns the same nested class referred to in the earlier messages.
Reproduction help from someone with a Mac and willing to dig into debugging compiler incompatibilities appreciated!

What cmake options were used in this build?

I ran a fresh build:
 $ clang++ --version
Apple LLVM version 7.0.2 (clang-700.1.81)
Target: x86_64-apple-darwin15.3.0
Thread model: posix

$ cmake .. -DCMAKE_PREFIX_PATH=~/osvr \
    -DCMAKE_INSTALL_PREFIX=~/osvr \
    -DCMAKE_OSX_ARCHITECTURES=x86_64

And compiled and linked successfully.
I did encounter an unrelated issue where it was finding old header files from a Homebrew installation of OSVR-Core, however. After removing those cached files, everything worked okay.


Review staging models

There are a few issues with the staging models as they are right now. Some of those issues are preventing the out-of-the-box compilation of those models. They generally related to CTEs not closing correctly.
We could also take this opportunity to support functions from other data warehouse providers (currently the functions are BQ-specific). Plus we could also take this opportunity to reinforce the use of style guide principles in those models.
┆Issue is synchronized with this Jira Task by Unito

Done, and the current release uses cross-database functions as much as possible, and supports Snowflake DW as a target warehouse too.

Done, and the current release uses cross-database functions as much as possible, and supports Snowflake DW as a target warehouse too.


Communicate to user overall increase when setting yearly increase

The ETM features many sliders which allow the user to indicate the yearly growth of, say, useful demand. Since these functions are exponential, it might be instructive to communicate to the user the cumulative increase or decrease as a result of the yearly increase set.
Such an implementation (as has already been done for carrier costs) could look like this (calculated for a 2050 scenario):

A 2.0% yearly increase might seem conceivable at first, but that actually amounts to a 100% overall for 2050. Especially for efficiency improvements it might be good to have this information - a 100% efficiency improvement for ICE engines seems quite unlikely, a 2.0% yearly increase perhaps less so. Of course it should be communicated to the user that this number is an increase.
I propose adding this to at least the mobility growth and efficiency improvement sliders in transport, but perhaps to all useful_demand updating and efficiency_improvement sliders.
@AlexanderWirtz , @jorisberkhout @ChaelKruip @dennisschoenmakers  do you think this is a good idea, and if so, do you have a preference for where to implement this?

I think this is a nice idea @RobTerwel. Big question is when we can find budget to pick this up.

I like it, @RobTerwel ! Putting on hold for now for the reason mentioned by @ChaelKruip .


⚠ ClamAV has degraded performance

In c7b780f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 54 ms

Resolved: ClamAV performance has improved in 281dc24.


feat:added refresh button for selected bucket

added refresh button for selected bucket
some change layout design

:tada: this issue has been resolved in version 1.0.0 :tada:
release is available on GitHub


reverse lat/long in the settings page

I think the titles are incorrect on your fields regarding LAT/LONG on the Settings -> Account window. I put in our lat and long (then log out and back in) and it puts me in Antartica. If I reverse what our LAT/LONG should be it puts me in the right spot. I double checked using this site http://www.darrinward.com/lat-long/
Can someone else check and confirm they have the same problem?

Yes, I can confirm that it's a known problem in version 3.1. It's already fixed in the code. Just need to wait for the next release.


Update Field.md

Fix the text, it seemed like a mistake.

Codecov Report

Merging #2811 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #2811   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files          67      67
  Lines        1395    1395
======================================
  Hits         1395    1395

Continue to review full report at Codecov.

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


[Windows] Fix Get-CodeQLBundleVersion output

Description
The line contains incorrect statement that cause the errors:
https://github.com/actions/virtual-environments/blob/b0f3f989abd744ea6bfcaf62fc5decf19f8853b6/images/win/scripts/SoftwareReport/SoftwareReport.Tools.psm1#L37
char:36
+     $CodeQLVersion = $($CodeQLPath version --quiet)
+                                    ~~~~~~~
Unexpected token 'version' in expression or statement.

Import-Module : The specified module
'..\virtual-environments\images\win\scripts\SoftwareReport\SoftwareReport.Tools.psm1' was
not loaded because no valid module file was found in any module directory.
At line:1 char:1

Related issue:
https://github.com/actions/virtual-environments-internal/issues/1213
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2016, windows2019


Android Basics: Activities and intents - Git commands to use the starters branch

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-activities-intents
(and others)
In which task and step of the codelab can this issue be found?
In the "Get the code" section(s) it mentions to download the github code in zip.
Describe the problem
I fetched the code using "git clone". This the usual way. It is not natural to fetch code from github using zip (!!). I realized that the examples are using the "DetailActivity.kt", which is not the "main" branch.
Of course, you can point to the other branch, but this is not obvious for the novice or the fast reader.
Please, make sure the examples are in sync with the "main" branch or indicate short instructions to point to a specific branch otherwise (e.g.: git clone <repo> & git checkout -t origin/starter) .
Example:
``
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

@migueldiez
Thank you so much for reporting this issue. This is fixed and will be published soon.

@migueldiez
Thank you so much for reporting this issue. This is fixed and will be published soon.


Question: GPortal

Question on: _docs/geyser/supported-hosting-providers.md
Need to contact support to open a UDP port, then use that as the bedrock port in the config. Also, uncomment bedrock address and make sure it is set to your Java server IP.
^^^^^^^^^^^^
Unfortunately, this is not correct. One does not need the support for the port.
You do not have to address the support.
Each MC server has 4 ports free after its query port.
So if you have 34565 as query port. You can use 34566 for Geyser or 34570 for Geyser.
The IP address must always be entered, because the GameClouds of GPORTAL do not allow to bind to <IP_ADDRESS>.
Your Quert port will always follow the scheme XXX65. The additional free ports are XXX66 - XXX70 respectively.
You can use these open ports for your mods, PlugIns, etc.. ... use.
Found on: https://www.g-portal.com/wiki/en/what-ports-can-i-use-for-minecraft-mods-and-plugins-that-require-an-extra-port/

Hey, feel free to make a PR to this file (https://github.com/GeyserMC/GeyserWiki/blob/master/_data/providers.json) to adjust the GPortal entry :)

Closed by #210


what is this?

This is an api wrapper the Noise Play Vlog Edition action camera. It supports remote access mode to control the camera. The camera creates a hotspot connection to which a mobile device connects and can control the camera using the official app.
But the app, is in short, bad. Also its been pulled out from the Google Play store.
Hence, I've been reverse engineering the app and making wrapper around the HTTP requests to control the camera


feat : Scaling and Translation on BoundingBox (AABB)

Add a Bounding Box for Component Particle and Component Line to make them take in count frustum to draw or not
Put the value "Draw Bouding Box" in ComponentTransform to true allow you to see it and scale or translate it as you wish
If you wand to load an old scene and apply the serialization modification, take a look at "InternalSave()" and "InternalLoad()" of ComponentTransform.cpp

As an outsider to this task, would you mind explaining why the bounding box in the transform may need to me moved or scaled? Conceptually, at least to me, that bounding box is kind of an internal thing of the engine, which represents the actual bounds of whatever the object has (be it a mesh or a UI element). As such, it should be immutable. To me, what you're doing here looks deserving of a different, new component (I'd recommend taking a look a Unity's Bounds). Let me know what you think ;)

Majoritarily for ComponentLine and ComponentLine but I add it to all of the Transform because
Firstly, I didn't need to repeat it to particle and line (and hypothetically for a new features). Particle and Line need it to don't be draw while not in the frustum because actually both of them, not having a mesh, didn't create a AABB depending of their vertices or more.
Secondly, sometime you will need to modify the bounding box for particle or componentLine to make it modifiable depending of how far the particle is going etc.
In our case with our implementation I don't think boundingBox should be immutable but I mostly do it like that because I think it's the faster an easiest way.
Hope it answer to everything !

Hope it answer to everything !

You did, thank you very much for taking the time to do so 🙂
I still don't quite understand why the bounding box needs to be modified for the particle. If it's tied to it, and the transform is moving, shouldn't the bounding box be moved implicitly as well? Or is that not the case currently? And even so, I'm not sure it should be modifiable by the user of the engine throught the UI. Would you mind giving me a use case for this?
But aside from that, I see why you did it this way, and agree with you that it's the best approach given our time constraints.

Hope it answer to everything !

You did, thank you very much for taking the time to do so 🙂
I still don't quite understand why the bounding box needs to be modified for the particle. If it's tied to it, and the transform is moving, shouldn't the bounding box be moved implicitly as well? Or is that not the case currently? And even so, I'm not sure it should be modifiable by the user of the engine throught the UI. Would you mind giving me a use case for this?
But aside from that, I see why you did it this way, and agree with you that it's the best approach given our time constraints.

Actually by default particle didn't have bounding Box because the bounding box of all models are create depending of their vertices but particles didn't have so the bounding box was 0 and then they was always drawn. For an exemple, if you make some particle falling from the ceiling or like lava particle that pop from the ground, if the bounding box is to small and you didn't have it in your camera frustum, you will not draw the component and then don't see the particle so make able to modify the it will draw it also if it's not in the camera and that can not be done before except with doing a big bounding box to all particle.
Also maybe it was confuse but by default the box is linked to the transform so moving it will move the box by the same case so don't worry this is not changing, it's just adding a possibility to move it independently !

it's just adding a possibility to move it independently !

I'm still unsure if that's something we want, but it's a very minor thing regardless. See if you can take a look at my other comments and I'll approve the PR. Thank you very much for taking the time to answer my questions 😄

Hi Alexis! Some things i've found. When entering play mode, the colliders for bix seem to be displaced, don't know if this is also happening to the other objects.

Also, the bbox origin por particles isn't centered, the pivot point should be the center of the bbox, not the lower left corner of the bbox.

Requesting changes for what I mentioned before

both fixed normally

looks good, but i have a question apart of the code, in this video we can see that the upper right particles stop rendering when we don't see the bounding box, but we should keep seeing them, is that the behaviour we want? or instead of having only into account the component we should have also in mind the particles itself? correct me if i'm wrong
particulas.mp4

It's the behaviour we want, It's just I didn't modify all the particle of the level manually right now, I add a bounding box that is a cube 1,1,1 so if we want to see it when the camera is down, we need to scale the bounding box in the bottom and then scale the bouding box to the particles itself like you said but it's only the component that will manage that size depending of where and how far the particles are intended to go !


Force the game to run stretch fullscreen

Hi there!
I'm running 800x600 game on 1920x1080 with DDrawCompat.
Everything works fine except the black border appear on the right and left side.
I want to the game to run forced into stretched fullscreen without modifying my graphic card settings. Because I've tried to modify my graphic card setting scaling into "Scale Fullscreen" option and the game performance drastically went down.
I've zero knowledge about ddraw and dx but I'm not really bad at cpp and I wish to contribute if possible.
Looking forward for the solution
Thanks for your awesome works, keep it up!
OS: Windows 10 Pro
Graphic Card: Intel HD Graphic 530 / Nvdia GeForce GTX 950M
EDIT:
Seems the lag is not caused by stretching game with my graphic card settings, but it's seems come from latest DDrawCompat, (I use it from the source, I compile the source directly in order to remove the logs).
I tried v0.21, the game run smoothly even with "Scaling Fullscreen" option turned on in within my graphic card settings. But still..
If you have pointer which code I should edit in v0.21 to stretch the game!
Or even better, make the latest one work proper as v0.21 do, because the game really stutter
Any both of them would be highly appreciated.
Thanks in advance.

I've been a bit inactive on this project lately, but I can certainly give you a few pointers.
Implementing manual stretching to native resolution is something I have been planning to do for the next major version (if I ever get to v0.3.0 first), but it's a lot more effort than it might first appear.
You'd have to override the resolution setting in DisplayMode.cpp (maybe it was still in DirectDraw.cpp in v0.2.1) and convince the game it's running at the correct resolution (IDirectDraw::GetDisplayMode, EnumDisplaySettings(Ex), maybe even SystemParametersInfo and others) without confusing the original ddraw module about the same (most likely with some extra IAT hooks for that). If you lie to ddraw about the resolution it will most likely end up in the primary surface creation/usage failing (resolution not matching the display mode).
Then you'd have to use some kind of scaling when presenting the internal primary surface to the real one in RealPrimarySurface.cpp compatBlt function. If you're fine with the builtin bilinear/nearest pixel scaling offered by DirectDraw then this is pretty simple, just use the Blt method of IDirectDrawSurface. Unfortunately the actual scaling method (bilinear vs nearest) is up to your GPU driver, but you can possibly influence it by changing the flags in the driver's Blt callback, which is now hooked in DeviceFuncs.cpp (though it wasn't in v0.2.1).
If you only need upscaling, the mouse would need to be restricted to the original resolution rectangle via the ClipCursor win32 function, to preserve the original mouse movement speed and to avoid having to convert mouse input coordinates.
Then if hardware cursor is used, you'd have to hide it and draw the cursor yourself before presentation, somewhere during compatBlt in RealPrimarySurface.cpp I guess, and make sure the primary is updated frequently enough (at least whenever the mouse is moving) via some timer.
Possibly alt-tabbing may need some fixes after all that in ActiveAppHandler.cpp to reset everything correctly when switching back and forth between other applications.
As for fixing the performance issue, I can possibly help with that if you let me know which game you're having problems with (assuming I have the game and can reproduce the issue). You can also try to debug it yourself, or at least narrow down the problem to a single commit using git bisect.

Hi! Thanks for your reply!
To be honest, after I open both sources, I would prefer to use latest source instead v0.21 lolol Well, I'm not good at c++ and ddraw / dx in the first place tho.
Anyway, the game what I was running is O2Jam.
I don't know whether you ever heard this game but this is an online multiplayer rhythm music game which heavily depends on stable performance.
There are several version out there you can try, such as O2V3, O2INT and O2PF, all of them are fanmade servers . This game originally made between Windows 2000 and Windows XP era.
Let me know if you are interested for the performance fix.
Meanwhile, I will check all the hints you told me on my free time.
Once again, Thanks in advance!

Hello @LumineBot, you might want to look at this thread here: #9.  That thread is about O2Jam also.  In addition, DxWnd has the ability to stretch a program to fill the screen.  I posted a download here compatibility on o2jam that should allow O2Jam to work with DxWnd.

@elishacloud Hi! thanks for the reply.
First of all, the issue that you referenced seems has nothing to do with my issue. Embedded WebControl ingame seems run perfectly starting with 0.21, none of my friends facing such issue.
Also I prefer to use DDrawCompat over DxWnd as it provide minimalist solution (only single binary, ddraw.dll without config files and it's open source!). This allow me to distribute easily among my friends, it also appear easier to maintain as it does not contains large codebase.

Yeah, I've worked on O2Jam issues before, but I'm not having any performance issues with the latest experimental release, either using my dedicated AMD or integrated Intel HD GPUs.
Do you know if the issue is specific to your Nvidia GPU? Can you try switching to Intel HD? Since you seem to have a mobile GPU I guess you have some kind of Optimus setup, so that may complicate things, but I guess it's possible to configure it to run on Intel only?

Display scaling has been added in v0.4.0. As for O2Jam, it went on my no-support list, along with all other online games.


Updated OpenMedia.4KStogram to <IP_ADDRESS>80

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Failed to validate against schema associated with property name 'Installers'
Failed to validate against schema associated with property name 'ProductCode'

We're looking at this.

/AzurePipelines run

/azp run


Querying dependecies

Hi Christoph,
I'm just trying to figure out the best way to query dependencies for a given application (in my case to make it standalone, but I guess the process is the same as for any given MINGW-package). There seem to be a lot of possibilities and I'm having a hard time to figure out the most secure variant, so since you seem knowledgeable about this topic maybe you can help me out a bit:

in your dllcheck command you're using objdump -p and then filter the output with the string "DLL Name:" which seems to work well but also a bit excessive.
on Linux I learned people use ldd. The version shipping with MSYS2 does not seem to work well but there is also mingw-w64-ntldd-git by @LRN which seems to perform quite nicely when used with the recursive option (ntldd -R)
last but not least I found cygcheck which is part of cygwin and seems to work perfectly for MINGW64 binaries (but apparently is broken for MINGW32 binaries?).

Any ideas on what method should be preferred to detect dependencies?

I have to admit I just tried a few things and used the easiest one where the result looked right to me.
What are you trying to do?

I'm basically trying to find a solution to reliably check if all dependencies for an application (specifically Inkscape) were correctly copied to the distribution directory.
I'm not exactly missing anything, but want to figure out the most reliable (and ideally most elegant) solution for this purpose. As there are a lot of options but none of them really seems to be tailored for this job.
My personal observations so far:

cygcheck seems like the easiest. It automatically finds dependencies recursively and excludes a lot of noise from Microsoft libraries (ext-ms-win-advapi32-registry-l1-1-0.dll and similar) that is usually not required. Unfortunately I have to rule it out as it's broken for 32-bit executables.
ntldd seems like the tool for the job. It also automatically finds dependencies recursively but includes all the noise from Microsoft libraries, so needs a lot of filtering to produce useful results. A minor downside is that it's an additional package.
objdump seems like the "hard" way and does not recurse automatically but obviously seems to work quite well, too, if used correctly.

Feel free to improve it if you find any problems.
I'm using a similar script in quodlibet to detect missing libs and to delete the ones not referenced by anything and it worked well so far: https://github.com/quodlibet/quodlibet/blob/master/win_installer/misc/depcheck.py
With quodlibet we also use "installed tests" of some sorts where we use the installer to run our test suite which covers various things which could go wrong there like missing audio/video/image decoders etc. That's been really helpful in that I don't have to worry about basic things being broken due to packaging errors.

I actually forgot to report back... (thanks for reminding me ;-) )
I decided to go with ntldd now as it automatically searches the path for dependencies. You can find the script I wrote at https://gitlab.com/inkscape/inkscape/blob/master/buildtools/msys2checkdeps.py.
It can be used to list dependencies for a single executable or a (distribution-)folder. It will show which libraries require a given dependency and can also be used in CI to check for missing or unnecessary libraries.

Cool, I'll give it a try.


Scoop direct install should follow 302 redirects

I already saw this issue #3155 but it’s not exactly the same. You can get similar errors when trying to install from JSON with a URL shortener.
scoop install https://opbs.at/cli-win
Couldn't find manifest for 'https'.

It is not possible in powershell PowerShell/PowerShell#2896
At least not without workaround, but workaround is not working for all use cases.

Sorry i was attached to wrong part of your problem.
Weird. This should work.
https://github.com/lukesampson/scoop/blob/b9b48182b4d921aa92e9b3e53ab2d2c376d467d7/lib/manifest.ps1#L13-L26

Thank you @Ash258 🙌
I also encountered this event in Emacs, but your comment helped me solve it by adding a file and copying the source piece 👍


AA Report is not included in the download package

The CQD Teams Auto Attendant & Call Queue Historical Report is not included in the downloadable package from GitHub https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/blob/live/Teams/downloads/CQD-Power-BI-query-templates.zip.
/Kenneth ML

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

ID: 628345c6-f5f9-fe8e-bfff-faea00a8a25c
Version Independent ID: 30f2d051-5af5-0063-7099-38b5b08ec1e4
Content: Auto Attendant & Call Queue Historical Report - Microsoft Teams
Content Source: Teams/aa-cq-cqd-historical-reports.md
Service: msteams
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@KennethMLdk Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Questions for meeting

What constraints are there on the user and data?

Can you have an Experience without a Location? Researcher? Note? Media?
Can you have a disconnected note, object, location, person?
If so, do these belong to the project? Experience? user?
Can you create an Experience for a place or time that you are not at?
How will users collaborate with each other?
Are the app users exclusively researchers? Is there any public facing exhibit?
How should the user encounter the moderation delay on uploaded media?

This impacts the data structure and the user interfaces. Workflow is fine to adjust, but usability is a higher priority on mobile.


save to folder doesn't work outside network page

From @friendica on September 5, 2013 21:30
Via mrjive:
@Mike Macgirvin a small bug: if I click on "SAVE IN A FOLDER" (sorry if the desccription is not correct, I'm translating from interface which is in italian and I couldn't find quickly wher to change it) it works only if I am in the "NETWORK" while it doesn't if I see the post via the "LINK TO ORIGINAL"
Copied from original issue: friendica/red#113

From @friendica on September 6, 2013 12:1
This is actually a very problematic bug. The functions to perform save to folder and tagging (and several other "actions on posts" ) are contained in jot-header.tpl, which is loaded whenever the status editor is loaded. We are going to have a lot of pages which allow conversation tools but which don't have a status editor. This makes "reshare" problematic in particular since it loads into the status editor on the same page. We may require a completely hidden status editor which we can popup if doing a reshare. Otherwise we have to pull all the javascript "actions on posts" into a separate file which is loaded with any editable conversations - which is a good idea anyway. We may in fact need to do both, or disable reshares outside pages with a status editor; which seems awkward and inconsistent.  It's a fair chunk of work and requires a bit of planning no matter how it's done.


Support dot notation in variable names

import?abc.def.ghi=>1
=> var abc = {};
abc.def = {};
abc.def.ghi = 1;
import?window.jQuery=jquery
=> var window = {};
window.jQuery = require("jquery");

Does this PR handle case of injecting multiple globals? For example: import?window.jQuery=jquery,window._=lodash

@korya It does handle multiple globals on window. I've added tests that illustrate various cases.

Great! Thanks!

@sokra What do you think? can it be merged?

It looks like this code change may override some variables which are already exist.
For example i have global / local variable noty.
import?noty.theme

will be compiled to
var noty = {};
var noty.theme = __webpack_require__(XXX);

@fredriks, looks like @artyomtrityak is right (disclaimer: I'm looking purely at the code). At the moment you have a specific check for var window (window || {}), perhaps that could be changed so it works on all variables?

Aye, good point! I'll update the PR with a more general approach.

looks good for me

Thanks a lot @fredriks!

Released<EMAIL_ADDRESS>with this, thanks again @fredriks, this has solved a lot of issues.


Bump go, controller-runtime and flux components versions

Signed-off-by: Sanskar Jaiswal<EMAIL_ADDRESS>

Please rebase with main, I've merged your Makefile changes.


ExampleScript.ps1 Imports xDiskImage instead of xStorage

The title should be self explanatory. The ExampleScript.ps1 does not work, as it imports what appears to be an old name for this module, xDiskImage, instead of the current name, xStorage.

I have resolved this as part of #53.


Package manager - add a package modal design updates + errors

Describe the bug
Design for Fluent consistency and the addition of error states in the "Install" or "Add a package" modal in Package manager.

The text of the modal and buttons has been updated.
However, it is not currently possible to easily identify if the package failed to install because of the name or version.
This will have to be a follow-up item.  I will open a second ticket to track that in r13.


fix issue 35

This patch fix this issue
https://github.com/a-vis/regl-line2d/issues/35#issuecomment-417993375

issue https://github.com/a-vis/regl-line2d/issues/35

Fixed in 3.0.11. Thanks for the PR!


train_task_classify is searching for .rec in wrong folder

When I am running train_task_classify.py
https://github.com/spcl/ncc/blob/4e9eaeb6e7d3687cc73e73ae0fdf513f4e429e83/train_task_classifyapp.py#L285-L291
This assertion is failing (seq_files list is empty).
This is because the .rec files are generated in separate folders prefixed by seq.
https://github.com/spcl/ncc/blob/4e9eaeb6e7d3687cc73e73ae0fdf513f4e429e83/task_utils.py#L259-L269
But train_task_classify is searching for .rec files in the original IR folder ir_train.

Hello, @sipian
I had the same problem and used a workaround described here, the second message: https://github.com/spcl/ncc/issues/4
i.e. just change a directory.

Thanks for reporting @sipian and thank you for mentioning the workaround, @natalymr.
I'll try with a clean build and replace the lines as necessary.

@sipian Did #9 fix the issue?

Closing the issue due to old age. Please reopen if necessary!


Add curl to test runner

cypress removed it from the base image :shrug:

/lgtm

/retest

/retest


Simplify readme

If I understand the dependency injection correctly, the example of the readme ca be simplified a bit: around the FloatingActionButton:

skip using a MomentumBuilder
getting the controller directly via Momentum.controller<CounterController>(context)

Does this make sense? Or did I miss something?

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   99.61%   99.61%
=======================================
  Files           3        3
  Lines         514      514
=======================================
  Hits          512      512
  Misses          2        2

Continue to review full report at Codecov.

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

It makes sense. I just wanted to show off Momentum's skip rebuild functionality using MomentumBuilder :)


proposal: strings: add Reverse function

This proposal aims to simplify string reversal tasks in Go by including a dedicated method in the standard library strings package.
Reversing a string is a very common operation in many situations. Currently, Go does not provide it in the standard library. Gophers need to write extra code to implement this functionality themselves in every project.
Adding a Reverse() method in the strings package could significantly improve productivity and code readability when string reversal is needed. The method would take a string as input and return its reversed version without mutating the original. It is a frequently needed low-level string operation suitable for including in the standard library.
The implementation could use a slice swap approach like:
func Reverse(s string) string {
  runes := []rune(s)
  for i, j := 0, len(runes)-1; i < j; i, j = i+1, j-1 {
    runes[i], runes[j] = runes[j], runes[i]
  }
  return string(runes)
}

Or consider performance and perform different processing based on string length
package strings

import (
 "sync"
 "unicode/utf8"
)

const threshold = 100  // This value needs to be discussed

var runePool = sync.Pool{
 New: func() interface{} {
  return make([]rune, 0, threshold)
 },
}

// Reverse returns the inverted string of s.
// Implemented through slice inversion, directly
// inverting bytes for small strings.
// Using rune array inversion for large strings.
func Reverse(s string) string {
 if len(s) == 0 {
  return ""
 }

 if len(s) < threshold {
  // Short string direct inversion
  bytes := []byte(s)
  for i, j := 0, len(bytes)-1; i < j; i, j = i+1, j-1 {
   bytes[i], bytes[j] = bytes[j], bytes[i]
  }
  return string(bytes)
 }

 // Inverting Long Strings with a Run Array
 runes := runePool.Get().([]rune)
 runes = runes[:0]
 runes = append(runes, []rune(s)...)

 for i, j := 0, len(runes)-1; i < j; i, j = i+1, j-1 {
  runes[i], runes[j] = runes[j], runes[i]
 }
 s = string(runes)
 runePool.Put(runes)
 return s
}

It has already been proposed and rejected in the past. See this comment in particular.

It has already been proposed and rejected in the past. See this comment in particular.

oh okay. Thanks. I will close this issue


Add isScrollingDisabled prop to NumberField to prevent value change on scroll

Closes https://github.com/adobe/react-spectrum/issues/5676
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue.
[x] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[x] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

Visit http://localhost:9003/?path=/story/react-aria-components--number-field-example&args=isScrollingDisabled:true&providerSwitcher-express=false&strict=true
isScrollingDisabled prop set to true and verify that scrolling does not change the value.
Prop set to false and ensure default behavior.

🧢 Your Project:

We are looking for this change as well, subscribing!

GET_BUILD


Add Conda Feedstock

For those who use the conda build system, is there a conda-forge feedstock?

Ah it looks as though the build failed and I didn't know why. If it fails again I'm now even further removed from understanding building pyhton x typescript packages so here's hoping it's not too tricky to solve

https://github.com/conda-forge/staged-recipes/pull/21385

This package is amazing and I'm tired of having it be the only package under "install with pip" section in my environment.yml

Perhaps a good idea would be to add a meta.yml to the repo so it stays up to date? I'm not sure whether my pull request will stay up to date.

Please see my attempt. I'm willing to amend my PR to make you maintainer.

yes please

Perhaps a good idea would be to add a meta.yml to the repo so it stays up to date? I'm not sure whether my pull request will stay up to date with the pypi version.

Maybe I'm misunderstanding your concern, but once the PR is accepted it will be moved to a diffrent conda forge repo and a bot will autodetect pushes to pypi and update conda-forge.

re the build system, I have experiments to run today, but towards the end of the week iwll likely have some time to look into it. Of course if you feel particularly inspired I'd also be happy to accept PRs that successfully update to the new system.

Yeah I'm also still getting build errors. As you can conclude I'm a heavy user of the conda infrastructure but haven't maintained packages myself.
I'll leave my pull request for now and let you focus on yours.
I checked the build logs for linux:
Don't know if you have tried to comb through the linux64 logs?


Bboxes do not contain center point

Hi,
The SOON dataset is offering bboxes defined as polygons of (heading, elevation) points. A predicted point is considered as valid if it belongs to its related polygon.
However, it seems that the polygons do not always contain the ground truth points.
The following code provides a success rate of 55.8% :
import json
data = json.load(open('data/task/SOON_train.json'))
total = 0
success = 0
for soon_item in data:
    for bbox in soon_item['bboxes']:
        t =  bbox['target']
        polygon = Polygon([
            [t['left_top']['heading']/ (2 * math.pi), (t['left_top']['elevation']+math.pi)/ (2 * math.pi)],
            [t['right_top']['heading']/ (2 * math.pi), (t['right_top']['elevation']+math.pi)/ (2 * math.pi)],
            [t['right_bottom']['heading']/ (2 * math.pi), (t['right_bottom']['elevation']+math.pi)/ (2 * math.pi)],
            [t['left_bottom']['heading']/ (2 * math.pi), (t['left_bottom']['elevation']+math.pi)/ (2 * math.pi)],
        ])
        p = Point(
            t['center']['heading']/ (2 * math.pi),
            (t['center']['elevation']+math.pi)/ (2 * math.pi)
        )
        success += int(polygon.contains(p))
        total += 1
print(success /total)

I am using the normalization as described here: https://github.com/ZhuFengdaaa/SOON/blob/master/r2r_src/utils.py#L156

This is a serious problem. This problem occurs when the boundaries of a bounding box are across 0 (360) degree (e.g., left side<0 and right side > 0). The center can be regarded as "outside" the bounding box, even if it is inside the bounding box. We are still looking for a solution to this problem, and we welcome any improvement.

If I understand correctly what you mean, the right side should be on the left side in this example:

So I guess one way to fix the dataset is to attempt every transformation until the point belongs to the bounding box.
I am not sure it would be enough though.


Restore patch in ciliumnetworkpolicies/status ClusterRole

The PR restores a missing verb (patch) in cilium-operator ClusterRole template.
The permission was removed in https://github.com/cilium/cilium/commit/10a96dd2659d5a79a10fc687a435ce8bb086928c, but the cilium operator periodic GC needs that to patch the status.nodes field in the ClusterNetworkPolicies when option.Config.DisableCNPStatusUpdates is set to false and operatorOption.Config.CNPNodeStatusGCInterval is greater than 0.

/ci-gke

/test-1.16-4.9

/test-1.24-net-next

I think that the ConformanceGKE CI failures (Timeout waiting for flow listener to become ready) are not related to the PR.

I think that the ConformanceGKE CI failures (Timeout waiting for flow listener to become ready) are not related to the PR.

Yup, the <IP_ADDRESS> related tests are sometime flanky, I just re-run it to make CI happy.

ci-aks is disabled manually, ci-l4lb is not required as there is not related changes. Marking this ready to merge as review is in.

Temporarily removed the v1.10 and v1.11 needs backport because of https://github.com/cilium/cilium/pull/20412
It seems that this commit https://github.com/cilium/cilium/pull/19673/commits/9c0965e1a345f553e6e739dc71cb458f074f11ab has not been ported to v1.10 and v1.11, thus this PR shouldn't be backported either?

Spoke offline with @aanm: the offending commit was not ported to v1.11 and previous versions because of its potential big impact. Thus, this PR should be backported to v1.12 only.


updated dingo/blueprint to satisfy requirements of Laravel 6

@specialtactics this is also required

please reopen this

Hey @lloricode I'm actually not able to re-open because the repository which this came from no longer seems to exist (check the PR details - says unknown repo).
Are there any issues with this? If so, can you explain, and I will fix them.

@specialtactics, done sorry i did not notice the repo is not ecist


public profile for 2018-06

2018-08-04: edited opening post (@joshgav)
This is the proposed June 2018 profile for public Azure, our first public profile. We will generate packages or other constructs in every SDK which target this group of APIs and versions persistently and stably.
The profile is intended to match services available not only in US Azure but in all other public Azures as well, e.g. China, Germany and US Government. It is also intended to ultimately provide a predictable, stable, supported programming interface for Azure developers.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

@sijuman please confirm that the resource versions specified in this profile actually exist in the repo specs

@sijuman I realized doing some automatic processing of the file that the structure was incorrect. I allowed myself to push a commit to fix it, to move forward, hope you don't mind :)
@joshgav This means we need schema to check this file now. It was JSON valid, but indented incorrectly.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-js

@sijuman what's the deal with this profile, should it be in should it not?
I will close it since there's no activity for a while now. please request reopening through email if still required


/monitors argument fails to respect provided ID

System: Debian 10 (Buster), amd64, stable branch
Version: "This is FreeRDP version 2.0.0-dev5 (2693389a+debian)"
Build Configuration: xfreerdp --buildconfig
Build configuration: BUILD_TESTING=OFF BUILTIN_CHANNELS=ON HAVE_AIO_H=1 HAVE_EXECINFO_H=1 HAVE_FCNTL_H=1 HAVE_INTTYPES_H=1 HAVE_JOURNALD_H=TRUE HAVE_MATH_C99_LONG_DOUBLE=1 HAVE_POLL_H=1 HAVE_PTHREAD_MUTEX_TIMEDLOCK=ON HAVE_PTHREAD_MUTEX_TIMEDLOCK_LIB=1 HAVE_PTHREAD_MUTEX_TIMEDLOCK_SYMBOL= HAVE_SYSLOG_H=1 HAVE_SYS_EVENTFD_H=1 HAVE_SYS_FILIO_H= HAVE_SYS_MODEM_H= HAVE_SYS_SELECT_H=1 HAVE_SYS_SOCKIO_H= HAVE_SYS_STRTIO_H= HAVE_SYS_TIMERFD_H=1 HAVE_TM_GMTOFF=1 HAVE_UNISTD_H=1 HAVE_XI_TOUCH_CLASS=1 WITH_ALSA=ON WITH_CCACHE=ON WITH_CHANNELS=ON WITH_CLIENT=ON WITH_CLIENT_AVAILABLE=1 WITH_CLIENT_CHANNELS=ON WITH_CLIENT_CHANNELS_AVAILABLE=1 WITH_CLIENT_COMMON=ON WITH_CLIENT_INTERFACE=OFF WITH_CUPS=ON WITH_DEBUG_ALL=OFF WITH_DEBUG_CAPABILITIES=OFF WITH_DEBUG_CERTIFICATE=OFF WITH_DEBUG_CHANNELS=OFF WITH_DEBUG_CLIPRDR=OFF WITH_DEBUG_DVC=OFF WITH_DEBUG_KBD=OFF WITH_DEBUG_LICENSE=OFF WITH_DEBUG_MUTEX=OFF WITH_DEBUG_NEGO=OFF WITH_DEBUG_NLA=OFF WITH_DEBUG_NTLM=OFF WITH_DEBUG_RAIL=OFF WITH_DEBUG_RDP=OFF WITH_DEBUG_RDPDR=OFF WITH_DEBUG_RDPEI=OFF WITH_DEBUG_REDIR=OFF WITH_DEBUG_RFX=OFF WITH_DEBUG_RINGBUFFER=OFF WITH_DEBUG_SCARD=OFF WITH_DEBUG_SND=OFF WITH_DEBUG_SVC=OFF WITH_DEBUG_SYMBOLS=OFF WITH_DEBUG_THREADS=OFF WITH_DEBUG_TIMEZONE=OFF WITH_DEBUG_TRANSPORT=OFF WITH_DEBUG_TSG=OFF WITH_DEBUG_TSMF=OFF WITH_DEBUG_WND=OFF WITH_DEBUG_X11=OFF WITH_DEBUG_X11_CLIPRDR=OFF WITH_DEBUG_X11_LOCAL_MOVESIZE=OFF WITH_DEBUG_XV=OFF WITH_DSP_EXPERIMENTAL=OFF WITH_DSP_FFMPEG=OFF WITH_EVENTFD_READ_WRITE=1 WITH_FAAC=OFF WITH_FAAD2=OFF WITH_FFMPEG=TRUE WITH_FFMPEG=TRUE WITH_GFX_H264=ON WITH_GPROF=OFF WITH_GSM=ON WITH_GSSAPI=OFF WITH_GSTREAMER_0_10=OFF WITH_GSTREAMER_1_0=ON WITH_ICU=ON WITH_IPP=OFF WITH_JPEG=ON WITH_LAME=OFF WITH_LIBRARY_VERSIONING=ON WITH_LIBSYSTEMD=ON WITH_MACAUDIO=OFF WITH_MACAUDIO=OFF WITH_MACAUDIO_AVAILABLE=0 WITH_MANPAGES=ON WITH_MBEDTLS=OFF WITH_OPENH264=OFF WITH_OPENSLES=OFF WITH_OPENSSL=ON WITH_OSS=ON WITH_PAM=ON WITH_PCSC=ON WITH_PROFILER=OFF WITH_PULSE=ON WITH_SAMPLE=OFF WITH_SANITIZE_ADDRESS=OFF WITH_SANITIZE_ADDRESS_AVAILABLE=1 WITH_SANITIZE_MEMORY=OFF WITH_SANITIZE_MEMORY_AVAILABLE=1 WITH_SANITIZE_THREAD=OFF WITH_SANITIZE_THREAD_AVAILABLE=1 WITH_SERVER=ON WITH_SERVER_CHANNELS=ON WITH_SERVER_INTERFACE=ON WITH_SMARTCARD_INSPECT=OFF WITH_SOXR=OFF WITH_SSE2=ON WITH_THIRD_PARTY=OFF WITH_VALGRIND_MEMCHECK=OFF WITH_VALGRIND_MEMCHECK_AVAILABLE=1 WITH_WAYLAND=ON WITH_X11=ON WITH_X264=OFF WITH_XCURSOR=ON WITH_XDAMAGE=ON WITH_XEXT=ON WITH_XFIXES=ON WITH_XI=ON WITH_XINERAMA=ON WITH_XKBFILE=ON WITH_XRANDR=ON WITH_XRENDER=ON WITH_XSHM=ON WITH_XTEST=ON WITH_XV=ON WITH_ZLIB=ON
Build type:          RELWITHDEBINFO
CFLAGS:              -g -O2 -fdebug-prefix-map=/build/freerdp2-Bj0zA6/freerdp2-2.0.0~git20190204.1.2693389a+dfsg1=. -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -Wall -Wno-unused-result -Wno-unused-but-set-variable -Wno-deprecated-declarations -fvisibility=hidden -Wimplicit-function-declaration -Wredundant-decls -g -DWINPR_DLL
Compiler:            GNU, 8.2.0
Target architecture: x64
Steps to reproduce:

xfreerdp /monitor-list

[0] 1680x1050        +0+0
[1] 1680x1050        +1680+0

xfreerdp /monitors:1 /f results in full screen over the terminal which called it (e.g. when the terminal is on screen:0, it opens in full screen on screen:0 and vice versa)

Workaround: Open a new terminal on the intended screen then call xfreerdp

Just tried with nightly, there is seems to work. can you confirm?
that means the debian package is due for an upddate.

@markfortma ok, then could you please add your window manager? might be related. Tried here with gnome3 (wayland and X11 sessions)


[WIP] [CI:DOCS] Split handlers and server packages

The files were split to help contain bloat. The api/server package will
contain all code related to the functioning of the server while
api/handlers will have all the code related to implementing the end
points.
api/server/register_* will contain the methods for registering
endpoints.  Additionally, they will have the comments for generating the
swagger spec file.
See api/handlers/version.go for a small example handler,
api/handlers/containers.go contains much more complex handlers.
Signed-off-by: Jhon Honce<EMAIL_ADDRESS>

/hold

/lgtm

/hold cancel


[viu:ott] Add support to download entire series with --yes-playlist

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Dear maintainers:
Fix:
Passing --yes-playlist is identical to not passing any --xxx-playlist option since there is a default False value.
Improvement:
The video info got from viu:ott has all the video ids of the entire series. So maybe we can support downloading of all the episodes with the --yes-playlist flag.

Hi @dstftw , could you kindly review this if you have time? Just wanna see if there are still problems.

Is there any update for this issue?
It would be nice if you could add support to download the entire series

The maintainer said this is intended behavior regarding the playlist flags, and will not change it. So if you want to download viu:ott as a list, either patch it yourself (e.g. fork my branch), or change the viu:ott extractor to ALWAYS return a playlist. Thus you can use --no-playlist every time you want a single episode, OR, omit the flag to have a full playlist.
If I didn't misunderstand @dstftw's opinion.

anyway i am still wondering how to handle if the url represents a single video, but it also expose a list. How can the user choose between?

The maintainer said this is intended behavior regarding the playlist flags, and will not change it. So if you want to download viu:ott as a list, either patch it yourself (e.g. fork my branch), or change the viu:ott extractor to ALWAYS return a playlist. Thus you can use --no-playlist every time you want a single episode, OR, omit the flag to have a full playlist.
If I didn't misunderstand @dstftw's opinion.

The maintainer said this is intended behavior regarding the playlist flags, and will not change it. So if you want to download viu:ott as a list, either patch it yourself (e.g. fork my branch), or change the viu:ott extractor to ALWAYS return a playlist. Thus you can use --no-playlist every time you want a single episode, OR, omit the flag to have a full playlist.
If I didn't misunderstand @dstftw's opinion.

ahm could you give me an example? I don't really understand :)
Here's the link  that I want to download all series at once
https://www.viu.com/ott/ph/en-us/vod/121181/The-Producers#tab2

Short answer
you cannot
long answer
the current viu:ott extractor does NOT return playlist result, although it has the playlist info already. To make it work, you must either MODIFY the code of either youtube-dl itself, or viu:ott extractor to return a playlist result instead.
The latest compiled binaries released by youtube-dl officially will NOT have the things you want, and you will have to download Python on your computer and run it locally, if you are to modify the code and run the customized program.

https://www.viu.com/ott/id/id/all/video-chinese-drama-movies-back_to_1942-1164195806

currently this extractor can only extract all the episodes of ONE entire series, such as https://www.viu.com/ott/hk/zh-hk/vod/259033/, and does not recognize any other pages including https://www.viu.com/ott/hk/zh-hk/category/4/ or whatever other listings e.g. https://www.viu.com/ott/hk/zh-hk/all.
so it is intentionally not matched.

Actually this PR has been here for 2.5 years already. The original suggestion is to give an option to download all the items of a series, since the series itself has NO standalone listing page, even until now 2020. Correct me if I am wrong.

https://www.viu.com/ott/id/id/all/video-chinese-drama-movies-back_to_1942-1164195806

currently this extractor can only extract all the episodes of ONE entire series, such as https://www.viu.com/ott/hk/zh-hk/vod/259033/, and does not recognize any other pages including https://www.viu.com/ott/hk/zh-hk/category/4/ or whatever other listings e.g. https://www.viu.com/ott/hk/zh-hk/all.
so it is intentionally not matched.

In my personal Viu tool, I can get the playlists. I don't know why youtube-dl doesn't have this feature
https://i.imgur.com/WINmha1.jpg

I don't know why youtube-dl doesn't have this feature

because there is yet no one submitted the code regarding the said feature here.
of course you can submit another PR if you have figured out how to do that, or a feature request to see if someone could try.

I don't know why youtube-dl doesn't have this feature

because there is yet no one submitted the code regarding the said feature here.
of course you can submit another PR if you have figured out how to do that, or a feature request to see if someone could try.

I only code on C#. I don't know yet python.
Wait, what are you talking about is the "Getting the entire playlists of the series?"
Or could you give me an example link and I'll check

what this PR  does:
given https://www.viu.com/ott/ph/en-us/vod/286677/Flower-of-Evil
return all the urls of episode 1 to 10, instead of just 286677 (ep1)

what this PR does:
given https://www.viu.com/ott/ph/en-us/vod/286677/Flower-of-Evil
return all the urls of episode 1 to 10, instead of just 286677 (ep1)
just like the dialog of your screenshot

Yeah, only 1 episode needed and automatically getting all the series links

take an example from my region (HK), which has 11 episodes currently:
C:\Users\lkho>youtube-dl https://www.viu.com/ott/hk/zh-hk/vod/254942/blablablablabla --get-url
https://stream-hk.viu.com/s/u8rqE9DiegA6KiRcxpjhoA/1599076637/UD/27645ce30c28a42e4d656dc1a1451242/27645ce30c28a42e4d656dc1a1451242_Layer4_vod.m3u8
https://stream-hk.viu.com/s/JoeNJ5AXUfBtG8nP2Eadtw/1599076638/UD/b42d3d204a6458b2d4e70b0bda7c7d4b/b42d3d204a6458b2d4e70b0bda7c7d4b_Layer4_vod.m3u8
https://stream-hk.viu.com/s/jJ0dlvmDrs_h02NqZopg5A/1599076639/UD/3ce7b96cffc5cb95d686dd652e16a042/3ce7b96cffc5cb95d686dd652e16a042_Layer4_vod.m3u8
https://stream-hk.viu.com/s/jJ0dlvmDrs_h02NqZopg5A/1599076639/UD/d66b9aa053940aa45b2ccace060b0cae/d66b9aa053940aa45b2ccace060b0cae_Layer4_vod.m3u8
https://stream-hk.viu.com/s/sZhuxJQvB_v16EmN41k2Qg/1599076640/UD/5c81d999e5a8296fa568be330b3eb2b5/5c81d999e5a8296fa568be330b3eb2b5_Layer4_vod.m3u8
https://stream-hk.viu.com/s/M3B8m9Hrp5gooBcDS3cUuQ/1599076641/UD/25ee0cd9689dd720c0a1e269fa3b8dc7/25ee0cd9689dd720c0a1e269fa3b8dc7_Layer4_vod.m3u8
https://stream-hk.viu.com/s/4oBCflvL8wkCgVqO0VWNvA/1599076642/UD/2b012655cd8dd4014361b5c2c17df423/2b012655cd8dd4014361b5c2c17df423_Layer4_vod.m3u8
https://stream-hk.viu.com/s/u1HHsbBET34B6U7dVldXvw/1599076646/UD/d429363047e56010a9d0a540d2ad7a40/d429363047e56010a9d0a540d2ad7a40_Layer4_vod.m3u8
https://stream-hk.viu.com/s/wGIkQW67MvYceHUJPzJDGw/1599076647/UD/835d423e7750b8c7cac2ea9fb00c1021/835d423e7750b8c7cac2ea9fb00c1021_Layer4_vod.m3u8
https://stream-hk.viu.com/s/8IYWSuRdTKPJM_UhPfeSrQ/1599076650/UD/00e28ceaebba81671af1751f914cc6fa/00e28ceaebba81671af1751f914cc6fa_Layer4_vod.m3u8
https://stream-hk.viu.com/s/XdHrt7GDfaizf2ssBkTO-w/1599076651/UD/bceadb5fdcda2688c4706977a075ea38/bceadb5fdcda2688c4706977a075ea38_Layer4_vod.m3u8

C:\Users\lkho>

so there are 11 .m3u8 urls already, corresponding to the 11 episodes


does Apple block Motrix?

for this apple hosted file (Sonoma OSX beta)
it seems Motrix is rate limited or restricted, or perhaps I'm doing something wrong
Chrome downloads it fine, but Motrix struggles to even get one going, 1 peer flashes on and off,
so seems strange.
https://swcdn.apple.com/content/downloads/23/44/032-94352-A_DB05J15QWT/4x91v0yzolyiat5cat76ieu0h78aeu3d03/InstallAssistant.pkg

Humm...I think this just a tracker list problem.Have you tried to add some extra tracker list in Motrix?Just on here:


Notification didn't dismiss when calling  FlutterCallkitIncoming.setCallConnected(id) in custom button

And i try make a log to trace, It should be like an endCall event but no event whatsoever is called when running FlutterCallkitIncoming.setCallConnected(id)

Hi,
Any updates on this, facing same issue.
Best Regards

facing the same issue! the connected event is called but the notification is not being dismissed. I wonder if the problem on Android is the hashCode being computed that is used as notification id.


LazyLoad: Add params for timeout period and timed-out template

It would be nice to have two more snippet params added to LazyLoad: one to specify the CometActor timeout and another to specify a second template to use in the case of a timeout.

Notably, Lift 3 supports a template parameter that can be passed to the lazy snippet. It doesn't support customization of the timeout, though. We'd need to support that directly in AsyncRenderComet, and then allow for setting it in the LazyLoad snippet.


Added location based infos

Uses the browser location feature to detect user location to show data.

it's already location based

it's already location based

not automatically since it showed Delhi while i live in Brazil

it's already location based

not automatically since it showed Delhi while i live in Brazil

okay

thank you for your contribution :)

hey there is some issue in UI structure and also background image is not loading

can you please check and edit it

hey there is some issue in UI structure and also background image is not loading

can you please check and edit it

oh ye

hm, the image is loading for me, and also the UI looks good.

hm, the image is loading for me, and also the UI looks good.

because I editd the , thank you for your contribution :)

hm, the image is loading for me, and also the UI looks good.

because I editd the , thank you for your contribution :)

Actually i was using the same code i uploaded to test

You can also contribute by designing the webiste for this repo- https://github.com/avinash201199/Paid-Internship-List