remove DOMParser(), decode compatible with Web Workers

Features like Offscreen Canvas use Web Workers. If we want to process a gainmap embedded in a JPEG, we won't have access to DOMParser() and thus need a compatible alternative. This is a simple Regex approach to get around it.
All test pass correctly :)

Hi! Thanks for your PR! Looks good to me but there's an eslint check failed, can you have a look (npm run check:eslint-src)?

Alright, there were trailing spaces in two lines 🥴 now it should be good

aww, shit, the commit message format didn't trigger an automatic release, I'm going to trigger one "artificially", thanks for this in the meanwhile!


autoValue not recognizing Session.get()

Spoke to a couple people who recommended I bring this here as a bug.  I am setting a session variable here in the onBeforeAction method
     Router.route('singleDetails', {
		path: '/speakers/singleDetails/:_id',
		template: 'singleDetails',
		data: function(){
			return Speakers.findOne({_id: this.params._id});
		},
		onBeforeAction: function(){
			console.log(this.params._id);
			Session.set('pathId', this.params._id);
			this.next();
		}
	});

	Template.singleDetails.helpers({

		singleDetails: function(){
			return singleDetails.find({});
		}
	});

But when I try to get it in my schema, it console.log's to null and it gives me a server error
    speakerId: {
			type: String,
			autoValue: function(){
				Session.get('idPath'):
			}
		}

In short: I am not able to dynamically set a schema field to this.params._id

Was a solution reached for this?

Yeah it was. I ended up pursuing a different approach. Thanks for following
up,
Nick
On Saturday, April 4, 2015, jakec729<EMAIL_ADDRESS>wrote:

Was a solution reached for this?
—
Reply to this email directly or view it on GitHub
https://github.com/aldeed/meteor-autoform/issues/677#issuecomment-89697585
.


[Java] Call bound check instead of capacity expansion in AbstractMutableDirectBuffer.getInt

Quite minor issue, the getInt method should call boundsCheck0 rather than ensureCapacity

@fbaro Thank you for the fix.


Fix CI/CD workflow issues

Workflows are failing because they try to use brew without installing it.
Hopefully this is enough to get it going.

Dropped some checks from the workflows because we can rely on the asdf-community hygiene workflow.


Is this CViViT implementation causal?

Thanks for your work! We are attempting to use your code as a base for implementing an existing paper but we meet a few issues.
One of the key differences of Causal ViViT with its predecessor is the use of causal attention. However, in this implementation we cannot find which part ensures of it, as the attention itself is not causal. We do notice that there is a PEG module with a causal option (by default). We see that there is padding happening but we do not understand the purpose. Can you elaborate what this module does? If it does not ensure causal attention, is there another part that does?
So far we see proof of our trained models not to be causal, we hope to find if we are doing something incorrectly with the code.
Thanks in advance! And thanks again for the code.

Just wanted to inform that we found some serious bugs, related to reshaping. They were causing, along with other issues, the lack of causality that we noticed. For example, the reshape in the PEG module should be a rearrange instead to allow for also transposing when necessary (such transposes happen when batching spatial and then temporal). After fixing those we were able to obtain a causal and hopefully correct CViViT (currently training). As we see this repo might no longer be maintained, if you are interested about more details, let me know.

@NSavov hey, thank you for attempting to train this! i would def welcome a PR for the PEG module, if you have found that it violated causality somewhere
it has been a while, so i can't seem to spot the issue

@NSavov also, i believe another group has already trained cvivit successfully

@lucidrains thanks for the response. We have noticed and used the weights of the mentioned group. However, it shares the same problems.
The problem is this line in the PEG module. We don''t use the reshape feature now and instead we make sure to rearrange ourselves. We also enable the causal attention and with those 2 changes our causality test passes. Once we get our project running and the time allows, we can prepare a pull request. For now just wanted to point this out.
Thanks again for the code, it is very useful to have something as a base.

@NSavov you are welcome, and a PR would be great
could you let me know if this change addresses the underlying issue?

@NSavov come to think of it, could you point me to the passage in the paper where they said they used causal?

Thanks! We will check it out tomorrow. We apply PEG for spatial and temporal, so it might, but I have to confirm. For enabling causal attention, that is what we do as well.
The causal attention requirement is mentioned in the "Encoder architecture" section on page 4:

@NSavov thank you! yes you are correct then, this is a misstep on my part, my bad

@NSavov  i can add back a spatial PEG if you find it effective

think it is resolved


[BSO] Add bank filter for KK, Nex and various fruits.

🚀 Feature Proposal
basically all the same idea so figured should be 1 suggestion.
--kk = returns all KK items on boss log (Drygores, chitin+pet)
--nex = Returns all nex items from log
--fruit/fruits = Returns all fruits used for Monkey rumble

Oh, --raw showing raw food (for tames) would be good too!

Going to take a crack at adding these myself


⚠ Bofrost.at has degraded performance

In 7a44b38, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 931 ms

Resolved: Bofrost.at performance has improved in b425181.


IA-2268: add PT translations for budget tool

Budget tool needs to be translated in PT as well
Related JIRA tickets : IA-2268
Self proofreading checklist

[x] Did I use eslint and black formatters
[x] Is my code clear enough and well documented
[ ] Are my typescript files well typed
[ ] New translations have been added or updated if new strings have been introduced in the frontend
[ ] My migrations file are included
[ ] Are there enough tests
[ ] Documentation has been included (for new feature)

Changes

added a pt.json file
added the missing translations in EN and FR

missing a few translations in EN and FR, will add them tomorrow morning

Closing as the ticket is back to waiting for client/specification status and will require more work than just adding the JSON


Check diff for all targets

Now that #2402 has been fixed, here's hoping we can enable diff checks for all targets.

So msvc is failing because unbeknownst to me, the linker is embedding its version number into the output. 14.35.32215 becomes 0E 00 | 23 00 | D7 7D. Let me go spelunking in the linker.

Looks like it emits the version string in link.exe!CImplib::EmitImportDescriptor. Doesn't look like it's tied into the existing undocumented /Brepro; that would have been too easy.


[FEATURE-REQUEST] Show case project

I think it would help if we had an other repo to show case kaffy, a bit like https://github.com/areski/ex-chitchat, this project is just for me to play with Kaffy.
It might be good to have your own, where we can show and try all the existing functionalities.
Ideally it should be what we show on the live demo.
Thoughts?

Yes I agree. I've been meaning to create a repo for the live demo and add as many features as possible but have been lazy in this regard. Also, the current demo is extremely simple and doesn't show most of the features.
I think the project we eventually use should have a simple interface for the "normal" page and a "/admin" endpoint to control that. I'll upload the bakery project (current demo) today so you can take a look. Then we can decide how to proceed.

Sounds good, Thanks!

Any progress on that? I really think that could be very useful, so people could see the code used by the demo, but also see example of how to use some functionalities.

I'll make a public repo for the current demo website today. Feel free to modify it as you wish.

Finally this is done at https://github.com/aesmail/kaffy-demo

I'm having some issues to get this working :face_with_head_bandage:

You could actually rewrite most of it.
It’s not important how it’s done.
The important thing is that it works.
I think you’re having issues because maybe it’s using features that were experimental and never shipped I was playing with them.
If you want to replace the whole thing with your experimental project, be my guest.
I will just “git push” to gigalixir 😄

My demo project is broken with the same error, I have opened a bug here with more detail https://github.com/aesmail/kaffy-demo/issues/1
It's certainly relate to the new build_resources, as it's working without it.


crc fails starting when dnsmasq started outside from NetworkManager

Hi all,
if dnsmasq is already started outside of NetworkManager, crc fails starting with the following error:
INFO Extracting the Bundle tarball ...
INFO Starting stopped VM ...
INFO Bridge IP on the host: <IP_ADDRESS>
ERRO Failed internal dns query: ssh command error:
command : host -R 3 foo.apps-crc.testing

crc tells NetworkManager to start dnsmasq. But if a dnsmasq service is already started, NetworkManager is failing with the following message:
-- Logs begin at Thu 2019-06-06 03:29:01 EDT.
Jun 19 07:15:05 mylaptop.mydom NetworkManager[16439]: dnsmasq: failed to create listening socket for <IP_ADDRESS>: Address already in use
Jun 19 07:15:05 mylaptop.mydom NetworkManager[16439]: <warn>  [1560942905.4433] dnsmasq[0x56074776c830]: dnsmasq exited with error: Network access problem (address in use, permissions) (2)

Leading to /etc/resolv.conf never getting correctly updated and remaining to something like:
# Generated by NetworkManager
search mydom
nameserver 192.168.x.y

instead of:
# Generated by NetworkManager
search mydom
nameserver <IP_ADDRESS>

Thank you @praveenkumar for helping sorting out the whole issue
cc @cfergeau

resolution here is to stop and disable dnsmasq . Restarting NetworkManager and maybe rebooting your laptop may be required:
systemctl stop dnsmaq
systemctl disable dnsmasq
systemctl restart NetworkManager

Once we have #248 in I hope then it should be more easy and we don't have to rely on host NetworkManager side.

you mean you use systemctl enable dnsmasq ?

you mean you use systemctl enable dnsmasq ?

for some reason my dnsmasq unit was enabled and started.
Probably because I installed and configured openshift 4 for for libvirt. Or maybe, dnsmasq is enabled by default with REHL8.
the fix is to disable and stop dnsmasq standalone unit and let NetworkManager start its own dnsmasq process.

I do not remember that OpenShift 4 describes to enable dnsmasq (we maintain this howto now), but we have to do some investigation for sure to why this was enabled.
Oftentimes people use this for caching reasons, and we should be able to work with this...

I too had dnsmasq standalone unit enabled, probably as result of attempting to troubleshoot bad DNS in oc cluster up and reading https://stackoverflow.com/questions/48057668/pods-cant-resolve-dns-after-oc-cluster-up advice, but not understand that NetworkManager is running dnsmasq anyway (?)

[x] https://docs.okd.io/3.11/install/prerequisites.html#prereq-dns does explain

Disabling the standalone unit wasn't enough to make crc work for me yet.  Nor oc cluster.  It now seems I have some wider DNS problem in all docker containers, outside openshift too...

@cben in response to:

Disabling the standalone unit wasn't enough to make crc work for me yet. Nor oc cluster. It now seems I have some wider DNS problem in all docker containers, outside openshift too...

Please file a new issue if you still have these issues. It seems you are using different things at the same time. Plus, be aware that docker has issues with some nameservers that refer to localhost as this gets replaced with <IP_ADDRESS>. This is a known issue for Docker that causes issues in a corporate environment.


tidyWrapper.pl: Pass -nst switch as $argv along to Perl::Tidy (fixes #36)

-pbp (Perl best practices) which is often used, includes -st, which is incompatible with this script. But when using perltidy from the command line, -nst is not wanted in most cases.
This change adds the -nst option to the arguments passed to Perl::Tidy, making it work for .perltidyrc files which contain -pbp but not -nst.
More details:

https://metacpan.org/pod/Perl::Tidy#NOTES-ON-FORMATTING-PARAMETERS
https://metacpan.org/dist/Perl-Tidy/view/bin/perltidy#-pbp,-perl-best-practices

Tested and merged. Thanks for the commit!


called during build

setState() or markNeedsBuild() called during build.
https://github.com/Invincible1996/flutter_tree/assets/4311975/7bb89862-d635-4713-a217-e457bb0136e4
OnChecked events should not be fired when initialized

https://github.com/Invincible1996/flutter_tree/assets/4311975/c7102889-ae7a-4705-801c-263c0d6de5be
Why does it send a request 3 times with one click?

@fakhravari please update to the latest version


Kw/undo weird type

Just reverting those commits should be fine?

LGTM. Merge when you have finished the tests.


three-argument SHADE (using cell vertices/bounds/edges) sometimes fails to plot cells

The following should SHADE nine cells, but instead does only six:
 	NOAA/PMEL TMAP
 	FERRET v7.44 (optimized)
 	Linux 2.6.32-696.30.1.el6.x86_64 64-bit - 12/10/18
 	 7-Mar-19 20:51

yes? let/title="longitude"/unit="degrees_e" xedge = x[gx=0:3:1] + 0*y[gy=0:3:1]
yes? let/title="latitude"/unit="degrees_n" yedge = 0*x[gx=0:3:1] + y[gy=0:3:1]
yes? let/title="temperature"/unit="K" val = reshape({1,2,3,4,5,6,7,8,9},x[gx=1:3:1]+y[gy=1:3:1])
yes? shade val, xedge, yedge

The range of the colorbar suggests that Ferret is aware of the data in the unplotted cells, it's just not plotting them.
But if we simply change the window aspect ratio, the cells now appear!
yes? set window/aspect=1
yes? shade val, xedge, yedge

What's going on?

This first shows up with Ferret v6.8, which was the change to use of double-precision data throughout Ferret - likely a clue!

Yes, this was a precision issue - comparing the coordinate data with the information about the translation of coordinates when putting data onto the page.  PPLUS is still in single precision, so that comparison needs to be done just to single precision tolerances but was done in double precision.
Fixed in  curv_coord_range_sub.F


Failed to update from 23.10 to 24.04 on WSL2, by 2024.6.13

Windows Version
10.0.22631.3737
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-microsoft-standard-WSL2
Distro Version
23.10
Other Software
N/A
Repro Steps
Step 1: sudo do-release-upgrade
Step 2: sudo systemctl restart snapd.service
Step 3: sudo service snapd start
Step 4: sudo service snapd.service start
Expected Behavior
I expect to see a successful update
Actual Behavior
Result of Step 1:
Reading cache

Checking package manager
Reading package lists... Done
Building dependency tree... Done
Reading state information...  Done
Building dependency tree... Done
Reading state information... Done
error: cannot list snaps: cannot communicate with server: Get "http://localhost/v2/snaps": dial unix /run/snapd.socket: connect: no such file or directory

Error in sys.excepthook:
Traceback (most recent call last):
  File "/tmp/ubuntu-release-upgrader-tx6qhtkv/DistUpgrade/DistUpgradeViewText.py", line 138, in _handleException
    apport_crash(type, value, tb)
  File "/tmp/ubuntu-release-upgrader-tx6qhtkv/DistUpgrade/DistUpgradeApport.py", line 57, in apport_crash
    apport_excepthook(type, value, tb)
TypeError: apport_excepthook() missing 1 required positional argument: 'exc_tb'

Original exception was:
Traceback (most recent call last):
  File "/tmp/ubuntu-release-upgrader-tx6qhtkv/noble", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/tmp/ubuntu-release-upgrader-tx6qhtkv/DistUpgrade/DistUpgradeMain.py", line 241, in main
    if app.run():
       ^^^^^^^^^
  File "/tmp/ubuntu-release-upgrader-tx6qhtkv/DistUpgrade/DistUpgradeController.py", line 2607, in run
    return self.fullUpgrade()
           ^^^^^^^^^^^^^^^^^^
  File "/tmp/ubuntu-release-upgrader-tx6qhtkv/DistUpgrade/DistUpgradeController.py", line 2412, in fullUpgrade
    if not self.doPostInitialUpdate():
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/tmp/ubuntu-release-upgrader-tx6qhtkv/DistUpgrade/DistUpgradeController.py", line 1437, in doPostInitialUpdate
    self.quirks.run("PostInitialUpdate")
  File "/tmp/ubuntu-release-upgrader-tx6qhtkv/DistUpgrade/DistUpgradeQuirks.py", line 102, in run
    func()
  File "/tmp/ubuntu-release-upgrader-tx6qhtkv/DistUpgrade/DistUpgradeQuirks.py", line 128, in noblePostInitialUpdate
    self._test_and_fail_on_tpm_fde()
  File "/tmp/ubuntu-release-upgrader-tx6qhtkv/DistUpgrade/DistUpgradeQuirks.py", line 1910, in _test_and_fail_on_tpm_fde
    snap_list = subprocess.check_output(['snap', 'list'])
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/subprocess.py", line 466, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/subprocess.py", line 571, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['snap', 'list']' returned non-zero exit status 1.

Result of step 2:
System has not been booted with systemd as init system (PID 1). Can't operate.
Failed to connect to bus: Host is down

Result of step 3:
snapd: unrecognized service

Result of step 4:
snapd.service: unrecognized service

Diagnostic Logs
N/A

Having same issue. I tried reinstalling snapd. $ snap list hangs for a long time.
I do not use any snaps, so I ran $ sudo apt remove snapd then $ sudo do-release-upgrade and was successful.

An alternative to removing snapd is to enable systemd in the distribution, see https://learn.microsoft.com/en-us/windows/wsl/systemd

Having same issue. I tried reinstalling snapd. $ snap list hangs for a long time.
I do not use any snaps, so I ran $ sudo apt remove snapd then $ sudo do-release-upgrade and was successful.

remove snapd
sudo apt remove snapd
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Package 'snapd' is not installed, so not removed
The following package was automatically installed and is no longer required:
  squashfs-tools
Use 'sudo apt autoremove' to remove it.
E: Could not read response to hello message from hook [ ! -f /usr/bin/snap ] || /usr/bin/snap advise-snap --from-apt 2>/dev/null || true: Success
E: Could not read response to hello message from hook [ ! -f /usr/bin/snap ] || /usr/bin/snap advise-snap --from-apt 2>/dev/null || true: Success

but still do-release-upgrade fail and can't update anymore
apt-get check
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
You might want to run 'apt --fix-broken install' to correct these.
The following packages have unmet dependencies:
 libnss-systemd : Depends: systemd (= 253.5-1ubuntu6.1) but 255.4-1ubuntu8.4 is installed
 libpam-systemd : Depends: systemd (= 253.5-1ubuntu6.1) but 255.4-1ubuntu8.4 is installed
 mailutils : Depends: libmailutils9t64 but it is not installable
 parted : Depends: libparted2t64 (= 3.6-4build1) but it is not installable
 systemd-sysv : Depends: systemd (= 253.5-1ubuntu6.1) but 255.4-1ubuntu8.4 is installed
E: Unmet dependencies. Try 'apt --fix-broken install' with no packages (or specify a solution).

some solution below don't work
sudo apt-get install -f
sudo dpkg --configure -a
sudo apt-get install -f

Having same issue. I tried reinstalling snapd. $ snap list hangs for a long time.
I do not use any snaps, so I ran $ sudo apt remove snapd then $ sudo do-release-upgrade and was successful.

remove snapd
sudo apt remove snapd
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Package 'snapd' is not installed, so not removed
The following package was automatically installed and is no longer required:
  squashfs-tools
Use 'sudo apt autoremove' to remove it.
E: Could not read response to hello message from hook [ ! -f /usr/bin/snap ] || /usr/bin/snap advise-snap --from-apt 2>/dev/null || true: Success
E: Could not read response to hello message from hook [ ! -f /usr/bin/snap ] || /usr/bin/snap advise-snap --from-apt 2>/dev/null || true: Success

but still do-release-upgrade fail and can't update anymore
apt-get check
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
You might want to run 'apt --fix-broken install' to correct these.
The following packages have unmet dependencies:
 libnss-systemd : Depends: systemd (= 253.5-1ubuntu6.1) but 255.4-1ubuntu8.4 is installed
 libpam-systemd : Depends: systemd (= 253.5-1ubuntu6.1) but 255.4-1ubuntu8.4 is installed
 mailutils : Depends: libmailutils9t64 but it is not installable
 parted : Depends: libparted2t64 (= 3.6-4build1) but it is not installable
 systemd-sysv : Depends: systemd (= 253.5-1ubuntu6.1) but 255.4-1ubuntu8.4 is installed
E: Unmet dependencies. Try 'apt --fix-broken install' with no packages (or specify a solution).

some solution below don't work
sudo apt-get install -f
sudo dpkg --configure -a
sudo apt-get install -f

install deb package version by force  which wget from https://ubuntu.pkgs.org/24.04/ubuntu-main-amd64
sudo dpkg -i  libmailutils9t64_3.17-1.1build3_amd64.deb  \
                      libpam-systemd_255.4-1ubuntu8.4_amd64.deb  \
                      libnss-systemd_255.4-1ubuntu8.4_amd64.deb  \
                      libparted2t64_3.6-4build1_amd64.deb

remove snapd, work for me.

I was facing the same issue and removing snapd using sudo apt remove snapd worked for me as well.


Compilation error - instanceof keyword detected

Summary
I encountered a compliation error - instanceof keyword detected when submitting my code on codecrunch.

Description:
However, the only instanceof keyword was used in ImList, which I did not edit. Also, my code passed all the test case when I run it manually in jshell.
Screenshots

Screenshots (if any)

Thank you for your feedback.  When I tested it, I did not submit ImList.java.  Anyway, I've tweaked the grader, and regraded your submission.  It should be fine now.

Hi Prof @hwkchia, I am currently facing the same issue!

OK everyone...
Do NOT submit ImList.java.
What they only need is your Circle.java, Point.java, and main.jsh.


[Proposal] Docusarus NodeJS api

🚀 Feature
Expose (some) docusaurus functionality through a NodeJS api.
Have you read the Contributing Guidelines on issues?
Yes
Motivation
At my work I have implemented an automated documentation flow using docusaurus, react-docgen, react-docgen-typescript and file-watchers.
We are aiming maintain a good DX in writing documents, so we implemented a flow that uses chokidar to watch multiple project's files (Files in our component library repository, our mobile repo, our main application repo, etc) and generate a bunch of .md files that then docusuarus renders.
However, there is ATM not an "official" way to programmatically start/stop a docusaurus process.
Pitch
ATM our solution is to reach inside docusaurus core
import { start as startDocusaurus } from "@docusaurus/core/lib";

/// much code

startDocusaurus(DOCS_ROOT, {
    port: undefined,
    host: undefined,
    locale: undefined,
    hotOnly: undefined,
    open: true,
    poll: undefined,
});

Unsure on the API exactly, but Ideally we would expose a method that would essentially return a a wrapper around WebpackDevServer 's .listen and .close methods, so expose specific controls over it and not return the whole devServer.
https://github.com/facebook/docusaurus/blob/3548686f596bba8dcd79f57f6a697d8f06a2fcd3/packages/docusaurus/src/commands/start.ts#L215
A rough schema of what I have in mind

type SomeDocusaurusInformation = {
	pid: number;
	port: number;
	filesWatched: string[];
	/// idk, something else.
}

export default async function start(
  siteDir: string,
  cliOptions: Partial<StartCLIOptions>,
): Promise<{
	start: Promise<SomeDocusaurusInformation>
	close: Promise<void>
}>

At my work I have implemented an automated documentation flow using docusaurus, react-docgen, react-docgen-typescript and file-watchers.

You could implement a plugin, the getPathsToWatch lifecycle allows to register a path like somePath/**/*.md and content will hot reload everytime a file changes
I'd also like to make it easier to integrate with CMSs. The issue is a bit messy but you may find relevant intos here:

https://github.com/facebook/docusaurus/issues/4138
https://github.com/facebook/docusaurus/issues/4492

We could have a way for plugins to trigger site content reloads on a per-plugin basis, eventually, listen for a CMS to send a notification for changes etc...
I don't think we'll expose an imperative Nodejs API, but rather expose the required APIs for such use-cases through plugin lifecycles. You will have to implement your own plugin


Increase control width to avoid truncating text

The text change in 887301db0f14d20f7a39cb6645f955bf32ac9a6d from "Updating VS Code..." to "Updating Visual Studio Code..." made it wide enough to be truncated and only show "Updating Visual".
This PR gives the text control the same width as the progress bar. It should fix https://github.com/microsoft/vscode/issues/81433 .
I have only tested it in a resource editor, and not by actually building and running the updater.

Oh that's what that was! Great fix, thanks!


US 02.04.01

As a player, I want to see that other players have scanned the same QR code.
User story points: 2

triet

Saturday 25 March

finished my part. See this PR for more info on how to use my method: https://github.com/CMPUT301W23T53/the-explorer/pull/80


Stop tile flickering

This appears to be the crux of our problem with flickering: https://github.com/uber/deck.gl/blob/master/modules/geo-layers/src/tile-layer/utils/tileset-2d.js#L272 "loaded" means the data is fetched, not that it is rendered.  I will probably ask them to either expose all of these utilities (the tile cache class, the tile class) so I can overwrite (not optimal) or let me (or them) use the success output from the draw() call instead of the loading of the tile for knowing when to hide parent tiles (more optimal).

Looks like it is being exported anyway with https://github.com/uber/deck.gl/pull/4246.  I will see how it is to extend/overwrite and go from there then.

Looks like this might have been a function of undefined behavior on the shaders.


pull columns_dict programatically??

Cant I just get the schema for the tables in the database and use them?

Cuz right now I have to make them match and it's a pain in the butt cuz I gotta write everything twice.


Ignore suggestions based on previously ignored

Fixes #37
Now suggestions are not added if a matching ignore is found.
Also rewrote the buffering of 404 to allow logging if threshold has passed as well. Meaning the buffer can be less than buffersize before being processed. Seems logical to me but perhaps there's a reason for waiting to fill the buffersize always.

@marisks shouln't LogQueue.Count be used instead of bufferSize in check of threshold as well?

We are working on rewriting the whole logging logic for Suggestions. This PR will be out of date soon.

How far away? It would be nice to avoid getting a lot of suggestions already ignored.

Closing this issue, due to the fix was provided here: https://github.com/Geta/geta-notfoundhandler/pull/134


"fs is not defined" in the prebuild version

Hi, I don't use NodeJS so I use the prebuild version of PDFKit to generate the PDF.
I imported the library in the HTML file
`
`
It worked well with built-in fonts.
When I registered a font from a file and used it
doc.registerFont('OpenSans-Regular', 'fonts/OpenSans-Regular.ttf', 'OpenSans-Regular'); doc.font('OpenSans-Regular');
I got this error
Uncaught ReferenceError: fs is not defined     at Object.fontkit.openSync (pdfkit.js:16213)     at Function.PDFFont.open (pdfkit.js:12888)     at PDFDocument.font (pdfkit.js:14673)
Can anyone please guide me how to solve this problem?
Thanks!

Hey!
To use embedded fonts in browser context, I needed to download it other way through http, and pass it to the  pdfkit.
I don't know what fancy framework are you using, but in angular, I did it like this:
$http.get('assets/fonts/LatoLatin/fonts/LatoLatin-Light.ttf', {responseType: "arraybuffer"})    .then(function(response){ /*..*/                      resultDocument.registerFont('Lato', font.data);                    /*..*/ })

Hi,
I did load the font files but the issue is still there.
I'm using Typescript + Jquery so I do this to load the file
$.when($.get("fonts/Open_Sans/OpenSans-Bold.ttf"),
            $.get("fonts/Open_Sans/OpenSans-BoldItalic.ttf"),
            $.get("fonts/Open_Sans/OpenSans-Regular.ttf"),
            $.get("fonts/Open_Sans/OpenSans-Italic.ttf")).then(function () {
                  doc.registerFont('OpenSans-Regular', 'fonts/OpenSans-Regular.ttf');
                  doc.font('OpenSans-Regular');
                  // ...
});

I think the problem is that my system can't understand the 'fs' Node package. For some reason, that package was not integrated to the prebuild file.

You did load the font files, but you did not point pdfkit to them.
Try
$.when($.get("fonts/Open_Sans/OpenSans-Regular.ttf"))
           .then(function (fontData) {
                  doc.registerFont('OpenSans-Regular', fontData);
                  doc.font('OpenSans-Regular');
                  // ...
});

or something like that (maybe fontData.data?)

Hi,
You are right, I didn't load the font data correclty.
Here's the code for those who need it (jQuery doesn't support binary datatype on ajax call)
var xhr = new XMLHttpRequest();
xhr.open('GET', font.url, true);
xhr.responseType = 'arraybuffer';

xhr.onload = function (e) {
  if (this.status == 200) {
    doc.registerFont(font.name, xhr.response);
    loaded[idx] = true;
  }
};

xhr.send();

@nguoihocnghe, I have the same problem. Where and how do you define font variable? Also, since I am getting the font file from local, I get CORS policy error.

It has been a long time since I worked with that project so I can't remember. However, I guess font is just my local object to define font name and url. You can define it the way you like, just make sure that you can pass it to the above function to load it and render your text.
You get CORS error because you dev site and your font are not on the same domain, I guess. I used localhost for my dev site so I had no problem with it.
It's not relevant to this topic though but I'm willing to help.

@nguoihocnghe, I have the same problem. Where and how do you define font variable? Also, since I am getting the font file from local, I get CORS policy error.

if you register a font with its path string, pdfkit will load it by fs module embed in nodejs, but it misses in the browser client. loading the font in the way by nguoihocnghe above can avoid the issue.

To load a font that is present in your source code or locally, you can use the below code,
const path = path.join(__dirname, '..', '..', 'fonts', 'Zilla Slab', 'ZillaSlab-Regular.ttf');
const font = fs.readFileSync(path);
doc.registerFont('Zilla Slab', font);


added uuid lib for format validation

Resolves #266

Additionally, please, fix rustfmt & clippy failures

ive made the changes we discussed. ran clippy and fmt locally and seem to be fine!
Thanks for your help, this was really interesting !!


⚠ Bofrost.at has degraded performance

In feb0dc8, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1758 ms

Resolved: Bofrost.at performance has improved in c318be0.


New version: Hesan.He3 version 1.3.7

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.4 schema?

Microsoft Reviewers: Open in CodeFlow


Error: sha256 mismatch: Kicad

kicad
There is a sha256 mismatch error, as you can see below:
brew cask install kicad
==> Downloading http://downloads.kicad-pcb.org/osx/stable/kicad-4.0.5.dmg
######################################################################## 100.0%
==> Verifying checksum for Cask kicad
==> Note: running "brew update" may fix sha256 checksum errors
Error: sha256 mismatch
Expected: b9747534aa29074eeb383dd4b506cf76e977f5b222dd08022d856a48d6137049
Actual: 4e15567c655176bd8d187d8d9c3e9413e7777e84bd5cef057f04b5895a5e786b
File: /Users/XXX/Library/Caches/Homebrew/Cask/kicad--4.0.5.dmg
To retry an incomplete download, remove the file above.

From the issue template that shows when opening a new issue:

Before opening your issue you must check our instructions for reporting bugs (…) [it] has several links to help diagnose and fix common issues.
(…)
if it is apparent you ignored the instructions for reporting issues, we may close the issue without review.

That seems to be the case, which is why I closed this issue. We worked hard  on those guides to keep things running smoothly, so I ask that you go back and follow them. It is OK to open a new issue for that.
If I was wrong in closing this issue, please do reply stating why! Closing an issue does not mean the conversation is over.
I understand sometimes we’re tired and don’t want to sift through a guide to make what we feel is a simple bug report. But when each user ignores the guides to save a few minutes, maintainers lose hours and get fatigued. I ask you take that into consideration.
Thank you for understanding.

Thx !
Sorry I did not see this sentence: It’ll likely need a new version, but it’s possible the version has remained the same in your issue template... My bad


RFP - Vacuum-IM

Full-featured cross-platform Jabber/XMPP client.
Downloads: https://github.com/Vacuum-IM/vacuum-im/releases

I'm looking into creating a package for this now.
I won't be providing "Nightly Builds" though (this may change in the future).
But I will support the GPO edition they provide(I have no idea what this is though, think it's related to perhaps a corporate edition somehow), with an optional package parameter.

This software seems pretty much dead at the moment. The latest release was more than 2 years ago.

I know, but you did request it when it was already in dorment state (when it comes to stable releases).
However, their latest nightly build was released last month it seems.

I can just close this issue if you are ok with it. :-)

their latest nightly build was released last month it seems.

No, it was released 2018-01-05.

I can just close this issue if you are ok with it. :-)

No need, I'm more or less done with the package.

No, it was released 2018-01-05.

Ops, mixed up the numbers (again 😞)

just pushed: https://chocolatey.org/packages/vacuum-im/1.2.5

thanks @pauby, I completely forgot to close this one when it was approved.


Fix to_utc_timestamp and from_utc_timestamp fallback when TZ is supported time zone

closes
to_utc_timestamp and from_utc_timestamp are not time zone aware.  It means they are not related to session time zone.
They both take a timezone parameter, usage:

from_utc_timestamp(ts_col, tz)
to_utc_timestamp(ts_col, tz)
So should not check the session time zone, only check time zone in the parameter is enough.

Signed-off-by: Chong Gao<EMAIL_ADDRESS>

build


Get an understanding of what is needed for the 3.0.0 release

Hi @aconrad
I think would be interesting what needs to be in for the 3.0.0 release.
It would also be interesting to see if some of the Issues could be closed and what the current PRs are doing to have a well-defined state of the repo when releasing the next major version.

All your PRs were merged so happy to work on a release. The issue about the XML being different between py37 and py38 would be nice to figure out since this release focuses on dropping py2. But maybe we can address that later.

tests for reporters as well (I only did the CLI tests)

CLI tests indirectly over them already, so definitely a nice-to-have but not a blocker for 3.0.

resolve inconsistency between Python versions

Yes, the fact that py37 and py38 are different bothers me, and I'd love for us to get that figured out before a 3.0 release.

clean-up the PRs (the one you opened is one line) and the issues we have (I close those that will be addressed or will not be done), because new issues could arise with the new release. Also there are issues that are quite old and we should check if there are still relevant or not.

I closed PR #75 which was no longer relevant. Then there's #109. The feature request is about grouping outputs on a per-directory and introduces a YAML output that you just introduced. But either way, I don't think that would be a blocker for 3.0.

some mentioned they want to regex match files
merging xml files was a topic

I also don't think any new feature should be a blocker. Those can come in as 3.1 if that's something we decide we want to have.
In general, I'd say most of the issues have stalled. I should start from the oldest issues and close them one by one if they aren't relevant anymore, or ask people to reopen the issues if they still care about them.

pycobertura 3.0.0 was released


[BUGFIX] Correct Nan's in the loss of transformer

Objective

Issue

Changes

Example

The root cause could be from the data preprocessing, i.e., vocabulary index building + padding.


strange face movements in mesh sequences from audio2mesh model

Nice Work! Thank you for opensourcing! 🙂
I have noticed that the face movements is quite strange in the mesh sequence output by audio2mesh model. For your information, you can see this video to see the mesh sequence:
https://github.com/Zejun-Yang/AniPortrait/assets/26021786/b4b41380-217b-4e7c-ab81-c59e3bcd8bb7
The eyebrows area and the contours of the face have a strange distortion. I suspect this is a problem introduced because real human mediapipe facial landmark data collecting.
However, in the final rendered video, although the landmarks in the middle are still distorted, the distortion becomes less noticeable in the final generated video.
https://github.com/Zejun-Yang/AniPortrait/assets/26021786/594ed486-cbf1-4fe1-9632-579fe6df1bc5
I wonder if you have done any post-processing on the mesh sequence to make this distortion disappear, or is there a special smoothing design in your rendering process?
Thank you for your reply!

To add on, I want to highlight that the eye sockets will suddenly enlarge and shrink, creating a bizarre effect.

To add on, I want to highlight that the eye sockets will suddenly enlarge and shrink, creating a bizarre effect.

I've seen that exact issue too, and I'm wondering how to solve it.

Thank you for your valuable suggestions! We believe that using face landmarks may not be the optimal method for controlling face motion, as it could introduce additional noise.


How does it work in ie9

ie9 doesn't support 'visibilitychange' ,but you said the lib works in ie9,how does it work?I didn't find the related code.

@LeviWangJS The library has some default strategies to work around the missing event in ie9 e.g. by listening on scroll/resize events.


Simple pip reqs

I updated the pip requirements file (in pip/simple_requirements.txt) to ignore all dependencies that are installed by the main programs we require. I haven't verified that it works on LC or with maestro, GMD or atomsci.clients so it's not ready to actually merge yet, but I wanted to ask for help in testing it out. It passed all the unit and integrative tests on my system.

Less packages to keep track of.
More up-to-date packages managed as others' dependencies instead of explicit requirements on our end.
This includes updated packages such as pandas and numpy that had a few small breaking changes. I fixed these.
PyYAML 6.0.1 is still unusable because it's referenced in several json readers; this requirement is frozen at 5.4.1

To do:

[ X] test on LC: cuda
[ ] test on LC: cpu only
[ ] test externally: cuda
[ ] test externally: cpu only
[ ] test rocm

HI, I was running this on LC and I ran into an issue with DGL libraries. I installed DGL like this
pip install  dgl -f https://data.dgl.ai/wheels/cu113/repo.html
pip install  dglgo -f https://data.dgl.ai/wheels-test/repo.html

and it complained
ERROR test_filesystem_perf_results.py - RuntimeError: DGL requires PyTorch >= 1.12.0

I'm guessing that I missed a step somewhere since others were able to run it successfully.

Hi @stewarthe6 @mauvais2 I have successfully tested the CPU and CUDA versions on LC and confirmed that CPU also works on my home (non-cuda) machine. I think it's ready for integration into 1.6.

CUDA now requires 11.8 in order for all packages to be compliant - this also allows training where it previously failed

Both requirements files required fiddling with the order of index URLs that pip searches in order to ensure the correct versions are installed

I majorly updated the readme.MD including adding a contributors section since the only authors listed are from the original AMPL paper

[ ] CUDA passes on Pascal pvis with only valid_r2 assertion errors:

[ ] Delaney_NN (Tensorflow)
[ ] dc_models (Tensorflow, Torch, DGL)
[ ] jax model notebook from DeepChem (Jax)
[ ] Delaney_RF (scikit-learn)
[ ] Delaney_XG (xgboost w/o GPU even though it has the option)

[ ] CPU passes on Pascal login node or pvis with only valid_r2 assertion errors:

[ ] Delaney_NN (Tensorflow)
[ ] dc_models (Tensorflow, Torch, DGL)
[ ] jax model notebook from DeepChem (Jax)
[ ] Delaney_RF (scikit-learn)
[ ] Delaney_XG (xgboost w/o GPU even though it has the option)


Description of “Valora would like to access the Camera” popup not seems as per expected in V1.14.3 on “Create Account” page

Frequency: 100%
Repro on build version: Test Flight build V1.14.3(62)
Repro on devices: iPhone SE (13.6.1) , iPhone 8 (14.5.1), iPhone 6+(12.4.5)
Repro Steps:

Launch the application & go to “Create Account” page
Tap on “Add Photo” icon & select “Take Photo” option
Now user should on “Valora would like to access the Camera” popup
Observed that ,
Actual Result: “Connecting your camera  allows you to scan codes for payment” description displayed
Expected Result User should able to see “Connecting your photo library allows you to save your code to your photos” popup as per bug fixes issue
Investigation:

Also we have observed same issue for Espano(America Latina) Language:

User able to see "Connectar tu camera te permite escanear codigos para hacer pagos" description instead of "Conectar tu galería de fotos te permite guardar tu código en tus fotos" on Camera access popup.

Similarly for  Portugues(Brasil) Language:

User able to see "Connectar a sua camera permite que voce possa escanear codigos de pagamento" description instead of "Conectar a sua galeria de fotos nos permite salvar seu código junto às suas fotos" on Camera access popup.

Attachment: AccessCameraPopupDescriptionChange

hey @MuckT  we have verified above issue and as per PR uploading profile picture has been removed while creating / restoring account hence we are closing this issue
Link : PR
Thanks..!


Reset Button

Description

There must be a reset button to quickly reset the date
There are 2 places which look suitable for the reset button to me (any new suggestions are welcome).
You must add the reset button in any one position, remember to maintain the responsiveness of the page.
The function of this reset button will be to reset the dates to current date.
The functionality must not be added right away only the button must be added in.

Screenshots

Additional information

Please maintain the mobile responsiveness.
Only use material UI components.
Do not install any new Package.
Make changes to the required file only.

Make sure to read the CONTRIBUTING and SETUP docs before proceeding
Happy contributing. 💐
Star my other Repositories here

Assign it to me

good! Assigning right away


pip install peptidebuilder fails.

pip install peptidebuilder  fails.  If I include both --allow-external PeptideBuilder and --allow-unverified PeptideBuilder it still fails.  I'd really rather not have to download this directly from github to use it, let alone trying to explain to others how to get it installed.
I'm thinking this used to work?  However, I don't really remember how I installed this earlier.

We never put this on pip, because the whole thing is really just two files, Geometry.py and PeptideBuilder.py. However, we can look into making this available through pip, shouldn't be too much work.

Please do.  I think my confusion comes from the fact that there is a page on pypi.python.org for the package (I guess to reserve the name?), but it is not installable through pip.
I'm trying to write some instructions for a colleague who is new to python and I'd rather not get into downloading things from github and adding by hand to site-packages and so on.  And as it looks like there are some changes since I've downloaded it, it might be nice for you to release those as well.
Let me say thanks while I'm here. I've found it to be quite useful. It has made some otherwise irritating things easy to do.

Yes, pypi lists the package, and me as maintainer. I have no recollection of registering the package with pypi. I presume it was just to reserve the name.
I'm glad you find the code useful.

Installing through pip should now work:
~wilke$ pip install peptidebuilder
Downloading/unpacking peptidebuilder
  Downloading PeptideBuilder-1.0.1.tar.gz
  Running setup.py (path:/private/var/folders/q8/wptgtbdn1pz0cfgrz39gq00m0000gn/T/pip_build_wilke/peptidebuilder/setup.py) egg_info for package peptidebuilder

Requirement already satisfied (use --upgrade to upgrade): Biopython in /Library/Python/2.7/site-packages (from peptidebuilder)
Installing collected packages: peptidebuilder
  Running setup.py install for peptidebuilder

Successfully installed peptidebuilder
Cleaning up...

However, the peptidebuilder test suite is not integrated in setup.py yet. As of now, tests still need to be run manually.

Great! Thank you very much.


Allow 'setup' configuration entries

Fixes #146

Thanks for adding this - do you mind also adding the field to the README and init.pp documentation section?


There is a similar project called OpenDevin

There is a similar project called OpenDevin and I don't know if it make sense that people working on two similar projects.

Thanks for the insight!

more is better

OpenDevin doesn't have any documentation yet


Background Delivery

I would like to implement the ability to receive data from the health connect integration in the background without using background tasks with periodic updates.
Is there any way to listen to an even in the background that will fire a synchronization method in my application and send new records to the server?
I was able to accomplish this in React Native Healthkit using NativeEmitter.

You can check the proof of concept of this in this tree:
https://github.com/matinzd/react-native-health-connect/tree/headless_poc

https://developer.android.com/health-and-fitness/guides/health-connect/develop/read-data
This page last updated 2024-09-30 UTC and states: "Health Connect allows apps to read data from the datastore when the app is in the foreground and background"
And it works ! I tested setting up a background-fetch that queries Health Connect for step counts in the background, and also triggering the task in the background using a silent push notification from my backend. Steps are returned as expected.

Very nice! Did the PoC help you? @lsandini

I wasn't even aware of the PoC, Here is how I did it, in my little test app (branch silent4)
https://github.com/lsandini/walker/tree/silent4
Health Connect queries returned an error as I tried to run them as a background task, now the step count is retrieved correctly


Added Blob name encoding

API changes have been detected in azure-storage-file-share. You can review API changes here
API changes
-             permanent_delete: bool = ...,
-             set_immutability_policy: bool = ...,

API changes have been detected in azure-storage-queue. You can review API changes here
API changes
-             permanent_delete: bool = ...,
-             set_immutability_policy: bool = ...,

/check-enforcer override

/azp run python - storage - ci

@scbedd any idea why this failure is happening?

@tasherif-msft it looks a random failure during the artifact upload stage early exited the build stage, the rest failed because the necessary packages artifact isn't actually present.
Exact Error Location
I'd give it a retry tbh. You should be fine.

/azp run python - storage - ci

@scbedd it seems even restarting the pipeline is not working :S

@tasherif-msft it's a NEW issue now!
Please check the Engineering System teams channel, github disabled a certain checkout style. I patched main but it looks like feature/storage-stg80 is your target branch, so it wouldn't have gotten that fix.
My recommendation? Update feature/storage-stg80 from main. If not that, then cherry pick the commit I linked above.

/azp run python - storage - ci


⚠ Bofrost.be has degraded performance

In 3eafb7b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 3465 ms

Resolved: Bofrost.be performance has improved in 8938721.


Make NavigationGrid Store NavigationGridData as pointers

This is required for further specializing grid square data in clients of GB.

Likely memory leak in Demo Region right now. initAllNavigationGridValues is called without freeAllNavigationGridData.
These functions are the new and delete for NavigationGrid instances and every initAllNavigationGridValues should have a matching freeAllNavigationGridData.
Add the matching free before merging this issue in.

@bauerbp @gyurgyma
For Review:
Pay special attention to the two functions in NavigationTools.
Should these functions be outside of Array2D? or is there an elegant way for Array2D to mass initialize when templateClass is a pointer?
Is NavigationTools in the correct sub folder? Should It be in util?


enhance: add auto complete to note lookup

enhance: add auto complete to note lookup
Note the tests currently hiccup around odd issue on initialization, haven't been able to pin point why this is happening yet.
Cmd is not a constructor: TypeError: Cmd is not a constructor
	at /usr/local/workplace/dendron/packages/plugin-core/out/src/workspace.js:476:25

DOC PR: https://github.com/dendronhq/dendron-site/pull/292
Here are some video usages, quite satisfying to use this addition:

https://www.loom.com/share/e2e6cd06c05d45fb9ed60542b96a069e
https://www.loom.com/share/4e2be8f5ddb842cfa8136f2c9ac3b118

General
Quality Assurance

[x] Add a test for the new feature
[ ] Make sure all the existing tests pass
[x] Do a spot check by running your feature with our test workspace
[x] If your tests involve running [[manual Testing|dendron://dendron.dendron-site/dendron.dev.qa.test#manual-testing]], make sure to update [[Test Workspace|dendron://dendron.dendron-site/dendron.dev.ref.test-workspace]] with any necessary notes/additions to perform manual test ^9jtWc6ov340p
[ ] After you submit your pull request, check the output of our integration test and make sure all tests pass

NOTE: If you running MacOS or Linux, pay special attention to the Windows output and vice versa if you are developing on Windows

Cmd is not a constructor: TypeError: Cmd is not a constructor
at /usr/local/workplace/dendron/packages/plugin-core/out/src/workspace.js:476:25

Possibly because NoteLookupAutoCompleteCommand depends on NoteLookupCommand, and there is an indirect circular dependency somewhere. This one's a super cryptic error I've ran into a while ago and it drove me crazy 😄
If you try / catch what's happening in _setupCommands in workspace.ts, you can see what Cmd it was trying to register that was undefined. It is undefined because the module loading order is mixed up and the Cmd doesn't exist yet.
A quick fix to this is pinpointing where the command is invoked (most likely in NoteLookupAutoCompleteCommand), and dynamically requireing the dependency instead of import
The real fix for this is to change our codebase so that we have full control over the module loading order, but that is a whole other big task so using the require hack for now should be good enough.
example of doing this: https://github.com/dendronhq/dendron/blob/a9ea0eaa4b44c7598831be9a625cd04df069725f/packages/plugin-core/src/utils.ts#L742
more info on this issue: https://medium.com/visual-development/how-to-fix-nasty-circular-dependency-issues-once-and-for-all-in-javascript-typescript-a04c987cf0de

@jonathanyeung , can you add https://stackoverflow.com/questions/55270915/vs-code-extension-hide-commands-from-command-palette to [[dev.cook.vscode]] inside our docs vault? (note doesn't exist yet)


to_if_alone sending events in non-deterministic order

I'm using this to_if_alone config with a build of Karabiner-Elements from the HEAD of the master branch a couple of days ago (specifically commit 97d6732e3aa59f4d934395d7d822eefc43c94602) to give me "Enter" when "Enter" is pressed alone and "Control" when it is pressed together with another key.
I noticed that sometime an "Enter" keypress produces this event log:
eventType:FlagsChanged    code:0x3e       name:right_control   flags:Ctrl                               misc:
eventType:KeyUp           code:0x24       name:return_or_enter flags:                                   misc:
eventType:KeyDown         code:0x24       name:return_or_enter flags:                                   misc:
eventType:FlagsChanged    code:0x3e       name:right_control   flags:                                   misc:

(ie. control down, then the "enter", then control up) and other times (about 50% of the time) it produces this:
eventType:FlagsChanged    code:0x3e       name:right_control   flags:Ctrl                               misc:
eventType:FlagsChanged    code:0x3e       name:right_control   flags:                                   misc:
eventType:KeyUp           code:0x24       name:return_or_enter flags:                                   misc:
eventType:KeyDown         code:0x24       name:return_or_enter flags:                                   misc:

(ie. control down then up, then the "enter"). This latter behavior is correct, I think, and the other one is not. This is noticeable in apps like Chrome in the location bar where there is a difference between Enter and Control-Enter; eg. if I type "tw" and the autocomplete suggests "https://twitter.com" (with "itter.com" selected):

Without Karabiner-Elements, pressing "Control-Enter" would actually navigate to "tw.com" and pressing "Enter" would navigate to "twitter.com". So, with the configuration above, pressing "Enter" goes to "tw.com" about 50% of the time and to "twitter.com" the other 50%.

Wondering if 62767e3482a4140ad17a is relevant here. The removed code comment said "Without wait, window system sometimes reorder modifier flag event.", but the commit doesn't say why the workaround could be removed.

So I just reverted 62767e3482a4140ad17a1468269ae216394d9184 on top of master (dc8dfbdedfe7eedff97e976573266c92aa13613b), and I can't reproduce the problem: the events always occur in the right order.

Thank you for feedback!
I confirmed this issue and reverted the change in v0.91.4.

Thank you!

Just opened #789 as a follow-up as I've discovered a couple more issues with to_if_alone.


HTML Summary does not show Reason & Client for Synced.

Need to fix up some field name and value format inconsistencies.

Summary v Description
duration is in hours or seconds need to pick one. (seconds)
activeCollabProjectId not always present?


Update demographics to use heading component

Issue Description

Issue Type

[ ] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Expected Behavior

Current Behavior

Steps to Reproduce

Environment

Component Version:
Browser Name and Version:
Operating System and version (desktop or mobile):

@mmalaker @bjankord the person name appears to be much larger in size in IE11 compared to the other browsers. I am guessing that should be fixed by this switch. Something we noticed when we switched to using the Heading component in our application.

The change needed for this conversion replaces
<h1> ... </h1>

with
<Heading level={1} > ... </Heading>

This changes the font sizes to 2.286 rem from 1.5 em. @neilpfeiffer what is the appropriate heading size to use to make this change? Below is a screenshot of the proposed changes on the left compared to current implementation.

The direct size conversion would be 2.14 rem which is not size option of the heading component.

I'd prefer we set the font-size for the heading element in DemographicsBanner.scss compared to using the <Heading /> component. This will give us more direct theming capabilities which will be scoped locally to the demographic banner styles.

Mike McCurdy working on this

Orion JIRA created

At any size larger than small: 1.286 rem
Small/tiny: 1.143 rem


theme: Cactus CN

Check List
Please read and check followings before submitting a PR.

[x] I want to publish my theme on Hexo official website.

[x] I have read the theme publishing doc.
[x] name is unique.
[x] link URL is correct.
[x] preview URL is correct.
[x] preview URL web site is rendered correctly.
[x] Add a screenshot to source/themes/screenshots.
[x] Screenshot filename is same as value of name.
[x] Screenshot size is 800 * 500.
[x] Screenshot file format is png.

[ ] I want to publish my plugin on Hexo official website.

[ ] I have read the plugin publishing doc.
[ ] name is unique.
[ ] link URL is correct.

[ ] Others (Update, fix, translation, etc...)

Languages:
[ ] en English
[ ] ko Korean
[ ] pt-br Brazilian Portuguese
[ ] ru Russian
[ ] th Thai
[ ] zh-cn simplified Chinese
[ ] zh-tw traditional Chinese

Currently, Hexo Team has not decided how to deal with forked theme yet.

@SukkaW  I'm not sure about the team's decision, but I did use the layout of the theme to adjust the style details based on the theme of fork cactus


Dead links to Mozilla

All links to the Mozilla web sites are dead, please correct.
-> Page not found
Thanx
Michael

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

ID: 55284993-672a-b950-38ec-f3e175dfbf93
Version Independent ID: 0617203d-5c57-037e-74b9-11c025917d07
Content: Progressive Web Apps on Windows - Microsoft Edge Development
Content Source: microsoft-edge/progressive-web-apps-chromium/index.md
Product: microsoft-edge
Technology: pwa
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Hi @bucmic - Sorry to hear about this issue and thanks for opening this issue.
I'm not able to reproduce the broken links. Can you retry please? All the links opened for me. Thank you.

Hi @bucmic - Sorry to hear about this issue and thanks for opening this issue.
I'm not able to reproduce the broken links. Can you retry please? All the links opened for me. Thank you.

Hello Reezaali,
ok, it's the DE version of the pages, EN seems to work. Just try to set DE as system language and you'll see. So it's a redirection issue and I guess I better contact Mozilla.
Regards
Michael

Thanks for confirming @bucmic.

@bucmic - I'll also reach out to them to let them know.


How can I remove comments from imports in compiled .exe

Infos:

Nuitka version: 0.6.13
OS: Windows
Arch: x86_64
Installed with: pip
In virtual environnement

This project is excellent and really helpful, the only threat is that, executables are really heavy. By curiosity i opened an .exe (generated by python -m nuitka --onefile --windows-company-name=Test --windows-product-version=<IP_ADDRESS> example.py) with npp and I found full chunks of doc and comments from my imports.

Example of what I mean
Do basic configuration for the logging system.

This function does nothing if the root logger already has handlers
configured, unless the keyword argument *force* is set to ``True``.
It is a convenience method intended for use by simple scripts
to do one-shot configuration of the logging package.

The default behaviour is to create a StreamHandler which writes to
sys.stderr, set a formatter using the BASIC_FORMAT format string, and
add the handler to the root logger.

A number of optional keyword arguments may be specified, which can alter
the default behaviour.

filename  Specifies that a FileHandler be created, using the specified
          filename, rather than a StreamHandler.
filemode  Specifies the mode to open the file, if filename is specified
          (if filemode is unspecified, it defaults to 'a').
format    Use the specified format string for the handler.
datefmt   Use the specified date/time format.
style     If a format string is specified, use this to specify the
          type of format string (possible values '%', '{', '$', for
          %-formatting, :meth:`str.format` and :class:`string.Template`
          - defaults to '%').
level     Set the root logger level to the specified level.
stream    Use the specified stream to initialize the StreamHandler. Note
          that this argument is incompatible with 'filename' - if both
          are present, 'stream' is ignored.
handlers  If specified, this should be an iterable of already created
          handlers, which will be added to the root handler. Any handler
          in the list which does not have a formatter assigned will be
          assigned the formatter created in this function.
force     If this keyword  is specified as true, any existing handlers
          attached to the root logger are removed and closed, before
          carrying out the configuration as specified by the other
          arguments.
encoding  If specified together with a filename, this encoding is passed to
          the created FileHandler, causing it to be used when the file is
          opened.
errors    If specified together with a filename, this value is passed to the
          created FileHandler, causing it to be used when the file is
          opened in text mode. If not specified, the default value is
          `backslashreplace`.

Note that you could specify a stream created using open(filename, mode)
rather than passing the filename and mode in. However, it should be
remembered that StreamHandler does not close its stream (since it may be
using sys.stdout or sys.stderr), whereas FileHandler closes its stream
when the handler is closed.

.. versionchanged:: 3.2
   Added the ``style`` parameter.

.. versionchanged:: 3.3
   Added the ``handlers`` parameter. A ``ValueError`` is now thrown for
   incompatible arguments (e.g. ``handlers`` specified together with
   ``filename``/``filemode``, or ``filename``/``filemode`` specified
   together with ``stream``, or ``handlers`` specified together with
   ``stream``.

.. versionchanged:: 3.8
   Added the ``force`` parameter.

.. versionchanged:: 3.9
   Added the ``encoding`` and ``errors`` parameters.

I'm conscient this is not really an issue and I don't know how heavy are all this texts but I doesn't find info about that anywhere.
Is it possible to not include this parts easily ?

This is solved by using this: https://nuitka.net/pages/commercial.html

In part you can use --python-flag=no_docstrings which removes those, much like you can do with -OO for Python bytecode, but that leaves your asserts intact, for that there is --python-flag=no_asserts to decide that separately.
The commercial offering allows you to obfuscucate the program data, and contains other features geared at Closed Source people. But if size is your worry, the above options help a bit.


Permission to put in South Kasai #571

Some of the newly implemented countries like Cyrenaica and Sarawak
lacked names in the nwo_text document and some parts of the South Kasai
commit I was doing were missing in the zip I sent you like decisions and
an edited province map depicting a redrawn Lusambo province to the Area
of South Kasai.

merged


[Bug]: The editor does not properly clear its displayed value when re-binding an empty string

I'm submitting a

[ ] Regression (a behavior that used to work and stopped working in a new release)
[X] Bug report
[ ] Enhancement
[ ] Feature request
[ ] Documentation issue or request

Description
I have an editor like so:
<form [formGroup]="billNoteTemplateModelForm">
	<app-ngx-editor formControlName="noteHtml"
			[placeholder]="'Enter text here...'"
			[(ngModel)]="noteTemplate.noteHtml"></app-ngx-editor>
</form>

The expected behavior is that whenever noteTemplate.noteHtml is changed via code, the editor should update its displayed value as well. However, if noteHtml is first set to any non-empty-string value, and then back to an empty string, the editor does not update its displayed value properly: it displays both the placeholder text and the previous non-empty-string value. It should instead display only the placeholder text.
The offending code appears to be the if-statement on line 199 of ngx-editor.component.ts. When the new value is detected to be an empty string, writeValue() does not call refreshView().
Version Information
ngx-editor: 3.3.0-rc.10
angular: 5.2.7

Browser

[X] Chrome - 64.0.3282.186
[ ]  Mozilla Firefox - x.x.x
[ ] Opera - x.x.x
[ ] Microsoft Edge - x.x.x

@chrisstaley. Since WriteValue is called twice I had a return statement which is removed now.


Test EDN parser against leading, trailing, and inter-element whitespace

I saw issues parsing  [1], [1] , and [ [1]].  These are simplified and from memory, so they may not all fail.  For what it's worth I was using r# raw strings, although I don't think that's relevant.

Mostly it should be the case that you just splash "__" around (see https://github.com/mozilla/mentat/blob/a152e6004013fcb5a0a51df5c38f9dad09964cef/edn/src/edn.rustpeg#L141)


Diagonal Movement Rule BUG

The optional diagonal movement DMG rule is not working.

What is your Foundry & DnD5e version?

What is your Foundry & DnD5e version?

Foundry V12.329 and DnD5e V3.3.0

We forgot the hide dnd5e's "Diagonal Movement Rule" setting in V12 (#3851), which doesn't do anything anymore. Now there's the core setting "Square Grid Diagonals". You'll need to set that to "Alternating (1/2/1)" if you want the DMG movement rules.


Would a Stretch flavor be possible?

I was using python:2.7-slim with stretch for a while and got hit by a version incompatibility b/t gnupg and taskd without a non-aptitude type solution.

I think that right now Debian Stretch is too early in the cycle to promote as a stable base for language stacks.  Since you are already using slim, have you tried the python:2.7-alpine version?  It usually has newer packages to install (though it may require using the edge apk repository).

This should probably be revised, now that Stretch is frozen.

I would like to see a 2.7 stretch image too; we can't use the alpine images because they are incompatible with manylinux1 wheels.

The list is of options is starting to get long, but what are the thoughts on having a slim variant based on stretch?

We do have stretch based images for 2.7, 3.6, and 3.7 (https://github.com/docker-library/python/pull/215).  As for slim variant of stretch, we'll track that in #219.


feat(mock): return QuotaCostList

The mock should return the list of QuotaCost.

We are missing env var to pick different AMS entitelements


Servable not found for request: Latest(mnist)

I am trying to run model.py with tfjob. Below is my yaml file to create a tfjob.
apiVersion: "kubeflow.org/v1alpha1"
kind: "TFJob"
metadata:
  name: "example-job3"
  namespace: tfworkflow
spec:
  replicaSpecs:
    - replicas: 1
      tfReplicaType: MASTER
      template:
        spec:
          serviceAccountName: tf-job-operator
          containers:
            - image: tensorflow/tensorflow:1.5.1
              name: tensorflow
              imagePullPolicy: Always
              command:
               - /usr/bin/python
              args:
               - /data/pijie/mnist/model.py
              env:
               - name: TF_DATA_DIR
                 value: /data/pijie/mnist/data-12345
               - name: TF_MODEL_DIR
                 value: /data/pijie/mnist/model-12345
               - name: TF_EXPORT_DIR
                 value: modelf-12345
               - name: TF_TRAIN_STEPS
                 value: "200"
               - name: TF_TF_BATCH_SIZE
                 value: "100"
               - name: TF_LEARNING_RATE
                 value: "0.01"
              volumeMounts:
               - name: data-persistent-storage
                 mountPath: /data
          volumes:
          - name: data-persistent-storage
            persistentVolumeClaim:
              claimName: tfworkflow-pvc
          restartPolicy: OnFailure
    - replicas: 2
      tfReplicaType: WORKER
      template:
        spec:
          serviceAccountName: tf-job-operator
          containers:
            - image: tensorflow/tensorflow:1.5.1
              name: tensorflow
              imagePullPolicy: Always
              command:
               - /usr/bin/python
              args:
               - /data/pijie/mnist/model.py
              env:
               - name: TF_DATA_DIR
                 value: /data/pijie/mnist/data-12345
               - name: TF_MODEL_DIR
                 value: /data/pijie/mnist/model-12345
               - name: TF_EXPORT_DIR
                 value: modelf-12345
               - name: TF_TRAIN_STEPS
                 value: "200"
               - name: TF_TF_BATCH_SIZE
                 value: "100"
               - name: TF_LEARNING_RATE
                 value: "0.01"
              volumeMounts:
               - name: data-persistent-storage
                 mountPath: /data
          volumes:
          - name: data-persistent-storage
            persistentVolumeClaim:
              claimName: tfworkflow-pvc
          restartPolicy: OnFailure
    - replicas: 2
      tfReplicaType: PS
      template:
        spec:
          serviceAccountName: tf-job-operator
          containers:
            - image: tensorflow/tensorflow:1.5.1
              name: tensorflow
              imagePullPolicy: Always
              command:
               - /usr/bin/python
              args:
               - /data/pijie/mnist/model.py
              env:
               - name: TF_DATA_DIR
                 value: /data/pijie/mnist/data-12345
               - name: TF_MODEL_DIR
                 value: /data/pijie/mnist/model-12345
               - name: TF_EXPORT_DIR
                 value: modelf-12345
               - name: TF_TRAIN_STEPS
                 value: "200"
               - name: TF_TF_BATCH_SIZE
                 value: "100"
               - name: TF_LEARNING_RATE
                 value: "0.01"
              volumeMounts:
               - name: data-persistent-storage
                 mountPath: /data
          volumes:
          - name: data-persistent-storage
            persistentVolumeClaim:
              claimName: tfworkflow-pvc
          restartPolicy: OnFailure

But when I used tf-serving to serve the trained model, and predicate a number.
/usr/bin/tensorflow_model_server --port=9000 --model_name=modelf-12345 --model_base_path=/model-12345/export/modelf-12345

python mnist_client.py

I got "Servable not found for request: Latest(mnist)" error. I am working on kubernetes 1.9.3. Can anybody help me?
root@77f321b127c5:/notebooks# python mnist_client.py
/usr/local/lib/python2.7/dist-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
Traceback (most recent call last):
  File "mnist_client.py", line 45, in <module>
    result = stub.Predict(request, 10.0)  # 10 secs timeout
  File "/usr/local/lib/python2.7/dist-packages/grpc/beta/_client_adaptations.py", line 309, in __call__
    self._request_serializer, self._response_deserializer)
  File "/usr/local/lib/python2.7/dist-packages/grpc/beta/_client_adaptations.py", line 195, in _blocking_unary_unary
    raise _abortion_error(rpc_error_call)
grpc.framework.interfaces.face.face.AbortionError: AbortionError(code=StatusCode.NOT_FOUND, details="Servable not found for request: Latest(mnist)")

However, it works when I follow the steps in here using argo.

/api v1alpha1
/kind question

I am not sure if it is the right place for the question since it is related to tf-serving.
/cc @jlewi

Hi @zuowang
I am not sure if the dir here is right:
/usr/bin/tensorflow_model_server --port=9000 --model_name=modelf-12345 --model_base_path=/model-12345/export/modelf-12345

Maybe it should be /data/pijie/mnist/model-12345. :thinking:
BTW, I think the error is not caused by tf-operator.

@gaocegege The dir is right. /model-12345/export/modelf-12345 is the exported model.

The error "Servable not found for request: Latest(mnist)  indicates a problem with TFServing and the client. I believe this error basically means that the request specified a model of Latest(mnist) and this was not found.
What do the logs for TFServing show?
/cc @elsonrodriguez @lluunn

Can you try changing --model_name=modelf-12345 to --model_name=mnist for the tf-seving command?


Add option to set statusline highlight group to be used when showing a message.

Since the default message is hardly visible to me, i wanted to make it more visible with the statusline highlight group. So i tried using this:
hi User1 guifg=#FFFFFF guibg=#FF0000
let g:keepeye_message = '%1*SAVE YOUR EYES, TAKE A BREAK'

But that's not working as intended because the padding is added to make the message centered.

So i added an optional variable that can be set to any number from 1 to 9, and it will use User1..9 respectively.
I'm just not sure how will this reflect airline users, since i'm not using airline.

Why using a highlight group ? I use this plugin with my own callback, and I just set highlight StatusLine guibg=#e45649 to get the full bar colored in red

How do you reset it back to normal?

It's done automatically when I change of mode, but it's thanks to my configuration, not to this plugin. You right, this option is a very good idea in fact

Readme updated.

Thank you :tada:


[Documentation]: Zephyr-7b-beta seems tested on 4-bit where there is 5-bit version possible

Documentation Issue Description
Found post on linkedin about evalution about Zephyr-7b-beta.
https://docs.llamaindex.ai/en/latest/module_guides/models/llms.html#open-source-llms
I saw that 4 bit quantization was used , and given I know there are 5 bit quantized version there (which we use on https://huggingface.co/spaces/coqui/voice-chat-with-zephyr  already, ) so I tested LlamaIndex Zephyr colab with 5-bit GGUF version.
Here is result notebook , hoping someone will validate results, as I am not sure when you set a test pass or fail
https://colab.research.google.com/drive/1ULelRgpKbCNIJwdP1vBvZtA7yTTHRwzK?usp=sharing
Quick edit here in case notebook fails to load:
GGUF version can be run on CUDA with following (zephyr-beta 5bit uses ~8Gb memory total), everything else on test same just llm replaced. Note that sometimes there are json parser errors and rarely produced prompt passes 3600 tokens , but I does rerun and pass.
Note that I did not change prompt, that is why there may be new lines on Q/A , but I guess we are checking final response only.

# need to install llama-cpp-python with cublas to make affective GPU usage
#!CMAKE_ARGS="-DLLAMA_CUBLAS=on" pip install llama-cpp-python

from llama_index.llms import LlamaCPP
from llama_index.llms.llama_utils import completion_to_prompt

llm = LlamaCPP(
    model_url="https://huggingface.co/TheBloke/zephyr-7B-beta-GGUF/resolve/main/zephyr-7b-beta.Q5_K_M.gguf",

    # optionally, you can set the path to a pre-downloaded model instead of model_url
    # first download will download into /tmp/llama_index/models/zephyr-7b-beta.Q5_K_M.gguf on colab
    model_path=None,

    temperature = 0.7, # set same as on original test

    max_new_tokens=256, # set same as on original test

    # llama2 has a context window of 4096 tokens, but we set it lower to allow for some wiggle room
    context_window=3900,  # note, this sets n_ctx in the model_kwargs below, so you don't need to pass it there.

    # kwargs to pass to __call__()
    generate_kwargs={"top_k":50, "top_p" : 0.95}, # set same as on original test

    # kwargs to pass to __init__()
    # set to at least 1 to use GPU  (depends on model for LLama base usually 35 total)
    model_kwargs={"n_gpu_layers": 35}, # 35 is full GPU, set 10-15 if you have lower VRAM , make sure llama_cpp compiled with cublas for better GPU speed

    # transform inputs into Llama2 format
    messages_to_prompt=messages_to_prompt, # this is overriden above
    #completion_to_prompt=completion_to_prompt, # imported via llama_utils
    verbose=True, # must be set True to use on llama_index
)

Documentation Link
https://docs.llamaindex.ai/en/latest/module_guides/models/llms.html#open-source-llms

@gorkemgoknar thanks for contributing this notebook! Feel free to make a PR to add it to our docs :)
I would edit some of the text to make it more correct (i.e. you are using a 5bit model with llama.cpp)
In terms of passing, it seems like all seem to work initially!


samples in java

Please. I know that nowadays they are using the kotlin a lot. But java is still being used a lot. And converting a kotlin project to java is very difficult. Could you make samples in java? would help me a lot.

Unfortunately, Android is Kotlin-first; we focus on creating and maintaining samples in Kotlin. We evaluate whether a sample needs a Java version on a case-by-case basis, and if there's enough developer interest we will port this sample to Java.

I'm also expecting the Java version.

old versions exists in the archive..

How is developer interest gauged? Is there somewhere we can bump/vote/push for this?


changedRows is English language-specific

The changeRows property is set from a regex that assumes an English language message response.  This functionality breaks in multi language environments.  For example, a French installation of MySQL will return modifié, which will be not be caught by the regex.
In my view, there are two solutions:

Attempt to improve the regex to be multi-language.
Document the language-specific nature of this property and suggest that non-English environments use affectedRows instead.

For further relevant information, see the discussion in https://github.com/mysqljs/mysql/issues/251 which pointed me to the changedRows property.

Ideally if it can be modified to support all languages that would be much better than a doc change. Docs can be easily overlooked and of course just working right is better anyway :)

That sounds like the best solution.  In light of https://github.com/mysqljs/mysql/issues/1819#issuecomment-329151763, would you like me to close https://github.com/mysqljs/mysql/pull/1820?

I can still merge the docs change, and keep this open for tracking getting it to actually work for all the languages.

Here is the translation strings from 5.7.17 for the message:
ER_UPDATE_INFO
        cze "Nalezených řádků: %ld  Změněno: %ld  Varování: %ld"
        dan "Poster fundet: %ld  Ændret: %ld  Advarsler: %ld"
        nla "Passende rijen: %ld  Gewijzigd: %ld  Waarschuwingen: %ld"
        eng "Rows matched: %ld  Changed: %ld  Warnings: %ld"
        est "Sobinud kirjeid: %ld  Muudetud: %ld  Hoiatusi: %ld"
        fre "Enregistrements correspondants: %ld  Modifiés: %ld  Warnings: %ld"
        ger "Datensätze gefunden: %ld  Geändert: %ld  Warnungen: %ld"
        hun "Megegyezo sorok szama: %ld  Valtozott: %ld  Warnings: %ld"
        ita "Rows riconosciute: %ld  Cambiate: %ld  Warnings: %ld"
        jpn "該当した行: %ld  変更: %ld  警告: %ld"
        kor "일치하는 Rows : %ld개 변경됨: %ld개  경고: %ld개"
        por "Linhas que combinaram: %ld - Alteradas: %ld - Avisos: %ld"
        rum "Linii identificate (matched): %ld  Schimbate: %ld  Atentionari (warnings): %ld"
        rus "Совпало записей: %ld  Изменено: %ld  Предупреждений: %ld"
        serbian "Odgovarajućih slogova: %ld  Promenjeno: %ld  Upozorenja: %ld"
        spa "Líneas correspondientes: %ld  Cambiadas: %ld  Avisos: %ld"
        swe "Rader: %ld  Uppdaterade: %ld  Varningar: %ld"
        ukr "Записів відповідає: %ld  Змінено: %ld  Застережень: %ld"

It looks like it's the only message that would match the language-neutral pattern /^[^:]+: ([0-9]+)[^:]+: ([0-9]+)[^:]+: ([0-9]+)/

Ok, so looks like that regular expression seems to work fine. If you want to try it out before the next release, you can use the instructions at https://github.com/mysqljs/mysql#install to install the master branch to test.

Awesome!  I'll give it a whirl and comment back when we have some results.


az bicep decompile should not overwrite existing files without confirmation

Is your feature request related to a problem? Please describe.
When using az bicep decompile, the target file (if already exists) is overwritten without confirmation.
Describe the solution you'd like
When the target file already exists, a confirmation for overwriting should be asked, and a flag --overwrite --allowOverwrite or something similar should exist

I think this is a fair ask. The question is what would be the default behavior. Today we override the file and going the opposite direction would be a breaking change.
I think we should get a bit more feedback on this, for what would be the default behavior.

Fair enough. Al I'm the first person to raise this, this might indicate that it is perfectly fair to hold this off until the next group of breaking changes (0.5 maybe?)

We'll discuss this in triage and we'll post the update here

A PR has been created to fix this: https://github.com/Azure/bicep/pull/6029.  However, it's not clear that the desired behavior has been resolved.  The PR changes the default behavior to be to fail if the output file already exists, unless the --allowoverwrite flag is added.  Thus a breaking change, but it's does seem like the right behavior to me.  Any objections?
I'm inclined to prefer the simpler "--overwrite" instead of "allowoverwrite", though.

Even if it's a breaking change I think it should have failed from the beginning.
From memory, others commands use --force, same think for the upcoming restore command. Wouldn't that make it more on-par with other experiences if it's --force instead?

I considered "--force" as well, decided "--overwrite" is more explicit, but don't have strong opinion.

Rethinking my original report, --force also makes a lot of sense as it is close to how many other commands out there work. If it's also used in other locations (restore command), that's another argument for --force I believe.


S4 m1 silicon support

added "mps" as device to support M1 and M2 apple gpu
fixed a couple of typos in the project description
fixed typo in logging

thanks @SebastianBitsch


Highlight select field with error to match Django style

Since the Django generated select box is hidden with select2, the red border style does not appear when there is an error. This fix adds that style the select2 box.

Nice one, thanks !

Released in 3.5.0


Should this work with dagger and kotlin/kapt?

I'm trying to make AssistedInject work with dagger and kotlin but no success so far.
It seems dagger generates module classes before assisted factories so it ends up with a error NonExistentClass  from the factory binding.
@Binds abstract fun bindHomeAssistedInjectFactory(factory: HomeAssistedInject_AssistedFactory): HomeAssistedInject.Factory
generates :
@org.jetbrains.annotations.NotNull()     @dagger.Binds()     public abstract HomeAssistedInject.Factory bindHomeAssistedInjectFactory(@org.jetbrains.annotations.NotNull()     NonExistentClass factory);
I remember trying with google AutoFatory a while ago and ending up with the same pb so i guess this is a kapt problem?
Thx for help!

Yes. You can perhaps follow Tivi's example: https://github.com/chrisbanes/tivi/commit/b7d976cec5b1a680d743da7f5bcee425acd5cc95.
Are you using a relatively modern version of Dagger and Kotlin?

Hum i'm was trying to add @AssistedModule on an existing non empty module, looks like we have to make that "Assisted Module"empty to make it works. So i created an empty module with @AssistedModule and include it to my existing module and now it works.
Thx Jake for the link, that helped me pointing out the pb.


Style improvements to no-var-keyword rule

Lint.isBlockScopedVariable now supports VariableStatements.

LGTM, pending the flags thing.

I tried to improve the readability of Lint.isBlockScopedVariable by adding a private helper function.

failing because of nested binding patterns... darn. will try and fix

fixed now

I'm not sure about the test case you added, other than that looks good.


Add FE changes to sidebar tasklist

styles for right hand task list including font size changes, indent fixes, improved spacing, highlighting for active page
styles for sticky heading link

https://trello.com/c/riC1fX76/152-design-build-a-sidebar-that-includes-steps-and-content-links-highlighting-the-page-youre-on

@andysellick I've noticed that when another section is clicked then we have two sections in yellow
Also, could you please highlight the whole section instead of only the title?
Thank you

Merging for show and tell


Chat eats my text

Description
I wouldn't be surprised if I'm the only one having this issue, but chat eats my text. Doesn't matter what I type in, after a certain threshold, it just deletes everything. Happens anywhere I can type, and when it happens, it's kinda global (happens in all typeboxes at once).
It is also seemingly random, coming and going as it pleases. All I can do when it happens is copy the text from somewhere else and send it.
I have tried reinstalling and all, no dice.
Screenshots
https://user-images.githubusercontent.com/52474532/163902127-f0935716-e318-4b5f-ba22-9669577e3d67.mp4

Is this specifically when writing the 'a'? I had a similar issue, ended up realizing my computer kept either alt or ctrl pressed down.
Smashing the key a couple of times should fix it.
If this is related to ss14 itself and not the window lagging after switching tabs and stuff, then not sure what's happening.

It is not specific to one letter, sadly. It seems to be a set number of characters (about 15), but the characters are all.

@Zymem You know, you were right the first time. It is actually related to writing 'a'. I can guarantee my other buttons are not pressed down tho, I installed a visualizer.

I am now inclined to believe it actually has something to do with my keyboard/pc. Doesn't register in any visualizers, but instantly goes away if I press ctrl.


Fixes coalesce operation race condition

Even though _identifierToOperations is a weak-weak map, we still need to remove it when we removed operation. Otherwise, it may exists race condition which leads new added completion not be called for example.
A case:
Let's assume L384 and L213 runs concurrently, L384 executes firstly, then L213 executes, new added completion would not be called anymore.
https://github.com/zhongwuzw/PINOperation/blob/eed328e43f706fa9e4b2cc38e7ed1ba83dbb1c8c/Source/PINOperationQueue.m#L384
https://github.com/zhongwuzw/PINOperation/blob/eed328e43f706fa9e4b2cc38e7ed1ba83dbb1c8c/Source/PINOperationQueue.m#L213

          1 Warning

:warning:
Any source code changes should have an entry in CHANGELOG.md or have #trivial in their title.

  Generated by :no_entry_sign: Danger

Do you mind rebasing against master for the new GitHub CI?

@garrettmoon Done.


Incorrect centroid location

Thank you for this great piece of work!
I spotted an issue in the latest version (0.4.0), that the centroid information returned by slic.slic_model.clusters is incorrect.
Minimal working example:
import numpy as np

# Much faster than the standard class
from fast_slic.avx2 import SlicAvx2
from PIL import Image
import cv2

with Image.open('fish.jpg') as f:
   image = np.array(f)

slic = SlicAvx2(num_components=512, compactness=10)
assignment = slic.iterate(image) # Cluster Map

x_centroid_map = np.zeros_like(assignment, dtype=np.uint16)
y_centroid_map = np.zeros_like(assignment, dtype=np.uint16)
for c in slic.slic_model.clusters:
    y, x = c['yx']
    x_centroid_map[assignment==c['number']] = x
    y_centroid_map[assignment==c['number']] = y

out = Image.fromarray((x_centroid_map.astype('float')/3000*255).astype('uint8'))
out.save('x.jpg')

out = Image.fromarray((y_centroid_map.astype('float')/3000*255).astype('uint8'))
out.save('y.jpg')

I would expect the (x, y) centroid map to look like this:
X:

Y:

But it gives me this for X:

and Y is fine.
Note that this works in version 0.3.5 (which I used to generate the "expected output")!
Hope this helps :smile:

I like the output of 0.4 more though, guess I will compute the centroid again in numpy for now.

Thanks for the report. I'll have a look on the version you mentioned. (it may take time though, because of the ongoing issue in my country😭)

Stay safe! I am in Hong Kong so 👀

Hello @Algy
As I understand the problem with centroid is still available. Can you advise me a fast solution to calculate it by myself?
Currently I'm using regionprops from skimage, but it's too slow

Hello @Algy ,
Thanks for this library, it is really fast and does the job very well.
I think that I met an issue that could be connected to the centroid one. If I show the image obtained by coloring every pixel of original image with the color of the assigned cluster I get the colors all shuffled up. I think that the indices of the clusters do not match the indices in the assignment matrix.
PS I checked the color space and it is not a problem of color conversion.

Any progress on the problem? I encounter this problem too.


Flutter Command cannot find GIT

When the flutter plugin Command is being used to generate sources, the flutter command cannot find GIT.
I added the GIT path to the Windows system PATH, and reboot the machine.
C:\Program Files\Git\cmd;
c:\Program Files\Git\bin\git.exe;
c:\Program Files\Git\bin;

This Flutter command had worked once or a few times before, but stopped working.
2024-01-17T16:58:24.2702575Z ##[debug]Evaluating condition for step: 'Flutter Command - Generate code'
2024-01-17T16:58:24.2704081Z ##[debug]Evaluating: SucceededNode()
2024-01-17T16:58:24.2704311Z ##[debug]Evaluating SucceededNode:
2024-01-17T16:58:24.2704716Z ##[debug]=> True
2024-01-17T16:58:24.2705195Z ##[debug]Result: True
2024-01-17T16:58:24.2705757Z ##[section]Starting: Flutter Command - Generate code
2024-01-17T16:58:24.2982402Z ==============================================================================
2024-01-17T16:58:24.2982548Z Task         : Flutter Command Task
2024-01-17T16:58:24.2982617Z Description  : Launch a custom Flutter command with custom arguments.
2024-01-17T16:58:24.2982715Z Version      : 0.3.2
2024-01-17T16:58:24.2982788Z Author       : Hey24sheep
2024-01-17T16:58:24.2982851Z Help         : Don't forget to insert a 'Flutter Install' task before. [More Information](https://github.com/hey24sheep/azure-flutter-tasks)
2024-01-17T16:58:24.2982998Z ==============================================================================
2024-01-17T16:58:24.3337213Z ##[debug]Using node path: D:\BuildAgentMobileWeb\externals\node10\bin\node.exe
2024-01-17T16:58:24.5509530Z ##[debug]agent.TempDirectory=D:\BuildAgentMobileWeb\_work\_temp
2024-01-17T16:58:24.5548681Z ##[debug]loading inputs and endpoints
2024-01-17T16:58:24.5573638Z ##[debug]loading ENDPOINT_AUTH_PARAMETER_SYSTEMVSSCONNECTION_ACCESSTOKEN
2024-01-17T16:58:24.5648832Z ##[debug]loading ENDPOINT_AUTH_SCHEME_SYSTEMVSSCONNECTION
2024-01-17T16:58:24.5653632Z ##[debug]loading ENDPOINT_AUTH_SYSTEMVSSCONNECTION
2024-01-17T16:58:24.5657386Z ##[debug]loading INPUT_ARGUMENTS
2024-01-17T16:58:24.5660690Z ##[debug]loading INPUT_FLUTTERDIRECTORY
2024-01-17T16:58:24.5663887Z ##[debug]loading INPUT_PROJECTDIRECTORY
2024-01-17T16:58:24.5670084Z ##[debug]loading SECRET_SYSTEM_ACCESSTOKEN
2024-01-17T16:58:24.5677118Z ##[debug]loaded 7
2024-01-17T16:58:24.5693658Z ##[debug]Agent.ProxyUrl=undefined
2024-01-17T16:58:24.5694157Z ##[debug]Agent.CAInfo=undefined
2024-01-17T16:58:24.5694523Z ##[debug]Agent.ClientCert=undefined
2024-01-17T16:58:24.5694822Z ##[debug]Agent.SkipCertValidation=undefined
2024-01-17T16:58:24.5705096Z ##[debug]FlutterToolPath=D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin
2024-01-17T16:58:24.5715375Z ##[debug]projectDirectory=D:\BuildAgentMobileWeb\_work\1\s
2024-01-17T16:58:24.5715722Z ##[debug]Moving to D:\BuildAgentMobileWeb\_work\1\s
2024-01-17T16:58:24.5774996Z ##[debug]Project's directory : D:\BuildAgentMobileWeb\_work\1\s
2024-01-17T16:58:24.5778726Z ##[debug]arguments=pub run build_runner build --verbose --delete-conflicting-outputs
2024-01-17T16:58:24.5779078Z ##[debug]argSeparator=undefined
2024-01-17T16:58:24.5783501Z ##[debug]which 'D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter'
2024-01-17T16:58:24.5802715Z ##[debug]found: 'D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat'
2024-01-17T16:58:24.5805408Z ##[debug]D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat arg: ["pub","run","build_runner","build","--verbose","--delete-conflicting-outputs"]
2024-01-17T16:58:24.5805951Z ##[debug]D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat arg: ["pub","run","build_runner","build","--verbose","--delete-conflicting-outputs"]
2024-01-17T16:58:24.5810526Z ##[debug]exec tool: D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat
2024-01-17T16:58:24.5810936Z ##[debug]exec tool: D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat
2024-01-17T16:58:24.5811269Z ##[debug]arguments:
2024-01-17T16:58:24.5811533Z ##[debug]arguments:
2024-01-17T16:58:24.5811919Z ##[debug]   pub
2024-01-17T16:58:24.5812163Z ##[debug]   pub
2024-01-17T16:58:24.5812427Z ##[debug]   run
2024-01-17T16:58:24.5812681Z ##[debug]   run
2024-01-17T16:58:24.5812940Z ##[debug]   build_runner
2024-01-17T16:58:24.5813209Z ##[debug]   build_runner
2024-01-17T16:58:24.5817730Z ##[debug]   build
2024-01-17T16:58:24.5817985Z ##[debug]   build
2024-01-17T16:58:24.5818244Z ##[debug]   --verbose
2024-01-17T16:58:24.5818503Z ##[debug]   --verbose
2024-01-17T16:58:24.5818781Z ##[debug]   --delete-conflicting-outputs
2024-01-17T16:58:24.5819070Z ##[debug]   --delete-conflicting-outputs
2024-01-17T16:58:24.5827286Z [command]C:\Windows\system32\cmd.exe /D /S /C "D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat pub run build_runner build --verbose --delete-conflicting-outputs"
2024-01-17T16:58:24.7863819Z Error: Unable to find git in your PATH.
2024-01-17T16:58:24.7874479Z ##[debug]Exit code 1 received from tool 'D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat'
2024-01-17T16:58:24.7875002Z ##[debug]Exit code 1 received from tool 'D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat'
2024-01-17T16:58:24.7880266Z ##[debug]STDIO streams have closed for tool 'D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat'
2024-01-17T16:58:24.7881381Z ##[debug]STDIO streams have closed for tool 'D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat'
2024-01-17T16:58:24.7928602Z ##[debug]task result: Failed
2024-01-17T16:58:24.7967636Z ##[error]Error: The process 'D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat' failed with exit code 1
2024-01-17T16:58:24.7978184Z ##[debug]Processed: ##vso[task.issue type=error;]Error: The process 'D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat' failed with exit code 1
2024-01-17T16:58:24.7979504Z ##[debug]Processed: ##vso[task.complete result=Failed;]Error: The process 'D:\BuildAgentMobileWeb\_work\_tool\Flutter\3.16.3\windows\flutter\bin\flutter.bat' failed with exit code 1
2024-01-17T16:58:24.7997090Z ##[section]Finishing: Flutter Command - Generate code

I put a workaround this by making the Windows user who is running the Windows agent as a user with admin rights.

The key indicator was running the "flutter.bat" that is being used on the Windows agent.  When running in non-admin mode, the batch command would not run at all, or it would hang.  When I ran the same "flutter.bat" in admin console mode, then it ran as expected.

I suspect that the reason why the Windows agent worked for the first few times was because the agent was started from the command line "run.cmd" from an admin command prompt.  And only when the server was restarted -- then it used the NT service to restart the agent.  Then that was when the agent was not able to invoke the flutter.bat as a NT Service.

@sk92129 I wish I could help you here but I do not have experience setting up a local windows agent. This issue is with the setup on a local windows agent more so than this extension, I will keep this issue open for now. But, I will close this issue within few weeks if no one else can help you.
It would be better if you ask for help on Stackoverflow or on Azure DevOps Help forums. Maybe someone smarter than me can help you there.

This problem was due to the fact that the Windows agent was not running in admin mode.  Once the agent was changed to run as Admin this problem went away.


Install pre-commit and use GitHub Actions

Description
-> Use pre-commit instead of flake8 for code-quality checks. Pre-commit provides more hooks for quality checking and it also helps to auto-correct the Code-quality errors.
-> Using GitHub actions to run the code quality check and install job CI. This helps to see the results directly on the GitHub pr, at present we are using TravisCI.
Sample runs of GitHub action -> https://github.com/amitsagtani97/kraken/actions
Fixes

Can one of the admins verify this patch?

LGTM, really appreciate the addition in the contribution doc! I haven't heard of this before and the documentation made it super easy and I'm sure will help others trying to contribute

LGTM


demo vulkan cube sdl2 invalid go

# github.com/vulkan-go/demos/vulkancube/vulkancube_sdl2
./main.go:33:31: cannot use surfPtr (type unsafe.Pointer) as type uintptr in argument to vulkan.SurfaceFromPointer
Tue 1 Dec 18:29:30 GMT 2020

fixed by adding type conversion...
surf := vk.SurfaceFromPointer(surfPtr)

->
surf := vk.SurfaceFromPointer(uintptr(surfPtr))

plus, when debugging enabled, i get...
closer.go:137: run time panic: runtime error: cgo argument has Go pointer to Go pointer

@splace Hi, thanks for reporting this. The issue with pointer to Go memory will be resolved when I re-generate bindings, since c-for-go improved lately.