Smart Charging : DST time change impacting charging plan

Since we went back to winter time, the charging plan is still showing proper hours, however the charging starts one hour ahead.
Expected Behaviour : Charging starts at 11pm

What's happening : Charging starts at 10pm

The charging station is obviously in supervised mode, and its hour is correct:

@yannpub In english please ;-)

@yannpub

Is it reproducible? Have you retried?
Is yes, could you then set you station timezone to UTC and see if it happens again?

And give us your feedbacks here.
Thanks.

I restarted a session today, and charging restarted at 22:00 and not 23:00 like in my charging plan.
I stopped the session, and changed time zone on the charging session:

I waited 10 minutes and restarted a session:

and the charging started immediately, instead of waiting until 23:00

Really set UTC in the timezone, nor Universal or etc/UTC, I still see Europe/Paris in your settings.
And reboot your station after the timezone change and check that the change is still here.

I've made the timezone change.

The station was still on 'Universal' this morning. I just changed it to UTC and rebooted.

Will give it a try this evening again.

You're the only one to have that issue. We have several users having the same Wallbox with similar recurring charging profile and none of them have that issue.
So to report it to Schneider, you will have to send us the charging station report.

You're the only one to have that issue. We have several users having the same Wallbox with similar recurring charging profile and none of them have that issue.

It would be actually good to double check with other colleagues (Florent...)

Set timezone to UTC -> KO
Factory reset with timezone to Europe/Paris -> KO

Changing start date for the charging plan (not changing anything else) from Sept 11 (Before DST change) to November 4th (After DST change) worked.
Old:

New:

Reproduce issue:
Changing start date for the charging plan (not changing anything else) from November 4 (After DST change) to September 1st (before DST change) reproduced the initial bug: the car started charging at 22:00 and not 23:00.
Old:
See previous post
New:
image

The issue is on the charging station then.

I don't think that the charging station is keeping a Daylight Saving Time calendar. It is getting its time either manually, either through NTP or through the supersivion. So I think the charging station is 'DST' agnostic.
Therefore, the backend should send the time according to the current day, and not according to the profile start date.

I don't think that the charging station is keeping a Daylight Saving Time calendar. It is getting its time either manually, either through NTP or through the supervision. So I think the charging station is 'DST' agnostic.

That's not the case on Schneider ones.

Therefore, the backend should send the time according to the current day, and not according to the profile start date.

As I said, it's already the case: all dates in charging profiles are relative to UTC and all schedules are relative to that date in seconds. And in recurring profile, the exact day is not taken into account and the start hour is relative to UTC.
Unless we have a bug at putting the start date to UTC (which does not seem to be the case), the issue is at handling the start date on the charging station properly.

As I said, it's already the case: all dates in charging profiles are relative to UTC and all schedules are relative to that date in seconds. And in recurring profile, the exact day is not taken into account and the start hour is relative to UTC.

When I look at the logs, for the same charging profile (32A from 11pm to 7am), the hours is not the same depending on the profile start date:
When setting a start date before DST change:
"chargingProfileKind": "Recurring",
"recurrencyKind": "Daily",
"chargingSchedule": {
"duration": 86400,
"startSchedule": "2020-08-31T22:00:00.000Z",
"chargingRateUnit": "A",
"chargingSchedulePeriod": [
{
"startPeriod": 0,
"limit": 32
},
{
"startPeriod": 25200,
"limit": 0
},
{
"startPeriod": 82800,
"limit": 32
When setting a start date after DST change:
"chargingProfileKind": "Recurring",
"recurrencyKind": "Daily",
"chargingSchedule": {
"duration": 86400,
"startSchedule": "2020-11-03T23:00:00.000Z",
"chargingRateUnit": "A",
"chargingSchedulePeriod": [
{
"startPeriod": 0,
"limit": 32
},
{
"startPeriod": 25200,
"limit": 0
},
{
"startPeriod": 82800,
"limit": 32

You're right, the UTC time calculation is wrong on the server at profile payload generation given the date inputed on the dashboard.
Since I'm vacation, I add @OliveGerste who is the charging profiles code master to the issue :)

Update:

Issue is definitely on the Schneider firmware side;
Fix expected in the next firmware versiom.

@yannpub: Schneider have released a new firmware version. Could you please install it to see if it fix that issue?
Thanks.

As expected with the DST change, again the start of the charging session did not start appropriately. Now that we changed from UTC+1 to UTC +2, previous charging profile, which was to start at 23:00 now starts at 24:00.
The issue is still that the charging profile is based on the daily planning start date and not on the actual date.
Issue is with Charging profile:
Start time : 23:00 winter time (daily planning start date November 1st 2020), meaning 22:00UTC. France time zone is UTC +1.
--> local start time 23:00 during winter
/|\ Change of time with DST
France time zone becomes UTC+2 --> start time still 22:00UTC --> local start time becomes 24:00 which is not what is expected from an user perpective. Charging profile has to be DST agnostic, high peak tarifs do not change with DST

The charging profile date format is timezone agnostic in the OCPP spec and we implement it correctly in e-Mobility.
The issue is that the Schneider firmware is does not handle it correctly given the timezone defined on the charging station.

The problem is that the usage of the 'start date' in the charging profile is a non-sense from an user prospective, and especially for a recurring daily profile.
Just send the time based on the day we are, and that will be correct independantly of any time change. What is the added-value of sending a time calculated in winter time when it is summer time? and vice-versa? none.


fail to clone (private) repo

System information

node version: 13.3.0
npm or yarn version: 6.1.0
OS/version/architecture: Win64 - Pro
Applicable nodegit version: latest

const fsj = require('fs-jetpack');
const path = require('path');
const Git = require("nodegit");
const private_token = 'XXXXXXXXXXXXX';
const cloneOptions = {
	fetchOpts: {
  		callbacks: {
    		credentials: function() {
      		return NodeGit.Cred.userpassPlaintextNew(private_token, "x-oauth-basic");
    		},
    		transferProgress: function(progress) {
         	progressCount++;
         	trace(progressCount)
       	}
  		}
  	}
};

let pr_dir = "testfolder\\westwood";

Git.Clone("https://gitlab.com/nova-nfp/westwood", pr_dir, cloneOptions).then(function(repository) {
  // Work with the repository object here.
}).catch((err)=>console.log(err));

results in
[Error: Method clone has thrown an error.] {
  errno: -1,
  errorFunction: 'Clone.clone'
}

I'm fairly confident there's no way this is going to work if you're leveraging private_token as username and x-oauth-basic as your password in userpassPlaintextNew.

i also thought so, but then there's
https://github.com/nodegit/nodegit/blob/master/examples/cloneFromGithubWith2Factor.js

I think that only works for github.com based remotes, because they're weird. For gitlab you probably want to do oauth2 for the username, and the accessToken as the password.

Ah, okay, that would make sense.
couldn't/can't find an example though ...

I think that only works for github.com based remotes, because they're weird. For gitlab you probably want to do oauth2 for the username, and the accessToken as the password. Check https://forum.gitlab.com/t/authenticate-using-access-token/9330.

That just worked for me on a bunch of Azure DevOps hosted repos, thanks!

I'm experiencing the same issue, would someone have an example?
import fse from 'fs-extra';
import nodegit from 'nodegit';

/**
 * Clone a repository locally.
 *
 * @param repositoryLocalPath
 * @param repoUrl
 *
 * @see https://github.com/nodegit/nodegit/blob/master/examples
 */
export const clone = async (repositoryLocalPath, repoUrl) => {
  // If the repo is public, you can use a callback instead
  const token = process.env.GITHUB_REPO_TOKEN;

  const opts = {
    fetchOpts: {
      callbacks: {
        credentials: function () {
          return nodegit.Cred.userpassPlaintextNew(token, 'x-oauth-basic');
        },
        certificateCheck: function () {
          return 0;
        },
      },
    },
  };

  await fse.remove(repositoryLocalPath);
  // @ts-ignore
  const repo = await nodegit.Clone(repoUrl, repositoryLocalPath, opts);

  if (repo instanceof nodegit.Repository) {
    console.info('We cloned the repo!');
  } else {
    console.error('Something borked :(', repo);
  }

  return repo;
};

export default clone;


webchat is not Loading

hello  everyone,
I am using the microsoft BotFramwork-webchat. I use the following webchat CDN
https://cdn.botframework.com/botframework-webchat/latest/webchat.js".
but I get the error while Load the webchat Component. It throws error from today itself. I working as expected before.
here is the snippet of error.

in network Tab the webchat Conversations request is Canceled.

the component look like this:

waiting for any solution.
thanks in advance.

Any update on this? We are having the exact same issue.

We are investigating the issue. Could you switch from latest to 4.14.1 in the meanwhile?

@compulim are you asking for change the webchat CDN to 4.14.1 ?

There is an issue when rendering Web Chat to a detached node. Detached node means the node does not belong to the HTML document.
Let's say:
const container = document.createElement('div');

window.WebChat.renderWebChat(
  {
    directLine: window.WebChat.createDirectLine({ token })
  },
  container
);

document.body.append(container);

The code above would fail because Web Chat is being rendered to a detached node.
The issue will be resolved when Web Chat is being rendered on an attached DOM node.
const container = document.querySelector('#webchat');

window.WebChat.renderWebChat(
  {
    directLine: window.WebChat.createDirectLine({ token })
  },
  container
);

In the meanwhile, please either downgrade it back to 4.14.1 and wait for a patch. Or render Web Chat to an attached DOM node.

hey @compulim it is really help full for me. working fine after downgrade version. thank for helping.

Any update on this? We are having the exact same issue.
@DainHeart  please check solution by @compulim  it working fine for me.

This is affecting us as well.


TypeError: path.replace is not a function.

Hello there!
I can't get the plugin to work…

Obsidian 0.13.1 on Windows 11
I'm correctly logged in in the plugin settings
I restarted Obsidian

If I POST a request to my webhook URL like this:
curl -X "POST" "https://us-central1-obsidian-buffer.cloudfunctions.net/webhook/censored?path=Test.md"
    -H 'Content-Type: text/plain'
    -d "Can you see me?"

nothing happens. In Electron's console I see this:
Uncaught (in promise) TypeError: path.replace is not a function
    at ObsidianWebhooksPlugin.eval (eval at <anonymous> (app.js:1), <anonymous>:20863:32)
    at Generator.next (<anonymous>)
    at eval (eval at <anonymous> (app.js:1), <anonymous>:43:71)
    at new Promise (<anonymous>)
    at __awaiter (eval at <anonymous> (app.js:1), <anonymous>:39:12)
    at ObsidianWebhooksPlugin.applyEvent (eval at <anonymous> (app.js:1), <anonymous>:20861:16)
    at eval (eval at <anonymous> (app.js:1), <anonymous>:20838:65)

You found a bug! I'll take a look shortly. Thanks for letting me know!

Wow, didn't expect such a prompt response haha. Thank you!

@yaserahmady I just shipped a 0.0.4 of the plugin can you let me know if it works for you? 🤞

It totally works now, great job and thank you!


Add support for Mars CosmWasm executions

For on-chain execution of Mars Protocol apps, we need to add support for CosmWasm MsgExecuteContract parsing for Mars contract addresses.
Helpful links
Blocked by #443

This issue will be worked on during the development of #443 to allow real-world testing

Dropping back to the backlog, we are prioritizing #552 first


Enable to open diagnostic link from problems panel with keyboard

This PR fixes #89562

Good fix. Thanks.


Fix gzip decompression

zlib package seems to fail to read all lines in some aws-created files

Hello @jyrivallan,
Sorry for the delay here.
Thanks a lot for your PR and to help us improving the Datadog product.
We are currently doing some final test on your suggestions and we will move forward very soon.
Once again, thanks a lot for your contribution.
Nils

With pleasure.
We also submitted https://github.com/logmatic/logmatic-lambda/pull/4, do you think you can merge it as well ?

Sure we will merge it as well.
Thanks


API check

Is there any error handling (or warnings) id the API is wrong ?

What error case exactly do you mean?
When is the API wrong?


Add missing libraries to toolchain

[ ] libffi
[ ] libgio-2.0
[ ] libglib-2.0
[ ] libgudev

These need to both be in the toolchain to link against, but they also need to be setup properly in pkgconfig so that tools like meson can properly identify that they are installed.


Fixes #27605: inline math blocks can't be preceeded/followed by alphanumerical characters

Inline math blocks couldn't be preceeded or succeeded by alphanumerical characters due to changes introduced in PR #21171.  Removed the condition that caused this (precedingCharacter condition) and added a new exit condition of the for-loop that checks if a specific '$' was escaped using '' so that the math expression can be rendered as intended.
Additionally this PR fixes another bug where math blocks of the type '$xyz$abc$' where the dollar sign was not escaped by the user, generated an error (shown in the screenshots below)
Altered the tests to accomodate for the changes

Former behaviour (from try.gitea.io):

Fixed behaviour (from my local build):

(Edit) Source code for the README.md file:
$x$ -$x$ $x$-

a$xa$ $xa$a 1$xb$ $xb$1

$a a$b b$

a$b $a a$b b$

$a a\$b b$

I tested what GitHub renders for your example and only the first one works. Would you say those are bugs in GitHub's implementation?

Also, maybe you could add a few more test cases, instead of only modifying the existing ones.

I tested what GitHub renders for your example and only the first one works. Would you say those are bugs in GitHub's implementation?

I think so since I don't see a particular reason why it wouldn't be allowed for characters to come directly before or after the math expressions.

Seems like GitLab rendering does agree with your solution:

Seems like GitLab rendering does agree with your solution, so 👍 from me on the rendering part.

We missed it but it seems that GitLab does not agree with my solution after all, the last expression of the second line is not rendered, and the rendering in the fourth line does not match

TBH, I still have question about why it needs this change.
Why it needs to handle $a a$b b$ / a$x$ that way? Why "inline math blocks should be preceeded/followed by alphanumerical characters"?
By reading #27605, I didn't get the answer, and I haven't seen a real world example yet ....
Could you elaborate the use cases by real examples?

For the a$x$ expression you could have for example something of the type a$x²$ where 'a' is a constant. If 'a' was inside the math expression it would have the same formatting as 'x²' making it less readable.
As for the $a a$b b$, it's an extra, it's a side effect of changing the render as mentioned above. Before, this expression would generate an error, as shown in the first comment of the conversation, and now it no longer does that.

For the a$x$ expression you could have for example something of the type a$x²$ where 'a' is a constant. If 'a' was inside the math expression it would have the same formatting as 'x²' making it less readable.

Why people would write such expr a$x²$? I think it is still an over-abstracted example, but not a real world example. I used some examples above like The price is between $1 and $10, because people really write that way. But for a$x²$ , could you show some real world documents/comments that why it is written?

As for the $a a$b b$, it's an extra, it's a side effect of changing the render as mentioned above. Before, this expression would generate an error, as shown in the first comment of the conversation, and now it no longer does that.

Yup, I agree that it is just a side-effect of this change. The root question is still the first one.

Should the math expressions not allowed to be followed by alphanumerical characters or also not allowed to be followed by any character besides ' ' or a newline?

IMO following by a punctuation or newline is fine, because that's also how a sentence is written. GitHub:

It is $1$, or maybe $2$. => It is $1$, or maybe $2$.

Guess 1.22 backport is enough.


GitHub REST API: add timeout and retry

add comment test

add comment test

One clarification, do we not want to use retries on "put" and "patch" requests?

We retries on "patch".
I don't retry on "post", because retry may add duplicate comments.


Release 55.0.0

Description
See changelog for details

@mcmire I added all your commits, thank you! Just a note I rebased this on main and pulled in two commits, one is your changes to add a missing test for network controller so I think no changes to changelog are needed. The other is one on assets-controller from @adonesky1 who is sending me the changelog entry.


Fix spelling, "general".

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


Vines show item.forgero.vines

And wrong texture I guess?

Yeah I I have forgotten translations and probably did a dirty copy when making the vine material palette. Should be easy to fix.


[bug] Doesn't work with Sortablejs 1.13.0 or 1.10.2

This package doesn't function properly when one of these to versions is installed with it.
In specific the swap doesn't work. The only way it works if version 1.12.0 is installed. Since 1.12.0 is deprecated on a fresh install the swap and the drag & drop doesn't function as it should.

I figured out, that in sortable.esm.js (it is used by Webpack bundler because it's written in module section) Swap plugin is not mounted.
So you have to add the following string in some of your index files:
import Sortable, { Swap } from 'sortablejs';

Sortable.mount(new Swap());

It doesn't solve all issues, but may be helpful.

I figured out, that in sortable.esm.js (it is used by Webpack bundler because it's written in module section) Swap plugin is not mounted.
So you have to add the following string in some of your index files:
import Sortable, { Swap } from 'sortablejs';

Sortable.mount(new Swap());

It doesn't solve all issues, but may be helpful.

I figured out, that in sortable.esm.js (it is used by Webpack bundler because it's written in module section) Swap plugin is not mounted.
So you have to add the following string in some of your index files:
import Sortable, { Swap } from 'sortablejs';

Sortable.mount(new Swap());

It doesn't solve all issues, but may be helpful.

Ah, so thats why it doesnt work with the newer version or the older one. Unfortunately mounting doesn’t work for my usecase. I run next and i get webpack module error when i try to mount Swap. I worked around my issue by using yarn and installing v1.12. Alternatively a package lock with v1.12

I figured out, that in sortable.esm.js (it is used by Webpack bundler because it's written in module section) Swap plugin is not mounted.
So you have to add the following string in some of your index files:
import Sortable, { Swap } from 'sortablejs';

Sortable.mount(new Swap());

It doesn't solve all issues, but may be helpful.

Ah, so thats why it doesnt work with the newer version or the older one. Unfortunately mounting doesn’t work for my usecase. I run next and i get webpack module error when i try to mount Swap. I worked around my issue by using yarn and installing v1.12. Alternatively a package lock with v1.12


Is MessageHeader necessary for each definition？

Is MessageHeader necessary for each definition？
and finish XML seems a little difficult。

The MessageHeader is required to encode what the following message type, version, and block length are required. Without this you could not have different messages and discriminate, or to have message evolve and add new fields and be backwards compatible.


fix: username generation for new user

Issue Ticket Number

closes #2133

Description

Documentation Updated?

[ ] Yes
[ ] No

Under Feature Flag

[ ] Yes
[ ] No

Database Changes

[ ] Yes
[ ] No

Breaking Changes

[ ] Yes
[ ] No

Development Tested?

[ ] Yes
[ ] No

Screenshots

Test Coverage

Screenshot 1

Additional Notes

#2147


Problem: missing explanation of used SDK modules

can link the official Cosmos SDK docs

Closed by #38


Missing vendor folder on clone

Laravel requires a vendor directory to install dependencies with composer. The fix is to mkdir vendor on the server, but we should consider adding an empty directory to the repo for easier installation.

When distributing the release of Lychee-Laravel, I think it would be wise to distribute it with the vendor folder (after having done the composer install)

Also include the Lychee-front in the 'public' folder, because it is missing when downloading this repo as a zip file currently... this results in e.g. missing svg-ressources...


Updates for failing tests

Description
This pull request fixes unit tests that are failing because of external changes to required files and dependencies.
Type of change
Please check all relevant options.

[x] Improvement (non-breaking)
[ ] Bug fix (non-breaking)
[ ] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Codecov Report

Merging #2184 (d69721c) into main (2111fe7) will decrease coverage by 57.03%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@             Coverage Diff             @@
##             main    #2184       +/-   ##
===========================================
- Coverage   85.63%   28.60%   -57.03%
===========================================
  Files         297      297
  Lines       26521    26520        -1
  Branches     4864     4807       -57
===========================================
- Hits        22712     7587    -15125
- Misses       2571    18612    +16041
+ Partials     1238      321      -917

Impacted Files
Coverage Δ

art/utils.py
57.09% <ø> (-16.32%)
:arrow_down:

... and 229 files with indirect coverage changes


Stat Name Null

After getting the StatisticEventType::GetLeaderboardComplete event, the stat name contained in the LeaderboardRow is null:
                LeaderboardResultEventArgs^ LbResultEventArgs = safe_cast<LeaderboardResultEventArgs^>(Event->EventArgs);
                LeaderboardResult^ LbResult = LbResultEventArgs->Result;

                for (LeaderboardRow^ Row : LbResult->Rows)
                {
                    FString RowText = FString::Printf(TEXT("Leaderboard row: User %s, Stats "), Row->Gamertag->Data());
                    for (unsigned int i = 0; i < Row->Values->Size; ++i)
                    {
                        LeaderboardColumn^ statCol = LbResult->Columns->GetAt(i);
                        Platform::String^ statName = LbResult->Columns->GetAt(i)->DisplayName;

                        RowText.Append(FString::Printf(TEXT("(%s : %s) "), LbResult->Columns->GetAt(i)->DisplayName->Data(), Row->Values->GetAt(i)->Data()));
                    }

                    UE_LOG_ONLINE(Warning, *RowText);
                }

Can you attach a fiddler trace?  https://docs.microsoft.com/en-us/windows/uwp/xbox-live/using-xbox-live/troubleshooting/how-to-set-up-fiddler-for-debugging

Is there another tool I can use?

You can route the HTTP calls to your own logging system:
https://docs.microsoft.com/en-us/windows/uwp/xbox-live/using-xbox-live/troubleshooting/troubleshooting-the-xbox-live-services-api

I am trying:
LiveContext->Settings->DiagnosticsTraceLevel = XboxServicesDiagnosticsTraceLevel::Verbose;
But the output in VS doesn't appear too helpful:
07/12/17 15:14:03 [19204] info  - Websocket connnection established.
07/12/17 15:14:03 [19204] debug  - websocket state change: connecting -> connected
07/12/17 15:14:03 [19204] debug  - Finish websocket connection task
Before that there are numerous Date Time entries in the log that are similar to those shown above, which leads me to believe they are relevant too. Would you like to see them all?

Try doing the EnableServiceCallRoutedEvents and ServiceCallRouted from that doc page.  It'll tell you what happened during the HTTP calls which you can log how you like

That didn't work as we hoped:
[2017.07.12-21.29.42:757][411]LogOnline:Warning: LIVE: [URL]:
[2017.07.12-21.29.42:757][411]LogOnline:Warning: LIVE: [Response]:
But there are many others above it that may be useful, for example:
07/12/17 15:29:42 [10540] info  - Presence service for the user already exsit, return
07/12/17 15:29:42 [11316] debug  - Token result properties:
07/12/17 15:29:42 [11316] debug  - Environment : prod
07/12/17 15:29:42 [11316] debug  - Gamertag : r2d2Electron
07/12/17 15:29:42 [11316] debug  - Restrictions : 195 229 230 234 251 252
07/12/17 15:29:42 [11316] debug  - Privileges : 188 191 192 193 194 196 198 199 200 201 203 204 205 206 207 208 211 214 215 216 217 220 224 227 228 235 238 245 247 249 254 255
Are these related?

No, you can turn off LiveContext->Settings->DiagnosticsTraceLevel as that isn't useful for this issue.
In your ServiceCallRouted lamda, what data is coming back?  Is it blank?

The code is as follows:
LiveContext->Settings->DiagnosticsTraceLevel = XboxServicesDiagnosticsTraceLevel::Verbose;
LiveContext->Settings->EnableServiceCallRoutedEvents = true;
LiveContext->Settings->ServiceCallRouted += ref new
Windows::Foundation::EventHandler<Microsoft::Xbox::Services::XboxServiceCallRoutedEventArgs^>(
[=](Platform::Object^, Microsoft::Xbox::Services::XboxServiceCallRoutedEventArgs^ args)
{
    UE_LOG_ONLINE(Warning, TEXT("[URL]: %s %s"), args->HttpMethod->ToString(), args->Url->AbsoluteUri->ToString());
    UE_LOG_ONLINE(Warning, TEXT("[Response]: %s %s"), args->HttpStatus.ToString(), args->ResponseBody->ToString());
});

VS does break at the args->HttpMethod->ToString() line, and I can view args at a high level from the debugger. . .but I can't see the details.

Okay, a fix up helped:
[2017.07.12-21.50.03:683][475]LogOnline:Warning: LIVE: [URL]: GET https://sessiondirectory.xboxlive.com/
[2017.07.12-21.50.03:683][475]LogOnline:Warning: LIVE: [Response]: 200 {"results":[]}
I added the ->Data() call.

Ok cool.  Can you send me the logs for the entire run, include where you set stat and get the leaderboard.

Do you mean the VS log with XboxServicesDiagnosticsTraceLevel::Verbose?
If not, there isn't much else to show.

No, XboxServicesDiagnosticsTraceLevel isn't useful.  I need your "LogOnline:Warning: LIVE: [URL]:", etc logs for the full run (set the stat, get the leaderboard, etc) to see what's going on.

Can we work on this tomorrow? I need to take off now.
The flow is this, at startup, when logging in, I read the stats, which is what you see above. After playing a game, then I update the stats, which does take a little longer. I will get this for you later and post it here.
Thank you for your help.

Yep

The log, as it is, may not be very useful
Start Game:
[2017.07.13-02.01.54:690][  0]LogOnline:Warning: LIVE: No SecureDeviceAssociationTemplateName specified in the engine ini file.
[2017.07.13-02.01.54:690][  0]LogOnline:Warning: LIVE: FOnlineAchievementsLive: Failed to find json OSS achievements config: ../../../LongshotHero/Config/OSS/Live/Achievements.json
Login:
[2017.07.13-02.03.31:544][297]LogOnline:Warning: LIVE: [URL]: GET https://sessiondirectory.xboxlive.com/
[2017.07.13-02.03.31:544][297]LogOnline:Warning: LIVE: [Response]: 200 {"results":[]}
[2017.07.13-02.03.31:999][323]LogOnline:Warning: LIVE: DoWork LocalUserAdded: r2d2Electron
[2017.07.13-02.03.47:250][419]LogOnline:Warning: LIVE: DoWork GetLeaderboardComplete: r2d2Electron
Complete Level:
[2017.07.13-02.09.47:837][381]LogOnline:Warning: LIVE: DoWork StatisticUpdateComplete: r2d2Electron
[2017.07.13-02.09.47:838][381]LogOnline:Warning: LIVE: DoWork Stat: HighScore
Exit Game:
[2017.07.13-02.10.03:492][242]LogOnline:Warning: LIVE: FSessionMessageRouter::UnsubscribeFromMultiplayerEvents failed - couldn't get XboxLiveContext for user<PHONE_NUMBER>546057.
When I perform a StatisticManager::SetStatisticIntegerData the StatisticUpdateComplete event triggers. How do I get the event results:
            case StatisticEventType::StatisticUpdateComplete:
                UE_LOG_ONLINE(Warning, TEXT("DoWork StatisticUpdateComplete: %s"), Event->User->Gamertag->ToString()->Data());

                IVectorView<Platform::String^>^ stats = mgr->GetStatisticNames(Event->User);
                for (Platform::String^ stat : stats)
                {
                    UE_LOG_ONLINE(Warning, TEXT("DoWork Stat: %s"), stat->ToString()->Data());
                }

                break;

It doesn't look like the stat value has been set on the server yet as there's no call going to statswrite.xboxlive.com.
By default it'll send the stat changes every 5 mins.  If you need it to go out faster or if the app is closing, call RequestFlushToService:
https://github.com/Microsoft/xbox-live-api/blob/master/Source/Services/Stats/Manager/WinRT/StatisticManager_WinRT.h#L46

Actually, after I write the HighScore I call flush, otherwise I won't get the update event:
Microsoft::Xbox::Services::Statistics::Manager::StatisticManager^ mgr = Microsoft::Xbox::Services::Statistics::Manager::StatisticManager::SingletonInstance;
if (mgr != nullptr)
{
for (FStatPropertyArray::TConstIterator item(WriteObject.Properties); item; ++item)
{
Platform::String^ itemName = ref new Platform::String(*item->Key.GetPlainNameString());
        int32 itemValue;
        item->Value.GetValue(itemValue);

        long long llValue;
        llValue = itemValue;

        mgr->SetStatisticIntegerData(LiveContext->User, itemName, llValue);
    }

    mgr->RequestFlushToService(LiveContext->User);

Hmm, that RequestFlushToService should call ServiceCallRouted.
If that's not happening, I recommend you set some break points inside XSAPI to see what's its doing.
You can link to source and debug XSAPI easily but following this:
https://github.com/Microsoft/xbox-live-api/blob/master/LINKTOSOURCE.md#how-to-link-against-the-xsapi-winrt-uwp-source
Then set a breakpoints at to ensure each of these is getting called:
https://github.com/Microsoft/xbox-live-api/blob/master/Source/Services/Stats/Manager/WinRT/StatisticManager_WinRT.cpp#L68
https://github.com/Microsoft/xbox-live-api/blob/master/Source/Services/Stats/Manager/stats_manager_impl.cpp#L265
https://github.com/Microsoft/xbox-live-api/blob/master/Source/Services/Stats/Manager/stats_service.cpp#L33

Thanks Jason. Let me give it a try. I'll report back later. . .

Where can I get the 140 build tools?
Microsoft.Cpp.Platform.targets(57,5): error MSB8020: The build tools for v140 (Platform Toolset = 'v140') cannot be found. To build using the v140 build tools, please install v140 build tools.  Alternatively, you may upgrade to the current Visual Studio tools by selecting the Project menu or right-click the solution, and then selecting "Retarget solution".

From that doc page:
If you are using VS2017 for WinRT, we only have a single WinRT project that supports both VS2015 and VS2017. To use it from source, you'll need to install 140 support in VS2017. Run the VS2017 installer, modify, and go to Individual components tab and choose "VS++ 2015.3 v140 toolset". Then you should be able to open

Yes, I did that first.

Hmm, if the v140 toolset is installed, VS2017 should be able build it.   You can try targeting it to v141 if that doesn't work or install VS2015 otherwise

Converting them to v141 allowed the build to work. Does this affect the API/DLL at all? I mean in terms of any software dependent on it.

Nah, its fine.  We just have kept it at v140 so folks using VS2015 can build it.

Thanks. I'll debug with it now. . .

After updating to VS 15.2 I can no longer build my project. I'm getting the following error:
failed on 'C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcpackages\platform.winmd'
'The system cannot find the path specified.'
By chance do you know of a fix for this?

Hmm, that's an odd one.  I'm running the latest VS2017 and VS2015 and our samples don't hit that error.  I don't know how to fix that.  Maybe someone in https://social.msdn.microsoft.com/Forums/en-US/home?forum=wpdevelop would know?

Hey Jason, I am able to build my game (UE4 with UWP) now using VS 2017 ver. 15.2. I'm running into some problem with replacement of the REST and XSAPI DLL's since they are part of the build package. I'm trying to work that out now.

Since this issue is getting pretty hard to parse, I'm going to close this one out.  If you need more help, feel free to open a new issue or better yet post on MSDN at https://social.msdn.microsoft.com/Forums/en-US/home?forum=xboxlivedev.  Github issues list are best for bug reports while MSDN is best for I'm stuck questions so others who get stuck can easily search/find the topic.

Jason, I just got debugging working again. I will follow the steps mentioned in the link, along with your recommendation, and see if I can get any additional information. It is worth noting that my Stat Updates and Retrieval are not working.


Codeception report failed when Test format is used

There is a difference in XML report when using Cest and Test formats in Codeception:
Cest:
<testcase file="/home/stalin/www/PHPCI/PHPCI/build/2/tests/unit/Service/Search/Adapter/StandardCest.php" name="testStandardAdapterFindUseRelevanceSortAndNoAlgorithmRelevance" class="Unit\Service\Search\Adapter\StandardCest" feature="test standard adapter find use relevance sort and no algorithm relevance" assertions="1" time="0.115426"/>

Test:
<testcase name="testBeforeTestProfiler" class="Unit\Library\Codeception\Extensions\ProfilerTest" file="/home/stalin/www/PHPCI/PHPCI/build/2/tests/unit/Library/Codeception/Extensions/ProfilerTest.php" line="53" assertions="1" time="0.008662"/>

As you can see there is no feature attribute in Test XML entry
Codeception plugin by default is using feature attribute so for Test testcases it will be empty.

How this have been addressed? It's fails for me and I'm in the 2.1.5 Codeception version.
Do we need to have both fixes? https://github.com/tankist/PHPCI/commit/ef21502e99893779f5b56cb281186ca210230918 and https://github.com/Codeception/Codeception/pull/1973? if so why this issue haven't been closed?

Just for the record my issue here was similar but not the same:
Exception: Notice: Undefined index: class in /Users/zot/Desktop/phpci/PHPCI/Plugin/Util/TestResultParsers/Codeception.php line 67

Because there is not class on my report.xml file


Use Xcode8.2 in Travis

Pull request checklist

[ ] All tests pass.
[ ] Demo project builds and runs.
[x] I have resolved merge conflicts.
[x] I have followed the coding style and contributing guidelines.

This fixes issue
What's in this pull request?
Simply bumped Xcode version to 8.2 in .travis.yml.

It appears that the iOS 10.2 simulators are duplicated in the new osx_image xcode8.2:

I'll try to investigate this when I have some time.

@Liquidsoul - ha 😄 yep.
We faced the same issue at https://github.com/Instagram/IGListKit/pull/350.
It's a known issue: https://github.com/travis-ci/travis-ci/issues/7031
Our workaround for 10.2 was to use iPad Air 2, since it wasn't duplicated.

Looks like a few different builds are failing because of simulator issues.
You can see supported sims at:
https://docs.travis-ci.com/user/osx-ci-environment/#Xcode-8.2
Or, we've managed to find a good combination with IGListKit:
https://github.com/Instagram/IGListKit/blob/master/.travis.yml

Current coverage is 90.57% (diff: 100%)

Merging #111 into master will increase coverage by 0.81%

@@             master       #111   diff @@
==========================================
  Files            11         11
  Lines           371        371
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            333        336     +3
+ Misses           38         35     -3
  Partials          0          0

Powered by Codecov. Last update 32003cb...e79cfa6


DS Material upgrade to MUI v5

Upgrade all @material-ui packages to the latest @mui packages.
This currently is blocked by other requirements on my side, as it's not possible in an soft-way (or I didn't saw it), most likely end of Q1/22 or even Q2/22.

Note: I need to adjust more than 20 packages, which are still on v4, with more than 3k files, to be able to use v5 in the end.

If you know a soft-upgrade path, I'm happy about feedback!

For all packages, except pickers and slate, the upgrade is so far complete and no further issues where recognized in production the last weeks.
Closing this issue - follow the individual ones for pickers #187 and slate #185  progress.


Trying to speed up revert

Attempts to speed up revert. There are three, what works best is the original simple algorithm, but somewhat aggressive pre-reservation of space in vectors when inserting and pushing_back. I think that version is usable, can speed-up things in the string solver.
We could talk about it with string people.
I wrote also some more complicated variants of revert wich strangely did not do better (also did not do much worse), maybe I was testing on too simple automata, but maybe simple automata are the most common, don't know, it would be good to test it on some string problem.
Apparently, allocation, destruction, even pushing_back, is quite expensive with vectors. Maybe some simple specialised implementation of these things with own memory management might change things ...

Is this PR ready for reviews?

The changes done to this branch were rebased onto the current devel in branch optimize_low_level_structure_operations. This PR will be closed and a new PR will be created from optimize_low_level_structure_operations later on when the changes are finalized.


fix($parse): Preserve expensive checks when runnning $eval inside an expression

When running an expression with expensive checks, there is a call to $eval or $evalAsync
then that expression is also evaluated using expensive checks

Aren't $watch and $watchCollection also affected ?
LGTM otherwise.

So, this works for $$postDigest(), $apply(), $watch() and $watchCollection(), but it doesn't seem to work for $applyAsync().
I would still add tests for all of those (better safe than sorry :smiley:). E.g.:
scope.foo = {w: $window};

expect(function() {
  scope.$eval($parse('$$postDigest($eval("foo.w && true"))', null, true));
}).toThrow();
expect(function() {
  scope.$eval($parse('$apply("foo.w && true")', null, true));
}).toThrow();

expect(function() {
  scope.$eval($parse('$applyAsync("foo.w && true")', null, true));
  scope.$digest();
}).toThrow();
expect(function() {
  scope.$eval($parse('$watch("foo.w && true")', null, true));
  scope.$digest();
}).toThrow();
expect(function() {
  scope.$eval($parse('$watchCollection("foo.w && true", $eval)', null, true));
  scope.$digest();
}).toThrow();

How about disallowing execution of nested $parseed statements. Would that be an option?

How about disallowing execution of nested $parseed statements. Would that be an option?

That would be too big of a breaking change

People actually do that? I've never even thought of it. Unless there is a less obvious way that I haven't thought of...

Updated the PR and fixed the comments. PTAL

LGTM

@petebacondarwin, should this be backported to v1.4.x ?

@gkalpak - yes I would say so. Please do make it happen :-)

Backported to v1.4.x as 96d62cc.


Add Kennewick, WA

To:
Don.Britain<EMAIL_ADDRESS>brad.beauchamp<EMAIL_ADDRESS>Chuck.Torelli@ci.kennewick.wa.us,John.Trumbo<EMAIL_ADDRESS>Subject: Public Safety Concern
Message
Dear Kennewick City Leadership,
My name is [YOUR NAME] and I am a resident of Kennewick. I am writing to demand that the City Council adopt a People’s Budget that prioritizes community well being and redirects funding away from the police.
We are in the midst of widespread upheaval over the systemic violence of policing. I will no longer accept empty gestures and suggestions of “reform.” I am demanding that my voice be heard now, and that real change be made to the way this city allocates its resources.
Rather than prioritize the quality of life for all of Kennewick with a proportional budget, Kennewick chooses to prioritize KPD. In the 2019-2020 budget flyer, the KPD was allocated a budget of $42,968,000 accounting for 39.4% of Kennewick’s total budget. This money can be spent in other ways that are proven to be more effective in improving community safety and wellness.
According to the Tri-City Herald, 223 Tri-Citians were homeless as of 2017, however the pandemic’s severe economic consequences has likely caused this number to increase. Support for communities in need is necessary now, more than ever. But instead, Kennewick has chosen to prioritize KPD over funding services for the homeless many of whom are facing mental health problems and drug addiction.
I demand the City Council defund KPD. Instead, I demand we redirect police funding to improving mental health response teams, homeless shelters, and funding drug addiction treatment centers. I join the calls of those across the country to defund the police. I demand a budget that adequately and effectively meets the needs of at-risk Kennewick residents. I demand a budget that supports community well-being, rather than empowers the police forces that tear them apart.
I am urging you to completely revise the Kennewick budget for the 2020-2021 fiscal year. You need to adopt a People’s Budget. Public opinion is with me.
Thank you for your time,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

let me know if i need to add anything else or if you need the sources, i have pdf's of everything required

Hi @JDot104, thanks for your interest in adding Kennewick to the website. Emails on defund12.org are all written by community members. To add your city, you must provide:

Email addresses (at least one) of the government officials you’d like to receive the message
Detailed message concerning the budgeting decisions at stake in your community

Until you provide this content, we cannot post your city.
Messages should follow our email style guidelines. Remember that messages must be specific to your locale and focused on defunding the police. If your message is copied from an external source, please provide a link to the original source.
Your request will be closed until you reply with the necessary content. Thank you!

The emails at the top are the emails of the council members in my city.
I detailed the budget decisions of my city as well as the amount of funding (and percentage of total funding) of the police in the third paragraph.

To:
Don.Britain<EMAIL_ADDRESS>brad.beauchamp<EMAIL_ADDRESS>Chuck.Torelli@ci.kennewick.wa.us,John.Trumbo<EMAIL_ADDRESS>Subject: Public Safety Concern
Message
Dear Kennewick City Leadership,
My name is [YOUR NAME] and I am a resident of Kennewick. I am writing to demand that the City Council adopt a People’s Budget that prioritizes community well being and redirects funding away from the police.
We are in the midst of widespread upheaval over the systemic violence of policing. I will no longer accept empty gestures and suggestions of “reform.” I am demanding that my voice be heard now, and that real change be made to the way this city allocates its resources.
Rather than prioritize the quality of life for all of Kennewick with a proportional budget, Kennewick chooses to prioritize KPD. In the 2019-2020 budget flyer, the KPD was allocated a budget of $42,968,000 accounting for 39.4% of Kennewick’s total budget. This money can be spent in other ways that are proven to be more effective in improving community safety and wellness.
According to the Tri-City Herald, 223 Tri-Citians were homeless as of 2017, however the pandemic’s severe economic consequences has likely caused this number to increase. Support for communities in need is necessary now, more than ever. But instead, Kennewick has chosen to prioritize KPD over funding services for the homeless many of whom are facing mental health problems and drug addiction.
I demand the City Council defund KPD. Instead, I demand we redirect police funding to improving mental health response teams, homeless shelters, and funding drug addiction treatment centers. I join the calls of those across the country to defund the police. I demand a budget that adequately and effectively meets the needs of at-risk Kennewick residents. I demand a budget that supports community well-being, rather than empowers the police forces that tear them apart.
I am urging you to completely revise the Kennewick budget for the 2020-2021 fiscal year. You need to adopt a People’s Budget. Public opinion is with me.
Thank you for your time,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]


Where does the difference between Actual Bid Amount and Bid Mask go?

Hi,
I did bid for a .eth name through MyEtherWallet and used for the bid mask a higher value than for the actual bid amount. This difference never returned to my account, I just see it as an internal transaction (the 1.45ETH):
https://etherscan.io/address/0xa1308820eacf53d56133c5980fEb4A862Cf678c1#internaltx
Thanks in advance,
Robert

You mean this transaction?
Well, the page says that it has been returned to your account.
"Internal" transactions is just EtherScan's way of tracking messages that were made by accounts with associated code (contracts), as compared to those without it, which are often called "externally owned", or "external".
The confusion that arises from this nomenclature is due to the thinking process of:

Well, there's "externally owned" accounts, let's call them "external" for short.
They send messages initiated by humans, which are logged in the chain directly, as "transactions" in blocks - let's call them "external", too.
Those messages sometimes go to other accounts, which are very different, let's call them something opposite... "internal"!
Contracts can send messages, too, which are not logged as "transactions", but whose effects are still recorded as part of "state change", or "state"... Let's call these... "Internal transactions"!

And - here we are. The "internal transaction" is in fact how the ETH got back to your external account, from the respective bid's Deed contract.


About the performance of certain categories of SemanticKITTI

I found that the results I reproduced are similar to yours, but for the same problem, the result of "motorcyclist" is 0.00, and the result of the same "other-ground" is much lower than the paper. I don’t know the problem, I hope you can help me

I found that the results I reproduced are similar to yours, but for the same problem, the result of "motorcyclist" is 0.00, and the result of the same "other-ground" is much lower than the paper. I don’t know the problem, I hope you can help me

There's no "motorcyclist" in the validation dataset, so the result is 0.00. I have no idea about the "other-ground" .


bug

note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× git clone --filter=blob:none --quiet https://github.com/sanchit-gandhi/whisper-jax.git /private/var/folders/nl/lb6vjd_x22sg124_rpd1kqrh0000gn/T/pip-req-build-_e68d4vt did not run successfully.
│ exit code: 128
╰─> See above for output.

Hey @secondbrain-note - do you want to clone the repo or pip install it from main?
Clone:
git clone https://github.com/sanchit-gandhi/whisper-jax.git
cd whisper-jax
pip install -e .

Pip install from main:
pip install git+https://github.com/sanchit-gandhi/whisper-jax.git

Thank you for your patience. I previously installed it by pip install in the macos terminal and report the bug above. Today I try to install it in VS Code and no bug is reported. I am not a programmer, so sorry for such a silly question.

Glad it's fixed!


if reopen a modal with an editor on it, it does not display unless i resize the window

if reopen a modal with an editor on it, it does not display unless i resize the window, any help? Thanks!

Try setting automaticLayout: true
https://microsoft.github.io/monaco-editor/api/interfaces/monaco.editor.ieditoroptions.html#automaticlayout

I had the same issue. I solved it by putting the editor in 'if condition'.
on opening the modal I make the if condition to true and the editor loads properly.

I have the same issue. I got around it by doing the following:
   $(mymodal).on('show.bs.modal', function () {
       editor.layout()
   });


new email ACS events

Adding new email events for ACS.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-messaging-eventgrid


refactor: consolidate compileElement and compileSurrogate

https://github.com/aurelia/templating-binding/issues/56#issuecomment-264728668

https://github.com/aurelia/templating/blob/1.1.4/src/view-compiler.js#L353
https://github.com/aurelia/templating/blob/1.1.4/src/view-compiler.js#L223

maybe can be closed
not sure if all those merges after the referenced comment did the refactoring mentioned

Not yet.


How to upgrade the operator on Openshift?

Hello,
The documentation is not very clear about the path to update the operator on Openshift.
According to this markdown , we need to apply CRD as a prerequisite.
What is the next step?
On operator page, we can see the latest version available (2.1.0) but it's impossible to upgrade to it.
Current version: 1.5.0
Update channel: alpha
Update approval: Manual (Don't have any button to upgrade to the latest)
Thank you

Hi @llomgui, can you please explain what you mean by  impossible to upgrade to it., are their any logs which you can share as it'll help us understand the issue better. Thanks!

Hello @vepatel,
There is no "seemless upgrade" step available on the operator, there is no button to update it.
Is there any way to do it manually? How to update to ClusterServiceVersion v1.5.2 for example?
Thank you

@vepatel Thank you, it worked. I deleted the operator then re-installed it.
I had one error about IngressClass being a string instead of an object. Quick fixed adding name attribute.
Do I have to deal with CRD as indicated by the documentation?

Hi @llomgui, or operator is helm based and unfortunately helm does not handle CRDs upgrades automatically so CRDs have to be applied manually as a pre-req.
see https://helm.sh/docs/chart_best_practices/custom_resource_definitions/#some-caveats-and-explanations

Thank you for you help. 👍


fix attr_encrypted? not working bug

fix bug for #73

Thank you for the bug report and contribution!


Add information about the fact we do not use Popper.js for Dropdown c…

Add information about the fact we do not use Popper.js for Dropdown contained in navbars
Related to this comment : https://github.com/twbs/bootstrap/pull/24092#issuecomment-339861457

Not sure my wording is right either, @mdo please confirm <3


Add current test file path in process.env

🚀 Feature Proposal
Absolute path to currently running test file in process.env
Motivation
When there are different "groups" of test files (e.g., usage in different environments or module systems), sometimes the difference between the groups is too minor to have an entirely separate Jest configuration file, separate run command etc. In this case, when having a simple if..else clause in setup files would be ideal, there is nothing to distinguish one setup from another.
Example
{
  "setupFiles": [
    "<rootDir>/test/setup.js"
  ]
}

// test/setup.js
console.log(process.env.JEST_TEST_FILE_PATH); // identifier is up to dispute

$ jest

● Console

  console.log test/setup.js:1
    /path/to/project/test/my-entity.spec.ts

Pitch
Not sure what's that

I don't think we should add this as an environment variable. If we add this, I think we should stick it on the injected jest object.
However, I'm not sure we want to add this regardless, seems to me like a leaky abstraction. Could you expand some on which concrete cases this solves a problem for you? That's the "pitch" part of the template 🙂

Sure. Actually, setting a prop in jest seems architecturally more befitting to me as well.
In my collection of 50-ish .spec.js files there is a handful of "rogue" ones that require different setup. Specifically, they test usage of an entity in browser environment (as opposed to Node.js), therefore, they require some additional DOM setup. The emphasis here is that these tests are not so different from the others, so the setup for them would only require something as minimal as additional if..else path with a couple of instructions.
In any other case, I'd create a separate configuration and separate tests command (e.g., jest.config.web.js and test:web, something like that), but here it seems to be an overkill.

Hi, that is a common situation, you can create your own testEnvironment (should be able to find something in the docs) and have those tests declare their test environment using a docblock comment at the top (also in the docs, hopefully 😄). If you can't find something, let us know, then (or anyway) the docs about this should probably be improved 😄

Okay, cool, thank you! I'll try that one

Note that test environments gets passed the test path already.
https://github.com/facebook/jest/blob/2e454103365885534ae86f7f9bbf1bf6ae171133/packages/jest-runner/src/runTest.ts#L137-L141
So you can use a single custom env if you want, and differ the setup within that environment based on the test path

Silly me 🤦 , Jest already has "jsdom" environment (the default one), which almost perfectly fits my needs.
I guess, I'm gonna read the docs again 😃

@SimenB, @jeysal I'm impressed, how friendly and helpful you guys were! Thank you very much 👏

Ah right, missed that part of your use case.
Yeah, we ship a browser like (backed by JSDOM) and a node like environment, so if that's all you need you should be good. in a setup file you can do something like if (typeof window !== 'undefined') {.
Thanks for the PR regardless, even if it wasn't merged! 👍


Add actor and lastModified to DatasetOwnership structure

Coverage decreased (-0.03%) to 22.974% when pulling 9a8b24dbe3554a4ed42ec03bfd1dc66fee523a5c on alyiwang:master into b0037ccbc22918de9fc9000bc4e1799345a36ff4 on linkedin:master.


NHD waterways overlaps buildings

We noticed several waterways overlapping buildings while working on Northwest.  This was an old import from the National Hydrography Dataset. Many of these streams were either diverted or don't exist anymore.  For now, we leave it as it is if we can't find any visible stream in the imagery.  In the future, we should think of ways to fix this.
@jschleuss @almccon @cityhubla @krishnanammala

Perhaps you can leave an OSM Note whenever you see these? Someone who goes there to ground-truth might be able to see evidence of a buried or diverted stream.


Update documentations [Navigation Tab, Mode]

Final updates for features in UG and DG. Added diagrams and screenshots.

Pull Request Test Coverage Report for Build 1137

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.162%

Totals

Change from base Build 1196:
0.0%

Covered Lines:
2759

Relevant Lines:
3358

💛  - Coveralls


FIX: Add check for line endings

Adds a check for line endings being correct. vispy has used something similar for a while.

travis not happy

Thanks @Eric89GXL for taking a shot at this!

@jasmainak comments addressed.

@Eric89GXL I cannot switch branches right now because I'm running a long script. If it works for you, I'm +1 for merge.

If your script is in the process of running you should be fine to switch branches actually (as long as no data files have been moved/removed and you aren't launching new interpreters). Once mne is imported, switching branches shouldn't matter.

test passes on my box :)

+1 for merge when travis is happy

merged by rebase. thanks


New sample: Display KML tour

Demonstrates how to consume KML tour content in ArcGIS Runtime.

The audio should be in sync. I believe there was a recently fixed issue
Yes, the placemark should float for those locations.
Yes, that workflow is expected. The API provides no way to know when the tour ends.


Completion fails to find subsequent declaration in incomplete ternary

1 ? f // does not work the autocompletion, `func` cannot select.
1 ? 1 : f // work the autocompletion, `func` can select.

function func() {}

Divides #6079.

Specifically, you will only see this if you have one ternary at a time. In other words, this works:
1 ? 1 : f/**/

function func() {}

and this does not work:
1 ? f/**/

function func() { }

I think this is actually due to the parse error where we expect a :, and then assume we must have a function expression in the false branch. Since a function expression doesn't create a binding in the rest of the function, it won't show up in your completion list.
I wonder if a reasonable thing to do would be to parse out a missing node if the colon is missing and the following token is on the next line. It'd be reasonable to try it out.

Sorry for my poor English. Thanks for your fix.

It's fine, I knew what you meant, I was just changing the title so whoever works on this knows exactly what the problem is. :smile:


Cannot install

Error during install:
sudo pip3 install dataserv-client
Downloading/unpacking dataserv-client
Downloading dataserv-client-1.2.1.tar.gz
Running setup.py (path:/tmp/pip_build_root/dataserv-client/setup.py) egg_info for package dataserv-client
Traceback (most recent call last):
File "", line 17, in
File "/tmp/pip_build_root/dataserv-client/setup.py", line 4, in
from esky.bdist_esky import Executable
ImportError: No module named 'esky'
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 17, in
UBUNTU 14.10

Sorry you pulled a development version. Use:
pip install dataserv-client==1.1.0


chore(main): release 0.2.0

:robot: I have created a release beep boop
0.2.0 (2024-11-11)
Features

Sort behaviors by label (#75) (a996e4e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.2.0
:sunflower:


Possibility to set compaction strategy in configuration

Feature request: Possibility to define compaction strategy when configuring the plugin.

That's a useful feature. Would you like to work on it and create a pull request?

I don't know Scala and SBT, but I might be able to look at the work done with "gc-grace-seconds", and do something similar. I will give it a try!

Just a summary of what I need to do, for reference:
I will have to make create-table with the following, read from config:
with gc_grace_seconds = 123
AND compaction = {
       'bucket_high' : 1.5,
       'bucket_low' : 0.5,
       'class' : 'SizeTieredCompactionStrategy',
       'cold_reads_to_omit' : 0.05,
       'enabled' : true,
       'max_threshold' : 32,
       'min_sstable_size' : 50,
       'min_threshold' : 4,
       'tombstone_compaction_interval' : 86400,
       'tombstone_threshold' : 0.2,
       'unchecked_tombstone_compaction' : false
};

with gc_grace_seconds = 123
AND compaction = {
       'class' : 'LeveledCompactionStrategy',
       'enabled' : true,
       'sstable_size_in_mb' : 160,
       'tombstone_compaction_interval' : 86400,
       'tombstone_threshold' : 0.2,
       'unchecked_tombstone_compaction' : false
};

with gc_grace_seconds = 123
AND compaction = {
   'base_time_seconds' : 60,
   'class' : 'DateTieredCompactionStrategy',
   'enabled' : true,
   'max_sstable_age_days' : 365,
   'max_threshold' : 32,
   'min_threshold' : 8,
   'timestamp_resolution' : 'MICROSECONDS',
   'tombstone_compaction_interval' : 86400,
   'tombstone_threshold' : 0.2,
   'unchecked_tombstone_compaction' : false
};

@anderius do you need all of those options for each of the compaction strategies? There are lots of defaults and the options change between versions. Alternatively we could be more agnostic and as people to include the entire block so we are future proof.

That would be easier... But how can we validate it? Against terminating the statement and doing something else...
Maybe that is not important, I have not yet secured my Cassandra installation very much..

It is a valid point however I assume that this is coming from user config rather than from an external request/source.

Hmmm... After thinking a bit, I am not so happy with my suggestion. :-)
Putting all these parameters in your config will make the config dependent on a more specific version of Cassandra than before. And these parameters could easily be changed runtime with DevCenter...
Changing the settings and restarting your application will not have any effect, even if changing them runtime makes sense. This is just like gc_grace_seconds, however: Changing it will only make the configuration differ from the actual database settings (if I read the code correctly).
I am not so sure anymore...

I understand your concerns as it it might violate the principle of least surprise that the plugin doesn't update your schema to what you've put in your config. Cassandra schema migration can be a controversial topic :)
I still think there is value in having the plugin create the table with your Compaction strategy of choice without any extra options + well documented. Then leave it up to whatever config management tool people use to modify it if required.

You mean only specifying the strategy to use, and not any parameters? That would be cleaner...
Is it possible to say something about which parameters are stable, and which might change?
The following would be the most useful for me, and they are the same across all strategies:
'tombstone_compaction_interval' : 86400,
'tombstone_threshold' : 0.2,
'unchecked_tombstone_compaction' : false

I think properties that are common across all compactions strategies would be useful. There's always the chance that new Compaction strategies will come that don't have them but can add validation for that later.

This lost focus from me, because my project took another path... I will try to complete it sometime, but not knowing Scala even the simplest task took time...
Anyone knowing Scala: Feel free to do this!

I'll try to work on this. :)

Thanks @jparkie, looking forward to see the pull request :)

I've been very busy with starting school again, so I started working on this today. :) I ported over all the compaction strategies as defined in https://github.com/apache/cassandra/tree/cassandra-2.2/src/java/org/apache/cassandra/db/compaction and http://docs.datastax.com/en/cql/3.1/cql/cql_reference/compactSubprop.html.

Continued here: https://github.com/akka/akka-persistence-cassandra/issues/10


CI: Add image definition

What type of PR is this?
Refactor
What this PR does / why we need it?
Adds base image definition to the repository for easier configuration.

Relates to: https://github.com/openshift/release/pull/41967

/lgtm


pin the copy controls section

this is actually on the difficult side with how the layout currently works, unfortunately


custom formatter

Allow for custom prefix builder

Refactor prefix builder from interceptor to configuration
Inspired by #3

Yeah, this works for me :+1:

awesome.  v1.2.0 has been released with this change.


feat: impl ExactSizeIterator and DoubleEndedIterator

This PR implements ExactSizeIterator and DoubleEndedIterator for RangeIter simply by deferring to the inner iter impls.

ack


Added Offenders

Created a list of Offenders which can be updated for more offenders and updated if statements to include them. Tested and it works as intended:

Using a special attack will add the counter
Using another special attack on the same boss will update the counter
Attacking one of the offenders will not reset the counter
Using a special attack on one of the offenders will not update the counter
Attacking another npc will reset the counter

Do you know what this dark core npc is?

Do you know what this dark core npc is?

My bad, thought the Dark Energy Core had more Ids, just tested and both of there are not the actual NPC Corp spawns:

DARK_CORE
DARK_CORE_388
Only the DARK_ENERGY_CORE is the one that's supposed to be in the Offenders list

My bad, thought the Dark Energy Core had more Ids, just tested and both of there are not the actual NPC Corp spawns:

DARK_CORE
DARK_CORE_388
Only the DARK_ENERGY_CORE is the one that's supposed to be in the Offenders list

I've taken this pr and made https://github.com/Adam-/runelite/commit/589a362d140b098a08316ddbed81512a2ee4b998 instead, can you test that?

I've taken this pr and made https://github.com/Adam-/runelite/commit/589a362d140b098a08316ddbed81512a2ee4b998 instead, can you test that?

On your version, specing one of the ignored npcs will still add to the counter but not remove it

On your version, specing one of the ignored npcs will still add to the counter but not remove it

On your version, specing one of the ignored npcs will still add to the counter but not remove it

hmm do you see why? It should be just resetting the counter and counting the new spec instead. Did you test it and get this behavior?

On your version, specing one of the ignored npcs will still add to the counter but not remove it

hmm do you see why? It should be just resetting the counter and counting the new spec instead. Did you test it and get this behavior?

LN177  if (!wasSpec && IGNORED_NPCS.contains(interactingId))
With this logic the ignored npcs will only matter for normal hits that aren't specs, let me test removing the spec part completely

..? My intent is if you spec one of the ignored npcs it still counts. Since that doesn't happen normally. Right? Except maybe for the dummy...

You are correct, and honestly, the dummy only mattered for Trailblazer as a make-do ornate pool.
I can't think of any other scenarios when we would Spec one of the Ignored NPCs

If the dummy is to be included then probably the !wasSpec should be removed. Maybe there is some niche use for it now or in future leagues.

added in aa1cd2c7ff79e368db6769d1c09f6b8a11c917d9

added in aa1cd2c7ff79e368db6769d1c09f6b8a11c917d9


Incorrect system_clock::time_point to gpr_timespec conversion

Hello Folks,
It seems there is a bug in system_clock::time_point to gpr_timespec conversion in grpc.
Particularly in Timepoint2Timespec() (src/cpp/util/time.cc::51) , we have this line:
to->tv_sec = secs.count().
The problem is that secs.count() has type int64_t, but gpr_timespec::tv_sec is declared as time_t, usually a typedef for  'long int'.
The bug leads to run-time errors when running on 32-bit machines or under ARMv7, when 'long int' is less than int64_t.
For example, the CompletionQueue::Next() specifies std::chrono::system_clock::time_point::max() as deadline, but this is much larger than available value of gpr_inf_future.tv_sec:
deadline.count()=9223372036854775807
secs.count()=9223372036
nsecs.count()=854775807
Leads to:
to->tv_sec=633437444
to->tv_nsec=854775807
As result we have absolutelly incorrect behavior of gpr_cv_wait() from sync_posix.c.
Fix it please!

Thanks for reporting. We discussed on this before but did not add a guard,
and the new AsyncNext method makes it more serious. I will fix it.
On Fri, Mar 20, 2015 at 9:43 AM, Andrey Myznikov<EMAIL_ADDRESS>wrote:

Hello Folks,
It seems there is a bug in system_clock::time_point to gpr_timespec
conversion in grpc.
Particularly in Timepoint2Timespec() (src/cpp/util/time.cc::51) , we have
this line:
to->tv_sec = secs.count().
The problem is that secs.count() has type int64_t, but
gpr_timespec::tv_sec is declared as time_t, usually a typedef for 'long
int'.
The bug leads to run-time errors when running on 32-bit machines or under
ARMv7, when 'long int' is less than int64_t.
For example, the CompletionQueue::Next() specifies
std::chrono::system_clock::time_point::max() as deadline, but this is much
larger than available value of gpr_inf_future.tv_sec:
deadline.count()=9223372036854775807
secs.count()=9223372036
nsecs.count()=854775807
Leads to:
to->tv_sec=633437444
to->tv_nsec=854775807
As result we have absolutelly incorrect behavior of gpr_cv_wait() from
sync_posix.c.
Fix it please!
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/issues/1103.

Resolved by PR #1108


Is it possible to create a new Merchant via this SDK?

To create a payment for a Shop-in-Shop, one of the required data is a Merchant ID. Is it possible to create a new Merchant using this SDK in order to get a Merchant ID?

Paytrail's Merchant Creation API is separate from the Payment API, so there is no support for creating the sub merchants with this SDK.

@loueranta-paytrail thanks! Can you provide a link to the API documentation or the SDK that supports this API call?

Unfortunately I'm not aware of any SDK that supports that functionality.
However, the Merchant Creation API is quite simple compared to the Payment API, so implementation should be pretty straight forward even without SDK: https://services.paytrail.com/merchants/ext/docs/index.html


Creating branch to work on

closes #10267
Co-authored-by: Rida Hamadani<EMAIL_ADDRESS>

This is already done in #16109


Fix leak in sslengine

In case of the openssl provider, if the ssl engine is unused, then it will leak some structures like Bio and SSL that were allocated in initialization. This makes it so that we clean them up if peeking fails and we are never going to use the Sslhandler.

LGTM


Remote v2

Second try. This time it uses the eventserver interface and reproduces every single key in the remote keyboard as if it was attached to the host machine itself. Should need some testing first but in general it seems to work for me.
I don't think this is as responsive as the JSON-RPC interface but for the moment is the best way to implement such a thing. Post-EDEN versions should have much better command on it tho so in the future it might be worth switching.

All seems to work very well. The only thing i can fault is that scrolling through file lists is a bit slow, but afaik thats a limitation of the eventserver?

I think so. There might be a better way to handle it but I need to do some more research. The problem is that the eventserver does not give a response so we cannot know when a command has been successfully sent which makes it a bit difficult to increase the complexity of the remote. Furthermore, from a quick analysis, this is heavily affected by the currently playing requests. I think it only issues a new request after the last has been finished so if it sometimes gets stuck on an ajax request it will start pilling up new requests until the current one is finished. This is definitely something we do no want which is why I've been thinking of maybe creating some threading for the remote and currently playing?

if you play a video then press tab to get back to file list then hold down for a while then press tab again it it return you to the video and keep backskipping while it catches up on the down presses. so as far as i can tell it will accept a new command bfore it has executed all previous requests.

Dear God, this is absolutely amazing. Maraschino just got twice as useful!

@N3MIS15 : ye it definitely accepts new commands but they queue up so if too many commands get queued your XBMC will eventually go nuts until they're all used up. There should be a way to prevent that but I haven't figured out how yet, I guess on an internal network it wouldn't be a huge problem tho unless something else is taking most of the bandwidth.
@mrkipling : thanks man! appreciate it. Any opinions regarding threading? Possible, useful, unnecessary? What do you think?


App crashed when I clicked on show details on a song being played

I clicked on show details option on a song which I was playing and the application crashed.
I tried multiple times but the result was same.
Video of the bug
https://youtu.be/dOkFSJ0fV2M
App version: <IP_ADDRESS>

Ok, fix this bug later, the crash caused by the null pointer

fixed try new version (release page)


QCoDeS can't register proxy parameters based on parameter_class

My VNA, the Copper Mountain M5180, has qcodes parameters for each s-parameter, defined like
self.add_parameter(name='s11',
                           start=self.start(),
                           stop=self.stop(),
                           npts=self.npts(),
                           parameter_class=FrequencySweepMagPhase)

The instrument server and client don't have any problems with that parameter, but trying to add that the proxy parameter with meas.register_parameter(vna.s21) gives ValueError: Can not register object of type <class 'method'>. Can only register a QCoDeS Parameter.. It seems the client doesn't properly make that function a qcodes parameter when it's created.

Ok, it turns out that class FrequencySweepMagPhase(MultiParameter): is a MultiParameter, which isn't yet supported in instrument server, so the server doesn't add s11 to the parameters blueprints. I'm going to see if I can get a handle on how hard it would be to add support for MultiParameters and ArrayParameters.

@ttshaw1 yeah, i totally agree. we just didn't have any multiparameter instruments in use when we initially wrote this so we skipped dealing with that. would be absolutely worthwhile to have.
do you want to give this a shot? :)

I've got it on my to-do list but don't know when I'll be able to spend the time on it. If anyone else scoops it up I certainly won't be offended

@wpfff please see linked pull request above


Fix Jukebox in Origin when joining

This is not a full fix, but fixes the crash. Looks like it tries to play a sound without a time of day suffix when joining, but that doesn't exist.
It plays fine when warping to Origin though, so needs more debugging.

Thank you


Nothing backed up on Error

When I try to backup my Documents folder, I get the following output:
C:\Git\restic>.\restic -r D:\Backups\restic backup "C:\Users\Jan Bader\Documents"
enter password for repository:
scan [C:\Users\Jan Bader\Documents]
error for C:\Users\Jan Bader\Documents\Eigene Bilder: open C:\Users\Jan Bader\Documents\Eigene Bilder: Zugriff verweigert
error for C:\Users\Jan Bader\Documents\Eigene Musik: open C:\Users\Jan Bader\Documents\Eigene Musik: Zugriff verweigert
error for C:\Users\Jan Bader\Documents\Eigene Videos: open C:\Users\Jan Bader\Documents\Eigene Videos: Zugriff verweigert
scanned 205 directories, 6787 files in 0:00 MiB
[0:00] 2.36%  0B/s  5.929 MiB / 251.367 MiB  33 / 6992 items  0 errors  ETA 0:00
duration: 0:00, 546.02MiB/s
snapshot 92730b9b saved

Although a snapshot is created, no data is backed up:
C:\Git\restic>.\restic -r D:\Backups\restic cat snapshot 92730b9b
enter password for repository:
{
  "time": "2015-11-03T19:02:30.594317+01:00",
  "tree": "ac08ce34ba4f8123618661bef2425f7028ffb9ac740578a3ee88684d2523fee8",
  "paths": [
    "C:\\Users\\Jan Bader\\Documents"
  ],
  "hostname": "...",
  "username": "..."
}

C:\Git\restic>.\restic -r D:\Backups\restic cat tree ac08ce34ba4f8123618661bef2425f7028ffb9ac740578a3ee88684d2523fee8
enter password for repository:
{
  "nodes": []
}

Hm, strange. Thanks for the report. It looks as if restic is not allowed to open (and therefore: list) the listed directories. Could that be a permission issue?

I guess it's because the three folders are "Junctions" (not sure what they're called). Similar to Linux's symlinks.
When I open Explorer or use dir in cmd, the folders are not shown. My minGW Terminal's ls shows the following output though:
lrwxrwxrwx 1 Jan Bader 197121   27 Aug 20 04:12 Eigene Bilder -> /c/Users/Jan Bader/Pictures/
lrwxrwxrwx 1 Jan Bader 197121   24 Aug 20 04:12 Eigene Musik -> /c/Users/Jan Bader/Music/
lrwxrwxrwx 1 Jan Bader 197121   25 Aug 20 04:12 Eigene Videos -> /c/Users/Jan Bader/Videos/

Default permission set for these legacy junctions include: Everyone (this folder only): Deny List folder / read data. Those junctions are used for legacy application compatiblity only. Microsoft keeps users from using those folder directly by denying everyone the List folder / read data permission on these special folders. Most legacy applications can can continue access files below e.g. My Documents, as long as they do not try to iterate files directly below the My Documents junctions. You can try this yourself: Enter C:\Users\<Your Username>\My Documents\AnySubfolder into the explorer address bar. It will show you a directory listing of Any Subfolder. Entering  C:\Users\<Your Username>\My Documents\ will however show you "Access is denied."
Any other junction (without the mentioned ACL entry) behaves transparently in windows explorer.

Hm, it sounds like that the correct behaviour for restic would be to save them as symlinks instead of reading the content.

The new title is probably wrong: The Documents folder backed up by C:\Git\restic>.\restic -r D:\Backups\restic backup "C:\Users\XXX\Documents" is a "real" folder, not a junction. However by default this folder contains some legacy name junctions ("My Music", "My Pictures", "My Videos") with permission List folder / read data denied for everyone. To me it rather looks like restic is not handling the access denied error correctly!
Nevertheless backing up NTFS Junctions (Junction Points), Symlinks, Volume Mount Points and Hard Links should be fully supported in the medium term. I suppose restic silently follows them right now (assuming necessary permissions).

Thanks for the clarification, I've changed the title. @JaCoB1123 was antyhing contained in your Documents folder? restic should try to continue despite of errors, so it should have saved everything besides the Junctions.
I suspect that other things like sym- and hardlinks are followed without any special handling. For people who want to help out: In node_windows.go code should be added to handle these special files.

The Backup is instant though and the resulting snapshot is empty.

@JaCoB1123 build your own version with:
e:\home\jf\src\github.com\restic\restic>go run build.go -tags debug
e:\home\jf\src\github.com\restic\restic>set DEBUG_LOG=restic.log
e:\home\jf\src\github.com\restic\restic>restic.exe -r e:\tmp\b1 backup e:\

Here is my debug executable.
@JaCoB1123 by the way, try to avoid file or folder names with blanks. That's a big command line pain.

@frederich Thanks, I already did that and got no meaningful output.
Can't you "step through" somehow and analyze variable values at runtime?

@JaCoB1123 you mean interactive debugger? No, I don't like debuggers (see Torvalds why). One interactive Go debugger is delve.
The best way is to add print statements to restic and to build a custom Go tool chain with more print statements at syscall packages. Or use the built-in detectors like race.
@fd0 is it possible to use the go tool instead of go run build.go? Then @JaCoB1123 can use some detectors.

Of course it's possible, build.go is just a wrapper around the normal Go toolchain that additionally builds a GOPATH on the fly. Instructions on how to create a proper development environment and be able to use the toolchain directly can be found here: https://github.com/restic/restic/blob/master/CONTRIBUTING.md#development-environment


Evading Mutes through GTS

When muted, players are still able to add named items and Pokemon to GTS to talk in the chat. They use this exploit to continue conversations during their mute and there's no way on our end for us to stop it.

Most plugins that offer a mute feature such as Essentials have a configurable blocked command list for when a player is muted. If yours doesnt, the only other workaround I can recommend is creating a muted rank using your permissions plugin and setting the GTS permission to false and applying it to the player. Otherwise, a fix for this isn't really in scope of the plugin itself.


JuliaCon 2021: Atomic fields: the new primitives on the block

Video link: https://www.youtube.com/watch?v=2rBv6sV4Xts
0:47 Introduction
1:07 What is @atomic?
1:32 Not atomic: Single-threaded example
2:27 Not atomic: Broken example
3:31 @atomic: Fixed example
3:50 What @atomic does not do
5:03 Other atomic operators
9:53 Conclusion

I just edited the first 45 seconds of silence, can you modify these slightly?

Sure. I subtracted 47 seconds, but had to round some of the timestamps up or down because the transitions don't usually happen at the exact second marks.
0:00	Introduction
0:21	What is @atomic?
0:47	Not atomic: Single-threaded example
1:41	Not atomic: Broken example
2:44	@atomic: Fixed example
3:04	What @atomic does not do
4:18	Other atomic operators
9:06	Conclusion

Added! Thank you for updating this


Clean up grammar and formatting

Just some simple grammar and formatting improvements! I can do another, more in-depth, clean up if this is something that's useful.

But why should they the names be in smaller letters?

I was just going for consistency, since some were uppercase and some were lowercase. There's no particular reason why I made them all lowercase versus all uppercase. I left terms like programming languages and acronyms uppercase, since those are typically styled like that.

Okay. I like it.


Fix up the WIFI connection dialogs, plus some misc GUI fixes

Hello,
I've been using the wifi connection dialogs for a while, but the fact that I have to enter passwords twice is very annoying. I can't think of a good security reason to do this (especially considering other operating systems don't ask for them twice, and the show key checkbox makes the second line edit redundant). Anyone using the GUI is going to be rate limited anyway.
So, I dug into my old Qt hacking skills and removed the second dialog in all the wifi dialogs.
I also made it possible to cancel out of the dialogs. In the current version, if you enter the wifi dialog by accident, you can't just close the dialog. You must enter a password (and a correct one at that). This forgives choosing an incorrect wifi.
While I was there I "modernized" the calling conventions (well, how we designed them back in 2005) and noticed the code was leaking the dialog everytime it was created so I fixed those as well.
WPA Enterprise is currently in libtrueos-ui though I don't know why. I have also moved it into here. (and created a corresponding pull request in libtrueos.
There's also a .gitignore file, because it was annoying to see all the object files when calling git status
I'm sorry if these come through as too many changes in one shot. Each commit builds on the last, so you can stop your rebase where you want.

Looks good! Thanks for sending that in!

On 09 Jan 2017, at 18:07, Ken Moore<EMAIL_ADDRESS>wrote:
Looks good! Thanks for sending that in!

No problem. It was something that was bothering me for a while, but just finally had time to sit down and fix it.
Glad I could bring the WPA Enterprise stuff in too. Looking forward to seeing it in the next release.
--
Trenton


refactor(combine-to-osv): update only changed files in output/ dir

Create a modified_time.csv file to track the last modified time of all output files.
Generate output files only when any parts/ or CVE content is modified after the last recorded output modification time.
Update modified_time.csv after rewriting any files.

Close this PR as https://github.com/google/osv.dev/pull/2611 has significantly improved the performance.


Should debug integrated terminal support pass the env directly to createTerminal?

Looking at this:
https://github.com/microsoft/vscode/blob/b51019b5508096e2dbd7ae3c7ac84a3a0c22816c/src/vs/workbench/contrib/debug/node/terminals.ts#L113-L122
It looks like it would be a lot simpler to pass in env to createTerminal instead of constructing a per-shell command to set/unset them:
https://github.com/microsoft/vscode/blob/4d02d83c4183abeba42061135b0cd6a976dabab3/src/vs/vscode.d.ts#L7106-L7109

Debug tries to reuse a terminal if it is "free", i.e. if there are no child processes running in it.
So the terminal is created for the first "runInTerminal" and then reused (if possible).
Consequently passing env to createTerminal only works for the first "runInTerminal". Subsequent "runInTerminal" requests still require the current approach.

@Tyriar for addressing this issue I'm trying to use the terminalService.createTerminalFromOptions API. When filling up the "options" arguments, I can use "getDefaultShell". But why is the corresponding "_getDefaultShellArgs" private?
const options: vscode.TerminalOptions = {
	shellPath: this._terminalService.getDefaultShell(configProvider),
	shellArgs: this._terminalService._getDefaultShellArgs(configProvider),
	cwd: args.cwd,
	name: args.title || nls.localize('debug.terminal.title', "debuggee"),
	env: args.env
};
delete args.cwd;
delete args.env;
this._integratedTerminalInstance = this._terminalService.createTerminalFromOptions(options);

@weinand it just didn't have any consumers outside yet, whereas the default shell is exposed in the API via env.shell, feel free to change it to public


Move argument parser back to data_utils.py and make sure recons_nns.py works

#1

Is this issue closed?


Errata: Possible typo in Chapter 6

Chapter: 6
Description: Possible typo https://github.com/TartanLlama/sdb/blob/881d145832bb46d22d1208eb2c8ca2ced16d0947/tools/sdb.cpp#L194-L195 I have a feeling this line should be inside the catch block, otherwise we send an error regardless if parsing success.
Reproduction steps: N/A

Thanks for getting in touch! It's intentionally outside of the block,
because the valid cases all return early, so control flow shouldn't reach
that line.
On Sun, 29 Dec 2024, 12:38 Wenqing Zong, @.***> wrote:

Chapter: 6
Description: Possible typo
https://github.com/TartanLlama/sdb/blob/881d145832bb46d22d1208eb2c8ca2ced16d0947/tools/sdb.cpp#L194-L195
I have a feeling this line should be inside the catch block, otherwise we
send an error regardless if parsing success.
Reproduction steps: N/A
—
Reply to this email directly, view it on GitHub
https://github.com/TartanLlama/sdb/issues/7 or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOFIZS64BAFBWWTUQNNIGD2H7UNZBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJLJONZXKZNENZQW2ZNLORUHEZLBMRPXI6LQMWBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTLDTOVRGUZLDORPXI6LQMWSUS43TOVS2M5DPOBUWG44SQKSHI6LQMWVHEZLQN5ZWS5DPOJ42K5TBNR2WLKJYGE4DEMBUGU3DLAVEOR4XAZNFNFZXG5LFUV3GC3DVMWVDENZWGIZDGNBUGQY2O5DSNFTWOZLSUZRXEZLBORSQ
.
You are receiving this email because you are subscribed to this thread.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.

Thanks for the fast reply, I'll close the issue then!


Sometimes check goal fails during a Jenkins build

Hi, our CI pipeline run o Jenkins occasionally fails with:
Failed to execute goal org.ec4j.maven:editorconfig-maven-plugin:0.1.0:check (check) on project parent: /srv/jenkins/<some-path>/jenkins-log.txt
I believe that the fileset being verified, as well as plugin configuration is correct as the above error happens only sporadically. A re-run of the build job always succeeds.
Is there any way to make editorconfig plugin more verbose to help identify the problem?
Currently nothing apart from that message appears in the build log. I am also not able to access the jenkins-log.txt file mentioned in the message as is is probably just deleted after the Jenkins job finishes.
I am using editorconfig plugin in version 0.1.0
Thanks.

It turns out that the problem was caused by running Maven build stage in parallel with some other build steps.


GUACAMOLE-326: Explicitly deny attempted reads/writes to Windows named data streams.

From GUACAMOLE-326:

When a file has been downloaded over the internet via Windows 10, and that file is later transferred over Guacamole using the magic "Download" folder, multiple files end up being transferred: "filename.ext" and "filename.ext:ZoneIdentifier".
This is due to the way Windows 10 tags files downloaded over the internet with secondary "streams", such that it can warn users of the risks prior to executing those files:
http://stackoverflow.com/questions/4496697/what-is-zone-identifier
As Guacamole does not actually use such files, and the underlying filesystem does not support them, they should be appropriately ignored/rejected rather than downloaded.

Looks like ':' is a reserved character for windows filenames? This makes sense to me.

By which I mean that no normal file should have a filename like that, so this check should be safe I think.


View for notification

This looks like an awesome library.  Is there a way to customize the view so it just shows the time around a specific timeframe?   I'd like to use this in a custom UI for a rich push notification but obviously I don't want to show the whole day, just a couple hours before and after the event that's pushed to the user.

Hi @grosch,
CalendarKit was meant to be a set of separate modules, so you could use it to create your own UI components.
For your task, I'd recommend using just TimelineView, it should serve the purpose perfectly.
If you'd like to have a higher-level abstraction over TimelineView, you may wrap it into TimelineContainer to support scrolling to specific hour or the first event.

Thanks, @richardtop.  Could you give a bit more of a specific example please?  I'm looking at TimeLineView and none of the stuff in there is public so I'm not sure how to use this by itself.

At first, I recommend you taking a look at the "parent" view, i.e. TimelinePagerView. The method important for us is updateTimeline:
  func updateTimeline(_ timeline: TimelineView) {
    guard let dataSource = dataSource else {return}
    let date = timeline.date.dateOnly()
    let events = dataSource.eventsForDate(date)
    let day = TimePeriod(beginning: date,
                         chunk: TimeChunk.dateComponents(days: 1))
    let validEvents = events.filter{$0.datePeriod.overlaps(with: day)}
    timeline.eventDescriptors = validEvents
  }

The last line shows, how  the information in form of [EventDescriptor] is passed to the Timeline.
Also, you'll need to position your view in the parent view, i.e. set it's frame's y to some arbitrary number:
let hourToScroll = 18 // Scroll to 18:00
let fullHeight = timeline.fullHeight
let yOffset = fullHeight * hourToScroll/24
timeline.frame = CGrect(x: 0, y: yOffset, width: ...., height: ...)

Thanks.  Still not there though.  Everything seems to be internal variables.  I can put a TimelinePagerView on my screen and it draws fine.  However, updateTimeline wants to use the internal timeline variable to get the internal date variable so I can't actually set what the date is that I'm looking at.

Is there any chance you can just provide an example view controller that displays a day view set to a specific date with some events?

I'll send you an example in a moment

@grosch I've added example in #126
It looks like this:

You may want to tweak the library to meet specific requirements. Feel free to change it and open a pull request.

Thanks!  Please remember to update the cocoapod as well.  It's still pulling 0.2.1

Done. Feel free to close this issue, if you don't have other questions related to the Notification view.

Thanks much.  I've almost got this working now.  Might need to make a couple more things public, like is24hClock.  I'll open a new item on what I'm finding though.


Fix for create-tamagui fail to detect proper package manager

This pull request fixes #1272. I also encountered an issue while running the program where it failed when cloning the tamagui master repo. since it spits out "fatal error: too many arguments", I had to double-quote the local directory to handle spaces in paths

amazing, thank you


Add support for the Galaxy S3 Mini (GT-I8200)

This is tested on a physical device.

This is not in Google's device list. It must be in Google's device list to be added to the library. The method is automatically generated from the list.
https://support.google.com/googleplay/answer/1727131?hl=en

It is on Google's device list now.

Added GT-18200 here: https://github.com/jaredrummler/AndroidDeviceNames/commit/9c45e7cc59621db26e214a322cde323da003070d
The official name is the Galaxy S3 Mini Value Edition so I will not merge this PR but appreciate your contribution. Thank you.


feature: 1) share a library to users 2) return full response in a function

feature:

share a library to users
return full response in a function

Hello, thanks for the PR. Please split up the PR. I am OK with 16580bb but c5bfac1 I would like to have changed.
Instead of returning a boolean and accessing the response body by reference I would rather have the method return the full Response object.
Regards
Rene

@rene-s OK. So should I just create two PRs, or I just committed the 3rd time to implenent return full response?

Hello, 2 PRs would suffice, thanks.

I cannot fully follow you. I guess you mean that I should give one PR for share a library to users. Am I right?
So should I create a new PR(and branch) and close this one or do in other ways?
Thanks.

@rene-s I have opened another pull request with only the library share function.


Archive differ widget refactored

wrapped around some function
writing full csv instead of snapshot
added comparison with merge

no longer using this script


raftLocalState is updated in an inappropriate method

raftLocalState is currently updated in doApplySnapshot (see below) when it has nothing to do with snapshot for normal cases. Move it to a more appropriate place.
https://github.com/matrixorigin/matrixcube/blob/766748e4ee72c27f9c105da3ae25910e6a842ce8/raftstore/peer_event_raft_ready.go#L364

local state will be removed as a part of the ongoing refactoring.


react-native-video shows blank screen in my app without any error log

Current behavior
Describe what happens when you encounter this issue.

I am using only one video file for each screen (I have four screens), when I switch to another screen the video not play on the next screen when I goto previous screen video is playing and shows the black (blank area) screen without throwing error.

Reproduction steps
A 1, 2, 3, etc. list of what's needed to see the issue happen.
Expected behavior

I just want to play video on each screen as I goto another screen

Platform
Which player are you experiencing the problem on:

Android ExoPlayer

Video sample

I have downloaded this video and use it from my project.
<Video source={require("../../assets/video/backgroundVideo.mp4")} />

https://github.com/react-native-community/react-native-video/issues/338#issuecomment-251856958

I used to run into stucked black screen all the time. The most strange one I experience is https://github.com/react-native-community/react-native-video/issues/1538
My feeling is this project manage the native video player instances very poorly (especially on Android). The project frequently mess things up when you have multiple native video player instances. In our project, we had to do a lot of difficult and hard things to ensure that we only have one instance of a screen that have the  tag, thus only one native video player instance, then it won't stuck.

Can you try on 4.4.4?

Does changing to 4.4.4 resolve the issue

Can you try on 4.4.4?

Does changing to 4.4.4 resolve the issue?
Have been stuck in this for a long time now

+1

"react-native": "0.61.5"
"react-native-video": "^5.0.2"
React Native Android Black Screen For Android

can someone tell me if this is related to https://github.com/google/ExoPlayer/issues/58, https://github.com/google/ExoPlayer/issues/2993 ?? because I do remember experiencing those issues before?

Can you try on 4.4.4?

can someone tell me if this is related to google/ExoPlayer#58, google/ExoPlayer#2993 ?? because I do remember experiencing those issues before?


fix: missing translations

Pull Request

Description

Added missing translations for optimiser tab
Modified stat weight sliders and E/S placeholders to adjust width as necessary

Related Issue

Checklist

[x] I have added commit messages that are descriptive and meaningful.
[x] I have tested the changes locally.
[x] I have reviewed the code changes.

Screenshots

I think the right CSS for the button previews is:

Parent: width: 'max-content'
Children: flex: 'auto'


provide signaling for arbitrary signals

I think that test should work, grep should hang since no stdin is supplied.

Coverage increased (+0.1%) to 85.78% when pulling df62b64719f8504216cfa50017623e58ce881914 on imgix:master into 0694053ec5afb3cdd857bbc38d3d84e8fa0e4acb on cablehead:master.

Coverage increased (+0.18%) to 85.86% when pulling 0c7804419f2e52e4194445140d0bd3ca5658963f on imgix:master into 0694053ec5afb3cdd857bbc38d3d84e8fa0e4acb on cablehead:master.

Coverage increased (+0.18%) to 85.86% when pulling ca4ab12d7e92311c3fd6f0827dbaa5005c1f8214 on imgix:master into 0694053ec5afb3cdd857bbc38d3d84e8fa0e4acb on cablehead:master.

Coverage increased (+0.18%) to 85.86% when pulling 1beb4cefb97f63c3c266ce2b8a7520d7654ae9dc on imgix:master into 0694053ec5afb3cdd857bbc38d3d84e8fa0e4acb on cablehead:master.

Thanks. I'm so happy right now, haha!


Fixed small typo in custom action template

I ran fastlane new_actionand spotted a typo in the generated file so I fixed the typo in the template file.

@iONsky Thanks! 🎉


Select: added CASE and OF for highlighting

Leave a comment

Nice Thank you!

@all-contributors please add WilliSeng for code