Error message "Unrecognized Controller ID:"

What does the error message Unrecognized Controller ID:  mean?

That happens when a controller that is not paired tries to form a secure connection.  Happens all the time since the new Apple introduced the new architecture.  Should not cause an issues.


Bff/token optimisation

PR to fix: https://github.com/clovaai/donut/issues/266

This is very useful!
One small findings:
add .DS_Store into gitignore or remove it.

Ah yes, thanks for catching that - removed the file

@bugface – let me know if you're happy to merge :)

@bugface – let me know if you're happy to merge :)

I think it looks great.

I think we're good to merge then, @bugface :)

news about this? @bugface

Ah yeah, I guess bad style. At the time I wanted to be explicit because this bug was so opaque. Are you happy to merge if I fix that?

I could be interested in patching it into my fork, if you fix and explain
the changes. Thank you.


Add client fixture for tests

Proposed change

SSIA
Additional information

This PR fixes or closes issue: fixes #

Checklist

[ ] I have updated the documentation if needed.
[ ] I have updated the tests if needed.

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

Comparison is base (6d74ab2) 100.00% compared to head (0ed4a9c) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #441   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          141       141
  Branches        18        18
=========================================
  Hits           141       141

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


app version doesn't update

I updated my appstore app with a new version (1.1.6)
call the api with the current version 1.1.5
https://check-version.flexible.agency/ios/com.pennstate.wwyc/1.1.5?country=us
always return with version 1.1.5
Looking through the server code "utils.js"
const axios = require("axios");

const cache = {}

const lookupVersion = async(platform, bundleId, country = 'us') => {
  const key = `${platform}.${bundleId}`;
  let res = cache[key];
  if (res) {
    return res;
  }

Apparently, it is always returning the result with platform and bundleId. Is this expected, that the server will only register the app info once?

Hi! The cache is actually not a long-term cache, since this code runs on AWS lambda, which doesn't let any instance of the code run longer than 15 minutes. So this caching is only to catch quick subsequent requests.
In this case, it seems like the App Store metadata API is running behind. Our API is only a thin caching layer around the metadata API that basically does something like this:
curl 'http://itunes.apple.com/lookup?lang=en&bundleId=com.pennstate.wwyc&country=us' | grep version

Running that in my terminal just now it returned version 1.1.5 as the latest version as well. So it doesn't seem to be a problem in our backend, but rather on Apple's side.
I expect it to be a little delay, but let's leave this issue open for a couple of days to see if it resolves automatically.

thank you for the quick reply. I'm surprised Appstore isn't updated to all regions. I'm getting 1.1.6 at the moment. I'll check again sometime if this has resolved the version correctly.


clearRegions  method  has bug

Bug description
The  regions array is not cleared after the clearRegions method is called
Environment

Browser: Chrome

Minimal code snippet
Expected result
Obtained result
Screenshots

this.regions.forEach((region) => (region && region.remove()))

Hmm, weird. Regions are supposed to be removed from the list on the remove event:
https://github.com/katspaugh/wavesurfer.js/blob/main/src/plugins/regions.ts#L581

So we can only remove regions using events, not clearRegions? Because clearRegions is going to be a problem next time.

Hmm, weird. Regions are supposed to be removed from the list on the remove event:
https://github.com/katspaugh/wavesurfer.js/blob/main/src/plugins/regions.ts#L581

So we can only remove regions using events, not clearRegions? Because clearRegions is going to be a problem next time

No, the event is emitted when a region is removed. So when clear regions removes each region, it's supposed to emit a remove event and get removed from the list.

No, the event is emitted when a region is removed. So when clear regions removes each region, it's supposed to emit a remove event and get removed from the list.

I see what you're said， How do I know when clearRegions has cleared all of them

No, the event is emitted when a region is removed. So when clear regions removes each region, it's supposed to emit a remove event and get removed from the list.

I see what you're said， How do I know when clearRegions has cleared all of them

I can only use other variables to control, I can't get this moment from an event？？？

You don't need to use the event. All I'm saying is that clearRegions is supposed to remove each region from the this.regions array thanks to an internal remove event subscription. If it doesn't, there might be a bug.

clearRegions

When clearRegions called, you should make sure that addRegions cannot be executed, otherwise it will get messy

I see. You can probably call addRegion with a delay (setTkmeout) as a workaround.

I see. You can probably call addRegion with a delay (setTimeout) as a workaround.

That doesn't work, I think of a way, could you have a look?
`
public clearRegions() {
// this.regions.forEach((region) => region.remove())
const regions = cloneDeep(this.regions)
regions.forEach((region) => region.remove())
}
`

That doesn't work, I think of a way, could you have a look?
` public clearRegions() { // this.regions.forEach((region) => region.remove()) const regions = cloneDeep(this.regions) regions.forEach((region) => region.remove()) }
`
That doesn't work, I think of a way, could you have a look?

`
public clearRegions() {
// this.regions.forEach((region) => region.remove())
const regions = cloneDeep(this.regions)
this.regions = []
regions.forEach((region) => region.remove())
}
`

I ran into the same issue and my workaround was this:
regionsPlugin.clearRegions();
regionsPlugin.regionsContainer.innerHTML = "";

I ran into the same issue and my workaround was this:
regionsPlugin.clearRegions();
regionsPlugin.regionsContainer.innerHTML = "";

I used id to delete it

@panhe-xue this is now fixed and can be closed right?

This issue is not fixed @katspaugh. I think the problem is that once the region is removed, the region is now null, so when you go through to remove it from the regions at https://github.com/katspaugh/wavesurfer.js/blob/main/src/plugins/regions.ts#L581 it's already null and thus no regions are removed from this.regions because no regions in this.regions match the region being removed.

Closing as fixed here: https://github.com/katspaugh/wavesurfer.js/pull/3909


Superpixel modal

Issue for app: #514
App pull request

[ ] This is a new app
[X] I am improving an existing app (redesigns/code "makeovers")

About

Playground deployment URL (new version): https://dash-playground.plotly.host/dash-3d-image-partitioning/
Current gallery app URL: https://dash-gallery.plotly.host/dash-3d-image-partitioning/

Workflow

[X] I have created a branch in the appropriate monorepo, and the
elements necessary for successful deployment are in place.
[X] If the app is a redesigned and/or restyled version of an
existing gallery app, I've summarized the changes requested in the
appropriate Streambed issue and confirm that they have been applied.
[X] If the app is on the Dash Gallery portal, I have added a link to
the GitHub repository for the source code in the portal description.
[ ] If the app is a reimplementation of a Python gallery app for the
DashR gallery, the app in this PR mimics, as closely as possible,
the style and functionality of the existing app.=
[X] I have removed all Google Analytics code from the app's
assets/ folder.

The pre-review review
I have addressed all of the following questions:

[X] Does everything in my code serve some purpose? (I have removed
any dead and/or irrelevant code.)
[X] Does everything in my code have a clear purpose? (My code is
readable and, where it isn't, it has been commented appropriately.)]
[ ] Am I reinventing the wheel? (I have used appropriate packages to
lessen the volume of code that needs to be maintained.)

One thing that we could consider adding here is to let users download the file with the brain itself. If someone wants to explore their annotations it would be more exciting to also have the brain to overlay them on.

One thing that we could consider adding here is to let users download the file with the brain itself. If someone wants to explore their annotations it would be more exciting to also have the brain to overlay them on.

Yes this would be pretty easy to do as all the logic is already there to download the 3D annotations. To avoid writing javascript code, you could follow what is done in github.com/plotly/dash-simple-image-processor/.
This app just performs some simple operations on an image and when it's finished applying an operation, it sets an html <a>'s href to a base64 encoded string representing a PNG.
So for the 3D app, you could either just do this conversion to base64 and setting of an href at the app load time, if you just want to users to download the 2 nii files separately (probably the easiest), or you could somehow combine the files every time an annotation is selected (a bit harder and maybe more confusing for the user). Take a look at the simple-image-processor app, maybe you'll see how you might be able to do a similar thing with .nii images.
The relevant code is in the process_image function in simple-image-processor/app.py.

@nicholas-esterer: as far as I can see "download the brain as well" functionality is working now. A couple of open questions:

I had to make the download buttons auto-width so they are still fully legible on smaller screens. Maybe there is a more elegant way?
The modal/tutorial doesn't explain what to do with the "Download Brain Volume" button. I think it is self explanatory. But for someone who has never used any brain image viewer software, getting the brain and segmentation to overlay may be non-trivial. Do you think that should go in, maybe with a separate screenshot?

@emmanuelle: two questions to be safe. is it true that:

[ ] the brain used in the demo is showing an stroke/infarct? I guessed that from the image but am not sure this is correct.
[ ] we can freely share the brain image/have users download it? most probably yes, since it's already on the github repo. Just want to double-check! Maybe there is also someone / some dataset we should thank / reference for this?

Looks good to me! This app never really worked with small screens (you'll see that changing the browser window size doesn't put the figures one on top of the other like the other apps), so unless we fix the whole app I wouldn't worry too much about the button.
As for users that would download the brain volume: this is for advanced users who probably have some other software for viewing the brain, so you could probably just say "the downloaded image is compatible with viewers that can open .nii files" or something like that.

@surchs the images are from MICCAI Brats 2019 challenge http://braintumorsegmentation.org/
The Brats website says
"""
You are free to use and/or refer to the BraTS datasets in your own research, provided that you always cite the following three manuscripts:
[1] B. H. Menze, A. Jakab, S. Bauer, J. Kalpathy-Cramer, K. Farahani, J. Kirby, et al. "The Multimodal Brain Tumor Image Segmentation Benchmark (BRATS)", IEEE Transactions on Medical Imaging 34(10), 1993-2024 (2015) DOI: 10.1109/TMI.2014.2377694
[2] S. Bakas, H. Akbari, A. Sotiras, M. Bilello, M. Rozycki, J.S. Kirby, et al., "Advancing The Cancer Genome Atlas glioma MRI collections with expert segmentation labels and radiomic features", Nature Scientific Data, 4:170117 (2017) DOI: 10.1038/sdata.2017.117
[3] S. Bakas, M. Reyes, A. Jakab, S. Bauer, M. Rempfler, A. Crimi, et al., "Identifying the Best Machine Learning Algorithms for Brain Tumor Segmentation, Progression Assessment, and Overall Survival Prediction in the BRATS Challenge", arXiv preprint arXiv:1811.02629 (2018)
"""
So indeed we should add these references both in the Readme and at the end of modal.

@nicholas-esterer or @emmanuelle: I think this can be merged now

Thanks you @surchs and @nicholas-esterer !


Disallow certain HTML elements from being used

Motivation
Svelte can be used as a strongly component-based system where most of the elements used are components. Therefore, it makes sense that an error is raised whenever an element is specified but should not be used.
Description
Disallows certain built-in HTML elements from being used, instead there's likely a component that should be used in its place. For example, instead of using the h1 element, import and use the Heading element.
Examples

<Heading type={1}>H1</Heading>
code

H1

code

Additional comments
No response

Thank you for positing issue.
That rule sounds good to me.
I think the new rule is similar to the vue/no-restricted-html-elements rule, so its implementation may be helpful.
https://eslint.vuejs.org/rules/no-restricted-html-elements.html
I think you can also use the no-restricted-syntax rule if you want to get it right now.
https://eslint.org/docs/latest/rules/no-restricted-syntax
  "rules": {
    "no-restricted-syntax": ["error", "SvelteElement[kind='html'] > SvelteName.name[name=/^(?:h1|code)$/]"]
  }

@ota-meshi This is a good reference, what I would add, however, is the ability to specify an array of elements when using the { element, message } syntax.
{
	'svelte/no-restricted-html-elements':
	[
		'error',
		{
			element: [ 'h1', 'h2', 'h3', 'h4', 'h5', 'h6' ],
			message: 'Prefer use of our custom <Heading /> component'
		}
	]
}

Welcome pull requests.


Warn or fail if logger called with no consumers

If the logger is called without registered consumers, it just drops the log messages. That was happening on daemon startup.
Should we warn or fail in that case? Maybe print the message to stdout, so it's not lost entirely?

Hm. I feel like we should throw an exception for this.

In any case, it should not fail silently.

We now use this feature to drop log messages during unit tests (via Logger.null). Closing.


Adding adapters

Hello,
would it be possible to add these tow packages to the composer.json file as requirements?
guzzlehttp/guzzle
php-http/guzzle6-adapter
I see it as a unnecessary room for error which can be eliminated. I beleive above packages can be installed when running composer require sparkpost/sparkpost
thank you

Hey!
So just to make sure I'm understanding this correctly, you want it install guzzle by default if there aren't other adapters already installed?

Correct.
I know how to do it, however this can be automated I believe by adding these adapters to required packages.


Support transformers-like api for woq quantization

Type of Change
feature
Description
quant command:
FORCE_DEVICE="cpu" python run_generation_gpu_woq.py --model "/mnt/cached_oses/lzw_data/model/Llama-2-7b-hf/" --eval_batch_size 52 --trust_remote_code --woq_algo Rtn --woq --weight_dtype int4_fullrange --bits 4  --group_size 32

accuracy:
python run_generation_gpu_woq.py --model saved_results/ --eval_batch_size 52 --trust_remote_code --accuracy

Tasks
Version
Filter
n-shot
Metric

Value

Stderr

lambada_openai
1
none
0
acc
↑
0.7202
±
0.0063

none
0
perplexity
↓
3.7429
±
0.0750

Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

@XuehaoSun we need to record recipe after LLM model validation complete in another pr.


Avoid pandas dependency for CelebA dataset

The CelebA dataset has a runtime dependency on pandas. Since all we need is pandas.read_csv, it would be interesting to see if we can replace pandas.read_csv by just the builtin csv module. The mergability of the PR would depend on how ugly / complex the code becomes... :)
cc @pmeier

Yes, that bothered me for a long time. Never enough to fix it though :innocent: The tests also manage without pandas. Go ahead @carolineechen.


not working v1.4.0-1

powershell spits out
Preparing patches...
Verifying files...
Reading file contents...
WARNING: Expected 1 match but found 0, in file '.\assets\js\game.compiled.js', for line:
else
if(b=="pixel-size"){window.IG_GAME_SCALE=(this.values[b]||0)+1;localStorage.setItem("options.scale",window.IG_GAME_SCAL
E)}
Execution have terminated! Press Enter to close the window:

Hi, I've made a PR to fix this. If you don't want to wait for it to be accepted, you can find a working patch here: https://github.com/someonesomewheredev/CrossCode-IntegerScaling

Hi, I've made a PR to fix this. If you don't want to wait for it to be accepted, you can find a working patch here: https://github.com/someonesomewheredev/CrossCode-IntegerScaling

Do you know if this works with the GOG release? I got an error while running the Powershell script.
Preparing patches...
Verifying files...
Reading file contents...
Get-Content : A parameter cannot be found that matches parameter name 'Raw'.
At C:\GOG Games\CrossCode\Install-IntegerScaling.ps1:64 char:54

    $Patch.Content = Get-Content -Path $Patch.File -Raw <<<<

CategoryInfo          : InvalidArgument: (:) [Get-Content], ParameterBin
dingException
FullyQualifiedErrorId : NamedParameterNotFound,Microsoft.PowerShell.Comm
ands.GetContentCommand

WARNING: The file was empty.
Execution have terminated! Press Enter to close the window:

The code has been updated to support the latest copy of the game.

Get-Content : A parameter cannot be found that matches parameter name 'Raw'.

@Tenome, the script requires PowerShell 3.0 (which added support for the -Raw parameter). Based on your error message you're running the script using PowerShell 2.0. You would need to update Windows to get PowerShell 3.0 or newer (we're like up to v5 or v7 now depending on the system).


Xpedite logs

Did something about PMC change for the probes in the allocatorApp? The tests that have PMC and allocatorApp are failing, I don't see a recent change that would have caused this - rebased against master.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1da6569). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master      #6   +/-   ##
========================================
  Coverage          ?   67.3%
========================================
  Files             ?      80
  Lines             ?    5747
  Branches          ?       0
========================================
  Hits              ?    3868
  Misses            ?    1879
  Partials          ?       0

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 1da6569...2c7004f. Read the comment docs.


Increase unit test timeout to 10m

The unit tests have started overrunning the 5 minute timeout when run across all packages, this should prevent this happening again for a little while.

This PR only affects the unit tests timeout, it doesn't touch the actual project code or the e2e tests.
Thus we are overriding E2Es for faster merge as the unit timeout change is blocking other PRs.
/override ci/prow/e2e-aws-operator
/override ci/prow/e2e-aws-ovn
/override ci/prow/e2e-aws-serial
/override ci/prow/e2e-aws-upgrade


Added CoTracker for download stats

Hey!
Thanks for the amazing library :)
We are hosting CoTracker model checkpoint here and would like to see the download count. This PR follows this one to enable it.
Hope it's correct way to do it, let me know if any changes are needed or there is a better way to do so.

@Wauplin I've fixed both things that you mentioned -- looks like good to go :)


Rename the numba_dppy module to numba_dpex.

The numba_dppy module was renamed to numba_dpex.
All remianing references to dppy or DPPY were removed.
Github actions updated to use new path for files.

@PokhodenkoSA I became a bit impatient and indiscriminate with the last set of changes. I am unable to push my fixes as AN servers are down right now. I will try to fix from my home system over the weekend.


Update for Julia 1.0

@travis-ci/julia-maintainers
This is roughly modelled on the successful Appveyor script (https://github.com/JuliaCI/Appveyor.jl/blob/version-1/bin/install.ps1) but I have no way to test this locally.
Would be great to have this in ASAP.

Unfortunately I don't understand how the tests in julia_spec.rb work: is there a way we could, e.g. run the script on https://github.com/JuliaLang/Example.jl?

Looks like this needs a rebase

@simonbyrne I ran a test of this on https://staging.travis-ci.org/JuliaCI/TravisTest.jl/builds/725336 (you can just login and hit "trigger new build" from the "more options" dropdown in the top right), and it looks like the auto-detection of $JL_PKG is failing.  I'm not sure why this would be, perhaps we should try printing out shurl before matching against it, just for debugging, so that we can see why it would be failing to determine the package name.

@joepvd would you be able to try adding this to staging again?

The new version was deployed to staging.

Okay, I'm fairly confident I have now learnt enough Ruby for this to work.
@joepvd Could you please stage this again?

The branch has been deployed.

The code is live on org-staging again

It works! It works! It works!
Thanks @joepvd!

https://travis-ci.org/simonbyrne/GitX.jl/jobs/414074129

https://staging.travis-ci.org/JuliaCI/TravisTest.jl/jobs/725339

For some reason, it seems that the staging environment is having trouble downloading the binaries, e.g.
https://staging.travis-ci.org/JuliaCI/TravisTest.jl/jobs/725357
127.29s$ curl -A "$CURL_USER_AGENT" -s -L --retry 7 'https://julialang-s3.julialang.org/bin/linux/x64/1.0/julia-1.0-latest-linux-x86_64.tar.gz' | tar -C ~/julia -x -z --strip-components=1 -f -
curl: (7) Failed to connect to julialang-s3.julialang.org port 443: Connection timed out

Any idea what is going on here?

Assuming the download issue is due to the staging environment, this is ready to go on our end.

The new master has been deployed to production.
Is there anything that needs to change in https://docs.travis-ci.com/user/languages/julia/? (Thinking example version numbers, yet there might be more :grimacing: )
PRs welcome in https://github.com/travis-ci/docs-travis-ci-com/ ;)


run this project with my own dataset

Hello author, how can I run this project with my own dataset?

you need to make a nuscenes-like dataset and the occupancy annotations, the training process is the same.

您好作者，请问我该如何使用自己的数据做测试？也需要构建像nuscenes的数据集吗？

you need to make a nuscenes-like dataset and the occupancy annotations, the training process is the same.

有制作nuscenes数据集的方法介绍吗？

Sorry, I'm not sure how to make a nuscenes dataset. However, you don't need to follow its format exactly, you just need to make sure you can make the same data_infos.

Sorry, I'm not sure how to make a nuscenes dataset. However, you don't need to follow its format exactly, you just need to make sure you can make the same data_infos.

ok thank you for you reply1


Updated cursor behavior

Closes issue #372
Cursor is now the text editing ibeam instead of general arrow.

Closing for now.. May re-open when I can figure out how to reproduce the odd cursor issue.


Remove out-of-support flags from deployment Yaml

How to categorize this PR?

/area usability
/kind cleanup
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

The flags which went out-of-support in MCM v0.49.0 have been cleaned up from MCM deployment yaml.

/test pull-gardener-e2e-kind-upgrade
/lgtm
/approve


Upgrade to rails 5.1.4

So far I have not seen any test failures similar to what are reported on #609 so I figured I'd push this up and let Travis try it too.

Looks good, thanks 👍


Make 3x3 matrix visible in README.rst

Coverage remained the same at 100.0% when pulling 83d735dcfe2b9729dc47fa45aacf3e3dff4387ab on mwtoews:readme into e90718c6464b1efd7d6aaf1ebc1e86b85032a2ea on sgillies:master.

Coverage remained the same at 100.0% when pulling 83d735dcfe2b9729dc47fa45aacf3e3dff4387ab on mwtoews:readme into e90718c6464b1efd7d6aaf1ebc1e86b85032a2ea on sgillies:master.


change: [M3-7540] - Improve tags experience

Description 📝
Update: it appears that the React 18 upgrade resolved the underlying bug! 🪄 Keeping this PR open without the workaround since it includes some improvements to the tag UX.
~Fixes a bug in the TagCell component's overflow detection logic that caused it to sometimes render incorrectly. This bug only appeared in production builds of Cloud Manager and was caused by a parent Grid component briefly rendering as display: none, breaking the overflow detection in TagCell.~
~To fix this issue, this PR introduces a somewhat convoluted workaround: upon mount, TagCell locates its nearest ancestor in the DOM with display: none and then attaches a MutationObserver (via the newly introduced useMutationObserver hook) to that element, re-computing overflow when it is finally displayed.~
This PR also includes improvements to the display of the TagCell component and AddTag input on small screens.
Changes  🔄

~Fix overflow detection using useMutationObserver on parent element~
Increase width of AddTag input on small screens
Always render "Add a tag" and ellipses buttons to the right of tags
Upgrade deprecated Select component to Autocomplete (this also fixes a momentary flicker when adding a tag)
Replace TagsPanel component with TagCell (these components shared a lot of logic)

Preview 📷

Before
After

Not detecting overflow
Correctly detecting overflow

AddTag unusably small on small displays:
AddTag takes up entire width:

"Add a tag" and ellipses buttons appear to the left of existing tags on small displays:
Buttons now appear to the right of new tags, consistent with the "shadow" effect:

How to test 🧪
Prerequisites
As far as I am aware, this bug is only reproducible in production builds of Cloud Manager. For reproducing or testing, use either:

https://cloud.linode.com / Preview link for this PR
yarn build && yarn workspace linode-manager serve build

Reproduction steps
(For overflow detection bug)

Add several tags to a Linode until they overflow in the Linode details footer
Navigate back to the Linodes landing page (/linodes)
Refresh the page
Return to the details page for the Linode
Observe that overflow detection fails, causing tags to clip instead of displaying the "shadow" effect and ellipses button

(For style changes on small screens)

In the details page for a Linode with overflowing tags, reduce the window size until the mobile layout is activated
Observe that the "Add a tag" and ellipses buttons appear to the left of the tags, inconsistent with how they're displayed on larger screens and unintuitive with the "shadow" UX metaphor indicating an overflow
Click on the the "Add a tag" button
Observe the tag input field is unusably small, with placeholder and tag labels clipped

As an Author I have considered 🤔

[x] 👀 Doing a self review
[x] ❔ Our contribution guidelines
[x] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[x] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[x] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

@abailly-akamai @hana-linode @jaalah-akamai
In a happy turn of events, it appears that @bnussman-akamai's work to upgrade to React 18 has resolved the underlying bug causing the overflow detection to fail 🎉. I am no longer able to reproduce the bug in develop under all previous conditions that I was aware of.
However, I'm keeping this PR open (but removing the workaround code) since it includes some improvements to the tags experience:

Fixing the broken display of AddTag in small screens
Listening to changes in window size, which can cause the tags to overflow
Replacing the deprecated Select component (which was causing some flickering) with Autocomplete
Tweaked the interaction when deleting tags (previously, all tags would be set to disabled; now only the tag that is being deleted is set to disabled)

In the tag drawer, the Autocomplete is opening randomly after I add a tag

Ah, good catch. Seems like this was caused by differences in how Chrome vs Safari trigger onblur.

Somehow I'm able to make a tag look more transparent than the others

This is a new indication for a tag that is in progress of being deleted. Since multiple tags could be deleted at the same time, sometimes this would cause a race condition leading to inconsistent states. Fixed with a new useAtomic hook.

@abailly-akamai Thanks for the feedback. Took it a step further and completely removed TagsPanel in favor of TagCell, since I noticed most of the logic was being duplicated (aa3c9).


Update Composer dependencies (2022-10-18-00-07)

Visual regression test passed!
View the visual regression test report


Imported findings from Kiuwan don't show in metrics dashboards

Bug description
After importing findings for a product from a Kiuwan scan CSV file, the list of findings appear in the findings list for the product and in the correspondent engagement (the Kiuwan scan the gets created automatically with the import).
However, in the dashboard those new findings are not counted and if you go to the product metrics screen they don't appear there either.
I have activated them after the import but still they don't show in any counter/dashboard mentioned above
Steps to reproduce
Steps to reproduce the behavior:

Select a product
Import findings from a Kiuwan CSV vulnerabilities file. You can  use any sample available in the project django-DefectDojo/dojo/unittests/scans/kiuwan_sample/ or use the one below
List all findings, see the findings
Activate all or some findings in bulk
Go to the metrics screen for the product. See that the number of findings are not updated, but if you click on the link for Open findings it'll take you to the list of findings including the ones not shown in the counter.
The same happens if you go to the global dashboard, the newly imported findings are not there
See screenshots

Expected behavior
All counters in all dashboards refreshed taking into account
Deployment method (select with an X)

[ ] Kubernetes
[X] Docker
[ ] setup.bash / legacy-setup.bash

Environment information

Operating System: [e.g. Ubuntu 18.04]
DefectDojo version: [use git show -s --format="%h: %s [%ci]"]
1.5.4

Sample scan files (optional)
If applicable, add sample scan files to help reproduce your problem.
2019-05-22_Kiuwan_VulnerabilitiesTable_rfc_4180.zip
Screenshots (optional)
If applicable, add screenshots to help explain your problem.

Console logs (optional)
If applicable, add console logs to help explain your problem.
Additional context (optional)
Add any other context about the problem here.

@jsalado are the findings also set to verified?

@aaronweaver I see I've set the status as Active AND Verified and now the dashboards show the information.
I see now that the import screen allows you to check both statuses, so they can automatically be created with both set. This obvious ly will depend on the process every user wants to implement and the level of confidence in Kiuwan results ;-)
Thanks. All clear now, I'm closing the issue.

@jsalado I think I'll add some text on the screen to indicate that it's only showing active and verified.


Prevent page flash

When loading a page on a static site, even if it is set to the dark theme, it becomes the light theme for a moment.
This can be difficult to debug because it only occurs when pages take a long time to fully serve. Please wait for an update as I am working on a fix.

Ah...It looks not resolved...Why???


Download as CSV is not working correctly.

We are using the apex blazor wrapper in our website.
One of the clients just notified us that the download as CSV option is not working correctly.|

You will see the issue is:
First of all the CSV is not properly breaking up the columns by the comma "," .  In our previous systems, we used to put sep=,   on the first line of the CSV and this fixed the issue.
Secondly the date is only showing the date, not the time.

Hi,
Please take a look at https://apexcharts.github.io/Blazor-ApexCharts/features/export#csv formatter
Fixed in v0.9.12-beta

Hi,
Please take a look at https://apexcharts.github.io/Blazor-ApexCharts/features/export#csv formatter
Fixed in v0.9.12-beta

Thank you very much.  This is EXACTLY what we needed.  I have literally had so many phone calls this week moaning about the csv,  this fix worked perfectly and your documentation was perfect.

Great that the issue was resolved.
Please do not forget to star the repo if you like it..


Remove unnecessary finalizers

Remove unnecessary finalizers which would end up in the finalizer queue, doing unnecessary work.
Objects with finalizers will survive at least one garbage collection and get promoted to a higher generation since the GC has to wait for their finalizers to get executed before reclaiming their memory. This means that they need at least 2 garbage collection before their memory is reclaimed which is especially bad for PageBuffer since it maintains a reference to a large byte[] object.
I did a simple test on a relatively large dataset (same dataset generated by LiteDB.Demo, that is, 1 million documents) and ran col.FindAll().Count() on it. It registered 29001 PageBuffer objects in the finalizer queue which is unnecessary, since PageBuffer does not do any meaningful work in its finalizer.
Before PR:
29001 PageBuffer objects registered and some other unnecessary ones by LiteDB.

After PR:

Potential fix for #1345.

Hi @FICTURE7, thanks for this great job. I saw you tests and is amazing. The only #if you keep it right: it's for debug proposes (on PageBuffer)
Thanks again!


Refactor data uri generation and remove space in generated string

What's this PR do?
Removes a space in the data uri that is generated for uploaded image. This was causing errors in rogue when trying to edit the image because the library we are using to do cropping and rotating in rogue could not read the malformed uri.
Also refactors how we generate these URIs. Before, there was one function that took in an image object but we also needed it to work for file objects. This generalizes the helper function so that we can use one function to generate a base64 encoded data uri instead of doing it two places.
Relevant tickets
Fixes https://github.com/DoSomething/rogue/issues/61

🚢


Fixes #82

Simple change for columns aggConfig construction.

Can you link it to #82?

Hi @mordekasg,
First, thanks for this PR!
Then, I just linked the PR to issue #82

I agree to merge it as is.
Can you just confirm me that you tested it with percentiles aggregation, and test was OK?

I agree to merge it as is.
Can you just confirm me that you tested it with percentiles aggregation, and test was OK?

Yes, done it and it`s working fine.

I added fix for #81 (tested) and #79 (not tested).

Nice!
LGTM ;)


Support for recurrent neural network training data

It seems that the Scan class assumes the input is two dimensional (m,n) for number of samples by number of features, and targets are (m,c) for number of samples by number of labels. I can't figure out how to use this for recurrent data, where the Keras models like LSTM assume your input is 3d: (m, t, c) for number of samples by number of timesteps by number of channels. Is there a way in the existing framework that this kind of data can be accommodated?

Can you share the Keras model you want to use and a little bit of the data.

I am also using talos with LSTM. Following the suggestions from #11 I try to input dummy data to Scan() and then use a data generator with fit_generator in the model function which accesses the generators directly from a common scope. This is my code:
_BATCH_SIZE = 4
_SET_SIZE = 20
_CHANNELS, _IMG_WIDTH, _IMG_HEIGHT = (3, 192, 192)
train_generator = create_generator("/path/to/data",
                                   batch_size=_BATCH_SIZE,
                                   set_size=_SET_SIZE,
                                   img_dim=(_CHANNELS, _IMG_WIDTH, _IMG_HEIGHT))  # Channels first!
def CNN_LSTM(x, y, x_val, y_val, params):
    [...]
    optimizer = optimizers.adam(lr_normalizer(params['lr'], optimizers.adam))
    model.compile(loss=params['losses'], optimizer=optimizer, metrics=['mae'])

    #print(model.summary())

    out = model.fit_generator(generator=train_generator, validation_data=val_generator,
                              epochs=params['epochs'], callbacks=[live()])

    return out, model

dummy_x = np.empty((1, _SET_SIZE, _CHANNELS, _IMG_WIDTH, _IMG_HEIGHT))
dummy_y = np.empty((1, _SET_SIZE))
scan = ta.Scan(x=dummy_x,
               y=dummy_y,
               model=CNN_LSTM,
               params=params,
               grid_downsample=0.1,
               dataset_name='img_data',
               experiment_no='1')

ta.Deploy(scan, 'experiment_lstm_1')

I am still debugging (the code above runs though), therefore cannot confirm that the deployed scan will be any good, but maybe this helps.

When you deploy, you have to do talos.Deploy( ... metric='mae' ... ) because you don't have 'val_acc' in your model.

Closing this and creating a new issue for removing the default value 'val_acc' so user has to choose.

I have an update for my code: metric='mae' gives you the same error as val_acc, you have to spell out 'val_mean_absolute_error'. Deploy then saves details.txt, model.h5 and model.json, but then throws an error. Here is my Deploy() command and the error:
ta.Deploy(scan, 'experiment_lstm_1', metric='val_mean_absolute_error', asc=True)

Deploy package talos_one_subject_test have been saved.
Traceback (most recent call last):
  File "lstm_talos.py", line 110, in <module>
    ta.Deploy(scan, 'experiment_lstm_1', metric='val_mean_absolute_error', asc=True)
  File "/home/pia/anaconda3/envs/venv_cuda/lib/python3.6/site-packages/talos/commands/deploy.py", line 34, in __init__
    self.save_data()
  File "/home/pia/anaconda3/envs/venv_cuda/lib/python3.6/site-packages/talos/commands/deploy.py", line 60, in save_data
    x = pd.DataFrame(self.scan_object.x[:100])
  File "/home/pia/anaconda3/envs/venv_cuda/lib/python3.6/site-packages/pandas/core/frame.py", line 424, in __init__
    copy=copy)
  File "/home/pia/anaconda3/envs/venv_cuda/lib/python3.6/site-packages/pandas/core/internals/construction.py", line 146, in init_ndarray
    values = prep_ndarray(values, copy=copy)
  File "/home/pia/anaconda3/envs/venv_cuda/lib/python3.6/site-packages/pandas/core/internals/construction.py", line 249, in prep_ndarray
    raise ValueError('Must pass 2-d input')
ValueError: Must pass 2-d input

Is this error related to passing dummy data instead of the dataset?

Sorry I missed this as the ticket had been closed. There was an issue with the Deploy codes that were causing this in all cases where x was not 2d. This is now fixed in v.0.6 onwards. You can access it with:
pip install git+http://github.com/autonomio/talos@daily-dev


Expected "http://localhost:xxxxx"; Actual: about:blank

Problem Statement
So, this is my first time using SpecBind. I read all the documentation, looked at the examples in the documentation and the full sample used here at the site, and finally proceeded to write my first feature spec that simply navigates to the Home page of a new ASP.Net MVC project and find the DIV element whose class attribute equals jumbotron. That's it. Very simple. But, it doesn't work.
Problem Setup
Below is the code in the test project that was used to produce the behavior.
HomePage.cs
[PageNavigation("/")]
public class HomePage : HtmlDocument
{
    public HomePage(UITestControl parent) : base(parent) { }

    [ElementLocator(Class = "jumbotron")]
    public HtmlDiv Jumbotron { get; set; }
}

UnderstandingSpecBind.feature
Feature: UnderstandingSpecBind
    In order to understand how SpecBind works
    As a developer
    I want to test that I can find the
 element on the Home page.

Scenario: Find the HTML element '
' element on the Home page.
    Given I navigated to the {Home} page
    When I am on the {Home} page
    Then there should be a div whose class attribute is jumbotron

StepDefinitions/UnderstandingSpecBind.cs
[Binding]
public sealed class UnderstandingSpecBind
{
    [Then(@"there should be a div whose class attribute is jumbotron")]
    public void ThenThereShouldBeADivWhoseClassAttributeIsJumbotron()
    {
        ScenarioContext.Current.Pending();
    }
}

What I Expected to Happen
I expected that the Given step would navigate the browser to the site's home page. After reading the SpecBind documentation, I understand that these steps use the past tense to signify that, to the reader's perspective, the action has already occurred. Accordingly, the browser navigates to my sites' home page.
I then expected that a verification (i.e. some sort of assertion) would occur to ensure that I am actually on my site's Home page when the When step executes.
Finally, I expected that I would get an inconclusive result for the Then step since I hadn't provided a definition for that step yet.
What Actually Happened
The test failed while processing the When step. Here's the output from the test runner:
Test Failed - FindTheHTMLElementDivClassJumbotronOnTheHomePage_
Message: Test method UnderstandingSpecBindFeature.FindTheHTMLElementDivClassJumbotronOnTheHomePage_ throw exception:
SpecBind.Pages.PageNavigationException: Cannot navigate to HomePage, Url: http://localhost:14512/, Actual: about:blank

If I comment out the When step in my feature file and re-run the test, then "it passes" (actually, it's inconculsive because of the ScenarioContext.Current.Pending(); in my Then step definition, but that's what I originally expected to occur as specified above).
Additional Findings
I know that for SpecFlow (or should I say Gherkin), Given and When are just keywords to help the Gherkin parser--it doesn't matter which order they appear in. So I thought, what if I omit the Given step in my feature file above, and only use the When step:
When I am on the {Home} page
Then there should be a div whose class attribute is jumbotron

After running this, however, the test fails again with a similar failure as above, but slightly different enough to make me think there may be a bug in the When step definition:
Test Failed - FindTheHTMLElementDivClassJumbotronOnTheHomePage_
Message: Test method UnderstandingSpecBindFeature.FindTheHTMLElementDivClassJumbotronOnTheHomePage_ throw exception:
SpecBind.Pages.PageNavigationException: Cannot navigate to HomePage, Url: http://localhost:14512/, Actual: http://www.msn.com/

Notice this time the Actual argument value: the home page I currently have set for Internet Explorer (not about:blank as in my original test with both a Given and When). In the test above, should I have expected it to fail, I would have expected it to fail in the same exact way. So why is the behavior different?

Took a look in the source code for the Given and When steps in question.
Firstly, I noticed that the same method is used for both Given I navigated to the (.+) page and When I navigate to the (.+) page steps. The method causes the browser to navigate to the requested page. Given the example feature file in my original issue post above, I would expect that the browser would, upon executing the Given step, navigate to the specified page, which it does. Then I would expect that the When step would navigate to the same page again, but it doesn't--or, at least, it tries, but the browser's location becomes about:blank instead of the expected page.
Even though I realize now that I don't need to specify both the Given and When steps that I have in my feature file, it still should have worked, even if its inefficient.

Hi, thanks for the really complete report! I know we fixed a bug in the past with navigation in the page, but it's a little difficult to tell if the navigation step is the problem. Here's a few things you can try:
(this won't fix it but it will complete your final step without code)
Then I see
         | Field     | Rule   | Value   |
         | Jumbotron | exists | true    |

The navigation code is optimized to no re-run navigation if it thinks it's already on that page which it may be doing incorrectly at this point. The way to check this is to add an appsetting to print out additional debug logging while running. Add this to the scenario and run it again. In the test output you should see what page it expects to be on and what the actual is:
<specflow>
   (other xml here)
   <trace traceSuccessfulSteps="true"/>
</specflow>

@dpiessens Thanks for the tip on the Then step!! Also, I think it's great that SpecBind tries to optimize unnecessary navigation. What's troubling, however, is that in my original spec with the Given, When, and Then steps, after executing the When step, the browser location appears (to the test runner) to be about:blank (the location shown in the browser itself did not change). Also, the fact that these two specs are not equivalent (when, by looking at the code it appears they should be) is a bit concerning, too:
Given I have navigated to the {Home} page
Then I see
    | Field     | Rule   | Value |
    | Jumbotron | exists | true  |

# The test below is not equivalent to the one above; in fact, it fails!! Is this expected behavior?
When I navigate to the {Home} page
Then I see
    | Field     | Rule   | Value |
    | Jumbotron | exists | true  |

I should mention that the browser in use during these tests was IE11 with latest updates (since I was using the CodedUITest SpecBind extension). I didn't try these tests using a different browser with the Selenium Web Driver extension for SpecBind.

I haven't tried any debugging with IE11 so that could be the problem. I'm also not sure is MS has tested all this yet as it's had bugs with CodedUI and new browsers in the past. If I have some time I'll run some tests.

Ah, I hadn't heard the Coded UI Test Tools were buggy. IE11 is supposedly supported by them according to MSDN; and, browsers change so fast, I can only imagine how hard it is to make sure testing against the "latest" is anything but easy. Heck, even MS doesn't release updates for CodedUITests for the latest versions of FF and Chrome that seem to ship nearly every other week.
If I get some time, I'll fork this project and see if I can find anything that may be causing this behavior.
Thanks again for the replies.

I think you're just missing the startUrl in your app.config as documented in Configuration.
For example:
<?xml version="1.0" encoding="utf-8"?>
<configuration>
  <configSections>

</configSections>
  <specFlow>
    <stepAssemblies>
      <stepAssembly assembly="SpecBind" />
    </stepAssemblies>
    <plugins>
      <add name="SpecBindGeneratorPlugin" type="Generator" />
    </plugins>
    <unitTestProvider name="MsTest" />
  </specFlow>
  <specBind>
    <browserFactory provider="SpecBind.CodedUI.CodedUIBrowserFactory, SpecBind.CodedUI" browserType="IE" />
    <application startUrl="http://localhost/MyAspNetWebSite" />
  </specBind>
</configuration>


Add a payment_state field to Visit

Add a payment_state field to Visit that contains (for starters) two states: Clear and Pending Payment. Clear would be the default state.
Then modify the following screens:

[ ] Don't show Outpatient Visits with a Visit payment_state of Pending Payment
[ ] Don't show Imaging Requests with a Visit payment_state of Pending Payment
[ ] Don't show Lab Requests with a Visit payment_state of Pending Payment
[ ] Don't show Medication Requests with a Visit payment_state of Pending Payment

@tangollama working on this


Change base 64 encoded GIFs to SVGs

Closes #23185
For reference, an example of the decoded SVG is as follows:
<svg width="1" height="1" xmlns="http://www.w3.org/2000/svg"><path fill="#00dc83" d="M0 0h1v1H0z"/></svg>

Minified using SVGOMG: https://github.com/jakearchibald/svgomg

Thanks for the patch!
But, you shouldn't base64 encode the SVGs. That being said, we need to make sure some characters are escaped otherwise FF can have issues.

Also data:image/svg+xml;base64,DQo8c3ZnIHdpZHRoPSIxIiBoZWlnaHQ9IjEiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyI+PHBhdGggZmlsbD0iIzc3NyIgZD0iTTAgMGgxdjFIMHoiLz48L3N2Zz4= has a linebreak at the start of the file.

@XhmikosR You can blame 36 hours of no sleep for that one! I'll change it to use SVG elements isntead

Also, we might as well have 2 instances for proper aspect ratio. It's worth looking into it.

What's the difference in size between the two formats?

@mdo encoded they have about twice the char length, amounts to about an additional 16 bytes per svg. I would guess that this number would decrease if they weren't encoded

Yup, that is why I said you shouldn't base64 encode the SVGs :)
Without that, it should be even shorter and more flexible with custom width/height etc.

@tom-sherman: any updates?

@XhmikosR changed to inline SVGs

@tom-sherman: do you plan to take care of the feedback above?

Closing this since we are going to go with another solution.


switch from target:server to target:serverless from (npm-)scripts

Feature request
Is your feature request related to a problem? Please describe.
I'm using "target:server" for development. I'm also testing "target:serverless" with serverless framework, and right now, i'm switching manually target:server to serverless manually.
Is there a way to switch that automatically ?

I'm using custom server (server/index and server/lambda), so maybe, there is a way to load custom next.config to change conf on the fly ?
something like :
server/index.ts

import config from 'next.config.js'
const conf = { ...config, target: 'server' };
const app = next({ dev, conf });

if so, right now, its not working or I'm doing something wrong.
(Maybe similar to https://github.com/zeit/next.js/issues/6219)

You can use phases like: https://github.com/zeit/next.js#custom-configuration


feat: Bump Firebase Web SDK version to 8.10.1 (CVE-2022-0235) for security patch purposes.

Description
There is a security vulnerability in version 8.10.0: https://nvd.nist.gov/vuln/detail/CVE-2022-0235
There is a security patch from Firebase in web SDK version 8.10.1 that fixes this vulnerability.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (melos run analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

@Salakar @russellwheatley when are plugins updated to include fixes like this one?

Release going out in a few hours


Null vs absent fields

Sometimes there's a difference between a null field and an absent field.
http://spaceapi.net/documentation
As an example, the sensors.people_now_present.names array may be absent, but if present it should contain at least 1 entry. (I think that's a bug though, should probably be reported to be fixed in a future API version.)
Also, state.open (which is stated to have the Array type, probably another bug) can be true (open), false (closed), null (we don't know, e.g. when the presence counter is down) and absent (when we don't tell).
In the end, we could probably get some of the details fixed in the next API version and use null values everywhere. But maybe there are still other places where null is not the same as the absence of the field.
Any ideas / opinions? @rnestler?

Reported both issues:

https://github.com/SpaceApi/spaceapiapp_documentation/commit/3c322db4dacdea1b7f5bf8d8e0306cd48c186b42#commitcomment-11054369
https://github.com/SpaceApi/SchemaValidator/issues/7

I think for optional fields the Option<T> is the most natural thing.

Depends on the implementation. Should it? Probably, yes.
Some implementations might think "if there's a field there must be a value", but then the implementations are broken.

Quickly reviewed the specs and indeed, the documentation isn't consistent with the defined types.
"state":{
      ...
      "properties":{
        "open":{
          ...
          "type":[
            "boolean",
            "null"
          ]
        }
}

Basically yes, it looks like a bug in the documentation generator.

@slopjong Regarding the API in general, can we assume that a missing field should semantically be the same as a field set to null?

The validator won't accept them in general.

Is that an API specification issue or a validator issue?

It's more like a JSON schema feature. If a non-required field is present it must conform to the specified type. And validators implementing the the JSON schema guidelines will all behave like that.

Alright, then we have to find a better solution than setting the fields to null :)

(@slopjong: Lists should still be allowed to be empty though.)

According to today's discussion we'll simply implement the ToJson trait for all structs containing custom types.

Fixed by PR #21.


Update Voice Assistant Test Tool to 1. allow setting up input Utterance, WavFile, and Activity in same one config file, 2. use an optional flag to specify which input should be used for testing. The optional flag could be effective at app/dialog/turn levels.

From Fan: Update Voice Assistant Test Tool to 1. allow setting up input Utterance, WavFile  in one config file, 2. use an optional flag to specify which input should be used for testing. The optional flag could be effective at app/dialog/turn levels.


Fix exit code and replicaset panic

address #9 , #10
I addressed issues at the same time since a fix of #10  is related to #9 .
If separating PR is required, please let me know.
Descriptions

chage test directory to tests to match Makefile and README.md
check a condition for each resource in the same workload
add checking a condition of daemonset
support ReplicaSet to avoid panic

there are several ways to avoid it e.g.) skip replicaset check, fallback to the pod handling, but I think adding it looks simple and useful.

Thanks so much for the PR and open issues. I'll test this next week.


When using the checkbox, ivh-treeview-on-item-selected has no effect

I added a callback event to ivh-treeview-on-item-selected, but did not enter the callback event when I clicked on the node.

The following is my callback method:

I think maybe you want ivh-treeview-on-cb-change? I don't recognize ivh-treeview-on-item-selected.
https://github.com/iVantage/angular-ivh-treeview#selectdeselect-handlers
Also, just FYI I also don't recognize ivh-treeview-use-img-icon.

Ok, thank you


Fork memory

Create a copy of the memory at a certain time index. The copy can be continued independently of the original memory.

This needs further discussion: What exactly should be copied? One option would be an eager copy of the process, which could be shallow (including side effects on the original process) or deep (very expensive). Another option would be a lazy copy via something like SimulationStudio's Sandbox, which would be slower and have unclear proceed semantics. In the latter case, there would also be an engineering issue to find out whether tracing simulators and sandbox simulators can be plugged together as simply as the Simulator CoR is designed.


SwiftUI Updated -> Drawing and graphics: GeometryReader3D [BETA]

Doc: https://developer.apple.com/documentation/SwiftUI/GeometryReader3D

According to @carson-katri we won't implement this


Support for oauth openid static keys endpoint

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
If the provider could support enabling and disabling of staticJwksEnabled
/pf-admin-api/v1/keyPairs/oauthOpenIdConnect

New or Affected Resource(s)

pingfederate_9.x >
suggested pingfederate_keypairs_oauth_openid_connect

Potential Terraform Configuration

resource "pingfederate_keypairs_oauth_openid_connect" "static_key" {

  staticJwksEnabled = true

}

I think this would be a very good resource to be included.


Transform Cognito Events to a Permission with SourceArn

Issue #, if available: Fixes #1253.
Description of changes:
When the AWS::Lambda::Permission resource was created for a Cognito event, it used the EventSourceToken property. This caused authentication (or, in my case, pre-authentication) events to fail to execute "due to error AccessDeniedException". This PR changes that to the SourceArn property.
Description of how you validated changes:
I deployed a transformed template of one of my applications and attempted to authenticate. I succeeded. 🎉
Checklist:

[x] Write/update tests
[x] make pr passes
[ ] ~Update documentation~
[x] Verify transformed template deploys and application functions as expected
[ ] ~Add/update example to examples/2016-10-31~

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1260 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1260      +/-   ##
===========================================
+ Coverage    94.48%   94.49%   +<.01%
===========================================
  Files           72       72
  Lines         3738     3739       +1
  Branches       740      740
===========================================
+ Hits          3532     3533       +1
  Misses         104      104
  Partials       102      102

Impacted Files
Coverage Δ

samtranslator/model/eventsources/push.py
89.83% <100%> (+0.02%)
: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 3725f00...4c57d11. Read the comment docs.


Fix issue with merge process and output labels (Merged and MergedError)

Fixes #10592
Status
not-tested
Description
This solution avoids tweaking both Merged and MergedError output labels, but only one of them, depending on whether --useErrorDataset was used or not when creating the process.
Is it backward compatible (if not, which system it affects?)
YES

@amaltaro I just made the requested changes. Let me know if you have any other request or question or I can squash the commits otherwise ;)

Update: This was tested and it worked.
There were additional errors after this, but nothing that seems related to PSet tweak changed (logcollect errors trying to write to CASTOR), which is described hr: #10110
@amaltaro Any objections in merging this ?

@amaltaro I made the changes (waiting for jenkins to finish)

@amaltaro Alright! Done.

Thank you very much for the prompt fixes, Kenyi.


where to find configuration documentation

Just discovered SLRIA.
Aim is to extract from one PDF the references (43).
Paper has no DOI.
Where can I find documentation on the config file, esp. the request section?
Thanks!
[Request] r0 = "Personal Identification with PPG" r1 = "Personal recognition with PPG" r2 = "Signature with PPG" r3 = "biometric identification with photoplethysmography" r4 = "Personal Identification with photoplethysmography"

Hello,
The request section is only here to help you organize the folder of your SLR. You simply enter all the requests you want to make to your search engine. If you only want to extract the references of one paper, SLRIA may not be appropriate. It is designed to extract, filter and merge all the references of several papers at the same time. If you really want to use this software to extract the references of one paper, just keep the request r0 and use any phrase you want to organize (like "paperXXX"). It will create a folder with that name and then follow the rest of the instructions in the readme file.
Have a nice day.

Thank you very much for the quick answer. I will try to understand the tool.


How I can get the timeline of a match?

Dear developers,
As a title, How I can get the timeline of a match?
I could not find out a method to call a URL as "/lol/match/v3/timelines/by-match/{matchId}"

You can use a_match.getTimeline().

@jjmaldonis That`s a just Timeline, not MatchTimeline. How can I use MatchTimeline?

Timeline is the core-level user-facing type that should be used to access information from "/lol/match/v3/timelines/by-match/{matchId}". MatchTimeline is a data container used internally to support Timeline. If you're using the core types in the rest of your code with Orianna, you definitely want Timeline rather than MatchTimeline.
You can also get a Timeline using Timeline.withId(matchId).get() in addition to match.getTimeline().
If you're sure you want to use the raw dto-level data containers that are used internally instead of the core Orianna API, there's no convenient user-facing API for getting that data. Let me know if that's the case and I can give you some details on how you would do that.


benchmark OpenHFT

For the benchmark, I wouldn't create additional implementations of Hash since that would change from the current monomorphic calls to bimorphic

For the benchmark, I wouldn't create additional implementations of Hash since that would change from the current monomorphic calls to bimorphic

No performance improvements.

No performance improvements.


'str' object has no attribute 'decode'

Dear Sir,
I'm having this issue when I tried to run the pretrained weights or my owned trained weights.
'AttributeError: 'str' object has no attribute 'decode'
Is the error I finally getting. Your advice would be much appreciated.
Thanks
Loading pre-trained weights in svhn/weights.h5
Traceback (most recent call last):
File "train.py", line 62, in
yolo.load_weights(config['pretrained']['full'], by_name=True)
File "/home/hasantha/Documents/Yolo-digit-detector-master/yolo/frontend.py", line 71, in load_weights
self._yolo_network.load_weights(weight_path, by_name=by_name)
File "/home/hasantha/Documents/Yolo-digit-detector-master/yolo/backend/network.py", line 58, in load_weights
self._model.load_weights(weight_path, by_name=by_name)
File "/home/hasantha/anaconda3/envs/yolo/lib/python3.6/site-packages/keras/engine/topology.py", line 2620, in load_weights
load_weights_from_hdf5_group_by_name(f, self.layers)
File "/home/hasantha/anaconda3/envs/yolo/lib/python3.6/site-packages/keras/engine/topology.py", line 3161, in load_weights_from_hdf5_group_by_name
original_keras_version = f.attrs['keras_version'].decode('utf8')
AttributeError: 'str' object has no attribute 'decode'

I dropped the decode('utf-8') part but that gave me another bug.
You can see the link that has solution: https://stackoverflow.com/questions/28583565/str-object-has-no-attribute-decode-python-3-error
Now I am getting ValueError: Cannot feed value of shape (30, 2048, 1, 1) for Tensor 'Placeholder_318:0', which has shape '(1, 1, 2048, 75)'
Thanks

Thanks for the help. Yes I have tried that too. But it gives that error code as you mentioned . But I think this will help for some extent.
https://www.kaggle.com/tustunkok/license-plate-detection
This repository describes a similar approach for localizing objects from regular cnn. ( I couldn't achieve this btw, hope you could figure this easily)

I've solved it, thanks to
https://github.com/keras-team/keras/issues/14265#issuecomment-720376773


feat(plugin-webpack): Package all webpack externals and their dependencies

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[ ] The changes are appropriately documented (if applicable).
[ ] The changes have sufficient test coverage (if applicable).
[ ] The testsuite passes successfully on my local machine (if applicable).

This brings the functionality of this plugin and enables it by default.

Collects Webpack externals from both main and renderer configs
Finds all their dependencies using flora-colossus
Ensures the correct directories in node_modules get included in the packaged app

This makes it particularly easy to get native modules working if they don't work well with Webpack. You just add them to Webpack externals and everything should work.
Notes:

This doesn't work with monorepos with hoisted modules
It currently uses private fields and methods of flora-colossus
Not yet tested in combination with Vercel loader

This PR is not finished and I haven't got time to finish at the moment.
Anyone else is welcome to take the reins though!


implemented multiple theme suppport

Hi @adarshzpatel
This is something next level. I didn't expect that this repo would get these next-level contributions. Thanks a lot!
But there is just 1 issue, can you please go through it:
Consider this scenario, someone creates a birthday wish and then shares the URL to send to them. Now, the default color is blue, and there is no way one can define the color without changing any parameter in the URL.
I would love to see something like this: The URL scheme can be changed for betterment in this scenario, right now it is hbdwish.ml/<name>. Can you do something like hbdwish.ml/<name>/<color-int> where color-int is the integer that defines the color that the theme should use.
Color indexes:
none for Blue, just the URL hbdwish.ml/<name> should mean blue colour
1 for green, hbdwish.ml/<name>/2 means green theme
similarly,
2 for purple
3 for yellow
and 4 for red.
Thanks once again! Really appreciated

Also, on the home page, by default, there is no shadow over the blue circle. It would be nice if it has the shadow by default :)

Yeah thanks for pointing it out . I forgot to take it into consideration that the  color would be reset when shared.
I will work on the url solution that you have suggested.

Amazing!

Implemented what you said , now you can use themes from url itself and the GO button changes to url if the theme is changed.

This looks great, but I was thinking if it could be changed from hbdwish.ml/<name>?colour=<number> to just hbdwish.ml/<name>/<number>
Is that possible? If yes that would look more professional and better!

Hey @gouravkhunger I looked into the official Next.js documentation and found a solution and implemented it .
Major Changes : -

Changed [name].js to [...name].js to catch multiple routes
Now name variable is an array which has 'name' in its 0th index and color id in 1st Index.

Thanks a lot, this was a really great feature!! Really really appreciated :)

No problem , this was a great learning experience for me as well ,before this I didn't know much next.js routing & theming but now small project of yours helped me to learn as well implement it .

Amazing! Even I am surprised that a small project of mine gained enough traction and now its becoming a really cool app. Happy to work with you :)


Having kill terminal set to true combined with command + shift + j does not open file at all

Like the title says having kill terminal set to true combined with command + shift + j does not open a file at all

Thanks for reporting! I managed to reproduce. Version 0.0.29 is now released and this is should be fixed.


"USING AUTO REPLY... keep showing until it match max_consecutive_auto_reply settings

My Question:  I've successfully get the code generated by Autogen, but the ">>>Using AUTO REPLY..." just keep showing on the terminal 10 times.  Until the end, I have to type "exit" to stop the code exectution.
My code:
# use 'pip install pyautogen' instad of 'pip install autogen'!!
import autogen
import OPENAI_KEY

config_list = [
    {
        'model': 'gpt-3.5-turbo-16k-0613',
        'api_key': OPENAI_KEY.myKey
    }
]

llm_config={
    "request_timeout": 600,
    "seed": 42, #for catching to saving money
    "config_list": config_list,
    "temperature": 0
}

assistant = autogen.AssistantAgent(
    name="assistant",
    llm_config = llm_config,
    system_message = "Your loyal assistant!"
)

user_proxy = autogen.UserProxyAgent(
    name = "user_proxy",
    human_input_mode = "TERMINATE",
    max_consecutive_auto_reply = 10,
    is_termination_msg=lambda x: x.get("content", "").rstrip().endswith("TERMINATE"),
    code_execution_config={"work_dir": "D:\stone\Documents\python\ms-autogen\web"},
    llm_config=llm_config,
    system_message = """
    Reply TERMINATE if the task has been solved at full satisfaction.
    Otherwise, reply CONTINUE, or the reason why the task is not solved yet."""
)

task = """
Write me a python code that add odd numbers from 1 to 1000 and print out the result. After that, save the Python code in a file.
"""

user_proxy.initiate_chat(
    assistant,
    message=task,
)

TERMINAL Output:
`Write me a python code that add odd numbers from 1 to 1000 and print out the result. After that, save the Python code in a file.

assistant (to user_proxy):
Certainly! Here's the Python code that adds odd numbers from 1 to 1000 and prints out the result:
# Initialize the sum variable
sum_of_odd_numbers = 0

# Iterate through numbers from 1 to 1000
for num in range(1, 1001):
    # Check if the number is odd
    if num % 2 != 0:
        # Add the odd number to the sum
        sum_of_odd_numbers += num

# Print the result
print("The sum of odd numbers from 1 to 1000 is:", sum_of_odd_numbers)

# Save the Python code in a file
with open("sum_of_odd_numbers.py", "w") as file:
    file.write('''\
# Initialize the sum variable
sum_of_odd_numbers = 0

# Iterate through numbers from 1 to 1000
for num in range(1, 1001):
    # Check if the number is odd
    if num % 2 != 0:
        # Add the odd number to the sum
        sum_of_odd_numbers += num

# Print the result
print("The sum of odd numbers from 1 to 1000 is:", sum_of_odd_numbers)
''')

print("Python code saved in 'sum_of_odd_numbers.py' file.")

This code will calculate the sum of odd numbers from 1 to 1000 and print the result. It will also save the Python code in a file named "sum_of_odd_numbers.py".

USING AUTO REPLY...

EXECUTING CODE BLOCK 0 (inferred language is python)...
execute_code was called with use_docker evaluating to True, but the python docker package is not available. Falling back to native code execution. Note: this fallback behavior is subject to change
SIGALRM is not supported on Windows. No timeout will be enforced.
user_proxy (to assistant):

exitcode: 0 (execution succeeded)
Code output:
The sum of odd numbers from 1 to 1000 is: 250000
Python code saved in 'sum_of_odd_numbers.py' file.

assistant (to user_proxy):
Great! The code executed successfully and the output shows that the sum of odd numbers from 1 to 1000 is 250000. The Python code has also been saved in the "sum_of_odd_numbers.py" file. If you have any more questions or need further assistance, feel free to ask!

USING AUTO REPLY...
user_proxy (to assistant):

Great! I'm glad the code executed successfully and the output is as expected. If you have any more questions or need further assistance, feel free to ask. Otherwise, the task has been solved at full satisfaction.

assistant (to user_proxy):
I'm glad to hear that the task has been solved to your satisfaction! If you have any more questions or need assistance in the future, don't hesitate to ask. Have a great day!

USING AUTO REPLY...
user_proxy (to assistant):

Thank you! I'm glad I could help. If you have any more questions or need assistance in the future, feel free to reach out. Have a great day too!

assistant (to user_proxy):
Thank you! I appreciate your help and support. I'll definitely reach out if I have any more questions or need assistance. Have a wonderful day ahead!

 I'm always here to help. Feel free to reach out anytime you need assistance. Have a wonderful day ahead as well!

assistant (to user_proxy):
Thank you for your kind words! I appreciate your availability to assist. I'll definitely reach out whenever I need help. Have a fantastic day too!

USING AUTO REPLY...
user_proxy (to assistant):

Thank you for your kind words! I'm always here to assist you whenever you need help. Have a fantastic day as well!

assistant (to user_proxy):
Thank you once again! Your willingness to assist is greatly appreciated. I'll keep that in mind and reach out whenever I need help. Have a fantastic day too!

 I'm always here to assist you, so don't hesitate to reach out whenever you need help. Thank you for your kind words. Have a fantastic day too!

assistant (to user_proxy):
Thank you for your continuous support! I truly appreciate your availability to assist. I'll definitely reach out whenever I need help. Have a fantastic day as well!

USING AUTO REPLY...
user_proxy (to assistant):

Thank you for your kind words! I'm always here to support you, so please feel free to reach out whenever you need assistance. Have a fantastic day too!

assistant (to user_proxy):
Thank you for your kind words and support! I really appreciate it. I'll definitely reach out whenever I need assistance. Have a fantastic day as well!
 I'm glad I could assist you. Thank you for your kind words. Remember, I'm always here to help, so feel free to reach out whenever you need assistance. Have a fantastic day too!

assistant (to user_proxy):
Thank you for your kind words and support! I'm grateful for your assistance. I'll definitely reach out whenever I need help. Have a fantastic day as well!

Please give feedback to assistant. Press enter to skip and use auto-reply, or type 'exit' to stop the conversation: exit`

Hi @stonez56 , try this:
assistant = autogen.AssistantAgent(
    name="assistant",
    llm_config = llm_config,
    # system_message = "Your loyal assistant!"  # remove this line
)

@thinkall I just tried to remove the line and it worked. Thanks a lot!

@thinkall I just tried to remove the line and it worked. Thanks a lot!
However, If I input some text at "Please give feedback to assistant. Press enter or type 'exit' to stop the conversation prompt, the same repeating message will happen again.

I'm not sure what do you mean. If you mean you replaced system_message of assistant with "Please give fee...", then you shouldn't do it. The reason is that autogen.AssistantAgent has its own pre-defined prompt, if you replace it then the termination function should be updated accordingly. You can put any instructions in your user proxy agent's prompt.

@thinkall Yes, If I didn't reply anything at user_proxy_agent prompt and it worked fine.
However, If I replied a text at the user_proxy_agent prompt, the repeating message of USING AUTO REPLY...will repeat the max execution times I've set in the config.

A user proxy's llm_config should be set to False. @stonez56

@thinkall Just tried setting below, but it didn't work.
user_proxy = autogen.UserProxyAgent(
    name = "user_proxy",
    human_input_mode = "TERMINATE",
    max_consecutive_auto_reply = 10,
    is_termination_msg=lambda x: x.get("content", "").rstrip().endswith("TERMINATE"),
    code_execution_config={"work_dir": "D:\stone\Documents\python\ms-autogen\web"},
    llm_config=False,#this prevents AUTO REPLY...repeats many many times!
    system_message = """
    Reply TERMINATE if the task has been solved at full satisfaction.
    Otherwise, reply CONTINUE, or the reason why the task is not solved yet."""
)

Here is the repeated reply on my terminal.
Great! The workshop introduction has been successfully saved to the `workshop_intro.txt` file. You can now open the file to view the contents.

If you have any more questions or need further assistance, feel free to ask.

--------------------------------------------------------------------------------

>>>>>>>> USING AUTO REPLY...
user_proxy (to assistant):

--------------------------------------------------------------------------------
assistant (to user_proxy):

Is there anything else I can help you with? If you have any more questions or need further assistance, please let me know.

--------------------------------------------------------------------------------

>>>>>>>> USING AUTO REPLY...
user_proxy (to assistant):

--------------------------------------------------------------------------------
assistant (to user_proxy):

If you have any more questions or need further assistance in the future, feel free to ask. Have a great day!

--------------------------------------------------------------------------------

>>>>>>>> USING AUTO REPLY...
user_proxy (to assistant):

--------------------------------------------------------------------------------
assistant (to user_proxy):

Thank you! Have a great day too!

--------------------------------------------------------------------------------

>>>>>>>> USING AUTO REPLY...
user_proxy (to assistant):

--------------------------------------------------------------------------------
assistant (to user_proxy):

Thank you! I'm glad I could assist you. If you have any more questions or need help in the future, don't hesitate to ask. Have a great day!

--------------------------------------------------------------------------------

>>>>>>>> USING AUTO REPLY...
user_proxy (to assistant):

--------------------------------------------------------------------------------
assistant (to user_proxy):

You're welcome! I'm here to help. If you have any more questions or need assistance in the future, feel free to reach out. Have a fantastic day!

--------------------------------------------------------------------------------

>>>>>>>> USING AUTO REPLY...
user_proxy (to assistant):

--------------------------------------------------------------------------------
assistant (to user_proxy):

Thank you! I'm glad I could assist you. If you have any more questions or need help in the future, don't hesitate to ask. Have a great day!

--------------------------------------------------------------------------------

>>>>>>>> USING AUTO REPLY...
user_proxy (to assistant):

--------------------------------------------------------------------------------
assistant (to user_proxy):

You're welcome! I'm here to help. If you have any more questions or need assistance in the future, feel free to reach out. Have a fantastic day!

--------------------------------------------------------------------------------
Please give feedback to assistant. Press enter to skip and use auto-reply, or type 'exit' to stop the conversation:

Can you share the whole code so I can reproduce. Thanks.

@thinkall  Here is my complete code. Thank you!
# use 'pip install pyautogen' instad of 'pip install autogen'!!
import autogen
import OPENAI_KEY

config_list = [
    {
        'model': 'gpt-3.5-turbo-16k-0613',
        'api_key': OPENAI_KEY.myKey
    }
]

llm_config={
    "request_timeout": 600,
    "seed": 42, #for catching to saving money
    "config_list": config_list,
    "temperature": 0
}

assistant = autogen.AssistantAgent(
    name="assistant",
    llm_config = llm_config,
    system_message = "Your loyal assistant!"
)

user_proxy = autogen.UserProxyAgent(
    name = "user_proxy",
    human_input_mode = "TERMINATE",
    max_consecutive_auto_reply = 10,
    is_termination_msg=lambda x: x.get("content", "").rstrip().endswith("TERMINATE"),
    code_execution_config={"work_dir": "D:\stone\Documents\python\ms-autogen\web"},
    llm_config=llm_config,
    system_message = """
    Reply TERMINATE if the task has been solved at full satisfaction.
    Otherwise, reply CONTINUE, or the reason why the task is not solved yet."""
)

task = """
Write me a python code that add odd numbers from 1 to 1000 and print out the result. After that, save the Python code in a file.
"""

user_proxy.initiate_chat(
    assistant,
    message=task,
)

I met the same problem with similar code, actually totally the same.
I said :" Write a code that can print 1~20 and save the code in a .txt file"
but it shows this all the time :

USING AUTO REPLY...

EXECUTING CODE BLOCK 0 (inferred language is python)...
execute_code was called with use_docker evaluating to True, but the python docker package is not available. Falling back to native code execution. Note: this fallback behavior is subject to change

EXECUTING CODE BLOCK 1 (inferred language is unknown)...
user_proxy (to assistant):

exitcode: 1 (execution failed)
Code output:
2
3
5
7
11
13
17
19
unknown language unknown
why it cannot automatically execute python in my computer?

"save the code in a .txt file" may be confusing. Do you mean save the code as .py or save the results as .txt?

@thinkall  It worked!
I modified the system_message as you suggested above and the code worked! Thank you!


chore: move share encoding logic from core -> app

Description
This PR moves the share encoding code from core to the app. It makes as few changes as possible, as to make the upcoming refactor for non-interactive defaults actually reviewable.
closes: #623

Codecov Report

Merging #627 (8a25644) into main (24bbf82) will increase coverage by 8.75%.
The diff coverage is 75.10%.

:exclamation: Current head 8a25644 differs from pull request most recent head 421f233. Consider uploading reports for the commit 421f233 to get more accurate results

@@            Coverage Diff             @@
##             main     #627      +/-   ##
==========================================
+ Coverage   29.78%   38.53%   +8.75%
==========================================
  Files          16       19       +3
  Lines        2055     2543     +488
==========================================
+ Hits          612      980     +368
- Misses       1382     1478      +96
- Partials       61       85      +24

Impacted Files
Coverage Δ

pkg/shares/shares.go
50.00% <50.00%> (ø)

pkg/shares/share_splitting.go
75.00% <75.00%> (ø)

pkg/shares/share_merging.go
77.11% <77.11%> (ø)

x/payment/types/payfordata.go
77.30% <100.00%> (+1.95%)
:arrow_up:

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


CoLLAs 2023

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

if you run the code w/ file_args = yaml.safe_load(f, Loader=yaml.FullLoader) the error is:
Exception has occurred: TypeError
safe_load() got an unexpected keyword argument 'Loader'
  File "/Users/massimo/dev/rl_4_cl/code/misc/utils.py", line 253, in overwrite_args
    file_args = yaml.safe_load(f, Loader=yaml.FullLoader)
  File "/Users/massimo/dev/rl_4_cl/code/main.py", line 271, in <module>
    args = overwrite_args(args, "methods", args.method_config)
TypeError: safe_load() got an unexpected keyword argument 'Loader'

if you run the code w/ file_args = yaml.safe_load(f) the error is:
Exception has occurred: ConstructorError
could not determine a constructor for the tag 'tag:yaml.org,2002:python/object:argparse.Namespace'
  in "code/configs/methods/ER-ID.yaml", line 1, column 1
  File "/Users/massimo/dev/rl_4_cl/code/misc/utils.py", line 252, in overwrite_args
    file_args = yaml.safe_load(f)
  File "/Users/massimo/dev/rl_4_cl/code/main.py", line 271, in <module>
    args = overwrite_args(args, "methods", args.method_config)
yaml.constructor.ConstructorError: could not determine a constructor for the tag 'tag:yaml.org,2002:python/object:argparse.Namespace'
  in "code/configs/methods/ER-ID.yaml", line 1, column 1


Failed to carry op.Do(): maxOp

The MaxOp still doesn't work.
func main() {
g := NewGraph()
xt := tensor.New(tensor.Of(tensor.Float32), tensor.WithShape(5), tensor.WithBacking([]float32{0.1, 0.2, 0.3, 0.3, 0.1}))
x := NewVector(g, Float32, WithValue(xt))

max := Must(Max(x))
z := Must(Gte(x, max, true))

m := NewLispMachine(g, ExecuteFwdOnly(), WithManualGradient())
if err := m.RunAll(); err != nil {
	//RunAll: Running Node: >= true(%0, %1) :: Vector float32: Failed to execute MaxAlong[0] in
	//node MaxAlong[0](%0) :: float32: Failed to carry op.Do(): maxOp.
	fmt.Println(errors.Wrap(err, "RunAll"))
	return
}
fmt.Println("max of x: %v\n", z.Value())

}

The main problem with max is that it's not particularly differentiable. Do you have any suggestions?

Hi,
twitter pa-m here, author of github.com/pa-m/sklearn
I'm very impressed with the work you did on gorgonia. I'm  still in phase of understanding how I can help
I'm screwed with this max problem too.
maybe unrelated:
ml activation funcs have often been chosen because you can compute their diff from y. its a key to perfs. for intance
func (Logistic) Fprime(y float64) float64 { return y * (1 - y) }
not sure you're using this. its a key yto ML perfs
returning to max
there are several problems with this op

sometime you must be able to know node children values when you diff it.
there're a shape problem in type inference:

the following test says "Input shape is ()"
func TestMax(t *testing.T) {
dt := G.Float32
g := G.NewGraph()
Xn := G.NewTensor(g, dt, 2, G.WithShape(2, 2))
//max :=
G.Must(G.Max(Xn, 0))
}
Best regards
Pascal

The Max function is not complete... @accenta . I'll come back to complete it by v0.10.0

@accenta @chewxy thank you very much. come on.

The simplest thing to do is pass through the gradients of the largest element and forget about the differentiability of the max operation as such. More complex solutions could involve using a continuous approximation like the concrete distribution https://arxiv.org/pdf/1611.00712.pdf. The former is a good stop-gap measure (as opposed to just throwing an error), until something more advanced can be implemented.


Custom cursor interaction with specific GuiScreens

Pseudo code:
public interface GuiCursorCallback {
public Vector2f guiCursorCallback(Vector2f previousCallback, GuiScreen currentScreen, CursorController cursor) {
    if(currentScreen instanceof MyScreen) // Special code for a special screen
        if(cursor.hoverMode) return ...; // Return special hover code
        else                 return ...; // Special jump code
     else return previousCallback; // Give back the old callback, nothing else to do...
    }
}

This makes it easy for special Gui like the inventory screen to have custom interaction, like easier crafting or Gui specific button actions.

#21


Add qwestive-client-sdk npm pkg integration

Removed browser script integration
Added npm package integration

Thanks a lot @abhidtu2014 ! Going to close this and re-open in https://github.com/jbx-protocol/juice-interface/pull/3525 so we can get build passing


Inspector View

Hey, i was trying to create a PR for the inspector view but my problem is, as far as I understand the Ultralight API, that I cannot replace a view or create an overlay with a view using the C API.
Do you have any ideas? My idea would be to replace a pointer, but with ulOverlayGetView there is only one structure copy.
Current approach:
func (view *View) GetInspectorView() *View {
	return &View{
		v: C.ulViewCreateInspectorView(view.v),
	}
}

From the browser Example:
void Tab::ToggleInspector() {
  if (!inspector_overlay_) {
    inspector_overlay_ = Overlay::Create(ui_->window_, *view()->inspector(), 0, 0);
  } else {
    if (inspector_overlay_->is_hidden())
      inspector_overlay_->Show();
    else
      inspector_overlay_->Hide();
  }

  // Force resize to update layout
  Resize(container_width_, container_height_);
}

If there isn't any approach i already created an issue for the CAPI. https://github.com/ultralight-ux/Ultralight/issues/252

Hi,
Your understanding is correct. Unfortunately, as it is not currently possible to create an overlay from a view using the CAPI, there is no way to display the inspector view.
For this reason, I have omitted the creation of inspector views from this wrapper for the time being, aiming to add in the functionality when it becomes available.

Alright. Lets hope @adamjs will implement this to the CAPI. 👍


Cassandra 2.0.15

support cassandra 2.0.15

@colinkuo Thanks for the patch. Everything looks fine, but I am really hesitant to take new commits into the code base directly as I don't want to give anyone the impression that there is active maintenance on the project (there isn't).
I've meant to update the readme to reflect that - apologies it was there already.

@zznate Totally understand your thoughts. That makes sense to me. Just share this for someone is still using hector and is planning to upgrade C* to 2.0.15 like us.
Anyway, let me keep it in my fork for now.


The abstractions available to integrate with external services aren't at the right level; they should be higher with less code, less work to get them woven in

The abstractions available to integrate with external services such as LUIS, QnA Maker, Azure Search, etc., aren't at the right level. They should be higher with less code, less work to get them woven in, where the bot developer simply points to the desired service at the right point in the conversational flow, with an easy interface linking inputs and outputs to the flow. For e.g., submitting a question to QnA Maker and expecting a response should be a simple bot SDK call that abstracts the integration, sending a question to the service and expecting a text response.

We'd like you to review the coming 4.5 preview work we will show at //Build.  We have some new constructs that we believe may be helpful here.


[WIP] show error message

When we construct QFunction, and model parameter is not initialized (it happens when some Link is instantiated with in_size=None), copy_param will fail because the target_link's param is None.
This PR is to show user-friendly error message to let the user know what is the cause of this error.
I'd like to get comment that what kind of message is better.
I'm not sure the performance degrade by checking this type error.

Thanks. I agree with you about showing error message.
I'd like to see message that starts with "target_link has parameter {}", since target_params is a local name.

And should we add some sentence to how to fix this error like:
'target_link parameter {} is None. Maybe the model params are not initialized.\n Please forward dummy input before hand to determine parameter shape.'.format(param_name)
or something?


Change OmniAuth flash wording (authorize->authenticate)

This is probably really nitpicky, but I changed this in my app and thought it might be useful for others.

Thanks, I've merged the pull request.
But, before sending it, ensure that all tests pass :)


Gnome Extensions App DOes Not launch

More details on NixOS/nixpkgs#220690
I am unsure on the fix.

It looks like the postFixup phase from here:
https://github.com/NixOS/nixpkgs/blob/645bc49f34fa8eff95479f0345ff57e55b53437e/pkgs/desktops/gnome/core/gnome-shell/default.nix#L225-L230
Is being overridden here:
https://github.com/danth/stylix/blob/a1c5d01a40ca65335925974741020fe8b217b7bd/modules/gnome/nixos.nix#L27-L32
I'll try fixing this.