Multi-thread receive of packets

Receive packets in multiply threads

Sorry for the late reply. Does it improve receiving packets in large host environment e.g. yours?

Yes it does. On Linux with 24-cpu-core it's now possible to do ping in 48000ip chunks with exact latency returned. It's much better than it was :)

It merged into 42755b0. Now both send and recv are multiplexed so procRecv can be removed and merged into sendICMP. After doing it and some more refactoring, I will merge the branch into master.

Does this fix #14 ?

@kkirsche, it should be. I will do more work on this.

I ping 800 ips  every 1 second, my machine has 4 core cpu and 16G memory , and it occured packets loss when pinging some hosts .
tcpdump results show that the host's  echo reply packet actually arrived .
is there  something wrong when i use the fastping package?
below is my code:
`package cping
import (
        "fmt"
        "github.com/kanocz/go-fastping"
        "log"
        "net"
        "os"
        "syscall"
        "os/signal"
        "strings"
        "time"
)
var pingIP string = ""
var rowcounter int
var recvcounter int
var maxPingCount int = 999 // maxPingCount should be greater than Count, or it will ping at maxPingCount, then quit
var ipList []string
var pingStats map[string]stats
type milliDuration time.Duration
func (hd milliDuration) String() string {
        milliseconds := time.Duration(hd).Nanoseconds()
        milliseconds = milliseconds / 1000000
        return fmt.Sprintf("%d", milliseconds)
}
func (hd milliDuration) Int() int {
        milliseconds := time.Duration(hd).Nanoseconds()
        milliseconds = milliseconds / 1000000
        return int(milliseconds)
}
type stats struct {
        min   int
        max   int
        avg   int
        count int
        rtts  []int
}
type PingStat struct {
        Pktloss  float32
        Resptime int
}
type response struct {
        addr *net.IPAddr
        rtt  time.Duration
}
func checkHostErr(host string, err error) {
        if err != nil {
                log.Fatal(host, ": ", err)
        }
}
func GoPing(ipListInput []string, hosts map[string]string)  {
pingIP = strings.Join(ipListInput, " ")
pingStats = make(map[string]stats, 1000)
result := make(map[string]string, 1000)
onRecv, onIdle := make(chan *response, len(ipListInput)), make(chan bool)
c := make(chan os.Signal, 1)
signal.Notify(c, os.Interrupt)
signal.Notify(c, syscall.SIGTERM)
p := fastping.NewPinger()
p.MaxRTT = time.Second
for , host := range strings.Fields(pingIP) {
ra, err := net.ResolveIPAddr("ip:icmp", host)
checkHostErr(host, err)
p.AddIPAddr(ra)
ipList = append(ipList, ra.String())
result[host] =  "null"
}
p.OnRecv = func(addr *net.IPAddr, rtt time.Duration) {
onRecv <- &response{addr: addr, rtt: rtt}
}
p.OnIdle = func( map[string]*net.IPAddr) {
onIdle <- true
}
//p.Debug = g.Config().Debug
p.RunLoop()
loop:
for {
select {
case <-c:
fmt.Println("get interrupted")
break loop
case res := <-onRecv:
addr_string := res.addr.String()
//log.Println("Receive echo reply from ", addr_string)
recvcounter ++
if _, ok := result[addr_string]; ok {
result[addr_string] = milliDuration(res.rtt).String()
stats := pingStats[addr_string]
stats.rtts = append(stats.rtts, milliDuration(res.rtt).Int())
pingStats[addr_string] = stats
}
case <-onIdle:
log.Println("---------------------------------------------")
log.Println("The total ip to ping is ", len(ipList))
log.Println("The receive counter is ", recvcounter)
// reset packet  receive counter
recvcounter = 0
for _, ip := range ipList {
if result[ip] == "null" {
log.Printf("Can not receive reply from %s at this turn \n", ip)
stats := pingStats[ip]
stats.rtts = append(stats.rtts, -1)
pingStats[ip] = stats
}
// reset result of every turn
result[ip] = "null"
}
rowcounter++
//              log.Println(pingStats)
if rowcounter == maxPingCount {
break loop
}
if (rowcounter > 0) && (rowcounter%g.Config().Count == 0) {
for hostname, ip := range hosts {
s := GetHostIpStat(ip)
fmt.Println(s)
}
// reset pingStats , rowcounter , recvcounter
pingStats = make(map[string]stats, 1000)
rowcounter = 0
}
log.Println("=============================================")
case <-p.Done():
if err := p.Err(); err != nil {
fmt.Println("Ping failed:", err)
}
break loop
}
}
signal.Stop(c)
p.Stop()
}
func GetHostIpStat(host string) *PingStat {
stat := pingStats[host]
ploss := 0
for _, entry := range stat.rtts[0:rowcounter] {
if entry != -1 {
if entry > stat.max {
stat.max = entry
}
if entry < stat.min {
stat.min = entry
}
stat.avg = stat.avg + entry
} else {
ploss++
}
}
if ploss == rowcounter {
stat.min = -1
stat.max = -1
stat.avg = -1
} else {
stat.avg = stat.avg / (rowcounter - ploss)
}
plosspct := float32(ploss) / float32(rowcounter) * 100
if g.Config().Debug {
log.Printf("%-38s: %4d %4d %4d %5d/%d (%.2f%%)\n", host, stat.min, stat.max, stat.avg, ploss, rowcounter, plosspct)
}
return &PingStat{
Pktloss:  plosspct,
Resptime: stat.avg,
}
}
`

@kanocz  i use your fork of the fastping. you say that you test 48000ip ping with exact result. could you tell me how to get it ?  looking forward to your reply . thank a lot.
@others or others  has the  answer?

@hackstoic I'm using much more modified (stripped down) version of go-fastping in live projects (most mutex locks removed, ipv6 support removed, only single run supported, single context, so no re-run until finished and so on). If you need I can little bit polish it and push commit to my repo

@kanocz  yes . I appreciate that if you push the new commit. I also use the code in live projects which need to ping more than 20, 000 ips and collect the packet loss and response time .   Thank you again.

@kanocz  i suffer the incollect loss rate these day . but i can't find the root cause.  i refactor my code  but it still don't receive the icmp reply sometime. (i made tcpdumps again and again, and be sure that the icmp reply had actually arrived).

@kanocz  paste my code here. (i hope it won't  annoy you )
package cping
import (
"fmt"

"github.com/kanocz/go-fastping"

"github.com/zabbix_ping/g"

"log"

"net"

"os"

"syscall"

"os/signal"

"strings"

"time"

)
var pingIP string = ""
var rowcounter int
var recvcounter int
var maxPingCount int = 999 // maxPingCount should be greater than Count, or it will ping at maxPingCount, then quit
var ipList []string
var pingStats map[string]stats
type milliDuration time.Duration
func (hd milliDuration) String() string {
milliseconds := time.Duration(hd).Nanoseconds()

milliseconds = milliseconds / 1000000

return fmt.Sprintf("%d", milliseconds)

}
func (hd milliDuration) Int() int {
milliseconds := time.Duration(hd).Nanoseconds()

milliseconds = milliseconds / 1000000

return int(milliseconds)

}
type stats struct {
min   int

max   int

avg   int

count int

rtts  []int

}
type response struct {
addr *net.IPAddr

rtt  time.Duration

}
func checkHostErr(host string, err error) {
if err != nil {

    log.Fatal(host, ": ", err)

}

}
func GoPing(ipListInput []string, hosts map[string]string)  {
pingIP = strings.Join(ipListInput, " ")

pingStats = make(map[string]stats, 1000)

result := make(map[string]string, 1000)

onRecv, onIdle := make(chan *response, len(ipListInput)), make(chan bool)

c := make(chan os.Signal, 1)

signal.Notify(c, os.Interrupt)

signal.Notify(c, syscall.SIGTERM)

p := fastping.NewPinger()

p.MaxRTT = time.Second * time.Duration(g.Config().Timeout)

for _, host := range strings.Fields(pingIP) {

    ra, err := net.ResolveIPAddr("ip:icmp", host)

    checkHostErr(host, err)

    p.AddIPAddr(ra)

    ipList = append(ipList, ra.String())

    result[host] =  "null"

}

p.OnRecv = func(addr *net.IPAddr, rtt time.Duration) {

    onRecv <- &response{addr: addr, rtt: rtt}

}

p.OnIdle = func(_ map[string]*net.IPAddr) {

    onIdle <- true

}

p.Debug = g.Config().Debug

p.RunLoop()

loop:
for {

    select {

    case <-c:

        log.Println("get interrupted")

        break loop

    case res := <-onRecv:

        addr_string := res.addr.String()

        //log.Println("Receive echo reply from ", addr_string)

        recvcounter ++

        if _, ok := result[addr_string]; ok {

            result[addr_string] = milliDuration(res.rtt).String()

            stats := pingStats[addr_string]

            stats.rtts = append(stats.rtts, milliDuration(res.rtt).Int())

            pingStats[addr_string] = stats

        }

    case <-onIdle:

        log.Println("---------------------------------------------")

        log.Println("The total ip to ping is ", len(ipList))

        log.Println("The receive counter is ", recvcounter)

        // reset packet  receive counter

        recvcounter = 0

        for _, ip := range ipList {

            if result[ip] == "null" {

                log.Printf("Can not receive reply from %s at this turn \n", ip)

                stats := pingStats[ip]

                stats.rtts = append(stats.rtts, -1)

                pingStats[ip] = stats

            }

            // reset result of every turn

            result[ip] = "null"

        }

        rowcounter++

//        log.Println(pingStats)

        if rowcounter == maxPingCount {

            break loop

        }

        if (rowcounter > 0) && (rowcounter%g.Config().Count == 0) {

            ZabbixSender(hosts)

            // reset pingStats , rowcounter , recvcounter

            pingStats = make(map[string]stats, 1000)

            rowcounter = 0

        }

        log.Println("=============================================")

    case <-p.Done():

        if err := p.Err(); err != nil {

            fmt.Println("Ping failed:", err)

        }

        break loop

    }

}

signal.Stop(c)

p.Stop()

}

Sorry, I'm little too busy at main work, but I hope to publish my version of go-fastping by the end of the week. I have removed almost all using of mutex and so on, so it several times less  syscalls to kernel and synchronizations between processor cores.

@kanocz  Thanks. I can understand. And I will keep watch to your repo and  try to solve this problem by myself at the same time.

@hackstoic sorry, it takes a little bit more time to prepare patch that can be publishes :-(
but please note that I've removed many functions like RunLoop(), any preparation have to be done in one thread + no controls about any modification of settings during the ping (this will end with panic)
this is little bit cleaned up version compared to that I'm using in real life


trans function CSP issue

obviously we don't want user to write such code...
it is most natural for user to write fun o -> Obj o, instead of fun o -> .<Obj .~o>.
https://github.com/marklrh/pot/blob/master/prototype/test_parser_generator.ml#L33

want a function f of type (a -> b) -> (a code -> b code) with no CSP issue...

maybe it is impossible.
For example,
in file Typedef,
type t = A of int

and in another file,
open Print_code
open Format

let pp_code code = print_code Format.std_formatter code

let test6 () =
  let open Typedef in
  let f = fun x -> A x in
  let g = fun x -> .<f .~x>. in
  pp_code .<fun x -> .~(g .<x>.)>.

let () = test6 ()

We get
Warning 22: The CSP value is a closure or too deep to serialize
.<fun x_1  -> (* CSP f *) x_1>.

semantic action, defunctionalization


chore: Set up the initial project

🚀 Pull Request
🔗 Linked Issue

Closes #11
📝 Description

This PR sets up an initial Svelte project for the landing page.
🛠 Changes Made

[x] Initial Svelte project is set up
[x] Zufall Labs specific files are added
[x] Initial styling is done

🧪 Testing

As this is only a project scaffold, there are no tests.
📸 Screenshots (if applicable)

🔍 Additional Notes

This PR does not contain any major features, instead it is only a project scaffold
✅ Checklist

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

👥 Reviewer Checklist

[ ] The code change is focused and makes sense
[ ] The code adheres to our established coding standards
[ ] The changes are adequately tested
[ ] The documentation is updated to reflect the changes (if necessary)

💖 Thank you for contributing to Zufall! 💖

This PR should probably not be merged alone, but instead together with https://github.com/zufall-labs/web/pull/9.
My suggestion would be to

review this PR
work on https://github.com/zufall-labs/web/pull/9 until it is finished
review https://github.com/zufall-labs/web/pull/9
merge this PR
merge https://github.com/zufall-labs/web/pull/9.

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


TouchableHighlight or TouchableOpacity doesn't work inside Callout

This is my current code setup in render
<View style={styles.view_container}>

              <MapView
                    style={styles.map}
                    region={this.state.region}
                    showsUserLocation={YEAH}
                    followsUserLocation={YEAH}
                    onRegionChange={this.onRegionChange}
                  >
                    {this.state.markers.map(marker => (
                      <MapView.Marker
                        coordinate={marker.latlng}
                      >

                         <MapView.Callout>
                            <CustomCallout {...marker}/>

                         </MapView.Callout>

                        </MapView.Marker>
                    ))}
              </MapView>
              <TextInput />

            </View>

I can't seem to get the custom callout to listen to presses done inside the CustomCallout. The code is this.
class CustomCallout extends Component{
  paypal(){
    console.log('presseeedd')
    }

  componentDidMount(){
    console.log('custom call props', this.props.title);
  }

  render(){
        return (
          <View style={{flex:1, flexDirection:'column'}}>
              <Text>{this.props.title}</Text>
              <Text>{this.props.description}</Text>
              <TouchableHighlight onPress={this.paypal.bind(this)}>
                <Text>Reserve Now</Text>
              </TouchableHighlight>
          </View>
        )
  }
}

Is this happening for Android and iOS?

It's happening on Android.

Seems to be linked https://github.com/lelandrichardson/react-native-maps/issues/211

Android map callouts don't support buttons inside of them, only an onPress event for the entire callout. This is a limitation of the Google Maps library on Android and react-native-maps can't do anything about it.

Closing this one too.

@jojonarte hey Jojo, were you able to solve it?

@eugenehp Read my previous comment.

Ross, thanks. Let me rephrase it, have you found any solution on top of the
maps that allowed you to do your initial intend? Basically I'm asking what
you end up using in it. Thanks in advance!
On Friday, June 24, 2016, Ross<EMAIL_ADDRESS>wrote:

@eugenehp https://github.com/eugenehp Read my previous comment.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lelandrichardson/react-native-maps/issues/226#issuecomment-228514021,
or mute the thread
https://github.com/notifications/unsubscribe/ABxW7xOtR8bcUnB8GeD7WjssQOeAK3Evks5qPMIdgaJpZM4IdujH
.

@eugenehp I wasn't able to, what I did to use it was just let the callout be tapped to view another scene with the full details of what's supposedly pointed by the callout.

Thanks Jojo, that makes sense. Thanks for your time! 👍
On Saturday, June 25, 2016, Jojo Narte<EMAIL_ADDRESS>wrote:

@eugenehp https://github.com/eugenehp I wasn't able to, what I did to
use it was just let the callout be tapped to view another scene with the
full details of what's supposedly pointed by the callout.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lelandrichardson/react-native-maps/issues/226#issuecomment-228532382,
or mute the thread
https://github.com/notifications/unsubscribe/ABxW72CsmrcO5Xpdar5Oea4iwNDxic04ks5qPQoJgaJpZM4IdujH
.

Glad I could help.

@jojonarte would you care to share an example of that? I'm trying to do something similar.

you can use CalloutSubView for this purpose
<Callout style={styles.calloutWidth}>
     <CalloutSubview
           style={styles.phoneContainer}
          onPress={() => {
       console.log('onPress Clicked')
      }}>

    <Icon
         name="directions-car"
         type="material"
         color={APP_THEME.APP_ORANGE}
         size={30}
          underlayColor="clear"
     />

       </CalloutSubview>
</Callout>

you can use CalloutSubView for this purpose
<Callout style={styles.calloutWidth}>
     <CalloutSubview
           style={styles.phoneContainer}
          onPress={() => {
       console.log('onPress Clicked')
      }}>

    <Icon
         name="directions-car"
         type="material"
         color={APP_THEME.APP_ORANGE}
         size={30}
          underlayColor="clear"
     />

       </CalloutSubview>
</Callout>

CalloutSubview works only on IOS

@heydabop
Is your answer still valid after 8 years or is there now a solution for Android too?


[Bulk] Validation: absolute links 7/11a

This PR fixes 26 absolute link issues.

Docs Build status updates of commit 44ac140:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/Outlook/object-model.md
:white_check_mark:Succeeded
View

api/overview/PowerPoint.md
:white_check_mark:Succeeded
View

api/overview/PowerPoint/object-model.md
:white_check_mark:Succeeded
View

api/overview/Project.md
:white_check_mark:Succeeded
View

api/overview/Publisher.md
:white_check_mark:Succeeded
View

api/overview/Publisher/object-model.md
:white_check_mark:Succeeded
View

api/overview/Visio.md
:white_check_mark:Succeeded
View

api/overview/Visio/object-model.md
:white_check_mark:Succeeded
View

api/overview/Word.md
:white_check_mark:Succeeded
View

api/overview/Word/object-model.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Microsoft Defender for Cloud simulation

Added the security threat simulation command on the readme, once ArcBox is ready, might me good to tailor it to specific resources names and maybe some screenshots.

added the screenshots


Adding PrincipalType property to RoleAssignments

Description
Include PrincipalType as part of the Role Assignment as per the documentation, if not set properly you might get intermittent deployment errors, especially when you work with service principals and managed identities. Link to documentation here.
(Shared by an external contributor<EMAIL_ADDRESS>

Agreed. We should do that asap. I just faced the same issue preparing a module for the public bicep registry.


Shapeable should not require a default serializer

If I have a client that's using Shapeable, I should be able to decide whether I want to include default serializers or not.

Marking this as closed. Shapeable 0.3.0 no longer requires default_version and default_shape.


test function is not working outside PBxplore folder

Hi,
By playing with conda to create a conda package, I realized that the test function I implemented in #122 doesn't work outside the PBxplore folder package :
(pbxplore)[santuz@temple:~]$ python -c 'import pbxplore; pbxplore.test()'
/home/temple/santuz/projets/depots/PBxplore/pbxplore/tests
PBxplore version 1.3.1
numpy version 1.11.0
numpy is installed in /home/temple/santuz/.virtualenvs/pbxplore/local/lib/python2.7/site-packages/numpy
MDAnalysis version 0.14.0
MDAnalysis is installed in /home/temple/santuz/.virtualenvs/pbxplore/local/lib/python2.7/site-packages/MDAnalysis
matplotlib version 1.5.1
matplotlib is installed in /home/temple/santuz/.virtualenvs/pbxplore/local/lib/python2.7/site-packages/matplotlib
weblogolib version 3.4
weblogolib is installed in /home/temple/santuz/.virtualenvs/pbxplore/local/lib/python2.7/site-packages/weblogolib
Python version 2.7.6 (default, Mar 22 2014, 22:59:56) [GCC 4.8.2]
nose version 1.3.7

----------------------------------------------------------------------
Ran 0 tests in 0.102s

OK

I found the bug and I'll create a PR

fixed in #130


Compile separated files?

Hi, I want to know if is possible to have something like app/styles/home.scss and having it compiled to dist/assets/home.css, without having to import it explicitly in app.scss

You should take a look at: http://www.ember-cli.com/user-guide/#configuring-output-paths

Is normal to specify explicitly in ember-cli-build.js the files that I try to compile?
Assets	                                       Output File
other CSS files in app/styles	same filename in /assets

I supposed that if I have a scss file in the css folder, it will compile automatically.

This is the way to create app in ember-cli-build.js.
var app = new EmberApp(defaults, {
  outputPaths: {
    app: {
      css: {
        'home': '/assets/home.css'
      }
    }
  }
});

Thanks!


Preselect Variant if Only One Variant

What I'm trying to achieve
Have the variant preselected on the product details page if there is only one variant for the given product.
I see that similar functionality was discussed briefly in this issue.
The conclusion was that preselecting a variant would cause users to accidentally checkout with the wrong color/size, etc.  I agree with this thinking but would like to propose a solution that I believe would allow this to be implemented and still mitigate accidental variant selections.
Proposed solution
If the product has only ONE variant, preselect it for the user and enable the "Add to basket" button.
If the product has MULTIPLE variants, do not preselect it for the user and keep the "Add to basket" button disabled as it is in the current implementation.
Why bother?
Implementing this would increase checkout conversions for shops with large product catalogs of one-of-a-kind items that only have one variant.  Decreasing the total clicks to checkout by 2 clicks is fairly significant.
I am learning TS and React and am working on implementing this myself.  However I could use some help!  If you help orient me as to how to accomplish this I would be very thankful.
Thank you.

I would like to see this feature too. In addition a variant marked as DEFAULT should be preselected too. Do you have an implementation yet? I would like to help you.

I don't have an implementation yet.  But I am still working on the storefront and would really appreciate some help with this feature!

I made it work. At least for my needs. Maybe not the best solution.
Add the following to the top of the useEffect() function in ProductVariantAttributeSelect.tsx
   if (productVariantsAttribute.values.length === 1) {
      defaultValue = productVariantsAttribute.values[0].value!;
    }

Change this  line to:
const selectedValuesList = selectedValue ? [selectedValue.value] : [defaultValue!];

I made it work. At least for my needs. Maybe not the best solution.
Add the following to the top of the useEffect() function in ProductVariantAttributeSelect.tsx
   if (productVariantsAttribute.values.length === 1) {
      defaultValue = productVariantsAttribute.values[0].value!;
    }

Change this  line to:
const selectedValuesList = selectedValue ? [selectedValue.value] : [defaultValue!];


keeps getting behind

It works fine on windows, but on mac, it very quickly will fall behind. The app will think I'm on a song that was already played 2 songs ago, and the only way to fix it is to disconnect and reconnect after every song. Incredibly annoying.

i think i know the fix, you have to have the iTunes RP as your primary window (you have to have it as the latest window opened, so having it in front of discord and any other applications, that seems to make it not fall behind [ik this can get annoying since youll constantly have to keep it in front of your discord when chatting every 15 seconds or so but yeah it fixes the issue, and it doesnt seem to appear to have this issue on a later mac model. it works for my M1 2020])


Allow users (User account Role) to delete / archive events

Description
This PR fixes: https://github.com/apache/cloudstack/issues/5519. Users unable to archive / delete their events.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[X] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?
Was able to delete / archive events  when logged in with user account role.

@blueorangutan ui

@blueorangutan ui

@rhtyd a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5520 (SL-JID-704)

@Pearl1594 somehow the qa.cloudstack.cloud URL with this PR fails on log in (says caught issue) but it works on the normal URL. Can you check?

Could it be some caching issue @rhtyd ? As I was able to login just fine:


Btm 536 refactor test helper types

refactored int test helpers to use own return type rather than aws ones and updated corresponding tests

While we're changing types could we stop using StringObject and just use Record<string, string>?


Add Bytes and BytesMut helpers

These make it more convenient to read/write Pod types, and harder to cause indexing panics when reading.
Fixes #32
Absence of indexing panics checked using findpanics.
@fitzgen src/pod.rs is probably the most in need of review (including the unsafe code).

Coverage increased (+1.6%) to 54.872% when pulling ba3602187584250a1029141b6662a624f7b70115 on philipc:bytes into 368adf180fd62ef6b8522d703a4425abb53f00d9 on gimli-rs:master.


upload method uploads as tmp file

Hi, on the example i find $bucket->upload(     fopen('/data/file.txt', 'r') );
Following that example, I do it from a html form to upload a file:
$bucket->upload(     fopen($_FILES['uploaded_file']['name'], 'r') );
But what I finish uploading is the tmp file, I mean, the file is named as the tmp file. How to get rid on this issue?
Regards.

Hey @NestorAcevedo, thanks for filing this issue.
All you should need to do to name the file differently is to supply the name optional argument.
$bucket->upload(
    fopen($_FILES['uploaded_file']['name'], 'r'),
    ['name' => 'my_file.txt']
);

I want to upload to storage using remote file (url), is it possible?

Thank you @dwsupplee it worked.

@dwsupplee you should use stream_get_contents to achieve it.

@dungvaph03374 fopen can work with URLs:
http://php.net/manual/es/function.fopen.php


Use insert-file-contents when reading process stderr

The function straight--process-call was using find-file-noselect to read in the standard error after the process exits. This apparently runs a bunch of hooks and potentially turns on minor modes, which can be disruptive and slow. (Context of the particular issue I had is below.)
It seems like the recommended way to just read a file to a string is with-temp-buffer and insert-file-contents. See, for example, top answer to “What's wrong with find-file-noselect?” on stackexchange.
So I think this should be an invisible, work-alike improvement to an internal function.
Now, for my context: I'm using undo-tree, and by default, undo-tree-auto-save-history so that undo info is saved across killing and reopening the file or restarting Emacs. I also configured the recommend advice to gzip the undo-tree files, which by default is verbose.
With that configuration in place, whenever I ran a batch straight process such as straight-freeze-versions or straight-normalize-all or straight-fetch-all, the useful echo-area progress output of straight would be completely overwhelmed by repeated "compressing undo-tree" messages referencing the straight-stderr-NNNN file... something like this:
Processing repository "rainbow-mode"...
compressing .straight-stderr-1462415.~undo-tree~.gz...done
Wrote /tmp/.straight-stderr-1462415.~undo-tree~.gz
compressing .straight-stderr-1462415.~undo-tree~.gz...done
Wrote /tmp/.straight-stderr-1462415.~undo-tree~.gz
[...Above 2 lines repeat about 14 times...]
Processing repository "rainbow-mode"...done
Processing repository "nix-mode"...
compressing .straight-stderr-1462415.~undo-tree~.gz...done
Wrote /tmp/.straight-stderr-1462415.~undo-tree~.gz
[...14 times...]
Processing repository "nix-mode"...done
Processing repository "echo-bar.el"...
compressing .straight-stderr-1462415.~undo-tree~.gz...done
Wrote /tmp/.straight-stderr-1462415.~undo-tree~.gz
[...etc...]

Obviously, undo-tree is not relevant when straight is trying to read its stderr file after each process call. At first I pursued ways to prevent undo-tree from activating on this file. That seems harder than I expected – something like this was apparently not enough:
  (add-to-list 'auto-mode-alist (cons "straight-stderr-" #'special-mode))
  (add-to-list 'undo-tree-incompatible-major-modes #'special-mode)

I thought about hacking undo-tree so it can prevent turning on by filename regexp and not just major-mode. But when I read the tips about find-file-noselect it seems to me that hacking straight as shown here is the Right Thing™ instead.  Thanks!

That looks great. I wondered about optimizing the nested with-temp-buffer, so the REPLACE is a great idea.  I'm surprised if buffer-substring-no-properties is making a difference, because there shouldn't be any properties (no sentinel manipulating the standard out). But indeed, if there were properties we wouldn't want them! I'll build and try with this, then add a commit, feel free to squash of course.
Should this have a changelog for an internal improvement?

I wondered about optimizing the nested with-temp-buffer, so the REPLACE is a great idea.

Seems to work good on my end with Emacs 27, 28, and 30.

I'm surprised if buffer-substring-no-properties is making a difference, because there shouldn't be any properties (no sentinel manipulating the standard out).

I think the actual C function is just faster.
That seems to be the general case.

Should this have a changelog for an internal improvement?

I think we're good here.

feel free to squash of course.

I've done so and merged to "develop". Thanks again!


Production Level Updates

If the maintainers of the repo are open to pull requests, would they be interested in a pull request for code cleanup and some production level updates? There are a lot of things in the code that just don't make sense, such as setting the max sequence length and then checking the max sequence length, or neurotically sending models to devices. This isn't great for a more production level deployment when we want to embed millions or billions of documents.
I am going to be making these changes, but I don't want to fork and diverge from the repo. I'd rather that my work be pulled into this repo so that I not only don't have to worry about downstream updates but also it will benefit anyone that uses this.

Hi, Thanks a lot for your interest in the INSTRUCTOR!
I welcome all levels of code cleanup! Feel free to open the pull request!

Raised one such PR
https://github.com/xlang-ai/instructor-embedding/pull/92


xml-generator: be compatible with both python2.7 and 3.x

The xml-generator scripts were using deprecated Python 2 constructs and were
unable to run with Python 3.x. This is an issue on Windows because CMake seems
to use the highest version available even when there is a version closer to
what was requested. If both 2.7 and 3.x are installed on the system,
find_package(PythonInterp 2.7 EXACT REQUIRED)

may fail and
find_package(PythonInterp 2.7 REQUIRED)

may find 3.x.
This is a known issue for some time now:
https://public.kitware.com/Bug/view.php?id=15661

This pull request is now managed with CodeReviewHub. Refresh.

Current coverage is 80.30%

Merging #358 into master will not affect coverage as of e380905

@@            master   #358   diff @@
=====================================
  Files          211    211
  Stmts         7184   7184
  Branches         0      0
  Methods          0      0
=====================================
  Hit           5769   5769
  Partial          0      0
  Missed        1415   1415

Review entire Coverage Diff as of e380905

Powered by Codecov. Updated on successful CI builds.

:+1:

@krocard Please review.

:+1:


Export FilterTypes enum

In order to be used in TypeScript, FilterTypes has to be exported. Before this change, we had to use @ts-ignore everywhere we used it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.411%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1164

Relevant Lines:
1557

💛  - Coveralls


⚠ Bofrost.fr has degraded performance

In 52dc4dd, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1649 ms

Resolved: Bofrost.fr performance has improved in 267f9e3.


Move the implementation of the RawTag from tags.impl to tags

Move the implementation of the RawTag from tags.impl to tags.

The implementation of ILTagHeaderImpl had to be moved to tags too.


[nitro-protocol] Tweak readme, add test folder to npm package

and patch version++

Rewrote the readme to make a bit more sense on npm https://github.com/statechannels/monorepo/blob/687a6c53b0a1e5f85834ce1f44e9b2efe6d0d100/packages/nitro-protocol/readme.md


Added error handle

Added error handle

Hello @rw209, thanks for taking time for this. As a major refactoring has been completed, I will close this PR now and I will consider creating a new one.


[TEST] Skip TensorCore TOPI test when it is not available

https://github.com/apache/tvm/pull/7146 Introduces a test on tensorcore for batchmatmul
Because tensorcore is only optionally supported in the topi testing stage, it would be great to skip it when it is not available.

cc @jwfromm @jcf94 @Meteorix

cc @jwfromm @jcf94 @Meteorix

(Posting here as I think it is related)
When running TVM tests on a CPU only machine, I'm getting an error when running /test_topi_batch_matmul_tensorcore.py:test_batch_matmul:
https://github.com/apache/tvm/blob/89e3688137b9d8dd0e431cbdadb84d42dae9eee3/tests/python/topi/python/test_topi_batch_matmul_tensorcore.py#L66-L71
The error I see is:

Check failed: allow_missing == false: Device API gpu is not enabled.

The test is marker as uses_gpu.
I wonder whether we should move it to either requires_gpu or even requires_tensorcore?

(Posting here as I think it is related)
When running TVM tests on a CPU only machine, I'm getting an error when running /test_topi_batch_matmul_tensorcore.py:test_batch_matmul:
https://github.com/apache/tvm/blob/89e3688137b9d8dd0e431cbdadb84d42dae9eee3/tests/python/topi/python/test_topi_batch_matmul_tensorcore.py#L66-L71
The error I see is:

Check failed: allow_missing == false: Device API gpu is not enabled.

The test is marker as uses_gpu.
I wonder whether we should move it to either requires_gpu or even requires_tensorcore?

I think requires_tensorcore is the right macro cc @tkonolige

I think requires_tensorcore is the right macro cc @tkonolige

Yes, it should be requires_tensorcore.

Yes, it should be requires_tensorcore.


Functions run multiple times - a way to force acknowledgement?

Description
I'm using the app_mention event like this:
app.event("app_mention", async ({ event, say}) => {
 (...)
 if (event.text.includes("hello")) {
await say(`something`)
}
(...)
});

However, occasionally my functions trigger multiple times (getting multiple slack posts on a single mention).
This is probably caused by Slack getting no acknowledgement of message received within 3 seconds and retrying.
From what I read, there's no way to send an acknowledgement from an event like the above because Bolt does that automatically. Can I force this though to react as soon as possible so it doesn't happen?
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[ x] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[ x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[ x] I've read and agree to the Code of Conduct.
[x ] I've searched for any related issues and avoided creating a duplicate issue.

Hey @darekmm!
As you mentioned, bolt-js will automatically send ack for incoming events. This is done before your listeners are even run. We don't provide a way send your own ack for events.
If you are using the processBeforeResponse flag when initializing app, then ack will be sent after your listener completes. Are you using this flag?

Hmm - thank you for explaining! No, I'm not using such flag.
I guess if this is sent before my functions run then there's no point looking for a manual way.
I'm using some potentially time-consuming processes though (fetch, axios.get) if that makes any difference, but then again the ack() is probably sent before these are run.
In that case I'm thinking it must be caused by actual delays between slack <-> glitch.me (where I'm testing).
It seems the only way to prevent is to implement some sort of a failsafe where the app saves the last request/response and skips if it's the same?

fetch & axios.get calls shouldn't make a difference for the ack in this case.
It might have something to do with glitch. Hard to say. How frequently are you seeing this problem?
Having to implement a failsafe is definitely not ideal.

It doesn't seem consistent. Sometimes it will work flawlessly for a couple of days and then there's an evening when all responses get duplicated (or appear 3 times). Seems like if the server is experiencing heavy load or something then lags can occur - but I might be wrong. Honestly if there was a setting on Slack to disable retrying after 3s it would solve the problem - I'd prefer not getting a response to getting multiplicated ones, especially if they also re-run various other things.

Are you on the free tier for Glitch?

Yes. Would that make a difference?

@darekmm I've heard of weirdness with the free tier of Glitch before with Slack Apps.
From https://glitch.com/pricing

For free users, Glitch apps go to sleep after five minutes of inactivity — if an app is waking up, your users might see a loading screen (we do this to keep our servers happy). Boosted apps don’t sleep and are always ready to go.

Pretty sure the app is asleep, so it isn't responding within 3 seconds, hence the duplicate messages

That explains! Yes come to think of it it mostly happens when the app has not been active for a time.
In that case I will get a paid tier or try hosting somewhere else like heroku and let's see.
Thanks!

ok so update on the issue:
Pinging periodically didn't really help.
So I now have a paid tier of Glitch with a boost for this app (keeps alive at all times)... and still  it tends to randomly multiplicate responses.  If it responds fast there's no problem, but sometimes there's a few seconds of delay which results in multiple identical responses.

Hi @darekmm, if I understand this issue here correctly, there is no further action we (as the maintainers of Bolt framework) can take for you. Let us know if you have any further comments. I will close this issue after waiting for your response for a while.

No, I think no action needed on your side. Thank you!
On Thu, 18 Mar 2021 at 06:22, Kazuhiro Sera @.***>
wrote:

Hi @darekmm https://github.com/darekmm, if I understand this issue here
correctly, there is no further action we (as the maintainers of Bolt
framework) can take for you. Let us know if you have any further comments.
I will close this issue after waiting for your response for a while.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/slackapi/bolt-js/issues/786#issuecomment-801635551,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHBMIBGGKNQPP6AU7JQG2VDTEGEY7ANCNFSM4XLNESNA
.


boto3 SQS set attributes fix.

In boto3, the SQS set_queue_attributes items are now presented as a list of values of the form Attribute.1.Name, Attribute.1.Value, Attribute.2.Name, and so on.
This patch ensures these are properly handled.

thanks!


refactor samples tables in frontend

Should have a vue component that displays a samples table and re-use that instead of existing duplication of code in SamplesView and AdminView

resolved by #66


migration - get localip

Please test on others setups. My ip address was blank.

Replaced this PR with the solution to get the local IP from the specialized config-script internet.sh and added the inet to its scope to look for: https://github.com/rootzoll/raspiblitz/blob/2c9924eac140239302a757f0ea5ceca85d8c79cc/home.admin/config.scripts/blitz.migration.sh#L32


How to mark some plugins as not updatable?

I use Vundle as git-submodule, and l need to mark this plugin as not updatable to ignore it when I do BundleUpdate, can I do it with Vundle?

Seems that you can use the pinned option for your plugin configuration, eg:
  Plugin 'gmarick/Vundle.vim', {'pinned': 1}

See https://github.com/gmarik/Vundle.vim/blob/master/doc/vundle.txt#L176.

@davidjb thank you, it works for me.


Implemented "Track" Feat

Description
It implements the 'Track' feature which closes #31 and resolves #34
How Has This Been Tested?
Unit and integration tests have been written to test these features
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 on my code, particularly in hard-to-understand areas
[x] I have added necessary inline code documentation
[x] I have added tests that prove my fix is effective and that this feature works
[x] New and existing unit tests pass locally with my changes

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

See the complete overview on Codacy


ROS 2 Rolling/Galactic: Can't see labels/icons on many (but not all) buttons

While doing testing for the upcoming Galactic release, I noticed that many of the buttons in RQt on my test platforms were empty.  For instance, running rqt, and then adding in the Plugins->Logging->Bag plugin shows a group of buttons across the top.  But there are no labels or icons on any of them.  They do all have a tooltip, so it is somewhat possible to figure out what they are for, but something is clearly missing.
@cottsay suggested it may have to do with the vendoring of tango_icons_vendor, but I have not yet had time to look into it.  @ahcorde, any thoughts here?

Examples of missing icons
https://github.com/ros-visualization/rqt_graph/issues/63
https://github.com/ros-visualization/rqt_service_caller/issues/19


Replace DirectionalLight3D's use_in_sky_only with sky_mode enum

Fixes: https://github.com/godotengine/godot/issues/53985
This PR builds on Calinou's commit and adds the option to exclude lights from sky rendering. This way, users can have lights that affect the scene without adding new suns to the sky.

Needs a rebase.

Done!

Thanks!


allow overriding of boilerplate hooks and shell flags in catalog config

Describe the enhancement
It would seem that when running terragrunt scaffold (through terragrunt catalog ) boilerplate hooks and shell scripts are disabled. That is, it seems the following flags are added to the boilerplate call:
--disable-hooks
--disable-shell

This doesn't seem to be documented anywhere, which would be the first suggestion for enhancement. The second suggestion would be to let the user override these in the catalog config.
catalog {
  disable_hooks: false,  # defaults to true
  disable_shell: false, # defaults to true
  urls = [
    "https://github.com/gruntwork-io/terragrunt-infrastructure-modules-example",
    "https://github.com/gruntwork-io/terraform-aws-utilities",
    "https://github.com/gruntwork-io/terraform-kubernetes-namespace"
  ]
}

Additional context
I understand the reasoning for disabling both hooks and shells, as they can introduce security concerns when run against third party terraform modules. If the end user has control over the catalog config however, and if they also have control over all the configured terraform module repos in the catalog config, it would seem to me that the user should be able to make the call on whether or not to allow boilerplate to execute scripts.
Scripts can significantly extend the possibilities of what can be achieved with boilerplate.  As long as safe defaults are set, the trade off of security versus functionality should lie with the end user.

Hi,
for reference, hooks and shell execution are explicitly disabled in:
https://github.com/gruntwork-io/terragrunt/blob/main/cli/commands/scaffold/action.go#L175
most probably we need to add CLI flags to enable it when doing scaffold too

We'll commit to addressing this in the new year.


AOI Error - Multiple Spread-Out Polygons

The following error message is encountered when trying to define an area of interest made up of multiple spread-out polygons:
Internal Server Error
The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.
Looks like the tool has some difficulty assessing buildings in dense cities such as Kathmandu and Dar es Salaam.

Thanks for opening this up @mataharimhairi. Yeah, I saw this last week in Dar as well when campaigns that are querying features like buildings in dense urban areas. This fails because of the same type of size limitations we see with the Export Tool and querying large, dense countries.
Right now we make a simple bounding box query and thus when the areas area spread apart, they make an initial call that retrieves a lot of data. I suspect that then the Lambda process times out.

This is related to #496


Create "How to Contribute" Section

Create a section for people to contribute and add solutions to different challenges. Include How to Contribute with guidelines/formatting criteria.

Solved by #5


Import text styles from external library

User may want to have Text Crop exist in a different file to the Library Styles.

No longer needed.


socket is connected, but not able to recieve data.

It has been 2 months without an example. This issue should be closed.


This page doesn't reflect the current user experience

This link no longer shows the current user experience.
There is no "Stop replication" option, there is only an option to promote or delete.
https://learn.microsoft.com/en-us/azure/mysql/single-server/how-to-read-replicas-portal#stop-replication-to-a-replica-server

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

ID: 151a5ac0-5f12-e4f5-4c14-1dcb78f80e61
Version Independent ID: 13e702f3-293d-801a-ea77-c3832bc41803
Content: Manage read replicas - Azure portal - Azure Database for MySQL
Content Source: articles/mysql/single-server/how-to-read-replicas-portal.md
Service: mysql
Sub-service: single-server
GitHub Login: @SudheeshGH
Microsoft Alias: sunaray

@airmnichols Thank you for your feedback. We will review and make an update if appropriate.

https://github.com/SudheeshGH Could you please help in fixing the documentation. This no longer exists in the portal.

#assign: @VandhanaMehta

Please close this. The link is for Single server and there is no change in single sever. The changes are there in flexible server and the same is already updated  https://learn.microsoft.com/en-us/azure/mysql/flexible-server/how-to-read-replicas-portal#stop-replication-to-a-replica-server


AlbumCover Image Size

Is there any way to get a bigger AlbumCover Image Size?
i found in utils.py the line
lib.SpGetMetadataImageURL(uri, lib.kSpImageSizeSmall, image_url, ffi.sizeof(image_url))
but even if i change it to  lib.kSpImageSizeLarge there is no change in the size of the resulting Album Cover File. What did i wrong?

The file probably is increasing in size, it's just the page that limits the display size (https://github.com/Fornoth/spotify-connect-web/blob/master/templates/index.html#L52)
I'll add an option for a larger player

I'm also looking to get a bigger AlbumCover Image Size. Unfortunately I'm not a programmer  to implement the promised change.


Added SASS to the list!

You gotta show it for them to know it, right?

(I see I removed the 'oxford comma', should I add it?)

@jeroenvisser101 I just edited that commit to be SCSS and merged as 38dca18. Thanks for catching this!

Cool! Thanks :)


add 2 options for background image: fit and full

Add 2 new options for scaling the background image: fit and full.
Both options keep the aspect ratio of the background image, unlike the option "Automatic".
Full option will scale the image to cover the window, and the top left part is used.
Fit option will scale the image to fit the window, and put it at the center of the window.
It seems to be hard to use QSS to achieve this, so I used QPainter for this.
PS. There seems to be a bug when changing the options. Please don't merge it now.

I think the problem is that QPainter should not be used outside of the paintEvent() function. I have considered replacing the QSS for background with QPainter before, but just have not had time to work on it.
Basically you just need to have two private variables (QPixmap image & QString scaletype) which you set within the background changed function and then call this->redraw() to force it to repaint (automatically triggering a paint event).

If you do switch to using a QPainter, I would actually prefer it if you re-implemented all the current scaling options as QPainter routines as well. That will keep the code much cleaner and easier to adjust later as necessary.

I've realized the problem. Whenever the themesettings.cfg is changed, QApplication::setStyleSheet is called, which overrides the palette setting.
I think that I'd choose another way: put a QLabel inside the bgWindow, and set its content to the image.

Code updated to use a saved QPixmap in paintEvent() for drawing.
The QPixmap is generated when the background is set.
Other modes are also implemented.
Currently, "Fullscreen" centers the image. We may also put it at the top left corner, but this is better in my opinion.
If the image is too large, "tile" mode would show the top left corner of the image (image is not scaled). We may change it to tile the image after scaling it to fit the screen, but anyway we do not scale it when the image is smaller than the screen, so maybe just keep it like this.

Looks great!
Thanks for working on that!


Remove env prop for API URL being cached

Changing env prop GRAPHQL_API_ENDPOINT is not working if you have accessed the project previously, unless you clear localStorage data.
This code is causing it:

I think we should remove this code using API url saved in localStorage, unless I'm not seeing the use case for it
@luizstacio you see any problems on removing it?
Ref: #76

I think the purpose of this was if the user had configured a custom endpoint, then it'll get saved and used by default?
Instead of removing the feature entirely, maybe make a default special case (maybe empty?) where the default is used, instead of saving the default as a custom endpoint.

We should add the feature: https://github.com/FuelLabs/block-explorer-v2/issues/74


[cloud_speech] Streaming response support

The cloud speech API supports streaming recognition via gRPC, but the dart cloud-speech package doesn't support streaming recognition, is there any planned support for this?

I've generated a library but have had issues getting it to work. It's here!

To all who are still looking for speech functionality with streaming support. I have created a package for this purpose that supports grpc and communicates with Google's Speech-to-text api.
You can find it here


Epic: Supporting Express 5

Instead of #374 but non-breaking
The express 5 beta 3 is fully operational and tested by community, but its release is delayed.
The current issue is that it's also outdated comparing to v4.18.2, having nested dependencies of older versions.
latest tracking issue:
https://github.com/expressjs/discussions/issues/233

previous one:
https://github.com/expressjs/express/issues/5205

Latest proposal:
https://github.com/expressjs/discussions/issues/160

Probably it will be ready in march 🤞🏽  (no, it's not)
https://github.com/expressjs/express/discussions/5111#discussioncomment-4811950

coverage: 100.0%. remained the same
when pulling e33287c0c77061c8e760d1e4264d7d6d60e085b2 on express5-2024
into 37fecfe2679394a9bdc8fd95af44e8b402afcb75 on master.

They made beta2 today...

very nice

beta3 now. I like the tempo.

Three things one can enjoy watching eternally:

1
2
3

I can't believe it!

These are expected changes in https://github.com/DefinitelyTyped/DefinitelyTyped/pull/69846/files#diff-5962769e8693df44c9e1083881fb1da149a67262026db939b7430536d514e7af


0/1 knapsack with set of items that form solution using matrix(DP)

I would like to contribute solution to standard 0/1 knapsack problem using cpp
Thanks!

#102

Sure @satyam-kalyane! You can work on it! 🎯


datetime.cpp with GCC 10.1 requires inclusion of stdexcept

Hi,
I've tried to build nana 1.7.3 (as downloaded on 2020 may 11) on windows 10 with msys2/mingw-w64/gcc 10.1, using -std=c++17 .

With the above configuration it seems to occur a problem : with this version of the compiler the class std::runtime_error is reduced in visibility (maybe it's included by less headers of the standard library) and so it's necessary, at least for the file datetime.cpp, to include its definition:

#include
I send you a copy of the modified file, just with this simple change I managed to compile again the library.
Thanks for your precious work.
     Marco

datetime.zip
     Marco

Thanks for reporting this bug. It has been fixed by PR#509.


pypiserver: Fix "No module named 'pkg_resources'"

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Can you update the commit and PR title from [pypiserver] Fix : No module named 'pkg_resources' to pypiserver: Fix "No module named 'pkg_resources'"?
See commit conventions.

I think this was superseded by https://github.com/NixOS/nixpkgs/pull/354480. My bad for not getting back to you in a more timely manner. Closing this, but if there's still an issue let me know.


MacOS performance and multithreading

Hi,
I'm trying to use multiple threads to render different parts of the window.
Right now I'm rendering into a shared buffer for a 300x300 pixels window, then updating the window once with that buffer.
On MacOS with a 2019 Macbook Pro 2.4 GHz 8-Core Intel Core i9 I'm seeing the update
window
        .update_with_buffer(buf, window_width, window_height)
        .unwrap();

taking initially 100-200 micro seconds. But as I fill up the buffer (replacing 0s with actual colors) the update takes up to 32ms - I can't even maintain 60fps.
I suspect I'm doing something (or many things) wrong. When the update starts taking so long I am not even updating the buffer with anything. I wonder:

Should I only update the window with the smallest buffer and pixel range that is actually changing for better performance?
Should I share reference to the window itself between my threads, and call update from each thread? Would that be a perf boost?

Would be great to have a bit more info about performance and mutli-threading on this repo.
Thank you for the awesome crate!

Hi,
Just making sure when testing the performance do you run with cargo --release ? default is debug mode which is much slower.

Ok release is incredible faster, thank you for that tip :)
But I'm still running into the same issue. Update takes 33ms - 34ms when I get to that situation.
Perhaps it's update_with_buffer sleeping? But I do have window.limit_update_rate(None); (as I'm maintaining framerate by sleeping on my own). I checked the code and this should indeed skip the sleeping update function. So I'm still bewildered why it would take so long :/

Just to double check, can you do something like this:
use std::time::{Duration, Instant};

fn main() {
    ....
    while window.is_open() ... {
       let start = Instant::now();
        window.update_with_buffer(...);
        let duration = start.elapsed();
        println!("Time for update_with_buffer is: {:?}", duration);
    }
}

And report what time you are seeing here, if its' really big (i.e several ms) it may be a good thing to try to use a profiler (such as Instruments that is included with XCode) to see where the time is being spent.

That's exactly how I'm measuring the time. I'll try to get you a minimal repro.

Well, I'm not having much luck simplifying this, but I have kinda found a work around.
In my loop, I also have a sleep() call to let the other threads besides the main one get a chance to do their work and acquire the lock on the buffer. The sleep() was set to 0 if the main thread took longer than my expected FPS (basically every 1/60 seconds I wanted to make sure the main thread wakes up, regardless of how long the main thread takes to do its job). When I did this the main thread eventually got to the place where the update_with_buffer takes 33 or 16ms, leading to 0 sleep, and this repeats. The fix is to always force the main thread to sleep. Then update_with_buffer stays fast.
🤷

This sound strange to me, the way I would design this loop is something like this
  start_work_on_threads(temp_buffer);
  while window.open() .. {
    buffer = sync_thread_work_to_buffer();
    start_work_on_threads(temp_buffer);
    window.update_with_buffer(buffer);
  }

Now the threads will work in the background while update_with_buffer is running, update_buffer will also sleep (if setup as in the examples) here to sync to 60 Hz, if sync_thread_work_to_buffer() ends up being slow (i.e the thread workers takes long time) the sleep time should be reduced and in worse case it will drop frame-rate (no sleeping)

The thing is, I need to acquire a lock on the buffer. This is what I think happens:

The main thread "hiccups" - for whatever reason the update takes mss instead of µss.
This starves the worker threads, as they can't acquire the buffer lock
When the main thread gives up control, the worker threads acquire the lock one by one
But this makes the main thread take longer to acquire the lock for itself
The longer the main thread takes (counting both acquiring the lock and updating the screen), the less it sleeps, and the more starved the buffer lock becomes

Eventually the main thread doesn't sleep at all, and essentially deadlocks the buffer.
I originally designed my loop exactly the same way you have it in UpdateRate. The code is identical, so both exhibit this.
Always sleeping manually works. As you're suggesting sync_thread_work_to_buffer would also work, if I knew how to do that :)

The hiccups are noticeable even when running noise example for example, btw.


[README] Howto enable Zuul in more repos

example review:
https://softwarefactory-project.io/r/18820

please also include the example review in the docs :)

Hmm, so either I created the mergeit label somehow wrong or Zuul can't merge PR in the 'tenant config' (this) repo?


Revert "fix(TDI-45883): CVE: calcite-core-1.16.0.jar (#1903)"

This reverts commit 31456634d2c66d3ddb9cd365ca509d7715b0b934.
What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-45883
What is the new behavior?
https://jira.talendforge.org/browse/TDI-45883
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[x] Other... Please describe: CVE

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Pull request test result: success :star:


Lecture 11 water vapor data

the water vapor database in lecture 11 is not working--it returns an "Error -3 while decompressing data: invalid stored block lengths"

The data servers we are running at UAlbany have been a bit temperamental lately. However I just tried running this notebook and it worked as expected. I suggest you try again!
If you encounter the same error, please let me know what version of xarray you are using. Thanks.

Thanks, Brian! It's working now so it might have just been down for a bit.


small-fixes

[x] fix printlevel
[x] fix summing over local reps without symbols
[x] fix evaluation os spin argument (closes #71)
EDIT:
[moved to issue #74]
fix implicit dtype=object (deprecated numpy => 2.*.* ?)

could you please add a test for this?

[ ] fix the input parsing
[ ] fix the spin evaluation (i.e. omods ARG)
[ ] ? move the dm computation to bond() ?

@briling checks fails because of numpy version I think this AttributeError
normally it should accept a callable since v. 1.23.0
Do you know if the version in the workflow is correct ?

[ ] apply changes to atom.py


For subskills that are orb converters, use emojis with an arrow to show the conversion

I think we need to add a mortal poison and a bomb emoji
Dependent on https://github.com/TsubakiBotPad/pad-data-pipeline/issues/391

This is no longer blocked.


About navigationView BindView

Hello,
I want to use @BindView() to bind a navigationView.getHeaderView(0).findViewById()
but I failed, How can I bind it?

Pass the result of getHeaderView to bind() with the object that has the bindings you want bound.


Compatibility with next: 12.3.0

Using "next": "^12.3.0"
I added the configurations as instructed:
 experimental: {
    swcPlugins: [["@nissy-dev/swc-plugin-react-native-web", { commonjs: true }]],
  },

but once i run the dev server i start getting the following errors:

could it be a compatibility issue?

that would be highly appreciated

Sorry for a late response. I released the new package (0.2.6) for Next.js v12.3.1, so you can use it!


Lampasas County, TX parcels

http://mapsatbis.com/bisgis/rest/services/LampasasWeb/MapServer

Already in state wide data, north of Austin.


Inaccurate Predictions and Inconsistencies in opal23_headpose_test.py

Hello,
I recently renamed the file "example.tif" to "example_bak.tif" to exclude its annotation file (example.json) and ran the script "opal23_headpose_test.py". However, I encountered significant variations in the predicted values depending on the dataset and rotation mode, resulting in inaccurate predictions.
I observed the following:

The predicted values varied substantially based on the dataset used.
Different rotation modes led to inconsistent results.
The overall accuracy of the predictions was poor.

I am seeking guidance on how to obtain more accurate and consistent predictions from the "opal23_headpose_test.py" script. Any suggestions or insights into improving the prediction accuracy would be greatly appreciated.

300wlp_euler

300wlp_6d

panoptic_6d

panoptic_6d_opal

panoptic_euler

Hello, thank you for your interest.
If I understood correctly, you are trying to call opal23_headpose_test.py without providing the example.json annotations, right?
Since the model needs bounding box annotations to crop the face, I assume you downloaded and configured this face detection repository to get the bounding box annotations. The code defaults to this repo if the JSON file is not found.
As for the wrong predictions you are getting, the models trained on 300W-LP should output the correct predictions. I could not reproduce your results even without the example.json  annotations. The models trained on CMU Panoptic, however, won't work on in-the-wild images because the dataset only includes one specific type of background / illumination.
Let me know if this answered your questions.

Thank you for your response!
Yes, I used the SSD detector by default. The 6D predicted values for 300W-LP are accurate, but the 300W-LP Euler predicted values differ from the 300W-LP 6D values. I noted that Panoptic may not perform well on in-the-wild images. Thank you for your response.

That's strange, I get the following results using SSD detector and 300wlp_euler model:

And with 300wlp_6d:

Maybe you have an old version of the repo?
Have you tried running git pull on opal23_headpose and images_framework repos?

I set up opal23_headpose and images_framework today, so they should be the latest versions. I really appreciate you taking the time to reproduce the test for me. Given the results, it seems I need to investigate further to determine if there are any issues with my code. Thanks again for your help!

Hi @jayong-sv @alejandro-cobo ,
I am trying to replicate the work to get head pose for custom images. I am getting confused at many steps. If any of you can share more details about the implementation it will be very helpful. After setting up this repo with image_framework, while running for custom images, I am getting stucked at image annotation portion, as there is nothing in detection.ssd16_detection.src. Do I need to add the repo for ssd16 there? And suppose I have landmarks from my own detection method, How to use them?

For custom data, you need to provide a bounding box of the face in the image, either by using a face detector such as SSD or by providing your own annotations.
For the detector, just run git clone https://github.com/pcr-upm/ssd16_detection.git inside images_framework/detection.
If you have landmark annotations you can use the bounding box of those landmarks in the format (x_min, y_min, x_max, y_max). Check out the test code to see how to add your annotations.

@alejandro-cobo
Thanks. I am able to run it now. I was testing it for AFLW2000 but could not figure out where to pass the data or directory. Can you help me with the command?

For the code to correctly import the modules, you should run the command from the parent directory where the images_framework repo is located:
python images_framework/alignment/opal23_headpose/test/opal23_headpose_aflw2000.py --anns-file images_framework/alignment/opal23_headpose/annotations/aflw2000_ann.txt --database 300wlp --gpu 0 --rotation-mode euler

For the code to correctly import the modules, you should run the command from the parent directory where the images_framework repo is located: python images_framework/alignment/opal23_headpose/test/opal23_headpose_aflw2000.py --anns-file images_framework/alignment/opal23_headpose/annotations/aflw2000_ann.txt --database 300wlp --gpu 0 --rotation-mode euler

Still getting same error: ValueError: Database does not exist.
Command running now:
images_framework/alignment/opal23_headpose/test/opal23_headpose_aflw2000.py --anns-file C:\Users\Girish\Desktop\FF-Projects\images_framework\alignment\opal23_headpose\annotations\aflw2000_ann.txt --alignment opal23_headpose --database 300wlp --gpu 0 --rotation-mode euler --target-dist 1.6

It seems like it is a problem with Windows, as I am able to replicate your error.
The quickest way to solve the error would be to use Linux instead (with WSL, for example).
Let me know if you are restricted to only using a Windows environment, and we can try to make it work on Windows soon.

It seems like it is a problem with Windows, as I am able to replicate your error. The quickest way to solve the error would be to use Linux instead (with WSL, for example). Let me know if you are restricted to only using a Windows environment, and we can try to make it work on Windows soon.

Yes, I am currently restricted to windows only. I will see if I can get my hands on a linux system. If you can provide it for windows , it will be of great help.
I am able to run it for custom images, but can not say how good it is performing as I don't have ground truth values for them.
Also, have you considered it to be converted in onnx?

The problem should be fixed with commit fef052091d2227fab59a703eeeb135a148f7e9b1.
However, we have found that the annotations file for AFLW2000 is outdated, so the code will still fail until we update it.
You can check if the ValueError: Database does not exist. error no longer appears after updating your local copy of the repo.
As for ONNX, be may convert it in the future if it makes the models more convenient to use.

The problem should be fixed with commit fef0520. However, we have found that the annotations file for AFLW2000 is outdated, so the code will still fail until we update it. You can check if the ValueError: Database does not exist. error no longer appears after updating your local copy of the repo.
As for ONNX, be may convert it in the future if it makes the models more convenient to use.

I made the suggested changes, and the error ValueError: Database does not exist. no longer appear. However, it was looking for images in image_framework\alignment\opal23_headpose\annotations\images folder, so I copied all the files (images and .mat files) of AFLW2000 dataset there. Now I am getting IndexError: list index out of range at line 430 in image_framework\src\dataset.py.

It is fixed now, I can replicate the same results on Windows and Linux.
Let me know if you encounter any other errors after updating to the latest version of the repo.

Thanks @alejandro-cobo , Its working fine now.


ffprobe-location

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
I want to use yt-dlp as a dependency in an electron application. The two most important dependencies of yt-dlp itself are supposedly ffmpeg and ffprobe. I do not want to have to add these to the PATH to get my application to work. There is an option to specify the ffmpeg path with ---ffmpeg-location, but considering that ffprobe is considered an important dependency, shouldn't there also be a ---ffprobe-location option? That is what I would like.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Nevermind, it turns out that ffprobe isn't that important for what I need to do.

yt-dlp looks for both binaries in the same directory


App won't start with plugin installed

The plugin completely breaks my app and I've tried installing the plugin several times even on a fresh install. I get this output:

Launching lib/main.dart on iPhone X in debug mode...
Xcode build done.                                           11.2s
Failed to build iOS app
Error output from Xcode build:
↳
** BUILD FAILED **
Xcode's output:
↳
=== BUILD TARGET audioplayers OF PROJECT Pods WITH CONFIGURATION Debug ===
/Users/paul/dev/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.10.1/ios/Classes/AudioplayersPlugin.m:74:47: warning: format specifies type 'double' but the argument has type 'int' [-Wformat]
NSLog(@"position: %f %@", milliseconds, call.arguments[@"positions"] );
~~      ^~~~~~~~~~~~
%d
/Users/paul/dev/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.10.1/ios/Classes/AudioplayersPlugin.m:104:61: warning: format specifies type 'double' but the argument has type 'int' [-Wformat]
NSLog(@"Seeking to: %f milliseconds", milliseconds);
~~                ^~~~~~~~~~~~
%d
/Users/paul/dev/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.10.1/ios/Classes/AudioplayersPlugin.m:232:5: error: use of undeclared identifier 'AVAudioSessionCategory'; did you mean 'AVAudioSessionCategoryRecord'?
AVAudioSessionCategory category;
^~~~~~~~~~~~~~~~~~~~~~
AVAudioSessionCategoryRecord
In module 'AVFoundation' imported from /Users/paul/dev/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.10.1/ios/Classes/AudioplayersPlugin.m:4:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.3.sdk/System/Library/Frameworks/AVFoundation.framework/Frameworks/AVFAudio.framework/Headers/AVAudioSession.h:642:28: note: 'AVAudioSessionCategoryRecord' declared here
AVF_EXPORT NSString *const AVAudioSessionCategoryRecord;
^
/Users/paul/dev/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.10.1/ios/Classes/AudioplayersPlugin.m:232:27: error: expected ';' after expression
AVAudioSessionCategory category;
^
;
/Users/paul/dev/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.10.1/ios/Classes/AudioplayersPlugin.m:232:28: error: use of undeclared identifier 'category'
AVAudioSessionCategory category;
^
/Users/paul/dev/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.10.1/ios/Classes/AudioplayersPlugin.m:234:9: error: use of undeclared identifier 'category'
category = AVAudioSessionCategoryAmbient;
^
/Users/paul/dev/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.10.1/ios/Classes/AudioplayersPlugin.m:236:9: error: use of undeclared identifier 'category'
category = AVAudioSessionCategoryPlayback;
^
/Users/paul/dev/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.10.1/ios/Classes/AudioplayersPlugin.m:239:34: error: use of undeclared identifier 'category'
setCategory: category
^
/Users/paul/dev/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.10.1/ios/Classes/AudioplayersPlugin.m:232:5: warning: expression result unused [-Wunused-value]
AVAudioSessionCategory category;
^~~~~~~~~~~~~~~~~~~~~~
3 warnings and 6 errors generated.
Could not build the application for the simulator.
Error launching application on iPhone X.
Exited (sigterm)

Confirming this issue as well, only fails on iOS. Downgrading to 0.9.0 fixes it.

@indiepaul have you tried running the plugin example app? Which xcode version are you using?

@erickzanardo I have tried the example but I get the same message.
I am using xcode version 9.3.1, could that be an issue? Everything else works fine though.
I will try it again after I update maybe.

I am not an ios expert, but this could be a reason, I guess xcode 9.3.1 is a little old already, can you upgrade it?

I will try and do that and let you know

Closing this due inactivity, please open a new issue in case you still have problems.


Documentation for name query parameter

Adding docs based on https://github.com/webpack-contrib/bundle-loader/issues/31

fixes #31

@michael-ciniawsky updated

@MobiusHorizons Thx

glad to help. Its a very useful plugin!!


I cannot call MatDialog.open method

My code is:
@Component({
selector: 'app-root',
templateUrl: './app.component.html',
styleUrls: ['./app.component.css']
})
export class AppComponent {
constructor(private solutionService: SolutionService, public dialog: MatDialog) {
}
showLogin(): void {
let dialogRef = this.dialog.open(LoginComponent, {
width: '250px',
});
dialogRef.afterClosed().subscribe(result => {
  console.log('The dialog was closed');
});

}
}
and:
@Component({
selector: 'app-login',
templateUrl: './login.component.html',
styleUrls: ['./login.component.css']
})
export class LoginComponent{
//noinspection JSAnnotator
constructor(@Inject(MAT_DIALOG_DATA) public data: any,public dialogRef: MatDialogRef){
}
}
when I call 'this.dialog.open' l got red error:

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.

I have tried gitter on this and no-one has addressed my comments.
I know the issue is related to the generics being used <R = any> because if i remove the equals any it gets rid of the error.  Can we state if this is an IDE issue, typescript issue, or something else?

I am having the same problem with the exact same code which is the sample code on material site....

only happens at runtime


Diacritical marks in bibliography

Have a general solution for converting bibtex style {\'e} accented characters to a unicode é in https://github.com/distillpub/distill-template/blob/master/components/bibliography.js.

The most official looking reference for Latex—Unicode conversion I found is at W3:
https://www.w3.org/Math/characters/unicode.xml
It might be enough to get the most common symbols from the first couple hundred lines at https://gist.github.com/piquadrat/798549 and error out on unknown escape sequences. We could then back test it on, say, all NIPS publications to check if we're missing anything.


Case Data Form Overhaul

See #445 and #572 and #658 and #550
@plscully to provide final form fields.
Also review order of fields and consider sub-sections / accordion solution.

Some notes as I work on mockups for data forms:

In general, spacing and typography issues need to be tightened up. I'll go through this with @andreadelrio
Some items in the full version (FV) are under quick submit (QS) headers, ie: Geographical Scope should be found under Location. Should clicking Do Full Version publish the case and reload the form with things in the right place? Or do we keep QS and FV separate in the form?
The proposed form updates are sorted by overarching categories (Overview, Location, Media, Purpose and Approach, etc) which sort of line up with QS categories but not exactly, ie: some things in Overview are currently their own categories/headers in QS, ie: title of case.

The following should be included in edit forms (maybe left side bar):
First submitted by xxxx
Most recent changes by xxxx
See History/Revisions for list of all contributors to this case

Most Recent mockup below. Current status - waiting for @plscully to clarify multi-component elements required before I can mock up.
Considerations:

Wide variation between multi-component examples
What fields are "repeated" for components? (why one field and not another?)
What section do components go under (ie: under 1-Overview vs )? Does it get it's own new section, if so, in what order? (currently the "repeated" fields are spread across sections - possibly, this doesn't matter as we can just repeat them in the component section, just wondering if it's confusing)

I think we should allow the addition of photos/videos that you don't own the copyright for but that you properly credit. Is it possible to have a "If not, please provide media credits [text entry]". On the page it could appear as "Photo courtesy of [text entry]". See this info graphic for 'fair use':

@plscully and @dethe both agreed that the check box is the way to go, and suggested I copy the way wikimedia treats their image release - which is shown in the mockup above. I still think this treatment is not ideal for participedia - unless someone was actually at a case event and took photos, it's going to be difficult for people to say that they "own" any image or have authority to release it to the creative commons.
The world of image use online is like the wild west. No one knows who really "owns" an image that has been used all over the place. I found this particularly challenging when dealing with images for the Participedia overview video - tracking down original sources was next to impossible.
Offering an image credit text box is one way of dealing with it, as @scottofletcher suggests. But I wonder, @plscully, has anyone complained about participedia's use of images so far? Can we keep going without an ownership / credit system? Seems like as a research platform we should fall under the "fair use" category for copyrighted images anyway, is there a way to vet that info graphic Scott?

The best of my knowledge, we have never received a complaint about the use of images on PP. Rather, it has been a concern about a hypothetical, and I think that some of it is grounded in concern that someone will take a photo at an event without asking the participants' permission. But as I see it, that is the user's dilemma, not ours. I like the approach of linking users to guidelines about "what is fair use", but not asking people to attest that they "own" an image.

One more thing -- This seems like it will be something that will be relatively easy to change later on if others in our team make a good case for doing so.  So don't worry about spending too much time on this when we have other complex things to figure out.

@jesicarson Jesi - Looking at this issue once again, and I will support whichever way you think is best. Please finalize this doc, but keep any of the ideas you developed in case user feedback suggests that we need to try another approach. https://docs.google.com/a/clearviewconsultingllc.com/document/d/1FB5BI7apjtRoml-lKzWDGOqJFA24_GrFsUDsNsbQSHk/edit?usp=sharing

We did actually have someone contact us about an image he owned the rights to (see https://participedia.net/en/cases/participatory-budgeting-la-marsa-tunisia). He was fine with us using it but wanted the proper attribution so having a way to provide that would have solved the problem

Maybe we can provide field linked to each image that allows users to attribute any image if they want to do so?

@jesicarson So the graphic seems to be accurate although there are slight differences between fair use (or 'fair dealing') in Canada and the US. Here's a guided two-step test for fair dealing published by the Council of Ministers of Education: https://goo.gl/5Qd2cb. I think we pass the first test (and first step in the guidelines) but not the second (ie. second step in the guidelines: attribution).
Also, this is from U Ottawa Law Professor Michael Geist's personal blog: "[T]the Court’s analysis suggests that virtually all educational uses would be covered by the existing categories without further legislative reform. In [ESAC v SOCAN], it concluded that the first stage purposes test has a low threshold:
'In mandating a generous interpretation of the fair dealing purposes, including “research”, the Court in CCH created a relatively low threshold for the first step so that the analytical heavy-hitting is done in determining whether the dealing was fair.'" (https://goo.gl/zBMI1J)

@andreadelrio this is the folder where the forms are "ready" https://drive.google.com/open?id=0Bzo3Gs74FNPwUVdJTnhCMjR0RzA

Thanks!
On Wed, Nov 22, 2017 at 1:20 PM, jesicarson<EMAIL_ADDRESS>wrote:

@andreadelrio https://github.com/andreadelrio this is the folder where
the forms are "ready" https://drive.google.com/open?id=
0Bzo3Gs74FNPwUVdJTnhCMjR0RzA
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/participedia/frontend/issues/659#issuecomment-346476149,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABBuabw0HPPuaOjvRQtuyzyAn-FeoLOks5s5JAagaJpZM4PvvpI
.

@andreadelrio is going to finish up the other stuff from our meeting this week, plan is to start on implementing the form overhaul next week.
Meanwhile, I'm going to go through the "ready" forms in detail and make sure all the logic is in place. ie: we just looked at quick submit, and realized there is some detail missing - like the fact that "media" is it's own section in the full form, but treated like just one field in the quick sub. I'm going to need to mock up quick sub i think to make sure all is clear.
@dethe, I just chatted with andrea and she thinks that the way forward would be for her to build the front end parts for forms and then be in touch with you about the database pieces that need to be changed. But i'll let you two discuss the best way forward on the building side.
Here we gooooo!!!

Excellent! Go team!
On Wed, Nov 22, 2017 at 3:39 PM jesicarson<EMAIL_ADDRESS>wrote:

@andreadelrio https://github.com/andreadelrio is going to finish up the
other stuff from our meeting this week, plan is to start on implementing
the form overhaul next week.
Meanwhile, I'm going to go through the "ready" forms in detail and make
sure all the logic is in place. ie: we just looked at quick submit, and
realized there is some detail missing - like the fact that "media" is it's
own section in the full form, but treated like just one field in the quick
sub. I'm going to need to mock up quick sub i think to make sure all is
clear.
@dethe https://github.com/dethe, I just chatted with andrea and she
thinks that the way forward would be for her to build the front end parts
for forms and then be in touch with you about the database pieces that need
to be changed. But i'll let you two discuss the best way forward on the
building side.
Here we gooooo!!!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/participedia/frontend/issues/659#issuecomment-346480340,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcJT0VCZSmphJuu5SRJcjtPZSv16YrVaks5s5JSagaJpZM4PvvpI
.

--
Patrick L. Scully, Ph.D.
President, Clearview Consulting, LLC
T<PHONE_NUMBER>
www.clearviewconsultingllc.com http://www.clearviewconsultingllc.com

current status:
@andreadelrio is working on building the ranking system with @dethe's help
I am going through the forms and labelling things for clarity for andrea, mocking up quick submit and mocking up components section.

For launch (Jan 8, 2018):

update as per strong drafts "ready" docs
including ranking widget
including components (but not stub creator feature)

@andreadelrio Question mark icon for more info - use bootstrap modal box. I will add copy where needed and notes in the strong drafts forms.

This is basically done, remaining things have their own issues to track them under.


Feature/cs 1

첫 번째 블로그 글

👷 Deploy Preview for deft-cobbler-e7aa7e processing.

Name
Link

🔨 Latest commit
85c0c250a34823c9cba8485186d20536a4bf9b6f

🔍 Latest deploy log
https://app.netlify.com/sites/deft-cobbler-e7aa7e/deploys/634f989c45dda500091c1a42


Converting SlowFast detection model to deployable form

I am trying to convert Slow R50 4x16 detection model to TorchScript .ptl format to use in the Android app using the model conversion script here.
convert_to_deployable_form(model, input_tensor)

Above API from pytorchvideo accepts only two arguments. Model and only one input tensor to the model. This works well for classification models but detection models like Slow/SlowFast take two inputs - input frames and bounding box details.
Is there a way to use convert_to_deployable_form( ) function with detection model ?
Any help is really appreciated.

@arunponnusamy, you could wrap the Detection model in your arbitrary custom nn.Module class that would take only one input argument and convert into 2 arguments needed by our detection model.
Hope this answers your question. Feel free to re-open the issue if you have more questions.


Power Platform Tools won't install on Visual Studio 2022.

[Enter feedback here]

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

ID: 2d709296-ee38-4b31-a3b3-2a532ed06908
Version Independent ID: ef0cedf7-60d7-b0ca-770f-a69759bb59e5
Content: Install Power Platform Tools - Power Apps
Content Source: powerapps-docs/developer/data-platform/tools/devtools-install.md
Service: powerapps
Sub-service: dataverse-developer
GitHub Login: @phecke
Microsoft Alias: pehecke

@jahmedhitachi A VS 2022 version of the tools has not been released. VS 2019 is required for now.
See https://marketplace.visualstudio.com/items?itemName=microsoft-IsvExpTools.PowerPlatformTools


Unable to assign field of type from @preconcurrency imported module from nonisolated constructor of class protected by global actor

Description
I have class which is protected by global actor.
I want to have nonisolated constructor to be able to instantiate class synchronously from any concurrency context.
I can't initialise state of class due to concurrency checker even so code neither unsafe nor have some data races.
Reproduction
@preconcurrency import Combine

@MainActor
private final class GlobalActorIsolatedClass {
  private let subject: PassthroughSubject<Void, Never> // adding nonisolated to let definition does not help

  nonisolated init() {
    subject = PassthroughSubject<Void, Never>() // Main actor-isolated property 'subject' can not be mutated from a nonisolated context
  }
}

Expected behavior
It should be possible to initialize state of class protected by global actor form non-isolated constructor of that class.
The compiler should still produce error in case non-sendable and copyable type is assigned from arguments of non-isolated constructor to an isoalted field.
The concurrency checker for regular actor's init and concurrency checker for non-isolated inits of classes protected by global actor should work the same way, isn't it?
Environment
❯ swiftc --version
swift-driver version: 1.115 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.10 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx14.0

Additional information
The current workaround is to have:
extension PassthroughSubject: @unchecked @retroactive Sendable where Output : Sendable, Error: Sendable {
}

The error is not triggered in case of state type is actually sendable:
private final class SendableClass: Sendable {
}
@MainActor
private final class GlobalActorIsolatedClass2 {
  private let state: SendableClass
  nonisolated init() {
    state = SendableClass()
  }
}

BTW, why Apple does not annotate Combine with proper sensibility annotations?

One might suggest to use default initialiser to fix mentioned issue, but the actual code I'm trying to fix or understand why it's illegal is the following:
@MainActor
private final class GlobalActorIsolatedClass {
  private let subject: PassthroughSubject<Void, Never>
  private let publisher: AnyPublisher<Void, Never>

  nonisolated init() {
    let subject = PassthroughSubject<Void, Never>()
    let publisher = subject.share().eraseToAnyPublisher()

    self.subject = subject
    self.publisher = publisher
  }
}

actor RegularActorClass {
  private let subject: PassthroughSubject<Void, Never>
  private let publisher: AnyPublisher<Void, Never>

  public init() {
    let subject = PassthroughSubject<Void, Never>()
    let publisher = subject.share().eraseToAnyPublisher()

    self.subject = subject
    self.publisher = publisher
  }
}

I expect that either both snippets either compile or both rejected by compiler. Or can anyone can explain why my expectation are wrong?


No scrcpy start-up

When I run scrcpy nothing happens, either phone plugged with USB debugging enabled or unplugged. I realized I can't even use it with the phone I need scrcpy as it has Android 4.4.2 (the problem is always being so strict about compatibility) but scrcpy can be useful for my other devices if only works. It's a kind of issue I had on other games, I double-click the shortcut/click taskbar shortcut and nothing happens. On a game I had the support to run it and it was told me to run a terminal command to install a folder with necessary stuff (dependencies) so looks like I need the same.
Samsung Galaxy Trend 2

What is the output in the console when you run scrcpy from a terminal?

I should have thought about running it on terminal before 🤦.
luca@Asus:~$ scrcpy
scrcpy 1.25 <https://github.com/Genymobile/scrcpy>
* daemon not running; starting now at tcp:5037
* daemon started successfully
ERROR: Could not find any ADB device
ERROR: Server connection failed

Well, maybe later I try running scrcpy again with my phone plugged, it's not supported but I should see it run anyway or show an error, for now looks like it doesn't start or misses of dependencies, I'll try again and see what happens.

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#device-not-detected

luca@Asus:~$ scrcpy
scrcpy 1.25 <https://github.com/Genymobile/scrcpy>
* daemon not running; starting now at tcp:5037
* daemon started successfully
ERROR: Device is unauthorized:
ERROR:     -->   (usb)  4203d267da21a100          unauthorized
ERROR: A popup should open on the device to request authorization.
ERROR: Check the FAQ: <https://github.com/Genymobile/scrcpy/blob/master/FAQ.md>
ERROR: Server connection failed
luca@Asus:~$ scrcpy
scrcpy 1.25 <https://github.com/Genymobile/scrcpy>
/usr/local/share/scrcpy/scrcpy-server: 1 file pushed. 0.0 MB/s (42151 bytes in 2.026s)
error: closed
ERROR: "adb reverse" returned with value 1
WARN: 'adb reverse' failed, fallback to 'adb forward'
Aborted
ERROR: Server connection failed
luca@Asus:~$ scrcpy
scrcpy 1.25 <https://github.com/Genymobile/scrcpy>
/usr/local/share/scrcpy/scrcpy-server: 1 file pushed. 4.7 MB/s (42151 bytes in 0.009s)
error: closed
ERROR: "adb reverse" returned with value 1
WARN: 'adb reverse' failed, fallback to 'adb forward'
Aborted
ERROR: Server connection failed

I hope that when I plug an Android 5 or above scrcpy works as intended otherwise I have nothing to do for it.

It could not work on Android 4, that's why you get these errors.
By the way, note that the latest version is 2.0.

Ah, well, I installed what GNOME Software had so I don't know, I see no 2.0 there.


[Runtime] Add ptrauth to standard concurrency descriptors.

Put the __ptrauth_swift_type_descriptor qualifier on the fields of ConcurrencyStandardTypeDescriptors.

@swift-ci please smoke test and merge