Use java module-info to improve visibility validation of java packages

During egeria development there have been a variety of cases where one java class has used another it rather shouldn't - breaking the intended 'architecture'.
Sometimes this is noticed when reviewing maven dependencies - especially as we now have additional checks.
Java 9 added modularity support to the language.
For example see the tutorial at https://www.baeldung.com/java-9-modularity
We could use this approach to more precisely define the provider/consumer relationship between our packages (or technically what we would now call modules)
Though egeria still has Java 8 as a minimum level, the package-info file will simply not have any effect in Java 8. But since our builds, including verification, require java 11 to pass also, we should benefit from the checks.
We may not benefit from any checks at runtime if using java 8, but this could bring us significant benefits.

This could give us extra architectural verification & spot misuse much earlier in the cycle than currently. This will probably reduce any follow on build/compatabilty/packaging issues. It could therefore be a useful investment, in particular at the inner layers of the stack @mandy-chessell

See also https://www.pluralsight.com/guides/getting-started-with-the-java-platform-module-system-part-3

Related tool (gradle) : https://github.com/jjohannes/extra-java-module-info

Also https://github.com/jjohannes/java-module-dependencies

YouTube video: https://www.youtube.com/watch?v=0xijKEeb5ug&list=PLPZy-hmwOdEWUR5OaOGs863rNS2DaIyXS&index=2

IntelliJ has support for adding module definitions (alternatively this can be done at the command line with jdeps).

Load up egeria in the latest verison of IntelliJ (v4, gradle build)
Ensure the latest java 17 compiler is being used, and the language level is set to 17
Select the top level egeria directory
Go to Menu->Code->Generate module-info descriptors

IntelliJ will then build, scan, reindex & create the module-info files based on the current packaging

Mostly clean, though notices this

It's important to bear in mind that the generated files reflect what is being used, and should then be reviewed for intent

A gradle build will then fail, for example:

We'll see a contextual fix option:

Running a build with ./gradlew clean build --no-build-cache --continue game 165 errors, but only across ~4 projects:

So we end up with lots of definitions such as:

So what this gives us:

protection against unexpected usage/changes in use
clarity on current dependencies and consumers
ability to review these contracts for correctness
potential pain in updating contracts (but could be a good thing)

See also:

https://www.oracle.com/uk/corporate/features/understanding-java-9-modules.html
https://medium.com/javarevisited/9-things-you-should-know-about-java-modules-43dd5fdeea6

An initial PR has been created for 'demonstration'. It includes more package renames then required, and inclusion should wait a proper review/agreement at our face to face meeting.
The code basically runs, but some more work is needed on test classes to complete the work

Agreed in face 2 face meeting Jan 2023 that this was something we should do, but we would not target 4.0. Rather we'd ensure the transition to java / gradle / v4 is smooth and understood first.
Module support can be added incrementally, and in any case remains under control of the consumer, so it is not a change requiring a major release.


Update Convolution parameters sequence

Updates code as proposed in
https://github.com/baidu-research/DeepBench/pull/51

In original version there was a typo in following config:
// std::make_tuple(151, 40, 1, 1, 32, 20, 5, 8, 8, 2, 8), ARM convolution kernels hang with this kernel
Since it is fixed in this PR, that config may work properly on ARM.

Closing this PR since these changes are now part of #51


Updated travis staged build following release meeting [Issue #109]

WHAT kind of change does this PR introduce?
...
HOW is this accomplished?
...
Checklist

[ ] Docs have been added / updated to reflect the changes
[ ] I have reviewed and tested the changes :white_check_mark:
[ ] I have squashed my commits to have a reasonable amount of commits
[ ] All of the code I have written adhere to the coding conventions outlined in the coding conventions section of the wiki

✅ Styling is adequate and follows coding conventions outlined by the Kotlin Foundation
✅ New methods and functionalities have appropriate tests (testing edge cases + good coverage)
✅ Branch has an acceptable amount of commits with descriptive names and
✅ PR description matches the code outlined a has mentions to issues
✅ This PR attempts to implement a specific feature (no code blob or epic pr)
Status: Approved

@mehranhassani updated it following the discussion in the release meeting


Not work iOS above 16.0.0

it doesn't work on newest iPhone with iOS above 16.0.0

Already enable vibration

@caturadjiprasetyo It is under "Sounds & Haptics" menu, at the bottom is "System Haptics" option

It worked, thank you so much @lpatrun


Introduce proper lifecycle-handling. Rework communication stack.

@jsangmeister Works best with https://github.com/OpenSlides/OpenSlides/pull/5487. You can start/stop the autoupdateservice with ./dc-dev.sh start autoupdate/./dc-dev.sh stop autoupdate to simulate a down server.


toolbar show/hide

toolbar.js:

[x] Open / close toolbar using the arrow button on the right
[x] Check that the django-cms logo redirects to homepage

:+1:


Little delays and Random lights when nothing is playing as well.

I have got this up and running and I can really see the led strip reacting to the music. But there are some lights showing up even when there is no song or sound playing in the PC. I will put up a youtube link as soon as possible. With that said sometimes there is a delay as well..
Is there any reason why it may be slow. I am running from a powerful PC with the GUI running in the PC as well.

Is the delay happening when processing or controlling the lights? As for the random lights, you can try upping the MIN_VOLUME_THRESHOLD in config.

By disabling the display of FPS am able to get a decent output. Not sure if the print is consuming some bit of time which adds on to the delay

Interesting. I've never thought of printing to console as a resource intensive task, but I guess it's significant enough to cause a delay. If fps is important to you, you could try lowering the frequency of its output. Just out of curiosity, what OS and terminal are you using, and what's your screen size/resolution?

I wouldn't have thought so as well. Mine is a Windows 10 pc running of i7
processor and 12 gigs of RAM.
On Aug 23, 2017 16:48, "nickmc1171"<EMAIL_ADDRESS>wrote:

Is the delay happening when processing or controlling the lights? As for
the random lights, you can try upping the MIN_VOLUME_THRESHOLD in config.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/scottlawsonbc/audio-reactive-led-strip/issues/68#issuecomment-324378596,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdWADcSy8hhBU5JzQ7N1fQW0ICVtRNeXks5sbEnngaJpZM4O-gNK
.

Yeah, that should be more than enough for this. Well, I hope you get it all sorted. I'm not home right now, but I can run some tests or something this weekend if it will help.

Yeah that would be great. I will set the mic threshold and see as well..

Ok cool. I set mine to 1e-4, but I'm sure it varies by pc and input device.

Tried that as well.. This is what happens....
https://www.youtube.com/edit?o=U&video_id=-rXyAfl9Co8

That's exactly what mine was doing before i upped the threshold. I'm also not using a microphone for audio in though. How familiar are you with python/coding? Can you step through the code and see what it's reporting as volume when there is no audio in?

I am very new to python, in fact only started looking into python because of this project. But am a good code builder, so if you let me know what i should do i may be able to do it.
With respect to your question, I tried putting the following print statement after the volume deriving formula
vol = np.max(np.abs(y_data))
print('Current Volume:', vol)
and the output was changing between 0.000183105 and 0.000213623

I will do another test and let you know in 12 hours. I just ran a sample again and can see the output even when the volume is below threshold

Ok. We should probably move this off of github so we don't clutter up the forum. If you want to give me your email I can start a Slack channel and invite you. Or you can start something and invite me if you don't want to post your email. Mine is [my github<EMAIL_ADDRESS>

Sure. Just mailed you...
On Aug 25, 2017 4:50 PM, "nickmc1171"<EMAIL_ADDRESS>wrote:

Ok. We should probably move this off of github so we don't clutter up the
forum. If you want to give me your email I can start a Slack channel and
invite you. Or you can start something and invite me if you don't want to
post your email. Mine is [my github<EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/scottlawsonbc/audio-reactive-led-strip/issues/68#issuecomment-324960492,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdWADbLcA1p0gHcsYZafbXrzrjdD9qcdks5sbu1CgaJpZM4O-gNK
.

Hello, i have the same issue decribe in the youtube video by  ramkithepower 👍
Same Os, same issue ...
Did you find the root cause ?

Yes it was the microphone volume
On Oct 11, 2017 14:18, "tettar"<EMAIL_ADDRESS>wrote:

Hello, i have the same issue decribe in the youtube video by ramkithepower
👍
Same Os, same issue ...
Did you find the root cause ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/scottlawsonbc/audio-reactive-led-strip/issues/68#issuecomment-335806250,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdWADSu9dD2GFxzo8B6ATBwPcu-AaQ5Cks5srMAWgaJpZM4O-gNK
.

thx  ramkithepower i set  up MIN_VOLUME_THRESHOL in config.py
MIN_VOLUME_THRESHOLD = 1e-2
I thought that the problem resulted from the background noise and that this could be filter in another way !!!
But like this, all is ok,
thx :)


Paste.py giving Undesired Result

Paste.py in the web3 sub directory giving additional config text when it actually works, Otherwise It Just works normally giving the resource unavailable message
The Undesired Text is shown in Bold Below
(blockboard) parzival979@Potluris-MacBook-Air web3 % node main.js
File clipboard_content.txt already exists
File clipboard_content.txt updated with contents: File clipboard_content.txt already exists
File clipboard_content.txt updated with contents: import subprocess
import pyperclip
def execute_command(command):
process = subprocess.Popen(command, stdout=subprocess.PIPE, shell=True)
output, error = process.communicate()
if error:
return error.decode(utf-8)
return output.decode(utf-8)
clipboard_data = pyperclip.paste()
print(clipboard_data)
if clipboard_data:
pyperclip.copy(execute_command("node main.js --text {text}".format(text = clipboard_data)))
stdout: Uploading 1 files
stdout: Content added with CID: bafybeihrf2yporm6ydibjsamovu4frivf5fpzqie3bd6iwkyozaetzgaem
child process exited with code 0
stdout: Uploading 1 files
stdout: Content added with CID: bafybeicdd3xkfu6wnzs4trypyspshzqyrdkccxuj53zyu4k25bmeybvsmy
child process exited with code 0

Created a better algorithm to manage copy and paste and solved the issue


Copy needed for migrate rep modal - HC links

We need two HC links for the rep migration modals

migrating v1 to v2 rep
general learn more about augur account address

two modals:
1

2

waiting for copy from @petervecchiarelli

@petervecchiarelli any updates? Can you ping me when the URL's are ready! thanks

links are in. closing


Error: add site dependencies: load resources: loading translations:

Language
bn
What's the issue?
Env:

linux,
hugo version: Hugo Static Site Generator v0.68.3/extended linux/amd64 BuildDate: 2020-03-25T06:15:45Z

Steps to reproduce:

hugo new site my-site
cd my-site
git init
git submodule add -b stable https://github.com/jpanther/congo.git themes/congo
update config file with theme="congo"
run: hugo
hugo generate Error (see below)
PS. Please note that when I run the same steps on Windows I see no issues.

Theme version
Stable & Dev
Hugo version
0.68.3
Relevant Hugo log output
Error: add site dependencies: load resources: loading translations: "/mnt/d/Repos/my-site/themes/congo/i18n/bn.yaml:1:1": failed to load translations: unable to parse translation #0 because invalid plural category externalurl_title
map[id:list translation:map[externalurl_title:অন্য ওয়েবসাইটের লিংক no_articles:এখানে তালিকাভুক্ত করার মতো আপাতত কিছুই নেই।]]

Thanks for the report. The fact that this is working on Windows but not on Linux suggests to me that it has something to do with the file encoding. I'll make a change that hopefully rectifies this for you and will include it in the next update.

Not sure, looks like the connected issue?
I've got this error in the attempt to install using Hugo:
hugo server
go: finding github.com/jpanther/congo/v2 v2.5.4
hugo: downloading modules …
go: finding github.com/jpanther/congo v1.6.4
go: downloading github.com/jpanther/congo/v2 v2.5.4
go: downloading github.com/jpanther/congo v1.6.4
go: extracting github.com/jpanther/congo v1.6.4
go: extracting github.com/jpanther/congo/v2 v2.5.4
hugo: collected modules in 4228 ms
Error: add site dependencies: load resources: loading translations: "/tmp/hugo_cache/modules/filecache/modules/pkg/mod/github.com/jpanther/congo/v2@v2.5.4/i18n/bn.yaml:1:1": failed to load translations: unable to parse translation #0 because invalid plural category copied
map[id:code translation:map[copied:কপি হয়ে গেছে copy:কপি করুন]]


There is __MACOSX directory present in archive on vanillaforums.org

What subject says.
This is a minor thing, but it would be nice to have no OS leftovers in release archives. Release source archives here on github do not have any extra files.

The version released on vf.org requires some extra modifications to avoid issues with folder naming and whatnot. I therefore have to modify the git-created archive locally prior to upload and have been too lazy to use zip so far; I'll ditch the built-in OS X compression feature for the next release :+1:

I believe that this happens when compressing things. You can remove it, nothing would change. It's just the way Apple likes compressing thing in their own special way.


How to re-render the component when changing locale?

I'm having some trouble with changing locales. I have the Timeline running with default english but when I change language the component doesn't update the labels to the new language until I scroll a bit to make it re-render or until I refresh the page. I've searched the documentation but found no way to make the component re-render when I change the locale on componentDidUpdate(). Any way to deal with this?

you can pass new items kist or group when changing the local that will definitely help

I'm having the same issue. The secondary header (which uses a unit of "week") isn't correctly updated when moment's first day of the week is updated according to locale.
As suggested, I tried changing the items/groups when this event happens, but it didn't work. Manually resizing the timeline (by changing the browser's window size) does make it correctly re-render.
What other possible solutions could you suggest? @Ilaiwi

Couldn't find better solution for this. So I decided to force render the view for time being.
componentDidUpdate(prevProps, prevState) {
    const { locale: prevLocale } = prevProps
    const { locale } = this.props
    if (!isEqual(prevLocale, locale)) {
      this.setState({ refreshing: true })
      const self = this;
      setTimeout(() => {
        self.setState({ refreshing: false })
      }, 0)
    }
  }

 return (
    <>
       { !this.state.refreshing && <Timeline ... /> }
    </>
 )


Bug: imdb_id support produces error - invalid literal for int() with base 10: ''

Describe the Bug
Using the develop branch (at commit  62532535c9d6d6b2d686716614c36e05c78efc0e), if library caches items with an IMDb id, but no TMDb id, subsequent runs fail with the following error:
invalid literal for int() with base 10: ''

This issue seems to stem from the changes in the mentioned commit here: modules/convert.py
This allows new library items to be added to the cache that have an IMDb id, but not a TMDb number. Then when the program tries to read back the cache on a later run, it tries to read out that empty TMDb entry as an Int starting here in modules/cache.py.
Relevant Collection Config

Having a problem with a collection include the collection config from your metadata file.

Not collection specific
Plex Meta Manager Info

Version Number (can be found at the beginning of your meta.log file.
1.13.0-develop1124

develop branch commit 62532535c9d6d6b2d686716614c36e05c78efc0e as mentioned above
Link to logs (required)

If having an error with a specific collection include the collection.log otherwise please include the full meta.log file on Gist. Do not upload attachments.

https://gist.github.com/snown/09c35bf2bf79097252acae75900c4231

just pushed a fix to develop thanks a bunch for the detailed report it helped alot

In my testing, the changes fixed this issue. Feel free to close. Thanks!

awesome thanks again for the help


Commenting out csproj elements moves them to top on save

Related to #1442. Moved from https://github.com/dotnet/roslyn-project-system/issues/932.
@onotovy said:

When I have a CSproj open for editing, if I comment-out a property group, when I hit save, it moves the commented area to the top of the file, right under project. That's startling and wrong as it moves items from where I want. Further, if it can change the meaning if they're uncommented and not restored to the correct location (which someone may not know where it belongs).

Again, I'm accessing the XML by going through the RawXml property: https://github.com/dotnet/roslyn-project-system/blob/dev15-rc2/src/Microsoft.VisualStudio.ProjectSystem.Managed.VS/ProjectSystem/VS/Utilities/IMsBuildAccessor.cs#L53.
FYI @AndyGerlicher @rainersigwald

Thank you for letting is know about this bug, @333fred. This sounds like a very worrisome developer experience. @cdmihai please look into this.

It's more than that...if you paste in an XML snippet that contains a comment and hit save, it appears to remove the comment from the editor. When you look on disk, it's really there.

Can you please write out a minimal repro in terms of MSBuild APIs that exposes this issue? That way I can debug and see what happens. See my comment from the related issue: https://github.com/Microsoft/msbuild/issues/1442#issuecomment-266917716

Should be fixed by #1507


Bug 429, 492, 551 : Resolved Issues on Download Collection, Premium Support Component & Popover Element

Description

Resolve the issues on Element Radio button unable to select:

Download Collection
Premium Support
Popover

Resolve issue on mouse hover on download icon hand cursor are not showing on Download Collection component.
Resolve spacing issue on small mobile view on Premium Support component.
Resolve issue of selecting small & large mobile view popover get cuts.

Screenshots :

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Changes Reviewed and PR Approved


Programatically setting playing to true or false does nothing

I want to use the player with custom controls so I've set controls to false and set playing equal to a state variable. Changing the variable does nothing, neither does just setting playing to true. Even if initialized with playing set to true there is no audio.
import React from 'react'
import { connect } from 'react-redux'
import ReactPlayer from 'react-player'

class Player extends React.Component {

  componentWillReceiveProps(nextProps) {
    console.log(nextProps)
   // logs as expected
  }

  render() {

    const { audioPlayer } = this.props

    return (
      <ReactPlayer
        url='https://s3.amazonaws.com/*************/30rocktheme.mp3'
        playing={audioPlayer.isPlaying}
      />
    )
  }
}

export default connect(store => ({
  audioPlayer: store.audioPlayer,
}))(Player)

On what device/browser are you seeing this happen? Changing the playing prop from true to false should pause the media, and the opposite should play it.
If you are both trying to play things on mobile devices, keep mobile considerations in mind. Mobile browsers are prone to not allowing media to be played (or played with audio) without deliberate user interaction.
If either of you have an example repo or jsFiddle where the issue is occuring consistently, feel free to reopen the issue.

Why is this closed??  I'm experiencing this on Chrome / Linux.
`export default ({ url, showCloseButton, onStart }: VideoProps) => {
const s = useStyles()
const [playing, setPlaying] = useState(false)
return (

<ReactPlayer
allowFullScreen="allowFullScreen"
frameBorder="0"
className={s.player}
url={url}
width='100%'
height='100%'
playing={playing}
config={{
youtube: {
playerVars: {
controls: 1,
allowFullScreen: 1,
showinfo: 1,
playsinline: 1,
}
}
}}
onStart={onStart}
/>
{showCloseButton && (
<IconButton aria-label="delete" className={s.button} onClick={() => {setPlaying(false)}}>

)}

)
}`

Why is this closed?

Explained in my previous comment:

If either of you have an example repo or jsFiddle where the issue is occuring consistently, feel free to reopen the issue.

It's much harder for me to help if I can't easily recreate the issue myself.


API should return the latest package of the same project

Task linked: QF-1577 When there is no need to trigger a new package job, the API should return the latest package of the same project, not the latest package in for all projects


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_94kakahctw/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_94kakahctw/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_94kakahctw/clone/recipe/meta.yaml:29: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_94kakahctw/clone/recipe/meta.yaml:29: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
ERROR: /tmp/abs_94kakahctw/clone/recipe/meta.yaml:35: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_94kakahctw/clone/recipe/meta.yaml:35: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_94kakahctw/clone/recipe/meta.yaml:35: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_94kakahctw/clone/recipe/meta.yaml:35: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_21bgrd7v2g/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_21bgrd7v2g/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_21bgrd7v2g/clone/recipe/meta.yaml:29: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
ERROR: /tmp/abs_21bgrd7v2g/clone/recipe/meta.yaml:29: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_21bgrd7v2g/clone/recipe/meta.yaml:35: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_21bgrd7v2g/clone/recipe/meta.yaml:35: missing_description: The recipe is missing a description
ERROR: /tmp/abs_21bgrd7v2g/clone/recipe/meta.yaml:35: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_21bgrd7v2g/clone/recipe/meta.yaml:35: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


Update stars on tap

Is there a way to update the stars using just a tap? It seems as though you either have to scroll across or hold down for longer than a tap.
If there is a way to do this I would love to know! Or if there isn't, could it be a possible future feature?

Library setup method: CocoaPods
Version of the library: 14.0.0
Xcode version. Example: 9.2
OS version. Example: iOS 11

Thanks for reporting, @dcortright. It looks like a bug because Cosmos is supposed to update the rating on just a quick tap as well. How are you using cosmos? There is a demo app in this repo, it would be helpful if you could check if the problem exists there for you as well. Let me know how it goes.

I have a view that I am presenting on a custom alert that provides the user the rating stars as well as a text box for entering a review. I tested a different view that also uses Cosmos and I am able to tap and update the stars there. So it is something with presenting it on the alert. I will report back once I determine the bug. Thank you for responding!

Interesting, something may be blocking the tap gesture.

I will report back once I determine the bug.

It will be great, thanks. :)

It the alert controller that I am using that is blocking the tap gesture. Off to find out which one will have to change. Thanks again!

No problem, feel free to share your solution here if you find out.

Alright. In case anyone else runs into this problem this is what I had to do:
I was using a combination of CFAlertViewController and Cosmos to present a view to the user that allows them to create a review on a product.
BUT...CFAlert listens for taps on its view so that if you tap outside the view it will dismiss it. So what I did was remove all of the gesture recognizers from the CFAlert view and instead will handle them on my own.
for recognizer in alertController.view.gestureRecognizers ?? [] {
            alertController.view.removeGestureRecognizer(recognizer)
        }

The stars are back to acting as they should! Thank you @evgenyneu


Improvement: Ignore abstract model classes

Closes #18

We're currently processing your upload.  This comment will be updated when the results are available.


AutoMLService training fails when there's error in one of trainers

Steps to reproduce
Dataset:
dataset-sqlserver.database.windows.net taxi-fare/Biance
select HighPrice as label, de-select ID and pick regression, click training

Having the same problem with own dataset:
Poisson regression must regress to a non-negative label, but label -0.2607227 encountered

Then whole process failes and I am not able to generate the code.
Workaround: Use a training time where it doesnt reach that specific algorithm and retrain after code gen.


Add codefix to generate types for untyped module (#26588)

Port of #26588

We'll merge to master to 3.1 at 4 PM


open an issue or PR re: the spec fix for BoundField[] fields [optional]

https://github.com/dart-lang/vm_service_drivers/pull/55/files#diff-68936077052390239c77a3732f47e9ccR1365

@danrubel, done?

https://github.com/dart-lang/sdk/pull/24481 ... still waiting

Fixed!


Enable & verify support for Bluz...

... which means going back to SPI0 pins (instead of SPI1) and deactivating DMA buffer transfer if Bluz is used

Implemented & enabled with initial release


NAN issue

When I try to apply this loss function to my model, modified from DCGAN,  I got NAN loss, I wonder will this modified vanilla function will lead to this issue? Thanks!

That shouldn't happen, maybe there's a bug in your code? Try getting the DCGAN code from improved WGAN Github (I give a link at the bottom of README here) and start from that.


Same issue as https://github.com/github/git-lfs/issues/833

Hi there,
It appears that our organization is having the same issue as the one seen in https://github.com/github/git-lfs/issues/833.
Please let me know what needs to be done to correct the issue.
 ~/Desktop/BloxelsSandbox   v1.4.3 ✚  git lfs fetch Fetching v1.4.3 Git LFS: (0 of 2 files) 0 B / 247.01 MB batch response: http: Git LFS is disabled for this repository. Docs: https://github.com/contact Warning: errors occurred  ✘  ~/Desktop/BloxelsSandbox   v1.4.3 ✚ 

I resolved this by moving to the $5/monthly plan for LFS. Will reopen if I find another issue.

weird, when I upgraded the plan, it fails uploading because of quota. Maybe it needs time to update with the plan.


mjml module not working for sending email notifications

Describe the bug
We have enabled email plugin to send out notification mails on ticket update. The status change of the dispatch ticket is triggerent slack messages fine. However, it is failing on sending email notifications.
ERROR:dispatch.messaging.email.utils:internal/modules/cjs/loader.js:818
throw err;
^
Error: Cannot find module '../lib/index'
Require stack:

/usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/.bin/mjml
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:815:15)
at Function.Module._load (internal/modules/cjs/loader.js:667:27)
at Module.require (internal/modules/cjs/loader.js:887:19)
at require (internal/modules/cjs/helpers.js:74:18)
at Object. (/usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/.bin/mjml:3:1)
at Module._compile (internal/modules/cjs/loader.js:999:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12) {
code: 'MODULE_NOT_FOUND',
requireStack: [
'/usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/.bin/mjml'
]
}

ERROR:dispatch.decorators:MJML template processing failed.
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/dispatch/decorators.py", line 85, in wrapper
result = func(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/dispatch/incident/flows.py", line 1132, in incident_update_flow
send_incident_update_notifications(incident, previous_incident, db_session)
File "/usr/local/lib/python3.9/site-packages/dispatch/incident/messaging.py", line 395, in send_incident_update_notifications
notification_service.filter_and_send(
File "/usr/local/lib/python3.9/site-packages/dispatch/notification/service.py", line 135, in filter_and_send
send(
File "/usr/local/lib/python3.9/site-packages/dispatch/notification/service.py", line 109, in send
plugin.instance.send(
File "/usr/local/lib/python3.9/site-packages/dispatch/decorators.py", line 106, in wrapper
result = func(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/dispatch/decorators.py", line 122, in wrapper
return func(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/dispatch/plugins/dispatch_sendgrid/plugin.py", line 89, in send
message_body = create_message_body(notification_template, notification_type, **kwargs)
File "/usr/local/lib/python3.9/site-packages/dispatch/messaging/email/utils.py", line 90, in create_message_body
return render_html(template.render(**kwargs))
File "/usr/local/lib/python3.9/site-packages/dispatch/messaging/email/utils.py", line 106, in render_html
raise Exception("MJML template processing failed.")
Exception: MJML template processing failed.
To Reproduce
Using v20211015 release.
The content of the file /usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/.bin/mjml is following

#!/usr/bin/env node
require('../lib/index')
require('mjml-cli')
However, the path '../lib/index' does not exist.
root@web-78c8cc85f8-l266l:/usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/.bin# cat mjml
#!/usr/bin/env node
require('../lib/index')
require('mjml-cli')
root@web-78c8cc85f8-l266l:/usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/.bin# ls -al ../lib/index
ls: cannot access '../lib/index': No such file or directory
root@web-78c8cc85f8-l266l:/usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/.bin#
Most probably it is a NPM issue, but was this email notification working earlier? Or is there any fix we can apply in Dispatch to pass this error.

Hmm, we are currently using that release and email notifications are working correctly. Judging by this old issue, maybe a re-install fixes it? https://github.com/mjmlio/mjml/issues/507

Yes, reinstalling npm plugin is solving the issue. Followed below steps:

rm /usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/.bin/mjml
npm uninstall mjml
npm i mjml

But strangely even with newly built image, this issue persists. Hence, i need to do above steps to make the notifications working.
Let me check mjml forums.

@kevgliss for me a different mjml config path is working.
I have changed following:
default=f"{os.path.dirname(os.path.realpath(file))}/static/dispatch/node_modules/.bin",
to
default=f"{os.path.dirname(os.path.realpath(file))}/static/dispatch/node_modules/mjml/bin",
in dispatch/src/dispatch/config.py

You can also set the MJML_PATH
See: https://hawkins.gitbook.io/dispatch/administration-guide/server#mjml_path-default-node_modules-.bin

ok, so in my case, can i set MJML_PATH to '/node_modules/mjml/bin' in .env file or do I need to specify full path '/usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/mjml/bin' ?

tried with both 'MJML_PATH=/node_modules/mjml/bin' and 'MJML_PATH=/usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/mjml/bin' in .env, but the changes are not reflecting there.

tried with both 'MJML_PATH=/node_modules/mjml/bin' and 'MJML_PATH=/usr/local/lib/python3.9/site-packages/dispatch/static/dispatch/node_modules/mjml/bin' in .env, but the changes are not reflecting there.

any comments on this?

@vikassinha2019 are you still experiencing issues with this? If so, please feel free to close.


⚠ Bofrost.at has degraded performance

In aad6cb1, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1707 ms

Resolved: Bofrost.at performance has improved in 22e1d37 after 15 minutes.


st.experimental_rerun -> st.rerun

Streamlit v1.28 gives a deprecation warning
`st.experimental_rerun` will be removed after 2024-04-01

Which version of the searchbox are you using? This deprecation warning should not appear with versions > 0.1.6

I should have checked if a new package is available. Thanks!


jacoco for tests.

Current coverage is 13.48%

Branch #10 has no coverage reports uploaded yet.

No diff could be generated. No reports for develop found.

Powered by Codecov. Updated on successful CI builds.


Exif image rotation

Some images have an exif rotation applied to them purely in metadata. This will provide those rotations if they exist.

Main thread previously:


PHOENIX-6470 Bump Tephra to 0.16.1 in 4.x

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 38s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ 4.x Compile Tests _

+1 :green_heart:
mvninstall
15m 33s
4.x passed

+1 :green_heart:
compile
1m 53s
4.x passed

+1 :green_heart:
javadoc
1m 24s
4.x passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
7m 44s
the patch passed

+1 :green_heart:
compile
1m 54s
the patch passed

+1 :green_heart:
javac
1m 53s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
1m 18s
the patch passed

_ Other Tests _

-1 :x:
unit
159m 10s
root in the patch failed.

+1 :green_heart:
asflicense
0m 55s
The patch does not generate ASF License warnings.

193m  6s

Reason
Tests

Failed junit tests
phoenix.monitoring.PhoenixTableLevelMetricsIT

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1230/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1230

Optional Tests
dupname asflicense javac javadoc unit xml compile

uname
Linux 02f8eab159a4 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
4.x / 9e91e5c

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1230/1/artifact/yetus-general-check/output/patch-unit-root.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1230/1/testReport/

Max. process+thread count
5607 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1230/1/console

versions
git=2.7.4 maven=3.3.9

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Closing this as it is committed here


Unable to load data from local "data-server" on the web app

Hi,
I want to use the standalone web application feature of Vitessce (the Vitessce app ).
And it seems that the local data cannot be accessed through it. And here is the screenshot of the error

In a really tedious way describing what I did that ended with this error:

I was able to make a Vitessce widget using Jupyter lab. So I exported the .json from that widget.
In the .json file I edited the url to be (say http://localhost:9000/path-to-data.zarr-dir)
I created a simple python http web server for the data.zarr-dir on port specified above (python -m http.server 9000 )
Upload this modified .json file to the Vitessce app website

I can't really find where it's been documented how to make local data accessible to the Vitessce web app, so I am likely doing something very stupid here. Any suggestion on how to serve the data correctly is greatly appreciated!
Thank you

These steps all make sense, so it seems like a simple issue with the local HTTP server lacking a CORS configuration. It is slightly involved to configure the python built-in HTTP server so that it responds with the correct CORS headers (according to this stack overflow answer).
I would recommend using https://www.npmjs.com/package/http-server rather than python -m http.server, then you can navigate to the directory that you want to serve and run
http-server --cors='*' --port 9000 .

Worked! Thank you so much! @keller-mark


Update MD file on new branch 2017-23-20_09-11-16

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Is api-auth version 1.0.3 compatible with rails 3.0.0?

We auto-run test against Rails 3 since #40 and the initial implementation of those tests targeted 1.1.0 (the successor of 1.0.3).
Looking at the commits I guess it's safe to say 1.0.3 is compatible even though I'd encourage you to upgrade to a recent version which is tested against Rails 3.0.20


CommutationAnalysis needs to be refactored

in 2018, #1500 introduced CommutationAnalysis pass. The pass does the trick while working with CommutativeCancellation. However, there are some aspects that are not very well covered and I think it is time to sit again with it and think it over again. Here there are a list of the problems:
property commutation_set is a messy dict
The property property_set['commutation_set'] is a non-uniformly typed dict:
from qiskit import QuantumCircuit

circuit = QuantumCircuit(1)
circuit.h(0)

from qiskit.transpiler.passes import CommutationAnalysis

property_set = {}
CommutationAnalysis()(circuit, property_set)

for k,v in property_set['commutation_set'].items():
    print(type(k), type(v))

<class 'qiskit.circuit.quantumregister.Qubit'> <class 'list'>
<class 'tuple'> <class 'int'>
<class 'tuple'> <class 'int'>
<class 'tuple'> <class 'int'>

suggested solution: property_set['commutation_set'] should be a qiskit.dagcircuit.dagdependency.DAGDependency.
does not handle intransitivity
The current commutation analysis assumes transitivity (I think), while the relation is intransitive. For example H commutes with I  and I commutes with Z, but H does not commute with Z. This is not reflected in the commutative set:
from qiskit.circuit import Qubit
from qiskit.dagcircuit import DAGOpNode

def pp_commutation_set(property_commutation_set):
    # pretty printer of the commutation set
    for qbit, sets in property_commutation_set.items():
        if not isinstance(qbit, Qubit):
            continue
        print(qbit.index)
        for commutation_set in sets:
            r=[]
            for node in commutation_set:
                if isinstance(node, DAGOpNode):
                    r.append(f"{node.op.name}_{node._node_id}{[arg.index for arg in node.qargs]}")
            if r:
                print('\t', r)

from qiskit import QuantumCircuit

circuit = QuantumCircuit(1)
circuit.h(0)
circuit.id(0)
circuit.z(0)

from qiskit.transpiler.passes import CommutationAnalysis

property_set = {}
CommutationAnalysis()(circuit, property_set)

pp_commutation_set(property_set['commutation_set'])

0
	 ['h_2[0]', 'id_3[0]', 'z_4[0]']

There is a lot of duplication with DAGDependency
The qiskit.dagcircuit.dagdependency.DAGDependency seems to do the same as qiskit.transpiler.passes.optimization.commutation_analysis.CommutationAnalysis. For example qiskit.dagcircuit.dagdependency._does_commute and qiskit.transpiler.passes.optimization.commutation_analysis._commute are similar in their goal, but one of them has a cache system that makes it better.

In continuation of  https://github.com/Qiskit/qiskit-terra/pull/8056, the following test currently fails due to unsound optimization:
    def test_intransitive_non_commutative_circuit1(self):
        """Test simple intransitive non-commutative circuit on 1 qubit"""
        circ = QuantumCircuit(1)

        circ.x(0)
        circ.i(0)
        circ.h(0)
        circ.i(0)
        circ.x(0)

        passmanager = PassManager()
        passmanager.append(CommutativeCancellation())
        ccirc = passmanager.run(circ)
        self.assertEqual(Operator(circ), Operator(ccirc))

due to transitivity assumption.
The following test is an example of missed optimization in case of naive fix:
    def test_non_pairwise_commutative_circuit1(self):
        """Test simple circuit where we can simplify though operators not commute pairwise on 1 qubit"""
        circ = QuantumCircuit(1)

        circ.x(0)
        circ.i(0)
        circ.z(0)
        circ.i(0)

        passmanager = PassManager()
        passmanager.append(CommutativeCancellation())
        ccirc = passmanager.run(circ)

        expected = QuantumCircuit(1)
        expected.x(0)
        expected.z(0)

        self.assertEqual(Operator(circ), Operator(ccirc))
        self.assertEqual(ccirc, expected)

A few questions.

The caching mechanism of qiskit.transpiler.passes.optimization.commutation_analysis._commute seems extremely useful. However (if I am not mistaken) the cached information that x(1) and cx(0, 1) commute would not help to detect that x(42) and cx(50, 42) commute (same gates up to relabeling qubits). Would be nice to avoid caching a quadratic amount of information. Also I am not sure whether this would detect that cx(0, 1) and x(1) commute (same gates, but in different order).
If I understand correctly, in qiskit.dagcircuit.dagdependency we compute and store the lists of predecessors and successors for every node in DagDependency, which seems that this may require a lot of memory for long and wide circuits. Is this correct or retworks does something clever when storing such lists?The algorithm to construct DagDependency can be easily modified to reason about direct_predecessors only, however it seems that predecessors are also needed for the template matching algorithm.


Manual inspection to make sure no more than three clicks are required to start a new code review.

frontend interface code output

already done


Emulator does not respond to keys when speed is set below 20%

With default 100% clock speed, load samp123a.s19 and DO 0000.  See the moving single segment.  As it runs, slow clock speed to 20%.  It slows down, good.  Press the emulator RESET key and see CPU uP.  Press "DO" and nothing happens.  "Reset" the speed to 100% and press "DO".  Then it works and shows "_ _ _ _ do.".

Key presses are handled by the ET-3400 software (Monitor ROM).  Keypress detection is not buffered and goes through a software "debounce" i.e. it loops for a few cycles and checks that the key is still pressed. If you slow down the CPU, keypress detection also slows down and so you have to hold the key down much longer.

I was surprised at the "not a bug" classification. The problem is understood and hard to fix, but seems like a big problem -- Normal operation, trying to make the emulator the same speed as the real device, and then clicking on the emulator key gives no response.  Dead end.  Holding it down for a longer time is not an obvious workaround.

In v1.0.7 Beta 2 I ran into this problem with all default settings and the first thing I tried -- Load RAM, samp123a.s19, click on "DO" (it saw this OK), click on "0" and it appears to not see my mouse clicks.  Pressing the "0" key on my keyboard has the same problem.  I can see it activating the "0" key in the emulator window, but it doesn't register in the emulator display until I hold down the keyboard key awhile.

Another addition: The title of the issue "set below 20%" no longer applies since it happens at the default 100% setting.  Also of note: After writing all the above and playing with the speed setting, I only just now noticed the explanation "Setting the frequency below 70 kHz..." in the window.  It explains things well, but I hadn't noticed it until now.  Suggestion: If the problem can't be fixed, change the default setting to about 150%.  At lease on my PC that is a nice trade-off between execution speed (too fast) and key response (usually works fine).  I'd rather have it execute too fast than miss my mouse clicks and key presses.  I would guess most users when they first use the emulator are developing new programs and don't care as much about execution speed matching the real device.


blockHandler feature request

Do you want to request a feature or report a bug?: Feature request
What is the current behavior?
Currently blockHandlers are somewhat limited, and the fields available differ quite a bit network to network. It would be ideal if the following fields were included for each chain type:
EVM:

transactionCount or list of transaction hashes
gasPrice (if possible)
contracts/block
contract calls/block
list of addresses that accessed a block (used to calculate addresses/block)

Ideally a blockHandler would return a list of Transaction types and then we could dive into a transaction to see to/from addresses, gasUsed, gasPrice, etc.
Arweave:

transaction count

Again, it would be ideal to dive into transactions to see the size of each transaction
Cosmos:

gas used/ gas wanted

if we can dive into transactions to see gas used/wanted even better
Near:

transaction list (for transaction count / gas used)
transactionFees

Hi @dmelotik ! Thanks for this, I am keen to understand your use case a little more, are you building subgraphs across networks? In general the schema / fields available is informed by the protocol, rather than generalising across them

Hey @azf20, we are making a network level subgraph across all chains supported by thegraph. So we are trying to capture as many useful network-level metrics as possible to drive data analysis and research.
If we were to get some of these features added to the blockHandlers the most pressing would be transaction count and gas price for EVM chains. That will drive metrics like transactions/block/day and then we are able to calculate rewards/block.
You can view a WIP solution here (see the schema for what we are grabbing, but certain metrics are not avail across other chains): https://github.com/messari/subgraphs/pull/298
lmk if I can clear anything up or provide more context

bumping this @azf20, not sure how much of a lift this is (ie, going to have to weigh the cost/benefit). But ideally we would be able to expose all of the data on each block. We want to have the most complete data set we can.
It has been something we have had to do to our best ability with what we have and then defer the rest to substreams.


Use pathinfo to get file extension instead of substr.

I think it's a bit safer to use pathinfo() instead of substr().

@spectrox

I think it's a bit safer to use pathinfo() instead of substr().

Can you explain it please? You think based on ... ?
As I have checked it is not faster for sure:
https://3v4l.org/Be0iY
$filename = 'fgdsahktyrewagfbdhzbvt74tf.gfdhsgfghsdf.gsdfhjgfdsg.gsfdghfdsh.dtasfghsdghg.tar.gz';

$t1 = microtime(true);
for ($i = 0; $i < 100000; ++$i) {
    $e1 = substr($filename, strrpos($filename, '.') + 1);
}
$t2 = microtime(true);
for ($n = 0; $n < 100000; ++$n) {
    $e2 = pathinfo($filename, \PATHINFO_EXTENSION);
}
$t3 = microtime(true);
for ($j = 0; $j < 100000; ++$j) {
    $e3 = substr(strrchr($filename, '.'), 1);
}
$t4 = microtime(true);

echo 'strrpos: ' . ($t2-$t1), PHP_EOL;
echo 'pathinfo: ' . ($t3-$t2), PHP_EOL;
echo 'strrchr: ' . ($t4-$t3), PHP_EOL;

Example results:
strrpos: 0.0073950290679932
pathinfo: 0.35552787780762
strrchr: 0.0089771747589111

very similar on different PHP versions.

Sorry, I missed your first reply.
My thinking is based on the fact that substr/strrpos functions aren't unicode-safe, while IIRC pathinfo is locale aware and will act normally as long as your locale will be set to the right one.

@spectrox can you then provide an example which is failing with substr/strrpos and is passing with pathinfo?


Remove ro from mount function and run container for npm test.

We need to remove ro parameter from mount command.
It fails on RHEL-8.
Also for checking if npm works properly with NPM_REGISTRY
we need to start a container and check npm by docker exec function.
Required: s2i-nodejs-container#221
Signed-off-by: Petr "Stone" Hracek<EMAIL_ADDRESS>

https://github.com/sclorg/s2i-nodejs-container/pull/221

[test]

[test]

[test][test-openshift]

Hmm seems like CI jobs for rhel8 version are stuck after finishing testing the python image

[test]

Narrowed it down to something introducing the hang inside the image tests, will revisit tomorrow in more detail.

Looks good, tests are green now and complete. Just some small details left before we can merge

[test]

[test-openshift]

Now, If NPM_REGISTRY variable is present and CA file is present as well, then if NPM_REGISTRY entry is not mentioned in jquery log file, then test fails and return 1.

In rhel7 test there is something weird. From log https://gist.github.com/rhscl-bot/b333e062505750485ec2f2b3bf55dfef#file-container-common-scripts-135-sh-L1211 I see, that centos7 image is used instead of rhel7. NPM_REGISTRY should not be present for centos7.
I will look at it deeply.

[test]

[test]

LGTM now, thanks!


Update Pathology Pipelines for MetaTensor Changes

Fixes https://github.com/Project-MONAI/MONAI/issues/5078
Description

[x] Update tumor detection pipeline
[x] Update Multiple-instance learning pipeline

Waiting for https://github.com/Project-MONAI/MONAI/pull/5107 and https://github.com/Project-MONAI/MONAI/pull/5115 to be merged.

All the related PRs in MONAI core is merged, so this PR is ready for review.


If there are less than two images to be displayed, nothing will be displayed.

Hi all, I have found a bug.
When using WaterdallGrid, if there are less than 2 data to display, nothing will be displayed.
Here is a piece of source code that reproduces the bug at my end
ScrollView {
                        WaterfallGrid(dataArray, id: \.self) { file in
                            VStack {
                                if let post = viewModel.posts.first(where: { $0.fileIds.contains(file.id) }) {
                                    NavigationLink(destination: ImageDetailView(item: post, file: file)) {
                                        // View to display images.
                                        ImageGridItem(url: file.url!)
                                    }
                                }
                            }
                        }
                    }

import SwiftUI
import Kingfisher

// View to display images.
struct ImageGridItem: View {

    private var url: String

    init(url: String) {
        self.url = url
    }

    var body: some View {
        KFImage.url(URL(string: url))
            .resizable()
            .placeholder{
                RoundedRectangle(cornerRadius: 30)
                    .fill(Color(0xe6e6e6))
            }
            .scaledToFit()
            .clipped()
            .cornerRadius(5)
            .aspectRatio(contentMode: .fit)
    }
}

3 or more
less than two data

Please some advice or fix

Have you find a solution?

Hi
 @yamanetaisei.  It's not limited by the exact number (2). Such things happen when the number of items are less or equal than number of columns in your grid. I guess you saw 2 is making the difference just becasue the default number of columns is 2.
Furthermore, I feel this bug should come from the logic how waterallgrid calcualtes the width of each item. @xiaoxidong  Then seems not easy to fix it. But a workaround is to add some blank item in your items to make its length larger than number of columns.

  Thanks
  Dongqing

hi. i used the following approach:
.gridStyle(
                columnsInPortrait: maxColumns(max: 2),
                columnsInLandscape: maxColumns(max: 3),
                spacing: 4
            )

func maxColumns(max: Int) -> Int {
       // images is an array holding the images and its data
        let highestMatchIndex = min(max, images.count - 1)
        return highestMatchIndex
    }


Clear test dependency management by create azkaban-db-{api, test} modules

This PR is a follow-up of #1361 , which aimed at clearing the submodule dependency management. Change being made in this pr:

create azkaban-db-test and azkaban-db-api sub modules.
move DB's abstract class to  azkaban-db-api , and move Test class to azkaban-db-test. In this way, we can call up azkaban-db-test in azkaban-common module.
azkaban-db will rely on azkaban-db-api.

We can discuss together with @HappyRay


GIS Day

Benefit
UGRC as the leading GIS agency in the State should contribute to this unofficial holiday. It is a great opportunity to highlight our work, enhance relationships with businesses and government entities, and contribute to the community.
Acceptance Criteria

[ ] Start an internal discussion to plan a small-scale GIS Day event this year or next.

Notes
Potential collaborations and collaborators:

Esri
UGIC
Municipalities
stemfest
#96
#72
#49

Risks
No response
Issue Reference
No response

Looks like we missed the due date. Should we close this or change the date for 2025?

Yep, that's on me. Let's move it to 2025
On Fri, Nov 29, 2024 at 10:20 AM steveoh @.***> wrote:

Looks like we missed the due date. Should we close this or change the date
for 2025?
—
Reply to this email directly, view it on GitHub
https://github.com/agrc/planning-queue/issues/493#issuecomment-2508180242,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BK3H5GFD4V247QIA3CEPPGD2DCO43AVCNFSM6AAAAABO5P55YOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBYGE4DAMRUGI
.
You are receiving this because you were assigned.Message ID:
@.***>

Discussed with Laura some possible ideas through project collaborations with higher education institutions such as Utah Tech.


Clarification on objectives documentation

Skeptical that it clarifies much... what is the issue here?

It never occurred to me that you can have a regression problem in which y is not scalar.
When I saw the cost 'rmse' I simply assumed that the name say what it does root-mean-square.
At least on the samples in a batch.
But this is not true. what rmse does is to do the RMSE over the internal dimension of y which in may case is one and then do a mean over the all samples (first over the batch and then over all batches.)
so if y is scalar then rmse is exactly the same as abs.
I find this very confusion and not clear from the docs.


keyboardHidesTabBar option doesn't work

Bug
Adding tabBarOptions={{keyboardHidesTabBar: true}} on either Tab.Navigator component from React Navigation 5.x or AnimatedTabBar from this library doesn't work - the tab bar still shows above keyboard.
Tested on Android v10 Emulator
Environment info

Library
Version

@gorhom/animated-tabbar
1.6.0

react-native
0.62.2

react-native-reanimated
1.9.0

react-native-gesture-handler
1.6.1

react-native-svg
12.1.0

Steps To Reproduce

Use the example with React Navigation 5.x;
Try adding tabBarOptions={{keyboardHidesTabBar: true}} to Tab.Navigator

Describe what you expected to happen:

Adding tabBarOptions={{keyboardHidesTabBar: true}} should hide the tab bar when using keyboard

Reproducible sample code
<Tab.Navigator
	tabBarOptions={{
		keyboardHidesTabBar: true
	}}
	tabBar={props => (
		<AnimatedTabBar style={{backgroundColor: '#272727'}} tabs={tabs} {...props}/>
	)}
>

My bad - I did not clear the recent build, after doing so everything works fine!

My bad - I did not clear the recent build, after doing so everything works fine!

Clearing recent build you mean npm cache clean ?


rev to base dockerfile to 1.5.3

while deploying to k8s 1.5.2, I was getting error:
error validating "manifests/hdfs-nn-statefulset.yaml": error validating data: API version "apps/v1beta1" isn't supported,

until I rev'd the hyperkube base to 1.5.3

ps. thanks for this repo - very useful

Looks great, thanks for the contribution. Glad you are finding it useful.


Could not connect to Windows pod by port 10250

Is this a request for help?:
No
Is this an ISSUE or FEATURE REQUEST? (choose one):
ISSUE
What version of acs-engine?:
latest master(commit 05ebc78b57ce2ef92a83bcf6b81b3a9b46ce7f0b)

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kubernetes
What happened:
Could not connect to Windows pod by port 10250 or query pod stats/summary by port 10255, it's a regression on v1.10.0 Windows cluster, it works well for Linux pod.
azureuser@k8s-master-20941235-0:~kubectl exec -it aspnet -- cmd
Error from server: error dialing backend: dial tcp <IP_ADDRESS>:10250: getsockopt: connection timed out
azureuser@k8s-master-20941235-0:~$ curl http://20941k8s9010:10255/stats/summary
curl: (7) Failed to connect to 20941k8s9010 port 10255: Connection timed out

While if I connect to that Windows node, and run following command by localhost, it works well, so I guess there is some network configuration not correct for Windows:
PS C:\> $response = wget http://localhost:10255/stats/summary -UseBasicParsing
PS C:\> echo $response
StatusCode        : 200
StatusDescription : OK
Content           : {
                      "node": {
                       "nodeName": "20941k8s9010",
                       "startTime": "2018-04-02T06:52:31Z",
                       "cpu": {
                        "time": "2018-04-02T07:17:35Z",
                        "usageCoreNanoSeconds":<PHONE_NUMBER>0
                       },
                       "memory": {
                        "t...
RawContent        : HTTP/1.1 200 OK
                    Transfer-Encoding: chunked
                    Content-Type: application/json
                    Date: Mon, 02 Apr 2018 07:17:40 GMT

                    {
                      "node": {
                       "nodeName": "20941k8s9010",
                       "startTime": "2018-04-02T06:52:31Z",...
Forms             :
Headers           : {[Transfer-Encoding, chunked], [Content-Type, application/json], [Date, Mon, 02 Apr 2018 07:17:40
                    GMT]}
Images            : {}
InputFields       : {}
Links             : {}
ParsedHtml        :
RawContentLength  : 2370

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):

use laest master branch(commit 05ebc78b57ce2ef92a83bcf6b81b3a9b46ce7f0b) to deploy windows cluster
create a simple pod on windows
kubectl create -f https://raw.githubusercontent.com/andyzhangx/Demo/master/windows/basic/aspnet-pod.yaml

watch the status of pod until its Status changed from Pending to Running
watch kubectl describe po aspnet

enter the pod container to do validation
kubectl exec -it aspnet -- cmd

Anything else we need to know:
@JiangtianLi @feiskyer

Seems kubelet is not listening at 10250? are there any errors in kubelet logs?

@feiskyer 10250 & 10255 are all being listened on that windows node
I0402 03:26:30.748044    2928 csi_plugin.go:61] kubernetes.io/csi: plugin initializing...
I0402 03:26:30.748044    2928 plugins.go:454] Loaded volume plugin "kubernetes.io/csi"
I0402 03:26:30.749044    2928 server.go:149] Starting to listen read-only on <IP_ADDRESS>:10255
I0402 03:26:30.749044    2928 healthz.go:74] Installing healthz checkers:"ping", "syncloop"
I0402 03:26:30.750044    2928 server.go:428] Event(v1.ObjectReference{Kind:"Node", Namespace:"", Name:"20941k8s9010", UID:"20941k8s9010", APIVersion:"", ResourceVersion:"", FieldPath:""}): type: 'Normal' reason: 'Starting' Starting kubelet.
I0402 03:26:30.750044    2928 config.go:99] Looking for [api], have seen map[]
I0402 03:26:30.750044    2928 fs_resource_analyzer.go:66] Starting FS ResourceAnalyzer
I0402 03:26:30.750044    2928 status_manager.go:140] Starting to sync pod status with apiserver
I0402 03:26:30.750044    2928 kubelet.go:1777] Starting kubelet main sync loop.
I0402 03:26:30.750044    2928 kubelet.go:1794] skipping pod synchronization - [container runtime is down PLEG is not healthy: pleg was last seen active 2562047h47m16.854775807s ago; threshold is 3m0s]
I0402 03:26:30.750044    2928 generic.go:183] GenericPLEG: Relisting
I0402 03:26:30.750044    2928 server.go:129] Starting to listen on <IP_ADDRESS>:10250
I0402 03:26:30.751044    2928 healthz.go:74] Installing healthz checkers:"ping", "syncloop"

I found I even could not ping windows node, windows version is
>ver
Microsoft Windows [Version 10.0.16299.248]

@andyzhangx Can you curl the aspnet POD?

@JiangtianLi  I could ping aspnet pod from master, while I could not ping windows node

@andyzhangx ping may be blocked on Azure. Can you use telnet or netcat to test connection to node on a specific port?

Was this ever resolved? I am facing the same issue.

@caseywatson what k8s version and acs-engine you are using? Do you have windows only cluster or hybrid cluster?

Any new update on this? I am experiencing the same issue with:
acs-engine version Version: v0.16.2
Has anyone went through this issue and finally connected to the pod?

I deployed from the master branch a few days ago and it seemed to work, do looks like a fix is in

@jakubmalin v0.17.1 should have fixed this issue. all firewall should be turned off for Azure CNI which is the default. You may also fix this issue by netsh advf set allprofiles state off on windows node.


Use pl.sink_* in LazyPolarsDataset._save

Description
When passing a lazy DataFrame to LazyPolarsDataset, it is currently collected into an eager DataFrame before writing it using the appropriate pl.write_* function. This can be skipped by writing the lazy dataframe using pl.sink_*.
Context
In some cases, it may be faster to collect the lazy DataFrame in streaming mode. Additionally, it is not always possible to collect the entire DataFrame (e.g., if the data is too large). Using pl.sink_*, the entire data set does not need to be loaded.
Possible Implementation
In the _save function, the input DataFrame could first be coerced into a lazy DataFrame and then written to disk using pl.sink_*.
Possible Alternatives

I would hold off such a change for now. As mentioned in the docs:

Streaming mode is considered unstable. It may be changed at any point without it being considered a breaking change.

On top of that, I heard the Polars team is working on completely rewriting their streaming engine. So I would just stick with the current implementation...

Indeed, streaming mode is unstable, but lazy non-streaming methods are considered stable, as far as I understand?
About what to do with remote storages, maybe we can offer a sink_* path for local files?

Actually, the sink methods also use the streaming engine under the hood. Hence these methods should also be considered unstable as is explicitly mentioned in the docs. So I wouldn’t recommend using it either.


[Solved] toArray() does not preserve Collection order

It seems that if I use the sort() method on a Collection (from Eloquent) and then I return it as array/json the Collection order is lost.
$models = MyModel::where('name', $name)->get();
$models->sort(function($a, $b) {
    return Version::compare($a->version, b->version);
});
return $models->toArray();

I'm I doing something wrong?
Thx.

being quite long.. But is helpful. thanks

For me as well calling
return $this->response->withJson( [
				'code' => 200,
				'data' => $mySortedCollection->values()
			] );

did it... :-) Thanks a lot @JoostK .


Express 5 Support?

I've been using the bull monitor with express 4.x and am upgrading my project to express 5.x. However, bull monitor stopped working after upgrading the express version. The dashboard loads fine but the graphql requests are stuck at pending status.
Any idea?

,

@robincsamuel hello, looks like apollo-server-express doesn't support express 5 (and it's still in alpha btw) so i'm closing this for now

It's currently in beta, fyi :)

I tried to debug the issue and figured out the root cause. It works fine if I pass bodyParserConfig: false to the applyMiddleware method (source). However, I don't see a way to pass this config via bull monitor.
@s-r-x Any idea?

Adding a middleware like this fixes the issue since body-parser checks for req._body to see if it's already parsed and skips parsing in that case.
router.post('/queues/graphql', (req, res, next) => {
      req['_body'] = true;
      return next();
});

I think it'd be great if we can pass the Apollo Server/applyMiddleware options via bull monitor configuration. I'd be happy to raise a PR if you think that's a good idea :)

@robincsamuel n1. I think we can add some config to the init method with something like useExpress5: boolean, and tweak bodyparserconfig based on that. wdyt?

I don't want to let the library users to deal with any internal dependency (like apollo server) directly, so let's stick with some abstraction for now

Okay, that makes sense. What about passing a more generic option like disableBodyParser and disableCors? I think a flag based on the express version may become irrelevant in the future or will end up adding more flags.
Or maybe, we can detect the express version and handle this without a flag.

sounds good

both are fine, do whatever you think fits the best


initial multiArch support

This should resolve #17. NixOS is only supported on x86 Linux, so this does not affect nixosConfigurations. This only affects the devShell and package outputs. NixOS is pinned to x86_64 for now.
Every package is tested to ensure it is compatible with a given system based on its platform metadata before it it added to that systems package set.

Hello! This is quite the turnaround from what I assumed to be a dead request.  Thanks!

NixOS is only supported on x86 Linux, so this does not affect nixosConfigurations. This only affects the devShell and package outputs. NixOS is pinned to x86_64 for now.

What do you mean by "NixOS is only supported on x86 Linux"? do you mean officially? I haven't seen any statements about official support, but nixpkgs does support multiarch, but maybe not as much non-x86 builds. For example I do still have a hard time updating from both channels 20.03 and unstable on my Pinebook Pro. I think it's mostly the newness of aarch64 using nixpkgs, like at some point i heard the builds are failing in Hydra but were recently fixed.
I'm referring to you for my understanding of flakes using a system configuration.

NixOS is pinned to x86_64 for now.

Thus I do not understand what this means, as nixpkgs has no defined arch? I'm still relatively new and only started tracking my own NixOS configs only recently, but this simple repo layout has been working fine in between x86 and aarch64 machines so far.

@tgunnoe, sorry for the delayed response on this. I made the assumption, based on the images available for download on nixos.org, that NixOS was only support on x86, since those are the only images available. It is possible that it would work on other architectures.
Although, since the flakes specification doesn't allow for defining nixosConfigurations.<some-system> without breaking, I'd assume the answer is no. I think I'm finally ready to implement this since flake-utils would make it rather trivial. I just sort of forgot about it with all the crazy stuff going on this year.


Fix: use action label instead of description in Steward cards

Related: https://github.com/GoodDollar/GoodCollective/issues/130

@krisbitney to confirm:

If the contract HAS a value, that value will display (e.g. "Stewards are paid G$ each time they measure a tree's status"
If the contract DOES NOT HAVE a value, the default text will display: "Please see the smart contract for information regarding payment logic." (per this ticket)

My understanding is that the Silvi contract does have this value, though I am not sure if any of the others displaying DO NOT in order for us to effectively test.
cc @L03TJ3 @patpedrosa @vldkhh to restest once redeployed!

@krisbitney to confirm:

If the contract HAS a value, that value will display (e.g. "Stewards are paid G$ each time they measure a tree's status"
If the contract DOES NOT HAVE a value, the default text will display: "Please see the smart contract for information regarding payment logic." (per this ticket)

My understanding is that the Silvi contract does have this value, though I am not sure if any of the others displaying DO NOT in order for us to effectively test.
cc @L03TJ3 @patpedrosa @vldkhh to restest once redeployed!

I added the property to the subgraph and named it infoLabel, but none of the collectives had it in their IPFS data.
See: https://github.com/GoodDollar/GoodCollective/issues/78#issuecomment-1888710889
Unless the IPFS data has changed, I don't think that value was present for any of the collectives

@krisbitney thank you!
@L03TJ3 @sirpy should add to our to-dos to update IPFS in this case. Who should take this on?

@decentralauren whoever created the pool will be responsible one for keeping the pool attributes updates because the owner/creator of the pool is the only one able to update the contract to new data on ipfs

Confirmed this works, added infoLabel to new pool created


[bug]  run dev, get a lots of error about serde-1.0.193

Describe the bug
cannot find function, tuple struct or tuple variant Err in this scope
--> /Users/natsugei/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.193/src/lib.rs:287:32
Err(err) => return Err(err),
|                                  ^^^ help: a local variable with a similar name exists: err
the error in serde-1.0.193
Reproduction
No response
Expected behavior
No response
Platform and versions
[✔] Environment
    - OS: Mac OS 14.0.0 X64
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.74.0 (79e9716c9 2023-11-13) (Homebrew)
    ✔ cargo: 1.74.0
    ✔ rustup: 1.26.0 (2023-04-05)
    ✔ Rust toolchain: stable-aarch64-apple-darwin (default)
    - node: 21.4.0
    - pnpm: 8.9.0
    - npm: 10.2.4

[build-dependencies]
tauri-build = { version = "2.0.0-alpha.12", features = [] }

[dependencies]
tauri = { version = "2.0.0-alpha.18", features = [] }
serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0"
tauri-plugin-shell = "2.0.0-alpha.4"
compiler_builtins = "0.1.104"

Stack trace
No response
Additional context
No response

try using cargo clean and cargo update inside src-tauri

try using cargo clean and cargo update inside src-tauri

I had tried, and then the error change to once_cell-1.19.0
cannot find function, tuple struct or tuple variant Ok in this scope
--> /Users/natsugei/.cargo/registry/src/index.crates.io-6f17d22bba15001f/once_cell-1.19.0/src/race.rs:236:39
|
236 |         match self.get_or_try_init(|| Ok::<&'a T, Void>(f())) {
|                                       ^^ not found in this scope

try using cargo clean and cargo update inside src-tauri

after more try, the error is:
Compiling parking_lot_core v0.9.9
error[E0463]: can't find crate for core
|
= note: the aarch64-apple-ios-sim target may not be installed
= help: consider downloading the target with rustup target add aarch64-apple-ios-sim
error[E0463]: can't find crate for compiler_builtins
For more information about this error, try rustc --explain E0463.
error: could not compile cfg-if (lib) due to 2 previous errors
warning: build failed, waiting for other jobs to finish...
Error [tauri_cli] Failed to run cargo build: command ["cargo", "build", "--package", "task", "--manifest-path", "/Users/natsugei/task/task/src-tauri/Cargo.toml", "--target", "aarch64-apple-ios-sim", "--features", "tauri/rustls-tls", "--lib", "--no-default-features"] exited with code 101
Command PhaseScriptExecution failed with a nonzero exit code
note: Run script build phase 'Build Rust Code' will be run during every build because the option to run the script phase "Based on dependency analysis" is unchecked. (in target 'task_iOS' from project 'task')
** BUILD FAILED **

how about just deleting the src-tauri/target directory all together then?

how about just deleting the src-tauri/target directory all together then?

not work

I am out of ideas, tbh try cleaning ~/.rustup or reinstalling rust through rustup again

I am out of ideas, tbh try cleaning ~/.rustup or reinstalling rust through rustup again

Could you tell how to do with the error Eo463.
rustc --explain E0463
A plugin/crate was declared but cannot be found.
Erroneous code example:
#![feature(plugin)]
#![plugin(cookie_monster)] // error: can't find crate for cookie_monster
extern crate cake_is_a_lie; // error: can't find crate for cake_is_a_lie
You need to link your code to the relevant crate in order to be able to use it (through Cargo or the -L option of rustc example). Plugins
are crates as well, and you link to them the same way.
Common causes

The crate is not present at all. If using Cargo, add it to [dependencies] in Cargo.toml.
The crate is present, but under a different name. If using Cargo, look for package =  under [dependencies] in Cargo.toml.

Common causes for missing std or core

You are cross-compiling for a target which doesn't have std prepackaged. Consider one of the
following: + Adding a pre-compiled version of std with rustup target add + Building std from source with cargo build -Z build-std +
Using #![no_std] at the crate root, so you won't need std in the first place.
You are developing the compiler itself and haven't built libstd from source. You can usually build it with x.py build library/std. More
information about x.py is available in the rustc-dev-guide.
(END)

about that, I don't know how to do.
I had tried: cargo add crate , fail: the crate crate could not be found in registry index.
so, about "The crate is not present at all. If using Cargo, add it to [dependencies] in Cargo.toml. " add what? I can't understand.

run create tauri-app --alpha, use new version to create repo

The issue is homebrew, which can't be invoked from xcode. run brew remove rust, then follow the directions for installing rust from the install page


Episode Idea: Kubernetes Admission Control and OPA

Specifically look at https://github.com/open-policy-agent/gatekeeper.  I'd like to do this one soon.

I'm going to do this this week: https://j.hept.io/tgik071
Monday or Tuesday:

[ ] Pick a topic and create/use an issue in the repo
[ ] If applicable coordinate with an upstream project if they're interested in attending the show
[ ] Create the graphic
[ ] Schedule on Youtube
[ ] Add scheduled episode to TGIK playlist on YT
[ ] Create short link in rebrandly
[ ] Tweet about it from your personal account

[ ] Post tweets on internal #tgik slack channel and cc @castrojo and @boschem
[ ] Get confirmation from @boschem that he's seen the tweet and is scheduling it

Before episode:

[ ] Create HackMD page for live notes.  Start with README template in this repo
[ ] Paste HackMD URL on internal #tgik channel and cc @castrojo or other assistant

[ ] Assistant adds ~5-10 news items from around the web as possible "News of the Week"

[ ] Get everything tested well before episode so you can debug
[ ] Use internal slack channel to ensure that AV setup is good

After show (ideally right away but can wait until Monday):

Create/submit PR for this repo

[ ] Episode directory and README from hackmd
[ ] Any other files from the episode that might be useful
[ ] Update playlist.md with episode
Feel free to self merge

[ ] Edit YT description to point to episode directory here
[ ] OBS/OS Upgrade if applicable (Never on the day of a show)

Links for the episode:

https://www.cncf.io/blog/2019/04/02/toc-votes-to-move-opa-into-cncf-incubator/
https://www.openpolicyagent.org/
https://www.openpolicyagent.org/docs/
https://github.com/open-policy-agent/gatekeeper
https://github.com/open-policy-agent/opa

I recommend getting started with the Kubernetes admission control tutorial on openpolicyagent.org: https://www.openpolicyagent.org/docs/kubernetes-admission-control.html. The tutorial takes you through deploying OPA from scratch, enabling the webhook, deploying a policy, exercising the policy, updating the policy, etc. This is how most people try out OPA for admission control for the first time.
In addition to the tutorial, we recently put together a primer/guide on writing admission control policies: https://github.com/open-policy-agent/opa/blob/master/docs/content/docs/guides-kubernetes-admission-control.md. The guide goes into detail on how admission control policies are structured and evaluated.
A couple other pointers that might be helpful:

Language Reference. This page lists all the various built-in functions for things like string manipulation, regexp, etc.

Rego Cheatsheet. Compact reference doc for common expressions in OPA (e.g., looking up values, iterating over arrays, etc.)

Lastly, when it comes to writing Rego, you could check out the VS Code extension and play.openpolicyagent.org. Both provide ability to evaluate the policies interactively.
As far as Gatekeeper goes, we're in the middle of restructuring the controller quite a bit and we don't have the same kind of docs yet. The policy authoring experience is more or less the same in Gatekeeper (i.e., users write deny rules that enforce constraints on input which gets bound to the AdmissionReview object.) Once Gatekeeper is ready, there will be additional features like a constraint library and audit capability that might be good fodder for a future episode :D

A more in depth article that @mauilion pointed me at: https://itnext.io/optimizing-open-policy-agent-based-kubernetes-authorization-via-go-execution-tracer-7b439bb5dc5b?gi=414d224ebdc7

HackMD: https://hackmd.io/e8NXFMdAQ06SeOuXkAZYmA?view


Question: Does this support RSA (RS256) algorithm?

Hello Team.
Thanks for this library. I am trying to convert JSON payload to COSE and vice versa. I am able to do it with ES256 but when I try to do it with RS256, it ends up with the error "The algorithm is not supported."
Please note I am using this dependency with version: 1.15
<dependency>
        <groupId>com.authlete</groupId>
	<artifactId>cbor</artifactId>
	<version>1.15</version>
</dependency>

Please see error below:
Caused by: com.authlete.cose.COSEException: The algorithm '-257' is not supported.
	at com.authlete.cose.COSESigner.sign(COSESigner.java:199) ~[cbor-1.15.jar:na]
	at com.authlete.cose.COSESigner.sign(COSESigner.java:137) ~[cbor-1.15.jar:na]
	at com.authlete.cose.COSESigner.sign(COSESigner.java:102) ~[cbor-1.15.jar:na]

Seems the RSA algorithm is not supported yet. If not, Is there any future plan to support?

Thank you for trying using this library. I'm sorry it's difficult for me to spare time to work on RS256 for now, but a PR to support other algorithms is welcome!

Thank for the reply and clarification @TakahikoKawasaki. I will have a look at it.


BUG: Attempt to concat string from object

Attempt to concat url from data object, instead of url
Only need to apply _t to query string if request is "GET"

In previous code, a GET would never have the _t query string appended

GET request should not have _t applied...would result in disabling cache - not really what you want.

From what I read of the original code:
if (!cache) { ...

The intention is to disable cache, when avoiding cache.

argh...just testing...pulled the last PR without checking to deep - but obviously messed up a lot. Looking deeper i think cache was appended wrong before already...should not be added to post - will result in wrong data added on send missing...

pulished in<EMAIL_ADDRESS>thanks for clearing that out


Completing with EMPTY observable

this is a continuation of discussion in #19
When an observable in an expression completes without emitting any value — it often makes whole expression impossible to emit. Here are some examples:
// source streams
const a = timer(0, 1_000);
const b = EMPTY; // imagine b comes from an external API

// combinations:
computed(() => ({ a: $(a), b: $(b) }))
computed(() => [$(a), $(b)])
computed(() => $(a) || $(b))

Currently we swallow this issue. And it's not that easy to detect this completion if computed doesn't report it in any way! Users would probably need to plug in some combination of tap pushing to Subject that would complete computed via takeUntil.
Though when it comes to conditions, it's not that obvious:
computed(() => $(a) < 5 ? $(a) : $(b)); // GOOD: values > 5 are endless
computed(() => $(a) > 5 ? $(a) : $(b)); // BAD: values > 5 are meaningless

But we don't know which of the two cases we face. So to choose what behavior is more obvious — I'd vote for completion with EMPTY. It's very much expected in simpler cases and in conditional cases user would be able to easily handle it with defaultIfEmpty operator or concat(NEVER), depending on the need.
Completion with EMPTY also has a feature to it: users would be able to intentionally complete their streams:
computed(() => $(a) < 5 ? $(a) : _(EMPTY));

That's like a.pipe( takeWhile(x < 5) ) in an expression!
@Jopie64 as to the case you've mentioned with unsubscription and filtering: IMHO, it makes sense that if user wants to unsubscribe eagerly — they would mark it as weak, just as we intended:
const r = computed(() => {
   switch($(a)) {
      case 0: return $.weak(b); // weak b
      case 1: return $(c); // intentionally normal c
      default: return $(NEVER); // skip emission if not 0 or 1
   });

While default: return $(EMPTY) would mean that user wants to complete the stream.
LMKWYT 🙂

The takeWhile example can be done without the new spec like this:
computed(() => _(a) < 5 ? $(a) : someFinalValue); // where someFinalValue could also be _(EMPTY)

Do you have another way to use this new spec as a feature?
My thinking is... if this spec is only there to help a user when he/she probably made a mistake, then this is less worth it if it blocks other users from using $(EMPTY) as a feature the way it is currently working (or could have been working when we would allow it to be used to pause the stream with unsubscribing normal bound observables). Sentence too long. Does it still make sense? :) I'd consider it less of a problem when a stream could be paused where it would unsubscribe unused streams, without explicitly binding it weakly.
But on the other hand, you might be correct that this semantic would help users significantly and maybe there will be other unforeseen use-cases on top of the takeWhile() like case. Also it explains nicely. NEVER for pause, EMPTY for stop.
These are my thoughts. So I guess I'd be ok when this is in the spec.

You're right, maybe a case with $(a) < 5 ? $(b) : $(EMPTY) ? But I'm just picking examples out of thin air to fit my point 🙂
And I agree that it can be a feature of a filter with completion unused normal subscriptions. But between the two feature options, I think the "NEVER for pause, EMPTY for stop" is simpler to understand, explain & reason about.
Yet, I'm most concerned about uncompletable expressions. Though it turns out that combineLatest stays open even if one of the two streams is EMPTY. Which confuses me 🙂
I'm not 100% sure now.
I'll let this hang for a few days and if no more thoughts come up — I'll create a PR (also to see it in action). Still debatable & revertible!


XDF.jl (Julia importer)

I know that this is not the best place, but I wanted to let you know that I've written a Julia importer for XDF:
https://github.com/cbrnr/XDF.jl
If you want to give it a try, let me know how you like it and be sure to report any issues and feature requests you might have.

You could open a PR to include it as submodule in the https://github.com/sccn/xdf repository

Good idea!

@cbrnr do you have any plans to migrate that to xdf-modules at some point?
If so, I'm wondering whether it incurs any hassle on the end-user side once that submodule is flipped from one upstream repo URL to another... does anyone have experience with that?

I would assume that GitHub redirects work even if it's a submodule. Even if not, as long as we update the submodule pointer within the parent xdf repo as soon as the move is made, then it should be transparent to users. The default operations of git pull && git submodule update will first update the submodule URL before pulling from the new URL.


Add os.Signal handling to mixs

This PR adds a simple handler for os.Signals to the Mixer server process.
In particular, this supports Interrupt (SIGINT) and SIGTERM. Upon such a signal,
a graceful stop of the gRPC server is attempted.
This PR is added to provide proper mixer container termination support for k8s.

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

Needs approval from an approver in each of these files:

mixer/OWNERS

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

Looking into this more than my initial superficial attempt, adding a more graceful interrupt handler to all the various subsystems -- including the runtime -- within Mixer is a bit more involved than I'm willing to dig into right now.

 Can you just add the handler go-routine in Run and document the problems with it?


Adding sample endorsement language with icon

Re https://github.com/social-science-data-editors/DCAS/issues/6 , uses https://github.com/social-science-data-editors/DCAS/issues/20

I made the change myself. I added dynamic version numbering, @larsvilhuber please check on DCAS-test if it is working.

Can't check on my test site, since it is hard-coded to work on the root domain name. You'll have to trust that it works.


"Check for duplicates" feature while reporting an issue

Add a button to check for duplicate issue while reporting a new issue, similar to this -

/assign


Added eslint-plugin-import to prevent circular dependencies

Addressing #10
Plugin found here: https://github.com/benmosher/eslint-plugin-import
Advice found here: https://railsware.com/blog/how-to-analyze-circular-dependencies-in-es6/
Plugin use described in package.json, this could be moved to .eslintrc.yml or similar at a later point

I suggest that you create a circular dependency in one commit to make sure that Travis fails, then correct it in another. Then we can be confident that this check is working.

Already found circular dependencies in this project with this rule, suggesting it works. Project was running fine but bad practice to have these circular dependencies hanging around.
Clean up underway before merging.


Make imports explicit, test with ExplicitImports.jl

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 83.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 88.83%. Comparing base (116f027) to head (376e79b).

Files with missing lines
Patch %
Lines

test/runtests.jl
71.42%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #188      +/-   ##
==========================================
- Coverage   90.77%   88.83%   -1.94%
==========================================
  Files          48       48
  Lines        2113     2078      -35
==========================================
- Hits         1918     1846      -72
- Misses        195      232      +37

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


Combine transaction

fix issue #132
comment: it is a little different from the designs, since every action has been written into a component that shows the whole information, if changed to what design shows, there will be many changes and create several new component to reuse and inspect for different situation. Have a look and give feedback about this

I like the designed version better as it does not have duplicated information which may confuse people.

@icerove I agree with @frol here. We want to communicate to the user that each action was one transaction,  by one user,  confirmed with one timestamp.

okay, I will do that.

@icerove Just noticed some missing formatting on the TX detail and block detail pages.

On these pages, we can remove the "Batch Transaction" text, timestamp, and sender (since it's listed above).

Forgot to spec this before!

@icerove Just noticed some missing formatting on the TX detail and block detail pages.
Make sence to TX detail page but block detail page still need batch I think

Make sence to TX detail page but block detail page still need batch I think

Good point!

single transaction is not correct on detail page

@icerove Let's center the action descriptions here:

@kcole16 You mean in detail page, every action is in the center? I am not very sure what do you want?

@kcole16 You mean in detail page, every action is in the center? I am not very sure what do you want?

Ah right, I meant align the action description (e.g. "New key added") with the icon on the left, like this:

@frol Think this is ready to merge?

I think it is ready to merge.
@icerove Please, review my changes.

I have been considering better naming for ActionRowBlock and realized that we use a similar block layout for AccountRow and BlockRow, so we should consider moving ActionRowBlock to utils, rename it to something more generic, and reuse it everywhere.

I will merge it and open a new PR to make changes for ActionRowBlock


[Enhancement] Optimize judgment logic when producing compaction task for single disk

Proposed changes
When producing compaction task for a particular disk, each tablet on BE will be traversed.  Usually, there are several disks on the BE, so that most of tablets locate on other disks which should be skipped.  To reduce some invalid judgment logic, the first thing we should do is to judge whether the tablet is on the target disk.
Types of changes
What types of changes does your code introduce to Doris?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[ ] Code refactor (Modify the code structure, format the code, etc...)
[x] Optimization. Including functional usability improvements and performance improvements.
[ ] Dependency. Such as changes related to third-party components.
[ ] Other.

HI @weizuo93 , I am try to solving this issue #6355, and it will cover the optimization your did here.

HI @weizuo93 , I am try to solving this issue #6355, and it will cover the optimization your did here.

OK. I'm looking forward to your PR.

#6365 solve this problem


[Bug/Feature Request] Resize Event Listener not appropriate when container option not window

Currently VirtualList attaches to the 'resize' event on the supplied container option.
This is fine when window is supplied as the container, but will not fire when an alternate element is supplied.
There are a number of approaches for detecting element size change of non-window elements that could be integrated to correctly trigger this.refreshState.
Happy to submit a PR if you wanted to choose which lib you liked best.

Personally, I think the ResizeObserver pattern with a note about requiring a polyfill would be best.


Associate DOI to surface

It would be useful to have allow a list of DOI attached to each surface that point to the publications where this surface has been used/described.

We should add an option to add a multiple DOIs to already published surfaces, because a paper often appears after the data has been published.

More explanation:

In a published paper, we point to a published publication using the contact.engineering/go identifier.

From contact.engineering, we should also be able to point back to all publications in which a certain surface was used. For this we should support adding a list of DOIs, or more generically URLs. (We should probably distinguish between generic URLs and DOIs.)

This means we have mutual pointers from data to publication and back.

Now it's clear, thanks!


A bunch of fixes for variadic tuple properties in structs

Builds on https://github.com/apple/swift/pull/64517.

@swift-ci Please test

@swift-ci Please smoke test macOS

@swift-ci Please smoke test macOS

@swift-ci Please test

@swift-ci Please smoke test macOS

@swift-ci Please smoke test

@swift-ci Please test

I think I spent an entire day not being able to land this PR despite it never breaking anything itself.

@swift-ci Please smoke test


Battery percentage and altitude percentage is always 0

Hello,
Firstly thanks for the great library. I am having some problems with functions such as getBatteryPercentage() and getAltitude(). They always return 0(even though battery is fully charged).
Do you have any idea what could be a cause to this?
Thanks,
Fiz.

Hi
I had the same problem when run the sample program on Ubuntu (or OS X?).
What about switching navdata_demo=TRUE ?
// ./src/ardrone/navdata.cpp
// AR.Drone 2.0
if (version.major == ARDRONE_VERSION_2) {
    // Disable BOOTSTRAP mode
    if (mutexCommand) pthread_mutex_lock(mutexCommand);
    sockCommand.sendf("AT*CONFIG_IDS=%d,\"%s\",\"%s\",\"%s\"\r", ++seq, ARDRONE_SESSION_ID, ARDRONE_PROFILE_ID, ARDRONE_APPLOCATION_ID);
    sockCommand.sendf("AT*CONFIG=%d,\"general:navdata_demo\",\"TRUE\"\r", ++seq); // HERE !!
    //sockCommand.sendf("AT*CONFIG=%d,\"general:navdata_demo\",\"FALSE\"\r", ++seq); // NOT USE
    if (mutexCommand) pthread_mutex_unlock(mutexCommand);
    msleep(100);

    // Seed ACK
    sockCommand.sendf("AT*CTRL=%d,0\r", ++seq);
}

If getBatteryPercentage() and getAltitude() work, it seems there is some bug in ARDrone::getNavdata().
Another solution is to update the firmware of your drone to the latest version.
Thank you,
puku0x

Hi,
Thanks for the reply. I am running it on OS X. I have read about previous issues that had been opened, and have already changed navdata_demo to true, but it seems like  it doesn't help. In addition, the latest version of firmware is installed  on the drone. Probably there is another bug in getNavData function.
Thanks,
Fiz

Hi,
Thank you for your report.
I will check it tomorrow.
Regards,
pukuox

Hi,
Sorry for my late reply.
Unfortunately, I could not reproduce the issue.
--
AR.Drone Ver. 2.4.8
OS X 10.11 El Capitan
OpenCV <IP_ADDRESS> (via MacPorts)
CV Drone https://github.com/puku0x/cvdrone/commit/15dd04305d6b5f9d3a11c14666aa88fe831c207f

Hi there,
Thanks for looking into this.
I am not sure if this is what causes the issue, but I have OpenCV 2.4.12 (via brew).
I will upgrade, check again and report back.
Thanks,
Fiz.

It worked after upgrading libraries. Thank you

Please tell me how to upgrade the libraries on windows, i am having the same problem.


Replace jieba3k with jieba

jieba3k package is outdated. It is last updated in 2014.
Main jieba is Python3 compatible since 2015.

Good idea @pistolero

Will look into investigating and integrating this

Any progress on this?

+1 waiting for fix due to this issue raised on mac os https://github.com/fxsjy/jieba/issues/250

+1 waiting for fix due to this issue raised on mac os https://github.com/fxsjy/jieba/issues/250