Sbee dev

Changes

Refactor GainDB
Resolve #183

Acceptance Criteria

Boot ✅✅
Get Spectra ✅✅
Switch to Hardware page, correct string shown next to [Save .ini File] ✅
Debug Gain visibility always True, set gain persist after restart ✅✅
Save measurement, to disk, and display in 'Clipboard/Measurements'  ✅
Load measurement from disk ✅

Two checkmarks means another check after merging main

TODO: "persistent GainDB via .ini" >> changelog
Also this shadows the device's 'default GainDB' eeprom setting. So the only time it gets loaded from eeprom is the first time this spec is connected to a particular computer. This isn't necessarily a problem, as it saves us unnecessary writes to the EEPROM, but it can be categorized as kind of weird default behavior-- and it does not support gain persistence across multiple computers, obviously.

changelog

It's in there.

this shadows the device's 'default GainDB' eeprom setting

Yeah.  So, history:
EEPROM availability: originally, Wasatch spectrometers lacked an EEPROM, so the .ini file was literally where such things were mastered. Then we added the EEPROM, but still had lots of spectrometers in the field without EEPROM, so we were kind of maintaining both. Then we dropped ENLIGHTEN support for the old (non-FID) spectrometers, so "nearly all" spectrometers now have EEPROMs (*XL).
EEPROM fields: the "startup_" EEPROM fields, supporting initial integration time, gain, and TEC setpoint are relatively new.  So we were still very much using the .ini to let users configure the "normal, intended" settings for that unit.  Now we have the startup fields, but as you say EEPROMs have a limited write lifetime.
Scalable configurability: we don't necessarily want to let all users configure all settings...some settings can lead to degraded measurement quality, others could potentially infringe safety, others could even damage equipment. Some users are highly-qualified scientist/engineers, some are station operators. The thought was that we would roughly group settings into 3 categories: 1. those anyone could configure in the .ini, 2. those OEM/advanced users could configure in the EEPROM ("edittable"), and 3. production/"everything".  And now we've stirred that pot again by adding "Expert Mode," which would be about a 1.5 (e.g. can fiddle excitation on-screen, but still not write EEPROM).
So yeah, this is a sliding scale across multiple dimensions with multiple vocal stakeholders, and we're trying to maintain a balance across different use-cases.
In this particular case, the EEPROM startup fields are arguably more important for people not using ENLIGHTEN (OEM integrators who want the unit to startup and operate in a specific mode on power-up).


Add support for CLI

We need to be able to bundle and execute the transformers using command lined interface.

Could you please be more specific about this? #45 wasn't documented well enough, but by running npm run build, the tarball is created in the dist directory, which can be installed in a client code by running npm install <path to tarball>.

The main objective is to be able to run something like:
npm run model-driven-data object-model ./object-model.json respec ./output-directory/ or model-driven-data object-model ./object-model.json respec ./output-directory/. The difference does not really matter now, it should be related to the npm install as you mentioned.
Before this it will be probably necessary to process TypeScript code into JS so it can be executed using node.

Basic CLI functionality is already implemented.


Even not working optimally?

The even mode in balance is the following:
even - This is a specialized type for clusters which have topics that have a number of partitions that are a multiple of the number of brokers. The module assures that every broker has exactly the same number of partitions for each topic.

Yet the code tries to SSH to my brokers?
[ec2-user@ip-172-31-12-216 ~]$ kafka-assigner  -z $ZK_HOST -e balance --types even
[INFO] Connecting to zookeeper <IP_ADDRESS>:2181
[INFO] Getting partition list from Zookeeper
[INFO] Closing connection to zookeeper
[INFO] Getting partition sizes via SSH for <IP_ADDRESS>
[INFO] Getting partition sizes via SSH for <IP_ADDRESS>
[INFO] Getting partition sizes via SSH for <IP_ADDRESS>
[INFO] Getting partition sizes via SSH for <IP_ADDRESS>
[INFO] Starting even partition balance

I would have guessed the even module could just connect to Zookeeper to distribute partitions evenly?

As explained on #63:

It evenly balances the partitions across the brokers, and any "extra partitions" are evenly distributed within the cluster.

I believe that's what SSH'ing is for -- to figure out where to place the remainder of partitions.


How to set focus to input from some other event?

I am trying to get onKeyboardChange  to set focus on the AutoSuggest input so that the suggestions show.  Right now they will not show unless I tab back into that input.  I get error on this.state.ref.current.focus() 'TypeError: this.state.ref.current.focus is not a function'.  Is this possible?
this.state = {
value: '',              // autosuggest
suggestions: [],        // autosuggest
layoutName: 'default',  // keyboard
input: '',              // keyboard
ref: React.createRef()
}
onKeyboardChange = (input) => {
this.setState({ value: input, input: input })
console.log('Input changed', input)
if (!this.state.ref) { return }
this.state.ref.current.focus()
}

I was able to accomplish what I needed after I RTFM.  I just added the alwaysRenderSuggestions setting in <Autosuggest ...  alwaysRenderSuggestions />


[vcpkg] Use SSH keys instead of password authentication when minting Linux scale sets

This is required by a future Azure DevDiv security policy which will fail VM creations that don't disable password auth.

I would really prefer if we minted an ssh key and then threw it away, like we do for Windows passwords

Me too but Powershell has no such capability and I don't want to require someone run this in a prompt with openssh tools on the PATH.

Why not? Aren't they automatically on the path pretty much everywhere?

I don't think that's a default Windows component?

Pretty sure; it's in C:\Windows\System32 for me, so it's at least an optional component, and I don't remember turning it on.

It's listed as an optional feature but no idea what the default is. ¯\_(ツ)_/¯. I guess installing that might be less annoying than needing to generate a key in advance...

/azp run


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit cc6c517:
:clock10: Full build: the average full build time is 5 min(s) 56 sec(s), this is based on the last 677 build(s) for this repository.

Docs Build status updates of commit cc6c517:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Example code does not work.

Thank you for sharing the amazing repo to community.
When I try to use examples, this error occured. How to set up for tutorial code?
[My script]
torchrun --nproc_per_node=2 run_train.py --config-file examples/config_tiny_llama.yaml
[Error Logs]
Traceback (most recent call last):
File "/root/nanotron/run_train.py", line 156, in
trainer = DistributedTrainer(config_file)
File "/root/nanotron/src/nanotron/trainer.py", line 127, in init
self.config = get_config_from_file(
File "/root/nanotron/src/nanotron/config/config.py", line 432, in get_config_from_file
config = get_config_from_dict(
File "/root/nanotron/src/nanotron/config/config.py", line 393, in get_config_from_dict
return from_dict(
File "/usr/local/lib/python3.10/dist-packages/dacite/core.py", line 81, in from_dict
instance = data_class(**init_values)
File "", line 14, in init
File "/root/nanotron/src/nanotron/config/config.py", line 336, in post_init
assert all(
AssertionError: You must have a training stage starting at 1 in the config's data_stages

Hello. Could you try again with the current main branch? I think we've just fixed it

ah yes. we have a new fix. not merged yet https://github.com/huggingface/nanotron/pull/120

Check this commit. The assertion can't be all, as you are expecting to have 1 data stage starting at 1.

Thanks:)


Bump checkstyle version

to support JEP-378 https://openjdk.java.net/jeps/378

Fixes #14906
Context

This will unblock using textblocks from java 15+
Contributor Checklist

[ ] Review Contribution Guidelines
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[ ] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@logart Can you please make sure you do the sign off of your proposed contribution? Otherwise we cannot accept it.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@logart please amend your comits so they are signed off

8.37 has been released, will open another PR for this.
Closing this one as obsolete, replaced by #15252 .


Separate external network from Kubernetes quoin

- Abstract external VPC creation out of Kubernetes into it's own quoin
- Update docs.md

In taking another pass, I'm realizing this PR is not complete yet. Will continue to work through it. Feel free to post feedback on things I'm missing.

Close, what we want to create is an internal layout only quoin. You can use the external-internal layout as a template and then remove the external bits, like the Internet Gateway and NAT Gateways.

Hmm, but I like the approach though and see what you're trying to do. We need to think about the tag though so the ELB's that K8S create get put on the correct subnet. Which is partly why I thought it would be rip out the network piece all together and not do the layout at all inside the K8S quoin.

Oh I see now. I had misunderstood. I assumed that because Kubernetes needed an internal network layout, it would be included in the Kubernetes quoin, and we would abstract the optional external network piece out into a new quoin. However, I do like separating the network component altogether away from Kubernetes. I'll work on some changes, and update this PR.

Ok, PR updated based on feedback. More feedback welcome. Once this looks good, I will also open a PR into our infra repo, and make the appropriate changes there.

@dragan @weitenghuang I worked this afternoon trying to test this locally by creating a k8 cluster in  our integration VPC, but was running into errors. I'm sure I was doing something wrong somewhere, but I'm not sure what (I was just trying to build a k8 cluster on existing infra layers that are already there).

@nicklathe No worries, if you could, include the error next time in your comment. If you could, show me Tuesday morning. @weitenghuang, if get a chance tomorrow, try testing this PR and providing feedback. However, no worries if you can't. We'll look at it together on Tuesday. Have a great weekend!

@dragan @weitenghuang this should be ready for a merge. Wei and I tested it yesterday, and other than the fact that building a k8 cluster with internal subnets only (no NATs) in our us-west-2 account couldn't reach out to quay, it appears the changes I made work. I will test this morning to create a k8 cluster using the external-internal-layout to finish confirming this PR works.

Ok, updated docs.md. @weitenghuang, let me know if I missed something.


Stack overflow error in 1.10.2

Hi Im the author of the mudpack Project psiloncrafted.
When i use the atomic dissembler i get stack overflow OpenGL error.
any idea why

We might have an idea if there are some logs/crashlogs available

funny thing there is no error log or message other that OpenGL error stack overflow 1382...
removed foam fix and better fps and it seems to be working fine now


fixing exported struct that needed comment

As requested (correctly) in #13 add comment description for the NATSConfig structure.

@jrossi Thank you!


perp: OpenPositon and ClosePosition should check if it's possible to interact with the perp by checking Params

Fixed by https://github.com/NibiruChain/nibiru/pull/1342.


Update readme for workshop

Coverage remained the same at 86.375% when pulling 3a36f66d96b19d323aa1d316a96023e67d88d4a0 on feature/PM-481-Refactor-GnosisDB-README into cc15d65099d00c47f83e2e6df30282e13b70811e on development.


nile deploy: Error while running deploy on local testnet

Hello I am using Python Python 3.9.7 on MacOS Monterey 12.3.1, (1.4 GHz Quad-Core Intel Core i5). I have install nile and nile init. I am able to compile my cairo contracts but I struggle and cannot deploy it on local testnet when I run nile node on another CLI.
nile deploy helloworld --alias helloworld
🚀 Deploying helloworld
Error: BadRequest: HTTP error ocurred. Status: 500. Text: <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>500 Internal Server Error</title>

Internal Server Error

The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.

Traceback (most recent call last):
  File "/Users/opt/anaconda3/bin/nile", line 8, in <module>
    sys.exit(cli())
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/nile/main.py", line 44, in deploy
    deploy_command(artifact, arguments, network, alias)
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/nile/commands/deploy.py", line 29, in deploy_command
    output = subprocess.check_output(command)
  File "/Users/opt/anaconda3/lib/python3.9/subprocess.py", line 424, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/Users/opt/anaconda3/lib/python3.9/subprocess.py", line 528, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['starknet', 'deploy', '--contract', 'artifacts/helloworld.json', '--gateway_url=http://localhost:5000/']' returned non-zero exit status 1.

This is the error I get on Nile Node CLI
(env) (base) ak@Ak-MBP nile-cairo % nile node
 * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)
[2022-06-03 12:19:43,961] ERROR in app: Exception on /gateway/add_transaction [POST]
Traceback (most recent call last):
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/flask/app.py", line 1953, in full_dispatch_request
    return self.finalize_request(rv)
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/flask/app.py", line 1968, in finalize_request
    response = self.make_response(rv)
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/flask/app.py", line 2127, in make_response
    raise TypeError(
TypeError: The view function did not return a valid response. The return type must be a string, dict, tuple, Response instance, or WSGI callable, but it was a coroutine.
<IP_ADDRESS> - - [03/Jun/2022 12:19:43] "POST /gateway/add_transaction HTTP/1.1" 500 -
[2022-06-03 12:25:13,968] ERROR in app: Exception on /gateway/add_transaction [POST]
Traceback (most recent call last):
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/flask/app.py", line 1953, in full_dispatch_request
    return self.finalize_request(rv)
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/flask/app.py", line 1968, in finalize_request
    response = self.make_response(rv)
  File "/Users/opt/anaconda3/lib/python3.9/site-packages/flask/app.py", line 2127, in make_response
    raise TypeError(
TypeError: The view function did not return a valid response. The return type must be a string, dict, tuple, Response instance, or WSGI callable, but it was a coroutine.
<IP_ADDRESS> - - [03/Jun/2022 12:25:13] "POST /gateway/add_transaction HTTP/1.1" 500 -

Hi. Sorry for the long wait, this fell through the cracks. I could not reproduce this, is the problem persisting?


ResourceManagementClient credentials require azure.identity ClientSecretCredential authentication method.

Latest ResourceManagementClient class works with one of the azure.identity authentication method. e.g. ClientSecretCredential
But the DataFactoryManagementClient class still works on the ServicePrincipalCredentials.
ClientSecretCredentials is not yet an option. Its work in progress as per my investigation.
I had to create both clients with separate authentication mechanisms to make the solution work.
I did not opt to downgrade the azure.mgmt.resource package.
An indication about this issue is needed in the article for running the sample successfully.

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

ID: 06b12410-3d72-8cc9-be6c-3b8e8521b90d
Version Independent ID: 9ef477fb-8321-4763-d340-0fee977f4beb
Content: Quickstart: Create an Azure Data Factory using Python - Azure Data Factory
Content Source: articles/data-factory/quickstart-create-data-factory-python.md
Service: data-factory
GitHub Login: @dcstwh
Microsoft Alias: weetok

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

Thank you for your feedback @Sanoobk  . This has been assigned to the content owner, @dcstwh , and will be updated as appropriate.

@nabhishek  can you help look into this? Thank you.

@nabhishek  can you help look into this? Thank you.

Please see whether the docs helps - https://docs.microsoft.com/en-us/azure/data-factory/quickstart-create-data-factory-python and close this issue once it address what you need.

Thanks for the update.


How would one create a Vertex Buffer (vulkan memory management)?

Hello, I'm trying to extend the vulkan example using this tutorial and the VULK engine as a guide. It was all and good until I've hit a memcpy. I tryied to gain some insight looking at vulk/vulkanobject.py, line 787, but I can't make heads of tail of it. This is the specific block of code that contains the memcpy:
void* data;
vkMapMemory(device, vertexBufferMemory, 0, bufferInfo.size, 0, &data);
memcpy(data, vertices.data(), (size_t) bufferInfo.size);
vkUnmapMemory(device, vertexBufferMemory);
How does one manage memory with vulkan?
Thank you :)

Hello @ompadu !
Yes I faced this issue too, not so easy to memcpy with Python...
So you can find my solution here: https://github.com/realitix/vulk/blob/master/vulk/graphic/mesh.py#L160
I use numpy to store my vertices array and I can memcpy it with numpy.copyto. Like you can see, I tell numpy that the format is uint8 to do it bit by bit.
I didn't test but I think you can use cffi to do it: https://cffi.readthedocs.io/en/latest/ref.html#ffi-memmove.
Did I answer well to your question or do you want more informations ?

I didn't even think to use numpy. I'll go with that as I don't really have any direct cffi experience. Thank you :)

Good luck!


Add support for Azure Container Apps

Waiting for Terraform support of ACA

https://learn.microsoft.com/en-us/samples/azure-samples/container-apps-azapi-terraform/container-apps-azapi-terraform/

Hey @SandraAhlgrimm,
Great catch! Would you like to work on this task? If so, this is were I would start:

[ ] Test the mentioned sample
[ ] Review the existing TF modules (I don't remember your level of fluency about Terraform)

Having the proper

[ ] Remove Dapr-specific things. Dapr support is interesting, but not all ACA users are DAPR users. Let's ship a simple version first
[ ] Look at our existing Application Insights template. I think these ones are outdated. It would make sense to create a new Azure Monitor / AI TF module, and leverage it in this ACA module.

What do you think?

Yes, please assign it to me. I’d love to get through the „nubesgen add a
new feature“ process first before thinking about the refactoring you
suggested.
Christopher MANEU @.***> schrieb am Mi. 16. Nov. 2022
um 17:09:

Hey @SandraAhlgrimm https://github.com/SandraAhlgrimm,
Great catch! Would you like to work on this task? If so, this is were I
would start:

Test the mentioned sample
Review the existing TF modules (I don't remember your level of
fluency about Terraform)

Having the proper

Remove Dapr-specific things. Dapr support is interesting, but not
all ACA users are DAPR users. Let's ship a simple version first
Look at our existing Application Insights template. I think these
ones are outdated. It would make sense to create a new Azure Monitor / AI
TF module, and leverage it in this ACA module.

What do you think?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/NubesGen/issues/322#issuecomment-1317266540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHZDW5OZDGQCSN627SYI23WIUBMFANCNFSM5YCFKPRQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yes @SandraAhlgrimm this is clearly the best approach! And ACA support would help a ton of people.

HI @cmaneu, let's wait until ACA is in the official azurerm terraform project https://github.com/hashicorp/terraform-provider-azurerm/pull/18008
It should be there soon, and it will ease adding this feature. Otherwise, we'll have to update it anyhow as soon as it gets released.


TypeError: fit() got an unexpected keyword argument 'epochs'

Please go to TF Forum for help and support:
https://discuss.tensorflow.org/tag/keras
If you open a GitHub issue, here is our policy:
It must be a bug, a feature request, or a significant problem with the documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
Here's why we have that policy:.
Keras developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.
System information.

Have I written custom code (as opposed to using a stock example script provided in Keras):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the problem.
I am getting ethe below error from model.fit() function.
TypeError: fit() got an unexpected keyword argument 'epochs'
---> 10 history = model_xgb.fit(X_for_training, y_train, epochs=25, batch_size=32, verbose=2,callbacks=[early_stopping])
File ~/miniconda3/lib/python3.8/site-packages/xgboost/core.py:620, in require_keyword_args..throw_if..inner_f(*args, **kwargs)
618 for k, arg in zip(sig.parameters, args):
619     kwargs[k] = arg
--> 620 return func(**kwargs)
TypeError: fit() got an unexpected keyword argument 'epochs'
Describe the current behavior.
I tried nb_epoch as well. same error.
tensorflow version is
TensorFlow: 2.4.1
Keras:      2.4.0
Python:     3.8.13
Xgboost:    1.7.5
numpy ver:  1.22.4
seaborn:    0.12.2
system   : Linux
Describe the expected behavior.
Contributing.

Do you want to contribute a PR? (yes/no):
If yes, please read this page for instructions
Briefly describe your candidate solution(if contributing):

Standalone code to reproduce the issue.
import xgboost as xgb
model_xgb = xgb.XGBClassifier()
#EPOCHS = 25
early_stopping = tf.keras.callbacks.EarlyStopping(
monitor='val_accuracy',
mode='max',
patience=6
)
history = model_xgb.fit(X_for_training, y_train, epochs=25, batch_size=32, verbose=2,callbacks=[early_stopping])
Source code / logs.

TypeError                                 Traceback (most recent call last)
Input In [60], in <cell line: 11>()
3 model_xgb = xgb.XGBClassifier()
5 early_stopping = tf.keras.callbacks.EarlyStopping(
6     monitor='val_accuracy',
7     mode='max',
8     patience=6
9 )
---> 11 history = model_xgb.fit(X_for_training, y_train, epochs=25, batch_size=32, verbose=2,callbacks=[early_stopping])
File ~/miniconda3/lib/python3.8/site-packages/xgboost/core.py:620, in require_keyword_args..throw_if..inner_f(*args, **kwargs)
618 for k, arg in zip(sig.parameters, args):
619     kwargs[k] = arg
--> 620 return func(**kwargs)
TypeError: fit() got an unexpected keyword argument 'epochs'
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

@Sreejith915,
I tried to execute the mentioned code on tensorflow v2.12 and it was failing due to a different error. Kindly find the gist of it here.
And also you are trying to execute the code on tensorflow v2.4 which was pretty old. So I request to try the latest stable v2.13 from this official document.
https://www.tensorflow.org/install
It's unlikely for TF 2.4 version to receive any bug fixes except when we have security patches. There is a high possibility that this was fixed with later TF versions. Perhaps you can use latest tf versions for your case. Thank you!


Changed packages are sometimes wrongly computed

For example if I have the following packages

instrumenters/nodejs
backend
testers/scanner
scanner

If I change the testers/scanner folders it will run hooks of the scanner folder as well

Fixed in https://github.com/Escape-Technologies/mookme/pull/6


Add support for PV-battery with DC-connected battery and multiple MPPT inputs:

Add support for PV-battery with DC-connected battery and multiple MPPT inputs:
https://sam.nrel.gov/forum/forum-general/3160
Originally posted by @cpaulgilman in https://github.com/NREL/SAM/issues/215#issuecomment-693702967

From https://github.com/NREL/ssc/issues/250 about DC-Connected batteries, we have the configuration for a single-MPPT inverter:

The simplest way to support DC-connected battery with multiple MPPT inputs may be to assume charge controllers for each PV array are connected to the battery, which can then charge from both PV arrays. Each PV array's power can charge the battery or be inverted. The power inverted is the sum of the PV arrays and battery. The rectifier shown in the first diagram is not explicitly modeled currently, but the power passing from the AC bus to the BMS is netted out of the power inverted.
Thoughts and suggestions welcome.
Mermaid markdown code for diagram:
graph LR
A[PV Array]-->C[DC/DC Power Optimizer]
B[PV Array]-->D[DC/DC Power Optimizer]
C-->E[Inverter]
D-->E
C-->F[DC/DC Battery Management System]
D-->F
F-->G[Battery]
G-->F

Some additional context:
This revised diagram from help shows how SAM models BTM PV-battery DC-connected systems in a way that is more consistent with SAM's inputs than the older version of the diagram:

Feedback from a user (with diagram):

Most residential battery based DC Coupled systems have the DC/DC converter that steps down the 600V to 24-48V and the inverter is then 24-48V based. It would be great to be able to model these types of systems in the future more accurately. Do you think this could be a future possibility?

See also https://github.com/NREL/SAM/issues/215


[Web] Composing Japanese letters disappear when shifting converting area

Steps to reproduce

launch an app with TextField on web platform
type some letters in Hiragana, Japanese.
before committing (by hitting the Enter key), hit the Space key to convert them to Kanji.
press Shift and hit the left arrow key twice
composing Hiragana letters disappear

Expected results
The composing Hiragana letters don't disappear, and the composing area would shift respecting the arrow key hit.
Actual results
All the composing Hiragana letters disappear.
Code sample
Code sample
void main() => runApp(const MainApp());

class MainApp extends StatelessWidget {
  const MainApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Padding(
          padding: const EdgeInsets.all(32),
          child: Container(
            width: double.infinity,
            color: Colors.white,
            child: const Padding(
              padding: EdgeInsets.all(16),
              child: TextField(),
            ),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
https://github.com/user-attachments/assets/50f5c200-8e3d-4ff1-9457-3c08224d8be6

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
$ fvm flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.5, on macOS 14.3.1 23D60 darwin-arm64, locale ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.94.2)
[✓] VS Code (version 1.61.2)
[✓] Connected device (4 available)
[✓] Network resources

• No issues found!

@chooyan-eng
I tried the same on latest master version in which I am unable to replicate this. Can you try the same to confirm at your end as well ?

@darshankawar Thank you for checking! And unfortunately, the same problem happens on master channel.
$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 3.27.0-1.0.pre.728, on macOS 14.3.1 23D60 darwin-arm64, locale ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.95.3)
[✓] VS Code (version 1.61.2)
[✓] Connected device (4 available)
[✓] Network resources

Thanks for the update. I tried with  Japanese-Kana input keyboard which seems to have Hiragana selected by default as input mode, but I am not sure if this is right setting to replicate the issue ?
Also, does it only occur in web or on desktop as well ?

It seems that's right setting. Could you let me see the screen capture? I'll possibly find the difference between what we are doing to replicate.
I'll try it on desktop(macOS at first) and write the result later.

@darshankawar I tried the same operation on macOS app visualizing keystrokes with keycastr. I found the same issue doesn't happen on macOS, but another issue that unnecessary charactors appear when commiting the composing (the last "は" letter in this case).
https://github.com/user-attachments/assets/83088cb6-3d26-4c05-9388-ca3d965accc7

Thanks for the update. I tried again with same setup I had and was able to replicate after few tries. This only seems to be occuring in web platform.
stable : 3.24.5
master : 3.27.0-1.0.pre.728

but another issue of unnecessary characters appearing when committing the composing (the last "は" letter in this case) happened.

Please file a new issue for this.

@darshankawar I'm glad to hear you replicated the issue.

Please file a new issue for this.

I'll do. Thanks.


Persistent retained messages in k8s deployment

Are there any recommended way how to make retained messages persistent between EMQ deployments?
For me it looks like there is no use for EMQ in Kubernetes because:

by default kubernetes will mount /opt/emqttd/data/mnesia as root:root.
That lead be to rather hacky attempt to solve this by:

      securityContext:
        runAsUser: 0
        fsGroup: 1000
      containers:
      - name: emqtt
        image: private.registry/emqttd-docker:v2.3.9
        command: ["/bin/sh", "-c"]
        args:
          - chown emqtt:emqtt /opt/emqttd/data/mnesia && su -l emqtt -p -c /opt/emqttd/start.sh
        ports:

even with correct permissions for /opt/emqttd/data/mnesia EMQ will just start with new node nam e on each restart:

# ls /opt/emqttd/data/mnesia/
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
setting EMQ_NAME, EMQ_HOST or EMQ_NODE__NAME make no effect to use persistent mnesia path

Hi @miks I'm not sure why setting EMQ_NODE__NAME did not work for you.
This worked to me:
docker run --name emq --hostname local.host --rm -it -e<EMAIL_ADDRESS>--add-host local.host:<IP_ADDRESS> -v (pwd)/tmp:/opt/emqttd/data/mnesia emqttd-docker-v2.3.11
The host name has to contain a dot because so far we do not support short names.


.NET 7: Code sample for Improved unit testability for minimal route handlers

Improved unit testability for minimal route handlers
Create sample app in this folder

I can take this 😸

I can take this 😸

Fantastic.


C++ Interop: import const methods as non-mutating

This change makes ClangImporter import some C++ member functions as non-mutating, given that they satisfy two requirements:

the function itself is declared as const
the parent struct doesn't contain any mutable members

get accessors of subscript operators are now also imported as non-mutating if the C++ operator[] satisfies the requirements above.
In the future we should allow annotating a C++ method as mutating explicitly, to make it possible to preserve the correct behavior for methods that are declared const, but mutate the object anyway, for example via const_cast.
Fixes SR-12795.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test macOS

Yeah, looks like the C++ interop performance tests currently verify object instantiation, but not method calls.

Hey @egorzhdan :)
It seems like this caused a failure on tests https://ci.swift.org/job/swift-PR-macos/28502/testReport/junit/Swift(watchsimulator-i386)/ClangImporter/cxx_interop_ir_swift/ when running targeting i386 watch simulator, noted when running tests for #38464. By the logs of why is failing, my guess would be since i386 is a 32-bit this alloca %TSo7MethodsV, align 8 expects 4 bytes align 4. Is that right?

To unblock CI, I created a PR to disable the test and filed a issue:
https://github.com/apple/swift/pull/38690
https://bugs.swift.org/browse/SR-14986

Sorry for breaking the test!
@LucianoPAlmeida yes, you're right, the test should accept align 4 on a 32-bit arch.
I submitted a patch with a fix: https://github.com/apple/swift/pull/38692.

Sorry for breaking the test!
@LucianoPAlmeida yes, you're right, the test should accept align 4 on a 32-bit arch.
I submitted a patch with a fix: #38692.

Thank you for the fix :)

Also, for the future, if you're changing code in the ClangImporter, IRGen or the runtime code, it is a good idea to run full testing instead of smoke testing as there are often platform differences which aren't caught by smoke testing.

Also, for the future, if you're changing code in the ClangImporter, IRGen or the runtime, it is better to err on the side of caution and run full testing instead of smoke testing as there are often platform differences which aren't caught by smoke testing.

Yes, that's reasonable, thanks!


Fix #45839: tf.distribute.CollectiveAllReduceStrategy can't load model from checkpoint

This code has been tested on Tensorflow v2.3, but when I tested on v2.4，
a new bug, not caused by the code I submitted,  was encountered.

Please change worker_context.should_checkpoint instead of duplicating multi_worker_util.is_chief. Evaluator should not be considered as chief, but it makes sense to enable checkpoint loading on evaluator.

@crccw
Thank you for your advisement, I have modified it according to your suggestion. Please check it again and point out the
deficiencies.
Thanks a lot.

It has been 15 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

@UrmsOne  Any update on this PR? Please. Thanks!

@gbaned Sorry for my delayed reply. As @crccw said, there is no problem if use NFS, but my datasets are all above 1TB, using NFS will brings additional storage overhead. So I suggest that a switch should be reserved, when using local storage, the checkpoint file can be written by themselves. This also needs to consider the issue of backward compatibility.

I'm not sure the reason that the size of the dataset matters. You only need to put your checkpoint files on NFS.

@UrmsOne  Can you please check @crccw's comments and keep us posted ? Thanks!

It has been 14 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

@gbaned

@UrmsOne Can you please check @crccw's comments and keep us posted ? Thanks!

Yes, I can.  But I want to use local storage instead of NFS for distributed training. Can I implement a custom CollectiveAllReduceStrategy to support local storage for distributed training and achieve the backward compatibility of CollectiveAllReduceStrategy? And then  PR again?

@crccw  Can you please assist on above comments from @UrmsOne. Thanks!

@crccw Any update on this PR? Please. Thanks!

@crccw Any update on this PR? Please. Thanks!

@crccw Any update on this PR? Please. Thanks!

@crccw Any update on this PR? Please. Thanks!

@crccw Any update on this PR? Please. Thanks!

@crccw Any update on this PR? Please. Thanks!

@crccw Any update on this PR? Please. Thanks!

@crccw Any update on this PR? Please. Thank you!

@crccw Any update on this PR? Please. Thank you!

@crccw Any update on this PR? Please. Thank you!

@crccw Any update on this PR? Please. Thank you!

@crccw Any update on this PR? Please. Thank you!

Sorry for the late response.
@crccw no longer works on tf.distribute, and this PR has fall off my radar.
Unfortunately I lean towards not merging this PR. The change, while adds support for checkpointing to multiple locally mounted filesystems, will greatly amply the amount of writes(which tends to be expensive) to the more frequently used single-shared-filesystem use case.
@UrmsOne for your use case, I suggest applying a monkey patch to the strategy as a work around, such that for your particular use case you can still get the symmetric checkpoint to every locally mounted file systems.
I wonder if we can generalize the monkey path as an optional flag that is disabled by default. A PR for that would be fine, because it won't affect single-shared-filesystem use cases.

Hi @UrmsOne Can you please check @rainwoodman's comments and keep us posted ? Thank you!


Error running example provided

I get the following error:
Unhandled rejection Error: Not Found
    at respond (/home/r0hit/Code/elasticsearch-csv-import/node_modules/elasticsearch-csv/node_modules/elasticsearch/src/lib/transport.js:232:15)
    at checkRespForFailure (/home/r0hit/Code/elasticsearch-csv-import/node_modules/elasticsearch-csv/node_modules/elasticsearch/src/lib/transport.js:200:7)
    at HttpConnector.<anonymous> (/home/r0hit/Code/elasticsearch-csv-import/node_modules/elasticsearch-csv/node_modules/elasticsearch/src/lib/connectors/http.js:155:7)
    at IncomingMessage.wrapper (/home/r0hit/Code/elasticsearch-csv-import/node_modules/elasticsearch-csv/node_modules/lodash/index.js:3095:19)
    at IncomingMessage.emit (events.js:129:20)
    at _stream_readable.js:908:16
    at process._tickCallback (node.js:355:11)

I have exactly the sample code setup, only changes being index, type, host and filePath. I have tried with a few different csv files in different formats, all exactly producing the same error. The files are sized between 1.3-1.8 MB.

Hi there @r0hitsharma,
Sorry for not getting to you sooner. Could you please provide me with the code/options you are passing, so that I can see what is going on?

Sorry closing this one. If you still have a problem, create a new issue.


Why not follow up update?

Now, go version 1.8
LiteIDE 31.1

Thanks for the reminder :-) https://github.com/josephspurrier/golang-portable-windows/releases/tag/1.8-r.1


Research possibility of namespaced jQuery/Bootstrap dependencies

Part of https://github.com/mapzen/styleguide/issues/94, we need to know whether Bootstrap can be asked to use jQuery from someplace other than window.$.

Kinda.
jQuery doesn't care where it is. So it's not it's fault.
Bootstrap does, expecting jQuery to be located at window.jQuery actually. So we still have to make that happen.
Here's a proposal:

In the bundle, load jQuery (let's call it jQuery B) and expose it globally.
if jQuery B detects that another version of jQuery is loaded (let's call it jQuery A), then jQuery A is stashed somewhere, and jQuery B becomes the canonical $.
Since we explicitly asked for jQuery B, this is the one that we load Bootstrap onto.

Next...
OPTION A.
After loading all the styleguide stuff, we attach jQuery B to a namespaced object MPZN. We then call $.noConflict(). This restores jQuery A from stash. jQuery A is now canonical $, and jQuery B, with Bootstrap attached, can be accessed at MPZN.$. By the time the Mapzen Common JS has finished loading, its jQuery (B) has come and gone, and the developer only sees jQuery A, unless they need to go around looking for jQuery B at MPZN.$.
OPTION B.
Don't do anything else. jQuery A is still stashed, but it is now up to the developer to restore it with $.noConflict(). This means the developer has the flexibility of writing additional code that depends on jQuery B, restoring it to jQuery A for other scripts on their own terms. However, the tradeoff is that unless the developer already knows that Mapzen Common JS is loading its own jQuery and overwriting $, then the version of $ seems to come from a magic place, and the developer must know to call noConflict on their own.

Another idea.
Something like this where the Bootstrap plugins are re-written in vanilla Javascript. Since none of the other parts of Mapzen Common JS requires jQuery (only Bootstrap does) this removes Bootstrap's jQuery dependency entirely, and allows the developer to utilize their own jQuery (or not) at their discretion with no other side effects.
Tradeoffs: this is a third-party library which means support and release schedule may not be as reliable as stock Bootstrap.

Huh, that Bootstrap Native concept is super interesting.
How sensitive to load order are the options above? In other words, do they work equally well when some other jQuery is loaded before or after?

Under Option A, both of these are equivalent:
  <script src='https://code.jquery.com/jquery-1.12.0.min.js'></script>
  <script src='scripts/mapzen-styleguide.min.js'></script>

  <script src='scripts/mapzen-styleguide.min.js'></script>
  <script src='https://code.jquery.com/jquery-1.12.0.min.js'></script>

The effect is that Bootstrap uses its own jQuery but the developer sees their own version of jQuery.
Under Option B, load order does matter. This means jQuery B is canonical unless jQuery A is manually restored:
  <script src='https://code.jquery.com/jquery-1.12.0.min.js'></script>
  <script src='scripts/mapzen-styleguide.min.js'></script>

This means jQuery A overwrites jQuery B:
  <script src='scripts/mapzen-styleguide.min.js'></script>
  <script src='https://code.jquery.com/jquery-1.12.0.min.js'></script>

The other observation I have about Bootstrap Native is that it's pretty recent, and the current 3.3.x branch is pretty stable. We could probably drop it in now and not have to worry too much about breaking changes for a whole year. By the time we want to do something like adopt Bootstrap 4, we will need to be revisiting the entire styleguide anyway.

So with Bootstrap Native, we’d skip using jQuery entirely? People could still include it, but it’d be 100% separate? That’s really attractive, if you think that Native is stable enough to use for a year.

Option A sounds vastly preferable to B by the way, since it’s independent of load order and leaves the developer with fewer choices.

The solution we decided on is to import jquery in noconflct mode which doesn't add it to the global namespace and using browserify-shim to bind bootstrap to our internal jquery. We're exporting the bootstrap-bound jquery at MPZN.$ for internal calling of bootstrap functions.
The pros of this seem to be:

Simplicity. If somebody wants access to $, then they import jquery. We don't have any surprising import behavior.
Frees us from providing jquery for the future. If we decide to remove our dependency on jquery in the future through something like bootstrap native, then we can without any fuss.

Cons:

Sometimes jquery will be double imported.
Calling Bootstrap javascript is uglier. (This may not really matter if styleguide users aren't exposed to it.)

If we start creating javascript heavy components, we'll have a better idea of what sort of api we want to expose and we may need to revisit this issue.

:+1: to this solution. The pros are great, and I like how it insulates the future from the ugliness of the present.


feat(schemas): Copy existing schemas to separate repo

Is this moving more json out of the core sdk code and into the tooling repo?

Ref https://github.com/rdkcentral/firebolt-openrpc/pull/18

There's two other PRs combined with this one that address at least a few of your concerns:
https://github.com/rdkcentral/firebolt-schemas/pull/2
https://github.com/rdkcentral/firebolt-openrpc/pull/18/files
https://github.com/rdkcentral/firebolt-core-sdk/pull/11
They move the following:

shared schemas from rdkcentral/firebolt-openrpc into rdkcentral/firebolt-schemas
core schemas from rdkcentral/firebolt-core-sdk into rdkcentral/firebolt-schemas so other SDKs may use them
add rdkcentral/firebolt-schemas as a dependency for firebolt-core-sdk so it can use them

I agree about the JS template code, but we can solve that in a separate PR.


Peer dependency Sinon is not provided

Hi Guys,
I am facing a warning issue as below:
markdown-spellcheck@npm:1.3.1 doesn't provide sinon (p2d3db), requested by sinon-as-promised.
It cant be fixed even after adding "sinon": "1 || 2" in the peerDependencies section. Could someone please advice and provide a fix. Thanks alot.

Read the docs please, the peer dependency is correct as is


Remove profiler

Bug
Expected: Profiler is not present
Actual: Is present
Fix
Remove profiler
Testing
Try to launch profiler. Verify it is not present
cc @billti @paulvanbrenk @mjbvz

Hi @RyanCavanaugh, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Ryan Cavanaugh). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

LGTM

It's gone


Problems with org.eclipse.xtend.lib.annotations.Accessors xtend-gen Java Classes

I added sprotty as a submodule to my personal project to play around a bit and I noticed some odd behavior with the @Accessors annotations, especially in the io.typefox.sprotty.server.xtext.LanguageAwareDiagramServer class.
It seems that the getter and setter for languageServerExtension are not generated in the Java class.
In LanguageAwareDiagramServer.xtend:
@Accessors DiagramLanguageServerExtension languageServerExtension
In the generated LanguageAwareDiagramServer.java are no getter/setter present:
/* @Accessors */private DiagramLanguageServerExtension languageServerExtension;
I get the following Error in the xtend class (I'm using Intellij):
Error:(27, 2) Error during annotation processing:
java.lang.NoSuchMethodError: org.eclipse.xtext.xbase.lib.CollectionLiterals.newArrayList()Ljava/util/ArrayList;
	at org.eclipse.xtend.lib.annotations.FinalFieldsConstructorProcessor$Util.getFinalFieldsConstructorArgumentTypes(FinalFieldsConstructorProcessor.java:99)
	at org.eclipse.xtend.lib.annotations.FinalFieldsConstructorProcessor$Util.hasFinalFieldsConstructor(FinalFieldsConstructorProcessor.java:77)
	at org.eclipse.xtend.lib.annotations.FinalFieldsConstructorProcessor$Util.needsFinalFieldConstructor(FinalFieldsConstructorProcessor.java:71)
	at org.eclipse.xtend.lib.annotations.DataProcessor.doTransform(DataProcessor.java:82)
	at org.eclipse.xtend.lib.macro.AbstractClassProcessor.doTransform(AbstractClassProcessor.java:42)

However gradle build finishes successfully and I can run the project without problems with gradle jettyRun. But If I want to run a class from inside my project, which depends on the xtend-diagram module, It won't compile.

Xtend support in IDEA is outdated and deprecated. It is likely you run into such problems.
If you want/have to use IDEA, I'd recommend to use Java and consume the sprotty/server projects from sonatype snapshots. See e.g. the yang-lsp project for an example:
https://github.com/theia-ide/yang-lsp/blob/master/yang-lsp/build.gradle (repo config)
https://github.com/theia-ide/yang-lsp/blob/master/yang-lsp/io.typefox.yang.diagram/build.gradle (dependencies)

Thanks for your quick response. I'll give it a try.
Closing this.

While my initial command still holds, this looks like the org.eclipse.xtext.xbase.lib is missing on the classpath. Or wrong version as there have been some changes to this part of the runtime lib in Xtext 2.13.

see https://www.eclipse.org/forums/index.php/t/1088834/


Memory leak in grid creation

I've only started using ipyaggrid but I'm exited about using it in standalone HTML reports.
However, I've found that creating grid objects seem to have a memory leak.  Every time I create a grid into the same variable name, the memory usage increases. Even if I delete the variable and do a garbage collection, it continues to climb. My use-case is a follows:

Run a query from a database into a pandas dataframe
Pass the dataframe to a function in a .py file (griddler, shown below) that returns a grid from the string representation of the dataframe with a list of columns to pin.  I.e.: Not much too it.

I'm using:
ipyaggrid                 0.5.1
ipykernel                 6.25.2
ipython                   8.14.0
ipywidgets                8.0.3
I'm having to cache my database query results to tinker around with ipyaggrid because I have to restart the kernel so often to get the RAM back down.
I'm open to workarounds if anyone can help with that.
def griddler(df, pin_list=[], height=400):

    col_defs = ipyaggrid.Util.build_colDefs_for_si_cols(df)

    for d in col_defs:
        if "field" in d.keys():
            if d['field'] in pin_list:
                d['pinned'] = 'left'

    grid_options = {
        'columnDefs' : col_defs,
        'enableSorting': True,
        'enableFilter': True,
        'enableColResize': True,
        'enableRangeSelection': True,
    }

    df_str = pd.DataFrame()

    for c in df.columns:
        df_str[c] = df[c].astype("string")

    g = None

    g = ipyaggrid.Grid(grid_data=df_str,
            height=400,
            grid_options=grid_options,
            quick_filter=True,
            sync_on_edit=False,
            show_toggle_edit=True,
            export_csv=True,
            theme='ag-theme-blue',
            show_toggle_delete=False,
            columns_fit='auto',
            index=False,
            keep_multiindex=False,
            sync_grid=False)

    return g

Just to help paint the picture on this:


CodersClan support button does not lead to support forum

The "getSupport()" button in the README links to http://codersclan.net/forum/index.php?repo_id=8, but this redirects to https://www.codersclan.net/?repo_id=8 (the CodersClan homepage).
This is confusing for people looking for a support discussion group who instead find a website where they are expected to pay for coding "tasks".
I can submit a pull request either removing the button or updating the language in the README to make things more clear, but I am not certain what you would prefer.

@martey If you don't, I may submit a PR for this...

I didn't submit a pull request because I was hoping for clarification from @omab whether removing the button was okay, or whether he would prefer to have the README's language updated.

The button was removed since it wasn't used anymore.


HADOOP-18516: [Backport to 3.4] [ABFS][Authentication] Support Fixed SAS Token for ABFS Authentication

Description of PR
Jira: https://issues.apache.org/jira/browse/HADOOP-18516
PR on trunk: https://github.com/apache/hadoop/pull/6552
Commit cherry-picked: https://github.com/apache/hadoop/commit/d8b485a51229392874eb54f86bc3fdc61ce6084e
Original PR Description:
This PR introduces a new configuration for Fixed SAS Tokens: "fs.azure.sas.fixed.token"
Using this new configuration, users can configure a fixed SAS Token in the account settings files itself. Ideally, this should be used with SAS Tokens that are scoped at a container or account level (Service or Account SAS), which can be considered to be a constant for one account or container, over multiple operations.
The other method of using a SAS Token remains valid as well, where a user provides a custom implementation of the SASTokenProvider interface, using which a SAS Token are obtained.
When an Account SAS Token is configured as the fixed SAS Token, and it is used, it is ensured that operations are within the scope of the SAS Token.
The code checks for whether the fixed token and the token provider class implementation are configured. In the case of both being set, preference is given to the custom SASTokenProvider implementation. It must be noted that if such an implementation provides a SAS Token which has a lower scope than Account SAS, some filesystem and service level operations might be out of scope and may not succeed.
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
18m 34s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
markdownlint
0m  0s

markdownlint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 7 new or modified test files.

_ branch-3.4 Compile Tests _

+1 :green_heart:
mvninstall
49m 25s

branch-3.4 passed

+1 :green_heart:
compile
0m 38s

branch-3.4 passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
compile
0m 34s

branch-3.4 passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
checkstyle
0m 30s

branch-3.4 passed

+1 :green_heart:
mvnsite
0m 39s

branch-3.4 passed

+1 :green_heart:
javadoc
0m 37s

branch-3.4 passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javadoc
0m 32s

branch-3.4 passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
spotbugs
1m  5s

branch-3.4 passed

+1 :green_heart:
shadedclient
38m 29s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 28s

the patch passed

+1 :green_heart:
compile
0m 30s

the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javac
0m 30s

the patch passed

+1 :green_heart:
compile
0m 26s

the patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
javac
0m 26s

the patch passed

+1 :green_heart:
blanks
0m  1s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 19s

the patch passed

+1 :green_heart:
mvnsite
0m 29s

the patch passed

+1 :green_heart:
javadoc
0m 25s

the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javadoc
0m 24s

the patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
spotbugs
1m  3s

the patch passed

+1 :green_heart:
shadedclient
38m 28s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
2m 13s

hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

161m 20s

Subsystem
Report/Notes

Docker
ClientAPI=1.44 ServerAPI=1.44 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6855/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6855

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets markdownlint

uname
Linux 161c5677ec56 5.15.0-94-generic #104-Ubuntu SMP Tue Jan 9 15:25:40 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.4 / b5b89926eebeb9c442e4f37e7d7b8977c6635e98

Default Java
Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6855/1/testReport/

Max. process+thread count
529 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6855/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@steveloughran @mukund-thakur
This should be good to merge.
Thanks


Add media session API support

Iterating on this here so I can use Netlify to get an easy public build that I can access from my Android phone.

@durasj I've indicated that __onTrackDidCange this as private by giving it a __ prefix for now. Maybe you could give it a try in webamp desktop and see how it works? If it works well for you like this, let me know and I can promote it to a real public API, with docs and types.
In the mean time, would you be interested in adding types/docs for the other public methods that this diff added? If not, no worries. Just let me know and I'll add it to my todo list.


[Bug] OpenAI proxy usage support.

Describe the bug
I set up LiteLLM as a caching proxy for OAI calls.
open ai open_api_base support
To Reproduce
Set OPENAI_API_BASE env value to custom proxy address
Create GptConversation object, call set_api_key
Function call fails on client.models.list() call which accesses default OpenAI URL
Stack trace
401 auth error due to usage of default OpenAI base URL, as a result GptConversation .chat attribute is not initialized
Expected behavior
add api_base hook
Additional context
Minor change required:
def set_api_key(self, api_key: str, user: str, api_base: str = None) -> bool:
client = openai.OpenAI(
api_key=api_key,
api_base=api_base,
)

Or rather, not to touch base class declaration, add to init along with model:
https://github.com/biocypher/biochatter/pull/203


Upgrade docs need to talk about application.yml

The 2.X => 3.0 upgrade wiki doesn't explain that you need to copy config/application.yml.example to config/application.yml and edit it to include your gitlab URLs.

Thanks for the issue report but we only support issues for the latest stable version of GitLab. I'm closing this issue but if you still experience this problem in the latest stable version, please open a new issue (but also reference the old issue(s)). Make sure to also include the necessary debugging information conforming to the issue tracker guidelines found in our contributing guidelines.


Webilder

I get this message that says Error occured while downloading images Unable to download webshots_d3287304.jpg what is the problem and how can I fix it. thanks

It appears like this problem can not be reproduced. Closing for now.

i encountered the same problem... i will open another issue.

discovered now that only premium users can download pictures from webshots :(

Okay so we all know what that means. oh well
--- On Fri, 12/9/11, purpleleaf<EMAIL_ADDRESS>wrote:
From: purpleleaf<EMAIL_ADDRESS>Subject: Re: [webilder] Webilder (#24)
To: "tleeonly"<EMAIL_ADDRESS>Date: Friday, December 9, 2011, 9:36 AM
discovered now that only premium users can download pictures from webshots :(

Reply to this email directly or view it on GitHub:
https://github.com/thesamet/webilder/issues/24#issuecomment-3080768


CephFS: Add clusterName to snapshot metadata and honor --setmetadata option

Describe what this PR does

Add clusterName as metadata on snapshots

Example:
sh-4.4$ ceph fs subvolume snapshot metadata ls myfs csi-vol-ba248f9e-0e75-11ed-b774-8e97192ff5ec \
                        csi-snap-ce24e3bb-0e75-11ed-b774-8e97192ff5ec --group_name csi
{
    "csi.ceph.com/cluster/name": "K8s-cluster-1",
    "csi.storage.k8s.io/volumesnapshot/name": "cephfs-pvc-snapshot",
    "csi.storage.k8s.io/volumesnapshot/namespace": "rook-ceph",
    "csi.storage.k8s.io/volumesnapshotcontent/name": "snapcontent-2e89e1b2-e6e9-48fe-b365-edb493d7022e"
}

honor --setmetadata option at CephFS metadata code
--setmetadata is false by default, honoring it will keep the metadata disabled by default

Signed-off-by: Prasanna Kumar Kalever<EMAIL_ADDRESS>

Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

/retest ci/centos/<job-name>: retest the <job-name> after unrelated
failure (please report the failure too!)
/retest all: run this in case the CentOS CI failed to start/report any test
progress or results

/retest ci/centos/k8s-e2e-external-storage/1.22

/retest ci/centos/k8s-e2e-external-storage/1.23

/retest ci/centos/mini-e2e/k8s-1.22

/retest multi-arch-build

/retest ci/centos/mini-e2e-helm/k8s-1.22

/retest ci/centos/mini-e2e/k8s-1.22

@nixpanic addressed the review comments, PTAL. Thanks!

@Mergifyio rebase

@Mergifyio rebase

/retest ci/centos/mini-e2e/k8s-1.23

/retest ci/centos/k8s-e2e-external-storage/1.22


Adding Serverless Stack

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you added your SVG to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[x] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
no headquarter addresses for sst-c030 at https://www.crunchbase.com/organization/sst-c030
Empty twitter for Plural (member): https://twitter.com/plural_sh
No cached entry, and Dave (supporter) has issues with twitter: https://twitter.com/davesavesyou, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Empty twitter for Spotinst Functions: https://twitter.com/spot_hq
Empty twitter for OpenGitOps: https://twitter.com/opengitops
Serverless Stack either has no crunchbase entry or it is invalid

Build failed because of:
no headquarter addresses for sst-c030 at https://www.crunchbase.com/organization/sst-c030
Empty twitter for Spotinst Functions: https://twitter.com/spot_hq
No cached entry, and Dave (supporter) has issues with twitter: https://twitter.com/davesavesyou, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Empty twitter for OpenGitOps: https://twitter.com/opengitops
Empty twitter for Plural (member): https://twitter.com/plural_sh
Serverless Stack either has no crunchbase entry or it is invalid

LGTM @caniszczyk

Thank you!


CameraOverhaul crashes on startup.

Mod Name
Camera Overhaul
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/cameraoverhaul
Describe the bug
Crashes on startup.
Steps to reproduce

Attempt to run Camera Overhaul
Crash

Logs
https://gist.github.com/PlayfulPiano/88c5df8df545ecee616fa8bc9ee2e482
Additional context
I do have an override file in use, which I believe should make the mod work on forge.
override below:
fabric_loader_dependencies.json

Attempt to run Camera Overhaul

You listed camera overhaul as the only mod that needs to be run, but your log shows over 400 mods installed. Does this crash also occur with only Camera Overhaul installed?

Can confirm it does crash on world creation with a minimized mod list:
crash-2023-12-02_14.58.44-client.txt

Can confirm it still crashes as of beta 29.

Can confirm that v30 fixed this issue and the mod works fine with no issues!

Can confirm that v30 fixed the crash and the mod works fine with no issues!

Hey I have this exact problem, what do you mean by v30? I've downloaded the mod for 1.21 and it doesnt seem to work, how'd you fix it?

Can confirm that v30 fixed the crash and the mod works fine with no issues!

Hey I have this exact problem, what do you mean by v30? I've downloaded the mod for 1.21 and it doesnt seem to work, how'd you fix it?

note the date this issue was made. this was referring to v30 of the mod back when it was for 1.20.1


Database modeling

In this issue we will model the database and comment on what the program's fields/tables will be.


use @Parameter(alias) to make deployment slot configuration more friendly

Codecov Report

Merging #258 into develop will decrease coverage by 0.06%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop     #258      +/-   ##
===========================================
- Coverage    55.84%   55.77%   -0.07%
===========================================
  Files           53       53
  Lines         1669     1671       +2
  Branches       188      188
===========================================
  Hits           932      932
- Misses         674      676       +2
  Partials        63       63

Impacted Files
Coverage Δ

...a/com/microsoft/azure/maven/webapp/DeployMojo.java
0% <0%> (ø)
:arrow_up:

...e/maven/webapp/handlers/DeploymentSlotHandler.java
0% <0%> (ø)
:arrow_up:

...crosoft/azure/maven/webapp/AbstractWebAppMojo.java
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Getting incorrect page orientation value

Bug description
I am working on OCR scenarios where documents can be rotated. I have set detect_orientation=True to observe the page orientation value. I am noticing inconsistent result for page orientation. I have a couple of documents where one is rotated 90 degree left and other is rotated 90 degree to right.
For both the documents I am getting orientation value {'value': 90, 'confidence': None}. How can I get the correct value of rotation which I can use to rotate the page to the right orientation. My goal is to reconstruct or rotate the page to it's right orientation so that I can display the pdf properly.
Here are the sample documents:

Code snippet to reproduce the bug
from doctr.models import ocr_predictor
from doctr.io import DocumentFile

pdf_path = "some path"
docs = DocumentFile.from_pdf(pdf_path)

# load model
predictor = ocr_predictor(
    det_arch="db_mobilenet_v3_large",
    reco_arch="parseq",
    pretrained=True,
    preserve_aspect_ratio=False,
    symmetric_pad=False,
    )

predictor.det_predictor.model.postprocessor.bin_thresh = 0.35
predictor.det_predictor.model.postprocessor.box_thresh = 0.3

result = predictor(docs)
ocr_json = result.export()

for page in ocr_json.get("pages", []):
      page_data = {
          "page_idx": page.get("page_idx"),
          "dimensions": page.get("dimensions"),
          "orientation": page.get("orientation"),
          "lines": [],
      }
      print(f"Page orientation: {page.get('orientation')}")

Error traceback
The page orientation value is incorrect.
Environment
Python 3.11.7
Deep Learning backend
Torch

Hi @vikasr111 :wave:,
You can easily fine tune the orientation models to your needs :)
See: https://mindee.github.io/doctr/using_doctr/custom_models_training.html#custom-orientation-classification-models
And https://github.com/mindee/doctr/tree/main/references/classification#usage-orientation-classification
Only images are required for training no annotations / all images for training shouldn't be rotated (+-10 degree would be fine) - labels are created by auto augmentation
Hope this helps :)
Best,
Felix

Thanks @felixdittrich92.
I handled this issue using a custom orientation fix using opencv. Would love to share the solution.

Hey @vikasr111 :wave:,
Sure would be great to share a snippet here where we can discuss if it makes sense to integrate in any way into docTR :hugs:

Hello @vikasr111,
Can you share with us how you fixed it pls?
Thank you,

@felixdittrich92 @aymen20002005 Earlier I had written a whole module which uses pdf2image to extract page image and run DocTR to find the orientation of the image. Later I realized that it's not the problem of DocTR but OnnxTR which I was using earlier.
I want to report a strange case where for the same image DocTR gives correct orientation of 0 as the image is vertical, while OnnxTR gives 180degree orientation.
Here's the image:

Here's the code snippet:
import os
import json
from doctr.io import DocumentFile
from doctr.models import ocr_predictor
# from onnxtr.io import DocumentFile
# from onnxtr.models import ocr_predictor

os.environ['USE_TORCH'] = '1'

img_path = "extracted_image/abc6_page_1.png"

predictor = ocr_predictor(
    det_arch="db_resnet50",
    reco_arch="crnn_mobilenet_v3_large",
    resolve_blocks=True,
    resolve_lines=True,
    paragraph_break=0.035,
    assume_straight_pages=False,
    detect_orientation=True,
)

doc = DocumentFile.from_images(img_path)
result = predictor(doc)
text_data = result.render()
json_data = result.export()
print(json_data)

parsed_file = f"doctr-output.txt"
with open(parsed_file, "w") as f:
    f.write(text_data)

Hey @vikasr111 :wave:,
Thanks for reporting i will have a look - OnnxTR uses the docTR models exported to Onnx while exporting it's possible that it leads to a small numerical difference - we check this already in our CI with an absolute tolerance from 0.0001 - but it's still possible that (especially the cases if the image is exactly 0, 90, 180, 270 degrees rotated) it differs (i expect it works in ~95% as the pytorch model).
I'm not sure if i can do anything about this but i will have a look :)
So do you think we can close the ticket here and move it to OnnxTR ? :)

@felixdittrich92 Yes. We can close this ticket. Thanks for all the help :)


Provision to set Custom Tags/Label for Devtron Deployment

For each Devtron Deployment, we can set custom tags / labels for the deployments where tags/label can be in a tabular form.
Reason -
This can helps in tracking the application with respect to owner / Team etc.

+1

user won't apply search filter based on app labels in app listing. this will be picked up in next release.


Does sharding make a difference?

See here https://github.com/zarr-developers/zarr-python/discussions/1338

If you can be bothered/have time


Allow multiple signing keys to be provided

Description
[Describe what this change achieves]

Category (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
This change is an enhancement which allows users to provide multiple signing keys when configuring JWT options.
Previously, users could only specify a single signing key when configuring their JWT options. As a result, if a user wanted to rotate their key, they would experience some amount of downtime since the cluster would have to reinitialize the update security configuration. This change does not stop the required bouncing, but because you can now provide multiple keys, you can provide backup(s) key(s) which means even when the cluster bounces you have the ability to send JWT requests with the backup key(s).

To specify the multiple keys, you just need to use the same method as before but separate the keys with a comma i.e. <key 1>, <key 2>, etc....
Issues Resolved
[List any issues this PR will resolve]
This PR addresses: https://github.com/opensearch-project/security/issues/4613
Is this a backport? If so, please add backport PR # and/or commits #, and remove backport-failed label from the original PR.
Do these changes introduce new permission(s) to be displayed in the static dropdown on the front-end? If so, please open a draft PR in the security dashboards plugin and link the draft PR here
No
Testing
[Please provide details of testing done: unit testing, integration testing and manual testing]
I added several new tests to the existing HTTPJWTAuthenticatorTest class.
Check List

[x] New functionality includes testing
[x] New functionality has been documented
[ ] ~New Roles/Permissions have a corresponding security dashboards plugin PR~
[ ] API changes companion pull request created
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks for creating a documentation issue as well. Do you plan to open a PR along with the issue?

@krishna-ggk Settings.getAsList is backwards compatible with Settings.get(). The same yaml configuration for jwt can be used with both Settings.get and Settings.getAsList for the signing_key portion of the config:
Example jwt config:
jwt_auth_domain:
    description: "Authenticate via Json Web Token"
    http_enabled: false
    transport_enabled: false
    order: 0
    http_authenticator:
      type: jwt
      challenge: false
      config:
        signing_key: "base64 encoded HMAC key or public RSA/ECDSA pem key"
        jwt_header: "Authorization"
        jwt_url_parameter: null
        roles_key: null
        subject_key: null

One potential issue that could arise would be trying to update the JWT authc config in a mixed cluster of 2.17 and <2.17 nodes.
If the config update hit the 2.17 node and updated the jwt config then the <2.17 nodes would not know how to parse the new config.

Thanks @cwperks for clarifying the backward compatibility aspects.

If the config update hit the 2.17 node and updated the jwt config then the <2.17 nodes would not know how to parse the new config.

Does the config update through API generate list or CSV? If so, it could potentially cause issue by generating incompatible version even if only single value is provided. Admittedly though this should be a remote scenario.
Potentially the least riskiest approach would be to introduce a pluralized attribute (signing_keys) which comes at expense of managing deprecation of older attribute. In interest of keeping things simple, I agree current approach is good enough.


Bug or something is broken for Jobs page

This page has some HTML formatting issues. To see content we have to scroll right. Would you please check this?https://kubernetes.io/docs/concepts/workloads/controllers/job/

/language en

/sig docs
/area web-development


Importing SVG / DXF doesn't take color into account

I have a dxf with two layers, one red and one blue (#FF000 and #0000FF). When I import or open it (the dxf or an svg export), everything goes in the same job. I can move one element to another job by doubleclicking it, but I can select several element (which would not be practical anyway).
Finally, even if I split it into two DXF, I can't import both of them in two different jobs, I have to close open the engraving then the cutting, and manage to move everything at the same place (which mean I need to add a bounding rectangle in each dxf).
Is there some functionnality I overlooked ? (I could change color of a set of elements with ctrl-shift-s, but could not change job with crtl-s as I saw on the wiki)

Like you want a boundary between the two events without physically making it two different jobs. Like load one, start. File, New. Load the other. Start?
Or am I missing something? Everything that is red gets classified in any red operations, anything blue stroke gets classified into blue operations. If you load multiple files this is still the case. I'm not sure what your trying to do workflow wise?

Ah, might mean dxf isn't loading the colors of the layers or something. If that's right I'd need a test dxf that fails.

Yeah, saved in Inkscape R14. Did not respect the colors in the dxf.
Will correct.

This should be corrected in the published beta. Waiting on full release.
https://github.com/meerk40t/meerk40t/releases/tag/0.6.19-beta2
Also corrected the units, which was kinda a nagging error.

Thanks ! I tested and it works for DXF. (SVG still load as bold black, but I use DXF anyway :) )


feat: set version constraint for flysystem to ^3.10.3

1. Why is this change necessary?
Currently, flysystem itself is just installed with maximum version of 3.10.4 (<3.11.0), but the adapters are at 3.15.0
2. What does this change do, exactly?
Set constraint to ^3.10.3
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

copilot:summary
copilot:walkthrough

please also adjust src/Core/composer.json


syntax/make.vim: curly brackets in a target break makeTarget and subsequent syntax highlighting

Steps to reproduce
@rohieb
Write a Makefile with a recipe such as
${HOME}/documents/example.out: ${HOME}/documents/example.in
		cat $< > $@

With the current makeTarget definition, syntax highlighting breaks from the first slash to the end of the recipe.
Expected behaviour
Adding curly brackets to makeTarget's regular expression fixes the issue, but the variable ${HOME} is highlighted as makeTarget and not as makeIdent
--- C:/Program Files/Vim/vim90/syntax/make.vim  Wed Oct 19 23:01:52 2022
+++ C:/Users/c9580/.config/vim/syntax/make.vim  Sun Oct 30 08:44:43 2022
@@ -45,11 +45,11 @@
 syn match makeImplicit         "^\.[A-Za-z0-9_./\t -]\+\s*:[^=]"me=e-2

 syn region makeTarget transparent matchgroup=makeTarget
-       \ start="^[~A-Za-z0-9_./$()%-][A-Za-z0-9_./\t $()%-]*&\?:\?:\{1,2}[^:=]"rs=e-1
+       \ start="^[~A-Za-z0-9_./$(){}%-][A-Za-z0-9_./\t ${}()%-]*&\?:\?:\{1,2}[^:=]"rs=e-1
        \ end="[^\\]$"
        \ keepend contains=makeIdent,makeSpecTarget,makeNextLine,makeComment,makeDString
        \ skipnl nextGroup=makeCommands
-syn match makeTarget<EMAIL_ADDRESS>$()%*@-]*&\?::\=\s*$"
+syn match makeTarget<EMAIL_ADDRESS>$(){}%*@-]*&\?::\=\s*$"
        \ contains=makeIdent,makeSpecTarget,makeComment
        \ skipnl nextgroup=makeCommands,makeCommandError

Version of Vim
Gvim 9.0.801
Environment
Windows 10 19044.2130
Logs and stack traces
No response

Thanks, looks good to me!

I'll include the change then.


Add UDP?

Should UDP be an option? If so, how do we resolve the issue of symmetry with the rest of the API? So far every protocol supports all four scenarios but with UDP it gets weird...
hookah -i udp://server:port
This kinda makes sense if the server already knows the address of the client to send to. Just connect and read all incoming messages.
hookah -o udp://server:port
This is easy, just send the messages to the server.
hookah -i udp-server://server:port
This is easy, just accept all incoming messages.
hookah -o udp-server://server:port
This is weird because we don't have an easy way to send the messages to all clients.
Does anyone else have any thoughts on this?

Nice package!
Just my 2c.
For the last case, -o udp-server, UDP multicast maybe?

Thanks!

UDP multicast maybe

I agree, multicast would be good to support. But it might be confusing to use the UDP-server proto for it.
https://godoc.org/net#ListenMulticastUDP
It would probably only work to have -i udp-multicast:// because the normal -o udp:// would still be the one responsible for dialing the multicast address.
With UDP protocols the client usually has to send some kind of request, which shares its address, before receiving messages from the server. Unless I'm overlooking something here.

udp://

works for -o (and kinda for -i)

udp-server://

works for -i

udp-multicast://

works for -i

UDP may need to be inconsistent in the rest of the API since it isn't symmetrical.

With UDP protocols the client usually has to send some kind of request, which shares its address, before receiving messages from the server. Unless I'm overlooking something here.

You are right, for an UDP server to reply to a client, it requires an address/port, either sent by the client via some kind of protocol, or agreed beforehand between both parties.

UDP may need to be inconsistent in the rest of the API since it isn't symmetrical.

It seems so. I don't have other examples in mind but I'm sure there are other examples of protocols for which writing and reading are not symmetrical. BTW it would be nice to be able to provide custom protocols when using hookah as a package, for example with a simple API like:
var custom Proto // implements io.ReadCloser + a way to provide options
hookah.registerInput("myproto", custom)

And be able to use it with myproto://foo=bar.
I don't have any use case for it though :-)

I completely agree with the idea of a RegisterInput / RegisterOuput API.


Allow connecting to multiple brokers[panic fixed]

Description
This PR was merged earlier with a skipped nil check causing Meshery server to crash.
This PR adds the nil check to avoid panic when no k8s context is present
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Unit test check...

Codecov Report

Merging #5513 (507065c) into master (bae09fa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5513   +/-   ##
=======================================
  Coverage   17.49%   17.49%
=======================================
  Files         107      107
  Lines        9974     9974
=======================================
  Hits         1745     1745
  Misses       7803     7803
  Partials      426      426

Flag
Coverage Δ

gointegrationtests
5.93% <ø> (ø)

integrationtests
∅ <ø> (?)

unittests
13.14% <ø> (ø)

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

Continue to review full report at Codecov.

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


[Entitas:] Add e.Destroy() and remove context.DestroyEntity()

This is the way you have to destroy entities currently:
context.DestroyEntity(entity);

It's more convenient to write
entity.Destroy();

I always try that the entity doesn't know its context and doesn't have a reference to it.
@mzaks idea is that entity.Destroy(); dispatches an event and the context subscribes to it and cleans up.

Good idea !

The new term 'Context' is starting catch onto me as well, actually im starting to prefer it to pool now when I think about the types of 'contexts' I want to use entities for. e.g. The terrain context (to handle all terrain entities/behaviour), DynamicObject Context (e.g. for NPC, Items, Players, monsters, things that exist on the terrain)... starting to make more sense.


Added support for JIT clients

Added the ability to specify a source in checks which enables JIT clients.

Eek! I'll work to fix up the tests.

Sorry about that, I forgot to move a couple of files from my test rig into the branch.

Thanks!


[NFR] Support multidate

Hi,
Would you consider adding multi-date selection support?
Thanks!

Not yet.


aer_simulator_statevector doesn't return a statevector when using execute

Environment

Qiskit Terra version:  0.22.2
Python version:  3.9.15
Operating system:  macOS 12.6.2

What is happening?
When using the aer_simulator_statevector with execute there are no counts nor state vector returned in the results. This works if instead you use
When wrapping the same backend in a QuantumInstance and using its execute method, a state vector is returned as expected.
How can we reproduce the issue?
Works:
from qiskit import execute, BasicAer, QuantumCircuit
backend = BasicAer.get_backend('statevector_simulator')
execute(QuantumCircuit(1), backend).result().get_statevector()

Doesn't Work:
from qiskit import execute, Aer, QuantumCircuit
backend = Aer.get_backend('aer_simulator_statevector')
execute(QuantumCircuit(1), backend).result().get_statevector()

Works:
from qiskit import Aer, QuantumCircuit
from qiskit.utils import QuantumInstance
backend = Aer.get_backend('aer_simulator_statevector')
QuantumInstance(backend).execute(QuantumCircuit(1)).get_statevector()

What should happen?
All examples given should work
Any suggestions?
No response

This is expected behaviour - the implicit snapshotting of the statevector was removed quite some time ago, in part because it causes more problems when people want to make mid-circuit snapshots, and expanded the memory cost of results for the more common use-case of not needing statevectors (especially for large circuits).  BasicAer and QuantumInstance are largely unmaintained, and QuantumInstance is fully scheduled for removal.
You can make the statevector available by running QuantumCircuit.save_statevector() in the place you want to make the snapshot.


chore: support node 18 && move to yeoman-generator^5

let's see if the tests pass...

All the test are passing locally, just a matter of getting it to work here...

Phew, that wasn't easy, but I think we now have a foundation to work with.
I had to update quite a few dependencies to finally support node v18 (see issue here), which required to turn the project into an ES module, as a few of the updated dependencies must be used with import - see previous investigations. I also considered https://github.com/SAP/generator-easy-ui5/pull/82 and fixed https://github.com/ui5-community/generator-ui5-project/issues/23 ➡ with yeoman-generator^5 the generator-ui-project now doesn't run its own npm install, but let's yeoman handle the autoinstall (see https://github.com/ui5-community/generator-ui5-project/pull/62/commits/f205b23e226d2122a3f6b61540d56f8234a96b77)
Very much looking forward to everyone's feedback!

With the new UI5 Tooling v3 requiring node >= 18 I think this PR has become more important. Looking forward to feedback your @vobu @petermuessig 😊

I was linked to this PR from the related question on the SAP Community, answered by @nicoschoenteich.
The described issue persists with Node v19 as well...
When following the "Create an App for SAP Build Work Zone, standard edition" Tutorial, running yo easy-ui5 project is recommended. This command currently fails with a Error: ENOENT: no such file or directory, scandir '/Users/currentuser/tutorial.products/uimodule/ui5-local.yaml'error using Node.js v19.9.0.

@petermuessig @vobu @nicoschoenteich any update on this PR, please? would appreciate if it was merged and closed to unblock us. Thanks!

additionally, please wait until #66 is in, as this has the move to ui5/cli^3


TFS side folder upper/lower case renaming not handled properly

Cloning TFS repo to Git by using git-tfs 0.12.1 release with the below command:
git tfs init "http://mycompany.com:2222" "$/path1/foo" -u "username" -p "password" foo
git tfs pull
it seems that everything was fine, until I run the below command:
git tfs verify
it reports a lot of "Name case mismatch:".
After tracing the cause of this problem, it seems that TFS side upper/lower renaming was handled correctly.
For example:
1, TFS checkin a folder "AAA"
2, TFS rename folder AAA to aaa
the Git side folder name still left to AAA.

good find.
This happens because git-tfs tries to be case-insensitive. I've seen changesets from TFS where the casing wasn't consistent across items. That said, if a folder gets renamed, git-tfs should try to get the new case right.

Thanks for the response, spraints. I try to work on the source code, but till now, it is far too complicated for me to make a patch.
Now I have manually fixed this case mismatch, and already pushed the change to my local Git server. While trying to checkin into tfs, it failed as in the tfs side renaming has already been done.
I am wondering whether there is any workaround, so git-tfs will skip some certain Git commit (such as this renaming commit)? This might be a Git usage problem, but please forgive me that I am new to either Git or git-tfs.
Best regards,
Roy

Hm. Is it just the case that changed? TFS tends to ignore case (e.g.
tf edit $/project/aaa/file and tf edit $/Project/AAA/File do the
same thing, outside tfs?), so I wouldn't have expected there to be a
problem on checkin.
Maybe I'm not following... what change did you try to make via git-tfs?
On Tue, Nov 15, 2011 at 8:48 AM, royx
<EMAIL_ADDRESS>wrote:

Thanks for the response, spraints. I try to work on the source code, but till now, it is far too complicated for me to make a patch.
Now I have manually fixed this case mismatch, and already pushed the change to my local Git server. While trying to checkin into tfs, it failed as in the tfs side renaming has already been done.
I am wondering whether there is any workaround, so git-tfs will skip some certain Git commit (such as this renaming commit)? This might be a Git usage problem, but please forgive me that I am new to either Git or git-tfs.
Best regards,
Roy

Reply to this email directly or view it on GitHub:
https://github.com/spraints/git-tfs/issues/104#issuecomment-2744735

Sorry for the confusion, spraints.
Actually I am trying to setup a git central repository for working against TFS with git-tfs bridge.
So after manually fixing the name-mismatch problem in git side, I have to synchronize back this fix commit to TFS, and this leads to problem in the first place.
Fortunately later yesterday, I found a way to make it work: "Since git-tfs checkin/shelve only checks the last commit, by rebasing the renaming commit and make sure the last commit is from git-tfs pull, git-tfs checkin/shelve will work correctly"
Regards,
Roy

Hi, spraints, please ignore my comments below, my workaround is not working, it made rebasing needed every time I do git tfs pull. I am still working on this issue.

I'm seeing this same issue - it happens from a "rename, source rename" TFS checkin - and on some files also, not just folders.
This is on win 64-bit from the latest git tfs as of Dec 1st.
(I'm also seeing a handful of "Only in TFS" messages, but I'm going to open a separate issue for that, it doesn't have to do with case).
Here's a breakdown of the issue I'm encountering:
--> Case not correct on HEAD as compared to TFS

"Name Case mismatch" for git tfs verify (file or folder)
tracing back to TFS checkin where it was renamed shows a "rename, source rename" in TFS
tracing back to sha in Git and calling log -p or diff on the sha shows no contents in patch

Would setting git config core.ignorecase false before running git tfs clone blabla... help in this scenario?

I wasn't aware of the core.ignorecase setting. Good find, and we should make git-tfs play well with it. For now, it might not do much to set core.ignorecase, depending on which parts of git use it. If it's mostly git status and the other diff tools, then git-tfs won't be affected, because it uses more plumbing than porcelain.

OK, turns out my little "trick" didn't change anything - which doesn't tell me if that's the cause yet - it's likely the command tfs clone runs with core.ignorecase true even though I changed the setting mid-execution.
Next step is making sure that tfs clone, on init, sets that config properly (or copies config from global, which it appears to ignore currently) - any quick fixes up for this yet? I guess I could hack at it...
To answer your question, it looks like the ignorecase item isn't just for git status but for various other things:
(look for ignore_case in these files)
https://github.com/msysgit/git/blob/master/read-cache.c
https://github.com/msysgit/git/blob/master/name-hash.c
https://github.com/msysgit/git/blob/master/dir.c

I would leave the branch there, and create a pull request from it.
Feel free to merge it into your master. That way, any more changes you make will be based off of it.
And, please let us know how the 12 hour operation works. If you're still happy with it, we'll get it merged into the mainline.


v4.7.0 Ship List

Warning
Temporary ship list in case of we release a v4.7.0 which is not planned atm by Bootstrap since EOL is supposed to be at the end of 2022

:rocket: Features
:bug: Fixes

CSS: set .mark and <mark> y padding to 0 (#1465)
Progress: reduce height to 20px (#1456)
Docs: consistent alignment at different breakpoints and no overlap in About > Brand (#1460)
Forms: change top margin, font size/color of .form-text (#1442)
RTL: remove .text-break from RTL Boosted CSS (#1459)
CSS: handle link hover state in dark background within bg-variant mixin (#1446)

:book: Docs

Utilities: substractive border example has 2px borders (#1458)

:toolbox: Chore

v4 is End-of-life since the beginning of 2023 so let's close this issue


i18N Translation missing

Environment
Docker
Version
0.15.7
Describe the problem
In the current version, the translations for PiHole TimerModal are missing:

That is caused by:
For the defintion of "t" you are using "modules/dns-hole-controls" as prefix (homarr/src/widgets/dnshole/TimerModal.tsx line:30),
const { t } = useTranslation('modules/dns-hole-controls');
but when accessing "t" you are using the prefix again:
title={t('modules/dns-hole-controls:durationModal.title')}
This is how it should look:

Logs
No response
Context
No response
Please tick the boxes

[X] I confirm that I attached the proper logs
[X] I've read the docs
[X] I've checked for duplicate issues
[X] I've tried to debug myself

Closed via #2188


RequestPromiseResponse json body

Describe the bug
RequestPromiseResponse throws TypeError: res.body.replace is not a function. This happens when request is made with json: true, thus the retuned value is not json string but object.
To Reproduce

Send request with json: true, i.e: { method: 'GET', json: true, url: 'your-url',  resolveWithFullResponse: true, simple: false}
* assert that response with expect(response).toSatisfyApiSpec()

Expected behavior
Handle json without throwing an error.
The problem is on line 7 in RequestPromiseResponse:
this.body = res.body.replace(/"/g, '');

thanks for this! Should be a straightforward fix. Would you like to submit a PR?
Otherwise I will see when I have time to test and fix this

@rwalle61 I might submit a PR.

thanks!

Released!


Rendering of a Page is currently dependent on the sidebar order

I want this feature to be more flexible.
For example, I don't want to show the "Charts" subtab as an option in the sidebar. I want this page to be rendered when the custom URL is triggered or when the tab_number and subtab_number match.
def RenderTabs(sidebar_list: SidebarList):

upgrade to 0.6.25