Index range tests in SparseVector are off by one.

There are redundant range tests in SparseVector so the previous behavior was observably correct and had test coverage already. This fix ensures the failures happen at a consistent location.
Example call stack prior to the fix, note that the exception is thrown from SparseArray.findOffset and not SparseVector.apply:
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index 12 out of bounds [0,12)
    at breeze.collection.mutable.SparseArray.findOffset(SparseArray.scala:198)
    at breeze.collection.mutable.SparseArray.apply(SparseArray.scala:53)
    at breeze.collection.mutable.SparseArray.apply$mcD$sp(SparseArray.scala:52)
    at breeze.linalg.SparseVector$mcD$sp.apply$mcD$sp(SparseVector.scala:75)
    at breeze.linalg.SparseVector$mcD$sp.apply(SparseVector.scala:73)
    at breeze.linalg.SparseVector$mcD$sp.apply(SparseVector.scala:49)
    at breeze.linalg.TensorLike$class.apply$mcID$sp(Tensor.scala:94)
    at breeze.linalg.SparseVector.apply$mcID$sp(SparseVector.scala:49)

thanks


Toggling off DocumentReference attachment processing

Looking for a configurable option to not process DocumentReference url attachments at all regardless of size or type, and didn't see anything that does that in the available params. The params used to dictate whether to inline the content of the URL attachment come close, but, in my case, even when when setting to inlineAttachmentSize to 0, individual bundles containing the attachment are placed in the 'attachment' folder (one for every line of the DocumentReference NDJSON file that has a URL attachment). The client seems to take a lot of time to iterate/process large numbers of these DocumentReference.content.attachment.url elements so I'm thinking that I'd like to be able to toggle that functionality off...
Thanks!

We made some updates and there is a downloadAttachments option now. You can set that to false to disable attachment processing. Feel free to reopen if that does not work for you. Thanks!


Generic, SMT-LIB-based backend for solvers

Currently we have a generic solver interface, realized by a wrapper for Z3 that uses the Java API of Z3. While using such APIs is efficient, it requires a significant effort to integrate a new solver. It would be nice to have a generic implementation, which serializes the expressions as SMT-LIB string (possibly to an in-memory file for efficiency) and passes it to an arbitrary binary that can consume the format. This would make it much more easier to integrate new solvers.

@as3810t I assigned you because you already started working on it on the smtlib-solver branch

This feature is rather huge, so it will be implemented over multiple pull requests to the master, to gather feedback between steps.
Road to SMT-LIB solvers

[x] Initial implementation ( #109 )

[x] Works on Linux
[x] Works with some, SMT-LIB compliant solvers (CVC4, recent Z3)
[x] Works with other SMT-LIB compliant solvers (older Z3, MathSAT5, Princess, SMTInterpol, Yices2, Boolector)
[x] Works with interpolation supported on some solvers (older Z3, MathSAT5, Princess, SMTInterpol)
[x] Works in the CFA cli frontend

[ ] Integration with the rest of the CLI tools

[ ] STS
[ ] XSTS
[ ] XTA

[ ] Works on Windows (maybe MacOS) with some solvers (not all support Windows / MacOS)
[ ] Testing, testing, testing

[ ] First of all, there should be a test suite targeting different solvers and versions and testing their supported features for regression testing. The "soul" of the integration is an ANTLR grammar, which can easily break. The problem is that the number of supported features are huge, plus there is a huge amount of solver versions, across which the supported features change. The test suite should be designed in a way, that most tests are reusable.
[ ] Second, a regression testing should be performed to check, that the algorithms indeed work, and there are no glaring mistakes in the implementation.

[ ] Code cleanup

Nice to have features
This list contains features that would be nice to have, but require big, architecture changing decisions to implement, so they are out-of-scope.

There is no theory detection, so the solving process is not always the best.
If using unsupported features, the failure is detected by the solver. It would be better, if Theta could catch these before sending the unsupported command down to the solver.

I think this was solved with #143, closing.


iOS Demo app is slowing down over time, and the first inference seems much slower

Running the iOS Demo app, I found the 1st inference in CPU and GPU will slower than 2nd inference and later.
When running inference with the CPU, it becomes noticeably slower. GPU and ANE also become slower but not very noticeable.
And if switching the delegates to and from each other, it will become noticeably slower, not only CPU but GPU and ANE, too.
Is this an issue with the MiDaS model, or is it a problem with the TensorFlow Lite Delegate?
The log is as the attached file. I only log the first 105 times inference without switching the delegates.
iphone_15_pro_max_midas.csv

I haven't tried the iOS/tensorflow demo myself, but I can say from using pytorch on desktop machines that it's very common for the first run of large models (not just midas) to be slower than follow up runs. Presumably this is due to some initialization/compilation/caching step that occurs on the first run, but I don't know any of the details.
As for slow down observed on later runs, it could be a thermal effect (i.e. phone/cpu overheating). The cpu takes much longer to finish the tests (and is probably more power hungry) which would explain why the effect is more obvious on cpu (though the ane has strange inconsistencies... may just be in use by something else at the same time?). It might be worth repeating the testing with longer gaps between runs (or active cooling) to see if that improves the pattern.

I think this should be a behavior of the Model. So I close the issue since it is not an issue.


Remove spm

Spm (Static package manager) has been removed and so should be removed from the moment's website and other relevant places.

I have removed the references to spm from our website and made a pull request, as it is clear to me that the spm people intend to shut down.
 Can we remove all spm related code from Moment and stop publishing releases? Or does spm still work, and they're just phasing it out? In that case we would, I suppose, want to continue to publish for a while as people moved off.

@maggiepint I guess the moment's maintainers can remove spm supporting code from the repo but it may induce a breaking change and might be possible at the next major release (if they follow semver)


Possible to slide through Laboratory windows due to step check behaviour

Cogito and Godot Engine Version:
Godot version: 4.2.2
Cogito version: beta 202409.04
Description:
Possible to slide through Laboratory windows due to step check behaviour
Reproduction steps:

Run at the Windows in the Laboratory
Should enter the windows, due to step check registering them as step
Also enters slide as player forced into crouch, whilst running

Expected behavior:
Not sliding through windows.
Can be fixed by lowering default step height
Screenshots:
The type of window:

I would also like to add that because this template supports auto-crouching, crouch jumping, stepped-geometry stairways and the like that there should be some guidelines for things like window height from the ground, stair step and table height.
I also think lowering the default step height a little would feel more accurate, because you can too easily walk up onto the coffee table in the lobby, and at the same time it should retain the smooth movement and capabilities of walking over ledges and grounded objects easily that come with a higher step height.
Because the feature is quite dynamic it might just be a matter of having some basic guidelines for minimum window and table height, otherwise it would just require lots of testing and iteration to fix all the exploits, which is expected regardless. 😭

I found what cause this problem:
This line is related to stair stepping:
https://github.com/Phazorknight/Cogito/blob/4c1df66d938d5ae3dbec67cf14546c0a54332fd9/addons/cogito/CogitoObjects/cogito_player.gd#L1082
and after this head's position change, in next frame, condition in this line will be true:
https://github.com/Phazorknight/Cogito/blob/4c1df66d938d5ae3dbec67cf14546c0a54332fd9/addons/cogito/CogitoObjects/cogito_player.gd#L877
and in the next frame this part of code will be executed because raycast is_colliding is true:
https://github.com/Phazorknight/Cogito/blob/4c1df66d938d5ae3dbec67cf14546c0a54332fd9/addons/cogito/CogitoObjects/cogito_player.gd#L853
Before:
https://github.com/user-attachments/assets/98b3fcab-7155-44fd-9b39-f2b7ce2e180f
After replacing these two lines of code with pass, issue will be solved but we need a better solution!
https://github.com/Phazorknight/Cogito/blob/4c1df66d938d5ae3dbec67cf14546c0a54332fd9/addons/cogito/CogitoObjects/cogito_player.gd#L879-L880
https://github.com/user-attachments/assets/5755706d-e409-40aa-919a-1c55e294a1a5
These lines added by @Phazorknight in this commit: https://github.com/Phazorknight/Cogito/commit/71fcf4c14104136af527a1a961c5a8ea2a4b1994
I think someone with better understanding of player's code can fix this bug without introducing new one!
I found another bug related to crouch/uncrouch and stair stepping, I'll send a PR for that soon.

Thanks everyone for flagging this and investigating.
For now, I've simply reduced the step height in the stair settings of the player controller (it was 0.5 i set it to 0.35 ) and that seems to do the trick. (see https://github.com/Phazorknight/Cogito/commit/97d53f1544a503623b1a192b24d7be003dbee161 )
This also helps with avoiding the player climbing on top of tables and couches in the Lobby scene.
It's still possible to jump-crouch through the Lab windows, which I think is fine. It seems like the kind of opening in a wall the player should be able to climb through.

Closing this as original issue has been resolved.


Implement ECS-Compatibility Mode

As a part of the effort to make plugins able to run in an ECS-Compatible manner
by default in an upcoming release of Logstash, this plugin needs to implement
an ECS-Compatibility mode that does not implicitly use fields that conflict
with ECS.
Absent target directive, this plugin populates fields verbatim from the
JSON-encoded payload it is given, and may benefit from requiring an explicit
target when run in ECS-Compatibility mode, or otherwise reduce the risk of collision.

delivered in https://github.com/logstash-plugins/logstash-filter-json/pull/49


Ninja Usagimaru - The Gem of Blessings (USA)

Game Name (and version if necessary)
Ninja Usagimaru - The Gem of Blessings
Hardware Specs (If necessary)
CPU: Processor
12th Gen Intel(R) Core(TM) i3-12100F
base speed : 3,30 GHz
Cores : 4
Threads : 8
GPU: GPU 0
NVIDIA GeForce RTX 3050
Dedicated Memory (Vram): 8,0 Go
RAM: Memory
16,0 Go DDR4
Speed : 3200 MHz
Operating System
Windows 11 Pro Insider Preview 23546-1546
Game State
Instant crash
Crash with the (famous): Fired exception oops
Log:

Screenshots
record of the crash:
https://github.com/Panda3DS-emu/Panda3DS-Games-List/assets/102532446/06592c07-8641-4335-ba28-69e23e77f0e3
Additional information
No response

Confirmed to be a decryption issue, game is still horribly fucked up if I manually decrypt it though but at least it shows "graphics"

Fixed

👍


Participant count continuously increments

I'm guessing this is because the backend is not correctly handling the disconnect event

Will need to replicate this on the dev by having people join and leave by closing out.

Person 1 joins, yielding 1 participants.
Person 2 joins, yielding 2 participants.
Person 2 refreshes, yielding 3 participants.

fixed


Grid barely responsive with 1000 rows, about 10 columns

Despite original delay during first render, I've also got few seconds delay when trying to use key navigation or change a cell.
How could I make it fast?
Also how could I show eg 20 rows on screen and use scroll or key navigation to show the rest?

Try to avoid passing the component with arrow functions or binding them in the render function. Otherwise 1000 rows is pretty extensive. A proper solution would be to use react-virtualized for the rendering. I'll get around to trying something like that eventually.

I think there is a lot of room for improvement, especially for selecting and editing, even without react-virtualized will try to get my head around this soon and send a PR.

I was able to do it without making react-virtualized a hard dependency by modifying the render in DataSheet.js to pass back a virtual row renderer that matched the react-virtualized api.
https://gist.github.com/cthurston/516117bb5143ed0e455143962fafef57
It feels a little circular.  With a small change in the DataSheet api something close to this may be a viable route to performance on large datasets.

Hey there. I created a contemporary variant at iddan/react-spreadsheet. Way more responsive on large datasets. Going to make it even more in the following week by fixing the new React Context API polyfill.

@cthurston i tried your solution and did not worked :/
Just throw a lot of Invariant Violation's errors.
The gist is correct?

@iddan i tried your solution and did not worked :/ is there any update

What did not work?

@iddan needed Help on virtual columns and Rows implementation please provide me some input

simply perform ｎａｔｕｒａｌ　ｔｒａｎｓｆｏｒｍａｔｉｏｎ of react-virtualized onto react datasheet


ZooKeeper Version Compatibility

The zookeeper plugin in cluster module is based on the apache curator. Curator 4.0 has a hard dependency on ZooKeeper 3.5.x.
But our users maybe just had ZooKeeper 3.4.x. Add description into config file and document about how to compatibility with it.

Added.


What should we do about pre-seeded games?

The prime offenders being nht's 235 turn AceHack ascension and their 363 turn unnethack asc (as unnht).
At least with the unnethack game, the seed is shown in the dumplog, allowing us to track which games are pre-seeded,
though the current infrastructure does not look at dumplog information, only xlogfile. See https://ascension.run/history/unn/users/unnht/dumps/eu/unnht.1429442870.txt.html
The AceHack game however, does not show any seed info in the dumplog.

i feel like acehack is effectively historical/defunct, both it and nitrohack were abandoned and essentially replaced by NH4.  it seems like the only cheaty run is NHT's superspeed run - why not simply exclude that one manually?  in other variants that allow seeding, the games are already unscored.
old conversations:

https://nethackwiki.com/wiki/Random_number_generator#Misuse
https://old.reddit.com/r/nethack/comments/3355z6/363_turn_16_second_set_seed_unnethack_win/
https://old.reddit.com/r/nethack/comments/31ifi4/prelude_to_an_unnethack_speedrun_set_seed_gametime/cq27bfn/

Yeah true. I'm still sorta undecided about what to do with the record because I don't want to remove it entirely,
but as you mentioned on IRC we could have a separate category/page somewhere on the scoreboard site for cool stuff ppl done with preseeded games...

there could be a "historically interesting games" page?  and they don't all necessarily need to be ones where some form of exploit happened, it could include things like some of the stuff from https://nethackwiki.com/wiki/Notable_Players
But at least there'd be a place for interesting games that shouldn't be scored, and/or games that caused the devteam to fix a critical flaw/vulnerability.


Implement apply_delta() on Account

This PR implements Account::apply_delta() method which enabled application of AccountDelta object to an account.

Only thing I could suggest is a test, but the code looks simple enough.

Agreed! I'll merge this PR and will create an issue for this.


[1.3.0-a0] First source to the next Armor v1.3.0

Resolves #6 and resolves #4 . Besides to these issues, this PR brings some code formatting refactorings/fixes.

Besides to what is said on the first PR comment, this PR implements some fixes made on the branch 1.2, fixes which has been added until the v1.2.11 and brings some new implementations of methods and documentation comments refactorings.


Some pictures with complex structure are very time-consuming to convert

Some pictures with complex structure are very time-consuming to convert. For example, 67kb image conversion takes 1521345ms

This project just wrap java code and dll files. You may go to source repo luciad/webp-imageio , and create issue about supporting new feature of animation webp.


Add the ProcessControlFlowGuardPolicy mitigation policy support.

This pull request is identical to the previous one, except the email address.

Thanks for doing this. Really appreciate it.

Thanks for your great tool !!
On 06.09.2017 14:44, James Forshaw wrote:

Thanks for doing this. Really appreciate it.
--
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [1], or mute the thread [2].

Links:
[1]
https://github.com/google/sandbox-attacksurface-analysis-tools/pull/7#issuecomment-327471733
[2]
https://github.com/notifications/unsubscribe-auth/AeMODCRSGhrc18XhTk1yet55IuKf4gKbks5sfpPFgaJpZM4POUBA


Traversal Trees

Closes #117
Closes #118 (hopefully! 🤞)
Closes #119
Closes #115
Try Again!
This PR is the result of iterating on #119. #119 attempted to resolve the same issue, but @amascolo generously raised examples that still failed.
This PR (maybe!) resolves these additional cases, which are now also included as tests.
I will include the identical "root cause" section here as #119 so that this PR may be readable on its own.
Root Cause
Consider the grammar:
<start> ::= <shortfail> | <longsuccess>
<shortfail> ::= <char> 'never'
<char> ::= 'a'
<longsuccess> ::= <long2>
<long2> ::= <long3>
<long3> ::= <long4>
<long4> ::= <char>

When parsing input "a", there are two routes: "shortfail" and "longsuccess". As the names suggest, "shortfail" requires fewer traversals, but always fails. "longsuccess" requires more traversal steps, and should eventually succeed.
The issue is caused because both paths predict the non-terminal "char". Practical Earley parsing requires de-duplicating predictions or else recursive grammars fall into infinite loops. The existing Earley implementation in BNF does roughly the following:
(work roughly alternates between the short and long routes because new traversals are appended to the end of the work queue)
* <start> ::= • <shortfail>
* <start> ::= • <longsuccess>
* <shortfail> ::= • <char> 'never'
* <longsuccess> ::= • <long2>
* <char> ::= • 'a'
* <long2> ::= • <long3>
* <char> ::= 'a' •
* <long3> ::= • <long4>
* <shortfail> ::= <char> • 'never' // <--- notice this does NOT succeed
* <long4> ::= • <char> // !!! this <char> prediction is IGNORED because an identical prediction was already made

All the <longN> productions are necessary because otherwise the <longsuccess> route is able to predict  before its completion.

I am sorry if I have not explained this super clearly. It is a tricky problem! Funny aside, I actually thought of this bug while developing v0.4.0 . But I (wrongly) assumed there was something about the Earley algorithm which resolved it. I attempted to write a test, but I did not realize it would require so many intermediate non-terminals to expose. Woops!

Existing Issues
Where this PR differs from #119 is its approach to "duplicate detection". Previously, duplicate Earley states/traversals were identified by which Production and how many Terms had been matched. This turns out to be insufficient, because partially "matched" Productions (i.e. <shortfail> ::= <char> • 'never') could have matched non-terminals via different paths.
New Duplicate Detection
Traversals now match/complete terms by building trees 🌳
A new prediction is the root trunk of a tree, and each matched/completed term adds a new branch. Assuming there are two different traversals which can complete base, a traversal tree segment may look like:
flowchart TD
0["dna ::= • base dna"] --> 1["dna ::= base=1 • dna"]
0["dna ::= • base dna"] --> 2["dna ::= base=2 • dna"]

Performance
On my machine, there was seemingly no performance cost to these changes. I believe there was a cost to adding the new logic for "prior completed" traversals. But that cost was offset by the improvement of traversal trees, instead of reference counted term matching vectors.
Extra
Basically every time I have worked on an Earley bug, I have ended up adding the same manual logging to help with debugging. I decided to commit that logging this time!
There is a new tracing::event! which by default is a noop, but with the "tracing" feature enabled adds logging events.
For Earley, traversal state events are logged when created and during predict/scan/complete. It helps quite a bit with debugging!

I again plan to leave this PR open to give some time for review. Also I still have to read it all over myself with fresh eyes, and add a lot of comments 😅

Coverage: 91.549% (-0.6%) from 92.114% when pulling e7b36fc6a6ba4bbcc21bb14c5f70fe1dc641f435 on right-recursive-failure-new into f3ca02101c71636be2d0e7e1605578e63767082c on main.

snapshot testing has exposed some nondeterminism in the Earley parsing. the parse trees are valid, but for ambiguous grammars are parsed in inconsistent orders between test executions.
I will investigate this (likely a Hashmap or something 🙏). Otherwise, this PR is holding up to my local scrutiny. Once I resolve the nondeterminism issue, I plan to merge 🎊


Theme Park: Invalid translation

Current translation: 카페 통로
It should be translated as follows: 카페 테라스

I will modify with new map translation. Thank you!


The Definition List Definition does not take styles

The field Style set for the DefinitionListDefinition does not take effect.
Expected behavior
Setting the value "pre" for a DefinitionListDefinition shall set the style white-space: pre; for all dds.

See also https://github.com/Xyna-Factory/xyna-zeta/wiki/Workflow‐defined-UI#definition-datatypes


IconLayer crashing on Safari when updating layers and auto packing atlas

Description
We are using IconLayer to represent live data with auto packing atlas (we generate a few hundred different icons depending on the data we receive). Everytime we receive an event with updated data, we set props with an updated layer and we get the changes represented in the map.
Updating process is working fine on Chrome and Firefox, however, with Safari we only get crashes.
It seems that it is a problem with WebGL2 not supported on Safari, but as first creation is working fine, it would be great to we able to update the layers the same we are creating them (creation works without problems) in the cases in which WebGL2 is not supported.
Repro Steps
We update the layer, using set props like this:
this.overlay.setProps({ layers: [       clustersLayer.clone({ data: clustersData }) ]});
The amount of data entries may change depending on the bounds of the map in the screen and the events we receive, so we don't always have the same length of data (pointing this out, just in case it is relevant).
It is always working fine in Google Chrome and Firefox, but with Safari, altough it starts working fine (first creation is ok), once we update the layers with new data which generates new icons, it crashes. I'm attaching the stack trace.
Environment (please complete the following information):

Framework Version: 7.3.3
Browser Version: Safari 13.0.3
OS: Mac OS Catalina 10.15.1

Logs

[Error] ERROR
Error: Requires WebGL2
assert — assert.js:3
assertWebGL2Context — webgl-checks.js:15
readPixelsToBuffer — copy-and-blit.js:78
resizeTexture — icon-manager.js:56
_updateAutoPacking — icon-manager.js:302
setProps — icon-manager.js:245
updateState — icon-layer.js:193
_updateState — layer.js:788
_update — layer.js:772
_updateLayer — layer-manager.js:501
_updateSublayersRecursively — layer-manager.js:383
_updateLayers — layer-manager.js:329
setLayers — layer-manager.js:189
setProps — layer-manager.js:148
setProps — deck.js:223
setProps — google-maps-overlay.js:48
updateOverlay — ng-components.js:4531
ngOnChanges — ng-components.js:4436
checkAndUpdateDirectiveInline — core.js:31905
debugCheckAndUpdateNode — core.js:45327
debugCheckDirectivesFn — core.js:45270
(anonymous function)
checkAndUpdateView — core.js:44270
callViewAction — core.js:44636
execEmbeddedViewsAction — core.js:44593
checkAndUpdateView — core.js:44271
callViewAction — core.js:44636
execComponentViewsAction — core.js:44564
checkAndUpdateView — core.js:44277
callViewAction — core.js:44636
execComponentViewsAction — core.js:44564
checkAndUpdateView — core.js:44277
callViewAction — core.js:44636
execEmbeddedViewsAction — core.js:44593
checkAndUpdateView — core.js:44271
callViewAction — core.js:44636
execComponentViewsAction — core.js:44564
checkAndUpdateView — core.js:44277
callWithDebugContext — core.js:45631
detectChanges — core.js:31125
tick — core.js:41044
(anonymous function) — core.js:40892
onInvoke — core.js:39698
run — zone-evergreen.js:124
next — core.js:40889
(anonymous function) — core.js:35335
__tryOrUnsub — Subscriber.js:183
next — Subscriber.js:122
_next — Subscriber.js:72
next — Subscriber.js:49
next — Subject.js:39
emit — core.js:35297
checkStable — core.js:39641
onHasTask — core.js:39718
hasTask — zone-evergreen.js:411
_updateTaskCount — zone-evergreen.js:431
_updateTaskCount — zone-evergreen.js:264
runTask — zone-evergreen.js:185
drainMicroTaskQueue — zone-evergreen.js:559
promiseReactionJob
	error
	View_MapComponent_1 (MapComponent.ngfactory.js:7)
	logError (vendor.js:99488)
	logError
	handleError (vendor.js:64138)
	run (polyfills.js:3359)
	tick (vendor.js:95185)
	(anonymous function) (vendor.js:95023)
	onInvoke (vendor.js:93957)
	run (polyfills.js:3359)
	next (vendor.js:95020)
	(anonymous function) (vendor.js:90285)
	__tryOrUnsub (vendor.js:257900)
	next (vendor.js:257839)
	_next (vendor.js:257789)
	next (vendor.js:257766)
	next (vendor.js:257552)
	emit (vendor.js:90247)
	checkStable (vendor.js:93900)
	onHasTask (vendor.js:93977)
	hasTask (polyfills.js:3646)
	_updateTaskCount (polyfills.js:3666)
	_updateTaskCount (polyfills.js:3499)
	runTask (polyfills.js:3420)
	drainMicroTaskQueue (polyfills.js:3794)
	promiseReactionJob

@ruizmarc  thanks for reporting, yes readPixelsToArray uses WebGL2 only API, we should replace this with copyToTexture , should be easy to update and happy to review if you can create a PR
copyToTexture:  doc and source

Hi!
Thanks for the suggestion. I will work on the pull request. I'm not sure if I'm walking to the right direction because it seems it works better than before, but I'm having a new error. I'm attaching it at the end the comment.
This is what I've understood from your suggestion about modifying resizeTexture function (let me know if this is what you were thinking about):
function resizeTexture(texture, width, height) {
  const oldWidth = texture.width;
  const oldHeight = texture.height;
  const tempTexture = new Texture2D(texture.gl, { width: oldWidth, height: oldHeight});
  copyToTexture(texture, tempTexture);
  texture.resize({width, height});
  copyToTexture(tempTexture, texture, {
    targetY: height - oldHeight,
    height: height,
    width: width
  });
  texture.generateMipmap();
  return texture;
}

The error I'm having now is this one. Sorry for asking, but it is the first time working with WebGL and I need a few clues to make this pull request happen:

Thanks for your feedback

Is new width and height are always greater than current texture width and height?
I would try this :
function resizeTexture(texture, width, height) {
  const newTexture = cloneTextureFrom(texture, {width, height});
  copyToTexture(texture, newTexture);
  texture.delete();
  return newTexture;
}

also you need to update this._texture explictly.

I have just tried this, and although is not showing any error in the console, at some point after a few data updates, icons stop being represented or just a few of them get represented. It is like it matches with empty pixels in the atlas.
I have also tried your approach, but with the offset that was previously being applied, but I get similar results:
function resizeTexture(texture, width, height) {
  var oldHeight = texture.height;
  const newTexture = cloneTextureFrom(texture, {width, height});
  copyToTexture(newTexture, newTexture, {
    targetY: height - oldHeight,
    height: height,
    width: width
  });
  texture.delete();
  return newTexture;
}

With this offset, they also eventually stop being represented or just a few of them get represented. Furthermore, with the offset I also get the error I was mentioning in my previous comment.
This is the evolution of resizes, just in case it may help:
[Log] oldHeight – 512 – "oldWidth" – 1024 – "newHeight" – 1024 – "newWidth" – 1024 (vendor.js, line 142583)
[Log] oldHeight – 1024 – "oldWidth" – 1024 – "newHeight" – 2048 – "newWidth" – 1024 (vendor.js, line 142583)
[Log] oldHeight – 2048 – "oldWidth" – 1024 – "newHeight" – 4096 – "newWidth" – 1024 (vendor.js, line 142583)

Fix is landed in 7.3.5

Yeah! Works great on Safari! Thanks a lot!


[WIP] Support for ConfigurationSetName

This PR resolves #1.
It introduces the ability to pass in a value for ConfigurationSetName which would otherwise be a default value (empty/""). The field has also been added to tests already supported by the application.
Testing will be organized prior to removing the draft flag, but I would appreciate some feedback from @blueimp in regards to how receptive they are to merging this and what they would like to see included in this PR leading up to a merge.
Also, for general knowledge, this field is natively supported by the API.

Marked as ready based on advice.
I'll still put it into testing and let you know how it goes 👍

Thanks @fubarhouse - I've tested and can confirm we can view metrics in CloudWatch and SES contents via Kinesis Firehose.
Note that to use the -r switch, we just overrode the entrypoint in Openshift.


Explanation for field enclosure?

Hello!
Can you explain to me what field enclosure does? It seems it requires some characters like \t\x1f but I don't know what this means.
I have the library working on my project, but I just wanted to know what this actually does.
Br,
Miika

@mkoske thanks for using the library if you want to know about csv options please refer to PHP documentation manual

https://www.php.net/fputcsv
https://www.php.net/fgetcsv

there you will have all the information you need.


Api - No result when too much results

Hello !
When I make PDO request, I can have a lot of result, but nothing is returned. I tried to var_dump tht fetchall, no problem, I have everything.
But in the end the API response is empty.
Thanks a lot

Can you see if there is anything in your error logs? I suspect that you might be running out of memory if the result set is too large.

Ok, I found what is going wrong.
UTF8...
If I have a bad encode character in my function return, no output.
I have to array_map_recursive("utf8_encode", $fetched_result)


Face detect not work with medium resolution preset

I'm making a face detect app on Android and it only detects faces with any resolution but medium. I don't know why because it wont work with all resolution if the problem came from my code. Anyone have the same problem?
By the way, when i change the orientation of device to landscape, the camera preview turn to opposite (180).
Sorry for my bad English.

@hoangquy4897 Please provide an example of code to reproduce the issue. Also your flutter doctor -v output.
Also I would advise you to post the issue to StackOverflow (there you may find help faster), as it may relate to the implementation, not to flutter itself.

I have same issue, see https://github.com/FirebaseExtended/flutterfire/issues/1518


"If there aren't any HTTPs URLs - the first HTTP URL"

Text says "If there aren't any HTTPs URLs - the first HTTP URL" - clearly wrong

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

ID: be16b5c5-40c4-3e6c-0b0d-840373c63f9f
Version Independent ID: ced6b3ba-8da1-dd3d-9d1c-8a21dabd44fe
Content: Restrictions and details of API formats support - Azure API Management
Content Source: articles/api-management/api-management-api-import-restrictions.md
Service: api-management
GitHub Login: @vladvino
Microsoft Alias: apimpm

Thanks for the feedback! I have assigned the issue to the doc author to update appropriately.

This issue has been resolved. Thank you! #please-close


[docker] Fix chip-build-nrf-platform Dockerfile

Problem
The 'west' tool no longer supports checking out repositories using a specific commit SHA. As a result, the nRF Connect Docker image build fails.
Change overview
Fix Dockerfile for the nRF Connect platform not to use that deprecated feature.
Testing
Built the Docker image manually.

Marking as hotfix:  this issue prevented us from building updated dockerfile images.
In hindsight it seems that not being able to fix 'west' version may hurt us (we cannot build old docker images probably either).


Proof of Concept: Introduce ServerFactoryContext

Description:
Introduce ServerFactoryContext which is an implementation of CommonFactoryContext. Comparing to ListenerFactoryContext, ServerFactoryContext has a wider lifetime.
In this PR I also prove ServerFactoryContext would allow multiple listeners share a RdsConfigProvider and save huge of memory and CPU
[x] Compile envoy-static with ServerFactoryContext: RdsConfigProvider, PerFilterConfig, Route, VhdsSubscription should use ServerFactoryContext
[ ] Safely implement ServerFactoryCxtUtil::generateServerFactoryContext
[ ] Server::Instance should implement ServerFactoryContext
[ ] RdsConfigProvider should own RdsSubscription
[ ] RdsConfigProvider should be able to notify multiple listeners and listeners could wait on RdsProvider in ready state
[ ] tons of broken tests due to signature change
Risk Level:
Testing:
Docs Changes:
Release Notes:
Fixes #8303

@lizan Please review https://github.com/envoyproxy/envoy/pull/8485 instead.
This PR will be updated to describe the full picture. It will be growing bigger and not suitable for review.

Split into #8485 and #8523


Missing left join in generated SQL

I've been trying to figure out a way to extract some "detail" data from a set of tables that have a Master/Detail relationship.
Programs MASTER has many ProgramVendors DETAIL.  I want to display the ProgramVendor has a field called Primary to signify it as the primary vendor for the program.  Here's the fields of both tables
ProgramVendor
Id	int
VendorId	int
VendorContact	nvarchar(50)
SubmittedBy	nvarchar(50)
[Primary]	bit
ProgramId	int	FK to Program Table
Programs
id	int
businessKey	varchar(30)
effOrderStart	datetime
effOrderEnd	datetime
effShipStart	datetime
effShipEnd	datetime
notes	nvarchar(MAX)
created	datetime
createdBy	varchar(50)
modified	datetime
modifiedBy	varchar(50)
CompanyId	int
ProgramTypeId	int
ProgramTimeFrameId	int
ProgramTermsId	int
ProgramDealClassId	int
ExcludeMonthEndCalc	bit
IsActive	bit
In my ProgramRow.cs class I defined this field and this class level join
[LeftJoin("jProgramVendor", "[dbo].[ProgramVendor]", "T0.Id = jProgramVendor.ProgramId")]
public sealed class ProgramsRow : Row, IIdRow, INameRow
{
....
[DisplayName("Primary Vendor"), ForeignKey("[dbo].[Vendor]", "id"), Expression("jProgramVendor.VendorId"), NotMapped]
[LookupEditor(typeof(VendorRow))]
public Int32? PrimaryVendor
{
get { return Fields.PrimaryVendor[this]; }
set { Fields.PrimaryVendor[this] = value; }
}
}
The join works fine in SQL server but when I look at the generated SQL it is not present at all when the grid is displayed.

Is there a better way to slice out data from a detail table to display on the master grid?  I've searched and searched and tried overiding the OnReturn of MyListHandler class to fetch what I want etc with no luck.
If this can't be done so be it I will tell the client we'll have to do some work-around etc.  But I'm at a loss.
Why is the class level join not sowing up and if there's a better way I'm all keyboard.
-Garrett

The columns added PrimaryVendor?

Yes I added a new field in the ProgramsRow for the PrimaryVendor that is not mapped to the Programs Table ... see code snippet...
    [DisplayName("Primary Vendor"), ForeignKey("[dbo].[Vendor]", "id"), Expression("jProgramVendor.VendorId"), NotMapped]
    [LookupEditor(typeof(VendorRow))]
    public Int32? PrimaryVendor
    {
        get { return Fields.PrimaryVendor[this]; }
        set { Fields.PrimaryVendor[this] = value; }
    }

Perhpas I'm not declaring the attribute correctly but the name of the leftjoin is jProgramVendor.
Thank you for your response and help

He is asking if you added it to "Columns", invisible foreign fields are not loaded by grid by default. Search for "getIncludeColumns" in source to force load a invisible property when needed.

Is there an example of how to properly specify the "columns"  is it part of the LeftJoin attribute?


Change GATEWAY_REQUEST_URL type from URI to UriComponents

https://github.com/spring-projects/spring-framework/issues/24439#issuecomment-579829363
netty HttpRequest doesn't rely on URI.

After an initial attempt, I'm not sure it's worth the time and effort to get all the nuanced encoding things right.


Focus gets confused, ends up with no split focused

Twin bug to https://github.com/vim-airline/vim-airline/issues/1131 as I'm still not sure which plugin is the actual culprit; they only seem to misbehave when together.

This is reproducible for me with just vim-stay and vim-airline installed, no vimrc except the pathogen line, and a clean view folder, so I don't think there are any special settings involved? When I get to my proper computer I'll post the full build features of vim just in case.

@eapache thanks for posting the issue here. As I said, my problem reproducing is that if I follow your steps literally, I end up having no open windows at all, so it seems I am missing something altogether. The output of :version would be quite useful, as I have failed to notice issues due to fixes in one of the recent 800 or so patches before…

Full version:
MacOS X (unix) version
Included patches: 1-1707
Compiled by Homebrew
Huge version with MacVim GUI.  Features included (+) or not (-):
+acl             +file_in_path    +mouse_sgr       +tag_old_static
+arabic          +find_in_path    -mouse_sysmouse  -tag_any_white
+autocmd         +float           +mouse_urxvt     +tcl
+balloon_eval    +folding         +mouse_xterm     +terminfo
+browse          -footer          +multi_byte      +termresponse
++builtin_terms  +fork()          +multi_lang      +textobjects
+byte_offset     +fullscreen      -mzscheme        +timers
+channel         -gettext         +netbeans_intg   +title
+cindent         -hangul_input    +odbeditor       +toolbar
+clientserver    +iconv           +packages        +transparency
+clipboard       +insert_expand   +path_extra      +user_commands
+cmdline_compl   +job             +perl            +vertsplit
+cmdline_hist    +jumplist        +persistent_undo +virtualedit
+cmdline_info    +keymap          +postscript      +visual
+comments        +langmap         +printer         +visualextra
+conceal         +libcall         +profile         +viminfo
+cryptv          +linebreak       +python          +vreplace
+cscope          +lispindent      -python3         +wildignore
+cursorbind      +listcmds        +quickfix        +wildmenu
+cursorshape     +localmap        +reltime         +windows
+dialog_con_gui  -lua             +rightleft       +writebackup
+diff            +menu            +ruby            -X11
+digraphs        +mksession       +scrollbind      -xfontset
+dnd             +modify_fname    +signs           +xim
-ebcdic          +mouse           +smartindent     -xsmp
+emacs_tags      +mouseshape      +startuptime     -xterm_clipboard
+eval            +mouse_dec       +statusline      -xterm_save
+ex_extra        -mouse_gpm       -sun_workshop    -xpm
+extra_search    -mouse_jsbterm   +syntax
+farsi           +mouse_netterm   +tag_binary
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
  system gvimrc file: "$VIM/gvimrc"
    user gvimrc file: "$HOME/.gvimrc"
2nd user gvimrc file: "~/.vim/gvimrc"
    system menu file: "$VIMRUNTIME/menu.vim"
  fall-back for $VIM: "/Applications/MacVim.app/Contents/Resources/vim"
Compilation: clang -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_MACVIM -Wall -Wno-unknown-pragmas -pipe  -DMACOS_X_UNIX  -F/usr/local/Cellar/python/2.7.11/Frameworks -I/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/include/python2.7 -I/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/include/python2.7 -fno-strict-aliasing -fno-common -dynamic -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: clang   -L. -L/usr/local/lib -L. -L/usr/local/lib -F/usr/local/Cellar/python/2.7.11/Frameworks -L/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/config -lpython2.7 -framework CoreFoundation -L/usr/local/lib -o Vim -framework Cocoa -framework Carbon       -lm  -lncurses -liconv -framework Cocoa   -fstack-protector  -L/System/Library/Perl/5.18/darwin-thread-multi-2level/CORE -lperl -F/usr/local/Cellar/python/2.7.11/Frameworks -framework Python  -F/System/Library/Frameworks -framework Tcl -framework CoreFoundation -framework Ruby```

I'm not sure how those steps could leave you with no window at all; you should have two splits open when you run the final `:bd` and there's no reason I can think of for `:bd` to close *multiple* splits at once.

there's no reason I can think of for :bd to close multiple splits at once.

Isn’t that the expected behaviour when multiple windows (splits) show the same buffer?

Not when there is another buffer in the background on the second split; the second split should (and does, for me) switch to showing the other buffer.

@eapache you are correct about the window behaviour, of course, I must not have followed your steps closely enough. Good news is I was able to reproduce the issue and am pretty sure I have pinpointed the issue, but to confirm, could you please tell me the output of :echo &viewoptions?

folds,options,cursor

These are the Vim defaults – thank you, you just confirmed my diagnosis. To make along story short, the issue is with options: with that flag in &viewoptions, view session files will store all local options, including &statusline, which confuses airline after loading a view session file (I haven’t investigated in detail, but something like autocmd SessionLoadPost * AirlineRefresh should mitigate the issue).
I have recommended a different setting in both the README and the help, but that obviously is not enough, so I will enforce set-=options while making views starting henceforth (and restore the user’s &viewoptions afterwards, obviously – no plug-in should permanently alter user settings).

So basically vim-stay was loading/restoring status-line values (since that's the default for mkview) which was confusing airline by changing its settings out from under it?

That’s the gist of it, yes.

Thanks, the 1.4 release branch fixes this nicely.

Reopening until merged into master.


Help window update

Codecov Report

Merging #102 (3e50ed9) into master (75fe5f7) will decrease coverage by 0.29%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #102      +/-   ##
============================================
- Coverage     41.70%   41.41%   -0.30%
  Complexity      365      365
============================================
  Files           108      108
  Lines          1983     1997      +14
  Branches        192      192
============================================
  Hits            827      827
- Misses         1082     1096      +14
  Partials         74       74

Impacted Files
Coverage Δ

...elr/logic/commands/DeleteEventFromTripCommand.java
0.00% <ø> (ø)

src/main/java/seedu/travelr/ui/HelpWindow.java
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[IT-4003] Auto-update pre-commit hook versions monthly

Change the frequency that PRs to update pre-commit hook versions are auto-generated from weekly (the default) to monthly.

coverage: 91.667%. remained the same
when pulling 6f19a89bbb23d79426ba0fd0aa07e135da2f61a4 on ConsoleCatzirl:it-4003
into 1076701a1f55ec59549c40af57d50298cf20c4a8 on Sage-Bionetworks-IT:master.


[develop_2.0] default values (for model properties) implemented

according to http://json-schema.org/latest/json-schema-validation.html#anchor101,
https://github.com/swagger-api/swagger-spec/blob/master/versions/2.0.md#schemaObject

this should be supported in 1.5.4-M1-SNAPSHOT.


Handle filenames with spaces

When running
~/go/bin/rac -output rac-out Payload\ RT_20191023-115059_20191024-125014.rac
files are created with ' as part of the filename, e.g.:
'Payload RT_20191023-115059_20191024-125014_PWR.csv'
Make sure spaces in filenames are handled in a better way, so that the resulting files get filenames that we can handle.

Flagging as invalid - I think this is probably just how ls displays it. Someone else could perhaps have a look before closing?

I don't have any with space in them, but I suggest we just close this and reopen if we know it is a problem / get a user complaint.


from xlsx from ods and from toml

They all work pretty much as in the old engine
〉cat ./Cargo.toml | from toml | get features | get plugin
╭───┬───────────────────╮
│ 0 │ nu-plugin         │
│ 1 │ nu-parser/plugin  │
│ 2 │ nu-command/plugin │
╰───┴───────────────────╯

〉cat ./Teste.xlsx | from xlsx -s [Planilha1] | get Planilha1
╭───┬─────────┬─────────╮
│ # │ Column0 │ Column1 │
├───┼─────────┼─────────┤
│ 0 │ foo     │ bar     │
│ 1 │ 1       │ 2       │
╰───┴─────────┴─────────╯

〉cat ./TesteODS.ods | from ods -s [Planilha1] | get Planilha1
╭───┬─────────┬─────────┬─────────╮
│ # │ Column0 │ Column1 │ Column2 │
├───┼─────────┼─────────┼─────────┤
│ 0 │ 1       │ 2       │ 3       │
│ 1 │ yo      │ Yo 2    │ Yo 3    │
╰───┴─────────┴─────────┴─────────╯

A few things to consider.
-1: I couldn't add tests examples to from xlsx and from ods because they use binary output which would be quite weird to add and to appear on the -h page.
-2: In the old engine, from xslx had a flag -n. Which was there to don't treat the first row as column names. The thing is, this is the default behaviour anyway, so i did some diging in how the codebase looked back when the command was added(28 of november of 2019) and it turns out this flag never did anything, i don't know why it was there. Maybe in the future i'm going to add a flag which adds the behaviour of treating the first row as the column name, but for now i think i will focus on porting all the rest of the from commands if thats ok.

looks good!


[How?] Debugging crash in jax.jit function when running under pytest-xdist

For NetKet we run the test-suite under pytest-xdist (for those not familiar with it, xdist is an extension that forks the python process and then splits the test suite among those processes).
For most of all our tests, we run calculations with relatively small matrix sizes, however a few tests involve compiling some fairly involved functions/gradients which take quite some time.
We run our tests both locally and under GitHub-actions (2 processes).
Between 2 and 3 months ago we started seeing spurious, non-reproducible failures where the worker (a pytest process) crashes without a stack trace. The test that crashes is always the same.
This only happens when running under this distributed pytest-xdist (pytest -nX). If you disable xdist those crashes never occur (pytest -n0).
At first I thought it could be an OOM error, because I could not reproduce them on my beefy workstation, but by running the tests several times I am able to reproduce the tests locally as well.
You can see here the test that is crashing https://github.com/netket/netket/runs/5978272340?check_suite_focus=true#step:8:5977
I tried to debug it for a while, and the culprit is a pretty large jai-compiled function.
Do you have any suggestion on what I could do to instrument jax to understand what is happening?

I suspect that means the job segfaulted, so it should dump a core if you raise the ulimit.
We don't have prebuilt wheels with symbols.
It would be most helpful if we can reproduce outside the github actions environment...

I don't have time this week but will try early next week.
Ill post here again.
Thanks for your help in the meantime.


Update show.sql to reflect macro name change

We renamed this macro in dbt-adapters and need to rename it here as well.
As a result dbt show some_table.sql --limit 1 will fail as this macro won't be picked up

Resolves #216


Create a zone that spans over multiple monitors?

Is it possible to create a zone that spans over multiple monitors? That would be great :-)

Hi @eliahreimers
not a present.
We have an issue to track the feature request https://github.com/microsoft/PowerToys/issues/480
Feel free to add your comments there or to up vote it.
I'm closing this one as duplicate.
Thanks.


Cheap Tuya temperature sensor support - offer to contribute

There are cheap Tuya temperature sensors like https://www.aliexpress.com/item/1005006010340464.html (can be had for $2 more often than not) - they have a well defined protocol, and if the deviceId/localKey is known, the advertisements can be decoded.
If there is willingness to include such feature in this library (given that the Tuya HHCCwhatever is already supported), I'd like to implement it and contribute it to this package & add the relevant code to the home assistant addon.

I think it would be better to create a copy of this package and release it as tuya-ble or something similar. This packages focusses on the devices that are sold as xiaomi brand and that mostly advertise with the xiaomi UUID (95FE). Some HHCC plant sensors were added, as these are sold under different names, including Xiaomi. There are also brands that use the MiBeacon format, which are included for that reason. But I think Tuya should get its own package, like we did for Qingping, Govee, etc. A copy of an existing package would be a good start, I guess. We can create create a repo under Bluetooth-Devices if needed.

Oh good, I didn't notice there were other libraries 🥲
I'll create a new repo under my user, then if the quality's good enough, it can be transferred to Bluetooth-Devices.
I guess the same goes for the home assistant component, correct?

Yep, each BLE brand has its own integration, but the integrations are mostly similar set up.


New Cel2 Version of Docs

Note: I did not review every page, relying on the other reviewers to have a look here.

remember that sweet bridging page @therealharpaljadeja made for the dango launch? lets bring that back


Indexeddb improvements

I accidentally delete my branch. This is the continuation of previous. I added a wire to make FS flushing available to database delegate. Further implementation of this will be in a new Drift PR.

Other libraries can go the same way and manage the file system for users too :D Thanks for your work on this and the drift PR!


Colours not showing up properly.

The color of the prompt is always like this no matter what theme I choose.

System specs
NAME="Ubuntu"
VERSION="18.04 LTS (Bionic Beaver)"
FLAVOUR="Kubuntu"

Could you record a .cast file with termtosvg --record and copy paste the content here?

Thanks. Your prompt uses a bold font so termtosvg displays it with bright colors (color8 to color15 of the color themes defined in ~/.config/termtosvg/termtosvg.ini). There are two solutions:

Tweak a color theme so that color8 through color15 look like what you want
If you don't care about bright colors, remove them altogether. For example you could add the following theme to the configuration file:

[solarized-dark-8]
foreground = #93a1a1
background = #002b36
color0 = #002b36
color1 = #dc322f
color2 = #859900
color3 = #b58900
color4 = #268bd2
color5 = #6c71c4
color6 = #2aa198
color7 = #93a1a1

and then call termtosvg --theme solarized-dark-8.
Let me know if it fixes your problem!

Great! It fixed my problem! Thanks!


new pre implementation

New implementation of pre function (Matlab version). Modifications happen in TransSyst.m and pre.m. The old implementation isn't removed. When fast_enabled flag in TransSyst class is on, the new implementation of pre is called automatically. Speed up the 'radiant' example 4 times on my computer.

Thanks! Didn't know my implementation was that slow. I removed the old one to keep the code cleaner.


ci: workflow all_plugins fails because of Flutter 3.0.0

See: https://github.com/firebase/flutterfire/runs/6398175458?check_suite_focus=true
Blocked by:

#8645
#8646
#8647
#8648
#8649

#8692 might be related

The all_plugins workflow should work again because all PRs got merged. Therefore, we can close this ticket.

The all_plugins workflow should work again because all PRs got merged. Therefore, we can close this ticket.


udpate integration test

We update example yaml last week (use new cluster version of yaml). As a result, primary-001 can not used alone for integration test since it will wait all clusters to start up. This pr update integration test to use single node primary easegress to do the test.
update github workflow, run test when change yaml file and shell file since these changes may fail integration test.

Codecov Report

Merging #442 (55b3192) into main (fb7d28c) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #442      +/-   ##
==========================================
- Coverage   80.44%   80.29%   -0.15%
==========================================
  Files          70       70
  Lines        8094     8094
==========================================
- Hits         6511     6499      -12
- Misses       1225     1239      +14
+ Partials      358      356       -2

Impacted Files
Coverage Δ

pkg/object/mqttproxy/session.go
74.23% <0.00%> (-7.98%)
:arrow_down:

pkg/object/mqttproxy/client.go
77.67% <0.00%> (-2.33%)
:arrow_down:

pkg/cluster/cluster.go
50.68% <0.00%> (+1.02%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks Samu, we may add that later. Test cluster may need some new tests. Objects like HTTPServer need open new port, so when run multiple easegress in one machine, some objects may failed to create.


generate base dotnet project files

DoD:

API base files generated
docker compose up should start a local development container of the API
add a README.md to help others to get started

Resources:

https://www.youtube.com/watch?v=PmDJIooZjBE

nice!


Test harness UI truncates output

When running a spec that fails due to a compilation error the UI truncates the tailing 3 lines of output.
Inspecting the requests shows the full output being returned from the /test endpoint, which points to this being potentially UI related.
As an example, a test run may end in:
There was a problem expanding macro 'macro_139728512985392'
Called macro defined in macro 'args'
 2 | {% if true %}
Which expanded to:
    1 |
    2 |     {
    3 |
    4 |         a: a
    5 |
 >  6 |         b: b
    7 |
    8 |     }
    9 |
Error: expecting token '}', not 'b'

This then renders as:
There was a problem expanding macro 'macro_139728512985392'

Called macro defined in macro 'args'

 2 | {% if true %}

Which expanded to:

    1 |
    2 |     {
    3 |
    4 |         a: a
    5 |
 >  6 |         b: b
    7 |

@MrYuion  whats the status here?

This should be fixed in the latest build.
Let me know if this is still an issue.


API Keys

Description
Please provide as much info as you can.
Where can I find all the needed api keys and how?

https://github.com/Dev-CasperTheGhost/ghostybot/blob/master/docs/INSTALLATION.md#requirements

Yea, use the list Xaliks posted in the message above, its not 100% up to date tho


added note about docker tag command

Just a note in case the user is using your own image, they have to tag the image with de registry url.

@EltonAlvess : Thanks for your contribution to the Azure documentation! The author, @SyntaxC4, has been notified to review your proposed change.

@EltonAlvess : Thanks for your contribution to the Azure documentation! The author, @SyntaxC4, has been notified to review your proposed change.

#sign-off


An internal DaCe error was encountered! (DaCe request failed with code 500)

I'm working under Windows and using remote-SSH to remotely connect to my Linux server.
After installing DaCe-vscode on the server, there is an error. Does this affect the use of DaCe-vscode?
An internal DaCe error was encountered! (DaCe request failed with code 500)

Thanks for reporting! We would need more context to understand the issue. If you open the terminal pane, a tab called "SDFG Optimizer" should be open there. What is the output log there?

The output log is:
/home/chenzhiqiang/anaconda3/bin/python /home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py -p 9604
(base) chenzhiqiang@GEM02_GPU:~/code-repo/dace$ /home/chenzhiqiang/anaconda3/bin/python /home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py -p 9604
 * Serving Flask app "DaCeInterface" (lazy loading)
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: off
[2021-08-19 20:32:14,727] INFO in _internal:  * Running on http://<IP_ADDRESS>:9604/ (Press CTRL+C to quit)
[2021-08-19 20:32:16,320] INFO in _internal: <IP_ADDRESS> - - [19/Aug/2021 20:32:16] "GET / HTTP/1.1" 200 -
[2021-08-19 20:32:16,337] ERROR in app: Exception on /get_metadata [GET]
Traceback (most recent call last):
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py", line 285, in _get_metadata
    return get_property_metdata()
  File "/home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py", line 67, in get_property_metdata
    from dace.sdfg.nodes import full_class_path
ImportError: cannot import name 'full_class_path' from 'dace.sdfg.nodes' (/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/dace/sdfg/nodes.py)
[2021-08-19 20:32:16,341] INFO in _internal: <IP_ADDRESS> - - [19/Aug/2021 20:32:16] "GET /get_metadata HTTP/1.1" 500 -
[2021-08-19 20:32:16,465] INFO in _internal: <IP_ADDRESS> - - [19/Aug/2021 20:32:16] "POST /transformations HTTP/1.1" 200 -
[2021-08-19 20:32:39,148] ERROR in app: Exception on /get_metadata [GET]
Traceback (most recent call last):
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py", line 285, in _get_metadata
    return get_property_metdata()
  File "/home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py", line 67, in get_property_metdata
    from dace.sdfg.nodes import full_class_path
ImportError: cannot import name 'full_class_path' from 'dace.sdfg.nodes' (/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/dace/sdfg/nodes.py)
[2021-08-19 20:32:39,150] INFO in _internal: <IP_ADDRESS> - - [19/Aug/2021 20:32:39] "GET /get_metadata HTTP/1.1" 500 -
[2021-08-19 20:32:39,152] ERROR in app: Exception on /get_metadata [GET]
Traceback (most recent call last):
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py", line 285, in _get_metadata
    return get_property_metdata()
  File "/home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py", line 67, in get_property_metdata
    from dace.sdfg.nodes import full_class_path
ImportError: cannot import name 'full_class_path' from 'dace.sdfg.nodes' (/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/dace/sdfg/nodes.py)
[2021-08-19 20:32:39,155] INFO in _internal: <IP_ADDRESS> - - [19/Aug/2021 20:32:39] "GET /get_metadata HTTP/1.1" 500 -
[2021-08-19 20:32:53,336] ERROR in app: Exception on /get_metadata [GET]
Traceback (most recent call last):
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py", line 285, in _get_metadata
    return get_property_metdata()
  File "/home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py", line 67, in get_property_metdata
    from dace.sdfg.nodes import full_class_path
ImportError: cannot import name 'full_class_path' from 'dace.sdfg.nodes' (/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/dace/sdfg/nodes.py)
[2021-08-19 20:32:53,338] ERROR in app: Exception on /get_metadata [GET]
Traceback (most recent call last):
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py", line 285, in _get_metadata
    return get_property_metdata()
  File "/home/chenzhiqiang/.vscode-server/extensions/phschaad.sdfv-1.0.3/backend/run_dace.py", line 67, in get_property_metdata
    from dace.sdfg.nodes import full_class_path
ImportError: cannot import name 'full_class_path' from 'dace.sdfg.nodes' (/home/chenzhiqiang/anaconda3/lib/python3.8/site-packages/dace/sdfg/nodes.py)
[2021-08-19 20:32:53,340] INFO in _internal: <IP_ADDRESS> - - [19/Aug/2021 20:32:53] "GET /get_metadata HTTP/1.1" 500 -
[2021-08-19 20:32:53,342] INFO in _internal: <IP_ADDRESS> - - [19/Aug/2021 20:32:53] "GET /get_metadata HTTP/1.1" 500 -

Is that the information you need?

Yes, this helps! To check if it's a compatibility issue with releases, could you please try using the latest master branch of dace ?
To do so, please run pip install --upgrade git+https://github.com/spcl/dace

Yes, it is a compatibility issue with releases.
Probably because of the network, I can't run the command you provided, the error message is:
(base) chenzhiqiang@GEM02_GPU:~$ pip install --upgrade git+https://github.com/spcl/dace
Collecting git+https://github.com/spcl/dace
  Cloning https://github.com/spcl/dace to /tmp/pip-req-build-hvxwslps
  Running command git clone -q https://github.com/spcl/dace /tmp/pip-req-build-hvxwslps
  git: 'remote-https' is not a git command. See 'git --help'.
WARNING: Discarding git+https://github.com/spcl/dace. Command errored out with exit status 128: git clone -q https://github.com/spcl/dace /tmp/pip-req-build-hvxwslps Check the logs for full command output.
ERROR: Command errored out with exit status 128: git clone -q https://github.com/spcl/dace /tmp/pip-req-build-hvxwslps Check the logs for full command output.

I downloaded the master branch source code for dace, python setup.py install, Installation via source code.
After successfully updating the dace, this problem did not appear anymore.

Thank you for the update! We will make a release very soon and set the required version so that this does not happen again.


Disstrack Bye Pewdiepie by Carryminati unsupported.

Check if issue with swaglyrics or whether song lyrics unavailable on Genius.

stripper -> Carryminati-Disstrack-Bye-Pewdiepie
version -> 0.2.6

I'm not sure I want to fix this but lyrics don't exist for this lol


strongswanNM: 5.9.7 -> 5.9.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/strongswan/strongswan/releases.
meta.description for strongswanNM is: "OpenSource IPsec-based VPN Solution"
meta.homepage for strongswanNM is: "https://www.strongswan.org"
meta.changelog for strongswanNM is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.9.8 with grep in /nix/store/fjazxbilmnjs3ik8sb3wnwzgpmsvnljk-strongswan-5.9.8
found 5.9.8 in filename of file in /nix/store/fjazxbilmnjs3ik8sb3wnwzgpmsvnljk-strongswan-5.9.8

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
gnome.networkmanager-l2tp
networkmanager-l2tp
networkmanager_strongswan
strongswan
strongswanNM
strongswanTNC

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fjazxbilmnjs3ik8sb3wnwzgpmsvnljk-strongswan-5.9.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A strongswanNM https://github.com/r-ryantm/nixpkgs/archive/ac864f6d7c4642f0dbde4e8ee26350e989dd1ed6.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fjazxbilmnjs3ik8sb3wnwzgpmsvnljk-strongswan-5.9.8
ls -la /nix/store/fjazxbilmnjs3ik8sb3wnwzgpmsvnljk-strongswan-5.9.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

5 packages built:

networkmanager-l2tp (gnome.networkmanager-l2tp)
networkmanager_strongswan
strongswan
strongswanNM
strongswanTNC

Maintainer pings

@ofborg build networkmanager-l2tp networkmanager_strongswan strongswan strongswanNM strongswanTNC


Bracket '>' on new line not parsed as part of HTML-tag, but as Block Quote instead

Example:

some content here

When option Parser.BLOCK_QUOTE_INTERRUPTS_PARAGRAPH is set to true (=default), the parser prefers parsing a block quote instead of greedily include it in the opening HTML-tag:

<span
class="abc"
data-role="example"

some content here

@Elmervc, thank you for reporting this. I will add logic to not let block quotes interrupt active HTML blocks.

@Elmervc, I took a look at the code and realized that this detection would be problematic.
Block parsing is done before inline parsing. Therefore at the time a block quote prefix is detected, the paragraph text is not parsed for inline HTML. Therefore it is not possible to with the CommonMark architecture to correctly parse dangling closing > of an HTML tag.
The only workarounds are to either disable block quotes interrupting paragraphs, requiring block quotes to be at the start of a block or to not leave dangling > at the start of a line.

Thanks for investigating. Not sure how HTML-block's (not the inline ones) are parsed. But I understand that this cannot be done easily before block-quote parsing. We go for disabling  the "block quote interrupting paragraphs" option now.

@Elmervc, fixed in pegdown compatibility mode with deep html parsing. 0.22.2


Validation errors don't disappear for valid data.

Hi all,
When using your kit in my app with fluentvalidation the red rectangle for the textbox don't disappear when i update the value in a valid way.
The only way to update the view is change the active tab and return to the previous one.
Repro steps:

Setup a view with a textbox with validation properties like this:

<TextBox Margin="12,101,8,0" TextWrapping="Wrap" VerticalAlignment="Top" Text="{Binding Ospite.TipoDocumento, Mode=TwoWay, UpdateSourceTrigger=LostFocus,NotifyOnValidationError=True,ValidatesOnDataErrors=True}" Grid.Column="1" Height="26" />

Enter some text that will fail validation
Trigger LostFocus by switching to the next input

Expected Behaviour: validation messsage appears
Actual Behaviour: only validation highlight around box appears
When switching between tabs, the validation message returns.
I've added two picture displaying the wrong state and the right state (after the tab switch)

Thanks


Fix  allow the use of secret names with capital letters

Vanguard Secrets Manager secret names have capital letters which are illegal in the ref field of K8s environment variables. The changes made are to allow the use of secrets with names containing capital letters by aliasing them first with lowercase aliases.

@CodiumAI-Agent /improve

PR Code Suggestions ✨
No code suggestions found for the PR.

@CodiumAI-Agent /review

PR Reviewer Guide 🔍
Here are some key observations to aid the review process:

⏱ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review
Template ExpansionEnsure that the Helm template expressions (e.g., {{ .Values.secretsStore.aws.dbCredsSecretName }}) are correctly expanded and validated in different deployment environments to prevent runtime errors.
Permissions ExpansionThe addition of 'patch' and 'list' verbs to the Kubernetes RBAC rules should be reviewed to ensure they are necessary and do not grant excessive permissions.

Persistent review updated to latest commit https://github.com/Multiwoven/helm-charts/commit/05b9db4e458bb64b22c1202db6c364baac97298a


linuxManualConfig: add pahole to moduleBuildDependencies

Description of changes
This seems to be needed for out-of-tree module builds since #221707. We do not yet understand why, but this will unblock the channels while we figure it out.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@OfBorg build linuxPackages.zfs

@ofborg build linuxPackages.zfs

Just want to state that I'm very grateful for your work on this!


Support for Unity's Localization

Is your feature request related to a problem? Please describe.
Unity has packages in preview that will provide a standard way to deal with localization of strings and assets, link. It also discusses a standard way of localizing the text of editor tools, link
Describe the solution you'd like
Once these are stable, it warrants investigation to determine if Fungus' existing Localization can be switched to be backed by Unity's package, or if the Fungus, localization can be removed entirely.
On the Editor side, we presently don't offer any localization and we should. We would have to rely on community or funding to do the translation work itself though.

@stevehalliwell  Quick question; would it be possible that, since the aforementioned Unity localisation package is not ready for production anytime soon, to extend the Localization.cs script instead, to have an extra option to pull all text from canvases and such from the scene, just like it pulls strings from flowcharts in the scene, and export it into a separate csv? I thought this is what the "Export Standard Text" button did at first, when I saw it (I know it doesnt).
So when you click the "export non-Fungus text" button lets say, it pulls all the non-Fungus text from the scene and offers you to export it into a csv file.

To add to my previous post; I'd like to better understand your original post/proposition; would this task involve to find a solution to support localising the non-Fungus text in games with the new official Localization package, or are you proposing to make all of Fungus be compatible with the Localization package instead of the localisation features it's currently using?


Display URLs in integration test logs

Before:
[Macaron] 2017-06-09 15:56:01: Started POST  for
[Macaron] 2017-06-09 15:56:01: Completed  302 Found in 43.305422ms

After:
[Macaron] 2017-06-09 15:56:01: Started POST /user/login for
[Macaron] 2017-06-09 15:56:01: Completed /user/login 302 Found in 43.305422ms

Will help in debugging, such as figuring out what caused https://drone.gitea.io/go-gitea/gitea/3457 to fail

LGTM

LGTM

make L-G-T-M work


Provider: Checkout

a checkout plan where you can build your own custom plans (Internet, TV and Telephone).

[x] Select the Internet Plan
[x] Select the Internet addon
[x] Select the TV plan
[x] Select the TV addons (E.G. HBO)
[x] Select the Telephone plan
[x] Select the Telephone addons

Complete the Checkout With Promotion Logic


HIVE-26227: Add support of catalog related statements for Hive ql

What changes were proposed in this pull request?
Implement the ddl statements related to catalog, the statements can refer to HIVE-26227.
Why are the changes needed?
To support basic ddl operation for catalog through Hive ql.
Does this PR introduce any user-facing change?

Yes, we should add these new statements to DDL Document.
How was this patch tested?

Add a qtest catalog.q, can be test by command:
mvn test -Dtest=TestMiniLlapLocalCliDriver -Dqfile=catalog.q

@pvary @deniskuzZ: Could you also review this PR?

A very great work! it's much easier for us to manage catalogs with DDL.

As iceberg REST catalogs are popular it would make sense to add a catalog type as well like HIVE_CATALOG, ICEBERG_REST_CATALOG etc. When adding  a new catalog this must a be mandatory.

@zratkai Does the catalog name meet the requirements? For example, hive corresponds to HIVE_CATALOG, and iceberg_rest corresponds to ICEBERG_REST_CATALOG.


Clojure classpath

We want a settable classpath and dynamic classpath, shared with the other JVM languages.

Classpath and imports are available to set up in the Clojure plugin settings


[SPARK-43476][SPARK-43477][SPARK-43478][PS] Support StringMethods for pandas 2.0.0 and above

What changes were proposed in this pull request?
This PR proposes to support StringMethods for pandas 2.0.0 and above.
Why are the changes needed?
Support the latest pandas for pandas API on Spark.
Does this PR introduce any user-facing change?
StringMethods.split, StringMethods.rsplit and StringMethods.replace is available with the latest pandas.
How was this patch tested?
Enabling the existing UTs.

merged to master


lib文件增加了12兆

很多.so文件,apk体积大大增加

English !

Adding this dependency results in an increase in apk volume.

@superqiao just read the doc:
https://github.com/barteksc/AndroidPdfViewer#why-resulting-apk-is-so-big

Adding any thing to the basic compile will increase the apk size.... its like a shopping bag you put bits in it and it will get heavier !


Ease of Setup improvements

Pull Request
Description
This issue includes some feature improvements to tackle issues I encountered when trying to run the module for the first time.
This is my first contribution to this code base so happy to get feedback on code style or any other aspect of my change
Features

Adds details of required environment variables required for running unit tests
Removes duplication of the folder name "latest" in utils by creating a function get_latest_subdir_path to retrieve the latest sub-folder name
In the new function to retrieve the latest sub-folder name, allows optionally for the latest sub-folder to be created if it doesn't already exist. This is used in the existing function move_older_files_to_different_location which requires the folder to exist
.gitignore update to cover files and folders created by the tool

How Has This Been Tested?

A unit test has been added for get_latest_subdir_path and run locally
Ran all unit tests locally to confirm they passed
Re-Ran the instructions "Downloading EUMETSAT Data" to ensure the latest folder was created automatically

No Changes to data processing
Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

@all-contributors please add @TomPughe for code


Are certain Object.create patterns supported?

I tend to use certain patterns for prototypical inheritance using Object.create.  I've spent several days trying to make this work and the only way I can do it, so far, is by casting through "any".  Is there a better way?

//@flow
let parent = {
	foo: 1,
	extend: function<T: Object>(args: T): {...typeof parent, ...T} {
		let child = Object.create(this, args);
		return (child: {...typeof parent, ...T});
	}
};
let child = parent.extend({
	bar: 2
});
child.bar;
// desired error
child.baz;
let grandkid = child.extend({
	baz: 3
});
grandkid.baz;
// undesired error
grandkid.bar;

flow.org/try

If you want the chaining behavior, use class inheritance instead.


ModuleNotFoundError, can't import modelci

Software and Hardware Versions
Ubuntu 18.04.5 LTS
CUDA 10.1
Docker version 20.10.5
Problem description
I follow the steps according to CONTRIBUTING.md and faced with the folllowing error.
(modelci) root@ML:/home/pengzhang/Learn/ML-Model-CI-master# python3.8 modelci/cli/init.py service init
Traceback (most recent call last):
File "modelci/cli/init.py", line 18, in
from modelci.cli import modelhub
ModuleNotFoundError: No module named 'modelci'
Steps to Reproduce the Problem
conda env create -f environment.yml
pip install torch==1.7.1 torchvision==0.8.2 torchaudio==0.7.2
pip install pytorch-lightning==0.8.5
pip install hummingbird-ml==0.2.1 torchviz==0.0.1
export PYTHONPATH="${PWD}"
python modelci/cli/init.py service init
Expected Behavior
Start Services
Other Information
I tried to check the environment variables, but I still cannot import modelci

Successfully resolved


Python-derived models cause a segmentation fault when compiled in multi-threading mode

When compiled with multi-threading enabled, deriving models in Python causes a segmentation fault unless nthreads=1 is set explicitly before calling solve().  This makes sense, but we should have a user-friendly way of not segfaulting.
Case to reproduce:
import crocoddyl
import numpy as np
from crocoddyl.utils import UnicycleModelDerived

if __name__ == "__main__":
    N = 200  # number of nodes

    def createProblem(model):
        x0 = np.matrix([[1.], [0.], [0.]])
        runningModels = [model()] * N
        terminalModel = model()
        xs = [x0] * (N + 1)
        us = [np.matrix([[0.], [0.]])] * N

        problem = crocoddyl.ShootingProblem(x0, runningModels, terminalModel)
        return xs, us, problem

    # Crashes
    xs, us, problem = createProblem(UnicycleModelDerived)
    problem.nthreads = 2
    # Fine as long as nthreads=1
    #xs, us, problem = createProblem(crocoddyl.ActionModelUnicycle)
    # problem.nthreads = 1
    solver = crocoddyl.SolverBoxDDP(problem)
    solver.solve()

I am aware of this issue from a long time, but without a clear solution.
To be more concrete, Crocoddyl doesn't yet support multi-threading in python-derived code (action, residuals, cost models, etc).
It seems we need to manage the Python GIL using boost python. Honestly, I have tried different solutions, but still not clear how to do it.
If someone has a better understanding on how tackling this issue, then please go ahead or comment here.

I'm aware that this is a limitation and a "nice to have" to be solved. However, as long as it segfaults, it's a bug. If we can simply show an error message instead, it's fine to leave it as an enhancement. I don't have a solution for it right now though either :-/

I'm aware that this is a limitation and a "nice to have" to be solved. However, as long as it segfaults, it's a bug. If we can simply show an error message instead, it's fine to leave it as an enhancement. I don't have a solution for it right now though either :-/

Agreed. We could sort out the segfault. One solution is to change (behind the scene) the number of threads to 1 when Python-derived code is in place. Do you have another proposition?

I think this post is relevant for anyone reading this thread: https://stackoverflow.com/questions/18618333/boost-python-wrap-functions-to-release-the-gil


Installation could not be completed normally.

Describe the bug
Although the console print done, when I reboot the machine in virtual machine, I could not enter the system correctly. It still shows the installation interface.
To Reproduce
This is my shell record: https://fars.ee/t/9A05.json
Expected behavior
It should enter the system correction.
System information
My nix config
https://github.com/god464/NeoDotfile

This looks like you aren't using kexec where, but booted an installer on the target machine before you ran nixos-anywhere. Is this correct?
If so you need to remove the installer medium (i.e. image attached to vm) yourself or configure the target machine to boot from the disk where nixos gets installed first.
If not, we'd need to check whether the machine actually uses EFI to boot, as the config wouldn't be compatible with legacy/MBR boot as far as i see from a quick glance.

Of course I'm not using kexec. But I don't think that's the main reason because of the logs that appeared during the installation process:
File system "/dev/sda1" has wrong type for an EFI System Partition (ESP).
Traceback (most recent call last):
  File "/nix/store/zqbxnxdp110mvq7ck614l0x2hjh18wzr-systemd-boot", line 341, in <module>
    main()
  File "/nix/store/zqbxnxdp110mvq7ck614l0x2hjh18wzr-systemd-boot", line 258, in main
    subprocess.check_call(["/nix/store/41d6dz6cj7kj52gfmbyqwhp4hkvbrmfg-systemd-253.6/bin/bootctl", "--esp-path=/boot"]
+ bootctl_flags + ["install"])
  File "/nix/store/jhflvwr40xbb0xr6jx4311icp9cym1fp-python3-3.10.12/lib/python3.10/subprocess.py", line 369, in check_ca
ll
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/nix/store/41d6dz6cj7kj52gfmbyqwhp4hkvbrmfg-systemd-253.6/bin/bootctl', '--esp
-path=/boot', 'install']' returned non-zero exit status 1.

I had this same error when trying to install with only  flake + disko, and I checked to make sure the install mode was under UEFI.

Ah, it helps "of course" to mention the actual error instead of expecting people to skim the recording above.

File system "/dev/sda1" has wrong type for an EFI System Partition (ESP).

Which looks quite clear to me, your disko config doesn't set a "type" for the ESP partition, e.g. https://github.com/nix-community/disko/blob/4015740375676402a2ee6adebc3c30ea625b9a94/example/simple-efi.nix#L11

It look good now. Thank you.


Web - Chat - BackHandler console error shows up when refreshing the page

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

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

Navigate to staging.new.expensify.com
Log in.
Refresh the page.

Expected Result:
No console error shows up.
Actual Result:
BackHandler console error shows up when refreshing the page.
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

This is an issue in react-navigation/native that is fixed in this version
It should go away when we update react-navigation/native

closing this out in that case


Error while exporting to an existing spreadsheet with existing pivot tables

I have an existing xlsx file that contains various pivot tables and charts based on data that I would like to update using Export-Excel.  I'm using this file as a template file of sorts; it contains the complex tables/charts based off of data in the Chassis worksheet and I don't have to try and recreate/organize the tables/charts every time - only refresh the data.
While trying to export an array of chassis information to this xlsx file, I receive the following error:
$chassisArray  | Export-Excel -Path $existingFile -worksheetname Chassis -BoldTopRow -AutoSize -Debug
Exception calling "Save" with "0" argument(s): "Error saving file C:\temp\test.xlsx"
At line:120 char:13
+             $pkg.Save()
+             ~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : InvalidOperationException

If I remove the worksheet that contains the pivot tables, the export is successful, however this defeats the purpose of leveraging a template file.

Thanks for reporting the issue. The underlying library that supports exporting to the xlsx has options for working with templates. I have not looked at how to expose that through the PowerShell interface. Could you post that and I can investigate the level of effort (and viability) to make that work.

Can you clarify what you'd like posted?

Optimally, the xlsx you use as a template, plus the data and how you want to export.


PowerShellStandard library nuget package name is problematic

the current name of the standard library has some issues with regard to the upcoming PowerShell 7 release. We should consider renaming the library to PowerShellStandard.Library.7.0.0 to show the terminus rather than the initial supported version, especially in the light that PowerShell 6 will not remain supported long term.

So the library name will have the end version and then the version will be the initial version? Am I understanding that right?
Or will we version based on the ending version? if so, what's the initial version?

PowerShell Standard is intended support a subset of Windows PowerShell and PowerShell Core versions.
In the past, the version of Standard (3.0 and 5.1) was tied to the floor version in the pair (Windows PowerShell). Given that all future version of Standard will only guarantee compatibility back to 5.1, that version nomenclature makes no sense anymore.
And given that new APIs are being added back to 7.x, we want to use the version now to say "this works with Windows PS 5.1 and PowerShell Core " (in this case 7.0).
In the future, a Std 7.1 would guarantee support with 5.1 and 7.1


Azure Alerts for Errors - 4xx status codes

Story
As a developer, I need CDC-TI 4xx status codes to be alerted on after a threshold, so that we can begin an incident.
Pre-conditions

[ ] Assumptions of prior or future work that's out of scope for this story

Acceptance Criteria

[x] 4XX alerts are configured for all non-pr environments and have been tested in the TI application in a deployed environment
[x] 4XX alerts are configured for all non-pr environments and have been tested in the RS SFTP ingestion service in a deployed environment

Tasks

[x] Set up 4xx Azure Alerts in the TI Application in all non PR environments

[x] Test 4xx azure alert in TI.

[x] Set up 4xx Azure Alerts in the RS SFTP Ingestion Service in all non PR environments

[x] Test 4xx azure alert in TI.
[x] Make sure to remove test400 endpoint from the RS SFTP Ingestion Service

Research

[ ] Research work needed to complete the story
[ ] Foundational: High-level research that will support this and future efforts

Engineering

[ ] Engineering work needed to complete the story
[ ] Foundational: Technical runway work to support this and future efforts

Definition of Done

[ ] Documentation tasks completed

[ ] Documentation and diagrams created or updated

[ ] ADRs (/adr folder)
[ ] Main README.md
[ ] Other READMEs in the repo
[ ] If applicable, update the ReportStream Setup section in README.md

[ ] Threat model updated
[ ] API documentation updated

[ ] Code quality tasks completed

[ ] Code refactored for clarity and no design/technical debt
[ ] Adhere to separation of concerns; code is not tightly coupled, especially to 3rd party dependencies

[ ] Testing tasks completed

[ ] Load tests passed
[ ] Additional e2e tests created
[ ] Additional RS e2e assertions created in the rs-e2e project for any new transformations.  Includes improvements to the assertion code required to make the new assertions

[ ] Build & Deploy tasks completed

[ ] Build process updated
[ ] API(s) are versioned
[ ] Feature toggles created and/or deleted.  Document the feature toggle
[ ] Source code is merged to the main branch

Research Questions

Optional: Any initial questions for research

Decisions

Optional: Any decisions we've made while working on this story

Notes

This story came out of #1379, and #1380.

For this alert I believe we need the following:

A metric alert (Azure monitor has metrics that monitor http status codes)
The alert is stateful (See: https://learn.microsoft.com/en-us/azure/azure-monitor/alerts/alerts-overview#types-of-alerts)
For the interval and lookback period I'm not 100% sure yet.

We attempted creating a list of variables for the lifecycle ignore changes block.
cdc_tags =
set(
    "business_steward",
    "center",
    "environment",
    "escid",
    "funding_source",
    "pii_data",
    "security_compliance",
    "security_steward",
    "support_group",
    "system",
    "technical_steward",
    "zone"
 )
  ignore_tags = [for tag in cdc_tags : "tags[\"${tag}\"]"]

With
lifecycle {
    ignore_changes = local.ignore_tags
  }

And we get the following error
 Error: Invalid expression
│
│   on ../../template/alert.tf line 111, in resource "azurerm_monitor_metric_alert" "azure_4XX_alert":
│  111:     ignore_changes = local.ignore_tags
│
│ A static list expression is required.

We'll revert back to the old lifecycle ignore changes list as there doesn't seem to be a better way to do this.
We could add it to the provider of azurerm but that would ignore all the tags in every azure resource and I think we use those tags in some resources.

The PR was merged and the alert was tested in the internal environment. The alert successfully fired and resolved itself correctly as well.

The DoD isn't done.  Moving back into Review column.

Created a new ticket to triage 4xx alerts popping up and going through DoD now to make sure this card is completed.


[models] gltf animation not animating correctly

here's the model
walk.glb.tar.gz
basically loaded and displayed as per examples/models/models_gltf_animation.c
It looks like the rotations are messed up somehow?
This was originally a download from cgtrader as a blend file which I then exported to gltf, I'm going to relook in blender as it could be just a case of some setting their...

i also took a walk cycle from mixamo.com exported as dfx and dea without luck :(

This issue seems a duplicate of #1799. I'm closing it.


[6.5.x] GUVNOR-2679 : GSS Custom changes to pom.xml are being overwritten (#463)

RHBPMS-4190 : GSS Custom changes to pom.xml are being overwritten
(cherry picked from commit 7229b74cf64702f2ab3ed0bb706be8eae9e13467)
https://issues.jboss.org/browse/RHBPMS-4190
https://issues.jboss.org/browse/GUVNOR-2679
Had to do some adjustments, mainly to make the tests fit 6.5.x branch.

@karreiro @manstis Please review when you have time.

Hmm, this might be missing few files. Rechecking.

Ok. All in.

@Rikkola you have a failing test - but it is fixed by https://github.com/droolsjbpm/kie-wb-common/pull/476/commits/c52590c3a4ba06dc35890623fb58a251706063f5 (in one of my pending 6.5.x PRs).

jenkins retest this.

+1


pipeline: record accepted files in database table

Codecov Report
Attention: 15 lines in your changes are missing coverage. Please review.

Comparison is base (6dbedc8) 48.91% compared to head (a6d225d) 49.20%.
Report is 3 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #213      +/-   ##
==========================================
+ Coverage   48.91%   49.20%   +0.29%
==========================================
  Files          87       89       +2
  Lines        4042     4093      +51
==========================================
+ Hits         1977     2014      +37
- Misses       1744     1755      +11
- Partials      321      324       +3

Files
Coverage Δ

internal/environment.go
50.00% <100.00%> (+0.41%)
:arrow_up:

internal/pipeline/pipeline.go
61.90% <100.00%> (ø)

internal/files/mock_repo_files.go
0.00% <0.00%> (ø)

internal/files/repo_files.go
87.50% <87.50%> (ø)

internal/pipeline/file_receiver.go
53.33% <53.33%> (+0.20%)
:arrow_up:

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