Empty column values now showing null

Please see screenshot

Any news on this issue?

HI Gabi,
Thankyou for the fix, When will it be available for download. I am doing a
customer demo early next week.. Happy new year btw 😊
From: Gabi<EMAIL_ADDRESS>Sent: Monday, 6 January 2020 10:11 PM
To: Servoy/aggridcomponents<EMAIL_ADDRESS>Cc: wvitpr<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Servoy/aggridcomponents] Empty column values now showing
null (#42)
Closed #42 https://github.com/Servoy/aggridcomponents/issues/42 via
afcd3f8
https://github.com/Servoy/aggridcomponents/commit/afcd3f895c1a6b2a9ac4714d4c8ed8d0d167b810
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Servoy/aggridcomponents/issues/42?email_source=notifications&email_token=AAAUP7VAR62TFUU7NF4KWK3Q4MGUDA5CNFSM4J5TN7A2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVZDMVBA#event-2923874948,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAUP7XK536TNQ5L6AYSQTTQ4MGUDANCNFSM4J5TN7AQ
.

Hi wvitpr,
It may be a little time before we make a new binary release, but you can always test the latest by running from source, either by pulling from git or downloading the zip.
Are you familiar with how to do that ?
Best,
Sean

Hi Sean,
Oh ok, I'm happy to download, but not sure what to do with it. What folder
does it go into?
On Thu., 9 Jan. 2020, 2:28 am Sean<EMAIL_ADDRESS>wrote:

Hi wvitpr,
It may be a little time before we make a new binary release, but you can
always test the latest by running from source, either by pulling from git
or downloading the zip.
Are you familiar with how to do that ?
Best,
Sean
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Servoy/aggridcomponents/issues/42?email_source=notifications&email_token=AAAUP7SA3UNJ33U5OR4UWILQ4XWIZA5CNFSM4J5TN7A2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIM553A#issuecomment-572120812,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAUP7XA756PMHKKC2VBP73Q4XWIZANCNFSM4J5TN7AQ
.

Hi wvitpr,
I missed your last message while traveling and didn't see it because the issue is marked closed. Sorry for that. To run (any) package from source,

click green download button > download as zip.
Extract the package folder "aggrid".
From eclipse, file >import >general > projects into workspace and select your "aggrid" folder. Then you should see it in the solution explorer.
Right-click Servoy Packages > Add Servoy Package Project Reference and select the project


feat(syncthing): ✨ add ignoring files management

This PR introduces a new feature in the plugin's settings for managing ignored files by Syncthing in the currently opened vault.
See #81 for reference.
Closes #81 .

#include is a thing
just fyi.


feature: Add concent

Add concent implement

Would you try removing - 10 in .travis.yml?

The master branch is updated.

I have merged the latest code of master branch

Hey, dude,  everything is ok now, can you accept my PR?

We have some notes in this thread: https://github.com/dai-shi/will-this-react-global-state-work-in-concurrent-mode/pull/36#issuecomment-829644387
What is your result like?
(Also, I need to follow up latest React experimental build, which turns out breaking change, so until it's fixed, no PRs should be merged to avoid further work on me. I feel like dropping some existing libs before working on it.)

Let me close this as we dropped many in #42. Feel free to work on your fork and share the result in https://github.com/dai-shi/lets-compare-global-state-with-react-hooks


Secret sync rebase

This deals with https://github.com/application-stacks/runtime-component-operator/issues/350
After this change, the pods will be deleted/recreated automatically if a pullSecret is added to the CR
Also, the pull secret will be deleted from the service account if it is deleted from the CR.

I'm closing pr #369 in favour of this as I've tweaked the commits a fair bit.
All the review comments from that PR have been addressed


[wip] update documentation

update documentation to reflect changes to pipeline current with the latest release v0.2.4

update to include tumor-only v0.2.5


Serialized columns

Hey Mat. I've spiked out an idea for what serialisation could look like. This isn't ready to merge, I just wanted to check if you were ok with the general direction before fleshing it out and speccing it in detail.
I initially tried putting the logic for serialization in cast, but that doesn't work since there's no distinction between casts on the way into the DB and on the way out. So I arrived at this idea, which is using an intermediate object that has a serializing to_s for insertion, and checking for the presence of that type in the attribute reader to ensure that the original object is what gets passed back.
I think it's likely the cleanest solution, but I'm not sure where you'd prefer the new classes live. I've given it a new second-level namespace, but perhaps you'd prefer it live in a sub-module inside of Record?
Anyway, let me know. If you're happy with the direction I'll wrap up the specs and add in messagepack to close off #33.

I am closing this PR due to inactivity. If anyone wants to work on this feel free to create a new PR.


Data Splitting, page-data by pages and namespaces.

I use namespaces and have a big page-data file size because of each page continents the same source for each localization. f.e:
page-data for page1 and page2 the same:
en: {page1: {...}, page2: {...}, ...}
I use
const { t } = useI18next('page1');
Is it possible to separate, to fetch data that will be relative to the exact page only?
page-data/en/page1/page-data.json
en: {page1: {...}}
Thanks a lot :)

Not for now, but this is on the roadmap, I'm sorry but I don't have time lately to work on this. PR's are always welcome.

Fixed by #43


Update favicon

Describe the bug
Currently on the favicon of the app it still using the vite favicon
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

I would like to help

Hi @abhishekpaturkar sure, I will assign this to you.

The Husky test case are failing. Even without any file change

Hey @frank-mendez can you provide me the icon?

The Husky test case are failing. Even without any file change

Hi @abhishekpaturkar can you send me the details of the failed husky. I will check it out. Thanks

Hey @frank-mendez can you provide me the icon? I have fixed it. here's the screenshot:

We can use this icon

I am able to change the icon but the test cases are failing

Mhmmm that is weird. Can you try to do a PR? The test is fine on my end.

I changes the icon


CBench: reduce switch or increase core error

Copying an email to this Issues so I can answer it publicly (docs for others, etc).
I cloned WCBench repo from git hub. Ran command vagrant up. It gave error saying box chef/fedora-20 is not found. So I added the box by
vagrant box add chef/fedora-20 https://atlas.hashicorp.com/chef/boxes/fedora-20
vagrant up then started a VM. sshed into the VM.
Ran ./wcbench.sh -ci
./wcbench -o
./wcbench -r
This is giving following error
Collecting pre-test stats
Running CBench against ODL on localhost:6633
Collecting post-test stats
Collecting time-irrelevant stats
WARNING: Error occurred: Failed to parse CBench average
This is an issue with CBench or ODL, not WCBench.
May need to reduce NUM_SWITCHES or allocate more CPU cores
See: /home/vagrant/cbench.log
See: https://github.com/dfarrell07/wcbench/issues/16
I have reduced NUM_SWITCHES to 6 in wcbench.sh file and no. of cores increased to8 in Vagrant up. Yet I am getting this error.

This is giving following error

You likely just need to kill/restart the ODL process. Once it gets into that error state it needs a restart before it starts working again. High values for NUM_SWITCHES and few CPU cores seem to make it happen more frequently.

I have reduced NUM_SWITCHES to 6 in wcbench.sh file and no. of cores increased to8 in Vagrant up. Yet I am getting this error.

That should be more than sufficient.
I wasn't able to replicate the failure on my system, for what it's worth.

Solved...Restarted ODL, reinstalled CBench. No. of switches = 16 and no. of cores = 4 works fine.

Solved...Restarted ODL, reinstalled CBench. No. of switches = 16 and no. of cores = 4 works fine.

Glad it worked out well. :)
I added the restart ODL tip to the relevant error output in a6b871bd02fa8722ce89f7de5ac042a28e401bd5.


[Improvement] Support local sort in the Spark's insertRepartitionBeforeWrite optimization rule

Code of Conduct

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

Search before asking

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

What would you like to be improved?
Currently, the Spark's insertRepartitionBeforeWrite optimization rule will be skipped when logical plans are Sort regardless of whether they are local or not: https://github.com/apache/kyuubi/blob/fa9e6be/extensions/spark/kyuubi-extension-spark-common/src/main/scala/org/apache/kyuubi/sql/RepartitionBeforeWritingBase.scala#L133.
It makes sense for global sort, since inserting repartition after the sort changes the semantics of the original plans and doing before that only introduces an additional shuffle.
However, inserting repartition before local sort will help to sort rebalanced partitions even if locally, and it follows the behaviors of queries that explicitly use both REPARTITION|REBALANCE and SORT BY.
How should we improve?
This issue proposes to support local sort in the Spark's insertRepartitionBeforeWrite optimization rule by inserting repartition before the sort.
Are you willing to submit PR?

[X] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
[ ] No. I cannot submit a PR at this time.

cc @ulysses-you @cxzl25 @cfmcgrady what do you think about this idea?


Don't call LifeCycleTracking for build < 14

Fix for #46
Shouldn't call any methods from LifeCycleTracking on build level < 14.

Hi @ranterle, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


Can't update wsl2 Error : 0x80041002

按照升级指引操作，转换的时候报0x80041002错误，无法转换。
OS Version:10.0.19041 N/A Build 19041 / 19041.264
subsystem：Ubuntu-20.04
Dodotry C:..\system32 ❯❯❯ wsl --set-version Ubuntu-20.04 2
正在进行转换，这可能需要几分钟时间...
有关与 WSL 2 的主要区别的信息，请访问 https://aka.ms/wsl2
错误: 0x80041002

you can only Set your distribution version to WSL 1 or WSL 2
wsl --set-version <distribution name> <versionNumber>
like wsl --set-version Ubuntu 2
version 1 and 2 is only available.
visit here for more details

you can't update the version of your ubuntu using wsl command.
update it from Microsoft Store.

you can only Set your distribution version to WSL 1 or WSL 2
wsl --set-version <distribution name> <versionNumber>
like wsl --set-version Ubuntu 2
version 1 and 2 is only available.
visit here for more details

you can't update the version of your ubuntu using wsl command.
update it from Microsoft Store.

I tried again, but the conversion still failed.

https://docs.microsoft.com/en-in/windows/wsl/install-win10
See this

https://docs.microsoft.com/en-in/windows/wsl/install-win10
See this

No use.

https://docs.microsoft.com/en-in/windows/wsl/install-win10
See this

wsl --list --online

https://docs.microsoft.com/en-in/windows/wsl/install-win10
See this

No use.

And also according to this web page: https://docs.microsoft.com/en-in/windows/wsl/install-win10 ,
there is no this command: wsl --list --online

the same to you！ help me！！！


codeAt in HumanStandardTokenFactory.sol does not work

This is the function:
    function codeAt(address _addr) internal returns (bytes o_code) {
      assembly {
          // retrieve the size of the code, this needs assembly
          let size := extcodesize(_addr)
          // allocate output byte array - this could also be done without assembly
          // by using o_code = new bytes(size)
          o_code := mload(0x40)
          // new "memory end" including padding
          mstore(0x40, add(o_code, and(add(add(size, 0x20), 0x1f), not(0x1f))))
          // store length in memory
          mstore(o_code, size)
          // actually retrieve the code, this needs assembly
          extcodecopy(_addr, add(o_code, 0x20), 0, size)
      }
    }

Its test fails. I think I have an idea as to why.
Lets enumerate the magic numbers in the line mstore(0x40, add(o_code, and(add(add(size, 0x20), 0x1f), not(0x1f))))
0x40 is the location of the free memory pointer.
0x20 is 32 bytes, which we want to leave free at the beginning of o_code so we can store the buffer size in it (mstore(o_code, size)).
0x1f appears to be a number with which we determine the amount of memory to reserve. Though its exact nature is mysterious to me.
Imagine size is 0. 0x20 + 0x1f is 0x3f. In decimal, the 32 byte size field plus the 31 byte magic number is 63 bytes. The negation of 0x1f (not(0x1f)) is 0xe0 (224). and(0x3f, 0xe0) is 0x20, or 32 bytes, which is what we would expect if we had an empty buffer: a size element storing zero followed by no data.
If the code is one byte in size and we do this math again we get 64 bytes, which also makes sense. A 32 byte size element followed by a 32 byte data element. If the code size is 33 bytes we'll get 96, which also makes sense: a 32 byte size element followed by two 32 byte data elements.
This all works until you get to code sizes greater than 192 bytes (which would correctly give you a 224 byte buffer). If you have a code size of 193 bytes, and(0x100, 0xe0) will give you 0.
0000 0001 0000 0000
0000 0000 1110 0000

And the answers will continue to be funky for all code sizes greater than 192 bytes.
So I think we need to understand where the magic number 0x1f comes from, and how we can make it not a magic number on the basis of the size of the code being examined.

This whole theory is wrong. See: https://consensys.slack.com/archives/C03DVPNTT/p1500487032466679

This is still a bug though! For whatever it's worth, if somebody else's eyes see something, calling the getter for humanStandardByteCode returns undefined. Writing a custom getter to get the length also returns undefined (in JavaScript world).

That's odd. It definitely worked when I tested it originally. Will retest.

Addressed in #58.


doc: mention feature toggling in "Specifying deps"

Basically copies info that was already on another page.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @steveklabnik (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+ 885501e136377c1620f185d91f6a8b9eae234176
Thanks!

:hourglass: Testing commit 885501e with merge e4b4471...

:broken_heart: Test failed - cargo-win-gnu-32

@bors: retry
seems spurious?

:zap: Previous build results for cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64 are reusable. Rebuilding only cargo-win-gnu-32...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: alexcrichton
Pushing e4b4471f9180bc2b94c06710a938147e52fcb9d5 to master...

@bors: retry
On Thu, Sep 1, 2016 at 10:29 AM, bors<EMAIL_ADDRESS>wrote:

💔 Test failed - cargo-win-gnu-32
https://buildbot.rust-lang.org/builders/cargo-win-gnu-32/builds/687
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang/cargo/pull/3065#issuecomment-244151634, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95J3-YDGCf1-yqGzyl4k0yeLG1f_cks5qlwt7gaJpZM4Jy2bk
.


Centos6 ius

Add links to the page and use literalinclude when possible
I have deliberately not used it in the set-up section,
This will be reviewed when we have the "auto generate" script in place.
cc @hflynn

Conflicting PR. Removed from build OMERO-DEV-merge-docs#252. See the console output for more details.
Possible conflicts:

PR #1410 snoopycrimecop 'Update auto-generated documentation'

omero/sysadmins/unix/walkthrough/step05_centos6_py27_ius_nginx.sh

The autogen PR is up to date, is this ready to be reviewed now?

@hflynn: it should be good now that the auto-gen PR has been merged.

Two edits needed to fix this up then we should be good to go @jburel

@hflynn: done

Rebuilding on staging now thanks.

All looks good now thanks.


Integration test for posh-runtime class

This is in continuation with Iox #177 unit test for posh runtime.
Test to be covered
findService
    reoffer service
    mix up of AnyService, AnyInstance, AnyEvent
offerService
    call offer service twice
stopOfferService
    call with invalid serviceDescription like {AnyService, AnyInstance, AnyEvent}
    call with non existing serviceDescription
    call stop offer service twice

@nihalchari this can be closed or?

yes @dkroenke , ill close it

closed by mistake, reopening,

@nihalchari Why was this reopenned ?

@ithier   This is in continuation with https://github.com/eclipse/iceoryx/pull/214  unit test for posh runtime., this issue was closed by mistake, actually 177 should hvae closed


Minor log message clarification

Supplying just a name, rather than a namespace/name, for a cainjector
source reference, results in the generic error message "invalid
certificate name". This condition is detected on its own branch so we
can be more specific.

/assign wallrj

/assign wallrj

/ok-to-test

/ok-to-test

Fails locally too, exact same failures on master and my branch...
Sorry, I only ran the tests that seemed to pertain to my change before submitting, but seems like an upstream error?

Fails locally too, exact same failures on master and my branch...
Sorry, I only ran the tests that seemed to pertain to my change before submitting, but seems like an upstream error?

Fails locally too, exact same failures on master and my branch...
Sorry, I only ran the tests that seemed to pertain to my change before submitting, but seems like an upstream error?

Hi Matt,
Yeah, we've got a couple of issues with our tests at the moment:

Our Venafi test server is offline for maintanence and
There's a problem with our code generation script since the turn of the new year.

We'll try and get those fixed ASAP.

Fails locally too, exact same failures on master and my branch...
Sorry, I only ran the tests that seemed to pertain to my change before submitting, but seems like an upstream error?

Hi Matt,
Yeah, we've got a couple of issues with our tests at the moment:

Our Venafi test server is offline for maintanence and
There's a problem with our code generation script since the turn of the new year.

We'll try and get those fixed ASAP.

No worries, it's hardly a blocking bug 😂 Happy new year!

No worries, it's hardly a blocking bug 😂 Happy new year!

Hey @mt-inside . Those tests are now fixed in trunk,  but you'll need to rebase (I think).
/retest

Hey @mt-inside . Those tests are now fixed in trunk,  but you'll need to rebase (I think).
/retest

Correction. It seems that we test against the result of merging into trunk and test now pass. Thanks!
/lgtm

Correction. It seems that we test against the result of merging into trunk and test now pass. Thanks!
/lgtm

/approve
/kind cleanup

/approve
/kind cleanup


Plugins detective is broken

I believe the plugin detective used to parse the minified ES5 server.js to try to catch how context was transformed, and then would seek for (that).plugins.(something) calls in order to detect calls to not supported plugins. Now that we are bundling and compiling with Babili the Regex doesn't work anymore: https://github.com/opentable/oc/blob/master/src/registry/domain/plugins-detective.js#L7
Maybe we can discuss how to improve that. If we can't, we can rather remove the check and let the registry throw (which would then fire a domain error with a component's 5xx response).

@matteofigus happy to have a look at this issue if you like

ok! thanks

@matteofigus it looks like detective is only used here:
https://github.com/opentable/oc/blob/master/src/registry/routes/helpers/get-component.js#L341
inside the handleError function that is raised only if domain.on('error', handleError):
https://github.com/opentable/oc/blob/master/src/registry/routes/helpers/get-component.js#L365
so the detective logic is only exercised if there is an error; if I do not register a plugin and I try to get a component that uses one, this on error is never raised because there is no error; I've got the feeling that the detective maybe still work but the get-component functionality doesn't behave as expected; or am I missing something?

@matteofigus as #495 has been merged shall we close this issue?

Oh yeah. Thanks!


PLAT-132784: Fix ui-test fail on Drawer

Enact-DCO-1.0-Signed-off-by: Dongsu Won<EMAIL_ADDRESS>Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[x] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added

Need to add delay for Draw UI-Test(Jenkins Fail).

Resolution

Add delay for Draw UI-Test.

Additional Considerations
Links
PLAT-132784
Comments

@dongsuwon Please rebase your branch with develop.

@dongsuwon Please rebase your branch with develop.


City of Columbia MO Police Data

Source Name (such as the name of a police department): City of Columbia MO Police Data
Website URL: https://www.como.gov/police/data-reporting-forms/
Does this source contain police data? (Yes/No):  Yes
What types of data does the source contain? (Use of force, traffic stops, officer demographics, etc.):  Vehicle Stop Data
Any other important details that you found?: No

This data has been added


chore: use new creds for roxbot

Description

#143  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dhaus67 and the rest of your teammates on  Graphite


[release/1.0 backport] Add /proc/asound to masked paths

Backport of https://github.com/containerd/containerd/pull/2846 for the 1.0 branch
While looking through the Moby source code was found /proc/asound to be shared
with containers as read-only.
This can lead to two information leaks.

Leak of media playback status of the host
Steps to reproduce the issue:

Listen to music/Play a YouTube video/Do anything else that involves sound
output
Execute docker run --rm ubuntu:latest bash -c "sleep 7; cat
/proc/asound/card*/pcmp/sub/status | grep state | cut -d ' ' -f2 | grep
RUNNING || echo 'not running'"
See that the containerized process is able to check whether someone on the
host is playing music as it prints RUNNING
Stop the music output
Execute the command again (The sleep is delaying the output because
information regarding playback status isn't propagated instantly)
See that it outputs not running

Describe the results you received:
A containerized process is able to gather information on the playback
status of an audio device governed by the host. Therefore a process of a
container is able to check whether and what kind of user activity is
present on the host system. Also, this may indicate whether a container
runs on a desktop system or a server as media playback rarely happens on
server systems.
The description above is in regard to media playback - when examining
/proc/asound/card*/pcm*c/sub*/status (pcm*c instead of pcm*p) this
can also leak information regarding capturing sound, as in recording
audio or making calls on the host system.

Codecov Report

Merging #2849 into release/1.0 will not change coverage.
The diff coverage is 100%.

@@             Coverage Diff              @@
##           release/1.0    #2849   +/-   ##
============================================
  Coverage        50.95%   50.95%
============================================
  Files               81       81
  Lines             7205     7205
============================================
  Hits              3671     3671
  Misses            2833     2833
  Partials           701      701

Flag
Coverage Δ

#linux
50.95% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

oci/spec_unix.go
98.37% <100%> (ø)
:arrow_up:

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 be280f4...84f3f2a. Read the comment docs.


DI for a job

Dependency injection for a job

In this form DI params are unavailable.
That has been discussed in #59.


Make inline notes compatible with other todo-management plugins

HI,
thanks for the awesome work on this plugin! I have played around with it recently and wanted to discuss an idea.
Could an additional summary type be implemented that creates inline todos as "real" todos (as Joplin understands it) in a separate notebook? That way, other plugins that extend Joplins todo management capabilities (for example joplin-agenda and joplin-note-overview) could be used in conjunction with inline-todos.
This combination of plugins would help me to resemble the todo management system used in emacs org-mode more closely, which would be a huge plus for me.
To clarify, lets assume a notebook "Notebook A" with a note "Note A" that contains a couple inline todos. Instead of listing the inline todos in an overview note, this new summary type would create a notebook called something like "Notebook A_inlineNotes" (or maybe one notebook per assignee in confluence style) that contains a todo-type note for each inline notes. Tags used in the inline notation can just be added to the todo note, as well as due dates.
I would be happy to help this with a pull request if any help is needed.

That's a nice idea! I'm a big fan of adding the ability to play nice with other plugins.
That said, I am a bit hesitant about adding this. I can see this either being really easy to add, or being a nightmare.
If you're quite keen on building this out then I'll happily review a PR from you. I can also answer questions you might have while building it. But I won't help other than that.


General Grievance: Watch sonic-menus.com

General Grievance requests the watch of the watch_keyword sonic-menus\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

sonic-menus\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.


Prioritize Pod error status over START_PREPARE_WORKSPACE

Our Jupyter image in capella-dockerimages has no error handling and never raises FAILURE_PREPARE_WORKSPACE in the logs. Therefore, the sessions hangs in the status Session preparation until session termination. It should display an error message instead.

Issue should be resolved after https://github.com/DSD-DBS/capella-dockerimages/pull/198 is merged

Resolved by https://github.com/DSD-DBS/capella-dockerimages/pull/198


fix: Broken link due to missing character

As per @wallrj's comment here, minor typo fix on the missing h on https.
/CC @wallrj

/CC @wallrj

Thanks Peter.
/lgtm
/approve


Login with Registration user

Description:
As a user, I want to be able to login (IDIR or bceid) with my user setup in registration.
Brief description of the work involved, including any user research context or key decisions.
Acceptance Criteria:
Given
When
Then
Development Checklist:

[ ] Keycloak wired in the server side of nextjs
[ ] Environment variables for KC
[ ] Login buttons work
[ ] Meets the DOD

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[ ] User story is included
[ ] User role and type are identified
[ ] Acceptance criteria are included
[ ] Wireframes are included (if required)
[ ] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

Definition of Done (Note: If any of these points are not applicable, mark N/A)

[ ] Acceptance criteria are tested by the CI pipeline
[ ] UI meets accessibility requirements
[ ] Configuration changes are documented, documentation and designs are updated
[ ] Passes code peer-review
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Ticket is ready to be merged to main branch
[ ] Can be demoed in Sprint Review
[ ] Bugs or future work cards are identified and created
[ ] Reviewed and approved by Product Owner

Notes:

One question: When a user tries to login, and they have never been in the registration app, what do we do?
Options:

create a user record in the registration app
refuse login and redirect to registration app
display a relevant error message with a link to the registration app
...?
@patriciarussellCAS @dleard @andrea-williams @mikevespi

One question: When a user tries to login, and they have never been in the registration app, what do we do? Options:

create a user record in the registration app
refuse login and redirect to registration app
display a relevant error message with a link to the registration app
...?
@patriciarussellCAS @dleard @andrea-williams @mikevespi

in Registration, we have already implemented the logic that as soon as a new user logs in with either IDIR or Business BCeID, a new user record is created. From what I saw in the wireframing party last week, the intention is that our 3 modules will all be asked from the same URL (unless I misunderstood entirely), so there might not be any work required on the reporting side for this... ?

That would be fantastic! So just hook up to the registration API for the user creation?

I will flag that they will not be able to do anything in the reporting app until they have requested and been given access to an operator in the registration app, so displaying a message or redirecting them may be what we want

...yessssss? Or maybe not even that? Assuming that users will be accessing the Reporting module from the same url as the Registration module (i.e., industrialemissions.gov.bc.ca), when they go to that link they'll see this page
https://cas-reg-frontend-dev.apps.silver.devops.gov.bc.ca/home
and so would have to log in anyway before they can access anything else. And as soon as a user submits their login, a new user record is created in our DB (assuming there isn't one already for them in there). From what I understand, we'll eventually be replacing the default page after sign in (that right now is the "Registration Dashboard" page) with a more generic home page for all 3 modules.
In Registration we're using next-auth to store user session data, including user id and their app_role, which determines what features they have access to on the frontend, and is used to audit all API requests the user sends. So we could just insert some logic that external users don't see the Reporting module unless they're assigned to an Operator.

My understanding is similar to Andreas. What we've been communicating to the BA is that this is going to feel like a single app to the end user. That should mean login is handled once by a common page, and while it still needs to be figured out what exactly it will look like, there will be some "homepage" that will provide paths to all of the different modules we're building. I think that if we're looking at a solution that involves logging into each module independently, we'll need to sync on this and communicate it to the business area, as well as work on defining more formal scopes for the individual modules.

Sounds great. We can use the registration landing page and login functions, and only expose login protected pages as part of the reporting app :)

Since we're not sharing login yet, we might want to investigate how to build both frontends and have a login page for each.


Example Code

Hello DarkStorm652,
for beginners, it's very difficult to use your library. Couldn't you add an example code which makes a player join or something like that? It would be very helpful!
Greetings, xTACTIXzZ.

It's actually not difficult at all. Instead of skimming over 5 files expecting to become a master at this, pull the code and read it properly.

It is poorly documented :( with so many issues to work out, it is hard to find time to write a wiki or provide docs


Data management folder restructure

Changes proposed in this PR:

According to recent agreements, data folder shall be divided into more granular pieces (table, io, graph, etc.). At now, only table component exists, so data renamed to table
table.hpp divided onto common.hpp (table interface) and homogen.hpp (homogen_table class)
accessor.hpp divided onto row_accessor.hpp and column_accessor.hpp
array.hpp moved to the root folder
type traits for tables moved from utils to table folder and named like the entities which they belong to (common_type_traits, homogen_type_traits)

:hourglass: files table_metadata and table_builder were not changed because are planned to change in next PRs

/intel-ci: run

/intelci: run


Remove duplicate "collection" from example

While technically valid JSON, having two "collection" keys generally means the first value is ignored, and is likely to cause confusion.
Related Issue(s): #
No related issue, figured a PR would be better than a issue
Proposed Changes:

Removes a duplicate "collection" key from a example

PR Checklist:

[x] This PR is made against the dev branch (all proposed changes except releases should be against dev, not master).
[x] This PR has no breaking changes.

SInce this was a minor change, I have not done the following:

[ ] I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.
[ ] This PR affects the STAC API spec, and I have opened issue/PR #XXX to track the change.

Thanks for the contribution!


Fixes #143, close-bilkent to cose-bilkent; fixes #119, added fcose layout

About

Fixing an issue.

Description of changes
Fixes #143, replaced close-bilkent with cose-bilkent
Pre-Merge checklist

[x ] The project was correctly built with npm run build:all.
[ x] If there was any conflict, it was solved correctly.
[ ] All changes were documented in CHANGELOG.md.
[ ] All tests on CircleCI have passed.
[ ] All Percy visual changes have been approved.
[ ] Two people have :dancer:'d the pull request. You can be one of these people if you are a Dash Cytoscape core contributor.

Reference Issues
Closes #143

@alexcjohnson Looking at the diff here, really feel like we should do something like the dev branch and main branch in this repository - just like we are doing with dash-core-components...

Fixes #165


Add hyperlink to issue guidelines

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@daveOrleans Thanks for your contribution. We welcome future contributions from you.


cl-jupyter and jupyterhub

Any idea how to run cl-jupyter with https://jupyterhub.readthedocs.io/en/latest/ ?

Here's a running example https://lisp.style/hub/login . It uses GitHub authentication and Docker. If someone else could provide a more generic setup (mine is for Kenzo), that's be very helpful.


Adopt edit sessions in github.dev/vscode.dev Continue Working On entrypoint

Testing microsoft/vscode#156211
To me, it seems like both of these are trying to do the same thing. If I go through the "Continue working on" flow, I see it creates an edit session. So as a user, it's really confusing how one is different than the other or whether these should all be combined.
FWIW, the "Continue working on" feels more natural than "Edit Sessions" from a concept perspective and might be easier for users to understand. For edit sessions, there's a lot of new terminology and paradigm from "storing" to "managing" and how that model compares to git stashes. I could see these two merging to become "Sessions" more generically since this is all happening in the editor and integrating them better like showing the sessions view when "Continuing..."

@joyceerhl I've been was using "continue working on" as the command for using edit sessions, are there any verification steps needed beyond that?

@connor4312 that's all the verification required :)


Add new K.TypeAlias element

Type aliases are currently modelled as variable declarations with a special typealias modifier on them. While this works syntactically, it doesn't make much sense from a model perspective, as a typealias doesn't really introduce a new variable and as such would end up confusing a lot of existing recipes. Instead, there should be a dedicated K.TypeAlias model element for this.

Please also see https://github.com/openrewrite/rewrite-kotlin/pull/549/commits/12f31a05ab935d5e95b89c905f60d7333d253f23 where I removed some legacy handling of typealias. It looks like we at some point had a dedicated typealias annotation instead.

PR is pending a decision on how to represent the whitespace, comments, and annotations before the keyword typealias.


iterate over the Item values rather than the Items string keys

Related Issue(s):

n/a

Description:
This is also a bug in SQLAlchemy backend, an issue has been filed.
The problem is that the Items iter implementation iterates over the keys of the items (dict[string,Item]) rather than the values, so it gets a string (which is the id) instead of the Item itself. This iterates over the Items instead.
PR Checklist:

[X] Code is formatted and linted (run pre-commit run --all-files)
[X] Tests pass (run make test)
[X] Documentation has been updated to reflect changes, if applicable, and docs build successfully (run make docs)
[X] Changes are added to the CHANGELOG, if applicable

@philvarner Should we just implement the /items endpoint being able to process a feature collection? We could then add the code to stac-fastapi too potentially?

I think we want to wait for it to be done in core stac-fastapi, and then update our impl as appropriate to however the api changes there.


AttributeError When Running QCGPU Examples

Sorry for the double issue, GitHub had a bit of a fit!
When attempting to run any of the examples, I get the following attribute error:
module 'qcgpu.backend' has no attribute  'create_context'
This happens when I call QCGPUProvider(), which calls create_context() on line 17 of init.py. I'm running Python 3.6.6 on a Linux machine, with Qiskit 0.6.1, Qiskit Aqua 0.3.0, pyopencl version 2018.2.1, and version 0.0.1 of this library.
Any idea what the issue could be?

Hello! I actually know exactly what that is - When I published the code to Pypi yesterday, I forgot to push the core QCGPU package also, so It calls a function that's not there yet.
I will try to fix it in the next few minutes

Will you try that again? Upgrade the 'qcgpu' package to 0.0.5.

That did it! Thank you for the quick fix.


Yagegemu patients is not showing on the daily graph

sorry it is there. just hidden behind the other dots i think

but it is worth sanity checking the running sums and numbers as people come in and go out. how that is reflected on the dashboard

it is fixed in the next update which will be pushed late in the afternoon or in the evening.

It is now fixed!


@gnilekaw: Revert typeahead to 0.10

It's unfortunate that typeahead is sprinkled so much throughout Force because the change from 0.10 to 0.11 is widely breaking in subtle ways. Typeahead hasn't been touched in almost a year either, so I don't think it's worth trying to keep it up to date if the author's aren't.
This reverts typeahead to the 0.10.x version to fix the issue you reported in slack. I think looking to a more React heavy future we can replace typeahead with a saner component or even just skip the third party all-together (it can be surprising how easy it is to implement simple UI patterns in React such as autocomplete and modals). I'll also work on an integration test in the meantime.

Thanks, @craigspaeth for the quick fix! 🙇

Awesome! Merging.


Challange.organizerIds does not accept empty array

The Challenge schemas has several {entity}Ids array that are required but my understanding of the OpenAPI object property required was that the array could have been empty, which @rrchai reported is not the case.
Tasks

[ ] Reproduce the issue reported by @rrchai
[ ] Fix the issue, possibly by updating the schemas.

Reproducing error
I tried posting the following Person object (easier than Challenge object):
{
  "email"<EMAIL_ADDRESS>  "firstName": "John",
  "lastName": "Smith",
  "organizationIds": []
}

The error (500) is the following:
{
  "detail": "ValidationError (Person:60d4f1c4b42cc85f11454004) (Field is required and cannot be empty: ['organizationIds'])",
  "status": 500,
  "title": "Internal error"
}

Solution
The issue comes from using the option required for ListField mongo objects. The only solution I found was to remove required from the mongo object (see above PR). Using the Swagger UI, I can confirm that the input and output specified now work as expected. Practically, an empty array is correctly saved in MongoDB and querying this object returns a JSON object that has Person.organizationIds = [], for example. Yet the Person creation request would fail if the property organizationIds is not specified, which is in compliance with the OpenAPI spec.
@rrchai Take home lesson: Flask takes care validating the input and output objects. Additional validation can be done at the level of the Mongo models using required. In Mongo required has the same behavior than required in OpenAPI for non-array properties. For array properties, the Mongo option required means that the array specified can not be empty.


get old refinery working.

old repo: https://github.com/dynamic-modeller/old-type-refinery

Hi @wildone
How could we clone "old-type-refinery" repo, as it's not accessible to us?


using numba with "from collections import Counter"

Hi, could you help me please to run this function using numba .
`from collections import Counter
@njit(nopython=True)
def TE(C):
pxxy = Counter(literal_unroll(map(tuple,C)))
pxy = Counter(map(tuple,C[:,1:]))
pxx = Counter(map(tuple,C[:,:2]))
px = Counter(map(tuple,C[:,1:2]))
    A = pxxy.keys()

    tot = float(sum(pxxy.values()))

    return sum((pxxy[k]/tot)* np.log2( (pxxy[k]/float(pxy[k[1:]])) / (pxx[k[:2]]/float(px[k[1:2]])) ) for k in A)`

It generates an error C is a matrix of 0 and 1 with 3 columns

Thanks for the question - a few remarks on the above:

You can write blocks of code in Markdown using three backticks on a line, instead of just the one - see e.g. https://github.github.com/gfm/#example-89
Using nopython=True with njit is redundant - njit is just a shorthand for @jit(nopython=True)
It would be a bit easier to look into if the code provided were executable to demonstrate the issue, with some explanation of the expected outcome from running it.

I've had a guess at what the executable reproducer might be:
from numba import njit, literal_unroll
from collections import Counter
import numpy as np

@njit
def TE(C):
    pxxy = Counter(literal_unroll(map(tuple, C)))
    pxy = Counter(map(tuple, C[:, 1:]))
    pxx = Counter(map(tuple, C[:, :2]))
    px = Counter(map(tuple, C[:, 1:2]))

    A = pxxy.keys()

    tot = float(sum(pxxy.values()))

    return sum(
        (pxxy[k] / tot) * np.log2((pxxy[k] / float(pxy[k[1:]])) /
                                  (pxx[k[:2]] / float(px[k[1:2]])))
        for k in A
    )

c = np.random.rand(5,5,5)

TE(c)

There are some difficulties with using this with Numba directly:

The generator expression ((... for k in A) is not supported in nopython mode - this would need to be converted to a loop. See the list of supported Python language features: http://numba.pydata.org/numba-doc/dev/reference/pysupported.html?highlight=generator expression#language
Collections.counter is probably also not supported in nopython mode. Standard library features that are supported are listed here: http://numba.pydata.org/numba-doc/dev/reference/pysupported.html?highlight=generator expression#standard-library-modules

Closing this issue as it seems to be resolved. If this is not the case please re-open with a comment about any item that appears to be unresolved. Many thanks.


Merge upstream

PULL REQUEST
Overview
Merge in upstream changes. This includes v3.0 changes which should make parallel uploads work with Node. See https://github.com/SkynetLabs/nodejs-skynet/pull/195#issuecomment-1221600861.
Someone should publish @skynetlabs/tus-js-client v3.0.0 after this.

@kwypchlo

@MSevey Looks like an issue with browserstack secrets. I think I sent you mine once, are they still set in the repo? Or can paste some secrets from your browserstack account or just ignore the failed run.


AadHttpClient CORS error "origin 'null' is not allowed"

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Expected or Desired Behavior
Calling a AAD secured API running on localhost should work with AadHttpClient
Observed Behavior
When calling my remote endpoint (https://localhost:44305/api/secured) I receive the following CORS error:

Failed to load https://login.microsoftonline.com/4bd4ff55-e72f-42e5-9cb5-f43aff2b2431/oauth2/authorize?client_id=412a1971-dada-434e-b230-c6f46ade0003&response_mode=form_post&response_type=code+id_token&scope=openid+profile&state=OpenIdConnect.AuthenticationProperties%3DEo7JqExfzJJaqy8G2wOZOr9QOWtV7gNbgVVEc6ah9ex9N4pvlkiWjNdZhgbNByJbk_SzcUkpPAECyLJkCLZ6Q1r-EF_D6qdvhUv7FjKgVFXsgKtZxcm_jhvZPd7dwRj4Yh-8NIv-bOPdJm6BtJvBVA&nonce=636724208806787290.ZTJiNjU5ZmYtNGMxNS00MDE1LWIyOTEtNjc0MDBlMTUyZjM0MzdjMjUyNTMtNzU1OS00ZmU1LTk4ZjItNGFlZWU3NzgxZDI0&x-client-SKU=ID_NET&x-client-ver=1.0.40306.1554: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'null' is therefore not allowed access. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

I followed the steps outlined at https://docs.microsoft.com/en-us/sharepoint/dev/spfx/use-aadhttpclient-enterpriseapi#consume-the-enterprise-api-secured-with-azure-ad-from-the-sharepoint-framework, except I created a new AAD application manually, and use that as authentication app for my web api running on localhost.
(I don't have an Azure subscription on my SP tenant subscription, so I couldn't create a Function app there to fully test the tutorial)
Also, when opening the https://login.microsoftonlint.com/4bd4....... url in a new tab, I'm successfully redirected to my API, without a login screen popping up or anything - so the security itself does seem to work.
Any help or ideas to fix this would be greatly appreciacted

Have you implemented CORS support in your API?

Yes I have, using the following in WebApiConfig.cs
var cors = new EnableCorsAttribute($"https://<tenant>.sharepoint.com", "*", "*") { SupportsCredentials = true };
config.EnableCors(cors);

I've used this same API before without issue in a custom scenario, but also on a modern SharePoint page.
What I'm mostly confused by is the origin 'null' part of the error message I guess...

@waldekmastykarz I did get that, yes. That's what resulted in a 302 response redirecting to the login.microsoftonline.com.... url.
Now, I haven't changed anything to code/settings at all, and I see it's working now. I'm not sure why?
Perhaps some sort of permission propagation that needed to be done overnight?

Great to hear it's working now. Is it a new tenant or one that you've had for a while?

It’s a new development tenant. I created it 3 days ago to test the aadhttpclient :)
Op 14 sep. 2018 om 19:40 heeft Waldek Mastykarz <notifications<EMAIL_ADDRESS>het volgende geschreven:
Great to hear it's working now. Is it a new tenant or one that you've had for a while?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/SharePoint/sp-dev-docs/issues/2537#issuecomment-421432393, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABZjf6yex5XB4e8iF4ejN-LGEsE0xLoYks5ua-oJgaJpZM4Wmv6b.

Maybe it indeed takes a moment for all resources to be propagated. Weird though that it was the CORS request failing. You haven't by any chance examined the access token in the failing request, have you?

I have the same problem here with my WebApplication trying to access my endpoint. So the solution to this problem is to wait a bit or?

Following the same procedure, which authentication mechanism as API works with aadhttps client. I have tried UseOpenIdConnectAuthentication but its not working

Make sure that in the O365 API Management there are no waiting requests for authorization, and on the azure side too.


update broken link in chainerx.md

Update a broken link.

Can one of the admins verify this patch?

Good catch and thanks for the fix.

jenkins, test this please.

Travis failures are unrelated.

Jenkins CI test (for commit 619ac5a8db6fb1b19b97d492dede17b5e739195d, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

CI failures seem unrelated. LGTM!

Jenkins CI test (for commit 619ac5a8db6fb1b19b97d492dede17b5e739195d, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)


Selection mouse:

Double click should select the word
Triple click should select the line
On a large buffer, starting the selection, starts from somewhere way back
Scrolling looks busted.

Implemented most of the modes, but Apple has a nice feature when double-clicking a parenthesis, bracket or brace, it selects everything the "expression" as a whole, with not quite obvious semantics, which are really nice.

Implemented the expression mode.


Version mismatched

Git tag v1.0.2
package.json 1.0.1
To fix: bump package.json  to 1.0.2, then retag it.
To automate the process of package release, please checkout

Using GitHub Actions: https://medium.com/openupm/how-to-maintain-upm-package-part-2-f352fbf5f87c
Using CLI which gives you more control: https://medium.com/openupm/how-to-maintain-upm-package-part-4-managing-package-release-with-cli-972ff5311163

Fixed


Create USGS_NLCD_RELEASES_2019_REL_NALCMS.jsonnet

Create NALCMS.jsonnet under USGS catalog area.

The failure is from an actual TAB character breaking |||.
Take a peak at #452 where I started working through some of the comments I have.  It's definitely not done.
Here is an example of gee:classes:
https://github.com/google/earthengine-catalog/blob/main/catalog/USGS/USGS_NLCD.jsonnet#L144
The long lines can be fixed with ||| or 'string one ' +\n'string two '\n + 'string three.'

@schwehr can we possibly get the color scheme in the Examples area to match the image in the Asset? See attached for how it looks in the Asset. We think it would be nice to have it match (or at least pretty close). Thanks!

@schwehr can we possibly get the color scheme in the Examples area to match the image in the Asset? See attached for how it looks in the Asset. We think it would be nice to have it match (or at least pretty close). Thanks!

Definitely. You can replace the color names in the script with the colors that you have from the gee:classes, that would be great.

@schwehr can we possibly get the color scheme in the Examples area to match the image in the Asset? See attached for how it looks in the Asset. We think it would be nice to have it match (or at least pretty close). Thanks!

Definitely. You can replace the color names in the script with the colors that you have from the gee:classes, that would be great.

I updated the colors and ran with the:
var image = ee.Image('projects/ee-nalcms/assets/nalcms/2020');
it looks pretty similar to this: https://www.mrlc.gov/sites/default/files/inline-images/2005_nalcms_large_0.jpg
but not too similar to the image in the Asset (screenshot) (that's more of a blue theme).
The colors still aren't quite right but I don't want to hold this up from just that. Maybe we can tweak them later after it's done. Let me know if you can see better color numbers to use that would match closer. Thanks!

Please leave this PR as is for now. It will likely be a week or two before we figure out how we plan to handle this PR.
Thank you for the PR!

Published and code is live in EE Data Catalog


please do not mix signed+encrypted headers with unsigned+cleartext headers

Over in https://0xacab.org/schleuder/schleuder/merge_requests/46 i learned that mail-gpg handles an encrypted PGP/MIME message this way when asked for decryption:

mail-gpg […] Use[s] the decrypted parts as new mail-object and add all headers from the "outer", encrypted message into it (without overwriting any headers).
Either way we end up with a message that contains the cleartext, and also headers from the "outer" message.

This means that the resulting decrypted message object (which is also associated with some sort of signature status, if there was a signature within the encryption as well) has a mixture of cleartext+unsigned headers and encrypted+signed headers.
This is potentially pretty bad news, if any subsequent message handler infers something about the body of the message based on the headers which it thinks are signed, or had been protected by encryption.  Is there no way to distinguish between the "inner" headers and the "outer" headers once decryption is done?
mail-gpg should provide the decrypted part independently from the outer cleartext/forgable layer so that handlers can distinguish between them.

That is an interesting point.
One of the design goals of mail-gpg was to be as transparent as possible, that's why it is the way it is (encrypted mail Message object in, clear text mail Message object out). I'd like to find a way to mark the headers somehow while keeping this transparency for systems that rely on it.

Unfortunately, the way that e-mail is constructed in general, it's not possible to reason clearly in this way.
Marking headers would be a good first step.  However, it's not only headers that a MUA needs to think about -- it could also be MIME parts as well.
For example, what does mail-gpg do if only a subpart of the message was encrypted+signed?  for example, you could have a message where only an inner part was encrypted+signed, but additional (spoofed) data is additionally attached.

One option could be to store the verification-result with every header and mime-part, I think. E.g. by monkey-patching the corresponding classes and adding methods like was_signed?, was_encrypted?, signature, etc.


generateContractTests give StackOverflowError with (no error messages)

Hi,
I have a bug using Spring-Cloud-Contract.  The error appears when I use, in the response body with a json file or body directly added, like in the next example :
response {
          headers {
              contentType(applicationJson())
          }
          status OK()
          body( file('reponseCarteADA.json') )
          /*body (
                  "statutProfilAcces": value($(producer("ACTIF")))
          )*/

      }

If the body is not the present, all my stubs are created and everything woth as expected.  The full error message is :
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':generateContractTests'.

java.lang.StackOverflowError (no error message)

springCloudVersion = "Hoxton.SR2"
Spring-Cloud-Contract : 2.2.2.RELEASE
Spring-Cloud-Services-Dependencies : 2.2.3.RELEASE (I have try with 2.2.2 as well)
Junit 5
spring-cloud-starter-contract-verifier : 2.2.2.RELEASE
That's it I think
Thanks
Daniel

Hi.  I found the solution to my problem but it will be great if the error message is more explicit.  In the Body of my Request, i used $(producer(... instead of consumer.  When using only producer, it give the error.  What's weird is that the error is their only if their is a body in the Response.  So, it may not be a bug but it could be a nice enhancement.  Tks

I can't replicate this with the latest snapshots. I'll close this for now and if you are able to replicate this we'll reopen it.


Doc pages missing / misleading directions in README?

Documentation Related To Component:
RxJS
Please check those that apply

[ ] typo
[x] documentation doesn't exist OR
[x] documentation needs clarification
[ ] error(s) in example
[ ] needs example

Description Of The Issue
In the doc/README.md page it says:

These files are not meant for reading directly in GitHub, they are just source code for generating the official page. You should find the docs at http://reactivex.io/rxjs/, containing all the documentation.

However, either most pages in the doc/ directory are missing from the reactivex.io site, or they're really hard to find. For example, I was looking for a page to match "writing-marble-tests.md" but can only see: Overview, Installation, Migration and the auto-generated API docs.
Would guess either:

README.md is wrong — you are better off reading directly in GitHub at the moment;
Some work needed to add these files to the docs site; or
The docs are there somewhere and navigation can be improved.

Sorry if this is in the wrong place or a duplicate, I spent a few minutes looking through existing bugs open and closed but didn't see anything similar.
P.S. It seems like the doc/ folder is in the process of being phased out in favour of the work in docs_app/ ? Might suggest a note is added to clarify.

@JWO719 please check this issue and let me know anything I can help with.

Hi @robations, so basically we are currently reworking the documentation. And yes you are right, docs_app is what we label "new docs". We don't want to migrate all the stuff, because many of those are outdated, or not easy to understand and stuff like that. But at this moment it would be really helpfull if you could say what you are struggeling with/ searching for so that we can cover those parts in some kind of tutorial or something likewise. Is there anything else the new docs "http://rxjs-dev.firebaseapp.com/" could help you with this issue?
Regarding the reactivex.io page. I think the root will redirect to the "new docs" but if you directly try to open some of the urls this should work.

So personally improving the developer experience is my main goal, therefore I try to improve the documentation :D So thank you very much for this valuable feedback, I'll take care of it

@JWO719 are there plans to migrate particular pages? The current docs are missing a lot of the useful "Overview" pages including the Observable and Subject (e.g. https://github.com/ReactiveX/rxjs/blob/master/doc/subject.md). I would like to help to update some of these documentation pages to new examples, but I'm not sure of the progress on that or whether there are plans to do away with some of these pages.

It is planed to migrate those informations to the new docs. I think we actually didn't started with that. So there are a lot of pages to migrate :) I would really welcome any contributions on that. If you need help with that, just ping me. I'm always glad to help.

@JWO719 as the PR is merged we could close this issue

Looks good to me.


[Snyk] Security upgrade io.streamnative:pulsar-broker from <IP_ADDRESS> to <IP_ADDRESS>

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

539/1000  Why? Has a fix available, CVSS 6.5
Denial of Service (DoS) SNYK-JAVA-IONETTY-5725787
io.streamnative:pulsar-broker:  <IP_ADDRESS> -> <IP_ADDRESS>
No
No Known Exploit

616/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.9
Integer Overflow or Wraparound SNYK-JAVA-ORGXERIALSNAPPY-5710959
io.streamnative:pulsar-broker:  <IP_ADDRESS> -> <IP_ADDRESS>
No
Proof of Concept

696/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 7.5
Denial of Service (DoS) SNYK-JAVA-ORGXERIALSNAPPY-5710960
io.streamnative:pulsar-broker:  <IP_ADDRESS> -> <IP_ADDRESS>
No
Proof of Concept

616/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.9
Integer Overflow or Wraparound SNYK-JAVA-ORGXERIALSNAPPY-5710961
io.streamnative:pulsar-broker:  <IP_ADDRESS> -> <IP_ADDRESS>
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)

 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.


export-* services don't support standard log settings

The export services in the California release use a different logging package (go.uber.org/zap)
than all the other services, therefore it's not possible to configure their logging behavior via consul or local configuration settings.
The snap build configures all of the service to write out log files to a $SNAP_COMMON (a write-able directory shared by all versions of the snap), but since the export-* services don't support a LoggingFile setting, their log files are not being written to disk when run inside the snap.  When run in docker containers, it's not possible to change the name of the log file, nor can the services be configured to use the support-logging service.
This bug is being reported against the latest code as of Jun 25 '18.

Export services were the first ones written, ment to serve as a blueprint for other services. Not all propositions from this blueprint were respected, so Export services diverged.
This should be fixed with the PR and Export services should be aligned to other services.


python311Packages.mmengine: 0.10.3 -> 0.10.4

Description of changes
Changelog: https://github.com/open-mmlab/mmengine/releases/tag/v0.10.4
cc @benxiao

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 306192 run on x86_64-linux 1

4 packages failed to build:

python312Packages.mmcv
python312Packages.mmcv.dist
python312Packages.mmengine
python312Packages.mmengine.dist

4 packages built:

python311Packages.mmcv
python311Packages.mmcv.dist
python311Packages.mmengine
python311Packages.mmengine.dist

Result of nixpkgs-review pr 306192 run on x86_64-linux 1

4 packages marked as broken and skipped:

python312Packages.mmcv
python312Packages.mmcv.dist
python312Packages.mmengine
python312Packages.mmengine.dist

4 packages built:

python311Packages.mmcv
python311Packages.mmcv.dist
python311Packages.mmengine
python311Packages.mmengine.dist

Should we mark the other platforms as broken?

Should we mark the other platforms as broken?

I would say yes.

Result of nixpkgs-review pr 306192 run on aarch64-darwin 1

8 packages marked as broken and skipped:

python311Packages.mmcv
python311Packages.mmcv.dist
python311Packages.mmengine
python311Packages.mmengine.dist
python312Packages.mmcv
python312Packages.mmcv.dist
python312Packages.mmengine
python312Packages.mmengine.dist

Result of nixpkgs-review pr 306192 run on x86_64-darwin 1

8 packages marked as broken and skipped:

python311Packages.mmcv
python311Packages.mmcv.dist
python311Packages.mmengine
python311Packages.mmengine.dist
python312Packages.mmcv
python312Packages.mmcv.dist
python312Packages.mmengine
python312Packages.mmengine.dist

Result of nixpkgs-review pr 306192 run on aarch64-linux 1

8 packages marked as broken and skipped:

python311Packages.mmcv
python311Packages.mmcv.dist
python311Packages.mmengine
python311Packages.mmengine.dist
python312Packages.mmcv
python312Packages.mmcv.dist
python312Packages.mmengine
python312Packages.mmengine.dist

Result of nixpkgs-review pr 306192 run on x86_64-linux 1

4 packages marked as broken and skipped:

python312Packages.mmcv
python312Packages.mmcv.dist
python312Packages.mmengine
python312Packages.mmengine.dist

4 packages built:

python311Packages.mmcv
python311Packages.mmcv.dist
python311Packages.mmengine
python311Packages.mmengine.dist

may i ask what the previous failures were? where they transitive failures from wasmer or was it mmengine itself that broke?

then i don't really se why mmengine is marked broken?

then i don't really see why mmengine is marked broken?

It's marked as broken on the other architectures. This is already the case on master.

Checked hydra, RuntimeError: internal error isn't really actionable i agree. LGTM


Wrap LAPACK.dgesvd for singular value decomposition

Currently, scipy only wraps dgesdd, which does essentially the same thing, but makes very mild assumptions about floating point arithmetic. At least in some cases, this approach fails and dgesvd works more robustly.
Also, it seems that other platforms (e.g. Matlab and Octave) use the dgesvd implementation.
http://www.netlib.org/lapack/explore-html/db/db4/dgesdd_8f.html
http://www.netlib.org/lapack/explore-html/d8/d2d/dgesvd_8f.html

The Cython LAPACK wrappers that are new in 0.16.x do wrap dgesvd.

The assumption:
The divide and conquer algorithm makes very mild assumptions about
 floating point arithmetic. It will work on machines with a guard
 digit in add/subtract, or on those binary machines without guard
 digits which subtract like the Cray X-MP, Cray Y-MP, Cray C-90, or
 Cray-2. It could conceivably fail on hexadecimal or decimal machines
 without guard digits, but we know of none.

Is that really a concern? Do you have an example where the assumption doesn't hold?

This is curious. I was trying to reproduce the problem I had before but it now seems that my results are essentially the same for both implementations. I'm not sure if something changed since I updated all relevant python packages. Sorry about this.
Regardless, when A has redundancy in the singular values, dgesvd and dgesdd return different representations of the subspaces (of course, the SVD solution is not unique in this case, so this is expected). However, it might be important to have dgesvd available in Numpy for cases where one would like to compare results to Matlab/Octave directly.
I did not find the wrapped dgesvd function in 0.17 SciPy while dgesdd is still there. Could you point me to the right place?

*gesvd is not wrapped on Python level on Scipy 0.17.dev. An ABI-stable
pointer is however exported to Cython, however:
http://docs.scipy.org/doc/scipy-dev/reference/linalg.cython_lapack.html
.
Adding the routine also to scipy.linalg.lapack would in principle be
straightforward; cf.
https://github.com/scipy/scipy/blob/master/scipy/linalg/flapack.pyf.src#L555
Due to historical reasons, these wrappers are not autogenerated, and do
not consequently cover all of LAPACK.

I'm hitting an issue with my code, which uses lots of SVD calls, that I get a fair number of numpy.linalg.linalg.LinAlgError: SVD did not converge's. I'm using Anaconda with MKL, and have [opened an issue with the Anaconda folks] (https://github.com/conda/conda/issues/2244), but I don't know how long it will take them to sort it out.
In the meantime, I'd like to try adding an option to compute SVD using the alternate algorithm. We could add a kwarg to linalg.svd(..., method='dgesdd' | 'dgesvd') (or really something more human-readable) for it. WDYT? It would be a potential workaround for this issue, and expose the alternative algorithm for people who want to use it.
If that's a no-go, I'd like to at least modify scipy, if necessary, to make my own derivative version of svd (it's not many lines anyway) that would do it. I take it that adding it to scipy.linalg.lapack, as mentioned in @pv's most recent comment, would do it?


broken pipe in pacman!

I am using manjaro linux. when I try to install any package with pacman, I get an error:
error: unable to write to pipe (Broken pipe)
what should I do to fix this error?
thank you.
sudo pacman -S maxima
resolving dependencies...
looking for conflicting packages...

Packages (1) maxima-5.46.0-5

Total Installed Size:  63.02 MiB
Net Upgrade Size:       0.00 MiB

:: Proceed with installation? [Y/n]
(1/1) checking keys in keyring                                              [##########################################] 100%
(1/1) checking package integrity                                            [##########################################] 100%
(1/1) loading package files                                                 [##########################################] 100%
(1/1) checking for file conflicts                                           [##########################################] 100%
(1/1) checking available disk space                                         [##########################################] 100%
:: Running pre-transaction hooks...
(1/1) Creating Timeshift snapshot before upgrade...
==> skipping timeshift-autosnap due skipRsyncAutosnap in /etc/timeshift-autosnap.conf set to TRUE.
:: Processing package changes...
(1/1) reinstalling maxima                                                   [##########################################] 100%
:: Running post-transaction hooks...
(1/6) Arming ConditionNeedsUpdate...
(2/6) Updating the MIME type database...
(3/6) Refreshing PackageKit...
(4/6) Updating the info directory file...
(5/6) Updating the desktop file MIME type cache...
(6/6) SELinux: relabel installed files
error: unable to write to pipe (Broken pipe)

This doesn't appear to have anything to do with Spark.


create package

Would it be possible to provide this library in form of a python-package? I am asking because FreeCAD is very interested in this project and people already forking PyFlow, but I guess it would be of more benefit for both project if PyFlow would be provided as a package and can be extended/used by external modules/packages.

Yes. PyFlow will be under pip eventually. And yes - pyflow will have own package system on next release. (it already have in master branch). It is completely modular

Is this solved already? Basically what I asked for was a setup.py to install the library via pip.

not it is not solved yet. we already haev issue for pip installation #13. So i closed this as duplicate

ah great, thanks.


Obfuscator auto crash without dialog box or pop up

I might be wrong about this part, and I do have screenshots. I just need to do a video test but whenever I have Obfuscation turned on, the character will lag out for maybe 2-3 seconds then the game will instantly crash without a dialog pop-up window with an issue for the fix. The best way I found to not auto crash is to play with Obfuscator turned off and then, it may be fine, or it may not. Let me do a video test to be sure. I will attach screenshots here as well.

I will do a video test via OBS and send another submission to make sure I am not crazy lol!!!! 😂 Anyways have a nice day!

Here is the first test I just did with Obfuscation turned on and you will see that the game did instantly crash.

I am making a video to see if I can reproduce the issue. This is with Obfuscation turned on in-game, and you will see that the game will lag a bit and then it will crash. https://youtu.be/S-B4pr-zQpM

Sorry, but 1) I have no idea what's going on in this 0.3 fps video and 2) I can't replicate it or most of the other crashes you're having. I assume this is just co-play of the 23101320 mods you have running in the game, especially if you don't get a crash dump.

https://youtu.be/hsV5BstqCvI - This was run without Obfuscation but yea just close this. I will test something else. https://youtu.be/hsV5BstqCvI


Adding NETFX_CORE compiler flag

Hello guys,
i have a app for multiple platforms which I also want to release for Hololens.
To be able to share assets and prefabs I want to use the same project, but I get compiler errors as soon as the Toolkit is included because it's uwp code. I used the #if NETFX_CORE flag a few times and asked myself if you think it would make sense to add this to every c# script in the sources.
What do you think?
Best regards
Dominic

There's already a PR https://github.com/Microsoft/HoloToolkit-Unity/pull/443 addressing some of these concerns (but instead it wraps everything in #if UNITY_WSA.
Here's a poll over on the MSFT Developer forums that asks the community what they think.
IMO I think it's messy and doesn't address the problem very well.  If you need the sharing services for the application to talk to each other, that should port over fine without any extra work.
The downside is that you'd need to import your shared assets into each project which can be time consuming, unless you're using asset packages and have good modules setup in git.
I think the ultimate goal is to make everything compile and play nicely across platforms as some point, but I think the HoloToolkit-Unity has a little ways to go before we can start thinking about optimizations like this.

Should we close that issue as it's duplicated? I understand the decision and of course it's not the best solution to use compiler flags, but maybe it's a workaround as long as the toolkit is not compatible....

I wouldn't say it's a duplicate, because of how the issue is worded and it's solution, but I think that's up to you on if it's closed.  If you decide to close it, just be sure to comment your solution in the other issue, so others have a way to see it.
Yeah I think this is something we all need to discuss as a community and see if anyone has some clever ideas to fix this, without touching every class.
With that being said, I think it's super important for us to get cross compatibility at some point.

I'd rather messy code than code that prevents building the application on all non-Microsoft operating systems.

While I'm all for making the project cross platform, I think it's important to keep a few things in mind.

Code of Conduct.
Maintainability (Messy code could potentially create more work for others in the future).
The main page clearly states that this project targets Windows Holographic.

The HoloToolkit is a collection of scripts and components intended to accelerate development of holographic applications targeting Windows Holographic.

Just because it targets Windows Holographics doesn't mean it should prevent compiling for other platforms. For example, I have a scene for mobile apps and a scene for hololens. The hololens uses scripts and features from the mobile scene. Due to unity not allowing scripts or folders to be excluded from specific builds, it means we have to do this via pre-compiler tags in the scripts.
Another solution could be to have the entire set of c# scripts compiling into one External Plugin for unity. Which can be excluded from all other platforms.
As for pre-compiler tags being messy, I honestly think all pre-compiler tags look bad, however, they are a requirement for multi-platform and Unity.

Another solution could be to have the entire set of c# scripts compiling into one External Plugin for unity. Which can be excluded from all other platforms.

I really like this idea. In fact I've played around with the some builds like this and it also helps cut down compile time.
I think the project could be refactored a bit more to to compartmentalize the platform specific stuff a bit more is all.  I think the general focus for MSFT has been Windows 10 Only, which is their OS so it makes sense.  FYI, the sharing service is definitely portable.  IMO it's really the only important part for cross platform support, where all the other features focus on Spatial Understanding and Windows Holographic Input support.

Also pre-compiler tags are a fact of life for Unity development

😆 Isn't that the truth!

@DominicBoettger is this still an open issue?

Hi @stephenhodgson,
I think this is solved....
Thanks!


Feature/optimize id sanitization

Codecov Report

Merging #91 into master will increase coverage by 0.32%.
The diff coverage is 91.07%.

@@             Coverage Diff             @@
##             master     #91      +/-   ##
===========================================
+ Coverage     98.37%   98.7%   +0.32%
- Complexity     1049    1107      +58
===========================================
  Files            62      62
  Lines          2528    2626      +98
  Branches        340     374      +34
===========================================
+ Hits           2487    2592     +105
+ Misses           27      19       -8
- Partials         14      15       +1

Impacted Files
Coverage Δ
Complexity Δ

...ightstep/WingtipsToLightStepLifecycleListener.java
94.01% <91.07%> (+14.35%)
63 <25> (+34)
:arrow_up:

...util/WingtipsToZipkinSpanConverterDefaultImpl.java
94.69% <91.07%> (-5.31%)
65 <25> (+24)

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 5005a16...725d7d1. Read the comment docs.


How to pass TF_DATA_DIR in terraform.Options

Tool set versions

Terraform v0.15.3
terratest v0.34.8

Directory structure
$ tree .
.
├── e2e_test.go
├── go.mod
├── go.sum
├── infrastructure_test.go
├── kubernetes_test.go

$ tree ../environments/infrastructure/terratest
../environments/infrastructure/terratest
├── terratest-backend.hcl
└── terratest.tfvars

$ tree ../environments/environments
../environments/environments
└── infrastructure
    └── terratest

infrastructure_test.go
package test

import (
	"github.com/gruntwork-io/terratest/modules/terraform"
	"testing"
)

func TestInfrastructure(t *testing.T) {
	environment := "terratest"
	infrastructureOptions := terraform.WithDefaultRetryableErrors(t, &terraform.Options{
		TerraformDir: "../environments/infrastructure/",
		Lock: true,
		Upgrade: true,
		BackendConfig: map[string]interface{}{
			"hostname": "app.terraform.io",
			"organization": "foo",
			"workspaces": "{name = terratest-infrastructure }",
			//"../environments/infrastructure/" + environment + "/terratest-backend.hcl",
		},
		EnvVars: map[string]string{
			"TF_DATA_DIR": "../environments/infrastructure/" + environment + "/.terraform",
			"TF_LOG": "trace",
		},
		VarFiles: []string{
			"../environments/infrastructure/" + environment + "/terratest.tfvars",
		},
	})
	terraform.InitAndPlanWithExitCode(t, infrastructureOptions)
}

What is the problem

terraform.Options is not taking TF_DATA_DIR correctly and as a result I end having .terraform directory at created at wrong location see output of$ tree ../environments/environments. Ideally it should create .terraform directory under ./environments/infrastructure/terratest
As per the official terraform docs TF_DATA_DIR is one of the environment variables.
On the other hand TF_LOG works as expected. But then why TF_DATA_DIR does not work.

Steps to reproduce

Create the directories as structured above.
Create a main.tf in the location mentioned as value for TerraformDir and add following code. terraform {
   required_version = "= 0.15.3",
   backend "remote" {}
 }
   variable "my_name" {
       type = string
       default = ""
   }

   output "name" {
       value = var.my_name
   }

Navigate to test directory and execute

$ go test -v infrastructure_test.go

Expected behaviour
./environments/infrastructure/terratest should have a .terraform directory.
Actual behaviour
Creates .terraform directory at ./environments/environments/terratest location.
Additional questions

Is this correct way to pass backend.hcl to terraform.Options

		BackendConfig: map[string]interface{}{
			"path": "../environments/infrastructure/" + environment + "/terratest-backend.hcl",

Here are the contents of terratest-backend.hcl
organization = "foo"
workspaces {
  name = "terratest-infrastructure"
}

Side notes

In the course of this issue please ignore following error

        	            	╷
        	            	│ Error: Invalid backend configuration argument
        	            	│
        	            	│ The backend configuration argument "workspaces" given on the command line
        	            	│ is not expected for the selected backend type.
        	            	╵
        	            	}
        	Test:       	TestInfrastructure
--- FAIL: TestInfrastructure (0.78s)

Very much looking forward for a quick response. Thank you !!!

	infrastructureOptions := terraform.WithDefaultRetryableErrors(t, &terraform.Options{
		TerraformDir: "../environments/infrastructure/",
		Lock: true,
		Upgrade: true,
		BackendConfig: map[string]interface{}{
			"hostname": "app.terraform.io",
			"organization": "foo",
			"workspaces": "{name = terratest-infrastructure }",
			//"../environments/infrastructure/" + environment + "/terratest-backend.hcl",
		},
		EnvVars: map[string]string{
			"TF_DATA_DIR": "../environments/infrastructure/" + environment + "/.terraform",
			"TF_LOG": "trace",
		},
		VarFiles: []string{
			"../environments/infrastructure/" + environment + "/terratest.tfvars",
		},
	})

I think the issue is the following:

You set TerraformDir to "../environments/infrastructure/". That means that, when Terratest runs terraform, it'll do so with with ../environments/infrastructure as the working dir.
You set TF_DATA_DIR to "../environments/infrastructure/" + (...). Well, if start in the working dir of ../environments/infrastructure, go up one folder due to the ../, you end up in ../environments... And then within that folder, you're telling Terraform to put the data dir in environments/infrastructure, so the path ends up as ../environments/environments/infrastructure.

So what you probably want is to set TF_DATA_DIR to either ../../environments/infrastructure or ../infrastructure.


Return to home button added

🛠 Fixes Issue
Fixes: #594
👨💻 Description
Added a button in login page that navigates to home page
What does this PR do?

Briefly describe the feature or bug fix you've implemented.
Highlight any major changes or added functionality.
Mention how it enhances the project.

📄 Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (adds or updates related documentation)

📷 Screenshots/GIFs (if any)
https://github.com/user-attachments/assets/d9b65e63-75f0-4624-b504-3bedf4ebb094
✅ Checklist

[x] I am a participant of GSSoC-ext.
[x] I have followed the contribution guidelines of this project.
[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have added documentation to explain my changes.

Mandatory Tasks

[ ] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

🤝 GSSoC Participation

[ ] This PR is submitted under the GSSoC program.
[ ] I have taken prior approval for this feature/fix.

@YadavAkhileshh Please review my PR


chore(deps): bump go-libp2p-kad-dht to v0.26.1

Update dht deps

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

Project coverage is 45.57%. Comparing base (2469e7a) to head (b5966c9).
Report is 178 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3698      +/-   ##
==========================================
+ Coverage   44.83%   45.57%   +0.73%
==========================================
  Files         265      281      +16
  Lines       14620    16042    +1422
==========================================
+ Hits         6555     7311     +756
- Misses       7313     7882     +569
- Partials      752      849      +97

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Set up period?

If we are going to have the users do the install, should we have a set up period scheduled?

Well, we do have times for a conda tutorial and a git/github tutorial. Activities don't get any more practical than cloning our git repo and setting up the conda environment.

OK, then it is implicit in the git and conda sections. Closing.


Delete input_data.py

Nagging Reviewer @petewarden: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @petewarden: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 29 days with no activity and the awaiting review label has been applied.

I'm not sure I understand the context of this change. It appears to just delete the test for input_data.py? Could you add some more information about the motivation for this and I'll look at it again.


Added right_click method for watir and location and size methods for watir and selenium

Right click on Watir was not defined, so added that. Also added methods to access the size and location of elements.
Previously these methods were all being passed to webdriver, and so giving a deprecation warning.

@DavidSelby I just merged a pr that had a location method.  Would it be possible for you to update this pr to use that method instead?  I plan to make a release today or tomorrow, so if you can do it soon that would be great.  Thanks for your help!

Thanks @cheezy. Sorry for the delay, I'm in GMT. Closing this pull request and created a new one to keep it tidy. https://github.com/cheezy/page-object/pull/349


how to Beagle AI factory reset

Hello.
I got BB AI and tried to update its image according to guild from beagle introduction, it says I must update BB image to latest.
I followed according to guide and all of sudden, my BB shutdown during "sudo apt upgrage" and terminal said that "bb shutdown automatically due to its internal temperature is too high..." so removed power and power on again after few minutes and it's working and I could access via <IP_ADDRESS>.
I started update image again but it requested some instruction to recover issue and I followed accordingly and I tried to update again but I couldn't connect at "http://repos.rcn-ee.com/debia" and keep failed to update or install something.
My BB AI seems to be cannot connect at server or internet so I would like to try factory reset.
How can I try factory reset with my BB AI?

I thought instruction at "http://beagleboard.org/getting-started#update" is just for SD card only but it has more information how to update eMMC.
http://beagleboard.org/getting-started#update
https://elinux.org/Beagleboard:BeagleBoneBlack_Debian#Flashing_eMMC