Mouseover highlighting issues with latest release of Chrome for Mac

Here's an example:

Chrome version: 37.0.2062.120
Mac OSX version: 10.9.4
All extensions are disabled.
This doesn't happen on my versions of Safari or Firefox. This also doesn't happen on Chrome for Chromebook, or on an older version of Chrome on another Mac that I have. But when I updated Chrome on that other Mac, the issue was reproduced (it runs OSX 10.7.5). So it probably has something to do with Chrome's latest stable release (or one that came out in the past couple months). Not sure if this was triggered by the recent bunch of commits, or if any version of CTH would have had the same issue. It occurs in both the Europe country quiz and the USA state quiz, so presumably it happens on all other quizzes, too.

This also happens in Chrome 83 on Linux for at least Europe and North America. Hovering over any of the countries gave a hover text with the country name on it. Kind of ruins the quiz.

The design goals were:
· if you’re moving really fast, the tooltip should not appear – this helps people who already know the neighbourhoods and want to perfect their time
· if you’re slowing down, the tooltips start appearing – this is for all the people who want to learn while playing (we got feedback that otherwise the game is too difficult and zero fun)

It's been six years so I can't remember, but to be clear I think my original issue was about the glitchy highlighting artifacts you see in my screenshot, not that mouseover reveals the answer. No idea if it still happens.

Thanks, @nluchs!

Ah, sorry I misunderstood what this issue was about. I could file a different bug for the timing?
Can I suggest making the tooltips a little slower to show up? It felt like cheating when I would mouse to a country that I was pretty sure was Moldova but before I clicked it would say "Slovakia" (for example).

After a few seconds of playing, mouseovers start appearing immediately. Mac, Mojave, Chrome 85.0.4183.121.


Make CIs available for forks

This PR makes a few changes to the config files for Travis CI and AppVeyor CI so that they work for forks of the repo.
P.S. Circle CI already works for forks and needs no change.

This looks simple enough! Though like all CI changes those simple fixes take a long time to get right!
What do you need to do to set it up on appveyor/travis/circleci?  Is it worth writing some instructions to go with it?  Perhaps a section in the CONTRIBUTING guide?

Though like all CI changes those simple fixes take a long time to get right!

You're right - I've already tried multiple times to get this PR working, especially for AppVeyor.
I have tested the following:

All the CIs build forks without running deploy
All the CIs build ncw/rclone with deploy
All the CIs build PRs on ncw/rclone without deploy

What do you need to do to set it up on appveyor/travis/circleci? Is it worth writing some instructions to go with it? Perhaps a section in the CONTRIBUTING guide?

People just need to go to those CIs' setting page and enable building for their forks. We can probably add a line or two in CONTRIBUTING.

Super job thanks :-)


feat(streaming): Introduce Column-Aware Row encoding, WIP

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
A new type of row encoding is introduced to support self-contained schema.

Summarize your change (mandatory)
WIP

Work Progress

[ ] Implement encoder and test
[ ] Implement decoder and test
[ ] Integrated test

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)
#7246

replaced by  #7301


Fix TileMapLayer emitting changed() signal after get_tree().quit()

If changed() callback of TileMapLayer accesses a tile_set property, game
hangs on close since changed() is emitted after the TileSet has been removed
from the scene tree.
This new check prevents changed() from being emitted when closing the
game.
Fixes #96898.

I am not sure this is the right way to fix it. This might be considered a breaking change, as the signal won't be emitted when the TileMapLayer is outside the tree. I think the right way to do it would require to disconnect from the signal when the TileMapLayer leaves the tree.


Copy Dashboard is not working properly

Hi Guys
just copied a dashboard from one Asset (which is a Group not a device) to another Group.
Result is that the widget shows the values of the former Asset and not the actual  Asset.
Assets are created with DTM.

issue was fixed in latest release https://github.com/SoftwareAG/cumulocity-asset-properties-widget/releases/tag/sag-pkg-asset-properties-widget-1.0.1


Importing data in Windows

I'm trying to run the MNIST notebook tutorial on a Windows machine, but it fails at the point where it is meant to build the database. I've run the tutorial successfully on an Ubuntu virtual machine and copied the database files over, however it fails at the point it runs the net:
return C.run_net_once(StringifyProto(net)) RuntimeError: [enforce fail at db.h:174] db_. Cannot open db: E:\nets\tutorials\tutorial_data\mnist\mnist-train-nchw-leveldb of type leveldb Error from operator: output: "dbreader_E:\\nets\\tutorials\\tutorial_data\\mnist\\mnist-train-nchw-leveldb" name: "" type: "CreateDB" arg { name: "db_type" s: "leveldb" } arg { name: "db" s: "E:\\nets\\tutorials\\tutorial_data\\mnist\\mnist-train-nchw-leveldb"}
Presumably this is because caffe2 was built without leveldb, as it is supposedly not available on Windows. I tried using an lmdb database instead, but the same error occurs. I was unable to build caffe2 with lmdb support, so presumably this is the reason.
So my question is: is it possible to build caffe2 on Windows with either leveldb or lmdb support, and if not what other options are there for importing data into caffe2 on Windows?

I think I've solved it. If anyone's interested, Caffe2 can be built on Windows with leveldb if you do the following:

Download Boost and build it, specifying static libraries (the default is shared) and 64 bit if necessary (32 bit is default)
Get the qdb branch of leveldb: https://github.com/bureau14/leveldb
Build leveldb, ensuring Runtime Library is set to 'Multi-Threaded (/MT)' in properties,C/C++ for both the leveldb and leveldbutil projects
Download the Windows port of Snappy for C++

Thanks @m00nkeh I'll add these notes to the Windows build instructions.


Possibly noncompliant OAI implementation breaks our Sickle overrides

When harvesting WOUDC at , we get the following traceback:
$ python globus_harvester.py
Traceback (most recent call last):
  File "globus_oai\harvester\OAIRepository.py", line 63,in _next_response
    super(FRDRItemIterator, self)._next_response()
  File "python3\tools\lib\site-packages\sickle\iterator.py", line 92, in _next_response
    oaiexceptions, code[0].upper() + code[1:])(description)
sickle.oaiexceptions.BadArgument: Missing metadataPrefix parameter

@usask-rc

This appears to also be a problem when using an unmodified Sickle, actually. Following up with the repository administrator...


[website] Move the React Engineer - X role to future roles

[x] I have followed (at least) the PR section of the contributing guide.

https://deploy-preview-42532--material-ui.netlify.app/careers/

I have unlisted the role https://app.ashbyhq.com/jobs/c86e52ce-0f3b-41cd-871b-5a7fd4044238/job-postings/77584e4c-9379-45f3-a294-6e50bf250383/info so it's not visible in https://jobs.ashbyhq.com/MUI.
On the hiring priorities, from https://www.notion.so/mui-org/0ea5085349464aefa17a6f56a6e25c2c, if the growth forecast matches the revenues, we could already start to work on the next two X roles. There are some roles that will have higher resources availability priorities, e.g. X sales roles > X engineering role at this point. So maybe risky, and instead, we can focus on onboarding of the new members.
For these X engineering roles, what I see:

A 3rd member on the charts team, give it more weight to bring revenues and be a serious open source option. It might take a while to find this 3rd person. While the charts team is small, every new joiners potential matters a lot, they will be the ones growing the team x10. Here are areas the team might need help on, I don't know how much Alexandre and Jose would want to focus on. Design engineering: bring blocks like tremor. Product leadership: be out there on Twitter to help/incentive Tremor, Mantine, etc to migrate from Recharts, go-to-market focused. The regular product engineering duties: bring more bandwidth. Noted added to the opening on Notion (better place to discuss further)
A new hire on the eXplore team. There, I could see us clearly playing the Mantine, Shadcn card. Wrap react-upload, full-calendar, slick, window-splitter, tiptap, and bring Material UI design to those. Push all support queries to these libraries if they are not style related. So this would be a very product design engineer role oriented. Otherwise starts a few small components: signature field, window splitter, barcode. I don't see us having bandwidth to start something large and in depth like RichTextEditor, no. A Scheduler/Gantt could almost be within range, but it feels like a too high opportunity cost, we don't know what it takes to do it well, if we are ressources strap, Grid, Charts, small components feels like should take resources propriety. If they do, then we would be left with not enough resources to bridge the gap needed to differentiate with those, so waste our time because we couldn't focus enough. Noted added to the opening on Notion (better place to discuss further).

Commit cherry-picked to master 8417e42c3f to clear potential confusions. https://x.com/rmaximedev told me he tried to follow the link on our career page but it lead to a closed Ashby role. We are good now on master: https://master--material-ui.netlify.app/careers/.

That's why I didn't close the Ashby role initially. I opted to wait until this was released, but thanks for cherry-picking it 🙏

@DanailH Right, I'm adding this to illustrate as an another instance of https://www.notion.so/mui-org/Update-mui-com-careers-to-sync-with-Ashby-6749a5ddbb7e4671b461e5c50d4476ff?pvs=4#66b79b82689a422ba97cfe3136e98f86. The problem with keeping the role open is that then people spend time to apply for nothing and we have to spend time to reject them.


Cannot figure out how to install on OSX

Need a bit of help installing it on macOS. I installed the npm package but it won't start.
What do you mean by ensure the 'nvim.exe' is available? I do have neovim installed via homebrew.

Uh, just mean you have to have a neovim executable available on MacOS. What error do you get when you type 'oni' in the shell?

I've run into the fact that the oni executable has Windows (CRLF) line endings, which screws up the shebang line and gives the error env: node\r: no such file or directory. I opened it in Neovim, did :set fileformat=unix and saved, and that was fixed.

Bummer - thought this was fixed in the last npmjs update. I'll revisit it, it seems like publishing from a Windows machine is problematic.  Thanks for the details
We might need a .gitattributes file to have git ignore the line ending conversion for this file.

I'm having similar problems, the install looks like it's erroring out at

npm run install:oni-plugin-typescript.

@Gajit - might be #19 also. I'll take a look at that one tomorrow, looks like quite a few people are hitting that

I published a new version to NPM - 0.1.2.
I believe the line-endings issue should be fixed now. I downloaded the tarball here:
https://registry.npmjs.org/oni-vim/-/oni-vim-0.1.2.tgz
And validated that the line endings are LF in the cli/oni file. Hopefully that means users won't need the :set fileformat=unix workaround. Let me know though if that is still showing up!

Ok I was able to npm install now but when Oni starts up it just gives a blank screen. Console errors with a Failed to load resources on

file:///usr/local/lib/node_modules/oni-vim/dist/browser/bundle.js Failed to load resource: net::ERR_FILE_NOT_FOUND


Flow crash

Create the dataset in #2307 using vars2dxy
Create a flow renderer
Select test for all the vars.
Enable

Vapor will crash

This issue can no longer be reproduced on the master branch as of 11/20/2020. I suggest we close it for now and verify later.
The created data set in question is attached as well.
issue_2461.tar.gz


feat: add C ndarray interface and refactor implementation for stats/base/dminsorted

Resolves None
Description

What is the purpose of this pull request?

This pull request:

adds C ndarray interface for stats/base/dminsorted
updates JavaScript implementation
refactors tests, docs, benchmarks and examples

Related Issues

Does this pull request have any related issues?

This pull request:

resolves None

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

same question here as #4181, what to do for generating random sorted arrays in benchmarks?

@Planeshifter, can you kindly give this a look and let me know what packages to use to generate random and sorted arrays for the JavaScript benchmarks? this would enable me to work on other packages that work with sorted arrays


Fix SUSPENDED channel reattach

Fixes #881.

Cherry picked fix done in another PR: 3cbca52.

@SimonWoolf Please review the last parts. Thanks


⚠ Bofrost.de has degraded performance

In 130e63b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 828 ms

Resolved: Bofrost.de performance has improved in 81c2116.


Breaks the Witcher 3

I installed this tool to change the color, since I didn't want the MSI bloatware. I tried to play the Witcher 3 after overclocking, and it wouldn't come up. uninstalled and reinstalled the Witcher, uninstalled and reinstalled different drivers, and restarted my computer a million times. The only thing that worked was removing the tool from the startup. It doesn't interfere with other games as far as I can tell.

That is interesting. Did the process (Witcher 3) appear in your task manager at that point?

Yup. Basically the screen turns black like it is starting up, and stays that way. I have to alt-tab and close it and the other Witcher process to close it. The Witcher also does that when I start up the MSI Afterburner, which I find to be weird.

I'm very sorry if you spent time trying to fix this issue. I just tried it without Afterburner on (after uninstalling some other possible conflicting programs), and the Witcher opened up just fine. Thank you for responding quickly!


Annoying Announcements from Amazon

This is kind of a follow up to my issue re: opening the Silk browser:
Using:
    - service: media_player.play_media
      data:
        entity_id: media_player.my_echo_show
        media_content_id: 'open mypage'
        media_content_type: custom

I am now able to have Alexa automatically open my HA dashboard via a trigger, which is progress!  There remains one small problem:

If I say "open mypage" using my voice (not the automation), Alexa silently opens Silk to my configured url (perfect!).
However, if I trigger "open mypage" using the above automation, it announces "OK, Here's mypage".
This would not be a problem, except that, due to limitations imposed by Amazon, after 5 minutes, it closes my page and reverts to is home screen.  There is no way around this that I have found.
So... I have to restart "open mypage" every 5 minutes, and every 5 minutes, it announces "Here's mypage", which is vary annoying and unacceptable to my wife.
I have tried setting the volume to 0, but the announcement forcefully turns the volume up before saying "Here's mypage".
So my question is, if triggering "open mypage" using my voice works with no announcement, perhaps there is a setting which can disable the announcement via the Alexa Media Player?

Here's hoping...
Thanks!

OMG, I discovered the solution!  If you just say "open mypage" it spouts off the announcement.  But if you use: "open mypage open dashboard"  it is SILENT!
Of course, dashboard must be setup in the mypage page list.
Issue closed!


Codecov dependency and script added

See manually imported results in https://codecov.io/gh/redhat-developer-tooling/developer-platform-install. I will update Jenkins jobs after this PR is merged, otherwise it would cause them to fail (non-existing npm script).

test this

@ppitonak is that possible to report coverage for PR on github the same way it is done for almighty-core PR's?

👍 it looks good

Thanks for merging. It's configured on Jenkins now so all new PRs will receive a comment from codecov.


[BUG] - couldn't find specified resource - resource might be unknown to client, or it's invalid

To Reproduce
Steps to reproduce the behavior:

Use /statistics or /admin-player-list command
Get error couldn't find specified resource - resource might be unknown to client, or it's invalid

Also error on command broadcast: invalid response code - message might not have broadcasted
I'm already authorized application for my server on cfcloud, but nothing works. But server-info works correct

Also in configs I'm already insert all ids and tokens

You are most likely using the wrong Server API ID, or an invalid combination of Application ID + Secret and Server API ID, please refer to the guide for a detailed overview of this information

Closing this as the wiki has all the information needed to resolve this issue


Incompatible with AngularJS

Hi,
Using JSqueeze on ownCloud, it fails to minify correctly the JavaScript file provided by the Notes application with the following error:
Error: [$injector:modulerr] Failed to instantiate module Notes due to:
[$injector:unpr] Unknown provider: t
http://errors.angularjs.org/1.4.0-build.3841+sha.ef894c8/$injector/unpr?p0=t
p/<@http://localhost/owncloud/assets/72773572308bad612ffce802e84db88d.js:104:418
be/p.$injector<@http://localhost/owncloud/assets/72773572308bad612ffce802e84db88d.js:104:24811
[ … more lines … ]
...="+encodeURIComponent(qt(n[r]))};return new t(i)}};var Xi=/^\/(.+)\/([a-z]*)$/,e...
-------------------------------------------^

(Firebug points the 418th character of the 104th line)
The unminified counterparts it comes from is:
for (i = SKIP_INDEXES, paramPrefix = '?'; i < templateArgs.length; i++, paramPrefix = '&') {
  message += paramPrefix + 'p' + (i - SKIP_INDEXES) + '=' +
    encodeURIComponent(toDebugString(templateArgs[i]));
}

return new ErrorConstructor(message);

I don’t believe the actual failure comes from here anyway: it’s just the error handling mechanism, as are the first hops (104:24811, 104:26521, etc.) pointed from the full debug message.
It may be related to the loading of angular.module (I really don’t understand much about JavaScript, sorry) as initiated by config/app.js and the way JSqueze shrinks this call and the actual AngularJS handling.
The three files corresponding to the three minified lines (104 to 106) are:

angular.js
angular-route.js
app.js

(Replacing those three lines by the content of those three files makes the application usable again.)
Please note that I’ve also reproduced the issue with two other versions (1.2.28 and 1.3.13) of AngularJS. I’d be happy if you could help finding a workaround if the issue is tough to fix. I’d also be happy to provide a more actionable bug report if you point me in the right direction (I have no idea how to properly debug such issue right now).

can you disable $singleLine mode? That make debugging easier...

Can you please disable $singleLine mode?

Sure, thanks for the tip. Please follow the links to the updated debug message and full JavaScript file.

I see no trivial pb for now, I'll need to dig deeper.
Any way I can run something an reproduce the issue locally?
Did you verify it works without JSqueeze? (by e.g. making JSqueeze return the original input string?)

Any way I can run something an[d] reproduce the issue locally?

I “just” installed the latest ownCloud (8.0.2), patched it to use JSqueeze instead of JSMin, and installed the Notes application that is unusable because of this issue (I’d expect any application using AngularJS —like News, Music or Tasks– to also fail the same way).
If you’re running Debian (or a derivative), I’m happy to provide the (currently WIP) packages I’m preparing, where I’m getting this issue. If you’re running Fedora (or a derivative), I guess @AdamWill may also help providing WIP packages.
I can also set up such a (broken) instance and give you a web access to it if that can help (I wish I could provide a minimalistic way to reproduce it, but I’m currently lost in JavaScript, maybe this weekend if I manage to learn how to use AngularJS…)

Did you verify it works without JSqueeze?

Yes, that’s what I meant by “Replacing those three lines by the content of those three files makes the application usable again.”

Yep, I can indeed provide Fedora or EL 7 packages with JSqueeze enabled if that'd be useful.

I'm used to Debian, so I could try with your packages on a VM

The packages not yet in the official Debian archive are on https://people.debian.org/~taffit/owncloud/.
You may add the following line to /etc/apt/sources.list (as well as experimental), and simply install owncloud-notes to get everything needed:
deb https://people.debian.org/~taffit/owncloud ./

(The apt-transport-https package is needed to use HTTPS repositories.)
Once installed, ownCloud should be available on http://localhost/owncloud. You need to activate the Notes application, and then, in the Notes interface, you should reproduce the issue.
Feel free to ping me or IRC if it you have any trouble setting up this test environment (I’m taffit in at least OFTC and FreeNode).

Here is a simple way to reproduce an error with AngularJS: with the simplified sample app/index.html from the beginning of their tutorial:
<!doctype html>
<html lang="en" ng-app>
<head>
  <script src="angular.js"></script>
</head>
<body>
{{'Yay' + '!'}}
</body>
</html>

Using an unminified version of angular.js, it correctly displays Yay!, but with a minified by JSqueeze version of angular.js, it displays {{'Yay' + '!'}} with an error in the FireBug JS console.
Hope this helps: even if the error message is different in the console, it shows at least an incompatibility, and involves a lot less to reproduce (even if angular.js is big…)
(Also, a minified by uglifyjs version of angular.js works fine, but I noticed it’s a lot bigger: original is 968K, JSqueeze’s version is 152K, while uglifyjs’ version is 263K).

Thanks! Comparing uglifyjs' vs jsqueeze's will help for sure!

I managed to create a currently failing test in #19
Fix pending :)

Btw, I get 145k with Uglify on my side

I managed to create a currently failing test in #19

Yay ;)

Btw, I get 145k with Uglify on my side

Maybe the version currently in Debian is not as effective as the latest upstream one…

Fixed in #19

[ Do not hesitate (to tell me) to open another issue if you prefer. ]
$scope should be protected
Still from their tutorial: controler.js is broken if handled via JSqueeze:
[…]
phonecatApp.controller('PhoneListCtrl', function($scope) {
  $scope.phones = [
[…]

becomes
phonecatApp.controller('PhoneListCtrl',function(t){t.phones=

then “Error: [$injector:unpr] Unknown provider: tProvider <- t <- PhoneListCtrl” is thrown in the JavaScript console and the page is not displayed correctly (UglifyJS doesn’t change $scope in t). That’s not exactly the same error as the one pointed in the beginning, but it’s getting closer.

Possible workaround
Hi,
I may have found a way to workaround this issue (or maybe that was just me misusing JavaScript…): I was actually using a plain concatenation instead of the improved (or protected) one. The differences are of this kind:
-app.controller('AppController', function ($scope, $location, is) {
+app.controller('AppController', ["$scope", "$location", "is", function ($scope, $location, is) {

Using the improved (or protected) version seems to work fine with JSqueeze, so I’m back being a happy user and tester (I suppose the issue is still worth fixing if it’s really an issue rather than a misuse on my side – in that case I apologize for being a bad tester and user).

This is it! Angular even has a doc about this new syntax being friendly to
minifiers. Not sure why Uglify works, maybe it has some rule for angular.
But for sure this is required for JSqueeze! Congrats, one bug fixed on each
side :-)

Here is the doc pointer: https://docs.angularjs.org/tutorial/step_05#a-note-on-minification

Fixed by #19

v2.0.2 and v1.0.7 tagged with fix embedded


Add description fields to all of the fields in our pydantic models

Docs Update Description
Pydantic models have a handy description field to document what the field is supposed to do. This is also included when using Pydantic for automated schema generation (i.e. json schemas). We should be leveraging this for better documentation and documentation/code colocation best practices.
Additional context
We are already exporting some schemas to our docs. We should include the raw JSON Schemas for all of our models as well.

@SteveDMurphy Assigning this to you since some of these more complex data-mapping related ones are fresh for you, but I'm going to hop in and help out as well :)

With all of the new fields, are we good to sneak this in as part of 1.5?


add type / content check to $articles argument in addPropertiesToAr…

…ticlesArray method
fixes fatal errors when products are out of stock or blocked for certain customer groups

Thank you @wizmobln for your Pull Request.
I pushed a simplified version, as the method already defines an array type hint and as such the only possible erroneous value would be an empty array.


verse within verse?

Just noticed: Guidelines state verse to be Contained by verse although it isn't allowed in verse.

This appears to be fixed in the development version of the Guidelines.


[google-sheet-sync] Fix return value error

Fix Error serializing return value: TypeError: Converting circular structure to JSON error from Cloud Functions logs. This is the continuation of pull request #341.

Thanks again @FluorescentHallucinogen !


Upgraded to DNN 8.0 and get the following error:

Line 6:
Line 7:
Line 9:
Source File: /new/DesktopModules/Events/SubControls/SelectCategory.ascx    Line: 7

Replaced the the 6.2.2 install code with the current github code, and get:
Parser Error
Description: An error occurred during the parsing of a resource required to service this request. Please review the following specific parse error details and modify your source file appropriately.
Parser Error Message: Could not load type 'DotNetNuke.Web.UI.WebControls.DnnComboBox' from assembly 'DotNetNuke.Web, Version=<IP_ADDRESS>9, Culture=neutral, PublicKeyToken=null'.
Source Error:
Line 1:  <%@ Control Language="vb" AutoEventWireup="false" Codebehind="SelectCategory.ascx.vb" Inherits="DotNetNuke.Modules.Events.SelectCategory" %>
Line 2:  <%@ Register Assembly="DotNetNuke.Web" Namespace="DotNetNuke.Web.UI.WebControls" TagPrefix="dnn" %>
Line 3:  <%@ Register TagPrefix="dnn" Assembly="DotNetNuke.Web.Deprecated" Namespace="DotNetNuke.Web.UI.WebControls" %>
Source File: /new/DesktopModules/Events/SubControls/SelectCategory.ascx    Line: 1
Version Information: Microsoft .NET Framework Version:4.0.30319; ASP.NET Version:4.6.1055.0

I'm having the same issue. Any fix or workaround?

Hello Gentlemen,
Are there any comments on this?  Will it be fixed?  I have two fairly important pages on my website using this module that I currently have to hide.

It will be fixed, but I don't know when. Since it seems that I am currently the only one doing work here,  it certainly has to wait till I am back from holiday (end march).
If someone could step in: a pull request with the fix is most welcome.

Cool.  Thanks,  What does it take to contribute?  I have written modules before.  All c# though, have not dealt with VB much in my professional life.

Well, interestingly, I downloaded the project from Github, recompiled it, put the DotNetNuke.Modules.Events.dll in the Bin directory of my site and now everything works.  So, I guess I am good.

Good to advthat things work. I am puzzled why it did not work im the first place.
Contribution can be done in any form of pull requests.
IMO editing existing VB.net code is not so difficult if you know C# (same apllies the other way around).


[Bug]: ts-jest[cli] (FATAL) Configuration file jest.config.cjs must be a JavaScript or JSON file. with config.migrate in vite react project

Version
^29.1.1
Steps to reproduce
I'm in the middle of a Parcel 1 > Vite upgrade.
After getting my tests running again, I got the following message:
ts-jest[backports] (WARN) Your Jest configuration is outdated. Use the CLI to help migrating it: ts-jest config:migrate <config-file>.
So I entered
ts-jest config:migrate jest.config.cjs
(File extension change was needed because type: "module" was added to package.json for vite).
Expected behavior
Some sort of upgrade wizard or something?
Actual behavior
ts-jest[cli] (FATAL) Configuration file jest.config.cjs must be a JavaScript or JSON file.
Debug log
None
Additional context
No response
Environment
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y)

The CLI migrate command does migration automatically. What kind of wizard would you suggest?

I would suggest that the migration should happen when the jest.config file has a cjs extension, since you need that with vite and it's a valid extension for a jest.config file.

Any movement on this?


Animation firing when the widget first appears

This is clearly visible when you open the window or after Hot Restart. In my app, I use a nested structure of Column and Row inside the ExpandablePageView and when the window is first opened (when the widget tree rendering is done), the Column height is animated at the speed of animationDuration. This is logical, because the Column changes its size from 0 px to N px. This is not a bug because it's in the logic of the ExpandablePageView widget (tween from previous height to the current), but this behavior for the first page in the PageView, that appears immediately after the widget tree is created, is clearly undesirable.
What do you think about that? One of the options: animate the size of the page when the page is scrolled, otherwise set the size of the page in the PageView without animation. This will also allow to place widgets inside the ExpandablePageView that already have a size animation (for example, ExpansionTile) and this animation may be inconsistent, i.e. the child's resize animation occurs before the ExpandablePageView animation. I have already encountered this, you you can check this behavior.
I created a special example in which I increased the animation duration for clarity and put the ExpandablePageView in a separate window.
https://user-images.githubusercontent.com/7840559/108893964-bdc98b80-7622-11eb-9639-4131392755cd.mp4
import 'package:expandable_page_view/expandable_page_view.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: OutlinedButton(
          child: Text('Press me'),
          onPressed: () {
            Navigator.of(context).push(
              MaterialPageRoute(
                builder: (context) => _ExpandablePage(),
              ),
            );
          },
        ),
      ),
    );
  }
}

class _ExpandablePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('ExpandablePageView'),
      ),
      body: SingleChildScrollView(
        child: Column(
          children: [
            ExpandablePageView(
              animationDuration: const Duration(seconds: 1),
              children: [
                ExamplePage(Colors.blue, "1", 300),
                ExamplePage(Colors.green, "2", 400),
                ExamplePage(Colors.red, "3", 500),
              ],
            ),
            const SizedBox(height: 20),
            Text("UNDER PAGE VIEW WIDGET"),
          ],
        ),
      ),
    );
  }
}

class ExamplePage extends StatelessWidget {
  final Color color;
  final String text;
  final double height;

  const ExamplePage(this.color, this.text, this.height);

  @override
  Widget build(BuildContext context) {
    return Container(
      height: height,
      color: color,
      child: Center(
        child: Text(text),
      ),
    );
  }
}

A quick fix that works: https://github.com/proninyaroslav/expandable_page_view/tree/animation_fix
This may not be a very pretty solution, but it animates the page widget only when needed, that is, when the page changes. The rest of the time, any resizing of the page widget will not be animated. If this solution suits you, I will make a pull request.
I also checked this with ExpansionTile inside ExpandablePageView, now only ExpansionTile is animating as it should.

Also video confirmation:
https://user-images.githubusercontent.com/7840559/108978915-28b6a900-769b-11eb-926c-ef4bda4ed242.mp4

By the way, another solution: leave the implicit animation of page sizes as an option. By default, it will be disabled, that is, the page animation will only be on scrolling.

Hi @proninyaroslav Thanks for the fix suggestion. I will take a look at it later this week since I am very low on time lately.

I've created a pull request including your fix @proninyaroslav : PR.
I have also added an additional estimatedPageSize variable which could help reduce the problem with shrinking and expanding of page view when you switch pages and they are expensive to build.
I think this PR also fixes the other issue


Rendering HTML as Text

Hello,
I have made my own column type but I have issue now.
here is my column.

namespace xxxx\Bundle\AdminBundle\Grid\Column;

class ThumbColumn extends \APY\DataGridBundle\Grid\Column\Column
{

    public function __initialize(array $params)
    {
        $params['filterable'] = false;
        $params['sortable'] = false;

        parent::__initialize($params);
    }

    public function getType()
    {
        return 'thumb';
    }

    public function renderCell($value, $row, $router)
    {
        if ($value) {
            return "";
        } else {
            return false;
        }
    }
}```

The result is a string with the HTML code but the HTML is not render...what is wrong ?

Use the safe attribute.

Hello...many thanks.
I have now create my own Template for my new column (thumb with AvalancheImage) and I have a last issue with the fact that the "value" data need to be a part of a filter
{% extends 'APYDataGridBundle::blocks.html.twig' %}
{% block grid_column_thumb_cell %}

{% endblock grid_column_thumb_cell %}

I have try different version but I don't find the way to print the value into the filter ?
What's the way the use a twig variable into a other twig variable ?

It's ok I have found my mistake


Improve the stability of e2e tests

Part of an effort to make Kopf-based operators more stable and resilient.

Issue : #13 #142

Description
E2E tests are flaky and unstable. This PR makes them more stable by:

Running in a standalone mode (on the same minikube as other, probably failed tests — to prevent freezing for minutes).
Limiting a time of termination of a background operator to 60s (and adding this feature).
Using a dynamic pod name in the pod-involving e2e tests (otherwise, it hits "409 Conflict" — when the pod already exists in the same cluster (actual for the local runs)).

Types of Changes

Bug fix (non-breaking change which fixes an issue)
Refactor/improvements

Codecov Report

Merging #149 into master will decrease coverage by 0.01%.
The diff coverage is 86.2%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   83.55%   83.54%   -0.02%
==========================================
  Files          30       30
  Lines        1545     1544       -1
  Branches      227      227
==========================================
- Hits         1291     1290       -1
  Misses        209      209
  Partials       45       45

Impacted Files
Coverage Δ

kopf/testing/runner.py
81.33% <86.2%> (-0.25%)
:arrow_down:

Codecov Report

Merging #149 into master will decrease coverage by 0.01%.
The diff coverage is 86.2%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   83.55%   83.54%   -0.02%
==========================================
  Files          30       30
  Lines        1545     1544       -1
  Branches      227      227
==========================================
- Hits         1291     1290       -1
  Misses        209      209
  Partials       45       45

Impacted Files
Coverage Δ

kopf/testing/runner.py
81.33% <86.2%> (-0.25%)
:arrow_down:


Cleans up Hydroponics content.

About the PR
Hydroponics content is very dirty and disorganized. I am cleaning it up.
= Putting stuff where it should be
= Adding stuff like inhands and updating some sprites
= Getting YAML and rsis up to standard

This is ready.

This is ready.

Fixed the one review and updated it from the last hydroponics PR.

Do not research

Pretty sure this messed up the spade texture.


Improving Keyboards

I'm thinking about improving keyboards since it happens a lot to me, that I'm a bit confused how to use keyboards. I always have to check the docs and I think we can make that a little bit more intuitive.
Status Quo
Currently Keyboards need to be defined either by mimicking the complete array structure of Telegram
Request::sendMessage([
    'chat_id'      => 12345,
    'text'         => 'Test',
    'reply_markup' => [
        'inline_keyboard' => [
            [ // Row 1
              [ // Button 1 in 1st row
                'text'          => 'A',
                'callback_data' => 'A'
              ],
            ],
        ]
    ]
]);

But it's possible to use a little bit syntactic sugar with the InlineKeyboard class:
Request::sendMessage([
    'chat_id'      => 12345,
    'text'         => 'Test',
    'reply_markup' => new InlineKeyboard(
        [ // Row 1
          [ // Button 1 in 1st row
            'text'          => 'A',
            'callback_data' => 'A'
          ],
        ],
    )
]);

I'm thinking about a few improvements.
Proposal 1
I already mentioned in #1239 that we should definitely use variable parameters with the ... notation instead of using func_get_args()
public function __construct(...$rows)

Impact: low
This is backwards compatible to the previous behaviour.
Proposal 2
Maybe we should split Keyboard and InlineKeyboard into sibling classes instead of parent and child. (Currently InlineKeyboard extends Keyboard). Main reason for this is, because InlineKeyboard has in fact no getResizeKeyboard, getOneTimeKeyboard, ... methods but it gets inherited from Keyboard.
Impact: low
This is backwards compatible to the previous behaviour.
Proposal 3
To be more consistent with current behaviour of the project we should create classes for ReplyKeyboardRemove and ForceReply. Currently you need to use the static methods on Keyboard: Keyboard::forceReply() and Keyboard::remove().
Since InlineKeyboard extends Keyboard you can use InlineKeyboard::forceReply() and InlineKeyboard::remove() too, which makes no sense semantically. This get's fixed with Proposal 2.
Impact: medium
We should mark the static methods as deprecated. But until it gets removed this is backwards compatible.
Proposal 4
We can change the __construct() methods of Keyboard, InlineKeyboard, KeyboardButton, InlineKeyboardButton and maybe even more to allow passing the required parameters as the first parameters of the constructor and the rest as an array after that.
Currently Keyboard and InlineKeyboard accept Buttons/Rows. KeyboardButton and InlineKeyboardButton accept an array with all arguments OR you can pass a string that get's automatically set as text property in the data array. But you can't pass additional parameters.
For Keyboard this means (as far as I know) that you need to instantiate a Keyboard object with the buttons, and after that you can call the set* methods for stuff like resize_keyboard and one_time_keyboard. I think you can't pass rows AND those properties.
This means the Keyboard constructor would look like this:
    public function __construct(
        array $rows,
        array $data = []
    )

It would not be possible to specify Keyboard rows like new Keyboard($row1, $row2, $row3) anymore.
Impact: high
The changes to Keyboard are not possible with backwards compatibility in mind. InlineKeyboard has no additional parameters, but I recommend changing it in a way so both classes behave the same.
Since InlineKeyboardButton and KeyboardButton already allow string $text as first parameter instead of array $data this stays backwards compatible. We would add an optional second parameter that is definitely array $data, but if this is not passed and the first parameter is an array, we take this as $data instead.
Partially backwards compatible
Proposal 5
Possible as soon as we bump the PHP dependency to 8.0
Like Proposal 4 but instead of a second array $data we make all those properties explicit arguments.
Pro: You can use them with named parameters like
'reply_markup' => new Keyboard(
    $rows,
    selective: true,
    input_field_placeholder: 'Type something...'
)

This would be even an awesome idea for other Entities/classes in the project. But this has one downside on the first look.
Currently most of the entities are perfectly usable when Telegram adds new fields to an object, but the library has not yet adapted it.
But I think we can still make this possible, if we always add a general array $data to the list of constructor parameters that can be used, if something is missing from the original parameter list. (In cases of Telegram Bot API Updates that are not yet reflected in the library.)
Impact: high
I think this has the same backwards compatibility notes as the previous Proposal.
Proposal 6
Adapt a more Laravel-like syntax.
Pro: It's awesome
Contra: We don't have this anywhere else in this project... But we could add this style in a few other areas too.
Example:
Keyboard::make()
        ->row([
            KeyboardButton::make('Test')->requestLocation()
        ])
        ->row([
            KeyboardButton::make('Second row')
        ])
        ->inputFieldPlaceholder('Type something...')
        ->selective();

or
InlineKeyboard::make()
              ->row([
                  InlineKeyboardButton::make('Login')
                                      ->loginUrl('https://example.com/login-with-telegram'),
              ])
              ->row([
                  InlineKeyboardButton::make('Previous')
                                      ->callbackData('prev'),
                  InlineKeyboardButton::make('Next')
                                      ->callbackData('next'),
              ]);

The awesome thing here is: We could implement a magic method that catches every additional call (like callbackData() and loginUrl()) and puts those (in snake_case) into the data array. And of course we would do that so we still have that awesome feature, that you can use new Telegram Bot API versions even if there is not yet a new release of this package.
Since InlineKeyboardButtons have this special case that you MUST use exactly one of the optional fields, it may be good to streamline this a little bit more like this:
InlineKeyboardButton::makeUrl($text, $url);
InlineKeyboardButton::makeLoginUrl($text, $loginUrl);
InlineKeyboardButton::makeCallbackData($text, $callbackData);
// ...

Impact: ultra high
Backwards compatibility: We could double-track this, leave the previous implementation in but mark as deprecated and add the new features like described.

Since we could add Proposal 6 without breaking anything that works now, that is my favorite for sure.

I discussed this with @noplanman:
We implement the following as described:

Proposal 1

We create new Keyboard classes in a new Entities\Keyboard namespace:

Proposal 2: Keyboard and InlineKeyboard as siblings
Proposal 3: Own classes for ForceReply and Remove
Proposal 4: new constructor signature with $data
Additional: We rename Keyboard to ReplyKeyboardMarkup and InlineKeyboard to InlineKeyboardMarkup to be closer to the Bot API type name.

We postpone the decision on Proposal 5 until we bump up the minimum PHP requirement.
We create a new package as a plugin for Proposal 6.

The new package (Proposal 6) can be found here: https://github.com/php-telegram-bot/fluent-keyboard/

I have no time to view all these, but i submitted a pull request may fix some of this.
Please view #1356


Max retires exceeded.

After installing gw2pvo I get the error:
HTTPSConnectionPool(host='globalapi.sems.com.cn', port=443): Max retries exceeded with url: /api/v1/PowerStation/GetMonitorDetailByPowerstationId (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f7a697ab978>: Failed to establish a new connection: [Errno -3] Try again',)
When I connect to globalapi.sems.com.cn I see the available API commands.
The command PowerStation/GetMonitorDetailByPowerstationId is not shown.
(I already changed the gw_api.py with data['msg'].lower() == 'success'.)
Is this a known issue?

I had this issue when using Docker on CentOS 8 with FirewallD.
Fixed by applying the following firewall rules:
firewall-cmd --permanent --zone=public --add-masquerade
firewall-cmd --permanent --zone=trusted --change-interface=docker0
firewall-cmd --permanent --zone=trusted --add-port=4243/tcp
firewall-cmd --reload

Also you might want to try this update:
sudo pip3 install https://github.com/markruys/gw2pvo/releases/download/1.3.2/gw2pvo-1.3.2-py3-none-any.whl

Please let me know if the latest release still has this issue. I expect it to be solved, so I close this issue for now.


Generate platform-specific gems from a single traject.gemspec

I looked at how Oga does it, and I think it's as easy as doing what I've done here. We just need to remember to release using both MRI and JRuby.
I ran rake build with both MRI and JRuby and untarred the resulting gemfiles (traject-2.0.pre-java.gem and traject-2.0.pre.gem) to look at them. The metadata  (e.g., list of dependencies) all looks good. I verified that require "traject" under jruby does, in fact, load up the marc4j and solrj stuff (required only under JRuby in index.rb).
I think all we need to do is decide what the defaults should be under JRuby, set them in a if defined? JRUBY_VERSION block and we're home free.

Yeah, that was my conclusion as well, I got to where you got to.
Initially I was hoping we could make it so a rake task like rake release would release both platforms, to avoid user error. But that doesn't seem to be possible, as you do need to switch to jruby to do the jruby release.
So I am a bit worried about us forgetting to do the jruby release (or vice versa) when we do a release, since you'll need to remember to do the release twice, once under jruby.  Do you think that's tolerable though, Bill?
Bah, I don't know.  Maybe I could live with the 'manual' solution instead, with one gem and the need to manually add teh marc4jreader gem.  We could have traject try to require the marc4jreader when on Jruby, and if it is found use it as the default, and if it's not found print a warning message recommending it. That would lessen the 'manual' steps someone had to do. I still don't like it without bundler, as users can run into dependency version problems and wind up really needing Bundler to save them the headache. (Without bundler, and with just a separate gem with no dependency from traject gemspec, if there's a bug in marc4jreader and a later version is released -- there's no way to automatically require that, the user just has to go and make sure the 'right' version is the latest installed on their system(s)).
I dunno, what are your current thoughts?

oh, and I actually wouldn't bother pulling in the traject-solrj_writer.
As far as we know, there are no general performance improvements for the solrJWriter, right? We don't actually recommend it?  If it's not the recommended-by-use for-most-use-cases solution, I don't think we need to automatically pull it in, and there are benefits to keeping MRI and Jruby on the same codebase as much as possible, when not required for performance. So I'd let people pull in SolrJWriter manually if they want it.

I'm happy to be Release master and in charge of remembering to release both. I think that's the way to go.  I'll take out solrj, set up the defaults for JRuby, add some tests, and ping you to take a look.

I've added some minor tests and changed the default JRuby reader to be Marc4J::Forgiving. I think there might be a problem where rubymarc outputs empty subfields? I'm gonna look into that, but for the moment I think this is a better default.
Also did a minor refactor of the default settings to more easily (if less prettily) set up with different defaults for MRI and JRuby.

changed the default JRuby reader to be Marc4J::Forgiving. I think there might be a problem where rubymarc outputs empty subfields? I'm gonna look into that, but for the moment I think this is a better default.

I have zero idea what you were talking about there, but that might be okay, just warning you that I got nothing from that sentance, haha.
I'd like to review your code, but I'm pretty slammed this week. I'm also still okay with releasing a pre1 at any point, if people need it.

Heh. I can see why that made no sense. What I meant:

I made the JRuby default the marc4j reader
I constructed a marc-binary file by using ruby-marc to read in a marc-xml file and spit out marc21
I tried to read in the resulting file with marc4j, but got "empty subfield" errors
...which leads me to believe that ruby-marc's writer outputs empty subfields in some way
...and also let me to make one of the default JRuby settings 'marc4j_reader.permissive' => true

Cool, so

There might be a bug in ruby-marc writer, it happens, and ideally we'd find it and write a failing test case and file it as an issue.

Was permissive the default for Marc4JReader in traject 1.x?  I think we should keep it the same. If permissive wasn't the default then, I wouldn't make it the default now -- I doubt reading files written by ruby-marc is a common use case?

I'll have to look to see if it was the default before. Permissive is also a
titch faster :-)
On Tue, Feb 10, 2015 at 4:22 PM, Jonathan Rochkind<EMAIL_ADDRESS>
wrote:

Cool, so

There might be a bug in ruby-marc writer, it happens, and ideally we'd
find it and write a failing test case and file it as an issue.

Was permissive the default for Marc4JReader in traject 1.x? I think we
should keep it the same. If permissive wasn't the default then, I wouldn't
make it the default now -- I doubt reading files written by ruby-marc is a
common use case?

—
Reply to this email directly or view it on GitHub
https://github.com/traject-project/traject/pull/83#issuecomment-73786402
.

--
Bill Dueber
Library Systems Programmer
University of Michigan Library


It's impossible to have both (CE and EE) version on Mac.

If you install CE version to Mac it will be impossible to install EE version using the Mac installer. You need to delete CE version firstly.

@serge-rider thanks for the info.

Mac installer and DMG image are fixed.


Entering Chat not showing while typing

When I am trying to type the input window is hidden.
To Reproduce
Steps to reproduce the behavior:

Hit enter
Chat window should show with text that user is inputting
Not showing, but if you type something the text will show in chat
See error

Expected behavior
Text should show while you are typing it.
Does the Glass work if you disable all other addons?
No.

Can you try to check that your chat style is on "Classic"? Interface -> Social -> Chat style. Options are IM style and Classic. Not sure if this is it but it's an idea anyway.

Hi thank you for responding.
I checked both and tried it each way with no luck. I’ve installed it on the
beta and it works well there. It’s just on retail where I’m having issues
On Mon, Aug 24, 2020 at 9:35 AM Mitchel Cabuloy<EMAIL_ADDRESS>wrote:

Can you try to check that your chat style is on "Classic"? Interface ->
Social -> Chat style. Options are IM style and Classic. Not sure if this is
it but it's an idea anyway.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mixxorz/Glass/issues/68#issuecomment-679129868, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQXFFZNSGXZNEBZKEWXRCHLSCJUCJANCNFSM4QJNTD4A
.

Mixx,
Sorry for the spam. I figured it out. SUI was causing an issue with it.
I disabled the chat functionality of that addon  and it works perfect now.
Thank you for the work you do with the add on.
On Mon, Aug 24, 2020 at 9:46 AM Gage McCoy<EMAIL_ADDRESS>wrote:

Hi thank you for responding.
I checked both and tried it each way with no luck. I’ve installed it on
the beta and it works well there. It’s just on retail where I’m having
issues
On Mon, Aug 24, 2020 at 9:35 AM Mitchel Cabuloy<EMAIL_ADDRESS>wrote:

Can you try to check that your chat style is on "Classic"? Interface ->
Social -> Chat style. Options are IM style and Classic. Not sure if this is
it but it's an idea anyway.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mixxorz/Glass/issues/68#issuecomment-679129868, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQXFFZNSGXZNEBZKEWXRCHLSCJUCJANCNFSM4QJNTD4A
.

Aha. No, this is useful info. Thanks! I'll add this to the README so people are aware they need to disable the chat module for SUI.

I, too, am having this issue without SUI installed. The issue persists even when all other addons are disabled. Likely a permanent setting from another mod as it works on a level 1 character with no addons enabled except for glass.


feature(generate module and routes with --lazy)

Proposal
ng generate component --lazy home should generate:
    home.component.css
    home.component.html
    home.component.spec.ts
    home.component.ts

// and new scaffolds:

    home.module.ts
    home.routes.ts

there is already a ng generate module scaffold

We are currently working on this.

lazy is not a product of a component, instead it is associated to an NgModule. Furthermore lazy loading of modules is configured with the routes that reference a module and because the CLI currently does not generate routes (for many reasons) this is not something that will be implemented. At least not until route generation makes sense.


[stable/redis] s/PostgreSQL/Redis

Bump to verify CLA

Note, this is a doc only change and could be merged without bumping the version. Bumping versions with docs is on the agenda for charts chat. After that meeting I'll come back with more (unless someone else comes and merges first)

/ok-to-test

Hey @mattfarina Do you guys need anything more for this PR?

Due to merge issues (from bumping the version) I manually merged (preserving the author). See 599b41c1031863635d0ec8fa4deb8cb8cc4b28fb


Add bundle one to helm chart for now

Add bundle one to the helm chart for now so we have something by default

/lgtm


Custommask warning when saving to h5 using built-in keras layers

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): ubuntu 18.04
TensorFlow installed from (source or binary): pypi
TensorFlow version (use command below): 2.5.0
Python version: 3.8.5

Describe the current behavior
When saving a model to h5 format, and the model contains one of the following layers:
tf.keras.layers.experimental.preprocessing.Resizing()
tf.keras.layers.experimental.preprocessing.CenterCrop()

there is a:
CustomMaskWarning: Custom mask layers require a config and must override get_config. When loading, the custom mask layer must be passed to the custom_objects argument.
When trying to load the keras model with tf.keras.models.load_model, it'll fail with:
TypeError: ('Keyword argument not understood:', 'crop_to_aspect_ratio')
Describe the expected behavior
It should both correctly save and load
Standalone code to reproduce the issue

@ghylander ,
In order to reproduce the issue reported here, could you please provide the complete code and the dataset you are using. Thanks!

unfortunately i can't, provide the dataset, but it's not relevant, it happens when saving an untrained model (only default weights from imagenet)
by doing further tests i found out that the issue just started happening on the AWS machine we use for training models
it just began happening even with model architectures that have been working for months.
There is specially something that confuses me, the code i am using to save the model is:
model.save('FULLPATHTODIRECTORY/tf_model')
model.save('FULLPATHTODIRECTORY/keras_model.h5')

and the output is:
WARNING:tensorflow:Compiled the loaded model, but the compiled metrics have yet to be built. `model.compile_metrics` will be empty until you train or evaluate the model.
/home/ubuntu/miniconda3/envs/VENVNAME/lib/python3.8/site-packages/tensorflow/python/keras/utils/generic_utils.py:494: CustomMaskWarning: Custom mask layers require a config and must override get_config. When loading, the custom mask layer must be passed to the custom_objects argument.
  warnings.warn('Custom mask layers require a config and must override '
INFO:tensorflow:Assets written to: FULLPATHTODIRECTORY/tf_model/assets
WARNING:tensorflow:Compiled the loaded model, but the compiled metrics have yet to be built. `model.compile_metrics` will be empty until you train or evaluate the model.

why is it rising a warning about compiling the model?
here gist with simplified code:
https://colab.research.google.com/drive/1GOYa3nBBmw27h7aayCfskz-p0SxElQ0S?usp=sharing
i used this dataset for the gist: https://storage.googleapis.com/mledu-datasets/cats_and_dogs_filtered.zip

Hi, i cloned my venv and installed tf 2.4.2 on it and i don't have this issue
is there something that changed in how models are saved?
previous to the stable 2.5.0 i had used both 2.5.0 nightly november release and march release, and i didnt have this issue before
i changed nothing in the code, just uninstalled 2.5.0 and installed 2.4.2
the issue is also present in the gist i linked, which i assume to be a "clean" install

@ghylander ,
The warning was added with TF 2.5 in commit 076b5be.Please refer this comment from the issue.Thanks!

i usppose i'll close it as it seems to be a duplicate then


Update generator options color

I Just found that the background color of generator options was almost the same as the text color so I changed it to bg-gray-800
####before

####after

Thanks, @MohanedAshraf really appreciate this! Just one minor thing to check.


Flocker driver for Ceph storage - error

Hi,
How to configure the flocker driver for Ceph ?
i have seen the below in your website :

version: 1
control-service:
hostname: "user.controlserver.example.com"
dataset:
backend: "ceph_flocker_driver"
cluster_name: "Ceph cluster name, defaults to ''"
user_id: "Ceph user ID, defaults to 'admin'"
ceph_conf_path: "path to ceph conf for rados, defaults to '/etc/ceph/ceph.conf'"
storage_pool: "name of storage pool to use, defaults to 'rbd'

here Cluster_name is IP address of Ceph server?
error in current ceph /flocker setup:
command: docker run -v test1:/data --volume-driver flocker busybox sh -c "echo hello > /data/file.txt"
response : docker: Error response from daemon: Timed out waiting for dataset to mount..

it is actually, just the cluster name. http://docs.ceph.com/docs/jewel/rados/deployment/ceph-deploy-new/#naming-a-cluster

This isnt really helpful. Please provide a full set of logs for us to help you. You can find more about how to do that here https://docs.clusterhq.com/en/latest/administering/debugging.html#debugging-flocker

Hi @wallnerryan , if we can use just the Cluster name here , where we have to give the IP of the Ceph cluster?

@rajkumar49 the ceph driver is not ready for production so it may not have all the bells and whistles yet. Right now I think it assumes it can talk to rbd client on the localhost which in turn runs against the cluster.

@wallnerryan is there an ETA for the ceph flocker driver to be production ready?

@galindro ClusterHQ has closed down and is therefore no longer working on Flocker or drivers for Flocker. However, a group of ex-ClusterHQ engineers are offering support plans and feature development at http://scatterhq.com.


Add support for audio/video files

please add support for mp3, wav, mp4, avi, and other audio/video files 🙏🏻


when attempting to configure proxmox to use mailrise I get a wierd error

thanks for looking at this.
<EMAIL_ADDRESS>works via powershell test: Send-MailMessage -from simon@local -to<EMAIL_ADDRESS>-subject "test" -body "test" -smtpserver <IP_ADDRESS> -port 8025

[2024-06-25 12:11:27] WARNING:apprise:Failed to send Telegram notification to<PHONE_NUMBER>: Bad Request: can't parse entities: Unsupported start tag<EMAIL_ADDRESS>at byte offset 34, error=400.
[2024-06-25 12:11:27] WARNING:mailrise.skeleton:Notification failed: address: [ Proxmox VE<EMAIL_ADDRESS>➤<EMAIL_ADDRESS>] subject: [ Test notification  ] body: [ This is a test of th (0.1K) ]
[2024-06-25 12:12:20] WARNING:apprise:Failed to send Telegram notification to<PHONE_NUMBER>: Bad Request: can't parse entities: Unsupported start tag<EMAIL_ADDRESS>at byte offset 34, error=400.
[2024-06-25 12:12:20] WARNING:mailrise.skeleton:Notification failed: address: [ Proxmox VE<EMAIL_ADDRESS>➤<EMAIL_ADDRESS>] subject: [ Test notification  ] body: [ This is a test of th (0.1K) ]
[2024-06-25 12:15:36] WARNING:apprise:Failed to send Telegram notification to<PHONE_NUMBER>: Bad Request: can't parse entities: Unsupported start tag "simon" at byte offset 34, error=400.
[2024-06-25 12:15:36] WARNING:mailrise.skeleton:Notification failed: address: [ Proxmox VE  ➤<EMAIL_ADDRESS>] subject: [ Test notification  ] body: [ This is a test of th (0.1K) ]
[2024-06-25 12:17:54] WARNING:apprise:Failed to send Telegram notification to<PHONE_NUMBER>: Bad Request: can't parse entities: Unsupported start tag "simon" at byte offset 34, error=400.
[2024-06-25 12:17:54] WARNING:mailrise.skeleton:Notification failed: address: [ Proxmox VE  ➤<EMAIL_ADDRESS>] subject: [ Test notification  ] body: [ This is a test of th (0.1K) ]

Try it with a valid email address in the 'From Address' field.  That works for me.

I assume that proxmox sets<EMAIL_ADDRESS>as sender.
du you get the same error with
Send-MailMessage -from<EMAIL_ADDRESS>-to<EMAIL_ADDRESS>-subject "test" -body "test" -smtpserver <IP_ADDRESS> -port 8025

I encountered a similar problem when sending from proxmox. There was an error in the logs:
WARNING:mailrise.skeleton:Notification failed: address: [ Proxmox VE<EMAIL_ADDRESS>➤<EMAIL_ADDRESS>] subject: [ Test notification  ] body: [ This is a test of th (0.0K) ]
By chance I managed to find out what the problem was. Mailrise does not work correctly with the author field. The problem goes away if you put a space there. It seems to me that the problem is on the mailrise  side, since alternative solutions handle this situation correctly.
P.S. Until the next edit of the submission

Unfortunately, although the above setting worked for testing, it did not work in real notifications. The problem is still the same: the sender's nickname is transmitted, which breaks the entire logic of parsing recipients.
[2024-08-25 10:38:18] WARNING:apprise:Failed to send Telegram notification to <chatid>: Bad Request: can't parse entities: Unsupported start tag<EMAIL_ADDRESS>at byte offset 73, error=400.
[2024-08-25 10:38:18] WARNING:mailrise.skeleton:Notification failed: address: [ Proxmox<EMAIL_ADDRESS>➤<EMAIL_ADDRESS>] subject: [ vzdump backup status (proxmox.domain.com): backup successful  ] body: [ <html>
 (3.4K) ]

I managed to solve the problem only partially (html is not displayed correctly in telegram messages), but the notifications themselves finally arrive with these settings:

configs:
<EMAIL_ADDRESS>    urls:
      - tgram://....
    mailrise:
      body_format: text

Is there any way to configure proxmox not to format these as HTML to begin with?


Missing License

Could not find any mention of license anywhere?
I hope the license is Apache 2.0 (cause does not need me to mention contribution, I think).

Same here.
I hope for MIT 😅. But Apache 2.0 would be good too.
But any way legally it looks like a no-permission license.

I've added Apache 2.0 license to the repo. Thank you everyone and sorry for the late update.


Why create unused parameters in functions?

With reference to extracted code below:
validator :: BankParam -> Validator
validator = Scripts.validatorScript . typedValidator
valHash :: BankParam ->  Ledger.ValidatorHash
valHash = Scripts.validatorHash . typedValidator
{-
scrAddress needs the BankParam (for the beneficiary!)
-}
scrAddress :: BankParam ->  Ledger.Address
scrAddress = scriptAddress . validator
Does these functions really need BankParam? Another confusing part is valHash. It is never called anywhere in the module, am I missing another Haskell rule?

BankParam is lifted into place and used in the data for the validator. It's just a record type that simplifies putparams containing only the needed data to parameterize the script (address).
valHash is not needed. Was thinking of using it, but it's extraneous.
Also, It's best, when opening issues, to reference line numbers so a reviewer can jump to the specified place.


K8s IT Test enhance

Fixes https://github.com/apache/druid/issues/10750.

Description

do the Druid/K8s cluster creation and cleanup in integration tests instead of relying on the Travis script

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[x] added integration tests.
[x] been tested in a test Druid cluster.

Key changed/added classes in this PR

MyFoo
OurBar
TheirBaz

Still need to deal with DIR properly

Still need to deal with DIR properly

Since Job 73 in CI is successful. I think this PR is ready for review.

Since Job 73 in CI is successful. I think this PR is ready for review.

Hi @himanshug  All modifications have been completed. Thanks for your review!

Hi @himanshug  All modifications have been completed. Thanks for your review!


Is there a way to mark code parts as unreachable and gray-out their colors?

I'm developing a extension with semantic tokens && language server support, and I want to make some parts of code grayed-out, as seen in this screenshot with Python:

I have no idea which token type/token modifier to use to get similar effect. Developer: Inspect Editor Tokens and Scopes shows no difference between "normal" and "grayed-out" code.

This is accomplished through Diagnostics, not Semantic tokens. You will want to use the tags property with DiagnosticTag.Unnecessary.

This is working, huge thanks! :wave:
Code snippet for researchers:
const diagnostic: Diagnostic = {
	range: {
		start: startPos,
		end: endPos
	},
	message: description,
	tags: [
		DiagnosticTag.Unnecessary
	]
}

diagnostics.push(diagnostic)

For some reason, VSCode marks code as error, even if DiagnosticSeverity is not set, and I don't want this behavior.
Screenshot:

My code looks like this:
var diagnostic: Diagnostic = {
	range: {
		start: startPos,
		end  : endPos
	},

	message: "Unreachable code test",
	tags: [ DiagnosticTag.Unnecessary ],
	// severity: DiagnosticSeverity.Error
}

diagnostics.push(diagnostic)

// ...

connection.sendDiagnostics({
	uri: 			uri,
	diagnostics: 	diagnostics
})

How to resolve this issue?

To fix that issue, I've had to use severity: DiagnosticSeverity.Hint with DiagnosticTag.


Flowise + Qdrant Load Existing Index (cloudhosted). Qdrant Receiver never returns any output.

Describe the bug
Flowise + Qdrant Load Existing Index (cloudhosted) never retuns any context.
When you create a basic QA setup with Qdrant Load Existing Index, the Qdrant Receiver never returns any output.
Qdrant Server URL is pointing to the cloud hosting.
When i test the same setup in a google colab, it works with no problems. When i use pinecone load index it also works, so it must be related to the qdrant load existing Index Features in combination with cloud based hosting of the qdrant vectorDB.
See my 2 screenshots attached for details.
Screenshot of the flowise setup

Setup in google colab which is working with the same index

@govind-kumarr Thanks for your implementation of qdrant. I am really a fan of their vector DB. Unfortunately it does not work for me. Can you take a look? This would be awesome. Thx.

Hey @cinemapara. I've got a setup similar to you and the generations work fine for me. Can you confirm the collection points have payloads with content to answer your query?

Hello @Anush008 thanks for your feedback. The Qdrant collection is correctly searched according to qdrant logs and response returned - but the response is never submitted to the conversational retrieval QA chain. You can test this by activating "return source documents" in your sample. It will not list any documents.

@cinemapara. I did get the flow to return source documents. Can you confirm that you've saved your flow after enabling "Return source documents"?

@Anush008 yes i saved it but still i do not get any output from the qdrant receiver. Do you specify anything under qdrant -> loading index -> additional parameters ? i tried adding the collection name there aswell but it did not help either (see screenshot). I am lost. Thanks a lot for taking your time supporting on this

No additional configurations. The only variability I can see among our setups are the collections.

Hello @Anush008 thanks for your feedback. I figured it out. My payload inside qdrant was page_content but flowise seems to expect payload = content. That was the problem. Thanks for your support.

Great👍👍


Discussion: Changes to API Page

I believe that the first code snippet in the "Interacting with the Environment" section of the API page is quite important because it serves to illustrates how gym is supposed to be used. I think there are a few changes that could make some points a bit clearer at a quick glance.

The output of step should be assigned to variables with meaningful names
Seeding should be demonstrated
The environment should be reset after a done signal has been emitted

Therefore, I would suggest something like this:
import gym

env = gym.make("CartPole-v0")
env.seed(42)
observation = env.reset()

for _ in range(1000):
  env.render()
  observation, reward, done, info = env.step(env.action_space.sample())

  if done:
    observation = env.reset()
env.close()

This snippet would demonstrate all essential methods of the API.
Of course, the animation won't be as pretty as the current one, but I think this snippet would do a better job of illustrating the usage of gym.
I would also suggest that each method is covered in a small subsection that covers parameters, effects and return values.
One thing that was quite unclear to me when I started using gym was how to tell whether a done signal was generated by the environment or a time limit. This is quite important in many RL algorithms, e.g. SAC. Although one might cover this in the Wrappers section, I think it is worth mentioning in the API section as well.

Can this be closed?


Ohu UI

I removed NotoSansOldHungUI dictionary totally, it had more rendering problems.
I added NotoSansOldHungarianUI font in glyphs.
The following command I run for testing:
./build src/NotoSansOldHungarianUI/NotoSansOldHungarianUI.glyphs
It said: Interpolating master UFOs from designspace
fontmake -g src/NotoSansOldHungarianUI/NotoSansOkdHungarianUI.glyphs
It said:
File  /"usr/lib/python3/dist-packages/booleanOperations/flatten.py", line 1073 in _flattenSegment
onCurve1, offCurve1, offCurve2, onCurve2 = segment
ValueError: not enough values to unpack (expected 4, got3)
@marekjez86  Any idea?

@marekjez86 Or should I upload a diff  or patch file, instead of PR?

@marekjez86 The shape of the letters of Old Hungarian font for UI is good according to the font design program. It has rendering comment problem only, I think.

@marekjez86 May I ask You, check, what is wrong in the second commit? The FontLab can generate font from NotoSansOldHungarianUI.glyphs file!
First commit remove totally the NotoSansOldHungUI structure, because of that, I believed, it would be better source code in glyphs file.

@marekjez86 I close it and open a PR with minimal changes


Running cell in Interactive Window gets stuck on "Pending"

Environment data

VS Code version: Version: 1.91.0
Jupyter Extension version: v2024.6.0
Python Extension version: v2024.10.0
OS: Microsoft Windows Server 2022 Standard
Python version: 3.7.8 (tags/v3.7.8:4b47a5b6ba, Jun 28 2020, 08:53:46) [MSC v.1916 64 bit (AMD64)]
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Jupyter server running: Local

Info
I am trying to run cells from a regular python (.py, not .ipynb) file. Sometimes, I run code in the cells directly.
Jupyter › Interactive Window: Creation Mode is set to Multiple
Expected behaviour
Code should run immediately.
Actual behaviour
SOMETIMES (this behavior is very inconsistent), the cell gets stuck on "Pending" for some period of time before executing.
In the logs below, the cell was Pending for 5+ minutes before it finally executed.
Steps to reproduce:
I had run all the cells in my file at once. Then I ran the following cell (%%timeit) in the Interactive Window. Half a minute later, while the cell was still pending, I ran another cell. Five minutes later, I finally got output.

Now can I run cells normally, but after a bit the problem occurs again, randomly.
It doesnt need to be this way, sometimes the pending behavior occurs when I send a code cell (from my file) to run. It can be literally any code, as simple as printing a variable. The pending time varies a lot.
Logs
11:34:26.108 [debug] Code execution cancelled by extension ms-toolsai.jupyter
11:34:26.108 [debug] Code execution cancelled by extension ms-toolsai.jupyter
11:34:26.108 [debug] Jupyter completion time: 1109
11:34:29.745 [debug] Code execution cancelled by extension ms-toolsai.jupyter
11:34:29.745 [debug] Code execution cancelled by extension ms-toolsai.jupyter
11:34:29.745 [debug] Jupyter completion time: 124
11:34:33.843 [debug] Handle Execution of Cells 31 for Interactive-1.interactive
11:34:33.843 [trace] Cell Index:31 sent to kernel
11:34:53.548 [debug] Code execution cancelled by extension ms-toolsai.jupyter
11:34:53.548 [debug] Code execution cancelled by extension ms-toolsai.jupyter
11:34:53.548 [debug] Jupyter completion time: 264
11:34:53.554 [debug] Handle Execution of Cells 32 for Interactive-1.interactive
11:40:20.889 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
11:40:20.927 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
11:40:20.967 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
11:40:21.003 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
11:40:21.040 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
11:40:21.078 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
11:40:21.115 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
11:40:21.152 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
11:40:21.168 [trace] Start cell 31 execution @<PHONE_NUMBER>168 (clear output)
11:40:21.168 [debug] Kernel acknowledged execution of cell 31 @<PHONE_NUMBER>168
11:40:35.891 [trace] Cell 31 completed in 14.723s (start:<PHONE_NUMBER>168, end:<PHONE_NUMBER>891)
11:40:35.891 [trace] Cell 31 executed successfully
11:40:35.892 [trace] Cell Index:32 sent to kernel
11:40:35.904 [trace] Executing silently Code (idle) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
11:40:35.908 [trace] Start cell 32 execution @<PHONE_NUMBER>908 (clear output)
11:40:35.908 [debug] Kernel acknowledged execution of cell 32 @<PHONE_NUMBER>908
11:40:35.911 [trace] Cell 32 completed in 0.003s (start:<PHONE_NUMBER>908, end:<PHONE_NUMBER>911)
11:40:35.911 [trace] Cell 32 executed successfully
11:40:35.936 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
11:44:26.948 [trace] Executing silently Code (idle) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
11:44:26.988 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)

This happened again just now, this time while running a code cell from my file
Here are the logs BEFORE
12:01:03.557 [info] Generated code for 34 =  with 100 lines
12:01:03.562 [trace] Cell Index:34 sent to kernel
12:01:03.572 [trace] Start cell 34 execution @<PHONE_NUMBER>572 (clear output)
12:01:03.572 [debug] Kernel acknowledged execution of cell 34 @<PHONE_NUMBER>572
12:01:06.021 [trace] Executing silently Code (busy) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
12:01:09.039 [trace] Cell 34 completed in 5.467s (start:<PHONE_NUMBER>572, end:<PHONE_NUMBER>039)
12:01:09.039 [trace] Cell 34 executed successfully
12:01:09.076 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
12:01:44.460 [trace] Executing silently Code (busy) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
12:01:45.742 [trace] Executing silently Code (busy) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
12:01:48.446 [debug] CodeLensFactory: Generating new code lenses for version 32 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:01:48.794 [debug] CodeLensFactory: Generating new code lenses for version 33 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:01:49.305 [debug] CodeLensFactory: Generating new code lenses for version 34 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:01:49.647 [debug] CodeLensFactory: Generating new code lenses for version 35 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:01:53.798 [debug] CodeLensFactory: Generating new code lenses for version 37 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:01:56.908 [debug] CodeLensFactory: Generating new code lenses for version 38 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:01:58.686 [debug] CodeLensFactory: Generating new code lenses for version 39 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:01:59.704 [debug] CodeLensFactory: Generating new code lenses for version 41 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:02:10.430 [trace] Executing silently Code (busy) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
12:02:12.604 [debug] CodeLensFactory: Generating new code lenses for version 42 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:02:17.959 [debug] CodeLensFactory: Generating new code lenses for version 53 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:02:18.409 [debug] CodeLensFactory: Generating new code lenses for version 55 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:02:19.178 [debug] CodeLensFactory: Generating new code lenses for version 57 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:02:19.862 [debug] CodeLensFactory: Generating new code lenses for version 58 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:02:20.404 [debug] CodeLensFactory: Generating new code lenses for version 59 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:02:20.888 [debug] CodeLensFactory: Generating new code lenses for version 61 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
12:02:21.332 [debug] CodeLensFactory: Generating new code lenses for version 62 of document file://proton/AlgorithmGroup/Users/~/projects/InstrumentDifferences/python/script_usepartial.py for commands jupyter.runcell, jupyter.runallcellsabove, jupyter.debugcell, jupyter.debugcontinue, jupyter.debugstop, jupyter.debugstepover
Here are the logs after I "Run Cell" (using ctrl+Enter)
In the middle, I hovered over some functions in my file, which is what caused those four logs at 12:04
12:02:21.631 [info] Generated code for 35 =  with 100 lines
12:02:21.634 [trace] Cell Index:35 sent to kernel
12:04:40.681 [trace] Executing silently Code (busy) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
12:04:42.326 [trace] Executing silently Code (busy) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
12:04:49.556 [trace] Executing silently Code (busy) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
12:04:54.056 [trace] Executing silently Code (busy) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
12:07:40.021 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
12:07:40.058 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
12:07:40.095 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
12:07:40.099 [trace] Start cell 35 execution @<PHONE_NUMBER>099 (clear output)
12:07:40.099 [debug] Kernel acknowledged execution of cell 35 @<PHONE_NUMBER>099
12:07:45.276 [trace] Cell 35 completed in 5.177s (start:<PHONE_NUMBER>099, end:<PHONE_NUMBER>276)
12:07:45.276 [trace] Cell 35 executed successfully
12:07:45.312 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
12:07:45.349 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
12:07:45.386 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)
12:07:45.423 [trace] Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)

Do you have the variable explorer open? Might be related to #15838

I dont have the variable explorer open in this case

Please could you enable logging as follows:

Open settings & go into Jupyter->Logging
Change the value to trace
Reload VS Code,
Attempt to repro the issue & then copy the output from the Jupyter output panel (use the command Jupyter: Show Output to get to the logs).


Track data

Proposed changes:

track which story blocks created a training tracker in tracker.block_history dict, including which checkpoints they went through to get there

Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog

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

@JoeyFaulkner I don't think we need this at the moment, right?

@tmbo nope, was deprioritised!


Feature/sql_err_handling

added sql error handling of unique violation.
added implementation for email validation

Thanks for sharing, I think this is a really useful addition!
Unique checks are common in my case and I hacked them by querying the resource first, to see if I have a duplicate 😅😅
Leveraging pq error codes is much more effective & elegant 💡

At quick glance it looks good. I need to look closer and will later today. Thanks.
In the meantime, please add yourself to the CONTRIBUTORS file.

Thanks for sharing, I think this is a really useful addition! Unique checks are common in my app and I hacked them by querying the resource first, to see if I have a duplicate sweat_smilesweat_smile Leveraging pq error codes is much more effective & elegant bulb

@ciprian-cimpan Thanks! It saves an extra call to the db and alleviates "some" bottlenecks due to network i/o being the most likely cause for drop is application performance; like what @ardan-bkennedy constantly reminds people of.
There is this repo https://github.com/omeid/pgerror/blob/master/pgerror.go that we can perhaps incorporate in this repo or our own private repo to usage. I didn't import it as a vendor because that's a decision is up to someone else to re-engineer. For the project to leverage this, i think Bill should have more situational awareness (SA) and decision making since he uses this repo a lot for his courses.

There is this repo https://github.com/omeid/pgerror/blob/master/pgerror.go that we can perhaps incorporate in this repo or our own private repo to usage.

Indeed, that's really useful! I bookmarked the repo & might use it in the future, but for my project I will only keep your unique constraint check addition, as that seems to be the most frequently needed.

Please add your name/email to the CONTRINBUTORS file and I will accept the PR.

Please add your name/email to the CONTRINBUTORS file and I will accept the PR.

Hi Bill,
I am currently in there already (twice):
https://github.com/ardanlabs/service/blob/master/CONTRIBUTORS#L43-L44


where is output_embedding concatenated in squad task?

i concatenated the elmo input embedding to glove.  but the performance is not increased. i used usage_token method. did i do something wrong?

Very likely -- you can find our SQuAD model here: https://github.com/allenai/document-qa/tree/master/docqa/elmo


Should messaging.destination_publish.* be in the stabilization scope for messaging?

There are no messaging systems that make any use of this namespace.
We don't know if there are any that could have the original destination available
We don't know if we'd ever have anything except the name attribute available to the consume
messaging.destination.original.name could be an alternative name.

Given all of this, I'd like to exclude messaging.destination_publish.* from the stabilization scope or even deprecate it as something we don't know where/how to use.

Looking at existing usage of this attribute, it seems we've created some confusion with it:

Both attributes are set, but somehow they are different for queues and topics?
https://github.com/kalaspuff/tomodachi/blob/ef036831d59d23e18280af91288ad2c6046c26db/README.md?plain=1#L1393-L1394
span.set_attribute("messaging.destination.name", queue_url.rsplit("/")[-1])
span.set_attribute("messaging.destination_publish.name", topic or queue_url.rsplit("/")[-1])

Different attributes  are set depending on the kind
https://github.com/airtai/faststream/blob/28f47833dd77509f4596fb582e6b539468c31fbc/tests/opentelemetry/basic.py#L102-L113
if span.kind == SpanKind.PRODUCER and action in (Action.CREATE, Action.PUBLISH):
    assert attrs[SpanAttr.MESSAGING_DESTINATION_NAME] == queue, attrs[
        SpanAttr.MESSAGING_DESTINATION_NAME
    ]

if span.kind == SpanKind.CONSUMER and action in (Action.CREATE, Action.PROCESS):
    assert attrs[MESSAGING_DESTINATION_PUBLISH_NAME] == queue, attrs[
        MESSAGING_DESTINATION_PUBLISH_NAME
    ]
    assert attrs[SpanAttr.MESSAGING_MESSAGE_ID] == IsUUID, attrs[
        SpanAttr.MESSAGING_MESSAGE_ID
    ]

Destination_publish is set on producer  for rabbitmq (to exchange)
https://github.com/open-telemetry/opentelemetry-php-contrib/blob/9f96b59bc483cc495edc49cef0d63b637a46fffa/src/Instrumentation/ExtAmqp/src/ExtAmqpInstrumentation.php#L61
->setAttribute(TraceAttributes::MESSAGING_DESTINATION_NAME, $routingKey)
->setAttribute(TraceAttributes::MESSAGING_DESTINATION_PUBLISH_NAME, sprintf('%s%s', $exchange->getName() != '' ? $exchange->getName() . ' ': '', $routingKey))

Given all of this, I'd like to exclude messaging.destination_publish.* from the stabilization scope or even deprecate it as something we don't know where/how to use.

I second that.
One thing we should ensure for stabilization: that those attributes (or similar) attributes are not needed on metrics. They're not in the current list of attributes on metrics, however, I'd like us to validate that again. If there's no need for those attributes on metrics, let's exclude them from initial stability.

Discussed on messaging SIG 7/11:
Let's deprecate messaging.destination_publish.* attributes since we have no real-life use-case for them and current usages are not correct.


ocamlPackages.carton: Tests require getconf

catched while reviewing another PR https://github.com/NixOS/nixpkgs/pull/196958#pullrequestreview-1150201632
getconf is used here https://github.com/mirage/ocaml-git/blob/471406dde6c3056ea49547de379fef12871da48b/bin/fiber/fiber.ml#L132 since it is a runtime dips I patched the source.
Description of changes

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg build ocamlPackages.carton


Testing issues - what is the proper way to verify work? (question/advice)

What is the proper way to test/verify work?
I would like to contribute to #526 and have changed the $data file names and the requires  statements that call them. All build fine, but I'm looking for any help, advice or suggestions on the proper way to test my work before thinking about a pull-request.
npm run build :  compiles fine.
npm run test : over 8000 CRLF to LF errors. i removed   linebreak-style: [2, unix] from eslintrc.yml and gets me through
npm run test again :  jshint says the following:

ERROR: Can't open lib/.js
ERROR: Can't open lib/**/.js

I tried unmarking the folders as readonly, no luck...
npm run test-spec :  I receive  --  bad option: --harmony-async-await)
npm run test-browser >'scripts' is not recognized as an internal or external command.
I've tried  performing this worklow in the readme.md file both before and after my changes with no luck.

npm install
git submodule update --init
npm test

Any help or advice would be appreciated.

Please run in some Linux VM in node 4 (other versions don't run browser tests)

Alternatively, when you make PR the tests will run in travis - you will see if they fail. Thank you.