Jumping between Open UDS files results in different camera positions

Describe the bug
A clear and concise description of what the bug is.
Occurs when you have 2  (or more) UDS files open in Vault at the same time.
If you double click on a UDS model that is not open you will jump into that UDS file at some random camera position in space.
When you double click the other UDS file (the original one that you were in) you are taken back into that previous model but you will not find yourself at the same location as you were before.
This is rather annoying especially if you are demonstrating the software to a potential client.
It forces you to waste time panning and zooming to get back to the point in space where you were before.
Steps To Reproduce
Steps to reproduce the behavior:

Open 2 separate UDS models

Take note of the position in space that you happen to find yourself (take a screen shot if you need to).

Double click the other model (that is not open) in the right side scene explorer.

Now go back to the original model by double clicking it in the right side scene explorer. You will not be in the same location in space as you were before.

Expected behavior
A clear and concise description of what you expected to happen.
Expect to be at the same camera position as before
Screenshots
If applicable, add screenshots to help explain your problem.
See attached video capture -
Note: Please allow for larger attachments in the future and support for wmv files. A 10 megabyte file limit is a bit small if you want to use a video capture to show an issue..
Desktop (please complete the following information):

Platform: [e.g. Windows, iOS, Chrome]
Camera-Position-Moves.zip

Additional context
Add any other context about the problem here.

I believe that double-clicking a model takes you to a central location within the model. If you wish to jump to specific camera locations/orientations then they can be set by adding Viewpoints to each model then double-clicking those, rather than the actual model names.

I think that's a good idea. The current behavior is consistent with the
design from a programmer's perspective however the idea presented is much
more intuitive for a user.
On Mon, Sep 30, 2019, 8:37 AM Carl Francis<EMAIL_ADDRESS>wrote:

I believe that double-clicking a model takes you to a central location
within the model. If you wish to jump to specific camera
locations/orientations then they can be set by adding Viewpoints to each
model then double-clicking those, rather than the actual model names.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Euclideon/vaultclient/issues/12?email_source=notifications&email_token=AC3GI46P2K2WBPQYJM7YQIDQMEU4NA5CNFSM4I3TLZOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD74AQJI#issuecomment-536348709,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC3GI47W3N43M6DRX2XBDRDQMEU4NANCNFSM4I3TLZOA
.

There is a lot of complexity with this as well.
For example; if you double click Brisbane and then manually fly to the Gold Coast (another city close by) then when you double click Brisbane again you would want it to be in Brisbane- not the Gold Coast but double clicking the Gold Coast you would want it to go to where you were when you left the camera when you went to Brisbane? This gets even more complicated when there are lots of models sort of overlapping each other- at what point does it stop remembering the nearby cameras?
Perhaps we just keep track of the camera over the last 5 'jumps' you did and have some UI to easily jump back to those?
@CarlCFX is right in that the Viewpoint system was designed to solve this problem.

Hi OsiJr - I don't think that the user will get confused since what I'm proposing is more in line with how other software works.
If I have multiple Word documents open and I jump between them I expect to go back to the exact same spot I was in before I jumped out of that document and into the other document. People are used to this behaviour so I don't think that they'll get confused by it. The software only needs to keep track of the last active position that the user was in.
Ideally, the position of the camera would be saved with the UDS file when the user closes it from the scene explorer such that the user is taken right back to their previous camera position when the file is re-opened. This would be the same behaviour as when saving an AutoCAD drawing for example. When you re-open the drawing it places the user in the last viewpoint that they occupied when they closed the drawing.

@CodeWarrior99, that's not the same metaphor as this bug is referencing though.
UDS files aren't different Word documents, they're different sections in the same document (different models in the scene). If you're in one section in a Word and jump to another section in the same document it doesn't remember where you were in the previous section.
We can make what you want work easily with different projects though? Where if you load another project it remembers where the camera is in the current project on disk so that when you open it next time it goes to the same place. This would be written to AppData (or equivalent) most of them time as most projects are read-only for most users.
If you want it to stay per model there are a lot of options but they all have downsides and overall seem more annoying than setting up proper viewpoints in the project (like how you set up bookmarks or a TOC in Word).

@OsiJr - I often have completely different UDS files open at the same time in the scene explorer. For example, I will often times flip quickly between a model of Vancouver and one of Bergen, Norway by double clicking them in the Scene Explorer. I guess this is what you mean by treating them as projects?

Hi Paul,
Ok - I can see the problem from your point of view now (all UDS files listed in the scene explorer are part of the same project and it's possible to manually fly between them even if they are really far apart).
Here's what I think could work:
Assume the user starts up Vault from scratch and no UDS files are loaded yet.
The user decides to load 3 separate UDS models which are named A, B and C for simplicity.
Step 1.  The user first drags in model A from windows explorer and A becomes the "active" model.
Step 2.  The user then drags in model B which then becomes the new "active" model.
Step 3. The user then drags in model C which becomes the newest "active" model and the camera position appears in some random location.
Now the user decides that she wants to open model A so she double clicks it in the Scene Explorer.
Vault then detects that she has double-clicked A.
However, before Vault does anything it realizes that C is currently the "active" model.
Vault then makes note of the current camera position and associates it with C.
Vault then allows the user to be teleported to model A.
Vault makes not of the current camera position of model A (which is the same as when A first loaded).
Vault then continues to monitor for double clicks of any models in the scene explorer list.
If it detects any it will then perform the same task as above.
If the user decides to manually fly from A back to C and then decides to double click C in the scene explorer they will find themselves at the last camera position when C was the active model before jumping to A. After making this jump the user could be confused and may like to go back to their previous position (when A was active and before they double clicked C) they can do so by simply double clicking A in the scene explorer list (as long as they can remember that A was the last active UDS file in the scene list).
This should be fairly intuitive to users (I think).
It also doesn't stop you from creating a different mechanism to allow the user to manually create and viewpoints (bookmarks) during their Vault viewing session.
Sketchup has an interesting way of creating viewpoints that can be animated that you might like to have a look at for inspiration (see attached).
Viewpoints in Sketchup.zip


missing include files

Recently, I got problems when trying to build paraview project, whick are described as follows:
218>  pqLineEditEventPlayer.cxx
218>e:\0_open_sources\kitware\paraview\qt\widgets\pqLineEditEventPlayer.h(35): fatal error C1083: can't open include file: “pqAbstractStringEventPlayer.h”: No such file or directory
218>  pqListWidgetCheckHelper.cxx
218>  pqTreeWidget.cxx
218>E:\0_open_sources\Kitware\ParaView\Qt\Widgets\pqTreeWidget.cxx(45): fatal error C1083: can't open include file: “pqTimer.h”: No such file or directory
218>  pqTreeWidgetItem.cxx
218>  moc_pqColorButtonEventPlayer.cpp
218>e:\0_open_sources\kitware\paraview\build\qt\widgets\../../../Qt/Widgets/pqColorButtonEventPlayer.h(35): fatal error C1083: can't open include file: “pqWidgetEventPlayer.h”: No such file or directory
218>  moc_pqColorButtonEventTranslator.cpp
218>e:\0_open_sources\kitware\paraview\build\qt\widgets\../../../Qt/Widgets/pqColorButtonEventTranslator.h(35): fatal error C1083: can't open include file: “pqWidgetEventTranslator.h”: No such file or directory
218>  moc_pqColorChooserButton.cpp
218>  moc_pqColorDialogEventPlayer.cpp
218>e:\0_open_sources\kitware\paraview\build\qt\widgets\../../../Qt/Widgets/pqColorDialogEventPlayer.h(35): fatal error C1083: can't open include file: “pqWidgetEventPlayer.h”: No such file or directory
218>  moc_pqColorDialogEventTranslator.cpp
218>e:\0_open_sources\kitware\paraview\build\qt\widgets\../../../Qt/Widgets/pqColorDialogEventTranslator.h(35): fatal error C1083: can't open include file: “pqWidgetEventTranslator.h”: No such file or directory
218>  moc_pqConsoleWidgetEventPlayer.cpp
218>e:\0_open_sources\kitware\paraview\build\qt\widgets\../../../Qt/Widgets/pqConsoleWidgetEventPlayer.h(35): fatal error C1083: can't open include file: “pqWidgetEventPlayer.h”: No such file or directory

After checking the source repository, i didn't find these include files either.
Would you tell me where to git-clone or download these include files.
Thanks very very much and appreciate your excellent work.

Hello,
It looks like some Qt files are missing. Are you using a system-wide install or building it as a VeloView sub-project ?
If you have a system install, then you should check that the version you are using contains all needed files.
If you are using the version automatically build from the VeloView sources, then ... you probably found a bug. Please report your full configuration so we can investigate.
Best regards,
Bastien Jacquet - Lead VeloView Developer

I'm really sorry that i didnot write correctly down my project environment. Actually, i was previously building up paraview sdk, then i can build up veloview project. However, i got the problems as described before, when i was building paraview, and i found there indeed are no those .h files in paraview repository.
Really apreciate your attention and replay , and happy thanksgiving day.


[EPL/TV] — Arsenal vs Liverpool LiVe 𝐒𝐭𝐫𝐞𝐚𝐦𝐬 FrEe BrOaDcAsT oN TV Guide & 𝐒𝐭𝐫𝐞𝐚𝐦𝐢𝐧𝐠 27 October 2024

The last match of Matchday 9 in the Premier League sees rivals Arsenal and Liverpool taking the pitch on Sunday, Oct. 27 at Emirates Stadium in London.
🔴🌍📺📱👉GO LIVE==►► CLICK HERE TO STREAMING
🔴🌍📺📱👉STREAMING==►► CLICK HERE TO WATCH LIVE
The match is scheduled to start at 12:30 p.m. EST and will be broadcast exclusively on Peacock, which provides highlights and post-match coverage across the Premier League.
Peacock also offers on-demand replays of Premier League matches shown on NBC, CNBC and USA Network in addition to having exclusive live games as well.
These are two of the top clubs in the league early into the season, with Liverpool entering the match at second on the table and Arsenal fourth. Liverpool is 7-0-1 and Arsenal 5-2-1 through eight matches.
Arsenal has a discipline issue and it could cost the team the Premier League title
By STEVE DOUGLAS AP Sports Writer
Mikel Arteta felt he had no choice.
The Arsenal manager saw his right back Ben White receive a yellow card in the first half of the Champions League game against Shakhtar Donetsk on Tuesday, and quickly came to a decision.
White needed to be removed at halftime.
“We have played enough with 10 men in the recent period,” Arteta said with a smirk.
It is becoming abundantly clear: Arsenal has a discipline problem — and it might yet cost Arteta’s team a shot at the English Premier League title.
Arsenal has had three players sent off in the opening eight rounds of a league campaign that is seeing yellow cards being dished out at an unprecedented rate.
It continues something of a running theme under Arteta. Since he arrived as its manager in late 2019, Arsenal has collected 18 red cards in the Premier League — five more than the next team.
Tellingly for Arteta, the only games where Arsenal dropped points this season — the 1-1 home draw with Brighton, the 2-2 away draw at Manchester City and the 2-0 loss at Bournemouth on Saturday — came when the team had a player dismissed.
“We cannot continue to play with 10 men, especially at this level. You see how we struggled,” Arteta said this week. “We need to eradicate it, it’s clear. Why, the reason, how — it doesn’t matter. We have to focus that it has to happen.”
It particularly has to be happen on Sunday, when Liverpool visits Emirates Stadium in the headline match of the league’s ninth round.
Liverpool is in first place, one point ahead of second-placed City and four clear of third-placed Arsenal. A win would put Liverpool seven points clear of Arsenal already — hardly an insurmountable deficit at this stage but one which would leave Arteta’s players with little wiggle room. Perhaps more importantly, it would likely leave Arsenal six points behind defending champion City, which is expected to swat aside winless Southampton on Saturday.
Arsenal’s disciplinary issues come at the start of a season that has seen an average of 5.1 yellow cards awarded per game so far, according to league statistics supplier Opta.
That is far more than any previous Premier League, says Opta, which points out that last season’s 4.2 yellows per game was a new record — surpassing 3.7 per game in the 1998-99 season.
Two of Arsenal’s dismissals — Declan Rice against Brighton and Leandro Trossard against City — saw the players in question each collect two yellow cards, the second for time-wasting by kicking the ball away.
William Saliba was handed a straight red against Bournemouth for bringing down Evanilson near the halfway line and denying what was adjudged to be a goal-scoring opportunity.
Saliba, perhaps Arteta’s most important defender, will miss the Liverpool match as a result, at a time when Arsenal is already without captain Martin Odegaard (ankle) and might also be missing star winger Bukayo Saka (hamstring) and summer signing Riccardo Calafiori, who came off against Shakhtar with a twisted knee.
Liverpool will arrive on the back of 11 wins from its first 12 games in all competitions under new manager Arne Slot. That run contains six straight away victories, which is a club record for the start of a single campaign.
“Arteta has done an amazing job in the last few years,” Slot said after Liverpool’s 1-0 win at Leipzig on Wednesday, “and we have to be on top of our game to get a result.”

frdsd

This is best option to Live Stream This Event, Follow Given Link for Free!👇👇👇

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

▶
➤►
▶
➤►
▶
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►
➤►
▶
➤►This is best option to Live Stream This Event, Follow Given Link for Free!👇👇👇

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

⭕🟢👉CLICK HERE TO LIVE STREAM for Free

⭕🟢👉CLICK HERE TO LIVE STREAM for Free


Integrate ROMS tools

For the InputDataset class, we should support the ability for the source attribute to point not only to a pre-created netcdf file containing the dataset, but also a yaml file containing instructions for creating the dataset with ROMS-tools. The file_hash attribute could still be used to verify that the created file is identical to the one that would be sourced remotely.
Related issue: roms-tools # 63

@dafyddstephenson I just merged a PR that adds yaml recipes to ROMS-Tools (https://github.com/CWorthy-ocean/roms-tools/pull/64). Obviously, I still have to work on "steps 7-10", but maybe we can already start the integration process and see if I need to change anything about the .to_yaml and .from_yaml methods on the ROMS-Tools side.


As an app developer I want to init a new function that includes an HTTP Trigger.

With the move to the new v2 API, where Routes are replaced with Functions and Triggers, the 'fn init' command along with the content of func.yaml need updating to meet the needs of the new model.
func.yaml
A v1 file looks like this:
name: myfunc
version: 0.0.1
runtime: java
cmd: com.example.fn.HelloFunction::handleRequest
build_image: fnproject/fn-java-fdk-build:jdk9-1.0.63
run_image: fnproject/fn-java-fdk:jdk9-1.0.63
format: http
A v2 file with an equivalent HTTP Trigger should look like this:
name: myfunc
version: 0.0.1
runtime: java
cmd: com.example.fn.HelloFunction::handleRequest
build-image: fnproject/fn-java-fdk-build:jdk9-1.0.63
run-image: fnproject/fn-java-fdk:jdk9-1.0.63
triggers:
    my-trigger:
        type: http
        source: /my-trigger-path
A v2 file without any Triggers should look like this:
name: myfunc
version: 0.0.1
runtime: java
cmd: com.example.fn.HelloFunction::handleRequest
build-image: fnproject/fn-java-fdk-build:jdk9-1.0.63
run-image: fnproject/fn-java-fdk:jdk9-1.0.63
fn init
With the v1 API, fn init my-func --runtime=java creates a Route. In v2, Functions can exist with or without Triggers, so the fn init command needs a little more flexibility.
Initialising a new function that includes a Trigger should be done like so:
fn init my-func --runtime=java --trigger=http
and creating a Function without a Trigger would simply be:
fn init my-func --runtime=java
Alternatively, we could preserve (or mimic) existing behaviour whereby fn init my-func --runtime=java will auto-generate an HTTP Trigger, and a --trigger=none could then generate a function without a Trigger, but this seems a little clunky and forcing compatibility at the cost of clean syntax.

A couple of things not quite right here to my mind, specifically around 'version'. The new V2 fn entity doesn't have a distinct 'version' field, the value is bundled into the 'image' field.
This actually presents an opportunity to retrieve 'version' for it's more regular usage as an api version. Therefore a V2 fn definition would look like:
fn.yaml
version: 2
name: myfunc
runtime: java
cmd: com.example.fn.HelloFunction::handleRequest
build-image: fnproject/fn-java-fdk-build:jdk9-1.0.63
run-image: fnproject/fn-java-fdk:jdk9-1.0.63:1.0.0
triggers:
    my-trigger:
        type: http
        source: /my-trigger-path

We should also start calling these things 'fn.yaml' to match the terminology used in the service. That would also really help differentiate the versions of things.

fn.yaml is an interesting thought, and an easy v1 v v2 detection mechanism.  I also took the leap of removing "format", which is really where we need to go with the cloud-event only direction.

I also changed "_" to "-"...

The version property is used by fn build|deploy|push to set the tag on the generated image.  I have no problem with distinguishing between image tag/version and file format/API version but don't we need both?
I like the idea of moving to fn.yaml as we embrace this new structure.
format should definitely be obliterated! ;)

format dying is a good thing but it does rather rely on having all the necessary cloud-event support in place first, which isn't very high up the priority list at this time. But that can be discussed.

 Just a Docker image in the repo which will never get called?

A function without a trigger can still be invoked using a new "direct invoke" endpoint, something like http://localhost:8080/v2/invoke/.


[Feature]: Implement event tracking

[ Y] Would you like to take up this task?
[Y ] Are you from GSSoC'23?

✨
Feature request

Is your feature request related to a problem?
Kindly provide an explanation of the problem that your feature aims to resolve.

Describe the solution you'd like
Please present a concise and precise description of the solution you envision.

i will add apis, which will track any event or action.

Describe alternatives you've considered
Elaborate on any alternative solutions or features that you have already evaluated as potential approaches.

Additional context
Include any supplementary information, contextual details, relevant screenshots, or supporting materials that can aid in better understanding and supporting your feature request.

Kindly explain what you want to implement?
@kharshita590


Fails on windows if calling on a directory that already exists

Error: EEXIST: file already exists, mkdir

worked for me with no issues, on windows 10

I get the same error with<EMAIL_ADDRESS>in the context of<EMAIL_ADDRESS>The real issue is related to a Maximum call stack size exceeded catched and thrown from this part of the code:
https://github.com/substack/node-mkdirp/blob/48e67fce39f1a5f2aba6196301b7be6d4545d6cf/index.js#L83-L94

The maximum call stack size exceeded is throwed by some of my code but I don't get how executing xfs.mkdirSync(p, mode) can lead to executing my code that is throwing an error catched by try/catch block around it
https://github.com/substack/node-mkdirp/blob/48e67fce39f1a5f2aba6196301b7be6d4545d6cf/index.js#L70-L74

You can reproduce doing something (ugly) like
const mkdirp = require('mkdirp')
global.Date = function() {
  throw 'here'
}
mkdirp.sync(process.platform === 'win32' ? 'C:||Users': '/Users')

Note that in further investigation of the issue I noted above in CodeceptJS, the root cause ended up being that that library was forming an invalid Windows path name (it had ; character in it).  This also produces ENOENT error which is indistinguishable from simple directory not present condition which is what is assumed here in the ENOENT case for the catch block here.  I don't believe this to be an issue for mkdirp library as it is only limited to being able to get information NodeJs gives to it.
If you are running into this error on Windows, make sure you have valid path name.


Upgrade Preconstruct

Hm, I run corepack pnpm i (just like in the other PRs targeting your repos). I re-run this and got a slightly different result now (maybe I've done something differently before, ain't sure) but it still fails. I also tried plain pnpm i rn and got the same result as with my latest run of corepack pnpm i.
Could you perhaps fetch this branch and try this out yourself? At a glance, I'm not sure what's happening.

Perhaps this lockfile is just subtly broken? It gets updated in a similar fashion with corepack pnpm i --fix-lockfile

Edit It also failed on my branch, and locally. Looks like something about the upgrade is causing trouble with the playwright dep. This update to the nested package JSON fixed it.

Pushed out this change, let's see what happens. Perhaps pnpm refetches the dependencies using a tag version (Playwright) when updating unrelated~ parts of the lockfile? 🤷♂

🤷🏼  Trying to update #10 in parallel.

Landed via #10!

It was a tight race! Brian: 🥇 Andarist: 🥈

😂


it worky

no issue js wanna say keep up the good work these people prob didnt do it right. update the followers and likes tho.

Thank you! I can't control Zefoy's website so it's up to them to make the services available


DBZ-7926 Update format of type field values in CloudEvent records

DBZ-7926
To align with an earlier code change (DBZ-7216), this issue makes the following changes to the CloudEvents documentation:

Updates the format that is used for values of the type field of a CloudEvents change record.
Implements other related minor formatting changes.

Changes were tested in a local Antora build.

@roldanbob Applied, thanks


Storage not survive across sessions

I tried to apply the persistence logic to the MsProject add-in.
For all methods, including "Local Storage", "Browser Cookies", and "Document", all stored value cannot survive after i closed the document and open it again.
Is persistency not available in MsProject? I have checked the references and it said MsProject should provide persistency "By document". Can someone help?

Sorry it took us so long to respond. We will try to verify/reproduce that issue and respond here very soon.

After looking at this issue further, it appears that you have a question about add-in capabilities in Project. This sample is an Excel add-in, so I can't reproduce your issue with the sample. Please post this question to Stack Overflow, and be sure to tag your question with "ms-project" and "office-js".


Dynamic trees crashing when breaking any tree.

My very first post here, so excuse if format isn`t correct :)
Describe the bug
Dynamic trees crashes when breaking any tree with-Exception in server tick loop java.lang.NoSuchMethodError:
To Reproduce
Install Project MMO https://www.curseforge.com/minecraft/mc-mods/project-mmo and
break any tree.
Screenshots
None, full crash log here: https://pastebin.com/EBGz8zpz
Rendering Glitches
No render glitches
Additional context
After some testing, project MMO seems to be the culprit, as once removed mod works just fine with no crashes.
Versions:

Minecraft: 1.16.5
Loader: Forge-36.2.39
Dynamic trees: Latest for said version

This appears to be on their end, I think they're calling a method we removed or changed the signature of during the beta. Please open the issue on their tracker.

Okay will do :)


Fix #17469: Fixed updating Yii logger instance when setting new logger via configuration

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
#17469

I have mixed feelings about this. It looks like ugly voodoo magic that assumes that you always have only one dispatcher and it is Yii::$app->log (which is really naive assumption). Shouldn't we at list check if $this === Yii::$app->log before implicitly assigning static global logger.

That will return false most of the time since logger is initialized early and then cached statically forever. That is indeed, a bit of voodoo magic but that is what service locator brings us :(

a bit of voodoo magic but that is what service locator brings us :(

Not it's not. This is the result of ugly shortcut which creates twisted relation between Yii::getLogger() and Yii::$app->log->logger. This PR just make it way more magic (by introducing BC break BTW - this is really ugly and unexpected side effect). You could have service locator without assuming that one class could be instantiated only once.

Yes but in this case it was already working that way (when getting it) but without the ability to configure it properly. That part was fixed.

There is a difference between using Yii::getLogger() as a fallback, and between implicitly setting global logger when you use setter on local object. Now if you use in your code new Dispatcher(['logger' => $myFakeLogger]); it will magically break your app by overriding global logger.

Right. Is there a better way to solve original issue?

I'm not sure. But I would rather do not touch it at all than introduce BC break that makes this part of framework even more magic. This issue is known from at least 3 years (#12163) with known (Yii::setLogger(new MyLogger()) before app initialization) and consistent (you should use Yii::setLogger() to change logger returned by Yii::getLogger()) workaround, I'm not sure it is a good time to make such changes.

I agree with @rob006, please no BC breaks in 2.x

OK. Will be reverted next release.


confusions about dump_bin.py

In  https://github.com/microsoft/qlib/blob/main/scripts/dump_bin.py#L352   and
https://github.com/microsoft/qlib/blob/main/scripts/dump_bin.py#L315
There are two classes that I don't know how to use, class DumpDataFix(DumpDataAll), and  class DumpDataUpdate(DumpDataBase), can anyone explain it for me? I am looking for some  ways to update dump data, but did not succeed till now.

me too when i update , it always wrong

The dump has three classes:

DumpDataAll: dump all data

Dump all data from csv_path into qlib_dir, which will overwrite all data (calendars/instruments/features)

DumpDataFix: adding a new field, symbol, rewriting of an existing field

Dumping data from csv_path into qlib_dir will overwrite existing features data; if new fields are added, they will be added according to qlib_dir/calendars/<freq>.txt and the qlib_dir/instruments/all. txt file, without modifying existing data

DumpDataUpdate: append new data to existing data

append the data from csv_path to the existing data

For example:

Data already available

# Data already available:
    qlib_dir/
        calendars/
            day.txt: [2021-06-01, 2021-06-02]
        instruments/
            all.txt:
                SH600000    2021-06-01  2021-06-02
        features/
            SH600000/
                close.day.bin: [1, 2]

csv_path

# csv_path:
    SH600000.txt
        datetime    symbol  close open
        2021-06-01 SH600000  1.1   1
        2021-06-02 SH600000  1.2   1.1
        2021-06-03 SH600000  1.3   1.2
    SH600001.txt
        datetime    symbol  close open
        2021-06-01 SH600001  2.1   2
        2021-06-02 SH600001  2.2   2.1
        2021-06-03 SH600001  2.3   2.2

After DumpDataFix

# After DumpDataFix:
  qlib_dir/
      calendars/
          day.txt: [2021-06-01, 2021-06-02]
      instruments/
          all.txt:
              SH600000    2021-06-01  2021-06-02
              SH600001    2021-06-01  2021-06-02
      features/
          SH600000/
              close.day.bin: [1.1, 1.2]
              open.day.bin: [1, 1.1]
          SH600001/
              close.day.bin: [2.1, 2.2]
              open.day.bin: [2, 2.1]

After DumpDataUpdate

# After DumpDataUpdate:
    qlib_dir/
        calendars/
            day.txt: [2021-06-01, 2021-06-02, 2021-06-03]
        instruments/
            all.txt:
                SH600000    2021-06-01  2021-06-03
                SH600001    2021-06-01  2021-06-03
        features/
            SH600000/
                close.day.bin: [1, 2, 1.3]
                open.day.bin: [1.2] # Incomplete data, should be processed again using DumpFix
            SH600001/
                close.day.bin: [2.3] # Incomplete data, should be processed again using DumpFix
                open.day.bin: [2.2] # Incomplete data, should be processed again using DumpFix

@wangze586 Hi,
Only numeric fields (int/float) can be dumped, if they are non-numeric (e.g. symbol), you can use --exlcude_fields to specify the fields that do not need to be dumped, for example:
# SH600000.csv
      date       symbol  close open     name
      2021-06-01 SH600000  1.1   1       test
      2021-06-02 SH600000  1.2   1.1      test
      2021-06-03 SH600000  1.3   1.2      test

python dump_bin.py dump_all --csv_path xxx --qlib_dir xxx  --exlcude_fields datetime,symbol,name
or
python dump_bin.py dump_all --csv_path xxx --qlib_dir xxx  --include_fields close,open

here is mydata
#SZ000001.csv
date | open | high | low | close | pre_close | change | pct_chg | vol | amount
2021/7/24 | 17.5 | 18.55 | 17.2 | 18.29 | 17.45 | 0.84 | 4.81 | 82161 | 147325.4
2021/7/25 | 18.35 | 18.85 | 18 | 18.06 | 18.29 | -0.23 | -1.26 | 93993 | 173475.2
2021/7/26 | 18.02 | 19.05 | 17.75 | 18.78 | 18.06 | 0.72 | 3.99 | 120222 | 221192.5
I delete non-numeric data , but it still can't be updated

@wangze586
Add parameters --exclude date:
python dump_bin.py dump_update --csv_path C:/User/wangz/Desktop/run/data --qlib_dir C:/User/wangz/Desktop/run/output/1day --exclude date

..........same problem

@wangze586  Sorry, the parameter name is wrong ......
python dump_bin.py dump_update --csv_path C:/User/wangz/Desktop/run/data --qlib_dir C:/User/wangz/Desktop/run/output/1day --exclude_fields date

--exclude_fields date  I know it's wrong and i also change to correct one , problem still remain the same

@wangze586
After this line https://github.com/microsoft/qlib/blob/main/scripts/dump_bin.py#L218 add :
print(_df.dtypes)
print(self.get_dump_fields(_df.columns))
print(self._exclude_fields, self._include_fields)

Output the result of these three lines, and see the result

python dump_bin.py dump_update --csv_path C:/User/wangz/Desktop/run/data --qlib_dir C:/User/wangz/Desktop/run/output/1day --exclude_fields date,symbol

thank you !!! it works

@wangze586
Would you mind sending a PR to improve the docs(or docstrings) about this Question?
Other users who have similar confusion will be grateful to you :)
Thanks

@you-n-g
problem seems can be easily solved. just add  "--exclude_fields date,symbol"  , when use "dump_update"

well done!


TensorFlow docker image JupyterNotebook does not start

I built the docker image using Dockerfile present in tensorflow/tensorflow/tools/docker/Dockerfile, When I run the image it does gives me url for notebook

Copy/paste this URL into your browser when you connect for the first time,
    to login with a token:
        http://(e01f14b69e8e or <IP_ADDRESS>):8888/?token=da541XXXXXXXXXXXXXXXXXXXXXX

However opening the url fails with error
<IP_ADDRESS> didn’t send any data.
Following is the port-mapping output
debug1: client_input_global_request: rtype<EMAIL_ADDRESS>want_reply 0
debug1: Connection to port 8888 forwarding to localhost port 8888 requested.
debug1: channel 2: new [direct-tcpip]
debug1: Connection to port 8888 forwarding to localhost port 8888 requested.
debug1: channel 3: new [direct-tcpip]
channel 2: open failed: connect failed: Connection refused
channel 3: open failed: connect failed: Connection refused
debug1: channel 2: free: direct-tcpip: listening port 8888 for localhost port 8888, connect from <IP_ADDRESS> port 51331 to <IP_ADDRESS> port 8888, nchannels 4

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


LPDC	Launchpad short texts

this is probably somewhere in transaction LPD_CUST, but I cant remember
closing for now


[session] Expose struct loading

Motivation
This operation loads the struct type information.  It differs from load_type which, when loading a generic struct type, will also instantiate type parameters.  Having this separate entry into the loader allows us to implement type loading support in the presence of package upgrades from the adapter layer.
Test Plan
$ env SOLC=... cargo nextest --no-fail-fast

Yeah I did consider it, but ended up going for reimplementing it, because we do expose get_type_abilities on the session.


Doesn't properly encode nested records

Summary
Using nested records doesn't get encoded properly.
Steps to reproduce
Run jwt encode -e<PHONE_NUMBER> -n 0 -S "fake" -P admin=false, test:{stuff=things,s=t}
Actual Behavior
Token claims
------------
{
  "admin": "false,",
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>,
  "nbf": 0,
  "test:s": "t",
  "test:stuff": "things"
}

Expected behavior
Token claims
------------
{
  "admin": "false,",
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>,
  "nbf": 0,
  "test": {"s": "t", "stuff": "things"}
}

Hey @JasonMFry! Thanks for reporting this. It looks like, however, your syntax is a little off. The way that jwt-cli supports objects is via a JSON string. My shell is fish so I have to put quotes around it, but this is how I got your token:
jwt encode -e<PHONE_NUMBER> -n 0 -S "fake" -P admin=false -P 'test={ "stuff": "things", "s": "t" }'

// eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJhZG1pbiI6ZmFsc2UsImV4cCI6MTUzOTE3ODc1MywiaWF0IjoxNTM5MTgzMzcwLCJuYmYiOjAsInRlc3QiOnsicyI6InQiLCJzdHVmZiI6InRoaW5ncyJ9fQ.lOrVPkgN-O01YpATw3ry4cth-bCLpVUqMGGiBcKvsbU

Decoding it looks like so:
jwt decode eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJhZG1pbiI6ZmFsc2UsImV4cCI6MTUzOTE3ODc1MywiaWF0IjoxNTM5MTgzMzcwLCJuYmYiOjAsInRlc3QiOnsicyI6InQiLCJzdHVmZiI6InRoaW5ncyJ9fQ.lOrVPkgN-O01YpATw3ry4cth-bCLpVUqMGGiBcKvsbU

Token header
------------
{
  "typ": "JWT",
  "alg": "HS256"
}

Token claims
------------
{
  "admin": false,
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>,
  "nbf": 0,
  "test": {
    "s": "t",
    "stuff": "things"
  }
}

If you still have trouble, let me know. Thanks!

It took us over 30 minutes of debugging and found that it absolutely doesn't take JSON, the error output even says the payload must take key=value which is not JSON. If it does take JSON then the error output and documentation should be updated. As it is we have begun using a different tool because of the time wasted.

@JasonMFry Thanks! I've been working on this and would like to get your opinion on two solutions.
The first allows you to add a JSON string behind a -j flag like so:
jwt encode -S 'fake' -j '{"test":"hello"}'

The second would be added as an argument without a flag. Because of the library I'm using, however, it would require a -- before the argument:
jwt encode -S 'fake' -P admin=True -- '{"test":"hello"}'

Let me know what you think is best. Thanks!

You said that because of the library you're using, it would require --, but is there a way to provide a default in your code? I imagine the most common use case would be json, but forcing the user to add a flag for the most common use case would be inconvenient and potentially error prone.
I would prefer the following:

some random flag (let's say -x) uses object notation {key: "value"}
no flags at all means json

If that is for some reason impossible, I would prefer -j because it is more memorable to me than --.

@JasonMFry I actually realized there was a bug in my code, so a JSON string can be the last argument without any flag. You could also augment with the -P or --payload flag if needed. So, thinking about your example above, any of these would be valid
jwt encode -e<PHONE_NUMBER> -n 0 -S "fake" -P admin=false '{"test":{"stuff":"things","s": "t"}}'
jwt encode -e<PHONE_NUMBER> -n 0 -S "fake" '{"admin":false,"test":{"stuff":"things","s": "t"}}'
jwt encode -e<PHONE_NUMBER> -n 0 -S "fake" -P admin=false  -P 'test={"stuff":"things","s": "t"}'
jwt encode -e<PHONE_NUMBER> -n 0 -S "fake" '{"admin": false, "test": { "stuff": "things", "s": "t" } }'

The only caveats that remain are that the JSON string is valid JSON. If not, serde (rust deserialization package) won't be able to parse it. Hopefully that makes sense and sounds good to you. Fighting with CI at the moment, but I hope to publish it soon. Thanks for working with me on this!

Looks great! 👍

@JasonMFry If you're on windows or macOS, you can now upgrade to 2.2.0 if you still use this tool or are interested in it again. If you're on linux, I'm going to try and release that tonight. Thanks


fix(router) several fixes related to sub-matching priorities

This patch intents to replace the following PRs:

https://github.com/Kong/kong/pull/4152
https://github.com/Kong/kong/pull/4153

Summary
It provides more appropriate fixes to each of the above issues. There may be more tests to write, which is something I will give more thoughts to when I have some more time on my hands. There are also several things we could do with the router to increase tests coverage and maintainability, but I am running short of time to write down ideas at the moment.
Full changelog

fix: edge-case resulting in failure to match regex paths
fix: edge-case resulting in failure to match wildcard hosts
perf: refine hosts reducer by accounting for the request's Host header
tests: fix many potential false-positive and confused usage of luassert assertions

Notes

The nature of these changes introduce breaking changes in the router, hence the PR is based on next. Nonetheless, those changes do fix edge-cases causing the router to not behave as expected/documented ("expected" meaning here that the behavior broke once the scenario was slightly more complexed than already tested; nonetheless basic matching rules were not being respected).
As of today (July 3rd, 2019), our proxy guide documentation states that plain paths are evaluated before regex paths, and implies that the former have precedence over the latter, which should not be the case, as already asserted by existing test cases. Note as well the invalid link resulting in plain markdown inlined in the rendered document (cc @Kong/team-docs).

Notes to reviewers

Reviewing commits individually and in chronological order will help (some changes will be invisible in the final diff, but should be looked at.
We need to think about more test cases for sub-matching priorities edge-cases and catch potentially other broken use-cases fixed by this patch; and add regression tests for those. More can be done for testing and maintainability, but that is outside the scope of this PR.
Each commit has some background and a detailed description of the issue.

Issues resolved
Fix #3094
Replaces #4152
Replaces #4153

cc @Tieske @RobSerafini - FYI


Fix release action

ActionLint was reporting that the action I was using to create a GitHub release was deprecated. So, I went back to the drawing board.
First, I found a fork of the origin GitHub release action, and I slapped that in place. It has significant differences from the one I was using, namely that it didn't support uploading files. So I then followed this blog post and manually added the files with the gh CLI.
Of course, now I'm wondering if the blessed path is, in fact, to use the CLI to create the release.

Leroy Jenkins


Prep release

@LinguList this dataset has not etc folder (so no /languages.tsv or ortho-profiles). Should I work on this for the Lexibank 2.0 release?

Yes, it cannot be done, it is only orthography :-( So we do not include it in the next release!


Redesign the application

Hello folks,
I am open to new designs for the application, hit me up on hangouts<EMAIL_ADDRESS>for suggestions.
Currently, i have been not working on this project, but seeing the contributions i might get back to this project. If anyone is willing to contribute to this project, and recode the whole app according to a new design, go ahead !
Thank you

Would be cool if u could just open the application and it would show you all of the courses and then you could choose which course and then a list of the videos would show up.

@Irantwomiles You can go ahead and try doing it yourself. As i said, i haven't been working on this project right now, so do whatever you feel like, and make it a success for the open source community.

@theIYD ok, I will once I get the chance

It seems this app is no longer working after CH updated the website. Any suggestions how to fix it?

It seems this app is no longer working after CH updated the website. Any suggestions how to fix it?

As reported by other users, the app seems to be working fine with the updated version of CH website

As you can see in the image, the app does not show the list and does not start downloading.

@Irantwomiles @ycnmhd Since i have not been working on this, please look into this matter
Thank you

@shinokada could u send the link you used, I don't have access to my computer for the next week but I'll try to help.

@Irantwomiles I tried different websites but none of them worked.
e.g. https://coursehunters.net/course/socket-io-s-websockets-podrobno-socket-io-v2

@shinokada can you try the fork I have on my profile and see if that works for you.

@Irantwomiles Thanks. From your comment I realized what was wrong. I used dmg version. So I clone it and installed npm and run npm start, then voila, it works. Thanks.


[github] CODEOWNERS: Organize entries by directory

This scatters the areas a particular person owns or is watching across the file and in turn makes it harder to outline them. However, it also appears to be the safest and most maintainable approach to organizing the map. First, accidental ownership overrides are pretty much excluded. Second, the uncharted parts are a lot easier to pin down.

@swift-ci please smoke test

@swift-ci please smoke test macOS


Continuously loading loading while register.

It's only loading while registering a new account. Why it's happening? Could you please check it?

did you went over setup of Firebase?

Yeah I've already setup fire base in the console and add .plist file in the
project too.
On Wed, 22 Feb 2017 at 8:01 PM, Haik Aslanyan<EMAIL_ADDRESS>wrote:

did you went over setup of Firebase?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aslanyanhaik/Quick-Chat/issues/19#issuecomment-281684760,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYwUrugTEzmUQzi5aO5yRPSIbNu7SuT4ks5rfEavgaJpZM4MItiL
.

did you changed the permissions of email signin? App will stuck if email sign in option is disabled

Yeah I've enabled email signin option in the firebase console.
On Sat, 25 Feb 2017 at 8:08 PM, Haik Aslanyan<EMAIL_ADDRESS>wrote:

did you changed the permissions of email signin? App will stuck if email
sign in option is disabled
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aslanyanhaik/Quick-Chat/issues/19#issuecomment-282487923,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYwUribb_IrXfrorqUMQcaF5Re6FDVyiks5rgDz1gaJpZM4MItiL
.

@aslanyanhaik please share your firebase rule.

Please find the attached file
GoogleService-Info.plist.zip

Any updates on this?

firebase auth rule
{
"rules": {
".read": "auth != null",
".write": "auth != null"
}
}

Me too what is the reason?

Enable Database and Storage in Firebase. Then re-install, re-create an account and try.


"default_transport.cpp" fatal error: sys/time.h: No such file or directory

Hi guys,
I'm facing with error when compiling the default_transport.cpp file that there is no file or directory for sys/time.h.
I supposed that my sys directory are missing time.h file but I can't insert this file here. So you guys please help me to fix it.
Thank you guys alot.
Regards,
Rabbit.

I exported wrong path for arduino libraries.


v2 doesn't work with CRA's browserslist setup

Bug Report
Describe the bug
When using the v2 of Docz with CRA v3's browserslist.
CRA's browserslist setup is for development and production.
{
	"browserslist": {
    	"production": [
      		">0.2%",
      		"not dead",
      		"not ie <= 11",
      		"not op_mini all"
    	],
    	"development": [
      		"last 1 chrome version",
      		"last 1 firefox version",
      		"last 1 safari version"
    	]
  	}
}

But Docz v2 is expecting something like:
{
	"browserslist": [
      	"last 1 chrome version",
      	"last 1 firefox version",
      	"last 1 safari version"
    ]
}

Expected behavior
I expect to be able to use Docz v2 with my current CRA browserslist setup

I have same error. What's the work around?

same +1

Hey,
This issue was fixed in version 2.0.0-rc.33.
Any version after that should work as you'd expect it.
Feel free to re-open if it happens again


[Docs]: Added reference to null-loader

Follow up of #25

Only for webpack@5, also we need provide an example how you achieve this using the configuration

@evilebottnawi You meant this one https://github.com/webpack/changelog-v5/pull/25


build-marist-ubuntu1604-s390x-2 offline : connection problem

https://ci.adoptopenjdk.net/computer/build-marist-ubuntu1604-s390x-2/log

Cleared out some directories. Theres 47g free on the machine now

Machine no longer exists - we do not build on Ubuntu. The jenkins definitions appear to be lagging behind reality which I have resolved via the above PR


Add timeout to writing to responses (#15831)

Backport #15831
In #15826 it has become apparent that there are a few occasions when a response can
hang during writing, and because there is no timeout go will happily just block
interminably. This PR adds a fixed 5 second timeout to all writes to a connection.
Fix #15826
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

make lgtm work


Allow users to customize the CA of impersonation proxy to use with ingress

Is your feature request related to a problem? Please describe.
Currently, we cannot use the impersonation proxy with an ingress controller and a custom DNS name. Here is my use case:
I have an Ingress with the host impersonation-proxy.example.com, and a wildcard certificate for *.example.com signed by a Certificate Authority. The Ingress is pointing to the service pinniped-concierge-proxy. Then here is my CredentialIssuer.
apiVersion: config.concierge.pinniped.dev/v1alpha1
kind: CredentialIssuer
metadata:
    name: using-custom-certificates
spec:
  impersonationProxy:
    # Enable the proxy
    mode: enabled
    # Provision a service
    service:
      type: ClusterIP
    # Advertise an external DNS name that points at a custom ingress
    externalEndpoint: impersonation-proxy.example.com

With this configuration, pinniped get kubeconfig complains that the certificate is signed by an unknown authority. I believe this is because the TLS endpoint in the concierge pod on port 8444 is served with an internal generated certificate.
Describe the solution you'd like
So I need something like:
apiVersion: config.concierge.pinniped.dev/v1alpha1
kind: CredentialIssuer
metadata:
    name: using-custom-certificates
spec:
  impersonationProxy:
    # Enable the proxy
    mode: enabled
    # Provision a service
    service:
      type: ClusterIP
    # Advertise an external DNS name that points at a custom ingress
    externalEndpoint: impersonation-proxy.example.com
    tls:
      # Point to some certificate/key to serve with, this would be my wildcard certificate for `*.example.com`.
      secretName: my-tls-cert

Or we could allow serving concierge in a non-TLS port, because the internal traffic between the ingress controller and the concierge could be non-TLS, with something like:
apiVersion: config.concierge.pinniped.dev/v1alpha1
kind: CredentialIssuer
metadata:
    name: using-custom-certificates
spec:
  impersonationProxy:
    # Enable the proxy
    mode: enabled
    # Provision a service
    service:
      type: ClusterIP
    tls:
      # Disable TLS
      enabled: false

This is already discussed in https://github.com/vmware-tanzu/pinniped/issues/617 (last use case) and https://github.com/vmware-tanzu/pinniped/issues/363#issuecomment-905710186.

Hi @antoineozenne, Thank you for submitting this issue. The maintainer team has other feature work that we're prioritizing right now, so we're not likely to pick this up soon. We'll let you know if that changes, but in the meantime if you're interested in making a PR, we'd be happy to review it.

In case anyone decides to pick this up:
The impersonation proxy uses client certificates for authentication, thus any form TLS termination is incompatible with it (i.e. if using Kube ingress, it must support TLS passthrough).  This of course disallows any non-TLS based approach.
We would need two new config knobs in the credential issuer: a required (when ImpersonationProxyTLS is not nil) secret with a serving certificate and an optional CA bundle to use for TLS verification - this will later get embedded in the generated kubeconfig.  We may want to perform validations to help the admin know if they have misconfigured the serving certificate (i.e. invalid SAN).
diff --git a/apis/concierge/config/v1alpha1/types_credentialissuer.go.tmpl b/apis/concierge/config/v1alpha1/types_credentialissuer.go.tmpl
index d1cb160b..51a4692f 100644
--- a/apis/concierge/config/v1alpha1/types_credentialissuer.go.tmpl
+++ b/apis/concierge/config/v1alpha1/types_credentialissuer.go.tmpl
@@ -5,6 +5,8 @@ package v1alpha1

 import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+
+	"go.pinniped.dev/GENERATED_PKG/apis/concierge/authentication/v1alpha1"
 )

 // StrategyType enumerates a type of "strategy" used to implement credential access on a cluster.
@@ -100,6 +102,14 @@ type ImpersonationProxySpec struct {
 	//
 	// +optional
 	ExternalEndpoint string `json:"externalEndpoint,omitempty"`
+
+	TLS *ImpersonationProxyTLS `json:"tls,omitempty"`
+}
+
+type ImpersonationProxyTLS struct {
+	v1alpha1.TLSSpec `json:",inline"`
+
+	SecretName string `json:"secretName"`
 }

 // ImpersonationProxyServiceSpec describes how the Concierge should provision a Service to expose the impersonation proxy.

Are you saying that the solution of exposing the proxy on a non-TLS port is not an option? It may be simpler, even if it is not the initial request.

Are you saying that the solution of exposing the proxy on a non-TLS port is not an option? It may be simpler, even if it is not the initial request.

Correct, this is not an option as client certificates are used for user authentication, i.e. mTLS.
The mTLS requirement aside, all existing non-TLS configuration for the supervisor is being deprecated and removed, i.e. #1094.  I do not see us adding such configuration for the concierge.


MSUnmerged failing to connect to RucioConMon due to a change in the endpoint URL

Impact of the bug
MSUnmereged
Describe the bug
We found  out that  MSUnmerged is failing to fetch RSE statistics from Rucio Consitency Montior [1]. Our service is trying to contact the following URl:
https://cmsweb.cern.ch/rucioconmon/WM/stats
While this one works:
https://cmsweb.cern.ch/rucioconmon/unmerged/stats
Same for the file lists per RSE API:
https://cmsweb.cern.ch/rucioconmon/WM/files?rse=T2_DE_DESY&format=json
vs.
https://cmsweb.cern.ch/rucioconmon/unmerged/files?rse=T2_DE_DESY&format=json
Obviously, at some moment, the endpoint prefix has changed from:

from: https://cmsweb.cern.ch/rucioconmon/unmerged/stats
to:  https://cmsweb.cern.ch/rucioconmon/WM
and the change has not been reflected in our code/configuration.

How to reproduce it
Just try to contact the old Url
Expected behavior
The Url for a service in production not to change without a notice.
Additional context and error message
[1]
2022-12-09 01:59:19,359:ERROR:MSUnmerged: Unknown exception while trying to fetch statistics from Rucio Consistency Monitor. Error: url=https://cmsweb.cern.ch:8443/rucioconmon//WM/stats, code=404, reason=Not Found, headers={'Date': 'Fri, 09 Dec 2022 00:59:19 GMT', 'Server': 'Apache', 'Content-Type': 'application/json', 'Content-Length': '135', 'X-WebPie-Request-Id': 'UHk.331', 'CMS-Server-Time': 'D=18447 t=1670547559340442'}, result=b'{"message": "The resource could not be found.

\\nInvalid path //WM/stats\\n\\n", "code": "404 Not Found", "title": "Not Found"}'
Traceback (most recent call last):
  File "/data/srv/HG2212d/sw/slc7_amd64_gcc630/cms/reqmgr2ms/1.1.5rc7/lib/python3.8/site-packages/WMCore/MicroService/MSUnmerged/MSUnmerged.py", line 216, in execute
    self.rseConsStats = self.rucioConMon.getRSEStats()
  File "/data/srv/HG2212d/sw/slc7_amd64_gcc630/cms/reqmgr2ms/1.1.5rc7/lib/python3.8/site-packages/WMCore/Services/RucioConMon/RucioConMon.py", line 104, in getRSEStats
    rseStats = self._getResult(uri, callname='stats')
  File "/data/srv/HG2212d/sw/slc7_amd64_gcc630/cms/reqmgr2ms/1.1.5rc7/lib/python3.8/site-packages/WMCore/Services/RucioConMon/RucioConMon.py", line 76, in _getResult
    with self.refreshCache(cachedApi, apiUrl) as istream:
  File "/data/srv/HG2212d/sw/slc7_amd64_gcc630/cms/reqmgr2ms/1.1.5rc7/lib/python3.8/site-packages/WMCore/Services/Service.py", line 218, in refreshCache
    self.getData(cachefile, url, inputdata, incoming_headers, encoder, decoder, verb, contentType, binary=binary)
  File "/data/srv/HG2212d/sw/slc7_amd64_gcc630/cms/reqmgr2ms/1.1.5rc7/lib/python3.8/site-packages/WMCore/Services/Service.py", line 362, in getData
    raise he
  File "/data/srv/HG2212d/sw/slc7_amd64_gcc630/cms/reqmgr2ms/1.1.5rc7/lib/python3.8/site-packages/WMCore/Services/Service.py", line 318, in getData
    data, dummyStatus, dummyReason, from_cache = self["requests"].makeRequest(uri=url,
  File "/data/srv/HG2212d/sw/slc7_amd64_gcc630/cms/reqmgr2ms/1.1.5rc7/lib/python3.8/site-packages/WMCore/Services/Requests.py", line 173, in makeRequest
    result, response = self.makeRequest_pycurl(uri, data, verb, headers)
  File "/data/srv/HG2212d/sw/slc7_amd64_gcc630/cms/reqmgr2ms/1.1.5rc7/lib/python3.8/site-packages/WMCore/Services/Requests.py", line 190, in makeRequest_pycurl
    response, result = self.reqmgr.request(uri, data, headers, verb=verb,
  File "/data/srv/HG2212d/sw/slc7_amd64_gcc630/cms/reqmgr2ms/1.1.5rc7/lib/python3.8/site-packages/Utils/PortForward.py", line 67, in portMangle
    return callFunc(callObj, newUrl, *args, **kwargs)
  File "/data/srv/HG2212d/sw/slc7_amd64_gcc630/cms/reqmgr2ms/1.1.5rc7/lib/python3.8/site-packages/WMCore/Services/pycurl_manager.py", line 341, in request
    raise exc
http.client.HTTPException: url=https://cmsweb.cern.ch:8443/rucioconmon//WM/stats, code=404, reason=Not Found, headers={'Date': 'Fri, 09 Dec 2022 00:59:19 GMT', 'Server': 'Apache', 'Content-Type': 'application/json', 'Content-Length': '135', 'X-WebPie-Request-Id': 'UHk.331', 'CMS-Server-Time': 'D=18447 t=1670547559340442'}, result=b'{"message": "The resource could not be found.

\\nInvalid path //WM/stats\\n\\n", "code": "404 Not Found", "title": "Not Found"}'

As discussed over Slack, we need to:

find out whether the uri API really changed?
if so, when was that? and where was that announced?
if there is any functionality/format change to the request and response object

Lastly, it might give us a clue that we should have an alert for these basic initial data that we collect before proceeding with the removal logic. Especially because the interval of MSUnmerged is pretty large (8h?). I think this can be done as an extra ticket for the future though.

That's a good idea @amaltaro
I'd say we may think of creating a general mechanism to upload all types of exceptions from our logs to monit in addition to the alerts we create.


Version 0.3

The last version tag was 0.2.56. We now begin version 0.3 with goals described in Jan 25 talk at the DESC CO meeting. The major goal is to enable users to start jobs on NERSC machines (including perlmutter batch and login nodes) by running an agent which has their NERSC credentials.

Our first use case is to specify a job in the SPIN UI (descprod.com) and have the user start an agent on a perlmutter login node. For this (and most other use cases), the agent will fetch the data for a job, start the job and send back results and updates. The JobData class has been modified to put this persistent data into a map for each job object. This is  tag 0.1.

We want the job data in a persistent DB. I added mysql-server to dockerfile-05 and built, pushed. I am able to start the service in SPIN descprod with

service mysql start

by hand for now.
The datadir for mysql is /var/lib/mysql which is not persistent (across server restarts). In the SPIN DESCprod service, I created a new volume dpvol03 with a directory mysql and copied the files from /var/lib/mysql there. I was not able to remove the latter and so I added a mount of dpvol/mysql over /var/lib/mysql.  After the server restart, I was able to create a DB/table descprod/jobdata. After another restart I verified that the DB and table still existed.

I added a start command mysql to start the mysql server in version 0.3.2. I also added this to the SPIN DESCprod service startup and verified that the service now starts with mysql running.

Trying to connect to the DB in python, I had this error:

>>> import mysql.connector
>>> con = mysql.connector.connect(database='descprod')
Traceback (most recent call last):
  File "/home/descprod/conda/lib/python3.10/site-packages/mysql/connector/connection_cext.py", line 288, in _open_connection
    self._cmysql.connect(**cnx_kwargs)
_mysql_connector.MySQLInterfaceError: Host '<IP_ADDRESS>' is not allowed to connect to this MySQL server

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "", line 1, in
  File "/home/descprod/conda/lib/python3.10/site-packages/mysql/connector/pooling.py", line 293, in connect
    return CMySQLConnection(*args, **kwargs)
  File "/home/descprod/conda/lib/python3.10/site-packages/mysql/connector/connection_cext.py", line 118, in __init__
    self.connect(**kwargs)
  File "/home/descprod/conda/lib/python3.10/site-packages/mysql/connector/abstracts.py", line 1178, in connect
    self._open_connection()
  File "/home/descprod/conda/lib/python3.10/site-packages/mysql/connector/connection_cext.py", line 293, in _open_connection
    raise get_mysql_exception(
mysql.connector.errors.DatabaseError: 1130 (HY000): Host '<IP_ADDRESS>' is not allowed to connect to this MySQL server

I resolved it with:

mysql> select User,host from mysql.user;
+------------------+-----------+
| User             | host      |
+------------------+-----------+
| debian-sys-maint | localhost |
| mysql.infoschema | localhost |
| mysql.session    | localhost |
| mysql.sys        | localhost |
| root             | localhost |
+------------------+-----------+
5 rows in set (0.00 sec)

mysql> create user<EMAIL_ADDRESS>Query OK, 0 rows affected (0.01 sec)

mysql> GRANT ALL ON *.* to<EMAIL_ADDRESS>Query OK, 0 rows affected (0.12 sec)

mysql> flush privileges;
Query OK, 0 rows affected (0.01 sec)

mysql> select User,host from mysql.user;
+------------------+-----------+
| User             | host      |
+------------------+-----------+
| root             | <IP_ADDRESS> |
| debian-sys-maint | localhost |
| mysql.infoschema | localhost |
| mysql.session    | localhost |
| mysql.sys        | localhost |
| root             | localhost |
+------------------+-----------+
6 rows in set (0.00 sec)

On my laptop, I instead added user descprod:

mysql> create user<EMAIL_ADDRESS>Query OK, 0 rows affected (0.08 sec)

mysql> GRANT ALL ON *.* to<EMAIL_ADDRESS>Query OK, 0 rows affected (0.04 sec)

mysql> flush privileges;
Query OK, 0 rows affected (0.02 sec)

mysql> select User,host from mysql.user;
+------------------+-----------+
| User             | host      |
+------------------+-----------+
| descprod         | <IP_ADDRESS> |
| debian-sys-maint | localhost |
| mysql.infoschema | localhost |
| mysql.session    | localhost |
| mysql.sys        | localhost |
| root             | localhost |
+------------------+-----------+
6 rows in set (0.02 sec)

Methods are added to check and optionally create a DB and table in that DB with default names "descprod" and "jobdata". The table schema includes that current identified for jobs with the index a primary key. there is also an option to drop the table. All in version 0.3.3.

After earlier dropping the jobdata table on SPIN, I create it with the new schema:

root@descprod-66cccfcd5d-z2dn7:~# cd dev/desc-prod
root@descprod-66cccfcd5d-z2dn7:~/dev/desc-prod# cd server
root@descprod-66cccfcd5d-z2dn7:~/dev/desc-prod/server# python
Python 3.10.8 (main, Nov 24 2022, 14:13:03) [GCC 11.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from descprod import JobData
>>> JobData.have_table()
False
>>> JobData.have_table(create_table=True)
JobData.have_table: Creating table jobdata
True
>>>

Class and object data and methods are added to JobData to enable a job to insert its data into a DB, track fields that have changes since a DB update and to set update values in the DB. All changes are in 0.3.4.

The wrapper is enhanced to add a heartbeat, i.e. poll the process at regular intervals (now 10 sec) and write the status to the wrapper json file. The job class reads this file when its return status, stop time or progress are read. The progress string is now a persistent data member and included in the DB schems.

I updated the service on SPIN.
Removing an old job, an exception was raised:

  File "/home/descprod/dev/desc-prod/server/descprod/jobs.py", line 734, in delete

    shutil.rmtree(rundir)

  File "/home/descprod/conda/lib/python3.10/shutil.py", line 724, in rmtree

    _rmtree_safe_fd(fd, path, onerror)

  File "/home/descprod/conda/lib/python3.10/shutil.py", line 657, in _rmtree_safe_fd

    _rmtree_safe_fd(dirfd, fullname, onerror)

  File "/home/descprod/conda/lib/python3.10/shutil.py", line 680, in _rmtree_safe_fd

    onerror(os.unlink, fullname, sys.exc_info())

  File "/home/descprod/conda/lib/python3.10/shutil.py", line 678, in _rmtree_safe_fd

    os.unlink(entry.name, dir_fd=topfd)

FileNotFoundError: [Errno 2] No such file or directory: 'taskerr0010.log'

in jobs.py line 734 from rmtree:

        if os.path.exists(rundir):
            shutil.rmtree(rundir)

Removing other jobs, I saw other unexpected errors including "Archive command failed".  I suspect the filesystem is not fully updated from the opertaion on one line, when the next python line is executed.
Th try to fix this, I added a loop check with delays (wait_for_path) after creating the archive and after deleting the run directory. Changes are in 0.3.6.

I could not open mysql on my laptop where default account is descprod. I fixed this by add user descprod:

mysql> create user descprod@localhost
    -> ;
Query OK, 0 rows affected (0.09 sec)

mysql> select User,host from mysql.user;
+------------------+-----------+
| User             | host      |
+------------------+-----------+
| debian-sys-maint | localhost |
| descprod         | localhost |
| mysql.infoschema | localhost |
| mysql.session    | localhost |
| mysql.sys        | localhost |
| root             | localhost |
+------------------+-----------+
6 rows in set (0.01 sec)

Jobs now update DB on creation, submission and running. Local progress is not working and remote progress has yet to be added. I added stups to fetch and update jobs from the server and verified I can access these with messages from requests.

The displayed job table is now read from the DB. Version 0.3.8

I had problems getting the new version running at NERSC because DB schema had changed. I finally used JobData.have_table to drop and then recreate DB. Now all appears fine there. Version is 0.3.11.

I fixed witing of job data to the DB to include wrapper info. And I added JobData.jmap() which returns a json map of the data for a job. Version is 0.3.12. Still need to test this at nersc.

Version 0.3.14 adds seconds to the time displayed in home page header and start time columns. Useful for meauring the time to refresh the table which is now (since the mysql switch) many seconds on my laptop an about a second at nersc. Something to keep an eye on.

For version, 0.3.15, the python support code in server/descprod is moved to ./descprod and setuptools interface added so we pip install the client parts of desc-prod. The package has to be installed and the client code should be useable outside the server env. A new image, version 06, is required to run the server.

For version 0.3.16, the server command get_job is fixed to return the job json map.
Also, command descprod-get-job is added to display that map.

For 0.3.19:

Introduce job.usedb which can be se false to have a JobData object disconnected from the DB. This is intended for use in client scripts.
Revamp JobData to allow remote running. Add host and run directory to job data. Run directory is specified in run method instead of ctor and the json config file is created there.

For 3.20, continue work on start_jobs:

Fix JobData.jmap_update to always return string.
Add JobData.ready_to_run() that returns blank if ready.
Drop command from DB data and change JobData.command() to evaluate it from jobtype and config.
Add option to start a job (iff it is ready to run) in the job table menu.
Add recording and display of session ID for each job.

For 3.21, toward using ptenv to set up for running parsltest:

JobData.run sets up for server env iff rundir is missing
Add pip install of ptenv utilities
Add create of ptenv-home-00 to startup if missing but
Add create of homeptenv00 to image version 7

For 3.22:

Eliminate need to run ptenv-create twice when miniconda is absent.
Add miniconda flavor for MacOSX so ptenv works on mac.
Set ptenv python version to 3.11.0. Older version had conflicts with that one on my laptop.
But finally give on on mac: workqueue and sysmon do not work there.

For 0.3.23:

Modify jobs.py to use command runapp-parsltest to run parsltest.
Fix getlogin in get_job.
Stop running jobs with sudo so we can use pre-installed ptenv.

For 0.3.24 fis remote running and start implementing wrapper-server job communication:

Drop setups of conda and parsl when starting jobs in runs.py. These only work on server.
Drop bash -login option when starting jobs in jobs.py so we have the descprod commands.
Modify descprod-wrapper to send update messages back to the server. Still need to ad check of reponse.

For 0.3.26, move toward sending job updates from remote to server.

Implement update_job in app.py
Add server url in start_job call to JobData.run
Update start_job to pass server url to descprod-wrap
Update descprod-wrap to send back messages to the received server if it is present.
Update JobData.get_job and server get_job and and update_job calls to take job from DB if not already in cache.
Modify JobData ctor to initialize update_time.
Modify server to allow missing update time.
Add host and update_time updates to wrapper.
Fix rundir (replace run_directory) update in wrapper.
Fir rc overwrite in wrapper

For 0.3.27, get remote running working at NERSC :

Deploy nerscprod server with version 0.3.26.
Fix handling of unspecified server (default descprod) in JobData.run.
Fix default server in wrapper.

I am now able to run nerscprod jobs on perlmutter login nodes.
I also ran a nerscprod job on my laptop (in linux window from descprod).

Next want to work on batch submission. This is discussed in #3 .


Desktop Verison

Hi,
Loved your project and would like to use it for another. Do you have a desktop version or something that can be run on a server for scalability? Preferably python, if not a problem. Would really appreciate it.

We don't have the code on other platforms than Android. I would probably try this https://github.com/davidsandberg/facenet


Changing back custom modifications to mod names should be avoided

If I change the name of mod (such as to remove platforms), then my custom name should persist when Packwiz upgrades the mod.

Duplicate of https://github.com/packwiz/packwiz/issues/52

Oops, I misread the title...


ETag header in mocked response missing

This is my uploaded configuration:
{     "httpRequest": {         "method": "GET",         "path": "/getETag",         "body": "",         "headers": [],         "cookies": []     },     "httpResponse": {         "headers":             [ 				{ "name": "ETag", "values": ["notInResponse"]}, 				{ "name": "ETagg", "values": ["works"]}, 				{ "name": "Content-Type", "values": ["text/html"]} 			]         ,         "statusCode": 200,         "body": "Some body",         "cookies": [],         "delay": {             "timeUnit": "MICROSECONDS",             "value": 0         }     },     "times": {         "remainingTimes": 1,         "unlimited": true     } }
and these are the response headers:
HTTP/1.1 200 OK
Date: Fri, 06 Oct 2017 15:22:45
Server: Apache-Coyote
ETagg: works
Content-Type: text/html
Content-Length: 22
Vary: Accept-Encoding
Keep-Alive: timeout=5, max=100
Connection: Keep-Alive
The ETag Header is missing in the response. The war file (v3.11) is running on a Apache Tomcat/8.0.41

Issue can be closed, there was a configuration problem in apache's httpd.conf:

Header unset ETag

FileETag None
Sorry for any inconvenience


makes sure animated example works correctly with the default App.tsx

Description
the latest react-spring does not have the Transition component API and also Transition duplicates the html it is applied on, and this does create weird jumping of the content unless position: absolute is applied
Motivation and Context
Make the example work correctly with the default html structure

This guide may be outdated for v7. Could you please test with v7 and make sure?

I did. Works well with version 7.

Actually, after a closer look, this should be working since the Routes child renderer has been deprecated.
I'm not sure on the exact example yet, but we'll need to update it to use the new Routes component together with the useRoutePath. It's a bit different, so I'll work on an example to show you.


Deep Learning

what is this issue?

@wz7710
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you.

Closing this issue due to lack of recent activity .Please feel free to reopen if you still have concern.Thanks!


removed org from contact vcard str

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

Project coverage is 83.10%. Comparing base (6770bea) to head (eb18d72).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   83.10%   83.10%
=======================================
  Files           8        8
  Lines         148      148
=======================================
  Hits          123      123
  Misses         25       25

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


Implement case where perpendicular distance isn't on short side of great circle

While the distance returned will always be the perpendicular distance from the location of the ionosonde to the great circle containing the path betwen the two QSO ham radio stations, it might be the distance from the ionosonde to the longer portion of the great circle (i.e. the portion of the great circle that the radio waves did not propogate along. For example, the Alaska ionosonde (Gakona) shown below

taken from the maximum usable frequency (MUF) map will not have a perpendicular path to the QSO path used in the original example from San Fancisco to the Netherlands shown below

Instead, the shorter distance to the path will be to the endpoint in San Francisco.
Modify the code to test which path is shorter, and return the shortest path distance.

Unit testing revealed my assumption about Gakona was incorrect. There is a perpendicular path to the QSO. Let's try Pt. Arguello instead.

That worked, even though I didn't think it did at first
FAILED tests/test_haversine_point_path.py::test_hav_not_perp[float-mi-1099.44] - assert 1099.44 ==<PHONE_NUMBER>9164974 ± 1.9e+00

Just to be completely sure, let's also test an ionosonde location past the rx side of the QSO, JULIUSRUH.
But that won't work because

so.....
How about Sopron

Yup!
FAILED tests/test_haversine_point_path.py::test_hav_not_perp[float-mi-228.7148841] - assert<PHONE_NUMBER> ==<PHONE_NUMBER>165612 ± 5.9e+00


My Account Grid displays blank after the bagisto 0.1.2 installation.

Framework Version: 0.1.2
https://prnt.sc/lwcmt2

This issue had been fixed and will you will get to see it in next release build.
Till then you can use the zip file for using bagisto.

Framework Version: 0.1.4
Fixed


DRAFT: feat(deps): Updated dependencies and added GitHub Actions

Summary
This PR contains the following updates:

Updated the package name so that it can be published as an org
Updated the minimum Node version to enable upgrading dependencies
Updated dependencies - new ones added as plugins have been separated into their own packages
Updated code to use latest version of imagemin (API change - expected with major version increase)

Description

Evidence

Any caveats, potential risks or outstanding actions?

There are still active CVEs in this change, so this should not be merged yet.


Attach col_stds and col_means to fitted GeneralizedLinearRegressor

These are computed anyways and nice to have for downstream analysis.

Can you elaborate a bit on the downstream analysis use case? I'm not against adding these new attributes, but it is also a commitment to keep them there in the future.

I mostly want to recover alpha * l1_ratio * np.abs(glm.coef_ * glm.col_stds_) and alpha * (1 - l1_ratio) * np.square(glm.coef_ * glm.col_stds_). I don't care about the means.


[charts - ecm] release update charts

Scope
Implement update options for the charts release process into ecm-distro-tools.
Parent issue:
Architect import charts into ecm-distro-tools
Command from charts-build-scripts that will be created and used by ecm-distro-tools:

release

Scope 2

The new release strategy is dependent on an old bash script called forward-port.
This script makes calls to charts-build-scripts and performs other operations.
Implement this script inside the release command in Golang, and deprecate the script.

Expected result
Upon execution of:
release update charts <branch> <chart(optional>

The following actions will be executed:

Fetch upstream repository
checkout release branch
create new branch from release branch
import target chart from dev branch
git add .
git commit

This must be done separately for each asset version.
i.e:
Suppose we have 2 versions for rancher-monitoring chart (v1 and v2).
The result must be 2 separate commits, one for each version.

Done


nixos/pam: Include the "cue" option to the pam_u2f.so invocation.

Motivation for this change
This makes it clear to the user that they are expected to touch the button on their U2F token to do the action that they're requesting.
I'm having some trouble finding resources for how to test this, so as of yet it's untested, however since it's just a string change I'm not sure there would be any issues.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

I've configured sudo to verify my action with my u2f token, however occasionally  I'll forget that I did that, or hit the token too soon before it's being requested, and thus the sudo command just hangs there for a while, eventually just timing out.
By default the pam_u2f.so module doesn't output any message, or show any prompt at all. The cue argument passed to it makes it show a prompt in the console, or in the gdm / polkit authorisiation dialogs (though neither of those cases actually work atm due to #21860 seeming to have fallen through the cracks)

Thanks for explaining.

I'm having some trouble finding resources for how to test this, so as of yet it's untested,

Why can't you test this with your u2f token? At least a console message should work.

I just now managed to figyure out how to replace the pam.nix with my fork's version (disabledModules + import) and here's the before and after -- in the "before" i tapped the u2f token after a couple seconds, then my echo "foo" went through, on the second you can see the prompt added by cue

I have extended the builtin u2f module with an option to make such customization possible. In my oppinion it would be much more flexible to implement configuration options for passing arbitrary arguments to PAM modules.


Exit Status: 74 when collecting logs on CircleCi, Xcode 8.3, on Fastlane 2.27.0

Issue Description
Since upgrading to Fastlane 2.27.0 we have seen failures in our CircleCi runs.
Using Fastlane 2.26.1 does not produce this issue.
Looking at the commits between the two, this one seems the likely culprit.
This issue also seems related to the discussion in https://github.com/fastlane/fastlane/pull/8829, specifically the comment made by mccarron.
+--------------------+------+
|       Test Results        |
+--------------------+------+
| Number of tests    | 5421 |
| Number of failures | 0    |
+--------------------+------+

[14:58:42]: ------------------------------
[14:58:42]: --- Collecting system logs ---
[14:58:42]: ------------------------------
[14:58:42]: $ xcrun simctl spawn ID_NUMBER log collect --output /Users/distiller/tracker-ios/fastlane/test_output/system_logs-iPhone\ 7_iOS_10.1.logarchive 2>/dev/null

[14:58:42]: Exit status: 74

Did you maybe upgrade to Xcode 8.3 and that caused the error?

Locally, where log collection works we are running XCode 8.3 (is 8.3 unsupported at this time?)
On CircleCI, where the build is failing, we are using Xcode 8.2.1 (build 8C1002), per their documentation.

I'm seeing this same failure. Failing on Circle, not on local builds. Upgraded both Xcode and Fastlane, but I can try to isolate one or the other if you think it will help.

Our failure output is very slightly different:
+--------------------+----+
|      Test Results       |
+--------------------+----+
| Number of tests    | 49 |
| Number of failures | 0  |
+--------------------+----+

[10:24:30]: ------------------------------
[10:24:30]: --- Collecting system logs ---
[10:24:30]: ------------------------------
[10:24:30]: $ xcrun simctl spawn 770CF0F5-5F6B-4E05-8CF0-E5A2BEFF0702 log collect --output /tmp/circle-junit.glEjmqS/junit/system_logs-iPhone\ 6s_iOS_10.0.logarchive 2>/dev/null

[10:24:30]: Exit status: 74

@lalunamel That code path should never actually be reached on 8.2.x, but CircleCI does support 8.3, the 8.3 image details can be found here https://180-74671181-gh.circle-artifacts.com/0/Users/distiller/macos-image-tests/pages/_site/index.html (sorry for the bad URI, since we moved to single-xcode-per-image, we didn't update the deployment of image docs yet)

We've been seeing that issue as well and temporarily disabling simulator logs solved the issue for us.
If you're ok with not having simulator logs for awhile, try adding the include_simulator_logs: false param in your scan() method.

@seanooi nice workaround! I looked at this a bit with @DanToml via ssh on a build, and it looks like the xcrun simctl spawn command is failing silently (due to the 2>/dev/null) if the path given to it doesn't exist. I'm not sure if this is a Circle or a Fastlane issue, but either way, now the right person knows about it 😀.

Setting

include_simulator_logs: false
worked for us too, but it would be nice to get that back when looking for back traces upon a crash to determine code or simulator.

Weirdly, removing include_simulator_logs from our Fastfile did not fix it for us.

This still seems to be an issue (Fastlane 2.33.0) and as a workaround I've had to ensure I create the log directory first before it is used. E.g. in my Jenkinsfile I have:
sh "mkdir -p ${env.WORKSPACE}/build/reports"
before attempting to run the tests.

Oops, I meant to follow up on this. The default value of include_simulator_logs changed to true some time after Circle 2.0 launched. You need to explicitly pass false in your Fastfile.

@lalunamel @ZevEisenberg @seanooi @jjacquet @hammertoe @DanToml @KrauseFx
Thank you for reporting and supporting. 👍
@lyndsey-ferguson
Thanks for the fix. I've also confirmed this is fixed in my project 🚀

👏 Glad to help!


URL not detected as vulnerable

This is the service and sample URL provided by trendmicro
Service: https://log4j-tester.trendmicro.com/
URL: http://ec2-44-199-245-240.compute-1.amazonaws.com:8080
The service says that the said URL is vulnerable.  But log4j-scan (on the sample URL) states that target is NOT vulnerable.
Where could be the gap?

Hi @gv2870,
This should be because of the HTTP request headers being used at log4j-scan are relatively extensive. Trend Micro is sending basic (2-4 insertion points per the entire HTTP request). LBs on odd cases may throw 4XX errors on abnormal number of http request headers (or its values). This seems to be the case here.
I pushed a PR to use basic headers when needed:
https://github.com/fullhunt/log4j-scan/pull/118

Closing the ticket for cleaning-up, the PR should be merged soon.
Thanks @gv2870 for bringing this!


mat1 and mat2 shapes cannot be multiplied (16x1024 and 4096x1152)

Hello,
Thanks for bringing pixart to comfyui!
I am trying to use the default workflow and I am getting a  mat1 and mat2 shapes cannot be multiplied (16x1024 and 4096x1152)
I am using PixArt-XL-2-1024x1024.pth
Here's the full trace:
got prompt
model_type EPS
adm 0
Warning: lewei scale: (2,), base size: 64
Loading T5 from 'D:\stable\comfyUI\ComfyUI\models\t5\ZET5'
Requested to load BaseModel
Loading 1 new model
0%|                                                                                | 0/20 [00:00<?, ?it/s]
ERROR:root:!!! Exception during processing !!!
ERROR:root:Traceback (most recent call last):
File "D:\stable\comfyUI\ComfyUI\execution.py", line 153, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "D:\stable\comfyUI\ComfyUI\execution.py", line 83, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "D:\stable\comfyUI\ComfyUI\execution.py", line 76, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "D:\stable\comfyUI\ComfyUI\nodes.py", line 1299, in sample
return common_ksampler(model, seed, steps, cfg, sampler_name, scheduler, positive, negative, latent_image, denoise=denoise)
File "D:\stable\comfyUI\ComfyUI\nodes.py", line 1269, in common_ksampler
samples = comfy.sample.sample(model, noise, steps, cfg, sampler_name, scheduler, positive, negative, latent_image,
File "D:\stable\comfyUI\ComfyUI\comfy\sample.py", line 100, in sample
samples = sampler.sample(noise, positive_copy, negative_copy, cfg=cfg, latent_image=latent_image, start_step=start_step, last_step=last_step, force_full_denoise=force_full_denoise, denoise_mask=noise_mask, sigmas=sigmas, callback=callback, disable_pbar=disable_pbar, seed=seed)
File "D:\stable\comfyUI\ComfyUI\comfy\samplers.py", line 715, in sample
return sample(self.model, noise, positive, negative, cfg, self.device, sampler, sigmas, self.model_options, latent_image=latent_image, denoise_mask=denoise_mask, callback=callback, disable_pbar=disable_pbar, seed=seed)
File "D:\stable\comfyUI\ComfyUI\comfy\samplers.py", line 621, in sample
samples = sampler.sample(model_wrap, sigmas, extra_args, callback, noise, latent_image, denoise_mask, disable_pbar)
File "D:\stable\comfyUI\ComfyUI\comfy\samplers.py", line 560, in sample
samples = self.sampler_function(model_k, noise, sigmas, extra_args=extra_args, callback=k_callback, disable=disable_pbar, **self.extra_options)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\utils_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "D:\stable\comfyUI\ComfyUI\comfy\k_diffusion\sampling.py", line 137, in sample_euler
denoised = model(x, sigma_hat * s_in, **extra_args)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "D:\stable\comfyUI\ComfyUI\comfy\samplers.py", line 284, in forward
out = self.inner_model(x, sigma, cond=cond, uncond=uncond, cond_scale=cond_scale, model_options=model_options, seed=seed)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1518, in _wrapped_call_impl
return self.call_impl(*args, **kwargs)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1527, in call_impl
return forward_call(*args, **kwargs)
File "D:\stable\comfyUI\ComfyUI\comfy\samplers.py", line 274, in forward
return self.apply_model(*args, **kwargs)
File "D:\stable\comfyUI\ComfyUI\comfy\samplers.py", line 271, in apply_model
out = sampling_function(self.inner_model, x, timestep, uncond, cond, cond_scale, model_options=model_options, seed=seed)
File "D:\stable\comfyUI\ComfyUI\comfy\samplers.py", line 252, in sampling_function
cond_pred, uncond_pred = calc_cond_uncond_batch(model, cond, uncond, x, timestep, model_options)
File "D:\stable\comfyUI\ComfyUI\comfy\samplers.py", line 226, in calc_cond_uncond_batch
output = model.apply_model(input_x, timestep, **c).chunk(batch_chunks)
File "D:\stable\comfyUI\ComfyUI\comfy\model_base.py", line 85, in apply_model
model_output = self.diffusion_model(xc, t, context=context, control=control, transformer_options=transformer_options, **extra_conds).float()
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "D:\stable\comfyUI\ComfyUI\custom_nodes\ComfyUI_ExtraModels\PixArt\models\PixArtMS.py", line 215, in forward
out = self.forward_raw(
File "D:\stable\comfyUI\ComfyUI\custom_nodes\ComfyUI_ExtraModels\PixArt\models\PixArtMS.py", line 169, in forward_raw
y = self.y_embedder(y, self.training)  # (N, D)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "D:\stable\comfyUI\ComfyUI\custom_nodes\ComfyUI_ExtraModels\PixArt\models\PixArt_blocks.py", line 399, in forward
caption = self.y_proj(caption)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "D:\stable\comfyUI\comf\lib\site-packages\timm\models\layers\mlp.py", line 27, in forward
x = self.fc1(x)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "D:\stable\comfyUI\comf\lib\site-packages\torch\nn\modules\linear.py", line 114, in forward
return F.linear(input, self.weight, self.bias)
R

I am running on windows, latest comfy and extension
torch  2.1.1+cu118,  transformers           4.36.0,  bitsandbytes           0.40.1.post1

The only real guess I have is that you're using the wrong T5 version? It should be t5v1.1 xxl.
That or you're using the diffusers version for the PixArt model instead of the one from here but that's unlikely since it wouldn't even load...

That was it!
I don't know how I took the wrong T5 but I did :)
It was 44gigs, that explains why I thought the t5 loader was slow.. :D
Thanks for your help!

No problem! Glad you got it sorted.


Chrome Web Store Violation: Having obfuscated code in the package

Checklist

[X] The issue can be reproduced in the auth0-react sample app (or N/A).
[X] I have looked into the Readme, Examples, and FAQ and have not found a suitable solution or answer.
[X] I have looked into the API documentation and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have searched the Auth0 Community forums and have not found a suitable solution or answer.
[X] I agree to the terms within the Auth0 Code of Conduct.

Description
I got a violation at the Chrome Web Store.
Code Readability Requirements
Violation reference ID: Red Titanium
Violation: Having obfuscated code in the package.
Violating content:
Code snippet:
ar Bt, Nt, Vt, st, Ir = (Bt = "Lyogcm9sbHVwLXBsdWdpbi13ZWItd29ya2VyLWxvYWRlciAqLwohZnVuY3Rpb24oKXsidXNlIHN0cmljdCI7Y2xhc3MgZSBleHRlbmRzIEVycm9ye2NvbnN0cnVjdG9yKHQscil7c3VwZXIociksdGhpcy5lcnJvcj10LHRoaXMuZXJyb3JfZGVzY3JpcHRpb249cixPYmplY3Quc2V0UHJvdG90eXBlT2YodGhpcyxlLnByb3R

How to rectify: Replace the obfuscated code with human-readable code and resubmit the item.
Reproduction

install npm module
Go to the dist folder and find the string starting with Lyogcm9sbHVwLXBsdWdpbi13ZWItd29ya2Vy

Additional context
No response
auth0-react version
2.1.1
React version
18.2.0
Which browsers have you tested in?
Chrome

@valyefimov were you able to find a solution for this? I've just encountered the same issue.

@valyefimov were you able to find a solution for this? I've just encountered the same issue.

Yes, I wrote a script that decodes the string and updates the final JS file. I run that script after making a production build for Chrome Web Store submissions.
    "postbuild-production": "node ./scripts/fix-auth0-violation.js",

https://gist.github.com/valyefimov/f2fc1c70c9bfd94fba98d962e1112c8c
Note: you have to update this line to specify your build folder
const assetsPath = path.join(__dirname, '../build/chrome-mv3-production');

@valyefimov were you able to find a solution for this? I've just encountered the same issue.

Yes, I wrote a script that decodes the string and updates the final JS file. I run that script after making a production build for Chrome Web Store submissions.
    "postbuild-production": "node ./scripts/fix-auth0-violation.js",

https://gist.github.com/valyefimov/f2fc1c70c9bfd94fba98d962e1112c8c
Note: you have to update this line to specify your build folder const assetsPath = path.join(__dirname, '../build/chrome-mv3-production');

Thank you so much for this, this is super helpful!!


Can I decode every frame in Python?

Now I can get every encoded frame from socket, but I want to decode them in Python. How can I do this? Thanks.

See #129.
Probably to be adapted with newer versions.

Thank you very much. It helped me a lot.


numpy.linalg.linalg.LinAlgError: Singular matrix

Running runfailuretest.py as is gives me a numpy.linalg.linalg.LinAlgError: Singular matrix

Hi @benoitpaillard, did you change the variables defined on the beginning of Test A method?
I've not worked on this project for a while but I just cloned it again & ran it normally. Also, I changed the default layup on master branch. Could you pull & try again?


How about merge My page with user detail?

I don't think there will be a big difference between these pages. The only different things is what to be shown, editable.

I agree with you.

great. We should update the design document.

Merged. please notify me if design document has modified.


fix: Add missing Embed::batch_size implementation

transformers::Embed was missing the Embed::batch_size implementation for BatchableTransformer, causing the pipeline to always use the default batch size when using this transformer.

Indeed, nice find! Thank you for your contribution


[MXNET-978] n-th order gradient test support.

Description
n-th order gradient test support function.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] Support test for n-th order.

@apeforest @larroy
While working on a PR (I don't exactly remember which one), test for incorrect implementation of second order gradient was passing due to the very small values. However as sanity check, when I checked for its third order gradient, the assertion failed allowing to catch the issue.
So I think it would be better to check for the order being implemented and one more (computed by autograd) as a sanity check.
If you think this is in the right direction, will replace check_second_order_unary or make it use check_nth_order_unary.
Would like to know your thoughts about the same.
Thank You.

Ping @sxjscience for review.

@sxjscience Gentle ping for review:)