Downloading multiple attachments consecutively

Hi.
I modified the "Downloading attachments" example to loop over messages to download multiple attachments consecutively. My Gmail inbox has 50+ messages and each has one video file (3MB). It starts nicely, but after 20-30 downloads, it stops. It stops at
res = self._service.users().messages().attachments().get(userId=self.user_id, messageId=self.msg_id, id=self.id).execute()
in attachment.py. I left it running for over 30 minutes, but it's simply stuck there. I used signal.larm to force it to time out, but the immediate next attempt gets the same (non) response. If I restart the entire script, it runs fine for a while again.
Any idea how to solve it? Thanks in advance!

@mkimura1971 I believe the issue may be hidden by however you modified it to download "consecutively". I just tested the example as is with 100 messages each with a 6MB video file, no issues.
The download is not trivially parallelizable. This has to do with complications of the underlying Google API resource object that is used to make API requests. Are you calling download() or save() in parallel? If not can you post the relevant code?

Thanks for the quick response! My code is essentially this (I simplified the original, but confirmed that this simplified version gives me the same symptom):
from simplegmail import Gmail
import os
gmail = Gmail()
messages = gmail.get_messages(query='in:inbox<EMAIL_ADDRESS>has:attachment')
for message in messages:
if message.attachments:
for attm in message.attachments:
file_path = os.path.join("data", attm.filename)
print('File: ' + file_path)
attm.save(file_path)

Does it always stall on the same file?

No, it's different each time.

I'm not able to reproduce the stalling on my end no matter how many messages or attachments I download, so I'm at a bit of an impasse. If you were hitting google API quotas, it wouldn't stall. My best guess is that it may have to do with your network, since not receiving a response is the most likely explanation for it stalling. If you have access to another network (or can run this on a remote machine with a different connection) that would be one way of testing this.
Otherwise, I recommend you write the script to only process a smaller amount of messages at a time, if you are saying that it consistently doesn't stall for the first X amount of messages.

Yes, I'm going to try it from a different network and perhaps a different Google account. Thanks!


MySQL: Not respecting AUTO_INCREMENT when inserting new records

sqlfuzz expects a table to be completely clean, so if the "seed" database already contains some data it then fails:
2021/04/24 15:08:26 Error 1062: Duplicate entry '1' for key 'PRIMARY'

In my case, i can't have a completely empty table/database due to various constrains, but I do want to add e.g. 100k new records.
SHOW CREATE TABLE table spits out AUTO_INCREMENT=XX so XX should be the starting point instead of 1.

Just checked the target table because it was suspicious to me that I got 4 errors for duplicate PK when I set it to insert 10 records. For some reason, it tried to insert 4 rows with value 1, which exists, but it then did insert 6 new rows starting from the correct AUTO_INCREMENT, which was actually 22 (so I got 6 new rows 22-27). Pretty weird?


update go.mod

1. Overview

cosmos-sdk v0.45.13, tendermint informal 0.34.26

This PR can be closed?
shouldn't this be merged? and the SDK should be updated further?
cosmos-sdk already has v45.14, and v45.15rc tagged


Create a "nice" landing page for devfile project

The devfile site (devfile.io) needs a proper landing page with appealing graphics, basic info, and links to repos, docs, comm. channels, etc. The Eclipse Che site could be an inspiration: https://www.eclipse.org/che/

JIRA: https://issues.redhat.com/browse/RHDEVDOCS-2790

Dupe: https://github.com/devfile/api/issues/379v


Non-blocking retries: partition rebalance causes emergency consumer stop

In what version(s) of Spring for Apache Kafka are you seeing this issue?
2.9.4, 3.0.2.
Describe the bug
When the "seek after error" recovery mode is disabled, KafkaMessageListenerContainer stashes the records returned from the most recent poll call into a member variable:
https://github.com/spring-projects/spring-kafka/blob/addb888088fcaa8acd375309e26a00cd46e60384/spring-kafka/src/main/java/org/springframework/kafka/listener/KafkaMessageListenerContainer.java#L2929-L2932
If a partition rebalance happens before they are processed, the affected consumer instance terminates with an emergency stop error:
https://github.com/spring-projects/spring-kafka/blob/addb888088fcaa8acd375309e26a00cd46e60384/spring-kafka/src/main/java/org/springframework/kafka/listener/KafkaMessageListenerContainer.java#L1672-L1676
It seems that the only possible reason for it is that the newly assigned partitions are not properly paused.
To Reproduce
I've uploaded a minimal example into this GitHub repository. Unfortunately, it lacks the Docker Compose setup as we are not allowed to use Docker Hub at work and, therefore, have a custom Kafka container that is not compatible with the open source one. Anyone wishing to reproduce this bug should create the following Kafka topics, with 4 partitions each:

input
input-retry-0
input-dlt
output

You may also have to adjust the bootstrap servers configuration in application.yaml. Assuming that this is done, the steps to trigger the problem are:

Send like 10 messages to the input topic via e.g. for i in {1..10}; do echo "test_$i" | kcat -b localhost:9192 -t input -P; done.
Clone the above example repository.
Start one instance of the service via mvn spring-boot:run. Wait for the partitions to be assigned and for the first Processing message= to appear in the console log.
Start another instance of the service and wait for the rebalance.

The first instance then fails with the emergency stop error, sometimes for multiple topics at once:

2023-02-10T17:48:49.149Z  INFO 40483 --- [ntainer#0-0-C-1] o.s.k.l.KafkaMessageListenerContainer    : rebalance-issue-consumer: partitions revoked: [input-0, input-1, input-2, input-3]
2023-02-10T17:48:49.149Z  INFO 40483 --- [0-retry-0-0-C-1] o.s.k.l.KafkaMessageListenerContainer    : rebalance-issue-consumer: partitions revoked: [input-retry-0-0, input-retry-0-1, input-retry-0-2, input-retry-0-3]
2023-02-10T17:48:49.149Z  INFO 40483 --- [ner#0-dlt-0-C-1] o.s.k.l.KafkaMessageListenerContainer    : rebalance-issue-consumer: partitions revoked: [input-dlt-0, input-dlt-1, input-dlt-2, input-dlt-3]
2023-02-10T17:48:49.173Z  INFO 40483 --- [0-retry-0-0-C-1] o.s.k.l.KafkaMessageListenerContainer    : rebalance-issue-consumer: partitions assigned: [input-retry-0-2, input-retry-0-3]
2023-02-10T17:48:49.173Z  INFO 40483 --- [ntainer#0-0-C-1] o.s.k.l.KafkaMessageListenerContainer    : rebalance-issue-consumer: partitions assigned: [input-2, input-3]
2023-02-10T17:48:49.179Z ERROR 40483 --- [ntainer#0-0-C-1] o.s.k.l.KafkaMessageListenerContainer    : Poll returned 7 record(s) while consumer was paused after an error; emergency stop invoked to avoid message loss
<...>
2023-02-10T17:48:52.831Z ERROR 40483 --- [0-retry-0-0-C-1] o.s.k.l.KafkaMessageListenerContainer    : Poll returned 1 record(s) while consumer was paused after an error; emergency stop invoked to avoid message loss

The second service instance may fail too, effectively causing an outage as only the DLT topic has consumers attached to it after that.
Expected behavior
I expect the rebalance events to be handled by the framework in a way that does not cause partial termination of the service.

I am not sure how this could happen because we re-pause any re-assigned partitions during the rebalance. I will investigate Monday using your app.
However, it does indicate another problem - we must remove any revoked partitions from the remainingRecords and not redeliver them to the listener.
I guess the work around, for now, is to not turn off seekAfterError.

Note that using seekAfterError=false with a legacy partition assignor will have no effect because all partitions are revoked and re-assigned during a rebalance; therefore all the remaining records will be purged.
With cooperative assignment, only a subset of partitions are revoked.
I tested my fix with the CooperativeStickyAssignor and it only purges remaining records for the subset of partitions that were revoked.

Thank you for such a fast fix and the advice regarding CooperativeStickyAssignor! I'll test it as soon as the next version of the library is released.

The release is currently scheduled for next Tuesday. In the meantime you can test it using the 3.0.3-SNAPSHOT or 2.9.6-SNAPSHOT.


fix scipy version check in cupyx.scipy.fft

It seems that the current code in cupyx/scipy/fft/fft.py that checks for SciPy >= 1.5.0 will always evaluate to False.
It seems from _scipy_fft._lib._pep440 import Version will always fail because the Version function lives in  scipy._lib._pep440. Also, should we really be importing anything from a private _lib folder? (We could use an alternative like numpy.lib.NumpyVersion or distutils LooseVersion instead)
@peterbell10, can you verify if this fix looks correct to you?

This looks like it should work. Though I agree that scipy._lib is a private subpackage so CuPy shouldn't be using it. Looks like this was added by @leofang in #2998.

Good catch, thanks @grlee77. I don't remember why I used _scipy_fft._lib._pep440, now that I read it again it's clearly a typo...😢
Thanks for the alternative solutions. numpy.lib.NumpyVersion seems legit. LooseVersion does not, because it doesn't support dev versions like '1.5.0.dev0+91b4020'. Would you like to switch to that?
Just curious, though, I remember I read it somewhere that modules under scipy._lib is considered stable. Is it not the case? I couldn't locate a pointer to this statement for now, though.

scipy._lib isn't listed as public in the docs. Maybe you're thinking of numpy.lib?

Yeah I don't remember...Anyway, @peterbell10 I just recalled your earlier PR (scipy/scipy#11290). I thought it meant scipy._lib._pep440.Version is preferred over NumpyVersion? Or it doesn't matter to CuPy?

NumpyVersion doesn't implement all of PEP440 but does include the subset used by NumPy and SciPy which is what matters here. The main motivation for removing NumpyVersion from SciPy is just avoiding having two ways to do the same thing.

Thanks Peter. I have updated to just use NumpyVersion here.

pfnCI, test this please.

Backport is not needed as this flag was introduced in v8.0.0a1 (#2998)

Jenkins CI test (for commit 8466b422887b1177ccd11a4a21de53e96d135b84, target branch master) succeeded!


Try other ports when port 9000 is already used

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Problem description
When running npm start in a just-created project, I saw:

Proposed solution
Try other ports when port 9000 is already used.
Alternatives considered

Not trying out other ports: status quo, 0 development required, but 0 support for other ports different than 9000.

Additional information
Please.

Or allow us to specify a custom port.

It looks like you can modify the default port by configuration.🤔 Can you try it?
https://github.com/electron/forge/blob/5b40a8dbeeebb388f5aa1a39c1f452b8b7800f45/packages/plugin/webpack/src/Config.ts#L144-L146

Thanks! Indeed this was useful, for sure there was a way to configure it, but the sole fact of it being non-trivial made me create this issue. I suggest one of the following alternatives:

Add the port-probing mechanism, check for availability of a port from 9000 to 9999, for example, and fail if none is available
Add the line with the loggerPort: 9000, to the config generated by the npm init electron-app@latest my-app or yarn create electron-app my-app commands (or any other command that generates a project from a template). It would be redundant, for sure, since 9000 is a default value, but it would also be very convenient since changing the port would be trivial. I assume everyone will try first CTRL + F, and then 9000, and this time they'll find something, and be able to change it to make it work.

Thanks again!


TST: use assert_allclose in test_bootstrap_alternative

assert_equal is not reliable for comparing floating point numbers and fails on i386
Closes gh-20043

I'll force-push rtol=1e-14 to match some of the other tests  in test_resampling.py

thanks for your first contribution to SciPy @drew-parsons !


iOS crash after attempting to enter turnip prices.

Unfortunately this happened after I entered all my inventory 😓 but the app crashed and it would not let me back in. I had been entering my turnip prices after signing up for AC Helper+. So I deleted the app and re-downloaded it but it's not picking up that I am a subscriber anymore :-/

There was a version for a very short time with this issue on the App Store, I'm very it happen to you. Be sure to be updated to the latest version from the App Store. For restoring subscription status you can go in the settings from the dashboard and there is a restore purchase button :)

No worries! I'm thinking maybe me not updating to the latest iOS caused these issues too. You can close this. This hasn't happened again yet.


Update to 1.4 version of Rack since Rails 3.2.0.rc2 has a dep on that.

All specs continue to pass.

Thanks for the pull request, however I decided to remove the fixed version dependencies from both, Thor and Rake, so guard-jasmine takes whatever version Rails provides.

Excellent!


Module subcommand

Implemented module add <name> and module remove <name>.
Currently, they add and remove 'features' from Cargo.toml of a project.
No tests, cause not sure how to test this. Maybe try to build a project?

@White-Oak Hmm, it is too early to properly test this feature since we don't have a concrete implementation of the engine modules concept.

@ebkalderon It's a part one of "modules/shards" project, featuring "features".

@White-Oak We should probably make an RFC for this, given the discussion we were all having on the general room. Implementing this properly might be an involved process.

@ebkalderon Certainly. But this is "the part one" as I mentioned eralier, so apart from naming it and naming commands it wouldn't have much alternatives (:

@ebkalderon if we agreed on CLI task in RFC for shards, this is ready to be merged.

@White-Oak It looks like there are some conflicts with this pull request? Could you please resolve them? I think this is ready to be merged. We can keep discussing specifics on the shards RFC (amethyst/amethyst#34) and iterate on this as needed.

@ebkalderon sure! I'll do it asap

@ebkalderon Renamed modules to shards and used amethyst add <name> as a syntax.

Sorry for the delayed response. I think this pull request is essentially done, despite our inability to effectively test it. I'll merge this now!


JRTB-3: Implement Command pattern for telegrambot #3

Implement Command pattern for telegram bot.
Acceptance criteria:

/start stub command added

implements in https://github.com/mrPropper911/telegram_bot/pull/8


Hello

Hi Lamesya,
Can you say if this module can be used to create tickets and retrieve the tickets back based on email address.
I would like to create an "in-app" ticket system where customer can make a report and in the app when we respond in freshdesk, they get to see an update in the "in app - inbox"

Hi @deonthomasgy,
This is still underdeveloped and it might create some issue.
Current function it has is to view and list the tickets, other than that is in progress.

ok, we're looking to had in our app the listing of tickets, the ability to view ticket and create. I understand that you don't have the create part as yet. Can we support you in anyway to get the create feature working?


如何预处理AAPM数据？

你好，非常感谢您开源了您的相关代码。
想请问一下，下载下来的AAPM数据，只有 projection data 和 image data ，但是网络的输入应该是正弦图？想请问一下如何生成正弦图呢？

The input to the network is only the CT image, and the dataloader will process it and generate simulated sinograms.

The input to the network is only the CT image, and the dataloader will process it and generate simulated sinograms.

Thank you for your reply. But the image data format of AAPM dataset  is .ima, not dcm. I think the code only can read dcm, so should I convert ima to dcm ?

The AAPM data format, which I downloaded two years ago, is .dcm. You can also try other datasets whose format is .dcm with the code. Otherwise, you can also rewrite a new dataloader, with the correct normalization.

OK. thank you for your reply


In case of PvP, countries that already contain a marker should be grayed out. No marker can be placed on them.

Resolved with commit: disabled country when in takenGuesses


Updates the license badge

Thank you for your Pull Request!
Before you submitting, please check that:

[x] You added a brief description of the PR, e.g.:

What does it do?
What important points reviewers should know?
Is there something left for follow-up PRs?

[ ] You labeled the PR with appropriate labels if you have permissions to do so.
[ ] You mentioned a related issue if this PR related to it, e.g. Fixes #228 or Related #1337.
[ ] You asked any particular reviewers to review. If you aren't sure, start with GH suggestions.
[ ] Your PR adheres the style guide

In particular, mind the maximal line length.
There is no commented code checked in unless necessary.
Any panickers have a proof or removed.

[ ] You bumped the runtime version if there are breaking changes in the runtime.
[ ] You updated any rustdocs which may have changed
[ ] Has the PR altered the external API or interfaces used by Polkadot? Do you have the corresponding Polkadot PR ready?

After you've read this notice feel free to remove it.
Thank you!
✄ -----------------------------------------------------------------------------[clabot:check].
The current license badge is not identifiable by github

[![GitHub license](https://img.shields.io/github/license/paritytech/substrate)](LICENSE)

Updates a new one contains both GPL3 and Apache2

[![GitHub license](https://img.shields.io/badge/license-GPL3%2FApache2-blue)](LICENSE)

✄ -----------------------------------------------------------------------------[clabot:check].
Preview the modification at clearloop/substrate:badge.

[clabot:check].

[clabot:help]


Add NameQualifier and SPNameQualifier to nameID

The nameID element of the authn response can contain the NameQualifier and
SPNameQualifier attribute optionally.
pysaml2 however, which we use to implement our identity provider, depends on these attributes being present on the logout request. So can we please add this to the logout request?

Thanks for the pull request, looks good to me.  Can you please add a test for this new behavior, though?

Thanks! I'll add the test.

@ploer I've added the test, let me know if it needs some work.


CAS-673: Fix update proposal payload

The update proposal payload was incorrect and did not provide the payload field containing the updated status.
I am not sure if the fix I made is the correct fix, but it works now. Assuming it worked before, so not sure whether the back-end was updated and that broke it, or the front-end.

@jbluks backend endpoint payload was updated causing the frontend to break:
This is the PR: https://github.com/DapperCollectives/CAST/pull/656
This is the line that changed:
https://github.com/DapperCollectives/CAST/pull/656/files#diff-7778dd3d4999c108290a8d41e171d479d56b27c482e31354ef84476a6e2a2095R48

how could we have caught this sooner? Backend test code can update without the client knowing. A front end cypress test file just for testing the api payloads?

Yea I think cypress tests of all the core functionality would work because we need to hit the actual backend to find an issue like this. If we were dealing with mocks on the front-end, this would still be missed.


Is it possible to send messages from another module to OPC Twin?

Hi, I have several modules on our Edge devices (OPC Publisher, OPC Twin, ... and other custom modules).
The output of one of our modules, you have to write the value of the result in the OPC.
We think that the best option is to send a message to the OPC Twin through the edgeHub, but we did not find if this is possible.
Which is the best alternative to do this? Invoke an OPC Twin Direct Method?

Hi @felixmondelo, to send messages from one module to another, you could use routing in IoT Edge. This doc would be helpful: https://learn.microsoft.com/en-us/azure/iot-edge/module-composition?view=iotedge-1.4#declare-routes
Hope this helps.

Thank you @monikavar, I know that. My problem is if OPC Twin has any input to receive messages from other modules or not.

@monikavar then there isn't any way to communicate with OPC Twin from another module without going to the IoT Hub?

@felixmondelo OPC Twin has a direct method interface, but this is limited to the use case OPC Twin was built for (being used as the edge side component of the Inudstrial-IoT microservices).

@hansgschossmann the requirement is (all is in the Edge):

I have the OPC Module getting signals from OPC
The output of the OPC Module goes through a custom module that in some cases changes the value of a signal.
That value changed has to be notified to OPC.

My idea was to use OPC Twin to notify OPC that change inside the Edge, but I suppose for your answer that this is not possible.

@felixmondelo thanks for explaining your scenario. Are you using only the IoT Edge modules or do you use the full Industrial IoT platform (with cloud microservices)?

Only the modules :-(

Yes, this is indeed not a supported scenario for OPC Twin. You might reach out to your MSFT contact to get an update on the Industrial product roadmap, there might be another solution better fitting your needs than the Industrial-IoT platform.

Closing for now. Please reactivate if you need further assistance.

This is now implemented in 2.9.0-preview2. The OPC Twin functionality has been exposed inside OPC Publisher and other modules can use the direct method API which is documented under /docs.


[WIP] Adds examples to the HTML docs

We want to add .html versions of the tutorial notebooks to the docs, so that they're easier to access.
Doing this is actually pretty simple! You just need to make sure the .ipynb file is in the /source directory, and it will create a .html file for it automatically. If, for example, in usage.rst I just direct to:
'Star notebook <Example-Star.ipynb>`
this will link in the docs to a .html version of the notebook.
In the future we need to decide on a suite of examples and probably create a specific tutorials page to tidy things up. But for now, we should focus on getting 3 clear examples with full markdown explanations that we can link to in the Usage section.

Closing this cos @nielsenmb is on the case!


[Request]  Add HLMV (Half Life Model Viewer)

[Request]  Add HLMV (Half Life Model Viewer)

Resolved in latest update..


[Feature Request]: support javascript in CodeHierarchyNodeParser

Feature Description
currenlt for language="javascript" i get :
KeyError: 'Language not yet supported. Please contribute!'
though it's tree sitter is there.
i see that the supported languages are just
"cpp"
"html"
"python"
"typescript"
"php"
is it really the case or just the activation for the other languages is missing in _COMMENT_OPTIONS in code_hierarchy.py
Reason
No response
Value of Feature
No response

You can just set typescript
To enable javascript, you'll have to duplicate all the settings from typescript

i f i replace it with typescript i get :
  File ".../lib/python3.11/site-packages/llama_index/packs/code_hierarchy/code_hierarchy.py", line 694, in _parse_nodes     raise ValueError(f"Could not parse code with language {self.language}.") ValueError: Could not parse code with language typescript.


Font setting on Plasma 5 is broken

I am running Plasma 5 on NixOS 23.11. When I try to set the global monospace font with plasma-manager it does not work, and I get a errors logged when opening the fonts section in system-settings. I suspect the issue here may be applicable to all font settings.
This looks to be due to the fact font changes in Plasma 5/6 (Qt 5 vs 6) are not accounted for. In 6, there are 6 new font fields, and Plasma 5 does not like them existing in a font configuration string. Additionally the font weight enum is changed for Plasma 6. The enum values are completely different. See: Qt5 vs Qt6 docs.
I have not ready for PR (no cleanup of dead code) working fix in my fork. Since all the new font options are at the end of the string, I just removed the last 6 items.

This should probably go to the plasma-5 branch, but I have had some difficulty maintaining it. The font setting was only really implemented taking plasma 6 into account so it's not really that surprising that it doesn't work on plasma 5.

@luna-1024  Now that NixOS 24.05 is available with Plasma 6 are you considering to just upgrade to Plasma 6?

Yes, I upgraded to Plasma 6 immediately after the 24.05 release.
Since I already did the research on this bug, I have opened PR #200. Though it does seem like the maintainability of the Plasma 5 support is in question. I plan to not use it going forward as I have upgraded.


unable to use mixed precision

Describe the bug
Unable to use automatic mixed precision
Code to reproduce the issue
if self.config.use_amp:
     policy = mixed_precision.Policy('mixed_float16')
     mixed_precision.set_policy(policy)

Describe the current behavior
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File<EMAIL_ADDRESS>line 87, in _run_code
    exec(code, run_globals)
  File "/Users/lei/.vscode/extensions/ms-python.python-2020.11.371526539/pythonFiles/lib/python/debugpy/__main__.py", line 45, in <module>
    cli.main()
  File "/Users/lei/.vscode/extensions/ms-python.python-2020.11.371526539/pythonFiles/lib/python/debugpy/../debugpy/server/cli.py", line 430, in main
    run()
  File "/Users/lei/.vscode/extensions/ms-python.python-2020.11.371526539/pythonFiles/lib/python/debugpy/../debugpy/server/cli.py", line 267, in run_file
    runpy.run_path(options.target, run_name=compat.force_str("__main__"))
  File<EMAIL_ADDRESS>line 265, in run_path
    return _run_module_code(code, init_globals, run_name,
  File<EMAIL_ADDRESS>line 97, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File<EMAIL_ADDRESS>line 87, in _run_code
    exec(code, run_globals)
  File "/Users/lei/Documents/Projects/NLP/新华社NLP/分类/hanlp_classification.py", line 42, in <module>
    history = classifier.fit(TRANIN_SET, DEV_SET, save_dir, **kwargs)
  File "/Users/lei/Documents/Projects/NLP/HanLP/hanlp/components/classifiers/transformer_classifier.py", line 119, in fit
    return super().fit(**merge_locals_kwargs(locals(), kwargs))
  File "/Users/lei/Documents/Projects/NLP/HanLP/hanlp/common/component.py", line 341, in fit
    model, optimizer, loss, metrics = self.build(**merge_dict(self.config, logger=logger, training=True))
  File "/Users/lei/Documents/Projects/NLP/HanLP/hanlp/common/component.py", line 255, in build
    self.model = self.build_model(**merge_dict(self.config, training=kwargs.get('training', None),
  File "/Users/lei/Documents/Projects/NLP/HanLP/hanlp/components/classifiers/transformer_classifier.py", line 174, in build_model
    model, self.transform.tokenizer = build_transformer(transformer, max_length, len(self.transform.label_vocab),
  File "/Users/lei/Documents/Projects/NLP/HanLP/hanlp/layers/transformers/loader.py", line 66, in build_transformer
    output = l_bert([l_input_ids, l_token_type_ids], mask=l_mask_ids)
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 925, in __call__
    return self._functional_construction_call(inputs, args, kwargs,
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 1117, in _functional_construction_call
    outputs = call_fn(cast_inputs, *args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/tensorflow/python/autograph/impl/api.py", line 258, in wrapper
    raise e.ag_error_metadata.to_exception(e)
TypeError: in user code:

    /usr/local/lib/python3.8/site-packages/bert/model.py:79 call  *
        embedding_output = self.embeddings_layer(inputs, mask=mask, training=training)
    /usr/local/lib/python3.8/site-packages/bert/embeddings.py:226 call  *
        embedding_output += tf.reshape(pos_embeddings, broadcast_shape)
    /usr/local/lib/python3.8/site-packages/tensorflow/python/ops/math_ops.py:1141 binary_op_wrapper
        raise e
    /usr/local/lib/python3.8/site-packages/tensorflow/python/ops/math_ops.py:1125 binary_op_wrapper
        return func(x, y, name=name)
    /usr/local/lib/python3.8/site-packages/tensorflow/python/util/dispatch.py:201 wrapper
        return target(*args, **kwargs)
    /usr/local/lib/python3.8/site-packages/tensorflow/python/ops/math_ops.py:1447 _add_dispatch
        return gen_math_ops.add_v2(x, y, name=name)
    /usr/local/lib/python3.8/site-packages/tensorflow/python/ops/gen_math_ops.py:495 add_v2
        _, _, _op, _outputs = _op_def_library._apply_op_helper(
    /usr/local/lib/python3.8/site-packages/tensorflow/python/framework/op_def_library.py:503 _apply_op_helper
        raise TypeError(

    TypeError: Input 'y' of 'AddV2' Op has type float16 that does not match type float32 of argument 'x'.

Expected behavior
amp mode enabled
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS
Python version: 3.8
HanLP version: LATEST

Other info / logs

[x] I've completed this form and searched the web for solutions.

Maybe one upgrade of TF broke this feature. Could you try out the solutions listed in https://github.com/tensorflow/tensorflow/issues/34406 ?

After debugging I found it is related to the package bert-for-tf2. This package also created another issue related to custom loss function from tfa. Have you think if using huggingface instead of bert-for-tf2?

After debugging I found it is related to the package bert-for-tf2. This package also created another issue related to custom loss function from tfa. Have you think if using huggingface instead of bert-for-tf2?

Yes, I was using huggingface  for the  initial 2.0 release but its downloading speed was too slow so I switched to that one. Now they have a fast global CDN and I've been heavily using theirs in 2.1. Please wait for a couple of days, I'll release 2.1 soon.

I tried huggingface but still got the same error. It's very easy to reproduce these two errors:

use_amp
use tfa.losses.SigmoidFocalCrossEntropy()
Can you take a look?

I can't find your codes using huggingface in https://github.com/hankcs/HanLP/pull/1595/files .
I don't think using tfa is a good idea. It's kind of a community supported package and tends to be buggy. I've encountered several glitches using their CRF implementation.
huggingface team confirmed this error is resolved in https://github.com/huggingface/transformers/issues/3320#issuecomment-730073845 so please make sure you are using the latest version.

I‘ve tried to upgrade hanlp in the anaconda prompt, but it still showed that cannot import name 'albert_models_tfhub' . Please help me to solve this problem. Thx a lot! :)

Traceback (most recent call last):
File "F:\Program_Files\Anaconda3\envs\tensorflow-gpu\lib\site-packages\hanlp-2.1.0a12-py3.6.egg\hanlp\utils\component_util.py", line 74, in load_from_meta_file
obj: Component = object_from_classpath(cls)
File "F:\Program_Files\Anaconda3\envs\tensorflow-gpu\lib\site-packages\hanlp_common-0.0.4-py3.6.egg\hanlp_common\reflection.py", line 27, in object_from_classpath
classpath = str_to_type(classpath)
File "F:\Program_Files\Anaconda3\envs\tensorflow-gpu\lib\site-packages\hanlp_common-0.0.4-py3.6.egg\hanlp_common\reflection.py", line 44, in str_to_type
cls = getattr(importlib.import_module(module_name), class_name)
File "F:\Program_Files\Anaconda3\envs\tensorflow-gpu\lib\importlib_init_.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 994, in _gcd_import
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 665, in _load_unlocked
File "", line 678, in exec_module
File "", line 219, in _call_with_frames_removed
File "F:\Program_Files\Anaconda3\envs\tensorflow-gpu\lib\site-packages\hanlp-2.1.0a12-py3.6.egg\hanlp\components\tok_tf.py", line 12, in
from hanlp.components.taggers.transformers.transformer_tagger_tf import TransformerTaggerTF
File "F:\Program_Files\Anaconda3\envs\tensorflow-gpu\lib\site-packages\hanlp-2.1.0a12-py3.6.egg\hanlp\components\taggers\transformers\transformer_tagger_tf.py", line 11, in
from hanlp.layers.transformers.loader_tf import build_transformer
File "F:\Program_Files\Anaconda3\envs\tensorflow-gpu\lib\site-packages\hanlp-2.1.0a12-py3.6.egg\hanlp\layers\transformers\loader_tf.py", line 9, in
from bert import albert_models_tfhub, fetch_tfhub_albert_model, load_stock_weights
ImportError: cannot import name 'albert_models_tfhub'

Sorry for the delay. This bug has been addressed in the latest TensorFlow release. Thanks.


Message: Developer token has expired. Please provide a new one.

Description of the Issue
I am trying to use https://github.com/box/box-python-sdk-gen?tab=readme-ov-file#installing to connect to box so that I can download and upload files. I have tried getting a Developer Token from the Configuration page  and inserting into code. I then try to run the code but get this issue: Message: Developer token has expired. Please provide a new one.
Steps to Reproduce

pip install box-sdk-gen
copy code:
from box_sdk_gen import BoxClient, BoxDeveloperTokenAuth

def main(token: str):
auth: BoxDeveloperTokenAuth = BoxDeveloperTokenAuth(token=token)
client: BoxClient = BoxClient(auth=auth)
for item in client.folders.get_folder_items('0').entries:
print(item.name)
if name == 'main':
main('INSERT YOUR DEVELOPER TOKEN HERE')

I Insert my developer token from configuration page
I run script
gives me error

Expected Behavior
I expected it to be able to connect to my box
Error Message, Including Stack Trace

Screenshots
attached

Versions Used
Python: Python 3.11.8

Developer tokens are usually short-lived. It is possible that yours has expired before the call was made. You can try making the same call with cURL to make sure it's not a problem with the SDK itself - https://developer.box.com/reference/get-folders-id/
curl -i -X GET "https://api.box.com/2.0/folders/0" \
     -H "authorization: Bearer <ACCESS_TOKEN>"

If problem still persist I recommend reaching out to our forums

Hi @marinabuezo,
did you manage to resolve your issue?


about nvidia VGPU

Can OmnisciDb run on nvidia VGPU ? If it can, please tell me how to do it.

When you say VGPU, do you mean in this context?
https://docs.nvidia.com/grid/latest/grid-vgpu-user-guide/index.html

@randyzwitch
yes, I just want to know if OmnisciDb can run on it.

It's probably the case that you can run using VGPUs, but it's not really a use-case we've focused much on, given that we're targeting high-performance vs sharing GPU resources in a mixed use pool. But assuming that each VGPU-enabled VM "looks" like a bare-metal install to the OS, then OmniSci should work the same way.


Abstract Tooltip HTML (and any other HTML) into external template files

We should abstract Tooltip and any other HTML into external templates which can be referenced from the source. A global config param should be created which allows developers to point to different templates (e.g. their own HTML).

+1

I'm fairly in support of this, particularly if all the default templates are precompiled and bundled with the concatenated/minified .js files.
Pros

Better access to C3 component styling and layout
Less DOM manipulation in the code
Probably easier maintenance long-term

Cons

Possible performance implications (not sure TBH; a precompiled template might be faster than all the .appends throughout the library)
Severe possibility of introducing new bugs and breaking stuff
We'd have to decide on a templating language

My personal preference for templating library is Handlebars, namely because it's familiar to me and allows precompilation (important for preventing users from having to include a bunch of HTML and stuff, which would lead to endless unnecessary issues in the queue). Granted, I haven't looked at templating languages in ages and have no idea what's current and good.

@aendrew
I don't think we need to open the templating language debate, and we shouldn't need to add any additional bloat (e.g. templating engines). The simple goal should be to abstract the HTML / markup into its own file(s), which the user could then override with their own templating as they see fit.
The issue today is that the html is hardcoded into the source and in order to override, the user must find it in the source and copy/paste/override in a fairly complex way.

@joshlangner Fair enough, but how would this get included with the library? Would the library have to load a lot of partials on page load or something? I can't help but feel there would be some performance implications until precompiled templates are included with the lib somehow, with those getting overridden if a template path is provided.


Upgrade open dependency

open 0.0.5 is vulnerable https://www.npmjs.com/advisories/663 - fixed in any version >0.0.5

Sure, though note bluebird has no dependencies and this is just a devDependency

For some reason Twistlock (CVE scanner from Palo Alto Networks) is alerting this CVE on our API image, possibly it's coming from some other dependency - will re-check.
/cc @samblackk

devDependency, closing


Regen (elastic and bean related)

Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

sounds like the camel-spring.xsd is genrated differently locally and on the CI. i will remove the changes from my PR


Revert "use correct status type when enforcing policy"

Reverts Kuadrant/kuadrant-operator#594
The Enforced status of policies that are partially enforced should be True, not False.

This will need removed https://github.com/Kuadrant/kuadrant-operator/pull/655/files#diff-8ca7bb2b5ec65645cb6668b37f1e07e74cb4fdbc880ded9b176b34303da44b3cR134-R146, @makslion fyi

Superseded by #677


chore: use pnpm shell-emulator instead of cross-env

Description
Use pnpm shell-emulator instead of cross-env dependencies
Additional context
pnpm uses @yarnpkg/shell to implement bash-like shell, so cross-env dependencie are no longer needed.
So use the Node glob syntax instead of leaving it to the shell expanded. So use ' wrap glob.
shell-emulator config in PNPM docs

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

I wasn't able to reproduce this CI error in a local approximation.


Implement the low-memory rendering pipeline.

Includes some fixes to the various stages.
The low-memory rendering pipeline is enabled by default. This does not
change the decoding output, and has minor effects on speed (that can
likely be compensated for by better optimizing the various stages and
removing remaining non-rp code).
Two TODOs are still left before being able to fully remove the non-rp
code: DC upsampling and encoder-side roundtrip still use code that
belongs to the non-rendering-pipeline path.
Speed measurements when decoding to uint8 (the slowdown is slightly more
significant when decoding to full float buffers, but that's not a common usecase):
x86 baseline: 54.481 MP/s
ARM baseline: geomean: 9.14383 MP/s [9.08262, 9.23499]

x86 after: 53.598 MP/s
ARM after: geomean: 9.25677 MP/s [9.21585, 9.31553]

Also includes various changes all over the place to make the rendering pipeline work properly.

Included in https://github.com/libjxl/libjxl/pull/1249.


Are training set and val set splitted correctly?

It seems that your code use subject 1, 5, 6, 7 and 8 to compose training set, and use subject 9,11 to compose val set. But in Human 3.6M official document, the training set is composed of S1, S7, S8, S9 and S11 (the val set is composed of S5 and S6), so is there any particular reason to do this?

Hi, please refer to paper A simple yet effective baseline for 3d human pose estimation， I follow his valuation

Sorry, it's my fault, your setting is right. However, Human 3.6M dataset contains two type of 3d coordinates, which are D3_Positions_mono and D3_Positions_mono_universal and they are both in camera coordinate system. My question is, which type of 3d coordinates do you use?

Hi, the dataset I use is the internal processed version, so I don't meet your problem. But I think you could back-project this two types of points to 2D plane to see which one should be used

Thanks a lot, I will figure it out myself.


ci: Use GitHub actions

Description
This makes use of GitHub actions for the pipeline instead of separate scripts for each platform.
Type of Change

[x] ✅ ci -- Build configuration change

Handling of error code 183 has been added in pub but is not in stable yet.
This is the commit which rolled the fix into the Dart SDK repo: https://github.com/dart-lang/sdk/commit/2aa637afce62adeba7a70310a6577160bcb6d776

Handling of error code 183 has been added in pub but is not in stable yet.
This is the commit which rolled the fix into the Dart SDK repo: https://github.com/dart-lang/sdk/commit/2aa637afce62adeba7a70310a6577160bcb6d776

That's great, do you know if this was included in Dart 2.17.5?

That's great, do you know if this was included in Dart 2.17.5?

I don't think so, unfortunately. Looks like this will land in 2.18.0.

I don't think so, unfortunately. Looks like this will land in 2.18.0.

Ah too bad. How do you know when different commits will be released?

I noticed that this specific commit adds to the changelog for 2.18.0 :)

Since Dart 2.18.0 now is released, can we try re-running this? :)

I see the Windows test isn't passing 🤔 But I cannot relaunch the CI, maybe @russellwheatley ?

Pushed an empty commit to force a re-run, but I forgot that the workflows require approval since I haven't had any PRs merged in this repo before. 😅

Should be good 👍


Debian 12 erroneously listed as supported distro

While Debian 12 is now listed as supported, i'm not seeing the aadsshlogin package in the Microsoft package repository. So honestly I'm not convinced that this will actually work. Has anyone confirmed this?

Originally posted by @LeonarddeR in https://github.com/MicrosoftDocs/azure-docs/issues/115244#issuecomment-1970550707

@MicrosoftGuyJFlo I have just tried this and even though listed, aadsshlogin is still not supported for Debian 12. Therefore the documentation is now showing incorrect information.

Originally posted by @LeonarddeR in https://github.com/MicrosoftDocs/azure-docs/issues/115244#issuecomment-1981177426

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

ID: a2ede563-b873-0bc3-959f-69e555f0efcc
Version Independent ID: 8471d9e4-6c79-0cf1-c5ed-50c58e1b92aa
Content: Log in to a Linux virtual machine in Azure by using Microsoft Entra ID and OpenSSH - Microsoft Entra ID
Content Source: docs/identity/devices/howto-vm-sign-in-azure-ad-linux.md
Service: entra-id
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@LeonarddeR
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

#label:"awaiting-product-team-response"
@SanDeo-MSFT Please take a look at this issue.

This was indeed fixed, the extension is now available for Debian 12.


Creating the structure

A class is created with the methods to create and display the buttons of each task and its description using the "for-each" loop and event listener.

A class is created with the methods to create and display the buttons of each task and its description using the "for-each" loop and event listener.


workflowtaskresults doesn't exists in namespace install

Summary
Workflowtaskresults are not installed within namespace install.
Diagnostics
I am using
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: argo-workflow
  namespace: argocd
spec:
  destination:
    namespace: clusterscanner
    server: https://kubernetes.default.svc
  source:
    repoURL: https://github.com/argoproj/argo-workflows.git
    targetRevision: v3.2.9
    path: manifests/namespace-install
  project: clusterscanner
  syncPolicy:
    automated: {}
    syncOptions:
      - CreateNamespace=true

to install argo worklows (see https://github.com/SDA-SE/cluster-image-scanner/blob/master/test/argowf/argoworkflow.yml). I have tried multiple 3.2.X versions, which all result in a missing permission for the SA argo to access workflowtaskresults. Logs:
W0303 15:53:10.028391       1 reflector.go:324] pkg/mod/k8s.io/client-go@v0.23.3/tools/cache/reflector.go:167: failed to list *v1alpha1.WorkflowTaskResult: workflowtaskresults.argoproj.io is forbidden: User "system:serviceaccount:clusterscanner:argo" cannot list resource "workflowtaskresults" in API group "argoproj.io" in the namespace "clusterscanner"
E0303 15:53:10.028425       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.23.3/tools/cache/reflector.go:167: Failed to watch *v1alpha1.WorkflowTaskResult: failed to list *v1alpha1.WorkflowTaskResult: workflowtaskresults.argoproj.io is forbidden: User "system:serviceaccount:clusterscanner:argo" cannot list resource "workflowtaskresults" in API group "argoproj.io" in the namespace "clusterscanner"

When I create the permission in my test env. (https://github.com/SDA-SE/cluster-image-scanner/blob/master/test/argowf/role-binding.yml), I get this:
│ time="2022-03-03T15:56:56.707Z" level=info msg="List workflowtaskresults 404"
│ W0303 15:56:56.707926       1 reflector.go:324] pkg/mod/k8s.io/client-go@v0.23.3/tools/cache/reflector.go:167: failed to list *v1alpha1.WorkflowTaskResult: the server could not find the requested resource (get workflowtaskresults.argoproj.io)
│ E0303 15:56:56.707941       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.23.3/tools/cache/reflector.go:167: Failed to watch *v1alpha1.WorkflowTaskResult: failed to list *v1alpha1.WorkflowTaskResult: the server could not find the requested resource (get workflowtaskresults.ar │
│ goproj.io)

To overcome it, I create the CRD https://github.com/SDA-SE/cluster-image-scanner/blob/master/test/argowf/workflowtaskresults.yml.

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

This feature is not available in v3.2. You're running the latest image, not the versioned images.
Do not install using Kustomize. Instead, install using the manifest attached to the release:
https://github.com/argoproj/argo-workflows/releases/download/v3.2.9/namespace-install.yaml

@alexec Thank you for the quick response.
That means this https://github.com/argoproj/argo-workflows/tree/master/manifests#kustomize is outdated?

Yes. This guide is incorrect. Would you like to submit a PR to correct it?

https://github.com/argoproj/argo-workflows/compare/master...wurstbrot:patch-1

Maybe we should warn people about Kustomize?

Installation with kustomize is not supported.
It would work with some customization. You think this is not offensive enough?

I think is frustrating to see kustomize.yaml files, but then not be able to use them. It looks like it is supported, but is not.
I think you could use Kustomize, but you must set kustomize edit set image to set the argo-server and workflow-controller image tags. Additionally, you must patch the executor version, by patching the workflow-controller deployment with the correct env var (I think it is something like ARGO_EXECUTER_IMAGE, but you’d need to check).

I think we are having a missunderstanding here, I understood that it is not supported.
Let is continue in the PR https://github.com/argoproj/argo-workflows/pull/8095.

Well, having just been bitten by the same problem, it seems I am running ":latest" because that's what the default install manifests actually have in them. I probably should've noticed that it said ":latest", but there's an argument to be made that ":latest" is a somewhat dangerous thing to leave in the "this is how you install it quickly" manifest(s).

Is there a reason why the yaml files from the releases are not moved to the manifests folder and you continue to support kustomize? I maintain all the argo components incl versions using kustomize so far by just defining the images. To add sso support e.g. the manifest needs to be patched anyways, which is pretty handy with kustomize.
# image version patch

images:
- name: quay.io/argoproj/workflow-controller:latest
  newName: quay.io/argoproj/workflow-controller
  newTag: v3.3.1
- name: quay.io/argoproj/argocli:latest
  newName: quay.io/argoproj/argocli
  newTag: v3.3.1

Further it seams like the version in the namespace-install.yaml within the release are not up to date.
Release 3.3.1: image: quay.io/argoproj/workflow-controller:v3.2.9
https://github.com/argoproj/argo-workflows/releases/tag/v3.3.1

We do not support kustomize. The reason can be found in the docs. Essentially, the tooling to do it was very buggy because it could not be tested as it only ran on release and would break every release.
If the manifests attached to the release do not have the correct image tag, that is a bug. Could you please raise a new issue?

Response appreciated. I opened an issue. https://github.com/argoproj/argo-workflows/issues/8429


Fickian Diffusion on Statoil Import

Hi Team OpenPNM!
I try to perform Fickian Diffusion from Statoil import Pore Network (on berea example), but it appears that there is a singularity in the matrix set (similar problem from this previous issue maybe?)... Can you help me? I join the output and the python script.
diffusionPost.py.txt
output.txt

Hi @Extremys
Looking at your supplied text file I see that you've not rn pn.check_network_health() on the network.  The maximum ball networks tend to have a lot of blind pores, which we call disconnected_pores.
Basically, just add this after your import: pn.trim(pn.check_network_health()['trim_pores'])

Hi @jgostick, it seems that it works thanks for your super-reactivity!

I just happened to be answering some other question on Github this morning...don't expect this kind of service all the time!  :-)


DRILL-7031: Add Travis job that runs protobuf generation command and checks if all protobufs are up-to-date

Example of a successful run: https://travis-ci.org/vvysotskyi/drill/builds/489799188
Example of job jailed because of non-updated protobufs: https://travis-ci.org/vvysotskyi/drill/jobs/489706919
For details please see DRILL-7031.

Output for the case of non-generated protobufs: https://travis-ci.org/vdiravka/drill/jobs/490016316

+1 Good improvement!


Not responsive

It should be responsive so that for we can use it in out mobile devices.

Duplicate of #435


limactl shell: ask whether to start the instance if not running

This is an edited version of #2763 with a simpler control flow.
It automatically picks the template corresponding to the desired instance name, if it exists. Otherwise it uses default:
$ limactl shell alpine
? Do you want to create and start the instance "alpine" using the "alpine" template now? Yes
? Creating an instance "alpine" Proceed with the current configuration
…

Could we add some "start-if-not-running" flag, perhaps? So that scripts could check that, to see that it's up OK.

How would this be different from running
$ export LIMA_INSTANCE=docker
$ lima true
? Do you want to create and start the instance "docker" using the "docker" template now? No
FATA[0002] open /Users/jan/.lima/docker/lima.yaml: no such file or directory
$ echo $?
1

I think this PR is now a possible replacement for #2763.
Changes to docker.lima, kubectl.lima, and nerdctl.lima should go into a separate PR.

It automatically picks the template corresponding to the desired instance name, if it exists.

This is inconsistent with limactl create.
Acceptable if they can be consistent

This is inconsistent with limactl create.
Acceptable if they can be consistent

Good point! I need to think about this for a while, as there are arguments for both choices.
The main reason I wanted to pick the template to match the instance name for limactl shell was because there is no way to say limactl shell foo --template docker; the only possible template would be default.
On the other hand, I think it is somewhat bad if commands have different semantics based on what files are installed. So limactl shell foo might create a template://foo instance on one machine, but a template://default instance on another.
However, this only happens in interactive use, the user is prompted, and the user can still change the template via the template picker in the next step.
Maybe we need to use a highlight colour for the template name in the message, to make it stand out better:

? Do you want to create and start the instance "docker" using the ${\color{blue}docker}$ template now?

And/or display a warning when the template name doesn't match the instance name?

Rather it should print a note when the template does match the instance name?
Instance names like "vm0", "vm1", ...,  "dev", "tmp", should not print any warning.

Could we add some "start-if-not-running" flag, perhaps? So that scripts could check that, to see that it's up OK.

I think using lima true || exit 1 works well:

Patch
--- cmd/docker.lima
+++ cmd/docker.lima
@@ -7,19 +7,14 @@ set -eu
 : "${LIMA_INSTANCE:=docker}"
 : "${DOCKER:=docker}"

-if [ "$(limactl ls -q "$LIMA_INSTANCE" 2>/dev/null)" != "$LIMA_INSTANCE" ]; then
-  echo "instance \"$LIMA_INSTANCE\" does not exist, run \`limactl create --name=$LIMA_INSTANCE template://docker\` to create a new instance" >&2
-  exit 1
-elif [ "$(limactl ls -f '{{ .Status }}' "$LIMA_INSTANCE" 2>/dev/null)" != "Running" ]; then
-  echo "instance \"$LIMA_INSTANCE\" is not running, run \`limactl start $LIMA_INSTANCE\` to start the existing instance" >&2
-  exit 1
-fi
+export LIMA_INSTANCE
+lima true || exit 1
+
 DOCKER=$(command -v "$DOCKER" || true)
 if [ -n "$DOCKER" ]; then
   DOCKER_HOST=$(limactl list "$LIMA_INSTANCE" --format 'unix://{{.Dir}}/sock/docker.sock')
   export DOCKER_HOST
   exec "$DOCKER" "$@"
 else
-  export LIMA_INSTANCE
   exec lima docker "$@"
 fi

If docker is not running at all:
$ docker.lima run hello-world
? Do you want to create and start the instance "docker" using the "docker" template now? Yes
? Creating an instance "docker" Proceed with the current configuration
…
------
Unable to find image 'hello-world:latest' locally
latest: Pulling from library/hello-world
…
Hello from Docker!
This message shows that your installation appears to be working correctly.

Full log
$ docker.lima run hello-world
? Do you want to create and start the instance "docker" using the "docker" template now? Yes
? Creating an instance "docker" Proceed with the current configuration
INFO[0005] Attempting to download the image              arch=x86_64 digest="sha256:0e25ca6ee9f08ec5d4f9910054b66ae7163c6152e81a3e67689d89bd6e4dfa69" location="https://cloud-images.ubuntu.com/releases/24.04/release-20240821/ubuntu-24.04-server-cloudimg-amd64.img"
INFO[0005] Using cache "/Users/jan/Library/Caches/lima/download/by-url-sha256/b2b185213b60ce48564393cf9eeb3c2ce4e92df4d2ca2edad8657c18d0e3052d/data"
INFO[0005] Run `limactl start docker` to start the instance.
INFO[0005] Using the existing instance "docker"
INFO[0005] Starting the instance "docker" with VM driver "qemu"
INFO[0005] QEMU binary "/usr/local/bin/qemu-system-x86_64" seems properly signed with the "com.apple.security.hypervisor" entitlement
INFO[0005] [hostagent] hostagent socket created at /Users/jan/.lima/docker/ha.sock
INFO[0005] [hostagent] Using system firmware ("/usr/local/share/qemu/edk2-x86_64-code.fd")
INFO[0005] [hostagent] Starting QEMU (hint: to watch the boot progress, see "/Users/jan/.lima/docker/serial*.log")
INFO[0006] SSH Local Port: 64363
INFO[0005] [hostagent] Waiting for the essential requirement 1 of 2: "ssh"
INFO[0024] [hostagent] The essential requirement 1 of 2 is satisfied
INFO[0024] [hostagent] Waiting for the essential requirement 2 of 2: "user session is ready for ssh"
INFO[0024] [hostagent] The essential requirement 2 of 2 is satisfied
INFO[0024] [hostagent] Waiting for the optional requirement 1 of 1: "user probe 1/1"
INFO[0024] [hostagent] Forwarding "/run/user/501/docker.sock" (guest) to "/Users/jan/.lima/docker/sock/docker.sock" (host)
INFO[0024] [hostagent] Forwarding "/run/lima-guestagent.sock" (guest) to "/Users/jan/.lima/docker/ga.sock" (host)
INFO[0024] [hostagent] Guest agent is running
INFO[0024] [hostagent] Not forwarding TCP <IP_ADDRESS>:53
INFO[0024] [hostagent] Not forwarding TCP <IP_ADDRESS>:53
INFO[0024] [hostagent] Not forwarding TCP [::]:22
INFO[0024] [hostagent] Not forwarding UDP <IP_ADDRESS>:53
INFO[0024] [hostagent] Not forwarding UDP <IP_ADDRESS>:53
INFO[0024] [hostagent] Not forwarding UDP <IP_ADDRESS>:68
INFO[0064] [hostagent] Waiting for the optional requirement 1 of 1: "user probe 1/1"
INFO[0073] [hostagent] The optional requirement 1 of 1 is satisfied
INFO[0073] [hostagent] Waiting for the guest agent to be running
INFO[0074] [hostagent] Waiting for the final requirement 1 of 1: "boot scripts must have finished"
INFO[0080] [hostagent] The final requirement 1 of 1 is satisfied
INFO[0081] READY.
INFO[0081] Message from the instance "docker":
To run `docker` on the host (assumes docker-cli is installed), run the following commands:
------
docker context create lima-docker --docker "host=unix:///Users/jan/.lima/docker/sock/docker.sock"
docker context use lima-docker
docker run hello-world
------
Unable to find image 'hello-world:latest' locally
latest: Pulling from library/hello-world
c1ec31eb5944: Pull complete
Digest: sha256:d211f485f2dd1dee407a80973c8f129f00d54604d2c90732e8e320e5038a0348
Status: Downloaded newer image for hello-world:latest

Hello from Docker!
This message shows that your installation appears to be working correctly.

To generate this message, Docker took the following steps:
 1. The Docker client contacted the Docker daemon.
 2. The Docker daemon pulled the "hello-world" image from the Docker Hub.
    (amd64)
 3. The Docker daemon created a new container from that image which runs the
    executable that produces the output you are currently reading.
 4. The Docker daemon streamed that output to the Docker client, which sent it
    to your terminal.

To try something more ambitious, you can run an Ubuntu container with:
 $ docker run -it ubuntu bash

Share images, automate workflows, and more with a free Docker ID:
 https://hub.docker.com/

For more examples and ideas, visit:
 https://docs.docker.com/get-started/

And if you refuse to create or start the docker instance you get proper errors:
$ docker.lima run hello-world
? Do you want to create and start the instance "docker" using the "docker" template now? No
FATA[0001] open /Users/jan/.lima/docker/lima.yaml: no such file or directory

$ docker.lima run hello-world
? Do you want to start the instance "docker" now? No
FATA[0001] instance "docker" status is not "Running" but "Stopped"

I think this is an argument in favour of selecting a template matching the instance name (when possible) for limactl shell.
@afbjorklund This means for kubectl.lima we would have to pick an instance name if we want to offer the auto-create/start functionality. Personally I would want to use k3s, but I think k8s is probably a more canonical choice.

Maybe we need to use a highlight colour for the template name in the message, to make it stand out better:

? Do you want to create and start the instance "docker" using the ${\color{blue}docker}$ template now?

I couldn't find a way to colourize substrings in the survey.AskOne prompt.

Rather it should print a note when the template does match the instance name?
Instance names like "vm0", "vm1", ..., "dev", "tmp", should not print any warning.

Ok, sounds reasonable. Should this be a WARN to stand out more?

Should this be a WARN to stand out more?

Maybe looks too scary to some users who want to dismiss all the warnings


query strings are stripped if wildcards match hits

customer escalation to update the public doc to make the behavior of wildcards match and stripping query string clear: https://identitydivision.visualstudio.com/Engineering/_boards/board/t/Supportability - IDEEE - Authentication/Features/?workitem=2445241 Feature 2445241: [DOC] query strings are stripped if wildcards match in redirect uri

@henrymbuguakiarie
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@cfzhang6688: Thank you for your contriution!
I've checked the proposed changes and verified the information. LGTM
#sign-off


Create fix

So this was actually pretty bad. Amazing that we never got it, probably because we never really cared about tokens. Basic "address.send()" from a contract was broken! This comment/contract wasn't true, since we were making calls without adding the account to the tx cache.
The new tests involve pre-compiled solidity contracts. It may be better to move them to epm, or to replace them with hand-written minimal byte code that captures the functionality.

Looks good to me;
At cost of greater overhead we can update at the call itself, but should be updated now as persisted state is nonce, balance, code and storage.


Add collection_select support

It would be great to have collection_select support.

Pull Request Test Coverage Report for Build 152

10 of 10 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 99.396%

Totals

Change from base Build 151:
-0.5%

Covered Lines:
987

Relevant Lines:
993

💛  - Coveralls

Thank you! Merged.


Support templates versioning format

The templates.json format has been updated to introduce a new root object with the following properties:

version: templates format version
templates: list of templates

Reference to the new file: https://github.com/portainer/templates/blob/master/templates-2.0.json
Reference to the old template format: https://github.com/portainer/templates/blob/master/templates-1.20.0.json
The front-end must be updated to support templates format versioning.

[ ] Blocked by https://github.com/portainer/portainer/issues/3707

Closed via #3729


Use camelcase instead of short variable name

Please use camelcase and longer name for any variable to improve code readability.

ok sure


Unstable_ClassNameGenerator (mui v6) didn't transform DataGrid classes

Steps to reproduce
Link to live example mui v5 (working): codesandbox
Steps:

Check classes of Button and DataGrid
Everything transformed (has toto- prefix)

Link to live example mui v6 (broken): codesandbox
Steps:

Check classes of button and datagrid
DataGrid classes not transformed, only Button class transformed

Expected behavior
DataGrid classes must be transformed
Your environment

npx @mui/envinfo
Happens in any browser

System:
    OS: Linux 5.15 Ubuntu 22.04.4 LTS 22.04.4 LTS (Jammy Jellyfish)
  Binaries:
    Node: 20.17.0 - ~/.nvm/versions/node/v20.17.0/bin/node
    npm: 10.8.2 - ~/.nvm/versions/node/v20.17.0/bin/npm
    pnpm: Not Found
  Browsers:
    Chrome: Not Found

Search keywords: datagrid, Unstable_ClassNameGenerator, generator, className
Order ID: 80084

@michelengelen The root cause is likely comes from missing v6 of @mui/utils in the DataGrid dependencies:
- "@mui/utils": "^5.16.6",
+ "@mui/utils": "^5.16.6 || ^6.0.0",

I am transferring this issue back to mui-x and will open a PR to fix it.


20/implement move button

Added file navigator and reused(copied) existing code for tree view.
Added Dialog window to move files along with "create new directory" function.
fixes #20

Looking for feedback and it is ready for the review!. I have also added API endpoint in frictionless framework.

Screenshot of the feature:

@roll it is ready for the review! Thanks!


[api-docs] Get Current User Guilds now returns up to 200 guilds

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/daa13f60634396296092da02bb8acd8f3360e03b

ncn


Fix the incorrect name of @dependabot in check-workflow-write-permission.sh

GITHUB_ACTOR name of @dependabot is not dependabot but dependabot[bot].

I've updated the script on the CI machines and dependabot[bot] was allowed to update workflow files.
https://github.com/line/armeria/actions/runs/7719907093/job/21056328357?pr=5429


Creates i18n infrastructure

Adds vue-i18n and nuxt-i18n, configured such that:

we'll have the locale in the url (/fr/admin) etc. This is easy to change if RMI wants to later.
we store the user's preference in a cookie, and that cookie works (at least locally) to deliver an SSR'ed i18n page.
we guess the user's location based on accept-lang and a few other heuristics, but honor a user's choice once made.
we offer english pages without prefix (/admin and /en/admin go to the same place)
the language files are lazy-loaded, so only current language gets pulled in by using VueI18nVitePlugin - you can see in the network tab when we switch langauges we fetch the lang/*.json?import file.
configures the fallback to be English, with the expectation that we won't actually write any of the other-language strings until the pre-launch stage of the project.

Creates a pattern that I hope we use everywhere for i18n.

uses objectNotation: true to allow Prefix.Suffix to be a nested property in JSON.
Groups language strings by file prefix - seems like an appropriate thing for now.
uses prefix with t in the template to create a helper tt to avoid having to write prefixes.
Generally uses the full string as the translation key, but uses a placeholder for longer strings.
Implements this pattern on the Nav + Footer + a few other places so you can see what it looks like.

Re-configures the footer to support a language selector. I decided on the footer over the header because as a I looked around and most functional websites put the selector in the footer, while most informational/one-time websites put it in the header.
I went with flags in the selector mostly for visibility/legibility. Without flags it's hard to draw the eye to the selector. I don't love it for a bunch of colonial reasons, but I think given how out-of-view it is after the first selection, I think it's probably a reasonable tradeoff.
Creates a one-time toast to alert folks to the fact that we support multiple languages.

Looks great, main general comment is that not all links have been updated [1], so you occassionally just get dumped back to the default language.
Is there a scheme (within vue/nuxt-i18n) we can use that's a little more bulletproof? It's easy to forget localePath when adding links/buttons/general nav
[1] /pages/admin/index.vue and /pages/admin/pacta-version/index.vue were the two I noticed

Great question + suggestion. I ended up creating some middleware that tests for this, and logs in the console when we fail to meet it. We can beef that up if/when we need to. That (in tandem with making the prefix consistent) will make these navigation errors a lot easier to spot and remediate, though I wish we could do something more compiley.


[explorer] Implement go-to-definition within modules

This uses the new normalized move module data to implement basic go-to-definition with the explorer. The approach we're taking now is super basic due to the limitations of Prism. The main issue that's currently observed is that we over-highlight pretty significantly. Any token that matches the normalized data will get highlighted, as opposed to a more granular approach of only typing move function arguments. This is solvable but requires us to either stop using react-prism-renderer, rework the syntax parsing of the move bytecode, or move off of prism for highlighting entirely.

I think the most impactful change to syntax coloring we could make would be:
find a way to set Move opcodes to a different color than type definitions
being the default page text color (or a color just for opcodes) would probably be fine for all the opcodes within the outline in this image

There's some inconsistency with which definitions offer click-to-jump.
Here, I can click Coin<SUI> but not Supply<SUI>, despite both being similarly formatted generic types, in similiar places, that we have definitions for.

SUI type itself is also not clickable, but has at least a dummy definition in the 0x02::sui module.


Complemented "SCOPE" column towards  "docker network ls " command-line

When I check network_ls.md for further study , I found the document description about "docker network ls" does not match with the terminal "docker network ls" , due to the document missed the "SCOPE" column.
So I complemented the missing part of  "SCOPE" column.
My docker version is 1.12.3
- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "commandline-pr" git@github.com:erxian/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "commandline-pr" git@github.com:erxian/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354412968
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Really sorry for my misuse of github, I will close this one to add a brand new one PR.
Sorry again for my bothering brought to this Repo.


qovery-cli: 0.55.2 -> 0.56.0

Diff: https://github.com/Qovery/qovery-cli/compare/v0.55.2...v0.56.0
Changelog: https://github.com/Qovery/qovery-cli/releases/tag/v0.56.0
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 224220 run on x86_64-linux 1

1 package built:

qovery-cli


Spelling error in naming the image built

Signed-off-by: Sven Dowideit<EMAIL_ADDRESS>sorry.
should have no functional effect, it'll just be the wrong image name

LGTM


[Feature] dotnet libraries not cache invalidated when package.json updates

Is your feature request related to a problem? Please describe.
I'm always frustrated when package.json updates and the cache invalidates on the dotnet libraries.
Describe the solution you'd like
Something along the line of using nx' 16.5 input: [{externaldependencies: nx-dotnet}] to not have the libraries trigger update only when @nx-dotnet/core updates
Describe alternatives you've considered
Other than adding the dotnet libraries to .nxignore or removing the package entirely, not really. It's a nuisance, not breaking.

Setting externalDependencies for them is the correct thing here, just remember that inputs arrays are not merged so you'll need to include your other namedInputs.
There are several caveats to how this would be done so I don't think its something that we will generate for folks, rather its something that will need to be done on a repo-by-repo basis since peoples inputs + outputs tend to be set up slightly differently and we would want to avoid false cache hits.
It seems like you know how to fix this for your repo, so I'm going to go ahead and close this out. Feel free to let me know if any more help is needed.

I tried fixing it myself, though this didn't seem to work. Maybe my premise is wrong or my setup is wrong:
My premise with this case:

Have "inputs": [{ "externalDependencies": ["@nx-dotnet/core"] }] on every task with a dotnet project
update package.json with dependencies not being @nx-dotnet/core.
Run npx nx run-many -t build lint test. The dotnet libraries should returned to be cached and not being built, linted or tested

When i tried this this morning with these targets:
"targets": {
		"build": {
			"executor": "@nx-dotnet/core:build",
			"inputs": [{ "externalDependencies": ["@nx-dotnet/core"] }],
			"outputs": ["{workspaceRoot}/dist/libs/shared/startup"],
			"options": {
				"configuration": "Debug"
			},
			"configurations": {
				"production": {
					"configuration": "Release"
				}
			}
		},
		"lint": {
			"executor": "@nx-dotnet/core:format",
			"inputs": [{ "externalDependencies": ["@nx-dotnet/core"] }]
		},
		"lint-fix": {
			"executor": "@nx-dotnet/core:format",
			"inputs": [{ "externalDependencies": ["@nx-dotnet/core"] }],
			"options": {
				"fix": true
			}
		}
	},

Do I need the namedInputs or is this a viable setup. If so, why is the package.json update still triggering these tasks to be run out of cache hit?


Lock judgments and get lock status for judgments

Trello: https://trello.com/c/l60TQkpe

The test failures are expected at this stage because we haven't linted the files yet.


The connection between some NSE and NSC does not work but no error

I am using NSM next-gen (multi-repo), not this repo, I don't know exactly which component of multi-repo is impacted so I opened the issue here.
Expected Behavior
I should be able to ping every NSCs from the NSE
Current Behavior
All interfaces on the NSC and NSE are injected and all IPs are configured correctly, but It is impossible to ping some NSC from the NSE.
From nse (the NSE of icmp-responder), I can ping all of its NSC except:
<IP_ADDRESS>

From nse-2 (the NSE of icmp-responder-2), I can ping all of its NSC except:
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

Failure Information (for bugs)
NAME                            READY   STATUS    RESTARTS   AGE     IP            NODE           NOMINATED NODE   READINESS GATES
forwarder-vpp-2jg58             1/1     Running   0          8m11s   <IP_ADDRESS>    kind-worker2   <none>           <none>
forwarder-vpp-m72xn             1/1     Running   0          8m11s   <IP_ADDRESS>    kind-worker    <none>           <none>
nsc-2-8554589b4c-5dvct          1/1     Running   0          7m46s   <IP_ADDRESS>   kind-worker2   <none>           <none>
nsc-2-8554589b4c-6l7mh          1/1     Running   0          7m46s   <IP_ADDRESS>   kind-worker2   <none>           <none>
nsc-2-8554589b4c-7tj57          1/1     Running   0          7m46s   <IP_ADDRESS>   kind-worker2   <none>           <none>
nsc-2-8554589b4c-8926j          1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker    <none>           <none>
nsc-2-8554589b4c-8zhzk          1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker2   <none>           <none>
nsc-2-8554589b4c-dws9j          1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker2   <none>           <none>
nsc-2-8554589b4c-f46vh          1/1     Running   0          7m46s   <IP_ADDRESS>   kind-worker2   <none>           <none>
nsc-2-8554589b4c-j6fjr          1/1     Running   0          7m46s   <IP_ADDRESS>   kind-worker    <none>           <none>
nsc-2-8554589b4c-jkkmq          1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker    <none>           <none>
nsc-2-8554589b4c-sggxd          1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker2   <none>           <none>
nsc-747d47bddf-2m2dq            1/1     Running   0          7m46s   <IP_ADDRESS>   kind-worker    <none>           <none>
nsc-747d47bddf-2rddf            1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker    <none>           <none>
nsc-747d47bddf-6tbfm            1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker2   <none>           <none>
nsc-747d47bddf-76v9b            1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker2   <none>           <none>
nsc-747d47bddf-kz4vj            1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker    <none>           <none>
nsc-747d47bddf-pdx5b            1/1     Running   0          7m46s   <IP_ADDRESS>   kind-worker    <none>           <none>
nsc-747d47bddf-pjgwv            1/1     Running   0          7m46s   <IP_ADDRESS>   kind-worker    <none>           <none>
nsc-747d47bddf-qj8vs            1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker    <none>           <none>
nsc-747d47bddf-rh4hz            1/1     Running   0          7m46s   <IP_ADDRESS>   kind-worker2   <none>           <none>
nsc-747d47bddf-rkztl            1/1     Running   0          7m46s   <IP_ADDRESS>   kind-worker    <none>           <none>
nse-2-6679ffc6f7-jgm24          1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker    <none>           <none>
nse-9b6588f9f-989mc             1/1     Running   0          7m46s   <IP_ADDRESS>    kind-worker2   <none>           <none>
nsm-registry-7d79995c87-9r5tv   1/1     Running   0          8m11s   <IP_ADDRESS>    kind-worker    <none>           <none>
nsmgr-pbq4b                     1/1     Running   0          8m11s   <IP_ADDRESS>    kind-worker    <none>           <none>
nsmgr-rsgxv                     1/1     Running   0          8m11s   <IP_ADDRESS>    kind-worker2   <none>           <none>

Steps to Reproduce
The bug happens randomly, sometimes I can deploy without an issue, and sometimes half of my NSC cannot be reached from the NSE. I never managed to get this bug with 1 NS and few pods, it always happened to me with multiple NS, NSE and NSC.
I deployed 2 NSEs of 2 different NS (icmp-responder (<IP_ADDRESS>/24) and icmp-responder-2 (<IP_ADDRESS>/24))
I deployed 20 NSCs using kernel mechanism, 10 request icmp-responder and 10 request icmp-responder-2
From the NSE, I try to ping each NSC.
Context

Kind (1 control-plane, 2 nodes)
Kubernetes 1.20.2
cmd-nsmgr (0f368c7a3252b9eab06ca4017cd7d6da605b303a)
cmd-forwarder-vpp (1067497c5ea12ad834d9b0e1e4b90a1c8cfe4b34)
cmd-registry-memory (5fe2e39f6ebd794ad330ca685f9222395f43d20d)
cmd-nse-icmp-responder (12dfd2ee17a5dcc1a8846c54ee73e4a3077dd264)
cmd-nsc (1cd011739e8eeddefa5026a1b90c1f7501981e39)

Failure Logs
I extracted the logs with kind using this command:
kind export logs /logs

logs.zip

Here are my last findings.
I connect to the forwarder which is on the same host as the NSE, and when I run this command: vpp show interface address
I get this result:
host-eth0 (up):
  L3 <IP_ADDRESS>/16
local0 (dn):
tap0 (up):
  L2 xconnect vxlan_tunnel0
tap1 (up):
tap2 (up):
  L2 xconnect tap3
tap3 (up):
  L2 xconnect tap2
tap4 (up):
  L2 xconnect vxlan_tunnel1
vxlan_tunnel0 (up):
  L2 xconnect tap0
vxlan_tunnel1 (up):
  L2 xconnect tap4

The TAP interface without “L2 xconnect” (tap1) is corresponding to the interface with the broken link (ping doesn’t work), I verified it with the MAC address in the pod and the mac address return in this command: vpp show tap. From what I have seen, the pod (NSC) where is the other interface, is always on another node (NSC and NSE on different nodes).
I also tried to run the command vpp show interface address, before and after this line https://github.com/networkservicemesh/sdk-vpp/blob/main/pkg/networkservice/mechanisms/vxlan/server.go#L55. And as we can see, there is no "vxlan_tunnel" created which I guess should have been created (like vxlan_tunnel1 for tap4), in order to be associated to the TAP interface.
To reproduce this bug, we can use the icmp-responder example with 2 NSE of 2 different NS and 20 NSC (10 connected to each NS). Sometimes (maybe 30% of the time), I get at least 1 non-working interface.
I updated my dependencies (sdk, sdk-vpp...), my vpp-forwarder and nsmgr with changes from this week, and I still have this bug.

When the vxlan tunnel is created, this VxlanAddDelTunnel function returns rsp with 0 as Retval, and a SwIfIndex such as 3, 4, 13... but when the vxlan tunnel fails to be created, Retval is always -75, and SwIfIndex is always<PHONE_NUMBER> (max uint32).


when press ctrl + C it report this... and can't do copy action

VSCode Version: Code 1.8.0 (38746938a4ab94f2f57d9e1309c51fd6fb37553d, 2016-12-13T17:45:32.595Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

vscode-eslint
dbaeumer
1.2.2

vscode-babel-coloring
dzannotti
0.0.4

Steps to Reproduce:

@kooboo-chenhai can you please describe the steps you did before this message appeared? Can you provide an animated gif please?

@egamma  Software to use an hour later will be that.
Uh,maybe i opened the translation software namded "youdao" (http://cidian.youdao.com/index.html)  at that time

@kooboo-chenhai are you saying this is now working for you and the issue can be closed?

no, i means the copy function always not work well when vscode has run some hours

@kooboo-chenhai thanks for your reply.
@peng what does the error message in the screenshot say, can you pls help to translate.

"WARN cannot copy terminal selection when terminal does not have focus"
#17232

Updating to v1.8.1 will fix the issue.


Alt.OffClient and AltAsync.OffClient don't work

Short description: registering OnClient in AltAsync or Alt registers fine, but when trying to unregister methods still get called when emitting event from clientside.
What happens: OffClient does not unregister event
What should happen: OffClient should unregister event and unregistered method souldn't be called when events are being emitted from clientside
Steps and code to reproduce:

Construct class
Emit events, only one line is being printed Async event called! 0  or sync for sync event
Call finish method, emit events, methods still get called, Async event called! 0 being printed

Version: 3.0-dev16, nugets are updated to their latest versions

When trying to construct class again, two instances of class receive events, and this results in code not working properly
using System;
using System.Threading.Tasks;
using AltV.Net;
using AltV.Net.Async;

    public class TestEvents
    {
        public static int InstanceNumber = 0;
        private int _instanceNumber;

        private TestEvents()
        {
            this._instanceNumber = TestEvents.InstanceNumber;
            TestEvents.InstanceNumber++;
            AltAsync.OnClient<AltCorePlayer, int>("testEventJob:asyncEvent", this.AsyncEvent);
            Alt.OnClient<AltCorePlayer, int>("testEventJob:event", this.Event);
        }

        private async void AsyncEvent(AltCorePlayer player, int n)
        {
            Console.WriteLine($"Async event called! {this._instanceNumber}");
        }

        private void Event(AltCorePlayer player, int n)
        {
            Console.WriteLine($"Sync event called! {this._instanceNumber}");
        }
        public void Finish()
        {
            AltAsync.OffClient<AltCorePlayer, int>("testEventJob:asyncEvent", this.AsyncEvent);
            Alt.OffClient<AltCorePlayer, int>("testEventJob:event", this.Event);
        }
    }
}

Next dev update will make the Off functions functional by accepting the return value of the On Functions as second parameter. Using the delegate reference won't work for it since they are transformed internally.


Add IP2Location.io

IP2Location.io Python SDK allows user to query for an enriched data set based on IP address and provides WHOIS lookup API that helps users to obtain domain information.

Sorry, this is too commercial in nature to include here.


Create new question type QuestionDict with a tutorial on how to create a new question type

It is possible to return a dict by manipulating the QuestionList question_text with instructions (eg, return a list of one or more dicts) but a separate question type would be useful.
Create a tutorial about adding a new type while doing this.

Basically, I think we want this functionality: https://pypi.org/project/instructor/
You explain what you want, structure-wise, and it takes care of it. Are you imagining it would always be a data extraction task & that you'd pass in their keys and types?


Improve tutorial and README

This PR makes several improvements to our README.
First, re #94, it adds to the top a clear list of the four core functions of eyecite: extraction, aggregation (resolution), annotation, and cleaning.
Second, re #95, it clarifies the tutorial regarding the hypothetical "database" and other confusing items. All the code in the tutorial is now definitely run-able by the reader, except for two sections that offer psuedo-code for expanded functionality (the resolutions2 and annotations2 sections). I also attempted to clarify those sections themselves by providing more details about how one might integrate with an external database or API. It should hopefully be clear that eyecite works fine without doing that, but the pseudo-code should still be helpful for users trying to exploit that functionality.
This PR shouldn't be merged before #96 since I added some links (that don't work) to the new API documentation. This also doesn't address all the open issues re the README; some things (re tests and bigger examples) may still need to be added.

Matt, you want to take a look at rebasing this and resolving conflicts now that #96 is in?

Just rebased and added real links to the new API documentation.

This is good. One small comment, but you took out the reference to Harvard. Can you get that back in?

Yes -- restored at the very top!


updated read Me file

@nazoorahamed Feel free to fully grab an issue from https://github.com/hotosm/OpenMapKitServer/issues?q=is%3Aissue+is%3Aopen+label%3Ahacktoberfest as there are updates to the docs that would be helpful to the project.


Eclipse can't recognize the class Message in Message.Recipient.TO

Eclipse is pointing to an error at the line :
email.addRecipient("mounaim", "m.zaryouhi@mymail.com",Message.Recipient.TO);
it seems it can't point to any Message class inside the simple java mail jar. What could be the problem here ? I'm using java 1.7 and simple-java-mail-4.0.0 version

Seems to be an issue with Eclipse. I have no idea. Have you tried IntelliJ?


state variables cannot use any kernel method names

because StateWrapper needs to inherit from BasicObject

This issue was moved to reactrb/reactrb#93


fix(repo parsing): considers cases where git repo has a trailing /

Describe what this PR does / why we need it
the automatic repository finding (when none passed as args) wouldn't work when the return of git remote -v had a trailing '/'. this is reproducible by adding a trailing '/' when cloning a repository
parser would split the lines with '/' as a delimiter. the trailing '/' made the result be an array of length 4 which wasn't considered.
Does this pull request fix one issue?
Fixes #605
Describe how you did it
added condition for when the split segments has a length of 4.
Describe how to verify it
git clone https://github.com/pwntester/octo.nvim/ && cd octo.nvim && nvim -c 'Octo issue list'

Special notes for reviews
additional checking that segment[4] == nil could be added to ensure this change fixes only this edge case
the opposite approach could be made and use if #segments > 2 so it becomes the default behavior for more edge cases

Thanks @Hashino can you please fix the linting errors?

@pwntester done

@pwntester since this is my first contribution you'll need to approve the run of the workflows

Thanks!