provide OAuth2 authentication for WebSpoon

Expected Behavior
allow WebSpoon to use OAuth authentication for Facebook, Google, LinkedIn, Twitter etc
Actual Behavior
Currently only Memory and LDAP authentication methods are supported.
Info
OAuth support in Spring Security 5.x has been improved/rewritten, e.g. https://www.baeldung.com/spring-security-5-oauth2-login

This is a great suggestion.
Currently, webSpoon uses Spring 4.X just because PDI-CE uses it.
I have to make sure

If webSpoon works with Spring 5.X
If OAuth is configurable by XML config files instead of Java code.

With OpenID, we might be able to do XML-based configuration.
https://docs.spring.io/spring-security/site/docs/current/reference/html5/#ns-openid

"OAuth2Login configuration with XML namespace" not supported yet.
https://github.com/spring-projects/spring-security/issues/4557

Version pinning to Spring 4 has been removed at https://github.com/pentaho/pentaho-kettle/commit/2723d9d2d89eb39c94e04e39139ae6296a285d3b.

@bamaer it seems a bit harder to configure OAuth2 by XML than I thought.
While it may take long to support OAuth2 at webSpoon or even it won't be easily possible, you could use a reverse proxy that supports OAuth2 in front of webSpoon.
           Auth providers (Google, Facebook, etc.)
            |
Client --> Proxy --> webSpoon

I quickly tested and confirmed that this architecture worked.
Here is my docker-compose.yml
version: '2'
services:
  webspoon:
    image: hiromuhota/webspoon
  auth:
    image: bitnami/oauth2-proxy
    ports:
      - "4180:4180"
    command: [
      "--provider=github",
      "--cookie-secure=false",
      "--upstream=http://webspoon:8080",
      "--http-address=<IP_ADDRESS>:4180",
      "--redirect-url=http://localhost:4180/oauth2/callback",
      "--email-domain=*"
    ]
    environment:
      - OAUTH2_PROXY_COOKIE_SECRET=hogehoge
      - OAUTH2_PROXY_COOKIE_DOMAIN=localhost
      - OAUTH2_PROXY_CLIENT_ID=<my secret client id>
      - OAUTH2_PROXY_CLIENT_SECRET=<my secret client secret>

Hope this helps.

Update: pentaho still uses Spring 4.X
https://github.com/pentaho/maven-parent-poms/blob/92f48c55a492f29759e01d99b5166d4e198e59a9/pom.xml#L151-L163


Fix recursive loop in OpenAPI example generation

This fixes an issue where the ExampleTypeFactory would not stop based on the configured depth. This change moves the depth check inside the hasExample helper so that we make sure to check the depth at the beginning of every call and immediately return false if the depth has been reached.

GitGuardian has uncovered 1 secret following the scan of your pull request.

This is just example data included in the API spec.

Superseded by https://github.com/fern-api/fern/pull/3086


Incorrect MAC address format

I am getting an "Incorrect MAC address format" error when trying to turn on a Samsung TV. It used to work in previous version, but no longer working in:
Home Assistant Core 2022.6.5
Home Assistant Supervisor 2022.05.3
Home Assistant OS 8.1
Full error message shown below. I am not sure what format the MAC address should be in in the configuration.yaml file. I have tried separating with colons (:) and with no separation between the pairs.

Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 193, in handle_call_service
await hass.services.async_call(
File "/usr/src/homeassistant/homeassistant/core.py", line 1704, in async_call
task.result()
File "/usr/src/homeassistant/homeassistant/core.py", line 1741, in _execute_service
await cast(Callable[[ServiceCall], Awaitable[None]], handler.job.target)(
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 204, in handle_service
await service.entity_service_call(
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 680, in entity_service_call
future.result()  # pop exception if have
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 964, in async_request_call
await coro
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 717, in _handle_entity_call
await result
File "/usr/src/homeassistant/homeassistant/components/samsungtv/media_player.py", line 460, in async_turn_on
await self.hass.async_add_executor_job(self._wake_on_lan)
File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/src/homeassistant/homeassistant/components/samsungtv/media_player.py", line 450, in _wake_on_lan
send_magic_packet(self._mac, ip_address=self._host)
File "/usr/local/lib/python3.9/site-packages/wakeonlan.py", line 52, in send_magic_packet
packets = [create_magic_packet(mac) for mac in macs]
File "/usr/local/lib/python3.9/site-packages/wakeonlan.py", line 52, in
packets = [create_magic_packet(mac) for mac in macs]
File "/usr/local/lib/python3.9/site-packages/wakeonlan.py", line 31, in create_magic_packet
raise ValueError("Incorrect MAC address format")
ValueError: Incorrect MAC address format

Configuration.yaml
media_player:
  - platform: samsungtv_tizen
    name: lounge_tv
    host: <IP_ADDRESS>
    mac: 54bd7933954e
    api_key: xxxxxxxxxxxxx
    device_id: xxxxxxxxxxxxxxxx

Also tried
 media_player:
  - platform: samsungtv_tizen
    name: lounge_tv
    host: <IP_ADDRESS>
    mac: 54:bd:79:33:95:4e
    api_key: xxxxxxxxxxxxx
    device_id: xxxxxxxxxxxxxxxx

Try
 media_player:
  - platform: samsungtv_tizen
    name: lounge_tv
    host: <IP_ADDRESS>
    mac: "54:bd:79:33:95:4e"
    api_key: xxxxxxxxxxxxx
    device_id: xxxxxxxxxxxxxxxx

I use dashes in my MAC address format. This is my configuration below which works just fine.
`media_player:

platform: samsungtv_tizen
host: <IP_ADDRESS>
mac: 1a-2b-3c-4d-5e-6f
name: "Living Room Samsung TV"
platform: samsungtv_tizen
host: <IP_ADDRESS>
mac: 11-2a-3b-4c-5d-6e
name: "Bedroom Samsung TV"
`

I also think using the quotes and colons as @numbnp used is also valid, as I may have used that at one time.


Docs: point to v3 in installation description

removed php-cs-fixer-v2 urls

Coverage remained the same at 92.175% when pulling 971ea1b7a7f6e14fcd361e5404d27bbda1f5c014 on Jimbolino:patch-1 into 82824602da2f4f2d75b1e4a272b24984790acbb5 on FriendsOfPHP:3.0.

Thank you @Jimbolino.


Possibly unneccessary requests to Lagoon API from Dashboard

Describe the bug
Opening a Lagoon Dashboard build page in a browser tab causes regular requests to the /graphql API.
To Reproduce
Steps to reproduce the behavior:

Open a build page in the Lagoon Dashboard.
Watch the network requests made by the page.
See regular requests to the Lagoon API (e.g. 2 requests/minute).

Expected behavior
I expected that because the build was complete and logs were loaded the page was not in its final state and no further requests would be made in the background.
Screenshots
Connection log shows this:

Console has:

Additional context
This issue causes significant load on large Lagoon installations where lots of people have the Lagoon UI open in a tab. It can result in thousands of requests per minute.
@rocketeerbkw advises that this issue may be related to https://github.com/uselagoon/lagoon/issues/1344

These are GraphQL subscriptions polling for changes over websocket connections which pulls in new build information automatically and seems the throughput is too high for node to handle. Although from the screenshots it looks as if multiple connects are opening when I assumed there should only be 1 per client 🤔 .  Perhaps we can look at setting up this with redis?


fix: go to workspace chat bug

Explanation of Change

Fixed Issues

$ https://github.com/Expensify/App/issues/53287
https://github.com/Expensify/App/issues/53252
PROPOSAL:
Tests

Case 1

Go to workspace settings > Profile.
Hover over workspace avatar under "Submit expenses using your workspace chat below:"
Note that the tooltip width does not cover the avatars and remember the width of workspace avatar tooltip.
Change the workspace name to longer name (as long as possible).
Hover over workspace avatar under "Submit expenses using your workspace chat below:"
Verify that: the workspace avatar tooltip should adjust accordingly after workspace name is updated.

Case 2

Log in to both new dot and old dot with the same account in different tabs
Create a new control workspace in OD
Navigate back to the tab where you are logged in new dot
Go to workspace list > Click on the control workspace you created
Verify that:  row "Submit expenses using your workspace chat below" is not displayed

[x] Verify that no errors appear in the JS console

Offline tests

same as above
QA Steps

// TODO: These must be filled out, or the issue title must include "[No QA]."
same as above

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I used JaimeGPT to get English > Spanish translation. I then posted it in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.ts or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I added unit tests for any new feature or bug fix in this PR to help automatically prevent regressions in this user flow.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/5a9c1686-2690-4a04-9deb-7d036a9be8c1

Android: mWeb Chrome

iOS: Native

https://github.com/user-attachments/assets/9b33c981-2ebf-4faf-8fc8-7dc28fde972a

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/9c4a6ca4-3a17-4f5d-bbcc-f4f5f2add10a
https://github.com/user-attachments/assets/2f48c9d8-11e8-4e5a-8b8b-34cf20f5cf5a

MacOS: Desktop

https://github.com/user-attachments/assets/785cb890-bc35-477d-950c-1078d5351e96

Please changes the following test steps

Note that the tooltip does not cover the avatars and remember the width of workspace avatar tooltip.

Note the tooltip width

Create a new control workspace

Create a new control workspace in OD

Is native build broke on main?

Is native build broke on main?

I am a little stuck on the Android and iOS Native builds. Will add soon

Please changes the following test steps
Note the tooltip width
Create a new control workspace in OD

@s77rt I updated

I updated step 3 here

It should be just: "Note the tooltip width". I find the other part especially the "does not cover the avatars" part confusing

thanks @s77rt, i updated


Better configuration options

Previously this code needed to be aware of the size of a Supernode when used in combination with GraphZeppelin. However, one could imaging using this code for another purpose where that wouldn't make much sense and more importantly the size of a Supernode changes and the size we had expected was hard-coded in this repo.
Therefore, users now specify the size of a guttering system leaf in bytes in the configuration. Additionally, we allow arguments to be unspecified in a way that can be seen by those inspecting the configuration. This allows GraphZeppelin to set the size of a leaf if it has not already been set by a user.

This will require changes in a lot of other repos. So please review but don't merge yet.


UserConfig.write is unsafe

The module UserConfig cannot serialize library versions correctly because the underlying OCaml binding of libyaml is serializing the string "1.0" as the floating point 1., "null" as null (unquoted), etc. This means users have to manually edit the configuration files.
The bug report to the upstream: https://github.com/avsm/ocaml-yaml/issues/39

Solved by #28 and #30.


[REQUEST] nodejs20.x runtime support

Now that node 20 is LTS and available in lambda, it would be nice to support that.
https://nodejs.org/en/blog/release/v20.9.0
https://aws.amazon.com/blogs/compute/node-js-20-x-runtime-now-available-in-aws-lambda/
Tested with
@sparticuz/chromium 119.0.0
puppeteer-core 21.5.0

Current Errors
Failed to launch the browser process!
/tmp/chromium: error while loading shared libraries: libnspr4.so: cannot open shared object file: No such file or directory

Unfortunately it looks like the node 20 runtime uses Amazon Linux 2023 and could have a few missing packages.

Yay! I saw the runtime was released yesterday. I'll be working on this soon!

Updated immediately but I'm getting the following error:
browserType.launch: Failed to launch: Error: spawn C:\Users\cspro\AppData\Local\Temp\chromium ENOENT
          Call log:
            - <launching> C:\Users\cspro\AppData\Local\Temp\chromium --disable-field-trial-config --disable-background-networking --enable-features=NetworkService,NetworkServiceInProcess --disable-background-timer-throttling --disable-backgrounding-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-background-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=ImprovedCookieControls,LazyFrameLoading,GlobalMediaControls,DestroyProfileOnBrowserClose,MediaRouter,DialMediaRouteProvider,AcceptCHFrame,AutoExpandDetailsElement,CertificateTransparencyComponentUpdater,AvoidUnnecessaryBeforeUnloadCheckSync,Translate,HttpsUpgrades --allow-pre-commit-input --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --force-color-profile=srgb --metrics-recording-only --no-first-run --enable-automation --password-store=basic --use-mock-keychain --no-service-autorun --export-tagged-pdf --disable-search-engine-choice-screen --headless --hide-scrollbars --mute-audio --blink-settings=primaryHoverType=2,availableHoverTypes=2,primaryPointerType=4,availablePointerTypes=4 --no-sandbox --allow-pre-commit-input --disable-background-networking --disable-background-timer-throttling --disable-backgrounding-occluded-windows --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-background-pages --disable-component-update --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --disable-sync --enable-automation --enable-blink-features=IdleDetection --export-tagged-pdf --force-color-profile=srgb --metrics-recording-only --no-first-run --password-store=basic --use-mock-keychain --disable-domain-reliability --disable-print-preview --disable-speech-api --disk-cache-size=33554432 --mute-audio --no-default-browser-check --no-pings --single-process --disable-features=Translate,BackForwardCache,AcceptCHFrame,MediaRouter,OptimizationHints,AudioServiceOutOfProcess,IsolateOrigins,site-per-process --enable-features=NetworkServiceInProcess2,SharedArrayBuffer --hide-scrollbars --ignore-gpu-blocklist --in-process-gpu --window-size=1920,1080 --use-gl=angle --use-angle=swiftshader --allow-running-insecure-content --disable-setuid-sandbox --disable-site-isolation-trials --disable-web-security --no-sandbox --no-zygote --headless='new' --user-data-dir=C:\Users\cspro\AppData\Local\Temp\playwright_chromiumdev_profile-nrcmSh --remote-debugging-pipe --no-startup-window

Package versions:
"@sparticuz/chromium": "119.0.2",
"playwright-core": "1.40.0",

Node version: 20.9.0
Code:
import chromium from "@sparticuz/chromium";
import { chromium as playwright } from "playwright-core";

...

      const browser = await playwright.launch({
        args: chromium.args,
        executablePath: await chromium.executablePath(),
        headless: !!chromium.headless,
      });
      const context = await browser.newContext();
      const page = await context.newPage();
      await page.setContent(html); // or 'await page.goto("...");'
      await page.pdf({ path: "public/exports/export.pdf" });
      await context.close({
        reason: "Exports finished, closing the context.",
      });
      await browser.close({
        reason: "Exports finished, closing the browser.",
      });

Any update on this?

Node 20 is supported with the latest version

Node 20 is supported with the latest version

Hi Spartacuz! So which packages exactly needs to be updated to fix this issue? and in which version it did get fixed?
I can see in the PR responsable for the fix there is a lot going on.. so wondering what exactly is the fix..
Simply updating
"@sparticuz/chromium": "116",
"puppeteer-core": "21.1.0",

To which version?
And nothing else?

Thank you!

I was able to use chromium and puppeteer on nodejs20 runtime with following config:
in package.json:
    "@sparticuz/chromium": "126.0.0",
    "puppeteer-core": "^22.13.1",

with the aws lambda layer chromium-v126.0.0-layer.zip (download here)

I also can confirm it works with Node.js v20, issue can be closed.


Performance-Problems?

When I have
format_on_save: true
it seems that the perfomance of my Sublime gets during days slower when I save files.
Honestly I have to say I installed the plugin on Sublime2 but maybe these issue occurs also on Sublime3.
OS: Win7 Professional, 64Bit

I'd be happy to accept a PR but unfortunately I'm not really doing active development on this anymore and don't have a windows box to test on.


Update build system to flit_core

Hi, I'm opening this pull request as part of a push to modernise how packages use Flit as a Python build backend.
Using flit_core as the backend in place of flit is recommended in the docs, and will make it faster for tools like pip & build to build your package from source, as it has fewer dependencies to install.
Specifying an explicit version range (>=2,<4) helps to ensure that your package can still be readily built from source despite changes in future major versions of Flit, because it will still use version 3.x. For instance, a future version is likely to drop support for the [tool.flit.metadata] table, in favour of the now-standardised [project] table for metadata. This is also in the docs, along with details of which versions support which features.
Most users probably install your package from a pre-built 'wheel' on PyPI, so this changes won't affect them at all. But people who install from a git checkout, for instance, will benefit.

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

flit also depends on flit_core - the easiest way to verify this is by installing flit into a fresh virtualenv, you'll see flit_core getting installed as well. The build actually uses flit_core either way; installing flit means some other packages are pulled in with it, which aren't really needed to build packages.


Point to feature branch of dbt-tests-adapter for testing, this should not be merged

We can avoid PRs like this in the future with this ticket: https://github.com/dbt-labs/dbt-adapters/issues/101


⚠ Empornium has degraded performance

In 3cb4b6f, Empornium ($EMP) experienced degraded performance:

HTTP code: 200
Response time: 1455 ms

Resolved: Empornium performance has improved in 4aebc16.


How to expand only some rows on the basis of some condition, hide/remove expandable icon of some rows

I am using "expandableRows". I want to expand some rows on the basis of some condition. I can achieve this with the help of "isRowExpandable". But it is not helping me to hide/remove expandable icon. I want to hide/remove expandable icon of some rows.

Hi @patorjk .
See this example https://codesandbox.io/s/dreamy-meadow-oghwd
I want to hide "Expandable icon" if there is no "inner row".  Like row with name "John Walshs". Is single row. No row inside it. So I want to hide "Expandable icon" of it.
Also one more issue here, if am hiding column of table from "view columns" feature, I want to hide column of "inner row" as well. Can we do this?

Added a fix for this in version 3.4.0. You can now override the ExpandButton component. Example: https://github.com/gregnb/mui-datatables/blob/master/examples/expandable-rows/index.js#L119

Thanks @patorjk
Thanks for providing solution quickly. It's working fine. Thanks

Added a fix for this in version 3.4.0. You can now override the ExpandButton component. Example: https://github.com/gregnb/mui-datatables/blob/master/examples/expandable-rows/index.js#L119

Hi, thanks for this! So I've tried this solution, but there's something that I can't do with it.
Here's the problem, I need to hide the expand button on specific condition, I need to see if the value on a column is null, for example, as of now, I'm unable to do that because the props to override the ExpandButton doesn't have the value for the column I'm refering to. There are only isHeaderCell, expandedRows, expandableRowsHeader, areAllRowsExpanded, onExpand inside the props.
I can just use the index to set isRowExpandable as false, but then the index is messed up if I filter or sort the table by clicking the table header.
Do you know a workaround for this? Thank you!

I have the same issue. I had a function which returns the indexes based on some condition, of my original data-source for which I would like the rows to be expandable. However when I filter, it changes the table internal data-source not my original data-source so the index references are wrong. Is there a way to pass the tablerowData to the isRowExpandable method?

I know this is a couple of years old but I am also having this issue where I'm trying to hide the dropdown based on the rowData but there is no way to access the rowData when using the solution above. Is there a workaround for this?

I know this is a couple of years old but I am also having this issue where I'm trying to hide the dropdown based on the rowData but there is no way to access the rowData when using the solution above. Is there a workaround for this?

I would try using React-Table. It is headless so you can style it however you want.


Update link in Contributions section

Previous link went to 404

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

As far as I can tell, https://github.com/dotnet/coreclr/pull/11944 intentionally removed that document, so I think that this link should be removed, not redirected to a branch that still has it.
cc: @richlander

You're right; https://github.com/dotnet/coreclr/pull/11944/commits/9a94cd2b5c8fe5143633c4823330e9d1c09fdef8 shows the deliberate cleaning out of that. I'll remove the link entirely.

@richlander did you remove the doc on purpose, or was it just moved to core repo?

Ya, I deleted the file on purpose. It wasn't very useful any longer.


Handling for really big text

@Andrew-Sem also, I think we could use a text box to answer instead of just input, what do you think?

@l3r8yJ yep, I think that we need more space for text, I'll think about it


Add form to insert comments and store them on the server.

Add form to insert the comments, store them in a deque in the server to maintain the order and use the gson library to create the JSON response.
At the moment it is possible to insert a comment without specifying an author and it is set to Anonymous, also it is possible to insert an empty comment, but I think I will modify it so that I won't allow this.

not sure if this is ready for a review but I anyway left a few comments. Please assign it to me once you think it's ready.

I added some controls for the empty comment and for the client side part I only added the required attribute, is that enough or do I have to implement a javascript function?


Upload photo

is posible upload photo? i try this:
var promise = Client.Session.create(device, cookiePath, 'username', 'pass')

promise.then(function(sessionInstance) {
  Client.Upload.photo(sessionInstance, 'test.jpg').then(function (out) {
    console.log(out)
  })
})

return Upload object, but no upload 😢

@jlobos of course its possible :P here is how to do it:
// import Client as Client.V1

var pathOrStream = './path/to/img.jpg';
Client.Session.create(device, cookiePath, 'username', 'pass')
.then(function(session){
  return [Client.Upload.photo(session, pathOrStream), session]
})
.spread(function(upload, session) {
  return Client.Media.configurePhoto(session, upload.params.uploadId, "you can provide caption or go let it go")
})
.then(function(medium) {
  console.log(medium) // -> return Instance of Client.Media
})

should work that way, try it out...

@huttarichard awesome, very thank!


2 factor authentication doesn't work

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

youtube-dl -v -f 'bestvideo[height<=?1080]+bestaudio/best' --username "won't say" --password "won't say" -2 901316 $v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-f', 'bestvideo[height<=?1080]+bestaudio/best', '--username', 'PRIVATE', '--password', 'PRIVATE', '-2', '901316', 'https://youtu.be/OvO_G5Q0aUI']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2024.05.31 [21924742f] (single file build)
[debug] ** This version was built from the latest master code at https://github.com/ytdl-org/youtube-dl.
[debug] ** For support, visit the main site.
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-6.5.0-35-generic-x86_64-with-glibc2.35 - OpenSSL 3.0.2 15 Mar 2022 - glibc 2.35
[debug] exe versions: ffmpeg 4.4.2, ffprobe 4.4.2, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] Downloading login page
[youtube] Looking up account info
WARNING: Unable to look up account info: HTTP Error 400: Bad Request
[youtube] OvO_G5Q0aUI: Downloading webpage
ERROR: Private video
Sign in if you've been granted access to this video
Traceback (most recent call last):
  File "/home/lieta/.local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 863, in wrapper
    return func(self, *args, **kwargs)
  File "/home/lieta/.local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 959, in __extract_info
    ie_result = ie.extract(url)
  File "/home/lieta/.local/bin/youtube-dl/youtube_dl/extractor/common.py", line 571, in extract
    ie_result = self._real_extract(url)
  File "/home/lieta/.local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 2219, in _real_extract
    raise ExtractorError(reason, expected=True)
youtube_dl.utils.ExtractorError: Private video
Sign in if you've been granted access to this video

Description

Downloading a private video from youtube with 2FA and code from Google Authenticator app doesn't work.

The username/password options almost never work because sites change their login systems and/or require something like a Captcha that can't be handled in the yt-dl login framework.
Try passing cookies exported from your logged-in browser session where you can access the video.


generateSchema(sails.models)

using
import { generateSchema } from 'sails-graphql';
 generateSchema(sails.models);

causes
TypeError: Cannot read property 'charAt' of undefined
Client.js
module.exports = {

  attributes: {
    id: {
      type: 'string',
      primaryKey: true,
      required: true
    },
    firstname: {
      type: 'string'
    },
    lastname: {
      type: 'string'
    },
    therapists: {
      unique: true,
      collection: "therapist",
      via: "clients"
    }
  }
};

Therapist.js
module.exports = {

  attributes: {
    id: {
      type: 'string',
      primaryKey: true,
      required: true
    },
    firstname: {
      type: 'string'
    },
    lastname: {
      type: 'string'
    },
    clients: {
      unique: true,
      collection: "client",
      via: "therapists"
    }
  }
};

+1

+1

The root of this problem appears to be when types, queries, and mutations are attempted to be created for waterline-generated join models. You can get around this by filtering the models with hasSchema == true. However, this just opens up the door for many other issues. Overall, the library is not ready for use.

+1

+1

Any updates on this?

any updates?

I'm sorry for not looking at this project for a very long time. I just created it for fun a few years ago and didn't expect that it can be useful for someone else. Honestly, I forgot that it even exists :)
I'll look into this issue in the near future.


Fix prometheus and grafana discovery

Signed-off-by: MUzairS15<EMAIL_ADDRESS>Description
This PR fixes #5967

Prometheus and Grafana discovery,
Missing hostname in connectivity test

https://user-images.githubusercontent.com/79860504/183157007-a5e860cb-b606-4bf1-8ff4-ed9393936bdb.mov
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Yay! They're back!


convert AllocationBatchJob#urn to string

Context
Changes proposed in this pull request
Guidance to review

 SCL bulk upload not saving the provision_urn with the job


Settings options for Kliva

We still need to expand the use of our Settings class, we'll add a feture list here as reference what's todo and what's done.

[ ] Keep track of the last selected Activity feed filter
[ ] Have a sorting option for Activity feed, ordered on activity start datetime or activity end datetime
[ ] Let user decide what metric unit he prefers

We need to add the StravaService.SetMetricUnits again, when returning Activities!
With the commit of the cached incremental collections we lost this.
Also we need to think of triggering this in the collections when the user changes his preference on the screen


Kwikset 916 not locking or unlocking from lock keypad

Hi Erik!
History: I had one Kwikset 916 lock & I was using a copy of the Lock Manager which was an older version which was only 2 modules (device type & lock-manager itself)...This lock could be locked/unlocked from the keypad as well as from the SmartThings app...Then I needed to make a few changes to my configuration:

Removed current lock from hub thru the app.
Deleted copy of SmartApp from The IDE.
Changed board in current lock (as batteries were draining in just a few days) & added it back to the hub thru the app.
Added a new Kwikset 916 (different color) to another door and added it to the hub using the app.
Added Github repository integration & linked to your LockManager folder (ethayer/Lock-Manager) using the IDE.
Added the device handler & the parent & 2 of 3 child apps from that repository thru the IDE & published all these apps...I did not add/publish the keypad child app as I don't have a separate keypad.
Configured

the LockManager app for both locks thru the SmartThings app...Wondering if I missed something here?
Issue: Both locks work properly (open/close & report back on manual operation) from the app...Neither lock responds properly when keypad is used...If I activate the lock keypad & hit the lock symbol, it beeps & "1-3-5-7-9" flashes...Some additional info:

5 codes were entered from app...None were entered at lock from keypad...The same beep / "1-3-5-7-9" flash happens if I try one of these codes when the lock is locked...I assume this will get fixed once I can get it to lock from the keypad.
No errors are displaying in the IDE log when the keypad in unsuccessfully used...The manual lock/unlock operation is being logged which is probably why I also see that notification on my phone.
Both locks are functioning the exact same way.

Any ideas?  Need any additional info or clarification?
Thanks for the help, Mike

I'm having the same issue with my kwikset 916.  It was working flawlessly for a long time then something broke.

I'm having the same problem keys you enter in the app don't push to the actual 916 to unlock the door.


Invalid time value

Describe the bug

First appearance: 2.4.3 after updating it from 2.3.1.

Thanks for the feedback!

Were you doing something particular when the error appeared?
What screen were you on? (main/update/etc...)
In case you can share the two log files (main.log and renderer.log) that you find in settings > open log files folder

I booted the application and the error appeared out of nowhere.
Main screen
Log file & Renderer File

Thanks!
Does the error reappear when you change the page? maybe it happens on a specific page?
May I ask if you can also attach the three databases with the .db extension that you find in (log folder > upper folder > appcache)?
Maybe with the same data I can reproduce the problem on my PC.
Probable cause:
The only thing that has to do with time (and that can give this kind of error) is the string-to-date conversion of the last game update (or main thread post). If that's what I think, some game has an incorrect string format and when the application tries to convert it this error occurs.

Here you go: https://cdn.knuffelbeestje.com/YAM-Appcache.rar

Fixed, the issue was due to a recommended game with a non-ISO 8601 date.
Patched in #61, closing


Unify embedding model / tokenizer for builtin source storages?

We currently used different embedding models and tokenizers for our builtin source storages:

Chroma:
https://github.com/Quansight/ragna/blob/e85129752682e38dc7f2ef9622446f3ba5a168e9/ragna/source_storage/_chroma.py#L39-L42
LanceDB
https://github.com/Quansight/ragna/blob/e85129752682e38dc7f2ef9622446f3ba5a168e9/ragna/source_storage/_lancedb.py#L36
https://github.com/Quansight/ragna/blob/e85129752682e38dc7f2ef9622446f3ba5a168e9/ragna/source_storage/_lancedb.py#L51-L52

So far I've just used whatever the documentation of the respective tool suggested. For Chroma that wasn't really an issue. However for LanceDB, added in #66, this added tons of heavy dependencies:
https://github.com/Quansight/ragna/blob/e85129752682e38dc7f2ef9622446f3ba5a168e9/ragna/source_storage/_lancedb.py#L21-L25
Since we build ragna.builtin_config at import time
https://github.com/Quansight/ragna/blob/e85129752682e38dc7f2ef9622446f3ba5a168e9/ragna/__init__.py#L35-L38
and PackageRequirement.is_available() performs the import, we now have a crazy overhead:

With LanceDB$ time python -c "import ragna"
python -c "import ragna"  6,23s user 2,41s system 124% cpu 6,930 total

Without LanceDB$ time python -c "import ragna"
python -c "import ragna"  2,28s user 1,63s system 156% cpu 2,497 total

Without LanceDB and Chroma$ time python -c "import ragna"
python -c "import ragna"  1,26s user 0,20s system 99% cpu 1,458 total

That in itself wouldn't be the issue if the specific embedding model / tokenizer is required for LanceDB. But it isn't.
My proposal is twofold:

We should use the same embedding model / tokenizer for all builtin source storages. Since the source storages basically just store vectors, I currently can't imagine a case where one would require a specific configuration. Even then, providing the same for all other source storages means that we keep our dependencies to a minimum and in turn also the import time.
Instead of using a "random" embedding model / tokenizer, we should lightweight ones. The ones used by Chroma look like a good starting point, but maybe we can do better? @dillonroach do you have insights here?

With #72 we no longer have the massive overhead at import, but still we pulling in multiple GBs as dependencies. A docker image based on python:3.11 is ~6GB big.
The folks over at chroma had the same issue and solved it:
# In order to remove dependencies on sentence-transformers, which in turn depends on
# pytorch and sentence-piece we have created a default ONNX embedding function that
# implements the same functionality as "all-MiniLM-L6-v2" from sentence-transformers.
# visit https://github.com/chroma-core/onnx-embedding for the source code to generate
# and verify the ONNX model.


Spellcasting bonus and penalties

If there is any + sign in front of the bonus for spell attack, damage or DC, the output for a spell is always 0. The math is there if you hover, but only 0 shows. if you only place the digit (i.e. 2) then the math works.

Don't put a + then. :P

Just letting you know :+1:  :)

I don't think there is any way to fix it besides stripping out a leading + sign


[MRG + 1] Tuning hyperparameters in a pipeline

I didn't see an example of optimizing over the hyperparams of several sklearn objects assembled into a pipeline (i.e. a pipeline generalization examples/hyperparameter-optimization.py).  I took a couple hours and made a notebook showing how to do this, if that example seems useful.
It's kinda drafty and terse compared to the other (really nice!) example notebooks.  I'm happy to add/modify if you can think of more stuff that might be interesting to show here.

Thanks!
A couple of comments:

pipeline.set_params(pca__n_components=N) is the recommended syntax for setting parameters.
instead of creating a new notebook, shall we rather expand or add a note in hyperparameter-optimization?

I would put links between the two notebooks ("if you liked this notebook on HP opt, maybe you want to keep reading and learn about pipelines" or something) but keep them separate. I think it makes it easier to discover what examples we have and keeps them a bit more self-contained.
@cmmalone can you add some plots, everyone loves plots 📈 📊. Not sure exactly what to plot ... how the performance changes over the iterations of the optimisation? Dependence on the parameters?
Who generates the warnings you are silencing? Is it from skopt or sklearn? If they are from us could you post them here/new issue so we can fix what ever they are warning about?

@betatim Well, in this case I dont agree. Tuning pipelines is not different from tuning a regular estimator. You just have to know the syntax. Everything else remains exactly the same.

I am +0 on splitting, if you guys have stronger opinions on this I go with your decision.

Thanks for the comments!  I'm working on them now.
@betatim the warnings I see are coming from the call to gp_minimize; here's the message:
/Users/civisemployee/src/scikit-optimize/src/sklearn/sklearn/utils/extmath.py:368: UserWarning: The number of power iterations is increased to 7 to achieve higher precision.

That warning is generated by something inside sklearn. Silencing that warning is being tracked in scikit-learn/scikit-learn#6746

Sorry if I was not clear, but my suggestion was not to concatenate the two notebooks together. At the moment the two are almost identical and I am not sure it is very helpful to repeat things like you propose. I think we could rather simply add a paragraph near the definition of the objective function to explain how to adapt the code to pipelines. The rest of the original content would stay the same.

@glouppe For sure, I think it can be merged much more elegantly--I got halfway through my work yesterday and then something else came up and I wasn't able to finish.
For how to actually make it clear, though, how simple it is to do pipelines once you've done e.g. a classifier--I've personally always found code snippets more helpful than prose explanations, but like you say, the code that would demonstrate this repeats a lot of what's already there.  What if I do something like this--keep the first part of the notebook largely the same, with just a note to see below if you're interested in pipeline stuff.  Then, where my pipeline example is now, replace the digits dataset with Boston housing (so it feels more like the same problem), build a pipeline with PCA and the same classifier that you used before, and then the convergence plot would show 6 traces: 3 classifier only, and 3 transformer+classifier.   Honestly I'd be a little curious to see how convergence changes as we toggle PCA on/off.
Let me know how that sounds to you.

Could we make the pipeline example (second half of the notebook) something that optimises parameters of algorithms and contents of the pipeline? At its simplest this would be one step of preprocessing (with parameters to tune) plus one step of classifications (with parameters to tune). You would end up optimising over the choice of preprocessing step and classifier as well as their respective parameters. I'd find that quite exciting.
Caveat: our API doesn't yet explicitly support conditional search spaces (#108) but maybe that isn't a problem in practice for this example.

For how to actually make it clear, though, how simple it is to do pipelines once you've done e.g. a classifier--I've personally always found code snippets more helpful than prose explanations

I agree with you on this. Actually, couldn't we maybe add a paragraph on this, then have below some code snippet defining a second objective function and then document that this new objective function can be plugged right in the minimizer? As Tim describes, you could even show that it may even be more complicated than that. E.g. something along the line of:
def objective(params):
    decomposition = params[0]   # categorical, either "pca" or "ica"
    if decomposition == "pca":
        p = make_pipeline(PCA(), SomeClassifier(...))
    else:
        p = make_pipeline(ICA(), SomeClassifier(...))

    # then set params of the classifier
    p.set_params(clf__XXX=params[1])
    p.fit(X, y)
    ...
    return score

Oh I really like that idea, of playing around with the fundamental pieces in the pipeline.  I was thinking to go in that direction anyway once this PR was complete, so yeah, I'm on board.
So where we stand now is, I've got my local version of the code into a place where it's comparing 6 pipelines (3 optimization schemes x switching PCA on/off) and it's close to being ready for a commit.  One thing I've found, though, is that PCA on the Boston dataset doesn't seem very methodologically well-founded, since (among other things) that data is fairly low-dimensionality to begin with.  Any objections to moving the example over to the digits dataset, where PCA makes more sense?
I should have an updated version over the weekend that finishes up what I've started, and then we can talk more about if we want to add @glouppe 's suggestion in this round or merge the PR and circle back for that later.

Ok, I've cleaned things up a bit, and compared the performance (on the Boston dataset) between GBR alone and PCA+GBR.  Hopefully the flow is a little better, although feel free to disagree :)
A few notes I made as I was working on this.

PCA isn't a particularly well-motivated tool to use on this dataset/problem.  As a result (presumably), the pipelines do worse than pipeline(PCA+GBR).

I played around with switching over to the digits dataset (and adjusting from GBR to GBC) because this seems like an instance where PCA is more theoretically well-motivated, but I found that the optimization then got really slow--sometimes up to 5 seconds per iteration.  Running 50 iterations was clearly a no-go, especially for demonstration/development purposes.  I'm not sure if this is because PCA was slow, or digits is higher-dimensional, or GBC is slower than GBR, or something else, but I wanted to note it--would be interesting (to me) to understand the cause a little better.

Also something weird in digits + GBC--the dummy optimization seemed to do really well?  Here's the convergence plot I got for GBC only (no pipeline/PCA):

Possibly this was just a coincidence, but it was so counterintuitive that I wanted to mention it.

Now that this commit should basically complete all the stuff I wanted to do with the original pipeline PR, I'd also be happy to talk more about optimizing over components of the pipeline (e.g. PCA vs. ICA).  It's really interesting and should be done.  But I might suggest that we save that for a future PR.  Thoughts?

The trouble with trees is that they are pretty robust wrt their inputs -> you don't benefit a lot from preprocessing. One thing that might help (a tree based model) is building additional, new features from existing ones (even constructing a + b requires a fairly deep tree). For polynomial features there is a tool in sklearn. Just an idea.
What happens if you increase n_components's range up to 13 (the original number of features in boston)?
Did you run it for more than 50 iterations to see if it eventually catches up with the no PCA setup?
Before going through the example for formatting and word smithing we (@cmmalone, @glouppe) should decide: are we happy with the general structure/problem setup? GBR followed by a pipeline example of PCA+GBR on boston.
I am kind of happy with PCA+GBR. My preference would be PolynomialFeatures+GBR.
Thanks for the patience @cmmalone.
It is strange that random search outperforms the others. Before drawing any conclusions I would run it several times because a single instance means nothing when random numbers are involved 😀 Though it seems in general, on low dimensional problems random search is hard to improve on.
In this particular run it looks like random search got lucky and picked a good point early on (and then doesn't improve on it any further)

Heya,
I ran a bunch of little studies.   Here's what I was finding:

Boston data, GBR alone, error = 2.78 (n_calls=30, GP optimizer)
Boston data, polynomial features + GBR, error = 3.05
Boston data, PCA + GBR, error = 3.67

Running out to n_calls = 75 doesn't really change the picture.
I also tried a bunch of analogous stuff with the digits dataset and a classification approach.  The same general trends held, albeit much more slowly because for some reason the GBC takes FOREVER on the digits dataset.
So, none of these approaches seems to really illustrate the point that we want to communicate, namely that doing feature engineering might be a good idea.  Upon some reflection I can see intuitively why this makes some sense for PCA (a well-validated model in general should do worse when you start doing dimensionality reduction) but I'm still scratching my head about polynomial features.
Now, here's one idea of where we can take this.  Sure, dimensionality reduction might not win you any advantages over just using the full dataset.  However, say we have a given feature budget, like max number of features = 10 or whatever--I bet that we'd see a difference between e.g. PCA for preprocessing vs. univariate feature selection, and of course n_components/n_features should make some difference.  I'd be happy to hack on something like we discussed earlier, where we optimize over which pieces to actually assemble in the pipeline, as well as the parameters of the pieces.  But I think that might be getting more toward a regime where we'd want a separate notebook for that, so as not to invite comparisons like the ones we're doing now (i.e. why even mess around with the pipeline if the raw estimator works better?)
Just an idea, I'm happy to keep wrangling this for as long as you guys don't mind working with me on it.
One last thing, that became viscerally clear in this work--imagine a pipeline where you have polynomial features + classifier.  And suppose that the optimal degree of polynomial features is 3.  In the case I was examining (digits--probably not benefitted by poly features, but that's not the point here), doing the poly features transformation for degree = 3 is SLOW.  But, the optimization is going to do it a lot, because it's giving the best results.  So you end up with a bottleneck where it spends a ton of time repeating the same poly feature transformation while it's trying to find the best parameters for the classifier--not ideal.  I'm imagining a possible algorithm especially for pipeline optimizations like this (well-defined interfaces between the pieces) where you can do the transformations once and then repeatedly just reload them while optimizing the downstream stuff.  I'm sure this is something you've thought about, and it seems really nontrivial to do for a general use case, but it would have been really nice to have here.

🚧 Extreme pragmatism ahead 🚧
Are @glouppe and @cmmalone happy with the example we have right now?
I would vote to merge the example as is. Yes it might not make the most amount of sense to assemble things this way. IMHO our examples should be more about the technical stuff (how do I do this?) and less about recommending the best approach. "We give you the fork and show you how to use it. You can use it to eat fries or steak or vegetables. Which one you should be eating is up to you."
In the end SMBO is about discovering things that you didn't know. So we can formulate the example as "Naively you might think polynomial features would help, turns out it doesn't. Who knew? Good thing you tried it."
🚧 Extreme pragmatism over 🚧
A bigger example that swaps in and out pipeline components and finds their best parameters would be interesting to me. Definitely a new notebook though. Let's start a new issue if we ant to discuss this further.
I would say that caching results that are repeatedly computed is a smart move, building the infrastructure for that is outside of the scope of this particular library though.
Personally I find both very interesting, however as scikit-optimize maintainer I would say "too specific to the hyperparameter tuning problem, skopt is about so much more. Probably useful and popular though so we might consider building an example to get people interested in skopt."

@cmmalone You might want to inherit cross_val_score from model_selection to avoid the deprecation warning.

Uggh I hate the fact that I cannot comment on the diff :/

I haven't read all the commentary but I'm fine with merging this as is. I agree that the purpose isn't to show that a pipeline with so and so performs better but just how to use a pipeline in combination with the minimize functions.
(Btw, this is a low dimensional problem with enough data and not sure we should do any dimensionality reduction except for showcase purposes)
@cmmalone Thanks for your work! This gets my +1!

@cmmalone Oh! Could you set pipe_x0 to [13] + x0? and search across range(1, 14). I'm curious to see what happens.

We should just remove that second import of cross_val_score.

Thanks for the comments, everyone, and I'm happy to merge too.  Appreciate your patience.
@MechCoder I've changed the n_components for PCA as you suggested, and gave the pipeline the same starting point for the pipeline and the solo regressor.  Also cleaned up the cross_val_score as suggested by @betatim

We probably want a graph instead of a NameError at the end :-)

@cmmalone if you re-run the notebook once more from scratch and update your last commit that would be great. Then merging 🎉 !

@betatim @MechCoder does it look better now on your end?

The start point should be [13], instead of 5 no?

@MechCoder good catch

Can you rerun it. for one last time.

Merging this and using my powers to update the notebook after.

🍝 ! Good job everyone and thanks for the patience on all sides.
There are a few ideas in this thread that we wanted to follow up. Create new issues or PRs for those :)

Umm. did you push the commit to upstream yet?

Oups! Done now


内存溢出

graphhopper:
restart:always
container_name: graphhopper
image: israelhikingmap/graphhopper:6.0
command: -i /maps/taiwan-latest.osm.pbf --host <IP_ADDRESS>
volumes:
- ./data/SiteMapRouting/maps:/maps
ports:
- 8989:8989
我使用此方式部署docker由于osm.pbf文件过大,导致报错:内存溢出,我该怎么修改这段配置

Google translate:

我使用此方式部署docker由于osm.pbf文件过大,导致报错:内存溢出,我该怎么修改这段配置
I use this method to deploy docker because the osm.pbf file is too large, resulting in an error: memory overflow. How should I modify this configuration?

As written in the readme here, you can set JAVA_OPTS: "-Xmx1g -Xms1g" environment variable to a bigger number to supporting import of bigger files.
Use
environment:
    - JAVA_OPTS: "-Xmx5g -Xms5g"


Clean MockedDependenciesRule

@TurpIF  changes look good to me. Let me know when I can review (resolve conflicts, quality gate).

@TurpIF changes look good to me. Let me know when I can review (resolve conflicts, quality gate).

@Amokrane: The quality gate fails because I moved the CdbMock class and Sonar doesn't get the good coverage rate because it is tested by PubSdkApiIntegrationTest class which is in the publicher-sdk module. This is the effect I mentioned in EE-1370 this morning.
I rebased, let's wait the end of the CI to verify if quality check is still the same.

Yep I confirm that quality gate is red because of the coverage wrongly detected on CdbMock. On 75 uncovered lines, 67 are in CdbMock.


Fixed copyright year in footer

I noticed that the year in the footer was out of date so I updated it.
I fixed the templates when I could so that it now automatically displays the current year.
However, I was not able to do that in the static HTML pages (error.html and maintenance.html) so on those pages I just incremented the year.
I also tried to use a dynamic year in the sphinx configuration file but I'm not sure if that's a valid solution.
Thanks!

Could you merge in the latest and I'll accept this pull request?

@pydanny done :sparkles:

Hmmm... tests are failing. I'll take a look at it today.

The test failure is unaffiliated with this pull request so I'm brought this in anyway.


Added 88.2KHz Sample Rate

This is particularly needed to allow MQA-enabled apps to have exclusive access to the audio device. MQA rendering uses 88.2Khz and 96Khz for 44.1Khz and 48Khz-encoded tracks, respectively.
Tidal can now have exclusive access to it and render correctly.
Tested using the Linux Renderer app which successfully changes the sample rate to 88200:

Thanks! Released new binaries.


No bounding box or ID

First of all, thanks for this clean code.
Problem - While running main_detector.py or main.py I don't see any bounding box or object ID on the output video. Can you please help me resolve it?

Hi!
I have the same problem. Try to change fp16 to "False" in Predictor.py: https://github.com/0w1Gr3y/MOT_ByteTrack/blob/725c6f26a7cee6ae098a61b377bd989cf71b3454/Predictor.py#L19
In my case this resolve problem. When I tried original YOLOX demo start with combined GPU and FP16 option i got:
Traceback (most recent call last):   File "tools/demo.py", line 320, in <module>     main(exp, args)   File "tools/demo.py", line 290, in main     model = fuse_model(model)   File "/root/MOT_ByteTrack/YOLOX/yolox/utils/model_utils.py", line 92, in fuse_model     m.conv = fuse_conv_and_bn(m.conv, m.bn)  # update conv   File "/root/MOT_ByteTrack/YOLOX/yolox/utils/model_utils.py", line 74, in fuse_conv_and_bn     fusedconv.bias.copy_(torch.mm(w_bn, b_conv.reshape(-1, 1)).reshape(-1) + b_bn) RuntimeError: expected scalar type Half but found Float
This give me an idea to change this line in Predictor.py

Hi! I have the same problem. Try to change fp16 to "False" in Predictor.py:
https://github.com/0w1Gr3y/MOT_ByteTrack/blob/725c6f26a7cee6ae098a61b377bd989cf71b3454/Predictor.py#L19
In my case this resolve problem. When I tried original YOLOX demo start with combined GPU and FP16 option i got:
Traceback (most recent call last): File "tools/demo.py", line 320, in <module> main(exp, args) File "tools/demo.py", line 290, in main model = fuse_model(model) File "/root/MOT_ByteTrack/YOLOX/yolox/utils/model_utils.py", line 92, in fuse_model m.conv = fuse_conv_and_bn(m.conv, m.bn) # update conv File "/root/MOT_ByteTrack/YOLOX/yolox/utils/model_utils.py", line 74, in fuse_conv_and_bn fusedconv.bias.copy_(torch.mm(w_bn, b_conv.reshape(-1, 1)).reshape(-1) + b_bn) RuntimeError: expected scalar type Half but found Float
This gave me an idea to change this line in Predictor.py

Thanks!
It worked for me


Jobs, fix Duration field format

Summary
Fixes #7758

Screenshot/Video

@torchiaf careful not to merge this yet as it's v2.7-next2.


pull-request: small speed improvement

Thanks for the great work: fast and good results!
Please find attached a small patch improving a bit the speed and memory usage by replacing IndexedFloat with a static plane index array. I fixed also some warnings.
patch.zip

Thanks a lot for the patch. Would you like to submit a pull request? That would be easier for me to updated the code. Thanks.

Sorry, I do not to submit a pull request without forking the project, and I do not plan to do that.
But I do not know what you mean by "easier". Applying the path is as easy as running:
git apply patch.diff

Thanks, I will try it.


Correct the 0.29 release to January

Signed-off-by: Matthias Wessendorf<EMAIL_ADDRESS>As per title

/assign @dprotaso


[RuntimeError] Unknown / unsupported VS version '17.0'

Hello, I faced with RuntimeError "Unknown / unsupported VS version '17.0'"
I install Ros2-Foxy follow the ros2 windows install instructions
Here is my environment information for re-produce error
OS: Window 11 Pro 22H2
ROS2 Distributor: Foxy
ROS2 Location: C:\opt\ros\foxy
WorkSpace Location: C:\foxy_ws
VisualStudio: Microsoft Visual Studio Community 2022 (64-bit) Ver 17.4.3
Command Prompt: Visual Studio 2022 Developer Command Prompt v17.4.3
command prompt
cd C:\foxy_ws
rosdep install -i --from-path src --rosdistro foxy -y
rosdep update
colcon build

error print
c:\foxy_ws>colcon build
[0.369s] root DEBUG Using proactor: IocpProactor
Starting >>> carter_description
[0.377s] colcon.colcon_ros.prefix_path.catkin WARNING The path 'C:\opt\ros\foxy\x64\tools\vcpkg\installed\x64-windows' in the environment variable CMAKE_PREFIX_PATH doesn't exist
Starting >>> carter_navigation
Starting >>> isaac_ros2_messages
Starting >>> isaac_ros_navigation_goal
Starting >>> isaac_tutorials
--- stderr: isaac_ros2_messages
Traceback (most recent call last):
  File "c:\opt\ros\foxy\x64\lib\site-packages\colcon_core\executor\__init__.py", line 91, in __call__
    rc = await self.task(*args, **kwargs)
  File "c:\opt\ros\foxy\x64\lib\site-packages\colcon_core\task\__init__.py", line 93, in __call__
    return await task_method(*args, **kwargs)
  File "c:\opt\ros\foxy\x64\lib\site-packages\colcon_ros\task\ament_cmake\build.py", line 59, in build
    rc = await extension.build(
  File "c:\opt\ros\foxy\x64\lib\site-packages\colcon_cmake\task\cmake\build.py", line 87, in build
    rc = await self._reconfigure(args, env)
  File "c:\opt\ros\foxy\x64\lib\site-packages\colcon_cmake\task\cmake\build.py", line 162, in _reconfigure
    raise RuntimeError(
RuntimeError: Unknown / unsupported VS version '17.0'
---
Failed   <<< isaac_ros2_messages [0.06s, exited with code 1]
Aborted  <<< carter_description [0.11s]
Aborted  <<< carter_navigation [0.08s]
Aborted  <<< isaac_ros_navigation_goal [0.05s]
Aborted  <<< isaac_tutorials [0.03s]

Summary: 0 packages finished [0.25s]
  1 package failed: isaac_ros2_messages
  4 packages aborted: carter_description carter_navigation isaac_ros_navigation_goal isaac_tutorials
  1 package had stderr output: isaac_ros2_messages
Unknown / unsupported VS version '17.0'

c:\foxy_ws>

thanks in advance

I already solve this porblem by uninstall VisualStudio2022 and install VisualStudio2019.
But i don't close this issue.


Android: New Team - Town lookup doesn't match when you enter the entire town name

If you enter Colcheste, it's in the dropdown.
If you enter Colchester, it isn't in the dropdown and it doesn't save it when you save the team

Duplicate


error when annotating tweets

When I try to annotate a tweet, I select some text and enter some annotation text. When trying to create the annotation, it gives an error:

I have deployed the annotation server. Configs fixed accordingly. PR made for relevant configs. After the merge and redeployment of the Linkyway this issue will be fixed.


move resource generation for fhir client to different repo

This repo is for mapping FHIR data to CAPACITY. There are a couple of files and scripts in here that were created to test out whether it was possible to use the fhir parser that is associated with the FHIR client to create models for new resources.
These files should not be deleted but moved to a separate repo.
This concerns the following files and directories:

fhir-parser/
generate_models.py
mappings.py
settings.py

AB#1907

Moved to https://github.com/FAIR-data-for-CAPACITY/custom-profile-generator, it's not working out-of-the-box though.


.gitignore: Add Windows intermediate files

Yeah this was pretty annoying. LGTM.

:+1:

Auto-merge toggled on

I continue to believe that growth of .gitignore files is a symptom of a larger problem: a disorganized build process that leaves junk files left where they don't belong. Am I alone?

@andralex : That's exactly the comment I would have written!

@andralex In this instance, yes. The Phobos win32.mak is less organized in where it creates temporary files than posix.mak, but I didn't want to risk breaking things by moving things around.

BTW, there are some entries in our .gitignore files that shouldn't be there. For example, the Makefile and GNUmakefile entries do not belong in .gitignore, they should be in .git/info/exclude. And I've no idea what the .*.sw* is for.

*.*.sw* is for the temporary files that vi creates during editing of a file.

Those should not be in our .gitignore either. They should be in users' global .gitignore.


Restructure project

This is just a project restructuring. No visual changes other than a new font. Go through commit by commit, and you should be able to follow along!

analysis_options: add more linter rules to
packages: introduce sail_ui, with basic ui components
lib/storage: new package for storing client preferences
routing: add dependency auto_route
rpc: move rpc to subfolder, and introduce GetIt
main: use auto router when starting app
pages: move home page code, introduce viewmodels
main: use monospaced font SourceCodePro
config: introduce class for getting runtime args
linux+macos+windows: automatic build changes
test: introduce test that can route our pages

Very good stuff! Are you able to get CI passing as well? (I know that I was the one who broke it originally...)

I don't think you broke anything. Those errors are all my fault. Fixed now though (more like supressed)

ready when you are torkel

let's go


Is there a flag to split the parquet output by size?

as per the title
i.e. i want to output a large table, but have several parquet files generated, that are no bigger than 1GB

Hello @samaguire,
Seems to me you are looking for the --file-size-threshold option of the query subcommand. See #190 , let me know if this solves your issue.
Cheers, Markus

FYI: To learn more about --file-size-threshold or to see all options and flags for the query subcommand, you can execute odbc2parquet query --help.

Ah. I thought it was odd to not have so ething like this. I didn't realize I could do the --help after query. 👍


Remove old notifications

Coverage decreased (-4.9%) to 10.062% when pulling a011d25cd91095a0c370b9a7f45be76c7d842fb9 on remove-old-notifications into 7838d5a2eb4af4816a9945533c1392bcbf93a1c1 on master.

If none of these classes are needed anymore that is amazing but it does look like there are some conflicts with the branch


[AutoPR security/resource-manager] Azure Security Center - Adding code generation attributes

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4315

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/302


Feature/app admin middleware

Summary
This pull request contains fixes and improvements related to the user authentication and restaurant management features.
Changes Made

Created appAdmin middleware and updated user role
Added extra validation to update user role
Modified the create restaurant service
Refactored restaurant update service
Modified user and restaurant models.
Implemented soft deleting functionality for user and restaurant

Additional Notes

Readme and license were added to the repository

New committ with the changes


added vertical scrollbar to popover

Can you also fix the popover Title for Pre-contract hours here? Right now it is rendering as Precontract.

fixed title to pre-contract


Why is yarn required for the build?

Is your feature request related to a problem? Please describe.
If user has node but not yarn the project can't be built.
Describe the solution you'd like
Don't have a hard dependency on yarn especially when node is the standard.
Check if yarn is installed before depending on it, otherwise use node.

Also, as of today, the recommended way to install is by git cloning. If someone wants to integrate it with Mason so it can be installed via MasonInstall, they will have to check for the presence of yarn which is unnecessary.

I get your point, and have already been trying to rely less on the use of yarn for distributing this project (prior to this issue). However, there are plenty of lsp projects using yarn, and I assume some of them are included in mason. For example: awk-lsp, typescript-language-server, etc...
I'm framiliar with the project Mason, and would love to have this lsp be supported there. Although, I'm not exactly sure what you're proposing. By node are you refering to npm?
Would removing the preinstall script resolve your issue?
I package this project on npm, and in earlier versions it did not seem to always install currectly for some users. That is why the recommended installation method is to clone this repo. Feel free to contribute to any fixes towards the projects installation process, it is extremely helpful.

[!Note]
I've tried porting the project to pnpm in favor of yarn but had difficulties getting everying working.

Having a GH release would help here as well in the mean time.


Dlib frontal_face_detector not providing consistent detections for the same image and code.

General Summary:
I'm 98% certain this has to do with my code, but since it's heavily inspired by the dlib example code and I couldn't find any similar issues on GitHub or StackOverflow, so thought I'd open an issue here. Anyhow, for some reason that I can't explain, the frontal_face_detector() behaves inconsistently. Using the same image, with the same block of code, dlib will sometimes will pick up the face and other times it won't. So far it seems to be completely random, but again it's probably something I'm overlooking in the code.
I've tried to eliminate every bit of deterministic code I can think and I've stripped it down to the block below.
import cv2
import dlib
import os
import numpy as np

video = cv2.VideoCapture('./5s_sample.mp4')

def detect_faces(success, frame):
    """Detect faces in a frame."""
    # Resize the frame
    #frame = imutils.resize(frame, width=400)
    if success:
        # Convert frame to gray and then Use dlib HOG front face detector
        # Note: The dlib CNN version will give more accurate results,
        # but will require more resources aka GPU

        gray = cv2.cvtColor(frame, cv2.COLOR_BGR2GRAY)
        detector = dlib.get_frontal_face_detector()
        detections = detector(gray)

        faces = []
        if len(detections) > 0:
            for d in detections:
                try:
                    l, t, r, b = d.left(), d.top(), d.right(), d.bottom()
                    l *= 2
                    t *= 2
                    r *= 2
                    b *= 2
                    w, h = r - l, b - t
                    cv2.rectangle(frame, (l,t), (l+w,t+h), (0,255,0), 2)

                except TypeError:
                    print(type(self.frame), self.frame)
                faces.append((l, t, w, h))

            faces.sort(key=lambda bbox: bbox[0])
            processed_frame['faces'] = faces

success, frame = video.read()
detect_faces(success, frame)
print(processed_frame)

Env:

Python 3.8.2
Dlib 19.19.0 (installed via pip)
Mac 10.15
OpenCV <IP_ADDRESS>

Expected Behavior

The expected output is the face coordinates, but that only happens 1 out of 10 tries or so. Dlib details and how it's being used are posted in the code above.
Current Behavior

The current behavior is just inconsistent detections for the same image. There aren't any errors and the frame data is, in fact, making it to the detector which I think eliminates opencv (but again I could be wrong). The video itself is PII so I can't share it exactly but here's the stream information.
Duration: 00:00:05.08, start: 0.000000, bitrate: 1568 kb/s
    Stream #0:0(und): Video: h264 (Main) (avc1 / 0x31637661), yuv420p(tv, bt709), 808x720 [SAR 8:9 DAR 404:405], 1578 kb/s, 29.97 fps, 29.97 tbr, 90k tbn, 180k tbc (default)

The image really just is close up of a person talking where the front face is 95% of the image (no pre-processing).
Steps to Reproduce

The code posted above combined with my env details should reproduce the issue.

Version:
19.19.0

Where did you get dlib:
pip3 install dlib

Platform:
macosx 10.15.4

Compiler:
3.8.2 (default, Mar 11 2020, 00:29:30)
[Clang 10.0.0 (clang-1<IP_ADDRESS>)]

I bet the same image isn't really being passed in.   You should log the
image and check that it's actually the same.

Thanks for the quick reply. You were correct, after switching to an FFmpeg stream and confirm the order of the frames, things worked perfectly.


Optimize authorization by caching authorization results

Motivation
To follow Kafka's behavior, KoP also performs authorization for each PRODUCE or FETCH request. If the custom authorization provider is slow to authorize produce or consume permissions, the performance will be impacted.
Modifications
Introduce caches for authorization:

PRODUCE: (topic, role) -> result
FETCH: (topic, role, group) -> result;

Add SlowAuthorizationTest to verify the producer and consumer won't be affected significantly by slow authorization.
Introduce two configs to configure the cache policy so that revoke permission can work:

kopAuthorizationCacheRefreshMs: the refresh timeout
kopAuthorizationCacheMaxCountPerConnection: the max cache size

Documentation
Check the box below.
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[x] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

@Demogorgon314 Done.


Move components back into API

Part of https://github.com/creek-service/creek-kafka/issues/56
On second thoughts...  the whole point of the API interface passed to the initialize method is to allow the API to be extendable, so let's not get into passing other things as well.
Reviewer checklist

[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure any appropriate documentation has been added or amended

Coverage increased (+0.04%) to 99.583% when pulling c3e073acd1a72a74c94448d0e56fc44eb954e7ef on api_comoponents into fc10a6f7733e6697c052aa45fbc6f5929e2d4f34 on main.


[firebase_ml_vision] Tensorflow lite custom models

Is your feature request related to a problem? Please describe.
As developer I need an easy and reliable path to use custom Tensorflow lite models in flutter apps.
Describe the solution you'd like
Add an example to cover Firebase with custom Tflite models
Additional context
https://firebase.google.com/docs/ml-kit/use-custom-models

See also https://codelabs.developers.google.com/codelabs/mlkit-android-custom-model/index.html

/cc @shaqian if interested in the topic.

/cc @devoncarew

Closing in favour of https://github.com/FirebaseExtended/flutterfire/issues/788


Don't set MaxMetaspaceSize on Java 8 by default

Resolve the issue mentioned in https://gitter.im/sbt/sbt?at=55d9963636e894436a9aa94f

This reminds me that on my machine Java version detection doesn't work. See https://github.com/sbt/sbt-launcher-package/pull/136


[BACKLOG-6682] fix injectable trim type for excel input field

@mdamour1976 please review

@bmorrise please review


'Extend all loans' for a patron in the back office (librarian)

Is your feature request related to a problem? Please describe.
As a librarian, in the back office, if a patron a several loans and would like to extend them all, we have to click on each loan and extend them one by one.
Describe the solution you'd like
A button "Extend all loans' on patron's profile as it is the case for patron in the front office where they can themselves extend all their loans at once.

disable, not hide the extend the all loans


chore: update schema definition

Summary

previously MR: #2397

Property jsxRuntime is not allowed:
But terminal output told me need to use jsxRuntime

The schema for jsxRuntime is not match on the Doc or the prompt on terminal, so I decide to update to the correct definition.🙏
Test Plan

We should update the serde attributes not directly the schema?


example

add more example for brute plz (VK,INSTA,GIThub or any)

I cannot provide such examples, it will violate the security policy. In addition, large services have serious protection, which will simply be unethical to bypass)
The project is a tool for further testing YOUR project


Add attestation handler component

Add an attestation handler component as described in #925
closes #925

@mullefel Can you have a first look at this? Is this more or less how you imagined it?
Some remarks:

Not quite sure about the sgx/std-compilation flags. Now most of the code only compiles with the sgx flag active
Also not quite sure about the error handling.
Nonce Cache, Extrinsics Factory and Ed25519 not (yet) changed

@murerfel I think I covered all your points except the issue with of the attestation handler compilation in std mode.


Collections docs

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Gatsby Cloud Clearing Cache Header does not work

Hey, i want to build a Gatsby Site hosted on Gatsby Cloud with a cache clear:
https://support.gatsbyjs.com/hc/en-us/articles/360053279533-How-do-I-clear-a-build-cache-
This is my config:
 "website-builder": {     enabled: true,     config: {       url: env("GATSBY_BUILD_HOOK"),       headers: {         "x-gatsby-cache": false,       },       trigger: {         type: "manual",       },     },   },
It seems that this does not work as expected (webhook triggers, but header is not working)

How are you checking if headers are working? I tested via webhook.site and headers are being sent and received properly.

Thanks for your investigation - it seems that gatsby cloud doesn't work as expected. I had a ticket open there and they said it should work..


Inventory payload request to use namespace

The API call to the inventory-payload app has changed slightly from:
/api/v1/extract?providers=provider1,provider2
to:
/api/v1/extract?providers=namespace1/provider1,namespace2/provider2

@mturley, @konveyor/virt-payload has been updated, could you please review?

Updated after http proxy addition.


Add Badge Component (and rename Badge to Tag)

Hey! I would like to add a Badge component which displays a count or just a colored dot overlayed over its children. The old Badge component should be renamed to Tag as its more descriptive of its use.

The component should take these props:

count: number - number to display in the badge
limit: number - the maximum count after which we get something like 99+
type: 'default' | 'secondary' | 'danger' | 'warning' | 'success'| 'light' - color
size: number - the size in px
shouldDisplayZero: boolean - badge should render even if count === 0
shouldDisplayCount: boolean - should render count or just an empty dot
offset: [number, number] - x and y offset in px

Question: What are your naming conventions for boolean props? I am personally not a fan of disabled or error when compared to isDisabled and hasError. Looking for your suggestions for the two boolean props here (showZero vs shouldDisplayZero etc.)

@henckellbach Hello, love your insights! Yes we can rename the current Badge component to Tag. Currently, Avatars can be given a hasBadge prop which only allows green color. I think we can make use of this Badge component instead!
Answer: I actually agree with you but when I started working on this project, I went with props disabled and error. So to keep it consistent  you can proceed with showZero and shouldDisplayZero

Awesome! Thanks for the quick reply, I will start working on this in the coming days 🎉

@henckellbach Sure! Keep me updated 💯

Opened a PR - https://github.com/karishmashuklaa/chic-ui/pull/58


Ability to Create Activity from Ruby Gem?

I see that the master branch has a create_an_activity method, but it is missing from published v0.7.0 gem. Are there any plans to publish this gem with the updates?

just use the master source in your gemfile

I've pushed 0.8.0 to ruby gems.


Compress responses in production

Adds the deflater middleware, but advised to add before SendFile to ensure everything is compressed.

I assume there's no way to do this at the PaaS http server level?

As far as I can tell, you can only provide an nginx config for static sites.


Extraction failed

I was not able to extract information from https://www.cdc.gov/coronavirus/2019-ncov/vaccines/booster-shot.html
I got null as a response from extractor

@gmocelj  sorry this parser can only extract the particular structured data such as blog post or news entry. The content from cdc.gov is another kind of structure which article-parser does not support.


v8.10.1

:tada: This PR is included in version 8.8.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error in execution of code in python micro course kernel

In the Micro course python's 7th Exercise for List and Tuples I completed the code and executed it and it shows that it is incorrect, When that is clearly correct as you can see in the picture.

The indexing during the execution is wrong even the binding to global from learntools.core and importing all the functions from learntools.python.ex4 setup is successfully completed.

There is another value read from the list error in the loop and list comprehension section

look at the blue rounded strings in the printed list and in the element and it,s neighbour printed section they both has to be same but are different hence resulting the error at the yellow line.
Instead of giving a True Boolean value it gives a False resulting to incorrect answer of docstring mentioned required result.

This doesn't seem to be an API related issue.  Please report such problems on the main Kaggle site (https://www.kaggle.com/discussion) or through our support (https://www.kaggle.com/contact).
Reopen if I'm mistaken here.  Thanks!


Minor design tweak for smartphones

This fixes #829 hiding the icon when browsing the site in a smartphone.

Anyway, good job!


Build does not auto-generate type

Describe the bug
See https://github.com/libp2p/js-libp2p/issues/1105
When upgrading aegir from 33.2.4 to 34.1.0, a type was not auto-generated anymore in the js-libp2p library.
js source code:
  /**
   * Peer object
   *
   * @typedef {Object} Peer
   * @property {PeerId} id peer's peer-id instance.
   * @property {Address[]} addresses peer's addresses containing its multiaddrs and metadata.
   * @property {string[]} protocols peer's supported protocols.
   * @property {Map<string, Uint8Array>|undefined} metadata peer's metadata map.
   */

Code generated with<EMAIL_ADDRESS>or 34.0.0, 34.0.1
/**
 * Peer object
 */
type Peer = {
    /**
     * peer's peer-id instance.
     */
    id: PeerId;
    /**
     * peer's addresses containing its multiaddrs and metadata.
     */
    addresses: Address[];
    /**
     * peer's supported protocols.
     */
    protocols: string[];
    /**
     * peer's metadata map.
     */
    metadata: Map<string, Uint8Array> | undefined;
};

No such type generated with 34.0.2, 34.0.3.
To Reproduce
Steps to reproduce the behavior:

Checkout latest https://github.com/libp2p/js-libp2p
npm i; npm run build
Check ./dist/src/peer-store/index.d.ts
No type Peer present
Downgrade aegir to ^33.0.0
npm i; npm run build
type Peer is present

Expected behavior
I expect the type Peer to always be generated.
Desktop (please complete the following information):

OS: Linux
Browser N/A
Version Isu

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version: issue starts from 34.0.2. Fine with 34.0.1.

Now resolved


Release error when no --file argument supplied in the CLI

Hey, giving the release branch a play.
I get this error:
➜  protocols git:(provision_fix) ✗ transcriptic release "package-id"
Traceback (most recent call last):
  File "/Users/Ben/anaconda/bin/transcriptic", line 9, in <module>
    load_entry_point('transcriptic==1.2.0', 'console_scripts', 'transcriptic')()
  File "/Users/Ben/anaconda/lib/python2.7/site-packages/click-4.0-py2.7.egg/click/core.py", line 664, in __call__
    return self.main(*args, **kwargs)
  File "/Users/Ben/anaconda/lib/python2.7/site-packages/click-4.0-py2.7.egg/click/core.py", line 644, in main
    rv = self.invoke(ctx)
  File "/Users/Ben/anaconda/lib/python2.7/site-packages/click-4.0-py2.7.egg/click/core.py", line 991, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/Ben/anaconda/lib/python2.7/site-packages/click-4.0-py2.7.egg/click/core.py", line 837, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/Ben/anaconda/lib/python2.7/site-packages/click-4.0-py2.7.egg/click/core.py", line 464, in invoke
    return callback(*args, **kwargs)
  File "build/bdist.macosx-10.5-x86_64/egg/transcriptic.py", line 198, in release
    @click.argument('protocol_name')
  File "build/bdist.macosx-10.5-x86_64/egg/transcriptic.py", line 157, in upload
    click.echo('Creating empty manifest.json...')
  File "/Users/Ben/anaconda/lib/python2.7/posixpath.py", line 121, in basename
    i = p.rfind('/') + 1
AttributeError: 'NoneType' object has no attribute 'rfind'

However if I provide the name of an existing archive via the --file argument.
It zips up all the files and uploads the most recent version, however the actual specified existing archive uploads with an error. So below an archive called release_v1.5.2.zip was created and uploaded just fine
transcriptic release "package-id" -f release_1.5.1.zip
Creating archive with all files within this directory...
Uploading to package with id package-id
Validating package...
Package uploaded successfully! Visit https://secure.transcriptic.com/ellis_lab/packages/package-id to publish.

Sorry I would try and fix it if my python was a little better!

Thanks Ben! This is still a WIP, I'm actually going to be splitting up the upload and release functions later.  I'll deal with this bug too :)

On Aug 15, 2015, at 3:12 PM, Ben<EMAIL_ADDRESS>wrote:
Hey, giving the release branch a play.
I get this error:
➜  protocols git:(provision_fix) ✗ transcriptic release "pk17p2u3upxhpf"
Traceback (most recent call last):
File "/Users/Ben/anaconda/bin/transcriptic", line 9, in
load_entry_point('transcriptic==1.2.0', 'console_scripts', 'transcriptic')()
File "/Users/Ben/anaconda/lib/python2.7/site-packages/click-4.0-py2.7.egg/click/core.py", line 664, in call
return self.main(*args, **kwargs)
File "/Users/Ben/anaconda/lib/python2.7/site-packages/click-4.0-py2.7.egg/click/core.py", line 644, in main
rv = self.invoke(ctx)
File "/Users/Ben/anaconda/lib/python2.7/site-packages/click-4.0-py2.7.egg/click/core.py", line 991, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/Users/Ben/anaconda/lib/python2.7/site-packages/click-4.0-py2.7.egg/click/core.py", line 837, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/Users/Ben/anaconda/lib/python2.7/site-packages/click-4.0-py2.7.egg/click/core.py", line 464, in invoke
return callback(*args, **kwargs)
File "build/bdist.macosx-10.5-x86_64/egg/transcriptic.py", line 198, in release
@click.argument('protocol_name')
File "build/bdist.macosx-10.5-x86_64/egg/transcriptic.py", line 157, in upload
click.echo('Creating empty manifest.json...')
File "/Users/Ben/anaconda/lib/python2.7/posixpath.py", line 121, in basename
i = p.rfind('/') + 1
AttributeError: 'NoneType' object has no attribute 'rfind'
However if I provide the name of an existing archive via the --file argument.
It zips up all the files and uploads the most recent version, however the actual specified existing archive uploads with an error. So below an archive called release_v1.5.2.zip was created and uploaded just fine
transcriptic release "pk17p2u3upxhpf" -f release_1.5.1.zip
Creating archive with all files within this directory...
Uploading to package with id pk17p2u3upxhpf
Validating package...
Package uploaded successfully! Visit https://secure.transcriptic.com/ellis_lab/packages/pk17p2u3upxhpf to publish.
Sorry I would try and fix it if my python was a little better!
—
Reply to this email directly or view it on GitHub.

You're welcome, I really like it.
Sounds great :)

I just made a ton of updates to this branch and outlined instructions in the README, check it out!


Support end-date on az ad app credential reset

Is your feature request related to a problem? Please describe.
I can't create a credential that matches the portal "Never" expires usage.
Describe the solution you'd like
Allow specifying a specific end-date when doing "az ad app credential reset" rather than just a relative number of years.
Describe alternatives you've considered
Specify a large relative number of years, but the expiration date won't match what the portal has for "Never" (always 2299-12-31).
Additional context
A fixed end-date would also be consistent with how az ad app create and update work.

I need this feature for another reason - currently, you cannot create credentials for less than a year. I want to create credentials that are valid only for 6 months - but you can pass 0.6 as value to --years parameter.


Change format from single VCFs to multi-sample VCF instead.

This will drastically simplify output and trying to stick the z-scores for enhancer activity and gene expression in the output.
It will also drastically improve performance for the motif checking

tf_expression.py, activity.py, and gene_expression.py are the files that will have to be reworked for this.

This has been handled for tf_expression.py, activity.py, and will be handled for gene_expression.py, which will borrow heavily from activity.py.


Fix of the wrong spelled event name TwinUpated

Description
In nanoFramework.Azure.Devices.Client.DeviceClient the event TwinUpated is spelled wrong.
Motivation and Context

Fixes nanoFramework/Home#1047

How Has This Been Tested?
Project Builds.
Types of changes

[X] Improvement (non-breaking change that improves a feature, code or algorithm)
[X] Bug fix (non-breaking change which fixes an issue with code or algorithm)
[ ] New feature (non-breaking change which adds functionality to code)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Config and build (change in the configuration and build system, has no impact on code or features)
[ ] Dependencies (update dependencies and changes associated, has no impact on code or features)
[ ] Unit Tests (add new Unit Test(s) or improved existing one(s), has no impact on code or features)
[ ] Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist:

[X] My code follows the code style of this project (only if there are changes in source code).
[X] My changes require an update to the documentation (there are changes that require the docs website to be updated).
[ ] I have updated the documentation accordingly (the changes require an update on the docs in this repo).
[X] I have read the CONTRIBUTING document.
[ ] I have tested everything locally and all new and existing tests passed (only if there are changes in source code).
[ ] I have added new tests to cover my changes.

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

This needs to be updated in de Samples after this PR is merged.
https://github.com/nanoframework/Samples/search?q=TwinUpated


Fix propagation docs

Fixes #525: wrong example provided in the "Stopping propagation" section.

Thanks!


Use OSContainerImage from OpenStackVersion

/test dataplane-operator-build-deploy-kuttl Internal error occurred: resource quota evaluation timed out


Docs update

replaces https://github.com/integr8ly/tutorial-web-app/pull/8

so I am thinking a structure like this will work much better for us since we still need formatting and links inside the verifications. Is there some way you can include the verifications within another ascii doc on your side easily? This is another path which I think will give you more flexibility with fewer steps than JSON...
https://github.com/priley86/integr8ly-prototype/commit/ee00eaffc9ebe5e22d5eacc2851adc63f30da1e1#diff-8dbd8cb7c1f7b2cd546ce1238ee20d19

@finp - one comments from me - we had a request to change the name of EnMasse to Red Hat AMQ Online (or just AMQ Online if you're allowed to use a short name in the docs).

closed in favor of https://github.com/integr8ly/tutorial-web-app/pull/17


Update uuid Formatters

Fixed building against github.com/twinj/uuid
Updated uuid formatters:
Clean to FormatHex
CleanHyphen to FormatCanonical
This duplicates the work in https://github.com/kabukky/journey/pull/74/commits/6c02d9445a1f5c8f365fa409c2de59b8d104a12e however it maintains the separate formats used, rather than just using FormatCanonical in all locations. I've also left the explicit formatting in there despite the default now being FormatCanonical

Rather than conflict with #74 I'm going to close this and open a new one for the master branch. I think the solution in #74 is better given that the only place the Clean format was used was a file path and the CleanHyphen/FormatCanonical format is faster according to https://github.com/kabukky/journey/pull/74/#commitcomment-19155313
I would just like to get the master branch patched to build with matching functionality at this point.


Clicking the month button sometimes doesn't jump to today

Problem: when clicking the large button with the month shown, the calendar doesn't always jump back to the current month if one's at the beginning of a new month...
Expected behavior: Should always jump to the correct current month!

Resolved in latest version.


Upgrade to Node.js LTS/18 + current/20

As Next.js v16 is EOL at Sept 11.

Shouldn't we use the lowest version that we support? Generally newer versions are more backwards compatible.
Unless we use specific features from the newer version

I guess you mean we use only the lowest version, then yeah would be a good idea, imho. Uses less resources too (looking at number of runs and remote caching for instance).
The irony is that just this week Node.js v20.6.0 came with a bug, impacting many popular projects. This is rare and usually fixed quickly, but it's not impossible. Still think one version is enough.


Support display of the concept DOI (DOI for all workflow versions)

TODO: create a new DB dump with the new conceptdoi column in the service, workflow and tool tables

Possibly not show that in STUBs, others should chime in

Restarted build on circle-ci, seems like build was stuck or something