Releases & Changelog

It would be helpful to:
Publish Releases on Github
Publishing releases on Github makes it easy to see what the latest and previous releases are, without having to go to NPM. To do this, you'll also have to git tag versions of Mirror.
Add a Changelog
As a project evolves, its helpful to know what has changed - especially any possible breaking changes. You might want to consider adding a CHANGELOG.md. This could be automated
with a variety of tools, such as github-changelog-generator.

If this seems like too much work, don't worry about it. But if you'd like any assistance, I'm happy to help.

Been busy for the last week 😂
Yes, releases and CHANGELOG is necessary, I'm working on it.

Oh, good stuff. Thanks :)
I’m on a trip this weekend, but I’ll send you the mostly finished presentation before Monday. We made quite a few changes this week.
I may also be able to finish the router. I’m having issues with creating a submodule when bundling with Webpack. Will figure it out.

Shawn

On Sep 16, 2017, at 3:18 AM, Linghao Li<EMAIL_ADDRESS>wrote:
Been busy for the last week 😂
Yes, releases and CHANGELOG is necessary, I'm working on it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/mirrorjs/mirror/issues/48#issuecomment-329959767, or mute the thread https://github.com/notifications/unsubscribe-auth/AEcds8FBedNn5u8ZVvitjsyAZQiP2lUMks5si6CFgaJpZM4PSZsA.

I should remind you that I've made a change about Router to handle props like basename etc, so your current work on submodule router might be influenced.

Hi @ShMcK , Change log is finally added :)


⚠ Mobitopup has degraded performance

In cbb14b7, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 7772 ms

Resolved: Mobitopup performance has improved in b9cf96f.


[HOPS-482] Block report load blancing

Make sure there is no duplicate PR for this issue

Please check if the PR fulfills these requirements

[x] Tests for the changes have been added and passed (for bug fixes / features)
[x] HOPS JIRA issue has been opened for this PR
[x] All commits have been squashed down to a single commit
[x] The commit message has the following format: [HOPS-XXX] message

Post a link to the associated JIRA issue
https://hopshadoop.atlassian.net/browse/HOPS-482

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
bug fix

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Can one of the admins verify this patch?

Jenkins test this please.


Missing PHP extensions in httpd module

Describe the bug
When I enable PHP in httpd using services.httpd.enablePHP = true, the file
<?php
var_dump(get_loaded_extensions());

when visited can only see 11 extensions
array(11) { [0]=> string(4) "Core" [1]=> string(4) "date" [2]=> string(6) "libxml" [3]=> string(4) "pcre" [4]=> string(4) "hash" [5]=> string(3) "SPL" [6]=> string(8) "standard" [7]=> string(10) "Reflection" [8]=> string(4) "Phar" [9]=> string(3) "xml" [10]=> string(14) "apache2handler" }

instead of the 52 I get when I run it through php CLI command.
To Reproduce
A failing NixOS test: https://github.com/NixOS/nixpkgs/pull/85402
Additional context
Likely caused by the great refactor: https://github.com/NixOS/nixpkgs/pull/83896
Notify maintainers
@etu, @talyz

Yep, the refactor relies on a configuration file with all the extensions to load being present and specified. For the binary, we have a wrapper that just tells PHP to look in the right folder for the config, but with the httpd module, it looks like that's not used, since it loads PHP as an apxs2 extension.
The solution would probably be to inherit the config from the php package as the first thing put into the php.ini it already generates; it should be accessible as ${php}/lib/custom-php.ini.


Override http.trusted_networks by auth_provider.trusted_networks

Description:
This should be done when we informed user http.trusted_network has been deprecated
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

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

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

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

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

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

If the code does not interact with devices:

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

This PR just a hot fix, I am going to rewrite setup_auth along with trusted_network auth provider enhancement.

Shouldn't we just remove the HTTP trusted networks feature ?

I want to remove it in next release, not in hot fix

So right. What am I saying.


Remove volume slider toggle

Similar to the option to remove the contrast slider, can you please add an option to remove the volume slider? I'm using a Dell U2515H and it doesn't have sound. The volume slider moves, but does nothing. I'm using a Dell USB sound bar thing that attaches to my monitor.

Hi, I added this in the new 3.0.0 version. You can check out the current beta or wait for the release! Please test and close the issue if it works. Thanks!

Hi, we have relased the 3.0.0 version. Could you please test if this issue is resolved and report back? Thank you!

I'll close this one since it should be solved. If there is any issue with this, please feel free to comment and ask for a reopen! Thank you!

Thanks it works great!


childSubmissions sends submission updates

Co-authored-by: Liliana<EMAIL_ADDRESS>Description
Fixes #

What work was done?
Why was this work done?
What feature / user story is it for?
Any relevant links or images / screenshots

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

[ ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

Has This Been Tested

[ ] Yes
[ ] No
[ ] Not necessary

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] There are no merge conflicts

 Need A Function that reaches out to the BE API and updates specified fields for the submission record


fix-vuln

OKTA-692039

Looks like I need to investigate the failures more closely 🤔

Extension request submitted.


Autosave: should save data every ? trials

Instead of saving all of the data at the end of the session, we should save data every ? number of trials
This should happen in a thread to prevent stalling trials, and we should measure inter-trial variability to ensure this is true
The interval of trials to save should be determined by the practical constraints of saving

I discussed this with Jeremiah. We reasoned that <1% data loss in the training room is probably ok (not great, but acceptable). Data loss during ephys recordings would not be. We think that the Harp messages are being logged and we can always try to reconstruct data if needed. We decided to hold off on this until we get 446 up and running and the new motor stages, then we can evaluate how often the session is crashing

I am reviving this thread. We lost an ephys session last week and so want to roll out a solution to this before we loose any more.
I think it makes the most sense to save data at the end of each trial. Because data arrays must, by necessity, grow on each trial, the safest thing is probably going to be writing trials one at a time and recombine at the end- that way sessions can go on forever without increasing saving load. We can do validation to confirm that this can be reconstructed exactly into the existing format.
In interest of getting something working before we do much more ephys, its probably easiest to start with a non-threaded version of this and work our way up to thread-safe version in the next iteration. I can take a stab a solution this week.

@yonibrowning Thanks for taking this issue on.
I would recommend appending each trial to a file, rather than creating individual files for each trial. That way no reconstruction is necessary.
I would also investigate what information is currently being logged from the Harp messages - it might be easier just to implement a function to recover lost data from the Harp logs.
What rig/time did you lose an ephys session? is the underlying issue documented?

I have created a pull request to address this here: https://github.com/AllenNeuralDynamics/dynamic-foraging-task/pull/484
At present, I opted for many small files because I do not know of a safe way to continuously write to a json. This could easily be changed if someone has a preferred format.
As to your other comments:
Harp logs won't have task state information, will they? They are better than nothing but not complete...
Our loss was on rig 2, the cause was disconnecting a harp device during a session (read "user error"). "Force save" did not work and the session couldn't complete with bonsai stopped, so we lost the data.

I didn't know anyone was running the task on ephys rig 2. Can you please enable automatic daily updates, install openssh server, and tell me the hostname so I can hook that rig into our system for tracking errors?

I didn't know anyone was running the task on ephys rig 2. Can you please enable automatic daily updates, install openssh server, and tell me the hostname so I can hook that rig into our system for tracking errors?

Long term yes, but I don't think we can in the short term. We have been piloting a breathing sensor that talks to bonsai on there, and until the code this sensor makes its way into the production repos I worry it will break with every auto update? If there is a way to do this then we can reconsider - it is all bonsai code (the passive sensor doesn't talk to the Gui at all), so one valid option might be to temporarily link to non-updating bonsai workflow?
I'll create a separate issue to discuss.


Handle URLs properly

The [https://github.com/NodeRedis/node_redis](Node.js redis client) supports (and explicitly documents) fully-specified URLs, i.e. [redis:]//[user][:password@][host][:port][/db-number][?db=db-number[&password=bar[&option=value]]].
The socket.io-redis adaptor can only handle URLs of the form hostname:port.

Thanks! The patch above is waiting for merge on PR #94. Hopefully someone can get to it soon.

That would be cool to see this patch merged.
Thanks @hgcummings for this fix.

Any chance of any movement on this? - It's becoming more inconvenient for me and I was hoping my PR would have been merged by now. Or is this blocked on issue #75?
Please let me know if there's anything I can do to help get the PR merged.


Possible memory on endpoint detail pages

Describe the bug
When I open an endpoint detail page endpoints/xxx in the browser, the memory consumption on the service increases on every refresh.
After some refreshes the container crashes with OutOfMemory.
It seems that the memory increases on each request on the backend. Are there some cached data, that is not released?
What do you see?
The memory consumption increases  on each refresh made on an endpoint detail page which leads to an OutOfMemory eventually.
What do you expect to see?
The memory consumption should not increase on each refresh.
List the steps that must be taken to reproduce this issue

Configure an endpoint
Open the detail page of this endpoint
Hit refresh several times

Version
v5.7.0
Additional information
We are running the docker image in kubernetes.
Starting the Container hat 64Mb memory, we extended the memory up to 256Mb but the OOM occurs still after some refreshes.
The used memory during runtime without much browser access is around 50Mb

I gave it a shot, and while I do see the memory increasing a bit more than I'd like to see, it eventually goes back down.
I do acknowledge that I am curious about why the memory increases so much, but there doesn't seem to be a memory leak.

same here. if i basically continue refreshing the status page all the time, i hit the same error.
i hit OOM because kubernetes deployment kills the pod if it consumes more than the limit set.
it's pretty consistent though. you see an increase with the number of refreshes, especially when checks are happening at the same time.
memory consumption decreases over time (maybe some timeout or garbage collection?) but very slowly, in my opinion.

I am not sure if my case is same, but pod gets crashed after some time due to memory threshold reach maybe. This was not an issue earlier.


.svg file can't transform to base64

Describe the bug
useage:
<img
  alt="Vue logo"
  class="logo"
<EMAIL_ADDRESS>  width="125"
  height="125"
/>

logo.svg size is smaller than limit, but it can't transform to base64;
it build to 'dist/assets/logo.da9b9095.svg'
Reproduction
npm create vite@latest
System Info
node:16.13.1
vite:3.0.4

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Duplicate of #1204

Duplicate of #1204

thanks


Big(1).divideWithMathContext(Big(3), new MC(0, RoundingMode.CEILING)).numberValue()

Describe the bug
Big(1).divideWithMathContext(Big(3), new MC(0, RoundingMode.CEILING)).numberValue()
                throw new RangeError('Non-terminating decimal expansion; no exact representable decimal result.');

1/3 ceiling
i want the result is 1. What should I do?

Closing because I think my reply solves your problem, reopen the issue if it does not.


Don't change relative location header on manual redirect

What is the purpose of this pull request?

[x] Documentation update
[x] Bug fix
[ ] New feature
[ ] Other, please explain:

What changes did you make? (provide an overview)
This removes the code that changes the location header to an absolute URL in the case of a manual redirect request. While doing this is helpful in most cases, it is too destructive in cases where the original, actual location header is needed.  See the situation described in #1086.
This change also documents the variation from the spec
This change reduces the total lines of code, thus reducing the overall test percentage for test coverage.  To keep that check passing, I added some appropriate c8 ignores to unrelated code (see https://github.com/node-fetch/node-fetch/pull/1105/commits/52f864bf57b3a7ada44e0f5a9aebd6f8641f2cbc)
Which issue (if any) does this pull request address?
Fixes #1086
Is there anything you'd like reviewers to know?
No

@tekwiz I think that the linting issue has been fixed on the latest master, could you rebase please?


interface, logging, process management, and testing improvements, as well as bug fixes

This pull request addresses the following issues:

#47 implements more robust handling of server processes, including cleaning up on exit; the library also no longer assumes the running server process is a child of caller process (by eliminating checks to the Dolt.server attribute); also added richer set of exceptions for helping user see root cause of failed server interactions
#49 we now expose logging configurations inside dolt.core via LOG_LEVEL and HANDLERS variables, these are passed to a function for the logger factory that the library uses
#16 README is now current
#28 clone_repo is now clone, and it correctly implements the same behavior as the CLI for cloning a repo, that is the directory does not have to be explicitly passed
retire command line shims that no one seems to use (they were awkward and ill considered)

These changes look good, but they don't exactly solve the core problem. As we discussed this morning, we have to weigh the tradeoffs of using the sql-server to read/write the repo vs using the CLI. As I see it:

Both approaches require managing subprocesses, but the sql-server potentially requires managing multiple subprocesses and connections.
The sql-server allows us to use type inference and type checking features from the connector library. Using the CLI requires manually validating data and formatting it for write/import.
In general, the sql-server is a much less tested pathway than other CLI commands. Using it here would provide more coverage, but at the cost of complicating development and testing of DoltPy.
I'm leaning toward the CLI out of simplicity/reliability. I think replicating the connector features is much closer at hand than stabilizing the sql-server

@timsehn / @zachmu - do you guys have an opinion here?

I like using the SQL server in doltpy. The problems are not going away because we avoid the path. This needs to work.
Maybe we can provide both interfaces and a switch? This means if one is blocked, you can try the other and see if it works. This will give us better repros.
For avoiding the MySQL client, we could use dolt sql -r csv as the read interafce. On the write path we could just assemble the queries and pipe in via dolt sql < <file>?

The switch idea definitely seems like a good way to reproduce and debug problems as we keep working on this.


Ethereum

Here's my Ethereum Wallet Address:
0x42Be3A83ff7D6444056d1702F936d12c7A1609e0

Here's my Ethereum Wallet Address: 0x42Be3A83ff7D6444056d1702F936d12c7A1609e0

#31


fix: prevent FP against brackets in User-Agent header

The Facebook for Android user agent uses a bracket sequence in the user agent string which triggers FPs for rule 932131. The analysis revealed that this issue had already been fixed in 932130, the sibling rule with the same regular expression, but that the regex-assembly file for 932131 did not exist, which is why the regular expression was not updated.
Fixes #3483

If 932130.ra and 932131.ra need to stay in sync, can't we source one from the other one?
Other than that this looks good to me. Thanks.

Done.

Thank you.


[DR-1408] Error dialog callback

Hi team (and mainly @mvillaseco),
Before to fix the root cause of DR-1374, I wanted to fix a UX issue (in my humble opinion) with our error dialogs.
Error dialog were always reloading the page on closing, but sometime we could want to keep the page with the error, for example when there are useful information there, or sometimes it generates an cycling error.
So, I have remove callback execution when the user closes the dialog with x icon.
I have also applied some minor code refactoring.
I am still testing but I am interested in know your opinion before spend so much time.

Hi,
Yes i agree with you on this!

Closing in favor of #142


Issues with FwLite release for Linux

Here are the issues I found testing the fw-lite-local-web-app-linux.zip release:

zip file didn't preserve Unix permissions. I assume the LocalWebApp and FwLiteProjectSync files are supposed to have execute-bit permissions, but they don't. If the zip file was created with GitHub Actions, this is a known issue. The workaround is to create the .zip file with Unix zip tooling, then upload that file, rather than let GHA create the zip file.

I've been trying to get a fix for this into GHA for nearly six months now, with no success so far.

Clicking the "Open in FieldWorks" button appears to do nothing (I don't currently have FieldWorks installed), but it does pop up a "This project is open in FieldWorks. Please close it to reopen" modal at the bottom of the viewer. I'll install the latest FieldWorks for Linux release and try again.

Everything else seems to be working to far. If I find any more bugs I'll update this issue.

Correction: I do have FieldWorks for Linux installed. The latest release is a flatpak, which can be launched by running flatpak run org.sil.FieldWorks.


CUPS: PPD version is not compatible with Gutenprint

Issue description
After upgrading to ef026e02fcaa92323be875d8c7146b0b86e31880 I got the following error:

"The PPD version (5.2.11) is not compatible with Gutenprint 5.2.13. Please run `/nix/store/7762kpyhfkcgmr3q81v1bbyy0bjhym80-gutenprint-5.2.13/sbin/cups-genppdupdate' as administrator." - http://localhost:631/jobs/

This was preventing me from printing anything with manual interaction. Executing the following command fixed it:
# /nix/store/7762kpyhfkcgmr3q81v1bbyy0bjhym80-gutenprint-5.2.13/sbin/cups-genppdupdate -p /etc/cups/ppd
Updated /etc/cups/ppd/XYZ.ppd using gutenprint.5.2://hp-lj_4350/expert
Updated 1 PPD file, 1 skipped.

It seems like CUPS is keeping a copy of the old PPD file in /etc/cups/ppd and we don't automatically update these files (e.g. with the command above). Is this something we could fix or am I using CUPS in an unintended way (I'm not really familiar with CUPS...)?
I assume this error was caused by #27867, but that commit should be fine and don't cause any problems if we could somehow trigger an update of these PPD files.
Steps to reproduce
Unconfirmed:

Build 4f66c0cf21613301f0932b93ee885cab7f933fa7
Add a printer via http://localhost:631/admin and use a PPD file from Gutenprint v5.2.11 (in my case HP LaserJet 4350 - CUPS+Gutenprint v5.2.13)
Optional: Try sending a job to the printer.
Build 30f96ed2ba2bd78dd46d49177fad63097b41e60a
Try sending a job to the printer.

My config:
  services = {
    printing = {
      enable = true;
      drivers = with pkgs; [ gutenprint hplip ];
    };
  };

cc @edolstra  @abbradar @cillianderoiste (CUPS & Gutenprint contributors)

@primeos We are aware of this problem; because of this you need instead to set printing.gutenprint = true which adds autoupdating of PPD drivers.
Maybe we can somehow detect gutenprint  in drivers instead? For example, meta.gutenprint = true.

@abbradar Thanks, I wasn't aware of this and it seems like I missed the note in the description... (or I haven't looked at it since d37729f4b8978af3ed2776cf706884f116997fb5)
Imho automatically detecting a gutenprint derivation in the drivers list is a good idea (assuming there isn't any real reason why one would not want auto-updating of the PPD files).

Would something like #28423 be ok or is that a bad idea/approach?

Should be fixed by ea23f8bb079ef11c2d11859994d804c4e4940be5.


Alt-Tab doesn't show Terminal if minimized

Windows Terminal version
1.14.1962.0
Windows build number
Version 21H2 (OS Build 19044.1826)
Other Software
No response
Steps to reproduce
Open and minimize terminal
Press Alt+Tab to switch back to it
Previous app loses focus, but Terminal doesn't show and remains minimized
NB: When this happened, I tested it multiple times, and it happened every time. I then restored the window and Alt+Tab worked normally, then minimized it and it didn't. However, now it's working fine, so I'm not sure if it was some weird Windows issue (W10 continues, to this day, to be extremely broken WRT focus issues, constantly leaving windows I click on behind other windows and requiring me to click on the foreground, unfocused window then click back in the other one to force it to come to the front). Still, I'm reporting just in case it is actually an issue, and maybe somebody else can reproduce it or will have it happen to them to confirm.
Expected Behavior
No response
Actual Behavior
See above

I have same issue I tried change preview to stable version but it didn't change anything, still happening this bug.

Does this only happen when running elevated (as Administrator)? this looks similar to #13592 which was just filed. I can't repro this here on Windows 11, but maybe the focus/foreground work attached to #2988 regressed this in Windows 10 😨

Sorry, I forgot to mention this in my OP, but it was NOT running as admin at the time. And, if it matters, I had about four or five tabs open, with a mix of PS and cmd.

Alright, I'm deduping here, this will be the master tracking thread.

Doesn't seem limited to Windows 10 (at least, I can repro this on win11 in an elevated window sometimes)
Only occurs when minimizing to the taskbar, using the taskbar icon itself.
Doesn't seem limited to elevated windows (but seems more frequent)
If you're stuck in this state, you should be able to hover the icon in the taskbar till you get a preview of the window, and then you can right click -> "Restore" the window that way

There's also MSFT:40530963 tracking internally.

Only occurs when minimizing to the taskbar, using the taskbar icon itself.

Ah, and there it is. Kinda surprised I didn't think to try this, but yeah, if I minimize this way, it happens every time, with a non-admin window. Doesn't get stuck though, just have to click the taskbar button again.

@zadjii-msft I narrowed it down to 6b936d9a74a5d677387f3edc9a68e6b33d566606. Commenting out this line fixes the issue:
https://github.com/microsoft/terminal/blob/feabe41a08604345a1eeb4b38fd1c99b811f8ecd/src/cascadia/TerminalControl/ControlCore.cpp#L1929
I tried various ways to debug this further and fix the issue, but failed at it so far. 😥

I wonder if it's because that results in a xproc call (sending a message to OpenConsole.exe via a pipe) on the main thread. I wonder if user32 doesn't like us doing that... A resume_background right before that would be an interesting experiment

I have the same issue. when run as Administrator, is not restored if it was minimized

i can always reproduce if i double click terminal icon on taskbar when its minimized. (in administrator mode)

Notes (ongoing)

I was able to get an unelevated dev build of the Terminal into this state, on Windows 11.
In that state, we only get a few messages when alt-tabbing to the window:

0x7f
WM_WINDOWPOSCHANGING

the flags are: 0x13 = SWP_NOACTIVATE | SWP_NOMOVE | SWP_NOMOVE
hwndInsertAfter is not a special value.

WM_WINDOWPOSCHANGED
and that's it. There's nothing else, no ShowWindow or anything...

We don't even get into ControlCore::_focusChanged even remotely. Checks out, the window never gets shown so no duh it's not getting focused.
The current window styles are... 34CF0000 -> WS_VISIBLE | WS_MINIMIZE is set...
The ConPty HWND seems to be owned correctly, and WS_VISIBLE | WS_MINIMIZE are both set
....

In the 10 VM, the taskbar icon is not immediately highlighted when launching the Terminal. As if it doesn't know that the terminal is in the FG. That would explain the initial SwitchToWindow

But then after the "restore", the terminal is indicated as being active. Hmm.

#13158
    // BODGY
    //
    // GH#13158 - At least temporarily, only allow the PTY to HIDE the terminal
    // window. There seem to be many issues with this so far, and the quickest
    // route to mitigate them seems to be limiting the interaction here to
    // allowing ConPTY to minimize the terminal only. This will still allow
    // applications to hide the Terminal via GetConsoleWindow(), but should
    // broadly prevent any other impact of this feature.
    //
    // Should we need to restore this functionality in the future, we should
    // only do so with great caution.

Oh hey look, that came back to bite 😠

Same problem here, windows 10 (20H2), elevated terminal.
This afternoon, every time after I built a project in vs2019, the terminal suddenly disappear.

@zadjii-msft  If you're stuck in this state, you should be able to hover the icon in the taskbar till you get a preview of the window, and then you can right click -> "Restore" the window that way

this doesnt work for me on windows 11

@satyapramodh Your screenshot only shows you hovering over the thumbnail. Did you actually right-click on the thumbnail and select restore from the menu that pops up? Granted, I'm on W10, but that does work for me and others, and it looks like zadjii is on Win11, like you, so it seems it should work for you as well.

Ok - restoring the window does actually work, but is quite annoying. Shouldn't have updated the terminal at all, the old version was working quite fine. 😅

Just to chime in that it happens on 22H2, too, though making it happen is tricky. It just seems to happen when you don't want it to.
I'll have to try the "right-click on the thumbnail and select restore" trick, which may be the only thing I haven't tried to get access to it once it's happened.

@zadjii-msft said:
If you're stuck in this state, you should be able to hover the icon in the taskbar till you get a preview of the window, and then you can right click -> "Restore" the window that way
No need to wait for preview of the window, for this "hover thing", simply shift-rightclick on the Terminal taskbar button and
click restore. A little bit simpler. :)

Those "handy" links are 404. They're missing the ".0" on the end.

Those "handy" links are 404. They're missing the ".0" on the end.

Yup, both links are very "handy". One misses .0 and another points to the store where they still keep older buggy version (200). 😁

The Store just got 1.14.2282.0.

The Store just got 1.14.2282.0.

thanks for letting us know

@zadjii-msft said: If you're stuck in this state, you should be able to hover the icon in the taskbar till you get a preview of the window, and then you can right click -> "Restore" the window that way
No need to wait for preview of the window, for this "hover thing", simply shift-rightclick on the Terminal taskbar button and click restore. A little bit simpler. :)

This is not an option on Windows 10 :(

@zadjii-msft said: If you're stuck in this state, you should be able to hover the icon in the taskbar till you get a preview of the window, and then you can right click -> "Restore" the window that way
No need to wait for preview of the window, for this "hover thing", simply shift-rightclick on the Terminal taskbar button and click restore. A little bit simpler. :)

This is not an option on Windows 10 :(

it works for Win10, also see the same workaround here  https://github.com/microsoft/terminal/issues/13592#issue-1316837614
But now it seems to  work as it should at least for me.


Convert environment Links to json in Resource.to_json()

There appears to be a bug where Entry instances aren't converting the 'environment' sys field Link instance to json when calling to_json().
Hopefully this solution makes sense in general, it fixed the particular issue I was having.
Demonstration here.

Hi @MikeFE, thank you for the contribution. Can you please add a test case for proposed changes?
Cheers


API Tokens

Users should be able to auth with a bearer token.
Workflow:

Call user/token/get with correct credentials, get a token returned.
Use token (which should have a long expire time).
Revoke token (not necessary) with user/token/revoke

Then the API can just check for either a bearer token in the header or the user/pass.
Explicitly provided user/pass overrides a token.
Relies on #1.

Tokens now exist as part of the course/server user split.


Awesome!

Thanks. This repo is so cool! I put it into my project and it runs well.
One thing, it will be more perfect if it can support fullscreen view mode.

Nice work
works like a charme much appreciated
have a question can you make the white background on the pages thumbnail transparent..??

Thanks. This repo is so cool! I put it into my project and it runs well.
One thing, it will be more perfect if it can support fullscreen view mode.

Thanks, I'm happy that this repo helps you somehow. If you check the code, you will see that the PDFView is actually a fullscreen view. I did enable "autoScales" property which will scale your pdf file properly in the view. So full-screen mode also depends on your pdf file too. So it seems better for me, if I scale the pdf file to fit full screen all the time, it may break for some content. Hope you understand what I'm trying to say :)

Nice work
works like a charme much appreciated
have a question can you make the white background on the pages thumbnail transparent..??

Thanks, I'm happy that this repo helps you somehow.
I did update the repo for your convenience, please check version 1.0.2 for this update.

nice thanks much appreciated

You're right. Thanks.


streamlink-twitch-gui-bin: 1.11.0 -> 2.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/streamlink/streamlink-twitch-gui/releases.
meta.description for streamlink-twitch-gui-bin is: "Twitch.tv browser for Streamlink"
meta.homepage for streamlink-twitch-gui-bin is: "https://streamlink.github.io/streamlink-twitch-gui/"
meta.changelog for streamlink-twitch-gui-bin is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/4vbclrrblgzxsgyqxim1lm85sar0divp-streamlink-twitch-gui-bin-2.0.0/bin/streamlink-twitch-gui had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/4vbclrrblgzxsgyqxim1lm85sar0divp-streamlink-twitch-gui-bin-2.0.0/bin/.streamlink-twitch-gui-wrapped had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 2.0.0 with grep in /nix/store/4vbclrrblgzxsgyqxim1lm85sar0divp-streamlink-twitch-gui-bin-2.0.0
found 2.0.0 in filename of file in /nix/store/4vbclrrblgzxsgyqxim1lm85sar0divp-streamlink-twitch-gui-bin-2.0.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
streamlink-twitch-gui-bin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4vbclrrblgzxsgyqxim1lm85sar0divp-streamlink-twitch-gui-bin-2.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A streamlink-twitch-gui-bin https://github.com/r-ryantm/nixpkgs/archive/5111b7b4682a9accb77a44c5917352707310a5c2.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4vbclrrblgzxsgyqxim1lm85sar0divp-streamlink-twitch-gui-bin-2.0.0
ls -la /nix/store/4vbclrrblgzxsgyqxim1lm85sar0divp-streamlink-twitch-gui-bin-2.0.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

streamlink-twitch-gui-bin

Maintainer pings
cc @rileyinman for testing.

I've learned my lesson, I'm leaving this one open so the bot won't make a new one. Please merge #162250 instead.

I merged https://github.com/NixOS/nixpkgs/pull/162250. I think maybe you will need an updateScript for r-ryantm to handle this package correctly.

Good point, the addition of gtk3-x11 had to be done manually but I'll see about adding a script to update both hashes. Thanks!


soca travis CI broken (in ufo)

Description
https://travis-ci.com/github/JCSDA-internal/soca/builds/216670131

-- Configuring done
CMake Error at /usr/local/share/ecbuild/cmake/ecbuild_add_library.cmake:294 (add_library):
Target "ufo" links to target "gsl::gsl-lite" but the target was not found.
Perhaps a find_package() call is missing for an IMPORTED target, or an
ALIAS target is missing?
Call Stack (most recent call first):
/usr/local/share/ecbuild/cmake/ecbuild_add_library.cmake:639 (ecbuild_add_library_impl)
src/ufo/CMakeLists.txt:125 (ecbuild_add_library)

This is likely because we started using ioda-engines in ufo. There's probably something missing in ufo CMakeLists; @travissluka, @rhoneyager, @srherbener  could you take a look?

https://github.com/JCSDA-internal/ufo/pull/705 broke it when ioda_engines was added to UFO_LIBS_DEP. It only breaks when ufo/ioda are not compiled within the same bundle.
I'm guessing that adding find_package(gsl-lite REQUIRED HINTS $ENV{gsl_lite_DIR}) to ufo would work, but perhaps someone with more cmake knowledge can tell me why the gsl library info is not found by ufo from the ioda install.

I've created a branch in ufo with @travissluka's proposed fix, and am running Travis CI in soca to test.
I agree with Travis, would be nice to know why the gsl library info is not found by ufo from the ioda install.

Now, with this ufo branch, it fails with a different error:

-- Configuring done
CMake Error at /usr/local/share/ecbuild/cmake/ecbuild_add_library.cmake:294 (add_library):
Target "ufo" links to target "MPI::MPI_C" but the target was not found.
Perhaps a find_package() call is missing for an IMPORTED target, or an
ALIAS target is missing?
Call Stack (most recent call first):
/usr/local/share/ecbuild/cmake/ecbuild_add_library.cmake:639 (ecbuild_add_library_impl)
src/ufo/CMakeLists.txt:125 (ecbuild_add_library)
CMake Error at /usr/local/share/ecbuild/cmake/ecbuild_add_library.cmake:294 (add_library):
Target "ufo" links to target "MPI::MPI_C" but the target was not found.
Perhaps a find_package() call is missing for an IMPORTED target, or an
ALIAS target is missing?
Call Stack (most recent call first):
/usr/local/share/ecbuild/cmake/ecbuild_add_library.cmake:639 (ecbuild_add_library_impl)
src/ufo/CMakeLists.txt:125 (ecbuild_add_library)

I have no idea how to fix it, unfortunately.


[R] Data loss with leading zero in partition column

Describe the bug, including details regarding any error messages, version, and platform.
Arrow does not correctly interpret some partition values as character columns, resulting in data loss.
library(arrow)
library(dplyr)

# character col with leading zero
mtcars2 <- mutate(mtcars, test = paste0(0, mtcars$carb)) |> as_tibble()

tmp <- tempfile()

write_dataset(mtcars2, tmp, partitioning = "test")

mtcars3 <- open_dataset(tmp) |> collect()

unique(mtcars2$test)
#> [1] "04" "01" "02" "03" "06" "08"
unique(mtcars3$test)
#> [1] 1 2 3 4 6 8

Created on 2024-01-17 with reprex v2.0.2
Component(s)
R

This is great stuff!  We should add this to the docs as it's not immediately obvious (i.e. I had no idea this was the case)


Fix customdomain for kista

Customdomain should show correct cname for kista zone deployments

Will try to sync this for the release of VM v2 in Kista


feat(deps): add fingerprintjs

:tada: This PR is included in version 3.0.0-alpha.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Please help

https://github.com/Nyr/openvpn-install/issues/258
Is my ISP that is trying to defraud me.
Do you think is okay to spend 5 € (5,37 USD) for 1 Gb of traffic? With 1 Gb of traffic I barely see one 1080p video on youtube... That's a thief. If you don't believe you can check the page of this theyr offert: https://www.tim.it/offerte/mobile/voce-e-internet/opzione-1gb-4g
I'm willing to pay for a flat LTE profile but they doesn't offer any flat profile. they're maximum traffic offer is 4 Gb for 20 €. That's a thief too.
I really need to use LTE connection... please it would be a dream for me...it's not my fault if they doesn't offer any decent plan for...
 But I cannot use my home ADSL connection anymore, even if it's flat the maximum speed is 1,5 Mbps and go off every night and time start raining. Please help me... It's a nightmare for me to live in this situation... Any provider is giving me a decent service... please understand my situation

In case this needed clarification: I have no sympathy for your ISP, but I don't appreciate your spam in this issue tracker either.
Both of your accounts had been blocked for this reason (spam after you were told to go elsewhere and your insults in Italian).


ignore appengine package

similar to what was done in https://github.com/deis/pkg/commit/153c471dcd407dc66f96cbab3ea076eea271aee7

The ignores were added in #128 (specifically here). Closing.


Weird deno version requirement

Why Lume has Deno version requirement now? It wasn't the case before.
What steps will reproduce the bug?
❯ deno run -A https://lume.land/init.ts

Lume needs Deno v1.46.0 or greater
Your current Deno version is 1.45.5
Run deno upgrade and try again

It always had version requirement but it was far below the latest version.
With Lume 2.3 I had to update to the latest Deno version due this bug https://github.com/denoland/deno/issues/25008 that was fixed recently.

Lume going multi-threaded?

Lume going multi-threaded?

Not exactly. The build process is run in a worker, so it can be reload after changing the _config.ts file. More info.

You have solved the reloading problem. Nice!


Allow favoriting certain locales

From the support forum:

I really like your extension - thanks! One suggestion: a previous language switcher extension I used had a feature where you could customize the list of languages that you saw. I don't really need the ability to fully customize the list, but it would be really useful to have the list of favorites or recently used languages at the top of the list, that way it would be even easier to switch between them. We support 7 languages currently in our product, so being able to flip between those 7 (versus searching each time) would be great!

We may be having our QA team toggle back and forth between languages to test our site.  A favorites list would be amazingly helpful for us.

I'd love this too! I think most people will only be switching between 2 (or maybe 3) languages and it's kind of a pain if the language you most often want to switch to isn't on the list.

One of the best locale switch extension I used 👍 , but I'm really missing this feature. Any change this will be added @athyuttamre  ?


Adds immutable stack implementation

Adds immutable stack implementation

Codecov Report
Attention: Patch coverage is 87.87879% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 90.66%. Comparing base (d4196d4) to head (41c593c).

Files
Patch %
Lines

src/data_structures/stack/core.clj
87.87%
0 Missing and 4 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   91.01%   90.66%   -0.35%
==========================================
  Files          18       19       +1
  Lines         267      300      +33
  Branches       16       20       +4
==========================================
+ Hits          243      272      +29
  Misses          8        8
- Partials       16       20       +4

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


CATCH_REGISTER_LISTENER fails to compile on vs2013

Description
Adding a event listener with CATCH_REGISTER_LISTENER results in:
Severity	Code	Description	Project	File	Line	Suppression State
Error	C2440	'type cast' : cannot convert from 'const Catch::ReporterConfig' to 'const MyCatchListener'	testsuite	catch.hpp	9638
Error	C2512	'MyCatchListener' : no appropriate default constructor available	testsuite	catch.hpp	9638

Steps to reproduce
class MyCatchListener : Catch::TestEventListenerBase
{
public:
    using Catch::TestEventListenerBase::TestEventListenerBase;

    void testCaseStarting(const Catch::TestCaseInfo& _testInfo) override
    {
    }

    void sectionEnded(const Catch::SectionStats& stats) override
    {
    }
};

CATCH_REGISTER_LISTENER(MyCatchListener);

Extra information
Tested on VS2013, with the latest stable catch release.
The same code compiles fine on Linux and OSX.
Please advise!

I don't think VS2013 has inheriting constructors, does it?

You're right, thank you.


Bump version to v0.9.2

Preparing for 0.9.2 (no ETA yet but it's "around the corner")

[x] Bump versions _config.yml: currentduckdbversion, currentjavaversion, currentsnapshotversion, nextjavaversion (if applicable)
[x] Add new version entry in _data/versions.csv
[x] Create new JSON file based on the current version: _docs/menu_docs_*.json
[x] Update the command line clients version number in docs/api/cli.md and docs/sql/introduction.md:

[x] Update the version you just archived
[ ] Update the dev version

[x] If the storage version changed (see https://github.com/duckdb/duckdb/commits/main/src/storage/storage_info.cpp), add the new version(s) to the storage site: https://duckdb.org/internals/storage
[x] Create archive pages using the scripts/archive_docs.py script.
[x] Generate new PDF version and link it
[ ] Test the site locally using manual testing
[ ] Run LinkChecker on the site locally
[ ] Test that all download links pointing to the new version work

This should merge once the OSX, Python, etc. releases are available for 0.9.2.


TernDef can't transverse files from nodejs project

Hopefully I'm missing something, but seems that TernDef can't figure out where a definition is when working within https://github.com/nodejs/node itself. Probably because all the require's are the same as that of core modules. e.g. require('buffer'). (note: I'm using tern_for_vim)
Is there any way to configure tern to look for core modules in, say, the lib/ folder of the project?

You could add a configuration option to the node plugin that can be used to disable the built-in packages, and enable that in the .tern-project for the node source itself. That's a rather specialized case, but if you submit a PR, I'll merge it.


Flickr likes contain the flickr link as u-like-of instead of the site link

My incoming backfeed from Flickr got stuck at webmention.io with the message: The source document does not have a link to the target URL
Example:

https://brid.gy/log?start_time=1541240937&key=aglzfmJyaWQtZ3lyRAsSCFJlc3BvbnNlIjZ0YWc6ZmxpY2tyLmNvbSwyMDEzOjQxNzMwODkyNzY0X2xpa2VkX2J5XzEzNjY5MDc2MUBOMDMM
https://brid-gy.appspot.com/like/flickr/36003160@N08/41730892764/136690761@N03
https://webmention.io/petermolnar.net/webmention/RIjpnlOTQ6FNijyt0ea8

The brid.gy webmention has the u-like-of of the flickr link.
I believe this is a bug, but it may as well be my site lacking syndication URL to flickr.

aha. did you change your flickr username from petermolnarnet to petermolnareu recently? bridgy uses username to determine flickr identity, and checks that each post it backfeeds was created by the bridgy user. here, it's seeing petermolnarnet vs petermolnareu, so it doesn't think you created this flickr post, so it doesn't use your synd link.
if you log into bridgy with flickr again, it should update your username.

@petermolnar looks like this got resolved. mind if i close?


feat: keywords

This appendix will list the reserved keywords in Cairo, and what they're used for.
For example, the return keyword is used to return from a function.
The expected output is similar to the Rust book's keywords appendix page.
You can refer to this page of the cairo documentation Your objective here is to provide context for these keywords and explain what they're used for.
Estimated time: 1 hour

want to try this one 👋

yo think you can close this issue :)


minor: support debian 12

Failed to find required executable \"rsync\" in paths

Failed to find required executable \"rsync\" in paths

Yep, the debian 12 container doesn't come with python so I configured molecule to prepare the image pre_build_image: false which should install python, etc.
But to do that the molecule docker driver uses the synchronize ansible module which requires rsync, where IMHO the copy module would be sufficient.
It's a catch 22 situation and I've actually thought about doing a PR to address this in the docker driver.


A JSONObject text must begin with '{'

Hi,
the JSON to XML conversion did not work for me. The following error appears:
System-ID: C:\Users\Stefan.Eike\DITA\de-DE\monitoring\strings\strings.json
Fehlerlevel: error
Beschreibung: A JSONObject text must begin with '{'

Anonymized file: strings.zip
Note: The anonymized file has duplicate key errors, which happens because of the anonymization. This does not happen in the original file. oXygen XML does not show any structural issues there (green light).

I made changes to your samples so that the keys are not duplicate (because the duplicate keys were hiding all other problems. I'm attaching the modified sample.
strings_fixed.zip
Unfortunately I cannot reproduce the problem using the latest plugin and my fixed sample JSON.

Hi @raducoravu, I can reproduce the error with your fixed file.
Software used:

 XML Editor 19.1, build<PHONE_NUMBER>
Batch Converter 1.1.1

@xephon2 Sorry, my bad, initially I tested our default JSON to XML conversion actions bundled with Oxygen instead of the plugin's ones.
We'll look into this.

The new release (1.1.2) contains this fix.
Thanks you for this report.


WARNING:tensorflow:A checkpoint was restored (e.g. tf.train.Checkpoint.restore or tf.keras.Model.load_weights) but not all checkpointed values were used. See above for specific issues.

> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._box_predictor._base_tower_layers_for_heads.class_predictions_with_background.4.10.gamma
> W0925 09:38:56.472692  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._box_predictor._base_tower_layers_for_heads.class_predictions_with_background.4.10.gamma
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._box_predictor._base_tower_layers_for_heads.class_predictions_with_background.4.10.beta
> W0925 09:38:56.473077  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._box_predictor._base_tower_layers_for_heads.class_predictions_with_background.4.10.beta
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.residual_blocks.0.0.kernel
> W0925 09:38:56.473470  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.residual_blocks.0.0.kernel
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.residual_blocks.0.0.bias
> W0925 09:38:56.473858  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.residual_blocks.0.0.bias
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.residual_blocks.1.0.kernel
> W0925 09:38:56.474293  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.residual_blocks.1.0.kernel
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.residual_blocks.1.0.bias
> W0925 09:38:56.474685  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.residual_blocks.1.0.bias
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.0.0.kernel
> W0925 09:38:56.475035  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.0.0.kernel
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.0.1.gamma
> W0925 09:38:56.475502  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.0.1.gamma
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.0.1.beta
> W0925 09:38:56.475951  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.0.1.beta
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.1.0.kernel
> W0925 09:38:56.476325  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.1.0.kernel
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.1.1.gamma
> W0925 09:38:56.476714  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.1.1.gamma
> WARNING:tensorflow:Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.1.1.beta
> W0925 09:38:56.477157  6844 util.py:150] Unresolved object in checkpoint: (root).optimizer's state 'momentum' for (root).model._feature_extractor._fpn_features_generator.conv_layers.1.1.beta
> WARNING:tensorflow:A checkpoint was restored (e.g. tf.train.Checkpoint.restore or tf.keras.Model.load_weights) but not all checkpointed values were used. See above for specific issues. Use expect_partial() on the load status object, e.g. tf.train.Checkpoint.restore(...).expect_partial(), to silence these warnings, or use assert_consumed() to make the check explicit. See https://www.tensorflow.org/guide/checkpoint#loading_mechanics for details.
> W0925 09:38:56.477508  6844 util.py:158] A checkpoint was restored (e.g. tf.train.Checkpoint.restore or tf.keras.Model.load_weights) but not all checkpointed values were used. See above for specific issues. Use expect_partial() on the load status object, e.g. tf.train.Checkpoint.restore(...).expect_partial(), to silence these warnings, or use assert_consumed() to make the check explicit. See https://www.tensorflow.org/guide/checkpoint#loading_mechanics for details.
>

i am getting this error while training steps are 25000. tensorflow v2.3 python 3.7.3

@muhosevo
Please refer to this link and let us know.

just tf.keras.Model.load_weights(path).expect_partial()
calls expect_partial() on the loaded status, since SavedModels saved from Keras often generates extra keys in the checkpoint. Otherwise, the program prints a lot of warnings about unused keys at exit time.


Client app gets 502 error at specific prompt

Describe the bug

We are getting a 502 error from DirectLine when we call EndDialog in the middle of a dialog that has "autoEndDialog": true
We have a composer bot deployed in Azure and a client app using the Direct Line web socket to send messages back and forth to the bot.
The client app was getting a 502 error when the conversation reached a specific step in one of our child dialogs, "GetPhoneNumber."
Steps:

GetPhoneNumber looks for the user's phone number in the conversation state
If it's in the state, it asks "Can I text you from the number you're calling from?"
If the user says "no", the dialog continues on to prompt for their phone number
If the user says "yes", EndDialog is called; however, when the client app sends the user's "yes" activity to the bot, it gets the 502 error

After reconfiguring the dialog to not use EndDialog, we stopped seeing the 502 errors. EDIT: We tried reconfiguring the response to the prompt, but we still get the errors.
BEFORE

AFTER

Version

v2.1.0
Browser

[ ] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[X] Edge

OS

[ ] macOS
[X] Windows
[ ] Ubuntu

To Reproduce
Here are the steps in our composer bot. Feel free to reach out to me for more in-depth walkthrough.

Add LUIS intent triggered dialog, i.e. "RescheduleAppointment"
Inside RescheduleAppointment, call a child dialog to gather user inputs, i.e. "GetAppointmentDetails"
Inside GetAppointmentDetails, call another child dialog to gather specific input, i.e. "GetEmail"
Inside GetEmail, when user input is not valid, call another child dialog, "GetSmsInput"
Inside GetSmsInput, call another child dialog, "GetPhoneNumber"
If user's phone number is in the state, and they confirm it's correct, call EndDialog (see above screenshots)

Here's an example from App Insights logs:

@lshade does this reproduce if you try to communicate with the bot running locally on your machine? If it does, maybe you could place breakpoints in your locally running bot and see what is going on or if an exception is being thrown that is causing the timeout.


[BUG] Bubble colums not functioning after new FluidLogged API update

It could be using old classes atm, so update to the newest one needs to be done, or some sort of communication to fluidlog the bubble columns by default in config.
I will double that on their side as well.

This was caused by updating my MCP mappings, not Fluidlogged, but thanks! Fixed in 1.12.1.


Deploy Checklist: New Expensify 2022-09-23

Release Version: 1.2.6-0
Compare Changes: https://github.com/Expensify/App/compare/production...staging
This release contains changes from the following pull requests:

[x] https://github.com/Expensify/App/pull/10491
[x] https://github.com/Expensify/App/pull/10860
[x] https://github.com/Expensify/App/pull/10984
[x] https://github.com/Expensify/App/pull/11013
[x] https://github.com/Expensify/App/pull/11110
[x] https://github.com/Expensify/App/pull/11140
[x] https://github.com/Expensify/App/pull/11155
[x] https://github.com/Expensify/App/pull/11217
[ ] https://github.com/Expensify/App/pull/11219
[x] https://github.com/Expensify/App/pull/11240
[x] https://github.com/Expensify/App/pull/11244

Deploy Blockers:

[ ] https://github.com/Expensify/App/issues/11264

Deployer verifications:

[ ] I checked the App Timing Dashboard and verified this release does not cause a noticeable performance regression.
[ ] I checked Firebase Crashlytics and verified that this release does not introduce any new crashes.

cc @Expensify/applauseleads

:rocket: All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.2.6-0 :rocket:

@luacmartins @roryabraham
Regression is completed
PR https://github.com/Expensify/App/pull/10491 is blocked https://github.com/Expensify/App/issues/11281. We are stuck with this PR and need some additional eyes here
PR https://github.com/Expensify/App/pull/11155 is failing https://github.com/Expensify/App/issues/11264
PR https://github.com/Expensify/App/pull/11219 is failing https://github.com/Expensify/App/issues/11237 blocker was removed. are we ok to check off?

I spent some time looking into it, and there were some problems with https://github.com/Expensify/App/pull/10491, but those problems either have been reported or were not caused by that PR. None of them are deploy blockers, so I'm going to check it off

https://github.com/Expensify/App/issues/11264 has been closed as expected behavior. Checking off https://github.com/Expensify/App/pull/11155

Also checking off https://github.com/Expensify/App/pull/11219

I am seeing a substantial spike in homepage initial render timing on iOS and Android:

But I believe that can be attributed to https://github.com/Expensify/App/pull/11206, which fixed a problem where TTI was captured noticeably before the app was actually interactable.

I am also seeing a fresh crash on 1.2.6-0: https://console.firebase.google.com/u/0/project/expensify-chat/crashlytics/app/android:com.expensify.chat/issues/391524c846c415da615800d7a0fe1346?time=last-seven-days&sessionEventKey=632F48100082000146271F7ACA92A9D9_1725626754260263079
Investigating...

This looks like a low-level issue, likely coming from Reanimated. Not going to block on it for now since it's only happened once.

:shipit:


Move state_dict serialization to Haliax

Currently Levanter has a bunch of machinery to support serializing to/from state_dicts, as well as writing to safetensors. I think it would be move that functionality to Haliax.
While doing it, I would like to revamp a few things:

[ ] remove/deprecate _state_dict_key_map, replace with a field property in eqx.field
[ ] add persistent=false as well.
[ ] ???

Also think this would be nice, b/c, e.g., in my nanoGPT implementation I have to install all of levanter just to use its serialization features. Seems like this may be a large-ish task, are there any areas I could potentially help with?

I think so!
I think we could start by:

[ ] just copying the main file over (though I'm about to finish a nontrivial change, so don't do this right away)
[ ] updating references in levanter
[ ] state_dict_key=foo
[ ] add persistent=False for buffers
[ ] port over the hf_checkpoint logic for loading/reading state dicts?

I don't think it's a ton of work, but it's not trivial either

@rohan-mehta-1024 i merged the change I was making into Levanter, so if you're excited about this, happy to let you take it! Happy to discuss more if you want

Sorry for the delay on this! I've just been a busy with starting school. I plan to work on it over the weekend.

no problem at all. Happy you're looking at it at all!

Sorry if this is obvious, but what did you mean by state_dict_key=foo? Also, just to confirm, when you say porting over the hf_checkpoint logic, you don't mean the whole file, right? Because some of the code in there seems to be specific to the case of language models.

sorry that was very unclear. By that I mean, you could write something like:
class MyModule(eqx.Module):
   weights: hnn.Linear = eqx.field(state_dict_key="foo")

and have the mapping work.
It's not a necessary step but I think it's a nicer api.
And re: the hf_checkpoints stuff, right. I was thinking of basically just a function to do this https://github.com/stanford-crfm/levanter/blob/16112cc003680e79e15fc7c62b63f917679e7e32/src/levanter/compat/hf_checkpoints.py#L403-L407 + the safetensors variant

Ok, so for now should I just go ahead with a "soft" version of hax.field and then we can modify this approach as we see fit later?

Not sure what that means? But my guess is probably :-)
On Wed, Dec 20, 2023 at 10:49 AM rohan-mehta-1024 @.***>
wrote:

Ok, so for now should I just go ahead with a "soft" version of hax.field
and then we can modify this approach as we see fit later?
—
Reply to this email directly, view it on GitHub
https://github.com/stanford-crfm/haliax/issues/35#issuecomment-1864975575,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAACLIIRL4G7E524ZYZPNRDYKMXKVAVCNFSM6AAAAAA4MEPMOKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRUHE3TKNJXGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm confused. How does it know what layer it is?
On Thu, Dec 21, 2023 at 8:30 AM rohan-mehta-1024 @.***>
wrote:

Yeah, that was poor wording, sorry (basically I think I'm just going to go
the route of explicitly using metadata and then once everything else has
been ported over successfully, I guess we can decide from there, since it
should just be a pretty minor stylistic change)! I was also wondering if we
should make the following stylistic change. When flattening/unflattening
layers, the type signature is (prefix, statedict: StateDict, layer:
hnn.Linear, out_dims_first_in_dict: Optional[bool]). Since we pass the
layer itself in, the function can use the state_dict_key_map to lookup any
necessary prefixes. So, if you have the following:
def _state_dict_key_map(self):
return {'attn' : 'c_attn', 'proj' : 'c_proj'}
Then you only need to do:
unflatten_linear_layers(prefix, state_dict, self.attn, None)
Instead of:
unflatten_linear_layers(apply_prefix(prefix, 'c_attn'), state_dict,
self.attn, None)
Since the function can infer this. However, for stacking/unstacking
layers, the function does not explicitly take in the block to be
stacked/unstacked. So even if you have:
def _state_dict_key_map(self):
return {
"blocks"              : "h",
"tok_embedding_table" : "wte",
"pos_embedding_table" : "wpe"
}
You still have do:
stacked_params = stack_state_dict(state_dict, prefix=apply_prefix(prefix,
"h"))
Would it make sense to rewrite the function to have it take in the block
to stack/unstack, so that you could do this instead:
stacked_params = stack_state_dict(state_dict, self.blocks, prefix=prefix)
It would probably complicate the underlying function, but it would also
allow for automatically inferring prefixes from state_dict_key_map. Also,
it seems like it would be a little more explicit, since you don't have to
reason through the prefixes to find out what is being stacked/unstacked. I
was just wondering if this line of thought makes sense to you, or if there
are some other reasons I haven't identified why it would be better to keep
things as they are?
—
Reply to this email directly, view it on GitHub
https://github.com/stanford-crfm/haliax/issues/35#issuecomment-1866609313,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAACLIMSZN2AODVAJCJYWYLYKRP4BAVCNFSM6AAAAAA4MEPMOKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRWGYYDSMZRGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Maybe I'm misunderstanding, but the following code, e.g., in unflatten_linear_layers:
tree_prefixes = leaf_key_paths(
        layer, prefix, is_leaf=lambda x: isinstance(x, hnn.Linear), use_state_dict_keys=True
  )

has a use_state_dict_keys argument which allows it to access the state_dict_key_map. And here is the relevant code from leaf_key_paths where it does access this mapping:
    elif isinstance(pytree, eqx.Module):
        names = []
        rec_values = []
        for field in fields(pytree):
            if field.metadata.get("static", False):
                continue
            field_name = field.name
            field = getattr(pytree, field_name)
            names.append(field_name)

            if use_state_dict_keys and hasattr(pytree, "_state_dict_key_map"):
                field_name = pytree._state_dict_key_map().get(field_name, field_name)

            rec_value = rec(field, field_name)
            rec_values.append(rec_value)
        return eqx.tree_at(lambda m: [getattr(m, name) for name in names], pytree, rec_values)

So I don't understand why you would explicitly have to do:  d.update(unflatten_linear_layers(apply_prefix(prefix, "c_attn"), state_dict, self.c_attn, None)). Why do we need to do the  apply_prefix(prefix, "c_attn") ourselves? Won't this code figure out that the attn module has to be renamed "c_attn", because it goes through all the fields of our given layer and updates the names of all of those which are also keys in our mapping? My understanding of how prefixes work is that they are automatically and recursively added on to more and more nested layers, so I'm a little confused why we sometimes have to explicitly add them on ourselves? Hopefully this makes a bit more sense!

oh well, the convention in all these methods is that they receive the path they're supposed to use. self.c_attn doesn't know that it's supposed to be named self.c_attn. It would need information from the parent (self) to know that.

Oh ok, I see, that makes sense now (sorry for the confusion). I think I've basically made all the necessary changes at this point (the only other thing I'm wondering about is what you meant by persistent_buffer = false, because anything to do with buffers seems to come from other parts of the hf_checkpoints file?). Also, I was curious what what you think a good idea is for testing? The Levanter tests for serialization/state_dict loading use full-blown models and are pretty extensive, but I was thinking of maybe just creating a small dummy model in both Haliax and PyTorch and then making sure it was possible to load in the PyTorch weights into the Haliax model and vice versa. Do you think this is a robust enough test?

Oh wow, thank you!
I think buffers thing can be a separate PR/future issue. I agree lighter weight tests are better here and I agree round-tripping to/from torch (and Haliax itself) is probably the best route.
Maybe like, a few tests that test only-Haliax roundtrip functionality (dicts, renames, reorders, whatever), and then a few tests that only fire with torch installed that make sure roundtrips work there?


Adds allocations to the payload for controller, view, and sql logs

Description of changes
Since Rails 6, allocations have been logged during ActionController request logging, ActionView rendering, and ActiveRecord querying. This change adds those metrics to the payload (similar to db_runtime and view_runtime).
A concern: allocations is on event rather than event.payload, so this doesn't quite match up like duration does, so we would need to make changes to SemanticLogger to move allocations up to the log struct.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Great addition, Thank you :tada:

Great addition, Thank you :tada:


Will GigaNVS dataset open Source？

Hi @lhc991025,
The dataset will be released in conjunction with the upcoming GigaVision challenge.


Gateway API: different route behavior between different GatewayClass

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
I am using Gateway API in Istio and the configuration is below. When using Istio GatewayClass, the traffic from gateway chooses the first rule, while it chooses the second rule under Contour GatewayClass. This case is deployed under Istio proxies, so which route behavior is more reasonable? What about the reason?
apiVersion: gateway.networking.k8s.io/v1
kind: HTTPRoute
metadata:
  name: hr
spec:
  parentRefs:
  - group: gateway.networking.k8s.io
    name: gateway
    kind: Gateway
  hostnames:
  - www.example.com
  rules:
  - matches:
    - path:
        type: Exact
        value: /productpage
    backendRefs:
    - name: ratings
      port: 8080
  - matches:
    - path:
        type: Exact
        value: /productpage
    backendRefs:
    - name: productpage
      port: 9080
---
apiVersion: gateway.networking.k8s.io/v1
kind: Gateway
metadata:
  name: gateway
spec:
  gatewayClassName: contour # or istio
  listeners:
  - name: port-1
    port: 80
    protocol: HTTP
    hostname: www.example.com
    allowedRoutes:
      namespaces:
        from: All

Version
client version: 1.21.0
control plane version: 1.21.0
data plane version: 1.21.0 (4 proxies)

Client Version: v1.28.2
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.27.3

Additional Information
No response
Affected product area

[ ] Ambient
[ ] Docs
[ ] Dual Stack
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Control Plane Revisions

I think the Istio implementation (first rule) is correct according to the spec: https://gateway-api.sigs.k8s.io/reference/spec/#gateway.networking.k8s.io%2Fv1.HTTPRouteRule, where it says:
If ties still exist within an HTTPRoute, matching precedence MUST be granted to the FIRST matching rule (in list order) with a match meeting the above criteria.

Closing as this appears WAI


WIP: Header component

Starting to think about this for digital workplace, so making a stab at a component structure for the header. Specifically wanted to make it easy to swap out parts of the header so we can handle use cases like https://github.com/citizensadvice/design-system/pull/1071 without resorting to workarounds.
Makes heavy use of slots to keep things flexible. Allows things like:

Opting out of the search form
Providing a custom block for the account link, e.g. to replace the plain link with a form.
Skip links and secondary links can be provided by slots too to keep the interface consistent

Definitely needs some thought around the API, currently quite a few slots with some variant of the word links. Needs some clearer names. Also need to consider how the header search form relates to the inline one; perhaps we can still share a component.
TODO:

[x] Convert logo to slot. Use slots for everything for a consistent API
[x] Improve names of slots for different kinds of links
[x] API for account link with custom HTML
[x] Settle on an approach for the search form
[x] Test including navigation as part of header
[x] Add rspec tets
[ ] Generate examples from previews
[ ] Document usage

Probably the pro outweighs the con so maybe give it a bash and see how it feels by npm linking to the public website and implementing it as a user would?

Ended up trying something where the navigation exists as a separate component, but we delegate to it as a slot within the header. Feels like a good mix of both worlds. You can still use the nav standalone; but there's a simpler API available if using with the header.
Will try it out within a project to get a feel for things.

Will try it out within a project to get a feel for things.

The API for the navigation ended up feeling pretty good. However, realised it's a chunky change to the docs so kinda stretching the scope of these migrations. Going to keep them separate for now but we should revisit down the line.

Going to raise a cleaned up version of this now the navigation component is in.


fix({,Readonly}Record): remove laziness from pick

related #46. Would close that issue since omit looks good, otherwise it doesn't close it.
Also added a CI commit because I ran it against v14 and it didnt' work because replaceAll does not exist in those environments.

This is a good change for the most common use case. :+1:
I still think we'd benefit from an additional pickFrom where the generic can't be inferred (or some other approach that allows us to provide a single generic somehow):
const f = flow(pick<A>(ks), etc)


Use sentence case for breadcrumbs and wizard steps

The text used in breadcrumbs and wizard steps should be 'sentence case'.
e.g.  'Connector Type' should be 'Connector type', etc

Pending PR #206

PR merged, closing


Fix values for non-matching tuples in Umbra Q2

Umbra should have been outputting 0s instead of nulls for non-matching tuples.
(Sorry for the back-and forth on this)

No problem, thanks for the PR!


Remove broken link to python USB script.

The link on the wiki's broken.
Can you either remove the 'canary' from 'canary.discord.com', or just put the script in a gist like the bat script (please).
(I made a gist before I realised I can't edit the wiki, but feel free to make another one).

Thanks, I linked to your gist.


interp: fix resolution of methods on aliased types.

The type.val field was always pointing to the final underlying type for aliased types, defeating a possible match if a method was attached to a type in between. Now the complete chain of aliases is always preserved.
Fixes #1411.

This PR is broken is broken by _test/issue-1408 from #1409. I made a new version #1417 which supersedes this one, to be closed.


Use webpack's built in types in bundle-size-tools

Description
Webpack now has built in types. This migrates bundle-size-tools to use them instead of using an out of date separate types package.
Part of #9568
PR Checklist

Use the check-list below to ensure your branch is ready for PR. If the item is not applicable, leave it blank.

[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My code follows the code style of this project.
[x] I ran the lint checks which produced no new errors nor warnings for my changes.
[ ] I have checked to ensure there aren't other open Pull Requests for the same update/change.

Does this introduce a breaking change?

[x] Yes
[ ] No

If this introduces a breaking change, please describe the impact and migration path for existing applications below.

Consumers should adopt the built-in types for webpack instead of using the separate @types/webpaack package.
Testing
This package appears to have no tests. It is unclear how to properly test it.
These changes should not affect non-error case runtime behavior, so the built should test that the type changes work, but this does not confirm that bundle-size-tools is still working.

Thanks for fixing this!


It should indicate the current branch when opening

When starting git bc, it should somehow indicate the currently active branch, e.g. using a different color - or even making it the default selection. Right now, it's simply defaulting to the first branch. If you don't have a branch indicator in your current prompt, it's hard to tell what branch you're currently on.

git uses a * to indicate the current branch on git branch, I'll try to change the current behavior to preserve this indicator.
in the current version all branch names are being cleaned up before invoking the interactive listing


Changing the color of favicon

The current favicon is black in color so it is not clearly visible. I wanted to enhance the look by changing its color which will match with the color combination of the website. I am attaching the sample .Let me know if I can go ahead with it.

I would like to contribute as a Gssoc21 participant in this project. could you please assign me this?

Dear mentors @smaranjitghose @anushbhatia waiting for your response on this. If it is suitable for our project kindly assign it to me. Thank you!

The current favicon is black in color so it is not clearly visible. I wanted to enhance the look by changing its color which will match with the color combination of the website. I am attaching the sample .Let me know if I can go ahead with it.

Dear mentors @smaranjitghose @anushbhatia waiting for your response on this. If it is suitable for our project kindly assign it to me. Thank you!

The current favicon is black in color so it is not clearly visible. I wanted to enhance the look by changing its color which will match with the color combination of the website. I am attaching the sample .Let me know if I can go ahead with it.

This is good but it is barely visible.

@sharvariiii Could you just integrate this into your copy of the web app's code and send a small clip of the same. Because in this zoomed view, it is difficult to judge since the actual size would be much less in practice

@sharvariiii any updates>?

Closed due to inactivity


infinite event loop

I get this error with testrpc:
"Error: Error: sender doesn't have enough funds to send tx. The upfront cost is:<PHONE_NUMBER>0000000 and the senders account only has:<PHONE_NUMBER>0000000
    at runCall (/usr/lib/node_modules/ethereumjs-testrpc/node_modules/ethereumjs-vm/lib/runTx.js:93:10)
    at /usr/lib/node_modules/ethereumjs-testrpc/node_modules/async/lib/async.js:718:13
    at Immediate.iterate [as _onImmediate] (/usr/lib/node_modules/ethereumjs-testrpc/node_modules/async/lib/async.js:262:13)
    at processImmediate [as _immediateCallback] (timers.js:383:17)

First of all, I don't get this error with geth.
But also after this error, an event which is supposed to be emitted at the very end of the smart contract function is emitted in a infinite loop:
{"name":"ballot-queue-worker","hostname":"local.cocorico.cc-local","pid":6571,"level":30,"balance":"19218460000000000","event":{"logIndex":0,"transactionIndex":0,"transactionHash":"0x966daa499879fe93a1c8c43cdb69dec5843166c47f3088d5af76fbb0c4f845c7","blockHash":"0xd6701b9c7eacb014cb2374012b22297e417ea83ae50844fdc0b75982db06d41c","blockNumber":7,"address":"0x01cf04804565c1f45aec92cf94a2a78e239aa231","type":"mined","event":"Ballot","args":{"voter":"0xb4d0b2358bd40fabd1d260eee60871e4c0c0e176","proposal":"0"}},"msg":"ballot event","time":"2016-04-03T20:33:07.479Z","v":0}
{"name":"ballot-queue-worker","hostname":"local.cocorico.cc-local","pid":6571,"level":30,"balance":"19218460000000000","event":{"logIndex":0,"transactionIndex":0,"transactionHash":"0x966daa499879fe93a1c8c43cdb69dec5843166c47f3088d5af76fbb0c4f845c7","blockHash":"0xd6701b9c7eacb014cb2374012b22297e417ea83ae50844fdc0b75982db06d41c","blockNumber":7,"address":"0x01cf04804565c1f45aec92cf94a2a78e239aa231","type":"mined","event":"Ballot","args":{"voter":"0xb4d0b2358bd40fabd1d260eee60871e4c0c0e176","proposal":"0"}},"msg":"ballot event","time":"2016-04-03T20:33:07.653Z","v":0}
{"name":"ballot-queue-worker","hostname":"local.cocorico.cc-local","pid":6571,"level":30,"balance":"19218460000000000","event":{"logIndex":0,"transactionIndex":0,"transactionHash":"0x966daa499879fe93a1c8c43cdb69dec5843166c47f3088d5af76fbb0c4f845c7","blockHash":"0xd6701b9c7eacb014cb2374012b22297e417ea83ae50844fdc0b75982db06d41c","blockNumber":7,"address":"0x01cf04804565c1f45aec92cf94a2a78e239aa231","type":"mined","event":"Ballot","args":{"voter":"0xb4d0b2358bd40fabd1d260eee60871e4c0c0e176","proposal":"0"}},"msg":"ballot event","time":"2016-04-03T20:33:07.837Z","v":0}
{"name":"ballot-queue-worker","hostname":"local.cocorico.cc-local","pid":6571,"level":30,"balance":"19218460000000000","event":{"logIndex":0,"transactionIndex":0,"transactionHash":"0x966daa499879fe93a1c8c43cdb69dec5843166c47f3088d5af76fbb0c4f845c7","blockHash":"0xd6701b9c7eacb014cb2374012b22297e417ea83ae50844fdc0b75982db06d41c","blockNumber":7,"address":"0x01cf04804565c1f45aec92cf94a2a78e239aa231","type":"mined","event":"Ballot","args":{"voter":"0xb4d0b2358bd40fabd1d260eee60871e4c0c0e176","proposal":"0"}},"msg":"ballot event","time":"2016-04-03T20:33:08.118Z","v":0}
...

Since the transaction cannot occur, it would be surprisingly to have this event emitted even once.
But here it's emitted in a loop.
Any idea what's wrong?
Thank you !

Calling event.stopWatching() when I get the event I want solves the issue.
Still, that's a weird behavior and it doesn't happen with geth so I think it's worth investigating.

First, it's hard to debug when I can't see the code that makes the
transactions, nor the contract code itself.
Second, that error means there's not enough Ether in the address you're
sending from.
I don't have enough information to start debugging, or to assume it isn't
an error on your end. You'll have to provide that before I can do anything
further.
On Apr 3, 2016 1:44 PM, "Jean-Marc Le Roux"<EMAIL_ADDRESS>wrote:

Calling event.stopWatching() when I get the event I want solves the issue.
Still, that's a weird behavior and it doesn't happen with geth so I think
it's worth investigating.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/ethereumjs/testrpc/issues/56#issuecomment-205051567

Second, that error means there's not enough Ether in the address you're
sending from.

I know that. But why does it happen with testrpc and not geth?
Here is the contract:
contract Vote {

    event Ballot (
        address indexed voter,
        uint8 proposal
    );

    event VoterRegistered (
        address indexed voter
    );

    event VoteError (
        address indexed user,
        string message
    );

    struct Voter {
        bool registered;
        bool voted;
        uint8 vote;
    }
    struct Proposal {
        uint voteCount;
    }

    address chairperson;
    mapping(address => Voter) voters;
    Proposal[] proposals;

    // Create a new ballot with $(_numProposals) different proposals.
    function Vote(uint8 _numProposals) {
        chairperson = msg.sender;
        proposals.length = _numProposals;
    }

    modifier onlyChairPerson() {
        if (msg.sender != chairperson)
            throw;
        _
    }

    modifier onlyRegisteredVoter() {
        Voter voter = voters[msg.sender];

        if (voter.voted || !voter.registered)
            throw;
        _
    }

    function registerVoter(address voterAddress) onlyChairPerson {
        Voter voter = voters[voterAddress];

        if (voter.voted || voter.registered)
            throw;

        voter.registered = true;

        VoterRegistered(voterAddress);
    }

    // Give a single vote to proposal $(proposal).
    function vote(uint8 proposal) onlyRegisteredVoter {
        Voter voter = voters[msg.sender];

        if (proposal >= proposals.length)
            return; // FIXME: emit a BallotError event

        voter.voted = true;
        voter.vote = proposal;
        proposals[proposal].voteCount += 1;

        Ballot(msg.sender, proposal);
    }

    function winningProposal() constant returns (uint8 winningProposal) {
        uint256 winningVoteCount = 0;
        for (uint8 proposal = 0; proposal < proposals.length; proposal++)
            if (proposals[proposal].voteCount > winningVoteCount) {
                winningVoteCount = proposals[proposal].voteCount;
                winningProposal = proposal;
            }
    }

    /* This unnamed function is called whenever someone tries to send ether to it */
    function () {
        throw; // Prevents accidental sending of ether
    }
}

Here is infinite event emitting loop in the very simple situation: http://ethereum.stackexchange.com/questions/2614/event-watch-gets-fired-infinitely-many-times

Hi, just stumble upon the same problem, but bug is not that the event got emitted infinitely, bug is that after calling eth_getFilterChanges testrpc reports the events, and forgets to that it reported it for current block, so when the next eth_getFilterChanges happens, testrpc reports the same events again, and again, infinitely, until the new transaction(block) happens, then it starts reporting the event from new latest block endlessly.

@kumavis @flyswatter Have you seen this?

i have seen this before https://github.com/MetaMask/provider-engine/issues/55
I can investigate

Fixed and published in v2.0.3.


Add support for className to RadioButtons

📥 Proposed changes
Introduce support for className prop in RadioButtons react component.
☑ Submission checklist

[X] I have read the CONTRIBUTING doc
[X] yarn build works locally with my changes
[X] I have added tests that prove my fix is effective or that my feature works
[X] I have added necessary documentation (if appropriate)

💬 Further comments
Added test in FieldGroup for checking className support.

@all-contributors please add @joms for code


GH-38503: [Go][Parquet] Make the arrow column writer internal

This makes it so the Arrow column writer is not exported from the pqarrow package.  This follows up on comments from #38581.

Closes: #38503

After the conflicts are resolved, i'll merge this, thanks!

I've rebased this and resolved conflicts.


Fix silent error for Columns with null values

When pushing log entries through the multiple sinks I had logs showing on console + log file but not on Postgres DB. After investigating the issue I noticed that null values were not getting assigned DBNull.Value therefore it would throw a silent exception and the log entries were ignored

Looks good, thanks :)

@all-contributors please add @mwlsilva for code test


Retry helper stub doesn't accept Throwable

Larastan Version: #.#.#
--level used: max

Description
When using retry, the when callable will pass the exception to the callable. PHPStan won't accept the code and throw this error:
Parameter #4 $when of function retry expects (callable(Throwable|null): bool)|null, Closure(Throwable): bool given

I managed to solve it by changing the stub and adding TValue to the callable like this:
/**
 * @template TValue
 * @param int $times
 * @param callable(): TValue $callback
 * @param int $sleep
 * @param null|callable(TValue): bool $when
 * @phpstan-return TValue
 *
 * @throws \Exception
 */
function retry($times, callable $callback, $sleep = 0, $when = null)
{
}

I'm happy to do a PR with tests. I'm just not familiar with PHPStan stubs so I don't know if my change is correct.
Laravel code where the issue was found
retry(5, function () {
  // Code to retry
}, 30 * 1000, function (\Throwable $exception): bool {
    return $exception instanceof \RuntimeException;
});

Let us compare those.

(callable(Throwable|null): bool)|null
Closure(Throwable): bool


[REQUEST] dynamic batch size with gradient accumulate

Is your feature request related to a problem? Please describe.
my truly minimal use case request: I have 2 datasets with resolutions 256 and 512, I want to build 2 dataloaders, one dataloader load
256x256 images with batch size 8, one load 512x512 images batch size 2.
it comes to a conflict with the note in the documentation:
Note: train_batch_size must be equal to train_micro_batch_size_per_gpu * gradient_accumulation_steps * number of GPUs. For simplicity, you can choose to only specify two of the three parameters, the last one will be inferred automatically by DeepSpeed.

so how to decide the train_micro_batch_size_per_gpu?
that is comes to a grounding problem: how deepspeed process gradient accumulation?

judge the forward access of the model, no matter the batch size is what, the gradient accumulation is logical right
judge the instance number, namely the number of data instances go through the model, such as if instances reaches to 32 to perform optimization, so for 512x512 and 256X256 data, the forward times will be different, and cause a logical promblem with mixed batch size training.

Describe the solution you'd like

describe how deepspeed process gradient accumulation in the document
it is better to judge the forward access of the model to perform gradient accumulation
unlock the limitation of train_batch_size must be equal to train_micro_batch_size_per_gpu * gradient_accumulation_steps * number of GPUs

thank you for your great work.

hi, is there anyone?


File Preview like windows Explorer

What's the Problem?
File preview is not dynamic like the windows explorer.
(Only reason why iam using win explorer :-))
Solution/Idea
scrolling, zooming  and a bigger layout (full size) of preview
Alternatives
--
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
No response
Windows Version
Edition	Windows 11 Pro Version	21H2 Installiert am	05.06.2021 Betriebssystembuild	22000.778 Leistung	Windows Feature Experience Pack 1000.22000.778.0
Comments
No response

We're working on a full screen preview option but in the meantime, we recommend using QuickLook for a similar behavior.


Double ripple on touch devices with delegated click

Clicking a button or notification on Snarl while using an iOS device (or simulator?) will produce one ripple on the touch event, and then one ripple on the delayed mouseup/click event, meaning it double-ripples with a 300ms delay.
This doesn't happen with Waves v0.5.5, so I'll see why it's happening with the delegated click handler and how to stop it. (Disabling mouse or ripple events after the first touch for the following 500ms would probably fix it? But I'll see if there's a cleaner way.)
I reckon using fastclick.js would also solve it and make pages/buttons feel faster on mobile, but I'll investigate the Waves code first.

Sorry, I think I forgot to mention you before that I have this problem earlier #44, and then I solved it by adding exclusion using if statement.
if (touch) {
   // attach touch event
} else {
   // attach click event
}

The problem is that some touch screen windows laptops will also have a mouse, so if we don't bind the mouse up event then mouse clicks won't initiate waves on those devices? We'll probably need a way of temporarily disabling waves for mouseup events if a touch event has occurred in the last 500ms or so.

It seems if the double ripple problem disappear if I disable event temporary.

Fixed this issue by adding a TouchHandler which stacks clicks and disables mousedown for 500ms when a touchstart event occurs (doesn't affect the listeners so there's no performance loss with adding/removing listeners). On the train right now so I don't have Internet on the laptop, but I'll push the change later tonight :)


looking at rscource buton crash

if i click on the eye buton behand the ore patch in the listi go to the patch. but there is no way to get back to the character. and you are stuck in an ore patch

Should be resolved by v0.8, which just opens the map/zoom to world with the eye icon.


Fixed #27439 -- Documented macOS locale configuration for Django tests.

I don't think the issue is specific to Mac OS. For example, it's mentioned here: https://docs.djangoproject.com/en/1.10/ref/unicode/#files.

Indeed, it's not specific to macOS.
However, on Debian-based systems, it is generally recommended to use dpkg-reconfigure, as already stated in the docs.
On macOS, you don't have this option, so I think it's beneficial to mention LANG and LC_ALL env variables in this "troubleshooting tests" chapter.

I did not verify this, but I suspect that it can also affect Linux (and *BSD?) systems with certain values for LANG / LC_ALL.
IMHO we should document the export way, because it also works on shells other than bash. In addition to that, maybe add a note like this:

You can add these commands to your shell's startup file (for example, ~/.bashrc) so you don't have to retype them.

@alix-, could you update as per the last comment or do you disagree with it?

I updated. Looks like the failing javascript step of the build is unrelated.


multi select with  dynamic values hang in infinite loop when check value in edit mode

Description
In edit mode, when I try to check a value of a multi select with dynamic values, the application is not responding anymore because Vue component hang in infinite loop
Steps to reproduce

Create a new Laravel project
install and configure twill 2.8.2
follow this documentation link to create a multi select with dynamic values
follow this twill guide  to add multi select in create modal
add new item and select one or more value from multi select during item creation
save the item
from the index list select the new created item
from the edit view try to check new value from multi select

Expected result
The checkbox change to checked
Actual result
The checkbox remain unchecked and the application is not responding anymore (hang in infinite loop and chrome ask to terminate the tab)
To discover the error I build Vue component in DEV mode following this twill guide
With dev mode enabled when I try to check a value from multi select the console show this error
vue.esm.js?a026:628 [Vue warn]: You may have an infinite update loop in watcher with expression "storedValue"

found in

---> <A17Multiselect> at frontend/js/components/MultiSelect.vue
       <A17Fieldset> at frontend/js/components/Fieldset.vue
         <Root>

The problem seems to be in the checkboxes,js code at set method that is called  in infinite loop
set: function (value) {
        if (!isEqual(value, this.currentValue)) {
          this.currentValue = value
          if (typeof this.saveIntoStore !== 'undefined') this.saveIntoStore(value)
          this.$emit('change', value)
        }
      }

Versions

Twill Version 2.8.2

Laravel version 7.3 & Laravel version 8.83.12

PHP version 7.4

Database MySQL
Tested in Google Chrome and Firefox

Hi @haringsrob ,
thanks for the reply.
I had already tried and did it again now but with no luck.

if you find it useful I am attaching part of the code
###views/admin/foodItems/form.blade.php
@extends('twill::layouts.form')

@section('contentFields')
    @formField('multi_select', [
        'name' => 'foodCategories',
        'label' => 'Categorie',
        'options' => $foodCategories,
        'min' => 1,
        'max' => 5
    ])

    @formField('wysiwyg', [
        'name' => 'description',
        'label' => 'Descrizione',
        'toolbarOptions' => [ [ 'header' => [1, 2, false] ], 'list-ordered', 'list-unordered', [ 'indent' => '-1'], [ 'indent' => '+1' ] ],
        'placeholder' => 'Descrizione della categoria',
        'editSource' => false,
    ])

    @formField('tags', [
        'label' => 'Proprietà',
    ])
@stop

###The associated controller
class FoodItemController extends BaseModuleController
{
    protected $moduleName = 'foodItems';

    protected $indexColumns = [
        'title' => [
            'title' => 'Nome',
            'field' => 'title',
        ],
        'foodCategory' => [ // relation column
            'title' => 'Categorie',
            'sort' => true,
            'relationship' => 'foodCategories',
            'field' => 'title'
        ],

    ];

    protected function formData($request)
    {
        return [
            'foodCategories' => app()->make(FoodCategoryRepository::class)->listAll()
        ];
    }

    protected function indexData($request) {
        return [
            'foodCategories' => app()->make(FoodCategoryRepository::class)->listAll()
        ];
    }
}

Please feel free to reopen an issue if you are still seeing this, but we haven't been able to reproduce.

Not selected on edit mode