Improve Python 3 Support

Currently Pyflame works with Python 3 only as long as the file names are ASCII. If the file names include Unicode characters, Pyflame will fail to decode them.
Some Background
If you download the source code for Python, you can find the implementation of strings (actually unicode objects) in Objects/unicodeobject.c. There are a few other files, but this is the main one.
A unicode object in Python actually has one of three representations; it can be any of the following:

PyASCIIObject
PyCompactUnicodeObject
PyUnicodeObject

Which one is actually used depends on what codepoints are actually in the string. If the string just contains ASCII characters a compact PyASCIIObject representation will be used. This representation is just like a Python 2 string: it has a size, and a pointer to bytes. Currently Pyflame assumes that string objects are really of type PyASCIIObject.
A Solution
A fix for this would include:

logic to detect what underlying type the unicode object actually is
logic for getting the raw bytes from the Unicode object
logic for converting the raw bytes to UTF-8 for non-ASCII strings

You should be pretty familiar with the internals of unicodeobject.c before attempting to implement a fix here.
As of this writing, the code here should probably go into src/frob.cc which is where the existing string handling code is. Although depending on the complexity of the solution, it may make sense to break out into another file.

Can you point to the part of the code where this change would have to be done?

@nichochar I'm not completely familiar with the code, but it seems that the Unicode support must be implemented somewhere around the pystring code: https://github.com/uber/pyflame/blob/master/src/pystring.cc#L38

I updated the top-level comment on this issue to provide more background, if anyone wants to take a stab.

I have got a first (very ugly) version of this to work. I will clean it up over the next couple of days, but then I would certainly like to get some feedback. I have made some assertions right now that may or may not be allowed, and it will probably not work for early versions of Python 3 (<3.3) either.
I also ran into the issue where FlameGraph does not really handle UTF-8 correctly either, so that will have to be fixed before people can actually use it.

I have pushed a commit which I would like some feedback on.
Besides the bunch of TODOs and asserts, The most important part I think is the discrepancy in the signature of Python 3's StringData() function. It currently returns a std::string, because it does all the heavy lifting itself. I think I have to do it this way, because the address of the second PtracePeekBytes depends on the result of some processing on the first PtracePeekBytes. I could move this processing and PtracePeekBytes stuff to the FollowFrame() function, but that would only make everything uglier and harder to understand.

@jackvreeken Sorry for not seeing this earlier. Your commit looks pretty good to me! Note that some stuff in frob.cc has changed very slightly, and #70 is also changing the interfaces for the Python 3.6 changes. But neither of those changes are too invasive.
As for testing, I think it's sufficient to create copies of the test Python scripts in the tests/ directory with unicode names, and then test against those as well. Ideally we could just do this using a symlink, so that we don't need to keep changs in files in sync. If that doesn't work (e.g. because Python resolves the file name after following the link) we could change runtests.sh to create copies of the files and then clean them up after.
I don't know how to test every file encoding (perhaps you do), but if we just confirmed that non-ASCII worked that would be good enough for me.

Alright, when #81 gets merged I will rebase on top and fix anything that has changed. For testing I would not just want to test unicode filenames, but also unicode function names (just in case).
What do you mean by every file encoding? Do you mean Latin-1, UCS-1, UCS-2 and UCS-4, or other exotic types like BIG5 and SHIFT-JIS? The latter two are handled by Python itself, so we only have to deal the "standard" set. I think Python just chooses the smallest one that fits to store strings, so I could come up with some filenames/function names that don't fit in UCS-2 but do fit in UCS-4. I would have to check that though.
Our output is always UTF-8 by the way, as that is what FlameGraph expects.

That change is landed now, and I released it as 1.4.0. If we can fix this issue, I will tag/release 1.5.0.
Good call on testing Unicode function names -- that's easier to test anyway (although I agree we should still have some Unicode file name tests).
For the encoding, what I remember when I first wrote the code for Python 3 string handling, is that Python internally uses one of four possible representations (e.g. ASCII, UCS-{1,2,4}, etc.) based on what bytes are actually in the string. Ideally we would have file/function names that test all possible internal representations (or at least, all of the ones Pyflame supports). I know the ASCII heuristic is just to use the ASCII internal representation if the string is representable as ASCII, for multi-byte encodings I'm not sure how it decides which multi-byte representation it wants to use.
For the very large characters, I know the newest emoji are large (I believe the terminology is that they're in the astral plane?). For instance, https://unicode-table.com/en/2603/ is three bytes in UTF-8, and https://unicode-table.com/en/1F356/ is four bytes in UTF-8.

Apparently not all unicode characters are supported. A certain set of them (=Unicode 4.1) are, which are defined in PEP-3131. See this link for the full list. I'll choose some exotic ones from this list to test with.


Upgrading clang-tidy to 7

Since clang-tidy became able to diagnose files without compile errors, it's now possible to upgrade it to 7 or later. Let's bump clang-tidy to 7, the default version on Debian "buster".
This is a follow-up of #24193

BTW, note that when you create a PR that you don't intend to be reviewed yet, you can create it as a draft PR, which will avoid adding reviewers from codeowners.

Thank you, Mark for the review. This PR was ready for the view but it started failing once it's rebased into the base PR. That's why I had to fix some errors after sending a review request.


update producer c ref

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #726 (473e893) into master (ade7dea) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #726   +/-   ##
=======================================
  Coverage   69.47%   69.47%
=======================================
  Files          32       32
  Lines        1661     1661
=======================================
  Hits         1154     1154
  Misses        507      507

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 ade7dea...473e893. Read the comment docs.


[Q]wandb: Network error (SSLError), entering retry loop.

This issue doesn't have any actionable information. i will be closing it. If you want help please provide more information.

When I run proxy software on Windows, like clash, wandb.init () function raises an error（SSLError）

This issue only recently occurred. Previously, I ran the proxy software and wandb simultaneously without causing errors

same error! When I run clash on Ubuntu, it raises the same error!


Use in typescript project

Is there a way to use this library in a Typescript project?

Funny thing we haven't updated with TS2 way to provide typings (and we never did with tsd or typings) . I'll make a PR fixing this today.

@codedoge I've been working on it on #90. While we finish reviewing it, would you care to try using/testing it with<EMAIL_ADDRESS>

I'll take a look, it's the first time I use this lib and I don't have much experience with typescript. I'll try and let you know.

Since PR #90 has been merged to master (waiting for next release), I'll be closing this.


Vulkan: Tile stretch mode doesn't work for TextureRect & TextureButton

Godot version:
Master 719609522a30fa9e3b96b028ac5a1d1a65f1678f
OS/device including version:
OS: Windows 10
Rendering: Vulkan (GLES can't be tested at the moment)
Graphic card: GeForce 940M
Issue description:
Setting the stretch mode to "Tile" in TextureRect or TextureButton doesn't result in the texture repeating itself.

I didn't find any setting on the texture to make it repeat on stretch, so it seems tiling is not supported at the moment.
Steps to reproduce:

Create new project
Add a TextureRect or TextureButton in the scene
Set texture property with icon.png
Check expand property
Select Tile in Stretch Mode property
Scale the control up
Observe tiling doesn't work properly

Minimal reproduction project:
texture-tiling.zip

I can confirm this as I also have this bug

I also noticed this when using an atlas resource on a texture rect and trying to tile it. Same thing happened. It just stretches.

Prior to Godot 3.3, I was able to work around this bug by adding a child to the TextureRect. After Godot 3.3, this workaround no longer works.
	var texture_rect_glitch_workaround: ColorRect = ColorRect.new()
	texture_rect_glitch_workaround.name = "WorkaroundForGodot40011"
	texture_rect_glitch_workaround.color = Color.transparent
	add_child(texture_rect_glitch_workaround)

@Poobslag This issue is Vulkan specific. If you have a similar issue in 3.3, this should be reported separately.

@Poobslag This issue is Vulkan specific. If you have a similar issue in 3.3, this should be reported separately.

Thank you! My mistake.

Doesn't seem to happen anymore in 83d2673


"Fills" for coinbaseEx

the coinbase exchange API has an endpoint to return trade fills. (example response below).
it would be useful to have a function in the wrapper getFills() which would return this. right now it seems all i can do is search the open orders list for the order i just placed and monitor when it disappears to know if i was filled. but that still doesn't provide me with extra detail i need like "fee" etc.
am i missing some existing function that provides this Fill detail?
[
{
"trade_id": 74,
"product_id": "BTC-USD",
"price": "10.00",
"size": "0.01",
"order_id": "d50ec984-77a8-460a-b958-66f114b0de9b",
"created_at": "2014-11-07T22:19:28.578544Z",
"liquidity": "T",
"fee": "0.00025",
"settled": true,
"side": "buy"
}
]

https://github.com/timmolter/XChange/blob/develop/xchange-coinbaseex/src/main/java/com/xeiam/xchange/coinbaseex/service/polling/CoinbaseExTradeService.java#L58
The function exists but nobody has written it yet.

@geigerzaehler242 Is this still relevant to GDAX?

Closing this as exchange changed name, API, etc. Please reopen if this persists for GDAX.


Legend Title does ot fit

First of all Documentation is still very annoying.
You just need to put a complete HTML file in there so people can have all the parts together. Anyone with basic knowledge of HTML,CSS,Javascript should be able to understand it. With the way that you have arranged your documentation, first you need top open 5 different pages to setup a working sample, then go to example pages and try to figure out which one of those chunks of codes goes where! This has been pointed out so many times.
Also in getting started only specifies use of CDN, but when even using cdn with version umd version should be used and not just chart.min.js version like any other library. cdnjs.com also does not have all the versions that are available in jsdelivr.com
ISSUE ITSELF
This issue not only applies to latest version of the Chart.JS (4.3.3 at the moment) but all the versions above 3.
Previously it was possible to get around this issue using generateLegend() function which was removed. The replacement method HTML Legend is to complicated and since the example is split in chunks with some missing and due to use of framework in examples, view source in browser wont work help to find what's going on either.
Following is a simple page that will show the problem (in screenshot too), I have tested this against all the versions from 4.2.1 up to 4.3.3
<!DOCTYPE html>
<html>
<head>
    <script<EMAIL_ADDRESS></head>
<body>
    <canvas id="pieChart" style="max-width: 980px; max-height: 340px; padding:10pt;border: 1px solid #000000"></canvas>
    <script type="text/javascript">

        const config = {
            type: 'doughnut',
            data: {
                labels: ['ABC', 'DEF'],
                datasets: [{ data: [50.00, 10.00,] },]
            },
            options: {
                plugins: {
                    legend: {
                        title: {
                            text: function () { return '4ABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZ'; },
                            display: true,
                        },
                        position: 'left'
                    }
                }
            },
        };

        let pie_chart = new Chart(document.getElementById("pieChart"), config);
    </script>
</body>
</html>

As you can see in following screenshot, the legend title is trimmed from both side. I could not find a property or method to fix this issue.

Note that there is enough space to show entire title of legend and push the chart to right, but I can't find any configuration allowing me to do that.


Virtcontainers documentation is outdated

Which feature do you think can be improved?
/kind documentation
Skimming over the documentation for virtcontainers, I found:

Outdated links mainly pointing to a deprecated Clearcontainers Org.
Good documentation (with diagrams) for CNM but minimal/missing documentation for CNI (which is IMO more important today).

How can it be improved?
The documentation needs to be updated with all state of the art links, and relevant frameworks.

@Adel, is this still relevant or you pushed a few of these fixes?
I will put this on your for now and we can work on this together :-).

@ariel-adam didn't push the fixes for this issue yet.

This issue is being automatically closed as Kata Containers 1.x has now reached EOL (End of Life). This means it is no longer being maintained.

Important:
All users should switch to the latest Kata Containers 2.x release to ensure they are using a maintained release that contains the latest security fixes, performance improvements and new features.

This decision was discussed by the @kata-containers/architecture-committee and has been announced via the Kata Containers mailing list:

http://lists.katacontainers.io/pipermail/kata-dev/2020-November/001601.html
http://lists.katacontainers.io/pipermail/kata-dev/2021-April/001843.html
http://lists.katacontainers.io/pipermail/kata-dev/2021-May/001896.html

If you believe this issue still applies to Kata Containers 2.x, please open an issue against the Kata Containers 2.x repository, pointing to this one, providing details to allow us to migrate it.


feat: added dependabot - auto dependecies update

Dependencies Management
Why?
To prevent security breach it is better to have a project always up to dates. With DependaBot, now part of the GitHub family, the updates come to you without the need to take time to check for them.

What?
The proposed configuration is taking care of all possible updates in the Java part of the project and only to the security updates for the NPM part.
Configuration can be naturally changed as desired.

@dktsni do you need this pullrequest on v4 too instead of master? Don't know if you are using dependabot but it is opening PR when a new version of lib you are using (npm or maven) is available.

Hi @mmornati
Dependabot was already added on version 4.0.0 (thx to @lusoalex ), you can find its configuration here :)
Master isn't subject to changes (unless a Hotfix on the current version is needed), so I think that we can skip Dependabot on it (cf discussions on PR #116 )
Thanks !

@dktsni can you close the PR? ;)

@dktsni @mmornati As dependabot was already added previously, this PR has been closed, but the branch has not been deleted. Could one of you delete it?
Regards,
Alexandre.

Done ;)


added github action, updates to pom for automatic release to maven central

Prerequisite steps:

https://central.sonatype.org/register/central-portal/
set all of the secrets for maven central, your GPG key and your SSH key in github settings

Notes:

the action is currently set up to trigger on a merge to main. this will version and publish the artifact every time a merge is made. not sure if that's what you want.

@robinbraemer This is essentially ready to go. In order to proceed and try to merge it you would need to:

Tell me if it is a correct assumption that you want it to automatically publish to maven central when a merge is made to main. If not, let me know how you want to trigger the action.
As in the prerequisites above,
a. Set up a maven central / sonatype account, and set the OSSRH_USERNAME and OSSRH_PASSWORD as repository secrets
b. Set your GPG_KEY_ID and GPG_KEY (public!) as repository secrets. You'll need to create a GPG key if you don't have one already
c. Set an SSH_PRIVATE_KEY as a repository secret. I prefer to make a separate SSH keypair for each repository so if one gets compromised, you can always just diable it. To generate and upload a new key to Github, you can follow this guide.

Please let me know if I can help or answer any questions. Thank you again!

Thank you! Why did you close?

I was under time pressure and I used a different approach to call the Cloudflare API, so I no longer need the library in Maven Central.


Copy, convert, and edit md guides to GLD

This includes converting to .Rmd and including the yaml code to render the md files into a user friendly directory -- making small edits where possible in the body of the document. Major revisions will come later.

resolves #82


Setting the admin password from ENV variables

In order to fully support Infrastructure-as-Code settings, changing the admin password without manual intervention would also introduce additional security.
In case someone is actively searching for a new umami instance on a URL, that person could be faster than the administrator of said instance and gain unprivileged access.
Having the option to set the admin password via an environment variable would solve this issue. (And allows to reset the admin password with a simple restart of the instance.)

To further support infrastructure-as-code settings, it would be great if one could start umami with a simple configuration file stating which websites should exist. Something like this:
websites.conf
some-website-name,predetermined-token
other-website-name,other-token

In case it's helpful to others, here's a way to accomplish both when deploying umami on kubernetes: https://stianlagstad.no/2022/08/deploy-umami-analytics-with-kubernetes/.

@mikecao seeing this issue going stale, are there any plans for improving the umami-initial setup without touching the database directly as @stianlagstad described?
If there are no plans, should we close this?

Any updates on this issue?
I'm currently facing a situation where I need to reset the admin password, and so far haven't been able to find how to do that. Having something like this would certainly make that easier.

I would still like to see this feature added :(


Default query editor connection to last-used user connection

As mentioned in #199

v3 does this!


Utility function to use the CLI interface to publish models to Hugging Face

Implementation of  the utility function to use the CLI interface to publish models to Hugging Face
Also a code refactor to use the same function publish_model_to_hugging_face using the CLI and the graphic tool.

@diegomiranda02 sorry for two reviews, hope it is not too confusing
one of us will re-review after addressing the comments, thx!

Thanks a lot @diegomiranda02 , will you work on this still? Otherwise, I may take it over and address the reviews.

@pascal-pfeiffer I've completed the code refactoring.

Safe serialization toggle is being ignored (always True)

Did this code snippet provide the value for Safe serialization? q.client["default_safe_serialization"]. I could propagate this as a function argument. What do you think?

Needing to provide model name + exp path feels redundant. Needs to be clear that the model_name is only used for HF. This should also not be a requirement but optionally automatically generated from the experiment name.

Previously, the code added the user ID and model name and passed the repository ID as a function argument. However, @maxjeblick pointed out that the following code snippet is necessary.
@maxjeblick  comments
Could you propagate user_id to a function argument?
Within the app,
user_id = q.client["experiment/display/push_to_huggingface/account_name"]
if user_id == "":
user_id = huggingface_hub.whoami()["name"]
is used, as the user may not have been logged in to hf hub.
When I passed the user_id as a function argument, I also passed the model name. I concatenated the two to create the repo_id variable. Do you have any suggestions on how to handle this situation?

Let's reuse hf_repo_friendly_name and force the name to be HF compliant.

Of course! Would you mind providing me with instructions on how to use this?

api_token = q.client["default_huggingface_api_token"] is wrong to pass from the form to the script. Needs to be q.client["experiment/display/push_to_huggingface/api_key"] as it could have been changed in the dialog.

I have a doubt about the difference between an api_token and an api_key, as the previous code only had an api_key. Would the value of the api_key from q.client["experiment/display/push_to_huggingface/api_key be sufficient?

why check for log-in? if not huggingface_hub.is_logged_in(): We probably need to force the relog if new credentials are supplied.

To avoid having users provide login credentials again, Hugging Face checks if they are already logged in. I was considering testing this code, but it may not be necessary and can be removed.

Thanks once more @diegomiranda02

Safe serialization toggle is being ignored (always True)

Did this code snippet provide the value for Safe serialization? q.client["default_safe_serialization"]. I could propagate this as a function argument. What do you think?

q.client["default_safe_serialization"] holds the value for the safe serialization toggle from the UI. So, this will need to be consumed by your new script. Let's also make it an optional argument when calling it from CLI (defaults to True).

Needing to provide model name + exp path feels redundant. Needs to be clear that the model_name is only used for HF. This should also not be a requirement but optionally automatically generated from the experiment name.

Previously, the code added the user ID and model name and passed the repository ID as a function argument. However, @maxjeblick pointed out that the following code snippet is necessary.
@maxjeblick comments Could you propagate user_id to a function argument? Within the app,
user_id = q.client["experiment/display/push_to_huggingface/account_name"] if user_id == "": user_id = huggingface_hub.whoami()["name"] is used, as the user may not have been logged in to hf hub.
When I passed the user_id as a function argument, I also passed the model name. I concatenated the two to create the repo_id variable. Do you have any suggestions on how to handle this situation?

We need to deal with two situations:

Use of your new function as a standalone upload from CLI
Use of your function from within the UI

My suggestion:
For 1., a minimum set of arguments should be needed, the rest shall be populated with meaningful defaults. Required is basically only model path. User may already be logged into HF. If so, use the following default values:

user_id from logged in user (optional organization ID for example)
model name on HF: apply hf_repo_friendly_name on the folder name of the model weights (only the last folder, not the full path) (optional custom name)
No authentication required, as already logged in (optional new token)
Safe tensor: True (optional)
device (cuda:0 by default, optional other devices such as "cpu" or "cuda:ANY NUMBER") also allow "cuda" which translates to the first device. Throw errors for any other device.

If an additional token is provided:

force a new login (it may be a new user!)

For 2, propagate all values from the dialog (user_id, experiment_path, model_name, safe tensor, device) and fall back to the same defaults as above.

Let's reuse hf_repo_friendly_name and force the name to be HF compliant.

Of course! Would you mind providing me with instructions on how to use this?

https://github.com/h2oai/h2o-llmstudio/blob/454b8dfc10a55aa2bd86ad0bd11e77e448b4d90a/app_utils/utils.py#L1980
Use this function on the model name before pushing it to HF

api_token = q.client["default_huggingface_api_token"] is wrong to pass from the form to the script. Needs to be q.client["experiment/display/push_to_huggingface/api_key"] as it could have been changed in the dialog.

I have a doubt about the difference between an api_token and an api_key, as the previous code only had an api_key. Would the value of the api_key from q.client["experiment/display/push_to_huggingface/api_key be sufficient?

Sorry for the confusion, please let me try to explain in more detail:
api_key or api_key is just a word here, my point is about the value that you are using.
q.client["default_huggingface_api_token"] stores the value that was set in the settings page. But the user may have provided a different key in the last dialog. In this case, it is available in q.client["experiment/display/push_to_huggingface/api_key"] and the former would be the wrong key. So, please use the latter.

why check for log-in? if not huggingface_hub.is_logged_in(): We probably need to force the relog if new credentials are supplied.

To avoid having users provide login credentials again, Hugging Face checks if they are already logged in. I was considering testing this code, but it may not be necessary and can be removed.

This is why the token should be optional. But if the token is provided, we need to force a relog, because the user may have changed.

@pascal-pfeiffer Could you take a look at the updated code I've made some changes to it? I'd appreciate your feedback.

Thank you for the quick changes @diegomiranda02 could you rebase the fork to newest main, please? I am seeing all the other latest changes mixed into your new code which makes it hard to read the PR.

@pascal-pfeiffer
I am currently performing the following steps. Please let me know if any corrections are necessary.
1 - I added the main repo
git remote add upstream https://github.com/h2oai/h2o-llmstudio.git
2 - Fecth the latest changes
git fetch upstream
3 - Checkout the main fork branch
git checkout main
4 - Rebase the changes
git rebase upstream/main
5 - Push the changes
git push

Thank you for the quick changes @diegomiranda02 could you rebase the fork to newest main, please? I am seeing all the other latest changes mixed into your new code which makes it hard to read the PR.

@pascal-pfeiffer
I am currently performing the following steps. Please let me know if any corrections are necessary.
1 - I added the main repo git remote add upstream https://github.com/h2oai/h2o-llmstudio.git
2 - Fecth the latest changes git fetch upstream
3 - Checkout the main fork branch git checkout main
4 - Rebase the changes git rebase upstream/main
5 - Push the changes git push

about right, yes.
You will need to force push, though.

Thank you for the quick changes @diegomiranda02 could you rebase the fork to newest main, please? I am seeing all the other latest changes mixed into your new code which makes it hard to read the PR.

@pascal-pfeiffer
I am currently performing the following steps. Please let me know if any corrections are necessary.
1 - I added the main repo git remote add upstream https://github.com/h2oai/h2o-llmstudio.git
2 - Fecth the latest changes git fetch upstream
3 - Checkout the main fork branch git checkout main
4 - Rebase the changes git rebase upstream/main
5 - Push the changes git push

about right, yes. You will need to force push, though.

@pascal-pfeiffer
After rebasing the code, I utilized the command "git push --force origin main" to push the changes.

Thank you for your help and support @pascal-pfeiffer @psinger @maxjeblick


Some recent work about topic mdoels

I find some intereseting work published in recent top conference. Can your guys include these projects into this library for convience?
The work list:
Alleviating “Posterior Collapse” in Deep Topic Models via Policy Gradient", NeurIPS 2022
HyperMiner: Topic Taxonomy Mining with Hyperbolic Embedding , NeurIPS 2022
Knowledge-Aware Bayesian Deep Topic Model, NeurIPS2022

I have inlcudes these works in our library and recommand the orginal authors of these work to reimplement their work in our framework.
The file link can be found https://github.com/BoChenGroup/PyDPM/tree/master/pydpm/example/Hybrid_GM

The issue has to be closed


[AutoPR cognitiveservices/data-plane/SpellCheck] Add cognitiveservices\data-plane\SpellCheck\readme.typescript.md

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4282

(message created by the CI based on PR content)


bugfix for templateName in dbca-create-db.rsp.<IP_ADDRESS>.j2

The DBCA creates the following error when templateName with '"'
are used:
[FATAL] [DBT-10325] Template file option is not specified.

Thanks!


Changed behaviour in Guice 7

the following thing is working in guice up to 6, but is not in 7
public class ADemo {

	public static final String CYCLIC_LINKING_DECTECTION_COUNTER_LIMIT = "CYCLIC_LINKING_DECTECTION_COUNTER_LIMIT";

	@Named(CYCLIC_LINKING_DECTECTION_COUNTER_LIMIT)
	@Inject(optional=true)
	protected int cyclicLinkingDectectionCounterLimit = 100;

	public static void main(String[] args) {
		Injector i = Guice.createInjector(new AbstractModule() {
			@Override
			public void configure() {
				super.configure();
				binder().bindConstant().annotatedWith(Names.named(CYCLIC_LINKING_DECTECTION_COUNTER_LIMIT)).to(0);
			}
		});
		ADemo demo = i.getInstance(ADemo.class);
		System.out.println(demo.cyclicLinkingDectectionCounterLimit); // should print 0
	}

}

is this an intentional change?

What imports do you use ?

right. i overlooked a javax named. will test again when i replaced these ones too

using import jakarta.inject.Named; indeed helps


add ability to set the emissions CLI flag to false

Currently, IFF the emissions CLI flag is set to true will the taro daemon be set to mint an asset with possible, subsequent emissions.  Users are currently assuming that the same field can be set to false to prescribe an asset not have subsequent emissions.  This is causing repeated user errors and is is a reasonable so assumption.
The issue will be satisfied by tarod allowing the emissions CLI field to also accept false

people want to use it
consider removing the flag and use a new term to reset

Digging in to this, having --enable_emission present in the cmd sets the internal bool to true. It can also be set with =false and =true, as expected. But a cmd using --enable_emission false sets the internal bool to true, as = is not used and --enable_emission is present.
I dug into the urfave/cli docs further and IIUC we don't have a straightforward way to reject cmds with bool flags like --enable_emission where the value is not explicitly set. We also don't have a great way to reject extra strings in a cmd, as true and false are here when not using =.
Also, right now these extra strings in the cmd seem to cause parsing issues for following flags.
For example, a command that should set the group_key and enable_emission flags:

tap-2-c-test assets mint --type collectible --name unlimited --supply 1 --enable_emission false --group_key 0349d60c6689bbbb2ffc9b30b45b96a3d6e5fd5fd01c867344c0054f6105048241
gRPC request made by CLI: (*mintrpc.MintAssetRequest)(0xc0002ec3c0)(asset:{asset_type:COLLECTIBLE  name:"unlimited"  amount:1}  enable_emission:true)

The group key is totally unset, and enable_emission is incorrectly set.

group_key is still unset even if we use = like this: tap-2-c-test assets mint --type collectible --name unlimited --supply 1 --enable_emission false --group_key=0349d60c6689bbbb2ffc9b30b45b96a3d6e5fd5fd01c867344c0054f6105048241

Once we remove false in the cmd it is parsed correctly:

tap-2-c-test assets mint --type collectible --name unlimited --supply 1 --enable_emission --group_key=0349d60c6689bbbb2ffc9b30b45b96a3d6e5fd5fd01c867344c0054f6105048241
(*mintrpc.MintAssetRequest)(0xc0002ecb80)(asset:{asset_type:COLLECTIBLE  name:"unlimited"  amount:1  group_key:"\x03I\xd6\x0cf\x89\xbb\xbb/\xfc\x9b0\xb4[\x96\xa3\xd6\xe5\xfd_\xd0\x1c\x86sD\xc0\x05Oa\x05\x04\x82A"}  enable_emission:true)

In practice, enable_emission and group_key are mutually exclusive settings, but the CLI should reject cmds that use both instead of mis-parsing them.

Draft PR has "Command with no arguments" are properly rejected
wanting to initial issuance

Other planned CLI changes re: emission - replacing enable_emission with new_grouped_asset and grouped_asset. new_grouped_asset would be used for creating a new asset with emission enabled / that will become a group anchor.
For reissuance, this would require grouped_asset and one of group_key or group_anchor. So the only valid combinations would be:
--new_grouped_asset
--grouped_asset --group_key
--grouped_asset --group_anchor
Which IMO is a bit clearer than the current flag name enable_emission. Strong thoughts @ffranr @GeorgeTsagk  ?

Will also need some RPC changes and a new type; right now we don't display if an asset added to a batch before finalization has enable_emission set.


[Incompatibility] Certain parts of the world look weird when the window is unfocused when using BlurredWindow for some reason.

I assumed the issues would only be on the title screen, yet somehow ingame is also affected (id brought this up previously on their end, decided to bring it here too)

(also sorry if you already went over this in the other issue i made on CWithoutB :p)

CW/oB is as simple, primitive even, as it gets. Therefore, I would hesitate to work on fixing actual bugs here (for example, I know for sure that it creates one very specific bug, but since it's non-obtrusive and can be triggered only in conjunction with other mods, I sacrificed it to the compatibility gods), and this particular one is certainly not this mod's problem :D
If I were to assume what's happening here, it's either that the lighting is messed up, or maybe your desktop colors are bleeding through translucent layers. Since BlurredWindow does touch some rendering logic here and there, this might be an actual problem with their implementation, which simply doesn't show itself when the window doesn't allow compositing under any circumstances (i.e., when Minecraft window uses regular, non-modified exclusive fullscreen). So, maybe this also causes those flickers when CWB is enabled?


Run with "Connection refused" error and 401 error

hi,
I am already export env vars use command like this  where run docker-compose:

export CONFIG_SERVICE_PASSWORD=1111 (other 4 variables also export)

but , when execute 'docker-compose ...' ,throw connection refused error.(all auth's client throw this error )
Caused by: org.springframework.web.client.ResourceAccessException: I/O error on GET request for "http://config:8888/registry/default": Connection refused; nested exception is java.net.ConnectException: Connection refused
I use command ping config host it's ok...
where i checked it ? thanks....

It's hard to guess without logs from config service and the service you are requested it from.
But it looks like config service is just still starting when you noticed this errors.
See https://github.com/sqshq/PiggyMetrics#notes-2


Slack大版本

多版本蓝鲸环境适配:

目前支持版本 v7

后台国际化支持:

支持中英文切换

多协议适配

企业微信
slack

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.neolei seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


refactor core, functions, pickle into single module

[x] core
[x] functions
[ ] pickle

won't be refactoring pickle


Math Exercises: Add initial math exercise support

⚠ Contains Migrations, proceed w/ caution

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[x] I followed the coding and design guidelines.
[ ] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I documented the Java code using JavaDoc style.

Changes affecting Programming Exercises

[ ] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[ ] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context
Math exercise support is to be added to Artemis. DB and server adaptions are needed in order to store and manage math exercises.
Description
This PR adds support for math exercises in the Artemis server:

Database migrations, adding tables math_exercise_details and math_submission_details.
Math exercise entities
Math exercise repository
Math exercise service
Math exercise API resource

Steps for Testing
TBD
Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress
Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Test Coverage

Can you extend the description and add some details on what makes a math exercise special (esp compared to text exercises)? This might help thinking about the code while reviewing.

Can you extend the description and add some details on what makes a math exercise special (esp compared to text exercises)? This might help thinking about the code while reviewing.

That's a good one really, already also thought of subclassing text exercises. However, I can already think about persisted fields for math exercises, which are not associated with text exercises, which makes them different (e.g. editor settings, selected feedback / verification mode, etc.)

@aplr To me this sounds like math exercises are text exercise with special properties which would make subclassing feasible and suitable.

@aplr To me this sounds like math exercises are text exercise with special properties which would make subclassing feasible and suitable.

I'll see what makes sense here. I don't know if the text blocks are needed, so to pull that into math exercises could be overhead.

I guess we should put all this behind a FF, both server and client, so it's not possible to corrupt the system w/ sending postman requests. WDYT?

@aplr To me this sounds like math exercises are text exercise with special properties which would make subclassing feasible and suitable.

I tried subclassing, but that did not really work. As Math exercises are also Text exercises, the ORM can't tell the difference, and text exercises will also be selected in queries.
Solutions:
1. Abstract Superclass
Create a abstract superclass TextBasedExercise, from which both TextExercise and MathExercise extend.
Pros: Shared fields are only defined once. Tests can be (partially) reused. Both types can still evolve independently.
Cons: Very little overlap, mostly just the entered text. Might diverge more in the future.
2. Separate entities
Leave two separate MathExercise and TextExercise.
Pros: Exercise types can evolve independently, without affecting the other.
Cons: Some duplicated code and database entities in the beginning. The relative share will reduce in the future, however.
3. Text exercise discriminator
Adding a discriminator to text exercises (text_exercise_type), w/ values math and text to tell them apart. I'd actually not do this, as there will be fields which only affect math / text exercises. Also, enum values may be only valid for one exercise type (e.g. feedback engine selection, math exercise UI settings, ...).
I'd probably choose (1) or (2), def. not (3)


Improve default chain strength in embed_bqm

Closes #300 .
Addresses the conversation in #326. After discussing with @alexzucca90, I went with an average of 1.33 and 1.5 as our default prefactor (using the formula described by @pau557 here). I didn't implement the topology dependency for reasons described here, but perhaps there are other spots that we could do that. For example, @alexzucca90 mentioned that the DWaveCliqueSampler may benefit from using a topology dependent coefficient (as opposed to an average or the current 1.5).
Also, a lot of the tests in test_embedding_transforms.py are using chain_strength=1. I wasn't entirely sure whether to change them, but can definitely do so once we're more certain of these changes.

Sorry, I know this is the PR that never ends, but probably we should also update DWaveCliqueSampler to use the new chain strength. In fact, if I am not mistaken, we can probably just remove that section and let FixedEmbeddingComposite handle it.


Test: Cleanup CIDR test on AfterEach

At the moment the cleanup of the interfaces are made in a defer, and
AfterFailed can't retrieve all the info correctly.
With this change the cleanup will happens on the AfterEach function.
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

This change is

test-me-please


[Feature] oam-runtime pod crash when webhook secret is not ready

Is your feature request related to a problem? Please describe.

When I work on #292 and e2e-test related to interacting with oam-runtime, each time set up a new vela runtime, the runtime pod always restarts several times before running stably.
As the below log from the runtime containers shows, the runtime startup crash for missing tls cert, because the secret (from cert-manager-webhook) volume is not mounted to the pod yet. The pod falls into failed phase from running phase. So we cannot use the phase of runtime pod to check it's ready or not. Even though the pod will run into stable finally once the secret volume is ready.
2020-10-10T01:32:38.978Z    INFO    controller-runtime.webhook.webhooks    starting webhook server
2020-10-10T01:32:38.978Z    DEBUG    controller-runtime.manager    non-leader-election runnable finished    {"runnable type": "*webhook.Server"}
2020-10-10T01:32:38.978Z    ERROR    vela-runtime    problem running manager    {"error": "tls: failed to find any PEM data in certificate input"}
github.com/go-logr/zapr.(*zapLogger).Error
    /go/pkg/mod/github.com/go-logr/zapr@v0.1.1/zapr.go:128
main.main
    /workspace/main.go:150
runtime.main
    /usr/local/go/src/runtime/proc.go:203

Describe the solution you'd like

Possible solution:  Add a looping check on webhook-cert-dir in /cmd/core/main.go. Only continue setuping controller manager when the secret volume is mounted successfully .

The main.go is running inside the pod that needs the secret so it's too late for the main.go to check. We need to add some logic in the vela install

fixed


Add string-width v3 and v4 to allowed dependency versions

Allow latest string-width versions.

I removed node v4 as it was causing issues but it's also isn't maintained anymore. Should consider removing v6 and (as of Jan 1st) v8.

@iarna Any chance to merge this and release? There is a vulnerability in ansi-regex that is fixed in v5 and v6 => newer string-width required:
--<EMAIL_ADDRESS>|             --<EMAIL_ADDRESS>|               --<EMAIL_ADDRESS>|                 --<EMAIL_ADDRESS>


Create Dockerfile

Create a Dockerfile so that the project can be deployed as a container on our infrastructure

@joao-paulo-parity won't existing file work?
https://github.com/paritytech/substrate-tip-bot/blob/master/Dockerfile

@joao-paulo-parity won't existing file work? https://github.com/paritytech/substrate-tip-bot/blob/master/Dockerfile

Yes, although it needs to be reviewed so I'll repurpose this issue

Done in #18


Analyzer: How to obtain the generic parameters of a ParameterElement?

Hello!
Consider the following:
abstract class Outer with _$Outer {
  const factory Outer({
    Whatever<int> innerData,
  }) = _Outer;
}

From the ParameterElement of innerData, how to obtain the generic parameters of Whatever (here int)?
parameterElement.typeParameters explicitly says that it works only for functions, and parameterElement.type doesn't seem to expose any way to read generic arguments.
I need to parse these generic parameters to fix a bug for a code-generator I maintain: https://github.com/rrousselGit/freezed/issues/128.

Actually I found the solution:
cast parameterElement.type to InterfaceType
We can then do:
InterfaceType type;
print(type.typeArguments); // [int]

Note that parameterElement.type as InterfaceType will sometimes throw a cast exception.
The getter parameterElement.type returns a DartType. You'll need to test the type to see whether it's a ParameterizedType. If it is, then you can use typeArguments to get the type arguments on the type.

Good to know.
In which situation would a parameterElement.type not be a ParameterizedType?

When it's neither a class nor a function, such as dynamic.


Mocha tests fail in GitHub Action

Not sure if this is something GitHub Actions does wrong since those tests work very well on my Mac and Windows box..
><EMAIL_ADDRESS>test /home/runner/work/destreamer/destreamer
> mocha build/test

  1) "before all" hook in "{root}"
  2) "after all" hook in "{root}"

  0 passing (2s)
  2 failing

  1) "before all" hook in "{root}":
     Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()"
     is called; if returning a Promise, ensure it resolves.

  2) "after all" hook in "{root}":
     TypeError: Cannot read property 'close' of undefined
      at Context.after (build/test/test.js:28:19)

##[error]The operation was canceled.

That's from our CI GitHub Action (which now fails). Any ideas?

it seems that puppeteer does not run, can you get some meaningful message in before or puppeteer?

Fails randomly - https://github.com/snobu/destreamer/actions
Sometimes it does come up, i think it's a Puppeteer on Docker thing but not sure where to start.

It's not. Sometimes the whole matrix (Node 8, 10, 12, 13) finishes with all greens. This could be a transient Actions failure. Let's wait it out.

Fixed somewhere down the line, allagedly the beforeAsync and afterAsync calls in mocha don't go very well with GitHub Actions so i removed them since they were unnecessary.


Allow base 4.15

The cabal file adds the constraint < 4.15 which is not necessary and should be updated accordingly (on hackage as well). Building with base 4.15 works like a charm without any further modifications.

Yes, it is.


wrong project platform is being used

An extra env variable Platform is being sent by collectDevBatVars function and it is causing the "test" project build to fail.

**********************************************************************
** Visual Studio 2019 Developer Command Prompt v16.10.0
** Copyright (c) 2021 Microsoft Corporation
**********************************************************************
[vcvarsall.bat] Environment initialized for: 'x86_x64'

C:\Program Files (x86)\Microsoft Visual Studio\2019\Community>echo %Platform%
x64

C:\Program Files (x86)\Microsoft Visual Studio\2019\Community>

test if we need to set platform when host and target kits are different?
test building cpptools using just 64 without setting platform (setting host and target the same)

I have no object to remove Platform, I am just telling where the Platform comes from

Thanks @lygstate, we just wanted to make sure removing it does the right thing for us, so these are a few notes @elahehrashedi wrote while we were discussing it this morning.  We were also hoping it doesn't break your fix for msvc2015 detection.


Add FreePass BidAdapter docs

🏷 Type of documentation

[x] new bid adapter

📋 Checklist

[x] Related pull requests in prebid.js or server are linked -> https://github.com/prebid/Prebid.js/pull/9827
[x] For new adapters check submitting your adapter docs

@bretg Thank you for the initial review. I've updated capabilities information and parameters table (none required). Please have another look if you get a chance.


Add manjaro as a supported distro

Add Manjaro as one of the supported distros.

Manjaro support was added!


Trivy &| Grype

https://github.com/anchore/grype

Closing for now. If this becomes necessary in the future, we will reopen.


[Textfield] Count remaining characters

This pull request replaces the [number] characters maximum message with a count down of how many characters can be entered.
Addresses MAR-3387
Before

After

this looks fine, I am just wondering if it is worth while to show it by default instead?
it looks like where this is configured to show is basically everywhere.

I've removed the prop and set it to display by default.


Handle tests with NIST data as integration tests

Unsure how best to handle these tests, but I think separating them is a good start to not have the data in the repo per #195.
Test suite will rely on environment variable, "STATRS_NIST_DATA_DIR" set as const NIST_DATA_DIR_ENV, to specify path where the files, some listed below, are located.

lew.txt
mavro.txt
numacc1.txt
...

Note: I am not married to the usage of an environment variable or its name.

I like the approach. Maybe add a helper script (bash?) to download the NIST files? Downloading the files manually could be a bit tedious.

We didn't distribute the integration tests with the last release, but if we did, would we want to share the script along with these tests?

I would logically see the script as a part of that test suite, so yeah.

Will add to CI in another PR, any feedback on this one?

I tried it on my machine and the script looks good. I think the environment variable works well, no reason to overthink this for a test IMO.
Few ideas to make the experience a bit smoother:

Add tests/*.dat to the .gitignore so the downloaded files don't show up in git
Add a bit of output to the script (e.g. Downloading <file>..., Processing <file>...) so we know what's going on. This gets more important when .gitignore includes the data files
Short comment (top of the nist_tests.rs) that explains how you're intended to actually run the tests would be nice. cargo test -- --include-ignored nist is not obvious IMHO
Use the current working dir when STATRS_NIST_DATA_DIR is unset. That'd make it easier to cd tests; ./gather_nist_data.sh; cargo test without requiring the env variable

Thanks for the feedback, think it will be clearer and nicer. Opted for default target directory to be tests since manifest and ignore target it and set some sense of working directory.


Fixing typo in settings.gradle section of README

Small typo in the settings.gradle section of the README.md file.

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

Thanks, @CRamsan!


Other tree types giving invalid resources

Other tree types giving invalid resources (spruce logs).
I think this was partially implemented new wood type that got cut from the last update, and some trees are giving a resource that doesn't have the assets added for it yet.
Can just wait until that content is finished and added.

Fixed for update 8. Other resource types still need implementing.


Add TorchPackedSequenceEqualityComparator

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

Comparison is base (0ab46c3) 100.00% compared to head (9fe7542) 100.00%.

:exclamation: Current head 9fe7542 differs from pull request most recent head deb0c0c. Consider uploading reports for the commit deb0c0c to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #387   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           62        62
  Lines         2181      2193   +12
  Branches       384       384
=========================================
+ Hits          2181      2193   +12

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


update ci-label gha for dependabot skip changelog

🚀 Pull Request
Description

This pull-request updates the labeller to automatically add the skip changelog label to dependabot generated pull-requests.

@tkoyama010 Are you happy to approve the PR?


Update decoration on value changes

I would like to give user feedback similar to an error but still allow the form to validate. One example of that is when input like a password is weak (but acceptable), give the user a warning or help.
Since validators and errors block saving a form, I thought to use helperText. To get the state updating on the helperText, the closest I've come is wrapping the ReactiveTextField with its own ReactiveStatusListenableBuilder.

Is this an efficient way to get a live updates to a text field's value as the user types?
Ideally, this helperText would be updated on the same timing as validators. The closest I can get is to watch for dirty and valid. In the code below, the unwanted behavior is that when the user is typing for the first time, the helperText appears from typing characters 4 to 7. (Watching 'touched' didn't work; it didn't trigger until the user reentered end edited the field.) What can I listen to for updating when focus is lost or when the user reenters and edits, just as it is with validators?

import 'package:flutter/material.dart';
import 'package:reactive_forms/reactive_forms.dart';

class ReactiveFormsLiveHelp extends StatelessWidget {
  const ReactiveFormsLiveHelp({Key? key}) : super(key: key);

  FormGroup get form => fb.group(<String, Object>{
        'name': FormControl<String>(validators: [Validators.required]),
        'password': FormControl<String>(validators: [Validators.minLength(4)]),
      });

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Testing Reactive Forms')),
      body: ReactiveFormBuilder(
        form: () => form,
        builder: (context, form, child) {
          return Padding(
            padding: const EdgeInsets.all(8.0),
            child: Column(
              children: [
                const SizedBox(height: 10.0),
                ReactiveTextField<String>(
                  formControlName: 'name',
                  decoration: const InputDecoration(
                    labelText: 'Name',
                    border: OutlineInputBorder(),
                  ),
                ),
                const SizedBox(height: 10.0),
                ReactiveStatusListenableBuilder(          // FOR GETTING STATE UPDATES
                  formControlName: 'password',
                  builder: (_, control, __) {
                    return ReactiveTextField<String>(     // TO THIS CONTROL
                      formControlName: 'password',
                      decoration: InputDecoration(
                        labelText: 'Password',
                        border: const OutlineInputBorder(),
                        helperText:                       // AND GIVING USER FEEDBACK
                          (control.dirty &&
                           control.valid &&
                           control.value.toString().length < 8)
                              ? 'Recommend using a password of at least 8 characters' : null,
                        helperStyle: const TextStyle(color: Colors.deepOrange),
                      ),
                      validationMessages: (errors) => {
                        ValidationMessage.minLength: 'Password must be at least 4 characters',
                      }
);})]));}));}}

I know this isn't great password control, but it shows my case. Other examples are to encourage users not to leave a field blank or to suggest a longer note.
Thanks for any help,

Check https://pub.dev/packages/reactive_fancy_password_field

Otherwise - you’ll have to create your own control with whatever bell and whistles you need and then write binding to the reactive forms

@tcpdev any feedback?

Thanks for the direction. Your answer kept me going to what looks like a workable solution. The fancy password showed well how two widgets may interreact, but since I'm live updating the helperText when the value of that same control is updated, I think I need the self-referencing ReactiveValueListenableBuilder - as in my code above.
Not finding a way to link into the validation event timing, I wrapped in a Focus widget in combination with control.touched to prevent the unwanted behavior of showing helperText when the user is doing the initial first entry.
Thanks

@tcpdev any update?


Not Up To Date

There are things in this doc, like "Save As", that don't exist anymore.  And doing the "Save a Copy" just gives an error.

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

ID: 719feeea-d688-78bf-fef5-8b58c2fb047f
Version Independent ID: 78793fa4-e2e2-7113-3652-b1165ee92145
Content: Monitor usage metrics in classic workspaces - Power BI
Content Source: powerbi-docs/collaborate-share/service-usage-metrics.md
Service: powerbi
Sub-service: pbi-collaborate-share
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Thanks for your comment, @chrislfish . This issue has been assigned to the doc author for review.

Hi @chrislfish ,
I see that this Git issue is a bit old, so I'm going to close it.  I hope that you were able to get your questions answered. If not, please revisit the article since it has been updated several times since this Git Issue was submitted.  Another option is to submit your question to the Power BI community (community.powerbi.com).
All the best,
the DOCS team


Parse dylib_commands and dylibs for mach-o

I plan on implementing some more mach-o parsing, but for now starting with dylibs and getting the information from them. 😸
Features implemented:

dylib_command parsed
dylib parsed
total number of dylibs in mach-o

Tests implemented for multiple executables:

dylib count
dylib names (paths)

After speaking with @plusvic, will port this PR over to rust and continue new features for mach-o on yara-x.
@plusvic, should I close this PR or do we want to still have open for review and possible merge given its already been written? :smile_cat:

The rust equivalent of this was merged into yara-x with https://github.com/VirusTotal/yara-x/pull/43

Closing this as progress is much heavier on the YARA-X side for Mach-O 😸


SyntaxError: Cannot use import statement outside a module

Version
nuxt-i18n: v6.15.1
nuxt: 2.11.0
Nuxt configuration

[x] universal
[] spa

Steps to reproduce
Update nuxt-i18n from v6.5.0 to v6.15.1
run npm run dev or npm run build
What is Expected?
Project should build without errors.
What is actually happening?
Getting this error
(node:6048) UnhandledPromiseRejectionWarning: ...\node_modules\nuxt-i18n\src\index.js:1
import { resolve, join } from 'path'
^^^^^^

SyntaxError: Cannot use import statement outside a module

fixed by deleting node_modules and reinstalling


Feature Request: Custom Query for Teradata

I am working on a project where there has been some customisation around how Teradata stores query log and schema information, making the queries here and here a blocker to being able to use data hub.
Would it be possible to add two fields to the configuration that would allow the user to provide custom queries? If so, I am happy to do the work and file the pull request. I see that the queries are parameterised so that would need to be mentioned in the documentation.
If this is possible, which option would be preferable for adding these fields?

Add two required fields with the existing queries as the default values. This would make the queries visible in the documentation, although they are long.
Add two optional fields that use the existing queries if the custom queries are not provided.

Seems like a reasonable idea as in the past a number of Teradata customers have used their own log archive table structures rather than the PDCR log archive service provided by Teradata.  These days, PDCR and Metric_Service are installed by default on Vantage Enterprise and Vantage Lake respectively.
I think the implementation here should be use the existing queries by default - the DBC views are on every implementation - and these optional queries be used in addition when specified.  I will defer to the Acryl team as to the best method of implementation, although my preference is a parameter to enable, and another parameter containing the query.
It should also be expanded to add in PDCR table queries and Metric_Service table queries, which could be enabled by parameters in the recipe configuration, as these are known table structures but optional implementations, and this parameter would be used to activate the additional queries which could be placed in the existing Teradata python module.
I can help providing the PDCR and Metric_Service queries, if you are actively making the change to support your request.

In the end I setup a monkey patch for the client because they have a highly customised setup and I hadn't heard back from Acryl. If someone else is dealing with this in Teradata feel free to contact me and I'll help you with the issue.

Ok. We are working with Acryl on improving the Teradata integration, and I may circle back to you on this later.
If you have any other Teradata requests or issues, please log them so that we are aware.


persistence : we need to know when starting which recipe are already pulled

When enabling/disabling the AI-Studio or the Podman-Desktop application, we probably need some kind of mechanism to persist the information of what recipes are already pulled.
We can probably have some kind of json saved somewhere, what do you think @jeffmaury

Can't we rely on the git repo presence and pod state ?

We can I guess

The how about this https://github.com/projectatomic/ai-studio/issues/234 what if the user want to provide either a git url, or a url folder

@jeffmaury then how about this #234 what if the user want to provide either a git url, or a url folder

I don't see the issue there as what we're trying to handle is the case where the repo is already download but on a different commit so I would say the git metadata would allow us to handle it properly


support block hash query

support both direct link (e.g. /block/hash) and search query.
etherscan.io already support /block/hash direct link like as following
https://etherscan.io/block/0x6ffdd1137835ccf41a28e753e64b99afd780ab53a3b791e433f20feb24214007
but etherscan.io does not support search query.

Thanks for your contribution! This works great!

Your change is now live on http://etherhub.io
Thanks again!


Sync of L515 and D435

Hi! I am wondering whatever it's possible to do hardware sync between L515 and D435? I saw a whitepaper on multiple L515 https://dev.intelrealsense.com/docs/lidar-camera-l515-multi-camera-setup and multiple D435 but it's unclear to me what to do if I want to connect them together. Any code examples/wiring schemas much appreciated!
Thanks!

Hi @zapaishchykova ,
Hardware sync is designed to work between the same cameras not across different models.
Thanks


Revert "Gracefully delete pods from the Kubelet"

Reverts GoogleCloudPlatform/kubernetes#8641
After merging #8641 latency metrics spiked from ~2s to ~16s during density e2e test. I have verified locally that reverting this PR solves the problem. It seems that the problem begins when we try to delete RC. All metrics suddenly spike to O(seconds). This includes simple operations like updating a node:
INFO: WARNING Top latency metric: {Verb:LIST Resource:pods Quantile:0.99 Latency:16.170251s}
INFO: WARNING Top latency metric: {Verb:LIST Resource:pods Quantile:0.9 Latency:11.774167s}
INFO: WARNING Top latency metric: {Verb:LIST Resource:pods Quantile:0.5 Latency:5.656098s}
INFO: Top latency metric: {Verb:DELETE Resource:pods Quantile:0.99 Latency:3.197476s}
INFO: Top latency metric: {Verb:PUT Resource:nodes Quantile:0.99 Latency:2.425735s}

Please merge it ASAP to fix jenkins e2e scalability job.
@bgrant0607 @smarterclayton @wojtek-t

e2e is failing, also.

If the density tests are spiking it may be a result of the use of TTL (and an issue on the etcd side), an issue with the design of the density test (graceful deletion means deletes take longer), or an unexpected server interaction.
How do I run the density e2e test directly?  If there's not an obvious fix I'm probably out of time to devote to this issue (and the risk is probably larger than we expected), so we'll probably have to drop it from 1.0z

On Jun 2, 2015, at 5:51 PM, Brian Grant<EMAIL_ADDRESS>wrote:
Merged #9143.
—
Reply to this email directly or view it on GitHub.

The other option (which I like better) is to set the default to 0 (on the apiserver) and the minimum on the kubelet to 30s, which lets us selectively test this (opt in).  The increase in minimum on the nose may result in larger numbers of failed pods (scheduler is more likely to double schedule a constrained resource or to cause OOMs on restart.  Feature flagging the behavior would make it easier to validate any etcd problems as well.

If TTL is the culprit I think that this will push this feature out of 1.0 as well - switching to create a graceful expiration controller is too large a change.

Although...


Solve failure in the automatic download in #3

Failure automatic issue was detoured by providing a specific filename in the personal repository (see https://github.com/LimHyungTae/kimera-multi-vocab)
It generally works as long as I don't change the repository to be private.
In addition, messages were not printed in the command line when we command catkin build, so I removed all the STATUS in the message, and fixed minor typo (ie., vocabularly- > vocabulary)

@yunzc, in addition to change directory, I also transferred the repository here: https://github.com/MIT-SPARK/kimera-multi-vocab.


[Hardening aspect] CoCo guest interrupt handling

Problem
The below is TDX specific:
Untrusted VMM can inject both non-NMI interrupts (via posted-interrupt mechanism) or NMI interrupts. However, TDX module does not allow VMM injecting interrupt vectors in range 0-30 via posted-interrupt mechanism, which drastically reduces the exposed attack surface towards the untrusted VMM. The rest of interrupts are considered controlled by the host and therefore the guest kernel code that handles them must be audited and fuzzed as any other code that receives malicious host input.
Solution
So far we have only address int80 case: i.e. we are making sure that a host/VMM cannot inject int80 interrupt for the guest (such case will be detected and guest will panic). Please see https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=55617fb991df535f953589586468612351575704 for details.
The rest of interrupt handlers (above vector 30) has been audited and no security implications has been found except for the issue discussed in https://github.com/intel/ccc-linux-guest-hardening/issues/142

Note: we have not done any fuzzing on the interrupt handlers.


Add spark-home to config file

SparkLaunch will now look at the config file for a spark-home parameter before falling back to the SPARK_HOME environment variable.  I also converted the submit command construction from a string to a more robust Seq of strings.

Fixes #69.

Codecov Report

Merging #73 into develop will increase coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop      #73      +/-   ##
===========================================
+ Coverage    86.94%   86.97%   +0.02%
===========================================
  Files           23       23
  Lines          452      453       +1
  Branches        23       23
===========================================
+ Hits           393      394       +1
  Misses          59       59

Impacted Files
Coverage Δ

...m/sparktc/sparkbench/sparklaunch/SparkLaunch.scala
70.37% <100%> (-2.05%)
:arrow_down:

...arktc/sparkbench/sparklaunch/SparkLaunchConf.scala
88.88% <100%> (+1.01%)
: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 236ee3e...a9221b8. Read the comment docs.


Unable to access environment variable within action code

Expected Behaviour
When setting "WSK_AUTH_SECRET" in .env, then running aio app deploy, the app should be deployed with the value of the env variable and when running 'npm run e2e' all 13 e2e test suites should pass
Actual Behaviour
The key being deployed is the plaintext from manifest.yml: '$WSK_AUTH_SECRET'.  9 e2e test suites are failing when the value of WSK_AUTH_SECRET is set to anything but "$WSK_AUTH_SECRET"
Reproduce Scenario (including but not limited to)
This is a relatively normal Firefly Deployment.  I am using Windows 10 and Powershell inside of VS code, but I've reproduced the same result in cmd, git shell, mingw, and WSL ubuntu all with the same results.
Steps to Reproduce

Clone Repo: https://git.corp.adobe.com/kelkingt/mkto-flow-lookup
Create new Firefly Workspace w/ IO runtime and download credentials
Add a value for WSK_AUTH_SECRET in your .env file, e.g. "WSK_AUTH_SECRET=some_secret"
Use aio app deploy
run 'npm run e2e'

Environment Info
  System:
    OS: Windows 10 10.0.19043
    CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
    Memory: 8.01 GB / 15.72 GB
  Binaries:
    Node: 14.15.5
    Yarn: Not Found
    npm: 7.14.0
  Virtualization:
    Docker: 20.10.7
  npmGlobalPackages:
    @adobe/aio-cli: Not Found

Sample Code that illustrates the problem
Logs taken while reproducing problem

@kelkington could you explain

What you need this secret for?
Where in your code are you trying to use it?

@meryllblanchet

I'm trying to set the value the require-whisk-auth annotation so that I can enable api-key based authentication
This is an API service, so it's only used by my test code and my manifest in this repo to run tests against the service: https://git.corp.adobe.com/kelkingt/mkto-flow-lookup/blob/9a519f920f91eae58f6ef07417280bb3aec9c316/test/lib/testUtils.js#L40 https://git.corp.adobe.com/kelkingt/mkto-flow-lookup/blob/master/manifest.yml

@kelkington You are trying to substitute an environment variable in an annotation. We follow the Openwhisk manifest.yml spec, which does not do environment variable substitution outside of input keys: https://github.com/apache/openwhisk-wskdeploy/issues/741

Closing this one, as @shazron's comment is solving the issue. @kelkington please reopen another issue in case you need further help.


Auto Dependency Updates ...

thank you! generally speaking it's okay to leave versions as they are unless there's a major change — the default behaviour is to install the highest version that matches the semver range, so e.g. ^2.1.4 would allow anything lower than 3.0.0


if exist(..)  is not executing as intended.

code:
`https://www.lowes.com
timeout 15
logged = exist('//*[@id="account-name" and not(contains(text(), "Sign In"))]')
echo logged
if exist('//*[@id="account-name" and not(contains(text(), "Sign In"))]')
echo Signin
else
echo Signout
if logged
echo Signin
else
echo Signout`
printed log:
https://www.lowes.com - Lowe’s Home Improvement
true
Signout
Signin
https://www.lowes.com/ - Lowe’s Home Improvement
FINISH - automation finished - 21.4s
Please tell me where I am wrong, thanks!

Hi @EvanYiLi you can try the following
https://www.lowes.com/
read //*[@id="account-name"]/text() to status
if (status!="Sign In")
	echo Signin
else
	echo Signout

Closing but please ping back if this is still relevant or an issue.


[stable/prometheus] Pushgateway deployment strategy type

Signed-off-by: Jeff Thompson<EMAIL_ADDRESS>
Is this a new chart
No
What this PR does / why we need it:
To allow the customisation of the pushgateway deployment strategy.
Which issue this PR fixes

fixes #16873

Special notes for your reviewer:
This replaces my original PR which I screwed up by rebasing incorrectly.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@jtnz
Would you mind to resolve the chart conflict please? Thank you.
stable/prometheus/Chart.yaml

/assign

/assign
/ok-to-test

/lgtm


belongsToMany({resource: 'batches', as: 'course'}) type signature does not exist

TypeError: jagql.Joi.required(...).belongsToMany is not a function

When will it be added?


please add to ekstensionn tachiyomi

Source name
Otascans
Source link
https://yurilab.my.id/
Source language
Indonesia
Other details
18+/NTSW=yes
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

please add


Hi! Thanks for your excellent work! I would like to know what directory structure the dataset should be stored in.

Hi! Thanks for your excellent work! I would like to know what directory structure the dataset should be stored in.

Hi! Thanks for your excellent work! I would like to know what directory structure the dataset should be stored in.

Thanks for your attention. We have updated the code for extracting patch features and the directory structure of the dataset. We hope it will be helpful to you.


server_connection_api_test fail->timeout on unittest-asserts-release-mac-arm64

build
logs

Also pkg/dds/test/devtools_server/instance_reuse_test and pkg/dds/test/devtools_server/server_connection_vm_service_test
from https://ci.chromium.org/ui/p/dart/builders/ci.sandbox/pkg-mac-release/31250/overview

//cc @bkonyi


Support for attributeSets

Adds support for attributeSets
Warning! attributeSets cannot be DELETED or RENAMED, they are permanent. (see here)
Related:
https://github.com/hashicorp/terraform-provider-azuread/issues/913

Hi @tiwood, just checking in to see if you're able to continue work on this PR? Thanks!

This project is being deprecated, and PRs are no longer being merged here.
Please refer to the new Microsoft Graph SDK published by HashiCorp, which inherits many of the same paradigms as this SDK, and which is now exclusively used in the Terraform AzureAD provider.


vc/qemu: add mutex to qmp monitor channel in qmpSetup()

Solve possible race condition in qmpSetup()
Fixes: #2139
Signed-off-by: Yves Chan<EMAIL_ADDRESS>

/test
thanks @YvesChan

Thanks @YvesChan!
Is there any way we can add a test (either a unit test here, or a test in https://github.com/kata-containers/tests) to give us some confidence the race has been fixed?

As I mentioned above, now the containerd-shim-kata-v2 process will hold the qmp.socket all the time, so there's no chance to reproduce the race condition in current runtime version. But it's not very convenient for user to debug the qemu issue if they cannot access qmp.socket since it's held by shim. So it's just better to make qmpSetup() more robust. What if we change the qmp connection to ephemeral(free after used) in the future? who knows :)

/test

/test

/test

/test

This is now ready to land but I'll let @bergwolf click the lovely green button as an ack to the comment about shutdown.


Adds code to configure MySQL instances and create cluster

WIP

@DnPlas @shayancanonical, should we close this PR then?

Since there are significant differences between @paulomach's implementation and this PR, I am going to close this PR. Thanks @DnPlas - we really benefited from using this PR as a reference


500 Internal Error when i try to capture screenshot of element

Hello everyone!
I try to make in-house framework for GUI testing and this the problem I have encountered:
When i try to capture screenshot of the element (or group of elements), server sends 500 internal error
==========================================
GET /session/53C6FE05-D076-48C7-9499-DDDF337DAC38/element/42.7149364/screenshot HTTP/1.1
Accept: application/json
Accept-Encoding: identity
Content-Type: application/json;charset=UTF-8
Host: <IP_ADDRESS>:4723
User-Agent: selenium/3.141.0 (python windows)

HTTP/1.1 500 Internal Error
Content-Length: 133
Content-Type: application/json

{"status":13,"value":{"error":"unknown error","message":"An unknown error occurred in the remote end while processing the command."}}
==========================================

But!
Look at this server responce
==========================================
GET /session/499FA856-097B-451C-BC2B-B56898F06A6B/element/42.2889676.4.-2147483562/screenshot HTTP/1.1
Accept: application/json
Accept-Encoding: identity
Content-Type: application/json;charset=UTF-8
Host: <IP_ADDRESS>:4723
User-Agent: selenium/3.141.0 (python windows)

HTTP/1.1 200 OK
Content-Length: 1818
Content-Type: application/json

{"sessionId":"499FA856-097B-451C-BC2B-B56898F06A6B","status":0,"value":"iVBORw0KGgoAAAANSUhEUgAAAXEAAAAYCAIAAACDaVrVAAAAAXNSR0IArs4c6QAAAARnQU1BAACxjwv8YQUAAATESURBVHhe7dxdSKpnHADw0tQ+WB8z08w6asVqsOiDFcEuNqqbg220Ilqcm4qyj0NFN6273WwQ0ceFO1At2E0rYTcFFSxbrJsOq7MF5ywvoiZqMjQzSyVNc3/04cCx0td4Paj8fxdvz8f/oXzw/b/P+1ViSUlJAkII0YRBfiKEEB0wpyCE6IQ55X3j+pAKQnHnnusp19fXbDabwaAh3TCZzNTUVFJ5l8Ph8Hg8pEJBWloah8O5vb29vLyELbTAX5ieng5bp9Npt9v9YdGvvr4etiq [...]

==========================================

This request works!
I have researched the issue and what I found:

request 1 (with 500 error) have elementID = 42.7149364
request 2 (with 200 OK) have elementID = 42.2889676.4.-2147483562
For example: in Appium Inspector it calls RuntimeId:

UPD. Both of these elements can be find in page_source and can interact with it (click, move_to, etc.)
This my pseudo code example:
driverDesiredCaps = {
    'platformName': 'Windows',
    'app': 'C:\\Path\\To\\MyApp.exe',
    'automationName': 'Windows',
}

winAppDriverServerUrl = 'http://<IP_ADDRESS>:4723'

driverSession = webdriver.Remote(command_executor=winAppDriverServerUrl, desired_capabilities=driverDesiredCaps)

## ELEMENT LOCATORS ##
myAppPanel_classname = "myAppPanel"
myAppPanel_xpath = f"//*[@ClassName='{myAppPanel_classname}']"

myAppPanel = driverSession.find_element(By.XPATH, myAppPanel_xpath)
myAppPanel.screenshot('myAppPanel.png')

driverSession.quit()

Why does request work in one case (with long ID) and not in another (with short ID)?
Please, help me to solve this problem
My stack:

Python (+ PyTest but not implemented yet)
Selenium==3.141.0 (because JWP protocol needed)
WinAppDriver (1.2.2009.2003)

Hi @AnatolyRyzhakov ,
Win app driver does not support direct element image capture, therefore what we do is we capture the whole screen shot and then we crop out the element image using its dimensions.
look at the code below:
public void highlightElement(WebElement element) throws ErrorMessage {
try {
// Get the bounds of the element
Point location = element.getLocation();
Dimension size = element.getSize();
        DesktopParamClass paramClass = new DesktopParamClass();
        //appending screenshot
        WindowsDriver  driver = ((WindowsDriver )(paramClass.getExecutionParamValue(DesktopParamKeyName.WIN_APP_DRIVER)));
        BufferedImage fullImage = ImageIO.read(new ByteArrayInputStream((
				(TakesScreenshot) driver).getScreenshotAs(OutputType.BYTES)));

        new ImageProcessing().highlightElementOnOnImage(fullImage, new DesktopImageProcessing().getHighlightColor(),
        		location.getX(), location.getY(), size.getWidth(), size.getHeight());

        paramClass.putExecutionParamValue(DesktopParamKeyName.BUFFERED_SCREENSHOT, fullImage);

    } catch (Exception e) {
        e.printStackTrace();
        throw new ErrorMessage("Unable to get element rect for highlights :" + e.getMessage());
    }
}

Also we have developed a tool regarding the same you can look into that as a solution over here 'https://soliterata.com/'

@HowTurnRight Hello! Sounds interesting. Thanks for the answer I will try it later


Dutch Language Files

I’ve translated the English language files to Dutch.

Hi @arietimmerman, Sorry for my late reply. Thank you so much for these translations, Fantastic work!
Unfortunately, I can't merge this directly due to the additional recent commit (Nescio support + NewUser command). I could merge the commits manually but, likely due to your local git email settings, your commits aren't linked to your github profile so you may not get proper credit.
I won't manually merge without confirmation from you due to this. Feel free to open a new full request without the latest commit or with the commits updated with new details.

Apparently I accidentally pushed some other changes to this pull request. I'll try to open a new - clean - shortly.