[BEAM-4099] Rebuild project before running Performance tests on Jenkins

Please see the detailed description in the JIRA issue.

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.

[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

@aaltay could you take a look?


Decouple geolocation translation

The async location code should probably be pulled out since this can be done in the browser, mobile app, node client, etc and isn't core to this library.
It just adds weight to this module.

If you would accept a pull request with adjusted library and example let me know and I'll get that done tonight.

@billyriantono

atm we prefer to use the location module inside the library because we don't need to handle locations outside it and we have updated information in playerInfo


[FancyZones editor] Close with ESC key

📝 Provide a description of the new feature
Would be nice to close the editor (when hot swapping between layouts) with the ESC key.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

Doesn't already work for you?

Nope :(. Not in the layout selection page, also not when editing a layout.

For editing layouts it's not implemented, so that is expected, but the "main" window should get dismissed when pressing esc, for me it works.

You are right, it does work: but there needs be to interaction with the editor window itself first (e.g. clicking the title bar). When it loads it's not in focus and thus not working.
For that use case it's not really an issue, only when opening it up accidently. I'll close this for now..


Allow Disabling Solidity Generation

It is useful for circuits that won't be published on-chain (either for external use, or testing purposes) to not generate solidity files that will never be used.

@sleepyqadir Any chance this could get merged? Thanks!


Get all stats for chart via API, not file

Since we migrated to Postgres we need to get a way to get statistic from there. Now this issue got real


Fails to compile with next dev --turbo

I found that when I switched from turbo (nextjs) to use Webpack. It was ok.
but when I use next dev --turbo again. It has the issues as I showed you above.

thank you for reporting this, investigating

still looking into this, we have a rough idea of what the problem is but no solution yet
as our own nextjs build times are getting up there (for local dev) we're definitely prioritizing it more :)

Take your time ;) I can use without --turbo. Hope you gonna find the solution soon. Cheer!

appreciate the patience!

Ok I have found that you guys already fixed it on 5.29.6


Wrong info

Information for netmask is just a copy of the destination info

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

ID: b5298689-e441-3788-b31d-967dbce4f735
Version Independent ID: 5b078c74-5b82-3b35-5842-8ce6142b2152
Content: route
Content Source: WindowsServerDocs/administration/windows-commands/route_ws2008.md
Product: windows-server
Technology: windows-commands
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

#assign:xelu86

Hy @KillerMZE,
Thanks again for contacting Microsoft. We value your feedback.
A documentation change has been made surrounding the mask/netmask descriptions and will be live after the next business day. We will now be closing this thread.
Thank you and have a most pleasant day.
#please-close


Dock - Tokens not displaying new currencies

Can you look into the dock - tokens not displaying the currencies of Legion?

Have you downloaded my latest changes? I believe this is working:

Dupe of #3


Auth.Basic.Removeheader:true does not work for Chrome

Do you want to request a feature or report a bug?
Bug
What did you do?

Enter basic auth (user:user) for traefik https endpoint and login to Concourse (admin:admin)
What did you see instead?
Enter basic auth (user:user) for traefik https endpoint. Concourse is loaded (static content), but there is still 401 for /sky/userinfo endpoint and Chrome keeps opening basic auth dialog.
Seems that Firefox is ok with this, but Chrome is not.
Output of traefik version: (What version of Traefik are you using?)

traefik:v1.7-alpine image

After digging around it seems, there is a similar problem with Graylog3 running behind Traefik with basic auth entrypoint.


Route order in App.tsx may cause catalog-import to fail

In App.tsx, if you put the /catalog/* route before the /catalog-import/* route, attempting to visit /catalog-import will yield a blank page.
Expected Behavior
The page should show the catalog-import page.
Current Behavior
A blank page is displayed. Seems the /catalog/* route is matched, but that component does not show anything since the URL is not as expected.
Possible Solution
Is this a bug in React Router? Or how backstage uses it? If not easily fixable, maybe a warning somewhere to avoid adopters accidentally arranging Routes in an order that causes this behaviour.
Steps to Reproduce

git clone git@github.com:backstage/backstage.git
In App.tsx, move the /catalog-import/* route below the /catalog/* route.
yarn dev
Visit http://localhost:3000/catalog-import

Context
Stumbled on this while bumping versions to use the new catalog-import. Cost me a few hours of troubleshooting. Might give others the same.
Your Environment

NodeJS Version (v12): v12.19.0
Operating System and Version (e.g. Ubuntu 14.04): OSX 10.15.7 (same issue when running Docker, node:12-buster image)
Browser Information: Chrome latest

Thanks for reporting! Took care of this as part of #3826 which will be rolling out soon


Ollie: Watch<PHONE_NUMBER>

Ollie requests the watch of the watch_number<PHONE_NUMBER>. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Spevacus in Charcoal HQ


Uninstall filter after get_logs() call

@pcppcp not required, but it would be nice to also add the uninstall function after this line: https://github.com/raiden-network/microraiden/blob/master/contracts/utils/logs.py#L146
Maybe after we settle on some libraries, we can use the same log handlers globally.

added. You still need to call it manually.
For deploy it doesn't really matter, but we should put this code i to a library common for both proxy and contracts deploy.


feat: adopt mandatory test 6.1.31 to reflect state of CS02

Update to reflect change in https://github.com/oasis-tcs/csaf/pull/552/files

Closing this in favor of #29


Best practices for integrating Dialogs with existing APIs / applications

I keep running into various serialization issues when attempting to integrate new Bot Framework Dialogs with my existing APIs and applications.
I understand the concept of making all the Dialogs Serializable, but how can I make my other code play nice without requiring [Serializable] attributes down through the whole stack?
Example:

Create a basic Dialog Chain with callbacks to cause a side effect in my own application using

Chain
    .From(() => new PizzaOrderDialog())
    .Do(async (ctx, result) => {
        var pizzaOrder = await result;
        this.pizzaService.Save(pizzaOrder)
    });

In the above example I want to take results from dialogs and store it in my own database.  If you were to modify the Pizza Bot for example and plug in the real code to persist it back to a database or other service where would it go? What are the best practices for writing something similar to the above
The PizzaService or containing business logic class isn't serializable and I don't think there should be a requirement to make it so.
Thanks in advance.

https://docs.botframework.com/en-us/technical-faq/#how-can-i-reference-non-serializable-services-from-my-c-dialogs has some suggestions - happy to discuss more.

@wbreza Did the documentation that @willportnoy referenced help you? Could you solve this issue?

I understand the gotcha's and I'm not completely happy about the hoops that are required to jump through.  I do have a hacky workround in place but I would urge you guys to figure out a more elegant solution.
It sounds like the Node side of the SDK uses a JSON serializer by default which could be a more lightweight solution here as well.

@wbreza feel free to share what your hacky workarounds are here - @willportnoy and I can see if we can help get your code be more elegant if it is possible. Seeing the specific hacks you needed to do might also help us understand how to improve the SDK moving forward :)


feat: Updates Stylelint to v14

Description
Updates dependency Stylelint to v14, which introduces breaking changes in the syntax parser. A project must update the Stylelint configuration object to use a shared syntax like @netcentric/stylelint-config (explained in the Migration guide).
Additionally, Stylelint v13 was not compatible with @netcentric/stylelint-config (which requires Stylelint v14).
Related Issue
Fixes #49.
Fixes dependency tree error due to eslint-plugin-import version mismatch with ESLint v7.
Types of changes

[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All new and existing tests passed.

Feat commit message prefix is not going to bump new major version, just minor.
Use:
BREAKING CHANGE: <breaking change summary>
<BLANK LINE>
<breaking change description + migration instructions>

https://github.com/angular/angular/blob/master/CONTRIBUTING.md#commit-message-footer


Playing Vimeo live stream results in CORS errors

Does the aframe-vimeo-component work with Vimeo live streams?   I tried to connect it to a public Vimeo live event that I was broadcasting, but the video would not play.  Other Vimeo videos that are not live are playing just fine, however.

Hi buddy, I'm looking for the same thing.
Apparently in here they have manage to solve the problem some how by creating a mask that can render HLS streaming; Never the less , I could not do it , therefore it you can I'll appreciate any insight.
https://github.com/aframevr/aframe/issues/1416
On this other link, you'll find a player that can stream HLS streaming by placing them on the search bar. I've test it and it works, however I can't find how to implement it on my own react app.
https://eyevinn-vr-player.herokuapp.com/
Cheers, happy coding....


Check if grid-gutter-width can be used for responsive size padding

Would it be appropriate to use @grid_gutter-width in place of 30px here?

I don't see why not, it's used for all sides to padding on L:785

There are several occurrences where @grid_gutter-width, (@grid_gutter-width / 2), and (@grid_gutter-width * 2) could be used in place of hardcoded values. See https://github.com/cfpb/cfgov-refresh/pull/401 for reference.


cant do c programming

Issue Type: Bug
my code runner is not working properly..............c extension is crashing again and again please help me
VS Code version: Code 1.67.1 (da15b6fd3ef856477bf6f4fb29ba1b7af717770d, 2022-05-06T12:37:03.389Z)
OS version: Windows_NT x64 6.1.7600
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-3550S CPU @ 3.00GHz (4 x 2993)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
undefined

Memory (System)
3.90GB (2.28GB free)

Process Argv
--crash-reporter-id 28567d50-a9bb-4594-b204-c7763d79effb

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

code-runner
for
0.11.7

cpptools
ms-
1.10.2

please solve my issue as fast you can

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines.
Happy Coding!


Vcenter hooks examples (hardenvm)

Bug Report
Expected behavior

Current behavior

Steps to reproduce

Impact

[ ] Low - Annoyance, but does not impact business or functionality
[ ] Medium - Issue can be worked around, but is causing pain
[ ] High - Blocker

Possible solution

Your Environment

Dispatch CLI version (or git commit):
Dispatch Chart version (or image[s] tag):
Operating System and version:
Kubernetes version (and distribution):
Etc (any other useful environmental information):

Merged in #774


Pressing just Ctrl opens help

While I am working on code, I often press Ctrl + S to save my code. But I realized, that if my cursor is on a token, it tries to show help for the token. This often just opens the help file in a new tab before I can even save my code.
With a little bit of more investigation, I found out that it happens on pressing just Ctrl (key-down event probably because it often happens before I even get to the S to save).
What's weirder is it is not there in any of my keybindings (screenshot attached) but is only an issue while the extension is installed/enabled.

This was fixed in the April release of VSCode (I think)
This is the same issue that is in https://github.com/QB64Official/vscode/issues/61
There used to be a setting added  that was added settings.json when it was created, but after the VSCode change the issue quit happening.  So it was removed.
Here is the setting to add to the launch.json "editor.multiCursorModifier": "ctrlCmd"  It doesn't fix the problem just moves to the alt key.   So somewhat better.
A few Qustions:

What version of VSCode are you using?
What version of the extension?
What OS is this happening in?

If you're still having the issue on the latest extension and latest VSCode then this is still an issue.

After some more testing I am able to recreate this with the latest VSCode and extension.
Add the setting does help with the issue.

I added the multiCursorModifier back to the auto created settings.json.
This will not updated existing settings.json.   You will either need to edit yours or all the extension to create a new one.
This is in the latest v0.9.1

Sorry for the delayed response but lgtm now, gonna close this.


The date type is converted to string in route defs

What version of Hono are you using?
4.4.5
What runtime/platform is your app running on?
Node
What steps can reproduce the bug?
The code below contains the ts error. I specified ctx.json, but it still doesn't work although it should.
new OpenAPIHono()
.openapi(
    createRoute({
      method: "get",
      path: "/example",
      responses: {
        200: {
          content: {
            "application/json": {
              schema: z.object({
                updatedAt: z.date(),
              }),
            },
          },
          description: "",
        },
      },
    }),
    async (ctx) => {
      return ctx.json(
        { updatedAt: new Date(), },
        200
      );
    }
  )

What is the expected behavior?
There shouldn't be any ts errors
What do you see instead?
The types of '_data.updatedAt' are incompatible between these types.
          Type 'string' is not assignable to type 'Date'.ts(2345)

Additional information
No response

In JSON when you stringify the content with a Date Object it converts it into an ISO string because there is no Date primitive in JSON. So, even tho you have defined that the response will be a Date instance (because you can do that in zod), but in the response, it will be a string, that you can convert into a Date instance.

@MathurAditya724 The route described above works fine and returns the next response data:
{
  "updatedAt": "2024-06-14T09:53:45.546Z"
}

So, why should I convert all my dates to strings when hono already does it out of the box? Why not just improve the types?

Hi @MichailShcherbakov ( @MathurAditya724 thank you for handling)
This is not Hono core matter, but @hono/zod-openapi matter. I will try to figure out how to handle this issue. Please wait a bit. Thanks.

Hey @MichailShcherbakov
Can you try it with<EMAIL_ADDRESS>It may be solved.

@yusukebe Yeah, there are no errors now, thank you

if i want  to share the schema between BE & FE . the backend schema defined has types as z.date() but gets converted into string type on fetching from FE.


feat(#39): add site for opeo-maven-plugin

Add site for opeo-maven-plugin:

Add templates for generating html pages.
GitHub pipeline to deploy the site.

Closes: #39.

History:

feat(#39): add site yaml pipeline to deploy site generated by maven
feat(#39): add doxia templates
feat(#39): add site.xml
feat(#39): format doxia templates

PR-Codex overview
Detailed summary

Added a new page 404.apt.vm with content for a Page Not Found error.
Updated the README.md file with a link to the Maven site.
Updated the pom.xml file with a new URL and description.
Added a new page index.apt.vm with content for the OPEO Maven Plugin.
Added a new workflow file site.yaml for building and deploying the site.
Added a new site.xml file with project information and menu structure.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@yegor256 reminder

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 3min)

@rultor release, tag is 0.0.3, title is 0.0.3.

@rultor release, tag is 0.0.3, title is 0.0.3.

@volodya-lombrozo OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.3, title is 0.0.3.

@volodya-lombrozo Done! FYI, the full log is here (took me 6min)


Simplify signature monomorphization

In monomorphization, we have a function to monomorphize a component's signature because this needs to be done as we add things to the queue. Externals and non-external components call the same function, which is a little messy because they are handled differently. For non-externals, we rewrite and replace any parameterized expressions that show up (like in signature port widths, for example). For externals, we don't rewrite anything and maintain the parameters, so basically just copy everything over and insert relevant information into our maps. Separating out these two cases more explicitly would be good.

I think there is a further opportunity to simplify things by merging the MonoSig and MonoDeferred structs. Since #310, a component is immediately monomorphized when we come across its instantiation which means there is no point in having a separation to have another struct with the deferral.


Assembly naming problems on Linux

I just tried to run a small console application that uses commandline on a Linux machine. VS publish copies a file CommandLine.dll to the output folder, capitalized with Pascal case. When I copy all files to my Linux machine and try to execute it, I get
The system cannot fnd the file specified
File name: 'commandline, Version=2.0.275.0, Culture=neutral ....
I noticed that the problem is solved if I rename the published file to all lowercase. And it is the only assembly that has this problem in my application!
Is this an issue with the nuget package maybe?

.Net Core or Mono? I glanced through the nuspec and csproj in the repository and both capitalize the dll correctly. I'm wondering if maybe your csproj/project.json references section is incorrectly capitalized. I would expect it to be correct based off the nuspec, but who knows on Windows.

.Net Core on Linux Ubuntu ARM32 (if of any importance). I could try on a Linux x64.
My Project.json has this entry:
"CommandLineParser": "2.1.1-beta"
but I did not enter this manually but used the Nuget packet manager from VS Solution Explorer.

How about in your xproj file? Is there a references element that mentions commandline?

No references at all in there.

Would you be willing to zip the whole thing (source & binaries) and send it to me so I can peek around? You can upload it to Dropbox or something or (if it's just a few hundred kb) feel free to send it to the email addr on my profile.
Core is pretty new, so I'm not too familiar yet with how the source gets turned into IL (w/ its refs)

My project is already quite complex with multiple cross project references and a complex directory structure. I will try to create a simplified version at least without any cross project references tomorrow and send it to you. Thanks for looking into this!

No problem. What version of Visual Studio are you trying this in?

That is Visual Studio 2015 Community Edition and .NET Core 1.1.0 SDK x64

Ok, it gets more complicated :-( The problem doesn't seem to occur on a normal Linux x64. It only occurs on a Raspberry Pi with ARM processor but already with a simple Hello World. I have not tested x86 systems though. I am not sure if you really want to mess around with this and if you even have a Raspberry Pi at hand. Since ARM is not yet officially supported, we should maybe wait until next year for .Net Core 2.0 which was promised to have offical ARM support.

Hmm, I have a Pi but I'm not able to find instructions on how to install .Net Core. Like you mentioned, it seems to not be supported yet.
You did actually solve the problem by lowercasing the filename, right? Would you be okay with that workaround for now (maybe add it as a postbuild step in the MSBuild xproj)? Once .Net Core is officially supported on ARM, feel free to reopen this if it hasn't been fixed.


ts-node v20.0.5 raises "ReferenceError: regeneratorRuntime is not defined" for TypeScript projects with generator functions

Issue
After adding initial Redux sagas (generator functions) to my project, one of my unit test modules failed to initialise with the error ReferenceError: regeneratorRuntime is not defined.
I found that this issue appears when upgrading from ts-jest v20.0.4 to v20.0.5.
Expected behavior
All test modules initialise and run properly, even if the test modules import other modules containing generator functions.
Additional details
My jest config in package.json:
  "jest": {
    "transform": {
      ".(ts|tsx)": "<rootDir>/node_modules/ts-jest/preprocessor.js"
    },
    "testRegex": "(/__tests__/.*|\\.(test|spec))\\.(ts|tsx|js)$",
    "moduleFileExtensions": [
      "ts",
      "tsx",
      "js",
      "json"
    ],
    "moduleNameMapper": {
      "^.+\\.(s?css|less)$": "<rootDir>/utils/styleMock.js"
    }
  },

My .babelrc file:
{
  "presets": [
    "es2015",
    "stage-3",
    "react"
  ],
  "plugins": [
    "react-hot-loader/babel"
  ]
}

The output from installing different versions of ts-node and running my tests:
C:\Users\<me>\WebstormProjects\<product>>yarn add -D<EMAIL_ADDRESS>yarn add v0.24.5
[1/4] Resolving packages...
[2/4] Fetching packages...
warning<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
warning<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
warning<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>[4/4] Building fresh packages...
success Saved lockfile.
success Saved 3 new dependencies.
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>Done in 9.07s.

C:\Users\<me>\WebstormProjects\<product>>yarn test
yarn test v0.24.5
$ jest
 PASS  app\views\components\ConnectionPane\ConnectionPane.spec.tsx
 PASS  app\state\ducks\connection\module.spec.ts
 PASS  app\views\containers\ConnectionPane\ConnectionPane.spec.tsx

Test Suites: 3 passed, 3 total
Tests:       19 passed, 19 total
Snapshots:   0 total
Time:        2.637s, estimated 3s
Ran all test suites.
Done in 3.51s.

C:\Users\<me>\WebstormProjects\<product>>yarn add -D<EMAIL_ADDRESS>yarn add v0.24.5
[1/4] Resolving packages...
[2/4] Fetching packages...
warning<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
warning<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
warning<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>[4/4] Building fresh packages...
success Saved lockfile.
success Saved 2 new dependencies.
├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>Done in 7.57s.

C:\Users\<me>\WebstormProjects\<product>>yarn test
yarn test v0.24.5
$ jest
 PASS  app\views\components\ConnectionPane\ConnectionPane.spec.tsx
 FAIL  app\views\containers\ConnectionPane\ConnectionPane.spec.tsx
  ● Test suite failed to run

    ReferenceError: regeneratorRuntime is not defined

      at Object.<anonymous> (app/state/ducks/connection/sagas.ts:7:15)
      at Object.<anonymous> (app/state/ducks/connection/index.ts:7:15)
      at Object.<anonymous> (app/views/containers/ConnectionPane/ConnectionPane.spec.tsx:7:20)

 PASS  app\state\ducks\connection\module.spec.ts

Test Suites: 1 failed, 2 passed, 3 total
Tests:       18 passed, 18 total
Snapshots:   0 total
Time:        2.934s
Ran all test suites.
error Command failed with exit code 1.

Please create a minimal repo that demonstrates this issue by running npm/yarn install & test.

Not sure what changed - no longer able to reproduce.


content inset from left edge of terminal

Describe the bug
each line of the player begins at between 27 and 32 spaces from the left edge of the terminal
To Reproduce
running zsh am.sh np (default usage)
Expected behavior
text should be left justified
Desktop (please complete the following information):

OS: iOS Monterey

I cannot determine from the code what's causing the indenting. This behavior is probably by design, not a bug, but either way I would appreciate knowing how to adjust the appearance of the player, particularly, the indentation. Thanks very much. Really love this player.

Hi @joeldcanfield, thanks for submitting an issue.  Could you please share a screenshot of your terminal running np?  the album art really should be flush to the left side.  Glad that you like the player, hopefully we can adjust it to your liking.

Indeed

Using Mac's native terminal.

Ah, did you install viu?  That's a dependency for np.  The text is padded to make room for the album art.  If you don't want to, you can run np with the text mode flag.

d'oh. used it on my first test, then forgot. looking for a pure text tool, so I'll alias and script it so it's text mode.
I'm munging the time played/left display and the volume/mute, shuffle/repeat buttons with text as well. I'm an old command line curmudgeon who likes what he likes.
For my edification, which part of the code does the padding for the album covers? I just couldn't find it.

Currently these calls at line 98 of am.sh:
<(printf %s "")
handle the padding.  BUT, you shouldn't need to remove these, you can just call np with am np -t so that it runs in text-mode, flush to the left side of your terminal window.
There isn't a keybinding to exit, I think CTRL-C is standard and it would be redundant to add a key for exiting for a bash script.  But if you think that's the wrong approach, I'd think about adding a q keybinding.

No need for a quit key unless the masses clamor. I sort of expected it, but only because it's common, not because it's necessary.
I do see the padding, and the space left for the art. Thanks.
Another question, but please, let me know when you tire of my pestering: the colors set on lines 64-66 use standard bash codes, and I'm wondering if there's a way to have access to all 256 xterm colors, as I do, for instance, with my zsh prompt.
Thanks again for being will to chat.

No worries!  I love talking about this stuff.  You can call any of the 256 xterm colors the same way, just replace the bash color codes in the bash variables.  For example:
cyan=$(echo -e '\e[00;36m')
Could instead include an xterm color code:
cyan=$(echo -e '\e[38;5;123m')
And that should display correctly if your terminal emulator supports xterm colors.

Perfect. It does.
I'd skipped past the 8-bit ANSI color coding. Useful. Thanks.

Rethinking my position on the 'q' option. Currently, I have to hit 'p', then CTRL-C. If I forget to pause the playback, I hit CTRL-C which kills the script but not the player, then after a moment of confusion while the music continues, I look for one of the other ways to stop the Music app.
An option to both stop playback and kill the script would be more elegant and intuitive.

np was written as a "widget" because it runs in parallel to Music.app, but it cannot run in place of it.  So the philosophy is that you can already have music playing, and decide to run the script.  It's an optional controller, but playback can be played or paused universally with the special Fn keys / Touch Bar, or within Music.app itself.  With that in mind, I wouldn't want the script to halt a user's playback just because they sent a kill signal to the np widget.
I feel that an option to just kill the script could be useful though, thank you for that suggestion.

Yes of course!  Feel free to fork it too!


fix: update text component styling

Description
Update text styles to make sure consumers can override the styles in the text component.
Links
https://jira-eng-gpk2.cisco.com/jira/browse/MOMENTUM-416

Fresh PR is created instead of this. Hence, closing this one.


trailing comma in enums generates spurious error messages

It seems that the fix for #93 is incomplete.
enum A
{
    a,
}

Enum member expected
Expected } instead of EOF

The grammar allows them. The problem becomes serious in a real context since libdparse generates more errors for the code that follows the enum declaration.

don't bother, will send a PR.


TFLite --- TypeError: names_to_saveables must be a dict mapping string names to Tensors/Variables. Not a variable: Tensor("MobilenetV1/Conv2d_0/BatchNorm/beta:0", shape=(32,), dtype=float32)

we can get mobilenet tflite&pb(Mobilenet_1.0_224) from https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/lite/g3doc/models.md
when I use the command:
bazel-bin/tensorflow/python/tools/freeze_graph --input_graph=/path/to/mobilenet_v1_1.0_224_frozen.pb --input_checkpoint=/path/to/mobilenet_v1_1.0_224.ckpt --output_graph=/path/to/freezed_mobilenet.pb --output_node_names=MobilenetV1/Predictions/Reshape_1 --input_binary=true
error occured:
/home/leve/anaconda3/lib/python3.6/site-packages/h5py/init.py:36: FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type.
from ._conv import register_converters as _register_converters
2018-07-16 11:04:48.613520: I tensorflow/core/platform/cpu_feature_guard.cc:140] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 FMA
2018-07-16 11:04:48.728926: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:898] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2018-07-16 11:04:48.729264: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1356] Found device 0 with properties:
name: GeForce GTX 1060 6GB major: 6 minor: 1 memoryClockRate(GHz): 1.7845
pciBusID: 0000:01:00.0
totalMemory: 5.93GiB freeMemory: 5.31GiB
2018-07-16 11:04:48.729276: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1435] Adding visible gpu devices: 0
2018-07-16 11:04:48.873004: I tensorflow/core/common_runtime/gpu/gpu_device.cc:923] Device interconnect StreamExecutor with strength 1 edge matrix:
2018-07-16 11:04:48.873030: I tensorflow/core/common_runtime/gpu/gpu_device.cc:929]      0
2018-07-16 11:04:48.873035: I tensorflow/core/common_runtime/gpu/gpu_device.cc:942] 0:   N
2018-07-16 11:04:48.873192: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1053] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 5086 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1060 6GB, pci bus id: 0000:01:00.0, compute capability: 6.1)
Traceback (most recent call last):
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/tools/freeze_graph.py", line 382, in
run_main()
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/tools/freeze_graph.py", line 379, in run_main
app.run(main=my_main, argv=[sys.argv[0]] + unparsed)
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/platform/app.py", line 126, in run
_sys.exit(main(argv))
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/tools/freeze_graph.py", line 378, in
my_main = lambda unused_args: main(unused_args, flags)
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/tools/freeze_graph.py", line 272, in main
flags.saved_model_tags, checkpoint_version)
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/tools/freeze_graph.py", line 254, in freeze_graph
checkpoint_version=checkpoint_version)
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/tools/freeze_graph.py", line 128, in freeze_graph_with_def_protos
var_list=var_list, write_version=checkpoint_version)
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/training/saver.py", line 1338, in init
self.build()
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/training/saver.py", line 1347, in build
self._build(self._filename, build_save=True, build_restore=True)
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/training/saver.py", line 1384, in _build
build_save=build_save, build_restore=build_restore)
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/training/saver.py", line 813, in _build_internal
saveables = self._ValidateAndSliceInputs(names_to_saveables)
File "/home/leve/tensorflow/bazel-bin/tensorflow/python/tools/freeze_graph.runfiles/org_tensorflow/tensorflow/python/training/saver.py", line 714, in _ValidateAndSliceInputs
variable)
TypeError: names_to_saveables must be a dict mapping string names to Tensors/Variables. Not a variable: Tensor("MobilenetV1/Conv2d_0/BatchNorm/beta:0", shape=(32,), dtype=float32)
System information

Linux Ubuntu 16.04:
TensorFlow installed from source:
TensorFlow version 1.8.0:
Python version:3.6:
Bazel version 0.11.1:
**GCC/Compiler version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.10) **:
CUDA/cuDNN 9.0/7.0.5:
GPU  1060-6G:
Exact command to reproduce:

@longchr123 why are you trying to freeze a frozen_graph?
bazel-bin/tensorflow/python/tools/freeze_graph --input_graph=/path/to/mobilenet_v1_1.0_224_frozen.pb --input_checkpoint=/path/to/mobilenet_v1_1.0_224.ckpt --output_graph=/path/to/freezed_mobilenet.pb --output_node_names=MobilenetV1/Predictions/Reshape_1 --input_binary=true

If what you want to do is to generate a new frozen graph from the mobilenet tflite&pb
(Mobilenet_1.0_224) file, try
bazel-bin/tensorflow/python/tools/freeze_graph --input_graph=/path/to/mobilenet_v1_1.0_224_eval.pbtxt --input_checkpoint=/path/to/mobilenet_v1_1.0_224.ckpt --output_graph=/path/to/freezed_mobilenet.pb --output_node_names=MobilenetV1/Predictions/Reshape_1

instead.

@freedomtan what you said is right, I have tested,but when I write a test code(#20824 ), convert pb to pbtxt, the problem also happend, when execute the command
bazel-bin/tensorflow/python/tools/freeze_graph --input_graph=/path/to/mobilenet_v1_1.0_224_eval.pbtxt --input_checkpoint=/path/to/mobilenet_v1_1.0_224.ckpt --output_graph=/path/to/freezed_mobilenet.pb --output_node_names=MobilenetV1/Predictions/Reshape_1`
I don't know why

I apologize, but I am having a hard time understanding what the problem is, where the problem is, and what version it affects. Please fill out the issue template (https://github.com/tensorflow/tensorflow/issues/new), and please re-paste using the error from trying to freeze a not-frozen graph rather than an already frozen one. Thanks--

@karmel  I should change mobilenet_v1_1.0_224_frozen.pb to mobilenet.pb  from
bazel-bin/tensorflow/python/tools/freeze_graph --input_graph=/path/to/mobilenet_v1_1.0_224_frozen.pb --input_checkpoint=/path/to/mobilenet_v1_1.0_224.ckpt --output_graph=/path/to/freezed_mobilenet.pb --output_node_names=MobilenetV1/Predictions/Reshape_1 --input_binary=true

like this:
bazel-bin/tensorflow/python/tools/freeze_graph --input_graph=/path/to/mobilenet.pb --input_checkpoint=/path/to/mobilenet_v1_1.0_224.ckpt --output_graph=/path/to/freezed_mobilenet.pb --output_node_names=MobilenetV1/Predictions/Reshape_1 --input_binary=true

Code to generate mobilenet.pb files:
def freeze_graph(ckpt_dir):
    output_graph = os.path.join(MODEL_DIR, MODEL_NAME)
    output_node_names = "output"
    saver = tf.train.import_meta_graph(ckpt_dir + '.meta', clear_devices=True)

    graph = tf.get_default_graph()
    input_graph_def = graph.as_graph_def()

    with tf.Session() as sess:
        saver.restore(sess, ckpt_dir)

        output_graph_def = graph_util.convert_variables_to_constants(
            sess,
            input_graph_def,
            output_node_names.split(",")
        )
        with tf.gfile.GFile(output_graph, "wb") as f:
            f.write(output_graph_def.SerializeToString())
        print("%d ops in the final graph." % len(output_graph_def.node))

        for op in graph.get_operations():
            print(op.name, op.values())

Is the action of graph_util.convert_variables_to_constants and command(bazel...freeze_graph ) the same?

@longchr123 not exactly the same, but one of the important steps in freeze_graph is graph_util.convert_variables_to_constants

Nagging Assignee @karmel: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Can you explain your motivation behind freezing the model mobilenet_v1_1.0_224_frozen.pb?
The model is already frozen, so it does not need to run through the freeze_graph tool again. Additionally, if you are trying to use this model in TensorFlow Lite, then you can use the provided mobilenet_v1_1.0_224.tflite file.


Use Either type to reduce object impl redundancy

This makes a significant improvement in the maintainability of the object module.
By including the either crate, we obtain a sum type for data which may be conditionally GZip encoded, that still implements Read.
Minor caveat: this now uses the Either type for reading the volume's data, even for in-memory volumes. This should hopefully not introduce a noticeable overhead.

Nice cleanup, thank you. lgtm.


Fix crash on save-branch-as-scene (2.1)

Fixes #15539.
Fixes #9367.

Sorry for the delay, seems like I merged the master version and forgot this one.

No problem!


How to evaluate android function call[feature]

Is the feature request related to a problem?
Ex. I'm like to see [...]
Describe the solution you'd like
When I run [X], I want to see [Y]
Additional context
Add any other context or screenshots about the feature request here.

I saw the evaluation dataset only has java, js, python but how about android app.


Merging v0.4.5 --> v0.5.0

Too big to fail 😄

Just for completeness - these changes were made under supervision and a final review will be done before a release of 0.5.0.


Drop cssnext bundle

widgets.built.css is the CSS of @jupyter-widgets/controls bundled with cssnext, which includes the polyfill for jupyterlab CSS variables (which we use for the classic notebook) and substitutes them by their values in all of the widgets CSS.
This PR makes thebelab use directly the bare CSS making use of the CSS variables that are already on the page in thebelab.

Note that the tool that was used for this substitution was cssnext which is now deprecated.
We still produce the old widgets.built.css in @jupyter-widgets/controls for backward compatibility.

For some reason, I don't manage to test this locally.


Replacement point formatter class

blah NK_SourceSystemID1 blah    /*NaturalKey_ReplacementPoint|NK_SourceSystemID1|AND*/
blah SampleColumnOne blah    /*DimensionAttribute_ReplacementPoint|SampleColumnOne|,*/
take a line with a tag in it like the above, an object with column lists for NK and for Dim, and then recreate copies of the line replacing that fragment and joining the copies with a newline and the join string (and the comment stripped out). Tag format in comment is pipe delimited: type of columns to put in | string to replace within the line (after comment removed) | join string for separating elements of this list, typically a comma or the word AND
This will the return a string of all these lines

closed by ceab6e40640b6d31a7606cedb4efc3063fdd6f6a


Renderless component

Added the Renderless Component pattern to the docs

I appreciate it, @ShailenNaidoo, and sorry for the late response.
Could you check https://github.com/learn-vuejs/vue-patterns/pull/28/commits/fec91f5d6cf55a9b9895b72dfe20028ecb802789 this commit.
It seems like something like your prettier or lint automatically changed all the files.
Thanks in advance.

@simsim0709 Sure thing! Will check it out 😄


yfinance 0.2.3: Cannot load Ticker Info

Dear Together,
I am facing the following situation:
yfinance module suddenly stopped providing Info today.
History request seems to be working however, while asking "actions" seems to provide "Empty Dataframe".
See the output below.
I found this issue from last month, resembling to my current situation, however its solution didn't solved my problem:
https://github.com/ranaroussi/yfinance/issues/1246
My questions would be:
Anybody facing this same problem currently?
Could you please help what may be the problem here and how to solve it?
What I tried so far:
-uninstall/reinstall yfinance package, with Pycharm restart (with reboot as well, just in case)
-update yfinance: pip install yfinance --upgrade --no-cache (it says "Requirement already satisfied" for all the packages)
-opening a new project with dummy requests
-debugging for the root cause, however I am quite a beginner in python, so no progress
Environment information:
I am using Windows, python 3.10, yfinance 0.2.3
The example code I tried:
asset = yf.Ticker("AAPL")
print(asset.history())
print(asset.actions)
print(asset.info["exchange"])
Its output:
Open        High  ...  Dividends  Stock Splits
Date                                               ...
2022-12-13 00:00:00-05:00  149.500000  149.970001  ...        0.0           0.0
...
2023-01-13 00:00:00-05:00  132.029999  133.960007  ...        0.0           0.0
[22 rows x 7 columns]
Empty DataFrame
Columns: [Dividends, Stock Splits]
Index: []
Traceback (most recent call last):
File "D:...\main.py", line 613, in
print(asset.info["exchange"])
File "D:...\lib\site-packages\yfinance\ticker.py", line 138, in info
return self.get_info()
File "D...\lib\site-packages\yfinance\base.py", line 894, in get_info
data = self._quote.info
File "D:...\lib\site-packages\yfinance\scrapers\quote.py", line 27, in info
self._scrape(self.proxy)
File "D:...\lib\site-packages\yfinance\scrapers\quote.py", line 58, in _scrape
quote_summary_store = json_data['QuoteSummaryStore']
TypeError: string indices must be integers
Thanks for the help in advance!

Duplicate of #1291

Sorry, wasn't aware that the other issue was already started.
Thank you, the package is working now!


[ConstraintSystem] Allow sequence element mismatch fix to produce new…

… holes
Type on the right-hand side of the element conversion/pattern match
should be allowed to have holes to be able to diagnose failures with
structurally incompatible types.
Resolves: rdar://problem/60832876

@swift-ci please smoke test


New validation method to find duplicated entries in observation table

Jira Ticket: VADC-62
New Features

New validation method to find duplicated entries in observation table

The validation will run for specific concept ids that can now be configured in the config .yaml (example added in this PR)

@kmhernan it runs at startup. We can decide later whether we want the process to crash or just to issue a WARNING in the logs (current behavior).


Latest release 1.0.13 have circadian-lighting inside the zip file instead of adaptive-lightning

Version information:
Description:
Latest release 1.0.13 have circadian-lighting inside the zip file instead of adaptive-lightning, causing the installation in HACS and the functionality in HA not to work.
Please try to include:

Second this.

I'm sorry. I've published the last git tag (which was 1.0.13) as a GitHub release. This triggered HACS to downgrade from the latest master commit to a much older version.
I just pushed version 1.0.14, which is based on the latest master commit. This should fix the issue

Just popping in to say this fixed the issue for me. Thanks for the fast fix!!

@beyondtenor well yeah, I broke it so I had to fix it fast. :)
Had some issues in my installation - it didn't want to just upgrade.
So you guys might need to remove it in HACS, restart HA, install it again in HACS, and restart HA - if you run into any trouble.

My experience: HACS wouldn't upgrade it. Restarted HA without removing. Upgraded, restarted HA. I didn't have to remove it and I think the first restart was HACS or cache related.

Ah okay thanks :)

Thanks for the quick fix :)

All good now, thanks for the quick turn on the fix and update!


Add support for deploy token

I use this client via the Terraform provider for GitLab, and have a use case where I'd like to be able to manage a deployment of certain images I have in my GitLab private image repository into Kubernetes.
In order to do this, I need a deploy token with read_registry permissions.
Looking at GitLab API Reference I was not able to find an endpoint that would be suitable to manage this...
However, it seems like this is just an undocumented feature: https://gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/29864
A quick test proved this to be true.
Creating a deploy token:
curl -d name=test -d expires_at="" -d username=test -d scopes[]=read_registry -d https://gitlab.com/api/v4/projects/:project_id/deploy_tokens

{
  "id": 149172,
  "name": "test",
  "username": "test",
  "expires_at": null,
  "scopes": [
    "read_registry"
  ],
  "token": "<redacted>"
}

And retrieving it:
curl  https://gitlab.com/api/v4/projects/:project_id/deploy_tokens

[
  {
    "id": 149172,
    "name": "test",
    "username": "test",
    "expires_at": null,
    "scopes": [
      "read_registry"
    ]
  }
]

A correction from my previous statement: it seems I just didn't look around well enough.
Here's the docs:
https://docs.gitlab.com/ce/api/deploy_tokens.html
I'll be opening a PR soon!
Cheers
Giorgio

Cool, sounds good 👍

I looked further into this... and I realised why I wasn't initially able to find the docs. Basically this is going to be released in version 12.9 in a couple of days. However, GitLab.com is already running this version, but the API is not entirely stable yet. So to anyone who'd like to use this, only GitLab.com works and there is still an issue with usernames.
See this for more details: https://gitlab.com/gitlab-org/gitlab/-/issues/21811


ARM version

Hi,
Actually I have been porting official Docker images for Elasticsearch, Logstash and Kibana to Raspberry with the intention of having a RPi version similar to officials ones.
Due to this reason I forked the repository with de purpouse of bulding a ARM (tested on RPi 2, ARMv7) version of Tini (https://github.com/ind3x/rpi-tini).
Would you mind to build and ARM version? I think that could be such a good idea to have option.
Kind regards!

Thanks; it's not the first time the issue of cross-compiling comes up (see: https://github.com/krallin/tini/issues/30). I'll take another look.
Thanks!

Thanks!
I read that issue before write this. But I was thinking that could be good idea to comment that there is a workaround until a official solution.
Kind regards!

Is there anyway to get this working on a Raspberry PI?  I don't mind a work around, but it appears this issue has been abandoned.  What needs to be done still to get the ARM compile done?

@schwab:
This issue is about providing pre built binaries for ARM, which is a build system issue - I need to cross compile for ARM, but testing the resulting binary is the harder part.
However, Tini compiles as-is for ARM, there's no extra work to do on that front. You have two options to use it on a Pi:

Use a distribution that packages Tini and install from a package. Alpine Linux for example has ARM Tini packages.
Compile Tini yourself. You'll need Cmake, xxd, and a C compiler. Instructions are in the README.

Cheers,

@schwab
Also, if you prefer, you can use my compilation that I share in my repo https://github.com/ind3x/rpi-tini.
It has been tested in a Raspberry Pi 2.

@schwab It'd be great if cross compilation can be added for arm64. For now I'm building and releasing them here https://github.com/Project31/tini. I have a jenkins instance on arm64, so if you want to setup some automated testing then please let me know.

hi, I'm start to port Dockerfile to armhf devices https://github.com/Valantin/rpimage based on armhf semiofficial image https://hub.docker.com/r/armhf/.
I'm intrested to a cross-compiled version of tini in the official repository

Hey everyone here, I have some good news: official ARM builds of Tini 😄
Please this give this latest pre-release a shot: https://github.com/krallin/tini/releases/tag/v0.11.0.pre1.
These are cross-builds, so while I've tested them myself on VMs, I'm interested in hearing a bit of feedback from ARM users before I merge those into an official release :)
I still have a little fix to do in https://github.com/krallin/tini/pull/54, but this is pretty close.
Thanks for your patience!
Cheers,

Great :). I'm traveling now but will give it a go later this week. Thx!
--Kurt
On Oct 30, 2016, at 08:06, Thomas Orozco<EMAIL_ADDRESS>wrote:
Hey everyone here, I have some good news: official ARM builds of Tini 😄
Please this give this latest pre-release a shot:
https://github.com/krallin/tini/releases/tag/v0.11.0.pre1.
These are cross-builds, so while I've tested them myself on VMs, I'm
interested in hearing a bit of feedback from ARM users before I merge those
into an official release :)
I still have a little fix to do in #54
https://github.com/krallin/tini/pull/54, but this is pretty close.
Thanks for your patience!
Cheers,
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/krallin/tini/issues/51#issuecomment-257156554, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACK-D5ByubxsEobaZGor7ulagBiCvpPks5q5LJlgaJpZM4J79iO
.

Great news!! I'm going to test it this days in my projects for being sure that works fine in my RPi 2.

Awesome; thanks for giving it a shot.
Cheers,

Yes! I let you know about result and also, in case of working fine, I will deprecate mi forked repository.

Hi again! I tested following the most common configuration:
ENV TINI_VERSION 0.11.0.pre1 RUN set -x \     && apt-get update && apt-get install -y ca-certificates curl vim \         --no-install-recommends \     && curl -fSL "https://github.com/krallin/tini/releases/download/v${TINI_VERSION}/tini-armhf" -o /usr/local/bin/tini \     && curl -fSL "https://github.com/krallin/tini/releases/download/v${TINI_VERSION}/tini-armhf.asc" -o /usr/local/bin/tini.asc \     && export GNUPGHOME="$(mktemp -d)" \     && gpg --keyserver ha.pool.sks-keyservers.net --recv-keys 6380DC428747F6C393FEACA59A84159D7001A4E5 \     && gpg --batch --verify /usr/local/bin/tini.asc /usr/local/bin/tini \     && rm -r "$GNUPGHOME" /usr/local/bin/tini.asc \     && chmod +x /usr/local/bin/tini \     && tini -h \     && apt-get purge --auto-remove -y ca-certificates curl \     && rm -rf /var/lib/apt/lists/*
and seems like it works fine.
As an anotation, when I execute tini -h the output prints tini (version 0.10.0 - git.271c097) instead of version 0.11.0
`+ tini -h
tini (version 0.10.0 - git.271c097)
Usage: tini [OPTIONS] PROGRAM -- [ARGS]
Execute a program under the supervision of a valid init process (tini)
-h: Show this help message and exit.
-s: Register as a process subreaper (requires Linux >= 3.4).
-v: Generate more verbose output. Repeat up to 3 times.
-g: Send signals to the child's process group.
-l: Show license and exit.`

Perfect, thanks for testing.
It's expected that the version reports a lower number since I have not
actually bumped it yet! :)
On Monday, 31 October 2016, Imanol Urra<EMAIL_ADDRESS>wrote:

Hi again! I tested following the most common configuration:
ENV TINI_VERSION 0.11.0.pre1
RUN set -x
&& apt-get update && apt-get install -y ca-certificates curl vim
--no-install-recommends
&& curl -fSL "https://github.com/krallin/tini/releases/download/v${
TINI_VERSION}/tini-armhf" -o /usr/local/bin/tini
&& curl -fSL "https://github.com/krallin/tini/releases/download/v${
TINI_VERSION}/tini-armhf.asc" -o /usr/local/bin/tini.asc
&& export GNUPGHOME="$(mktemp -d)"
&& gpg --keyserver ha.pool.sks-keyservers.net --recv-keys
6380DC428747F6C393FEACA59A84159D7001A4E5
&& gpg --batch --verify /usr/local/bin/tini.asc /usr/local/bin/tini
&& rm -r "$GNUPGHOME" /usr/local/bin/tini.asc
&& chmod +x /usr/local/bin/tini
&& tini -h
&& apt-get purge --auto-remove -y ca-certificates curl
&& rm -rf /var/lib/apt/lists/*
and seems like it works fine.
As an anotation, when I execute tini -h the output prints tini
(version 0.10.0 - git.271c097) instead of version 0.11.0
`+ tini -h
tini (version 0.10.0 - git.271c097)
Usage: tini [OPTIONS] PROGRAM -- [ARGS]
Execute a program under the supervision of a valid init process (tini)
-h: Show this help message and exit.
-s: Register as a process subreaper (requires Linux >= 3.4).
-v: Generate more verbose output. Repeat up to 3 times.
-g: Send signals to the child's process group.
-l: Show license and exit.`
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/krallin/tini/issues/51#issuecomment-257388263, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABqD1oXKdl6IMoLxs_Iqfw2DICIAvv5Cks5q5js8gaJpZM4J79iO
.

I just released Tini 0.11.0, which now includes ARM builds! 😄
Let me know if you run into any trouble of course.
Cheers,

Hi @krallin, just for notify, I've deprecated the forked repo I created a month ago in favor of yours :D .

Sounds good; thanks @ind3x

All looks good on arm64 too. Cheers! --Kurt
On Sun, Oct 30, 2016 at 1:20 PM, Kurt Stam<EMAIL_ADDRESS>wrote:

Great :). I'm traveling now but will give it a go later this week. Thx!
--Kurt
On Oct 30, 2016, at 08:06, Thomas Orozco<EMAIL_ADDRESS>wrote:
Hey everyone here, I have some good news: official ARM builds of Tini 😄
Please this give this latest pre-release a shot:
https://github.com/krallin/tini/releases/tag/v0.11.0.pre1.
These are cross-builds, so while I've tested them myself on VMs, I'm
interested in hearing a bit of feedback from ARM users before I merge those
into an official release :)
I still have a little fix to do in #54
https://github.com/krallin/tini/pull/54, but this is pretty close.
Thanks for your patience!
Cheers,
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/krallin/tini/issues/51#issuecomment-257156554, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACK-D5ByubxsEobaZGor7ulagBiCvpPks5q5LJlgaJpZM4J79iO
.

--
Kurt T. Stam
twitter: @KurtStam
google+<EMAIL_ADDRESS>


Expose debug pprof port 6060

This pull request exposes port 6060 in the Dockerfile to allow the port to be accessed when the http.debug-pprof-enabled flag is set.

shall we merge this?


Fix incorrect groupId and challenge token code example in README

Signed-off-by: Erko Hansar<EMAIL_ADDRESS>

Good catches, thanks!


finish dataTypes zh translate

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
finish dataTypes zh translate.

Looks like you haven't merged master to your branch for a while and your PR does not show diff from current translation...
cc @sundy-li @zhang2014

I'm trying to solve the conflicts.

@lamber-ken now PR shows that it has way too many unrelated changes

@blinkov  I have a question, should we keep the [Original Artical](https://clickhouse.yandex/docs/en/data_types/boolean/) <!--hide-->  links  in zh docs?

@blinkov I have a question, should we keep the [Original Artical](https://clickhouse.yandex/docs/en/data_types/boolean/) <!--hide--> links in zh docs?

@sundy-li It's supposed to be a link to where the rendered version of current article is published. So you can translate the words in [...] and replace /en/ with /zh/ in url, but if you don't it's no big deal.

I'll apply comments above after merge.

@sundy-li , @zhang2014, @blinkov  thanks for review and suggestion. :+1:


Group text styles in custom classes

Problem

The text styles defined by @panchovm in mockups normally involve more than a font-size and we find ourselves often missing other properties like font-weight and line-height. We can simplify the work of applying these styles correctly (and making future global modifications) by grouping these styles in simple CSS classes or (maybe even adding custom Tailwind utilities?) with the names he has already defined in Figma.

Additional context

This will prevent and solve quickly issues like #1587.
Later I found this related issue #1489 but I find overkill to create components for plain headings, paragraphs, labels, etc.
Implementation

[ ] 🙋 I would be interested in implementing this feature.

Thanks for creating this issue.
I was planning to start working on this and come up with a style update proposal. The current design definitions have some inconsistencies when applied to components and other pieces of text.
To tackle this from the design, I will add the needs design label.

I created this file where I am exploring and testing the styles, but before sharing the approach and some outcomes, I would like to test them online.
How can we mirror the current site and see what components and layouts look like? My first thought was coding them on codepen but for sure there is a better way. It would be great to have a css file or similar and adjust the values to see how everything changes.
Any ideas @krysal?

@panchovm Sorry, I haven't had the chance to look for options. Tagging other folks for any ideas @obulat @dhruvkb  @zackkrida

Thank you for tagging, @krysal!
The difficulty with this issue is that the heading styles are different on different screen widths. Sometimes, the size changes on the md breakpoint, but sometimes - on lg breakpoint. An even larger complication is that the same sizes on desktop can correspond to different sizes on the mobile. For example, Heading 5 on desktop can correspond to mobile style ### Description bold style (for image title on the single detail page) or to Heading 5 (for "Related images" heading)
I tried creating a functional component for Heading that would take the semantic level and the style level for the heading. Functional - to reduce the amount of overhead and prevent worsening of performance. But I couldn't get it to work with Typescript, and found that the different styles for mobile and desktops made this complicated, too. Besides, sometimes we need text styles for buttons or tabs that are not headings.
So, I tried out the utility styles you linked to, @krysal, and they seem perfect for this!
I am going to open a new PR adding utility styles for the styles from Figma designs.

I'm glad that the custom utilities worked out for this case 😃 I'm sure we will find situations when we can't use our custom utilities due to difficulties matching styles but I hope those are exceptions more than the norm.
One thing needed here before a PR, is that @panchovm wanted to review (and likely change some of) those styles @obulat, given the final result often looks a bit different on Figma than on the web browsers. Could you coordinate with him to check the changes applied to the components?

Do I understand it correctly, @panchovm, that in New styles you marked the styles that were changed with pink and the ones that are no longer used with lighter charcoal color?

Yes. But that is still pending. Please do not take it as a final decision.
I wanted to put my hands on some front-end code, but since I will take AFK days soon, perhaps it is worth keeping the Figma test and moving to a simple code test once I have the first outcome.


I've downloaded an existing skill, how do I connect to my remote dynamodb when debugging locally

How do I configure my skill to use my dynamodb table when testing locally.
First time I've tried using this extension for VS code. I downloaded an existing skill of mine, but when I try to run it I get the following error:
Uncaught AskSdk.DynamoDbPersistenceAdapter Error: Could not create table (tablename): Missing region in config
Can the docs explain how we should set this up? I don't know what config this is referring to. A region is specified in my launch.json
I can call AWS.config.update({region:'region'}); in my code but then this falls the wrong thing to do if I want dynamodb tables in multiple regions. I also then get an error about  missing credentials in the config file.
I've found one other issue that references this error but it was closed due to inactivity and no one ever commented on it. I've not used the CLI before, so not sure if I'm missing any additional setup other than installing this plugin and signing in

Hi @TommyBs - We've had some team turnover. Can have @Shreyas-vgr or @sam-goodwin take a look once back from OOTO.


Add limit parameter to single address and batch geocoding

Summary
Implements the possibility to add limit optional parameter in the geocode function, used in both geocode_address and batch_geocode.
Issue
Solves #57.
Testing
curl
In the terminal:
curl -X POST \
  -H "Content-Type: application/json" \
  -d '["4410 S Highway 17 92, Casselberry FL", "17015 Walnut Grove Drive, Morgan Hill CA"]' \
  https://api.geocod.io/v1.7/geocode\?api_key\=03335b96223739b5bb780288b5735b9b43b6b60\&limit\=0

That's to ensure limit=0 is the correct default value ✅
And then:
curl -X POST \
  -H "Content-Type: application/json" \
  -d '["4410 S Highway 17 92, Casselberry FL", "17015 Walnut Grove Drive, Morgan Hill CA"]' \
  https://api.geocod.io/v1.7/geocode\?api_key\=03335b96223739b5bb780288b5735b9b43b6b60\&limit\=1

Python tests
To ensure we're only getting the amount of results we requested ✅
I used my own API key and ran:
python setup.py test

Result:
----------------------------------------------------------------------
Ran 46 tests in 0.338s

OK

tox
On the other hand, I ran tox as well, getting the following result:
  py37: OK (6.64=setup[5.16]+cmd[1.48] seconds)
  py38: FAIL code 1 (2.22=setup[1.85]+cmd[0.37] seconds)
  py39: SKIP (0.03 seconds)
  py310: SKIP (0.00 seconds)
  py311: SKIP (0.00 seconds)
  flake8: FAIL code 1 (2.94=setup[2.64]+cmd[0.30] seconds)
  evaluation failed :( (12.08 seconds)

I think it wasn't working correctly, at least if you follow the CONTRIBUTION readme. I don't think the changes I introduced break anything.

Thank you! Just want to confirm one thing first.
The only thing I'm slightly concerned about is whether limit=0 is documented API behavior for having no limit as opposed to maybe meaning 0 results. @MiniCodeMonkey is that something you could corroborate? 😀
@bruno-uy I don't know why tox didn't work for you, but once I approved the workflow here it all ran fine. And by the way, you do not need your API key for the tests, since by design they don't actually interact with the live API.

Thank you! Just want to confirm one thing first.
The only thing I'm slightly concerned about is whether limit=0 is documented API behavior for having no limit as opposed to maybe meaning 0 results. @MiniCodeMonkey is that something you could corroborate? 😀
@bruno-uy I don't know why tox didn't work for you, but once I approved the workflow here it all ran fine. And by the way, you do not need your API key for the tests, since by design they don't actually interact with the live API.

Thanks @bennylope! Yeah, it would be great if @MiniCodeMonkey could confirm the limit 0 behavior. Empirically it's okay, but I found nothing in the API docs. I'll email him 😅

I can confirm that setting the limit parameter to 0 is the same as asking for no results limit. So it's safe to pass limit=0 per default.
I have also updated the API documentation at https://www.geocod.io/docs/#single-address to clarify this.
Thanks for adding this feature to the library 👍

Thanks for confirming @MiniCodeMonkey 👍🏼
@bennylope, we're good to go with this change, then.

I'll update a release shortly, thanks @bruno-uy
And thank you @MiniCodeMonkey for confirming!


missing ssh option : -J

Hi,
thanks for your fork.
Could you add "-J" option in the ssh available options ?

Good catch! I'll add that. Thanks for the poke.


'exiftool.exe' is not recognized as an internal command.

After run pip install -r requirements.txt, I tried to run the thermal_analysis.py script passing a jpg image as input. But I have the following error:
'exiftool.exe' is not recognized as an internal command. Traceback (most recent call last):   File ".\thermal_analysis.py", line 25, in <module>     obj = CFlir (flir_image_path,color_map=cv2.COLORMAP_JET)   File "C:\Users\user\Documents\thermal\Thermal_Image_Analysis\CThermal.py", line 44, in __init__     self.thermal_np, self.raw_thermal_np, self.meta = self.extract_temperatures()   File "C:\Users\user\Documents\thermal\Thermal_Image_Analysis\CThermal.py", line 145, in extract_temperatures     meta = json.loads(meta_json)[0]   File "C:\Users\user\OneDrive\Documents\Anaconda\envs\enviroment\lib\json\__init__.py", line 348, in loads     return _default_decoder.decode(s)   File "C:\Users\user\OneDrive\Documents\Anaconda\envs\enviroment\lib\json\decoder.py", line 337, in decode     obj, end = self.raw_decode(s, idx=_w(s, 0).end())   File "C:\Users\user\OneDrive\Documents\Anaconda\envs\enviroment\lib\json\decoder.py", line 355, in raw_decode     raise JSONDecodeError("Expecting value", s, err.value) from None json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
What exactly happened here?

Hello @andredavys . Forgot to mention it in the README, but if you're running this on Windows you'll have to do the following:

Download exiftool (for your OS) from here
Extract 'exiftool(-k).exe' to the folder where you're running the code from
Rename it to 'exiftool.exe'

Let me know if this works for you

Hello @andredavys . Forgot to mention it in the README, but if you're running this on Windows you'll have to do the following:

Download exiftool (for your OS) from here
Extract 'exiftool(-k).exe' to the folder where you're running the code from
Rename it to 'exiftool.exe'

Let me know if this works for you

It's work for me

Okay thanks for letting us know, closing this issue then


Need a variable for workspace ID & for user OSC commands

It would be great to be able to get the workspace ID in to a variable.  And also to ba able to make user OSC messages, with or without arguments.  There are so many features of Qlab 5 accessible via OSC that aren't included in this module, but that we could access.  The Problem is that we can't assign passcodes to the generic OSC module in companion, so even though we can send the message, Qlab won't respond.  This module is the only way to get a passcode sent with the messages, so to be able to build our own OSC messages would solve that.
Also any workspace OSC messages need the workspace unique ID as part of them, so if this was available as a variable it would open up a lot more user control.

Due to family and work issues I have not been able to add any QLab5 features, yet.
You can set the workspace ID on the same screen as the IP address.
As you have discovered, you end up hitting limits with the generic OSC module.
Could you make a list of QLab5 features you would like to have added to the module?

Thanks,  I hadn't noticed the workspace option on the setup page.
A lot of the features that would be good to have added aren't even Qlab5 specific.  They could be added to the regular Qlab module as they don't need feedback, it's just that that module doesn't allow for passcode either.  `This is why I suggest adding the ability to write our own OSC messages, with option to send string, integer, or float arguments, much like standard companion OSC module.
The features that need OSC feedback that would be great to have options for are things like, being able to get output names from Qlab.  I know it's possible to get the names, if these could be stored in a variable, so they could be displayed on a button, you could easily see which output fader you were going to bump the level of, PA L, PA R, Sub, Surround etc etc, rather than just 1, 2, 3, 4, etc.
Toggles for things that don't have a control to toggle, where it is possible to query the state.  Such as loop last slice, or integrated fade on off.
Similar to output fader names mentioned above it would be good to be able to get output patch names for things like network, MIDI and anything with an audio output.
A variable for unique cue ID.  Then the ability to make new cues.  When you do this Qlab returns the unique cue ID of the new cue, this could be saved in that variable and then used..  For example, for moving cues into a recently created group.  Or for retargeting cues to the new cue.
Command to save the unique cue ID of the selected cue.  It could go in the same previously mentioned variable, but we could then access that variable in other commands.
Really most of the commands that Qlab can receive via OSC it would be good to be able to access, there are an awful lot of them, so if we could just write them ourself it would surely simplify your coding.  but to have some more variables, as mentioned above that could be integrated into user messages would help hugely.
Sorry, this was a long response.  There is more I'm sure, but this was what came to the top of my head.

I appreciate you taking the time to respond!

A variable for unique cue ID. Then the ability to make new cues. When you do this Qlab returns the unique cue ID of the new cue, this could be saved in that variable and then used.. For example, for moving cues into a recently created group. Or for retargeting cues to the new cue.
Command to save the unique cue ID of the selected cue. It could go in the same previously mentioned variable, but we could then access that variable in other commands.

These have been in the module for quite a while.
Unique IDs are awkward, but sometimes necessary.
I often work on a workspace with over 10000 cues, very few have actual Cue Numbers.
Look for these in the Help document:
n_id and r_id are the playhead(next) and running Cue IDs
There is an action available to 'capture' the current playhead Cue ID that wil autofill the next Goto/Start/Sop/Panic (ID) action added to a button.
The others are more audio related and I'll look into them when I get some free time. Using as few 'custom' commands a possible makes simulating a 'toggle' much easier.

Workspace ID variable added

Transferred remaining ideas to a new Issue


Converting models to Kotlin.

Those models were converted to Kotlin using the Android Studio convert to kotlin action:

Account
AccountType
BaseModel
Book
Commodity
Money
PeriodType
Price
Recurrence
ScheduledAction
~Transaction~
TransactionType.

I've started converting the models to Kotlin. This PR includes only changes that won't require major refactorings only.
I'm trying to keep PRs small, so they are not too tedious to review. A few others are going to follow.

@djbrown: if we're merging this, please squash merge. I'm trying to avoid to rebase + force-push because sometimes github does funny stuff with the PR.

@reul the Transaction model wasn't converted, so I updated the PR description to reflect that.
I guess this class will be converted in an upcoming PR. Thank you so far 😊

Thanks. I'm going to send a PR with the remaining ones later today.


[native filters][cascading filter] Parent level filter component is hidden

How to reproduce the bug

Create a new dashboard with native filters like this:

in the dashboard, 1 chart is only filtered by "Region", 1 chart is only filtered by "Country".  Country filter is dependent on Country filter.

Open dashboard, you will only see "Region" filter.
select a value for region filter, you will only see "Country" filter.
Things became even worse, if multiple children filters depends on one parent filter, but children filters are not in the same scope (out of scope children filters should be folded in the filter bar).

Expected results
Since both "Region" and "Country" filter are used in dashboard, it should all show up in the filter bar.
Environment
Latest master
Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
CC @michael-s-molina @junlincc @rusackas Since we are redesign / fix cascading filter behavior, please consider this use case.

I just stumbled on this issue in 1.4.0.
That's really a pretty awkward bug, as newbie users will be easily confused by this.
On the other hand, if we implement the dashboard filters in a non-cascading fashion, users will be confused by filter options that will not have intersecting result sets.
Is there any progress on the redesign?

This doesn't appear to be a problem anymore. Closing as resolved.


[BUG] Unusually named license_files not included in dist-info

setuptools version
setuptools===57.4.0
Python version
3.9.6
OS
Fedora Linux 33, x86_64
Additional environment information
wheel===0.36.2

Description
I've tried to test out https://github.com/pypa/setuptools/pull/2645 but including license files not named according to this pattern:
https://github.com/pypa/setuptools/blob/0c302d232dd1a1f4ec531050532a84274edac2f4/setuptools/dist.py#L596
Are not included in dist info.
Expected behavior
I expected that files explicitly listed in license_files will always be listed in METADATA under License-File: and will always be included in the dist-info directory regardless of their filename.
How to Reproduce
Consider this setup.py:
from setuptools import setup

setup(
    name='nested_dist',
    version='666',
    license_files=['LICENSE'],
)

And an empty LICENSE file.
Run:
$ python -c 'import setuptools.build_meta; setuptools.build_meta.__legacy__.prepare_metadata_for_build_wheel(".")'

Or:
$ python setup.py bdist_wheel

Or:
$ pip wheel .

You'll see something like:
running dist_info
...
adding license file 'LICENSE'
...
adding license file "LICENSE" (matched pattern "LICEN[CS]E*")
...

The LICENSE file will be listed in METADATA under License-File: and it will be included in the dist-info directory.
Now rename LICENSE to DIFFERENT and adapt license_files accordingly:
from setuptools import setup

setup(
    name='nested_dist',
    version='666',
    license_files=['DIFFERENT'],
)

You'll get:
...
adding license file 'DIFFERENT'
...

The DIFFERENT file will be listed in METADATA under License-File: and it will not be included in the dist-info directory.
Output
See above.
Code of Conduct

[X] I agree to follow the PSF Code of Conduct

Agreed that looks like a bug. Thanks for the report.

@cdce8p Would you be willing to work on a fix?

@hroncok Thanks for the report, but this isn't a setuptools issue. The second entry is added by wheel.
Keep in mind that for wheel to be able to add license files based on patterns, you need to add license_files to the [metadata] section in setup.cfg, not define in in setup(). That will also add the license file to the .dist-info folder, once it's matched. For details, please refer to the wheel documentation: https://wheel.readthedocs.io/en/stable/user_guide.html#including-license-files-in-the-generated-wheel-file
As for the License-File: field in the METADATA: This one isn't in the official packaging spec yet. We only added it because it is useful for other project to be able and easily find the license files form the metadata. There is an effort to make it official, but it will take some more time. Until then, I don't expect wheel to support it.
https://www.python.org/dev/peps/pep-0639/#license-file-multiple-use

I kinda expected that setuptools supports PEP 639 before it is accepted, as a "preview". I understand that the PEP is still a draft.
I don't understand what is the point of listing missing files as License-File. If wheel cannot add files like this, setuptools should not list them and fail instead.

With setup.cfg, it kinda works.
However, when the licenses are in nested directories, as listed in the given example: https://wheel.readthedocs.io/en/stable/user_guide.html#including-license-files-in-the-generated-wheel-file
``[metadata]
license_files =
3rdparty/*.txt`

The license files are included directly in the dist-info directory (e.g. as `license.txt`) and the `License-File:` entries are incorrect (e.g. `3rdparty/license.txt`).

I don't understand what is the point of listing missing files as License-File. If wheel cannot add files like this, setuptools should not list them and fail instead.

If you're referring to the license_files argument in setup(), I agree this should be deprecated / removed.

With setup.cfg, it kinda works.
However, when the licenses are in nested directories, as listed in the given example: https://wheel.readthedocs.io/en/stable/user_guide.html#including-license-files-in-the-generated-wheel-file
The license files are included directly in the dist-info directory (e.g. as license.txt) and the License-File: entries are incorrect (e.g. 3rdparty/license.txt).

That is still a point that needs to be addressed. IMO wheel needs to update their implementation here. Although they do mention the behavior in their docs, it can cause name conflicts, e.g., if a file named LICENSE is present in a sub dir in addition to the main one.
No matter the path, all the matching license files are written in the wheel in the .dist-info
directory based on their file name only.

In that case 3rdparty/LICENSE would overwrite the previous one.
--
I'm open to suggestions if you have an idea how to better resolve this conflict.

Also, what happens if someone names the license file RECORD?
Raise an error if an existing file in dist-info would be overwritten? That'd allow adding better solutions in the future.

The original problem seems to be already solved with the latest versions of setuptools and wheel:
> docker run --rm -it python:3.11 /bin/bash

mkdir /tmp/myproj
cd /tmp/myproj
cat <<EOF > setup.py
from setuptools import setup

setup(
    name='myproj',
    version='666',
    license_files=['DIFFERENT'],
)
EOF
touch DIFFERENT
python -m pip install 'setuptools==66.0.0' 'wheel==0.38.4'
python -c 'import setuptools.build_meta; setuptools.build_meta.__legacy__.prepare_metadata_for_build_wheel(".")'
ls myproj-666.dist-info  # => DIFFERENT  METADATA  top_level.txt
rm -rf myproj-666.dist-info
python -c 'import setuptools.build_meta; setuptools.build_meta.prepare_metadata_for_build_wheel(".")'
ls myproj-666.dist-info  # => DIFFERENT  METADATA  top_level.txt
python -m pip install -U build
python -m build
unzip -l dist/*.whl
# Archive:  dist/myproj-666-py3-none-any.whl
#   Length      Date    Time    Name
# ---------  ---------- -----   ----
#         0  2023-01-20 10:48   myproj-666.dist-info/DIFFERENT
#        73  2023-01-20 10:48   myproj-666.dist-info/METADATA
#        92  2023-01-20 10:48   myproj-666.dist-info/WHEEL
#         1  2023-01-20 10:48   myproj-666.dist-info/top_level.txt
#       367  2023-01-20 10:48   myproj-666.dist-info/RECORD
# ---------                     -------
#       533                     5 files

I think that for the problems regarding name clashes and etc we should wait for PEP 639 implementation.


backport some build changes from Micronaut 4 PR branch

@alvarosanchez I think this PR is good, not sure about the build.

* What went wrong:
Execution failed for task ':oraclecloud-bom:checkVersionCatalogCompatibility'.
> Version catalogs are not compatible:
  The following libraries were present in the baseline version but missing from this catalog:
    - micronaut-oraclecloud-atp

I suppose this is being produced by using the latest micronaut-build version


Trivial default.nix setup times out on Binder

Bug description
The following default.nix setup times out on Binder:
{ pkgs ? import
    (
      fetchTarball {
        name = "21.11";
        url = "https://github.com/NixOS/nixpkgs/archive/a7ecde854aee5c4c7cd6177f54a99d2c1ff28a31.tar.gz";
        sha256 = "162dywda2dvfj1248afxc45kcrg83appjd0nmdb541hl7rnncf02";
      }
    )
    { }
}:
(
  pkgs.python3.withPackages (
    ps: [
      ps.bash_kernel
      ps.jupyterlab
    ]
  )
)

Expected behaviour
Should work as well as nix-shell --pure --run 'jupyter lab intro.ipynb' works locally.
Actual behaviour
Times out after a few minutes; see logs.
How to reproduce

Go to https://mybinder.org/v2/gl/victor-engmark%2Fbash-scripting-intro/HEAD?labpath=intro.ipynb
Wait for it to time out

Your personal set up
Not applicable, and I don't know what the values are on Binder.

This issue has been mentioned on Jupyter Community Forum. There might be relevant details there:
https://discourse.jupyter.org/t/trivial-nix-based-notebook-fails-to-load/13172/5

This is still an issue. Not even the examples repository works.

Yup, I've run into this when trying some things out thinking how #1300 might be implemented.
The weird thing is that running repo2docker locally for a nix-based repository works fine so I guess it's something to do with how JupyterHub detects that the container is ready?


fixed some public api stuff and clippy warnings

Legit, thanks :)
I should add a cargo clippy GitHub Actions, I often do bug somehow forgot on this project.


eqJoin Documentation joinData argument "type"

Even though this JS isn't typed, I take type hints seriously.
In the documentation, it says joinData must be an array, see http://lokijs.org/#/docs#joins
Looking into the code, it can be a ResultSet or even another collection... A little confusing.
I would have done this myself, but I couldn't find the lokijs.org sources here on github :(

If your up for it, you can submit a p.r. to lokijs.org repo here :
https://github.com/techfort/lokijs.org
@techfort has to update the website so that might help him keep the site up to date.

I think I've already submitted the PR for this... Please verify and if so close this issue :-)

great, thanks


Offline downloads happen on a separate thread now.

This Introduces an OfflineDownloadThread class which handles the offline region downloading and limits the number of callbacks that occur on the main thread (mainly status updates). I'd like to take this a step further in a different branch by allowing multiple regions to be downloaded at the same time.

This plugin is now blocked by https://github.com/mapbox/mapbox-gl-native/issues/11461. While the issue is being resolved, I'll be continuing work on cleaning this PR up and documenting the plugin in preparation for release.

I'm ready for another review on this PR when you get a chance @tobrun

A couple of more things that needs to be fixed before we can merge this PR:
Cancel
Clicking cancel does stop the download but doesn't dismiss the notification

MapSnapshot
I create two downloads with different styles but the same snapshot is used

Detail activity
The boundable detail activity is broken

Multiple threads
Since there still some more work to be done before we can merge this let's move to a single thread model instead of creating a new thread for each download.


Master

Hi, it is real good that you hand it in so fast , yet did you finished the protocol part, in markdown file of lec-3 folder, I think that protocol writing is the HW for this week.


Drawer.open causes browser to jump to top of page

I have a nav item that animates into view once a user scrolls down the page with an option to view the cart included. I included the js-drawer-open-right class one of the nav options in order to trigger the Cart drawer to open, which works, but the browser also jumps to the very top of the page. Is there any way to get around this?
Current behavior can be seen here by scrolling down some and clicking on the cart button once it pops into view: https://skinfood-dev.myshopify.com/

This behavior also can be seen when adding products to the cart in a collections page, the cart drawer is opened and the browser jumps all the way to the top of the page. Kind of an annoying side effect of using the drawers.


Unable to add accessory, the setup code is incorrect

I am trying to run this project: https://github.com/unreality/FujiHK  which has a dependency on this HomeSpan repository.
When I try to do the procedure to add the accessory, both iPhone and iPad (iOS 16 and iOS 15.7 respectively) return the error "Unable to add accessory, the setup code is incorrect" and the terminal tells me "SRP Proof Verification Failed".
Could these warnings during compilation be related to the problem?

Am I doing something wrong? Can you help me out?

What pairing code are you using?  Also, you may want to run some of the HomeSpan tutorials first - they are designed to test out various components of the system.

I'm using a custom pairing code because I changed it following the instructions in the repository I linked to earlier, and i've even tried 11122333 without success. What code should I try?

If you type 'S' into the Serial Monitor you can reset the code to whatever you want. Then, type that code into the Home App when pairing.

That's exactly what I did but it doesn't work!

One other thing to check: when HomeSpan first starts it will print out its LPK. It's a really long hexadecimal number.  Does it happen to begin with more than one zero?


Gradient2D Texture crashes Android Application

Godot version
3.5.rc2
System information
HUAWEI Y9 2019 kirin710 arm64-v8a
Issue description
The android application crashes when using a Gradient 2D In a TextureRect node. The order and visibility don't matter, just place a TextureRect node anywhere on the Scene tree, export it to android and the application won't start.

Steps to reproduce

Create a 2D scene (preferably).
Create a TextureRect node.
In properties, select Gradient2D.
Export to android.
Crash.

Minimal reproduction project
No response

CC @Geometror

Could you provide a backtrace or crashlog?
That would be of great help for tracking down the cause.

How do I do that?


1.11.2 Conflict

Whenever I click the search tab with both Rftools Dimensions version 1.1x-4.54 and Blood Magic version 2.1.4-73, my game completely crashes. Here's the log.
http://pastebin.com/rcYHjUGd

Did you report this to bloodmagic? I think it may be a bug with them. Not sure though.


Possible regression

Around two months ago I was experimenting with DoWhy. I installed from conda and set up a simple experiment with some synthetic data.
At that point in time, the estimand expression produced this:
d
───────────(Expectation(Spend|Month,Transactions,Primary Store))
d[Treatment]
Estimand assumption 1, Unconfoundedness: If U→{Treatment} and U→Spend then P(Spend|Treatment,Month,Transactions,Primary Store,U) = P(Spend|Treatment,Month,Transactions,Primary Store)
I recently updated the package, and now when I run the exact same code I see the following:
d
────────────(Expectation(Spend))
d[Treatment]
Estimand assumption 1, Unconfoundedness: If U→{Treatment} and U→Spend then P(Spend|Treatment,,U) = P(Spend|Treatment,)
I've tested this code in your binder setup and it's producing the same as above, so I think we can rule out an environment issue on my end.
Has something changed with regards to how I need to set the DAG up perhaps? This seems to suggest the confounders are being disregarded. Is this really the case? Perhaps there's something I'm not understanding here.
Here's the graph it produces:

I've included the necessary code to reproduce the issue below.
import pandas as pd
import numpy as np
import pydot
import logging
from io import BytesIO

import matplotlib.pyplot as plt
import matplotlib.image as mpimg

import warnings
warnings.filterwarnings("ignore")

month_cols = ['January', 'February', 'March', 'April', 'May', 'June', 'July', 'August', 'September', 'October', 'November', 'December']

def create_spend_data():

    df = pd.DataFrame([[np.random.randint(10,10000) for x in np.arange(12)] for i in np.arange(10000)],
                columns=month_cols)

    df['Treatment'] = [np.random.choice([True, False, False, False, False, False, False, False, False, False]) for x in np.arange(10000)]
    df['Primary Store'] = [np.random.randint(0,100) for x in np.arange(10000)]

    #make Treatment members on average spend ~5% more (some more, some less... but on average 5% more)
    df.loc[df['Treatment']==True, month_cols]  = df[month_cols] * (1 + np.random.random(2).sum()/20)

    df.reset_index(inplace=True)
    df = df.rename({'index': 'Customer'}, axis=1)

    df = df.melt(id_vars=['Customer','Treatment', 'Primary Store'],
           value_vars=month_cols,
           var_name='Month',
           value_name='Spend')

    return df

def create_transactions_data():
    df = pd.DataFrame([[np.random.randint(1,10) for x in np.arange(12)] for i in np.arange(10000)],
                columns=month_cols)

    df.reset_index(inplace=True)
    df = df.rename({'index': 'Customer'}, axis=1)

    df = df.melt(id_vars=['Customer'],
           value_vars=month_cols,
           var_name='Month',
           value_name='Transactions')

    return df

spend = create_spend_data()
transactions = create_transactions_data()
df = spend.merge(transactions, on=['Customer', 'Month'])

graph = """digraph {
"Treatment" [label="Treatment" color="darkgreen"];
"Month" [label="Month"];
"Spend" [label="Spend" color="blue"];
"Transactions" [label="Transactions"];
"Unobserved Confounders" [label="Unobserved Confounders" color="grey"];
"Treatment" -> "Transactions";
"Transactions" -> "Spend";
"Month" -> "Transactions";
"Month" -> "Spend";
"Treatment" -> "Spend";
"Treatment" -> "Primary Store";
"Month" -> "Primary Store";
"Primary Store" -> "Transactions";
"Primary Store" -> "Spend";
"Unobserved Confounders" -> "Primary Store";
"Unobserved Confounders" -> "Treatment";
"Unobserved Confounders" -> "Transactions";
"Unobserved Confounders" -> "Spend";
}"""

pydot_graph = pydot.graph_from_dot_data(graph)

# render pydot by calling dot, no file saved to disk
png_str = pydot_graph[0].create_png(prog='dot')

# treat the dot output string as an image file
sio = BytesIO()
sio.write(png_str)
sio.seek(0)
img = mpimg.imread(sio)

# plot the image
fig = plt.figure(dpi=300, figsize=(2,2))
imgplot = plt.imshow(img, aspect='equal', interpolation='bilinear')
plt.axis('off')
plt.show(block=False)

pydot_graph[0].write_dot('graph.dot')

model = CausalModel(
    data=df,
    treatment='Treatment',
    outcome='Spend',
    graph= 'graph.dot',
    logging_level = logging.INFO
)

# Identify causal effect and return target estimands
identified_estimand = model.identify_effect()
print(identified_estimand)```

As a follow up; for me to get to the original formulation where the confounders are all considered I need to change the way the graph is set up to this. The previous DAG conceptually makes a lot more sense to me though.
graph = """digraph { "Unobserved Confounders" [label="Unobserved Confounders" color="grey"]; "Treatment" [label="Treatment" color="darkgreen"]; "Month" [label="Month"]; "Spend" [label="Spend" color="blue"]; "Transactions" [label="Transactions"]; "Transactions" -> "Treatment"; "Month" -> "Treatment"; "Primary Store" -> "Treatment"; "Transactions" -> "Spend"; "Month" -> "Spend"; "Primary Store" -> "Spend"; "Treatment" -> "Spend"; "Unobserved Confounders" -> "Primary Store"; "Unobserved Confounders" -> "Transactions"; "Unobserved Confounders" -> "Treatment"; "Unobserved Confounders" -> "Spend"; }"""

@paulds8 this is an interesting example. In your first graph, the month variable need not be conditioned on since it lies on a  back-door path that has a collider, so it is not actually confounding the estimate. Primary Store and Transactions do lead to a back-door path between treatment and outcome, so they should be conditioned on. But at the same time, they are caused by treatment and therefore are mediators, and should not be conditioned on, assuming that the goal is to find the total effect of treatment on outcome. This is what creates confusion and the library decides to not condition on them.
Your second graph is cleaner and establishes a separate back-door path for each of the confounders. Assuming that the second graph also captures the relationships between your variables, I suggest to use the second graph.

@amit-sharma, thank you for the resource; I'll give it a read! It should definitely help with my understanding of this.
Would you mind if we unpack this particular case further? I'd really appreciate your views on it.
In this example you can think of the treatment as a customer actively using some kind of rewards/loyalty program that they can subscribe to (there's a monthly cost). Not all customers do so. Customers that do, get discounts in return which in the grander scheme of things potentially leads to more total spend. We want to measure the ATT of this program on total monthly spend over a full year.
The thinking with having the month variable is it allows us to deal with seasonal variations in shopping patterns; e.g. differences around Easter & Christmas vs the rest of the year. This acts on the control and treatment groups equally.
Using the loyalty program potentially has an effect on where people choose to shop and can impact the number of transactions for a given month; but these variables obviously also exist for shoppers that aren't on the program so we'd like to make sure the control/target matching accounts for this as we're expecting differences in total spend as a result of using the program. Where you shop and the number of transactions in the month doesn't necessarily impact whether or not you decide to join the program.
Given this explanation, does the second graph still seem reasonable to proceed with?

@paulds8 sorry I missed your reply somehow. Thanks for the details---I understand the problem better. I am not sure if either of the graphs are accurate here.
Time is the most crucial variable in such a problem. Suppose the customer's signup month is i. The primary store and num_transactions in the months before i are causes of the treatment (signup), and the primary store and num_transactions in the months after i are outcomes of the treatment. So you'll need to split up your data: the easiest is to consider users who signed up in each month separately. Then compute the pre- and post- transaction activity for that month, include the pre-month activity as the confounders and compute the causal effect. Then average over the causal effects over all months (you may also find different effects of signing up in November versus May, for example)
So the correct way is to condition on the activity before the treatment (signup) as confounders. And then ignore the activity after---since it will be a mediator on the path from treatment to final outcome.
To help describe the concept, I've shared a Jupyter notebook with a similar example: https://github.com/microsoft/dowhy/blob/master/docs/source/example_notebooks/dowhy_example_effect_of_memberrewards_program.ipynb
Does this help?
(When trying it, I recommend pulling the latest dowhy from Github master branch. I fixed a few bugs with the backdoor criterion.)

@paulds8 checking if the reply was useful to you.
Let me know if you have any more questions for this issue.

Apologies, I have been on leave. Thank you. This is very useful! I need to make some time to circle back to this particular case in the coming days. I'll re-open if there are any questions so we can continue a coherent conversation.


Update fontMetrics, fontMetricsData, Options, Settings, Style to use flow

This is being implemented in #848.

Implemented with #848