Load css files for each app

Load css files per each app in angular - like js files.

Can we then split up these massive app.js and projector.js? I would like this much..

Can we then split up these massive app.js and projector.js?

You mean *.css... Yes, we should do it for 2.2

We should setup Sass by the way. But first we should think about #3011 because jspm has also a css loader plugin which can be used easily.

Why using a css loader, if we can just concat all css together?
gulp.task('css', function () {
    return gulp.src([
            path.join('openslides', '**', 'static', 'css', '**', '*.css'),
        ])
        .pipe(concat('openslides.css'))
        .pipe(gulp.dest(path.join(output_directory, 'css')));
});

@FinnStutzenstein I think we can close this issue because #3507 is merged, right?


Client Allocation Request for: Maxius

Client Allocation Questions
Core Information

Name: Maxius
Website / Social Media: www.taejin.co.kr
Region: Asia excl. Greater China
DataCap Requested: 32TiB
Addresses to be Notarized: f12ojuuh62mdjadkox6ccgre5tw7coq2ydoc2vmwa
Notary Requested: MasaakiNawatani

@IsTandER Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Closing this duplicate issue. Please refer to #117


Restrict who can create apps when using SSO

Describe the solution you'd like
I would like to have an option to restrict app creation when SSO is enabled via Active Directory. The options would be set on Organisation. I would also like to an ability to create app creation under users when they are authenticated via SSO. Ideally all of this would be LDAP role based.
Describe alternatives you've considered
Unfortunately, there are no alternatives, as Admins, Collaborators and Members all can create apps.
Additional context
We are migrating from HockeyApp to AppCenter and although I'm an admin of our Organisation there are apps created by users that I cannot see. The presence of the same apps, but in different buckets also causes confusion as neither AppCenter nor HockeyApp check if the bundleId/packageId are unique.

Hello! Thanks for your feature request.
We re-tested our funcionality and this is how it's working:

I created an organization and invited a userA as Member
userA accepted the invitation and created a new App in the Org
I logged in back as myself and was able to see the app created by userA inside the Org

If a user created the app outside of the org, then you as an org admin, would not bar able to view the app.
We can take a deeper look if you reach out to us via support (blue chat button in the App Center portal) and provide us with some additional data about your apps/orgs. If you do, please make sure to refer to this GitHub issue.
Thank you,
Blanca


Incorrect URL under Contributing in README.md File

Describe the bug
When I clicked on the issues hyperlink under Contributing, I got a 404 not found error.
To Reproduce
Steps to reproduce the behavior:

Go to 'README.md'
Click on 'issues' under Contributing
See the error

Expected behavior
Discord-js issue page should show.

Please Assign me.
@TheLearneer @MrKrishnaAgarwal

The link works completely fine to me. 🤔

The link is absolutely fine 😬

The issue url under Contributing is:- https://github.com/HackSquadDev/discord-js/blob/main/issues
but URL should be:-
https://github.com/HackSquadDev/discord-js/issues
@TheLearneer @MrKrishnaAgarwal

@AkashSharma001 Thank you for your concerns, but we are not looking forward for any improvements to the docs as of now 🙇

Okay.
@TheLearneer


apollo3.5编译成功但是打不开localhost：8888

@zhuyingjie1 please refer to this document. Also, read through some of the previous issues for recommendations. Thanks!

Closing this issue. Please continue the discussion on #7147


Can’t Enable Plugin on iOS14 Device

Hello, I’ve installed the plugin (v1.1.5) on an iPhone SE 2020 on iOS14.8, and get an error whenever I try to enable the plugin. I’ve tried on a brand new vault & still have the same issue. Is this an expected incompatibility?

What is the error exactly? I think backwards compatibility for legacy OS-versions is generally out of scope for this project.

Not sure if there’s a debug mode where I can get more info, but here’s a video of what happens: https://d.pr/v/Xre758

Not sure if there’s a debug mode where I can get more info, but here’s a video of what happens: https://d.pr/v/Xre758

Found the dev tools plugin and managed to catch this error


fix(crud-update): use new state constructor instead of the old state

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
Update use's the old state constructor instead of the new state constructor.
What is the new behavior?
Update use's the new state constructor.

:tada: This PR is included in version 1.12.4 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Search Dropdown - Doesn't exist behaviour

It would be very useful to be able to set behaviour for dropdown searches where a searched value does not exist. such as ui dropdown search addable then the searched value would appear as an option that can be clicked and this would add the clicked value as a select option and select it.

I don't believe this use case is general enough to be added. You can modify noResults message by updating $.fn.search.settings.templates.message.
I do however think though there should be a onNoResults callback that fires only when no results are returned.

Oops misread this to be about ui search
ui search dropdown already has onNoResults callback. This can be used to attach behavior like you are describing.


Tooltip does not update

A tooltip does not update when the directive's smDirTooltip attribute changes.
Perhaps the directive could use ngOnChanges to update the data-tooltip attribute?

For anyone experiencing the same problem: I found a solution by using attr.data-tooltip="" instead. This works when the tooltip needs to reflect dynamic changes

@flemeur I believe your solution would defeat the purpose of the directive itself, if you set the data properties directly.
I would image that having the smDirTooltip and smDirPosition as setters instead of class properties would be a good solution to this problem.  For instance:
@Input()
set smDirTooltip(toolTip: string) {
    this.element.element.nativeElement.setAttribute("data-tooltip", toolTip);
}

@Input()
set smDirPosition(position: string) {
    this.element.element.nativeElement.setAttribute("data-position", position);
}

And you could have some extra logic for defaults much like is already in the code.  Just a suggestion.

@tspayne87 That would definitely be a good solution. I only mentioned my temporary "solution" because others might find it useful, until a proper fix is implemented :)


Frozen screen on Redhat 7.7 using version 1.55.2

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.55.2 and 1.5.7
OS Version: Redhat 64 bit 7.7

Steps to Reproduce:
1.install vscode from rpm
2.code --disable-extensions
3. vscode window opens, but no operations on the screen work. Can not expand files, can not select from menus.
Note do not have this problem with version vscode 1.5.2

Can you see if it still reproduces when disabling the GPU support in VSCode? You can do that by running VSCode from the command line like this: code --disable-gpu
If that helps, you can permanently configure this option via:

Command Palette search for "Configure Runtime Arguments"
Set disable-hardware-acceleration: true
Restart

tried running with the code --disable-gpu option and no change. When I try to close the window I get a message visual studio code is not responding. (not unique to running with --disable-gpu). Have also tried running with extensions disabled. I had read on stack trace that as of 1.53 centos 7 was not supported. I was wondering if this also applies to redhat.

Merging this to https://github.com/microsoft/vscode/issues/115784


Added a coding style

Kotlin and Tests area are still very small, but we should add more rules as we go


[SPARK-50106][PYTHON] Upgrade protobuf Python package to 5.28.3

What changes were proposed in this pull request?
This PR aims to upgrade protobuf Python package from 4.25.1 to 5.28.3 for Apache Spark 4.0.0 on February 2025.
Why are the changes needed?
To bring the latest bug fixes and support Python 3.13 which requires grpcio*>=1.66.2 (grpcio-status>=1.66.2 requires protobuf>=5.26.1).

https://github.com/protocolbuffers/protobuf/releases/tag/v28.3 (2024-10-22)
https://pypi.org/project/protobuf/5.28.3/ (2024-10-22)

Does this PR introduce any user-facing change?
This is a dependency change with no behavior change.
How was this patch tested?
Pass the CIs.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @HyukjinKwon

All tests passed.

Could you review this PR too, @zhengruifeng and @xinrong-meng ?

Could you review this PR, @viirya ?

Thank you, @HyukjinKwon . Merged to master for Apache Spark 4.0.0 on February 2025.

Thank you, @viirya .


Ability to add tabs for boxes

A common use for laser cutting is to make boxes. A box can be made by create "tab" or fingers that allow different sides of the box to slot together. It would be great to add some code to this repo that would let us add slots to a path or shape. For example, like what http://www.makercase.com/ allows you to do.

Absolutely! I am intending on doing this, just need to find the time. Meanwhile there is an example in http://microsoft.github.io/maker.js/docs/api/modules/makerjs.layout.html#clonetorow - just click the "Play" button. I just realized how buried this example is :(

Ok, here you go: http://microsoft.github.io/maker.js/playground/?script=tongue-and-groove

Interesting example @danmarshall. I ended up implementing this myself in a module that accepts a model and traverses it path by path to add a notch on each line segment. Still figuring out the use cases that I have for this project.

Is it private or can you share your example? Would like to see it :)

I can share it once I am in a working state again. Just ran into a bug. I am trying to expand a model using outline(), then originate() and simplify() and then add notches. What I am seeing however is that the direction of some lines is getting reverse at some step along the way. For example, a line that used to go from point A to point B now goes from point B to point A. It renders the same visually, but my logic was relying on direction of lines not changing as I am using connectTheDots() to product the final notched model. Hope that makes sense.

Oh interesting. Are you using findChains() ?

No, I am not aware of that API. I was using the following code to traverse the paths.
var keys = Object.keys(model.paths)
keys.forEach(function(path_name) {
   var path = model.paths[path_name]
   // and so on
})

Take a look at http://microsoft.github.io/maker.js/docs/advanced-drawing/#Chains
There is also findsinglechain which is easier to call if you only have one chain and one layer.

And for traversal there is http://microsoft.github.io/maker.js/docs/api/modules/makerjs.model.html#walk
probably another thing which could use better documentation...

@danmarshall I think i can get this to work, but I did notice that sometimes the origin/end points of a path are reversed. You can use this GIST as an example: https://gist.github.com/objectiveSee/02c24932ef2258b99e1a0361517dd704
I added a log to check whenever the origin of a path does not equal the end of the previous path. You can see that this happens sometimes in the console where the end and origin are flipped. Not sure if thats a bug or not, but it goofed me up ;)
Some output:
----
ShapeLine1_Top Origin: (-0.32,9.32) End:(0.68,9.32)
----
ShapeLine2_Top Origin: (0.68,9.32) End:(0.68,10.32)
----
ShapeLine2_EndCap_ShapeLine1 Origin: (0.68,10.32) End:(1.68,10.32)
----
ShapeLine4_Top Origin: (1.68,10.32) End:(1.68,11.32)
----
ShapeLine4_EndCap_ShapeLine1 Origin: (1.68,11.32) End:(5.32,11.32)
----
ShapeLine5_EndCap_ShapeLine1 Origin: (5.32,10.32) End:(5.32,11.32)
:: warning :: path is broken.(5.32,11.32) should equal (5.32,10.32)
----
ShapeLine7_Top Origin: (5.32,10.32) End:(7.68,10.32)
----
ShapeLine8_Top Origin: (7.68,10.32) End:(7.68,11.32)
----
ShapeLine8_EndCap_ShapeLine1 Origin: (7.68,11.32) End:(11.32,11.32)
----
ShapeLine9_EndCap_ShapeLine1 Origin: (11.32,10.32) End:(11.32,11.32)
:: warning :: path is broken.(11.32,11.32) should equal (11.32,10.32)
----

Ok, notching is working now! Here is the working code to paste into the playground. There is a hard-coded "pixel heart" model which I then expand using outline() and then add notches to. Both models shown in the playground output.
https://gist.github.com/objectiveSee/24cabe403c60266ecfed2feae262358b

Nice! That is impressive =)
Does it only work with "squared" shapes?

So, were you expecting that this code would traverse your paths in order?
keys.forEach(function(path_name) {
   //these will not be in order
})

Because, it sometimes works, but order isn't guaranteed: http://stackoverflow.com/questions/5525795/does-javascript-guarantee-object-property-order
This got me too. The only time order is guaranteed is in an Array. But I don't recommend using arrays as MakerJs objects.

I was expecting that order to be true (assuming that the keys were in order based on how they were named or generated). Does makerjs.model.walk guarantee that the order of traversing the paths is continuous?
The actual bug I ran into was about end and origin being opposite of what I had expected. This seems to only happy when the x or y movement is in the negative direction. For example in ShapeLine5_EndCap_ShapeLine1  above, its start point should have been (5.32,11.32) since that was the end of the previous line, ShapeLine4_EndCap_ShapeLine1 .

@danmarshall Yes, my notching algorithm only works on straight lines. I took some shortcuts since my design allowed for it.

No, makerjs.model.walk does not guarantee order, but .makerjs.model.findChains' does (also makerjs.model.findSingleChain), in its links property which is an array.

Whoa let me back up - the order is not the order of your objects keys. The order is in how the paths connect to each other. If your keys are sortable, then you might just sort them as an array then loop over that array. Watch out for things like p11 coming prior to p2 when sorting strings.

Thanks for the heads up on p11 bug. Chains are the way to go, and these two look like they will get me all the way there:
toKeyPoints(chainContext: IChain, maxArcFacet?: number): IPoint[]

toPoints(chainContext: IChain, distanceOrDistances: number | number[], maxPoints?: number): IPoint[]

Hi Danny, I'm going to close this for now, please re-open if there is anything else you'd like to add.

Thanks @danmarshall! I did find the Joinery tool (https://clementzheng.github.io/joinery/) which might be promising for others to try.


Allow dropping columns with a foreign key (in SQLite)

I'd suggest re-opening the issue of dropping columns with a foreign key in SQLite. Now that SQLite creates  table, the problem is that it generates SQL like: foreign key() references "users"("id"), which isn't valid. This is a remaining case of #214.
The problem is at: https://github.com/tgriesser/knex/blob/master/src/dialects/sqlite3/schema/ddl.js#L157, where the .replace() can result in an empty string when dropping a column, generating the bad SQL above. In this case, the entire foreign key clause can be omitted, so the new table no longer has a foreign key. After the rename, all is then as expected.
Without this, any column with a foreign key in SQLite is a blocker on schema changes, and the only workaround is a total rewrite of the table, which is less than satisfactory, especially when it is the parent for other foreign keys.

See also discussion in #1598

@rijkvanzanten Since you are quite deep into SQLite support internals already, could you by any chance take a look at this one as well? Not sure if it still relevant after your latest changes.

Yes, still very relevant 🙂
I'll see if we can update this:
https://github.com/knex/knex/blob/07ae5c072578de6b8e72036c45369ac768eecf07/lib/dialects/sqlite3/schema/tablecompiler.js#L96-L102
to actually be implemented for on alter table 👍


Replace the child_pty Library with pty.js

As quite a few people have noted, including me. There seems to be an issue with the child_pty module, which prevents Black Screen from building on all three of these platforms on several different occasions:

Windows: https://github.com/black-screen/black-screen/issues/24
Linux: https://github.com/shockone/black-screen/issues/49
Mac OSX: https://github.com/black-screen/black-screen/issues/27

There seem to workarounds but it would be better if the problem didn't arise in the first place.
Any ideas what could be causing this? Some have said its the NaN version that the module seems to require.
Solving this problem would be a major step in development for Black Screen. This could essentially enable full multi-platform support for it.

child_pty is the only native module, that's why it causes problems. Also the fact, that building it with Electron's headers isn't the usual workflow.
This is too level for me, but as far as I understand, it creates a pair of device files in order to pretend being a PTY. Since both the author of child_pty and the author of pty.js use a native extension for that, I guess it's not possible to do it using only node.js platform, but I might be wrong here.
So, the only option I see is to switch to pty.js. It has better support, and now it includes all the features we need. However, after rebuilding it with the Electron's headers, for some reason it becomes insanely slow. It wasn't like that a month ago.
I'll create an issue for that.

If you don't mind, I'll rename this one instead.
The replacement is really easy: in package.json we need to substitute child_pty with pty.js (two places) and change its version. And then there are two requires, which should be renamed. After that
npm run install-all to install and rebuild the new dependency. No other code changes are required.
But now executing any command has a 2 second delay, which we need to debug.

@shockone Maybe a temporary dev branch for testing is in order.

@flawiddsouza, surely, we create a branch for each feature.

@shockone Hopefully we won't have too many issues after this.

I'll start working on it. It seems to be the most painful issue.

@shockone Can you signup for https://gitter.im/ so we can talk about Black Screen without creating unecessary issues? [I'll delete this comment as soon as you acknowledge it]

@flawiddsouza, I'm there.

@shockone I can't seem to find you or Black screen.

A fix: https://github.com/black-screen/black-screen/pull/88


Is there any demo for ORC format file read/write operation using c++?

Is there any demo for ORC format file read/write operation using c++?

Have a look at the unit tests https://github.com/apache/arrow/blob/master/cpp/src/arrow/adapters/orc/adapter-test.cc
If you'd like to write some documentation please feel free to submit a PR


Only delete getters that have the canonical form

https://github.com/mplushnikov/lombok-intellij-plugin/issues/688
This is just a proof of concept, to get some early feedback.
Of course the same work would have to be done for the setters as well.
And maybe for the equals and hashCode methods. It might get trickier to test whether these are of the few established standard forms, as produced by Eclipse, IntelliJ and a few other IDEs.
My main focus are the getters and setters for now since manually checking for typos in that code is absolutely boring and error-prone.
While testing I noticed that Lombok will add a getter for fieldWithoutGetter. This may be intentional or not.

@mplushnikov Do you have an opinion on the code I have written? If you like it, I would implement similar code and tests for the setters as well.

Hello @rillig
thank you very much for your contribution. I'll look on your PR next days!
Please add yourself to contributors list in readme.md file, we forget to do it previously.


begginer-TMHMM2.0.model-etc-problems

Hello, sorry but I have to ask as a beginer:
If trying to use tmhmm.py as a library in Python 3.8.2:
import tmhmm
annotation, posterior = tmhmm.predict(sequence, 'mymodel.model')
I do not know how to use it

especialy "mymodel.model".

I tried TMHMM2.model file getting:
"TypeError: predict() missing 1 required positional argument: "model_or_filelike"
Shall I use "TMHMM1.0.model" or "PDBseqs.fasta"?
Could you give me some advice, please?
Thank you very much.
Petr Adamek

MaybeMaybe, I should give some additional information:
during installation appeared this warning:
WARNING: The script tmhmm is installed in '/home/oem/.local/bin' which is not on PATH.
Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
And in Python 3.8.2 I was able to install only this version:
tmhmm.py 1.2


Sparse transfer learning training error: AssertionError: min nan should be less than max nan

Describe the bug
After training the first batch before starting the testing the training stops with "assert min_val <= max_val, "min {} should be less than max {}".format(
AssertionError: min nan should be less than max nan"
Expected behavior
A clear and concise description of what you expected to happen.
Environment
Include all relevant environment information:

Ubuntu 18.04
Python version 3.8
SparseML version or commit hash [e.g. 0.1.0, f7245c8]:
ML framework version 1.9.1+cu102
Other Python package versions [e.g. SparseZoo, DeepSparse, numpy, ONNX]:
Other relevant environment information [e.g. hardware, CUDA version]:

To Reproduce
python3 train.py --data x-data.yaml --cfg ../models_v5.0/yolov5l.yaml --weights zoo:cv/detection/yolov5-l/pytorch/ultralytics/coco/pruned_quant-aggressive_95?recipe_type=transfer --hyp data/hyps/hyp.finetune.yaml --recipe ../recipes/yolov5.transfer_learn_pruned_quantized.md --batch-size -1 --img 1024 --epochs 10
Errors
Epoch   gpu_mem       box       obj       cls    labels  img_size
0/51     6.06G   0.02741   0.01176         0         2      1024: 100%|██████████| 10211/10211 [1:48:05<00:00,  1.57it/s]
Class     Images     Labels          P          R<EMAIL_ADDRESS>mAP@.5:.95:   0%|          | 0/116 [00:00<?, ?it/s]
Traceback (most recent call last):
File "train.py", line 681, in
main(opt)
File "train.py", line 577, in main
train(opt.hyp, opt, device, callbacks)
File "train.py", line 402, in train
results, maps, _ = val.run(data_dict,
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 28, in decorate_context
return func(*args, **kwargs)
File "/home/mate/Developer/recog/yolov5-neural/sparseml/integrations/ultralytics-yolov5/yolov5/val.py", line 192, in run
out, train_out = model(im) if training else model(im, augment=augment, val=True)  # inference, loss outputs
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/home/mate/Developer/recog/yolov5-neural/sparseml/integrations/ultralytics-yolov5/yolov5/models/yolo.py", line 128, in forward
return self._forward_once(x, profile, visualize)  # single-scale inference, train
File "/home/mate/Developer/recog/yolov5-neural/sparseml/integrations/ultralytics-yolov5/yolov5/models/yolo.py", line 151, in _forward_once
x = m(x)  # run
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/home/mate/Developer/recog/yolov5-neural/sparseml/integrations/ultralytics-yolov5/yolov5/models/common.py", line 208, in forward
return self.conv(torch.cat([x[..., ::2, ::2], x[..., 1::2, ::2], x[..., ::2, 1::2], x[..., 1::2, 1::2]], 1))
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/home/mate/Developer/recog/yolov5-neural/sparseml/integrations/ultralytics-yolov5/yolov5/models/common.py", line 47, in forward
return self.act(self.bn(self.conv(x)))
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/quantization/stubs.py", line 56, in forward
X = self.quant(X)
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1076, in _call_impl
hook_result = hook(self, input, result)
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/quantization/quantize.py", line 83, in _observer_forward_hook
return self.activation_post_process(output)
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/quantization/fake_quantize.py", line 131, in forward
_scale, _zero_point = self.calculate_qparams()
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/quantization/fake_quantize.py", line 126, in calculate_qparams
return self.activation_post_process.calculate_qparams()
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/quantization/observer.py", line 410, in calculate_qparams
return self._calculate_qparams(self.min_val, self.max_val)
File "/home/mate/Developer/recog/yolov5-neural/venv/lib/python3.8/site-packages/torch/quantization/observer.py", line 250, in _calculate_qparams
assert min_val <= max_val, "min {} should be less than max {}".format(
AssertionError: min nan should be less than max nan
Additional context
This is a 1 class training, .yaml has been changed according to this.

I have the same problem

你好@mate-hegedus @GhYang0519看起来这个问题可能是由于经理尝试两次初始化量化而出现的 - 我们已经在内部跟踪了这个问题，并且正在积极努力修复

Thanks for your reply, looking forward to an early repair

@mate-hegedus @GhYang0519 thank you for reporting this issue. A patch for Yolov5 is up here and should be incorporated into release 1.0 and nightly later today.

@mate-hegedus @GhYang0519感谢您报告此问题。Yolov5 的补丁已在此处发布，应该会在今天晚些时候并入 1.0 版和每晚发布。
OK, thank you.

Closing as changes have landed to address the reported issue. If you run into the issue again please feel free to re-open.


BVH/ImplicitGrid API refactor

Summary

Refactors spin::BVH public API and internal implementation to make use of axom::Array/axom::ArrayView

Public BVH::findPoints/findRays/findBoundingBoxes() methods now accept offsets and counts as ArrayViews, and output candidates as an axom::Array
Internal BVH data is stored as axom::Arrays, which Traverser object now points to with axom::ArrayViews

Cleans up spin::ImplicitGrid API use of Array/ArrayView

spin::ImplicitGrid::getCandidatesAsArray() method now accepts offsets and counts as externally-allocated ArrayViews
Adds overloads for spin::ImplicitGrid::getCandidatesAsArray() which accept the array of query primitives as an ArrayView
Use ArrayView to replace raw pointers in ImplicitGrid::QueryObject

Fixes a bug with mesh_tester binary on blueos by enabling unified memory

I noted the change in the release notes for BVH -- since ImplicitGrid's vector API hasn't been in a release yet, I didn't document that change.


[SPARK-11110] [Build] Remove transient annotation for parameters.

@transient annotations on class parameters (not case class parameters or vals) causes compilation errors during compilation with Scala 2.11.
I understand that transient parameters make no sense, however I don't quite understand why the 2.10 compiler accepted them.
Note: in case it is preferred to keep the annotations in case someone would in the future want to redefine them as vals, it would also be possible to just add val after the annotation, e.g. class Foo(@transient x: Int) becomes class Foo(@transient private val x: Int).
I chose to remove the annotation as it also reduces needles clutter, however please feel free to tell me if you prefer the second option and I'll update the PR

Can one of the admins verify this patch?

ping @marmbrus for any thoughts. But I think removing them makes sense. If someone makes it a val later they will have to reason about whether it should be transient or not as we would for any new field.

Unfortunately, I think these annotations may not be worthless and we might be hitting a bug in the scala compiler.  Consider the following class: class A(a: A) { def b = a }
While a isn't explicitly declared as a val, the compiler is still silently promoting it to a class field so that it can be used both in the constructor and in function b.  In Scala 2.10 at least this results in the following bytecode:
...
  private final int a;
    flags: ACC_PRIVATE, ACC_FINAL
...

Where as class A(@transient a: Int) { def b = a } results in:
...
  private final transient int a;
    flags: ACC_PRIVATE, ACC_FINAL, ACC_TRANSIENT
...

You can play around with this yourself in the REPL:
$ build/sbt catalyst/console

scala> class A(a: Int) { def b = a }
defined class A

scala> org.apache.spark.sql.catalyst.expressions.codegen.DumpByteCode(new A(1))
Classfile /private/var/folders/36/cjkbrr953xg2p_krwrmn8h_r0000gn/T/spark-292994a3-dd85-4f32-bf2e-ba6f1365712e/$line3/$read$$iw$$iw$A.class
  Last modified Oct 14, 2015; size 532 bytes
  MD5 checksum f2a93222131c1a1c914498fd7d22ec13
  Compiled from "<console>"
public class $line3.$read$$iw$$iw$A
  SourceFile: "<console>"
  InnerClasses:
       public static #23= #20 of #22; //$iw$=class $line3/$read$$iw$ of class $line3/$read
       public static #23= #25 of #20; //$iw$=class $line3/$read$$iw$$iw$ of class $line3/$read$$iw$
       public static #26= #2 of #25; //A=class $line3/$read$$iw$$iw$A of class $line3/$read$$iw$$iw$
    Scala: length = 0x0

  minor version: 0
  major version: 50
  flags: ACC_PUBLIC, ACC_SUPER
Constant pool:
   #1 = Utf8               $line3/$read$$iw$$iw$A
   #2 = Class              #1             //  $line3/$read$$iw$$iw$A
   #3 = Utf8               java/lang/Object
   #4 = Class              #3             //  java/lang/Object
   #5 = Utf8               <console>
   #6 = Utf8               a
   #7 = Utf8               I
   #8 = Utf8               b
   #9 = Utf8               ()I
  #10 = NameAndType        #6:#7          //  a:I
  #11 = Fieldref           #2.#10         //  $line3/$read$$iw$$iw$A.a:I
  #12 = Utf8               this
  #13 = Utf8               L$line3/$read$$iw$$iw$A;
  #14 = Utf8               <init>
  #15 = Utf8               (I)V
  #16 = Utf8               ()V
  #17 = NameAndType        #14:#16        //  "<init>":()V
  #18 = Methodref          #4.#17         //  java/lang/Object."<init>":()V
  #19 = Utf8               $line3/$read$$iw$
  #20 = Class              #19            //  $line3/$read$$iw$
  #21 = Utf8               $line3/$read
  #22 = Class              #21            //  $line3/$read
  #23 = Utf8               $iw$
  #24 = Utf8               $line3/$read$$iw$$iw$
  #25 = Class              #24            //  $line3/$read$$iw$$iw$
  #26 = Utf8               A
  #27 = Utf8               Code
  #28 = Utf8               LocalVariableTable
  #29 = Utf8               LineNumberTable
  #30 = Utf8               SourceFile
  #31 = Utf8               InnerClasses
  #32 = Utf8               Scala
{
  private final int a;
    flags: ACC_PRIVATE, ACC_FINAL

  public int b();
    flags: ACC_PUBLIC
    Code:
      stack=1, locals=1, args_size=1
         0: aload_0
         1: getfield      #11                 // Field a:I
         4: ireturn
      LocalVariableTable:
        Start  Length  Slot  Name   Signature
               0       5     0  this   L$line3/$read$$iw$$iw$A;
      LineNumberTable:
        line 7: 0

  public $line3.$read$$iw$$iw$A(int);
    flags: ACC_PUBLIC
    Code:
      stack=2, locals=2, args_size=2
         0: aload_0
         1: iload_1
         2: putfield      #11                 // Field a:I
         5: aload_0
         6: invokespecial #18                 // Method java/lang/Object."<init>":()V
         9: return
      LocalVariableTable:
        Start  Length  Slot  Name   Signature
               0      10     0  this   L$line3/$read$$iw$$iw$A;
               0      10     1     a   I
      LineNumberTable:
        line 7: 0
}

scala> class A(@transient a: Int) { def b = a }
defined class A

scala> org.apache.spark.sql.catalyst.expressions.codegen.DumpByteCode(new A(1))
Classfile /private/var/folders/36/cjkbrr953xg2p_krwrmn8h_r0000gn/T/spark-292994a3-dd85-4f32-bf2e-ba6f1365712e/$line7/$read$$iw$$iw$A.class
  Last modified Oct 14, 2015; size 532 bytes
  MD5 checksum 0dc1337c7731bd1a88bf8f33bbfb9e4b
  Compiled from "<console>"
public class $line7.$read$$iw$$iw$A
  SourceFile: "<console>"
  InnerClasses:
       public static #23= #20 of #22; //$iw$=class $line7/$read$$iw$ of class $line7/$read
       public static #23= #25 of #20; //$iw$=class $line7/$read$$iw$$iw$ of class $line7/$read$$iw$
       public static #26= #2 of #25; //A=class $line7/$read$$iw$$iw$A of class $line7/$read$$iw$$iw$
    Scala: length = 0x0

  minor version: 0
  major version: 50
  flags: ACC_PUBLIC, ACC_SUPER
Constant pool:
   #1 = Utf8               $line7/$read$$iw$$iw$A
   #2 = Class              #1             //  $line7/$read$$iw$$iw$A
   #3 = Utf8               java/lang/Object
   #4 = Class              #3             //  java/lang/Object
   #5 = Utf8               <console>
   #6 = Utf8               a
   #7 = Utf8               I
   #8 = Utf8               b
   #9 = Utf8               ()I
  #10 = NameAndType        #6:#7          //  a:I
  #11 = Fieldref           #2.#10         //  $line7/$read$$iw$$iw$A.a:I
  #12 = Utf8               this
  #13 = Utf8               L$line7/$read$$iw$$iw$A;
  #14 = Utf8               <init>
  #15 = Utf8               (I)V
  #16 = Utf8               ()V
  #17 = NameAndType        #14:#16        //  "<init>":()V
  #18 = Methodref          #4.#17         //  java/lang/Object."<init>":()V
  #19 = Utf8               $line7/$read$$iw$
  #20 = Class              #19            //  $line7/$read$$iw$
  #21 = Utf8               $line7/$read
  #22 = Class              #21            //  $line7/$read
  #23 = Utf8               $iw$
  #24 = Utf8               $line7/$read$$iw$$iw$
  #25 = Class              #24            //  $line7/$read$$iw$$iw$
  #26 = Utf8               A
  #27 = Utf8               Code
  #28 = Utf8               LocalVariableTable
  #29 = Utf8               LineNumberTable
  #30 = Utf8               SourceFile
  #31 = Utf8               InnerClasses
  #32 = Utf8               Scala
{
  private final transient int a;
    flags: ACC_PRIVATE, ACC_FINAL, ACC_TRANSIENT

  public int b();
    flags: ACC_PUBLIC
    Code:
      stack=1, locals=1, args_size=1
         0: aload_0
         1: getfield      #11                 // Field a:I
         4: ireturn
      LocalVariableTable:
        Start  Length  Slot  Name   Signature
               0       5     0  this   L$line7/$read$$iw$$iw$A;
      LineNumberTable:
        line 7: 0

  public $line7.$read$$iw$$iw$A(int);
    flags: ACC_PUBLIC
    Code:
      stack=2, locals=2, args_size=2
         0: aload_0
         1: iload_1
         2: putfield      #11                 // Field a:I
         5: aload_0
         6: invokespecial #18                 // Method java/lang/Object."<init>":()V
         9: return
      LocalVariableTable:
        Start  Length  Slot  Name   Signature
               0      10     0  this   L$line7/$read$$iw$$iw$A;
               0      10     1     a   I
      LineNumberTable:
        line 7: 0
}

That makes sense, it would also explain why the compiler didn't shout in 2.10. Could it be related to this bug?
As a workaround we could just declare the parameter as a private val.

Sounds good to me.

Ok I'll update the PR, I'll also check with the scala compiler people.

Hmm, actually I saw that some errors were due to a "transient parameter forwarding" of an existing field in an inherited class. For example HiveContext has a parameter called listener which is defined a a transient field in its base class SQLContext.
Other parameters however can be worked around with a private val

Update: scrolling through the error logs, I found that the @transient errors were also printed as warnings. Going through the sbt build script, I found that warnings are actually treated as errors but due to a bug in the script (which I'll address in a separate issue) the user is not informed that they are dealing with fatal warnings.
The bottom line is that we're not dealing with a compiler bug and, if acceptable, this PR should be ready to be merged in.

LGTM

ok to test

Merged build triggered.

Merged build started.

Please hold

Test build #43793 has started for   PR 9126 at commit e13a215.

Merged build triggered.

Merged build started.

You were right about NettyRpcEndpointRef, actually conf is not defined as a field in the base class.
Sorry I missed this, it is however the case for the listener in HiveContext

Test build #43795 has started for   PR 9126 at commit b505bb6.

Test build #43795 has finished for   PR 9126 at commit b505bb6.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #43793 has finished for   PR 9126 at commit e13a215.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Its strange that the python and R test fail, I didn't touch any related code.
Please note that I pushed a fix during the first build, maybe that somehow corrupted the tests?

Almost certainly spurious. Retesting ...

Test build #1909 has started for   PR 9126 at commit b505bb6.

Test build #1909 has finished for   PR 9126 at commit b505bb6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #1912 has started for   PR 9126 at commit b505bb6.

Jenkins, retest this please

Merged build triggered.

Merged build started.

Test build #43856 has started for   PR 9126 at commit b505bb6.

Test build #43856 has finished for   PR 9126 at commit b505bb6.

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

Merged build finished. Test PASSed.

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


M1 Mac python install error when installing openjtalk>=0.3.0.dev2

error msg like:
Collecting unidecode (from -r requirements.txt (line 6))
Using cached Unidecode-1.3.6-py3-none-any.whl (235 kB)
Collecting openjtalk>=0.3.0.dev2 (from -r requirements.txt (line 7))
Using cached openjtalk-0.3.0.dev3.tar.gz (24.9 MB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Discarding https://files.pythonhosted.org/packages/15/15/fb29ebea31d9b0fbe72b8547c9221dda17df8e23e3b4f2eb326eec07966a/openjtalk-0.3.0.dev3.tar.gz (from https://pypi.org/simple/openjtalk/): Requested openjtalk>=0.3.0.dev2 from https://files.pythonhosted.org/packages/15/15/fb29ebea31d9b0fbe72b8547c9221dda17df8e23e3b4f2eb326eec07966a/openjtalk-0.3.0.dev3.tar.gz (from -r requirements.txt (line 7)) has inconsistent version: expected '0.3.0.dev3', but metadata has '0.3.0'
Using cached openjtalk-0.3.0.dev2.tar.gz (24.9 MB)
Installing build dependencies ... done
Getting requirements to build wheel ... error
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> [18 lines of output]
Traceback (most recent call last):
File "/Users/zhonghaoli/miniforge3/envs/py30/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in
main()

try to install pyopenjtalk


XPath & document.evaluate

XPath & document.evaluate

This issue was imported from Google Moderator
Moderator votes: +11
Issue added by Daniel15 on 2011-11-11
To vote this issue up or down, simply include +1 or -1 in your comment.

Duplicate of #1558.

+1

+1

+1

+1

+1

+1

+1

+1

+!

+1, found a broken link on Stack Overflow: http://code.google.com/p/doctype/wiki/DocumentEvaluateMethod so looking to a trusted source like caniuse

+1

+1

Now available at http://caniuse.com/#feat=document-evaluate-xpath


adding a Gradle command for running application on port 9000

@infiniteshi Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@infiniteshi Thank you for signing the Contributor License Agreement!

@Buzzardo Hi Jay, thank you very much for encouraging me to make the pull request and I would appreciate it if you could review it. Thanks. --Wenyu

Thank you, and congratulations on your first Spring contribution. Make more contributions. :)

@Buzzardo  Thanks, and I will!

Hello again, Shi,
(Shi is your given rather than family name, right? I know another Shi, in Singapore, so I think that's a given name. Things get tricky across languages and cultures, and I don't wish to offend.)
One of my colleagues pointed out that the JavaScript has to be changed, too. Then we talked about how the index.html file and the JavaScript were not where we expected them to be. Consequently, I have some more changes to make to that guide. You can take credit for making that first change and for making us look more closely at that guide, too. If we find it to be confusing, what must our readers think?
Thanks again.
J

From: Wenyu Shi<EMAIL_ADDRESS>Sent: Friday, October 2, 2020 11:48 AM
To: spring-guides/gs-rest-service-cors<EMAIL_ADDRESS>Cc: Jay Bryant (Pivotal<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [spring-guides/gs-rest-service-cors] adding a Gradle command for running application on port 9000 (#29)
@Buzzardohttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FBuzzardo&data=02|01|jbryant%40vmware.com|718a4868ed834c16c7f508d866f2f510|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637372540963898294&sdata=1TJMALy7UzI9mPeuLEkEl7Ox9ZTAE2EWpssuLAxv1bw%3D&reserved=0 Thanks, and I will!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fspring-guides%2Fgs-rest-service-cors%2Fpull%2F29%23issuecomment-702840504&data=02|01|jbryant%40vmware.com|718a4868ed834c16c7f508d866f2f510|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637372540963903295&sdata=SsRm8FKsoxlvOrOYL4Uxp0MdvwWylu3KE%2BZcO%2FlM02k%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABH3VSSKTHY7HMJF63CAVTTSIX74XANCNFSM4SB2XPQQ&data=02|01|jbryant%40vmware.com|718a4868ed834c16c7f508d866f2f510|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637372540963908291&sdata=GmI1OJeunzDQs8BOC1L966w90rULTS5i5mCFnZ%2F4%2Fuk%3D&reserved=0.

Hi Jay,
Shi is my last name (pronounced "she") and Wenyu is my first name
(pronounced "when-you" - it keeps me alert in the
classrooms/meetings/conversations because people often start a sentence by
"When you (do this or do that)..."  I grew up in China and you were right
that in many Asian cultures people place the family name before the last
name. However, I adopted the American English's convention:-)
It's exciting to know more updates are being made to the Guide project! I
would love to be a volunteer tester for you guys. I have gone through all
the following tutorials most recently and remember only two I felt
that might need a bit further examination.
====What I've gone through without major difficulties====sorry, the list
is random order====
/* Definition of "major difficulties": Well, I definitely stumbled on
dozens of things but I don't think the issues I had were directly related
to or within the scope of the Spring Guides per se but they ranged from
Gradle/Maven build issues, local configuration, browser cache etc.
*/

Building a RESTful Web Service
Consuming a RESTful Web Service
Building Java Projects with Maven
Building Java Projects with Gradle
Accessing Relational Data using JDBC w/ Spring
6.Uploading Files
Validating Form Input
Securing a Web Application
Accessing Data with JPA
Serving Web Content w/ Spring MVC
Handling Form Submission
Building an Application with Spring Boot
Enabling CORS for a RESTful Web Service

====The one I felt might need a bit further examination was====
====Building a RESTful Web Service w/ Spring Boot Actuator====
Source URL: https://github.com/spring-guides/gs-actuator-service
The problem is this: the name of the application, as it was defined in
settings.gradle was "actuator-service"
rootProject.name = 'actuator-service'
In the meanwhile, the command listing reads as follows in the guide:
./gradlew clean build && java -jar build/libs/gs-actuator-service-0.1.0.jar
As a result, I used to get trapped in a build error complaining that
gs-actuator-service-0.1.0.jar couldn't be found.
Below is my understanding, and PLEASE CORRECT ME if it is wrong - I even
shared that as a solution on StackOverflow.com - if it is wrong, I got to
correct my posting there.
[image: image.png]
====The other one I couldn't get running without deviating from the guide
was====
==== Authenticating a user w/ LDAP ====
Here is a StackOverflow post where some other users and myself discussed
our struggles, in one way or another.
https://stackoverflow.com/questions/58239499/issue-with-spring-boot-guide-authenticating-a-user-with-ldap/63697051#63697051
And until now I'm still not sure how to make the following lines work for
my local as they are in the guide today, hence the two orange comments
@Override
public void configure(AuthenticationManagerBuilder auth) throws Exception {
auth
.ldapAuthentication()
.userDnPatterns("uid={0},ou=people")
.groupSearchBase("ou=groups")
.contextSource()
.url("ldap://localhost:8389/dc=springframework,dc=org")//CHANGE
8389 to 8399
.and()
.passwordCompare()
.passwordEncoder(new BCryptPasswordEncoder()) //REMOVE this line
.passwordAttribute("userPassword");
}
}
Only with the changes marked by the orange comments could I run the
application with the expected results. To summarize, my major struggle was
(1) port (preoccupied or not listening, but this can be out of the scope of
the Guides per se). (2) Which is the bigger struggle of the two, was with
the BCryptPasswordEncoder. Some other posts on StackOverflow suggested
that additional Spring configuration code has to be in place, but I didn't
succeed with many of their suggestions and haven't found the way to make it
work as it is in the guide by now.
Best regards,
Wenyu
On Fri, Oct 2, 2020 at 12:12 PM Jay Bryant<EMAIL_ADDRESS>wrote:

Hello again, Shi,
(Shi is your given rather than family name, right? I know another Shi, in
Singapore, so I think that's a given name. Things get tricky across
languages and cultures, and I don't wish to offend.)
One of my colleagues pointed out that the JavaScript has to be changed,
too. Then we talked about how the index.html file and the JavaScript were
not where we expected them to be. Consequently, I have some more changes to
make to that guide. You can take credit for making that first change and
for making us look more closely at that guide, too. If we find it to be
confusing, what must our readers think?
Thanks again.
J

From: Wenyu Shi<EMAIL_ADDRESS>Sent: Friday, October 2, 2020 11:48 AM
To: spring-guides/gs-rest-service-cors <
<EMAIL_ADDRESS>Cc: Jay Bryant (Pivotal<EMAIL_ADDRESS>Mention <
<EMAIL_ADDRESS>Subject: Re: [spring-guides/gs-rest-service-cors] adding a Gradle command
for running application on port 9000 (#29)
@Buzzardo<
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FBuzzardo&data=02|01|jbryant%40vmware.com|718a4868ed834c16c7f508d866f2f510|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637372540963898294&sdata=1TJMALy7UzI9mPeuLEkEl7Ox9ZTAE2EWpssuLAxv1bw%3D&reserved=0>
Thanks, and I will!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fspring-guides%2Fgs-rest-service-cors%2Fpull%2F29%23issuecomment-702840504&data=02|01|jbryant%40vmware.com|718a4868ed834c16c7f508d866f2f510|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637372540963903295&sdata=SsRm8FKsoxlvOrOYL4Uxp0MdvwWylu3KE%2BZcO%2FlM02k%3D&reserved=0>,
or unsubscribe<
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABH3VSSKTHY7HMJF63CAVTTSIX74XANCNFSM4SB2XPQQ&data=02|01|jbryant%40vmware.com|718a4868ed834c16c7f508d866f2f510|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637372540963908291&sdata=GmI1OJeunzDQs8BOC1L966w90rULTS5i5mCFnZ%2F4%2Fuk%3D&reserved=0

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spring-guides/gs-rest-service-cors/pull/29#issuecomment-702851635,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AINEMZLFTQOGZPQUIZI7RCDSIYCWJANCNFSM4SB2XPQQ
.

Hi Jay,
Sorry about the corrupted image in the previous email. Below is a re-send
of the image only:
[image: image.png]
https://stackoverflow.com/questions/52484926/error-unable-to-access-jarfile-build-libs-gs-spring-boot-0-1-0-jar/63745747#63745747
Best,
Wenyu
On Fri, Oct 2, 2020 at 3:11 PM Wenyu Shi<EMAIL_ADDRESS>wrote:

Hi Jay,
Shi is my last name (pronounced "she") and Wenyu is my first name
(pronounced "when-you" - it keeps me alert in the
classrooms/meetings/conversations because people often start a sentence by
"When you (do this or do that)..."  I grew up in China and you were right
that in many Asian cultures people place the family name before the last
name. However, I adopted the American English's convention:-)
It's exciting to know more updates are being made to the Guide project! I
would love to be a volunteer tester for you guys. I have gone through all
the following tutorials most recently and remember only two I felt
that might need a bit further examination.
====What I've gone through without major difficulties====sorry, the list
is random order====
/* Definition of "major difficulties": Well, I definitely stumbled on
dozens of things but I don't think the issues I had were directly related
to or within the scope of the Spring Guides per se but they ranged from
Gradle/Maven build issues, local configuration, browser cache etc.
*/

Building a RESTful Web Service
Consuming a RESTful Web Service
Building Java Projects with Maven
Building Java Projects with Gradle
Accessing Relational Data using JDBC w/ Spring
6.Uploading Files
Validating Form Input
Securing a Web Application
Accessing Data with JPA
Serving Web Content w/ Spring MVC
Handling Form Submission
Building an Application with Spring Boot
Enabling CORS for a RESTful Web Service

====The one I felt might need a bit further examination was====
====Building a RESTful Web Service w/ Spring Boot Actuator====
Source URL: https://github.com/spring-guides/gs-actuator-service
The problem is this: the name of the application, as it was defined in
settings.gradle was "actuator-service"
rootProject.name = 'actuator-service'
In the meanwhile, the command listing reads as follows in the guide:
./gradlew clean build && java -jar build/libs/gs-actuator-service-0.1.0.jar
As a result, I used to get trapped in a build error complaining that
gs-actuator-service-0.1.0.jar couldn't be found.
Below is my understanding, and PLEASE CORRECT ME if it is wrong - I even
shared that as a solution on StackOverflow.com - if it is wrong, I got to
correct my posting there.
[image: image.png]
====The other one I couldn't get running without deviating from the guide
was====
==== Authenticating a user w/ LDAP ====
Here is a StackOverflow post where some other users and myself discussed
our struggles, in one way or another.
https://stackoverflow.com/questions/58239499/issue-with-spring-boot-guide-authenticating-a-user-with-ldap/63697051#63697051
And until now I'm still not sure how to make the following lines work for
my local as they are in the guide today, hence the two orange comments
@Override
public void configure(AuthenticationManagerBuilder auth) throws Exception {
auth
.ldapAuthentication()
.userDnPatterns("uid={0},ou=people")
.groupSearchBase("ou=groups")
.contextSource()
.url("ldap://localhost:8389/dc=springframework,dc=org")//CHANGE 8389 to 8399
.and()
.passwordCompare()
.passwordEncoder(new BCryptPasswordEncoder()) //REMOVE this line
.passwordAttribute("userPassword");
}
}
Only with the changes marked by the orange comments could I run the
application with the expected results. To summarize, my major struggle was
(1) port (preoccupied or not listening, but this can be out of the scope of
the Guides per se). (2) Which is the bigger struggle of the two, was with
the BCryptPasswordEncoder. Some other posts on StackOverflow suggested
that additional Spring configuration code has to be in place, but I didn't
succeed with many of their suggestions and haven't found the way to make it
work as it is in the guide by now.
Best regards,
Wenyu
On Fri, Oct 2, 2020 at 12:12 PM Jay Bryant<EMAIL_ADDRESS>wrote:

Hello again, Shi,
(Shi is your given rather than family name, right? I know another Shi, in
Singapore, so I think that's a given name. Things get tricky across
languages and cultures, and I don't wish to offend.)
One of my colleagues pointed out that the JavaScript has to be changed,
too. Then we talked about how the index.html file and the JavaScript were
not where we expected them to be. Consequently, I have some more changes to
make to that guide. You can take credit for making that first change and
for making us look more closely at that guide, too. If we find it to be
confusing, what must our readers think?
Thanks again.
J

From: Wenyu Shi<EMAIL_ADDRESS>Sent: Friday, October 2, 2020 11:48 AM
To: spring-guides/gs-rest-service-cors <
<EMAIL_ADDRESS>Cc: Jay Bryant (Pivotal<EMAIL_ADDRESS>Mention <
<EMAIL_ADDRESS>Subject: Re: [spring-guides/gs-rest-service-cors] adding a Gradle command
for running application on port 9000 (#29)
@Buzzardo<
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FBuzzardo&data=02|01|jbryant%40vmware.com|718a4868ed834c16c7f508d866f2f510|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637372540963898294&sdata=1TJMALy7UzI9mPeuLEkEl7Ox9ZTAE2EWpssuLAxv1bw%3D&reserved=0>
Thanks, and I will!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fspring-guides%2Fgs-rest-service-cors%2Fpull%2F29%23issuecomment-702840504&data=02|01|jbryant%40vmware.com|718a4868ed834c16c7f508d866f2f510|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637372540963903295&sdata=SsRm8FKsoxlvOrOYL4Uxp0MdvwWylu3KE%2BZcO%2FlM02k%3D&reserved=0>,
or unsubscribe<
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABH3VSSKTHY7HMJF63CAVTTSIX74XANCNFSM4SB2XPQQ&data=02|01|jbryant%40vmware.com|718a4868ed834c16c7f508d866f2f510|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637372540963908291&sdata=GmI1OJeunzDQs8BOC1L966w90rULTS5i5mCFnZ%2F4%2Fuk%3D&reserved=0

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spring-guides/gs-rest-service-cors/pull/29#issuecomment-702851635,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AINEMZLFTQOGZPQUIZI7RCDSIYCWJANCNFSM4SB2XPQQ
.


Can I use Custom Component?

I want to add a link to one column. but I can not find the attribute to define owner custom component. I just find the custom editor, but it need me to click it, I want to display a link when it is initialing.
Can I finish the function with react-bootstrap-table? If i can, can you tell me the attribute?
Thank you very much!
Thank you very much!
Thank you very much!

yeah, I finish it by using the attribute dataFormat.thanks


fix: json decode bug

Description
This PR fixes #816 , and it a retry PR.
Summary
This PR is try again. I'm having a conflict, and retrying met a can't pass banchmark😥. Maybe I need help to Pass test.
Define a function for processing json_string to PyObject

PR Checklist
Please ensure that:

[x] The PR contains a descriptive title
[x] The PR contains a descriptive summary of the changes
[x] You build and test your changes before submitting a PR.
[x] You have added relevant documentation
[x] You have added relevant tests. We prefer integration tests wherever possible

Pre-Commit Instructions:

[x] Ensure that you have run the pre-commit hooks on your PR.

Hey @asamaayako 👋
Could you explain the confilct you are facing? I will be happy to help :D

Hey @asamaayako 👋
Could you explain the confilct you are facing? I will be happy to help :D

Previously encountered a conflict, merge conflict (failed to pass test after merging the main branch into this branch). I think the main branch at the time had code that couldn't pass. So Froce reverted the merge.
Now it ok :D.

Hey @VishnuSanal ,
Could you review this PR?

Hey @asamaayako 👋
Could you explain the confilct you are facing? I will be happy to help :D

Hey @sansyrox Previously encountered a conflict, merge conflict (failed to pass test after merging the main branch into this branch). I think at the time the main branch had code that couldn't pass test. So I force push rolled back the merge. Now it ok :D. This PR is ready to merge I think.

@asamaayako ,
thanks for the PR. Apologies for the super late acknowledgment. Having a look now


Add an event handler

This can be used by applications to better monitor their connection
pool usage. For example, you could maintain a histogram of pool checkout
times and alert if the p95 gets too high, indicating oversaturating.
cc #75

cc @Diggsey

This seems like a great starting point - the next thing I would add is some kind of opaque connection ID, just so that it's possible to match up the different events.

Oh, interesting. I've always thought about the events in aggregate, but I guess there's no reason not to allow you to correlate various events.
We could have an opaque connection ID, but I wonder if it'll be simpler for it just to be a u64? We're going to be generating them ourselves either way so it's not like that's really limiting in any significant way.

Yeah, I have no opinion on that. As long as I can correlate the events somehow it doesn't really matter.


Preliminary Download Support

Implements #2.
Adds API endpoints such as /version/1.1.8/ and /version/1.1.8/download/
Doesn't support snapshot versions but if people want that I can probably bodge it in.

Please add snapshots!

/version/:ver should include snapshots as they are versions!


[0.9.1] Unable to create series once it is dropped

I am using influxdb-0.9.1-1.x86_64.
Scenario:

Created a series using curl
Executed "drop series from "
Then again tried to create series with same name using curl. Executed show series from CLI but did get any output.

Is it expected behavior?

Can you please show the entire sequence using the actual curl commands? That would be most helpful.

curl -i -XPOST 'http://ip:port/write?db=mydb&precision=s' --data-binary Usage,applicationtype=WebBrowsing,application=google Upstream=10000,Downstream=200000<PHONE_NUMBER>'

What I mean @buro1983 is the entire sequence -- creating the database, writing a point, demonstrating the data is there, dropping the series, and then re-attempting the write. Please include the output of each command.
https://github.com/influxdb/influxdb/blob/master/CONTRIBUTING.md

i'm experiencing the same problem as described above. to reproduce:
# create database
curl -G http://localhost:8086/query --data-urlencode "q=CREATE DATABASE drop_bug"

# send a point
curl -i -XPOST 'http://localhost:8086/write?db=drop_bug' --data-binary 'my_metric,host=myhost value=0.64<PHONE_NUMBER>000000000'

# prove that the series is there.
curl -G http://localhost:8086/query  --data-urlencode 'db=drop_bug' --data-urlencode 'q=SHOW SERIES'

# DROP
curl -G http://localhost:8086/query  --data-urlencode 'db=drop_bug' --data-urlencode 'q=DROP SERIES FROM my_metric'

# send a point - yes, the same point.
curl -i -XPOST 'http://localhost:8086/write?db=drop_bug' --data-binary 'my_metric,host=myhost value=0.64<PHONE_NUMBER>000000000'

# Try to show series again. It should show the series just re-created, but comes back empty.
curl -G http://localhost:8086/query  --data-urlencode 'db=drop_bug' --data-urlencode 'q=SHOW SERIES'

my system: Mac OSX 10.10.4, InfluxDB 0.9.1 installed via Brew

so is there currently any way to delete data from a db without restarting afterwards? I see from #1647 that DELETE is not yet implemented (ERR: invalid query) - so how are we to accomplish deletions?

@samhatchett this bug relates to being unable to insert new points into the dropped series. The data in the series is dropped successfully.

[0.9.1] I'm experiencing the same problem.

select * from cpu
name: cpu
tags: host=local
time				value

2015-07-09T16:54:40.740627311Z	0.6

drop series from cpu where host='local'
select * from cpu
insert cpu,host=local value=9.0
select * from cpu
show series

It seems DROP does not succeed. When restarting the influxd process and writing data, the series that I dropped appear again.

select * from cpu
insert cpu,host=local value=7.0
select * from cpu
name: cpu
tags: host=local
time				value

2015-07-09T16:54:40.740627311Z	0.6
2015-07-09T16:55:27.193514255Z	9
2015-07-09T16:56:54.869625619Z	7
...

@beckettsean I get it - what I should have said is that obviously this is not currently a preferred method of deletion since it renders a time series defunct for future writes. Therefore it's off the list of things I can use to delete data.

@samhatchett that is correct, it's not currently a good way to delete. It is supposed to work, though, so starting with 0.9.2 or at worst 0.9.3 this should be the correct way to do what you're trying to accomplish.

Additional details on this problem in #3283

Fixed by https://github.com/influxdb/influxdb/pull/3651. Thanks for the reports @buro1983 and @samhatchett.

Fix will be in nightlies and 0.9.3.


test

Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

Codecov Report

Merging #104 (b780834) into main (67e2b52) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #104   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          1       1
  Lines          4       4
=====================================
  Misses         4       4

Flag
Coverage Δ

unittests
0.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 67e2b52...b780834. Read the comment docs.


Allow multiline strings to be rendered as comments

Picking this up from #45
So, docstrings are an important part of Python code and one may wish to render them as comments instead of normal strings. However, differentiating when a multiline string is being used as a docstring and when not, is tricky and probably slow.
In absence of a perfect solution, I propose the following. We split out multiline strings into their own class pythonMultiString. By default, it is linked to String like we do with so many other string types:
highlight link pythonMultiString String

So, nothing changes out of the box.
However, if users want to assume all multiline strings are docstrings, they can add this to their configs:
highlight link pythonMultiString Comment

and everyone can be happy.

Highlighting all multiline strings as comments seems really strange to me. Not sure about this, since it adds complexity and possibly making highlighting less performant.
Also, there are a few string groups that haven't been split into multiline yet in this PR.

Highlighting all multiline strings as comments seems really strange to me.

And for me, not highlighting docstrings as comments looks weird in most colorschemes. I work a lot on data analysis code and docstrings spanning 50 lines are common.
Hence a user configurable variable to turn it on/off depending on your preference.

I understand the docstring usecase, but like I said highlighting all multiline strings as comments is weird and hacky.

I think the solution here is quite useful: https://stackoverflow.com/a/43001535/210945
syn region pythonDocString start=+^\s*"""+ end=+"""+ keepend contains=...

It doesn't match all multi-line strings, only ones that fall at the start of a line (e.g. not directly following an assignment).
This will false-positively catch multi-line strings that fall inside parentheses where the string starts on a new line. I think that those cases are negligibly rare though, I think it would be worth including.

@naught101
I think it's a little more complicated, because of optional string prefices (e.g. r"""). For example, I currently use the following snippet in .config/nvim/after/syntax/python.vim to enable docstring folding:
syn region  pythonString matchgroup=pythonTripleQuotes
      \ start=+[uU]\=\z('''\|"""\)+ skip=+\\["']+ end="\z1" keepend
      \ contains=pythonEscape,pythonSpaceError,pythonDoctest,@Spell
      \ fold
syn region  pythonRawString matchgroup=pythonTripleQuotes
      \ start=+[uU]\=[rR]\z('''\|"""\)+ end="\z1" keepend
      \ contains=pythonSpaceError,pythonDoctest,@Spell
      \ fold

I think it comes from this SO post originally.

That seems irrelevant to me.. why would anyone use unicode or raw string types as docscrings?
The key thing is that docstrings are always at the start of the line, and not part of an expression. And they should be marked differently to normal multiline strings, because they are used differently.

After further consideration I think you're right. Better to use the simpler regex, and it's true I've never used raw or unicode for docstrings (I think I've seen it only once).
There are still some people who use triple single quotes (not me), but I suppose it's not recommended so overall I'm fine with start=+^\s*"""+ end=+"""+. And I agree that this would be nice to have (maybe even with support for folding...)
Anyway, thanks for pointing me to a better SO answer :)

Closing in favor of #83


[YOM-438] glow effect on nodes?

https://www.reddit.com/r/ObsidianMD/comments/177czwa/comment/k4slylu/?utm_source=share&utm_medium=web2x&context=3
YOM-438

this is also fancy thing, see #38


Adds Sky Lift Dash Game 🕹

PR Description 📜
This Pull Request addresses issue  #5110.
It introduces a new game titled Sky Lift Dash, to the repository, with the following updates.

Implemented the core game logic where players navigate a ball through a moving gap in horizontal walls.
Added dynamic wall movement and gravity mechanics.
Included a scoring system and game-over conditions.

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in GameZone's README.md
[x] I have added README.md in my Game folder
[x] I have specified the respective issue number for which I have requested the new game.
[x] I have added my Game to gamesData.json file and it is loading properly on main website as intended.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @SanskarSinghiit ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Ok, mam.

Hi there, the game logic files are not in the pr you made maje sure that files of the game are visible in the pr including readme. Md update of the repo refer contribution guidelines.


fix: target to use for cross build and bump version to 0.12.4-rc3

Testing the *-musl requires more configs, like [target.x86_64-unknown-linux-musl.linker] in .cargo/config.toml. So testing fails. See:
      - name: Test tinymist
        run: |
-          cargo test --profile=gh-release --workspace --target ${{ matrix.rust-target }}
+          cross test --profile=gh-release --workspace --target ${{ matrix.rust-target }}

Maybe we can just use cross for testing, and get rid of dealing with the complex configs...

https://github.com/Myriad-Dreamin/tinymist/actions/runs/11975350731/job/33388515141?pr=885

Seems that cross-testing *-musl targets would also require vendored openssl.

Seems that build_alpine is similiar (same?) with build linux-x64 now?

@Coekjan I've run cross test ... --features vendor-openssl, it now at least starts successfully, but it seems to be failing when downloading packages.
called `Result::unwrap()` on an `Err` value: MalformedArchive(Some("failed to create `/.cache/typst/packages/preview/tidy/0.3.0`"))

Seems that build_alpine is similiar (same?) with build linux-x64 now?

Pretty same. Now they built the same target. The only difference is in that build alpine is built in an alpine image. I suspect whether we can remove it right now, but I would rather not touching them until release 0.12.4.

I'll close it and leave this issue for next release.


WARNING: Tried to unregister TextField which is not registered

Hello, there is a lot of noise being generated by filter table by my application in the logs, because of $SUBJECT.
What causes this is having a hidden filter field for some of my columns (because it's unnecessary to filter on and causes confusion).
With this I get the warning, without I don't: table.setFilterFieldVisible("column", false);

Hi,
I also get this same warning in one application, but not in older one.
App that shows this warning uses Vaadin 7.7.5 and FilteringTable 0.9.15.v7
The older app is still using Vaadin 6.8.17 and FilteringTable 0.9.0

Maybe related to #87 ?


feature request: support for node's --env-file args

Thanks for hereby. It is an awesome small package and the best task runner I have used.
Currently, using --env-file with hereby is a little verbose,
hereby --env-file=.env test # does not load env file
node --env-file=.env node_modules/hereby/bin/hereby.js test # does load env file

It would be nice if there were a streamlined way to send an env file into the hereby process
https://nodejs.org/dist/latest-v20.x/docs/api/cli.html#--env-fileconfig

Thanks for hereby. It is an awesome small package and the best task runner I have used.

Thank you! ❤

It would be nice if there were a streamlined way to send an env file into the hereby process

Truthfully, I'm not sure how to do that; by the time hereby is running, Node's flags are already processed, and Node doesn't offer the code behind --env-file programmatically (it's implemented in C++). I'd have to reexec another process to pass those flags in, or, take a dependency on a different lib like dotenv, which I'd prefer not to do.
It may be easier to just do something like env-cmd hereby, which would auto-load .env and run any command; I honestly expect every Node CLI to have this problem...

understood, thanks :)


bump version of node-gyp up to 4.0.0

So that we're not using insecure version of tar anymore.  (see https://nvd.nist.gov/vuln/detail/CVE-2018-20834 )

Please search closed issues before opening new ones. There is a dozen identical closed issue.

My apologies, I did do a brief search and found nothing, but maybe I didn't look for the right thing...

So, the fact that so many people did not and still have not seen what you're doing about this problem would seem to indicate that another way of communicating it is in order.  Maybe a brief note in the readme? Maybe leaving an issue open? Maybe...?

@steevhise we did have the pinned issue and I had it in the PR template (but removed it after the upstream issue was resolved), but neither seemed to discourage people from opening PRs. Do you want to suggest a PR to the Troubleshooting.md maybe?

oh I wasn't aware of the resolution. Github still gave me an alert on one of my repos. But now I see that after I re-installed I get node-gyp 3.8.0 which uses a safe version of tar.  sorry! thanks.

I think the issue is really not yours but with the security audit systems (for both npm and github). there should be more information about troubleshooting coming from them, rather than just an alert and an "autofix" button (which I don't trust and i assume a lot of others don't either).

no problem, thanks for the feedback. Maybe we'll keep the issue pinned for longer after the resolution. Not sure what a reasonable timeframe is, but we can keep experimenting


Unable to RDP other than within same computer

After further troubleshooting I noticed that Windows Firewall was blocking my connection, I was using a custom RDP port and had to make a rule for it under MS Firewall.


Update -shard option for PR 22998 for InfluxDB OSS 1.10

If InfluxDB OSS 1.10 is released, then then -shard backup documentation needs to be updated to match the improvements in https://github.com/influxdata/influxdb/pull/22998/
Closes #
Describe your proposed changes here.

[ ] Rebased/mergeable

@gwossum --want to loop back around on this one --give we never released OSS 1.10, and this option was added to the Enterprise docs--- to be sure, this option was not back ported to OSS 1.8? thanks!


Bug: v2_playbook_on_start - 'list' object has no attribute 'startswith'

Seeing the following when using the latest version of this callback plugin w/ Ansible 2.4.0 (devel 53c46df323), python version = 2.7.9
2017-06-03 09:15:16,728 p=25592 u=root |   [WARNING]: Failure using method (v2_playbook_on_start) in callback plugin (</tmp/ansible/callback_plugins/datadog_callback.CallbackModule object at 0x7f38477cef90>): 'list' object has no attribute 'startswith'

Happy to provide any other information that might be useful here.

Hi @jaredledvina, thanks for reporting this issue!
I've been able to reproduce this issue on a local Ansible 2.4.0 devel environment, queuing some work to solve this in the next release.

Hey @olivielpeau, Ansible 2.4.0 is officially released (yay!), any word on an ETA with the fix for this?
Here's the entire traceback I was able to pull:
 [WARNING]: Failure using method (v2_playbook_on_start) in callback plugin (<ansible.plugins.callback.datadog_callback.CallbackModule object at 0x75db31d0>): 'list' object has no attribute 'startswith'
Callback Exception:
  File "/opt/ansible/lib/ansible/executor/task_queue_manager.py", line 366, in send_callback
    method(*args, **kwargs)
   File "/tmp/ansible/callback_plugins/datadog_callback.py", line 219, in v2_playbook_on_start
    self._handle_playbook_on_start(playbook_file_name, inventory)
   File "/tmp/ansible/callback_plugins/datadog_callback.py", line 125, in _handle_playbook_on_start
    inventory_name = os.path.basename(os.path.realpath(inventory))
   File "/usr/lib/python2.7/posixpath.py", line 382, in realpath
    path, ok = _joinrealpath('', filename, {})
   File "/usr/lib/python2.7/posixpath.py", line 388, in _joinrealpath
    if isabs(rest):
   File "/usr/lib/python2.7/posixpath.py", line 61, in isabs
    return s.startswith('/')
1 plays in /tmp/ansible/host-test.yml

Hi guys, any update on this fix? We're also on ansible 2.4, and this is breaking our integration.

Hi @jaredledvina,
Sorry for the very late fix ! I just merge a PR that fix the inventory for ansible 2.4 and tagged with 1.0.2.
Closing this issue now, but feel free to reopen it if needed.


⚠ Bofrost.de has degraded performance

In 30028ae, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1509 ms

Resolved: Bofrost.de performance has improved in b2f1765 after 6 minutes.


CNV-35782: Disable deleted NIC when VM is stopped

📝 Description
If a NIC is deleted when the VM is stopped it will not be removed until the VM is started. This PR disables the NIC to avoid confusing the user.
To Do: Determine if there are any other items that don't change until the VM is started.
Jira issue: https://issues.redhat.com/browse/CNV-35782
🎥 Demos
Before
https://github.com/kubevirt-ui/kubevirt-plugin/assets/8544299/1344aa39-b2ba-480f-891e-faaa60b43e72
After
https://github.com/kubevirt-ui/kubevirt-plugin/assets/8544299/f9f6a886-1d66-47c8-8ff4-07b8d423fd9e

Looks good, tested, works for me. And after the VM got started, the deleted NIC disappeared from the list as expected 👍🏽


[WIP] terraform: provide plugins for 0.10

Obviously very much a WIP, but this is the sort of thing I had in mind. I don't really care how we plug the plugins into it, but this way seemed to work and didn't take too much hackery.
To use, try:
terraform_0_10.withPlugins [ terraform_0_10.providers.aws ]
That should get you a version of terraform that:

Uses the aws provider compiled from source by Nix
Doesn't automatically download precompiled providers from their plugin repository

Feedback welcome! I've been considering making withPlugins take a function from the available plugins for that version of terraform, so you could write something like withPlugins (p: [ p.aws ]) or something.
Motivation for this change
https://github.com/NixOS/nixpkgs/issues/28537
Things done

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

[ ] NixOS
[x] macOS
[ ] Linux

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

I like the haskell set convention of the withPlugins taking the set itself, to cover @zimbatm's case we should just include p.callPackage or something like that that can be used to import a plugin expression with the current terraform as the base.

Any thoughts on withPlugins vs. withProviders? The only plugins I've heard of are providers but I don't know if it's planned to be exclusively providers.
Also @shlevy, are you saying you'd prefer withPlugins to take a function like I had in my example?

No strong feelings on the name, yeah I think it should take a function

No strong objections either. If Haskell takes a function we could do the same for consistency's sake.

A function it is!

@shlevy @zimbatm any thoughts on how to handle the override* cleanly here? I want this stuff to stack naturally, but the most obvious way to do that would also cause TF to recompile whenever changing plugins and that seems shitty. I can add special handling for overrideAttrs and override to shuffle around the composition order a bit (so that withPlugins can still be overridden as usual and it composes nicely) but it feels messy.
Otherwise I think this is ready.

Also, y'all feel welcome to add other plugins!

Okay, I now have AWS, Google, Azure, and K8s plugins, and I've got a not-beautiful-but-not-awful solution to the override question. You can now write this monstrosity and have it work:
$ nix-build -E "with import ./. {}; (((terraform_0_10.withPlugins (p: [ p.aws ])).withPlugins (p: [ p.azurerm p.google ])).overrideAttrs (orig: { name = orig.name + \"foo\"; })).withPlugins (p: [ p.kubernetes ])"

I'm going to merge this but I welcome improvements, new plugins, or suggestions on how to do the override crap more nicely.

Actually, the way I homogenized the withPlugins stuff means that it won't ever auto-download anymore. I'll fix that.

Fixed in 11753b3273ac492a4efdd44ae98f1969f554b126

LGTM & composable! ❤


Convert int64 type to Dart string

For some reason, int64 is returned as string from the Telegram server.
Adjust the generator to generate string type for int64 fields.
This fixes https://github.com/i-Naji/tdlib/issues/15.

Yes you are right, but let them be annoniated as int (avoiding fault in convert to json!) and convert that string to int when parsing data (in fromJson method)


KAFKA-3070: SASL unit tests dont work with IBM JDK

Use IBM Kerberos module for SASL tests if running on IBM JDK
Developed with @edoardocomar
Based on https://github.com/apache/kafka/pull/738 by @rajinisivaram

@ijuma @rajinisivaram can you have a look ?

Btw, please don't squash commits. It makes it hard to know what has changed since the previous review.

I've pushed an update that should address the reviews

@rajinisivaram I rebased and it's now clean to merge

@mimaison Thanks for the PR. Merging to trunk.


Appstudio purge rhtap-demo-component-byyo

Pipelines as Code configuration removal

Pipelines as Code CI has failed.There was an error creating the PipelineRun: rhtap-demo-component-byyo-on-pull-request-
creating pipelinerun rhtap-demo-component-byyo-on-pull-request- in namespace rhtap-demo-dev-upho-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Are the results recorded? If so are they accessible?

I am very interested to see what the data could tell us about the political views of people, and is the data generated from the site available?

Duplicate of #29.


[Chore]: Inject presenter into handlers

Background
The api/handler tests against JSON output. This means test cases with different results need to check different JSONs. This is verbose and moderately painful to write.
If the presenter were injected into the handlers, we could fake it in the handlers tests. This would mean the tests could assert the records or record lists passed into the presenters were correct, and that the faked output from the presenter was returned as JSON, simplifying the tests.
Action to take
No response
Impact
...
Dev Notes
We did a brief spike in spike/extract-presenter

We didn't inject the presenter in the end. It made more sense just to test the presenter output in the presenter package, and to prune down the tests in handlers never to test the whole JSON. The new MatchJSONPath gomega matcher helped things a lot.


Everywhere: Inconsitent identifier formatting

Currently snake_case is mixed with camelCase in variable and function naming all over the project.

i wouldnt call a single file "all over the project" (githubAPI), but yes naming should be consistently camelCase :)

Oh its actually just githubApi, the references to it made it seem everywhere


Wrong swagger definitions for POST /admin/v2/namespaces/sample/ns1/deduplicationSnapshotInterval

This endpoint doesn't have DELETE method. The undefined request body is used instead to remove this policy.
Source: https://github.com/apache/pulsar/blob/e6b12c64b043903eb5ff2dc5186fe8030f157cfc/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java#L1395
The topic-level policy has DELETE method. Probably there is a sense to implement it for namespace-level policy.
Source: https://github.com/apache/pulsar/blob/e6b12c64b043903eb5ff2dc5186fe8030f157cfc/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/PersistentTopics.java#L552

Closing as stale and no one worked on it. Please open a new issue if you volunteer to do it.


[res] Sample circle model for unit testing

This will introduce sample circle model for CircleGraph unit testing.
ONE-vscode-DCO-1.0-Signed-off-by: SaeHie Park<EMAIL_ADDRESS>

from #966


Improve deploy script to manage owners

In the current situation all deployed contracts are owned by the admin address, therefore we have to manually transfer the ownership of the nft contract from admin to minter contract.
Using starknet invoke we should be able to handle thise automatically after deployments.

protostar cannot invoke transferOwnershop yet (protostar#606).
Based on this, we have the following choices:

Do nothing and wait for protostar update
Use starknet invoke until next protostar update


3963 switch beanstalk ec2s to imdsv2

what
Add setting to disable IMDSv1
why

references

 Switch beanstalk EC2s to IMDSv2


Allow custom marshalers to provide schema information

This PR allows types that implement custom marshaling to provide a schema for their custom-marshalled data, to take advantage of the recursive processing of structs and complex types

@emicklei any thoughts on this?

I seemed to missed this one. Why was it closed? Is it still applicable to the current master?


proxy support

Could you please add proxy support. Its very easy.
example
#proxy = {
'http': "socks5://<IP_ADDRESS>:1080",
'https': "socks5://<IP_ADDRESS>:1080",
#}
class Client:
def init(self, cookie, proxy=None):
self.proxy = proxy
and all request.get(post etc) add proxies=self.proxy
response = requests.get(url, headers=headers, proxies=self.proxy)

Sure @theurs , this is most requested and will update in next version of code.
Thanks too 😄 !


Unreachable code in dai::DataOutputQueue::get() -- unclear intention

Unreachable code in dai::DataOutputQueue::get() -- the intention is unclear. Needs a review.
On line 175-6, do you want to throw, or return nullptr. Can't do both.
https://github.com/luxonis/depthai-core/blob/efa3deb7c17e223238696416cd45d2d3eb35674f/include/depthai/device/DataQueue.hpp#L171-L179
Setup

all OS
all compilers
depthai-core v2.13.2

Repro
MSVC compiler told me it is unreachable.
Verified by code review. It is unreachable.
Result
The code will throw. But it is unclear if that is the dev's intention.
Expected
Code matches dev's intention.

return nullptr is redundant in this case - the underlying queue was destructed in this case.
If the underlying queue was destructed, then the caller is given the reason why and gets out of the blocked call.

verified it is merged into develop codebase


Move fluent-bit persistent folder from /opt/fluent-bit to /var/fluent…

/area logging
/kind bug
This PR moves the fluent-bit persistent storage from /opt/fluentbit to /var/fluentbit. This is required to fix the case where we have a read-only root file system on the node, usually due to security hardening of the nodes. In those cases where the /opt is in readonly mode the fluent-bits will not be able to start.
Moves the fluent-bit persistence folder from /opt/fluentbit to /var/fluentbit.

Thanks!
/milestone v1.71


Update to GTK 4

Please move from GTK 3 to 4

duplicate of https://github.com/PolyMeilex/rfd/issues/108


red tiles like errors showing up near the line numbers indentation.

Type: Bug
red tiles like errors showing up near the line numbers indentation.
 Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


CRUD Fields with tab on CREATE not on UPDATE

Hey,
Weird issue here I think.
When addField with parameters 'tab' => 'My Tab' AND with view only 'create'

If you try to addField without parameters 'tab' for view UPDATE, then Backpack CRUD is not showing is as it seems it assumes it should be in a tab:

Result: nothing

HTML code added tab for reason I ignore:

That's odd but it seems that once one has a mention of any tabs, then everything needs a tab parameter/key.
Would you consider this a:

bug?
gotcha?
something else?

I think it's more of a gotcha myself.

@lloy0076 Well yes if one has 'tab' then all should have too, if not they will be added in the last TAB. Which also create another weird behavior that you currently cannot mix TAB with other type. (Ex have an input for title at the top of the page and below few tabs).
I think this is a bug because CREATE and UPDATE fields can be set/ should be set separately. Or now if I have TABS in CREATE then it forces me to use TABS in UPDATE.

@tabacitu - what do you think? It's somewhat broken - and I agree with @MickaelTH - but I think:

In the meantime we might look at the documentation and at least make it obvious that if you have tabs then EVERYTHING needs a tab; and
Fix as suggested so that the vraious fields can (or should) be able to be set separtely.

Hi,
I agree as well that a user should be able to set-up tabs differently on create and update.
What I'm not feeling so confident about is having tabs plus fields outside of the tabs. That just seems a little bit too much and for cases like the example above with an input for a title at the top of the page, I would usually just go with a 'General' tab or equivalent.
I am not familiar with all the use cases though, so I would really appreciate more input on this.
@tabacitu, any views on this?

If we decide to go with tabs only, a form can then either have tabs or not, like the current behavior. This should then become a crud panel configuration.
I see the crud panel API like this:
setFieldTabs(array $tabNames, $form = 'both')
getFieldTabs($form) // either 'create' or 'update'
clearFieldTabs($form = 'both')
All fields from a certain form that contain a valid tab name go in the specific tab, otherwise go in a generic tab at the end.
If the crud panel field tabs array is empty, there should be no tabs regardless of fields having the 'tab' property.

Hi guys,
I wish I saw this sooner. I merged a PR on the upgrade branch this week which introduced a feature - not specifying a tab will now place that field before the tabs:

I now think this is a more intuitive behaviour. When you don't specify a tab, you'd expect it not to be in a tab... Wouldn't you agree?
--
About adding fields in a certain tab on only create or only update, I think @MickaelTH 's syntax is spot-on. I would have expected that to work, so I'd say this is a bug.
@tumf87 I think adding public setFieldTabs(), getFieldTabs(), clearFieldTabs() would just make it more difficult to understand how to use tabs. I'm ok with them as private, if we need them to make it work, but not public methods, since I don't believe they're actually needed. I think @MickaelTH 's syntax is pretty darn intuitive - if you'd like separate fields on separate tabs, just add them twice, specifying a different tab each time.

My opinion on this:

we keep the option to have fields above the tabs, which is already merged in upgrade
we "make it work" with @MickaelTH 's syntax

Do you guys agree/disagree?

@tabacitu, agreed. Like I said, those methods would have only worked in case we decided to go with tabs only, not fields + tabs.

Please re-open if more discussion is needed.

This seems to be broken...?
In a project I defined tabs for update, but don't want them in create. I get this error then:

Thank you @tswonke - you’re right, there was a bug when the tabs were only defined for update, or create. Fixed in
https://github.com/Laravel-Backpack/CRUD/commit/6de4545dc44d8f7afa852678dcf89db24d2d3e98 and tagged. A composer update should fix it for you.
Cheers!


fix Chinese path issue

fixed resource load problem with Chinese characters in the path.

Microsoft's documentation states that the file path must be percent encoded (especially for slash characters), but it seems their percent-decoder does not correctly support Unicode. The best solution for this issue is likely going to be to use a different URIEncode function that leaves Unicode characters alone, not removing the encoder altogether

Should be closed when a better URIEncode method becomes available.


Reading providers.fileContents(regularFileProperty) doesn't make the value of the property finalized

Current Behavior
When the value of providers.fileContents() on a RegularFileProperty with finalizeValueOnRead() called is read, the value isn't actually finalized.
Expected Behavior
When the value of providers.fileContents() on a RegularFileProperty with finalizeValueOnRead() called is read, the value should be finalized.
Context (optional)
No response
Steps to Reproduce
val file = objects.fileProperty().apply {
    convention(layout.projectDirectory.file("file1")).finalizeValueOnRead()
}
providers.fileContents(file).asText.get()
file = layout.projectDirectory.file("file2")

This doesn't throw any exception.
Gradle version
8.7
Build scan URL (optional)
No response
Your Environment (optional)
No response

The issue is in the backlog of the relevant team and is prioritized by them.

It might actually affect all ValueSource implementation, we need to check.


Generated GPX files with empty tags

Is it mandatory to generate large GPX files with these empty tags ?

gpxtpx:TrackPointExtension
</gpxtpx:TrackPointExtension>

Thanks ! Files size is really reduced.


Signal shifted by phase is not properly drawn

I'm using the web editor on the latest mozilla firefox.
The following json
{signal: [
  {name: 'clk', wave: 'P..'},
  {name: 'EN', wave: 'X01', phase: 0.5},
]}

produce this visualization:

However, if I change the phase to something else:
{signal: [
  {name: 'clk', wave: 'P..'},
  {name: 'EN', wave: 'X01', phase: 0.2},
]}

The EN signal does not properly start from the beginning of the plot

X is not supported. Use x instead.

X is not supported. Use x instead.

Besides 'X' and 'x', with any phase that is not a multiple of 0.5 the signal does not start from the beginning of the plot but it is slightly shifted to the right.
If you compare the two images I have uploaded above you should see how with phase 0.5 the signal start from t=0 while with a different phase it starts from an arbitrary position.
Another example:
{signal: [
  {name: 'OK', wave: '010....', phase: 0.5},
  {name: 'NOT_OK', wave: '010....', phase: 0.1}
  ]
}

wavedrom diagram is constructed from standard building blocks (bricks) of fixed size.
With period=1 there are 2 bricks per period.
With period=2 there are 4 bricks.
Choosing narrow skin and period > 1 will give you more phase accuracy.
Like this:
{
  signal: [
    {name: 'OK0', wave: '010....', period: 2, phase: 1.5},
    {name: 'OK1', wave: '010....', period: 2, phase: 1},
    {name: 'OK2', wave: '010....', period: 2, phase: 0.5},
    {name: 'OK2', wave: '010....', period: 2},
  ],
  config: {skin: 'narrow'}
}


Popper with Pigeon Maps

Hello,
I read a blog that said they used popper with pigeon maps to create tooltips.  I am trying to create tooltips that give the name of the location when a user hovers over or clicks (haven't decided) a marker.  I am following a tutorial but it is not rending a tooltip for me so I am unsure how to proceed.  It may be a pigeon maps issue but I thought it was worth asking. I originally tried the ref on the marker but then wondered if I needed something surrounding the marker.  Neither worked!  Here is my code:
`const markerRef = useRef()
const tooltipRef = useRef()
const {styles, attributes} = usePopper(markerRef.current, tooltipRef.current)
return (
<Typography variant="h4" sx={{ m: 0 }}> Routes
    <Grid item sx={{ width: '100%', height: '45rem' }}>
      <Map provider={maptilerProvider} defaultCenter={[44.92306458149522, -93.30491066897952]} defaultZoom={14}>
       {markers.map((oneMarker, index) => (

         <Marker
            width={50}
            anchor={[oneMarker.lat, oneMarker.long]}
            key={index}
       />

              Example Tooltip

       ))}
      </Map>
    </Grid>
</Grid>

);
}
`

Hi 👋 , this seems more like a question for Stack Overflow or the blog author, as I don't know what Pigeon Maps is and there's no reproducible demo.
react-popper has its own repo located here.


Push/Pull Freeze on Windows

The problem I experienced trying to get SublimeGit/ST2 to run on Windows was that any remote git call caused ST2 to lock up and freeze. This seems to be a known issue, and the problem is caused by Git requiring the ssh passphrase for any push/pull operation. The documentation points here, which uses ssh-agent to store credentials. This allows Git Bash to only ask for the passphrase on first run and not any time after. However, I still experienced the same issue.

Thanks for the tip @albertosantini. What a great integration into the windows workflow! For some reason I just needed an extra backslash after '%USERPROFILE%:
C:\My\Path\pageant.exe %USERPROFILE%****.ssh\id_rsa.ppk -c C:\My\AnotherPath\SublimeText\sublime_text.exe


Updating environment.yml used by RTD for a release after release has been made

Details

Read the Docs project URL: https://readthedocs.org/projects/aospy/
Build URL (if applicable): https://readthedocs.org/projects/aospy/builds/8103255/
Read the Docs username (if applicable): spencerahill

Expected Result
n/a
Actual Result
We recently released v0.3 of aospy.  After the release, we realized that we had failed to correctly update the dependencies in our docs/environment.yml, such that the v0.3 builds on RTD are failing.  I have since updated it, such that our master and 'latest' builds are passing (e.g. here).
I suspect that the answer is 'no', since it's tied to the v0.3.0 release on Github that has already been finalized, but: Is there someway on the RTD side to modify the builds for v0.3.0 only to work around this?
I hope that makes sense.  Thanks in advance for the help.

Sorry, we can't edit any files on the servers, that would break the build process (fetching the repo, changing branch/tag and build).
What about doing a 0.3.1 release?
You may be interested in https://github.com/rtfd/readthedocs.org/issues/1340 to catch this kind of errors early.

Sorry, we can't edit any files on the servers, that would break the build process (fetching the repo, changing branch/tag and build).

Thanks @stsewd .  I figured as much, but worth checking!

What about doing a 0.3.1 release?

Yes, this is probably what we'll end up doing.  Lesson learned!

What about doing a 0.3.1 release?

Cool, will look into this.
I'll close this.