Python 2: two tests fail with DID NOT RAISE class 'semver.InvalidTypeIncluded'

Hi,
Using the latest git master, two tests fail with Python 2.7.16 and PyPy:
==================================================================================================== FAILURES =====================================================================================================
_______________________________________________________________________________________________ test_max_satisfying _______________________________________________________________________________________________

    def test_max_satisfying():
        def _callFUT(versions, range_):
            from semver import max_satisfying
            max_satisfying(versions, range_)

        from semver import InvalidTypeIncluded
        with pytest.raises(InvalidTypeIncluded):
>           _callFUT([b"1.0.0"], "1.0.0")
E           Failed: DID NOT RAISE <class 'semver.InvalidTypeIncluded'>

semver/tests/test_passing_bytes.py:11: Failed
_________________________________________________________________________________________________ test_satisfies __________________________________________________________________________________________________

    def test_satisfies():
        def _callFUT(version, range_):
            from semver import satisfies
            satisfies(version, range_)

        from semver import InvalidTypeIncluded
        with pytest.raises(InvalidTypeIncluded):
>           _callFUT(b"1.0.0", "1.0.0")
E           Failed: DID NOT RAISE <class 'semver.InvalidTypeIncluded'>

semver/tests/test_passing_bytes.py:28: Failed
================================================================================ 2 failed, 1124 passed, 5 skipped in 1.61 seconds =================================================================================

Yes, but python 2.x is not supported officially, sorry.

Ah I didn't notice that, sorry. In that case, I think this should be removed: https://github.com/podhmo/python-semver/blob/b988cb05a25318a3709e666d8393632121f534d5/tox.ini#L9


How to run sv_*_header.txt samples

What is the configuration to run SocketQueue with, so that it accepts the samples with headers?

Hi!
SocketQueue uses headers for upstream connection only. It does not read headers sent by clients. By default the length header is present.
Below is the example of self test mode where a binary client sends an echo packet that is being passed to an upstream with the length header. The response also comes with the header. The header data is marked with bold.
$ node socketQueue.js --upstreamHost=localhost --upstreamPort=5000 --listenPort=2014 --listenHttpPort=8080 --vv --echoServerPort=5000 --testClients=1 --testTargetHost=<IP_ADDRESS> --testTargetPort=2014
2020-04-07 22:49:08 - info: Important: starting...
2020-04-07 22:49:08 - info: Remote host configuration name: smartVista
2020-04-07 22:49:08 - info: Connecting to upstream server localhost:5000
2020-04-07 22:49:08 - info: Relay raw ISO-8583 server is now running on port 2014
2020-04-07 22:49:08 - info: Relay HTTP server is now running on port 8080
2020-04-07 22:49:08 - info: Echo server is now running on port 5000
2020-04-07 22:49:08 - info: Connected to upstream <IP_ADDRESS>:5000!
2020-04-07 22:49:08 - info: Processing queue [pending 0 / total 0]
2020-04-07 22:49:08 - info: The queue is empty
2020-04-07 22:49:09 - info: Client ::ffff:<IP_ADDRESS>:55314 connected
2020-04-07 22:49:09 - info: Client ::ffff:<IP_ADDRESS>:55314 sent data (45b)
2020-04-07 22:49:09 - verbose: [0800.......]
2020-04-07 22:49:10 - verbose:
::ffff:<IP_ADDRESS>:55314
 [Echo Request]

 Message Type Indicator [0].......................0800
 Bitmap [1].......................................2220010000800000
 Processing Code [3]..............................990000
 Transmission Date and Time [7]...................0407224909
 System Trace Audit Number [11]...................000001
 Function Code [24]...............................831
 Card Acceptor Terminal Identification [41].......00000001

================================================================================================
2020-04-07 22:49:10 - info: Writing to queue ::ffff:<IP_ADDRESS>:55314 [0]
2020-04-07 22:49:10 - info: New queue item 0
2020-04-07 22:49:10 - info: Processing queue [pending 1 / total 1]
2020-04-07 22:49:10 - verbose: Queue keys dump: 0
2020-04-07 22:49:10 - info: Upstreaming data for ::ffff:<IP_ADDRESS>:55314
2020-04-07 22:49:10 - verbose: [00450800...]
2020-04-07 22:49:10 - info: Echo server got data
2020-04-07 22:49:10 - info: Replying to client 00000001
2020-04-07 22:49:10 - info: Echo server sent response
2020-04-07 22:49:10 - info: Got data from ISO-host (61b)
2020-04-07 22:49:10 - verbose: [00570810...]
2020-04-07 22:49:10 - verbose: Releasing terminal 00000001 with the key 0

Thank you for providing a sample log for it.


[H] Undeprecate BigInteger support, but document what it actually does.

Also, some code cleanup.

See also https://github.com/flutter/engine/pull/6903

cc @amirh


Crashes Nova 8 anytime I save a file

See video... with the extension enabled, saving any TS file causes Nova 8 to crash.
Screenshots
https://user-images.githubusercontent.com/3246342/141155278-d06c05dc-f7ec-408c-b39f-3afb3854f883.mov
Environment

Nova version: 8.0 (Build 332649)
Extension version: 2.4.0
TypeScript version:  4.4.4
Sidebar information:  4.4.4
macOS version:  macOS 11.6.1 (20G224) ARM-64 (Native)
NodeJS information:

node version: v16.6.2
npm version: 7.21.1
installation method: homebrew

Extension console output
Tailwind CSS[8:43:43.003000] No Configuration File Detected

TypeScript[8:43:43.560000] activating...

TypeScript[8:43:43.565000] dependencyManagement: claimed lock

ESLint[8:43:43.579000] activating...

npm imports[8:43:43.585000] Extension encountered an uncaught exception:
/Users/timfee/Library/Application Support/Nova/Extensions/mockadillo.npmimports/Scripts/main.js (Line 1, Column 99)
Error: Arguments must all be strings
    [native code]
    file:///Users/timfee/Library/Application%20Support/Nova/Extensions/mockadillo.npmimports/Scripts/main.js:1:99

ESLint[8:43:43.798000] ESLint used without a workspace path, this is unlikely to work properly

ESLint[8:43:43.798000] No ESLint path

ESLint[8:43:43.798000] activated

TypeScript[8:43:43.868000] dependencyManagement: installing: npm WARN old lockfile

TypeScript[8:43:43.870000] dependencyManagement: installing: npm WARN old lockfile The npm-shrinkwrap.json file was created with an old version of npm,

TypeScript[8:43:43.870000] dependencyManagement: installing: npm WARN old lockfile so supplemental metadata must be fetched from the registry.

TypeScript[8:43:43.872000] dependencyManagement: installing: npm WARN old lockfile

TypeScript[8:43:43.872000] dependencyManagement: installing: npm WARN old lockfile This is a one-time fix-up, please be patient...

TypeScript[8:43:43.873000] dependencyManagement: installing: npm WARN old lockfile

TypeScript[8:43:44.388000] dependencyManagement: installing:

TypeScript[8:43:44.388000] dependencyManagement: installing: up to date in 702ms

TypeScript[8:43:44.389000] dependencyManagement: installing:

TypeScript[8:43:44.390000] dependencyManagement: installing: 10 packages are looking for funding

TypeScript[8:43:44.391000] dependencyManagement: installing:   run `npm fund` for details

TypeScript[8:43:44.393000] dependencyManagement: clearing lock

TypeScript[8:43:44.396000] using tslib at: /Users/timfee/Library/Application Support/Nova/Extensions/apexskier.typescript/dependencyManagement/node_modules/typescript/lib

TypeScript[8:43:44.406000] activated

TypeScript Language Server[8:43:45.075000] [tsserver] /bin/sh: /opt/homebrew/Cellar/node/16.6.2/bin/npm: No such file or directory

Nova console log
11/10/21, 8:08 AM Nova[12594] getattrlist failed for /System/Library/Frameworks/OpenGL.framework/Resources//GLRendererFloat.bundle/GLRendererFloat: #2: No such file or directory
11/10/21, 8:35 AM Nova[13903] getattrlist failed for /System/Library/Frameworks/OpenGL.framework/Resources//GLRendererFloat.bundle/GLRendererFloat: #2: No such file or directory
11/10/21, 8:37 AM Nova[14065] getattrlist failed for /System/Library/Frameworks/OpenGL.framework/Resources//GLRendererFloat.bundle/GLRendererFloat: #2: No such file or directory
11/10/21, 8:39 AM Nova[14194] getattrlist failed for /System/Library/Frameworks/OpenGL.framework/Resources//GLRendererFloat.bundle/GLRendererFloat: #2: No such file or directory

This appears to be happening for a number of extensions in Nova 8 - see https://github.com/apexskier/nova-eslint/issues/231#issuecomment-964699302 for more context.

Nova 8.1 is out with a probable fix

Resolved a crash when two or more extensions are installed that interact with document saves

Nova 8.1 fixed it for me


If a number is 0 this property will missing

Version of protoc (protoc --version)
3.0.0
Version of ProtocolBuffers.framework
4.0.1
Description
this is code for parse data to pb boject
let recogResult = try CloboticsRetail.GetImageRecognitionResultResponse.parseFrom(data: response.data)                     DEBUGLOG(message: recogResult)
this is server's return
boundingBox {
xmin: 0
ymin: 0.201082751)
xmax: 0.0288858321
ymax: 0.327146173
}
because xmin is 0  ,  xmin is missing from pb object
this is what i got .

if a number is 0 , can i parse to object and not ignore this property ?
if anyone can give some tips or solution , it will be a great help to me. thanks

i found every property can use hasXXX to check. close this.


Chaos v1 CLI Client

@ehsandeep

[ ] permission
[ ] "Status: In Progress" label

Working client for testing - https://github.com/projectdiscovery/chaos-client-v1


Remove naked returns

https://github.com/heptio/ark/blob/17b3a3b0730a23e1903663a21efb511ac4e1f95f/pkg/controller/restore_controller.go#L491
The return values for this function should be changed to not be naked returns.

@carlisia I have opened up a PR for this https://github.com/heptio/ark/pull/960


Sqlite quickfix

Quick version bump to address bug causing failed builds.
If this is affecting your build, see here for a temporary workaround:
https://status.heroku.com/incidents/1802

Thanks for the quick fix 👍

Thanks you for the fix 🙏
Just an FYI. In the linked issue https://status.heroku.com/incidents/1802 it says it only happened on the heroku-16 stack, but I had the issue on the heroku-18 stack while creating review apps.


ECR - Tags should be unique

Closes #5548

Codecov Report

Merging #5566 (b851ed2) into master (56ca48c) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #5566      +/-   ##
==========================================
+ Coverage   96.12%   96.14%   +0.01%
==========================================
  Files         736      742       +6
  Lines       72497    72622     +125
==========================================
+ Hits        69691    69822     +131
+ Misses       2806     2800       -6

Flag
Coverage Δ

servertests
37.00% <0.00%> (+0.12%)
:arrow_up:

unittests
96.08% <100.00%> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

moto/ecr/models.py
96.96% <100.00%> (+<0.01%)
:arrow_up:

moto/apigatewayv2/responses.py
100.00% <0.00%> (ø)

moto/servicequotas/resources/default_quotas/vpc.py
100.00% <0.00%> (ø)

moto/servicequotas/exceptions.py
100.00% <0.00%> (ø)

moto/servicequotas/__init__.py
100.00% <0.00%> (ø)

moto/servicequotas/models.py
100.00% <0.00%> (ø)

moto/servicequotas/urls.py
100.00% <0.00%> (ø)

moto/servicequotas/responses.py
100.00% <0.00%> (ø)

moto/apigatewayv2/models.py
98.17% <0.00%> (+<0.01%)
:arrow_up:

moto/rds/responses.py
99.30% <0.00%> (+0.01%)
:arrow_up:

... and 4 more

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


Docker image does not bind to <IP_ADDRESS> when only running azurite-blob

Which service(blob, file, queue, table) does this issue concern?
azurite-blob
Which version of the Azurite was used?
Azurite v3.2.0-preview (docker image)
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
DockerHub
What's the Node.js version?
N/A
What problem was encountered?
When running the full Azurite server from Docker, I am simply able to run the following command:
docker run -p 10000:10000 -p 10001:10001 mcr.microsoft.com/azure-storage/azurite

Which produce the following output:
Azurite Blob service is starting on <IP_ADDRESS>:10000
Azurite Blob service successfully listens on <IP_ADDRESS>:10000
Azurite Queue service is starting on <IP_ADDRESS>:10001
Azurite Queue service successfully listens on <IP_ADDRESS>:10001

Indicating that both the Azurite Blob service and Azurite Queue service are listening on <IP_ADDRESS> (which makes sense given that this is running in a Docker container).
However, if I try to run just the azurite-blob service with the following command that is listed in the Azurite README:
docker run -p 10000:10000 mcr.microsoft.com/azure-storage/azurite azurite-blob

I get the following output:
Azurite Blob service is starting on <IP_ADDRESS>:10000
Azurite Blob service successfully listens on <IP_ADDRESS>:10000

Indicating that the Azurite Blob service is now listening on <IP_ADDRESS>, which, because this is inside of a Docker container, rejects all connections from the host machine.
I had originally tried to run this using docker-compose, which resulted in a fair bit of confusion until I realized that the bind address had changed when running just the Azurite Blob service.
Steps to reproduce the issue?
See above.
Have you found a mitigation/solution?
Yes, I'm able to get this working by supplying the blobHost argument as follows:
docker run -p 10000:10000 mcr.microsoft.com/azure-storage/azurite azurite-blob --blobHost=<IP_ADDRESS>

It would be great if this was the default behavior when running within Docker (similar to the situation where you are running both the blob and queue service), however, if that isn't possible or easy, simply denoting this difference in behavior in the README would be very helpful.

Thanks! When doesn't specific azurite-blob command to start the docker image, there is --blobHost defined in dockerfile. That's why it uses "<IP_ADDRESS>".
azurite-blob command basically cannot detect it's under docker container or not, it needs the parameter.
Will update the readme documentation.

Sounds good, I figured it was probably something like that. Thanks for the info and reply.

Updated description in readme, will close this issue, thanks!


Task specific adapters

Thank you very much for an excellent repo. I am a bit confused on how to use task specific adapters. Let's assume we have 2 classification tasks. Here is a MWE for the model.
#Load the base Bert model from huggingface
model = BertModel.from_pretrained(model_dir, config=config)
tokenizer = BertTokenizer.from_pretrained('bert-base-cased')

config = AdapterConfig.load("pfeiffer", non_linearity='gelu')
model.add_adapter("dummy", config=config)
model.active_adapters = "dummy"

#This adds an adapter module in all the layers.

class ClassificationModel(nn.Module):
    def __init__(self, model, num_labels):
        """
        Torch model that uses the BERT and adds in a classifiers at the end. Num labels is a list of labels
        """
        super(ClassificationModel, self).__init__()
        self.model = model
        self.num_labels = num_labels
        self.dummy_classifier_task_0 = nn.Linear(self.model.config.hidden_size, num_labels[0])
        self.dummy_classifier_task_1 = nn.Linear(self.model.config.hidden_size, num_labels[1])

    def forward(self,input_ids,attention_mask):

        outputs = self.model(input_ids=input_ids,attention_mask=attention_mask)
        sequence_output = outputs[1]
        return self.dummy_classifier_task_0(sequence_output), self.dummy_classifier_task_1(sequence_output)

model_classifier = ClassificationModel(model, [3,4])
model_classifier.train()
#Then turn off all gradients except for the final layer and the adapters.
for name, param in model_classifier.named_parameters():
    if "dummy" in name:
        param.requires_grad_(True)
    else:
        param.requires_grad_(False)

#Then I pass in the parameters inside the optimizer.
opt = AdamW(model.parameters(), lr=lr, weight_decay=weight_decay)

This is how I wrap the model inside a pytorch nn module. But I do not know how to make the adapters specific to my two tasks. Also is there a way to add adapters at specific layers instead of all layers?
Thank you.

Hi @arijitthegame,
I'll give a short example how to handle two adapters with different classification heads for different tasks that hopefully makes things a bit clearer:
# create a model (note the AutoModelWithHeads class for flexible heads)
model = AutoModelWithHeads.from_pretrained("bert-base-uncased")
# add two adapters with two heads for different tasks
model.add_adapter("task1")
model.add_classification_head("task1", num_labels=x)
model.add_adapter("task2")
model.add_classification_head("task2", num_labels=y)

# Now, we can pass inputs to the model for training, e.g. by first training "task1" and then "task2":
# freeze all the weights except those for the current adapter with head
# this also tells the model to only this adapter and not the other one during a training pass
model.train_adapter("task1")
# training loop task 1
for ...:
    output = model(input)

model.train_adapter("task2")
# training loop task 2
for ...:
    output = model(input)

This example uses a model with flexible prediction heads as described here.
Here, both tasks would be trained sequentially. Alternatively, it would also be possible to pass data to different adapter within one batch of training data using BatchSplit.
Hope this helps!

@calpt  Thank you so much for your reply. This definitely clears things up. I am going to close the issue now.


Add breadcrumb in the content to suggest where the user has navigated

Is your feature request related to a problem? Please describe.
Add breadcrumb in the content to suggest where the user has navigated.
Currently all the simple graphs are just shown as Simple in the heading and the user cannot quickly gauge where they are just by looking at the content.
Describe the solution you'd like
Add a breadcrumb in the content above the graph.
Describe alternatives you've considered
N/A
Additional context
Multiline

Line

Paired Result

:tada: This issue has been resolved in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Failed to create GitHub repo. Error: 422 Unprocessable Entity

OS?
Windows 10
Versions.
angular-cli: 1.0.0-beta.25.5
node: 6.9.2
os: win32 x64
@angular/common: 2.4.3
@angular/compiler: 2.4.3
@angular/core: 2.4.3
@angular/forms: 2.4.3
@angular/http: 2.4.3
@angular/platform-browser: 2.4.3
@angular/platform-browser-dynamic: 2.4.3
@angular/router: 3.4.3
@angular/compiler-cli: 2.4.3
Repro steps.
use cli to create app.
create repo on github.
add repo as remote to newly created app
try ng github-pages:deploy
follow instructions
=> error: Failed to create GitHub repo. Error: 422 Unprocessable Entity
The log given by the failure.

Normally this include a stack trace and some more information.

Mention any other details that might be useful.
It looks like this happens because the cli attempts to create a repo itself (which already exists). Can the cli not check to see if a remote is already attached and if so just create the new branch with the deployed app?  This error is not clear and requiring users to delete their repo on github (my workaround) seems like it is not the best solution.

@goleary the cli already checks if the remote is set up as you can see here.
Can you manually run git remote -v in the project dir and see if there is an origin defined?

I did and there was a remote defined. I deleted my repo on github, removed the remote, and then re ran the github-pages command and it worked.

If you had a remote defined then it sounds like a bug.

just repeated the steps listed in my repro on the original and another machine and was unable to repro the error.
Not sure what the cause was yesterday, perhaps this can be closed until seen again?

I got the same error right now, my setup:
MacOS Sierra Version 10.12.2
angular-cli: 1.0.0-beta.26
node: 7.2.0
"@angular/common": "^2.3.1",
"@angular/compiler": "^2.3.1",
"@angular/core": "^2.3.1",
"@angular/forms": "^2.3.1",
"@angular/http": "^2.3.1",
"@angular/platform-browser": "^2.3.1"
Error is the same as one mentioned above, no stack trace.

Closed via https://github.com/angular/angular-cli/pull/4385.


Syn protocol query exception on windows ,V2.0.9328

I build the newest master code with version 2.0.9328 both on windows and Ubuntu.
In my tsl file , there is a syn protocol , it ok when use 1.0.8482 version , and throws exception on windows with the 2.0.9328 version:

Syn method is work fine on ubuntu.
the config V2.0 file seems not work, too.

the exception throws after this line:

Client side:

Server side:

and the client side throws the exception, but the server side didn't get the query request .

@BenDerPan can you step into the handler?

@yatli  I can not stop into the handler, I set breakpoints, but no response,I think the client request didn't send to the server side

More like a configuration problem. Could you please attach some output log? I think the client is configured as a server.

@BenDerPan could you please provide a minimal repro? I can help debug this case. :)

@yatli  I think you are right, maybe it's a configuration error, when I write configuration in C# code ,it's ok,
the server side catch the exception: cell id not found, and the client exception changed to Remote:
Server side exception fired:

Client side exception changed to Remote:

My configuration file as below:
Client and Server trinity.xml:

Is my trinity configure file correct? Sorry about to build a minimal repro, I try to build one ,but the exception changed to above (I changed to configure in code) , seems like the error never exists, caused by configuration file?... :(

@yatli  My hard code configuration:

@BenDerPan trying a minimal repro now. hang on.

@yatli  ok，I have a try, thank you. :)

@BenDerPan does it work for you?

@yatli  It's work now, but still some other problems

the Global.LocalStorage.Contains(long cellid) at the server side always return true, even though the cell is not exists,I have to use UseGenericCell(long cellid) method instead;
the trinity.xml(Version 2.0) configuration file will not work correctly, echo on console will not work, and  log dir, storage dir seems not work too.

@yatil sure, but I can't make a repro for you today, I am not on the computer now, tomorrow will be ok.  sorry about that...:)

@BenDerPan sure no problem!

Hi @yatli , I rebuild the latest master, and I can see log on console now :),
the contains minimal repro
TSL Contains  protocol:

Server side Contains implement:

Client side Contains request with random cellid , but always true:

(@BenDerPan Friendly reminder that @yatli is not me and that your @-replies don’t reach them but me. Feel free to delete this comment.)

@yatil, so sorry... entry errors...:)

@yatli does this fixed now?    :)

issue resolved. closing due to inactivity.


Hardcoded references to SURFconext wiki pages

Various pages in Teams have hardcoded references to SURFconext wiki pages
See:
https://github.com/OpenConext/OpenConext-teams/blob/master/coin-teams-war/src/main/webapp/WEB-INF/tags/genericpage.tag
https://github.com/OpenConext/OpenConext-teams/blob/master/coin-teams-war/src/main/webapp/error.jsp
https://github.com/OpenConext/OpenConext-teams/blob/master/coin-teams-war/src/main/resources/ftl/macros_htmlmail.ftl
https://github.com/OpenConext/OpenConext-teams/blob/master/coin-teams-war/src/main/resources/ftl/macros_plaintextmail.ftl

When does this need to differ? By language?

No, it needs to be configurable in case you are a different organization than SURFnet and want to run Teams. You want to be able to configure your own documentation URLs.

Ok. @surfnet-niels There are many more context dependent references that are hardcoded (like logos, titles, link names). Do you need it for something specific?

Initially I would consider everything that is in CSS as configurable. Optionally we could make the location of the CSS itself a config param:
https://github.com/OpenConext/OpenConext-teams/blob/master/coin-teams-war/src/main/webapp/WEB-INF/tags/genericpage.tag#L36 and https://github.com/OpenConext/OpenConext-teams/blob/master/coin-teams-war/src/main/webapp/WEB-INF/tags/genericpage.tag#L38
Most of the stuff in pages mentioned above is hardcoded contact data which should be in the config files I think.
The logo could be in CSS, not hardcoded into the pages

New issue on OpenConext-Teams-NG backlog.


[BUG]  如何用vllm部署qlora后的模型

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[X] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[X] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
No response
期望行为 | Expected Behavior
请问下如何使用vllm部署qlora微调后的模型呢？
复现方法 | Steps To Reproduce
No response
运行环境 | Environment
- OS:
- Python:
- Transformers:
- PyTorch:
- CUDA (`python -c 'import torch; print(torch.version.cuda)'`):

备注 | Anything else?
No response

同问


Disallow input values if not in the provided suggestions

Is it possible to disallow input values if not in the provided suggestions? I see there is a .filter() method but I don't think I want to overwrite the default functionality, is that what I should use? Does anyone have any idea how to achieve this?
If my suggestions were like this:
suggestions: ['sports', 'drama', 'romantic comedy', 'science fiction', 'thriller']

and a user tried to add action as a field value, I would like to disallow that.

See free option.


[MID PRI][Good First Task] Stop augmenting PATH for irrelevant packages.

I previously commited a change that prevents irrelevant packages from populating any of the variables for install/bin/man etc. Currently, however, all of those packages still populate the PATH, which means that even packages that don't perform any builds still complicate the PATH. I believe this makes invoking binaries with that environment slower (but also harder to debug).

You can confirm by doing esy sh -c 'echo $PATH' when you have a dependency on lodash or some totally JS package.

PR here since history doesn't track across repos.


jewel: remove rbd check

The value of doing this is fairly low compare to the added value.
So we remove these tasks, if rbd pool on Jewel doesn't have the right PG
value you can always increase it.
Signed-off-by: Sébastien Han<EMAIL_ADDRESS>

jenkins test luminous-ansible2.3-centos7_cluster


Draw from compendium?

Is there a way to draw from compendiums, or just imported/local tables? Using compendiums improves performance so keeping too many local tables (especially ones with many results) is undesirable.

I'll check into this and try to implement. I think it should be easy enough

That's awesome! Your module is great for my campaign but since I've moved everything to compendiums I've been struggling without it, I look forward to seeing what you come up with!

Fixed in latest release


in addition to closed issue #1145

Describe the problem
No JK BMS data is shown in the Venus OS display / logging, see #1145. I now installed fresh latest Venus OS + serial batery software and RS485 JK-BMS adapter (should be 8S-24S) version.
However there is an extension of the error message, between the 2 brackets [ ]; goes like this, the number seems te be every cycle different:
...
2024-12-04 08:35:15.932140500 INFO:SerialBattery:Testing Jkbms
2024-12-04 08:35:31.863215500 ERROR:SerialBattery:>>> ERROR: No reply - returning [len:222/27777]
...
What does it mean? Please advice for next steps. thx!
Driver version
venus-image-large-raspberrypi4-20241120085113-v3.52
Venus OS device type
Raspberry Pi
Venus OS version
v3.52
BMS type
JKBMS / Heltec BMS
Cell count
16
Battery count
16
Connection type
Serial USB adapter to RS485
Config file
same as in #1145

Relevant log output
root@raspberrypi4:/# lsusb
Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 001 Device 003: ID 1a86:7523 QinHeng Electronics HL-340 USB-Serial adapter
Bus 001 Device 002: ID 2109:3431 VIA Labs, Inc. Hub
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub

root@raspberrypi4:/# tail -F -n 100 /data/log/dbus-serialbattery.ttyUSB0/current | tai64nlocal
2024-12-04 08:33:12.734016500 INFO:SerialBattery:|- on slave address 14
2024-12-04 08:33:13.563226500 INFO:SerialBattery:|- on slave address 15
2024-12-04 08:33:14.383916500 ERROR:SerialBattery:>>> No reply - returning
2024-12-04 08:33:14.884950500 INFO:SerialBattery:-- Testing BMS: 2 of 3 rounds
2024-12-04 08:33:14.885305500 INFO:SerialBattery:Testing Daly at address "\x40"
2024-12-04 08:33:15.017033500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:15.017382500 INFO:SerialBattery:Testing Daly at address "\x80"
2024-12-04 08:33:15.146941500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:15.147283500 INFO:SerialBattery:Testing Daren485 at address "\x01"
2024-12-04 08:33:15.558275500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:15.558578500 INFO:SerialBattery:Testing Ecs
2024-12-04 08:33:15.971871500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:15.972220500 INFO:SerialBattery:Testing EG4_Lifepower at address "\x01"
2024-12-04 08:33:16.240581500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:16.508066500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:16.508608500 INFO:SerialBattery:Testing EG4_LL at address "\x01"
2024-12-04 08:33:16.776134500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:16.776675500 INFO:SerialBattery:Testing HeltecModbus at address "\x01"
2024-12-04 08:33:20.468181500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:20.468473500 INFO:SerialBattery:Testing HLPdataBMS4S
2024-12-04 08:33:21.579662500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:21.579966500 INFO:SerialBattery:Testing Jkbms
2024-12-04 08:33:37.510409500 ERROR:SerialBattery:>>> ERROR: No reply - returning [len:218/27777]
2024-12-04 08:33:37.511905500 INFO:SerialBattery:Testing Jkbms_pb at address "\x01"
2024-12-04 08:33:37.781230500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:37.782682500 INFO:SerialBattery:Testing LltJbd at address "\x00"
2024-12-04 08:33:38.054437500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:38.055907500 INFO:SerialBattery:Testing Renogy at address "\x30"
2024-12-04 08:33:38.326231500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:38.327624500 INFO:SerialBattery:Testing Renogy at address "\xf7"
2024-12-04 08:33:38.598076500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:38.599549500 INFO:SerialBattery:Testing Seplos at address "\x00"
2024-12-04 08:33:39.611992500 ERROR:SerialBattery:>>> ERROR: No reply - returning
2024-12-04 08:33:39.612272500 INFO:SerialBattery:Testing Seplosv3
2024-12-04 08:33:39.623819500 INFO:SerialBattery:|- on slave address 0
2024-12-04 08:33:40.454603500 INFO:SerialBattery:|- on slave address 1
2024-12-04 08:33:41.283968500 INFO:SerialBattery:|- on slave address 2

Any other information that may be helpful
No response

Check the FAQ.


Throws error when using React 0.14.3

When updating React I noticed that ReactList throws this error.
Warning: ReactList is accessing getDOMNode or findDOMNode inside its render().
render() should be a pure function of props and state.
It should never access something that requires stale data from the previous render, such as refs.
Move this logic to componentDidMount and componentDidUpdate instead.

Fixed in 0.7.5, thanks for the report.

Thanks!

Still see this problem. Reprocude by scrolling list with variable height.
I can of course provide an itemSizeGetter function and the error goes away.
But it would be neat if this library could handle all of that for me :smile:

Bah this warning is annoying because it's not applicable for this case.

Alright try 0.7.6, the warning should be gone.

Works great. Thanks!!


added support for cudnn flash attention

Implemented cudnn flash attention with Transformer Engine
Currently it supports head_dim till 128 and does not support GQA yet. It's an unstable API and would soon change it to a stable one with more feature support
Added a flag --multiprocess_gpu to support multiprocess SLURM tasks

Hi @rwitten I have made the changes. Please let me know how it looks.
Also I have been added to the contributor's list. My current commit email is verified but my inital commit email is not. Is there any way to remove it? Just wondering what could be the easiest way to fix this. Thanks!


RN 0.42.0 Fails to compile the Xcode project

Description
Action Taken  : Upgraded from RN version 0.41.0 to version 0.42.0.
Expected        : RN iOS project should compile and run properly (using Xcode 7.3).
Actual             : The RN iOS project fails to compile the file "react-native/React/Profiler/RCTProfile.m" (#line 386)
Here is the code that fails :
#if !TARGET_OS_TV
      UIActivityViewController *activityViewController = [[UIActivityViewController alloc] initWithActivityItems:@[[NSURL fileURLWithPath:outFile]]
                                                                                applicationActivities:nil];
      activityViewController.completionWithItemsHandler = ^(__unused UIActivityType activityType,
                                                            __unused BOOL completed,
                                                            __unused NSArray *items,
                                                            __unused NSError *error) {
        RCTProfileControlsWindow.hidden = NO;
      };
      RCTProfileControlsWindow.hidden = YES;
      dispatch_async(dispatch_get_main_queue(), ^{
        [[[[[UIApplication sharedApplication] delegate] window] rootViewController] presentViewController:activityViewController
                                                                                animated:YES
                                                                                completion:nil];
      });

#endif

Reproduction
Steps to reproduce : Upgrade the RN project from version 0.41.0 to 0.42.0
Solution
After commenting out the code mentioned above, the project compiles and runs successfully, however this by no means should be considered a solution. The developer who did those changes would be in a better position to address this (refer the point 1 in more information below)
More Information

Github commit which has caused this issue
Xcode version : 7.3
People here are facing this issue too

Additional Information

React Native version: [RN 0.42.0 : YES]
Platform: iOS only
Operating System : MacOS

@akashdmahali thanks for the detailed information. I ran into this recently as well, and while it didn't get to the bottom of the issue, updating Xcode from 7.3.1 to 8.2.1 allowed me to build successfully.

You'll need Xcode 8. Our tests are done using Xcode 8, therefore we cannot guarantee older versions are fully supported. You can still target older iOS versions than iOS 10 using Xcode 8.
Thanks for bringing this to our attention, the docs will be updated.


Commit transaction does nothing and produces no error messages

Using git revision 329651391ce1c209d1e56cafafb07002978989c7, I follow the procedure described in https://github.com/orientechnologies/orientdb/issues/3770#issuecomment-83228774
When I run the Java program, I get this output:
Mar 23, 2015 5:02:18 PM com.orientechnologies.common.log.OLogManager log
INFO: OrientDB auto-config DISKCACHE=1,137MB (heap=910MB os=4,096MB disk=7,120MB)
Exception in thread "main" java.lang.Exception: Matter not found with id0.43298458363146575
	at com.tkbt.Main.main(Main.java:42)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:483)
	at com.intellij.rt.execution.application.AppMain.main(AppMain.java:134)

Neither instance of orientdb shows any error messages. Is it just me or does this happen for other people?

With asynchronous replication the TX is executed on local node: if succeed the OK is sent to the client. If then, during the asynchronous replication any errors occur, the transaction is rollbacked. With asynch replication you can see dirty records (the isolation level is relaxed). Tried with synchronous replication and works.
Now why this fail with asynchronous? I'm logging the reason if that.

Reproduced. The tx worked, but cluster id was the default one: 3. That's why record can't be found. Fixed.


Signing in loop

Issue Type: Bug
I open Live Share, click "Sign in with GitHub" it redirects me to "Joining Visual Studio Live Share session with (user)" then it doesnt open up Visual Studio Code, and the bottom right bar ("Signing in...") is there stuck in a loop.
Extension version: 1.0.1293
VS Code version: Code 1.40.2 (f359dd69833dd8800b54d458f6d37ab7c78df520, 2019-11-25T14:54:40.719Z)
OS version: Linux x64 4.15.0-70-generic snap

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz (8 x 3100)

GPU Status
2d_canvas: unavailable_softwareflash_3d: disabled_softwareflash_stage3d: disabled_softwareflash_stage3d_baseline: disabled_softwaregpu_compositing: disabled_softwaremetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: disabled_softwareviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
4, 3, 3

Memory (System)
15.56GB (9.56GB free)

Process Argv
--force-user-env --no-sandbox --unity-launch --no-sandbox

Screen Reader
no

VM
0%

@Iskawo : thanks for reporting this issue! Sorry to hear you're having trouble signing in. Can you give us more details about the actions you are taking that lead to the sign in problems? If you can describe the sign in flow you're experiencing it will help us isolate the issue.
ex:

Clicked on Live Share button from VS Code task bar
Selected "Sign in with Microsoft"
Was presented with a prompt and clicked Open Link
...

#3035

We've changed our authentication service. See #4128 and if the new authentication is giving you an issue, please create a new issue. Thank you for your patience.


Shared fns

@mlap just a quick pass here trying to tidy things up a little with a bit more consistent use of shared functions in the gym.  Would be even better to add class inheritance so the methods can inherit their shared bits directly.  One thing I'm not clear on is how to do inheritance with custom init methods; e.g. since the init method for, say, the tipping point gym will take different structure.
Really this probably suggests the init method itself ought to just be refactored into something a little more generic (e.g. pass a dictionary of pars for the growth function, instead of hardcoding K, r etc into the init.  Unfortunately that would be a breaking change to user-facing code at this point so not sure if it's a good idea.
Also in the PR, I snuck in an additional method to try and estimate what "policy function" the agent is following.  Instead of running a simulated management scenario, it just steps through the state space on a uniform grid, and sees what action is proposed for each possible state (and does this reps number of times).  Not sure if that's helpful or not (obviously the agent will not have a well-tuned policy for states that it sees only rarely or never during training) but I think it can be useful in understanding the agent's behavior more generally (i.e. in transfer learning, or in comparison to standard theory).  Lemme know what you think.

All looks good, re inheritance, you can override the __init__ from the parent class but still keep the parent's methods. I'll get to this soon. And the policy function definitely can be useful, I did something like in the early days of this project.


odin: 0.13.0 -> dev-2023-05

Description of changes
updated odin package from 0.13.0 to dev-2023-05
since may 2021 odin has moved to monthly dev releases
I plan on keeping this up to date monthly for the foreseeable future as I use odin a lot
https://github.com/odin-lang/Odin/releases/
https://github.com/odin-lang/Odin/releases/tag/dev-2022-11
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

x86_64-darwin does not build and aarch64-darwin still needs to be added as a supported platform, working on a fix...

Given the github activity it would not surprise me if @luc65r has become inactive

Awesome! Thanks for this PR. It was exactly what I needed.

Works great! You may want to post it to discourse, ie. https://discourse.nixos.org/t/prs-ready-for-review/3032, or tag somebody whos more active than the current maintainer.
In the meanwhile, if anybody needs, here's an overlay for the newest version of odin:
    odin-overlay = self: super: {
      odin = super.odin.overrideAttrs (_: rec {
        version = "dev-2023-05";
        src = super.fetchFromGitHub {
          owner = "odin-lang";
          repo = "Odin";
          rev = "${version}";
          sha256 = "sha256-qEewo2h4dpivJ7D4RxxBZbtrsiMJ7AgqJcucmanbgxY=";
        };

        nativeBuildInputs = with super; [ makeWrapper which ];

        LLVM_CONFIG = "${super.llvmPackages.llvm.dev}/bin/llvm-config";
        postPatch = ''
          sed -i 's/^GIT_SHA=.*$/GIT_SHA=/' build_odin.sh
          sed -i 's/LLVM-C/LLVM/' build_odin.sh
          patchShebangs build_odin.sh
        '';
      });
    };

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/982

Just did the rebase. This should hopefully now work on Macos (worked in my vm). Will correct the wrapper logic later

Alright, this should now be everything!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2365


Remove MonadicValue2 and MonadicValue1

These are artefacts of this bug (https://github.com/aol/cyclops-react/issues/370) , and prevent a proper design for MonadicValues

Merged


Change enabled in all Enpoints API demos

It should be boolean not string

We have a trello card linked to this issue. Closing


chore(main): release 0.7.0

:robot: I have created a release beep boop
0.7.0 (2023-08-04)
What's Changed

docs: readme by @jitsedesmet in https://github.com/Mouwrice/proguard-core-visualizer/pull/105
feat: editor component by @jitsedesmet in https://github.com/Mouwrice/proguard-core-visualizer/pull/102

Full Changelog: https://github.com/Mouwrice/proguard-core-visualizer/compare/v0.6.0...v0.7.0

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Mouwrice/proguard-core-visualizer/releases/tag/v0.7.0 :sunflower:


Update

Good morning,
Your mod is amazing and my Discord team like to make it popular since it's getting more play since it's not updated, can you update it?
I ask you very kindly and it would make us very happy
I am French = text created with google translate

Yes, but so in your mod I have to update Reactor and lobbyOptionsAPI? and how do I update a file?
Je suis français = texte créé avec google translation

The parts modified by this mod are unlikely to have changed much in the game. So hopefully if you manage to update reactor it'll work.
You can follow the reactor documentation (get started) to install all dependencies:
https://docs.reactor.gg/
Then you can edit the AmongUsUnknownImpostors.csproj file with the correct versions.
Regarding optionsAPI, it is not mandatory, so you can either update it as well, or delete everything related to it from this mod

Thank you, I'll try it myself but maybe I won't be able to do it, it's a shame you don't have the time... I'll check it out over the weekend. If I succeed I will send you the .zip file so that you can upload it to Github.
Je suis français = texte créé avec google traduction


Add h3-pyspark to bindings.md

https://github.com/kevinschaich/h3-pyspark :)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Coverage remained the same at 98.2% when pulling fd86b3ea3c757d95e37f2aac126a8ea26e6bb1be on kevinschaich:patch-1 into 76bbde1dbf2280ca1675ec21c3ec195dcec8434e on uber:master.

Oh, based on the branch conflict, can you move this change to the main docs instead of the version-3.x docs (since we made the bindings page a symlink in version-3.x)

@isaacbrodsky @dfellis made both requested changes 👍


Tag classes dont set class to img

Hi. When configuring Tag classes, everything is ok (no error in the console), but it is not added class for the img tag in the editor. When can there be a problem?
Code configuration:
tagClasses: {
		h2: 'highline2Editor',
		h3: 'highline3Editor',
		img: 'imageEditor',
		p: 'paragraphEditor',
	},

Trumbowyg v2.25.0

It also doesn't work for paragraph (p) elements unless you explicitly press the paragraph button in formatting menu.

Duplicates #1227


Removed SQUASH COMMIT from the PR template

Removed SQUASH COMMIT from the PR template

Codecov Report

Merging #142 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #142   +/-   ##
=======================================
  Coverage   83.25%   83.25%
=======================================
  Files          56       56
  Lines        2789     2789
  Branches      295      295
=======================================
  Hits         2322     2322
  Misses        378      378
  Partials       89       89

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 469f132...be7cf5e. Read the comment docs.

could you do the same for other repos?

Doing it as we speak. lol


limit value above the maximum advertised (or allowed) should return that max limit rather than 400

This was recently resolved as an ambiguity in the OAFeat and STAC API specs for limit. If a limit parameter value above the max allowed limit is sent, the server should respond as if the max limit value were sent, rather than with 400 error.

I was taking a look at this and noticed that the BaseSearchPostRequest has a comment that it exists because it was waiting on a PR in stac-pydantic that has been merged for a while now. While there are some slight differences between the two functions, would it make sense to address this in stac-pydantic instead of here?
I played with a fix for this on my fork here but wanted to ask about that before submitting a PR anywhere.
There are a few other minor differences between the models, but it would be easy enough to port them all over there if that would be the better place for them to live.

While there are some slight differences between the two functions, would it make sense to address this in stac-pydantic instead of here?

I'm +0 on switching BaseSearchPostRequest to use stac-pydantic -- while it makes sense to put pydantic models in stac-pydantic, it does increase the time-to-fix for any changes, as we wait for bugfixes to propagate into a stac-pydantic release.
I think the best course of action would be to

Fix this issue in stac-fastapi using your branch
Reconcile any differences BaseSearchPostRequest and Search in stac-pydantic, get those changes released
In a new PR, update stac-fastapi to the new stac-pydantic version and remove BaseSearchPostRequest in favor of Search

That course of action seems totally reasonable. The delay in fixes certainly can definitely be a pain point. Especially with as many moving parts as there are. I understand why it ends up happening. Hopefully coordination can be streamlined over time.
I have one question about the tests in other parts of the code: This broke a few of the tests which are expecting a 400 error when the limit is set to 10001. For example: sqlalchemy/tests/api/test_api.py#L202 Would the right thing to do be just swap the 400 to a 200? I don't think it actually tests if the limit itself is functional, but it would catch a regression if it starts throwing errors in the future. Is there additional testing which should be added for this somewhere else?

I'm +0 on switching BaseSearchPostRequest to use stac-pydantic -- while it makes sense to put pydantic models in stac-pydantic, it does increase the time-to-fix for any changes, as we wait for bugfixes to propagate into a stac-pydantic release.

I agree with this.  For historical context, the original goal was to write stac-pydantic (get the data models right first) then build stac-fastapi on top of this.  But in reality the spec has changed quite a bit and the interpretation of the spec changes too which makes it difficult to always update stac-pydantic first because there is high coupling between the data models (stac-pydantic) and API validation (stac-fastapi).  I think its better to keep changes in stac-fastapi then backport to stac-pydantic where appropriate.

I have one question about the tests in other parts of the code: This broke a few of the tests which are expecting a 400 error when the limit is set to 10001. For example: sqlalchemy/tests/api/test_api.py#L202

Update tests to match the semantics of the spec.  So I'd expect a limit of 10,001 to return a 200 with limit being set to the server default maximum.

I think its better to keep changes in stac-fastapi then backport to stac-pydantic where appropriate.

Understood, thanks for the context. After the v1.0 release of STAC-API and a prolonged period of stability in stac-fastapi, it may make sense to refactor the models back down to stac-pydantic, but it does not seem like now is the right time.


Update and rename triage-issue-comments.yml to Astonprod

Why:
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

Hel

Why:
Closes:
What's being changed (if available, include any code snippets, screenshots, or gifs):
Check off the following:

[x]  I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x]  For content changes, I have completed the self-review checklist.

Who follows codes

Hubdue

Yee

Why:
Closes:
What's being changed (if available, include any code snippets, screenshots, or gifs):
Check off the following:

[x]  I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x]  For content changes, I have completed the self-review checklist.

D

👋 Hey there spelunker. It looks like you've modified some files that we can't accept as contributions. The complete list of files we can't accept are: .devcontainer/** .github/actions-scripts/** .github/workflows/** .github/CODEOWNERS assets/fonts/** data/graphql/** Dockerfile* src/** lib/redirects/** package*.json scripts/** content/actions/deployment/security-hardening-your-deployments/**
You'll need to revert all of the files you changed in that list using GitHub Desktop or git checkout origin/main <file name>. Once you get those files reverted, we can continue with the review process. :octocat:


User reader can drop class

On fresh 2.1-rc3 installation running console script below causes class C dropping.
set ignoreErrors true
drop database remote:<IP_ADDRESS>:2424/test root root
create database remote:<IP_ADDRESS>:2424/test root root plocal
connect remote:<IP_ADDRESS>:2424/test root root
create class C
connect remote:<IP_ADDRESS>:2424/test reader reader
drop class C
set ignoreErrors false
select from C

fixed in develop closing

Thank You very much.
Is it possible to apply fix to 2.0.11 also ?

Backported on 2.0.x, it will be in 2.0.12.

Backported on 2.0.x, it will be in 2.0.12.

Great, Thank You very much.


add support for fedora linux

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.04% :tada:

Comparison is base (4965906) 76.29% compared to head (206d46a) 76.34%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
+ Coverage   76.29%   76.34%   +0.04%
==========================================
  Files          97       97
  Lines        7046     7048       +2
==========================================
+ Hits         5376     5381       +5
+ Misses       1670     1667       -3

Files Changed
Coverage Δ

util/fibers/fibers_test.cc
99.06% <100.00%> (+<0.01%)
:arrow_up:

... and 1 file with indirect coverage changes

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


What's the problem? unrecognized %destructor

d:\php-sdk\phpdev\vc14\x64\php-src-php-7.0.14>nmake

Microsoft (R) Program Maintenance Utility Version 14.00.24210.0
Copyright (C) Microsoft Corporation.  All rights reserved.

        bison.exe --output=Zend/zend_ini_parser.c -v -d -p ini_ Zend/zend_ini_parser.y
zend/zend_ini_parser.y:295: unrecognized: %destructor
zend/zend_ini_parser.y:295:    Skipping to next %
NMAKE : fatal error U1077: 'd:\php-sdk\bin\\..\bin\bison.exe' : return code '0x1'
Stop.

Thanks for the report. Could it be, that you're on a DBSC system? Otherwise, I see that the old SDK is used. It would make sense to check with the new SDK branch as well.
Thanks.

windows 10，the sdk download from php.net, https://wiki.php.net/internals/windows/stepbystepbuild
http://windows.php.net/downloads/php-sdk/

Ups, DBCS system i meant. Or simply - is your default system codepage some of 932, 936 or alike? That is known to cause issues with some source files and tools, including Visual Studio. It might be good to fix this, but has not a big impact. The compilation on SBCS systems isn't known to cause any issues, so switching the code page to something like 437 should be fine.
Otherwise, the new_binary_tools branch is rather in focus now, about to move to master soon. It also has newer tools, so might work better.
Thanks.

Ah, ok then. So it indeed seems to be the codepage issue, i've met it sometimes with Korean and Japanese versions as well. With the legacy SDK, it's probably ok to fail in this constellation. Switching to 437 or alike should help, if the legacy SDK has to be used. The new branch is based on the tools from the current MSYS2, so that probably explains it. I think this one can be closed as well.
OFC you can use 2017, it's even already added into the build system. But with the SDK you'll need to go the old way as in the step by step doc. The new SDK reads all the data from the registry and the Visual Studio RC versions use separate registry entries, so they don't break the existing stable installs. OFC it'll work out of the box with 2017 stable with the phpsdk-starter.bat. We've no info yet, whether 2017 is going to be used for next PHP and all the deps are vc14 now, so be aware. But otherwise - just use the new SDK, but go the old way with vcvarsall.bat and co, to get the vc15 environment set.
Thanks.

Closing this one, since there's no issue with the new SDK.
Thanks.


Full text search

Is there any interest in adding full text search in the future?

Of course, it's a must-have feature. I hope you don't need it right now because I'm planning to do other things in the near weeks. It doesn't seem to be a large feature, maybe I could do it earlier.

I'm planning to add a search key in Prisma for a basic use-case, but more importantly to change the existing raw method that takes SQL string and then variables, into sql method that accepts a template literal, and to make where and other methods to be able to accept SQL:
const query = 'some & search';

db.article.where`
  to_tsvector(title || ' ' || body) @@ to_tsquery(${query})
`

It should simplify using various postgres functions and building custom queries.

I could make another GH issue about this if you like, but expanding on this point
From the docs:

Support for JOIN LATERAL is in the plans, it will enable an interesting optimization of joining and selecting relations.
Cannot select record with relations in the create, update, and delete queries.
No helpful methods available for PostGis, full text search, trigram search. Need to use raw SQL pieces for this.
Range database types, composite custom types aren't supported.
There are some methods for searching in JSON columns, but just very basic, need more.

This is unfortunately why I'm going to continue to use slonik, and the reason why I have been bitten by ORMs,  especially Prisma.  Prisma treats things like composite keys, PostGIS, and custom types as a "nice to have" rather than a core requirement.  Composite keys, PostGIS, etc. are core things that real world apps need.  Really, if you look at any ORM, it's where they become worthless and a regretted choice as you need to hack together custom queries and then the ORM itself becomes a burden.
My previous comment on this issue:
https://github.com/romeerez/orchid-orm/issues/12#issuecomment-1505870570
I encourage you to really make sure that the overall design of the library accounts for these scenarios early in the project and bump up the priority.  Users should always be able to expand the library themselves with custom types and raw SQL BUT also be able to fold it into the higher level parts of the ORM.
Anyway, thanks for letting me pound on this issue.  I really really want to use Orchid, but can't pull the trigger until I know that all the work I put into utilizing Postgres is available to me.

Neither I nor the big teams behind other ORMs are physically able to cover everything. That's a turn-off in the case of Prisma, as it's simply impossible to build specific queries with it, rather than writing them completely as a raw SQL. But it's not such a big deal with Orchid ORM and some other ORMs that allow using ORM models/interfaces and injecting only some pieces of raw SQL where needed.
Composite keys are indeed something that is currently lacking if we are talking about a situation where one table is referencing the other table by multiple columns. This is important and I'll try to add support for this in the near future.

I encourage you to really make sure that the overall design of the library accounts for these scenarios

That was an idea from the beginning: adding gradually various helpers to simplify common cases, and allow to inject only a piece of raw SQL where needed.

Users should always be able to expand the library themselves with custom types and raw SQL

Actually, that is in there: column types are customizable, and you can add your own type, and set a custom function to encode and decode it.

until I know that all the work I put into utilizing Postgres is available to me.

That's the goal, it will take more time. Right now, it's not a secret, some features aren't implemented, and different functionalities even weren't taken into account, like table inheritance for instance, there are tons of features that Postgres has.

but more importantly to change the existing raw method that takes SQL string and then variables, into sql method

The sql method I mentioned in the message above is implemented and published.
It is similar to what slonik does.

I looked closer into Postgres FTS capabilities, and it's so overwhelming. The good news: other ORMs don't even try to support this decently, the Orchid ORM could be the first.
But, it will take a lot of time and effort.

Ready! Here are the docs.


Add variants to Magic Missle

Make it so Magic Missle spell allows you to choose 1,2 or 3-action variant rather than 1-action being the only one avaliable.
It also works with feats like dangerous sorcery or sorcer's blood magic

Issue is that magic missile can be split amongst targets. The PF2e Workbench has a macro that handles it, dangerous sorcery included.

While it's true, the fact that a module makes it work, doesn't mean it should not be fixed, and most of the time all the missles go on the same target anyways. Maybe it isn't perfect, but it's still way better than what's right now at the core.

Issue is that magic missile can be split amongst targets. The PF2e Workbench has a macro that handles it, dangerous sorcery included.

While it's true, the fact that a module makes it work, doesn't mean it should not be fixed, and most of the time all the missles go on the same target anyways. Maybe it isn't perfect, but it's still way better than what's right now at the core.

We'll try to come up with something proper for this one, it's going to be such a unique case that it will likely just need hardcoding.


Not working with my own matrix server.

It seem the bot doesn't work with self hosted matrix server... :(

Hi, I haven’t tried this myself. Do you have any logs of the errors?

Yup:
[2022-04-15T09:51:03Z DEBUG crunch::matrix] response Response { url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("matrix.org")), port: None, path: "/_matrix/client/r0/login", query: None, fragment: None }, status: 403, headers: {"date": "Fri, 15 Apr 2022 09:51:03 GMT", "content-type": "application/json", "transfer-encoding": "chunked", "connection": "keep-alive", "synapse-trace-id": "a583afd31132411b", "cache-control": "no-cache,
no-store, must-revalidate", "access-control-allow-origin": "*", "access-control-a
llow-methods": "GET, HEAD, POST, PUT, DELETE, OPTIONS", "access-control-allow-headers": "X-Requested-With, Content-Type, Authorization, Date", "permissions-policy
": "interest-cohort=()", "cf-cache-status": "DYNAMIC", "expect-ct": "max-age=604800, report-uri=\"https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct\"", "server": "cloudflare", "cf-ray": "6fc3c7cc5c7c9bfe-FRA"} }
[2022-04-15T09:51:03Z ERROR crunch::crunch] Invalid password

Also there is no variable to set custom matrix server instead of matrix.org...
Bear in mind the username and password are set in .env file, but the debug log displays empty values, not sure if that's for security reasons or it fails to properly get the variables from file... If I put CRUNCH_MATRIX_BOT_USER=@:matrix.org all works fine...


MAINT: tweak to new available_apps() change

[NEW] added docstring of arg
[CHANGED] added type hint for return value

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
25 unchanged lines in 9 files lost coverage.
Overall coverage increased (+0.001%) to 91.593%

Files with Coverage Reduction
New Missed Lines
%

.nox/test-3-11/lib/python3.11/site-packages/cogent3/core/alignment.py
1
94.86%

.nox/test-3-10/lib/python3.10/site-packages/cogent3/app/init.py
3
97.41%

.nox/test-3-10/Lib/site-packages/cogent3/app/init.py
3
97.41%

.nox/test-3-11/lib/python3.11/site-packages/cogent3/app/init.py
3
97.41%

.nox/test-3-11/Lib/site-packages/cogent3/app/init.py
3
97.41%

.nox/test-3-8/lib/python3.8/site-packages/cogent3/app/init.py
3
97.41%

.nox/test-3-8/Lib/site-packages/cogent3/app/init.py
3
97.41%

.nox/test-3-9/lib/python3.9/site-packages/cogent3/app/init.py
3
97.41%

.nox/test-3-9/Lib/site-packages/cogent3/app/init.py
3
97.41%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
236373

Relevant Lines:
258068

💛  - Coveralls


Race cond fixes

Attempts to resolve some race conditions. Also added a Win32 writeable file that uses the Win32 API directly.

We are getting really close. I merged the PR locally and had to apply a little fix, but beside from that, it seems to work. I will try to break it, but so far it's holding strong.

I am closing ths PR, as I already merged it to the feature/memory_mapped_files branch


Update to vee-validate 3

Hi, vee-validate 3 was released, are there any plans to update this nuxt plugin?

Will be super great!

Yes! This weekend I will start the rewrite for V3 compatibiity

@lewyuburi Any update on this? Thanks!

Hi all, @aaharu is working on it: https://github.com/lewyuburi/nuxt-validate/pull/22 :D Help wanted for testing. Thanks to all for this colab <3

Is there any update on this?

I see #22 has been merged. @lewyuburi do you have any plans to do a new npm release?

@lewyuburi I wanted to let you know that I tested the master branch and it worked perfectly. I never used vee-validate before, so I couldn't test it with some advanced configuration.

I forked and published this package... 😐
If you need it immediately, use https://www.npmjs.com/package/@mole-inc/nuxt-validate

I guess this issue should be closed?

The version of vee-validate currently released on NPM is not using version 3 of vee-validate.

The version of vee-validate currently released on NPM is not using version 3 of vee-validate.

@jonkri Wow... Great.. however it's been merged months ago. What a waste of good work ;)
Sent with GitHawk

ping @lewyuburi

Any update on that after almost one year?

I'm under the impression this package is no longer maintained. Main issue is the version wasn't bumped from 1.0.1.
Anyone have any alternatives they recommend when working with nuxt? I have to admit I'm not a big fan of vee-validate to begin with, but I haven't played with v3+ 😅


Simplify send, request and reply help options

For these 3 commands I think we should move these two items to -hm. It will shorten the list and most folks wont' need to mess with these for getting started.
  --time-to-live <MILLISECONDS>            the time before a message is discarded or moved to a DMQ
  --dmq-eligible [BOOLEAN]                 the DMQ eligible flag (default: true)

Fixed in v0.0.36


Update changelog with MonoTimer frequency fix

Following up on #56 to also update the changelog.

Cool, thank you :)


Authentication library decoupling

Description
Decouples authentication library from the server. Now it's possible to load alternative authentication libraries. Microsoft internal user story 91775.
Testing
Manual by assigning authority and audience in appsetings.json
TAGS: "Enhancement", "Azure API for FHIR", "Azure Healthcare APIs"

This does pr not change behavior in any way so there's no need to add any additional tests. It just moves initialization of the authentication library to a different place.


3D Attribute and Hit Test Samples

Sample
Add 3D Attribute Sample and Hit Test Sample
Summary

Add 3D Attributes Sample
3D Attributes has UI to Match other Samples
Add Hit Test Sample
Hit Test Sample has UI to Match other Samples
Hit Test Sample uses New Input System

Tests
Windows
ArcGIS Maps SDK Version
Unity LTS 2021.3 W/ Maps SDK 1.3

With this going to main, we need readmes for these samples and updates to the ToC readme.

I can make the readme's for these samples but Tomo will need to edit the main one or will need to tell me what to do for it. Additionally, now that the Split Samples Project is merged in, the VR sample readem in ToC is broken and we need to update it anyway to explain the splitting the project for XR vs Normal samples. So that likely needs to be it's own PR. But I will do the readmes for this.

The main readme for the repository still does not have the new samples added to its table of contents.

As I mentioned above, I was told not to touch the main readme because the doc website pulls data from that readme. So I need to confirm with Tomo it is okay for me to do so.


network: add RMRK2 protocol to ZERO network

User Story
As #creator I want to be able to mint collectables on zero network so I can build new applications.
Acceptance Criteria

add RMRK protocol to zero network
enable creation of classes
enable minting of items
also a realm feature is needed to provide a map for classes created in a DAO context

Additional Information
How to test

a
b
c
d

This issue belongs to zero-network, and it was already finished. RMRK was added, creation of classes and items was enabled. Closing it as DONE.


Search Results page -- make nested checkboxes (subfacets) keyboard-accessible

From NYU A11y Review:
Under the Resource Types heading, the Publications menu is not constructed in a way that the keyboard-only user can access the sub-filter elements. Review the WAI-ARIA authoring practices for checkbox.
Also, the element that signifies that the Publications menu has more options:

is not actionable and not associated with the checkbox. Also, the sub-filter elements cannot be selected unless the Publications element is selected. Consider removing the "show/hide" design of this element so that all options are exposed.

This issue has returned.  The disclosure arrows for the search pages' facts are no longer keyboard accessible.


[improve][misc] Disable OTel by default when running the pulsar-perf tool

Fixes #23475
Motivation

Since pulsar 3.3.0 (https://github.com/apache/pulsar/pull/22179), pulsar-perf constantly print warning message on the console if user haven't configured a OTel-compatible metric collector in their environment. To align with broker, proxy, and other components, OTel metric collection should be disabled in pulsar-perf by default. https://github.com/apache/pulsar/pull/22010
With this patch, the following log will appear before running the performance test:
Nov 10, 2024 3:36:19 PM io.opentelemetry.api.GlobalOpenTelemetry maybeAutoConfigureAndSetGlobal
INFO: AutoConfiguredOpenTelemetrySdk found on classpath but automatic configuration is disabled. To enable, run your JVM with -Dotel.java.global-autoconfigure.enabled=true

Verifying this change

[x] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

An alternative would be to have OTel configured, but disabled by default as it's done in Pulsar's OpenTelemetryService: https://github.com/apache/pulsar/blob/dd1b57944b117d16ebd371996b44c02af2ce325c/pulsar-opentelemetry/src/main/java/org/apache/pulsar/opentelemetry/OpenTelemetryService.java#L74-L75 .
@dragosvictor Please review this PR and suggest a suitable approach.

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

Project coverage is 74.20%. Comparing base (bbc6224) to head (55b8b36).
Report is 749 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23585      +/-   ##
============================================
+ Coverage     73.57%   74.20%   +0.63%
+ Complexity    32624     3353   -29271
============================================
  Files          1877     1945      +68
  Lines        139502   149262    +9760
  Branches      15299    16698    +1399
============================================
+ Hits         102638   110766    +8128
- Misses        28908    29917    +1009
- Partials       7956     8579     +623

Flag
Coverage Δ

inttests
27.28% <ø> (+2.69%)
:arrow_up:

systests
24.39% <ø> (+0.07%)
:arrow_up:

unittests
73.76% <ø> (+0.92%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 670 files with indirect coverage changes

Workaround for 4.0.0
export OTEL_SDK_DISABLED=true


Improper Error Handling

Describe the bug
The following code has been added - which exposes the internal error. This should not be surfaced to the UI.
return (jsonify({"error": str(e)}), 500)

Expected behavior
We should adhere to these guidelines https://owasp.org/www-community/Improper_Error_Handling
Steps to reproduce
There are 7 occurrences of this in chat_history.py

Thanks @ross-p-smith  for this reporting this issue w'll look into it.

We are evaluating a fix for this Issue once fix is identified will post an update here. Thanks for your patience !!!!

Hi @ross-p-smith, just a friendly reminder that we've completed and tested the fix. Could you please review it and confirm?

Hi @ross-p-smith, Bug fix code changes are completed and tested. Hence Closing the Bug.


Newly added bank account which is in draft review status still can be used on invoice

Adding new bank account, with workflow setup and Bank account approval (create) parameter to yes produces following behaviour:
Bank account is created, review status is Draft, Status is Active
This bank account can be selected used (i.e. selected on Vendor invoice and posted)
So the statement "The new bank account record won't be available for use until an approver has approved it." is not correct
Furthermore, this whole functionality has very limited use unless it stops ability to add new bank account and use it without approval

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

ID: 82d19f25-f8e8-7b5a-ad68-281f49b644a3
Version Independent ID: 82d19f25-f8e8-7b5a-ad68-281f49b644a3
Content: Vendor bank account workflow - Finance | Dynamics 365
Content Source: articles/finance/accounts-payable/vendor-bank-account-workflow.md
Service: dynamics-365-finance
GitHub Login: @GalynaFedorova
Microsoft Alias: gfedorova

Bug 818704 has been logged to address this issue.

@cjokanov, thanks for contacting us. Unfortunately, we're unable to handle Support requests for the product here. To contact Support, consider one of the options described here: Support contact options. In particular, you might want to go to the Dynamics 365 community page to get help with your questions from the Dynamics community.


⚠ Processors has degraded performance

In f842e66, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 41 ms

Resolved: Processors performance has improved in 46d3dea.


benchmark dependencies are outdated

The benchmark optional dependencies are outdated, and xgboost==1.3.3 doesn't play nice with newer versions of pandas, which are installed by default.
Running flaml after a flaml[benchmark] installation results in an error:
...
  File "/Users/pietergijsbers/repositories/automlbenchmark/frameworks/flaml/venv/lib/python3.9/site-packages/xgboost/data.py", line 242, in _from_pandas_df

    data, feature_names, feature_types = _transform_pandas_df(

  File "/Users/pietergijsbers/repositories/automlbenchmark/frameworks/flaml/venv/lib/python3.9/site-packages/xgboost/data.py", line 192, in _transform_pandas_df

    from pandas import MultiIndex, Int64Index

ImportError: cannot import name 'Int64Index' from 'pandas' (/Users/pietergijsbers/repositories/automlbenchmark/frameworks/flaml/venv/lib/python3.9/site-packages/pandas/__init__.py)

Hi @PGijsbers, thank you for raising this issue!

I will make a PR to resolve the conflicting versions of pandas and xgboost.
You mentioned "optional dependencies are outdated". Could you please clarify what specifically are the  problems (in addition to the version problem in mentioned in 1)?

About the timeline: flaml will be releasing a new version soon (approximately in a week or two), which has a major change on how the lib shall be installed and requires corresponding changes in automlbenchmark. I plan to make a PR to make sure the new version of flaml works properly here, and also address problems 1 and 2 mentioned above in that PR.
Please let me know if you have any comments or suggestions! Thank you!

Could you please clarify

Sorry about this miscommunication, it shouldn't have been an "and" there. The only issue I identified is that the outdated xgboost does not work with the more modern pandas.
As for the benchmark, we will run experiments soon. I will try to hold off with running flaml experiments to see if we can have it run with the fixed new version instead, but I can't make promises there as we are on a tight schedule.

Thanks for the information. One easy workaround I can think of at this time point is to resolve the dependency issue in flaml's setup.sh file. i.e., adding the following command line below [line 13] (assuming that you will be running the stable version in your experiments)(https://github.com/openml/automlbenchmark/blob/master/frameworks/flaml/setup.sh#L13),
PIP install pandas==1.1.4

I have tested it and it works well. Please let me know how I could further help. Thank you!

Is there a specific reason that you wish to use the outdated xgboost, then?

@PGijsbers, thank you for your questions and response.
"Is there a specific reason that you wish to use the outdated xgboost?"
This is mainly because we haven't evaluated the performance of flaml with newer versions of xgboost. As a result, we've added this requirement as a more conservative measure. However, it's possible that newer versions might function just as well. Therefore, I'm comfortable with disregarding this requirement in your PR.
I have reviewed your PR and left some other comments. Thank you!


AG Listener creation failing

I am getting an error with Listener creation. I am setting up 2 node cluster.
Any idea from where does the Availability group reads the WSFC cluster config?
An exception occurred while executing a Transact-SQL statement or batch. (Microsoft.SqlServer.ConnectionInfo)
------------------------------
The Windows Server Failover Clustering (WSFC) resource control API returned error code 5057.  The WSFC service may not be running or may not be accessible in its current state, or the specified arguments are invalid.  For information about this error code, see "System Error Codes" in the Windows Development documentation.
The attempt to create the network name and IP address for the listener failed. The WSFC service may not be running or may be inaccessible in its current state, or the values provided for the network name and IP address may be incorrect. Check the state of the WSFC cluster and validate the network name and IP address with the network administrator.
The attempt to create the network name and IP address for the listener failed. The WSFC service may not be running or may be inaccessible in its current state, or the values provided for the network name and IP address may be incorrect. Check the state of the WSFC cluster and validate the network name and IP address with the network administrator. (Microsoft SQL Server, Error: 41009)

Finally able to create the Availability Group.


Add IR page titles

What is the context of this PR?
Individual response currently has missing page titles, they should be updated as per the spreadsheet in the card.
How to review
Use test_individual_response.json Ensure the page titles now match those in the spreadsheet.
Checklist

[x] New static content marked up for translation

 Add page titles when requesting an individual response (M)


Feature: State watcher

Hi, great work on the redux-starter-kit, really loving it!
My requested feature is a state watcher utility, something like subscribe() but on a higher-level. While it is entirely possible to make it your own, it's very hard for beginners to not mess it up. And currently there is no function that lets you observe the store for changes, and as stated in the docs, it's not practical to use subscribe() directly as stated here https://stackoverflow.com/questions/36212860/subscribe-to-single-property-change-in-store-in-redux
So maybe, to help non-advanced users, a utility function to watch (or observe) either a single property in the state or the whole state.
EDIT: I'd like to add while this still exists https://github.com/jprichardson/redux-watch , it's better to have an updated version of it which adds more features and a better syntax maybe.

It's an interesting idea, but I don't think it's something I want to add directly to RSK.
As you noted, redux-watch exists, and there's several other similar libraries as well:
https://github.com/markerikson/redux-ecosystem-links/blob/master/store.md#store-change-subscriptions
Most of the time, store state changes are only needed in components, in which case you know use mapState or useSelector and know that it's changed when the component re-renders.
Out of curiosity, what specific use case do you have for needing to watch changes to a given piece of state?

I'm using Redux on the server-side of a data-driven framework, I have a lot of modules and it's easier to communicate all that data using redux (and eventually rsk), my server opens a websocket to be used on the client via an interface or anything (to monitor the data). Since the store holds a lot of data, and a big chunk of that data doesn't actually update, I wanted to observe a set of properties that are frequently changing to not overload the client and only send what changed IF changed.
I understand that on React, most of the time store state changes are indeed only needed in components. While this is completely true, it's important to remember that Redux is gaining a big popularity in the backend as an alternative for the classic pub-sub system.

Hmm.  To be honest, I haven't seen or heard much about Redux being used in the back end, so I'm not nearly as familiar with that use case.
I don't immediately know of a good addon that would produce a diff of just the changed fields.  One might exist, and there might even be some in that list I linked, but if so it's not something I specifically remember seeing off the top of my head.
I think building one would be entirely feasible, but I also don't see that as something that needs to be added to RSK directly, so I'll go ahead and close this issue.  If you do find or build something like that, feel free to comment here to let me know.


Set exit status based on success/failure?

For integrating pandaPIengine into pipelines, scripts, etc. it would be helpful if it was easy to tell whether it had succeeded in finding a plan or not.
For this, it would help if the exit status was set to 2 (or some other designated value) on search failure.  Looking at the code, it seems like an exit status of 1 is reserved for an error condition.
I have had a start at trying to make a MR for this. It seems to be straightforward for translation mode, and mostly straightforward for the heuristic search mode, but for the SAT planner option, I can't figure out all of the ways that solve_with_sat_planner and its subsidiary function solve_with_sat_planner_linear_bound_increase can exit (the while(true) loop in the latter I can't figure out -- how does it exit on failure here?)
Any suggestions?

Thanks for starting the work!
As for the SAT solver, basically whenever the call returns anything other than 10 or 20, this is an error (except if we set terminations, which we don't). I've just pushed a commit that checks for this and exits the program with code 2 if this happens.
Could you add your PR?

I have a shot at it in #14


add ai extra setup

Closes https://github.com/holoviz/lumen/issues/583

No I think it's a good suggestion!


Add Examples For Rules

It would be very nice to have code examples showing correct and incorrect code for the rules.
Could use ESLint docs for reference

I had a look through the test codes.
It may not be a hard work to add those to the website

@clemtrek This may be in progress already. Is that PR what you're after?

Duplicate of #1329.


Scrubbed internal links

got rid of http://go/ links according to this.
see #1065

This approach isn't quite viable, but I'm going to pull it in and see if I can massage the internal version sufficiently.

可以拉取合併嗎？謝謝

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


[FancyZones Editor] hold-shift for rotating splitter fix.

Summary of the Pull Request
Fix unexpected behavior on pressing shift.
PR Checklist

[x] Applies to #7636, #750
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed

Info on Pull Request
Added key event handlers in GridEditorWindow and layout windows.
After opening the editor the GridEditorWindow is activated and catches key events. When a user starts to edit the layout, a layout window becomes activated. Without any of these handlers behavior would be broken.
Validation Steps Performed
Edit any layout of the grid type (Columns, Rows, Grid, Priority Grid), hold shift, and verify that the splitter has changed orientation. Split layouts and verify that shift behavior still works as expected.

What's the diffrence to #8279

What's the diffrence to #8279

It's the same but for the stable branch.


Mahad Aly Nurul Jadid

Add Campus Mahad Aly Nurul Jadid , Paiton - Probolinggo - Jawa Timur - Indonesia
Institution/School Name
Mahad Aly Nurul Jadid
Instiution/School Website
https://mahadaly-nuruljadid.ac.id/
Email Users
Please place an x between the square brackets if yes

[x] Students
[x] Academic/Teaching Staff
[x] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[x] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@abunaum First stage of verification passed, official domain is mahadaly-nuruljadid.ac.id as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@abunaum Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course

@abunaum You might miss my previous comment. Please react.

@abunaum I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


test: fail on unraisable exceptions

See https://docs.pytest.org/en/latest/how-to/failures.html#warning-about-unraisable-exceptions-and-unhandled-thread-exceptions
Unraisable exceptions and unhandled thread exceptions are normally considered bugs, but may go unnoticed because they don’t cause the program itself to crash. Pytest detects these conditions and issues a warning that is visible in the test run summary. This PR changes the warnings to errors instead.

Is there a current test that suffers this issue or is this PR more like a general enhancements for our test suite?

Is there a current test that suffers this issue or is this PR more like a general enhancements for our test suite?

Well if there were any issues in the current tests, the CI wouldn't pass. So, by turning on this option we prevent future issues :slightly_smiling_face:


Release time and plans

Hi,
Is the time of the next release already known? Is there a criteria or schedule for future releases when fixes and new features are added?
Thanks,
João

Hi João,
we do not have specific release planning in place at the moment.  Fixes and also small feature may be released very soon, especially fixes that we think are critical.  Larger scale feature enhancements typically depend on manuscripts prior or in conjunction with the publication.
HTH,
Hannes.

Thanks! Yes, I was more wondering for fixes at the moment, since I've seen you pushing some fixes to the main branch, and I was wondering when they would be part of a release. OK, I continue working with 4.0.11 at the moment, and update when new releases are available.

We use semantic versioning and follow a traditional open-source strategy which means that fixes will be pushed as as early as possible and then increase the patch level.  Any fix will be available in the next minor release of course, but we do not have specific releases with accumulated patches.  I would therefore recommend to keep an eye on changelog and update fairly frequently.  Since 4.0.11 we had quite a few fixes which otherwise would lead to crashes or even lead to wrong results in certain circumstances.

Oh, I see. There's already version 4.0.28, but there's no tag for it. I only see a tag for 4.0.11. With the tags, it's easy to git-checkout the right version to refer to it, and install from it.


Session id cannot be set to a value different from the session id of the message currently being processed

Is this a bug report?
Yes
Can you also reproduce the problem with the latest version?
Yes.
Issue is present in versions 7.3.0 & 7.3.1
Issue is not present in version 7.2.4
Environment

Dotnet version: net core 6
Operating system: Win 11
IDE (Rider, VS Code, Visual Studio): Visual Studio

Steps to Reproduce
When a consumer receives a message with a session id, and tries to publish a message with another session id, then an exception is thrown stating that the sessionid cannot be set to a value different from the partition key.

Register a consumer requiring session for a message
Send a message matching the consumer one, with a given session id
In the consumer, publish a message with another session id

Expected Behavior
Message with new session id should be published.
Actual Behavior
An exception is thrown:
System.ArgumentOutOfRangeException: SessionId:ACK-TEST cannot be set to a different value than PartitionKey:TEST. (Parameter 'value')
         at Azure.Messaging.ServiceBus.ServiceBusMessage.set_SessionId(String value)
         at MassTransit.Azure.ServiceBus.Core.Transport.ServiceBusSendTransport.SendPipe`1.CreateMessage(AzureServiceBusSendContext`1 context)
         at MassTransit.Azure.ServiceBus.Core.Transport.ServiceBusSendTransport.SendPipe`1.Send(SendEndpointContext clientContext)
         at MassTransit.Azure.ServiceBus.Core.Transport.ServiceBusSendTransport.SendPipe`1.Send(SendEndpointContext clientContext)
         at GreenPipes.Agents.PipeContextSupervisor`1.GreenPipes.IPipeContextSource<TContext>.Send(IPipe`1 pipe, CancellationToken cancellationToken)
         at GreenPipes.Agents.PipeContextSupervisor`1.GreenPipes.IPipeContextSource<TContext>.Send(IPipe`1 pipe, CancellationToken cancellationToken)
         at GreenPipes.Agents.PipeContextSupervisor`1.GreenPipes.IPipeContextSource<TContext>.Send(IPipe`1 pipe, CancellationToken cancellationToken)
         at MassTransit.Transports.HostConfigurationRetryExtensions.Retry(IHostConfiguration hostConfiguration, Func`1 factory, ISupervisor supervisor, CancellationToken cancellationToken)
         at SessionIdBug.MessageConsumer.Consume(ConsumeContext`1 context) in C:\Code\Git\_Temp\SessionIdIssue\SessionIdBug\MessageConsumer.cs:line 32
         at MassTransit.Scoping.ScopeConsumerFactory`1.Send[TMessage](ConsumeContext`1 context, IPipe`1 next)
         at MassTransit.Scoping.ScopeConsumerFactory`1.Send[TMessage](ConsumeContext`1 context, IPipe`1 next)
         at MassTransit.Pipeline.Filters.ConsumerMessageFilter`2.GreenPipes.IFilter<MassTransit.ConsumeContext<TMessage>>.Send(ConsumeContext`1 context, IPipe`1 next)

Reproducible Demo
SessionIdBug.zip

And if you set the partition key to the same value before setting the sessionId?
V7.3 changes to the latest azure SDK which is likely the reason an exception is thrown by the SDK when you attempt to change it.

Hi,
Thanks for the feedback.
If we set the PartitionKey, either before or after (but I believe it only sets values on the SendContext before them being applied on the SDK objects), then the issue goes away.
I understand that SesssionId is used as the partition key when set, while PartitionKey is used to partition messages when sessions are not used.
Wouldn't this mean that SetSessionId should also set the PartitionKey on the messages, while at the same time maybe SetPartitionKey should clear any previously set SessionId if different ?

Sure, that would be an easy PR :)

Do you want me to give this a try ? (master is still the 7.3 branch right ?)
Or do you prefer to do it ?

You can, but it would need to be on develop. There will be no further v7 releases.

To me this seems fine !

Good, cause that's what I committed to the branch :)


UsePythonVersion@0 doesn't work with service containers

Type: Bug
Enter Task Name: UsePythonVersion@0
Environment

Server - Azure Pipelines:
account name: pluckd
team project name: pluckd
build number: #20220410.8

Issue Description
Whenever I use the UsePythonVersion@0 task in conjunction with service containers, I get one of the following error whenever I call pip, python3 or python:
/__w/_temp/114bae27-4b79-449b-a114-c10064c5ee12.sh: /__t/Python/3.9.12/x64/bin/pip: /opt/hostedtoolcache/Python/3.9.12/x64/bin/python: bad interpreter: No such file or directory
or
python3: error while loading shared libraries: libpython3.9.so.1.0: cannot open shared object file: No such file or director
Task logs
logs_94.zip

I'm experiencing the same issue. Is there any workaround at present?

A possible workaround is running your job in a Python container. My particular use case is running Django unit tests against a PostgreSQL container
`- job: Test
displayName: Run unit tests
container: python:3.10
services:
  db: db
steps:
- task: PythonScript@0`
   ...

Is there an update or a workaround that doesn't involve making a container?
We are transitioning some services from cloud to on-premise, and we need to be able to use UsePythonVersion@0 on local agents.
From what I see the issue seem pretty old as I see people posting on it close to 4 years (!!!) ago.
We are running the agent on a kubernetes cluster, everything is running Ubuntu server 20.04. The container itself is running Ubuntu 20.04 as well. The problem seems to be that, either the libraries aren't linked properly, or that they should be on $PATH but they aren't.


Css question/guidance

I am trying to style the number of the active desktop (3 in the picture). What would be the css property to do this ? As you can see I have already styled the active content as such:
.Active .Contents, .Visible .Contents {
     background-color: @active-window-color;
}

.Active .WorkspaceLabel {
	font-size: 12pt;
}

You can use https://gitter.im/taffybar/Lobby to ask one off questions like this.


Provide footer-links to the tickets-shop

This PR fixes and adds the feature #204  of adding customizable links to footer.

There are commits here that should not be there.

There are commits here that should not be there.

No! only the second last one is from my local branch


Fullscreen thumbnails

Hi,
not an issue, more a question: is it possible to remove thumbnails when switching to fullscreen somehow?
Many thanks.

Hi,
It's possible to set the thumbnails' height to 0 with this code:
$( document ).on( 'fullscreenchange webkitfullscreenchange mozfullscreenchange MSFullscreenChange', function() {
    var isFullScreen = document.fullscreenElement || document.webkitFullscreenElement || document.mozFullScreenElement || document.msFullscreenElement ? true : false;

    if ( isFullScreen ) {
        $( '.slider-pro' ).sliderPro( 'thumbnailHeight', 0 );
    } else {
        $( '.slider-pro' ).sliderPro( 'thumbnailHeight', 100 );
    }
});

Best,
David

Works fine, many thanks!