Language selector callback

As per a meeting, if dotcom will provide a language rather a country selector,
KC will need also a callback on selection click so we can handle the url rewrite.
Right now, the footer takes the href from link rel="alternate" hreflang="cn-zh" href="/cn-zh"
we need to suppress this behavior and get a callback instead.

Hi @kyjak ,
I want to clarify what the scope of the feature request is:

Create a property on the footer level to replace the locale button with a languageOnly option
Provide a data structure for populating the language selector. Example:

languageData = [
  {
    label: 'English',
    url: 'https://www.ibm.com/',
    callback: () => {}
  },
  ...
]

The logic will be url by default, or if callback is defined, will use that instead
The language selector will be a standard dropdown

hello,
yes, we can provide an array with available languages to the footer.
there is no need for a separate callback on each entry, a global one with a parameter what language was chosen is enough.
so something like
languageData = [
{
label: 'English', code: 'en'
},
{
label: 'Slovak', code: 'sk'
}
]
and after picking a common callback (set as a prop for example for the footer) would be called as callback('sk')
but other web projects might have different needs, maybe they need also url and separate callback, so we are open to a dialog on this. not sure if KC is the only one for this feature.
we also have a filtering mechanism to search for a language, but that needs confirmation
from our designers, but 90% sure it will stay. look on the current KC page for inspiration

Right, we may just have an open callback method that is injected into the Footer, which is used by the item on select. We can just pass in the event object so you can do whatever is needed with it. Something like this:
languageData = [
  {
    label: 'English', value: 'en'
  },
  {
    label: 'Slovak', value: 'sk'
  },
  ...
]

and:
<Footer
    languageOnly={true}
    languageCallback={event => yourCallbackMethod(event)}
    languageData={languageData}
/>

We were thinking possibly to use the Carbon ComboBox for filtering:
https://react.carbondesignsystem.com/?path=/story/combobox--default
Thoughts @ljcarot @wonilsuhibm ?

@RobertaJHahn we probably need to turn this into an epic. We will will stories for:

design/visual specs
functional spec updates
dev story for implementation
prod qa
design qa

this seems to be implemented already, closing


drivers: flexspi_nor: Fix Flash failures seen on FRDM-MCXN947

The flash_flexspi_nor_check_jedec() checks the lower 16-bits of the manufacturer ID and installs a custom LUT table. Add an exception to check for the whole 32-bits so we do not accidentally install custom LUT for flash devices that do not support this LUT table.
Fixes https://github.com/zephyrproject-rtos/zephyr/issues/80223

@de-nordic , can you help take a look at this bug fix. Appreciate your prompt attention so we can include this in 4.0. This bug is preventing bootup on the QSPI variant of the MCXN947 FRDM board.


how to receive notification when app running in foreground

When the app is in foreground this does not work at all FCM.on("notification", notif => {

    FCM.presentLocalNotification({
          id: "UNIQ_ID_STRING",                               // (optional for instant notification)
          title: "Title",                     // as FCM payload
          body: "Body",                    // as FCM payload (required)
          sound: "default",                                   // as FCM payload
          priority: "high",                                   // as FCM payload
          click_action: "ACTION",                             // as FCM payload
          badge: 1,                                          // as FCM payload IOS only, set 0 to clear badges
          number: 1,                                         // Android only
          ticker: "My Notification Ticker",                   // Android only
          auto_cancel: true,                                  // Android only (default true)
          large_icon: "ic_launcher",                           // Android only
          icon: "ic_launcher",                                // as FCM payload, you can relace this with custom icon you put in mipmap
          big_text: "Show when notification is expanded",     // Android only
          sub_text: "This is a subText",                      // Android only
          color: "red",                                       // Android only
          vibrate: null,                                       // Android only default: 300, no vibration if you pass null
          tag: 'correcto',                                    // Android only
          group: "group",                                     // Android only
          picture: "https://google.png",                      // Android only bigPicture style
          my_custom_data:'my_custom_field_value',             // extra data you want to throw
          lights: true,                                       // Android only, LED blinking (default false)
          show_in_foreground: true                                 // notification when app is in foreground (local & remote)
      });

this line  show_in_foreground: true

@azamkth you have to describe more about your environment. what os? what os version? local or remote notif?

i have the same question.   FCM.presentLocalNotification  is not the solution
i'm sending a message from my server, and when the app is in foreground   the notification is not showed on my notifications bar

@darkkase you can use the local notification like on the on notification call back like this :-

@evollu is what @ansal-exp wrote valid? I tried it but it shows the local notification twice!

@danielfx90 not my intended usage.
in 99% cases you don't need to call local notification specifically in notification callback


[edpm_prepare] Ensure the ctrlplane CR has latest content

Initially the edpm_prepare role apply logic wasn't designed to be called more than once, but now it's possible, leading to the unexpected situation of ci_kustomize picking always the initial install_yamls CR as the input instead of the last applied content. Forcing the ci_kustomize output to the same file always will ensure we consume always the same CR instead of creating the ci_kustomize default one.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

recheck

Not needed anymore.


Adjust "The in operator narrowing" heading level

"The in operator narrowing" header must be one step larger to match the other headers on this page. Otherwise the user thinks it is a subheader.

Agree! Thanks


visibleNodes does't return always proper values

Hey!
I'd like to clarify if this is my mistake or there is a bug. When I type searchTerm that is not found in data for every second change of searchTerm visibleNodes is returning data even if there should be no match. With actual behavior sometimes "no search result" is not appearing. The expected behavior is that visibleNodes returns always only nodes matching with search phrase.
export const Tree = ({ initialData, searchTerm }) => {
  const treeRef = useRef(null);

  const [isNoSearchResult, setIsNoSearchResult] = useState(false);

useEffect(() => {
    const treeCurrent = treeRef.current;
    const isNoSearchResult =
      treeCurrent?.visibleNodes &&
      isEmpty(treeCurrent?.visibleNodes) &&
      searchTerm !== "";

    setIsNoSearchResult(isNoSearchResult);
  }, [searchTerm]);

  return (

      {isNoSearchResult ? (

No search result

      ) : (
        <TreeArborist
          initialData={initialData}
          ref={treeRef}
          searchMatch={(node) =>
            node.data.label.toLowerCase().includes(searchTerm.toLowerCase())
          }
          searchTerm={searchTerm}
        >
          {Node}
        </TreeArborist>
      )}

  );
};

Same issue for me. Any updates on how to fix this?


Fix datachannel accept EOF check logic

Fix the logic inverted by c1467e4
Reference issue
https://github.com/pion/webrtc/pull/2194/files#r870839078

Done, tagged v3.1.38
I really want to get v3.2.0 out soon. Just haven't found the time. I will start a issue this week and try to get everyone excited/onboard with it.


[Turbopack] [500] error: no storage space

What version of Turbopack are you using?
How to check?
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Linux
Describe the Bug
Vanilla next with turbopack install:
npx create-next-app --example with-turbopack
followed by devmode:
npm run dev
or
yarn dev
App starts, hitting root dir at post 3000 gives 500 error:
no storage space
server console shows:
[500] error: no storage space (151ms)
[500] error: no storage space (0.387ms)
This is Debian:
lsb_release -a
Distributor ID: Debian
Description:    Debian GNU/Linux 11 (bullseye)
Release:        11
Codename:       bullseye
Expected Behavior
Display the ui in dev mode
To Reproduce
as described
Reproduction Repo
No response

@MTyson works for me. Well, there are issue, but not when I'm opening root directory.
$ node --version
v18.11.0
$ npm --version
8.19.2

i am having the same issue, with differences in the environment.
i am using yarn, and this is my OS
Distributor ID:	Ubuntu
Description:	Ubuntu 22.04.1 LTS
Release:	22.04
Codename:	jammy

Is anyone still seeing this with recent versions?

Closing this as I haven't heard of anyone experiencing this recently. Let us know if we need to reopen.


Sagemaker docker build error

I have been following the scikit_build_your_own tutorial and am facing issue when building the docker image in Sagemaker. When I run build_and_push.sh, I get the following errors:
Login Succeeded
Sending build context to Docker daemon  24.06kB
Step 1/10 : FROM ubuntu:16.04
---> 4b22027ede29
Step 2/10 : MAINTAINER Amazon AI<EMAIL_ADDRESS>---> Using cache
---> 41070ab442d9
Step 3/10 : RUN apt-get -y update
---> Using cache
---> e34c45df332b
Step 4/10 : RUN apt-get install -y --no-install-recommends          wget          python          nginx          ca-certificates     && rm -rf /var/lib/apt/lists/*
---> Running in 5e7bea544eac
Reading package lists...
Building dependency tree...
Reading state information...
E: Unable to locate package wget
E: Unable to locate package python
E: Unable to locate package nginx
E: Unable to locate package ca-certificates
The command '/bin/sh -c apt-get install -y --no-install-recommends          wget          python          nginx          ca-certificates     && rm -rf /var/lib/apt/lists/*' returned a non-zero code: 100

It was a permissions error in SageMaker


Multipart Upload With 1 file and 1 parameter

I have a Spring MVC method that accept application/multipart with the following parameters:
@RequestPart("file") MultipartFile file,
@RequestPart(value = "aString", required = false) String aString
The MultipartFile is well upload but for the aString variable, I cannot get it. After investigation, I've found that the root cause is the fact that for the multipart request, the Content-Type is not set for that part. And Spring doesn't seems to consider that the Content-Type is optinal.
For sure, a work around is to use the @RequestParam but content is not send the same way using this.
Hope the issue is easy to undertstand.
Thanks a lot.
Jeremy

@jcarnus it is! Thanks for reporting, just to clarify tho' swagger-ui does not support multipart very well nor the swagger 2.0. Not sure about the 3.0 support in the spec


New commits api params support

Hi, NARKOZ
There are two new params 'since' and 'until' in commits api, so could you make a new release to support this feature?
http://docs.gitlab.com/ce/api/commits.html
GET /projects/:id/repository/commits

since	string	no	Only commits after or in this date will be returned in ISO 8601 format YYYY-MM-DDTHH:MM:SSZ
until	string	no	Only commits before or in this date will be returned in ISO 8601 format YYYY-MM-DDTHH:MM:SSZ
Thank you very much

We allow arbitrary options to be passed to the commits method so this should already be possible with the current release.

Please re-open if you're unable to use the new params.

Got it! Thank you


Eliminating Multiple Ways to Define Center Pivots

At one point we extended the C# model to allow for full-circle center pivot prescriptions being defined either as a center + radius or 3 points on the circle.   This arose out of different OEMs preferring different methods and not wanting to depart from the source data.
In the spirit of creating a standard, we should streamline this into a single method, and I would propose center + radius is the more conventional of the two.   While the logic to convert 3 points into a center+radius requires a couple calculations, it is arguably no different than our requiring implementers convert geospatial coordinates to WGS84 should they originate in an an alternative datum.

Confirmation at Sept 7 2022 meeting

Further from the Sept 7 meeting was that Pivot should be a single type for Full and Partial pivots with attributes defined accordingly.   Changes made.


Release 10.0.4

@PVince81 review on https://staging.owncloud.org/changelog and https://staging.owncloud.org/install#edition - is this fine?
not sure if the web updater is already 10.0.4 - should I manually leave it at 10.0.3?
Anyway, as soon as this is live I can publish the announcement blogpost, calling it a release :)

Looks fine please deploy :+1:


fix(repo): fix coverage scripts & setup

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #1   +/-   ##
=====================================
  Coverage     2.6%   2.6%
=====================================
  Files          11     11
  Lines         115    115
  Branches       14     14
=====================================
  Hits            3      3
  Misses        112    112

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8879713...265a6fe. Read the comment docs.


dma/cavs_hda: Configure DGMBS only for host DMA configuration

The DGMBS should be only set for the host side and not on the link DMA.
Fixes: #46632
Signed-off-by: Peter Ujfalusi<EMAIL_ADDRESS>

Current explanation is based on experiment and reference to the sof code.
So far I cannot derive an authoritative conclusion from the specs.
I am still digging into the related spec.
Hope some Intel HDA guru can shed some light on this.

@andyross, it is still unclear why tests/boards/intel_adsp/hda/src/smoke.c fails if DGMBS is configured.
The smoke is testing the host DMA, the smoke uses SPIB. The same conditions applies to SOF implementation.
The DGMBS is used internally by the hardware to calculate the FIFOS register (afaik) which is the information register used (or supposed to be used) by host to know the minimum buffer requirement.
44ef4c4737c05fb4c5ce9931badf4cdf3065d1bf broke smoke test and based on the commit message the DGMBS configuration is needed via normal use (using the DMA API).
This patch will unblock the smoke testing while keeping the 44ef4c4737c05fb4c5ce9931badf4cdf3065d1bf cited requirement to configure DGMBS.
In parallel we will try to figure out the reason why smoke fails and why SOF/Linux works in seemingly identical configuration regarding to host only DMA use.


Lit protocol tutorial & wagmi v2 updates

Summary

Adds a walkthrough for how to use Lit for encrypting/decrypting data with your private key.
Updates the wagmi/viem examples to use the new v2, which has breaking changes.
Some small tweaks to clean up SQL FAQs and showcase pages.
Small marketing updates to use the word "onchain" more.

@joewagner nice typo catch, fixed! and i just squeezed in a GH action to fix an issue where Algolia search was not indexing new docs pages; it calls the Algolia API to trigger reindexing upon merges to main


"Send all [asset]" wants to send unrelated assets too

Hi everyone,
Trying to send some assets between wallets, there might be a bug when using the the 'Send all" checkbox.
When checking the "Send all [asset]" for a given transaction

The wallet shows a warning, however displays that it wants to send all the assets in the wallet

The feature works properly with the browser extension.

Hi, @AliHoussein ! Thank you for the report, we are investigating this top priority at the moment.
Can you please clarify and that would help a lot with resolving the problem, in this case - are these ALL the tokens you have at all in that wallet, or only some of them? It would be really really helpful to us to clarify if it tries to send ALL tokens you have or only selects SOME of them.

@AliHoussein , which version of the app do you have, btw, and which operating system?

The issue is investigated. This is a displaying problem. The actual constructed transaction is correct and will only send the selected asset, but it incorrectly displays some accidental assets just because they are getting mixed in with the inputs. But they are not leaving the wallet in reality. This displaying is being fixed atm and the new version 4.5.2 will be pushed right after

Fixed in the version 4.5.2 which is bein deployed at the moment


Cleanup props on User Schema and DB records for a User

Fields to Remove

[ ] github

Not set anymore, and not used on server/client

[x] isGithub
[x] isLinkedIn
[x] isTwitter
[x] isWebsite
[x] password
[ ] timezone

Not currently used. Do we plan to?

[ ] verificationToken

Appears to be entirely related to LB
Not set this anywhere except to null in User.confirm which is never directly called

[ ] badges

Not currently used. Do we plan to?

[ ] rand

Do we need this?

Transformation Strategy
We have concluded an initial (needed) normalization will involve transforming optional array types into [].
completedChallenges.files is an array type, but does not exist on every completed challenge. Do we want to add files: [] to every single completedChallenges element? If not, we will not be able to make use of strong typing using Prisma.

completedChallenges

remove all __<> properties

profileUI

remove all __<> properties

progressTimestamps

[{ timestamp: Double }] -> [Double]
[Null] -> []
[Int64] -> [Double]

partiallyCompletedChallenges

Undefined -> []
[[]] -> []
Does this have any actual values? What is this used for?

yearsTopContributor

Undefined -> []
[String] -> [Double]

A string of length 4 takes 24 bytes, a double takes 8 bytes

savedChallenges

Undefined -> []

badges

Undefined -> []
Remove?

The is* flag seems to be present in the code, last I checked. Does it have any impact?

Do you think removing has any impact?

Thank you for making me double-check. The is<> fields are never used, but are still being set:
https://github.com/freeCodeCamp/freeCodeCamp/blob/b24c939322430cdee2daa51e258e2f8991353241/api-server/src/common/models/user.js#L808-L827
At one point, I assume they were used for logic on the client like:
if (isGithub) {
  return <h1>{user.githubProfile}</h1>
}

Now, we just use:
https://github.com/freeCodeCamp/freeCodeCamp/blob/b24c939322430cdee2daa51e258e2f8991353241/client/src/components/profile/components/social-icons.tsx#L90

Did you get a chance to profile the memory and CPU usage?

Yes, but only from a whole-system perspective - I did not see an easy way to log MongoDB-specific usage other than htop.

Process
CPU [%]
Memory [GB]

mongod
20 < x < 30
18 < x < 23 *

script
60 < x < 70
6 < x < 8

4GB of swap was used, but I do not know who ate this

Do you have the specs for the machine you run this on? How many CPUS, total memory?

Yes, but I do not think it is remotely comparable. As discussed, we should go with a staging test. Then, the only thing we need to decide is what machine runs the script.

Then, the only thing we need to decide is what machine runs the script.

I'd suggest spinning up a DigitalOcean droplet to run the script and destroying it once done.

The normalization script is ready (for review at least).
Action Plan

[ ] Merge normalization script
[ ] Generate hammer script to 🔨 the API

Typical Camper behaviour causing reads/writes to DB
Match typical concurrents
[ ] Merge

[ ] Seed staging with production

Upgrade to match prod

[ ] Deploy normalization script to droplet 💧
[ ] Deploy hammer script possibly to same 💧
[ ] Run and observe

Is this post-mvp?

It doesn't seem strictly necessary, so post-MVP seems best.

@ShaunSHamilton based on our recent discussions, I think we're likely to do this after deploying the new api. As such, I'm moving it to Post-MVP. Please move it back if I've misunderstood.

To prevent LB from adding back stuff we do not want, this should be Post MVP. Thanks, @ojeytonwilliams


[Feature] http proxy with headers

Verify steps

[X]  我已经在 Issue Tracker 中找过我要提出的请求 I have searched on the issue tracker for a related feature request.
[X]  我已经仔细看过 Documentation 并无法找到这个功能 I have read the documentation and was unable to solve the issue.

Description
Please add feature for http proxy with custom headers
Possible Solution
https://github.com/MetaCubeX/Clash.Meta/compare/Alpha...Dreamacro:clash:master#diff-893a0ac2bbce893e1c788c7b741cedeeda39663f7970cf10686dd4a5277c528b

https://github.com/MetaCubeX/Clash.Meta/blob/55255faa52455fdc51ae2037f3fc885f2cecb5b8/adapter/outbound/http.go#L42 This is a feature that already exists


CI: Add workflow to calculate build size diff before & after PR/commit.

This PR uses Google's Bloaty McBloatface to analyse the MICROBIT elf files for memory consumption.
It also builds the PR base commit, or the commit parent, to do a diff and see the additional memory consumed or saved.
In the case of being a PR from a branch in this repository (not forks) it will post the diff as a comment as well.
To show a representative diff in this PR (instead of an output just showing "no change in size"), the initial commit will contain a bit of code in MicroBit.cpp to create a 4000 byte array and a loop to write some data into it. So, the first bot comment will show the PR as increasing the .bss and .text sections.
I will then force push an update removing this code changes and the bot will post another comment showing a zero size increase.
Both bloaty analyses are added to to the Action work summary, so even PRs from forks will have this info (just not as a PR comment):

Okay, so now I'll force push without the test C++ code to make the build bigger and the bot should comment again indicating the build size for this PR hasn't changed.


Update prepare-network.md

adding the new IP range <IP_ADDRESS>/14

this is a fairly straight forward pull request - if you have any questions/concerns reach out to me internally at roshanp


Fix lint/clippy on main

Just a simple cargo fmt.

@TimonPost is this ready to be merged? Would be nice to solve incessant CI failures on main :)


connection refused error

hi i have managed to build the project but habving issues when running
./level-editor-debug ../../../samples/01-physics/
here is the outputs:
(level-editor-debug:29795): GLib-GObject-CRITICAL **: g_value_get_object: assertion 'G_VALUE_HOLDS_OBJECT (value)' failed
** (level-editor-debug:29795): CRITICAL **: gee_abstract_collection_foreach: assertion 'self != NULL' failed
Could not connect to <IP_ADDRESS>: Connection refused
Could not connect to <IP_ADDRESS>: Connection refused
Error receiving data: Connection reset by peer
Error sending data: Broken pipe
Error sending data: Broken pipe
Error sending data: Broken pipe

Can you pull latest and paste here the output?

hi this seems fixed now, fixed now. by the way nice engine.

Thank you. :)


doc: force docutils version below 0.17

Even though sphinx_rtd_theme forces this condition, it seems to be
ignored by the pip dependency resolver due to a ">=0.12" found in
other dependencies.
.venv2/lib/python3.8/site-packages/breathe-4.28.0.dist-info/METADATA:Requires-Dist: docutils (>=0.12)
.venv2/lib/python3.8/site-packages/sphinx_rtd_theme-0.5.2.dist-info/METADATA:Requires-Dist: docutils (<0.17)
.venv2/lib/python3.8/site-packages/Sphinx-3.3.1.dist-info/METADATA:Requires-Dist: docutils (>=0.12)
.venv2/lib64/python3.8/site-packages/breathe-4.28.0.dist-info/METADATA:Requires-Dist: docutils (>=0.12)
.venv2/lib64/python3.8/site-packages/sphinx_rtd_theme-0.5.2.dist-info/METADATA:Requires-Dist: docutils (<0.17)
.venv2/lib64/python3.8/site-packages/Sphinx-3.3.1.dist-info/METADATA:Requires-Dist: docutils (>=0.12)

current changes seem to work, docutils 0.16 gets installed:
Collecting docutils<0.17
  Using cached docutils-0.16-py2.py3-none-any.whl (548 kB)

thanks, now this is fixed, at least by looking at the artifacts of this PR


ERROR WHILE REVIEWING 3D MODEL IN 3D SCENE

To whom it may concern,
I followed the instruction but I received an error with the message "Error loading model -
This could be a CORS issue, invalid blob url or a network error. Please contact the owner of the container where this asset file is stored or try again later".
Please help me if any possible solution for it.
Thank you so much.
Sincerely.

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

ID: 222007c7-92a3-9075-f54f-8e76532141d2
Version Independent ID: 382238d5-75d2-6e48-d9be-d7e1267e0fc4
Content: Use 3D Scenes Studio (preview) - Azure Digital Twins
Content Source: articles/digital-twins/how-to-use-3d-scenes-studio.md
Service: digital-twins
GitHub Login: @baanders
Microsoft Alias: baanders

Hello @ChiAnLE Greetings. Thank you for reporting the issue. We will test the steps provided in the article and update you our observations here.
Best regards,
Leela

I have fixed that and understood it

@chianle67  can you provide the solution for this issue. I am facing the same issue Thanks!!

Hi @chianle67 Can you please open a new thread regarding this issue on Microsoft Q&A platform? The community and the SMEs on this topic on the platform would be able to assist you on this. Kindly share additional information such as any error messages you see in the platform to help the community better understand where you got stuck.
Thank you,
Best regards,
Leela Rajesh.


[Documentation] Create storage module README

There's a need to create a README for the storage module and populate it with the documentation regarding the functions to list and delete directories.

I believe this is done, right @cristianacarpinteiro?

Yes @begaiym-k! Closing it :)


Refactoring Patches

Use Patch class
Add dataclass
Remove legacy code

Codecov Report

Merging #643 (5487724) into main (bde0ef2) will increase coverage by 6.65%.
The diff coverage is 95.56%.

:exclamation: Current head 5487724 differs from pull request most recent head cdfb5a2. Consider uploading reports for the commit cdfb5a2 to get more accurate results

@@            Coverage Diff             @@
##             main     #643      +/-   ##
==========================================
+ Coverage   82.78%   89.43%   +6.65%
==========================================
  Files         159      159
  Lines       12678    12847     +169
==========================================
+ Hits        10495    11490     +995
+ Misses       2183     1357     -826

Impacted Files
Coverage Δ

tests/start/test_start_hf.py
100.00% <ø> (ø)

dataquality/integrations/keras.py
94.15% <93.46%> (+94.15%)
:arrow_up:

tests/integrations/semantic/test_torch_semantic.py
94.07% <94.07%> (ø)

dataquality/integrations/torch.py
92.10% <95.00%> (+3.11%)
:arrow_up:

dataquality/utils/torch.py
82.55% <98.19%> (+2.12%)
:arrow_up:

dataquality/analytics.py
77.60% <100.00%> (-0.53%)
:arrow_down:

...uality/integrations/torch_semantic_segmentation.py
94.65% <100.00%> (+94.65%)
:arrow_up:

dataquality/integrations/transformers_trainer.py
94.73% <100.00%> (+0.09%)
:arrow_up:

tests/conftest.py
99.18% <100.00%> (+0.02%)
:arrow_up:

tests/integrations/keras/test_end2end_keras.py
99.20% <100.00%> (ø)

... and 1 more

... and 23 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


GloDroid on RPi CM4

I'd like to test out GloDroid on the Raspbery Pi CM4.  The installer for Pi 4B only comes with an installer for SD cards but not the eMMC related files.  Is this something that I can take from another device or are there contents that are hardware specific?

I am going to post this in the manifest repo.  Unfortunately, I am unable to delete the duplicate post here

Duplicate of https://github.com/GloDroid/glodroid_manifest/issues/186


[RESOLVED] Unavailable by ICMP ping

Problem started at 21:03:29 on 2023.06.27
Problem name: Unavailable by ICMP ping
Host: HP_iLO
Severity: High
Operational data: Down (0)
Original problem ID: 31155
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23266&eventid=31155

Problem has been resolved in 7m 0s at 21:10:29 on 2023.06.27
Problem name: Unavailable by ICMP ping
Host: HP_iLO
Severity: High
Original problem ID: 31155


Unable to tell what bit-architecture of Devcon that I'm using.

So at work our computers comes with devcon pre-installed but nothing else from the WDK that I'm aware of. I'm trying to make use of it with a USB-connected device but it seems that all of the commands are failing for it. I've found an answer on SO mentioning that I might be using the wrong version which makes sense since it's a common issue around here. The problem is I have no way of confirming this, because nowhere I've looked tells me anything about which bit-architecture I'm using.
I understand that I might be digging down the wrong rabbit hole here, but shouldn't there be easy way of telling what version I'm using? I.e. Put in a version option for the command?

It is recommended that you use the inbox pnputil.exe instead of devcon.  pnputil supports most of the functionality that devcon does.  Please see https://learn.microsoft.com/en-us/windows-hardware/drivers/devtest/devcon-migration


chore: bdd-nh-1590489290 to 0.0.1

chore: Promote bdd-nh-1590489290 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Automatically source the virtual env when entering the development docker container

This is a tiny quality of life thing. I think it'd be handy to do this so you don't have to remember to do it yourself! I also updated the readme to use docker-compose to exec into the development container so it's also easier to remember and less sensitive to the container naming issue.

@airhorns my only concern here was when you need to do something not inside the venv, but then I realized that that is far less of an occurrence than using the venv, and you can deactivate.


No Channel/app/input selection in Homekit nor Homebridge

After installing the plugin, I have the icon to turn on/off, changing volume, but channel selection is not available in both Homekit and Homebridge.
HomeKit show something like this
https://i.imgur.com/UggoMrT.jpeg
Homebridge website show something like below
https://i.imgur.com/axlWFNF.jpeg
The channel does show up in the configuration screen like below, it just not showing a tile that allow you to change channel/app/input in homebridge nor home kit.
https://i.imgur.com/oZWdlBh.jpeg
Am I missing something here?

I got exactly same issue.
Homebridge version: 1.3.8
Homebridge Bravia Tvos version: 5.0.8

I had the same issue but resolved it. I am running the plugin as a child bridge and yet is seemed to be to the main bridge. I removed the accessory from homkit then removed the TV accessory from hombridge in hombridge settings and the plugin bridge which was added somewhere along the way.

Thanks @valiquette, child bridge is the trick here. I remove the TV accessory from both HomeKit and HomeBridge (delete it from the plug-in). Then I changed the plug-in to be child bridge mod. Restart HomeBridge. Pair child bridge with HomeKit. Then add my TV in the plug-in again along with all inputs and apps configured. Restart HomeBridge again. Pair the TV with HomeKit. It should tell you it finds all inputs and apps. If not make sure those are shown as separated accessories inside TV accessory in HomeKit. Then force close HomeKit app and restart it. It should now work.

Thanks @valiquette, child bridge is the trick here. I remove the TV accessory from both HomeKit and HomeBridge (delete it from the plug-in). Then I changed the plug-in to be child bridge mode. Restart HomeBridge. Pair child bridge with HomeKit. Then add my TV in the plug-in again along with all inputs and apps configured. Restart HomeBridge again. Pair the TV with HomeKit. It should tell you it finds all inputs and apps. If not, make sure those are shown as separated accessories inside TV accessory in HomeKit. Then force close HomeKit app and restart it. It should now work.

I finish "Pair child bridge with HomeKit. "
Then, how to "add my TV in the plug-in again along with all inputs and apps configured. "?

I finish "Pair child bridge with HomeKit. "
Then, how to "add my TV in the plug-in again along with all inputs and apps configured. "?

Same way how you configured your TV with the plug-in initially. https://github.com/seydx/homebridge-bravia-tvos#configuration-config-ui-x. Note that you have to remove the TV from plug-in before switching to child bridge.

You may not have to remove the TV from the plugin, just remove the accessory from the cached accessories from the homebridge settings, then you will not have to re-configure your TV. It is not required as far as I can tell to pair the child bridge, since there are no bridge accessories in the plugin. Force restarting HomeKit does seem to be a must.

Ok, there are more tricks needed in order to get it to work.

When you add the TV in the plug-in once again after you change to child bridge mode , you HAVE to add the TV in a different name than what you were used on the child bridge.  Just use some other name than the first time you used.   Otherwise, it won't see the newly add TV.

After you add the TV and able to choose from the drop down list and click on the edit button, you will have to manually add every single channel you want to see one by one there.  And then add all the application one by one in there also.

Then, you will need to click on the "Configs" link in the plugin, this is where you manually add the channel once again which you do that in step 2.  Every single app and channel, need to add one by one manually.  Yes, all that once again.

After restarting the Homebridge the last time, it should show the channel, volume and on/off switch in one single page like below


feat(dark-mode): implement dark mode and centralize CSS in WebUI

This Pull Request introduces significant changes to the styling and functionality of the web pages. Most notably, it performs a clean-up of repetitive CSS across multiple HTML files (add.html, index.html, links.html, and log.html) by removing duplicated style declarations. These styles were explicitly defining font families, background colors, and table styles directly within each HTML file. This redundancy is removed, and instead, the styling definitions are centralized and now reside within main.js.
Furthermore, the PR adds a comprehensive dark mode feature across the website. Two key additions facilitate this:

A toggle button (🌙 for dark mode and ☀  for light mode) is introduced in the navigation. This button allows users to switch between light and dark themes.
CSS definitions for dark mode styling are included, targeting various elements such as the body background, text color, navigation, tables, buttons, inputs, and placeholders. This ensures a consistent look and feel when dark mode is activated.

The implementation also respects the user's system preference for dark or light themes through the prefers-color-scheme media query, providing an adaptive and user-friendly experience. The dark mode state is persisted across sessions using localStorage, ensuring that users' preferences are remembered.
Additionally, JavaScript logic is added to dynamically update the dark mode state and the Ace editor theme based on the user's selection or system preference. An event listener for the DOMContentLoaded event ensures that the dark mode toggle and theme are correctly initialized when the page loads.
Overall, this Pull Request enhances the website's accessibility and user experience by introducing a much-requested dark mode feature and simplifying the CSS structure for easier maintenance and consistency across web pages.

Thanks!

https://github.com/AlexxIT/go2rtc/releases/tag/v1.9.0


Configurable marathon uri prefix

In our DCOS deployment we have multiple marathon with different names. These names are used in the marathon API URL.
For example:

...:9090/service/marathon/v2/deployments
...:9090/service/mon-marathon-service/v2/deployments

The library uses fixed name marathon https://github.com/gambol99/go-marathon/blob/master/client.go#L247. I think it should be configurable to be able to use multiple marathon services.

@vishr thanks for creating the issue. This sounds like a reasonable request. My proposal would be to extend the configuration so that custom names can be defined.
Happy to review any PR you may want to submit. :-)

hi @vishr does https://github.com/gambol99/go-marathon/issues/179 fix your issue?

I'll close this issue for now. @vishr, if you think #179 doesn't solve your problem, please don't hesitate to reopen and add more details.
Thanks.


Add go modules

Add go.mod, go.sum and cached dependencies in vendor directories for each package
Pull requests into this repository require the following checks to be completed by the submitter and reviewers.
Submitter:

[x] Administrative tasks are complete.

This pull request describes why these changes were made.
The JIRA ticket links to this PR.
The JIRA ticket for this PR is:
https://oncprojectracking.healthit.gov/support/browse/LANTERN-68
https://oncprojectracking.healthit.gov/support/browse/LANTERN-69

[x] Tests are complete. NA

Tests are included and test edge cases.
Tests have been run locally and pass.

[x] Code review has been performed.

Code diff has been done and checked to ensure only expected code is being committed.
A linter for the language has been run.

Primary Reviewer:

[ ] Code works.

Code has been run locally and the basic functionality affected by the PR has been checked.
Edge cases have been checked to ensure behavior is as expected.

[ ] Code review has been performed.

Code is maintainable, reusable, efficient, and correct.
Code accomplishes the tasks purpose.
Code follows style guidance appropriate for the language, including passing linter checks.
Code is well commented (which does not mean verbosely commented).

[ ] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

Secondary Reviewer:

[ ] Code works.

Code has been run locally and the basic functionality affected by the PR has been checked.
Edge cases have been checked to ensure behavior is as expected.

[ ] Code review has been performed.

Code accomplishes the tasks purpose.

[ ] Tests are complete.

Tests have been run locally and pass.

Merger:
Prior to merging, ensure that all checks have been completed. Delete the branch after the merge is complete.

Is there are a reason why you pushed all the dependencies? It seems like that isn't necessary since it doesn't seem difficult to set up the dependencies locally, but maybe I'm missing something.

Is there are a reason why you pushed all the dependencies? It seems like that isn't necessary since it doesn't seem difficult to set up the dependencies locally, but maybe I'm missing something.

Seems like there is no clear consensus among go users on whether or not to commit the vendor directory. Pros are that you have a cache of all your deps so if one goes away your project will still work, also reduces the number of deps that need to be retrieved on install. Cons are that the repo takes up more space (even though those deps would have to be pullled down eventually). I followed the convention in a couple other go projects https://github.com/kardianos/govendor/tree/master/vendor https://github.com/prometheus/prometheus/tree/master/vendor and decided to keep the vendor directory.


Module

I would like to move this into a module in the /backend folder, i have moved it to that folder and rewriten all the namespaces but i seem to have done something wrong, i get this error while posting the login form..."PHP Fatal Error", message: "Call to a member function where() on null", code: 1".
Can you please help?

There is not tutorial on this, and I doubt that it will created any time soon.


[Bug] Spring MVC @RequestBody 调用错误

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
Java Agent (apache/skywalking-java)
What happened

接收的对象中有 BigDecimal 字段，经过 agent 字节码加强后，调用该方法报错：

What you expected to happen
去掉skywalking后方法恢复正常。
How to reproduce
上面有代码。
Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please use English on the GitHub discussion.


Fixed saving of rest headers from response if body is empty.

Closes #541

Question - do we want to fail or return null when header that is not present is being saved? Current fix makes the value being saved in context null (instead of previous NPE)

Question - do we want to fail or return null when header that is not present is being saved? Current fix makes the value being saved in context null (instead of previous NPE)

I'd say, it should behave the same way as for saving parts of the response body, when the path is missing.

I'd say, it should behave the same way as for saving parts of the response body, when the path is missing.

I'd say, it should behave the same way as for saving parts of the response body, when the path is missing.

Good point, currently we do not solve it on our side and let json path implementation handle it, eg:
com.jayway.jsonpath.PathNotFoundException: No results for path: $['asdName']
So I added assert to the headers... Do we want to add assert to json path as well (to keep the exceptions the same?)

I'd say, it should behave the same way as for saving parts of the response body, when the path is missing.

Good point, currently we do not solve it on our side and let json path implementation handle it, eg:
com.jayway.jsonpath.PathNotFoundException: No results for path: $['asdName']
So I added assert to the headers... Do we want to add assert to json path as well (to keep the exceptions the same?)

Do we want to add assert to json path as well (to keep the exceptions the same?)

I think it is not necessary.

Do we want to add assert to json path as well (to keep the exceptions the same?)

I think it is not necessary.


Alt key not working properly for control characters

Alt+Enter, Alt+Tab, Alt+Ctrl+C, etc do not work properly: they are returned as KeyMsg{Type: KeyRunes, Alt: true} instead of the proper Type.

Because of this bug, to define a key binding for KeyEnter one has to do this instead:
  AlwaysComplete:  key.NewBinding(key.WithKeys("alt+\r")),

which is ugly and disappointing. I wish alt+enter would work instead. The patch in #428 fixes it.


Error fetching terraform module in blast-radius

I am trying to create a graph using blast-radius but it's throwing below error, can anyone please help me.
I think it's unable to fetch terraform module.
Thank you
` * Serving Flask app "blastradius.server.server" (lazy loading)

Environment: production
WARNING: This is a development server. Do not use it in a production deployment.
Use a production WSGI server instead.
Debug mode: off
Running on http://<IP_ADDRESS>:4004/ (Press CTRL+C to quit)
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET / HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/js/jquery.slim.min.js HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/js/bootstrap.min.js HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/js/d3.v4.js HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/js/fontawesome-all.min.js HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/js/blast-radius.js HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/js/d3-tip.js HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/js/categories.js HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/js/selectize.js HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/css/bootstrap.min.css HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/css/selectize.css HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/js/svg-pan-zoom.js HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /static/css/style.css HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 15:56:46] "GET /favicon.ico HTTP/1.1" 404 -
[2020-09-02 15:56:53,479] ERROR in app: Exception on /graph.svg [GET]
Traceback (most recent call last):
File "/home/harsh896/.local/lib/python3.8/site-packages/blastradius/handlers/dot.py", line 455, in _module
return m.groupdict()['module']
AttributeError: 'NoneType' object has no attribute 'groupdict'

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/flask/app.py", line 2446, in wsgi_app
response = self.full_dispatch_request()
File "/usr/lib/python3/dist-packages/flask/app.py", line 1951, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/lib/python3/dist-packages/flask/app.py", line 1820, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/usr/lib/python3/dist-packages/flask/_compat.py", line 39, in reraise
raise value
File "/usr/lib/python3/dist-packages/flask/app.py", line 1949, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/lib/python3/dist-packages/flask/app.py", line 1935, in dispatch_request
return self.view_functionsrule.endpoint
File "/home/harsh896/.local/lib/python3.8/site-packages/blastradius/server/server.py", line 36, in graph_svg
dot = DotGraph('', file_contents=run_tf_graph())
File "/home/harsh896/.local/lib/python3.8/site-packages/blastradius/handlers/dot.py", line 56, in init
self.nodes.append(DotNode(e.target))
File "/home/harsh896/.local/lib/python3.8/site-packages/blastradius/handlers/dot.py", line 415, in init
self.module         = DotNode._module(self.label) # for module groupings. 'root' or 'module.foo.module.bar'
File "/home/harsh896/.local/lib/python3.8/site-packages/blastradius/handlers/dot.py", line 457, in _module
raise Exception("None: ", label)
Exception: ('None: ', '[root] module.services (close)')
<IP_ADDRESS> - - [02/Sep/2020 15:56:53] "GET /graph.svg HTTP/1.1" 500 -
`

I have the same issue, guessing it'll be a tf0.13 issue

Same issue, terraform 0.13.5

I have it with Terraform v0.14.9

I have the issue with Terraform v0.15.3

I have a same issue with 0.13.6 :( So sad I just found this great project which may help me visualizing Terraform graph.

I just did some debugging. It seem that in my case (and maybe the case of @NeerajJ-addweb also), there is a module in the TF state file that does not exist in the code anymore. I don't know how it happen. The last modification on the module's resources was last year. It seems to be an issue from TF that blast-radius cannot handle.

Same error.
AttributeError: 'NoneType' object has no attribute 'groupdict' in blastradius/handlers/dot.py.

Same here on v1.0.4:
[2021-08-17 11:18:46,902] ERROR in app: Exception on /graph.svg [GET]
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/blastradius/handlers/dot.py", line 455, in _module
    return m.groupdict()['module']
AttributeError: 'NoneType' object has no attribute 'groupdict'

I have the issue with Terraform v0.15.5

same issue with terraform v1.2.9
[2022-09-23 22:09:27,966] ERROR in app: Exception on /graph.svg [GET]
Traceback (most recent call last):
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/blastradius/handlers/dot.py", line 455, in _module
    return m.groupdict()['module']
AttributeError: 'NoneType' object has no attribute 'groupdict'

During handling of the above exception, another exception occurred:

Not sure if this will help, but you can try using my version of Blast Radius which I've tried to increase compatibility for.
Source Code:
Blast Radius Fork
Installation:
python3 -m pip install git+https://github.com/Ianyliu/blast-radius-fork
docker run --rm -it -p 5000:5000 \   -v $(pwd):/data:ro \   --security-opt apparmor:unconfined \   --cap-add=SYS_ADMIN \   ianyliu/blast-radius-fork
Docker image:

Not sure if this will help, but you can try using my version of Blast Radius which I've tried to increase compatibility for.
Source Code: Blast Radius Fork
Installation: python3 -m pip install git+https://github.com/Ianyliu/blast-radius-fork
docker run --rm -it -p 5000:5000 \ -v $(pwd):/data:ro \ --security-opt apparmor:unconfined \ --cap-add=SYS_ADMIN \ ianyliu/blast-radius-fork
Docker image:

got error like this
[2022-09-25 07:48:26,018] ERROR in app: Exception on /graph.svg [GET]
Traceback (most recent call last):
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/lark/parsers/lalr_parser.py", line 59, in get_action
    return states[state][token.type]
KeyError: '__ANON_3'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/flask/app.py", line 2077, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/flask/app.py", line 1525, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/flask/app.py", line 1523, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/flask/app.py", line 1509, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/blastradius/server/server.py", line 134, in graph_svg
    dot = initalizeDotGraph(content=run_tf_graph(),
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/blastradius/server/server.py", line 215, in initalizeDotGraph
    tf = Terraform(os.getcwd())
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/blastradius/handlers/terraform.py", line 29, in __init__
    self.config = hcl.load(config_io)
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/hcl2/api.py", line 9, in load
    return loads(file.read())
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/hcl2/api.py", line 18, in loads
    return hcl2.parse(text + "\n")
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/lark/lark.py", line 464, in parse
    return self.parser.parse(text, start=start)
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/lark/parser_frontends.py", line 115, in parse
    return self._parse(token_stream, start)
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/lark/parser_frontends.py", line 63, in _parse
    return self.parser.parse(input, start, *args)
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/lark/parsers/lalr_parser.py", line 35, in parse
    return self.parser.parse(*args)
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/lark/parsers/lalr_parser.py", line 88, in parse
    action, arg = get_action(token)
  File "/home/vagrant/venv/terraform/lib/python3.8/site-packages/lark/parsers/lalr_parser.py", line 66, in get_action
    raise UnexpectedToken(token, expected, state=state, puppet=puppet)
lark.exceptions.UnexpectedToken: Unexpected token Token('__ANON_3', 'output') at line 302, column 3.
Expected one of:
        * __ANON_0
        * __ANON_2
        * __ANON_1

<IP_ADDRESS> - - [25/Sep/2022 07:48:26] "GET /graph.svg HTTP/1.1" 500 -

Do you happen to know if you're using HCL or HCL2?
I've updated one of the dependencies, python-hcl2 to see if it will help.
The problem occurred when parsing HCL files, and it's not the first time someone told me about it

another finding, it will work without any issue if there are no modules imported. @Ianyliu

Hi @xiaopeng163,
Could you try testing out this version of Blast-Radius (fork)?
It should resolve the problems for remote modules
I integrated changes from Jrc356/blast-radius
The changes are on my dev branch, so to install it you would have to put:
pip install -U git+https://github.com/Ianyliu/blast-radius-fork@dev
If you use Docker and would like me to create a dev or testing image, please let me know as well

@Ianyliu seems working with the remote module, for example
module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = ">= 3.14.2"

  name           = var.aws_vpc_name
  cidr           = var.aws_vpc_cidr
  azs            = var.aws_availability_zones
  public_subnets = [cidrsubnet(var.aws_vpc_cidr, 8, 1)]

  create_igw = true

  manage_default_security_group  = true
  default_security_group_egress  = var.aws_sg_egress_rules
  default_security_group_ingress = var.aws_sg_ingress_rules
}

but to be honest, this is not readable, seems it shows all resources defined in the module?
btw, the code still doesn't work for local modules

@xiaopeng163
Was there a problem before with local modules? I assumed that your problem was just due to the modules being remote.
If the source is from another local file, I doubt it will work right now. But perhaps that is something I should work on.
Could you describe the problem in more detail? I don't really have any experience using local modules in Terraform.


compatibility with twbs 4

Hey,
Am trying to integrate twbs alpha 4 with the typeahead.
Any settings I need to change to make it work with twbs 4?
Appreciate any help.
It gives me:
meteor add<EMAIL_ADDRESS>=> Errors while adding packages:
While selecting package versions:
error: Conflict: Constraint<EMAIL_ADDRESS>is not satisfied by twbs:bootstrap 4.0.0-alpha2.
Constraints on package "twbs:bootstrap":

<EMAIL_ADDRESS><- top level
<EMAIL_ADDRESS><- sergeyt:typeahead 0.11.1_8

As a workaround, I have downloaded the source and put it in a local package.
And I updated the package.js to
<EMAIL_ADDRESS>'client', {weak: true});
And also updated the css to [typeaheadjs.css] (https://github.com/bassjobsen/typeahead.js-bootstrap4-css)
This seems to work with twbs 4.
I am wondering whats the right way to achieve the same.


TypeError when send post request with multiparty/form-data

when i use form-data to send a post request, it has this problem.
how can i fix it?
thanks!


South hemisphere reminder

Updates #1 , creating a reminder when south hemisphere is set using -S.
I created a mutually exclusive argument group with -S and -s, so that only one can be provided.
It changed the way to set the default value for the variable.
# pyphoon -S north -l en -n 6 2020-01-01
       -..
         .`.     New Moon +
         O ).    5 21:44:28
        ) ||.    First Quarter -
        ..'/     2  1:46:52
       -''

# pyphoon -S south -l en -n 6 2020-01-01
    ,,-
  /,``   New Moon +
 `|| (   5 21:44:28
 `( O    First Quarter -
  `.`    2  1:46:52
    ``-  (Southern Hemisphere)

# pyphoon -s south -l en -n 6 2020-01-01
    ,,-
  /,``   New Moon +
 `|| (   5 21:44:28
 `( O    First Quarter -
  `.`    2  1:46:52
    ``-

https://github.com/chubin/wttr.in/issues/247 has some considerations from @iago-lito that should be addressed/answered before merging.

Fixes #1 and #27
I'm sorry for merging both in a single PR.

Reverting back to draft to include @iago-lito comments in https://github.com/chubin/wttr.in/issues/247.

I think that after all fixes we should release the next (0.2) version, and ask @lucasrodes
to update the PyPI package. After that we can mention direct pip installation with pip install pyphoon in README.md

Hi @chubin ,
If I understand you correctly, you'd like to have the name "pyphoon" available on PyPi, right? I can check whether this is possible on PyPi. On my side, it is fine, as the library is actually not being versioned (hence the repository should suffice).

@lucasrodes , I think you already are the maintainer there, or is that another lucasrodes ?
Anyway, I think @chubin wants to finish all the hemisphere translations before publishing it there. Is that correct? We are still missing bg, ca, cs, da, et, el, es, fi, hr, ia, it, ja, nl, pl, ro, sk and sr.

That's me, for another project I was working on (typhoon related). Flexible on giving the pypi domain to you!

@lucasrodes Lucas, feel free to contribute :-)
If you don't need the pyphoon package on pypi it would be very cool, if we could have it.
As we chosen the name for the project (pyphoon) 5 years ago
this word was not taken, and it was a good pun for phoon written in Pascal in 1979
that was reimplemented in Python now.
Fist time I saw this name on PyPI I even thought that someone packaged our pyphonn
and uploaded it. I didn't realize at first that it is completely unrelated project that
have the same name (because I remember how I checked if the name is not already used).
Of course it would be very good, if you could give the package name to us.
Thank you very much for it

@chubin Sorry I did not answer before, please feel free to reach me out so I can transfer this to whoever is to be in charge!
Have a nice day :)!

@lucasrodes I've contacted you via email


Add OpenShift Online single sign-on SAML container

This container runs a preconfigured version of simpleSAMLphp with PHP and Apache HTTPD. This initial revision is geared towards SSO to Amazon AWS accounts that have been configured to use the service provided by this container as a SAML Identity Provider (IdP).

:+1:


cURL backend error on l5 branch

Hello,
Not sure if I’m doing it wrong, but following the readme lead me to having the following error on php artisan deploy
+ --------------- § --------------- +
» Server: developmentUnable to check if file/folder exists using cURL backend. This functionality has not yet been implemented.

Is it due to a missing dep ? A misconfiguration ? Or is it simply that my site is unreachable ?

I'm using Bridge for the backends and it hasn't implemented cURL fully yet. Actually Maneuver it's supposed to be used with FTP or SFTP. Anyway, if you want, you can open an issue at Bridge requesting to fully implement cURL.

@rigelk you need to install php_ssh2 extension and everything will work fine. cURL is meant to be a fallback when php_ssh2 is missing.

@fadion do you mean Maneuver actually doesn’t support FTP and sFTP as advertised ?
@banago i installed php_ssh2 yet still have the same error. php -m | grep ssh2 shows ssh2 activated and so does the phpinfo().

I meant just the opposite, that it supports FTP and sFTP. @banago is the creator of Bridge and will provide better help than me.

@rigelk seems something is wrong on your system. FTP does not fall back to cURL as far as I can remember, only SFTP and SSH do. What system are you on?

@banago i’m running on archlinux. I installed php-ssh2 through AUR via the php-ssh-git package. I then activated extension=ssh2.so and did a systemctl restart httpd to restart the php dæmon.

@rigelk seems you are set. However, my last guess now would be you check extension=ssh2.so is set for both php.ini, webserver and CLI. At least in Ubuntu I had to update both of them. To test you really have php_ssh2 installed, please run this in a little file and see what you get:
	if(!function_exists('ssh2_connect')) {
		return false;
	}

If you get false, you are not done with configuration yet.

@banago i did a php -r 'echo function_exists("ssh2_connect") ? "ssh2 supported\n" : "ssh2 unsupported\n";'. Seems like the function exists.

I don't have any clues. This is not an expected behavior if php_ssh2 is active and works well. Sorry I cannot help further.

Thanks anyway ! I will try investigating further on the php_ssh2 side

@rigelk any update on this issue?


Options for GPU Sharing between Containers Running on a Workstation

Describe the support request
Hello, I'm trying to understand options that would allow multiple containers to share a single GPU.
I see that K8s device plugins in general are not meant to allow a device to be shared between containers.
I also see from the GPU plugin docs in this repo that there is a sharedDevNum that can be used for sharing a GPU, but I infer this is partitioning the resources on the GPU so each container is only allocated a fraction of the GPU's resources. Is that correct?
My use case is a tool called data-science-stack that is being built to automate the deployment/management of GPU-enabled containers for quick AIML experimentation on a user's laptop or workstation. In this scenario we'd prefer the containers have the ability to each have access to the full GPU resources - much like you'd expect for applications running directly on the host. Is this possible?
System (please complete the following information if applicable):

OS version: Ubuntu 22.04
Kernel version: Linux 5.15 (HWE kernel for some newer devices)
Device plugins version: v0.29.0 and v0.30.0 are the versions I've worked with
Hardware info: iGPU and dGPU

sharedDevNum is mostly intended to be used when either:

User running the workloads manually takes care of not oversubscribing the GPUs (e.g. in test cluster),
Cluster runs only GPU workload(s) which can share GPU up to sharedDevNum (e.g. cluster dedicated to running a single GPU workload), or
Pod specs indicate how much resources each container uses from GPU, and cluster is running GAS [1] that shares GPUs based on the specified resource consumption

There's no enforcement of limiting workload to the specified amount, those values are used only for scheduling / device selection

[1] GPU Aware Scheduling: https://github.com/intel/platform-aware-scheduling/tree/master/gpu-aware-scheduling

In this scenario we'd prefer the containers have the ability to each have access to the full GPU resources

If each container (in cluster) is supposed to have exclusive access to the GPU device, use 1 for sharedDevNum.

If each container (in cluster) is supposed to have exclusive access to the GPU device, use 1 for sharedDevNum.

But this does not allow the GPU to be shared between containers, correct?
Maybe a bit more context about the use case would help. We are building an application that simplifies the deployment of GPU-enabled containers (for example, using Intel's ITEX and IPEX images). This is not meant for deployments across a clusters of nodes. There is just a single node (user's laptop or workstation).
Each container runs a Jupyter Notebook server. Ideally, a user could be on a workstation with a single GPU and multiple containers running, with each provided full access to the GPU. Notebook workloads are typically very bursty, so container A may run a notebook cell that is very GPU intensive while container B is idle. In cases where both containers are simultaneously requesting GPU acceleration, ideally that would be handled the same way (or close to the same way) as two applications running directly on the host OS requesting GPU resources.

@frenchwr sharedDevNum is the option you would most likely want. Any container requesting the gpu.intel.com/i915 resource with sharedDevNum > 1 will get "unlimited" access to the GPU. "unlimited" in a sense that there's no hard partitioning etc. in works. Obviously if two containers try to run on the same GPU, they will fight for the same resources (execution time, memory).

@tkatila Thanks for clarifying! I agree this sounds like the way to go. A few more follow up questions:

Am I right that running with resource management disabled (default behavior) would make the most sense for our use case?
Is there any performance impact on the number you choose for sharedDevNum? For example, using 2 vs. 10 vs. 100. I guess not if there is no partitioning of the GPU resources but just want to confirm. Is there any reason not to choose an arbitrarily large number if our goal is to expose the full GPU to each container?

Am I right that running with resource management disabled (default behavior) would make the most sense for our use case?

Yes, that's correct, keep it disabled. To enable resource management you would also need another k8s component (GPU Aware Scheduling, or GAS). It's setup requires some hassle and I don't see any benefit from it in your case.

Is there any performance impact on the number you choose for sharedDevNum? For example, using 2 vs. 10 vs. 100. I guess not if there is no partitioning of the GPU resources but just want to confirm. Is there any reason not to choose an arbitrarily large number if our goal is to expose the full GPU to each container?

I don't think we have any guide for selecting the number, but something between 10 and 100 would be fine. The downside with an extremely large number is that it might incur some extra CPU and network bandwidth utilization. GPU plugin will detect the number of GPUs, multiply the number with the sharedDevNum and then create duplicate resources for the node. Carrying all those resources in resource registration and during scheduling will have some minor effect, but if the sharedDevNum is within a sensible range, the effect shouldn't be noticeable.


Possibility on cross-source point cloud registration between pedestrain and vehicle

Hi author, thanks for opening such a great work. I am curious about whether G3Reg can be used for cross-source point cloud registration such as pedestrain and vehicle, since the point clouds from pedestrains are not stationary as much as those from vehicles. Can author share more ideas about this scenario? Thanks a lot!

Hi, do you mean registering pedestrain and vehicle or scene point clouds containing pedestrain and vehicles? If the former, G3Reg can't since they are different objects. If the last, it can because the ped and vehicles will be regarded as outliers if they don't obey rigid transformation constraints.

I think you can have a try. There are some tricks that may improve performance. For example, use IMU to align the frame with the gravity; modify the hyperparameters of TRAVEL or DCVC (the segmentation method we used in this method)

Okay I plan to try it later. BTW have you ever  handled some scenarios like this before?

No, but I think it's not a very challenging issue, especially when you use the same LiDAR type. BTW, which type do you use?

No, but I think it's not a very challenging issue, especially when you use the same LiDAR type. BTW, which type do you use?

I use ouster LiDAR, but I found it unstable to use scan context and icp for registration in fact, the reason lies on that the changing lidar fov of helmet influences parameters of above methods seriously and it is hard to find one suitable case.

OK, besides my method, try this one as well. https://github.com/sparolab/solid

Great!. I will test both your work and this work. Thanks for your advice!


UnicodeEncodeError: 'ascii' codec can't encode character '\u25cf' in position 9: ordinal not in range(128)

When using Bullet I always get this error.

  File "bin/extract_genotypes", line 28, in <module>
    result = cli.launch()
  File "/home/endrebak/anaconda3/lib/python3.6/site-packages/bullet/client.py", line 670, in launch
    self.result.append((ui.prompt, ui.launch()))
  File "/home/endrebak/anaconda3/lib/python3.6/site-packages/bullet/client.py", line 213, in launch
    self.renderBullets()
  File "/home/endrebak/anaconda3/lib/python3.6/site-packages/bullet/client.py", line 157, in renderBullets
    self.printBullet(i)
  File "/home/endrebak/anaconda3/lib/python3.6/site-packages/bullet/client.py", line 165, in printBullet
    utils.cprint("{}".format(self.bullet) + " " * self.margin, self.bullet_color, back_color, end = '')
  File "/home/endrebak/anaconda3/lib/python3.6/site-packages/bullet/utils.py", line 119, in cprint
    forceWrite(on + color + s + colors.RESET, end = end)
  File "/home/endrebak/anaconda3/lib/python3.6/site-packages/bullet/utils.py", line 101, in forceWrite
    sys.stdout.write(s + end)
UnicodeEncodeError: 'ascii' codec can't encode character '\u25cf' in position 9: ordinal not in range(128)

Does it work if you run it with the environment variable PYTHONIOENCODING=utf8?

Yes! Thanks. But the reason I'm using bullet is so that noobs do not need to use scary command line apps. I do not want people having to rely on setting env variables. Is there a way to fix this in my python code? You can use sys to set the default encoding, but this was a no-no. Perhaps bullet should have a helpful error message that explain why this happens at least?

Definitely, this isn't a proper solution, just a easiest way of tracking down the issue.
It is strange though because utf8 should be the default for Python 3. Do you have a different file encoding specified for your own code?
I think the issue is that unicode ● character (\u25cf) in the source file. If you've copied that into your own code with a different file encoding, that could problems.
Also, what do you get from import sys; print(sys.getdefaultencoding())?


RFC for MergeOperator.

This is currently a draft, and there are a few points that I'd love to get input on.
In my view, the primary use case for the Merge Operator is buffering (essentially an append operation). However, the resulting list can become quite large in cases where there is per-key data skew. None of the existing implementations really address this, as they generally accept it as a fair limitation or memory constraint.
I'm considering extending the API to allow for limiting the size of merged values. In the case of an append operation, if the list grows beyond the defined limit, we could simply "close the batch" and start a new one. This would also improve performance of partial iterations.
Any thoughts?

 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.

👋 I've updated the RFC and incorporated all the discussions. It's now ready for another pass. 🚀


add operators in create-plugin template

Add starter operators to create plugin template

Closing in favour of #29


Rebuild Meshes without Gates

Apparently BSG generates NavMeshes without gates, elevator doors, etc. in place. If they aren't removed before generating the mesh, those objects create breaks in it, and bots will be unable to travel behind them. However, if the NavMesh is generated without those objects, NavMeshObstacles need to be attached to them.
I've confirmed this is happening in the following areas in SPT 3.7.6:

All exfil gates in Labs (i.e. Parking Gate)
All elevator doors in Labs (i.e. Cargo Elevator)
D2 sliding door in Reserve

3.8.0 generation of Labs will have elevator and rolling shutter doors (Garage and hangar) removed
Will be looking at D2 when I get to Reserve.
Will be looking at dynamic blockers as well

Done


Update Database Schema for Custom Back-Halves

Description:
This sub-issue deals specifically with modifying the database schema to accommodate custom back-halves in URL records.
Tasks:

Modify the database schema to include a field for custom back-halves in the URL records.
Ensure that custom back-halves are unique per user to prevent conflicts.

@DhananjayThomble Let me work on this.


Adding EthTransactionValidator::validate_all function

close https://github.com/paradigmxyz/reth/issues/5001

great

Thank you , really happy to participate on all this. Please don't hesitate to open more issue i take as much as i can :) Thank you so much for all your work @mattsse !


Update CSV Schema Documentation to Include New Standard Schema Column for Primary Language

Problem Statement
The standard schema being used for senders who use the CSV Upload feature has been modified to include an optional column to hold the patient's primary language. We need to update the CSV Upload Guide and the accompanying template to account for this new field.
We need to get the exact ReportStream field name from Rick Hood.
Acceptance Criteria

[ ] Add the new column to the "CSV Schema Documentation" page on the website
[ ] Add the new column to the "standard CSV with example data" csv file available for download on the website

Q: is this necessary as this is an optional field??????

[ ] Update website content
[ ] Update the "last updated" date for the guide

@drew-usds @BerniXiongA6 Can you please coordinate with Rick Hood on getting these changes made to coincide with the release of the new standard schema column
cc: @rachelhanster

@loripusey Moving this back into the sprint backlog. Update from Rick from today:
This is still in progress on their end, so no rush from us. Mo is working on changes that Rick will need to test. They'll keep us posted.

Update from @rhood23699 via Slack on 2.18.2022:
"@Berni Xiong I just updated my CSV Std File Format & Notes document at https://cdc.sharepoint.com/:w:/r/teams/USDSatCDC/Shared Documents/PRIME ReportStre[…]cx?d=w0bee04468e4f4736bd604831fb1d75ec&csf=1&web=1&e=TE79q9 .  My changes are highlighted in RED.  I also updated the StandardCSV.csv file at https://cdc.sharepoint.com/:x:/r/teams/USDSatCDC/Shared Documents/PRIME ReportStre[…]sv?d=w90d420991c5744a7a9ca556973422202&csf=1&web=1&e=vXfhK1
Changes are:
Added an optional field to the Patient Data Elements named patient_preferred_language
Added an optional field to the Patient Data Elements named patient_email."
cc: @loripusey @rachelhanster @drew-usds

Updated 2.22.2022: "content updates" have been made via shared working doc. @drew-usds , it's ready to be populated onto RS website. Lmk if you have questions, happy to help!