feat: optional vertical alignment [t|b|c] for array and aligned environments

What is the previous behavior before this PR?
\begin{array} and \begin{aligned} rendered any optional argument like [t].
What is the new behavior after this PR?
Matching LaTeX, \begin{array}, \begin{darray}, \begin{aligned}, \begin{alignedat}, \begin{gathered} (but not \begin{align}, \begin{alignat}, \begin{equation}, \begin{gather}, \begin{split} and starred variants, \begin{matrix} and variants, \begin{smallmatrix}, \begin{cases}) support custom vertical alignment via [t], [b], or [c] optional argument.
At least, that's the goal.  I haven't been able to test this yet, or write tests, because of #3189, but I theorize that it or a slight variation will work.
Optional arguments in environments: Along the way, I had to add support for optional arguments in environments, which was silently ignored before.  This only gets used in \begin{alignedat}[t]{2} because the optional argument is in the "correct" position before the required argument. We can't use it (without some modification) in \begin{array}{cc}[t] because the optional argument comes later.  Turns out I was wrong, and the correct order is \begin{array}[t]{cc}, so I can use the optional argument parser in all cases.
Demo:

BREAKING CHANGE: Optional arguments used to be rendered in aligned environments, but now they are correctly processed (as they would in LaTeX).  This is arguably a bug fix, but could break existing code like
\begin{aligned}
[hello] & [world] \\
\end{aligned}

Fixes #2172, fixes #2760

This is good stuff. It should get recognized in the documentation. Suggest modification of supported.md file:
In the Other KaTeX Environments table, to the row labeled gathered, etc., insert a line .".. take an optional argument [t|b|c] to set vertical alignment."
Revise one example environment to read
\def\arraystretch{1.5}
\begin{array}[c]{r|c:l}
   ab & c & de \\ \hline
   f  & g & h \\ \hdashline
   i  & j & k
\end{array}

There is a sentence that begins: "The {array} environment supports | ... . It's an unusual sentence for this document, which doesn't do much explanation. It just enumerates every KaTeX function and shows a working example of each. So that sentence is out of sync with the rest of the document. It should be either deleted or augmented with the {darray} environment and the new optional argument. I would be okay with either choice.

it would be nice if this got approved!

actually, I applied the patch and it doesn't seem to work too well for me:
$X=\begin{array}[t]{cc} 1 & 2 \\ 3 & 4\end{array}$

produces this:


Zoom breaks, if data has no .date attribute.

While running eventdrops with data like
{ "name" : [{ "date_sent": "2017-08-11" }] }

and
.date( d => new Date(d.date_sent) )

the graph is shown nicely.
After a zoom event, all cx values are set to 0 while exceptions like
Error: <circle> attribute cx: Expected length, "NaN".

show up in the console.

I may be having a similar issue, with cx attribute problem:
Unexpected value NaN parsing cx attribute.

But I'm using (from example):
    var data =  [
        { name: "http requests", data: [new Date('2014/09/15 13:24:54'), new Date('2014/09/15 13:25:03'), new Date('2014/09/15 13:25:05')] },
        { name: "SQL queries", data: [new Date('2014/09/15 13:24:57'), new Date('2014/09/15 13:25:04'), new Date('2014/09/15 13:25:04')] },
        { name: "cache invalidations", data: [new Date('2014/09/15 13:25:12')] }
    ];

Graph renders, when zoom or scrolling, the JS error is shown x7 (once for each element).

This is how the zoom function looks like in the compiled code:
            e.default = function(e, r, n, a, i) {
                var s = function(r) {
                        var s = d3.event.transform.rescaleX(n.x);
                        (0, u.boolOrReturnValue)(a.hasTopAxis, r) && e.selectAll(".x-axis.top").call(d3.axisTop().scale(s)), (0, u.boolOrReturnValue)(a.hasBottomAxis, r) && e.selectAll(".x-axis.bottom").call(d3.axisBottom().scale(s));
                        var c = (0, o.default)((0, l.default)(e.select(".labels"), {
                            x: s
                        }, a), 100);
                        t.requestAnimationFrame(function() {
                            e.selectAll(".drop-line").selectAll(".drop").attr("cx", function(t) {
                                return s(new Date(t.date))
                            }), c(r), i && i(r)
                        })
                    },
                    c = d3.zoom().scaleExtent([a.minScale, a.maxScale]).on("zoom", s).on("end", a.zoomend);
                return e.call(c), c
            }

I would expect it to be simmilar to the compiled draw function
     function(t, e, r) {
        "use strict";
        Object.defineProperty(e, "__esModule", {
            value: !0
        }), e.default = function(t, e, r) {
            return function(n) {
                var a = t.selectAll(".drop-line").data(n).enter().append("g").classed("drop-line", !0).attr("width", e.x.range()[1]).attr("transform", function(t, r) {
                        return "translate(0, " + e.y(r) + ")"
                    }).attr("fill", r.eventLineColor),
                    o = a.selectAll(".drop");
                o.data(function(t) {
                    return t.data
                }).enter().append("circle").classed("drop", !0).attr("r", 5).attr("cx", function(t) {
                    return e.x(r.date(t))
                }).attr("cy", r.lineHeight / 2).attr("fill", r.eventColor).on("click", r.click).on("mouseover", r.mouseover).on("mouseout", r.mouseout), o.exit().on("click", null).on("mouseout", null).on("mouseover", null).remove(), a.exit().remove()
            }
        }

but it seems not to use config.date at all.

I do not understand the compile machinery yet, but maybe changing
export const onRequestAnimationFrameFactory = (...)

to
export onRequestAnimationFrameFactory = (...)

in zoom.js would fix this.

After building the distribution myself my error vanished.
Without further thinking I followed the current readme and used the source from https://unpkg.com/event-drops for my experiments, which resolves to<EMAIL_ADDRESS>

@thoka  Are you saying that following the README doesn't work, and that https://unpkg.com/event-drops isn't up to date?
I'm using:
<script src="https://d3js.org/d3.v4.min.js"></script>
<script src="https://unpkg.com/event-drops"></script>

And getting the same problem you initially described.

It seems so.
unpkg is at 0.3.1-alpha4, the last tagged version from 2017-06-19 11:45:24.
The code in the repo is some commits newer.
Quoting unpkg:

You may also use a semver range or a tag instead of a fixed version number, or omit the version/tag entirely to use the latest tag.

I've put the current build at
<script src="https://gist.github.com/thoka/ca508531e9a016fb815477621f9d6aae.js"></script>

or not uglified at
<script src="https://gist.github.com/thoka/f66e1b154b851fb8dd4e2ab30b19b5b8.js"></script>

I have the same problem. Where can I get the full compiled code without error? Your links are not working.
Thanks

Ok, I copied the files and saved them and now its working. You should update the README with the newest build


Unable to get property 'parent' of undefined or null reference

function foo(input:

export class Foo {
    bar() {
        let f = new Foo();
        return f.bar();
    }
}

This code causes compiler error:
TypeError: Unable to get property 'parent' of undefined or null reference
  at appendOuterTypeParameters (:14140:17)
  ...


SDC-1752 feature/remove-tags-from-metrics

SDC-1752

just some code smell that should be easy to clean 😉


JSON Web Requests for Suggestions

Bot should have a "web API" for getting suggestions via a web request. The suggestions should be returned in a JSON format to be further processed.
There should be two separate requests. One that grabs data for Watched movies, and one for suggestions that have not been watched.
Data that should be in the requests;
Suggested Movies
{
server_id:<PHONE_NUMBER>421,
suggested: [
{
imdb_id: ######,
title: Title,
year: ####,
poster_url: URL,
full_size_poster_url: URL
total_score: ####
total_votes: ####
num_votes_entered: ####
date_suggested: DATE
suggestor: USERNAME
},
]
}

For servers that have the IMDB check off, this will return null for any IMDB-based info that is not available.


Add a test for the examples

This PR was originally written by @dnephin in https://github.com/ServiceWeaver/weaver/pull/287. The tests were
working locally but not on GitHub Actions. After some debugging, I found
that running subcommands using "sh -c" was causing things to hang. I
still have no idea why that is, but things are working now :)
I also tweaked logger_test.go because it was printing a lot (and I mean
a LOT) of stuff. When debugging on GitHub Actions, these logs were very
annoying.
Original commit messages from @dnephin:

This commit adds a test case for running each of the examples as both a
single deploy and multi-deploy. This test should ensure that the
examples are always in a working state, and provide some end-to-end
coverage of the single and multi deployers.
It also provides some test coverage of the integration of codegen with a
real application.
Register signal handler before listening
TestExamples only waits for a listener to be listening and handling HTTP
requests.
The signal handler must be registered before the listener is started,
otherwise a test waiting on the listener could send a SIGTERM before
the signal handler is registered.

Thank you for debugging this and getting it merged!


DISK does not create a workflow on wings.

*Describe the bug
DISK does not create a workflow on wings.
To Reproduce
Steps to reproduce the behavior:

Go to (https://disk.isi.edu/hypotheses/Hypothesis-eyhHQoDP0qhP)
Click on Update

Expected behavior
Create the workflow
Screenshots
Quering hypothesis: Hypothesis-6f28AYbUooqn
[(http://localhost:8080/disk-server/admin/hypotheses/Hypothesis-6f28AYbUooqn#goal http://disk-project.org/ontology/hypothesis#associatedWith Temperature), (http://localhost:8080/disk-server/admin/hypotheses/Hypothesis-6f28AYbUooqn#goal http://disk-project.org/ontology/hypothesis#associatedWith Humidity), (http://localhost:8080/disk-server/admin/hypotheses/Hypothesis-6f28AYbUooqn#goal http://disk-project.org/ontology/hypothesis#associatedWith Wind)]
LOI LOI-mOFYR83dYZpf got 1 results
Warning: file https://raw.githubusercontent.com/mosoriob/bikes_rent/master/bikes-part1.csv does not contain hash on Bikes-rental
Checking TriggeredLOI-3yMYjp38Z21U (LOI-mOFYR83dYZpf)
TLOI TriggeredLOI-3yMYjp38Z21U will be trigger
[R] Running execution thread
[R] Executing bikes with:
[R] - measure_per_day: bikes-part1.csv (1)
[R] - variable2: Temperature
[R] - variable3: Temperature
[R] - variable1: Temperature
Error parsing:
Can not deserialize instance of java.util.ArrayList out of VALUE_STRING token
at [Source: org.glassfish.jersey.message.internal.ReaderInterceptorExecutor$UnCloseableInputStream@4a8c0c09; line: 1, column: 708] (through reference chain: edu.isi.wings.portal.classes.util.TemplateBindings["parameterBindings"]->java.util.HashMap["https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#variable1"])
Error parsing:
Can not deserialize instance of java.util.ArrayList out of VALUE_STRING token
at [Source: org.glassfish.jersey.message.internal.ReaderInterceptorExecutor$UnCloseableInputStream@1e1fcc2a; line: 1, column: 708] (through reference chain: edu.isi.wings.portal.classes.util.TemplateBindings["parameterBindings"]->java.util.HashMap["https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#variable1"])
Error parsing:
Can not deserialize instance of java.util.ArrayList out of VALUE_STRING token
at [Source: org.glassfish.jersey.message.internal.ReaderInterceptorExecutor$UnCloseableInputStream@7f2c0799; line: 1, column: 708] (through reference chain: edu.isi.wings.portal.classes.util.TemplateBindings["parameterBindings"]->java.util.HashMap["https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#variable1"])
+ users/admin/bikes/plan/getExpansions
+ toPOST: {"templateId":"https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#bikes","componentBindings": {},"parameterTypes": {"https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#variable1":"http://www.w3.org/2001/XMLSchema#string","https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#variable3":"http://www.w3.org/2001/XMLSchema#string","https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#variable2":"http://www.w3.org/2001/XMLSchema#string"},"parameterBindings": {"https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#variable1":"Temperature","https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#variable3":"Temperature","https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#variable2":"Temperature"},"dataBindings": {"https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/workflows/bikes.owl#measure_per_day":["https://wings.dev.disk.isi.edu/wings-portal/export/users/admin/bikes/data/library.owl#bikes-part1.csv"]}}
[R] Error: Could not get run id
[E] Finished: Something when wrong.

Additional context

107e36f303a6dafd64b87b93996eb31d446147fc doesn't fix the problem

the logs say nothing
2022-07-22T19:46:57.708Z | Quering hypothesis: Hypothesis-eyhHQoDP0qhP
2022-07-22T19:47:06.712Z | Quering hypothesis: Hypothesis-eyhHQoDP0qhP
2022-07-22T21:53:07.850Z | Quering hypothesis: Hypothesis-eyhHQoDP0qhP
2022-07-22T21:53:13.399Z | Quering hypothesis: Hypothesis-eyhHQoDP0qhP
2022-07-22T21:53:33.249Z | Quering hypothesis: Hypothesis-eyhHQoDP0qhP
2022-07-22T21:53:57.506Z | Quering hypothesis: Hypothesis-eyhHQoDP0qhP
2022-07-22T21:54:14.472Z | Quering hypothesis: Hypothesis-eyhHQoDP0qhP

DISK is not creating the files

Fixed by #57


Do not merge: 4.x.x

This is a temporary fix until vuex-next goes to 4.0.X instead of rc.1/rc.2 otherwise. Can be installed with:
npm instal https://github.com/1beb/vuex-persistedstate#4.x.x
Will probably resolve itself in a month or so as versions mature.

The peer dependency issue is fixed in v4.0.0-beta.2

The peer dependency issue is fixed in v4.0.0-beta.2


Error when loading fancy zones editor

ℹ Computer information

PowerToys version:  0.23.0
PowerToy Utility: Fancy Zones
Running PowerToys as Admin: Yes
Windows build number: version 1909 build 18363.1082

📝 Provide detailed reproduction steps (if any)
While power toys is running press win + ` to access fancy zones editor
✔ Expected result
Fancy zones editor should open
❌ Actual result
An error message box appears with the following text.
Please report the bug to https://aka.ms/powertoysreportbug
Error loading custom layouts: Could not find a part of the path 'C:\Users\bpruett\AppData\Local\Microsoft\PowerToys\FancyZones\zones-settings.json'.
The editor opens after this but the custom zones are not showing in it. The zones however are still applied
📷 Screenshots

I am also receiving this error. Do you need log files?

@enricogior adding this into 0.25

@crutkas
very unlikely that we will have time for this in 0.25.

@legolasdimir
is the problem still happening?

I just tested and no it's working. I have not actually seen the error since
I reported it
On Mon, Nov 16, 2020, 12:17 AM Enrico Giordani<EMAIL_ADDRESS>wrote:

@legolasdimir https://github.com/legolasdimir
is the problem still happening?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/7163#issuecomment-727786589,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFC3L3K7ZHNMP45MYDXUUGLSQDG2LANCNFSM4SH2ZSFQ
.

@legolasdimir
thanks for the follow up, I'll close the issue, but please reopen it if it happens again.
Thanks.


make conda installs in CI stricter

Description
Contributes to https://github.com/rapidsai/build-planning/issues/106
Proposes specifying the package version in conda install calls that install CI artifacts, to reduce the risk of CI jobs picking up artifacts from other releases.

/merge

Thanks all! 🙏


The app keeps asking for API Key every time the page is refreshed

Hi. Thank you very much for this projects, it's very useful.
Everything works fine, but there seems to be annoying bug - every time I refresh the browser I see the form asking me to provide OpenAI key, even if I provided it before and I already have a working assistant chat.

Any ideas how to prevent it?

We will add a fix for this in the next update

Great, thanks. By the way - wouldn't it be better and safer to store the api key in an .env file?

While you develop the fix, is there a temporary workaround to this ?

@ascensus-mdb @Rohanmrao This has been fixed now


There is no step 28 through 32

Despite its being mentioned with BOLD in the tutorial

There certainly is...

On what part? I have searched every parts and can't find any :(

Searching for 32 only give that sentence https://imgur.com/X1TFth7

Directly above, beginning with
"Launch the Health and Safety Application (which is now FBI) on EmuNAND"
On Fri, Mar 11, 2016, 9:39 PM orangpelupa<EMAIL_ADDRESS>wrote:

On what part? I have searched every parts and can't find any :(
—
Reply to this email directly or view it on GitHub
https://github.com/Plailect/Guide/issues/9#issuecomment-195644536.

This one? It says steps 8 and 9 on me. Maybe the public served a stale version of the page?
https://imgur.com/IV20pkl

Github's mobile site cuts off the very left, try using landscape mode.
On Fri, Mar 11, 2016, 9:48 PM orangpelupa<EMAIL_ADDRESS>wrote:

This one? It says steps 8 and 9 on me. Maybe the public served a stale
version of the page?
https://imgur.com/IV20pkl
—
Reply to this email directly or view it on GitHub
https://github.com/Plailect/Guide/issues/9#issuecomment-195645154.


How to use Custom Fonts?

I tried adding Roboto font.
doc.addFont('Roboto-Regular.ttf', 'roboto', 'normal', 'Identity-H');     doc.setFont('roboto');
But it doesn't seem to work? What is the issue here?
Also, how can I use Google Web Fonts? I read that it should be possible by converting font to base64 format but I couldn't find any working implementation or couldn't get to convert google web fonts to base64 format.
Any ideas?

@nahomtefera Thanks. That made the custom fonts work. But I am having the font spacing issue that you had even after using the font squirrel tool with the settings you showed.
Can you share what logic to apply when changing those settings? Also, what is the different between encoding types?
doc.addFont("Roboto.TTF","Roboto","normal","WinAnsiEncoding");
How to know which encoding type to choose?

https://github.com/sphilee/jsPDF-CustomFonts-support/tree/v2
Can you check this branch?
I'm now fixing this bug.

I followed the above steps and get:
jspdf.customfonts.debug.js:11984 Font does not exist in FileInVFS, import fonts or remove declaration doc.addFont('ocrb_regular.ttf')
are there additional steps to makethis work?

https://github.com/sphilee/jsPDF-CustomFonts-support/issues/16#issuecomment-307294932
@idevN Delete "WinAnsiEncoding" to avoid font spacing issue.

Hi everybody! I included jspdf.customfonts.min.js, default_vfs.js and<EMAIL_ADDRESS>in order to use an specific ttf from my repository (OpenSans-Regular.ttf) which has '€', but pdf stills apppearing 'þÿ'.
doc = new jsPDF();
doc.addFont('{{asset('bundles/inspinia/fonts/OpenSans/OpenSans-Regular.ttf')}}', 'OpenSans-Regular', 'regular');
doc.setFont('OpenSans-Regular');

Any clue about what I'm doing wrong.
Thank you!


Not working.

it doesn't work. The map is always opened and a few seconds later the google maps map is opened, not the one of xamarin. How can I avoid opening the google maps and only see the map of xamarin ?? I followed all the steps in the documentation.
[Enter feedback here]

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

ID: d967c4fc-27ef-2712-f902-db53cef7f5dc
Version Independent ID: ee85bf33-2ce2-fc75-3806-50167b47a1f5
Content: Xamarin.Forms Map Pins - Xamarin
Content Source: docs/xamarin-forms/user-interface/map/pins.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @arsoftcr
If you look at the sample that the page links to, you'll see that native maps are used on each platform, which means that Google Maps are used on Android.


Relocate some yaml files to the proper directory (1.11)

Fix mislocation of yaml files happened in #8635 on release-1.11 branch.
Closes #10218
cf. patch for release-1.10: #10219
Note: this is not on master any more due to code->codenew work (#9363)

/assign @zacharysarah


added check for  [bins] argument in fft method of P5,  along with unit Test( in es6 style)

issue :

the bins arguments is very vulnerable , as there is no check to what user provides as  the input to bins argument  , suppose the user enters a negative value of bins by mistake , then it's a crash !
so there must be a check to what user gives input as  "bins" arguments"
solution :

added a filter (safeBins) to what user enters as argument .

checks are esablished as shown

added a unit test for the function

@therewasaguy
please  review this Pr!

Also, just a note:
In the Web Audio API, the AnalyserNode.frequencyBinCount is Read only (I'm looking at the MDN documentation for reference https://developer.mozilla.org/en-US/docs/Web/API/AnalyserNode). It is always half of the fftSize. So we're kind of doing things backwards in p5.Sound by allowing users to set the binSize directly. We're doing that because it seemed like it would be more intuitive to set a value that represents the array size you get from the analysis, rather than a value that is twice that amount... but I'm having second thoughts about that now.

yeah ! you are right ! but it is very intuitive and generous to provide user the flexibilty  to choose length of array in which analysed data will be served to them !, and we are no where assigning the value to "AnalyserNode.frequencyBinCount" that means it is good to use ,
infact we are creating a level of abstraction that provides ease to user !, so we don't need to remove that ,however we can rename it , name other than bins because it seem to  represent "AnalyserNode.frequencyBinCount"
how about name "length" which represents the length of array !

I suggest testing the public methods, and making this a private method.
i was actually thinking the same , means i need to create a wrapper for this method ? or you have any other  suggestions?

Do you think the functionality of safeBins could be incorporated into safeBufferSize, rather than making it a separate method?

we can create a safeCheck function which can be placed inside of both of these methods and
rather writing the same check twice , also safeBufferSize have other checks than  safeBIns has

Also ,
https://github.com/processing/p5.js-sound/pull/419#issuecomment-603586872
i agree to the w3 specs and not giving our user the ability to change the bufferSIZe() and hence we don't need to add tests for it like in safeBins !!

i would like to draft this one until the module system work is done !


Problem with the program not creating config files for modules

I was running the pflood module with the following line: java - jar hydrazine.jar -h localhost -m pflood -c
and it led me to the configuration thing where i put in the info, once i submitted it I got this error:

I have the same problem
is there a fix?

@xTACTIXzZ

If you are trying to run it on Desktop or in Documents, it may not work because of onedrive. Try in the Downloads folder, or in any other folder that is not synced with onedrive.

The thing is, that I already am running it from downloads folder...

Have you tried using a command promt as an administrator?

No but I can't do that anyways. I am not on administrator account


Send video

When I tried to send video, it cannot work.
Code
    if msg.text() == 'ding':
        file_box_3 = FileBox.from_file('51961638877671026199.mp4')
        print(file_box_3)
        await msg.say(file_box_3)

Error info
<wechaty_puppet.file_box.file_box.FileBox object at 0x7f3737263d50>
2021-08-05 10:04:06,614 - Message - INFO - say() <<wechaty_puppet.file_box.file_box.FileBox object at 0x7f3737263d50>>
Exception in callback AsyncIOEventEmitter._emit_run.<locals>._callback(<Task finishe...ertionError()>) at /home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/pyee/_asyncio.py:55
handle: <Handle AsyncIOEventEmitter._emit_run.<locals>._callback(<Task finishe...ertionError()>) at /home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/pyee/_asyncio.py:55>
Traceback (most recent call last):
  File "/home/ubuntu/miniconda3/envs/p2/lib/python3.7/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
  File "/home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/pyee/_asyncio.py", line 62, in _callback
    self.emit('error', exc)
  File "/home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/wechaty/wechaty.py", line 284, in emit
    super().emit(event, *args, **kwargs)
  File "/home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/pyee/_base.py", line 116, in emit
    self._emit_handle_potential_error(event, args[0] if args else None)
  File "/home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/pyee/_base.py", line 86, in _emit_handle_potential_error
    raise error
  File "wechatyrun.py", line 243, in on_message
    await msg.say(file_box_3)
  File "/home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/wechaty/user/message.py", line 175, in say
    await message.ready()
  File "/home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/wechaty/user/message.py", line 465, in ready
    self.payload = await self.puppet.message_payload(self.message_id)
  File "/home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/wechaty_puppet_service/puppet.py", line 428, in message_payload
    response = await self.puppet_stub.message_payload(id=message_id)
  File "/home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/wechaty_grpc/wechaty/__init__.py", line 322, in message_payload
    "/wechaty.Puppet/MessagePayload", request, puppet.MessagePayloadResponse
  File "/home/ubuntu/miniconda3/envs/p2/lib/python3.7/site-packages/betterproto/__init__.py", line 1125, in _unary_unary
    assert response is not None
AssertionError

I'm sorry to tell you that wechaty can't send gif image file beacuase of some wechaty structure reasons, and there will be some changes in sending file implementation. please refer to : https://github.com/wechaty/python-wechaty-puppet-service/pull/64#issuecomment-911103851
Please keep eyes on this issue to get latest progress of this problem.


improve test file

The sample.py file that tests the checker is quite ad-hoc (and not comprehensive) in the coverage of constructs. It would be better to replace the file with smaller files, one per unit, that only use the constructs introduced in that unit. A file that uses all Python constructs not allowed in M269 is also needed.

[ ] create a file 02-m269.py that only uses the constructs introduced in unit 2, as defined in allowed.py
[ ] create files 03-m269.py, etc. Don't create files for units that introduce no constructs, like unit 5.
[ ] create a file 33-m269.py that uses some Python language constructs and builtin functions not allowed in M269, i.e. not defined in allowed.py.
[ ] run python allowed.py *m269.py and check that violations are only reported for 33-m269.py

Do we still need these test files? I had writen a few of these, but then started to think they are very specific to M269 and the default configuration. Of course allowed was primary built for M269 (I am assuming) but we now have the ability to customise it through configuration.

You're right. It's better to change sample.py so that it includes more Python constructs in a more concise way. Currently, the file has 'realistic' code, e.g. functions that actually compute something, but that leads to lots of repeated constructs and makes it harder to realise which constructs aren't covered. So the file should be more like
1 + 2 - 3 * 4 // 5 % 6 ^ 7
round(3 / 2, 3)
... # more numeric ops and builtins
True and False or not False
3 < 5 <= 7 > 6 >= 1  # comparisons
if 3 == 5:
    pass
elif 3 != 3:
    print("no")
else:
    ... # test some other construct

By including most (ultimately all) Python constructs in a single file, users could test it against their configuration with different unit numbers. The file only needs to contain all M269 constructs and a couple of other widely used constructs not used in M269, like +=, tuple unpacking, list comprehensions. Other users can then extend sample.py as they need.

@densnow I did have an idea for refactoring the check_tree() function and splitting the message creation and display into a separate class. I've put together a little demo script to show you how it could work:
import ast
from _ast import AugAssign, Call
from typing import Any

STUDENT_CODE = """
animals = ['cat', 'dog', 'fish', 'bird']

def foo():
    for i, animal in enumerate(animals):
        print(i, animal)

animals += ['horse', 'cow', 'pig']
"""

# This is just a simple example - we will categorise constructs further.
DISALLOWED_CONSTRUCTS = ['+=', 'enumerate']

class CLIMessenger:
    """This class will handle all of the command line output and the construction
    of messages that will be displayed to students."""

    def display_message(self, node: ast.AST) -> None:
        print(f"Found disallowed construct: {ast.dump(node)}")

    # We would obviously want to make this more sophisticated, with clearer messages.
    # We could have methods for each type of disallowed construct. We could even
    # consider using a library like rich to make the output more colorful and
    # easier to read. See https://rich.readthedocs.io/en/stable/introduction.html

class ConstructChecker(ast.NodeVisitor):
    """This class will replace the old check_tree function. It will be responsible
    for checking the AST for any disallowed constructs that are passed to its
    constructer. If a disallowed construct is found, the relevant information will
    be passed to the CLIMessenger class to be displayed to the student."""

    # The ast.NodeVisitor class has a generic visit method that will be called
    # for every node in the tree. We can override this method to check for
    # disallowed constructs. We can also override the visit_* methods for
    # specific nodes that we want to check for disallowed constructs.
    # See https://docs.python.org/3/library/ast.html#ast.NodeVisitor

    def __init__(
        self,
        disallowed_constructs: list[str],
        messenger: CLIMessenger
    ) -> None:
        self._disallowed_constructs = disallowed_constructs
        self._messenger = messenger

    def check_tree(self, tree: ast.AST) -> None:
        self.visit(tree)

    def visit_AugAssign(self, node: AugAssign) -> None:
        # We would check if the operator is in the list of disallowed constructs
        # here. For now, I just check if it is an addition for the sake of example.
        if isinstance(node.op, ast.Add):
            self._messenger.display_message(node)
        self.generic_visit(node) # Visit all child nodes

    def visit_Call(self, node: Call) -> None:
        if isinstance(node.func, ast.Name) and node.func.id in self._disallowed_constructs:
            self._messenger.display_message(node)
        self.generic_visit(node)

def main():
    tree = ast.parse(STUDENT_CODE)
    messenger = CLIMessenger()
    checker = ConstructChecker(DISALLOWED_CONSTRUCTS, messenger)
    checker.check_tree(tree)

if __name__ == '__main__':
    main()

Let me know what you think.

To get back on topic re testing...
What I can do is redo the sample file to include all constructs that are allowed in M269, plus some additional commonly-used constructs, but in a more systematic way, i.e. one construct per line with comment, so it's obvious which construct is being tested. Then I could create tests that will compare the ouput of allowed.py against some additional 'expected output' files.
I would have to have an output file that matches the expected output when checking against each unit, with or without the -m tag and for both .py and .ipynb files. So there will likely be quite a few files to check against.
Is that the kind of thing you had in mind?
I could then automate the running of all test cases for all units.

@jurassic-squid @mwermelinger  Do you think it could be worthwhile to test different cases for each configuration?
E.g. with the M269 config have:

a file that includes all the allowed M269 constructs and no more. The expected output would have no warnings.
a file that includes all the allowed M269 constructs plus some commonly used not allowed constructs. The expected output would be warnings from each not allowed construct.
a file that includes non of the M269 constructs plus some commonly used extras. The expected output would be warnings from the extras

I am trying to think in terms of edge cases for allowed and what they are. Can you think of anything else that might be considered an edge case?
There is also now a tm112.json configuration which can be used for testing.
A couple of resources that might be of use here are the exec_tests and eval_tests lists used for testing the ast library. And of course the ast documentation

I think we should still keep a single sample.py test file, but be more systematic in terms of Python 3.10 coverage (we don't have to cover the advanced Python 3.10 stuff like async). One line per construct makes the file too long and it won't be needed once #29 is fixed. The file can be totally meaningless in terms of what it produces, e.g.
3 + 4 - 5 * 6 / 7 // 8 % 2 ** 3
while True:
    for i in range(10, 1, -1):
       if i > 10:
          pass

(The current sample.py is repetitive in what it tests.)
The test script should automatically search for all configurations (find all JSON files with a IMPORTS dict etc.), and for each configuration find which units introduce constructs, then run allowed for each of those units.
As now, the script can be run in generate mode (create the output file) or check mode (compare against existing output).
The test script can be improved independently of the sample file.

This discussion has ranged wide and far beyond the scope of the issue. I'm closing it to open better scoped issues.


Add support for scales in keys

I.e,. transposing the MIDI conversion of the scales. This might simply entail adding another arity for um.scale:midi ...

This is done; just needs to be committed and pushed.


[vendor-patcher] The service "Symplify\PackageBuilder\Parameter\ParameterProvider" has a dependency on a non-existent service "Symfony\Component\DependencyInjection\ContainerInterface".

I'm getting this error when trying to run vendor/bin/vendor-patches generate
The service "Symplify\PackageBuilder\Parameter\ParameterProvider" has a dependency on a non-existent service "Symfony\Component\DependencyInjection\ContainerInterface".
I didn't create any patches yet, yet still this error comes up. Funny thing is that it worked yesterday and I didn't change much.
PHP Version: 8.1.0RC6
New Laravel Project (composer create-project laravel/laravel example-app) - ("laravel/framework": "^8.65")
symplify/vendor-patches ^9.4

Also happens with PHP 8.1

Thanks for reporting!
This is probably due to Symfony 6 break in autowiring.
WIP: https://github.com/symplify/symplify/pull/3797

@TomasVotruba Sweet, thanks. As fast as a racing car!


fix: upgrade standard, and a few other dependencies. fix standard nits

:apologetic bump: Sorry to be a pain but any chance someone could look at merging this? I need the package.json update of istanbul-lib-source-maps to be released to fix a build issue.

@sc0ttyd if it's urgent you can just npm i nyc#upgrade-standard I believe. @bcoe just curious, is anything blocking this? it LGTM since it's pretty minor?

@sc0ttyd @JaKXz could you give the release candidate a spin:
npm cache clear; npm i nyc@next
@sc0ttyd should have these updates you need, and I'd love to have some people test it because it has some fancy new features via @addaleax.

@bcoe sure, I'll give it a whirl on Monday when I've access to TC.


Sometimes samples start playing but immediately stop

On both mobile and desktop, and happens arbitrarily. Once in every like 10 tries or so. Does not happen to samples that have already been played, even the once that had this bug occur to. So second click will always play, regardless of whether it errored before. Makes me think it is a howler bug (on 2.0.0-beta14), also happened on beta13.

Might be related to #48.

602howler.min.js:2 Uncaught TypeError: Failed to set the 'buffer' property on 'AudioBufferSourceNode': The provided value is not of type 'AudioBuffer'.i @ howler.min.js:2

Seems to happen mainly on longer samples.

onStop event is emitted when sample is loaded (why though?), which clears the sample from the list of samples that are supposed to be playing. For bigger samples this can lead into the sample cutting off after it has begun playing (not in the latest commit, as I've updated a check for that).

Fixed in 2d45465a35d60a90ec4732590a6c048f5f57b2ff.


CNN + LSTM problem

Hi, I'm trying to build a model for video classification. I'm treating videos as a sequence of images, so the model takes features from pre-trained cnn and passes them to lstm layer.
video  = Input(shape=(None, 256,256,3),name='video_input')
face_encoder = MobileNet(weights=None, include_top=False, input_shape=(256, 256, 3))
face_encoder.load_weights('vgg2_mobilenet.h5')
# add a global spatial average pooling layer
face_encoder.trainable = False
# wrap cnn into Lambda and pass it into TimeDistributed
encoded_frame = TimeDistributed(Lambda(lambda x: face_encoder(x)))(video)
encoded_vid = LSTM(256)(encoded_frame)
outputs = Dense(7, activation='relu')(encoded_vid)
model = Model(inputs=[video],outputs=outputs)

but while running I got a following error message:
2018-04-09 02:04:45 PST encoded_vid = LSTM(256)(encoded_frame)
2018-04-09 02:04:45 PST File "/usr/local/lib/python3.5/site-packages/keras/layers/recurrent.py", line 262, in __call__
2018-04-09 02:04:45 PST return super(Recurrent, self).__call__(inputs, **kwargs)
2018-04-09 02:04:45 PST File "/usr/local/lib/python3.5/site-packages/keras/engine/topology.py", line 552, in __call__
2018-04-09 02:04:45 PST self.assert_input_compatibility(inputs)
2018-04-09 02:04:45 PST File "/usr/local/lib/python3.5/site-packages/keras/engine/topology.py", line 451, in assert_input_compatibility
2018-04-09 02:04:45 PST str(K.ndim(x)))
2018-04-09 02:04:45 PST ValueError: Input 0 is incompatible with layer lstm_1: expected ndim=3, found ndim=5

I have no idea how to fix it, need your help :)
P.S.
I'm also using ImageDataGenerator for fitting, maybe the problem is here?
train_datagen = ImageDataGenerator(
    shear_range=0.2, zoom_range=0.2, horizontal_flip=True)
test_datagen = ImageDataGenerator()
train_generator = train_datagen.flow_from_directory(
    train_data_dir,
    target_size=(img_height, img_width), )

validation_generator = test_datagen.flow_from_directory(
    validation_data_dir,
    target_size=(img_height, img_width), )

Thank you!

[X] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

[X] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[X] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

You have a shape mismatch, not an issue with Keras. To get a better answer try StackOverflow. In short, the tensor returned from face_encoder is not (samples, timesteps, features). You can print(face_encoder) to see what shape the symbolic tensor is, you know it is 5 dimensional from the error message. And you don't need lambda layer as models can be used as layers already.


Old API version (V2)

Although the API's are working fine but V3 parameters are not supported by this package. Inside the package I found that it uses V2 version of woocommerce API's

I think it would be a great addition if we can make the version configurable as well. Any pull requests are welcome.


added Oid type wrapper around primarray

hooray implementation hiding!

actually, hold off on this until I've ported some utilities into this module as well (to/from String/ShortText, whatever I find as I work my way through the stack)

Cool. Just merge this whenever you're ready. It looks good.


Set connection timeout

Is there an option to set connection timeout?
Currently it seems the timeout is hard coded:
https://github.com/tidwall/SwiftWebSocket/blob/master/Source/WebSocket.swift#L197

Currently it's a hardcoded value.


Install dependency libs on Ubuntu when install polybar

Since getdeb died, here is my command to install dependency libraries to build polybar from source
sudo apt-get install python-xcbgen linxcb-ewmh-dev libxcb-icccm4-dev libxcb1-dev \
xcb-proto libxcb-util-dev libxcb-image0-dev libxcb-randr0-dev ibxcb-xkb-dev \
libalsaplayer-dev wireless-tools libcurlpp-dev  libcairo2-dev

Please write a script to install these libs or write the command in the Install section of the README file.

Which version of Ubuntu is this for?

There are already instructions on the Wiki, most of the packages you listed are also listed there.
The only differences are wireless-tools, libcurlpp-dev and libalsaplayer-dev.
I have checked those packages on Ubuntu 17.04 and it seems they are not really necessary
wireless-tools only provides a few executables but no header files that polybar uses. The package to use is libiw-dev which provides the proper header files and also has the same dependencies as wireless-tools
As for libcurlpp-dev, I don't see how polybar uses this. Polybar uses the the standard curl headers provided by libcurl4-openssl-dev. The same goes for libalsaplayer-dev.
As already mentioned, those checks were made on Ubuntu 17.04, so it is possible that on other versions those three packages are actually needed, can you check if you are still able to compile without those thre packages.


WIP: Containerfile for Konflux

Description
Containerfile for Konflux
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/hold

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.46%. Comparing base (32c2c70) to head (b5af0fd).
Report is 6 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1163   +/-   ##
=======================================
  Coverage   95.46%   95.46%
=======================================
  Files          56       56
  Lines        2380     2380
=======================================
  Hits         2272     2272
  Misses        108      108

/close


[WIP] upgrade mason, downgrade lua to 5.2.4

Solves #250

Current coverage is 95.00% (diff: 100%)

No coverage report found for master at 27cfbd8.

Powered by Codecov. Last update 27cfbd8...824e43c


fix MySQL replication status aquisition bug

WHY
mackerel-plugin-mysql fails on replica status aquisition (MySQL major version is not related)
https://github.com/mackerelio/mackerel-plugin-mysql/issues/7
WHAT
fix replica status aquisition bug
NOTE

show slave status returns only 1 row (which has multiple columns) but former implementation assume it is multiple rows like show status query
there isn't test for replica metric (also for this test) but I confirmed the test.sh on my localhost

taka-h@H37R9YYYGV mackerel-plugin-mysql % bash test.sh
cba1da4cf1213b79f8bdd5cc9e394c2e55f7ef2324b23c656cfd7d4e94125594
Connecting 1...
test-mackerel-plugin-mysql
test-mackerel-plugin-mysql
taka-h@H37R9YYYGV mackerel-plugin-mysql % echo $?
0

@yseto would you please take a look when you have time?

I will review this pull request ASAP.

@takaidohigasi Thanks! We will release new version in the next few days.


CV Restyled

I have redesigned the CV to have some options that I have in my pdf CV.
I'm not too inspired to include more example data, if anyone wants to add data, go ahead.
With my data the CV looks like this: https://jmrplens.github.io/cv/.
My data: VIEW FILE
Some screenshots:

Mobile:

I have a small bug in the markers in the table of contents list.
I'll try to solve it soon, if someone is clear about it, please propose a solution ;)
iPhone 12 screenshot in chrome:

LGTM, but I would like opinion from more maintainers. @alshedivat, @pourmand1376, @rohandebsarkar

@jmrplens thanks for the nice update. I used your code in my site and it looks like this,

The new design looks good to me. Thank you for your collaboration.
Let's see what others say.

@jmrplens anyway to make the table of contents horizontal?? When there are many contents it leaves the right side empty which looks a bit odd. I think having an option for a horizontal table of contents would be nice.

@awsaf49 I agree with you. Can you open another issue regarding this matter? If you could also submit a PR would be great.

@george-gca I will open a issue but not sure about PR as Web Dev is not my expertise. Perhaps @jmrplens  could help...

@awsaf49 I am also not a web dev. But sometimes you can understand how some other things are done on the code and just kind of replicate them elsewhere. Also stackoverflow is really helpful about jekyll liquid stuff.


Include module in demangled name in opt remarks

This allows filtering noisy generic specialization and inlining remarks for
stdlib functions based on the module name.

@swift-ci Please test and merge


Support OpenStack provider

This PR adds support OpenStack provider. Currently this provider supports the following resources:

Compute

compute_instance_v2

Networking

networking_secgroup_v2
networking_secgroup_rule_v2

@yukirii Thank you!


Move the location of the "Build" button after "Targets in Preset"

Brief Issue Summary
I think that placing "Build" after "Targets in Preset" will be a more logical layout:

What do you think?
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

I'm thinking of a new order:

Active folder
Configure preset
Build preset
Default Build Target
Build
Launch Target
Debug
Launch

I'm not sure about the Test Preset location.
Also in CMake 3.25.0 (Presets V6) we get:

The cmake-presets(7) schema version has been bumped to 6.
The cmake-presets(7) format now supports a packagePresets field to specify presets for cpack --preset.
The cmake-presets(7) format now supports a workflowPresets field to specify presets for cmake --workflow.

What about those? I feel that the status bar being used as a toolbar is becoming more and more inadequate.

The status bar for Kits has the build button on the left of the target.  I don't think we want to change it.
If it helps, we want to move stuff out of the status bar anyway and put it in the side panel with the project outline.  #1582

(I would prefer if the stuff gets moved to the side panel.)
Yet let me suggest my new "order' for the new layout too. It also would make sense to have different layouts for different contexts, which are optimized for the specific workflow.

Can you add your suggestion in https://github.com/microsoft/vscode-cmake-tools/issues/1582 , when we have a design for it, we will share our proposal .


Inference speed between the self trained model and the model provided by model_zoo

@lukemelas  Hi, Luke,
Thanks for your great work! I have written a training script to train my own model EfficientNet-B0 (based on the pre-trained model on ImageNet). The only modification to the model is that I changed the last fc layer (i.e. the number of classes). And then I tested the inference time using the self trained model, the results are:
CPU time cost: 0.772s
GPU time cost: 0.021s

And you can see that the model is so slow on CPU. Then I tested the  inference time with /examples/simple/example.ipynb you provided, the results are:
CPU time cost: 0.162s

Do you know why there is a big gap between the time cost on CPU of these two models?
Thanks a lot !

Hi @liuqk3,
I'm not sure what could be causing the difference in inference times. Have you managed to resolve the issue?

@lukemelas Yes, I have solved this issue.
It is becasue there are too many denomal digits (the digits whose magnitude is less than 1e-32) in myself trained model, and the computation of denormal digits in CPU is much expensive than normal digits. See this  and this for more info. about this phenomena.
I statistics on myself trained and ImageNet pretained models are as follows:
                    self trained    ImageNet pretrained
number of denormal:  0                   0
number of denormal:  169                 0
number of denormal:  0                   0
number of denormal:  1920                0
number of denormal:  401                 0
number of denormal:  28960               0
number of denormal:  31760               0
number of denormal:  16640               0
number of denormal:  68873               0
number of denormal:  71236               0
number of denormal:  53321               0
number of denormal:  214271              0
number of denormal:  215808              0
number of denormal:  218688              0
number of denormal:  188933              0

After the model para. is loaded, I set denomal digits to 0. The inference time can be optimized 0.772s-->0.160s.
The PyTorch has also provided a function to avoid denormal digits torch.set_flush_denormal(True), then the inference time is optimized to 0.160s-->0.111s.

Great to hear you solved the issue! Interesting, I didn't know that about the denormal digits thing, thanks for linking those threads.
I'm closing this issue as you've solved it.


[esco-content-grid] favorited portlets call addFavorite when trying to remove favorite

Describe the bug
When clicking the star on favorited portlet(s) to un-favorite it, the components calls addFavorite instead of removeFavorite
To Reproduce
Steps to reproduce the behavior:

Load a portlet containing an esco-content-grid component such as the admin tools portlet in uPortal-start
Favorite a portlet
Reload page and open browser's dev tools
Attempt to un-favorite portlet

Expected behavior
Clicking the star of a favorited portlet will result in

removeFavorite being called
the star to change from solid to outline

which version of the component are you using ?
I can't reproduce it inside the esco-content-menu.

Ok from the default deployement (master branch of uPortal-start) I reproduce the problem, but not with the esco-hamburger-menu (that's why I wasn't able to reproduce it).


Missing some necessary code under the taming folder

Dear authors, is there some operating code missing during the test and training stage? it seems that the lack of VQVAE (or something else) in the taming folder.

Problem solved with the new update


Support a Team as a valid contact entry

It was always the case that we could have a team but we were not explicit about in the schema docs. As well, our app templates added some new fields which would be valuable to document.
Related: https://github.com/telus/platform-cmd/pull/7
To note, semantic-release is removed in this PR. Most linting was also removed or replaced with standardjs. There is a significant decrease in packages used. In a follow up PR, will be adding back our new standard for deploying npm packages.

Dry run fails, but all other tests pass. Will sort this out in the followup pr


Canis Minor not defined?

I've been trying to create a Map Plot of Canis Minor. When I do:
test = Constellation.get(name="Canis Minor")
print(test)

I get the output 'None'.
If I try the same for, say, Canis Major (or any other constellation I've tried), I get the correct output, i.e.
Constellation(iau_id=cma, name=Canis Major, ra=6.95, dec=-23.18)
This made me suspect that perhaps Canis Minor has been accidentally omitted. (Unless, of course, I'm doing something silly.)
Environment Details

Starplot version: v0.12.2
Python version:
OS: macOS

ah, thanks for reporting this. Looks like there's a small bug in preparing the constellations reference file: it uses the constellation name as-is from here (which adds a new line char for some names to help with zenith plots) -- so I just need to remove those new lines before writing the name to the ref file.
Should have a fix out soon 👍


TZ localization in date picker should be optional

We have an application which operates solely on UTC time despite of the fact that we have works in PST as well as EST. We are running into issues where we really just want the date picker to show the date in UTC mode rather than localizing it by default.
I am proposing that we offer this as a bindable configuration at the date picker level and pump the configuration into the ng-date-filter. I have a separate ticket to represent the changes needed at the date filter.

+1

Any update?

Just noting, I have a current work in progress where the datepicker defaults to the timezone of the date bound to the model initially - I should be able to have this ready by the end of the week.

Currently, one can set an initial model value of the desired timezone and that timezone will be used throughout the datepicker.
If one wants to dynamically change it, that still needs work, but is also captured in other issues.


Notifications with Sound error when app is open

2015-03-19 11:59:22.960 MappSkeletor[5166:2285775] Notification received
2015-03-19 11:59:22.961 MappSkeletor[5166:2285775] Msg: {"alert":"This notification has been sent from the SERVER","badge":"11","sound":"www/application/communicator.wav","ejson":"{\"notificationId\":\"There Should Be An Id Here\"}","messageFrom":"push",foreground:"1"}
2015-03-19 11:59:22.972 MappSkeletor[5166:2285775] Uncaught Error: TypeError: Object is not a constructor (evaluating 'new Media(notification.sound)'):366:http://meteor.local/packages/raix_push.js?1331304cc9edac408362af29980480e0569c0980

Offending line:
https://github.com/raix/push/blob/05d1bab8eb7a57c12472af98d5396d7dbeffcf1b/lib/client/cordova.js#L172
Looking to debug further.

OK, so - has a collection called 'Media' - that's this error.  Still ran into issues once that was resolved.
Next error in line was that Media is undefined.
Had to put an if else around that line to check for the different browser cases.


Will the data be released or requested?

Very much looking forward to it, we are doing something similar, can you ask further questions about the data please ?

All relevant data is accessible via the Internet, including official train/test splits. However, due to storage limitations, I am unable to upload some of the larger datasets to cloud storage. Additionally, accessing certain datasets on PhysioNet requires certification, but they become available once the certification exam is passed.
Could you please specify the types or formats of data you are seeking? I am not fully understand your request. Are you referring to the post-preprocessing .tsv files?
By the way, if you have any questions regarding the data, please feel free to ask.


Regression in multipart parsing

After switching from Debian Stretch to Buster (ie. from 2.6.4 to 2.7.1) on some of our company servers we noticed one of our applications corrupts encoding in multipart messages. When investigating, git bisect showed commit 21222e1b48f08f6f848d26643d6b6fb1d873d18c (Eliminate attachment corruption caused by line ending conversions) as a culprit.
Consider this example message:
To: me@email
Subject: email
From: someone@email
Content-Type: multipart/alternative;
  boundary="b1_7d9a9542dac6553ec140c72503fdec38"
Content-Transfer-Encoding: 8bit
--b1_7d9a9542dac6553ec140c72503fdec38
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
UTF-8 text ě
--b1_7d9a9542dac6553ec140c72503fdec38
Content-Type: text/html; charset=utf-8
Content-Transfer-Encoding: 8bit
<body>UTF-8 html ě</body>
--b1_7d9a9542dac6553ec140c72503fdec38--
Example script:
require 'mail'
mail = Mail.read_from_string(
File.open('mail-multipart.txt', 'r:binary') do
|h| h.readlines.join
end
)
puts mail.parts.length # Output is "1"
Our application serves as a bridge between PHP apps (which think they are talking to sendmail) and the actual sendmail. It parses the e-mail using mail gem and performs some checks before forwarding it. Message is thus received from standard input, which needs to be set to binary encoding as the PHP apps themselves may pass data in whichever encoding their authors choose (making anything except binary encoding on Ruby side potentially invalid.)
Mail gem defaults to quoted-printable and converts message body accordingly. However - since both parts are parsed and considered as one, ie. second part is considered to be a body of the first part - part headers are converted as well, which results in this:
Content-Type: text/html; charset=3Dutf-8 (note the "=" character converted into "=3D")
Unsuprisingly, HTML version of the message is not showing correctly in mail clients after that.
Since this regression manifested after eliminating LF into CRLF conversion, my quick-fix was an alteration of the regular expression "parts_regex" in extract_parts method in mail/body.rb - changed "\r\n" into "\r?\n". To my knowledge this should not break anything, so I am posting this change for consideration.
(Sorry for absence of formatting, the editor decided to not cooperate for some reason.)

Thanks for your thorough reporting and diagnosis, @tlghacc.

alteration of the regular expression "parts_regex" in extract_parts method in mail/body.rb - changed "\r\n" into "\r?\n".

This is implemented in #1319 👍🏼

This is implemented in #1319

Oh, didn't spot that one, thank you for pointing it out.


Set lines for "Allowed Roles" in pv-inverter to active

Hello,
as written in https://github.com/freakent/dbus-mqtt-devices/issues/23 the PR activates the "allowed roles" lines for the pv-inverter to change it's position between AC-IN and AC-OUT:

This is necessary for easier integration of Hoymiles inverters to push there values with the dbus-mqqt-devices driver automatically from Open-DTU to VRM portal. See also PR https://github.com/tbnobody/OpenDTU/pull/377
Thanks,
Peter

Hi Peter, my apologies for not seeing your pull request earlier. I'll review and merge asap if all looks OK.

Hi Peter, my apologies for not seeing your pull request earlier. I'll review and merge asap if all looks OK.

is it possible ti have it merged?

The changes proposed in this PR have already been reflected in the new beta release. Generally I am not comfortable merging changes that I have no tests for. However since you have verified the changes proposed in the PR I have rolled this into the main branch.


[BUG]: TypeError: Do not know how to serialize a BigInt

What version of drizzle-orm are you using?
0.30.10
What version of drizzle-kit are you using?
0.21.4
Describe the Bug
Putting here in case it is of help:
When using Drizzle with fastify/view (8.2.0) and nest js (10), bigserials have a mode of bigint which croaks due to json stringify (see this fastify issue and stringify
The following was created by drizzle-kit with a postgres schema that has id as a bigserial and it will croak with TypeError: Do not know how to serialize a BigInt
export const users = pgTable(
  'users',
  {
    id: bigserial('id',{mode: 'bigint'}).primaryKey().notNull(),
    username: varchar('username').notNull(),
[..]

The quick solution is to change the mode from bigint to number.
   id: bigserial('id',{mode: 'number'}).primaryKey().notNull(),

I'm not sure if it is perhaps a good idea to either document or set the default drizzle-kit output to mode: number
I note that when drizzle-kit sees a bigint column that is not (big)serial, then it sets the mode to number by default and provides a comment.
So perhaps this could apply to bigserial too (since these two things are linked in the database: a bigserial id may have a bigint foreign key somewhere.
DRIZZLE-KIT output from introspect:
  // You can use { mode: "bigint" } if numbers are exceeding js number limitations
  user_id: bigint('user_id', { mode: 'number' })
    .notNull()
    .references(() => users.id, { onDelete: 'cascade', onUpdate: 'cascade' }),

Expected behavior
No response
Environment & setup
"dependencies": {
    "@fastify/static": "^7.0.4",
    "@fastify/view": "^8.2.0",
    "@knaadh/nestjs-drizzle-postgres": "^1.0.0",
    "@nestjs/common": "^10.3.8",
    "@nestjs/config": "^3.2.2",
    "@nestjs/mapped-types": "*",
    "@nestjs/platform-fastify": "^10.3.8",
    "@nestjs/serve-static": "^4.0.2",
    "@nestjs/swagger": "^7.3.1",
    "class-transformer": "^0.5.1",
    "class-validator": "^0.14.1",
    "dotenv": "^16.4.5",
    "drizzle-orm": "^0.30.10",
    "handlebars": "^4.7.8",
    "js-yaml": "^4.1.0",
    "postgres": "^3.4.4",
    "reflect-metadata": "^0.2.2",
    "rxjs": "^7.8.1",
  },
  "devDependencies": {
    "@fastify/static": "^7.0.4",
    "@fastify/view": "^8.2.0",
    "@knaadh/nestjs-drizzle-postgres": "^1.0.0",
    "@nestjs/cli": "^10.3.2",
    "@nestjs/common": "^10.3.8",
    "@nestjs/config": "^3.2.2",
    "@nestjs/core": "^10.3.8",
    "@nestjs/platform-fastify": "^10.3.8",
    "@nestjs/schematics": "^10.1.1",
    "@nestjs/serve-static": "^4.0.2",
    "@nestjs/swagger": "^7.3.1",
    "@nestjs/testing": "^10.3.8",
    "@swc/cli": "^0.3.12",
    "@swc/core": "^1.5.7",
    "@types/express": "^4.17.21",
    "@types/jest": "^29.5.12",
    "@types/js-yaml": "^4.0.9",
    "@types/node": "^20.12.12",
    "@types/supertest": "^6.0.2",
    "@typescript-eslint/eslint-plugin": "^6.21.0",
    "@typescript-eslint/parser": "^6.21.0",
    "class-transformer": "^0.5.1",
    "class-validator": "^0.14.1",
    "drizzle-kit": "^0.21.4",
    "drizzle-orm": "^0.30.10",
    "globals": "^15.3.0",
    "handlebars": "^4.7.8",
    "jest": "^29.7.0",
    "postgres": "^3.4.4",
    "prettier": "^3.2.5",
    "reflect-metadata": "^0.2.2",
    "source-map-support": "^0.5.21",
    "supertest": "^6.3.4",
    "ts-jest": "^29.1.3",
    "ts-loader": "^9.5.1",
    "ts-node": "^10.9.2",
    "tsconfig-paths": "^4.2.0",
    "tsx": "^4.11.0",
    "typescript": "^5.1.3"
  },

This is not just a fastify issue. Using bigints for the id columns makes queries error in node <=20.

The issue here is not with drizzle. Drizzle returns the correct datatype for the column as specified in the schema. The issue is when you want to send it via fastify to the client, and it doesn't only affect fastify, express too would be affected. You can serialize the data yourself before sending it to the client. See options here

Perhaps. But if you notice my post you will see that foreign key references are set the mode to number even if the referenced column is a bigserial/bigint.
It would work out of the box if drizzle was consistent and set the mode to number for bigserial during column definition and provided the same comment as it does for columns with foreign key references.
Up to you and how you want to interact with the user community. After an hour I learnt how to fix it on my system.