Secondary CIDR outing with Pod IP.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

Please describe your question here
Using vpc-cni-custom-networking example PODs from secondary CIDR is no outbound with NodeIP. Why ?
Docs referenced saying that outbound will work with NodeIp... :(
Provide a link to the example/module related to the question
https://github.com/aws-ia/terraform-aws-eks-blueprints/tree/main/examples/vpc-cni-custom-networking
Additional context
As the example I created 10.0 and 10.99 subnet.
I create a pod and everything is working and running as 10.99.x.x
I created an EC2 instance on a peering VPC and all requests receiving there is 10.99.x.x from pods.
Thanks

I create a pod and everything is working and running as 10.99.x.x
I created an EC2 instance on a peering VPC and all requests receiving there is 10.99.x.x from pods.

Since the pods run in private subnet in 10.99.x.x range; what's the issue that the receiving end sees 10.99.x.x as the source? Note that traffic do not go out via the NodePort.

for pods on extended subnets, they will go out to the subnet with that same IP CIDR.
resources in the same VPC will need to allow traffic from that CIDR in their SGs.
the other option is to have a NAT GW and route in the routing table to make all traffic go over it (expensive)

When custom networking is enabled, the VPC CNI creates secondary ENIs in the subnet defined under ENIConfig. The primary ENI is not used by custom networking.
What outcome are you looking to achieve?


[Bug]: Wrong timezone used in timestamps received from barman-cloud-backup-list

Is there an existing issue already for this bug?

[X] I have searched for an existing issue, and could not find anything. I believe this is a new bug.

I have read the troubleshooting guide

[X] I have read the troubleshooting guide and I think this is a new bug.

I am running a supported version of CloudNativePG

[X] I have read the troubleshooting guide and I think this is a new bug.

Contact Details
No response
Version
1.23.1
What version of Kubernetes are you using?
1.27
What is your Kubernetes environment?
Self-managed: k0s
How did you install the operator?
Helm
What happened?
In some cases when k8s worker node have non-UTC timezone set timestamps displayed in backup resource could be inconsistent
Cluster resource
No response
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

This will be solved when #6252 will be closed


Chrome hasn't updated

I noticed that the latest version of Chrome at the time of writing - 87.0.4280.141 - was released on 2021-01-09 and that the version on Scoop is still at 87.0.4280.88.
Hopefully this isn't complicated to solve.
Thank you in advance.

I tried to install google_chrome_portable but failed with version 87.0.4280.141，so I guess the address is not available

I tried to install google_chrome_portable but failed with version 87.0.4280.141，so I guess the address is not available

Fix in #5547

Fix in #5547

The manifest is at 91.0.4472.101, the new version ist 91.0.4472.114.
This time there is an important bugfix in chrome, so it would be important to update the version. The url for checkver is still out of date and there are alternatives. I think it makes sense to switch or integrate the scripts for checking the version in scoop.

The manifest is at 91.0.4472.101, the new version ist 91.0.4472.114. Right now .124 .
This time there is an important bugfix in chrome, so it would be important to update the version. The url for checkver is still out of date and there are alternatives. I think it makes sense to switch or integrate the scripts for checking the version in scoop.

With the current upstream, Chrome can be updated with some delay.

extras Chrome isn't update to 93 but still 91 and PR #6070 seems stop for a while. Is any plan to collect issue like 42wim does?

@guitarrapc I have already fixed my api. An auto-update will work and update Chrome to the newest version.

@Lutra-Fs Thank you, Waiting for merge to master <3

The collaborators have no response to these two PRs. I also contact SukkaW as I create a PR to fix his end. But he doesn't want to maintain the repos now...


Update to pyro 1.8.1

I've started the process of updating the dependencies and added an example for classification.
Things changed to enable move to pyro-ppl 1.8.1:

[x] fixed reference to deep_setattr in pyro.infer.autoguide.guides.deep_setattr
[x] updated reference for torch.cholesky to torch.linalg.cholesky
[x] added trange dependency for fit in svi to bring it in line with mcmc behaviour from pyro tqdm is a pyro dependency, so no new dependency is added to TyXe.
[x]  figured out why vs in tests/test_bnn.py:69 has changed value (to look more like old vs + torch.eye)
[x] fixed error in example/classification.py for likelihood.data shape
[ ] check that all examples still work (some don't work for me with pyro==1.4.0)

bnn.py:207 triggers
ValueError: at site "likelihood.data", invalid log_prob shape
  Expected [20], actual [20, 20]
  Try one of the following fixes:
  - enclose the batched tensor in a with pyro.plate(...): context
  - .to_event(...) the distribution being sampled
  - .permute() data dimensions

I haven't found what it refers to as when i step through it, all pyro.sample statements are in plates. The relevant example in pyro eight_schools has also been updated for the newer versions of pyro to include these pyro.plate contexts.
I'd appreciate input on the approach for the classification model as well as help with fixing these last two errors.

I have made a few decisions regarding the new names of the pyro parameters. It adds net. to the parameters, but not for all methods invoke.
The vcl.py example fails when the prior is updated to the DictPrior. Either a workaround can try names or net. + names or the example needs more adjustments.
There is also a script to test all the examples (bar two) (but not all options) to make sure they run for future development. Please have a look and let me know where you disagree with the direction. And any help on the bnn test would be much appreciated.

So a bit more digging and it turns out that net_guide.scale_tril returns a unitriangular matrix rather than a lower triangular in the newer versions of pyro. This is not due to the change from  torch.cholesky to torch.linalg.cholesky as I've checked and they return the same as expected and documented.
This can be remedied by multiplying the result with the scale vector, recreating the old triangular matrix. I don't know where else this change would have implications and I've not found the part in the pyro changelog where this change was documented.
Tests now pass. Last remaining issue is the vsl example's dictprior naming and I'm tempted to leave that for now and fix it after the PR.

I've done my best to address your comments.
See the comments in the files for the remaining issues.
I'll contribute the github actions for release to pypi and testing after this PR, if you like.

Left you a few comments. Thanks again for the updates. The big TODOs left at this point for me are:

[ ] Remove the new examples to submit them in a separate PR. I'd prefer to keep the changes minimal and focused in a single PR to make it easier to rollback in case anything does break.
[ ] Remove the examples execution script for the same reason for now. Happy to set up continuous integration after this PR is merged.
[ ] Remove bits of code that account for the possibility of different pyro versions and assume 1.8.1.
[ ] Add deprecation warnings for deep_getattr and deep_settattr in util and use the pyro functions in the rest of the code base.
[ ] Solve the prefix issue for the continual learning case. This one might require a little bit more discussion.

Let me know if I forgot anything :)
Btw should I have access to your branch? I think there's a github option to set up PRs in that way. In that case I can obviously make some code changes as well.

Left you a few comments. Thanks again for the updates. The big TODOs left at this point for me are:

Thanks a lot, i'll see what I can chew through in the next few days, most are fairy minor. I agree with all your points.

Btw should I have access to your branch? I think there's a github option to set up PRs in that way. In that case I can obviously make some code changes as well.

the branch isn't protected, but I guess the way GitHub works you'd have to make PRs against my branch. If this turns into a continuing effort, I guess I could be added as a collaborator on the original repo and then make branches there. But if you know of a setting I'm missing let me know, I'm not that familiar with GitHub.

Ok I think I've adressed everything i can. Only the deprecation warnings are left adn then the actualy push to PyPi but i guess they  are for after the PR is merged. Let me know if there is anything else I've missed.

Thanks could you just also remove the regression example?
Also for the prefix, I meant having it as an argument on the get_detached_distributions of the AutoNormal autoguide class. So if you could move that functionality there and revert the changes to DictPrior that would be great. Sorry if I hadn't been clear/am being pedantic, but I think it makes a lot more sense to pull these distribution names correctly out of the guide in case anyone wants to use them downstream. And have a look at how pytorch handles prefixes, there's no need to include a "." in the string just add it when concatenating it with the name -- makes calling the function easier by one character :)
Other little bits to clean up (I can handle those as well after merging):

[ ] remove the --test argument in the vcl.py example and replace it with a --num-epochs one.
[ ] remove the comments in the multivariate_svi test.
[ ] revert the change from range to trange in the bnn fit loop.
[ ] change occurrences of util.deep_setattr to pyro.infer.autoguide.utils.deep_setattr.
[ ] revert removal of deep_getattr and deep_hasattr (breaks backward compatibiltiy).
[ ] add deprecation warning to all deep_attr functions.

[x] remove the regression example.

* [x]  remove the `--test` argument in the `vcl.py` example and replace it with a `--num-epochs` one.

* [x]  remove the comments in the `multivariate_svi` test.

* [x]  revert the change from `range` to `trange` in the bnn `fit` loop.

* [ ]  change occurrences of `util.deep_setattr` to `pyro.infer.autoguide.utils.deep_setattr`.

* [ ]  revert removal of `deep_getattr` and `deep_hasattr` (breaks backward compatibiltiy).

* [ ]  add deprecation warning to all `deep_attr` functions.

Not 100% sure I follow. These functions are only accessed internally in tyxe functions and the import in utils is just a renaming. Unless you want to add the check for the version of pyro back, this won't run anyway. Also as there is no PyPi package, I'm not sure how you would push the deprecation warning to users before it is actually deprecated. but maybe I'm jst misunderstanding the whole thing, in which case please just change as you see fit.

Other than the depreciation warning, where I don't quite understand what you're trying to achieve, I think I have addressed all your points. I appreciate the detailed, nitpicking feedback, that is how things progress well.

Cool, looks good to merge now. I was playing around with the whole prefix issue for the prior update and found a nicer way of fixing this directly in the BNN class, I'll push that after merging your PR (I was actually wrong that the issue came from the guide, the net prefix is handled correctly there out of the box now).
Regarding the deprecation warning: I don't want to remove public facing functions without a warning. I don't know if anyone might be using them and if they would pull updates to the codebase, it just seems like the safer choice to me than outright deleting them.
Glad you appreciate the feedback, I hope it wasn't too much back and forth about small details!


Errors in console.

Hello. I would like some assistance with those errors that pop.

Plugins that I use that affect UI ; Aetherment , BigPlayerDebuffs , EnemyListDebuffs, EnemyListHP, Party Icons.

I admit, I have no idea what this means. Time to asm around!

Weeeiiirrdd, moodles shouldnt show in Duties

I can see a new update on moodles right now.

Yepp, this one should fix another error that might have influenced this one

Alright. I will check it out 💯

What about this one?

Take over moodles he said :catkiss: It'll be fun he said :catkiss:
Istg... Kite wrote not a single line of (null) safety into Moodles. I'm going to lose it.
Is IPC so people can send w/e they like so who knows if this will ever resolve properly:
[EzIPC("ClearStatusManagerByPtr")] private void ClearStatusManager(nint ptr)
Can literally be null:
Svc.Objects.CreateObjectReference(ptr)
(It literally returns an IGameObject?)
Can fail to cast in cases where the given object is null OR JUST NOT A PLAYER?????:
(IPlayerCharacter)Svc.Objects.CreateObjectReference(ptr)
Literally doesn't check if the player is null, valid or exists and just grabs the status manager:
void ClearStatusManager(IPlayerCharacter pc) {     var m = pc.GetMyStatusManager(); ...
(You can't assume pc is null if you don't use nullability OR check for null in the method that calls this one out of safety!!!)
[EzIPC("ClearStatusManagerByPtr")]
private void ClearStatusManager(nint ptr) => ClearStatusManager((IPlayerCharacter)Svc.Objects.CreateObjectReference(ptr));

[EzIPC("ClearStatusManagerByPC")]
private void ClearStatusManager(IPlayerCharacter pc)
{
    var m = pc.GetMyStatusManager();
    foreach(var s in m.Statuses)
    {
        if(!s.Persistent)
        {
            m.Cancel(s);
        }
    }
    m.Ephemeral = false;
}

I have no clue what you've just said but thank you and your similars for making this game a better , more enjoyable place for everyone <3

Alright, thank you for that.

akshually, turns out this is not reliably fixed... somehow...
Also depends on what plogon is throwing errors

It was Sonar plugin.

Then we cannot help 😔

Yeah I know. The errors regarding moodles have stopped so far


bug: Service worker may loop indefinitely in some browsers

Reported here: https://github.com/dfinity/internet-identity/issues/255

now that this repo is public, I could just transfer the original issue; it now lives as #453


feat: Modified notification nature when the tab is active

Fixes #366
My PR closes #issue_number_here
👨💻 Changes proposed(What did you do ?)

Modified js so that notification don't work while the tab is active

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title and description of the PR is clear and explains the approach.

Note to reviewers

📷 Screenshots

PR Compliance Checks
Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project. The results are listed in the following section.
Issue Reference In order to be considered for merging, the pull request description must refer to a specific issue number. This is described in our Contributing Guide. This check is looking for a phrase similar to: "Fixes #XYZ" or "Resolves #XYZ" where XYZ is the issue number that this PR is meant to address.
Conventional Commit PR Title
In order to be considered for merging, the pull request title must match the specification in conventional commits. You can edit the title in order for this check to pass. Most often, our PR titles are something like one of these:
* docs: correct typo in README

* feat: implement dark mode"

* fix: correct remove button behavior

Linting Errors
* Found type "Feat", must be one of "feat","fix","docs","style","refactor","perf","test","build","ci","chore","revert"

@Virtual4087 Please fix this error, if you don't know how, try this:

Read the error carefully or read the contributing file
Checkout past merged Pull requests to see how they were done for it to be merged

Note: you can edit your PR message, this PR won't be merged if the error isn't gone and you have a week max to fix it before the PR gets closed without being merged

My bad I'm new to this I didn't realize I had to resolve the convo I thought only you had the authority😂

My bad I'm new to this I didn't realize I had to resolve the convo I thought only you had the authority😂

i'm not taking about the convo

PR Compliance Checks
Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project. The results are listed in the following section.
Issue Reference In order to be considered for merging, the pull request description must refer to a specific issue number. This is described in our Contributing Guide. This check is looking for a phrase similar to: "Fixes #XYZ" or "Resolves #XYZ" where XYZ is the issue number that this PR is meant to address.

i'm talking about this, error isn't gone

My bad changed "Fixes issues #366" to "Fixes #366"

Finally😅


Use correct container name for Hive uninstall jobs

https://github.com/stolostron/backlog/issues/24302

/cc @RobDolares


Change .apk download for Get it on Github badge

I see that you're making similar requests on a lot of repos. Can I ask why?

closing this for now. feel free to reply. I'm unlikely to do this without justification. Even tho I use github, I don't like it bc Microsoft is an awful company... so I don't really care about this unless it poses an acctual issue wrt accessibility/etc.


Add methods for converting dict to product or zip sweep and deprecate implicit cartesian product

Fixes #2698 .
Fixes #3018 .

Parameter keys can be of type sympy.Basic in addition to string. We use TParamKey to represent the type of a parameter so we can define it in one place and also make the code more readable.


Bug: ReferenceError: Vue is not defined - Missing exports declaration ?

Describe the bug

Hello, I'm kind of reopening #234 here as I'm having some trouble upgrading a software to Vue3 which was running completely fine in Vue 2 with Karma test runner and Vue Test Utils V1.x latest version.
Once everything is updated, unit tests can't run anymore and a ReferenceError: Vue is not defined is thrown.
I tried to follow what was written in this post and investigate the different possibilities and found that the easiest way around was to simply add a "module" exports inside the package.json of VTU like this:

Is there a good reason not to declare the module exports type here ?  I'm not an expert at all in bundling so I don't know if there could be some bad side effects... If not, could it be inserted here ?  I'm O.K. to create a simple PR with this if needed...
To give more information on the first reply that was given by @lmiller1990 at that time, Karma bundles the unit tests, the project source code and its test runner using Webpack then executes the result inside a browser directly and not using node and JSDom as lots of "modern" test runners do. The target parameter inside webpack config file is set to default (so browserslist or web).
Thanks in advance ;-)

We already have the module export, it's on the very first line in your screenshot. I thought the bundler would look there - it certainly should.
What version of webpack are you using? I wonder if an old webpack version does not respect the top level module.
https://github.com/vuejs/test-utils/blob/844f5088ff5cde60c578b2aa5956d38901132d43/package.json#L8

Hi @lmiller1990 and thanks for your reply, I alredy saw the module export and that's why I kept this line in my screenshot. As you said, clearly, Webpack seems to 'miss' the top level module.
I'm using an almost up-to-date stack here, Webpack is on version 5.91.0. Latest version is 5.93.0.
I completely agree with you that Webpack should look there, but it also clearly seems it doesn't... I'm not that much in bundling technologies but what is sure is that top level module is not part of NPM standard but was part of a proposal. It's true that it's commonly used by some bundlers.
I'm aware this looks in a sense like a Webpack issue, however, I opened the issue here as adding a single line can fix the problem... If this is has no impact on other bundlers (I suppose it shouldn't but once again, I'm not an expert at all with the different bundlers available), perhaps it could be added here ?

What does your project have for type in package.json?
I'm hesitant to merge this until we understand the core issue. This package has millions of downloads for week - it's not really ideal to merge a quick fix for one person's project, until we actually understand what's going on.

Hi @lmiller1990,
Sure, I clearly understand your point of view and wasn't expecting a quick fix. Let's hope we can find out what happens here...
Our project has no type declared in package.json which is equivalent to commonjs as explained in Node documentation.

Just a little word to say that I just edited my last comment... passing too much time on it made me miss the fact that Webpack was indeed not behaving as expected !
I'd like to make one or two more tests on my side to see why the import condition is not set...
Also wanted to say that I'll soon be on summer break, so if I don't reply for a certain time, please don't consider the issue as stalled... I hope to finish my tests beforehand but have some other obligations so I'm not sure I'll be able to do them...
Thanks again for your valuable replies.

No problem, will leave this open. We can merge webpack specific code as long as we are confident it won't break other bundlers, webpack is still the most widely used bundler. I know webpack often overreaches and does some weird things, mainly because it was doing that before any of these standards really existed.
Let me know if you need any more info or if you are ready for a review - if we could just try the patch against some other popular tool chains / boilerplates, that'd be fine.

Hi @lmiller1990 and thanks for your understanding and time on this issue.
I couldn't resist to make all the tests I wanted before my leave... here's what I've found and some other proposals that could perhaps better suit depending on your experience with all the tools chains out there...
So, with a current version of Webpack and on my specific case, I saw that the conditions that were set were require, module, webpack, production and browser.
I made some tests with "type": "module" inside package.json. Unfortunately, this is not an option in my case as I can't manage to make all dependencies compatible with this, but I could see that, depending on the context where a dependency is imported, the set conditions are either the same, either import, module, webpack, production and browser.
Doing some tests with a node-related test runner such as Jest instead of Karma, I could see that the conditions were require, module, webpack, production and node.
To summarize, my personal problem comes from the fact that browser has higher priority than require in VTU exports field. Someone who is using Jest should not be impacted because browser condition is replaced by node. I'm not sure someone with a "type": "module setting is safe netiher as sometimes require condition is set, sometimes it's import...
My first proposal was indeed to include module inside exports like this:
"exports": {
    ".": {
      "types": "./dist/index.d.ts",
      "node": "./dist/vue-test-utils.cjs.js",
      "module": "./dist/vue-test-utils.esm-bundler.mjs",
      "import": "./dist/vue-test-utils.esm-bundler.mjs",
      "browser": "./dist/vue-test-utils.browser.js",
      "require": "./dist/vue-test-utils.cjs.js",
      "default": "./dist/vue-test-utils.cjs.js"
    },
    "./package.json": "./package.json"
  },

However, the main reason in the end is that require has lower priority to browser as Webpack chooses the first exports to meet one of the conditions. So this simple switch also does the trick:
"exports": {
    ".": {
      "types": "./dist/index.d.ts",
      "node": "./dist/vue-test-utils.cjs.js",
      "import": "./dist/vue-test-utils.esm-bundler.mjs",
      "require": "./dist/vue-test-utils.cjs.js",
      "browser": "./dist/vue-test-utils.browser.js",
      "default": "./dist/vue-test-utils.cjs.js"
    },
    "./package.json": "./package.json"
  },

This second proposal could perhaps be safer and would makes more sense in all cases... except if I miss some use cases, I suppose an IIFE is only used in pure browser context without using modern bundling technologies ?
And eventually, what can also be done as a safety net is to nest multiple conditions in the exports. That's not very clean and nice but perhaps you'd prefer and think it's safer:
  "exports": {
    ".": {
      "types": "./dist/index.d.ts",
      "node": "./dist/vue-test-utils.cjs.js",
      "import": "./dist/vue-test-utils.esm-bundler.mjs",
      "webpack": {
        "module": "./dist/vue-test-utils.esm-bundler.mjs"
      },
      "browser": "./dist/vue-test-utils.browser.js",
      "require": "./dist/vue-test-utils.cjs.js",
      "default": "./dist/vue-test-utils.cjs.js"
    },
    "./package.json": "./package.json"
  },

This last possibility can be mixed with the second one if you prefer:
  "exports": {
    ".": {
      "types": "./dist/index.d.ts",
      "node": "./dist/vue-test-utils.cjs.js",
      "import": "./dist/vue-test-utils.esm-bundler.mjs",
      "webpack": {
        "require": "./dist/vue-test-utils.cjs.js"
      },
      "browser": "./dist/vue-test-utils.browser.js",
      "require": "./dist/vue-test-utils.cjs.js",
      "default": "./dist/vue-test-utils.cjs.js"
    },
    "./package.json": "./package.json"
  },

So after all those tests and thinking about it, I'd personally propose to take the second option (switching require and browser), however, as I don't have a lot of experience with other bundlers than Webpack and certainly don't have all the knowledge of VTU internals as you've gathered throughout all those years, I prefer to let you evaluate if this could be applied and which version you'd prefer... The second proposal is the one which is the cleanest and it really makes sense I think, but perhaps you'd like the "safety net" of the last proposal...
Just take your time to think about it and tell how you feel, I'll happily open a PR when coming back from my break in case you think one is worth considering...

Second options seems the best imo - what to make a PR?


Explicitly set dataflow runner v2 test shuffle mode to appliance

DO NOT MERGE
Only for testing.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.


Regression: skipping primitive properties fails with NullPointerException

The issue discussed in #242 reoccurs in 2.3.5.
When running the example mentioned in #242, I' see the following exception:
Exception in thread "main" java.lang.NullPointerException
	at org.modelmapper.internal.ReferenceMapExpressionImpl.skip(ReferenceMapExpressionImpl.java:85)
	at org.modelmapper.internal.ConfigurableConditionExpressionImpl.skip(ConfigurableConditionExpressionImpl.java:71)
	at com.example.Main.lambda$thisOneDoesNotWork$1(Main.java:33)
	at org.modelmapper.internal.TypeMapImpl.addMappings(TypeMapImpl.java:266)
	at com.example.Main.thisOneDoesNotWork(Main.java:33)
	at com.example.Main.main(Main.java:10)

Is there a known workaround for that?

I believe I have the exact same issue on 2.3.7:
java.lang.NullPointerException
	at org.modelmapper.internal.ReferenceMapExpressionImpl.skip(ReferenceMapExpressionImpl.java:85)
	at org.modelmapper.internal.ConfigurableConditionExpressionImpl.skip(ConfigurableConditionExpressionImpl.java:71)
	at com.redacted.redacted.lambda$someToSomeEntity$11(RedactedClass.java:78)
	at org.modelmapper.internal.TypeMapImpl.addMappings(TypeMapImpl.java:266)

I have same problem with boolean setter in 2.3.7.
This might be due to Java version, since same code that doesn't work with JDK 14 or JDK 12 works fine with JDK 1.8

I am using ModelMapper v2.3.7 with JDK 11 and I also need the possibility to skip a setter with a primitive boolean parameter. Currently ModelMapper throws a NullPointerException.

This is most definitely related to https://github.com/modelmapper/modelmapper/issues/480. The same workaround I posted there also works for skipping (call skip in the overridden method instead of map).


CAMEL-9257 route stop/start doesn't work for camel-websocket producer

merged - ee3deceb902e944c54a1a87d4ea47e50ad21e47d


youtube video play in 360p. it remain fixed for all youtube video

i have solution contact me on whatsapp
+971557474526

i have solution for get all quality contact with me :+971557474526


Automatic Filtering Logic Doesn't Retain Multiple Status Codes with Same Content-Length

Recently, I encountered an issue. When the automatic filtering is enabled and the status code matched is set to "ALL", if the Content-Length of both 404 and 403 are the same, the 403 won't be retained, which doesn't match my expectations.
After spending some time debugging, I realized that when the automatic filtering is enabled, not all matching filters are added with an "AND" connection. Instead, only the first matching filter is added. I believe this is unexpected behavior. A more reasonable approach would be to add all of them to the "Filters" and use "AND" for filtering decisions.
Below is the modified code I've come up with, just for your reference:
func (j *Job) calibrateFilters(responses []Response, perHost bool) error {
	// Work down from the most specific common denominator
	if len(responses) > 0 {
		// Content length
		baselineSize := responses[0].ContentLength
		sizeMatch := true
		for _, r := range responses {
			if baselineSize != r.ContentLength {
				sizeMatch = false
			}
		}
		if sizeMatch {
			if perHost {
				// Check if already filtered
				for _, f := range j.Config.MatcherManager.FiltersForDomain(HostURLFromRequest(*responses[0].Request)) {
					match, _ := f.Filter(&responses[0])
					if match {
						// Already filtered
						return nil
					}
				}
				_ = j.Config.MatcherManager.AddPerDomainFilter(HostURLFromRequest(*responses[0].Request), "size", strconv.FormatInt(baselineSize, 10))
			} else {
				// Check if already filtered
				for _, f := range j.Config.MatcherManager.GetFilters() {
					match, _ := f.Filter(&responses[0])
					if match {
						// Already filtered
						return nil
					}
				}
				_ = j.Config.MatcherManager.AddFilter("size", strconv.FormatInt(baselineSize, 10), false)
			}
		}

		// Content words
		baselineWords := responses[0].ContentWords
		wordsMatch := true
		for _, r := range responses {
			if baselineWords != r.ContentWords {
				wordsMatch = false
			}
		}
		if wordsMatch {
			if perHost {
				_ = j.Config.MatcherManager.AddPerDomainFilter(HostURLFromRequest(*responses[0].Request), "word", strconv.FormatInt(baselineWords, 10))
				return nil
			} else {
				_ = j.Config.MatcherManager.AddFilter("word", strconv.FormatInt(baselineWords, 10), false)
				return nil
			}
		}

		// Content lines
		baselineLines := responses[0].ContentLines
		linesMatch := true
		for _, r := range responses {
			if baselineLines != r.ContentLines {
				linesMatch = false
			}
		}
		if linesMatch {
			if perHost {
				_ = j.Config.MatcherManager.AddPerDomainFilter(HostURLFromRequest(*responses[0].Request), "line", strconv.FormatInt(baselineLines, 10))
				return nil
			} else {
				_ = j.Config.MatcherManager.AddFilter("line", strconv.FormatInt(baselineLines, 10), false)
				return nil
			}
		}
	}
	return fmt.Errorf("No common filtering values found")
}

When using AND to connect several filters, if the size is the same, it will
not be filtered out. Only all filters that meet the conditions will be
filtered out.
ayu @.***> 于2023年10月16日周一 09:37写道：

Hi, for directory scanning, if there are 5 or 10 files of the same size
under the same domain name, then this size will be filtered out. User can
customize 5 or 10 or other values. Can this function achieve similar needs ?
—
Reply to this email directly, view it on GitHub
https://github.com/ffuf/ffuf/issues/738#issuecomment-1763596075, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALZC53USEPWEZGA4YE2CZ3DX7SFW7AVCNFSM6AAAAAA5YMLL6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRTGU4TMMBXGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


remove strawberry.enum class decorator in object-is-not-an-enum.md

Remove @strawberry.enum class decorator in object-is-not-an-enum.md
Fix typo in negative example for object-is-not-an-enum.md
Our error example doesn't show an error :) the class NotAnEnum is really still an enum as we decorated it in the negative example. This PR removes that and leaves a comment that it is missing that enum decorator.

Thanks for contributing to Strawberry! 🎉 You've been invited to join
the Strawberry GraphQL organisation 😊
You can also request a free sticker by filling this form: https://forms.gle/dmnfQUPoY5gZbVT67
And don't forget to join our discord server: https://strawberry.rocks/discord 🔥


Revert https://github.com/apple/swift/pull/58731

The changes from https://github.com/apple/swift/pull/58731 break compilation for compiler developers who choose to bootstrap with hosttools. Apparently older Swift compilers import std::string::c_str() as mutating.
This reverts commit d83c17e8324b3fbf69e2504d8d48c8ec160b621f.

@swift-ci please smoke test and merge

@tshortli Can we define a String initializer taking a std::string as argument (in an extension). And then do the magic there, e.g. #if on the compiler version or whatever is needed.

I like that idea. I'd like to get this in to unblock folks but I'd definitely love to find some way to resolve the warnings. What do you think @zoecarver?

I think @egorzhdan already added some such extensions. He would be the best person to advice on how to proceed.

Hmm, wouldn't SwiftCompilerSources always be built with the just-built Swift compiler, even in BOOTSTRAPPING-WITH-HOSTTOOLS mode?
I think having Swift extensions for these type conversions is a great idea regardless of the c_str mutability issue.
I added some trivial Swift.String <-> std::string conversion logic in https://github.com/apple/swift/pull/58452. However, those extensions won't immediately be available in SwiftCompilerSources, since the overlay is going to be built & shipped similarly to the Swift stdlib, and would sometimes be out-of-sync with the compiler (e.g. when building in BOOTSTRAPPING-WITH-HOSTTOOLS mode with a slightly outdated macOS SDK).

wouldn't SwiftCompilerSources always be built with the just-built Swift compiler

No, not with the HOSTTOOLS mode, which is used for local development (to speed up the build).


Port rejseplan

I've ported rejseplanen.dk to pyhafas.
Errata: Busses are labelled wrongly as trains, due to Leg's mode being trains by default.
But other than that, it seems to work.
I've also extended location queries to be more flexible than just Stations: Now one can ask from any address to any other address.
Motivation: I'm doing a digital dashboard with a Raspberry Pi Zero W.
It is not powerful enough to run Chromium(+Puppeteer), so scraping is a no-go.
pyhafas seems nice and extensible, and let's my Pi poll and display the next departures to my workplace :+1:

Hi, thanks for your contribution! A created a PATCH that should fix the failing tests (they fail because you've added lid to Station): https://gist.github.com/em0lar/c28fd830feb94a9f643b302473577f90 Please apply the PATCH or something else to let the tests pass.

Thanks for the patch to fix the tests!
It's been squashed with the commit that adds lid, so everything is atomicly updated together :+1:
And thanks for making pyhafas :-)


add search trie definition

Coverage increased (+0.7%) to 84.1% when pulling f31532067883fd536f5c2f15baa16368837771c6 on search-trie into 300437efa9e1e89366b439c85214d2671a6004fa on server-staging.


Windows testing

I changed runcmd_test.go to use a build constraint to only be used when GOOS isn't windows, and I added runcmd_windows_test.go with windows-specific testing. I feel that this helps keep the tests uncluttered by (possibly multiple) operating system checks.

Thanks, I tried to find that information - if you could have system specific tests - without sucess. Can you point me to it?

http://golang.org/pkg/go/build/
Down where it says "Build Constraints."


Revert [SPARK-8654] [SQL] Fix Analysis exception when using NULL IN

This reverts commit dcbd58a929be0058b1cfa59b14898c4c428a7680 from #8983
The tests never ran and this broke them.

Merged build triggered.

Merged build started.

Test build #43424 has started for   PR 9034 at commit 73c7048.

Test build #43424 has finished for   PR 9034 at commit 73c7048.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class BinaryHashJoinNode(
case class BroadcastHashJoinNode(
trait HashJoinNode

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43424/
Test PASSed.


Bump version of elkm1_lib

Description:
Bump base library (elkm1_lib) version.
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Not sure why the pylint build is failing. I can't tell from the terminal log. It doesn't appear to be my change. Anything I should do?

They fail sometimes on a good day for no reason, and there's been a couple flaky tests causing extra failures lately. I restarted the job so hopefully it will complete this time 😃


Optimism SDK withdrawl proof fails

System information
Running on BSC/opBNB mainnet (56, 204)
Expected behaviour
Proofs for L2 BNB and token withdraws should be calculated and submitted to BSC L1.
Actual behaviour
Theres a bug in the CrossChainMessenger getMessageBedrockOutput. BSC <> opBNB needs the most recent output index, but the optimism sdk uses the most recent index after the transaction. I have built a template dapp for BSC <> opBNB bridges with custom tokens, it fixes this bug by overriding getMesageBedrockOutput. The dapp is live at https://microscopic-continent-rhythmic.on-fleek.app/ and contains the fix. The broken code in Optimism SDK is at https://github.com/ethereum-optimism/optimism/blob/ee058e413e0c7ff92d000593af648f3d79ca1a31/packages/sdk/src/cross-chain-messenger.ts#L1076.
You can see the working code fix in my template dapp https://github.com/chinese-zodiac/bridge-opbnb-cz-cash-fe/blob/main/src/components/account/WithdrawAccount.js#L94

Thanks for your information!
This is not a bug; both proofs work for the withdrawal. The reason we use the most recent output index on BSC is that BSC has more historical state data than Ethereum, making it harder to preserve and retrieve the proof of an older block.
The @eth-optimism/sdk has been replaced with op-viem. op-viem uses a new interface for the withdrawal process, breaking down the withdrawal process. This allows us to choose to use the proof of the latest index or the next index after the initWithdrawal as desired.
There is a proof of concept available as a reference:
https://gist.github.com/owen-reorg/568ae7e5ae24df1db2ef2f58a623621b


Add installation and configuration to README

Adding installation and configuration instructions to README

Thanks for information, thought my machine was no working when selecting C# Godot didn't do anything. :heart:

Thanks! Sorry I overlooked this until now.


make discord-bot and netlify-functions-ecommerce work with tables

General idea:

Disable _id and __v because of underscore property names issue
Use insertMany() to create new documents and Model.updateOne() to update existing, as opposed to using save() and create(), because those rely on _id
Use id instead of _id as unique identifier
Comment out any logic that uses deleteMany(), dropCollection(), createCollection()
Any nested properties are stored as JSON strings

Outstanding questions:

Any workaround for lack of deleteOne() and deleteMany()? Hard to write tests without those.

Current outstanding issues:

addIndex not handling camelCased names (stargate/data-api#1404)
$in support for Mongoose populate (stargate/data-api#1415)
Sorting in find()
skip

In ae819af, I showed that Mongoose studio's basic functionality of displaying documents works with tables as well.

The missing features are:

estimatedDocumentCount()
countDocuments()

I worked around this limitation using the following code:
  const numDocuments =
    await Model.find(filter).then(docs => docs.length);


Providing all versions of pretrained weights

Hi, could you also provide all versions of the pretrained weights for BERT,CLIP,VideoSwin? And could you explain how different version of these backbone corresponds to the version of the VALOR weights? Thanks!

pretrained weights for BERT,CLIP,VideoSwin are already provided. VALOR-base use videoswin and bert. VALOR-large use CLIP and bert


Problem with opening the Rhino from Revit

Hello,
Today I installed Rhino Inside and I have a problem with Licence Manager initialization.
Here is a window which shows up:

Also, I tried to open Revit as Administration, but nothing changed.
I attach the error message "RhinoDotNetCrash":
[ERROR] FATAL UNHANDLED EXCEPTION: System.AccessViolationException: An attempt was made to read or write protected memory. It often indicates that another memory is damaged.
w Options.setSSOServer(Options* , Int32 )
w UIFrameworkServices.WebServiceWrapperImp.{dtor}(WebServiceWrapperImp* )
w UIFrameworkServices.WebServiceWrapperImp.__delDtor(WebServiceWrapperImp* , UInt32 A_0)
w UIFrameworkServices.WebServiceWrapper.!WebServiceWrapper()
w UIFrameworkServices.WebServiceWrapper.Dispose(Boolean A_0)

[END ERROR]

I downloaded again the newest installers and still the same problem.

Can you run Rhino 7 WIP outside of Revit, just by itself.  Does it request a license?  Once it runs outside, can you then run it inside Revit?

Hi,
Could you please follow instructions there are here and tell us what you found, it would be so helpfull.

Can you run Rhino 7 WIP outside of Revit, just by itself. Does it request a license? Once it runs outside, can you then run it inside Revit?

Yes. I can easily open just Rhino 7 WIP. The error occurs only when I am in Revit and when I try to click RhinoInside Plugin

Gentelmens...
Plugin starts working !!!. I disable all addins from Revit and leave only RhinoInside.
Now I have to figure out which one cause the problem. I let you know when I will find it.
Thanks a lot

Ok. I found the problem. The plugin called Naviate.addin cause a problem. When I disabled it, Rhino.Inside runs perfectly.

We can contact Naviate.  Do you have know someone there that may be able to help?

Do you know which Naviate plugin you are using specifically?

@marcinwoch Would you be able to open the Naviate.addin file in text editor, follow the path set under <Assembly> in your windows explorer, and share a screenshot of all the files shipped with the Naviate plugin? Another great adjustment you can do is to show the Product Version in the detail view of windows explorer

I just tested the newest version and it still clashes with Dynamo.
Maybe it will have to be run separately after all..
Starting Dynamo - closing it - then firing up Rhino.Inside gives error message.
Starting Rhino.Inside - then starting Dynamo (works up to this stage, but then) - Revit freezes.

@marcinwoch Would you be able to open the Naviate.addin file in text editor, follow the path set under <Assembly> in your windows explorer, and share a screenshot of all the files shipped with the Naviate plugin? Another great adjustment you can do is to show the Product Version in the detail view of windows explorer

When I was turned on a row you ticked, there is no information about the version.
Here is a info from Naviate Start Window:


StructExpression Ident's differ between AstNode and TypedAstNode

Given the following small example:
struct X{}

fn main() {
  let var = X{};
}

If we inspect the Ident of our let expression as an AstNode we get.
name: Ident {
    name_override_opt: None,
    span: Span {
        src (ptr): 0x000000012a64ea90,
        path: Some(
            "/Users/joshuabatty/Documents/rust/fuel/sway/sway-lsp/test_programs/array_basics/src/main.sw",
        ),
        start: 45,
        end: 46,
        as_str(): "X",
    },
    is_raw_ident: false,
},

And if we inspect this line as a TypedAstNode we can see the Ident is actually pointing to the struct declaration and the Span information of the token has been discarded.
struct_name: Ident {
    name_override_opt: None,
    span: Span {
        src (ptr): 0x0000000121ed1f70,
        path: Some(
            "/Users/joshuabatty/Documents/rust/fuel/sway/sway-lsp/test_programs/array_basics/src/main.sw",
        ),
        start: 16,
        end: 17,
        as_str(): "X",
    },
    is_raw_ident: false,
},

We need to carry forward the original Span information during type checking as this is required for the language server.

What if Span didn't have the source and only had the (non-optional) path?  I'm guessing you'd then have to maintain your own cache of source strings based on the paths, yeah?
I've never been a fan of the source + start + end spans that we have, their consistency is not enforced.  We are able to concoct them out of strings, give them random start and end values, do whatever we did to cause the above problem for this issue.
As far as the IR goes (and I imagine this would apply to other tooling) I'd rather just have a path and indices.  If the path doesn't point to a file (any more) then too bad, no span for you, oh well.
(This comes up every few weeks and I have this whinge and nothing much comes of it, from me or others, but I might as well go again... :wink:)

What if Span didn't have the source and only had the (non-optional) path? I'm guessing you'd then have to maintain your own cache of source strings based on the paths, yeah?
I've never been a fan of the source + start + end spans that we have, their consistency is not enforced. We are able to concoct them out of strings, give them random start and end values, do whatever we did to cause the above problem for this issue.

Sure, that sounds reasonable. I'm not using the src field for anything in the language server so can't really comment on their usefulness (or not).
This issue above is that the start and end  fields of the Ident are coming from the TypedStructDeclaration, in this case: start: 16 & end: 17. Where they originally are start: 45 and end: 46.
TypedStructDeclaration {
    name: Ident {
        name_override_opt: None,
        span: Span {
            src (ptr): 0x0000000121ed1f70,
            path: Some(
                "/Users/joshuabatty/Documents/rust/fuel/sway/sway-lsp/test_programs/array_basics/src/main.sw",
            ),
            start: 16,
            end: 17,
            as_str(): "X",
        },
        is_raw_ident: false,
    },
    ..,
},

I'm not sure why the local Ident is being replaced by the Ident in the TypedStructDeclarationbut we need to carry the localIdent` information forward.

Actually, yeah, take the Span out of Ident altogether here, IMO.  But that mightn't change anything here if a whole new Ident is getting created for some reason.

It looks like the Ident of the declaration is coming from this line
The correct Span can actually be found like this though. call_path_binding.inner.suffix.1
just tagging @emilyaherbert for any comment / insight on this?


GUACAMOLE-559: Add support for reading/writing clipboard contents using the Asynchronous Clipboard API.

Finally: the Holy Grail.
https://developers.google.com/web/updates/2018/03/clipboardapi
https://www.w3.org/TR/clipboard-apis/#async-clipboard-api

Cripes it even works on mobile Chrome.


Specify the format for azure-image specification

The format for azure-image is not specified anywhere in the docs. The link for more information talks about how to get information on publisher,offer,sku and version, but doesn't specify format for azuere-image option for azure driver.
cc @ahmetalpbalkan

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 "patch-1" git@github.com:ppadala/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending 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 "patch-1" git@github.com:ppadala/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

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

Lgtm. The default value and the "[?]" link in the docs probably help a bit, but this is better. Users still have to run the azure CLI to find the image name, SKU, publisher etc. it's hard to guess. And cli actually prints the full image name.

LGTM


[3.6][Bug] Using the parameter select_all on a second select2_multiple breaks the buttons events

Bug report
What I did:
Add a 'select_all' parameter to a second select2_multiple field in the same form. The first one doesn't has the 'select_all' parameter.
What I expected to happen:
The 2 added buttons to have a click event to trigger their related behavior.
What happened:
The click events are not set on the buttons.
Related JS there https://github.com/Laravel-Backpack/CRUD/blob/master/src/resources/views/fields/select2_multiple.blade.php#L79
What I've already tried to fix it:
Reverse the two select2_multiple fields to declare the one that has 'select_all'  first. It fixes the behavior (but will need a proper fix)
Backpack, Laravel, PHP, DB version:
Laravel 5.5, Backpack 0.3

Hello @edhelas
I am so sorry for taking so much time to handle this one.
I am really happy you found a fix, but as you said it was a temporary one.
I have now submited a PR #2243 that fixes this issue for good.
So i will be closing this now, if you need to add something to this issue please report on PR #2243 thread.
Once again, thank you very much for bring this up to the table.
Wish you the best,
Pedro

Ouch! Took so long to fix this, indeed sorry @edhelas ! Must have slipped through the cracks.
Thank you @pxpm . We're finally getting to fix minor issues that I've neglected for so long, I LOVE THIS!

No problem ;) Happy to see that it's now resolved!


refactor: enhance Alert and Toast components

This PR primarily focues on the funciontality. I will create another PR to change severity colors if the visual spec is still uncertain in this moment.
Todos

[x] Add redacted fonts, a simple font alternative to the Lorem Ipsum text,
[x] Severity levels
[x] Add more icons
[x] Enhance documentation for the useToast hook
[x] Closeable & CloseButton components
[x] useCloseable Hook
[x] Wrap children with Closeable component
[x] Add isCloseable and onClose props to Alert, AlertToast, Toast components

Recommended layouts for a toast notification

Increase the vertical padding space to 4x (or 1rem) for a multiline paragraph.
Set the minimum horizontal margin space to 4x (or 1rem) before the close button.
Set the minimum horizontal margin space to 4x (or 1rem) between the side icon and the content.
Apply vertical margin space with 2x (or .5rem) between title and its content.
Apply vertical margin space with 6x (or 1.5rem) between the end of the content and the action button (or action link).

Components
Alert (offers severity levels)
https://trendmicro-frontend.github.io/styled-ui-demo/pr-112/alert
Toast
https://trendmicro-frontend.github.io/styled-ui-demo/pr-112/toast
AlertToast (offers severity levels)
https://trendmicro-frontend.github.io/styled-ui-demo/pr-112/alerttoast
useToast Hook
https://trendmicro-frontend.github.io/styled-ui-demo/pr-112/usetoast

Toast
目前的 severity prop 是跟 Toast 綁在一起，我有想把 Toast 拆成 Toast 和 AlertToast 這兩個 components，這樣使用起來會比較明確
Toast is a simple toast without severity
<Toast />

AlertToast is a toast plus a severity level (success, info, warning, error)
<AlertToast severity="success" />
<AlertToast severity="success" icon="_core.severity-success" />
<AlertToast severity="success" icon={<TMIcon name="close-s" />} />

TODO: 在 Toast 那頁加個提示有 AlertToast component


[V2.7] Brazilian Portugese provider

Delayed, need to add some working English providers first, before adding additional providers.

Now feasible as I can add catalog providers.


CocoaPods Badges: Invalid Response Data

Are you experiencing an issue with...

[X] shields.io
[ ] My own instance
[ ] gh-badges NPM package

:beetle: Description

:link: Link to the badge

https://img.shields.io/cocoapods/v/Cotter.svg?style=flat
https://img.shields.io/cocoapods/l/Cotter.svg?style=flat
https://img.shields.io/cocoapods/p/Cotter.svg?style=flat

:bulb: Possible Solution

The reason these are all failing is because we expect the JSON response from Cocoapods to include the keys version, license and platforms (we use the same validator for all badges).
When we call https://trunk.cocoapods.org/api/v1/pods/Cotter/specs/latest we get a response without a license key.
{
  "name": "Cotter",
  "version": "1.0.1",
  "summary": "Cotter is the client SDK for Cotter authentication services",
  "swift_versions": "5.0",
  "description": "CotterIOS simplifies client's integration with Cotter. It is an SDK that provides necessary pages and API services that connects your application to Cotter authentication services.",
  "homepage": "https://github.com/cotterapp/ios-sdk.git",
  "authors": {
    "Cotter Team"<EMAIL_ADDRESS>  },
  "source": {
    "git": "https://github.com/cotterapp/ios-sdk.git",
    "tag": "1.0.1"
  },
  "platforms": {
    "ios": "10.0"
  },
  "source_files": "Source/**/*.swift",
  "resource_bundles": {
    "Cotter": [
      "Source/**/*.{png,storyboard,lproj}"
    ]
  },
  "frameworks": "UIKit",
  "swift_version": "5.0"
}

You can probably fix this by declaring a license for your project, but if this is a valid response object, maybe we need a different validator for the cocoapods license badge.

Hi Chris, thank you for your reply.
Thinking about it, a license should probably be a required parameter for an open-source CocoaPods project. Perhaps you're right about having a different validator since the JSON response is valid.
Thank you so much for your help!

By the way @rayandrie , we're open to pull requests if you want to help us out! Changing the schema should be quite simple even for a first time contributor. 😉


Unable to load native connector on windows.

On windows it seems that neither memflow 0.2.1 or 0.2.0 are able to load the native connector from the user/Documents/memflow/ folder or from the executeable folder. Also adding to the env path didnt make the connector available

or

Native is an OS layer, not a connector. Use --os native (with no connector)


Memory leak

Using:
StrictMode.setVmPolicy(new StrictMode.VmPolicy.Builder().detectAll().penaltyLog().penaltyDeath().build());
And creating blank activity which launches CameraActivity by tapping button and pressing back, tapping button -> repeat process there is memory leak.

com.example.android.camera2basic E/StrictMode: A resource was acquired at attached stack trace but never released. See java.io.Closeable for information on avoiding resource leaks. java.lang.Throwable: Explicit termination method 'release' not called at dalvik.system.CloseGuard.open(CloseGuard.java:180) at android.view.Surface.setNativeObjectLocked(Surface.java:511) at android.view.Surface.(Surface.java:179) at com.example.android.camera2basic.Camera2BasicFragment.createCameraPreviewSession(Camera2BasicFragment.java:684) at com.example.android.camera2basic.Camera2BasicFragment.access$400(Camera2BasicFragment.java:74) at com.example.android.camera2basic.Camera2BasicFragment$2.onOpened(Camera2BasicFragment.java:194) at android.hardware.camera2.impl.CameraDeviceImpl$1.run(CameraDeviceImpl.java:134) at android.os.Handler.handleCallback(Handler.java:751) at android.os.Handler.dispatchMessage(Handler.java:95) at android.os.Looper.loop(Looper.java:154) at android.os.HandlerThread.run(HandlerThread.java:61) 11-12 12:52:19.506 16069-16078/com.example.android.camera2basic W/System.err: StrictMode VmPolicy violation with POLICY_DEATH; shutting down.

This sample has been migrated to a new location where we can accept Pull Requests (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


Flutter Web: stack trace lists JS instead of Dart files.

When an exception is thrown in a Flutter Web application, the stack trace lists .js files instead of .dart files, which makes it difficult to reason about and debug.
For example, type in the following application:
void main() {
  throw "My Hello World Exception";
}

When I run this I get:
Error: My Hello World Exception
    at Object.throw_ [as throw] (http://localhost:43337/dart_sdk.js:5041:11)
    at main$ (http://localhost:43337/packages/test2/main.dart.lib.js:12:15)
    at main (http://localhost:43337/web_entrypoint.dart.lib.js:33:29)
    at main.next (<anonymous>)
    at http://localhost:43337/dart_sdk.js:37403:33
    at _RootZone.runUnary (http://localhost:43337/dart_sdk.js:37274:59)
    at _FutureListener.thenAwait.handleValue (http://localhost:43337/dart_sdk.js:32530:29)
    at handleValueCallback (http://localhost:43337/dart_sdk.js:33057:49)
    at Function._propagateToListeners (http://localhost:43337/dart_sdk.js:33095:17)
    at _Future.new.[_completeWithValue] (http://localhost:43337/dart_sdk.js:32943:23)
    at http://localhost:43337/dart_sdk.js:32194:46
    at _RootZone.runUnary (http://localhost:43337/dart_sdk.js:37274:59)
    at _FutureListener.then.handleValue (http://localhost:43337/dart_sdk.js:32530:29)
    at handleValueCallback (http://localhost:43337/dart_sdk.js:33057:49)
    at Function._propagateToListeners (http://localhost:43337/dart_sdk.js:33095:17)
    at _Future.new.[_completeWithValue] (http://localhost:43337/dart_sdk.js:32943:23)
    at async._AsyncCallbackEntry.new.callback (http://localhost:43337/dart_sdk.js:32964:35)
    at Object._microtaskLoop (http://localhost:43337/dart_sdk.js:37526:13)
    at _startMicrotaskLoop (http://localhost:43337/dart_sdk.js:37532:13)
    at http://localhost:43337/dart_sdk.js:33303:9

Breakpoints work correctly, so it seems that the JS <--> Dart mapping infrastructure does work correctly.
I'm using Android Studio and Chromium on Ubuntu 20.04.
Flutter doctor:

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on Linux, locale en_US.UTF-8)
    • Flutter version 2.2.3 at /home/user/flutter
    • Framework revision f4abaa0735 (7 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /home/user/Android/Sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: /home/user/Downloads/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+0-b944-P17168821)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/linux#android-setup for more details.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = chromium

[✓] Android Studio (version 4.2)
    • Android Studio at /home/user/Downloads/android-studio
    • Flutter plugin version 58.0.1
    • Dart plugin version 202.8488
    • Java version OpenJDK Runtime Environment (build 11.0.8+0-b944-P17168821)

[✓] IntelliJ IDEA Ultimate Edition (version 2020.2)
    • IntelliJ at /snap/intellij-idea-ultimate/current
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Chromium 86.0.4240.75 snap

HI @obeobe
This is working as expected, the flutter's dart converted code is running from js
Read https://flutter.dev/docs/resources/architectural-overview#flutter-web-support
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

Hi @TahaTesser ,
I realize that it's running in JS. This is besides the point.
JS stack traces can be converted to reference the relevant Dart files/functions.
I am not an expert on this, but I believe that this is done with source-maps, and I have seen it working correctly when working with Typescript (where I get stack traces that reference my Typescript code, even when the actual JS code is concatenated, obfuscated, and minified, to the point it is considerably different from the original Typescript).
This is obviously also done in Flutter/Dart to some extent, as I can debug Dart code in the IDE even when what actually gets executed in the browser is the generated JS.
Please re-open the issue, as it's not a complaint about the code running in the JS, but rather a complaint about a bug (or a gap) in the tooling. Stack traces can and should reference my actual source code rather than the compiled JS.

Hi @TahaTesser ,
Can you please re-open this issue and send it to triage?
This is bad developer experience and a serious shortcoming (if not bug) in Flutter tooling for web.
It's possible that proper support was just not developed, which would make this a feature request, but it's unlikely that this is the behavior the Flutter team deems "correct".

@obeobe

I am not an expert on this, but I believe that this is done with source-maps,

You can source maps,flutter build web -h
    --[no-]source-maps                 Generate a sourcemap file. These can be used by browsers to view and debug the original
                                       source code of a compiled and minified Dart application.
    --pwa-strategy                     The caching strategy to be used by the PWA service worker.

          [none]                       Generate a service worker with no body. This is useful for local testing or in cases
                                       where the service worker caching functionality is not desirable
          [offline-first] (default)    Attempt to cache the application shell eagerly and then lazily cache all subsequent
                                       assets as they are loaded. When making a network request for an asset, the offline cache
                                       will be preferred.

    --base-href                        Overrides the href attribute of the <base> tag in web/index.html. No change is done to
                                       web/index.html file if this flag is not provided. The value has to start and end with a
                                       slash "/". For more information:
                                       https://developer.mozilla.org/en-US/docs/Web/HTML/Element/base

Run "flutter help" to see global options.
PS C:\Users\Taha\StudioProjects\triage>

@TahaTesser ,

I do have source maps enabled. They DO generally work.
I can view and debug Dart code in the browser.
The exception is rendered correctly in the browser, with references to Dart files, that open right in the browser's dev tools.
But in the Flutter debugger back-end itself seems to ignore the sourcemaps or otherwise fail to utilize them.
See SS with correct behavior in the browser, and incorrect behavior in the IDE console (whose content comes from the Flutter debugger, I believe):

Hi @obeobe
Source maps are only supported in the build, not at run

Hi @TahaTesser ,
Not sure I understand what you mean.
If you mean that it's just not supported - then fine, but this is sub-par developer experience, and not in line with other modern languages, including (I suspect) Flutter itself when used on mobile platforms.
So maybe this issue is a "feature request" rather than a bug report, but I don't understand why it should be closed.

@TahaTesser can you please re-open and re-triage this issue, perhaps by someone who is more familiar with this subject? (you said you weren't).

cc: @yjbanov

This is a known issue: https://github.com/dart-lang/webdev/issues/1265
@DanTup do you know if there is something that can be done on vscode side to convert displayed stack traces to dart using source maps, or does it need dwds support?

@annagrin thanks for responding. please just note that I experience this with Android Studio, not vscode...

@obeobe thanks for clarifying!

I wrote a small tool to work around this issue until it's fixed.
I post it here in case it may help others: https://github.com/obeobe/flutter-web-exception-mapper

I believe this should have been fixed by https://github.com/flutter/flutter/issues/101888. Please reopen if you are still experiencing this. Thank you!


chore(main): release 0.2.5

:robot: I have created a release beep boop
0.2.5 (2024-06-19)
Bug Fixes

deps: Remove unused 'stringcase' dependency (1d2a9db)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/BLSQ/openhexa-toolbox/releases/tag/0.2.5 :sunflower:


openbangla-keyboard: 2.0.0 -> unstable-2023-07-21; add fcitx5 support

Description of changes

2.0.0 -> unstable-2023-07-21

needed for fcitx5 support

fix meta

remove use of with lib; (see #208242)
add self to maintainers

fix IME support (ibus, fcitx5)

Closes #182388

Things done

Built on platform(s)

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

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

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

[ ] 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.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Perhaps the openbangla-keyboard derivation should be deprecated (placed in aliases.nix with a message suggesting ibus-engines.openbangla-keyboard and fcitx5-openbangla-keyboard) since the build fails with
 > CMake Error at CMakeLists.txt:79 (message):
>   Please set atleast ENABLE_IBUS or ENABLE_FCITX to be ON to build a backend

Besides that, both of ibus-engines.openbangla-keyboard and fcitx5-openbangla-keyboard build.

Thanks, I completely blanked on the package deprecation process. I'll take a look at that

I've removed the existing attribute, thanks for the suggestion!

It does not work with fcitx5.

@eclairevoyant... fcitx5 will not work with OpenBangla Keyboard 2.0. It will work with OpenBangla Keyboard built from the develop branch of OpenBangla Keyboard's github repository till now.


add: Custom scripting for tween values

Added a new CUSTOM_SCRIPT tween value type to the Tween Leaf. It allows you to create a custom script (that is saved inside your scene) to have non-hardcoded tween values.
Template for the custom script
# Custom Tween Value
static func get_tween_value(_actor: Node, _blackboard: Blackboard) -> Variant:
	# Compute your custom logic here
	return 0

This template should automatically appear when creating a new empty script resource.

@SirPigeonz If you have any free time check this out. I am considering adding this to more nodes.

We could do something similar like this for custom scriptable guards in FSM Transitions!
Where you want to use it additionally?

This system is pretty flexible, but afaik you can't edit internal scripts in an external IDE, so I would suggest using this for small scriptable stuff like conditions and values.


[CMO]: Navigate user to View Single Shipment page after CMO press the button "Prepare Shipment"

Now the the CMO is redirect to Dashboard-Orders tab.
Redirect him to the new created shipment page and change the text of the redirecting modal to
"Shipment initiated, redirecting to View Shipment page"

I have raised PR for this issue.


Fix relation and error on default total and default total value type

Signed-off-by: Derrick Mehaffy<EMAIL_ADDRESS>Modified relation to be a oneWay instead of a oneToOne (not sure if this will work entirely as I haven't fully tested the plugin yet)
Also fixed an issue when creating a notification without any tokens which would try to write a string "0" instead of an integer 0

I try if this can be better, and I will publish this in next beta.5 release :)


#3981 - Model SDPR API

Created a new module for SIMS API from external access;
Created a segregated swagger for external endpoints: http://localhost:3000/external/swagger ;

Created Postman collection to authenticate and request a student by SIN (Teams folder);

Documented on: https://github.com/bcgov/SIMS/wiki/Keycloak-Config#external-api-access-service-accounts

Just a minor comment. Great work so far. Please take a look at the remaining comments.


Specifying schema for DB connection

Our project has several schemas in a single postgres database. Is there an option to target only specific schema? I already tried to set schema in database url like below,
DATABASE_URL=postgres://username:password@yourdomain.com/database?schema=my_schema
and
DATABASE_URL=postgres://username:password@yourdomain.com/database?search_path=my_schema
but neither worked. Is there any idea?

It should be doable with --include-pattern=my_schema.*. That option appears to be broken at the moment, but it should be fixed for the next release.

Try it out in<EMAIL_ADDRESS>

Just tried with<EMAIL_ADDRESS>and didn't work.
Created a schema named demo and created table test_foo in both public and demo
CREATE TABLE demo.test_foo (
	id bigserial NOT NULL,
	"name" varchar(64) NULL,
	is_enabled bool NOT NULL DEFAULT true,
	CONSTRAINT test_foo_pk PRIMARY KEY (id)
);

Generated without --include-pattern:
export interface DemoTestFoo {
  id: Generated<Int8>;
  name: string | null;
  isEnabled: Generated<Boolean>;
}

export interface TestFoo {
  id: Generated<Int8>;
  name: string | null;
  isEnabled: Generated<Boolean>;
}

export interface DB {
  "demo.testFoo": DemoTestFoo;
  testFoo: TestFoo;
}

Generated with --include-pattern 'demo.*':
export interface DemoTestFoo {
  id: Generated<Int8>;
  name: string | null;
  isEnabled: Generated<Boolean>;
}

export interface DB {
  "demo.testFoo": DemoTestFoo;
}

In my use case, there will be multiple schemas with the same structure. I don't want the Demo or demo. prefixes at all.

Right now filtering by schema and removing schema prefixes is possible due to https://github.com/RobinBlomberg/kysely-codegen/pull/56 like that:
kysely-codegen --schema auth --include-pattern 'auth.*'

But passing schema name like this is a bit awkward and inconsistent with .env file config when we define DATABASE_URL.
Maybe we could support another .env var to enable filtering with setting of something like DATABASE_SCHEMA=auth?

I would say that this issue has been fixed with --schema, but I welcome anyone to create a new issue for supporting an env-based schema.


Style number inputs

Thanks, was going to do this myself. While you're at it, do you mind adding type date too?


update modulename

update for the test suite

Hello Robert,
thanks for the PR. Looks good to me!
I noticed that you added a dynu secret. Is this an valid API key?
If it is please remove it and invalidate it on dynu, otherwise your dynu account might get compromised.
Thanks & Regards

Of course this ist not a valid one!
I did not found how to easily replace dynamicly with an environment
variable...
Thao-Nguyen Do @.***> schrieb am Do., 17. Feb. 2022,
18:33:

Hello Robert,
thanks for the PR. Looks good to me!
I noticed that you added a dynu secret. Is this an valid API key?
If it is please remove it and invalidate it on dynu, otherwise your dynu
account might get compromised.
Thanks & Regards
—
Reply to this email directly, view it on GitHub
https://github.com/Dopingus/cert-manager-webhook-dynu/pull/3#issuecomment-1043222887,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEWHASYHUDJAAXOGO2Q4Z43U3UWONANCNFSM5OVHEJUQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Btw, I am workung on all these issued to make everything is workung perfekt.
We, Red Hat, will release very soon a supported Operator based
cert-manager, 1.7.1. This will require that the webhook is also operator
based. The developer promised that only very few things has to bei
updated...
Thao-Nguyen Do @.***> schrieb am Do., 17. Feb. 2022,
19:00:

Merged #3 https://github.com/Dopingus/cert-manager-webhook-dynu/pull/3
into master.
—
Reply to this email directly, view it on GitHub
https://github.com/Dopingus/cert-manager-webhook-dynu/pull/3#event-6092299616,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEWHAS3IUAOVSVA4H6ULOWTU3UZTRANCNFSM5OVHEJUQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***
com>


REXML::ParseException when Partial Has More Than 1 Root

When I include a partial that includes more than one top level element, I get a REXML::ParseException.
My partial is:

Partial content 1

Partial content 2

And here's a partial stack trace:
#<RuntimeError: attempted adding second root element to document>
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/document.rb:95:in `add'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/element.rb:879:in `add'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/child.rb:21:in `initialize'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/parent.rb:13:in `initialize'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/element.rb:59:in `initialize'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/element.rb:877:in `new'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/element.rb:877:in `add'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/element.rb:297:in `add_element'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/document.rb:102:in `add_element'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/parsers/treeparser.rb:33:in `parse'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/document.rb:287:in `build'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rexml/document.rb:44:in `initialize'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/inky-rb-<IP_ADDRESS>/lib/inky.rb:35:in `new'
/Users/keokilee/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/inky-rb-<IP_ADDRESS>/lib/inky.rb:35:in `release_the_kraken'
/Users/keokilee/Code/InkyTest/app/views/inky_mailer/_inky_partial.html.inky:1:in `_app_views_inky_mailer__inky_partial_html_inky__4142895012036450841_70155048147260'

As a workaround, we're just wrapping the content of the partial with an outer div.

I think we need to make inky-rb smart enough to insert a root node when it needs one.  I believe this can be handled by catching exactly this exception and wrapping the content in a div... we can probably even re-remove it after the end.
I'll hopefully have a chance to tackle this shortly, but would happily accept a PR that implements this.
If you want to tackle it, I'd add a test for it in the components_spec and then look at making this modification in the release_the_kraken method of lib/inky.rb.

I think this has been fixed in <IP_ADDRESS>, now that we use nokogiri instead of REXML. @keokilee Is it easy for you to re-test this?

Actually, it's not fixed yet, sorry...

I'm checking with our HTML emails expert about the XHTML/HTML question for emails, but in the meantime I'd thought one possible workaround for this is to wrap everything in a root element prior to parsing that we then strip out after transformation.

@kball Check #19d8f83. Using an HTML parser instead of XML fixes a whole bunch of issues, including this one.

Should be completely addressed (for real! :smile:) with <IP_ADDRESS>. Let us know if that's not the case! Thanks for reporting this.


Labels not showing in Pie chart

Hi,
I'm trying to implement the Amcharts Mobile Library, and as the screenshots below show, the labels for the pie chart fail to render in the IOS Simulator. The labels render fine for the chart on my website. The IOS simulator is for the iphone. Even when I copy and paste the code for the sample Pie Chart in the examples, the labels still dont show up.
This is the code in Objective C. I've tried enabling "labelsEnabled" but that doesn't work either.
AmPieChart *pieChart = [[AmPieChart alloc] init];
pieChart.type = @"pie";
pieChart.theme = @"dark";
pieChart.backgroundColor=@"#3f3f4f";
pieChart.backgroundAlpha=[NSNumber numberWithInteger: 1];
pieChart.percentPrecision=[NSNumber numberWithInteger: 1];
pieChart.depth3D=[NSNumber numberWithInteger: 10];
pieChart.labelsEnabled=YES;
pieChart.balloonText=@"[[title]]: [[num_grams]] grams/day";
NSMutableArray dataProvider = [[NSMutableArray alloc] init];
[dataProvider addObject:@{@"nutrition" : @"Protein", @"calories" : @(rounded_protein4), @"num_grams" : @(rounded_protein)}];
[dataProvider addObject:@{@"nutrition" : @"Carbs", @"calories" : @(rounded_carb4), @"num_grams" : @(rounded_carb)}];
[dataProvider addObject:@{@"nutrition" : @"Fats", @"calories" : @(rounded_fat9), @"num_grams" : @(rounded_fat)}];
[dataProvider addObject:@{@"nutrition" : @"Alcohol", @"calories" : @(rounded_alcohol*7), @"num_grams" : @(rounded_alcohol)}];
pieChart.dataProvider = dataProvider;
pieChart.valueField = @"calories";
pieChart.titleField = @"nutrition";
AmMobileChartView *new_nutrition_graph=[[AmMobileChartView alloc]initWithFrame:CGRectMake(0, 0, 300, 250)];
///////this is in tableview, and each cell has an AmMobileChartView named nutrition_chart
cell.nutrition_chart= new_nutrition_graph;
[cell.nutrition_chart setChart:pieChart];
[cell.nutrition_chart drawChart];
//////////nutrition_chart_view is the UIView container that will hold the nutrition_chart
[cell.nutrition_chart_view addSubview:cell.nutrition_chart];

Found the issue. It seems like the graphs have a size limit, at which point they start omitting the labels. I've found that it depends on the device, as well as the graph. For example. iPhone 5's pie chart has a limit of ~415 pixels. If you go over that, the labels disappear. My workaround was making the frame bigger than 415, even though it goes off screen, and then shifting the frame, margins, and center of the graph.


Add support for mysql2/promise

When using library mysql2, instrumentation does not pickup calls made through mysql2/promise (https://www.npmjs.com/package/mysql2#using-promise-wrapper).
AFAIK, it does not look for promises require (https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/plugins/node/opentelemetry-instrumentation-mysql2/src/instrumentation.ts#L51), only 'mysql2'.
This is only for Node since databases are not normally used in browsers.
Weekly Downloads: ???
I would expect a large percentage of people using mysql2 would want promises. Over time. the usage of promises will continue to increase.
Versions: latest

This might be as simple as intercepting the require and adding tests. As far as I know the promise wrapper uses the same internals we already patch.

@mpoitras Did you make any progress on instrumenting mysql2?

This seems to be working now?


canBackstab() fails on unlinked tokens who's actor lacks a cclass.

HMActor.canBackstab() is being specifically called for the actor, disregarding token data.

resolved by 9785b65.


Remove deprecated PW API

Deprecated direct calls in PW API is removed.

 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.Ali Haydar Sümer 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.


bump uuid to 1.0

Codecov Report

Merging #1345 (77cc3d8) into main (2e255c4) will increase coverage by 0.23%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1345      +/-   ##
==========================================
+ Coverage   94.05%   94.28%   +0.23%
==========================================
  Files         230      233       +3
  Lines       39350    41868    +2518
==========================================
+ Hits        37010    39475    +2465
- Misses       2340     2393      +53

Impacted Files
Coverage Δ

src/core/segment_id.rs
70.00% <100.00%> (ø)

src/schema/json_object_options.rs
85.71% <0.00%> (-14.29%)
:arrow_down:

src/fastfield/writer.rs
90.12% <0.00%> (-5.84%)
:arrow_down:

src/aggregation/agg_req_with_accessor.rs
93.70% <0.00%> (-4.11%)
:arrow_down:

src/aggregation/agg_req.rs
95.55% <0.00%> (-3.69%)
:arrow_down:

src/error.rs
22.03% <0.00%> (-2.50%)
:arrow_down:

src/aggregation/bucket/range.rs
96.22% <0.00%> (-2.16%)
:arrow_down:

src/schema/numeric_options.rs
95.51% <0.00%> (-1.93%)
:arrow_down:

src/aggregation/intermediate_agg_result.rs
97.00% <0.00%> (-1.88%)
:arrow_down:

src/fieldnorm/writer.rs
98.48% <0.00%> (-1.52%)
:arrow_down:

... and 45 more

Continue to review full report at Codecov.

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


Force Graph Border Does Not Change Dynamically with Screen

BUGS

Force Graph border does not fill out the whole space for the services container.
When the view port height and width change, the border stays the same.

Resolved in PR #84


bottled package installs broken with non-standard install prefix

brew config output
HOMEBREW_VERSION: 3.4.9
ORIGIN: https://github.com/Homebrew/brew
HEAD: d0a0bbef8db45cca303c607037338d02d8eaa8ca
Last commit: 4 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 89bb4ce3b7b86c85a8bd2d60db8fa829e8ff9c1a
Core tap last commit: 9 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /Users/jacob/homebrew
HOMEBREW_REPOSITORY: /Users/jacob/homebrew
HOMEBREW_CELLAR: /Users/jacob/homebrew/Cellar
HOMEBREW_CASK_OPTS: []
HOMEBREW_CORE_GIT_REMOTE: https://github.com/Homebrew/homebrew-core
HOMEBREW_EDITOR: vim
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.8 => /Users/jacob/homebrew/Library/Homebrew/vendor/portable-ruby/2.6.8/bin/ruby
CPU: octa-core 64-bit westmere
Clang: 13.0.0 build 1300
Git: 2.32.0 => /Applications/Xcode.app/Contents/Developer/usr/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 11.6-x86_64
CLT: <IP_ADDRESS>.1.1638488800
Xcode: 13.2.1
Rosetta 2: true

brew doctor output
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Your Homebrew's prefix is not /usr/local.
Some of Homebrew's bottles (binary packages) can only be used with the default
prefix (/usr/local).
You will encounter build failures with some formulae.
Please create pull requests instead of asking for help on Homebrew's GitHub,
Twitter or any other official channels. You are responsible for resolving
any issues you experience while you are running this
unsupported configuration.

Warning: Homebrew's "bin" was not found in your PATH.
Consider setting your PATH for example like so:
  echo 'export PATH="/Users/jacob/homebrew/bin:$PATH"' >> ~/.zshrc

Verification

[X] I ran brew update and am still able to reproduce my issue.
[x] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
We're currently managing local development environments via a bootstrapped homebrew install in a custom location, isolated from users' system brew install. To speed up required package installs, we use the --force-bottle flag to install bottled binaries rather than building from source.
What happened (include all command output)?
Any install using --force-bottle fails with the following error:
~ ➤ homebrew/bin/brew install --force-bottle gettext
==> Downloading https://ghcr.io/v2/homebrew/core/gettext/manifests/0.21
Already downloaded: /Users/jacob/Library/Caches/Homebrew/downloads/66569d4eb02ec67195446d3dcba7d7806a35f88f629a55b43ba53fb1febb1f56--gettext-0.21.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/gettext/blobs/sha256:a025e143fe3f5f7e24a936b8b0a4926acfdd025b11d62024e3d355c10653
Already downloaded: /Users/jacob/Library/Caches/Homebrew/downloads/56f8e2a346bf8ce27f5f2aadbc0b3163e78d8cae1fcfd9f462477cd4b821479f--gettext--0.21.big_sur.bottle.tar.gz
==> Pouring gettext--0.21.big_sur.bottle.tar.gz
Error: Patching failed!  Original and patched binary sizes do not match.
Original size: 68912
Patched size: 68923

Running with --debug additionally spits out a bunch of messages along the lines of
==> Changing install name in /Users/jacob/homebrew/Cellar/gettext/0.21/bin/envsubst
  from @@HOMEBREW_CELLAR@@/gettext/0.21/lib/libintl.8.dylib
    to /Users/jacob/homebrew/Cellar/gettext/0.21/lib/libintl.8.dylib
==> Changing install name in /Users/jacob/homebrew/Cellar/gettext/0.21/bin/gettext
  from @@HOMEBREW_CELLAR@@/gettext/0.21/lib/libintl.8.dylib
    to /Users/jacob/homebrew/Cellar/gettext/0.21/lib/libintl.8.dylib
==> Changing install name in /Users/jacob/homebrew/Cellar/gettext/0.21/bin/msgattrib
  from @@HOMEBREW_CELLAR@@/gettext/0.21/lib/libgettextsrc-0.21.dylib
    to /Users/jacob/homebrew/Cellar/gettext/0.21/lib/libgettextsrc-0.21.dylib

What did you expect to happen?
the package installs successfully just as it does with the default prefix:
~ ➤ which brew
/usr/local/bin/brew
~ ➤ brew reinstall --force-bottle gettext
==> Downloading https://ghcr.io/v2/homebrew/core/gettext/manifests/0.21
Already downloaded: /Users/jacob/Library/Caches/Homebrew/downloads/66569d4eb02ec67195446d3dcba7d7806a35f88f629a55b43ba53fb1febb1f56--gettext-0.21.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/gettext/blobs/sha256:a025e143fe3f5f7e24a936b8b0a4926acfdd025b11d62024e3d355c10653
Already downloaded: /Users/jacob/Library/Caches/Homebrew/downloads/56f8e2a346bf8ce27f5f2aadbc0b3163e78d8cae1fcfd9f462477cd4b821479f--gettext--0.21.big_sur.bottle.tar.gz
==> Reinstalling gettext
==> Pouring gettext--0.21.big_sur.bottle.tar.gz
🍺  /usr/local/Cellar/gettext/0.21: 1,953 files, 19.8MB
==> Running `brew cleanup gettext`...
Disable this behaviour by setting HOMEBREW_NO_INSTALL_CLEANUP.
Hide these hints with HOMEBREW_NO_ENV_HINTS (see `man brew`).

Step-by-step reproduction instructions (by running brew commands)
1. set up a new homebrew install in a custom location following the steps [here](https://github.com/Homebrew/brew/blob/664d0c67d5947605c914c4c56ebcfaa80cb6eca0/docs/Installation.md#untar-anywhere):

mkdir homebrew && curl -L https://github.com/Homebrew/brew/tarball/master | tar xz --strip 1 -C homebrew

use it to install a package with --force-bottle:

homebrew/bin/brew install --force-bottle gettext

Digging around, it looks like this is possibly related to #13138 which landed recently? Installing packages from source seems to be unaffected, so for the time being we're just omitting the --force-bottle flag as a workaround.

CC @danielnachun
Just FYI, @jvandew: we don't provide support for non-default prefixes since there are just too many things that can go wrong with them, particularly if you use the --force-bottle flag [1]. I'll leave this open for now but I suspect another maintainer will close this issue soon.
[1] The --force-bottle flag overrides the relocatability check, and will install a bottle even when we have strong reason to believe that the bottle won't work, so having problems with it is expected.

The reason this bug is happening is because the new patching code we've added will try to replace prefix references with the new prefix in binaries.  It is not supposed to run unless you've set HOMEBREW_RELOCATE_BUILD_PREFIX to a non-empty value and your non-default prefix is shorter than the default prefix for your platform (in your case /usr/local), but I didn't consider the scenario where --force-bottle is used.  This is a simple fix I will make ASAP as this shouldn't be happening.
On a related note, if you are able to use a prefix that is shorter than /usr/local (/opt/hb is the one I've used for a lot of testing) and you set HOMEBREW_RELOCATE_BUILD_PREFIX, you don't need to use --force-bottle anymore and most bottles will work without issue.  This is still a highly experimental feature and there are definitely bottles that don't work, but it's moving us towards greatly reducing the need to build from source for non-default prefixes.
I'm okay with keeping this open until I've fixed this as I do consider this a bug, but I do understand if other maintainers want to close it.

Awesome, thank you for the additional context and looking into a fix. Let me know if there is anything else I can do to help.

I've opened https://github.com/Homebrew/brew/pull/13217 to fix this.  I tested it using --force-bottle without setting HOMEBREW_RELOCATE_BUILD_PREFIX and it did not attempt to do any patching.

Fixed in #13217.

brew developer on && brew update didn't seem to pick up the fix for me, but reinstalling via https://github.com/Homebrew/brew/tarball/master did the trick.
Thanks for quick turnaround!


refromated the paint command, created get_color_name(), formatImg() updated README.md

creating color name averaging function to be able to use Karels revised commands on codeHS.
created a formatting function to auto-size the image to 40x40 pixels to allow for images of any size to be used in the command line
updated README.md

Thank You! I no longer use CodeHS but the changes look good!


Improving book section

The feature I am suggesting is a way to fetch books from the a books API. This would allow users  view books from the books catalog.

Hey @M-A-N-I-K do you know about any book API? If yes I'll assign this issue to you.

Yes there are plenty of API's like google books api and open library API that i can use to fetch books data and display it on page

@M-A-N-I-K I have assigned the issue to you. You can work on it.

Hi @kailashchoudhary11,
I have received the issue that you assigned to me. I will start working on it right away. I will keep you updated on my progress and let you know when I have finished.


RotoGrad: Gradient Homogenization in Multitask Learning

I just noticed PCGrad in the list of optimizers modifiers. I had used this with success in the past on a very specific task. It may be useful.
RotoGrad: Gradient Homogenization in Multitask Learning
Paper: https://openreview.net/pdf?id=T8wHz4rnuGL

Multitask learning is being increasingly adopted in applications domains like computer vision and reinforcement learning. However, optimally exploiting its advantages remains a major challenge due to the effect of negative transfer. Previous works have tracked down this issue to the disparities in gradient magnitudes and directions across tasks when optimizing the shared network parameters. While recent work has acknowledged that negative transfer is a two-fold problem, existing approaches fall short. These methods only focus on either homogenizing the gradient magnitude across tasks; or greedily change the gradient directions, overlooking future conflicts. In this work, we introduce RotoGrad, an algorithm that tackles negative transfer as a whole: it jointly homogenizes gradient magnitudes and directions, while ensuring training convergence. We show that RotoGrad outperforms competing methods in complex problems, including multi-label classification in CelebA and computer vision tasks in the NYUv2 dataset.

A Pytorch implementation can be found in https://github.com/adrianjav/rotograd.

It's now on v2.7.0. Thanks for the request!


Dt 1466 oai unit tests

This adds unit tests for three classes in the OAI harvester.  We can add tests for other classes later -
this was a timeboxed ticket.  This also adds ScalaMock.  There are other mock testing frameworks available if we decide we don't like ScalaMock - I only needed it here for a very simple use case.
This addresses DT-1471.

This looks great to me!
Reminds me that we should get Codacy set up to calculate and display test coverage...

@AudreyAltman do you want to squash these up before we merge?

Squashed, and coday error fixed.  Ready to merge!


feat(api): encrypt/decrypt api

Signed-off-by: seeflood<EMAIL_ADDRESS>
What this PR does:
encrypt/decrypt api
Which issue(s) this PR fixes:

Fixes #794
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

@kevinten10 Hi, I modified the proto according to your proposal.
Please help review it :)


Fix connpool

Issues associated with this PR
Your PR should present related issues you want to solve.
Sign the CLA
Make sure you have signed the CLA
Solutions
You should show your solutions about the issues in your PR, including the overall solutions,
details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases.
And you need to show the ut result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #744 into stable will increase coverage by <.01%.
The diff coverage is 62.5%.

@@            Coverage Diff            @@
##           stable    #744      +/-   ##
=========================================
+ Coverage    51.5%   51.5%   +<.01%
=========================================
  Files         205     205
  Lines       17011   17009       -2
=========================================
- Hits         8761    8760       -1
+ Misses       7353    7352       -1
  Partials      897     897

Impacted Files
Coverage Δ

pkg/stream/sofarpc/keepalive.go
94.11% <100%> (+5.88%)
:arrow_up:

pkg/upstream/cluster/cluster_manager.go
62.3% <57.14%> (-0.91%)
:arrow_down:

pkg/log/logger.go
55.05% <0%> (-1.69%)
:arrow_down:

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 2dac0d9...eeddea1. Read the comment docs.


chore: update .snyk to ignore SNYK-JS-MINIMATCH-3050818

 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.


favicon and iOS icon fixed

I delete the yeoman icons, and replace with de oficial node school stgo for the ios icon and nodeschool.io for the .ico

Also, LGTM
@jorgevenegas FWI

:+1:

Thanks @s0wk !


Alpha Input Validation

Validate the users input to only allow positive integers.

Done! I also fixed UI freezes that I saw happening on Android.
Thanks for the report.


Site Crash events not showing for sites

https://applensv2.azurewebsites.net/sites/HackthonKustoConnector/detectors/sitecrashes?startTime=2016-10-21

cannot reproduce