TLS/ldaps ConnectionError: Can't contact LDAP server. (unknown error code) with Asyncio

Hello! I really appreciate the work you have put into this library!
I am getting an issue where if I try to enable ldaps/TLS in my code, it fails to connect to the server.
When I was using the ldap3 library it would be able to connect to the server with ldaps so I am unsure why I am having this issue.
For example, when I just have TLS enabled:
#!/usr/bin/env python3

import asyncio
import bonsai

async def do():
    client = bonsai.LDAPClient(f"ldap://{remote}", True)
    client.set_credentials("SIMPLE", user=username, password=passphrase)
    async with client.connect(is_async=True) as conn:
        who = await conn.whoami()
        print(who)

loop = asyncio.get_event_loop()
loop.run_until_complete(do())

It returns this Traceback:
Traceback (most recent call last):
  File "./testing.py", line 15, in <module>
    loop.run_until_complete(do())
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "./testing.py", line 10, in do
    async with client.connect(is_async=True) as conn:
  File "/home/testing/.local/lib/python3.8/site-packages/bonsai/asyncio/aioconnection.py", line 25, in __aenter__
    return await self.__open_coro
  File "/home/testing/.local/lib/python3.8/site-packages/bonsai/asyncio/aioconnection.py", line 59, in _poll
    raise exc
  File "/home/testing/.local/lib/python3.8/site-packages/bonsai/asyncio/aioconnection.py", line 54, in _poll
    return await asyncio.wait_for(fut, timeout)
  File "/usr/lib/python3.8/asyncio/tasks.py", line 455, in wait_for
    return await fut
  File "/home/testing/.local/lib/python3.8/site-packages/bonsai/asyncio/aioconnection.py", line 40, in _ready
    res = super().get_result(msg_id)
bonsai.errors.ConnectionError: Connect error. (unknown error code) (0xFFF5 [-11])

It shows the same results when I use ldaps.
Do you have any insight into this issue?

Hi, could you check the debug logs? You can turn it on right after the module import with bonsai.set_debug(True, -1).

Wow! Those debug logs are really nice. The issue was due to the certificate expecting a domain name such as website.com but I was giving it the IP address xxx.xxx.xxx.xxx


DOC: PchipInterpolator: missing integrate function

Issue with current documentation:
Documentation for PchipInterpolator misses integrate function in methods section.
Doc link: https://scipy.github.io/devdocs/reference/generated/scipy.interpolate.PchipInterpolator.html
The function itself is available for this class, it was inherited.
Link: https://scipy.github.io/devdocs/reference/generated/scipy.interpolate.PchipInterpolator.integrate.html
Idea or request for content:
No response
Additional context (e.g. screenshots, GIFs)
No response

Would you be interested in making a pr?

i am intrested in this issue..

Please feel free to open a pr @Nayan2003


[#32] Add raw tokens and semantic tokens for elevation

Review and fix draft raw and semantic tokens for elevation. Add unit tests for elevations raw tokens.
Add unit tests for eelvation semantic tokens overriding. Add unit tests for inverse theme colors.
Note: Please transform - [ ] into - (NA) in the description when things are not applicable
Related issues
#32
Description

Review and fix draft raw and semantic tokens for elevation.
Add unit tests for elevations raw tokens.
Add unit tests for eelvation semantic tokens overriding.
Add unit tests for inverse theme colors.

Definition of Done

[x] Check and update raw tokens
[ ] Implement all raw tokens
[x] Check and update semantic tokens
[ ] Implement all semantic tokens
[x] Add unit tests for raw tokens (order relationship for example)
[x] Add unit tests for semantic tokens (override by subtheme for example)
[x] If colors, add unit tests for inverse theme (same colors in light and dark modes)

Motivation & Context
See #32
Types of change

New feature (non-breaking change which adds functionality)

Previews

Checklist

Contribution

[x] I have read the contributing guidelines

Accessibility

(NA) My change follows accessibility good practices

Design

(NA) My change respects the design guidelines of Orange Unified Design System

Development

[x] My change follows the developer guide
[x]  I have added unit tests to cover my changes (optional)

Documentation

[x] My change introduces changes to the documentation and/or I have updated the documentation accordingly

Checklist (for Core Team only)

[x] The evolution have been tested and the project builds for iPhones and iPads
[x] Code review has been done by reviewers according to CODEOWNERS file
(NA) Design review has been done
(NA) Accessibiltiy review has been done
(NA) Q/A team has tested the evolution
[x] Documentation has been updated if relevant
(NA) Internal files have been updated if relevant (like CONTRIBUTING, DEVELOP, THIRD_PARTY, CONTRIBUTORS, NOTICE)
[x] CHANGELOG has been updated respecting keep a changelog rules and reference the issues

Pull requets is still ind raft mode because some raw and semantic token is missing ("Box shadow (Composite)").
Need more details to implement them, see https://github.com/Orange-OpenSource/ouds-ios/issues/32#issuecomment-2269180064


Remove pytest from environment requirement file

This isn't going to work. Pytest is needed for the tests to run


Perform full syntax rendering

The current method of rendering the minimap doesn't draw the full syntax highlighting as seen in the document, only the base theme colors.
#2 is the first step, but ideally I would like to fully render the document just as it is done in the editor.
Unfortunately, the APIs used to render the full document with syntax don't appear to be available to plugins.

Possibly related:
The feature I miss the most is double-click highlighting instances of a word throughout the document, and seeing that reflected in the atlas. This is by far the fastest way to search for instances in a file.
I really look forward to seeing if Rider makes the necessary queries possible. Happy to encourage them in this direction on your behalf!


Need for ENaddrule and ENaddcontrol functions

We already have the get\set functions for rules and controls but the ability to add (and also remove) them is still needed.

Been thinking about the ENaddcontrol function. My idea is to have it like the ENsetcontrol one and to return the new control index. Any thoughts on this?

@LRossman I already added the EN_addcontrol API, do you have the EN_addrule API in the work you are doing now or we are moving this to v2.3?

@eladsal I can add an EN_addrule and associated functions (EN_addpremise, EN_addthenaction, EN_addelseaction) by the end of the week if that's ok.

@LRossman I think that would be perfect, thanks!


Release of version <IP_ADDRESS> of xWebAdministration

Releasing version <IP_ADDRESS>

This change is

Codecov Report

Merging #436 into master will not change coverage.
The diff coverage is 37.5%.

@@           Coverage Diff           @@
##           master     #436   +/-   ##
=======================================
  Coverage   90.77%   90.77%
=======================================
  Files          17       17
  Lines        2438     2438
=======================================
  Hits         2213     2213
  Misses        225      225

Impacted Files
Coverage Δ

...sFeatureDelegation/MSFT_xIisFeatureDelegation.psm1
100% <ø> (ø)
:arrow_up:

...FT_xWebConfigKeyValue/MSFT_xWebConfigKeyValue.psm1
92.5% <ø> (ø)
:arrow_up:

...WebVirtualDirectory/MSFT_xWebVirtualDirectory.psm1
88.63% <ø> (ø)
:arrow_up:

...SCResources/MSFT_xIisLogging/MSFT_xIisLogging.psm1
97.74% <ø> (ø)
:arrow_up:

...SCResources/MSFT_xWebAppPool/MSFT_xWebAppPool.psm1
94.16% <ø> (ø)
:arrow_up:

DSCResources/Helper.psm1
90% <ø> (ø)
:arrow_up:

...SCResources/MSFT_xIIsHandler/MSFT_xIisHandler.psm1
100% <ø> (ø)
:arrow_up:

...Resources/MSFT_xSSLSettings/MSFT_xSSLSettings.psm1
78.43% <ø> (ø)
:arrow_up:

DSCResources/MSFT_xIisModule/MSFT_xIisModule.psm1
2.91% <0%> (ø)
:arrow_up:

...s/MSFT_xWebSiteDefaults/MSFT_xWebSiteDefaults.psm1
6.66% <0%> (ø)
:arrow_up:

... and 3 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 91ef234...70aad44. Read the comment docs.

merging since this is code coverage and not functional


Itk 5.4.0, Python CI

Windows packages fail with Cannot find path 'D:\a\im\dist' because it does not exist.


Show "downstream" jobs in lab ci view

https://docs.gitlab.com/ee/ci/multi_project_pipelines.html

May be blocked by https://gitlab.com/gitlab-org/gitlab-ce/issues/44209

Looks like the "list bridge pipelines" api would expose a path to implementing this https://docs.gitlab.com/ee/api/jobs.html#list-pipeline-bridges

Support for multi-project pipelines were added through PR #579


[precommit] fixed yapf compatiblity issue with pyproject

📌 Checklist before creating the PR

[x] I have created an issue for this PR for traceability
[x] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[x] I have added relevant tags if possible for us to better distinguish different PRs

🚨 Issue number

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

Fixed #3016 .
📝 What does this PR do?

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

This PR fixed the pre-commit error due to the compatibility issue between .pre-commit-config.yaml and pyproject.toml.
Below is a screenshot of successful pre-commit run after this bug fix.

💥 Checklist before requesting a review

[x] I have linked my PR to an issue (instruction)
[x] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[x] I have performed a self-review of my code
[x] I have added thorough tests.
[x] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[x] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

This PR is no longer needed due to #3022


NECO-185: Added meta APIs on README

Ticket

https://pflabo.atlassian.net/browse/NECO-185

Changes

README

Added meta APIs
Added Requirements and References
Deleted License
Fixed some sentences.

Source code

Added empty methods (addMeta, removeMeta)
Modified meta method.

Tests

See the ticket.

How It Looks Like

https://github.com/okunoyuki/media-storage-rb/tree/feature/NECO-185

I reviewed all the changes.
I'm going to merge this pull request.


Исправил ошибку indicies

hi Dmitry! Thanks for your PR, I noticed quite a few things were not up to date here anymore, so I decided to bump a few version myself see: https://github.com/wimspaargaren/yolov5/commit/c740066188630e1584dab48bcef1ba94a110eb8a
Also created a v0.0.1-alpha version, hope that helps!


Downloading latest, make it executable, and use relative pathing for …

…the executable.

This will still not work as the release seems to be dynamically linked. I am not sure how to statically link this program, nor all the required system level packages in the container. Would you be able to implement that at all?


Not really helping on 4k display

I tried it on Linux Mint Cinnamon 18, gimp 2.8.16, with my 4k monitor at 3840x2160 resolution. Icons are still too small but the worst thing is that gui controls are not scaled at all. I googled a bit but didn't find any workaround solution. Am I missing something?

Yeah I've never tested a 4k display. In the readme I mention (Macbook Pro Retina / X1 Carbon), which is what I lightly tested with.
Also from the readme:

In summary, I just replaced the icons with flat-style SVGs and changed the background color.

So yeah the GUI controls are still bad.

https://raw.githubusercontent.com/brion/gimp/0ddbf9b3732e11a5e7033418c17732295183a885/themes/HiDPI/gtkrc

https://raw.githubusercontent.com/brion/gimp/0ddbf9b3732e11a5e7033418c17732295183a885/themes/HiDPI/gtkrc

A little explanation would be helpful. Where does this go?


fds init

I successfully pip installed fastds, but anytime I did fds init, it always gives this import output after displaying some Traceback information:
ImportError: cannot import name '__version__' from 'fds.__init__' < c:\users\<MyComputerName>\pneumonia-detection\.venv\lib\site-packages\fds\__init__.py>

@Chibuzo75 strange! I would expect this to happen to use as well.
Maybe it's a windows problem.
Could you please share the Traceback information?

And also maybe print the contents of c:\users\<MyComputerName>\pneumonia-detection\.venv\lib\site-packages\fds\__init__.py ?

I think this error might be resolved if we move __version__ into its own file version.py

Hi Guy,
Thank you for reaching out to me. Here are the trace back calls from usin both fds init and fds add.d: they gave the same output
On Wednesday, 15 September 2021, 03:13:48 GMT-4, Guy Smoilovsky @.***> wrote:
@Chibuzo75 strange! I would expect this to happen to use as well.
Maybe it's a windows problem.
Could you please share the Traceback information?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

@Chibuzo75 I hope this will be fixed in the next deployed version, I'll update here when it's deployed and you can test again

@Chibuzo75 Hey, it would be great if you could test it with the newly deployed 0.5.0 version or newer, I'm interested to learn whether it fixed things.
I have to say I'm also not familiar with how conda runs things on windows (or in general) with runpy.py and how it affects things. Maybe a normal python installation would work? Can you ensure that the version of python you're getting is python 3?

@guysmoilov Yes I do have python 3.8.2. The initial error is not displaying again. Now, it's most likely my local computer issue since it shows this when I type the fds init,

FYI: Dean Pleban has assisted me in completing the Pneumonia-Detection project but without the fds init.

Thanks @deanp70 !
@Chibuzo75 well I would be happy to hear updates here if there are any, thanks!


OODT-925 Update schema.xml field definitions for SOLR 5.X

This is a new PR for master which addresses https://issues.apache.org/jira/browse/OODT-925

+1 from me here to commit thanks @lewismc


PDF bookmark encoding screwed up

As demonstrated in tests/pdf.sil, bookmarks are indeed generated by packages/pdf, but the character encoding of the titles is goofed up. PDF requires some crazy encapsulation for that data (you can't just cram plain text in that field). I can't remember exactly it has to be done, but I do remember having this problem back in the day when I wrote my own PDF library. In any case the current output of that test is actually a borked PDF.

Huh. I thought we'd fixed that - @shirat74 added UTF8 to UTF16-BE conversion, and UTF-16 should be acceptable. The PDF (and bookmarks) look fine here in Preview and in Adobe Acrobat, and I can't get any of the Acrobat preflight checks to show any errors. In what sense borked?

Borked as in neither Acrobat or Evince show a readable index:

Huh. What Lua version and do you have the utf8 library installed?

I default Lua runtime that I'm using for everything but regression testing is 5.3.1. And I have luautf8 installed (although it's 1.0 not 1.1) but not utf8. BTW should that be an install time dependency? If it matters for the core packages to function as expected I think it should be...shall I add it to #119?

If you're using 5.3, then it comes with the utf8 library baked in. And I don't trust it yet, because it was throwing strange errors like this. (That was actually failing to decode a "\n")

So is the utf8 luarock required for lua < 5.3?

Should not be required; lua 5.3 contains a utf8 library, which has a compatible interface to luautf8, but different code.

My question was about for Lua 5.2 etc. Do the lower versions of lua have an extra dependency that 5.3 doesn't? In other words should we be installing something more as a luarock for systems with older Lua versions?

No. utilities.lua contains implementations of all the UTF8 functions we need; Paul K noticed that these could also be implemented by the utf8 library, so if that library (either the 5.3 built-in one, or the luautf8 rock) is available its functions are used instead. But our functions are available as a fallback.

Got it. So where does that leave my long strings of Arabic hanging off the page?  But I guess we should go back to #123 for that since that's a different issue.

OK, so revenons à nos moutons, I have PDF bookmarks working on Lua 5.2, but you don't have them working on 5.3. Could you try (again) disabling the require("utf8") in utilities.lua and seeing if that helps? If not, could you then try printing out the value of ustr in pdf.lua:28 (piping through hexdump or moral equivalent if necessary) and seeing if it is correctly encoded UTF-16BE?

Commenting out require("utf8") has no effect.
Dumping the value of ustr does not net me a valid UTF16BE string as far as I can tell. Lua dumps it to the console as feff000100020003000400050006000700080009000a000b, and if I have Lua write it to a process pipe inconv declares it to be bogus:
local fp = io.popen("iconv -f UTF-16BE -t ASCII", "w")
fp:write(ustr)

Gives:

iconv: illegal input sequence at position 0

On the other hand feeding title to iconv and asking it to convert from UTF-8 works as expected.

It's the utf8 library again. Specifically, utf8.codes does not have the same interface as the utilities.utf8codes that we supply, so the fallback line:
utilities.utf8codes = utf8.codes or function (ustr) ... end
is false. Lua 5.2:
> for uchr in SU.utf8codes("hello") do print(uchr) end
104
101
108
108
111

Lua 5.3:
> for uchr in utf8.codes("hello") do print(uchr) end
1
2
3
4
5

I've removed the calls out to the utf8 library in [master cad2fe4].

Confirmed fix working for me here.
At some point the PDF package needs to be compatible with the debug outputter so we can test it. It's not yet. I started to get it setup the other day but I think I started at the wrong end of the stick. Then got sidetracked.


remove box shadow

👷 Deploy Preview for fancy-puffpuff-df775f processing.

Name
Link

🔨 Latest commit
022725b151e123c40c070734a8a44f88149d7143

🔍 Latest deploy log
https://app.netlify.com/sites/fancy-puffpuff-df775f/deploys/6644f1f4a7b84f0008f4be88


fixed exercise

I only fixed a typo
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks again!


tests/kernel/threads/no-multithreading: Not booting

Describe the bug
Cannot see any sign that the board is booting up
To Reproduce
Steps to reproduce the behavior:

mkdir build; cd build
cmake -DBOARD=altera_max10
make
make run

OS: fedora29
Toolchain zephyr
Commit SHA or Version used:  f5a4a698ec17e6a78299966a1a3fe73c3404b78d

boot delay was removed when running in a single thread. So it is a test setup issue, not a zephyr bug. Need to exclude this from testing or test manually.


Understanding the train procedures

Thanks for your fantastic work, I am reading your paper and the codes. There is one thing I can't understand: according to your paper, it seems that the train process involves in both segmentation decoder and depth decoder simultaneously, but in the experimental setup section, it says that the semantic segmentation is trained on Cityscapes dataset and the depth estimation is trained on Kitti dataset, which indicates the trainings of segmentation and depth are separate, right? Otherwise, do we need use the segmentation mask of x_t in Fig.2, as ground truth to train the segmentation decoder? Would you please talk a little more on the training procedure?
In addition, if I need to train the depth on my own dataset, how should I prepare the dataset? (use my own dataset to train depth and citiscapes dataset to train segmenation)  Thanks.

Hi @wwtu, so what is basically happening is that both datasets (KITTI for depth and Cityscapes for segmentation) are loaded simultaneously. Then batches from both datasets are sampled and concatenated. This concatenated batch is passed through the encoder, splitted into the depth-specific and segmentation-specific parts again and then passed through the respective decoders. Afterwards the losses are only calculated for the task-specific batches, so the depth loss is calculated for the KITTI images and the segmentation loss for the Cityscapes images (using the segmentation masks as groudn truth).
If you would like to train your own dataset for depth training using the code, then you would mainly have to define a new dataloader in loaders/depth/train.py. Note that the function returns a loader that when used supplies a dictionary containing key-data-pairs. You just need to take care that the correct keys are provided. You also would have the possibility to get familiar with our dataloader repository, which is linked in the README.

Hi @klingner, thanks for your quick reply. I will try to understand your points by reading the corresponding codes. By the way, can you tell me how much time you spent for the training process (My PC has 1 GeForce GTX 1070 GPU)? Thanks again.

For me the training process usually takes about 40 hours on a GeForce GTX 1080 graphics card. However, the bottleneck there was actually the speed at which the images are loaded/preprocessed, so in case you happen to have more CPUs and a faster storage device, it should be even faster.

@klingner OK, thanks a lot.

For me the training process usually takes about 40 hours on a GeForce GTX 1080 graphics card. However, the bottleneck there was actually the speed at which the images are loaded/preprocessed, so in case you happen to have more CPUs and a faster storage device, it should be even faster.

How can I make use of the more CPUs that I have. The code doesn't seem to automatically use all my CPUs. Is there any argument that I need to pass?

My understanding is that the num_workers argument passed to the Dataloader() class from pytorch sets the number of processes per Dataloader instance and that these would be automatically distributed across the CPUs. Please correct me, if I am wrong here, as I am no expert on that topic.


Fix stats report

Fix two minor issues in the output of -d stats where things were not counted properly:

The node stat metric appeared twice with different values, which was a bit confusing :-)
The .ninja_parse value was double-counting the parsing time due to recursive manifest parser calls.

In short:
BEFORE:
metric                  count   avg (us)        total (ms)
.ninja parse            27304   372.6           10172.2
node stat               119145  2.9             340.7
node stat               270673  3.0             819.3
...

AFTER:
metric                  count   avg (us)        total (ms)
.ninja parse            1       4109354.0       4109.4
node stat               270673  2.9             774.0
...

friendly ping

ping ?

Thanks!


Add a CHANGELOG file

To make it easier for people upgrading to bleeding edge version of your great django-webpack-loader it makes sense to check CHANGELOG first, before upgrading.

I can take a shot at an initial PR if you'd like; I can pull the content out of the tags and scrape the changes for v.0.4 and v.0.5 if you'd like; let me know if that's something you want @owais

@shapiromatron there are some tools which can create CHANGELOG files out of commit history but starting with a simple CHANGELOG.md file - in which every new feature/bugfix/change must be mentioned - would already help people upgrading.

Sound great @shapiromatron @saily. A PR would be great. Let's start with something simpler like @saily said.

Thanks @shapiromatron


remove notebooks used to generate figures in EM geosci

remove notebooks and figures to generate figs in EM geosci

notebooks/maxwell1_fundamentals/Viz_Dipoles_1D.ipynb
notebooks/maxwell3_fdem/Accuracy tests.ipynb
notebooks/maxwell3_fdem/AccuracyTests_TD.ipynb
notebooks/maxwell3_fdem/EMcircuit.ipynb
notebooks/physical_properties/Chargeflow.png
notebooks/physical_properties/ConductivityGenFigs.ipynb
notebooks/physical_properties/DoubleLayer.png
notebooks/physical_properties/Generate_AC_I_and_V.ipynb
notebooks/physical_properties/LogCond.png

thanks @sgkang!


Api Response sugaring

Coverage remained the same at 46.617% when pulling 2db225f9570147d3f666cbfd6a3a5c05f277bbfc on api_response_json_route into 8f08c75b908827b59dbf8b1546f4ebcaf2d4e788 on master.


Use new TypeScript Extensibility for code completion in html templates etc.

As far as i understand this thread:
https://github.com/Microsoft/TypeScript/issues/6508
It should be possible to have code completion in html templates. This would be awesome!
The Language Extensibility is available in TypeScript since 2.3. Whats missing is some code in atom-typescript, am i right?

I believe TypeScript has decided this a domain related issue and won't be adding it in.
That discussion has lead to the creation of language-service  with the support of angular language service
I'm currently toying with creating a package that would help enable the angular language service.
I currently can't find anything that really fits this criteria

This issue should be close as it shouldn't be part of typescript's main client.

@guncha @TypeStrong I'm planning to create a package that extends this for angular language service.
But I'm finding it a hard time to see where the language client gets initialize when a pane is open to.
Could i get a rough guide on how this package works on a high-level?

The nice thing about using the Typescript plugin feature is that it's not editor specific, i.e., you can write it once and it will work in atom-typescript, vscode, sublime text, vim plugins, etc. I'd say look at some already written plugins (the simpler the better, there's a few listed at the bottom of the plugin wiki) and try to modify them to do what you want.
That said, I should still document how this package works, but that's a different issue..

That's true for the language service.
But atom-typescript has a huge IDE feature (ie autocomplete, codefix, etc)
which is not part of the plugins.
I am planning to use the angular language service plugin. But i need to get
the IDE (ie atom-typescript) to work with it.
At the moment this package supports ts files. Which it should only. There's
bit of code in the grammar that regex on template:. Imo that shouldn't be
in the project but in a plugin to atom-typescript.
That's my intention to figure out how to make this package extendable and
not modify it just for a framework.
On Fri., 18 Aug. 2017, 12:10 pm Guntars Ašmanis-Graham <
<EMAIL_ADDRESS>wrote:

The nice thing about using the Typescript plugin feature is that it's not
editor specific, i.e., you can write it once and it will work in
atom-typescript, vscode, sublime text, vim plugins, etc. I'd say look at
some already written plugins (the simpler the better, there's a few listed
at the bottom of the plugin wiki
https://github.com/Microsoft/TypeScript/wiki/Writing-a-Language-Service-Plugin)
and try to modify them to do what you want.
That said, I should still document how this package works, but that's a
different issue..
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TypeStrong/atom-typescript/issues/1278#issuecomment-323240873,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKXAhvIYRuGP5nv9a7sR7aAdsJoTcmftks5sZPKTgaJpZM4N-RUM
.

I am still interested in this. If anyone has a workaround or hints how to get this, please post it here
✌


Wider class support for entities

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[x] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior

Currently, the constructor for classes accepts only 1 parameter - the merged state of old + new.
Expected behavior

In my use case, I need to pass additional parameters to the class constructor which I don't want to use as class members.
What is the motivation / use case for changing the behavior?

My idea - before calling update, store additional constructor parameters as an array in some local variable (in the store?) and then pass it to the constructor with destructing operator (or by calling apply to support < ES6) .
After updating, they should be cleared.
const additionalArgs = [1, 'text', {key: 'value'}];
this.store.setConstructorParams(additionalArgs); // name?

if (isPlainObject(newState)) {
  newEntity = new oldEntity.constructor(merged);
} else {
  newEntity = new (newState as any).constructor(merged, ...args);
  args = [];
}

I don't think we're going to add this as we don't recommend using classes as the model. Why you don't add them as class members, it seems like an anti-pattern.

Why you don't recommend on using classes?
I do understand though that my case is very specific and might sound odd - but generally speaking, it might be a good addition to have more arguments passed in constructor.

We can’t store classes in the server. A good case is when you need to save the stores snapshot in the server.
The typical pattern in Redux is to work with plain objects.
Harder to use with web workers. (serialization)
There are various third party (check immer for example ) that only works with plain objects.

Some good points over there.
I know it's not an Akita related discussion now - but how would you implement class methods as functions when you need to reuse them in multiple components in template bindings and also keep DRY?

Can you give me an example? I'm not sure what you mean.

Sure. My current class (which has an Akita store for it ) is of a generic type and one of its members is the type
export class A<T extends SomeClass>{
  id: number;
  type: T;
  ... // other common props
}

export interface SomeClass{
  getCode();
  ...
}

I have multiple presentation components which receive A as an input, and displays some properties of its type where each type can implement it differently.
@Input field: A;
...
<div>{{field.type.getCode()}} </div>

As far as I know I cannot refer to static functions from the template and I probably would have to add a method to call them explicitly.
Thanks!

As far as I know I cannot refer to static functions from the template

Why do you need static functions? I'm not sure I understand the use case. A little demo on stackblitz will help.

In order to make them plain object - I need to implement getCode() method for each type.
Will prepare a stackblitz

Updated the question in previous comment, added SB

Object composition?

Probably.
Thanks


Create MkDocs documentation

Roact has nice documentation that I've been working on rewriting in MkDocs. Doing the same treatment for Rodux would be excellent, since there is no documentation right now.

Initial docs are up, based on #18!


JW peer review

This is amazing!! You've taken a huge, complex data set and used map() variants and function writing in a variety of ways to pull out some interesting information.
My comments are within the Rmd, commented out. Hope it's helpful.

Really nice review, Joanna!


Script names proposal

This is the naming that @SophieHerbst proposes for the moment. This is open to discussion once we have an updated flowchart.
This issue is just to keep track of them

[x] 01-import_and_filter.py

[x] 02-apply_maxwell_filter.py

[x] 03-extract_events.py

[x] 04a-run_ica.py

[x] 04b-compute_and_apply_ssp.py

[ ] 05a-reject_ica_components.py

[x] 06-make_epochs.py

[x] 07-make_evoked.py

[x] 08-group_average_sensors.py

[x] 09-sliding_estimator.py

[x] 10-time_frequency.py

[ ] 11-anatomy.py

[ ] 12-make_cov.py

[ ] 13a-make_forward.py

[ ] 14a-make_inverse.py

[ ] 15a-group_average_source.py

[ ] 13b-lcmv_beamformer.py

[ ] 14b-group_average_lcmv.py

[ ] 99-make_reports.py

are we still renaming the scripts based on this issue?

Sure

close issue?

@sappelhoff I would wait for @SophieHerbst


[pure k8s] pai runtime plugin design

PAI Runtime Plugin is PAI functional module that can take effect in users’ job containers. It has the following features:

Independent from user container image. The plugins are built in PAI init container image and works in user container.
Hide data resource from user container. The init container can handle resources and do not need to expose these resources to user container.
Organize all PAI runtime modules as Plugins. Have a policy to add runtime module.

How PAI Runtime Plugin work in user container?

When submit job, the plugin params should be included in job config. The plugin params should be extra:com.microsoft.pai.runtimeplugin:[plugin name]
In init container, parse job config, check plugin params in extra field, and prepare runtime content under shared volume /usr/local/pai:
a.	Generate codes to shared volume. Under /usr/local/pai/plugins/[plugin name]/run.sh
b.	Inject call of /usr/local/pai/plugins/[plugin name]/run.sh to /usr/local/pai/runtime.sh
In user container, before execute user commands, we call /usr/local/pai/runtime.sh to prepare all runtime environments, mounts and other settings.

The structure of PAI runtime plugin
Every plugin should follow the structure:
pai/src/kube-runtime/plugins/[plugin name]
---- init.sh
---- run.sh
---- dataSchema.yaml
---- (libs, etc, )
init.sh: In init container, init.sh will be called to with parameters. It may generate runtime codes into run.sh, copy required libs, and prepare environments for plugin to run in user container.
run.sh: Executable file copied to user container, may be modified by init.sh. For more detailed infomation, run.sh will be copied to shared volume /usr/local/pai/plugins/[plugin name]/run.sh, and will be called in user container.
dataSchema.yaml: The schema of plugin parameters. It has two usages: 1. Verify data in job config. 2. When deploy kube-runtime service, save plugin schema in kubenetes, and webportal may use the schema to generate default input UI for the plugin.
PAI runtime plugin scenarios

Runtime environments
Team-wise storage
Customized mount
Python SDK
Ssh between job containers
Tensorboard integration
Code Debug

The schema of plugin in job config

name
Type
Comments

plugin
String
Plugin   name

stage
String
When plugin code is called in user   container. Could be onstart(before user commands) / onend(after user   commands)

taskroles
List
Optional.   The taskroles who would use the plugin.   If   no taskroles defined, all taskroles will use the plugin.

deployments
List
Optional. If no deployments defined,   all clusters can use the plugin.

parameters
Dict<String,   Obj>
Plugin   parameters, handled by init.sh of the plugin

Done, close this issue


Adding possibility of specifying listening interface

When docker "host" network is used and the host has more than one interface, it is desirable to select the interface you want ntp to be listening to.
This small change allows to set a new variable.

thanks for your patience with my review here... while i can appreciate this is an important feature for you, it deviates a bit away from the original intent, which is to expose as little as possible from the host system. network host is not something in this project, but instead just --publish: 123:123/udp.
i understand you have a need for this and would encourage you to keep you fork up to date with this main project repo. if others in the community think i am completely of the mark here, i'd be open to hearing it, so please feel free to comment on this PR.


Resource are shared only for one test

Hi,
I'm using rf-shared-resources to import my created library.
The problem I encounter is :
When I run the robot command on a folder containing multiple suites (.robot), the library is imported successfully for the first executed suite, but it seems like the same library is not being available for the second one (which is called the same way).
I tried to run the pabot command (with one or multiple processes) on the same folder containing the same multiple suites, and everything works fine.
Is there a restriction on the number of suites you can import your library into using the robot command (which is the default one for RF)? I am the only with this issue?
Regards,

Hi!
Thanks for using the library and reporting this bug.
I was able to replicate the issue as well: Seems that the __init__() method of the library class is not called again after the library is first imported, because Robot Framework caches libraries internally (See https://github.com/robotframework/robotframework/issues/2106)
I added import_library listener that should hande resource loading on second and further imports. See #4 for details.

Hi,
It seems to work fine with the new version.
Thanks for fixing it quickly.


Add e2e test to verify median pruning is working correctly

Set the oracle MaximumMedian stamps to a really low number and check medians for at least two median period iterations.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Completed.


Deviation Between Yosys RTLIL EBNF Docs and RTLIL Lex/Yacc Frontend

Link to page
https://yosyshq.readthedocs.io/projects/yosys/en/latest/appendix/rtlil_text.html
Build number
Build #26545028
Issue
I'm currently working on a tool that ingests RTLIL, to this end
I've been referencing the RTLIL EBNF grammar found in the Yosys
docs as well as the RTLIL Lex/Yacc Yosys frontend code.
I can understand if RTLIL was not meant to be a source or target
for other tools, however, I think RTLIL may find new uses outside
of its original Yosys ecosystem. In light of this, I'd like to
draw attention to the following notable deviations I've
encountered that could have meaninful implications on how RTLIL
is interpreted for the tool I'm building.
Switch Statements in Process Body
As of Yosys commit 8148ebd, the Lex/Yacc RTLIL frontend allows
attribute statements, switch statements, and assignment statements
to appear in any order at the root level of a process body.
The relevant snippet of Yacc code can be found
here.
By contrast, the EBNF grammar doc as of commit 8148ebd allows
multiple switch statements at the root level of a process body,
but requires that all assignment statements occur before the
first switch statement. The EBNF grammar also effectively
requires that attribute statements be placed above their respectve
switch statement. In practice, this second deviation is not an
issue as I've never seen a tool that emits RTLIL that violates it.
The revelant snippet of the EBNF grammar can be found
here.
Assign statements in a process body to me indicate some sort of
default value. What would it mean for an assign statement to occur
after a switch value?
Furthermore, I also understand that the Lex/Yacc frontend is built
to be permissive and that just being able to call read_rtlil
doesn't mean the RTLIL source is valid.
EBNF <module-body> Definition Presumably Missing <conn-stmt>
The current EBNF definition for <module-body> found in the Yosys
docs is:
 <module-body>       ::= (<param-stmt>
                      |   <wire>
                      |   <memory>
                      |   <cell>
                      |   <process>)*

Compare to the following from rtlil_parser.y:
module_body:
	module_body module_stmt |
	/* empty */;

module_stmt:
	param_stmt | param_defval_stmt | attr_stmt | wire_stmt | memory_stmt | cell_stmt | proc_stmt | conn_stmt;

The above effectively translates to:
module_body ::= (module_stmt)* ;
module_stmt ::= param_stmt | param_defval_stmt | attr_stmt | wire_stmt | memory_stmt | cell_stmt | proc_stmt | conn_stmt ;

I think the correct fix for the Yosys docs might look like:
 <module-body>       ::= (<param-stmt>
                      |   <wire>
                      |   <memory>
                      |   <cell>
                      |   <process>
                      |   <conn-stmt>)*

As of right now, <conn-stmt> is defined in the Yosys EBNF doc, but is not employed anywhere.
Expected
I'm not sure if assign statements should be allowed to appear after switches. With regards to module body, I think the definition should be update to the following:
 <module-body>       ::= (<param-stmt>
                      |   <wire>
                      |   <memory>
                      |   <cell>
                      |   <process>
                      |   <conn-stmt>)*

As of Yosys commit 8148ebd, the Lex/Yacc RTLIL frontend allows
attribute statements, switch statements, and assignment statements
to appear in any order at the root level of a process body.

This was discussed in #4765 that introduces 8148ebd. Previously both the EBNF and rtlil frontend allowed arbitrary ordering between assigns and switches, but the yosys internal datastructures did not capture this and would only represent things as if all assigns had come before the switches. We chose to emit a warning instead of an error when tightening up the frontend in case there are some users relying on this behaviour currently.
The parser rules require arbitrary ordering because otherwise the grammar would not be parseable without lookahead - it is not possible to distinguish <attribute> <switch> and <attribute> <assign> when you have just seen the <attribute> token. Although the grammar allows arbitrary ordering, there are assertions that actually enforce (or at least warn on) violations of the ordering https://github.com/YosysHQ/yosys/blob/87736a2bf9710e307fbf9e57e6cece7586314cf7/frontends/rtlil/rtlil_parser.y#L345-L355
As to the other issues, I would be happy to see PRs tightening up the EBNF docs/parser - I am sure there are plenty of places where they differ. The source of truth for what is allowed should come from the kernel/rtlil.h datastructures as those define what is actually representable internally.

I should also say that while much of RTLIL is probably suitable as an IL for other tools, I would be wary about using processes if you can at all help it. There have been various discussions about getting rid of processes as they have confusing semantics and inference rules. read_verilog is about the only place they are still generated, and it looks like yosys-slang (which doesn't use RTLIL processes nor the textual format) is likely to replace many end user uses of read_verilog.

I know it is possible to squeeze processes out using the Yosys proc pass. The trouble I've found with this is I usually end up with a large amount of muxes.
It would be nice to both be able to squeeze out processes whilst retaining the higher level semantics present in switch in process.

Whoops i completely missed a word there, i mean sync processes in particular, comb only processes are slightly less problematic.

comb only processes are slightly less problematic.

Comb only processes are fine, or should be fine if they somehow aren't. The decision of which columns to pick when generating a muxtree should not be the responsibility of each frontend individually.


Allow root zone queries and fall through

This PR allows to query the zone root domain and fall through if it doesn't match a resource.
Tests needs to be adjusted after agreement on actual behaviour.
Fixes #35

@cubic3d thanks for the PR, can you add some tests to this ? It'd be easier to see the corner cases.

Sure, as soon as time allows ;)

@cubic3d check out this commit https://github.com/ori-edge/k8s_gateway/commit/5966d8d6bfa9dcb6a1093b953dfe27e316f3ac8b
I've made some changes to make your PR pass the existing tests. I for some reason couldn't push into this PR so created another branch instead.
So now you just need to add a set of tests to cover the fallthrough behaviour

Thank you, strange that you can't push - edits are allowed. I'll get back to it at weekend.

thanks, will check it out later this week

LGTM. how does it look to you @cubic3d ?

LGTM too! Thank you! I'll mark ready


Clear up which bible translations we can use

bible.api

https://docs.api.bible/common-questions
esv.org

https://api.esv.org
OpenBibleProject

none apparently :////////


evaluate with UX designers if Jinja2 template

more discussion is required

Hello @Pyscho79, I leave here the link to the prototype with the authentication flow to be implemented (or to be discussed for a future implementation phase):
https://www.figma.com/file/wvXrFloFfHi6kd0176yqBK/Accesso-%2F-UI-kit-baset?type=design&node-id=4705%3A302709&mode=design&t=hj7npyQvpghH43HP-1
p.s. consider text and icons as placeholder

Hi @RosaliaGaleano could you provide me with a svg version of the wallet icon?
it would be much easier to integrate it to the discovery page, alternatively if you only have the png version available I would ask you for a white version and a blue version with no background.
Unfortunately I don't have the necessary permissions to download the item from figma.

@MurruAlessio, yes sure! Find the svg icon attached. Let me know if you need other icons or assets.

Hi Rosalia! just a note regarding figma, the session expiration value will not be a static number but will be decided on the backend side depending on the specification. so that value could be different.

Hi Alessio! Yes, you are right! The prototype just shows an illustrative number in bold (120 sec represents the expiration time for SPID) but it's a dynamic one as you said


I'm in the process of describing the change in more depth.
In the meantime, please see https://github.com/jswanner/DontFuckWithPaste/issues/52#issuecomment-372863598 for an explanation as to what the extension does with the tabs permission (which is the permission Chrome describes as meaning the extension has access to your browsing history).


Set rubyLintDiff to false

To ensure all of the code is linted all the time.

Thank you @cbaines 👍


How to increase the performance

Hi there,
I've done some performance test of sending notifications use pushy. and i got the result as followings,
i send about 1,000 notifications to the same device token and it costs 135 ms,but it takes about 41s to get all notifications (1,000) accept by APNs.I wonder if i got the normal result in receiving APNs accept message or whether there's some way to increase the performance of sending notifications.Your advice will be highly appreciated.
here's server code:

public Future<PushNotificationResponse<SimpleApnsPushNotification>> sendNotificationByDeviceToken(String deviceToken, String payload, String messageId, String scenario, long ttl) {
        LOGGER.debug("sendNotificationByDeviceToken begin. ");

        Future<PushNotificationResponse<SimpleApnsPushNotification>> future = null;
        if (!apnsClient.isConnected()) {
            LOGGER.error("ios service is init failed. ");
        } else {
            if (StringUtils.isEmpty(deviceToken)) {
                LOGGER.warn("device token is null or empty");
            } else {
                //send the notification to APNs server with device-Token no need to reply
                LOGGER.debug("send device token = {} message_id ={} . ", deviceToken, messageId);
                future = send(buildNotificationFromRequest(deviceToken, payload, messageId, scenario, ttl));
            }
        }

        future.addListener(new GenericFutureListener<Future<PushNotificationResponse<SimpleApnsPushNotification>>>() {

            @Override
            public void operationComplete(final Future<PushNotificationResponse<SimpleApnsPushNotification>> future) throws Exception {
                if (future.isSuccess()) {
                    final PushNotificationResponse<SimpleApnsPushNotification> response = future.getNow();

                    if (response.isAccepted()) {
                        LOGGER.debug("send device token = {} message_id ={} is accept. ", deviceToken, messageId);
                    } else {
                        Date invalidTime = response.getTokenInvalidationTimestamp();
                        if (invalidTime != null) {
                            // If we have an invalidation timestamp, we should also stop
                            // trying to send notifications to the destination token (unless
                            // it's been renewed somehow since the expiration timestamp).
                            LOGGER.error("device token {} is invalid, time = {} message_id = {} ",
                                    deviceToken, invalidTime.toString(), messageId);
                        } else {
                            // Something went wrong; this should be considered a permanent
                            // failure, and we shouldn't try to send the notification again.
                            LOGGER.error("Rejected by the APNs {}, device token ={} ,message_id = {}",
                                    response.getRejectionReason(), deviceToken, messageId);

                        }
                    }
                } else {
                    LOGGER.error("send device token = {} message_id ={} is failed {} ", deviceToken, messageId, future.cause().getMessage());
                    if (future.cause() instanceof ClientNotConnectedException) {
                        // If we failed to send the notification because the client isn't
                        // connected, we can wait for an automatic reconnection attempco
                        // to succeed before sending more notifications.
                        LOGGER.error("Client Connection failed, try to reconnect ...");
                        Future reconnectFuture = apnsClient.getReconnectionFuture().await();
                        //todo if reasonable
                        if (reconnectFuture.isSuccess()) {
                            sendNotificationByDeviceToken(deviceToken, payload, messageId, scenario, ttl);
                        }
                    }
                }
            }
        });

        return future;
    }

    private Future<PushNotificationResponse<SimpleApnsPushNotification>> send(SimpleApnsPushNotification simpleApnsPushNotifications) {

        return apnsClient.sendNotification(simpleApnsPushNotifications);

    }

here's test code:

    for (int i = 0; i < 1000; ++i) {
        iosPushMsgService.sendNotificationByDeviceToken(deviceToken,payload, message_id, scenario, 1000)
    }

Are you in China? Regrettably, we've seen a pattern of poor performance for users in China; it appears that all of Apple's servers are in the United States(?), and so latency is much higher than expected.
Still, if you're parallelizing your notifications (and it looks like you are), I'm not sure why it's taking so long. If you can profile your application and share any insights, that might be helpful.

@jchambers I have a question, which one is common practice?

create only one apns client, and then share it among all push threads.
each push thread has its own apns client

which method is prefered?

Yes , the server is deployed in china.
I use a BlockingQueue to store the notification then set mutil-thread to send it.
 private class ApnsNotifier implements Runnable {
        int id = 0;

        ApnsNotifier(int i) {
            id = i;
        }

        @Override
        public void run() {
            LOGGER.info("notify-worker-{} start", id);
            while (true) {
                NotificationItem tmpItem;
                try {
                    LOGGER.debug("apns-worker-{} wait to take", id);
                    tmpItem = apnsItemQueue.take();
                } catch (InterruptedException e) {
                    LOGGER.warn("apns-worker-{} is interrupted", id);
                    break;
                }

                List<NotificationItem> items = new ArrayList<>();
                if (tmpItem != null) {
                    items.add(tmpItem);
                }
                for (int i = items.size(); i < maxBatchNum; ++i) {
                    tmpItem = apnsItemQueue.poll();
                    if (tmpItem != null) {
                        items.add(tmpItem);
                    }
                }
                CountDownLatch latch = new CountDownLatch(items.size());
                for (NotificationItem item : items) {
                            Future future = apnsService.sendNotificationByDeviceToken(
                                    item.deviceToken,
                                    item.notificationTask.payload,
                                    item.notificationTask.messageId,
                                    item.notificationTask.scenario,
                                    item.notificationTask.ttl);
                            future.addListener(future1 -> {
                                LOGGER.debug("apns count down latch: {}", latch.getCount());
                                latch.countDown();
                            });
                }

                try {
                    long start = System.currentTimeMillis();
                    latch.await(5_000, TimeUnit.MILLISECONDS);
                    LOGGER.debug("apns-notifier-{} left latch {}, cost {}ms",
                            id, latch.getCount(), System.currentTimeMillis() - start);
                } catch (InterruptedException e) {
                    LOGGER.error("apns-notifier-{} latch was interrupted", id);
                }
            }
        }
    }

Like @peacecoder said for now i use mutil-thread to send the notificaition with one single ApnsClient
      apnsClient = new ApnsClientBuilder()
                    .setClientCredentials(new File(apnsCrtPath), apnsCrtPassword)
                    .setMetricsListener(apnsClientMetricsListener)
                    .build();

Should i set the eventLoopGroup for the client or just use certain number of clients , i've no idea about how to increase the performace , seems i got stucked here , really need your help here , thanks . :)

Folks, questions about best practices should probably go to the mailing list instead of in an issue so that a wider audience can benefit from the discussion. That said, you should probably share a single client across multiple threads (clients are thread-safe), and you do not need to supply your own event loop group. Supplying an event loop group with multiple threads will have no effect on performance for a single client.
@DreamyWen Again, I don't see anything wrong here. I think we need more data to be able to figure out what's going wrong. It'd be helpful if you could give us some profiling data. Failing that, the three main questions for performance issues are always:

Are you using netty-tcnative or alpn-boot for ALPN?
Are you CPU-limited? When you run your demo app, does CPU usage spike dramatically?
Are you bandwidth-limited? When you run your demo app, is your network connection running at full capacity?

Hi @jchambers sorry about that. I have post the question in the mail thread. please check https://groups.google.com/forum/#!topic/pushy-apns/5nOvqbJLXN8 thanks.

Closing due to a lack of new information. @DreamyWen, we'll happily re-open this if you can provide some more information later!


Rust: load_trusted_setup() API shouldnt care about BYTES_PER_*_POINT

Rust clients had to know the BYTES_PER_G1_POINT and BYTES_PER_G2_POINT constants to be able to use load_trusted_setup(). Ideally this would not be the case. The clients should not be exposed to cryptographic details.
This small modification makes the function take flat byte vectors as input which is also what the c-kzg backend expects. This way Rust clients don't need to chunk bytes into group points themselves, they just supply the flat byte vector and c-kzg takes care of the rest.

Right now it's only reth who uses load_trusted_setup() and they haven't integrated the das branch yet because as an EL client they don't care about PeerDAS.
When we merge das into main we will need to communicate with them to make sure they adapt their use of load_trusted_setup() accordingly.
Regardless of this PR they will most likely have to make changes anyway so that they load the g1_monomial points from a file (which was not the case for 4844).


Traefik package in the init package

Is your feature request related to a problem? Please describe.
Currently we use --disable-traefik on our k3s server run leaving it up to customers to bring their own ingress.  This is fine on more complex deploys but it would be nice to have an easy default for those who don't need the complexity.
Describe the solution you'd like
A new (optional) init package that installs the Traefik chart into a cluster to setup ingress.
Describe alternatives you've considered
We could make the --disable-traefik flag configurable more directly but this could open things up to more complexity with our configuration.
Additional context
This will be useful in making the init package more a-la-carte (see #1062)

closing since we support --set K3S_ARGS="" now


include and exclude are not passed to field serializer contexts

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
When using model_dump with include and/or exclude, I would expect the parameters to be available in the field serializer SerializationInfo object, like it is for model serializers.
However, it seems info.include and info.exclude are always empty.
Is it intentional? If so I could not find documentation about this behavior.
Example Code
>>> from pydantic import BaseModel, field_serializer, SerializationInfo
>>> from typing import Any
>>> class Foo(BaseModel):
...     bar : str
...     @field_serializer("bar")
...     def ser_bar(value: Any, info:SerializationInfo):
...         if not info.include:
...             raise ValueError("no include")
...         return value
>>> Foo(bar="baz").model_dump(include={"bar"})
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/eloi/.virtualenvs/pydantic-scim2-dlnh/lib/python3.12/site-packages/pydantic/main.py", line 347, in model_dump
    return self.__pydantic_serializer__.to_python(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
pydantic_core._pydantic_core.PydanticSerializationError: Error calling function `ser_bar`: ValueError: no include

Python, Pydantic & OS Version
pydantic version: 2.7.2
        pydantic-core version: 2.18.3
          pydantic-core build: profile=release pgo=true
                 install path: /home/eloi/.virtualenvs/pydantic-scim2-dlnh/lib/python3.12/site-packages/pydantic
               python version: 3.12.3 (main, Apr 23 2024, 09:16:07) [GCC 13.2.1 20240417]
                     platform: Linux-6.9.2-arch1-1-x86_64-with-glibc2.39
             related packages: typing_extensions-4.12.0 pydantic-settings-2.2.1
                       commit: unknown

@azmeuk,
Thanks for reporting this. Definitely looks like a bug - probably in pydantic-core. PRs welcome with a fix - should be a good first issue!

@sydney-runkle I think I know what to do with this one. Can you assign me ?

ahhhh @davidhewitt,
Thanks so much for the clarification here. That makes a lot of sense. Closing as resolved.


Gitea Migrate failing with external Postgres

Gitea version (or commit ref):
1.12.4
Git version:
Operating system:
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No
[ ] Not relevant

Log gist:

Description
I am trying to automatically install gitea automatically using the INSTALL_LOCK flag. My understanding is that this invokes the gitea migrate command. I have tried removing the flag and manually running gitea migrate but this results in the exact same error:
Server listening on :: port 22.
Server listening on <IP_ADDRESS> port 22.
2020/09/25 14:59:21 cmd/web.go:107:runWeb() [I] Starting Gitea on PID: 19
2020/09/25 14:59:21 ...dules/setting/git.go:93:newGit() [I] Git Version: 2.24.3, Wire Protocol Version 2 Enabled
2020/09/25 14:59:21 routers/init.go:118:GlobalInit() [T] AppPath: /app/gitea/gitea
2020/09/25 14:59:21 routers/init.go:119:GlobalInit() [T] AppWorkPath: /app/gitea
2020/09/25 14:59:21 routers/init.go:120:GlobalInit() [T] Custom path: /data/gitea
2020/09/25 14:59:21 routers/init.go:121:GlobalInit() [T] Log path: /app/gitea/log
2020/09/25 14:59:22 ...dules/setting/log.go:279:newLogService() [I] Gitea Log Mode: Console(Console:info)
2020/09/25 14:59:22 ...les/setting/cache.go:70:newCacheService() [I] Cache Service Enabled
2020/09/25 14:59:22 ...les/setting/cache.go:81:newCacheService() [I] Last Commit Cache Service Enabled
2020/09/25 14:59:22 ...s/setting/session.go:63:newSessionService() [I] Session Service Enabled
2020/09/25 14:59:22 routers/init.go:63:initDBEngine() [I] Beginning ORM engine initialization.
2020/09/25 14:59:22 routers/init.go:70:initDBEngine() [I] ORM engine initialization attempt #1/10...
2020/09/25 14:59:22 ...rm/session_schema.go:25:Ping() [I] PING DATABASE postgres
2020/09/25 14:59:22 ...m.io/xorm/core/db.go:154:QueryContext() [I] [SQL] SELECT tablename FROM pg_tables WHERE schemaname = $1 AND tablename = $2 [public version] - 4.145925ms
2020/09/25 14:59:22 ...m.io/xorm/core/db.go:277:ExecContext() [I] [SQL] CREATE TABLE IF NOT EXISTS "version" ("id" BIGSERIAL PRIMARY KEY  NOT NULL, "version" BIGINT NULL) [] - 1.302281ms
2020/09/25 14:59:22 ...m.io/xorm/core/db.go:154:QueryContext() [I] [SQL] SELECT column_name FROM INFORMATION_SCHEMA.COLUMNS WHERE table_schema = $1 AND table_name = $2 AND column_name = $3 [public version id] - 8.25945ms
2020/09/25 14:59:22 ...m.io/xorm/core/db.go:277:ExecContext() [I] [SQL] ALTER TABLE "version" ADD "id" BIGSERIAL PRIMARY KEY  NOT NULL  [] - 2.568822ms
2020/09/25 14:59:22 routers/init.go:76:initDBEngine() [E] ORM engine initialization attempt #1/10 failed. Error: migrate: sync: pq: column "id" of relation "version" already exists
2020/09/25 14:59:22 routers/init.go:77:initDBEngine() [I] Backing off for 3 seconds
2020/09/25 14:59:25 routers/init.go:70:initDBEngine() [I] ORM engine initialization attempt #2/10...
2020/09/25 14:59:25 ...rm/session_schema.go:25:Ping() [I] PING DATABASE postgres
2020/09/25 14:59:25 ...m.io/xorm/core/db.go:154:QueryContext() [I] [SQL] SELECT tablename FROM pg_tables WHERE schemaname = $1 AND tablename = $2 [public version] - 3.877293ms
2020/09/25 14:59:25 ...m.io/xorm/core/db.go:277:ExecContext() [I] [SQL] CREATE TABLE IF NOT EXISTS "version" ("id" BIGSERIAL PRIMARY KEY  NOT NULL, "version" BIGINT NULL) [] - 1.279001ms
2020/09/25 14:59:25 ...m.io/xorm/core/db.go:154:QueryContext() [I] [SQL] SELECT column_name FROM INFORMATION_SCHEMA.COLUMNS WHERE table_schema = $1 AND table_name = $2 AND column_name = $3 [public version id] - 7.861266ms
2020/09/25 14:59:25 ...m.io/xorm/core/db.go:277:ExecContext() [I] [SQL] ALTER TABLE "version" ADD "id" BIGSERIAL PRIMARY KEY  NOT NULL  [] - 2.661133ms
2020/09/25 14:59:25 routers/init.go:76:initDBEngine() [E] ORM engine initialization attempt #2/10 failed. Error: migrate: sync: pq: column "id" of relation "version" already exists
2020/09/25 14:59:25 routers/init.go:77:initDBEngine() [I] Backing off for 3 seconds

I am running this with an external postgres database. When I run the installation via the UI I am able to successfully setup gitea with my postgres database. I believe when running gitea migrate manually I am successfully connecting to the postgres database as this does create a single table. Any help would be greatly appreciated! Thank you.
...
Screenshots

We need to know what version of Gitea you are attempting to install. Gitea is under very active development so without that information we are extremely hamstrung. There is a reason why the default issue template explicitly asks for it...

Looking at your errors:
You appear to already have a "version" table in your public schema - have you considered using a different schema?

ah, whoops I missed the version, thought I filled out the template accurately.
I'm trying to use the docker image tagged as version 1.12.4
That version table gets created when gitea runs its migrate command.
I'm still getting familiar with gitea so if there is any information that might help resolve this I'll get it right away. Thank you

Quite old, I think it can be closed.


Remove service account limit per firewall rule.

Multiple target or source service accounts can be specified per firewall
rule.
https://github.com/GoogleCloudPlatform/magic-modules/pull/1048

Hi @julianvmodesto , thanks for opening the PR, do you mind adding a test case for your changes?

@joecheuk modified + added a test, PTAL


Webpack

Changes

Use Webpack for easy AMD to ES6 transition. It's the best of both worlds! Webpack is able to consume AMD, CommonJS, or ES6 module sources and export them as a nice UMD package for legacy clients (which is everything for the moment, but this will make it easy for us to start to use ES6 features where it makes sense).
Import individual Lodash functions (as CommonJS modules) from NPM.
Quick pass at ES6-ify Neue scripts.
Remove outdated code style guidelines.

For review: @DoSomething/front-end

Exciting! Looks good to me. Will need to text a bit, but liking the Webpack stuffs!

Made a few bug fixes, and tested that this continues to work in AMD build system in the main app.

:+1:

Weeeeeeeird. This merged (see @3b21b3f9e8c4cdca1c46207d31c09f2f667bab1f), but Github doesn't seem to have realized. Closing.


How to format the tagnames

I'm trying to figure out how to format the tagnames for my api call. It appears from my tests in Swagger that this has to be an exact match (not case sensitive).  Is there a way to format a 'contains' type search?
It seems like it would be possible to filter the results by something like 'partial tag name 1'. I just haven't been able to get this to work.
Would that be possible under the current structure?

@benrwoodard you are correct, the search isn't a contains search, it's an exact match.  We've recently expanded the tags documentation: https://www.adobe.io/apis/experiencecloud/analytics/docs.html#!AdobeDocs/analytics-2.0-apis/master/tags.md
Maybe you'll find some methods there useful.


On-board Maps

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

Refer to https://github.com/Azure/azure-cli/pull/5716
We were having a service rename from Location Based Services to Maps.
Since the timeline was short and that there was an embargo on "Azure Maps" name, the PR was canceled.
We are ready to on-board Azure Maps command module now.
@lmazuel @tjprescott

View a preview at https://prompt.ws/r/Azure/azure-cli/6303
This is an experimental preview for @microsoft users.

@jp94 which dev are you working with on the CLI side?

@tjprescott package is released:
https://pypi.org/project/azure-mgmt-maps/

See https://github.com/Azure/azure-cli/pull/5716 for intial review.

@derekbekoe what is the fix for the bdist wheel issue?

@jp94 This is the azure_bdist_wheel.py file in your PR - https://github.com/Azure/azure-cli/pull/6303/files#diff-3423f41d513e4e7b0cb1d04b1d23b5c7.
This is the one that it compares against - https://github.com/Azure/azure-cli/blob/dev/src/azure-cli-core/azure_bdist_wheel.py.
They are not the same.

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@a3be22d). Click here to learn what that means.
The diff coverage is 0%.

@@          Coverage Diff          @@
##             dev   #6303   +/-   ##
=====================================
  Coverage       ?      0%
=====================================
  Files          ?      12
  Lines          ?     400
  Branches       ?      52
=====================================
  Hits           ?       0
  Misses         ?     400
  Partials       ?       0

Impacted Files
Coverage Δ

...ommand_modules/azure-cli-maps/azure_bdist_wheel.py
0% <0%> (ø)

...c/command_modules/azure-cli-maps/azure/__init__.py
0% <0%> (ø)

...li-maps/azure/cli/command_modules/maps/__init__.py
0% <0%> (ø)

.../azure/cli/command_modules/maps/_client_factory.py
0% <0%> (ø)

...-maps/azure/cli/command_modules/maps/validators.py
0% <0%> (ø)

...e-cli-maps/azure/cli/command_modules/maps/_help.py
0% <0%> (ø)

...cli-maps/azure/cli/command_modules/maps/_params.py
0% <0%> (ø)

...-cli-maps/azure/cli/command_modules/maps/custom.py
0% <0%> (ø)

...li-maps/azure/cli/command_modules/maps/commands.py
0% <0%> (ø)

...mmand_modules/azure-cli-maps/azure/cli/__init__.py
0% <0%> (ø)

... and 2 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 a3be22d...498ca9c. Read the comment docs.

Looks like I didn't pull the latest code from the dev branch, and I was still seeing the old azure_bdist_wheel.py
Removed validators.py. This results in error stacktrace with a generic response "Operation returned an invalid status code 'Bad Request'". We will be adding checks in REST API specs and our backend routes to avoid redundancy. I'll talk to my manager about getting this in. Let me know if this change should be reverted.

I've been trying to resolve azdev verify module-load-perf (PURPOSE='Module Load Performance').
I was able to fix the servicefabric module failing by reverting merge remote-tracking branch 'upstream/dev' into rename. Do you know what may have caused it to fail?
Obviously, I don't want to push the merge reverts.
For example, my previous commit passed for module-load-perf: https://travis-ci.org/Azure/azure-cli/jobs/376134725 But after merging with dev branch, it failed: https://travis-ci.org/Azure/azure-cli/jobs/376565890

@troydai I sent an email about it.  It is essentially the same problem, yet identified, that caused us to raise the SQL threshold.  Now you'll note SQL is at 3 ms and the problem manifests in servicefabric.  I sent @jp94 the workaround.

thanks, guys.


Please provide example showing segmentationBrush with stackScroll

Simply adding "cornerstoneTools.brush.activate()" to the stackScroll example seems not to work.
Could you please provide an example that demonstrates how one would actually segment a volume (e.g. a tumor).

I added one here: https://rawgit.com/chafey/cornerstoneTools/master/examples/segmentationBrush/index.html
There are still some bugs to sort out though.

Great, that works and looks clean & easy. I should have tried to add stack functionality to the brush example. I instead tried to add the brush functionality to the stackScroll example, which somehow did not work. Thank you very much!


restic hangs forever on any operation involving b2

Output of restic version
restic 0.12.1 compiled with go1.16.6 on linux/amd64
How did you run restic exactly?
export B2_ACCOUNT_ID="xxx"
export B2_ACCOUNT_KEY="yyy"
export RESTIC_REPOSITORY="b2:xxxx
export RESTIC_PASSWORD=yyyy
export XDG_CACHE_HOME=/aaa/bbb/ccc
restic -v -r b2:xxx backup /xxx/yyy/zzz/ --host xxxx
restic -v forget --host xxxx --keep-daily 7 --keep-weekly 4 --keep-monthly 12 --keep-yearly 10
even a
restic --verbose=3 unlock
prints nothing.

What backend/server/service did you use to store the repository?
b2
Expected behavior
operation is performed

Actual behavior
restic just hangs

Steps to reproduce the behavior
see above

Do you have any idea what may have caused this?
no. worked like a charm until a few days ago
Do you have an idea how to solve the issue?
no
strace.log

Seems the same as #3636

Closing this issue as it seems to be resolved by the solution in #3636.


Embedding visualization in tf.keras / TensorFlow 2.0

I'm trying to visualize some embeddings in Tensorboard. In pure keras this was possible, for example:

https://keras.io/examples/tensorboard_embeddings_mnist/

There was already a regression in tf.keras, breaking this functionality, see this bug.
I thought I'd try out tensorflow 2.0 and it seems that the functionality is removed altogether. I now get the message:

WARNING:tensorflow:embeddings_layer_names is not supported in TensorFlow 2.0. Instead, all Embedding layers will be visualized.

I generally don't care about visualizing my Embedding layers. Instead, I'd like to visualize the output of my bottleneck layer, which is typically just a Dense layer.
As I'm writing this, I do realize that the way tensorboard/projector works is that is extracts the embedding Variable (weights) from a checkpoint. In the case of an Embedding layer this is very natural. The interesting case, I would argue, is the visualization of activations, i.e. a generic Tensor.
Is this supported by tf.keras or should I write a custom callback class to do this?
Thanks!

@KristianHolsheimer,
Can you please refer to the Embedding Projector and let us know if this is what you are looking for? Thanks!

@KristianHolsheimer  It looks like you are using an older Version of Tensorflow 2.0. Many bugs have been fixed in the latest version. Can you please execute your code using Latest stable Version(2.5) and let us know if the issue still persists? Thanks!


grpc.WithTimeout is deprecated: use DialContext and context.WithTimeout instead

https://godoc.org/google.golang.org/grpc#WithTimeout
This affects #157 . These lines were excluded from the linter for now, but should be reenabled once fixed.


Support wav for AudioSource

Puff scene doesn't work becuse the wav support is not working. The AudioStreamWav::set_data expects the samples, not the audio file. To support it, a importer is needed.

Godot 4.4 will solve this
https://godotengine.org/article/dev-snapshot-godot-4-4-dev-6/#runtime-wav-file-loading


TypeError: linear(): argument 'input' (position 1) must be Tensor, not tuple

When I run infer.py  in fgvc, I get this error.
File "infer.py", line 100, in visualize
attention_maps = net.visualize(X)
File "/kaggle/working/CAL/fgvc/models/cal.py", line 180, in visualize
p = self.fc(feature_matrix * 100)
File "/opt/conda/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/opt/conda/lib/python3.7/site-packages/torch/nn/modules/linear.py", line 96, in forward
return F.linear(input, self.weight, self.bias)
File "/opt/conda/lib/python3.7/site-packages/torch/nn/functional.py", line 1847, in linear
return torch._C._nn.linear(input, weight, bias)
TypeError: linear(): argument 'input' (position 1) must be Tensor, not tuple
Can you help me fix this? Thank you!

Hi @anandhiray, thanks for your interest in our work. I have fixed the typo in our visualization code. There should be two returned variables for the self.bap module here.

Thanks a lot for your immediate reply. But now it gives this error.
main()
File "infer.py", line 75, in main
visualize(data_loader=validate_loader, net=net)
File "infer.py", line 101, in visualize
attention_maps = torch.max(attention_maps, dim=1, keepdim=True)[0]
TypeError: max() received an invalid combination of arguments - got (tuple, keepdim=bool, dim=int), but expected one of:

(Tensor input)
(Tensor input, Tensor other, *, Tensor out)
(Tensor input, int dim, bool keepdim, *, tuple of Tensors out)
(Tensor input, name dim, bool keepdim, *, tuple of Tensors out)

Is it because of any version issue?
Thank you very much for your  immediate response.

It is working great now! Thank you for your help, without which I would not have been able to do this!


Define SCAP_HANDLE_T before including scap_open.h

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap-engine-udig

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
TBH I don't know why these are only build warnings instead of errors in the CI.
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gnosek
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.10.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, gnosek
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 8372e8368d6b22cdb10ca59c0237393fa900a4ef

/hold until CI is fixed.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, gnosek
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, gnosek, hbrueckner
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/unhold
We are still investigating why the CI is failing. The issue seems related to user-related e2e tests, i think it's pretty fair to merge this one.


[Feature request] Sort movies

I think an useful feature would be to add a sort option for movies to be able to see movies by raiting, newest first, oldest first, etc.
What you think?

this is actually a great idea, we will try to include it in the next update

@snisnik Definitely will try to include this in the next update.


Mass listing success modals

continuation of:

https://github.com/kodadot/nft-gallery/issues/9700

its divided into two modals, one for mass list and one for single list
since mass minting in drop page is coming soon, it will be nice to have both
It has a lot in common with buy and mint success modals
Single list

its basically single buy with different copy

full-width button to view item
under the main heading there is price what is listed at and the approximation in USD
view nft redirects to nft

Mass listing

next to every item there is price (new one)
copy changed to "Bulk Listing Successful"
main button (view nft) directs to profile with listed filter on

Something is missing here 👀

👋


Prettier now doesn't do anything

I have VSCODE:
Version: 1.45.0
Commit: d69a79b73808559a91206d73d7717ff5f798f23c
Date: 2020-05-07T16:10:24.511Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.4.0-29-generic snap
I installed Prettier now version 1.4.9
I have the following code:

Document

If I use the built in format with html language feature, it will format it correctly. But when I try to do it with Prettier Now, it doesn't change anything at all and the file stays the same.

When I installed the Prettier Now, I get the following error in debug:
[Embedded Page] Refused to apply inline style because it violates the following Content Security Policy directive: "style-src 'nonce-908337a8-9de0-4128-ad5d-77a71945942a'". Either the 'unsafe-inline' keyword, a hash ('sha256-u7IexyLDIp75z35KFbvJ/fh7N1FG0ehwP6fiagQQAXg='), or a nonce ('nonce-...') is required to enable inline execution.


setContents did not work?

After resizing an image, quill.getContents() does add attributes to the ops
But quill.setContents() did not seem to apply the attributes to the image
Can anyone help me try?

I found out that if I added formats to the quill options, setContents just stopped working.
Even if I put formats: ['image']

Finally got it. For anyone who needs this,
You need to add 'width' to formats


pseudo growth truth depth maps?

Hello author, I am curious if there is an image in your paper that should be pseudo growth truth depth maps? How should this be generated?

We captured multi-view video of a person asked to stand still. We then used SFM software (https://www.agisoft.com) to get the ground truth geometry.

thank you very much！


Upgrade to new zondax project/repo structure

This is on-hold until Ledger releases current codebase that is under review.

[ ] Restructure projects and unify repositories
[ ] Migrate all issues to the new repository


Sym links go from typings to root directory, not the other way around

Sym links go from typings to root directory, not the other way around
This is how things are done in celery-types and django-types. Having sym links go from root directory to typings meant that packages directive in pyproject.toml wasn't working -- these packages weren't being installed, for example, in the virtual env managed by poetry when installing this library with poetry
This PR fixes this problem

Thank you!


locks in parallel phases are incorrectly handled

Hi,
We have noticed, that the locks were not handled / cleaned up properly during parallel phase executions. Looking at the sample scripted pipeline script below, trigger pipeline build 1 with error "error " commented out and then trigger a pipeline build 2 with the error "error" un-commented before the build 1 finishes. You can see that as soon as the build 2 fails, build 1 gets superseded and is aborted. This behavior causes a lot of operational issues. Please let me know if there is an alternative way of doing this or a fix is needed?
Sample Scripted Pipeline Script
node {
  timestamps{
    milestone()
    lock(resource: "my_bld_lock", inversePrecedence: true) {
      milestone()
      stage("Bld") {
        sleep 10
      }
    }

    milestone()
    lock(resource: "my_pkg_lock", inversePrecedence: true) {
      milestone()
      stage("Pkg") {
        sleep 15
      }
    }

    parallel([
            "Testing"           : {
              lock(resource: "my_test_lock", inversePrecedence: true) {
                stage("Test") {
                  sleep 6
                }
              }
            },
            "Second Level Testing": {
              lock(resource: "my_second_test_lock", inversePrecedence: true) {
                stage("Second Level Test") {
                  sleep 4
                }
              }
            },
            "Create & Deploy"          : {
              lock(resource: "my_create_lock", inversePrecedence: true) {
                stage("Create") {
                  sleep 5
                  // error "error"
                  sleep 30
                }
              }

              lock(resource: "my_deploy_lock", inversePrecedence: true) {
                stage("Deploy") {
                  sleep 70
                }
              }
            },
    ])
  }
}

Basically, the issue is that a pipeline build should not get superseded if the later builds fail.
Pipeline builds should get superseded only if the later ones pass and produces usable artifacts.

looks to be fixed / obsolete


Remove creation of ragged sequences in MultivariateGrouper

Issue #, if available: #2612
Description of changes:
Latest numpy versions raise an error when ragged sequences are passed to np.array. An array of ragged sequences is generally not a good idea anyway. This PR fixes the MultivariateGrouper by sidestepping the creation of a ragged array. Instead, a list is used, which is batched based on the number of evaluation dates. The sublists are then stacked to create multivariate time series.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Please tag this pr with at least one of these labels to make our release process faster: BREAKING, new feature, bug fix, other change, dev setup

cc @kashif

LGTM! I will test it out on monday

@abdulfatir if this is backwards compatible, could you label it for v0.11 and v0.12 backport?


Styling of brackets matching highlighting

The only reason I am still using Bracket Pair Colorizer in VS Code is ability to change styles of brackets matching highlighting.
Instead of native square I left only a bottom border

It would be greate if it would be possible to change styles of this highlighting square

/duplicate https://github.com/microsoft/vscode/issues/19534


(security bug) Fix for the case when using proxies

Closes #25
The uri related properties on a request instance are copied from the parsed request.uri by Request.prototype.init. Particularly request.href and request.host which is not always an alias for request.uri.hostname. The host is recorded so it may be checked when a redirect occurs. During a redirect, If the host changed and there is an authorization header, that header will be removed. Relying on request.host to make subsequent requests can achieve what the request library was trying to prevent in the first place, that is sending an authorization header to the wrong host. This PR relies on request.uri.hostname directly. We should try to get this merged ASAP.
https://github.com/request/request/blob/master/request.js#L307-L313
  if (self.proxy && !self.tunnel) {
    self.port = self.proxy.port
    self.host = self.proxy.hostname
  } else {
    self.port = self.uri.port
    self.host = self.uri.hostname
  }

Edit: Added tests, request.uri is now testable.

Relevant diff: https://github.com/codemanki/cloudscraper/pull/101/files/ad936797ecdbfac2e4c40bd30cd678b72f3a1677..935992cb78362c83d6dd3a1f5b9a363799bba4d0


Deprecate network travelsal package

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

Please merge from main to fix the build issues as the azure-xml javadoc issue is fixed now.


Open source LLM integration

It would be great to see an integration of LLM (Large Language Models) with Hugging Face models instead of the locked CHATGPT and CLAUDE.

this project requires a llm with atleast 4000 context tokens and needs to work in realtime with most consumer hardware, there aren't a lot of options but it's good idea nonetheless . Maybe can be implemented in the future with better and smaller models.


Gitlab demo base64 encoding step incorrect

Bug Report
The base64 encoding of credentials are incorrect at:
https://github.com/crossplaneio/crossplane/blob/master/docs/gitlab/gitlab-gcp.md

The following instructions should use a cat command rather than echo

echo gcp-credentials.json | base64 -w0
and
echo gcp-credentials.json | base64 -w0

Minor issue: It should used the term encoded rather than encrypted in "Generate BASE64ENCODED_GCS_PROVIDER_CREDS encrypted value"
It would be good to note that Mac does not have a base64 -w0 flag and that base64 is sufficient on Mac.

@infinitecompute @ichekrygin this seems fixed can it be closed?

I believe so, @infinitecompute could you please confirm?


sim._pyrtl: work around Python's new integer-string convertion limits

Hi,
some of my code recently started failing with this exception after a Python version bump:
Traceback (most recent call last):
[...]
  File "/usr/lib/python3.10/site-packages/amaranth/hdl/xfrm.py", line 209, in on_statement
    new_stmt = self.on_Assign(stmt)
  File "/usr/lib/python3.10/site-packages/amaranth/sim/_pyrtl.py", line 349, in on_Assign
    gen_rhs = f"({(1 << len(stmt.rhs)) - 1} & {gen_rhs_value})"
ValueError: Exceeds the limit (4300) for integer string conversion

Here's a small reproducer:
from amaranth import Module, Signal, sim
dut = Module()
dut.d.sync += Signal(1).eq(Signal(1) << Signal(14))
sim.Simulator(dut)

This fails because the generated code uses a large mask value, which raises an ValueError during decimal formatting in Python versions that include a mitigation for CVE-2020-10735. Formatting to hexadecimal instead avoids the algorithmic complexity and is not impacted by the new conversion limits.
I'm not sure if this is worth merging as affected modules are already near the existing value limits enforced by pyrtl, so feel free to close/reject this patch ^^
Note: There's an open proposal that changes int's repr implementation to hex for large values and would make this patch a bit more self-explanatory ({v!r} vs {v:#x}): https://github.com/python/cpython/issues/96601

Could you rebase on top of main, please? The CI should pass afterwards.

Thank you!


babel-regenerator-runtime6.5.0

Type: Missing
Summary:
babel-regenerator-runtime6.5.0
Details:
ClearlyDefined license file is BSD-2-Clause
GitHub license is MIT: https://github.com/babel/babel/blob/v6.5.0/LICENSE
Resolution:
BSD-2-Clause
Affected definitions:

babel-regenerator-runtime 6.5.0

Added AND OTHER for the PATENTS file included in the package.
https://github.com/babel/babel/tree/v6.5.0/packages/babel-regenerator-runtime
https://github.com/babel/babel/blob/v6.5.0/packages/babel-regenerator-runtime/LICENSE
https://github.com/babel/babel/blob/v6.5.0/packages/babel-regenerator-runtime/PATENTS


Maintenance/use latest rustfmt

Coverage increased (+0.9%) to 93.85% when pulling c2eeb0c16f2616d482457099e47e638133c8988d on maintenance/use-latest-rustfmt into da5fda3628e10dc04f1cfebbf86ca8981ffd1977 on development.

Coverage increased (+2.6%) to 95.556% when pulling 18773d6518cf2415af1431c9c8cda51a8cdf7274 on maintenance/use-latest-rustfmt into da5fda3628e10dc04f1cfebbf86ca8981ffd1977 on development.


Add lpkg as java package format

also cleaned up linter warning
Closes: #682
Signed-off-by: Jonas Galvão Xavier<EMAIL_ADDRESS>

@kzantow while exploring the project my editor found places that were not properly formatted, I will try not to pollute PR too much going forward.
On testing: we don't add jar/war/hpi files within the repo, that is why there are automated building steps for most formats but war. War files are a super set of jar plus Servlet, JSP, HTML, JavaScript, but we only care about their jar content. Lpkg files are jars plus a property file. I wrote lpkg tests based off war validation that seems sufficient.


Refactor GeodeTypeIds enums...

… into DSCode, DSFid, and InternalId enums, make enums enum classes
Co-authored-by: Ivan Godwin<EMAIL_ADDRESS>Co-authored-by: Jacob Barrett<EMAIL_ADDRESS>

Hit CLI breaks in Windows after applying a small requested change from JB, so closing this while I fix things.


Scrollbar overlap console output

When compiled code length does not fit into the screen it adds a scrollbar, which can overlap console output. Visit Babel repl for more info.

Thanks for PR. Could you send a screen capture of this issue?
It's fine for me:

@xtuc

@artemgurzhii it's still weird if debug is enabled in your example:

Short fix is to bump z-index, more involved is to use flexbox in babel-repl-right-panel to deal with the console output being variable height.

@existentialism Noticed that my fix only works for one line errors. How does this look for you?
P.S. I has tried to increment z-index property and this doesn't look for me as a good solution because it overlaps output section scrollbar.

Will be fixed in #1297

Fixed in 1297


Address review

User @romkatv provided a wonderful review of the code: https://www.reddit.com/r/zsh/comments/cajcni/wakatime_zsh_integration_track_how_much_time_you/eta6ti1/
You want to hook preexec, not precmd.
Don't define globals with names bound to clash. E.g., PLUGIN_NAME.
It's better to use : {X:=Y} instead of X=${X:-Y}. However, in your code it's even better to not modify WAKATIME_TIMEOUT at all.
That $() in _wakatime_call is not doing what you think it does. Remove the parens.
Use &! instead of & to avoid polluting jobs.
Forks are slow. Avoid them. E.g., something like _should_work_online is definitely not worth the cost of a fork.
There is ${foo:t} to get the basename of a directory.
To check whether wakatime command exists, use if $(( $+commands[wakatime] )); ....
Don't trap signals. Traps are global, it's rude to hog them for yourself.
Prefix your functions with your project name such as _wakatime_. Try to limit the number of functions you define.

Related: http://zsh.sourceforge.net/Doc/Release/Functions.html


Viewport with ViewportTexture causes OpenGL error messages

Operating system or device - Godot version:
Linux 64 bit (Xubuntu 17.04) - Godot 3.0 alpha1
Issue description:
When I create a Viewport with 3D elements and display the Viewport through a ViewportTexture, the engine spams the debugger with the following error:

GL ERROR: Source: OpenGL	Type: Undefined behaviorID: 131222	Severity: Medium	Message: Program undefined behavior warning: Sampler object 0 is bound to non-depth texture 0, yet it is used with a program that uses a shadow sampler. This is undefined behavior.

Steps to reproduce:
Create a 3D viewport and a Sprite or TextureRect with a ViewportTexture. Start the scene and watch the debug output.
Link to minimal example project:
ViewportTexture.zip

What's your GPU and drivers?

GeForce GTX 750 Ti with proprietary Nvidia drivers (version 384.47)

This was fixed some weeks ago, also minimal example project works, so safely closing this


Style command prompt / messages

This theme seems to be missing styles for the command prompt: : and the dialogs.
I have styled them with the following:
# Style command prompt / dialogs
set -g message-style 'bg=#181825 fg=default'

Hey @druskus20
I'm not a tmux user but does this line not style the command prompt / messages?
https://github.com/catppuccin/tmux/blob/0be44ee9bd226cf6b9779d4ebd7f3d95362efce5/catppuccin.tmux#L332

Fixed in #193


Feature/strm 1389

Data subjects

:tada: This PR is included in version 2.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Can't auxiliary click on posts (web)

Describe the bug
Clicking on a post body takes you to the post thread, but since the links are are fake (not anchor tags), there's no auxiliary click action on web (aka middle click to open in new tab)
To Reproduce
Steps to reproduce the behavior:

middle click on a post
observe nothing happening

Expected behavior
It opens the post thread in a new tab
Details

Platform: web
Platform version: firefox
App version: any

Additional context
Just need to add an auxclick handler (weirdly I can find next to no info on this for RNWeb)
Turns out Firefox is the issue here! Doesn't support auxclick properly

I actually have a local branch where I fixed this afaik. I'll look and clean it up.


following error is coming when i run composer update or run any commnd

following error is coming when i run composer update or run any command
Fatal error: Call to undefined method Fxp\Composer\AssetPlugin\Package\Version\VersionParser::parseLinks() in /root/.composer/vendor/fxp/composer-asset-plugin/Repository/VcsPackageFilter.php on line 272

Update global fxp installation:
composer global require "fxp/composer-asset-plugin:~1.1.1"

When i run this command they show same error.

composer self-update && composer global require "fxp/composer-asset-plugin:~1.1.1"

 @ now!..

Weird.

looks like composer broke api on update and now it can not run. try removing global composer vendor dir, which is /root/.composer/vendor in your case and run the commands again.

Thanks....

i did everything mentioned above
still i m getting error
The "yiisoft/yii2-composer" plugin requires composer-plugin-api 1.0.0, this WILL break in the future and it should be fixed ASAP (require ^1.0 for example).
./composer.json has been updated
The "yiisoft/yii2-composer" plugin requires composer-plugin-api 1.0.0, this WILL break in the future and it should be fixed ASAP (require ^1.0 for example).

Check out the solution suggested by @cebe in this comment