Check SQLite.framework reference -- iOS?

Summary
I had some trouble getting an initial build to work in Xcode after cloning SQLite.swift.
I had to update the (relative path) reference to SQLite.swift from within Xcode (took an embarrassingly long time because I'm not a daily Xcode user...) but then I ran into another issue: a build failure due to a missing build directory:
Showing All Messages
No such file or directory: '/Users/<username>/Library/Developer/Xcode/DerivedData/rem-gzgveujhyqjmemcyrgrqjanzsmqi/Build/Products/Debug-iphoneos/SQLite.framework/SQLite'

Note: Debug-iphoneos - I don't have anything except the macOS SDKs installed.
I poked around on the internet and tried to read between the lines, but ultimately fixed it by deleting SQLite.framework in the rem target Build Phases section under both Link Binary with Libraries and Embed Frameworks and re-adding it, taking care to add the macOS SDK, not the iPhone version. After that my builds succeeded.
Screenshot:

Additional Info
I'm not sure if it's portable, but the relevant git diff in my .xcodeproj file is:
a/rem.xcodeproj/project.pbxproj b/rem.xcodeproj/project.pbxproj
index 6f2ec5e..9a66d6f 100644
--- a/rem.xcodeproj/project.pbxproj
+++ b/rem.xcodeproj/project.pbxproj
@@ -7,6 +7,8 @@
 	objects = {

 /* Begin PBXBuildFile section */
+		78349C542B3DE8630009C28F /* SQLite.framework in Embed Frameworks */ = {isa = PBXBuildFile; fileRef = 96E66BC32B2F5745006E1E97 /* SQLite.framework */; settings = {ATTRIBUTES = (CodeSignOnCopy, RemoveHeadersOnCopy, ); }; };
+		78349C582B3DE9050009C28F /* SQLite.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = 96E66BC32B2F5745006E1E97 /* SQLite.framework */; };
 		961C95DA2B2E19B30093F228 /* remApp.swift in Sources */ = {isa = PBXBuildFile; fileRef = 961C95D92B2E19B30093F228 /* remApp.swift */; };
 		961C95DC2B2E19B30093F228 /* ContentView.swift in Sources */ = {isa = PBXBuildFile; fileRef = 961C95DB2B2E19B30093F228 /* ContentView.swift */; };
 		961C95E12B2E19B40093F228 /* Preview Assets.xcassets in Resources */ = {isa = PBXBuildFile; fileRef = 961C95E02B2E19B40093F228 /* Preview Assets.xcassets */; };
@@ -19,8 +21,6 @@
 		961C96152B2EBEE50093F228 /* DB.swift in Sources */ = {isa = PBXBuildFile; fileRef = 961C96142B2EBEE50093F228 /* DB.swift */; };
 		969BA2EC2B3D1D46009EE9C6 /* SettingsManager.swift in Sources */ = {isa = PBXBuildFile; fileRef = 969BA2EB2B3D1D46009EE9C6 /* SettingsManager.swift */; };
 		969F3EFF2B3A8C4D0085787B /* HotKey in Frameworks */ = {isa = PBXBuildFile; productRef = 969F3EFE2B3A8C4D0085787B /* HotKey */; };
-		969F3F042B3B70560085787B /* SQLite.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = 96E66BBF2B2F5745006E1E97 /* SQLite.framework */; };
-		969F3F052B3B70560085787B /* SQLite.framework in Embed Frameworks */ = {isa = PBXBuildFile; fileRef = 96E66BBF2B2F5745006E1E97 /* SQLite.framework */; settings = {ATTRIBUTES = (CodeSignOnCopy, RemoveHeadersOnCopy, ); }; };
 		969F3F082B3B7C7C0085787B /* RemFileManager.swift in Sources */ = {isa = PBXBuildFile; fileRef = 969F3F072B3B7C7C0085787B /* RemFileManager.swift */; };
 		969F3F0B2B3CB2110085787B /* Field.swift in Sources */ = {isa = PBXBuildFile; fileRef = 969F3F0A2B3CB2110085787B /* Field.swift */; };
 		969F3F0D2B3CCEC30085787B /* Ask.swift in Sources */ = {isa = PBXBuildFile; fileRef = 969F3F0C2B3CCEC30085787B /* Ask.swift */; };
@@ -162,7 +162,7 @@
 			dstPath = "";
 			dstSubfolderSpec = 10;
 			files = (
-				969F3F052B3B70560085787B /* SQLite.framework in Embed Frameworks */,
+				78349C542B3DE8630009C28F /* SQLite.framework in Embed Frameworks */,
 			);
 			name = "Embed Frameworks";
 			runOnlyForDeploymentPostprocessing = 0;
@@ -201,7 +201,7 @@
 			isa = PBXFrameworksBuildPhase;
 			buildActionMask =<PHONE_NUMBER>;
 			files = (
-				969F3F042B3B70560085787B /* SQLite.framework in Frameworks */,
+				78349C582B3DE9050009C28F /* SQLite.framework in Frameworks */,
 				969F3EFF2B3A8C4D0085787B /* HotKey in Frameworks */,
 			);
 			runOnlyForDeploymentPostprocessing = 0;

Other
OMG thank you for posting this. I've been meaning to take a poke at it, but it's much easier to have a skeleton to work inside since I'm not a Swift/Objective-C user normally.

Nice catch. I wonder if some of the changes in https://github.com/jasonjmcghee/rem/pull/8 fixed it?

Someone just opened: https://github.com/jasonjmcghee/rem/pull/8. I'm taking a look at it now - in a very broken state locally 😅 so it's taking a few.
Any opinions on Cocoapods vs Carthage vs. Swift Package manager vs. local?

I am personally not the biggest fan of Xcode so will always default away from using it but the rational decision is likely to just use XCode's internal package manager.
E.g.,

Can confirm, #8 fixed me (and also fixed some resulting code signing issues I was having and hadn't started to debug yet). 🙇♂


4.3.1 please

Hi there,
Would you mind running a 'rake release' for a 4.3.1
Looks like you've got some useful updates available.
Thank you!

Released a 4.4.0. Thanks for keeping up on this. I meant to get better with this by releasing smaller versions every few PRs.


Collect pandoc urls from github api

Hey @nstrmx  thanks for the PR.
Can you take a look at the failing checks. Do you think it's a PR issue, or something else?

I've limited collected urls to a host processor_architecture at pypandoc/pandoc_download.py:73. Guess I was wrong there, tests run on linux, thus only linux urls are collected. Fixed it (hopefully).

Fixed it (hopefully).

Interestingly @nstrmx  it seems that the single test failing, is actually because of a 403 rate limit.
What would you say we could do about that? Asking because Idon't know that much about GHs api, when used in this way.

If we add support for the GITHUB_API_KEY as an environment variable to be used in requests, will that solve this?

yes, that would be acceptable. But I feel like that if that key isn't set, we should fall back to the old behaviour.
Due to the nature of pypandoc, where it's used a lot in CI/CD workflows, I do think that if it's not set, we should fall back to old behaviour so stuff doesn't break

Should work now.

Great work. Lastly, since we now should have 2 download behaviors, we should probably make a test casefor each behavior


[KO]: update site/ko/tutorials/distribute/save_and_load.ipynb

Fix for #184.

Correct some typos in translation.
Update by nbfmt and nb_code_syn.
Check with Colab.

Thanks

@googlebot I consent.

LGTM, thank you for updating the notebook!


Upgrade ECP CR definition from ECC

This upgrades to using the api (sub)module of enterprise-contract-controller.
If this goes after #222 it needs to be updated to remove the kcp-related replace statements from go.mod.

hmm I was hoping for a drastic reduction in indirect dependencies! Looks like no change?

hmm I was hoping for a drastic reduction in indirect dependencies! Looks like no change?


Google Map integration

Hello, Thank you for such an amazing library. I used this to show google map on this. But when i embade google map on the same. It seems to have some problem. It splits the whole google map into number of vPanels i define.
Here is my example code i am using:
(function() {
  document.addEventListener('DOMContentLoaded', function() {

    if (!OriDomi.isSupported) {
      document.getElementById('unsupported').style.display = 'block';
      return;
    }
     var folded = new OriDomi('.demo3', {
        maxAngle: 60,
        vPanels: 5,
        // ripple: true
      }).accordion(60, 'left');

      var elemt = document.getElementById("demo3");
      elemt.addEventListener("mouseenter", function(){
          folded.accordion(100, 'left').unfold();
      }, false);
      elemt.addEventListener("mouseleave", function(){
          folded.accordion(60, 'left');
      }, false);
  });
}).call(this);

accordion(-20)

fold

Please let me know if i am doing something wrong in here. Or is there any solution so after unfold it will behave as a single google map element.
Looking forward to know this!
Thank you!

Unfortunately this isn't a great use case for OriDomi since behind the scenes it's cloning the elements according to the number of panels you define. It might work for simple static map embeds but any map interactivity won't work. Sorry for the bad news.


Feature/bimpm

#183 Not done yet.

Codecov Report

Merging #423 into 2.0 will decrease coverage by 6.23%.
The diff coverage is 22.22%.

@@           Coverage Diff            @@
##            2.0     #423      +/-   ##
========================================
- Coverage   100%   93.76%   -6.24%
========================================
  Files        37       41       +4
  Lines      1135     1234      +99
========================================
+ Hits       1135     1157      +22
- Misses        0       77      +77

Impacted Files
Coverage Δ

matchzoo/layers/__init__.py
100% <100%> (ø)

matchzoo/utils/__init__.py
100% <100%> (ø)
:arrow_up:

matchzoo/layers/multi_perspective_layer.py
14% <14%> (ø)

matchzoo/models/bimpm_model.py
24.24% <24.24%> (ø)

matchzoo/utils/util_backend.py
35.71% <35.71%> (ø)

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 5669260...e5973e7. Read the comment docs.

@lixinsu Cool!
The major difference between the implementation and the paper is the cosine operation. For paper, it decodes as m_k = cosine(W_k \odot v_1, W_k \odot v_1). However, we need to broadcast to tensors. In other words, we need the  cosine operation as below:
def COS(H_P, H_Q; W):
    """
    Params:
      H_P: (time_steps, d)
      H_Q: (time_steps, d)
      W: (l, d)
    Returns: (time_steps(p), time_steps(q), l)
    """
    pass

Maybe the code from the author's repo will help us.
https://github.com/zhiguowang/BiMPM/blob/master/src/match_utils.py#L101-L132

Add me to the reviewers once you think this branch's ready.

@lixinsu @uduse Pls help to review this branch.
The basic framework is ready but I am worried about it.


not connecting to jolly mec agua iot anymore since a few days

Anyone else in the same situation here ?

 :(

I would guess resetting your password is possible via the original app?

Good pick ! I totally forgot obout that terrible app ! :D
Thing is, I just did open it again, and it's working fine ! I can see my stove when using the official app.
I'm a little scared to logout of the app, as I might loose the last security net I have to get back in touch with my stove.


AnimatedPlotterly doesn't accept a mix of offset-naive and offset-aware datetimes

Title. Doing this will lead to a TypeError when Stone Soup tries to do arithmetic using the datetimes. This can be fixed in a couple of lines by converting using the tzinfo property, maybe giving a warning if that's necessary?

So this issue will be caused with the input data. If all data that is read into Stone Soup objects is TZ aware, this shouldn't be an issue. Having mix of both will cause issues in a number of places, not just with plotter.

OK, fair enough.


P.R. China is not parsed as a country

Given the following query string:
MAOMIAO DEPT OF PROMOTION CENTER FOR INTERNATIONAL COMMERCE & BUSINESS ROOM 305 ZODIAC EXTRAS COURT 36 BAOSHAN JIUCUN, BAOSHAN DISTRICT 201900 SHANGHAI P.R. CHINA
the following was parsed:
[
{
"label": "house",
"value": "maomiao dept of promotion center for international commerce & business room 305 zodiac extras court"
},
{
"label": "house_number",
"value": "36"
},
{
"label": "suburb",
"value": "baoshan"
},
{
"label": "city",
"value": "jiucun"
},
{
"label": "state_district",
"value": "baoshan district"
},
{
"label": "postcode",
"value": "201900"
},
{
"label": "city",
"value": "shanghai"
},
{
"label": "state",
"value": "p.r."
},
{
"label": "country",
"value": "china"
}
]
Note that "p.r" was parsed as a state rather than as part of the country.

Generally for toponyms that libpostal gets incorrect, the first place to look is OSM, which doesn't appear to have that name variant. Feel free to add it under alt_name:en and it will get picked up by libpostal on the next import.

I've added the alternate names.

Awesome, thanks!


Are Disable / Enable charger safe to use?

I have a Hyundai Kona. Charger current limiting doesn't work with that (at least not while the car is plugged in). Neither does start/stop charging.
But disable/enable works just fine! So I've made a "smart night charge" mode based on that.
It's quite easy to do as well, since you have a HA switch for it!
Question: Are there any problems with this way to stop/restart charging?

I am a bit puzzled by what you are wrting.
Please try to describe in more detail what you have tried and why it is not working.
I.e. what service for what limit etc have you tried called?
Did you get an error message in your logs?
Does the corresponding sensor for that limit change after you called the service?
I assume you are talking about the "is_enabled" switch?
It will end up calling the https://api.easee.cloud/api/chargers/{id}/settings API and change the "enabled" setting to true or false.
I am honestly not sure what that setting does, it may write to flash memory which would be a bad thing.
Anyway, it is not the recommended way to control the charger, unless you have some sort of specific requirement you should use the circuit current limit or the pause/resume.
I.e the easee.set_circuit_dynamic_limit, easee.pause or easee.resume service.

Yes, it's the is_enabled switch.
When I use set_charger_dynamic_limit with current=10, charging stops and status changes from charging to awaiting_start, and reason_for_no_current=undefined.
Then, after I try start or resume, nothing happens at the charger.
The only way to get the charger back to life is a restart.
After the restart, I've tried stop: No response.
But: pause/resume works!
After pause, it pauses, and the reason_for_no_current is max_dynamic_charger_current_too_low, and that's strange, but
resume restarts it again anyway, so my automation now uses pause/resume and works just fine.
My car is a Hyundai Kona 2020.

Pausing sets the dynamic charger current to 0 , hence why you get max_dynamic_charger_current_too_low, totally fine. Any property with "dynamic" in it, is safe because its stored in volatile memory, not written to the flash.
Enabling / disabling it, is more akin to switch it off, i.e. don't allow charging at all. Its not something you should be doing often though. Better to stick with pause / resume.

There is some information on the easee developer page that is kind of required reading to understand what the different limits and functions actually does.
(I guess we should link to this in the readme).
https://developer.easee.cloud/docs/controlling-a-charger
https://developer.easee.cloud/docs/current-limits-and-control
https://developer.easee.cloud/docs/load-balancing


PowerToys Run to execute Powershell Code

Description of the new feature / enhancement
Allow execution of Powershell Cmdlets from within PowerToys Run
Scenario when this would be used?
executing quick powershell commands.  It currently works for Cmd functions, however Powershell has replaced Cmd as the default windows 10 shell and has far more capability.
Supporting information
Executing Powershell commands from the PowerToys Run should be self-explanitory as far as what ability this would provide.  Cmd but better.

Thanks for the feedback! This should be doable in the settings today for the shell plugin. I may be missing the use case, but I'm pretty sure you're covered by just changing the setting within PowerToys:

If you change that to powershell setting, that will work... although then you do have to choose between behaviors... may be nice to be able to do both easily.
Does this help with your ask?
/needinfo


Fix Cyrillic TOC

Short description

This PR tries to solve the problem that table of content in cyrillic letters is not shown in exported PDF.
Though there is an on-going discussion about PDF (library), I open this PR because this issue is probably urgent for the current situation.
Proposed changes

Replace the font set with a version which includes also cyrillic.
Raleway font is used as before, but this does not support greek letters (also as before). The problem stays unsolved for greek letters but will be sloved for cyrillic.

Side effects

so for not known

Resolved issues

Fixes: #1532

Pull Request Review Guidelines

@JoeyStk Could you please provide a description about how we can reproduce this? I did the same and had no issue with creating two russian pages that appear in the TOC.

I'm having this issue only in Ukrainian for specific page titles, not all of them. That's what I think is so strange. The two pages that broke the TOC were "Ласкаво просимо до міста Аугсбург" and "Відомство у справах іноземців". I copied their content from the app

Since both of you did not seem to have any issues with Russian language, I did not check again, but I tried to reproduce @JoeyStk 's issue with Ukrainian. For me it worked fine:

I also just copied the whole content blocks from the app. I tried a few more combinations, but could not break it.

@JoeyStk @ulliholtgrave @membralala
Thank you for the test and report!
It's a strange bug 🤔 I'll do a further research on it.
@JoeyStk
Entry to Changelog is done.

@timoludwig Thank you for your feedback! Yes, that seems to fix the issue! I'm sorry if I cause some unnecessary research/troubleshooting.  I guess that this is then ready to be merged 🎉 And fix the conflicts of course :D

Done!


API: Publish update slow since 0.9.6 / failing on BadGateway

After upgrade from 0.9.5 to 0.9.6, publish update of large collection takes extremely long time (respectively it fails before it completes on 504 Bad Gateway on side of Nginx proxy).
I have noticed it calls xz --decompress --stdout as a part of this process which seems to be new in this release and connected with slow down. What was exactly changed?
Raising timeouts for both proxy and API client didn't help ending up with:
2016/02/08 12:20:45 [error] 12951#0: *13 upstream prematurely closed connection while reading response header from upstream, client: <IP_ADDRESS>, server: apt01.prd.hv1.cloudlab.cz, request: "PUT /api/publish/nightly/trusty HTTP/1.1", upstream: "http://<IP_ADDRESS>:8080/api/publish/nightly/trusty", host: "apt01.cloudlab.cz:8081"

Request looks like this:
DEBUG:aptly.client:PUT http://apt01.cloudlab.cz:8081/api/publish/nightly/trusty, data={"ForceOverwrite": false, "Snapshots": [{"Component": "galera-s9s", "Name": "galera-s9s-1440419251"}, {"Component": "tcp-salt", "Name": "tcp-salt-1454695280"}, {"Component": "juno", "Name": "_juno-1454929768"}, {"Component": "extra", "Name": "_extra-1454929768"}, {"Component": "liberty", "Name": "_liberty-1454929768"}, {"Component": "tcp", "Name": "_tcp-1454926929"}, {"Component": "kilo", "Name": "_kilo-1454929768"}, {"Component": "security", "Name": "trusty-security-1454929528"}, {"Component": "main", "Name": "_main-1454929768"}, {"Component": "oc21", "Name": "oc21-1440419251"}, {"Component": "oc22", "Name": "oc22-1447341942"}]}

Everything works fine again after downgrade.

Problem is Contents generation, it's very slow... Probably it should be disabled by default

Is it possible to disable/enable it over API?
I don't see this option in API documentation: http://www.aptly.info/doc/api/publish/

yeah, that's missing in 0.9.6, I'll add that option and probably disable it by default in config

Contents publishing can now be disabled with "SkipContents": true in publish APIs.

aptly_0.9.7~dev+17+gffafed4 nightly build includes this fix

Great, thank you :-)
Will implement the option in python-aptly's publisher.

Option --publish-contents introduced in tcpcloud/python-aptly 0.7.2


Changing licence

Codecov Report

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

@@           Coverage Diff           @@
##             master    #28   +/-   ##
=======================================
  Coverage       100%   100%
  Complexity       66     66
=======================================
  Files            21     21
  Lines           332    332
=======================================
  Hits            332    332

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 886dd9e...f10178d. Read the comment docs.


Could upstream be merged into this fork?

Hi Pechy,
Could you possibly merge upstream back into this branch as there's also been some performance improvements happening there.

It looks like this has been done in https://github.com/struxoje/chiapos/commits/main but I have not tried it

Ok thanks :)

Well, I would still like this to be merged here before @pechy does more optimization @struxoje just does not have in his/her repo.

Hi, I now rebased on top of upstream (instead of merging, should make merge requests for official chiapos easier). On both main and combined branches.

Hi Pechy, you forgot to deallocate thread_pool ...after finish plotting seems 4 thread are active in system
in chiapos.cpp you call thread_pool pool(4);
where you close these threads?
thanks.


Document IPv6-only install

Currently the docs site seems to mostly have IPv4 things, and I'm having issues trying to get routing working without an tunnel, and without IPv4 enabled.
Helm install command: helm install cilium cilium/cilium --version 1.12.1 --namespace kube-system --set operator.replicas=1 --set ipv4.enabled=false --set ipv6.enabled=true --set ipam.mode=kubernetes --set tunnel=disabled --set ipv6NativeRoutingCIDR=2001:67c:1104:c000::/56 --set autoDirectNodeRoutes=true
Nodes have addresses in 2001:67c:1104:ac::/64 prefix and cluster cidr is that /56 one so have I understood this right?
cilium-sysdump-20220912-222822.zip

@samip5 I am currently using
version=1.12.2

helm upgrade --install cilium cilium/cilium --version $version \
  --namespace kube-system \
  --set ipv4.enabled=false \
  --set ipv6.enabled=true \
  --set enableIPv6Masquerade=false \
  --set bgpControlPlane.enabled=true \
  --set ipam.operator.clusterPoolIPv6PodCIDRList=2a0a:e5c0:13:cccc::/108

Something that also hasn't been documented is the proper/maximum pool size, however a /64 is "too big".
My setup is currently blocked by #21538, as the hosts are IPv6 only.

Something that also hasn't been documented is the proper/maximum pool size, however a /64 is "too big".

That cames from the Kubernetes itself, and it's documented in Kubernetes own site.

--set bgpControlPlane.enabled=true \

Unfortunately, one cannot have that and MetalLB in bgp mode.. :(

Not stale

+1

Have a look in this options:
https://github.com/cilium/cilium/blob/fb7e01b4875cbb47b9c8cec8c4da3dfd96ba6490/.github/workflows/tests-smoke-ipv6.yaml#L122


Add a plugin to enable pygments in 3bmd

Could you add description of what it does and how to use it in docs/plugin-use.md?

Sorry, I forgot about that. Done now.


Delayed applications can appear to be running

Originally discovered in this environment:
10-node cluster, Mesos 0.25, Marathon 0.14.0, Marathon UI 0.14.2
To reproduce:
Create two app configurations which are greedy of resources (eg require a large number of CPUS). Scale both to zero, then scale both to thousands of instances.
Repeatedly refresh the app list. The task count will be correct (0/3000), but the application will sometimes appear to be delayed (correct behaviour) and sometimes running (incorrect behaviour).

CC @jgehrcke who discovered this bug

I observed an app in status Running which had only one task, which was Staging and not Unknown or Healthy.

I'm struggling to reproduce this bug using the current marathon and marathon-ui master. @glkeiman can you provide any more information on how to force that bad state?

Appears to be behaving itself:

H/T @orlandohohmeier for helping me out with reproducing this one, I've edited the original post to reflect this.
The actual bug seems to be that after a deployment is stopped, even if an instance was never created, the groups endpoint returns instances: 1, which is incorrect.

This was fixed by https://github.com/mesosphere/marathon-ui/issues/733


add generate-cask-ci-matrix command

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

This is a draft of moving the commands that generate the matrixes for our homebrew-cask CI to a DevCmd in Brew.

Testing this locally using the CodeQL CLI, the errors relate to the [^\s",] and trailing \s parts of [^\s",]+)"?,?\s* in the DEPENDS_ON_MACOS_ARRAY_MEMBER regex snippet. I'm not super knowledgeable about this regex issue but using possessive quantifiers (++ and *+  respectively) resolves the errors and continues to match in the same way.
We don't have any depends_on macos: calls using an array in homebrew/cask but I'll create a PR to update the regex there first (as a smoke test). I've manually tested the regex locally (in brew irb) to make sure it matches in the same way but better safe than sorry. If nothing explodes, I'll push the change here as well.

The tests aren't exhaustive, I'm not exactly how much coverage we are looking for here, as any major concerns will be revealed in CI which we can't quite replicate in the test suite.

Thanks @bevanjkay! Let's get this merged and iterate in master.


Is the pages router supported in the latest version with web vitals?

Hello, thanks for helping me out :)
is the pages router supported on the latest version and with web vitals? I am asking because the documentation says:
Note Using the Pages Router? Use version 0.*, which will continue to get security patches. Here's the [README for 0.x](https://github.com/axiomhq/next-axiom/blob/v0.x/README.md).

But the github issues seems to suggest otherwise? Also I tested the latest version with the pages router and web vitals and it SEEMS to be working?
Thanks again!

oh, I think that should be 'Not using the App router?'.
The latest version of next-axiom is designed for app router, so can't guarantee that it works properly for pages router, we have the 0.x versions to keep support for it.

Ok, I see. So is the 0.x version guaranteed to support it?


BUG CLIENT SEND STICKER

When I send a sticker to the sticker bot, the bot replies " ⚙ Generating Sticker ⚙"

I need to see the console to see what error it caused

I had the same problem and I found a temporary fix: You can put the codeblock bellow at the start of the checkMessage() method in lib\whatsapp.js and turn off "notifyUrlNotDetected" in the config
if (!config.customActiveCommand.includes(this.message.caption)) { 		return "Url-No-Detected"; 	}

go to the downloaded directory and edit the config.json file, then change active command to false.

@L0neAsura that must be a user friendlier way to do it lmao, thx for telling

actually just realizing my response is for something else, it will make it automatically generate a sticker without putting !sticker.

im trying to fix the same issue as OP, i dont want it to generate a sticker when a sticker is sent, when i put that command that @Zi5han posted, it doesnt generate any stickers at all

so i found a fix i guess here: #48
changed my code to match and it doesnt generate a sticker when someone sends a sticker now.
you can try that.
PS: i know nothing about coding lol, im just doing guesswork


[TESTS] Add secp integration tests

TITLE
Description
Description of the pull request changes and motivation.
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report

Merging #370 (e1729c9) into main (3a7140f) will increase coverage by 0.03%.
The diff coverage is 100.00%.

:exclamation: Current head e1729c9 differs from pull request most recent head 442f58d. Consider uploading reports for the commit 442f58d to get more accurate results

@@            Coverage Diff             @@
##             main     #370      +/-   ##
==========================================
+ Coverage   97.85%   97.88%   +0.03%
==========================================
  Files          44       44
  Lines       11723    11602     -121
==========================================
- Hits        11471    11357     -114
+ Misses        252      245       -7

Impacted Files
Coverage Δ

tests/cairo_run_test.rs
98.42% <100.00%> (+0.07%)
:arrow_up:

src/vm/hints/usort.rs
98.80% <0.00%> (-1.20%)
:arrow_down:

src/vm/hints/keccak_utils.rs
96.38% <0.00%> (-0.32%)
:arrow_down:

src/vm/hints/secp/secp_utils.rs
89.83% <0.00%> (-0.17%)
:arrow_down:

src/vm/hints/cairo_keccak/keccak_hints.rs
98.40% <0.00%> (-0.09%)
:arrow_down:

src/vm/hints/find_element_hint.rs
98.51% <0.00%> (-0.04%)
:arrow_down:

src/vm/hints/secp/signature.rs
98.52% <0.00%> (-0.03%)
:arrow_down:

src/vm/hints/math_utils.rs
99.49% <0.00%> (-0.02%)
:arrow_down:

src/vm/hints/squash_dict_utils.rs
99.87% <0.00%> (-0.01%)
:arrow_down:

src/vm/hints/blake2s_utils.rs
99.70% <0.00%> (-0.01%)
:arrow_down:

... and 16 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Nice one @Juan-M-V Just let me mention some things
* Let's try to avoid writing hints by hand, instead use the function that implements that hint

* Some functions are not directly tested `reduce, verify_zero ,is_zero, ec_mul_inner, div_mod_n, get_point_from_x`. I checked and some are tested because they are called by other functions.  Can you check that all the hint functions are tested?

The functions mentioned are all tested now, after the rework.
https://github.com/lambdaclass/cleopatra_cairo/pull/370/commits/07730b2c50e1abaaf01ff5df2eb2ad250e381809


Release 0.13.0b Error in template path:  Template 'projects/base_project.html' not found

Original setezor/app_routes/pages/project.py line 46
"return aiohttp_jinja2.render_template('projects/base_project.html', request=request, context=context)"
After my corrections it works well:
"return aiohttp_jinja2.render_template('projects.html', request=request, context=context)"

Original setezor/app_routes/pages/project.py line 46 "return aiohttp_jinja2.render_template('projects/base_project.html', request=request, context=context)" After my corrections it works well: "return aiohttp_jinja2.render_template('projects.html', request=request, context=context)"

Dear Pavel Stepashkin, thank you for your interest in our project! We have found out the cause of the problem and already fixed it.


shapelib: 1.4.1 -> 1.5.0

Motivation for this change
https://lists.osgeo.org/pipermail/shapelib/2019-February/000647.html

s in 1.5.0 notes, no longer need 'proj' dep (so drop)

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build xastir navit


WIP: Optimizer closure

What does this PR do?

Fixes #4188 and also fixes #2785
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Fixed https://github.com/PyTorchLightning/pytorch-lightning/issues/4083 too right?

@rohitgr7 Not yet tested, net to get it working first :D

nice. can you add tests that use the mock + count to verify that we perform the expected number of backward passes? also with a non-lbfgs optimizer to make sure we don't break current functionality

@williamFalcon I think @awaelchli is currently looking into this.

Hello @justusschock! Thanks for updating this PR.

In the file tests/trainer/test_trainer.py:

Line 1377:1: E402 module level import not at top of file
Line 1379:83: W292 no newline at end of file

The GPU tests did not run, because of horovod installation failing, so we did not see the problem with amp closure.


Color shorthand value

Is there a way to prevent hex color values written in shorthand from expanding on compile?
i.e. #fff turns into #ffffff
I would like it to remain as #fff
I'm working with a team, and the expansion makes merging the CSS more dicey.

https://github.com/less/less.js/pull/2643
There was a change in the way less compiles. It looks like this issue can be solved by just getting the latest version.


HAWCLike.calc_p_value and bin-dependent top-hat radii for HAWCLike.display

Added method threeML.HAWCLike.calc_p_value, which performs a simple goodness-of-fit calculation for the best-fit model.  Also modified HAWCLike.display to allow different top-hat radii for different bins.  Because the calc_p_value method relies on functionality recently introduced to AERIE (this morning), this revision may break the build tests unless the test version of AERIE is updated.  Then again it may not break anything since the build test doesn't call the method in question.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a5d1a4c). Click here to learn what that means.
The diff coverage is 69.23%.

@@            Coverage Diff            @@
##             master     #240   +/-   ##
=========================================
  Coverage          ?   75.38%
=========================================
  Files             ?      134
  Lines             ?    14475
  Branches          ?        0
=========================================
  Hits              ?    10912
  Misses            ?     3563
  Partials          ?        0

@sammarinelli thanks for this contribution. Could you please elaborate a bit on the method used to compute the p-value?

@giacomov The p-value is calculated by computing D = 2 ln[(saturated likelihood)/(best-fit likelihood)], where the "saturated model" is the model corresponding to setting each bin's expectation equal to its observed count.  The quantity D should then obey a chi^2 distribution with (number of bins) - (number of free parameters) degrees of freedom under the hypothesis that the fit is good, so the p-value is calculated by integrating the chi^2 distribution from D to infinity.  The bin contents are computed by integrating over a top hat rather than pixel by pixel to ensure there are good enough statistics to apply Wilk's theorem.  I didn't try to set this up to interact with threeML/classicMLE/goodness_of_fit.py because I believe that module tries to do the calculation using Monte Carlo rather than analytically.

I implemented the changes requested here and also implemented the relative tests in the master branch. I am closing this.
thanks for the contribution!

Thanks!


Operator name required field error is displayed as an error below the input field, but Config error is show as an error in the modal. So when both errors are matched we can only see the Name one.

Operator name required field error is displayed as an error below the input field, but Config error is show as an error in the modal.
So when both errors are matched we can only see the Name one.

Issue migrated to JIRA.


Atom.Object.defineProperty.get is deprecated.

atom.workspaceView is no longer available.
In most cases you will not need the view. See the Workspace docs for
alternatives: https://atom.io/docs/api/latest/Workspace.
If you do need the view, please use atom.views.getView(atom.workspace),
which returns an HTMLElement.
Atom.Object.defineProperty.get (/Applications/Atom.app/Contents/Resources/app.asar/src/atom.js:57:13)
Object.activate (/Users/Sosoyan/.atom/packages/script/lib/script.coffee:30:9)
Package.activateNow (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:222:19)
<unknown> (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:203:30)

@Sosoyan This is a duplicate. Please see #407 which resolves it.


Give two dict() in chemstry.Reaction() not work

Documentation is clear on format Reaction(react,prod). React and prod are two dicts but is you try to give a dictionary object (like a variable) you'll see an error lambda for checking type.
Example
We try to take stoichiometry:
import chempy
react,prod = chempy.balance_stoichiometry({'H2', 'O2'}, {'H2O'})
print(chempy.chemistry.Reaction(dict(react), dict(prod)))

This will give:
line 577, in check_all_integral
    if v != type(v)(int(v)):
            ^^^^^^^^^^^^^^^
TypeError: Singleton.__init__.<locals>.<lambda>() takes 1 positional argument but 2 were given

This error will not occur if you pass result (we use dict() because belance create a OrderDict) dict in clear:
chempy.chemistry.Reaction({'H2': 2, 'O2': 1}, {'H2O': 2})

Is there a way for bypass? (store dict(react) or dict(prod) in another variable will give (of course) same error.

Interesting, I think SymPy's Integer class might have changed:
>>> type(react['H2'])
sympy.core.numbers.Integer

or it has always been broken, I'm not sure. Either way there's plenty of things to improve on here (PRs most welcome).
But the short term work around is to perform no checks when creating Reaction:
>>> print(chempy.chemistry.Reaction(dict(react), dict(prod), checks=()))
2 H2 + O2 -> 2 H2O

or selective disable the "all_integral" check:
>>> print(chempy.chemistry.Reaction(dict(react), dict(prod), dont_check={"all_integral"}))
2 H2 + O2 -> 2 H2O

Interesting, I think SymPy's Integer class might have changed:
>>> type(react['H2'])
sympy.core.numbers.Integer

or it has always been broken, I'm not sure. Either way there's plenty of things to improve on here (PRs most welcome).
But the short term work around is to perform no checks when creating Reaction:
>>> print(chempy.chemistry.Reaction(dict(react), dict(prod), checks=()))
2 H2 + O2 -> 2 H2O

or selective disable the "all_integral" check:
>>> print(chempy.chemistry.Reaction(dict(react), dict(prod), dont_check={"all_integral"}))
2 H2 + O2 -> 2 H2O

Thank you so much, I'm gonna work on a pull to help you soon as can. I meantime this seem work fine.

No stress! I reported the change in behavior in sympy/sympy#24268 too.

Could we just change chempy/chemistry.py#L577 from
if v != type(v)(int(v)):

to
if v != int(v) and v != type(v)(int(v)):

It does work.

Sounds promising! would you mind opening a pull request?


HacktoberFest: Safer resource handling

I was looking through the code and noted that the file creation here was
at risk of leaving resources open if an exception occurs, so I added a
private helper function to safely handle creating and writing the bytes
to the files.

Nice! Thanks for noticing that!


Using the model, Complete Training and Optimize Hyperparameters

First, I want to thank you for your work.
Reading the future work to be done I got a lil bit confused: Complete Training and Optimize Hyperparameters    isn't training already done? am seeing that U have already used the validation dataset but as a training dataset. Can U tell a lil bit more about the last part of the code "Training and Validation".
I want to try the model on a new article, however am new to tensorflow. I didn't get how to use the model generated on a new input. I used a usual command line model.predict (the one used for sklearn for example) but didn't get a result. Can U lead me to the right command to use to model files generated?

isn't training already done?

Kind of....I guess you could run for more epochs (I don't think I ran till convergence) with more data.

am seeing that U have already used the validation dataset but as a training dataset. Can U tell a lil bit more about the last part of the code "Training and Validation".

Validation is not done by training dataset. Validation is being done on a separate validation dataset after going through an iteration over all training data. (may be better to add validation after every 1000 steps or something)
This section is for training
           cost,prediction,\                 acc, _ = sess.run([cross_entropy,                                    outputs,                                    accuracy,                                    train_op],                                   feed_dict={tf_text: train_batches_text[j],                                              tf_embd: embd,                                              tf_summary: train_batches_summary[j],                                              tf_true_summary_len: train_batches_true_summary_len[j],                                              tf_train: True})
This section is for validation with separate validation data (val_batches_summary)
           cost,prediction,\                 acc, _ = sess.run([cross_entropy,                                    outputs,                                    accuracy,                                    train_op],                                   feed_dict={tf_text: train_batches_text[j],                                              tf_embd: embd,                                              tf_summary: train_batches_summary[j],                                              tf_true_summary_len: train_batches_true_summary_len[j],                                              tf_train: True})

Can U tell a lil bit more about the last part of the code "Training and Validation".

What do you want to know in particular.

Keep the model descriptions (the tensorflow functions and everything) the same,
use
       train_saver.restore(sess, 'Model_Backup/Seq2seq_summarization.ckpt')         sess.run(tf.global_variables())         sess.run(tf.tables_initializer())
to restore saved variables (after they are saved).
and then instead of training/vallidation run something like:
prediction, = sess.run(outputs, feed_dict={tf_text: your_text, tf_embd: embd, tf_train: False})
in your_text you put your own personal data with But the data must be reshaped to [batch_size, sequence_size] format, if it's just one data use batch_size as 1, and the data must be vectorized
use something like your_text = [vocab2idx.get(word,vocab2idx['']) for word in your_text]
Then reshape to [1, sequence_size].

Thank you so much for ur reactivity and detailed answer.

What do you want to know in particular.
"cost, prediction,\ acc = sess.run([cross_entropy, outputs, accuracy], feed_dict={tf_text: val_batches_text[i], tf_embd: embd, tf_summary: val_batches_summary[i], tf_true_summary_len: val_batches_true_summary_len[i], tf_train: False})"
is used for validation but isn't validation supposed to be using the model generated by the training? in which part of this command line is it calling the trained model?

Thank you a lot for your detailed answer about using the model, I have tried a lot before asking, I'll try out your solution and get back to you.

is used for validation but isn't validation supposed to be using the model generated by the training?

validation is used for model selection. we check validation performance periodically while training (we don't train on validation, only check) to decide when to checkpoint, when to save, when to stop training etc.
Test set is finally used in the end. I didn't implement testing. But one should.

in which part of this command line is it calling the trained model?

       saver.restore(sess, 'Model_Backup/Seq2seq_summarization.ckpt')
        sess.run(tf.global_variables())
        sess.run(tf.tables_initializer())

This part does restores the trained model from checkpoint.
            cost,prediction,\
                acc, _ = sess.run([cross_entropy,
                                   outputs,
                                   accuracy,
                                   train_op],
                                  feed_dict={tf_text: train_batches_text[j],
                                             tf_embd: embd,
                                             tf_summary: train_batches_summary[j],
                                             tf_true_summary_len: train_batches_true_summary_len[j],
                                             tf_train: True})

This part does the training.

Do you have any idea why it's showing this error " 'train_saver' is not defined" each time I wanna use "time_saver"? (even answering [y] to ur code is showing this error).

Sorry; typo. It should be "saver" not "train_saver", since I used "saver = tf.train.Saver()".
I didn't notice the error because I think I never tried to restore saved points with y.  I will update the repo.

in which part of this command line is it calling the trained model

For a bit more details:
This part initializes the tensorflow variables if you are not restoring any pre-saved variables.
        init = tf.global_variables_initializer()
        sess.run(init)
        sess.run(tf.tables_initializer())

Otherwise this part restores pre-saved variables when answering "y":
       saver.restore(sess, 'Model_Backup/Seq2seq_summarization.ckpt')
        sess.run(tf.global_variables())
        sess.run(tf.tables_initializer())

After this the code will load the model with its variables in the memory. You don't need to call the trained models explicit anymore, because the model graph and its variables should be already loaded. The previous cells with the tf functions and variables should generate the tf graph,
"with tf.Session() as sess:" should start a tf session with the specific tf model graph and the above codes should initialize/restrore the variables.
            cost, prediction,\
                acc = sess.run([cross_entropy,
                                outputs,
                                accuracy],
                                  feed_dict={tf_text: val_batches_text[i],
                                             tf_embd: embd,
                                             tf_summary: val_batches_summary[i],
                                             tf_true_summary_len: val_batches_true_summary_len[i],
                                             tf_train: False})

What this part does is compute the specified tensorflow terms (cross_entropy, outputs, accuracy) under the current session. cross_entropy, outputs, accuracy refers to pre-defined terms from previous cells. To compute does terms, tensorflow library automatically runs any other model components that are necessary to compute them. For example, cross_entropy at least requires you to compute this first:
tf.nn.sparse_softmax_cross_entropy_with_logits(
        labels=tf_summary, logits=decoder_outputs)

for this you need tf_summary data, and the decoder_outputs. But for the decoder outputs you have to essentially run the whole model. Tensorflow library will automatically take care of that and run the necessary components. Same for computing 'accuracy', and 'outputs'.
Note that when using validation as in this case, we are not using 'train_op' in sess.run. Computing 'train_op' includes computing gradients and running the optimizer -essentially doing backpropagation based updates (in other words, training). So if you include 'train_op' or not determines whether you are training the model, or just checking it.
Other than that, to compute cross_entropy, outputs, and such you also need to feed necessary data to the model. tf_summary, tf_text, tf_embd are placeholders - i.e they 'hold' the 'place' for certain datastructures based on which the model graph is created. During training, testing, validation or prediction you have to 'fill' in the places held by placeholders to actually make the model work.
So cross_entropy requires tf_summary data (so you need to feed it). Cross_entropy also requires decoder_outputs which in turn requres tf_text (the input data) so you have to feed data to tf_text too. Similarly other placeholders are need to be fed with necessary data. This is the explanation behind this part:
                                  feed_dict={tf_text: val_batches_text[i],
                                             tf_embd: embd,
                                             tf_summary: val_batches_summary[i],
                                             tf_true_summary_len: val_batches_true_summary_len[i],
                                             tf_train: False})

When making a prediction  (predicting a new summary) without having any label data (labeled summary), you don't have the tf_summary data or the tf_true_summary_len data. You won't be able to compute accuracy or cross entropy either. In that case, you will be only computing the outputs of the model based on tf_text. tf_embd, and tf_train will be necessary to feed with data as well though.

Didn't want to get back to you until searching more.
But even after trying to find the answer my own, still the part of reshaping is not clear.
Am always trying to predict one summary using the model generated.
So after using this code and generating a vector from my text
S=[vocab2idx.get(word,vocab2idx['']) for word in tokinput]
And this to reshape SA=tf.reshape(S,[1,len(S)])
where S is a list of int (the comment to summarize)
I used this code for prediction:
prediction = sess.run(outputs, feed_dict={tf_text: SA, tf_embd: embd, tf_train: False})
I have received this error:
TypeError: The value of a feed cannot be a tf.Tensor object. Acceptable feed values include Python scalars, strings, lists, numpy ndarrays, or TensorHandles. For reference, the tensor object was Tensor("Reshape_148:0", shape=(1, 23), dtype=int32) which was passed to the feed with key Tensor("Placeholder:0", shape=(?, ?), dtype=int32).
I have tried different formats of text but still the same error.
Can you tell me more about reshaping or do you have any idea why am still getting this error.
Thanks again!

Intuitively, you can think of the Training and Validation portion as 'outside Tensorflow' in a sense, where you merely run tensorflow graph component through tensorflow sess.run, or save parameters with tf.train.saver but everything else is normal python.
So in this case, the reshaping should be through something like numpy (np.reshape) instead of tf.
What is happing here is that we are reshaping our original data in some multi-array format (could be numpy, or perhaps in a form of a nested list) and then feeding it to the 'tf_xyz' placeholders - essentially we fill the 'held place' by the tf placeholders (like tf_text) with numpy/multi-array data having the same shape structure as defined for the placeholder.
As an example, take note of the  train_batches_text[j] and such, they are not tensor objects but numpy multi-arrays.
You can 'import numpy as np' and then do np.reshape
also this code should be:
S=[vocab2idx.get(word,vocab2idx['<UNK>']) for word in tokinput]

A simple way to do what you want, for testing a single text without any numpy and reshaping is to simply feed '[S]' as 'tf_text: [S]' ([S] would be automatically interpreted as 1 x len(S) shape because of it's a nested-list of that shape)

Thank you so much! That helped (the simple way you suggested didn't work, I guess reshaping is always needed). However, using simply numpy is working perfectly. Thanks a lot for all your detailed answers!


0.5.2 broken throws CLI error: cli.rb:32:in get_command_class': undefined method camelize' for "help":String

Our usage:
oauth \
  --verbose \
   --query-string \
   --consumer-key v484mnii2jyswedm6uo2cfcjay7uy49snws \
   --consumer-secret koay6845che7giy5lr17gnrhckkbhf8h5 \
   --access-token-url https://www.yourstore.com/oauth/token \
   --authorize-url https://wwws.yourstore.com/admin/oauth_authorize \
   --request-token-url https://www.yourstore.com/oauth/initiate \
   authorize

On 0.5.2, the above bash script for Magento, fails:
/home/my-folder/.rvm/gems/ruby-2.2.5/gems/oauth-0.5.2/lib/oauth/cli.rb:32:in `get_command_class': undefined method `camelize' for "help":String (NoMethodError)
    from /home/my-folder/.rvm/gems/ruby-2.2.5/gems/oauth-0.5.2/lib/oauth/cli.rb:20:in `initialize'
    from /home/my-folder/.rvm/gems/ruby-2.2.5/gems/oauth-0.5.2/bin/oauth:11:in `new'
    from /home/my-folder/.rvm/gems/ruby-2.2.5/gems/oauth-0.5.2/bin/oauth:11:in `<top (required)>'
    from /home/my-folder/.rvm/gems/ruby-2.2.5/bin/oauth:23:in `load'
    from /home/my-folder/.rvm/gems/ruby-2.2.5/bin/oauth:23:in `<main>'
    from /home/my-folder/.rvm/gems/ruby-2.2.5/bin/ruby_executable_hooks:15:in `eval'
    from /home/my-folder/.rvm/gems/ruby-2.2.5/bin/ruby_executable_hooks:15:in `<main>'

We thought this issue could be an incompatibility with Ruby 2.2.5, so we downgraded to Ruby 2.0.0, and got the same error.
The work around for this issue is downgrading to 0.5.1.
gem uninstall oauth
gem install oauth -v 0.5.1

@cdcooksey I've detected it. I've had it solved but there's another small underlying issue. I'll get it sorted out and ping you about it. Thanks for the report.

Sounds good.  Thanks, @thejamespinto !

@cdcooksey Thanks for reporting this.
Let me know if there's anything else.
https://rubygems.org/gems/oauth/versions/0.5.3


Show warning if emitting event with invalid name

What problem does this feature solve?
Recently I tried to namespace events (poorly admittedly) by using dots to emit events gallery.share and gallery.success. Of course these event names are invalid, as they cannot be used in event listeners, since dots are used to indicate modifiers.
It would be nice, if the $emit Method would print a warning to the console when using invalid event names.
What does the proposed API look like?
(There are no API Changes needed)

Technically they are not invalid though, because you can listen to them in JavaScript with $on('gallery.share'). But I suppose we can add a note in the docs.


Why iproducer?

It looks as though producer is the only class that inherits from iproducer (my apologies if I missed another). If so, can't you eliminate the iproducer class? It seems like unnecessary complexity.

This is a design pattern that is embraced by some people. It does add some complexity. But it should not add too much difficulty for people to understand the code.

Could you explain the reasoning?

The design pattern separates the interfaces from the implemention. For example, we can easily replace the current impelementation of ObjectaFactory in a separate module without making any changes to the codes in the main program, which emphasizes the communications among the producer and consumers.  When users  work on other algorithms, they can focus on the implementations of the producer and consumers instead of the communication part.
As said in our previous comment, this is the preference of our implementions. It does not prevent others from using simpler design patterns.  It is just a choice of the programmers.

Thank you for the explanation.


Why is CapSolver so expensive.

How am I supposed to use CapSolver if the prices for FunCaptcha is $200-300.

This is not a bug or related issue. It would be more suitable for discussions or the community forum. Nevertheless, I am unsure where you checked the price. According to the official capsolver documentation, the pricing for FunCaptcha is as follows: 'FunCaptcha $1.6/1000 requests $0.6/1000 images'.


psycopg2 - New Meltano users struggle with this error a lot

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/3033
Originally created by @vischous on 2021-10-29 12:49:40

I had one person run into this while trying Meltano (It's pretty difficult to understand if you aren't a python dev what the error actually is)
Does anyone know what the root issue is here, and could we fix the root problem instead of telling pepole to install psycopg2 manually?
Search slack for psycopg2 there's 110 messages across 5 channels, 7 people.

View 12 previous comments from the original issue on GitLab


[Window Walker] Enable tabbing to scroll

Summary of the Pull Request
Allow scrolling the list of windows in Window Walker using the tab key
PR Checklist

[X] Applies to #1806
[X] CLA signed. If not, go over here and sign the CLA
[]Tests added/passed
[] Requires documentation to be updated
[X] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Validation Steps Performed
Ran it and tried tabbing through a list of windows and it works fine.

Code looks good.
Tested and work as expected


Improve message dispatching & small refactorings

Hi,
I made some small improvements dispatching messages to a listener. Basically:

Drop locks early, especially drop them before the dispatch ie. on_receive call
Dispatch is done on a separate task so that a slow listener does not slow down.
Todo to handle the connection drop case correctly.

Cheers,
Jan

Hey @jjj-vtm -- sorry, I missed this! I'll make some time to review

No problem, I merged the changes.


Flutter build apk --release fails with Kotlin compilation error in MainActivity.kt

Steps to reproduce
When running flutter build apk --release, the build fails due to unresolved references in the MainActivity.kt file. This issue appears to be related to the Kotlin dependencies or Flutter Gradle integration. Below is the error log and the relevant configurations.
Error Logs
e: file:///Users/sayedmoataz/StudioProjects/mobile/android/app/src/main/kotlin/com/metachain/matchbuy/MainActivity.kt:3:8 Unresolved reference: io
e: file:///Users/sayedmoataz/StudioProjects/mobile/android/app/src/main/kotlin/com/metachain/matchbuy/MainActivity.kt:5:21 Unresolved reference: FlutterActivity

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:compileReleaseKotlin'.
> A failure occurred while executing org.jetbrains.kotlin.compilerRunner.GradleCompilerRunnerWithWorkers$GradleKotlinCompilerWorkAction
   > Compilation error. See log for more details

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

BUILD FAILED in 49s
Running Gradle task 'assembleRelease'...                           51.5s
Gradle task assembleRelease failed with exit code 1

Steps to Reproduce:

Create a Flutter project with the following Android configuration (detailed below).
Run the command flutter build apk --release.
Observe the build failure.

Expected results
The project should compile successfully when running flutter build apk --release.
Actual results
The build fails with unresolved references for:

io in the MainActivity.kt file.
FlutterActivity in the MainActivity.kt file.

Code sample
code sample
setting.gradle:
pluginManagement {
  def flutterSdkPath = {
    def properties = new Properties()
    file("local.properties").withInputStream { properties.load(it) }
    def flutterSdkPath = properties.getProperty("flutter.sdk")
    assert flutterSdkPath != null, "flutter.sdk not set in local.properties"
    return flutterSdkPath
  }()

  includeBuild("$flutterSdkPath/packages/flutter_tools/gradle")

  repositories {
    google()
    mavenCentral()
    gradlePluginPortal()
  }
}

include ":app"

android/build.gradle:
buildscript {
  ext.kotlin_version = '1.9.0'
  repositories {
    google()
    mavenCentral()
  }
  dependencies {
    classpath 'com.android.tools.build:gradle:8.2.1'
    classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
    classpath 'com.google.gms:google-services:4.4.2'
  }
}

plugins {
  id 'org.jetbrains.kotlin.jvm' version '1.9.0' apply false
}

allprojects {
  repositories {
    google()
    mavenCentral()
  }
}

rootProject.buildDir = "../build"
subprojects {
  project.buildDir = "${rootProject.buildDir}/${project.name}"
  evaluationDependsOn(":app")
}

tasks.register("clean", Delete) {
  delete rootProject.buildDir
}

android/app/build.gradle:
plugins {
  id 'com.android.application'
  id 'kotlin-android'
  id 'com.google.gms.google-services'
  id "dev.flutter.flutter-gradle-plugin"
}

android {
  namespace "com.metachain.matchbuy"
  compileSdk 34
  ndkVersion flutter.ndkVersion

  compileOptions {
    sourceCompatibility JavaVersion.VERSION_1_8
    targetCompatibility JavaVersion.VERSION_1_8
  }

  kotlinOptions {
    jvmTarget = '1.8'
  }

  sourceSets {
    main.java.srcDirs += 'src/main/kotlin'
  }

  defaultConfig {
    applicationId "com.metachain.matchbuy"
    minSdkVersion 26
    targetSdkVersion flutter.targetSdkVersion
    multiDexEnabled true
    versionCode flutterVersionCode.toInteger()
    versionName flutterVersionName
  }

  buildTypes {
    release {
      signingConfig signingConfigs.debug
    }
  }
}

flutter {
  source '../..'
}

dependencies {
  implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8:$kotlin_version"
  implementation platform('com.google.firebase:firebase-bom:33.7.0')
  implementation 'com.google.firebase:firebase-analytics-ktx'
}

gradle-wrapper.properties:
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-8.2-all.zip

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Flutter build apk -v
flutter build apk -v pastebin

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.24.0, on macOS 14.3 23D56 darwin-x64, locale en-EG)
    • Flutter version 3.24.0 on channel stable at /Users/sayedmoataz/fvm/versions/3.24.0
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 80c2e84975 (5 months ago), 2024-07-30 23:06:49 +0700
    • Engine revision b8800d88be
    • Dart version 3.5.0
    • DevTools version 2.37.2

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/sayedmoataz/Library/Android/sdk
    • Platform android-35, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • 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
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)

[✓] VS Code (version 1.96.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.102.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 14.3 23D56 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 131.0.6778.141

[✓] Network resources
    • All expected network resources are available.

• No issues found!

I'm tried to check this answer but it doesn't working for me

Thanks @darshankawar  for your help
I've already done the gradle sync many times today but it always builds successfully and there is no problem with syncing
new distributionUrl
distributionUrl=https\://services.gradle.org/distributions/gradle-8.4-all.zip
sync is done successfully
problem still exist

Please check https://docs.flutter.dev/release/breaking-changes/kotlin-version to see if you have same config or update as mentioned in it to see if it helps to resolve the case or not.

thanks @darshankawar for your help
I'm trying to use ext.kotlin_version = '2.0.0' and also trying to run flutter upgrade to 3.27.1
but it still the same problem
flutter build apk --release
e: file:///Users/sayedmoataz/StudioProjects/mobile/android/app/src/main/kotlin/com/metachain/homely/MainActivity.kt:3:8 Unresolved reference 'io'.
e: file:///Users/sayedmoataz/StudioProjects/mobile/android/app/src/main/kotlin/com/metachain/homely/MainActivity.kt:5:21 Unresolved reference 'FlutterActivity'.
e: file:///Users/sayedmoataz/StudioProjects/mobile/android/app/src/main/kotlin/com/retrytech/homely/MainActivity.kt:3:8 Unresolved reference 'io'.
e: file:///Users/sayedmoataz/StudioProjects/mobile/android/app/src/main/kotlin/com/retrytech/homely/MainActivity.kt:5:22 Unresolved reference 'FlutterActivity'.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:compileReleaseKotlin'.
> A failure occurred while executing org.jetbrains.kotlin.compilerRunner.GradleCompilerRunnerWithWorkers$GradleKotlinCompilerWorkAction
   > Compilation error. See log for more details

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

BUILD FAILED in 8m 30s
Running Gradle task 'assembleRelease'...                          513.4s
Gradle task assembleRelease failed with exit code 1

there is a something strange for me, I don't know what is the retryTech or homely why there are folders for them!!!


Fix initialise_driver to work in monitored mode

We should probably look at whether we are using the managed_mode flag correctly in blockdevice_zfs.py:BlockDeviceZfs.initialise_driver
Currently, in monitored mode we don't stop the services (zfs.target/zfs-import-scan/zfs-import-cache/zfs-mount), that was a patch created and implemented from the conversations we had with Don Brady regarding protective measures to avoid undesired consequences when importing zpools Read-only. We should probably be disabling (and stopping, which it doesn't look like we do at the moment!) those services in monitored mode as we still do the import-export in device scans (not just in managed mode).

@tanabarr can this be closed? Reactive architecture will remove the import-export.


[Question]   最新的preview版本是否删去了服务器切换的功能

Checklist

[X] I have already read docs/Tips.md, but my question is not solved.
[X] My question was not mentioned by others, and it is not a duplicate question.

Description
如题，更新了版本之后发现无法官服切换国际服和b服了。选择国际服只有【一键修复】的按钮，选择b服启动的同样是官服。
Starward Version
0.11.7-preview.4
Windows Version
19045.4529
Log
No response
Additional Context
No response

临时删除了客户端区服切换的功能
0.11.7 Preview 1


Clean up the log4j vs slf4j mess and consolidate the dependency resolution into a gradle script

Coverage remained the same at 0.661% when pulling 99ea255c28f825a399035792e407a4464c2aa49c on mars-lan:log4j into 4c39b546f95eec2ecc3149ca359f0dbff596fb8c on linkedin:master.

Coverage remained the same at 0.661% when pulling 99ea255c28f825a399035792e407a4464c2aa49c on mars-lan:log4j into 4c39b546f95eec2ecc3149ca359f0dbff596fb8c on linkedin:master.


Fix macaddr, optional + computed is problematic

We have had several fixes already for this problem. The key issue is that macaddr is calculated by Proxmox automatically in the majority of cases, but some might want to pass a specific mac address. When using a computed + optional property you have a remaining limitation.
Suppose a resource was created without supplying the mac address manually. The mac address is thus computed. Then an update is executed. Now the mac address is missing. Because we are using an optional + computed property, Terraform sees this as a missing value, and wants to update the resource. However, there is no update, because macaddr was never set, and is still not set.
What is the solution? To include the intent in the code. Use mac_manual if you want a manual address, or have it automatically calculated.

@alterway-ansible Could you have a look at this PR? We need to fix this once and for all.

I was thinking, maybe the variable should be macmanual, without underscore. We do not seem to have a real guide how to name a variable in the project.

I like this approach so far. Does it work for anyone who can test it?

This does not work. I got this error
proxmox_vm_qemu.vm: Creating...

Error: Error creating VM: 400 Bad Request, error status: {"data":null,"errors":{"net0":"invalid format - format error\nnet0.mac_manual: property is not defined in schema and the schema does not allow additional properties\n"}} (params: map[agent:1 bios:seabios boot:cdn cores:4 cpu:kvm64 description: hotplug:network,disk,usb ide2:none,media=cdrom memory:8192 name:worker-1 net0:virtio=7E:48:FF:1B:02:60,bridge=vmbr0,mac_manual=DE:73:91:B4:55:21 numa:false onboot:true ostype:l26 scsihw:virtio-scsi-pci sockets:1 virtio0:media=disk,size=100,file=pve-data:88077017/vm-88077017-disk-0.raw,format=raw vmid:88077017])

  on main.tf line 24, in resource "proxmox_vm_qemu" "vm":
  24: resource "proxmox_vm_qemu" "vm" {

This is my .tf file
resource "proxmox_vm_qemu" "vm" {

  name = "worker-1"

  # Node name has to be the same name as within the cluster
  # this might not include the FQDN
  target_node = "pve2"

  vmid = 88077017
  onboot = true
  preprovision = false
  cores = "4"
  memory = "8192"
  scsihw = "virtio-scsi-pci"
  cpu = "kvm64"
  agent = "1"

  # Setup the disk. The id has to be unique
  disk {
    id = 0
    size = 100
    type = "virtio"
    storage = "pve-data"
  }

  network {
    id = 0
    model = "virtio"
    bridge = "vmbr0"
    mac_manual = "DE:73:91:B4:55:21"
  }

  iso = "none"
}

PS. I'm using your vmid PR combine with this PR.

The PR is updated. At the moment, I cannot test this out. Probably next week. Maybe @winggundamth can try it.

Is there any ongoing work on this? I could really use this feature, as I am currently ignoring changes on the entire network block in order to stop terraform from rummaging with my mac addresses
This is less than ideal because it stops me from changing network settings of existing vms with terraform


Options - option to specify which column(s) to automagically search

The Automagic Search is great.  However as far as I can tell, having the search automagically setup, there is no way to specify which column(s) to search.
As such I propose an option for this in the Options object.
Thanks
Charlie

You can use searchColumns in the options when you initialize the list like this:
var options = {   valueNames: [ 'name', 'lastname','birthdate' ],   searchColumns:[ 'name', 'lastname' ] }; var userList = new List('users', options);
In this example you would have 3 fields (name, lastname, birthdate) and the search function would use only name and lastname.

ah, yes, thanks.  easier than I thought.thanks.

just going to point out this isn't mentioned anywhere in the docs.


enableEdgesRendering crash on scene dispose in 4.0.3

scene dispose crashes when using Edgesrenderer in 4.0.3 release (commit 9deb1df98eed29fe2b503c1201f2150dba3ebcb5) because of missing modification made here : 2e96c10092abc1705cbc4c98ee94d4b36ddc4140
It would be nice to add this modification to 4.0 and make a new release.

We do not port fix back (4.1 will be out this Thursday)


Update SQL reserved Key Words list for Postgres 13

Add all key words listed as "reserved" in PostgreSQL v13 docs to the reserved list.
See https://www.postgresql.org/docs/13/sql-keywords-appendix.html
Also adds a link to the docs to make keeping the list up-to-date easier.
Closes #1

Hey @alxndrsn, I made a fork/port of this package. If you submit your PR to our repo, I'll merge it promptly.
Issue: https://github.com/ScaleLeap/pg-format/issues/4

@moltar unless I'm missing something, it looks like your fork already has all the keywords in this PR

@alxndrsn possibly, I did not review them. I "forked" from another repo.


Can't open documentation menu on mobile browser

When I view the documentation at https://tmuxp.readthedocs.io/en/latest/ on a mobile browser, I cannot open the menu because the "Fork me on GitHub" banner is covering the control.

Although the hamburger menu button is visible, it's actually covered by the invisible part of the "Fork me on GitHub" banner.  I was able to reproduce this in Chrome by narrowing the browser window until the view gets switched.

Fixed here and across libtmux/vcspull/libvcs
Thanks for catching this

Perfect! Thank you!


It's not possible to install version 0.3.0

Hi
I removed the previous version and installed version 0.3.0. It seems that the adapter doesn't work correctly.
Here is the installation log with debugging enabled:

Even though the adapter has been installed, there is no entry for the instance.  There are no object entries as well.

Hi
I've done an update today and pushed it to npmjs.com. Tomorrow Version 0.3.0 should be in the latest repo of ioBroker. Can you please do a retest with this version?

Hello
Thanks for a new version.
It doesn't work. Same behaviour. Sorry.

Hi
Weird. Not sure if this is an adapter problem or a problem with the ioBroker installation. This is because the installation of the adapter obviously runs without errors, but then no instance appears. Does the configuration overview appear after the installation of the adapter, like this?

Ok, I made some more tests. So if I add an additional instance, it works as expected and the configuration page pops up. But it seems when I install the adapter no instance was created for the first time. I had to add an instance manually, by clicking the "plus" symbol next to the installed adapter. I currently can't put in my secret because the developer page from SRG is down at the moment. I'll try it later. Thank you for your assistance.

Oh, okay. ...And s soon as the developer site of SRG is online again, the rest will work, too. :)


Tokens not working in Python files

This issue has been moved from a ticket on Developer Community.

Tokens are not working in Python files. Neither custom or default tokens work. To reproduce:

Create or open any Python file
add the line "# TODO token test"
open Task List and this will not show.
do the same with a cpp file and add the line "//TODO token test"
Note the TODO will show in the Task List for the cpp file but not for the py file, even in the same project.

Original Comments
Feedback Bot on 3/28/2023, 06:57 PM:
(private comment, text removed)

Original Solutions
(no solutions)

Hi there, we have checked in the fix recently, see https://github.com/microsoft/PTVS/issues/7151. Could you update to the latest VS version and try again? Please reopen the issue if it persists. Thanks.


Install Bricks

Previously, you had to install a “Brick” (i.e. a decoupled React container component with its own Redux saga, reducer and selectors) by wiring it up in three different locations in your application, and you had to do all this when the application was initialized. There was no way to add Bricks to an application at a later state, preventing optimization practices like code splitting.
This pull request introduces a new package “bricks” with a new class “BrickManager” that allows to install Bricks at any time, by calling a single function.
The demo app was updated to showcase the usage of this new feature. In addition to the existing Hacker News brick, a new demo module was added that shows Gists.

@epegzz Thanks for taking the time to review. Yes, well, it would have been misleading to call it a module, since that's already used for many, many things. I would have preferred “apparillo”, but maybe that term is a bit childish 😀


Acroform checkbox and radio controls

Added fake checkboxes and radio buttons with a pinch of css.
Tested against http://www.cic.gc.ca/english/passport/forms/pdf/pptc153.pdf on Chrome and Firefox:

@timvandermeij I'll take your remarks into account and add unit tests.

I have been looking into this a bit more and I now realize why you chose divs. I forgot that it's not really possible to style input elements other than those with type="text". We do need input elements, however, both for accessibility as well as for easily reading the selected options, so I think your proposal for using hidden inputs is the only possible way to both make this look good and work like regular input elements, so I'm okay with that.

[...] so I think your proposal for using hidden inputs is the only possible way to both make this look good and work like regular input elements, so I'm okay with that.

My suspicion is that using hidden elements will unfortunately do nothing to alleviate the keyboard accessibility problems!
What I'd suggest is to use regular HTML form elements for now (e.g. checkbox, radiobutton, etc.), since they should be easy to implement and can be improved later on. (Given that there's still a number of major roadblocks to enabling forms by default, e.g. printing/saving the entered data and Appearance stream support.)
However, one possible way forward instead of using HTML form elements (e.g. checkbox, radiobutton, etc) could perhaps be to utilize <button> elements instead, since they are possible to style as you want and should still give proper accessibility.

Right, it's a bit unfortunate, but the current state of the web platform won't really let us do what we want. Using <button> elements might help, along with some "checked" state in the form of a class as is done now.
@benweet Could you check if using <button> elements might help, or if you can improve accessibility (tab the hidden input fields) somehow? It could help determine what the best way to get is here. If those do not work, I also suggest to just use regular checkboxes and radio buttons without styles.

Here is a new commits with hidden inputs. Keyboard accessibility works fine, though I didn't manage to have the highlighting on focus.
@timvandermeij I've taken your comments into account but I still have to add tests.

Added unit tests and CSS outline on focus.

Let's remove the noToggleToOff code if we're not implementing it. Then, please squash the commits into one commit so we can do a new review round. If you're not familiar with squashing commits, here is how to do it easily: https://github.com/mozilla/pdf.js/wiki/Squashing-Commits

Done.
Cheers!

Everything that blocked this PR has landed, so it's now good to rebase and fix up, @benweet. Thanks.

Fixed. Cheers!

Closing as replaced by #7898, which contains the commit from this PR and follow-up fixes so we can land it.


Extend mlx with Additional Optimizers

Issue Type: Enhancement
Proposal:
I would like to propose an enhancement to the framework by adding a variety of optimizers. While the current implementation includes Stochastic Gradient Descent (SGD) and Adam optimizers, the addition of more optimization algorithms could greatly benefit users with diverse requirements.
Potential Optimizers to Include:

Adagrad:
RMSprop:
Adadelta:
Nadam:
AdamW
etc ...

I would appreciate your guidance on the next steps for contributing to this enhancement. Whether it's providing additional details, discussing the proposed optimizers, or collaborating on the implementation, I am eager to contribute to the growth of this fantastic framework.

Thanks, we'd be happy to take PRs for any of the above optimizers. I would suggest starting with a draft PR for one of the above, maybe AdamW and we can go from there?
Generally we follow the PyTorch API for mlx.optim. If there are any involved design questions then we can use this thread to discuss (or even better a draft PR) to discuss.

I'm currently working on a PR to integrate the AdamW optimizer (and other next!). Once ready, I'll submit it for reviews. Your feedback is highly appreciated! 🚀 Thanks for your support! :p

Hi - thanks a ton! But FYI there is a PR for AdamW already https://github.com/ml-explore/mlx/pull/72 !
Maybe try Adagrad / RMSProp / or adding Nesterov style momentum to SGD instead?

On it!


Not able to authenticate using AAD

@saumyabahuk Could you give more info on the Storage Explorer version, platform running on and exact steps you tried to hit the issue?

@saumyabahuk Could you give more info on the Storage Explorer version, platform running on and exact steps you tried to hit the issue?

Hi Jason,
The version of storage explorer being used is 1.14.1
To repro the issue- tried to connect to a gen2 preprod resource using an alternate account on storage explorer. On using the pin for authentication, it gives an error mentioning no valid credentials found, inspite of the alt card working fine.

@saumyabahuk please upgrade to the latest version of Storage Explorer. You can download it at https://www.storageexplorer.com .

Closing due to a lack of response. If you are still seeing this problem please open a new issue. Thanks.


Now serializing also overlays in SceneSerializer

is the loader taking care of them?

Good catch !
I completely forget to check, commit is comming

I tested and it's fine now :)


Move this Repo to the HEC GitHub space

approve the readme, and I will push it. After that if you dont have any branches open, I will move the repo. We will then both need to update origin in our local git repos.

You're approved. Sail Away

done


Fix documentation links in README.md

Goal
It looks like the Gradle plugin documentation was moved from "gradle" to "android-gradle-plugin" folder, updated links accordingly and verified that they work.
Design
N/A
Changeset
Documentation links.
Testing
N/A

Hi @Egorand
Thanks for sharing!
We will look to merge your PR when priorities allow.


[Swagger] Review account properties returned format

Review and document how the account properties endpoint returns the data after the latest changes in catapult-rest.

Solved in #131


Potential request spamming after initial cooldown surpassed

Hey Folks.
We recently had a situation in one of our environments, where usage of the refresh-api somehow overwhelmed our azconfig instances, which then gave HTTP 429 Too-many-requests, occasionally - especially during peak-load times of our app.
We used 1.0.0-preview4 at this time and thx to @lorenzadomat found out, that the refresh-interval might be ignored, somehow.
I then took the opportunity to narrow this even further down. Seems like starting with 1.1.0 / b1cf8099fbf8737de3d38299be2871d4547c7cac , this behavior dissappeared.
Looks like prior 1.1.0, sentinel keys got queried every time, after the initial refreshIntervalInMs has been surpassed - before this has happened, everything works as expected - that's why this was not that easy to reproduce ;)
Just wanted to let you know about this, especially as this might have been fixed "by accident" and not on purpose.
I have a simple repro over at my repro-repo, which hopefully showcases the problem clear enough.
Let me know if this somehow might help in preventing some future regressions. We will probably notice, at least.

I guess this is related to this comment

Hi, @earloc Thank you for reaching out! I also released this issue. Could you check this PR #136. Did the code snippet in the description illustrate the problem you met?

@zhiyuanliang-ms No, the scenario you outlined in #136 seems to be different, as it only strikes when callers may not actually await the refresh() (which we do) - and therfore falling into an even more tighter problematic scenario with macro / micro-tasks.

As @lorenzadomat  suggested, the fix for us was probably a side-effect of this comment, as the internal timer was not reset when there where no changes to the watched sentinel-keys. The fix you suggest over at #136 wouldn't prevent our scenario (which doesn't happen with 1.1.0 forward). But it might have an effect on others, which are not awaiting - not sure if a library is responsible for preventing such bad callers, though ;).

@earloc Oh, I see. I didn't notice that you await the refresh call. Taking a look at it.

@zhiyuanliang-ms just to make sure: Our problem disappeared with 1.1.0. But as there was no issue about this, I just wanted to document it here (and maybe higher the chance that this won't accidently get re-introduced in future Version).
I might have a look into providing a unit-test similar to the one I used in my repro to guard against this - when I find the time 😇).
Heads up, and thx for coming back on this so fast 💪.

@earloc Big thanks! 😃 Glad to hear that using 1.1.0 can solve your problem. I will add more testcase in this repo to guard against it. I will close this issue now.


[5.10] Serialization: Ignore missing potentially non-required dependencies when listing imports

Cherry-pick of ##68480.
The service ModuleFile::getImportedModules is called after the dependency loading logic runs. We can trust the dependency loading logic to have correctly loaded dependencies depending on the context. Later in getImportedModules, we can just ignore any missing dependencies if it could have not been required.
Indexing looks through all dependencies recursively to index to find all system modules to index. It could lead to a crash when looking at dependencies that weren't loaded for a transitive client.
rdar://115372249

@swift-ci Please test


clarify core support of secret reference

What this PR does / why we need it:
This PR clarifies implementation of core support of Secret reference.
Which issue(s) this PR fixes:
https://github.com/kubernetes-sigs/gateway-api/discussions/1059
Does this PR introduce a user-facing change?:
I think it does not.
NONE

Thanks @jakubdyszkiewicz!
/ok-to-test

@jakubdyszkiewicz this change LGTM, but I think you'll also need to commit the results of make generate to get our tests to pass.

/approve

@robscott thanks, I fixed the autogenerated files.

Thanks @jakubdyszkiewicz!
/lgtm


LKE-723 POC: UI for code snippet on Cluster Details

Description
2 options both in this PR to make viewing easier (will only go with one option though)

Inline version:

Tooltip version:

Type of Change

Non breaking change ('update')

I like the tooltip version with a bit more width so that command fits on one line.

Thanks @asauber for the feedback, I addressed that here.


Utulize config log format within gc

This ensures that the GC command uses the same logging format as specified in the registry's YAML config.

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 "gc-logging" git@github.com:tonyhb/distribution.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.

@tonyhb you have a couple of tests failing it seems:
registry/garbagecollect_test.go:164: not enough arguments in call to markAndSweep
registry/garbagecollect_test.go:196: not enough arguments in call to markAndSweep
registry/garbagecollect_test.go:330: not enough arguments in call to markAndSweep

Current coverage is 25.56%

Merging #1557 into master will decrease coverage by -32.39% as of bed3c29

@@            master   #1557   diff @@
======================================
  Files          122     341     +219
  Stmts        10633   34663   +24030
  Branches       840    1270     +430
  Methods          0       0
======================================
+ Hit           6162    8863    +2701
- Partial        840    1270     +430
- Missed        3631   24530   +20899

Review entire Coverage Diff as of bed3c29

Uncovered Suggestions

+0.58% via ...mespath/functions.go#125...325
+0.15% via .../yaml.v2/scannerc.go#2365...2416
+0.15% via .../yaml.v2/emitterc.go#1460...1509
See 7 more...

Powered by Codecov. Updated on successful CI builds.

@dmp42 thanks, fixed. janky seems to time out a fair amount with the tests right now

Disregard Janky.

LGTM :clap:

LGTM


Deleting policies set by 'Trustee'

only the users with roles of Admin, provider, or delegate are allowed to delete policies.
'Trustee' role is not allowed to delete policies they may set for providers.

Fixed by #165


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_15a52a8b-d8b3-4b2f-9787-f8ae702ae29d/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_15a52a8b-d8b3-4b2f-9787-f8ae702ae29d.git

Docs Build status updates of commit 4d821d9:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-01-26-03-00-06 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 4d821d9:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-01-26-03-00-06 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Added StopWatch

I have added stopwatch with all the features mentioned.
Pls review and approve. #39
Uploading StopwatchVideo.mp4…

@SandeepKrSuman
I've made the changes you requested for.
Can you please label as Hacktoberfest
please review and approved.
#40

I cannot find any changes in this PR. Kindly push the changes if you have made them locally.
PS.: hacktoberfest label will be added only after approval.

See above, your last commit was 4 hours ago and there's no changes after that.

@SandeepKrSuman Sorry, for the inconvenience sir. Please review it again.
#40


Factorize and solve the Laplacian on the GPU

PR #75 added GPU support to CombinatorialSpaces.jl via the CUDA.jl API.
Currently, we directly offer GPU support for the classic DEC primitive operators: $d, \star$, and $\wedge^{pp}$. And Decapodes.jl is able to automatically create bindings for higher-order operators on the GPU - such as the Laplacian - by expanding their definitions - e.g. $d\star d\star$ - and pre-multiplying those matrices. The branch llm/cuda-wedge-music is adding GPU support for $\wedge_{10}^{p d}, \wedge^{dp}_{01}$, and the interpolating musical operator $\flat\sharp$. These operators are implemented as matrix-multiplications are relatively straight-forward to port to the GPU.
However, some operators - such as the geometric Hodge star for 1-forms - are instead implemented by solving a matrix implementing that operator. $\star^{-1}$ was implemented by using gmres from Krylov.jl, and performantly solves this system, since it is "mostly" diagonal.
However, when we need to solve the heat equation by solving the sparse $\Delta_0$ matrix, gmres is not able to provide adequate performance. Further, even the factorization of this matrix on the GPU is prohibitively-expensive.
So, we need to provide a canonical way of solving this problem quickly in the CombinatorialSpaces library. The current prototype factors the matrix on the CPU, sends its components to the GPU, and in-houses an LU-solve.
We are also experimenting with approximations to the Laplacian on well-structured meshes that can be exploited for superior factorizations.

Looking at the CUDA.jl source I found that they actually do implement in-place sparse solvers on GPU with QR and Cholesky. They also have a eigenvalue solver. These are called csrlsvqr!, csrlsvchol! and csreigvsi respectively.
They seem to have sparse factorization support but I'm not sure how to use those to solve.
Link here: https://github.com/JuliaGPU/CUDA.jl/blob/master/lib/cusolver/sparse.jl#L101


Node bundling

Other bundlers (like webpack) can target node environment instead of browser. That largely involves leaving things like require('fs') and process intact in the code.
Is there a way to do the same in fastpack?
If no, is it on the roadmap?
The use case is when writing a Node application using ReasonML, but don't want to bring the compiler with it when deploying, therefore need to bundle the js code.

Hi, @mrvicadai. Thanks for the request! Unfortunately, it is impossible in the fastpack at the moment. We've been targeting browsers first. There are some builtin (not pluggable) features which prevent an easy addition of the node target at the moment:

Fastpack considers the browser field.
Requiring fs, path and other builtin Node.js packages leads to an error with the suggestion of the browser-friendly mocks
process.env.NODE_ENV is replaced across the codebase with the constant (development or production) as well as non-matched if/ternary branches are pruned.

If I understand the feature request correctly, for the node target you definitely don't want (1). (2) should leave require statements intact. What about (3)? Is it desired behaviour or it should be skipped as well? Is there anything else specific about the node target?

I'm also interested in targeting node. Another difference that comes to mind is window vs global as the global object. Not sure if that's builtin or not

Hi @zindel thanks for the reply! I think you're correct that (1) and (2) are preventing node environment from working. Not necessarily (3) though since the application might need different configuration profiles depending on the environment.
@juancampa also made a good point that global is the "Pervasive" variable in Node, as well as __dirname and __filename as seen here.

@juancampa @mrvicadai could you suggest a reasonably simple (but still with some dependencies) app we could use for testing?

@zindel I was looking at the source tree of the parcel bundler and found some integration test cases that might just serve as a SSCCE for this:

fs-alias
fs-assign-alias
fs-assign
fs-buffer
fs-destructure-assign
fs-import
fs-inline

In terms of real world example, I can think of this one as reasonably small, but highly used module: https://github.com/isaacs/rimraf/blob/master/rimraf.js


Security Flaw in ListenerInventory Logic

There's a major security flaw inside the Listener-Logic, which enables Players to open and use the GUI.
Why?
The main reason is, because if you rename a container, the title of the View gets set to the name.
If you rename the container to "Ban Home GUI", you can successfully bypass the if check on Line 30, and thus open the other ban-inventories, and use their features.
https://github.com/valkyrienyanko/BanGUI/blob/9ec0f12c35ba7409b98a7319283d6eee64f36b5c/src/main/java/com/bangui/listeners/ListenerInventory.java#L30
Reproducing
There are only few simple steps on how to exploit this logic.

Rename a Chest to "Ban Home GUI"
Now you only have to place a item at slot 12 or 14 (as defined in the switch block at Line 32 to 41), to open the other menus, and use their functions.

https://github.com/valkyrienyanko/BanGUI/blob/9ec0f12c35ba7409b98a7319283d6eee64f36b5c/src/main/java/com/bangui/listeners/ListenerInventory.java#L32-L41
Fixing
I suggest keeping track of players who opened the menu with the "guiban" command, and only letting them use the functions of the menus, else, cancelling or even kicking / banning the players who try to use this exploit,

A quicker solution may be to check user permission in addition to the GUI title. This would not require a means of keeping track of players, however it will still work for people with permission. It will no longer be a security issue but it will remain as a bug.

Plugin now keeps track of all users who use the ban gui command.
Release with exploit removed.


chore: fixing issue when metric time is 0 in document-load plugin

Which problem is this PR solving?

During working on web collector I discovered a bug with document load and when metric is 0 (for example "secureConnectionStart"). The plugin was treated this metric as available and it was shown as metric with negative value

error image

Codecov Report

Merging #546 into master will increase coverage by 5.5%.
The diff coverage is n/a.