Update regrid.pl for GOCART2G

Per @pcolarco and @mmanyin in email, there is a desire to regrid the new GOCART2G restarts. Looking at a set of them, the (possible) new files seem to be:

achem_internal
cabc_internal
cabr_internal
caoc_internal
du_internal
hemco_internal
ni_internal
ss_internal
su_internal

Now, normally we could just add these to The List™ that's in regrid.pl but it's not that simple.
I looked at these restarts with @bena-nasa and we noticed that:

hemco_internal is a 2D restart (no levels)
du_internal and ss_internal are 4D restarts (ungridded dims)

The main issue is the underlying regridding code was not set up for files like these.
So, I suppose the first question for @pcolarco or @amdasilva or @christophkeller is: Do we need to worry about hemco_internal? If not, we just just always bootstrap it and focus on the 4d restarts?

CC @weiyuan-jiang, @tclune, @aoloso in re the regrid refactoring effort

Adding the ability for interp_restarts.x to handle these new cases an ungridded dimension + level is straightforward from a scientific perspective (just treat each index of the ungridded dimension as a 3-D variable in it's own right and do the usual horizontal and vertical regridding. However, the code has gotten to the point that to add new capabilities like this calls for a refactoring. At least splitting out the binary and netcdf into separate codes so I can focus on adding these capabilities to the netcdf restarts without breaking the binary path. Splitting this apart and adding new capabilities is a non-trivial exercise so lead time and dedicated time to do this will be necessary.

It had been a long time since I looked at the code; I split interp_restarts.x into a binary and netcdf code. Adding support for these 4-D variables is actually more straightforward than I thought. Once I have set of restarts provided that I can use to test I can make this change.

@mmanyin @pcolarco @gmao-jstassi @mathomp4
I've updated interp_restart.x to handle these new restarts (as well as refactoring and spliting interp_restart.x into a separate binary and netcdf version to make my life easier going forward). I've confirmed it works with the new gocart2g restarts that are either 2d only or 4d with the unknown_dim + level.
The issue now is regrid.pl itself. It needs to know about the restart names but I think we are getting to the point where it needs some more flexibility. Since each species in gocart and each instance can have it's own restart like
cabc_internal_rst
cabr_internal_rst
caoc_internal_rst
adding these explicitly in regrid.pl seems problematic. What is somone else has other instances of ca for example?
It seems like regrid.pl needs to be able to do some sort of wild carding. Like instead of adding these explicitly, you had wild card like this:
ca*_internal_rst
and it finds any restarts of that form.
Pinging Joe in this as my perl is shaky to do this.

Does this only affect the input name ? Is there any special option?  For the future, the python regrider only cares about the names that are listed in the yaml file.

Sounds like good progress.  Will it eventually be possible to regrid from the old GOCART 1G format to the GOCART 2G?

I would second @mmanyin. I think we would want to be able to regrid to GOCART2G from MERRA-2 for instance.

Sounds like good progress. Will it eventually be possible to regrid from the old GOCART 1G format to the GOCART 2G?

@mmanyin Can you elaborate what that would entail or what that even means?  Is it just a matter of splitting an old gocart 1g restart into separate restarts?
That's beyond the scope of what the underlying regridding code (interp_restarts.x) would handle, it just regrids what is there already.
It would have to be some other script but someone who understands this would have to write it or give me a precise recipe for what that operation would entail.

I would second @mmanyin. I think we would want to be able to regrid to GOCART2G from MERRA-2 for instance.
@pcolarco what does this mean? Would every field in a gocart 1g restart (merra2 or otherwise) go to a specific field in a specific gocart2 split restart? If not, I'm not sure what this operation of going from GOCART1G to GOCART2G means.

As I said, I will not support this in interp_restarts.x, that's outside the scope of that program. If is as simple as spliting the fields into separate restarts that is another programs job,  it would be a trivial python script.
As far as MERRA2 This is a can of worms. Regridding directly from MERRA2 to GOCART2G is complicated by the fast that MERRA2 was binary, so there is no metadata in the file. I painstakingly figured out the order of the fields in the MERRA2 binary restarts and wrote a converter here:
https://github.com/bena-nasa/GEOS5_restart_converter
If you want to do something with MERRA2 in this hypothetical GOCART1G to GOCART2G operation, you would need to use my tool to convert this to NetCDF then, if there is a solution to go from GOCART1G to GOCART2G with a NetCDF file use that.

I understand. A boy can dream.
Splitting a netcdf legacy GOCART to GOCART2G should be straightforward with NCO or some other tool.

@bena-nasa  I have been impressed in the past when needing to convert legacy restarts to a more recent version, that regrid.pl could identify what was missing and provide at least a bare bones set of restarts.  Without really knowing what the limitations are, I was posing the general question -- can the program generate a set of G2G restarts that approximates an older G1G set?  Sounds like this is not the tool for doing it.

@bena-nasa I have been impressed in the past when needing to convert legacy restarts to a more recent version, that regrid.pl could identify what was missing and provide at least a bare bones set of restarts. Without really knowing what the limitations are, I was posing the general question -- can the program generate a set of G2G restarts that approximates an older G1G set? Sounds like this is not the tool for doing it.

No, regrid.pl just calls other program that regrid the restarts that are there using the boundary conditions it thinks are appropriate based on your answers to the questions; no more, no less. If you have a set of gocart_internal_rst in, you get one out.
After I fix up regrid.pl, if you have a set of restarts from gocart 2g in you get a set out.
@mmanyin @pcolarco This does bring up point, what are all the "base" component names it need to be aware of. I'm going to try to implement a wild card feature in regrid.pl
So looking in the restarts pete provided we have
ca,ss,du,ni,su
and these could potentially have multiple instances (in the restarts I have only ca actually does) that I will use the wildcard feature to find.
What am I missing if any? I'll code it to the list above so please let me know if I need to include others.

@bena-nasa Your list looks complete for GOCART2G, although note it is "cabr," "cabc," and "caoc". You might anticipate an eventual refactoring of the remainder of the legacy GOCART which would split what is still left in gocart_interal_rst into subsequent things like co, co2, ch4, ... _interal_rst.

@bena-nasa Your list looks complete for GOCART2G, although note it is "cabr," "cabc," and "caoc". You might anticipate an eventual refactoring of the remainder of the legacy GOCART which would split what is still left in gocart_interal_rst into subsequent things like co, co2, ch4, ... _interal_rst.

@pcolarco Oh, did  misread that? I thought, cabr, cabc, caoc were separate instances of one species, but now I'm reading again, that is just brown carbon, black carbon, and organic carbon. So per species there is only one restart? Do we have to worry about a case when a single component has multiple instances. If not then I can just hard code the names and I was making a problem out of nothing.

@bena-nasa Hmm... Each instance has its own restart. But default we are running three carbonaceous instances: brown, black, and organic carbon. We need to be able to regrid each such instance. Some guidance then for how to handle multiple instances for later (i.e., so far not tried out) cases will be helpful. Does that make sense?

@pcolarco @jstassi I was only asking about the instances because currently the way regrid.pl works is that is has hard coded restart names it looks for. So if the name is not in the list, it won't regrid it. So this could be a problem if someone runs a new gocart case with multiple instances for example and wants to regrid those but the script is unaware.
I see two solutions possible solutions for this in regrid.pl

Allow the user to specify "extra" restarts that the program is unaware of on the command line or when answering the questions when running interactively
Implement some wildcard type thing where the program is given something like this ca*_internal_rst and it tries to find all the restarts that match that pattern. This would require the instance pattern to have been added and for the instances to be consistently named (I've never run gocart2g so how it works is a total mystery to me).

It sounds like for now the PR I've made handles the current uses but still needs another extension. Any thoughts on which method sounds better as an end user?

@bena-nasa Hey, Ben, is this functionality available now in some more recent model version for me to try out? Thanks

@pcolarco I think it should be in GEOSgcm v10.21.0 for sure.

Thanks, Matt. I did go check out and see it in the CHANGLOG, so I’ll give it a try.

Peter Colarco
NASA GSFC
Code 614
NASA Goddard Space Flight Center
Greenbelt, MD 20771
301.614.6382 (ph)
301.614.5903 (fax)
<EMAIL_ADDRESS>http://acd-ext.gsfc.nasa.gov/People/Colarco
http://www.researcherid.com/rid/D-8637-2012
From: Matthew Thompson @.>
Reply-To: GEOS-ESM/GMAO_Shared @.>
Date: Thursday, January 13, 2022 at 4:23 PM
To: GEOS-ESM/GMAO_Shared @.>
Cc: Peter Colarco @.>, Mention @.***>
Subject: [EXTERNAL] Re: [GEOS-ESM/GMAO_Shared] Update regrid.pl for GOCART2G (Issue #228)
@pcolarcohttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpcolarco&data=04|01|peter.r.colarco%40nasa.gov|384d5c0d076d4e1f481408d9d6dae82a|7005d45845be48ae8140d43da96dd17b|0|0|637777057950609750|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=GYFupCgFRv8XC%2FlQgjksCn4toJvjeiqXjp4781rEgzw%3D&reserved=0 I think it should be in GEOSgcm v10.21.0 for sure.
—
Reply to this email directly, view it on GitHubhttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FGEOS-ESM%2FGMAO_Shared%2Fissues%2F228%23issuecomment-1012526119&data=04|01|peter.r.colarco%40nasa.gov|384d5c0d076d4e1f481408d9d6dae82a|7005d45845be48ae8140d43da96dd17b|0|0|637777057950609750|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=SwUuh6lSXjRW3fBbE1I69BbIm2LgR4gJ%2FMBNAT2YejA%3D&reserved=0, or unsubscribehttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANW73YARKTWE3AI2GMKBZ6LUV47D7ANCNFSM5HI7R2DQ&data=04|01|peter.r.colarco%40nasa.gov|384d5c0d076d4e1f481408d9d6dae82a|7005d45845be48ae8140d43da96dd17b|0|0|637777057950609750|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=DpoQuJ8CFnDykS4zhy0azsRTcbdTcLlEpOhcENZlTiQ%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|peter.r.colarco%40nasa.gov|384d5c0d076d4e1f481408d9d6dae82a|7005d45845be48ae8140d43da96dd17b|0|0|637777057950609750|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=SBkW7lMnsrY5TxL15rtpNmQpbqrjXuDzM87OonR%2BjlI%3D&reserved=0 or Androidhttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|peter.r.colarco%40nasa.gov|384d5c0d076d4e1f481408d9d6dae82a|7005d45845be48ae8140d43da96dd17b|0|0|637777057950609750|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=7gD5uyZMXBg1J1xkkIG1G7%2Bhlz2hGml2yGXU9%2F8PUIQ%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


Improve the Release status dashboard

I've spent some time making the Release Dashboard closer to my daily status reports (and include the info needed for them). Here's the result.

Outdated static demo: https://encukou.github.io/buildbot-status-dashboard/
WIP repo, with caching and hacks needed to iterate on this locally: https://github.com/encukou/buildbot-status-dashboard
Technically it should be drop-in upgrade for the existing dashboard, and that's what this PR does. But I now think it would be better to have both, at least for a while, in case this falls over. But, code is here.
Does this look reasonable?

Quick branch info (status, RM name, release schedule link) for all branches at the top, with detailed per-branch sections after that.
Details can generally be expanded/collapsed.
Show tier-3+ failures: these aren't considered release-blocking but we tend to fix these ASAP. These mark a "green" branch/builder as "yellow".
Show low-priority problems: disconnected builders, build warnings, unstable builder failures. These don't affect the overall status.
Show changes and test results for failing builds (when available).
Show build history overview for builders; this is useful context.
Conceptually, the branch sections list "problems", not builders, in preparation for adding blocker issues from GitHub. Problems are sorted by severity & grouped by type; the highest-severity problem determines the overall branch status.
Pull Request builders are, for now, shown in a no-branch section (hiding at the end with no link to it).
Based on Pablo's existing dashboard and his BuildbotHammer tool.

Wow Petr, this is absolutely fantastic work! 🎉
I'm really excited to see the release dashboard coming together like this! This is going to make everyone's lives so much easier.
I made a first pass but given how big this PR is I probably missed stuff so I may do one more later.

Do you think it's OK to replace the existing dashboard, or should we leave them side-by-side for a while?

If it's quick to load, IMO it's ok to replace the existing one.

I've been using this for a week, and saw a minor display issue (and many out-of-sync results as the local version needs heavy caching).
Integrating with the live buildbot might be a bit rough, of course.
I plan to merge this around Wednesday (if there are no objections) and iterate from there.

If it's quick to load, IMO it's ok to replace the existing one.

It should be on the same order of magnitude as the current dashboard.
Quite slow, but cached, and re-generated by a cron job every 5 minutes.

I'm currently on vacation for two weeks so unfortunately I was not able to find the time to review this again so feel free to land it if you are happy with it and we can fix problems later :)

Thanks for this great enhancement Petr!

Thanks for the help & reviews!
It's up now and works, knock on wood :)


Some applications don't deregister protocol handlers when uninstalled

Summary
Some applications, e.g. Steam, leave behind their Windows registry records when uninstalled. Therefore user can install an application, then delete it and still have protocol registration which is picked up by the demo. Since the demo produces results which appear as false positives (reporting that an application is installed when it ctually is not), users might dismiss the demo thinking it is inaccurate. It would be nice if you could add a note or a mark (e.g., asterisk) to programs prone to false positives.
Repro steps

Install Steam so that it registers protocol handler in HKEY_CLASSES_ROOT\steam.
Uninstall Steam and note that the registry record is still present.
Run demo and see Steam among "installed" applications.

You should probably file it as a Windows bug.
The demo is designed to detect if the custom protocol is supported by the system. Fancy UI is made to make the demo less boring.

It's not a Windows "bug" that an application can leave registry entries behind on uninstall, that's just how it works.
/Arguably/ a browser could do work to verify that a protocol registration is still valid (by e.g. checking for the target executable before prompting) but this would be error prone for a variety of reasons, so I don't think you're likely to land such a change in browser.

So, the demo isn't "designed to detect if the custom protocol is supported by the system" but rather "designed to detect whether the browser /thinks/ the protocol is supported by something on the system."

Good points. I've rushed to conclusions, sorry.


Misspelling in README.md

Should be module.exports instead exports.exports

@a7IadXE1c9 Thats a good catch, thanks for the PR.


BUG FIX load_flash - use a copy of jobs when cancelling.

Save and load cancel_after using ISO format.
Minor changes to debug logging.
uasyncio replaced by asyncio since Micropython 1.21.

@dcox761 thanks for the PR looks good.


fix: Add missing dev dependency from recent PR

Description
In a recent PR we missed adding a dev dependency for the source-map-support package used in .mocharc.js. This adds it so the e2e tests pipeline works again.

Need it in actual dependency?

Ah, good catch. Updating.


Upgrade to the latest version of the Sierra API

Rather annoyingly, holdings aren't available in the v3 API (which is what we currently use).
The newer versions API have a breaking change: previously IDs were returned as strings, now they're returned as ints. This is going to knock our transformers for six – I think we need to fix our libraries, then reharvest all the Sierra data to use the new format.
This is blocking https://github.com/wellcomecollection/platform/issues/5003

Done!


Fixed #24835 - exists() incorrect after annotation with Count()

https://code.djangoproject.com/ticket/24835
I followed the same approach as in QuerySet._values(). Seems hacky but I couldn't figure out a better way.

Buildbot test this please.

I think the approach you've taken is mostly correct. It's unfortunate that similar processing has to be done in so many places, but that's a task for another time.
I feel like the majority of the work should be performed by db.sql.query.has_results rather than db.query.exists. Is there a technical reason for this?

I second jarshwah's opinion - the logic should be in has_results(), not in exists(). (Seems like the ._values() logic should also be in sql.Query class, but this doesn't need to be solved here). We could even implement a dedicated helper method to be used both from ._values() and has_results().

Please also add 1.8.3 release notes.

I second jarshwah's opinion - the logic should be in has_results(), not in exists().
I moved it to has_results().

Please also add 1.8.3 release notes.
Done.

Could you rebase and squash your commits? Thanks.

Rebased on master and squashed.
As a rule, should I do that with every update or only at the end of review process?

For small patches, squashing after each update is fine. For larger patches, it can be helpful to review only the changes.
buildbot, add to whitelist.

buildbot, add to whitelist.

:shipit: LGTM

I've opened a new ticket for cleaning up the _values() method and potential factoring with has_results here: https://code.djangoproject.com/ticket/24854

merged in 801a84ae329a24d9adf01b700429fe8f1285b2b8, thanks!


Changelog not updated anymore?

Hi!
Changelog is not updated anymore?
From code you have a button directly to the changelog... and it is a bit outdated... 2.3.0....
Thanks!

You can view the main changelog here: https://github.com/pybricks/pybricks-micropython/blob/master/CHANGELOG.md
We do need to fix the changelog of the editor and the links to it, so we'll keep this one open. Thanks!


(Suggestion) Beeping to alert nearby players

This script is amazing. I think it could add a lot to the RP if the bomb played a beeping sound or something every once in a while to alert nearby players of the bomb.

Is not already added?

there is a red flashing light, you can also easily add export to your police dispatch to alert the police for any active bombs, from there the police can alert the city and cut off the area.

Beeping has been added to the script, update.

Beeping has been added to the script, update.


License

Hello.
I am doing my university homework, part of which is to implement some graph layout algorithm on the plane. I found your very interesting article about tsNET algorithm and wanted to try to implement it in my work. But there is no license specified in your repository, so I have no possibility to use your code. Could you provide me with such an opportunity? Or just specify the license in the repository. I will be extremely grateful to you.

Hi there,
Thanks for bringing this to my attention. I have added a license to the repository.
Good luck with your university work!


[bug]: Sidebar icon size is not taken in consideration

Describe the bug
When I set the size=lg to the <SidebarMenuButton>component, only the spacing between rows is updated.
By default, Icons have a size of 16px. In my case, i'm looking for bigger icons like 20 or 24px.
The size=lgdoes not work as expected.
Affected component/components
Sidebar
How to reproduce

relay on the default icon size:

<SidebarMenuItem key={item.title}>
                  <SidebarMenuButton asChild>

                      <item.icon width="8" height="8" />
                      {item.title}

</SidebarMenuButton>

update the size of each SidebarMenuItem:

<SidebarMenuItem key={item.title}>
                  <SidebarMenuButton asChild size="lg">

                      <item.icon width="8" height="8" />
                      {item.title}

</SidebarMenuButton>

Analyse the difference

The icon size are the same
Codesandbox/StackBlitz link
No response
Logs
No response
System Info
macOS 15.0.1
Latest Chrome
NextJS 14
latest Shadcn

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

I hava the save issue

I am also experiencing this issue where i would like to have my collapsed icon bigger, but the button restricts it and even shows a blank area
Collapsed State
Expanded state:

I have the save issue, when I want to change the size

I have the same problem, did you find any solution?

I could only fix it with css.
className="my-menu-item-link"
.my-menu-item-link svg {
    height: 24px !important;
    width: 24px !important;
}

@ledevmacedo @Timmy0o0 @anmoroz
I did some tinkering on the sidebar.tsx and I was able to hard set the size of the collapsed icon state. The part in green was part of my change, so it sets the collapsed size to the exact size I want

Then i used the Avatar component in a brute way to render a png

Also notice the changes to the className of the SidebarMenuButton component, which I added for preference

seems that the svg 's css is overided by gap-2 [&_svg]:pointer-events-none [&_svg]:size-4 [&_svg]:shrink-0 when use  <SidebarMenuButton /> with asChild.
I wrapped  with another
 for workagound, e.g.:
<SidebarMenuButton asChild>

      <item.icon size={24} />

{item.title}

</SidebarMenuButton>

about svg in button, may blame to this

This is the implementation of my collapsible sidebar icon.
Again, please ignore the use of the Avatar component 😅😅
                <SidebarMenuItem className="group/item group-data-[collapsible=icon]:flex group-data-[collapsible=icon]:justify-center">
                        <div
                                className={`absolute top-1/2 -translate-y-1/2 -left-[1px] w-2 rounded-full transition-all duration-100 ${
                                        selected_tab === label.toLocaleLowerCase()
                                                ? "h-10 bg-discord-blue"
                                                : "h-0 group-hover/item:h-5 bg-white opacity-0 group-hover/item:opacity-100"
                                }`}
                                style={{
                                        clipPath: "inset(0 0 0 50%)",
                                }}
                        />
                        <NavLink to={to} onClick={handleClick}>
                                <SidebarMenuButton
                                        tooltip={label}
                                        className="w-full gap-3 group-data-[collapsible=icon]:ps-0 text-base h-fit group-data-[collapsible=icon]:[&>span:last-child]:hidden p-0 ps-3"
                                >
                                        <Avatar className="size-[50px] flex justify-center items-center">
                                                <AvatarImage
                                                        src={`/icons/${icon}.svg`}
                                                        alt={`${label} icon`}
                                                        className="w-[30px]"
                                                />
                                        </Avatar>
                                        {label}
                                </SidebarMenuButton>
                        </NavLink>
                </SidebarMenuItem>

Just in case these previous solutions don't work for someone, what worked for me was overriding the svg size for the  in the className, like so:
<SidebarMenuButton     className='[&>svg]:!size-6'> ... </SidebarMenuButton>
This changed the icon to the desired size, and then I made other adjustments to fit them in a way that worked for me.
Hope it helps someone! :)

The problem is that the icon size is hardcoded like in the Button Components. And the icons that are nested are all with [&_svg]:size-4 I think it is better to remove this from the component itself and decide it in the buttons' icon instead

I fix the problem for the lg size by adding this class group-data-[collapsible=icon]:[&>svg]:ml-2 to sidebarMenuButtonVariants in sidebar.tsx

Is an action is scheduled to upgrade this component to let users managed icon sizes without hacks?

I couldn't solve my problem even using these "hacks", has anyone found an alternative?

Hey @ledevmacedo You could drop snippets of the specific parts of the code; maybe we'd find a solution to it. Also before that, you can kindly make sure that you are using the latest shadcn cli version; there might or might not have been updates that render these "hacks" redundant

Esta é a implementação do meu ícone da barra lateral recolhível com alteração no tamanho do icon
  <SidebarMenu>               {items.map((item) => (                 <SidebarMenuItem key={item.title}>                   <SidebarMenuButton asChild size="lg">
                         <item.icon size={24} />
                       {item.title}                                        </SidebarMenuButton>                 </SidebarMenuItem>               ))}             </SidebarMenu>

Same

@shadcn Is it expected to be possible to define the size of the icons?


Update catalogue for compatibility with latest s3fs

Coverage remained the same at 80.591% when pulling 34779c34ed14c96a2fe1841161ec2accd21a44f1 on fix_catalogue into f6275e04bd71d8aad78043b370914ccb7aea11d9 on master.


Rename nullable to required in InterfaceProperty

We should rename nullable to required in the InterfaceProperty definition, as semantically those two words are very different.
nullable means that a value can either have any type or null, while required indicates if a value must be defined or not.
  export class InterfaceProperty {
    name: string
    type: InstanceOf
-   nullable: boolean
+   required: boolean

+1 to clearly distinguish and disambiguate nullable ("that accepts null as a valid value") and required/optional ("may not be present" or "can be undefined").
However:

nullable means that a value can either have any type or null

I guess you mean "have a type or be null" and this property will be defined using a union, e.g. null | string.
Let me give an example in Rust that does not have null nor undefined values to illustrate why a clear distinction is needed:
// Enums are Rust's union types
enum NullableString {
    Null,
    String(String)
}

struct Foo {
  required_non_nullable: String,
  required_nullable:     NullableString,
  optional_non_nullable: Option<String>,
  optional_nullale:      Option<NullableString>,
}


Remove -Force check in ReconcilePreexistingPropertyNames method in Export-Csv.

PR Summary
Now, when you use Export-Csv with -Append parameter and new properties don't exist in existing file:

-Force:$false: error is thrown
-Force:$true: cmdlet silently skips adding new properties.

Should we remove -Force check in ReconcilePreexistingPropertyNames method, so that error is thrown in both cases - when -Force:$true and -Force:$false.
Or is it better to make cmdlet overwrite existing file with new data if -Force:$true?
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

Can you restart tests?

I restarted CI Travis MacOs tests.

@mklement0 Could you please review the change too?

My vote is for not making this change, which is certainly a breaking one:
While the docs don't really say what -Force is for - which should definitely be added - de facto it has served to force addition of objects even if they have unrelated / additional properties, which is a useful option to have if applied intentionally (that's where the docs come in).
Note that the error message you currently get with incompatible properties clearly states that -Force can be used to force appending.
@sethvs:
Please make it clearer up front when your changes will be breaking, and ideally create an issue before starting a PR, ideally by identifying the "bucket" that the change falls into.
Also, sometimes there's an alternative to a breaking change: in #6797 you tried to simply rename a parameter, but the better course of action is to add an alias.

I've added a docs issue to request documentation of the current behavior: https://github.com/PowerShell/PowerShell-Docs/issues/2392

@mklement0 Agree about -Force.
This is what discussion for.
Closing.

@sethvs:
Glad to hear it.
The reason I was suggesting creating an issue before submitting a PR is that the issue allows a broader discussion as to whether and how to implement a change / feature first.
I know that this PR was simple, but generally you run the risk of wasted effort without having consensus on what should be done first.


@cosmincatalin this jar just contains pointers to the sub-package jars and you can see this in the pom.xml here:

@cosmincatalin this jar just contains pointers to the sub-package jars and you can see this in the pom.xml here:
https://mmlspark.blob.core.windows.net/maven/com/microsoft/azure/synapseml/0.9.1/synapseml-0.9.1.pom
SynapseML isn't designed to be easily installable by uploading Jars as it has several dependencies. Please consider installing via spark.jars.packages using maven coordinates. Hope this helps and feel free to re-open if needed
Originally posted by @mhamilton723 in https://github.com/microsoft/SynapseML/issues/1227#issuecomment-960261697

My machines have no network access, How can I install the synapseml on spark cluster?  mmlspak 0.18 in one jar ,I can use --jar upload to install. Thanks for your attention.

Hi @liuyonglang, even with mmlspark 0.18, not all dependencies are packaged into one jar. There are many external dependencies that have to be installed at Spark runtime. It may have worked for you because you were not calling into these dependencies.
One thing you may want to try is to install all the dependencies in a docker image, export the image to your offline machine, and run through docker.

@liuyonglang, perhaps you can clone the repository locally and use sbt assembly to make a fat jar?

@liuyonglang I'm in a similar situation on a spark cluster with no internet access, have you found a solution?
@mhamilton723 I'll play around with that idea and see if I can get it to work

Can we get this in Maven Central. I think we can all agree that would be easiest for everybody, with no obvious downsides?

@cosmincatalin that is coming in our next release if all goes according to plan!

Feel free to re-open if needed

@liuyonglang I'm in a similar situation on a spark cluster with no internet access, have you found a solution?
@mhamilton723 I'll play around with that idea and see if I can get it to work

I clone this locally and use sbt build some jars。Then I add all jars to spark dependency。

@liuyonglang I'm in a similar situation on a spark cluster with no internet access, have you found a solution?
@mhamilton723 I'll play around with that idea and see if I can get it to work

I clone this locally and use sbt build some jars。Then I add all jars to spark dependency。

@liuyonglang I'm in a similar situation on a spark cluster with no internet access, have you found a solution?
@mhamilton723 I'll play around with that idea and see if I can get it to work

I clone this locally and use sbt build some jars。Then I add all jars to spark dependency。


Doc comments & generated API docs

This adds a bunch of jsdoc comments to most of the code in src/peer, and an npm run build:docs command to generate HTML docs using the jsdoc cli tool & a docstrap template.
I also added npm run check:style and npm run check:types scripts, mostly so that I can easily run the version of flow that's bundled as a dev dependency; @parkan I think that the Buffer / Array errors you were seeing only occur on flow 0.37, but things seem to check out with 0.38 which is installed as the flow-bin dev dependency.
The 107 files changed 😛 is due to committing the generated docs... I kind of went back and forth on that but it will let us point github pages at the docs dir and easily host them without using a gh_pages branch.  I'm fine with adding a script to build and commit to the gh_pages branch though, if having the built docs on master is too annoying.

do we have a copy of the generated docs up anywhere public?

yeah, it's at https://mediachain.github.io/aleph/

:+1:


MP4 boxing never starts

Have fallen back to 6.5.1.
Video is captured when motion occurs, file added to queue, but boxing never starts. GPAC is installed, I have checked. Queue now has many items. Is there a way of forcing them to process?
Where is end_vid macro so I can check it? Does not appear in /var/www/html/macros or equivalent RPi_Web_Cam_Interface directory

end_vid macro is for user extensions and does not need to exist for normal operation.
Your problem with Boxing may be due to your failed upgrade and then attempt to revert.
If possible I would suggest a clean install starting from Buster

Buster update ongoing now as per other issue.
Can you suggest a way of boxing the h264 files?
I tried running MP4box but got a message that it did not like the input file.

MP4Box is the standard way. What command line did you use?
This software runs
MP4Box -fps %i -add %s %s
where %i is substituted by the frame rate e.g 25, the first %s is the input file (full path like /var/www/html/media/myvid.h264 and the second %s is the full path to destination file like /var/www/html/media/myvid.mp4

Thanks Robert - I suspect frame rate was my error! I’ll try again anon.

Have now updated to 6.6.4 and Buster.
Got the following at end of running update.sh
pi@DDhomepi3:~/RPi_Cam_Web_Interface $ cPHP Notice:  A non well formed numeric value encountered in /var/www/html/schedule.php on line 650 d  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current                                  Dload  Upload   Total   Spent    Left  Speed 100   121  100    54  100    67     88    109 --:--:-- --:--:-- --:--:--   197   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current                                  Dload  Upload   Total   Spent    Left  Speed 100   121  100    54  100    67     72     89 --:--:-- --:--:-- --:--:--   162   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current                                  Dload  Upload   Total   Spent    Left  Speed 100   121  100    54  100    67     73     90 --:--:-- --:--:-- --:--:--   164   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current                                  Dload  Upload   Total   Spent    Left  Speed 100   121  100    54  100    67     65     81 --:--:-- --:--:-- --:--:--   146   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current                                  Dload  Upload   Total   Spent    Left  Speed 100   121  100    54  100    67     95    118 --:--:-- --:--:-- --:--:--   213
Some older captures were then boxed but not all, so deleted them
Still getting same problem. Motion is detected, file captured, but never boxed and left marked “busy”.

Boxing problem Solved! Turns out that the residual h264 files block boxing from occurring.
When I deleted the, suddenly got the flow of IFTTT notifications I have set up using end_box.sh for those I deleted and subsequent detections work and create MP4 files ok.
Closing this and opening separate issue for message after update


alacritty freezes after extremely long output without newline

I'm on Linux, X11, Alacritty 0.3.3
Running any command which produces a ton of output without newlines (~900K-1MB?) causes alacritty to completely freeze up, hanging at 100% CPU usage and very little memory.
I'm using the default scrollback configuration (my config file is at https://github.com/lunarlambda/dotfiles/blob/master/alacritty/alacritty.yml)
The simplest way of reproducing the issue is running while true; do; echo -n a; done and waiting a few seconds. It doesn't always happen though, if you introduce colorization it becomes way more consistent. For example, using fd -0 (sharkdp/fd) or piping things through lolcat.
Another one that triggered it consistently was ls -R --color=always | tr -d '\n'
In the ls and fd cases, the command finished completely and the shell prompt displayed before alacritty froze.
Memory usage stayed low, so not an OOM error, I believe.
Endlessly outputting lines via yes or similar didn't do anything, even after lengthy waiting.

Sounds like a duplicate of #2217, feel free to give #2438 a shot, that should fix it.

Read through #2217 and this doesn't sound like the same issue at all: No lag/delays, alacritty just freezes completely. Vim/Nano/Tmux and similar all run perfectly fine, even if used for hours, but here it would sometimes freeze after 3 seconds of output or so.
print-events also stopped outputting anything. I'll see if #2438 fixes it

Just try it out, I'm pretty confident this is the same issue.


remove delayload from mkldnn

remove delay loading of mkldnn.dll library.

remove delay loading of mkldnn.dll library.

curious - why was delayload added initially and why are we removing it now? :)


opt support binlog for rename operator

Rename operator doesn't have binlog now. This PR will create binlog when execute rename. The rename operator means :

rename table : ALTER TABLE table1 RENAME table2;
rename rollup : ALTER TABLE example_table RENAME ROLLUP rollup1 rollup2;
rename partition : ALTER TABLE example_table RENAME PARTITION p1 p2;

After SQL analyzing, we can get the old and new table name (rollup name or partition name) , then record the info to binlog, so we can use the info from binlog.

run buildall

no need to branch-2.1 so close


Add the missing tool_prompt parameter to Python client

What does this PR do?
This PR adds the missing tool_prompt parameter in Python client

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[x] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@Narsil

LGTM, thanks a lot !


Payment Date Is Incorrect in Contract

When my buying node calls ob/order/ for one of its orders, placed on November 28th, it shows the payment as being made today, December 18th.
The block explorer shows the payment as having been made 18 days ago.
https://live.blockcypher.com/ltc/tx/9899c344a3a68cdefdf8d25f7679f1bf6eeda73a3cef710a5abc4ae1c3edb819/
The contract shows the order was confirmed by the vendor on November 28th, which would seem to indicate the payment must have happened before that.
The selling node sees the same incorrect date for the payment.
{
    "contract": {
        "vendorListings": [
            {
                "slug": "ltc-star",
                "vendorID": {
                    "peerID": "QmaWkAWZ5ZS1dGmYkkJRBMVGMmAbjpRCPCNeGsxgYLCcx9",
                    "handle": "",
                    "pubkeys": {
                        "identity": "CAESINXF2v3WZg8zp5Vg9iZ3C40ubrgMQVbTGSWDrqzzP5qv",
                        "bitcoin": "AzLQ2ndfo2dv0vBrZ0ClEPYJqG5doYkUFwZ+MC7YaXpN"
                    },
                    "bitcoinSig": "MEQCIBJNKELJeBjRBFa8m/kf6C1wQGjeAPnl2nuQzbO5zYUXAiApE57qbLT6IzOIe0MEM5nexDWGM5pYHGeugdfzoKFInQ=="
                },
                "metadata": {
                    "version": 4,
                    "contractType": "CRYPTOCURRENCY",
                    "format": "MARKET_PRICE",
                    "expiry": "2037-12-31T05:00:00.000Z",
                    "acceptedCurrencies": [
                        "LTC"
                    ],
                    "pricingCurrency": "",
                    "language": "",
                    "escrowTimeoutHours": 1080,
                    "coinType": "STAR",
                    "coinDivisibility": 100000000,
                    "priceModifier": 100
                },
                "item": {
                    "title": "LTC-STAR",
                    "description": "Just a test. Do not buy.",
                    "processingTime": "",
                    "price": 0,
                    "nsfw": false,
                    "tags": [
                    ],
                    "images": [
                        {
                            "filename": "awesome_products_you_will_want_to_buy_640_19.jpg",
                            "original": "zb2rhbDUjzdEnhU9752tTnKc3QFTAWXDdUVF6Sr1noUVngLhM",
                            "large": "zb2rhYjfaSLD4HFwhNs1tGF87s8dta3BKECGcM2UroHALa8Cd",
                            "medium": "zb2rhY2Pv2tD9wkgi2t3csATFdm2jg4zWvcHrX59gYPQCochC",
                            "small": "zb2rhhUzqoXzjsEAq9ofR3xgrTvikyyDSWqT4skT4ozUBsF3c",
                            "tiny": "zb2rhh6XWNWnQjKqMGjt9PPhnoc3h9vF9dqjDjZRs7PeLE1b5"
                        }
                    ],
                    "categories": [
                    ],
                    "grams": 0,
                    "condition": "",
                    "options": [
                    ],
                    "skus": [
                        {
                            "productID": "",
                            "surcharge": 0,
                            "quantity": 0
                        }
                    ]
                },
                "shippingOptions": [
                ],
                "coupons": [
                ],
                "moderators": [
                ],
                "termsAndConditions": "",
                "refundPolicy": ""
            }
        ],
        "buyerOrder": {
            "refundAddress": "LMa8pnfCd4tgxZoCsEsdokyW3zTL2Ypztv",
            "refundFee": 0,
            "shipping": {
                "shipTo": "",
                "address": "",
                "city": "",
                "state": "",
                "postalCode": "",
                "country": "NA",
                "addressNotes": ""
            },
            "buyerID": {
                "peerID": "Qmdy44rwpPQ2opSJi9LjKSzChMgHXgdC5vPq5zfVUGNYbh",
                "handle": "",
                "pubkeys": {
                    "identity": "CAESIKdDOlxfCHPOS42kcE3MoShJV2jiqvwIBM2eDzdodnTa",
                    "bitcoin": "A3Mkw6CUFQeLvMCDM9qjbztfUOaQ3QngDMD/7mHp2mrc"
                },
                "bitcoinSig": "MEUCIQDJPiX0Bo4jsLqA1Bcxiu109ZfvZWxwQEY1RGiEOxID0gIgbKQVZjV1p+eErG6PeJmqNhCgImLnWxyIPAgJY4x/BBE="
            },
            "timestamp": "2018-11-28T19:47:50.858130007Z",
            "items": [
                {
                    "listingHash": "zb2rhn6zFDE34XD32Jsof7dW31tr2kbgAtcsRf2wEsZ3ToyrY",
                    "quantity": 0,
                    "quantity64": 100000000,
                    "memo": "",
                    "paymentAddress": "dgfgfgsdfg"
                }
            ],
            "payment": {
                "method": "ADDRESS_REQUEST",
                "moderator": "",
                "amount": 34812,
                "chaincode": "",
                "address": "",
                "redeemScript": "",
                "coin": "LTC"
            },
            "ratingKeys": [
                "ArLuYZw+V/kat4bTiGelXXobyGa+mFPB1h1nOSX2C6jg"
            ],
            "alternateContactInfo": "",
            "version": 2
        },
        "vendorOrderConfirmation": {
            "orderID": "QmbMSo9QURyyCg4gP3y4ao1dTdbBiN8prddkWGjqk5SeWp",
            "timestamp": "2018-11-28T19:47:52.186188167Z",
            "paymentAddress": "LN94E96dDyUpUQa6vQ4c5hhxgVdUCkxVJt",
            "requestedAmount": 34812
        },
        "signatures": [
            {
                "section": "LISTING",
                "signatureBytes": "LRQV6QR863+MhpY4VdSDBlNkIrDTbu3LeQQGBEDnFn2N7tnUjmpVNyvTVjViBvCx+h36l0+yRjuvJdU0+8bECg=="
            },
            {
                "section": "ORDER",
                "signatureBytes": "g4Ef76ywF6D6D2SVQZ1LmimoWWvlfMfPAioq8+nUvODvZe3naFoRBBAhOB1OfnL6QzW3VOWANMODtjq6H164DA=="
            },
            {
                "section": "ORDER_CONFIRMATION",
                "signatureBytes": "KK8iW1ZFuPQCVSFc4r78upcNTYy7QId3PVePWiU5S2hlAkCbb8HrCQB89N23713xVSAV7u1OQMmWlbfD0lbRCA=="
            }
        ]
    },
    "state": "AWAITING_FULFILLMENT",
    "read": true,
    "funded": true,
    "unreadChatMessages": 0,
    "paymentAddressTransactions": [
        {
            "txid": "9899c344a3a68cdefdf8d25f7679f1bf6eeda73a3cef710a5abc4ae1c3edb819",
            "value": 34812,
            "confirmations": 10190,
            "height": 1535409,
            "timestamp": "2018-12-18T17:12:36.757816Z"
        }
    ]
}

Timestamps have been fixed


Does the wrapped logger write need to be synchronize?

I found the SendToLog have mutex inside, does logger's write function need synchronize then?
https://github.com/google/glog/blob/c8f8135a5720aee7de8328b42e4c43f8aa2e60aa/src/logging.cc#L1715-L1720
the LogToAllLogfiles will callback wrapped logger's write function, does the write need to synchronize if there is a mutex before?

The comment above the function implies that there's no need for an additional lock:
https://github.com/google/glog/blob/c8f8135a5720aee7de8328b42e4c43f8aa2e60aa/src/logging.cc#L1714
Does this help?


Viewport property "Handle Input Locally" gets set to false for no apparent reason, including in the editor, when it is a child of ViewportContainer.

"Handle Input Locally" won't stay checked on 3.4.2-stable. The inspector will show the "revert to default" icon, but even if the property is manually written into the file as being explicitly true, it will always appear as false in the editor when you stop looking at the scene and look back. When running a project, checking it it during the ready function of the viewport's container will return false. Attempting to set it to true during the ready function succeeds, and getting it during the same function call returns true, but it gets reverted at some point between that and the first call of the container's process function, where it will return false. Calling the set function deferred from ready will not be reverted, however, nor will setting it at any point after that... Unless there's some trigger that'll cause it that I simply haven't tripped, like changing the viewport container size or something.
I have no idea why this would happen. Malevolent spirits?
Can be reproduced with the same project as in the below issue.
Originally posted by @SuperSuit12 in https://github.com/godotengine/godot/issues/25525#issuecomment-1005330138

(I don't frequently open issues - hopefully I did this right.)

Oh! And there's an old project I have where this issue doesn't come up, and I don't. know. why.
Why is it exempt? What is different there? I cannot figure it out.

Confirmed previous comment - tried it in a separate project and it's DEFINITELY because of the ViewportContainer, not Viewports in general. So it seems... intentional? I would expect a little ⚠ in the SceneTree pane if it's set up in a potentially harmful way, but not forcefully setting the option like that. As far as I can tell, it works fine when set to true... Better than fine, even, because it being set to false results in some very strange behavior, like buttons not being clicked for no apparent reason because something visually behind them is getting the input first.

handle_input_locally doesn't have documentation in either 3.x or master

I thought I'd mentioned this here, but apparently not... I checked, and it's very clearly intentional. There's a bit of code in ViewportContainer (at least in 3.x) that intentionally disables the option, labelled something along the lines of "don't do this." It appears to have been added at the exact same time the handle_input_locally property itself was.
I'm not sure if it solves #25525. I'm not seeing any absolutely critical issues, and most of the time input goes to exterior elements as you'd expect. However, buttons inside the Viewport still enter the "hovered" state when there's something that should be blocking them, and I had the occasional issue with still clicking "through" an outside element to an inside one - although why the latter happens I can't discern, because I cannot figure out how to reproduce it. I had a couple of elements that were identical in all but position, and one got clicked through outside elements while the other did not. I switched their positions and rearranged them in the hierarchy a few times, and nothing changed, until I detatched the script from both to make sure it wasn't something about my custom input code. Suddenly the outside button always got the click. I have no idea why.

I just got this issue and spent couple of hours wondering why my viewport input was conflicting outside the viewport container.
Currently I have a script that just toggles it off then back on again just to fix it.
onready var viewport_node = find_node('Viewport')

func _ready():
	# Bypass for issue #56502
	viewport_node.handle_input_locally = false
	yield(get_tree(), "idle_frame")
	viewport_node.handle_input_locally = true

https://github.com/godotengine/godot/blob/3.5/scene/gui/viewport_container.cpp#L123
Seems to be where the issue is happening. I turn handle_input_locally back on with a script and I just did quick test with gui nodes over the viewport. So far it seems to be just functioning fine?

The ViewportContainer will also silently change the update_mode depending on the visibility. I don't think it should do that, either.

I just ran into this issue of handle_input_locally being set to false after running the scene. This is really confusing behavior. I added this to my scene as a workaround and it seems to mostly work:
func _ready():
    sub_viewport.handle_input_locally = true


API Connection keeps closing

Hello,
The API often returns this error:
"ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host"
When I try to except it by re-connecting to the API:
except:     API_CONNECTION = API(API_KEY, request_params={'timeout': 30})
It returns:
"ProtocolError: ('Connection aborted.', ConnectionResetError(10054, 'An existing connection was forcibly closed by the remote host', None, 10054, None))"
then it ends the program.
However, when I re-run the program / call the API again, it works just as expected. I have only just added the timeout parameter, but without it, the error still happens.
All calls are in the format of API_CONNECTION.request()

Here is the full error log:
Submit_Open_Buy_Order_Details = API_CONNECTION.request(orders.OrderDetails(accountID = ACCOUNT, orderID = Submit_Open_Buy_Order_ID))
Traceback (most recent call last):

  File ~\Anaconda3\lib\site-packages\urllib3\connectionpool.py:703 in urlopen
    httplib_response = self._make_request(

  File ~\Anaconda3\lib\site-packages\urllib3\connectionpool.py:449 in _make_request
    six.raise_from(e, None)

  File <string>:3 in raise_from

  File ~\Anaconda3\lib\site-packages\urllib3\connectionpool.py:444 in _make_request
    httplib_response = conn.getresponse()

  File ~\Anaconda3\lib\http\client.py:1377 in getresponse
    response.begin()

  File ~\Anaconda3\lib\http\client.py:320 in begin
    version, status, reason = self._read_status()

  File ~\Anaconda3\lib\http\client.py:281 in _read_status
    line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")

  File ~\Anaconda3\lib\socket.py:704 in readinto
    return self._sock.recv_into(b)

  File ~\Anaconda3\lib\ssl.py:1241 in recv_into
    return self.read(nbytes, buffer)

  File ~\Anaconda3\lib\ssl.py:1099 in read
    return self._sslobj.read(len, buffer)

ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

  File ~\Anaconda3\lib\site-packages\requests\adapters.py:440 in send
    resp = conn.urlopen(

  File ~\Anaconda3\lib\site-packages\urllib3\connectionpool.py:785 in urlopen
    retries = retries.increment(

  File ~\Anaconda3\lib\site-packages\urllib3\util\retry.py:550 in increment
    raise six.reraise(type(error), error, _stacktrace)

  File ~\Anaconda3\lib\site-packages\urllib3\packages\six.py:769 in reraise
    raise value.with_traceback(tb)

  File ~\Anaconda3\lib\site-packages\urllib3\connectionpool.py:703 in urlopen
    httplib_response = self._make_request(

  File ~\Anaconda3\lib\site-packages\urllib3\connectionpool.py:449 in _make_request
    six.raise_from(e, None)

  File <string>:3 in raise_from

  File ~\Anaconda3\lib\site-packages\urllib3\connectionpool.py:444 in _make_request
    httplib_response = conn.getresponse()

  File ~\Anaconda3\lib\http\client.py:1377 in getresponse
    response.begin()

  File ~\Anaconda3\lib\http\client.py:320 in begin
    version, status, reason = self._read_status()

  File ~\Anaconda3\lib\http\client.py:281 in _read_status
    line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")

  File ~\Anaconda3\lib\socket.py:704 in readinto
    return self._sock.recv_into(b)

  File ~\Anaconda3\lib\ssl.py:1241 in recv_into
    return self.read(nbytes, buffer)

  File ~\Anaconda3\lib\ssl.py:1099 in read
    return self._sslobj.read(len, buffer)

ProtocolError: ('Connection aborted.', ConnectionResetError(10054, 'An existing connection was forcibly closed by the remote host', None, 10054, None))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

  Input In [25] in <cell line: 1>
    Submit_Open_Buy_Order_Details = API_CONNECTION.request(orders.OrderDetails(accountID = ACCOUNT, orderID = Submit_Open_Buy_Order_ID))

  File ~\Anaconda3\lib\site-packages\oandapyV20\oandapyV20.py:337 in request
    response = self.__request(method, url,

  File ~\Anaconda3\lib\site-packages\oandapyV20\oandapyV20.py:266 in __request
    raise err

  File ~\Anaconda3\lib\site-packages\oandapyV20\oandapyV20.py:262 in __request
    response = func(url, stream=stream, headers=headers,

  File ~\Anaconda3\lib\site-packages\requests\sessions.py:542 in get
    return self.request('GET', url, **kwargs)

  File ~\Anaconda3\lib\site-packages\requests\sessions.py:529 in request
    resp = self.send(prep, **send_kwargs)

  File ~\Anaconda3\lib\site-packages\requests\sessions.py:645 in send
    r = adapter.send(request, **kwargs)

  File ~\Anaconda3\lib\site-packages\requests\adapters.py:501 in send
    raise ConnectionError(err, request=request)

ConnectionError: ('Connection aborted.', ConnectionResetError(10054, 'An existing connection was forcibly closed by the remote host', None, 10054, None))

The error message seems pretty clear: ConnectionError, it also says 'forcibly closed'.
Since you did not show any code my guess is that you instantiate an API-client multiple times. OANDA has limits on several connection related things. Setting up connections in a certain time span is one of them.
The pattern is:
client = API(access_token=access_token)

r = orders.Order(...)

try:
    rv = client.request(r)
except Exception as err:
    print(err)
else:
    print(rv)

re-use the client. So, in case you place multiple orders:
client = API(access_token=access_token)

for instrument in [...]:
    r = orders.Order(...)
    try:
        rv = client.request(r)
    except Exception as err:
        print(err)
    else:
        print(rv)


mj-button ignores align attribute

The code for mj-button always generates a td with align='center'
See https://github.com/mjmlio/mjml/blob/master/packages/mjml-button/src/index.js#L110
The button defines an align attribute at https://github.com/mjmlio/mjml/blob/master/packages/mjml-button/src/index.js#L7
but it is never used.
To Reproduce
Steps to reproduce the behavior:

Create a file with this MJML code:

hello

Render it to HTML using https://mjml.io/try-it-live
Inspect the code

Expected behavior
Setting align to left would override the align="center"

@iRyusa Setting text-align on an mj-button element doesn't appear to actually do anything. Looking at the source for the mjml-button component, there is no reference to text-align.

Ah - sorry about that. I was using mjml-component-boilerplate and didn't realize it comes with a very outdated version of MJML.


NotImplementedError: Cannot copy out of meta tensor; no data!

System Info
2.0.1+cu118, driver 535.86.05
Information

[X] The official example scripts
[ ] My own modified scripts

🐛 Describe the bug
When you run on single GPU, you will encounter this error if you do not quantise.
To fix this, change this line from:
        model = LlamaForCausalLM.from_pretrained(
            train_config.model_name,
            load_in_8bit=True if train_config.quantization else None,
            device_map="auto" if train_config.quantization else None,
        )

to fix:
    else:
        model = LlamaForCausalLM.from_pretrained(
            train_config.model_name,
            load_in_8bit=True if train_config.quantization else False,
            device_map="auto", #if train_config.quantization else None,
            torch_dtype=torch.float16,
        )

basically, setting the torch_dtype will then I guess load the meta device.
later on, the loss won't be stable, you will get a lot of NAN loss. So I suggest to change it to bf16 in this section:
    if train_config.quantization:
        model = prepare_model_for_int8_training(model)
    else:
        model.to(torch.bfloat16)

Error logs
NotImplementedError: Cannot copy out of meta tensor; no data!
Expected behavior
Should allow to train on one gpu without quantisation

Let me know what you think needs to be in the official changes, I can make a PR.

Hi!  Since our repo has been updated, can you try it again with the latest repo?


Use React 16's forwardRef for most/all components

reactstrap is a perfect use-case for React 16's new forwardRef API. Since most components in this library have predictable and well-defined DOM they produce, it would be very useful to have them forward their refs along to the underlying elements so consumers like myself don't have to wrap access in ReactDOM.findDOMNode when we want to e.g. implement a scroll-related feature.

Totally Agree!! We're using some reactstrap components in our project and in order to expand the current behavior it would be really handy (and much more consistent) being able to select the DOM elements via ref selections in the same components
For example, doing this in the Tooltip component
class Tooltip extends React.Component {
  constructor(props) {
    super(props);

    this.refTooltipInner = React.createRef();
    ...
  }

  render() {
    ...
     return (
      <PopperContent
        ...
      >
        <div
          ref={this.refTooltipInner}
          ...
        />
      </PopperContent>
    );
  }
}

Would allow to have access to this DOM element easily from the outside

I'm all for this, but it will be a breaking change since it will change the require version of react needed as a peerDep.

If adding refs is all that is needed, we can do that like how we did https://github.com/reactstrap/reactstrap/issues/1054, so that we don't have to change react version, right? So wouldn't it be better to do it like that so we don't have to bumb the version up?

True. You could put innerRef on everything, and use this issue as a tracking issue for the eventual deprecation and replacement with forwardRef as noted in #1054.

@TheSharpieOne when would you accept a PR for this breaking change? I keep coming up against this cause I want to attach refs to things that often turn out to be stateless components. I'd be open to making a PR to apply the change as well. We might be able to keep innerRef around as a deprecated alias to preserve API-compatibility even as the minimum React version is raised.
It's a shame that 16.3 was introduced not 6 days after #916 dropped support for versions below 16. :(

there is a v7 branch, make PRs with breaking changes to that branch, eventually it will be merged and released.

When you start using forwardRef in a component library, you should treat it as a breaking change and release a new major version of your library. This is because your library likely has an observably different behavior (such as what refs get assigned to, and what types are exported), and this can break apps and other libraries that depend on the old behavior.
https://reactjs.org/docs/forwarding-refs.html#note-for-component-library-maintainers

Yes, it would be nice to get forwardRef into the next major release. This missed v7 as well as v8

@TheSharpieOne Now I'm working on https://github.com/reactjs/react-transition-group/pull/559 to migrate from deprecated findDOMNode then I think I'll create a new PR from #1356, to migrate to forward ref, and use new react-transition-group api, so we can finally use React.StrictMode in our app.

Any updates on this issue?
It is really needed.


E10M20-T1 PCIE Support

Any way to get E10M20-T1 PCIE support added? I have a DS2422+ with the E10M20-T1 PCIe NVMe card tried the current version on DSM 7.2 guessing only internal NVMe are supported at the minute.

It should work. Did you reboot after running the script? In storage manager do the drives show as "Cache drive" or "M.2 drive"?
If you run the Synology_HDD_db script it will add the NVMe drives to the ds2422+_e10m20-t1_v7.db and ds2422+_v7.db drive database files.

Yeah I rebooted. I have also ran the  Synology_HDD_db script. I have tried the Synology enable M2 volume which does work it would just be nicer if I could do it via the GUI on demand.

Are you referring to Synology_M2_volume script or this Synology_enable_M2_volume script?
Synology_enable_M2_volume lets you create and manage the storage pool and volume all from within the storage manager GUI. It works on DS3622xs+ and RS4021xs+ with DSM 7.2-64561 and DSM 7.1.1 so it should work on a DS2422+... unless DSM 7.2-64570 has changed something.

The Synology_M2_volume script is working for me but the Synology_enable_m2_volume script is not working for me not sure if its the latest update since I've never tried it before or possibly because I'm using the E10M20-T1 PCIE adapter for my NVMe drives.

Does Synology_HDD_db detect the E10M20-T1 and the NVMe drives?
Can you paste the output from Synology_HDD_db in a reply.

Synology_HDD_db v2.2.47
DS2422+ DSM 7.2-64570
Using options: -nr
HDD/SSD models found: 4
MG07ACA12TE,0101
WD161KFGX-68AFPN0,0A83
WDS100T1R0A-68A4W0,00WR
WDS100T1R0A-68A4W0,10WR
M.2 drive models found: 1
WD Red SN700 2000GB,112000WD
M.2 card models found: 1
E10M20-T1
No Expansion Units found
MG07ACA12TE already exists in ds2422+_host_v7.db
MG07ACA12TE already exists in ds2422+_host_v7.db.new
WD161KFGX-68AFPN0 already exists in ds2422+_host_v7.db
WD161KFGX-68AFPN0 already exists in ds2422+_host_v7.db.new
Updated WDS100T1R0A-68A4W0 to ds2422+_host_v7.db
Updated WDS100T1R0A-68A4W0 to ds2422+_host_v7.db.new
Updated WDS100T1R0A-68A4W0 to ds2422+_host_v7.db
Updated WDS100T1R0A-68A4W0 to ds2422+_host_v7.db.new
WD Red SN700 2000GB already exists in ds2422+_host_v7.db
WD Red SN700 2000GB already exists in ds2422+_host_v7.db.new
WD Red SN700 2000GB already exists in ds2422+_e10m20-t1_v7.db
WD Red SN700 2000GB already exists in ds2422+_e10m20-t1_v7.db.new
Support disk compatibility already enabled.
Support memory compatibility already disabled.
M.2 volume support already enabled.
Drive db auto updates already disabled.
DSM successfully checked disk compatibility.
You may need to reboot the Synology to see the changes.

Yeah, thats what I've done for now and it is working correctly. Cheers buddy


Clearing worker processor instances of reactor state after every interaction

What is the change?
The PR creates a method in Operator that is called _finalizeInteract which is called inside interface loop of _interactAll. This method is overridden by OperatorMPI to "reset" worker processors by broadcasting a command to force execution of _resetWorker.
Why is the change being made?
The purpose of this change to force worker resets after each interaction is complete, instead of relying on plugins to return the correct mpiActionRequiresReset during MpiAction invocations. The goal was to reduce memory when going to subsequent interfaces that don't require a distributed state. However, the addition of this reset call is not reducing memory on worker processors, which means the "reset" implementation may be flawed. However, the change proposed in this PR is still appropriate in principle and improvements to the reset functionality can be improved in the future.

Checklist

[x] This PR has only one purpose or idea.
[x] Tests have been added/updated to verify any new/changed code.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[ ] The release notes have been updated if necessary.
[ ] The documentation is still up-to-date in the doc folder.
[x] The dependencies are still up-to-date in pyproject.toml.

Quick Question:
You added new code, why are there no tests of this?  (Did you try, and it was too hard to test?)

Quick Question:
You added new code, why are there no tests of this? (Did you try, and it was too hard to test?)

I just didn't think of it. I can add a test.

@john-science @opotowsky Changed log message to extra and added a test in test_mpiFeatures.


Change modified file badge in the explorer tab for consistency

VSCode Version: Code - Insiders 1.18.0-insider (65c08338e4bece35fb53f5a1cdf56d30b7706157, 2017-10-20T15:35:45.848Z)
OS Version: Windows_NT x64 6.1.7601
Extensions:

Extension
Author (truncated)
Version

vscode-eslint
dba
1.3.2

githistory
don
0.2.3

xml
Dot
1.9.2

Go
luk
0.6.66

code-beautifier
mic
1.7.2

vscode-icons
rob
7.16.0

vscode-table-formatter
shu
1.2.1

addDocComments
ste
0.0.8

vscode-import-cost
wix
2.5.1

(1 theme extensions excluded)

Use in the explorer opened files list the modified svg badge from the source control tab instead of a new creation.
My suggestion:

For testing, add these css values to the modified element and remove the class monaco-decorations-style-2.
background-image: url(/resources/app/extensions/git/resources/icons/dark/status-modified.svg);
background-repeat: no-repeat;

Current design:

Yes, we have done that yesterday and aligned both representations. Tho we did pick a lighter design based on user feedback. See https://github.com/Microsoft/vscode/issues/178#issuecomment-339077269 for more details.

Thanks, but what do you think about adding a right margin (2px) to the 'U' element, that it looks like that:

Instead of

Because the 'M' icon is wider than the 'U' icon.

Tricky... That is rendered as text with the option for more text...


Add badge for GitHub user/org total stars received.

Add another GitHub badge that indicates the total stars received for all repositories owned by a user or organization.
Perhaps rely on a module like starz, or a fork thereof, to get the data. See yyx990803/starz#2.

Thanks for the suggestion!

Do you have an example of a place where a badge like this would be used?

@paulmelnikow A personal website or an organization site advertising their GitHub presence.

Makes sense. Do you know if there's a way to get the data from the API without making a ton of requests?

I'm marking all the needs-upstream-help issues closed.
If there's new information from the upstream service, please post in the thread. If they're actionable by Shields, a maintainer will reopen the issue.
If anyone wants to follow up with these vendors, feel free to do that! Even though the unresolved issues are closed, they are easy to find:
https://github.com/badges/shields/issues?q=label%3Aneeds-upstream-help+is%3Aclosed

Hi all!
I've made a new repository to provide an API~
https://github.com/idealclover/GitHub-Star-Counter
Example URL: https://github-star-counter.herokuapp.com/user/idealclover
So we can add badge like this:

Hope it can help you!


how to generate key frame?

in this repo, i just see the inference script for QA, but where can i use key frame generation function?

Hi, we are currently working on a more easy-to-use code and will release the key frame generation code in this section https://github.com/Deaddawn/MovieLLM-code?tab=readme-ov-file#pipeline. You can check the raw data we just release.

in this repo, i just see the inference script for QA, but where can i use key frame generation function?

Hi, we are currently working on a more easy-to-use code and will release the key frame generation code in this section https://github.com/Deaddawn/MovieLLM-code?tab=readme-ov-file#pipeline. You can check the raw data we just release.

in this repo, i just see the inference script for QA, but where can i use key frame generation function?

Hi, we are currently working on a more easy-to-use code and will release the key frame generation code in this section https://github.com/Deaddawn/MovieLLM-code?tab=readme-ov-file#pipeline. You can check the raw data we just release.

thx. I am looking forward this function.


mean style requires cuda tensor but got ''cpu'' while runing demo.py

https://github.com/mit-han-lab/anycost-gan/blob/19229bdc525ecd00ca7c2322b192504325bda9e0/models/anycost_gan.py#L85
here 'z' is "cpu" device while self.style needs "cuda" tensor???
Error is: RuntimeError: input must be a CUDA tensor
i figured out the reason is the code then running to FusedLeakyReLUFunction ,which requires CUDA tensor. But in demo.py, the device is "cpu". Any idea how to fix this problem?

Hi, sorry about that. The demo was tested on a CPU-only machine, which fails to expose the problem.
You can run the demo with FORCE_NATIVE=1 python demo.py

@tonylins thanks! It works！
BTW， what is this FORCCE_NATIVE=1 refers to ? is that means force to disable all GPUS?

thank you! that works!

hi,want to know my pc cant understand FORCE_NATIVE=1. thanks
but my pc run stylegan3 using cuda is ok

@tonylins  need your help,thanks


graph executes as many times as there are previous runs

This relates to https://github.com/noflo/noflo-ui/issues/383:
When I open a graph, and run it several times in a row (say, click the "connected" button 3 times), and then change one of the graph's initial values, the graph runs automatically the same number of times. Not just once, but a few times in a row  (3 times if I clicked the button 3 times before). Is this the way it is supposed to be?

confirmed similar behavior in my project (FWIW)

No it should run only once. What runtime are you using? noflo-browser?

I use nodejs runtime. I used "math" components connected to the "output" component. I judged about the number of runs based on the number of outputs. I think it is related to that the components do not reset the state when the graph is stopped so it appears that the graph is run several times.

ditto, nodejs runtime.

@jonnor can you confirm for us whether you'd expect this behavior if components aren't properly cleaning up/disconnecting?
The whole "when to call disconnect" thing is certainly something that's confusing me. When I look at the source of many of the core components I see a mixed bag as far as when disconnect is explicitly called vs just forwarded from in ports. Should we disconnect after every send? Is this even potentially related to this issue? (...where oh where are all the great user tutorials we should have by now? </whiney voice>)

Most components are state-free and should not remember previous "firing".
Not sending disconnect may prevent the network from stopping, but the old
network should be thrown away when doing stop/start again anyway.
Disconnect should be sent in component when it has stopped sending packets,
and new packets will not be sent until there are new packets coming in.
On Feb 24, 2015 1:11 AM, "ryedin"<EMAIL_ADDRESS>wrote:

@jonnor https://github.com/jonnor can you confirm for us whether you'd
expect this behavior if components aren't properly cleaning
up/disconnecting?
The whole "when to call disconnect" thing is certainly something that's
confusing me. When I look at the source of many of the core components I
see a mixed bag as far as when disconnect is explicitly called vs just
forwarded from in ports. Should we disconnect after every send? Is this
even potentially related to this issue? (...where oh where are all the
great user tutorials we should have by now? )
—
Reply to this email directly or view it on GitHub
https://github.com/noflo/noflo-ui/issues/384#issuecomment-75669057.

for example, noflo-math does not clean up primary and secondary values on shutdown, and since it invokes calculate when either primary or secondary data are received (e.g. https://github.com/noflo/noflo-math/blob/master/lib/MathComponent.coffee#L36) I see two results for add, sub, etc. on subsequent runs. this is perhaps I mistook it for multiple runs.

Ok, so if anything this is a noflo-math issue. Feel free to open an issue there.


X.app PiP URL scheme

Hello!
My app is "X.app": https://apps.apple.com/us/app/x-app/id1533525753
It's basically a browser with a Picture-in-Picture and Fullscreen modes + matching extension, supports YouTube, Vimeo, etc.
All web URLs are allowed.
URL scheme:
xapp://?videoURL=Encoded_URL
Twitter handle: @xapp_company
Thanks!

Added in https://github.com/timonus/OpenerManifest/commit/e2db817d3fceda0fa66ea406297ec11cf86b1ce8


How to hide status bar in navigation bar?

In my code, status bar is hidden,but after show that component, status bar will appear...

@Pozitive23  Subclass MWPhotoBrowser and then override prefersStatusBarHidden to hide or show status bar
- (BOOL)prefersStatusBarHidden
{
    return true;
}

If you always hide the status bar then you can set View controller-based status bar appearance to YES in your info.plist and then select "Hide status bar" in your target's general tab. Otherwise, yes subclassing would be the other option.


Update Posts “music-monday-the-mars-volta-maida-vale-sessions”

Automatically generated by Netlify CMS

👷 Deploy Preview for benevolent-muffin-215770 processing.

Name
Link

🔨 Latest commit
81b9c69d9d9d59f55c4c00f71e65c82b8df63d38

🔍 Latest deploy log
https://app.netlify.com/sites/benevolent-muffin-215770/deploys/6286e083ae82170009b329ff


version-based dependency overridden by path-based from unselected configuration

Given the following:
name "dubtest"

configuration "dynamic" {
    dependency "allegro" version="~>2.0.3+5.0.10"
    targetType "executable"
    libs "allegro"
}

configuration "static" {
    dependency "allegro" path="DAllegro5"
}

dub build --config=dynamic will get the allegro dependency from ./DAllegro5 rather than looking in the cache for the versioned dependency.
If you delete the dependency from the unused configuration, it works.
Full output:
[rcorre@desktop-arch dubtest]$ dub run --config=dynamic -v
Using dub registry url 'http://code.dlang.org/'
Refreshing local packages (refresh existing: true)...
Looking for local package map at /var/lib/dub/packages/local-packages.json
Looking for local package map at /home/rcorre/.dub/packages/local-packages.json
Note: Failed to determine version of package dubtest at .. Assuming ~master.
Refreshing local packages (refresh existing: false)...
Looking for local package map at /var/lib/dub/packages/local-packages.json
Looking for local package map at /home/rcorre/.dub/packages/local-packages.json
Adding local /tmp/dubtest/DAllegro5
Note: Failed to determine version of package allegro at .. Assuming ~master.
Found dependency allegro >=0.0.0 @DAllegro5
Checking for missing dependencies.
Using fixed selection allegro 2.0.3+5.0.10
Refreshing local packages (refresh existing: false)...
Looking for local package map at /var/lib/dub/packages/local-packages.json
Looking for local package map at /home/rcorre/.dub/packages/local-packages.json
Adding local /tmp/dubtest/DAllegro5
Found dependency allegro >=0.0.0 @DAllegro5
Checking for upgrades.
Using cached upgrade results...
Generating using build
Generate target dubtest (executable /tmp/dubtest dubtest)
Generate target allegro (staticLibrary /tmp/dubtest/DAllegro5 dallegro5)
Performing "debug" build using dmd for x86_64.
allegro ~master: target for configuration "no-libs" is up to date.
Using existing build in /tmp/dubtest/DAllegro5/.dub/build/no-libs-debug-linux.posix-x86_64-dmd_2070-F918032832AACEBCEA74E06E9194D292/.
Copying target from /tmp/dubtest/DAllegro5/.dub/build/no-libs-debug-linux.posix-x86_64-dmd_2070-F918032832AACEBCEA74E06E9194D292/libdallegro5.a to /tmp/dubtest/DAllegro5
dubtest ~master: target for configuration "dynamic" is up to date.
Using existing build in /tmp/dubtest/.dub/build/dynamic-debug-linux.posix-x86_64-dmd_2070-BD38FC2B9D3466932F0883331E42F1A8/.
Copying target from /tmp/dubtest/.dub/build/dynamic-debug-linux.posix-x86_64-dmd_2070-BD38FC2B9D3466932F0883331E42F1A8/dubtest to /tmp/dubtest
To force a rebuild of up-to-date targets, run again with --force.
Running ./dubtest

Using fixed selection allegro 2.0.3+5.0.10

This should come from the cache, but later ...

Using existing build in /tmp/dubtest/DAllegro5/.dub/build/no-libs-debug-linux.posix-x86_64-dmd_2070F918032832AACEBCEA74E06E9194D292/.

Is this happening for 0.9.24? I can confirm it there, but on master it appears to be fixed already. I've added a test case to ensure that this is/stays fixed.

Yeah, that was 0.9.24. Looks fine on master. Thanks!

issue was reintroduced it seems, but is now being tracked in #1217


Write Arrow buffer into host shared memory in CPU mode

cc: @andrewseidl
This PR is a starting point for writing the serialized Arrow buffer to host shared memory in CPU mode via shmget. I've tested it works locally on macOS Sierra 10.12.5, though it should work on any System V platform.
My C++ fu is not strong, so I cargo-culted the ThriftHandler. I implemented new code paths to reduce the risk of introducing new bugs. I expect someone from the team will want to delete or rewrite this code to conform to the coding guidelines.
The core logic is in ResultSet::getArrowCopy. I've added comments inline with the following questions/todos:

I've used stdlib rand() to generate unique shmid keys. Are there any flaws in this approach?
if shmget or shmat fail, the error should be surfaced to the caller (needs rewrite following mapd-core guidelines)
it should be possible to serialize the arrow buffer directly to shared memory, but I'm not familiar with the C++ Streams API. I've hacked it with a memcpy for now

Here's an example of calling the sql_execute_df method from node (see more in our graphistry/rxjs-mapd repo):
// Assuming mapd-core is running with default settings and flights_2008_10k test database
import * as shm from 'shm-typed-array';
import { createClient, createConnection, TBinaryProtocol, TBufferedTransport } from 'thrift';
// These libs aren't in npm yet
// Arrow's JS impl has a bug that prevents it from reading data from
// columns that don't contain nulls. I patched it locally, and will PR them soon
import { getReader } from 'apache/arrow';
// run `thrift --gen js:node mapd.thrift` to get this code
import { Client as MapDClient } from 'mapd-thrift';

const client = createClient(MapDClient,  createConnection(`localhost`, 9091, {
  protocol: TBinaryProtocol, transport: TBufferedTransport
}));

client.connect(`mapd`, `HyperInteractive`, `mapd`)
  .then((session) => client.sql_execute_df(session, `SELECT origin_lat, origin_lon FROM flights_2008_10k WHERE dest_city ILIKE 'dallas'`, -1))
  .then(({ df_handle }) => getReader(shm.get(+df_handle, 'Buffer')))
  .then((arrow) => console.log(`col names: ${arrow
      .getSchema()
      .map(({ name }) => name)
      .join(', ')
    }`)
  );

 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.

Hi @trxcllnt, thanks for your commits, we will review and merge them soon.

Thanks! We'll keep refining this in master.


anti afk cannot detect roblox windows

After i opened roblox and ran the script, it says no windows found. And i'll be kicked after 20 mins. How can i solve this? Thanks

Did you add your game into the config?

you mean the edit script function?

what you mean by config included in script

if i see "Monitoring: RobloxPlayerBeta.exe - 1" then is it working

Sorry for the late response, I didn't get a ping.

what you mean by config included in script

By "in the script" I meant opening it in a text editor. Everything is well-commented so you'll be fine from there.

if i see "Monitoring: RobloxPlayerBeta.exe - 1" then is it working

Yes, if by "working" you mean waiting for the window to time out.
While it does see the window, it will wait for a bit of inactivity before it starts to send keypresses.
This should change to "Managing" once keystrokes are being sent.
A surprising amount of people have raised similar issues, where they haven't specified which processes to watch and are surprised when no windows are found.
When I get back from holiday, I'll update the README to be a bit more explicit in the matter and include instructions on how to find the config.
Hopefully this clears up the issue!

thanks for your response!


Documentation bug for sign up funnels

Per our support emails, in the docs (https://mixpanel.com/help/reference/ios#user-identity), the following sentence needs to be more explicit to ensure the Mixpanel generated distinct id is set for the signup funnel.  Otherwise, if you set the identify to your own user ID upon signup, it's possible to hit a race condition in which the events are associated between two different buckets (one is the IFA/IFV and the other is your own internal user ID)
Please consider changing this:

The recommended usage pattern is to call both createAlias: and identify: when the user signs up, and only identify: (with their new user ID) when they log in.

to this:

The recommended usage pattern is to call both createAlias: and identify: (with the Mixpanel generated distinctId, as shown in the example above) when the user signs up, and only identify: (with their new user ID) any time they login thereafter.

Thanks for pointing this out @iwasrobbed -- went ahead and made the change you requested in the docs. If you see anything else which should be changed please let me know!


Could I use the conditional boundaries from InterFaceGAN?

Hi!
First of all, thanks for Genforce's kind share! It's really great work!
Recently, I am reading papers about Idinvert (this project) and InterFaceGAN and they really make me believe that these two works together would make some contribution if we want to do some real image editing in this field. So could I apply the conditional boundaries in the InterFaceGAN to the sematic manipulation of Idinvert project? What I found in the Idinvert project are some single boundaries. Besides, the shapes of the conditional boundaries in the InterFaceGAN project ($Z$ or $W$ space) are not compatible with the latent code of stylegan_ffhq256 in Idinvert project (I suppose they are $WP$ with (1, 14, 512) shape?). I am not sure whether I need to retrain the conditional boundaries for this? If you could offer some suggestions, that would be much help :)
By the way, I have another little quesion: there is one sentence called "Change repleated $w$ for all layers to different $w$s (Line 428-435 in file training/networks_stylegan.py)" in the repo. What does the word "repleated" mean? Or you mean "repeated"?
Thank you again for your great work!

The model used in this work is different from the model in InterFaceGAN. That's why the boundaries in InterFaceGAN cannot be applied to this project. But the project_boundary() function (here) can be reused.
Yes, it is a typo. The word should be "repeated". In particular, in official StyleGAN, the w code, with shape (1, 512), is repeated multiple times before fed into each layer. We slightly modify the model by directly learning the wp code, with shape (14, 512).

@ShenYujun Thanks a lot! I would look into the "project_boundary()" function later.


updated code_ql action

Updated action for node16.
(closes https://github.com/macrosynergy/macrosynergy/issues/645)

Codecov Report
Base: 58.94% // Head: 58.94% // No change to project coverage :thumbsup:

Coverage data is based on head (c9d3dfb) compared to base (f1a46db).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #658   +/-   ##
========================================
  Coverage    58.94%   58.94%
========================================
  Files           29       29
  Lines         3064     3064
========================================
  Hits          1806     1806
  Misses        1258     1258

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


live reload bug.

🐛 Bug Report
https://www.youtube.com/embed/5uqJjNFazPo?start=19
To Reproduce
react-native init testlivereload
cd testlivereload
react-native run-android
Then, I will do the video.
Expected Behavior
Code Example
Environment
React Native Environment Info:
System:
OS: Linux 4.15 Ubuntu 18.04.1 LTS (Bionic Beaver)
CPU: (8) x64 Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz
Memory: 3.26 GB / 15.56 GB
Shell: 4.4.19 - /bin/bash
Binaries:
Node: 8.12.0 - ~/.nvm/versions/node/v8.12.0/bin/node
Yarn: 1.10.1 - ~/.nvm/versions/node/v8.12.0/bin/yarn
npm: 6.7.0 - ~/.nvm/versions/node/v8.12.0/bin/npm
SDKs:
Android SDK:
API Levels: 23, 24, 25, 26, 27, 28
Build Tools: 23.0.1, 23.0.2, 23.0.3, 24.0.2, 25.0.0, 25.0.1, 25.0.2, 25.0.3, 26.0.1, 26.0.2, 26.0.3, 27.0.1, 27.0.3, 28.0.3
System Images: android-23 | Google APIs Intel x86 Atom_64, android-27 | Google APIs Intel x86 Atom
npmPackages:
react: 16.6.3 => 16.6.3
react-native: 0.58.3 => 0.58.3

It is good to test it by creating new test.


[Anomaly Detector] New SDK release v1.1.0-preview.1

Request a new release of .Net SDK for cognitive service Anomaly Detector.

I'd like to close this PR because we want to postpone this release.


New package: EnvironmentMigrators v0.1.0

Registering package: EnvironmentMigrators
Repository: https://github.com/mkitti/EnvironmentMigrators.jl
Created by: @mkitti
Version: v0.1.0
Commit: dac72a7c3426f472ee37087b1c8aa163ff59ca43
Reviewed by: @mkitti
Reference: https://github.com/mkitti/EnvironmentMigrators.jl/commit/dac72a7c3426f472ee37087b1c8aa163ff59ca43#commitcomment-65979995
Description: Utility package to help migrate environments by copying Project.toml and Manifest.toml

Please close both.


Multiple SQL Statements with combined output in 1 result tab.

I would like to be able to run many sql statements and have the results all send to the same output tab.
Query script to run with (Ctrl+Alt+Shift+X)
select 'table_1', count() from table_1;
select 'table_2', count() from table_2;
...
select 'table_400', count(*) from table_400;
Desired output (echoing sql commands optional) (dots added for justification)
'TABLE_1'|COUNT(*)|
---------+--------+
table_1  |     5400|
'TABLE_2'|COUNT(*)|
---------+--------+
table_2  |        100|
.....
'TABLE_400'|COUNT(*)|
---------+--------+
table_400  |   33009|
Is there a way of sending all the output to a single results tab or can that be added as a feature?

Does SQL Terminal satisfy your needs?

there have been no updates for a long time. If the issue is still actual - feel free to reopen it.


Could not find ComfyUI under Directory "". The server will NOT be started

Like the title says.
I installed it via the Automatic1111 extensions list and after giving me that error, I tried installing it from the github link, giving me the exact same error. I see the tab on Automatic1111 UI but nothing is in it. Not sure what's going on.
I am using:
Windows 11
Automatic version: 1.6.0
GPU: RTX 3090
CPU: Ryzen 7 5800X3D

Normally, right after installing the extension, you have to install comfyui. You can either do this manually or by following the instructions in the ComfyUI tab.
Here it seems that the tab does not load for some reason. Does it work if you git clone comfyui at extensions/sd-webui-comfyui/ComfyUI? To do this, you can either run the install_comfyui.py script manually from a command line interface or clone https://github.com/comfyanonymous/ComfyUI after changing directory to extensions/sd-webui-comfyui.

Closing this for now as there is no answer from op. Feel free to open it again if the issue is still there.


Downgrade AndroidX Test Espresso to 3.4.0

IMPORTANT: Where possible all PRs must be linked to a GitHub issue
Fixes #1932
Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[x] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

awesome this is looking good, will merge on pass

Thanks for merging this @pld
I can see subsequent PRs having their checks passing accordingly 🙌🏾


chore: add a .devcontainer.json file with rust support

Description

Problem*

Resolves
Summary*
I've been using a github codespace to review some PRs from external contributors recently. One issue I've been running into is that the default codespace image doesn't have rust installed so I need to install it each time.
This PR sets the codespace to use an image which installs rust by default to avoid this step.
Relevant links:
https://docs.github.com/en/codespaces/setting-up-your-project-for-codespaces/adding-a-dev-container-configuration/introduction-to-dev-containers#about-dev-containers
Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

This image doesn't have yarn, etc. by default so this needs to be added.

Closing in favour of https://github.com/noir-lang/noir/pull/3097


⚠ AGEG website (prod) has degraded performance

In a3acfed, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 1837 ms

Resolved: AGEG website (prod) performance has improved in 7a92be9.


November 2019 meetup

[x] Speaker: @pyxld-kris, @chungthuang
[ ] Sponsor:
[ ] Confirm venue
[ ] Meetup post
[ ] Blog post
[ ] Tweet
[ ] Order pizza
[ ] Record stats
[ ] Invoice sponsor

The proposed talk is an introduction to DevLaunchers, an initiative to teach high school students in underserved communities about game development. We'll discuss dive into how we build the website, activities, internal tools and slack event handler on Cloudflare Workers, CodeSandBox, Github Pages and Travis CI.


Add timeout to network requests

Network requests should have a timeout in case of an outage. This could happen anywhere between user systems to statsig. This will prevent the network request from taking down app incase it stays long running
Slack thread for context: https://statsigcommunity.slack.com/archives/C01QVL20EDD/p1637613724280900

Looks like one of the tests failed due to an invalid key but dont think thats related to this code change.

The test failure is unrelated - it will pass once I merge


(Maybe) Consume block after running

There should be a boolean which is set to true, after which, further calls to run kill the block.
Maybe this is overkill? Perhaps just preventing two blocks from running at the same time is enough? Is there a usecase for creating a block and running it multiple times?

We should probably do this because I'm not being very strict about checking that when the KonsoleBlock leaves that it's in a clean state. For example, onFinished callbacks. SUre we can fix that, but maybe a future update will break it again.


[SP-6120] Backport of PDI-19423 Calculator step: The result of SQRT(A…

…A+BB) is incorrect (9.2 suite)

8.3: https://github.com/pentaho/pentaho-kettle/pull/8313
9.2: https://github.com/pentaho/pentaho-kettle/pull/8314
<IP_ADDRESS>: https://github.com/pentaho/pentaho-kettle/pull/8315


Add sparse fieldset capabilities on task resources

Closed for the moment, will be rewritten from this base.


Nuxt Axios Fails on POST to route

While I do form submit to some routes with POST with data from FORM, if that route is consuming server side fetch on asyncData and calls some API via store using axios, response fails, or sometime it hangs and gives "Socket Hung Up" error in node console
Step to produce issue

visit https://codesandbox.io/s/codesandboxnuxt-oqk58
Open preview window in new window
Just reload page from browser, you will see ip address fetched from api
But if you click submit button of form, same api fails, and in my project, it hangs, so why this happened and what is this behaviour? What exactly happened there? Looks like nuxt or nuxt/axios issue

This question is available on Nuxt community (#c9311)

Thanks for the report. Should be fixed in 5.5.4


WFLY-8934 Support legacy AUTH configured via property resources

https://issues.jboss.org/browse/WFLY-8934
Requires https://github.com/wildfly/wildfly/pull/10150

Retest this please


Update: Renode.Renode version 1.11.0

Removing old versions to reduce clutter
Microsoft Reviewers: Open in CodeFlow

/azp run


Invalid assets

These assets report errors with glTF validator:
Elf/Elf.gltf
meshes/Yeti/MayaExport
meshes/Demos/retargeting/riggedMesh.glb

I'm using these files for unit testing, which I think it's the purpose of this repository... I would like these to be fixed, or moved to a separate directory for "malformed files", or removed altogether.

I will update the Yeti model, as this is a very old model. However, I don't have the source for the Elf.glTF or the riggedMesh.glb as neither was created in-house. We cannot move them as there are playgrounds which could reference them and are immutable so would break. But I would also say that our assets library is for assets for demos, not for unit testing so I would not rely on this repo for unit testing.


Can't add a file (Using SourceTree)

When I tri to add a file, I get this message:
git -c diff.mnemonicprefix=false -c core.quotepath=false add -f -- app/AppProperties/database.properties
Unknown option -C:/Program Files (x86)/Git/xsltp
error: external filter c:/Users/Jakob/bin/xsltproc xsl/DXLClean.xsl - failed 3
error: external filter c:/Users/Jakob/bin/xsltproc xsl/DXLClean.xsl - failed
fatal: app/AppProperties/database.properties: clean filter 'dxlmetadata' failed
Do you know what is wrong?

Roland, I can't believe that worked, but thanks!


[JENKINS-66428] Prepare SmileHub Notifier for core Guava upgrade

See JENKINS-66428 and JENKINS-65988. Jenkins core is using Guava 11.0.1, which was released on January 9, 2012. Jenkins core would like to upgrade to Guava 30.1.1, which was released on March 19, 2021. Plugins must be prepared to be compatible with both Guava 11.0.1 and Guava 30.1.1 in advance of this core transition.
In particular, this plugin has been identified as using the com.google.common.base.Objects#toStringHelper method, which existed in Guava 11.0.1 but was removed in later versions.
To facilitate the Jenkins core transition, this plugin must be prepared and released such that it works with both Guava 11.0.1 and latest. This PR migrates away from Objects#toStringHelper and rewrites the code to use the native functionality provided by the Java Platform.
CC @hieule2ntq

@hieule2ntq Would it be possible to take a look at this PR?

@hieule2ntq Gentle ping. It would be nice to get this PR merged and released to facilitate the Guava upgrade in Jenkins core (JEP-233).