[Suggestion] Location Presets/Saves

Just a quick idea, I have been toying with this app since v.003. I feel like this project has the most potential for user functionality. Part of using these early builds involved me saving a slew of GMAP coords to a text file because the app has yet to allow me to change locations while it is live. I managed to get my monstrous laptop into my tiny impreza today and drive around with it. It doesnt update dynamically and there is some strange error that causes the geolocation setting to pull locations from random areas. So far I have ended up logging into Houston, Baton rouge and east of the city I live in. So I saved a list of coords I like to check on and logout to change them in the app. Changing the coords in app would be great but saving those coords in a manner that made changing them on demand much easier would be also pretty awesome. Anways, just an idea, I love what you guys are doing.
NOTES from v.007
at one point I was able to get it fully integrated with google so that the in-app browser would show the pokemons on googlemaps, however some error cause gmaps to shift into lite mode and it completely died. I had to close and reopen and the magic was gone. I was never able to repeat whatever I did to have it act like that. IT invloved middle clicking the operational button to open a second window, and somehow I was able to drag the little yellow dude from the second window back to the first to go into street view. Then I did the same to the second. The result was that each refreshed independently and I could change my location manually within the app without any problems (until it kicked into lite mode). I look forward to all the trial and error discovers I'll make with this newer version

This would be a killer feature. For some reason i cannot get the location changer to work. so have to log out every time. If you find the time then could you please add it in the next release.
Thanks a million for the awesome work you have done here :-)


How to do paging?

Hi ,
Is it possible to do paging with angularfire2? How to switch between both angularfire and firebase?
Sridhar

Hello! This answer is more appropriate for Stackoverflow as we troubleshoot library issues on Github. Thanks :)

Sadly, there are no reasonable answers on SO about this :(


cmd/pprof: panic on debug=0 goroutine profiles

go version devel +be8a6fd2e3 Wed Dec 7 17:09:37 2016 +0000 darwin/amd64

The go tool pprof command panics when retrieving /debug/pprof/goroutine?debug=0 profiles.
What did you do?
Test cases are adapted from #16892.
main.go:
package main

import (
	"net/http"
	_ "net/http/pprof"
)

var sink []byte

func main() {
	for i := 0; i < 10; i++ {
		sink = make([]byte, 1<<20)
	}
	panic(http.ListenAndServe("<IP_ADDRESS>:8080", nil))
}

test.sh:
#!/usr/bin/env bash

go version

go build -o ./server ./main.go
./server &
server="$!"
sleep 1

echo '--- debug=1 ---'
go tool pprof -raw -output=/dev/null 'http://<IP_ADDRESS>:8080/debug/pprof/goroutine?debug=1'
echo '--- debug=0 ---'
go tool pprof -raw -output=/dev/null 'http://<IP_ADDRESS>:8080/debug/pprof/goroutine?debug=0'
echo '--- end ---'

kill -KILL -- "$server"
rm ./server

What did you expect to see?
To begin, I expected go tool pprof to not panic on any input.
Second of all, I expected the runtime/pprof and net/http/pprof packages to not produce content that would trigger panics in cmd/pprof or github.com/google/pprof.
What did you see instead?
However, cmd/pprof (be8a6fd2e3) and github.com/google/pprof (google/pprof@01ea337ec1f9cb900b4df99128cb8c4b09a59af2) both panic.
Here's the output of the cmd/pprof panic:
$ ./test.sh
go version devel +be8a6fd2e3 Wed Dec 7 17:09:37 2016 +0000 darwin/amd64
--- debug=1 ---
Fetching profile from http://<IP_ADDRESS>:8080/debug/pprof/goroutine?debug=1
Saved profile in /Users/rhys/pprof/pprof.<IP_ADDRESS>:8080.goroutine.017.pb.gz
Generating report in /dev/null
--- debug=0 ---
Fetching profile from http://<IP_ADDRESS>:8080/debug/pprof/goroutine?debug=0
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x118f774]

goroutine 19 [running]:
internal/pprof/profile.(*Profile).preEncode(0xc420184000)
	/usr/local/go/src/internal/pprof/profile/encode.go:65 +0xc04
internal/pprof/profile.(*Profile).Write(0xc420184000, 0x1609a40, 0xc42000e048, 0x0, 0x0)
	/usr/local/go/src/internal/pprof/profile/profile.go:216 +0x47
cmd/pprof/internal/driver.grabProfile(0xc42007de80, 0x33, 0x0, 0x0, 0x0, 0x0, 0x14622d0, 0x1462978, 0x160ee00, 0xc420077fb0, ...)
	/usr/local/go/src/cmd/pprof/internal/driver/driver.go:410 +0xa95
cmd/pprof/internal/driver.PProf.func1(0xc420144260, 0xc420146000, 0x14622d0, 0x1462978, 0x160ee00, 0xc420077fb0, 0x160f700, 0xc4200860b8, 0xc420144228, 0xc420144250, ...)
	/usr/local/go/src/cmd/pprof/internal/driver/driver.go:73 +0x102
created by cmd/pprof/internal/driver.PProf
	/usr/local/go/src/cmd/pprof/internal/driver/driver.go:81 +0x3bd
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x8 pc=0x1202292]

goroutine 1 [running]:
cmd/pprof/internal/driver.processFlags(0x0, 0x160f700, 0xc4200860b8, 0xc420146000, 0x14622d0, 0x1462978)
	/usr/local/go/src/cmd/pprof/internal/driver/driver.go:794 +0x762
cmd/pprof/internal/driver.PProf(0x1610d20, 0x16964e8, 0x14622d0, 0x1462978, 0x160ee00, 0xc420077fb0, 0x160f700, 0xc4200860b8, 0x0, 0x0, ...)
	/usr/local/go/src/cmd/pprof/internal/driver/driver.go:108 +0x559
main.main()
	/usr/local/go/src/cmd/pprof/pprof.go:29 +0x14a
--- end ---
./test.sh: line 17: 51773 Killed: 9               ./server

/cc @matloob @rauls5382

The profile generated is invalid as it has locations with id = 0, eg:
sample: [
location_id: [ 0x0000000000000000 ]
value      : [ 1 ]
}     # sample[1]


enhance combineReducers()

There are a couple of features that I need from combineReducers().

support reducers with Symbol keys
pass additional arguments to reducers

If these enhancements are generally a good thing then please consider my pull request.
I do see that there is a branch to remove combineReducers() so this PR may be uninformed. If so please let me know what the near-future plans are and let me know if I can help.

Alternative
I just took a look at #79 and the WIP on the version 3.0 branch.
The new Effects API is seriously good. Cmd.batch() will make it a lot easier to write your own combineReducers().
As an alternative to the PR could I suggest:

The remaining 3.0 features be pushed to 4.0 so that the new Effects API can be released ASAP.
Merge the /remove_combine_reducers branch into 3.0 if you still want to be rid of it.

Additional Bike shedding
If I look at<EMAIL_ADDRESS>I see the code:
if (isLoop(nextStateForKey)) {
  cmds.push(getCmd(nextStateForKey))
  nextStateForKey = getModel(nextStateForKey)
}

If this code can be made more terse then the remainder is next to nothing. Perhaps:
nextStateForKey = getModelPushCmd(cmds, nextStateForKey);

Just a thought.

I'm going to close this since I'm considering removing combineReducers completely for the next version, but if we decide to keep it around I'll come back, reopen, and figure out a way to get this merged in. Passing additional arguments to reducers is a good strategy for sharing parent state and something I'd like to encourage.
Here's roughly what the next API is going to look like
https://github.com/redux-loop/redux-loop/pull/95


[BUG] Unable to get a player's stats data

Describe the bug
get_data() method in Player class returns a 403 when it tries to access the html page
To Reproduce
p = Player(47270)
print(p.get_data(None, 1, 'batting', 'match'))
Expected behavior
Should be able to access the html page and cotinue with the parsing process.
Screenshots
I've put a print command to show the contents of the html page

This is what the output and stack trace is.

The Response <403> is the output from printing the HTML.
Additional context
I'm guessing this is espncricinfo blocking odd requests not coming from browsers? Is there a way to get around this?

should be able to use a driver that is recognized as a user


Linux: help users understand why running as sudo does not work

It turns out that running sudo code results in a blank window. At the very minimum we should show a dialog telling the user that running as sudo is not supported and/or print this info to the console.
Some related issues:

https://github.com/microsoft/vscode/issues/151187
https://github.com/microsoft/vscode/issues/150111
https://github.com/microsoft/vscode/issues/149077
https://github.com/microsoft/vscode/issues/148680
https://github.com/microsoft/vscode/issues/126747

Ah ok, then maybe this issue should be renamed to reflect the support for running with --no-sandbox.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


How are the icons being set?

Thank you so much for sharing this. Super helpful.
Out of curiosity, how are these icons being set? I couldn’t find any image files or references to images in the framework.

Cheers!

The Preferences View Controller scene is a tab view controller, with a Toolbar style.

Each of trhe tab view controller's tabs has its image set to one of the system-provided images.


extra string variables added when previous variable exceds 8

data

I think index 2 and 4,  shouldn't be there. or I'm missing something?
thanks

@arielgk can you take a look if this pull: https://github.com/tiamo/spss/pull/51 solved your problem?

fixed


(#2195) Added timeout to registry download call

resolves #2195 (potentially)
Description
Added a 10 seconds timeout to every download call during dbt deps. I think the timeout is only for waiting until the response will be started to emmiting (the download process may be longer than timeout). See requests - Timeouts for more info.

You can tell Requests to stop waiting for a response after a given number of seconds with the timeout parameter. Nearly all production code should use this parameter in nearly all requests. Failure to do so can cause your program to hang indefinitely:

Checklist

[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

I have run this code in development and it appears to resolve the stated issue

The testing part looks awful I think (mock the requests internals or create /etc/hosts with desired low connection service). And as it looks to be a general good practice to add a timeout to requests and we actually don't know the 1:1 reason why the connection hangs maybe we don't need to add unit/integration test to these parts? 🤞

@beckjake good point! I was thinking about using a global config or something which should be used here but (maybe I'm wrong) there is nothing available in registry.py throw self or something and I need to add a lot of refactors. Do we have some policy here (for ex. static CONNECTION_TIMEOUT at beginning of file etc.)?

Do we have some policy here (for ex. static CONNECTION_TIMEOUT at beginning of file etc.)?

Global constants are always nice, though I felt this was unambiguous enough to just leave it. I agree it'd be quite tough to get it through from a config file given the current state of registry.py, and it's why I thought of an environment variable.

I'd be comfortable with adding a DBT_HTTP_TIMEOUT env var which we use whenever we download data with requests. I just want to make sure it's clear that this controls http web requests, not database connections, as those timeouts are going to be handled separately.
For the purposes of this PR, it would be great if we could apply this env var override in the scope of downloading data from the package hub

In the future, we may want to make this timeout configurable, even if it's just via an environment variable. While 10s is a totally reasonable maximum value for most places, it might not be great if you are both far away and have a bad connection. We'll see how this shakes out for now.

I'd be comfortable with adding a DBT_HTTP_TIMEOUT env var which we use whenever we download data with requests. I just want to make sure it's clear that this controls http web requests, not database connections, as those timeouts are going to be handled separately.

Done

I'll move the environ variable usage inside system.py insted of registry.py.


Create tool that load dumped event

fluent-binlog-reader is a command line tool to read the data which is saved as Message Pack format.
Usage of fluent-binlog-reader's Commands
head
head is used to display the beginning of a text file.
$ fluent-binlog-reader head packed.log
2016-08-12T17:24:18+09:00       packed.log      {"message":"dummy"}
2016-08-12T17:24:18+09:00       packed.log      {"message":"dummy"}
2016-08-12T17:24:18+09:00       packed.log      {"message":"dummy"}
2016-08-12T17:24:18+09:00       packed.log      {"message":"dummy"}
2016-08-12T17:24:18+09:00       packed.log      {"message":"dummy"}

You can specify the number of events to display.
$ fluent-binlog-reader head -n 1 packed.log
2016-08-12T17:24:18+09:00       packed.log      {"message":"dummy"}

cat
cat reads files sequentially, writing them to standard output.
$ fluent-binlog-reader cat packed.log
2016-08-12T17:24:18+09:00       packed.log      {"message":"dummy"}

...

2016-08-12T17:24:18+09:00       packed.log      {"message":"dummy"}

formats
formats displays plugins that you can use.
$ fluent-binlog-reader formats
out_file
stdout
json
hash
msgpack
ltsv
csv
single_value

Global Options
--format
You can specify the formatter as shown below.
$ fluent-binlog-reader cat packed.log --format=json
{"message":"dummy"}
...
{"message":"dummy"}

Format params
formatter has configurable params.(e.g. CsvFormatter has delimiter, force_quotes, and fields)
You can pass these params by using -e option.
$ fluent-binlog-reader cat --formats=csv  -e fields=message packed.log
"dummy"
...
"dummy"

--plugin
You can add new plugins path.
# $ touch my_plugins/fomatter_csv2.rb
# $ vi my_plugins/fomatter_csv2.rb

$ fluent-binlog-reader cat formats=csv2 --plugin=my_plugins
...

TODO

[x] Subcommands

[x] head
[x] cat
[x] formats

[x] adding plugins path
[x] specifed format
[x] specifed format params
[x] add tests

I added review comments.
And... fluent-unpacker looks the name too internal (users don't take care about the fact that dumping the content of buffer files is equal to unpacking msgpack event stream).
Do you have another idea about naming of this tool?

I think it is better standard-format-reader or  dumped-log-reader.
How do you think?
@tagomoris

I prefer dumped-log-reader. But fluent- prefix is needed.
fluent-dumped-log-reader, fluent-read-dump, ...
@repeatedly Any thoughts?

@tagomoris
I updated without the name of this command.

Your design to configure formatter plugins looks:

(before) fluent-unpacker -f out_file FILE_NAME -- -output_time=false -output_tag=false
(after) fluent-unpacker -f out_file FILE_NAME -- output_time=false output_tag=false

Is it correct?
If so, using -- to separate configuration parameters for formatters and others looks very curious. Normally -- is used to separate options and other arguments (may includes - as heading char).
@ganmacs How do you think about to use another command line option to specify formatter configuration parameters?
e.g.,) fluent-unpacker -f out_file -e output_time_false -e output_tag=false FILE_NAME
In this way, we can use -- to specify FILE_NAME with heading -.

@tagomoris
I changed command line design like below.
fluent-unpacker -f out_file -e output_time_false -e output_tag=false FILE_NAME

It's too late to point... but, head and cat can share code :P

head and cat can share code

Done.

I added some minor comments.
The last major problem is naming...
@repeatedly do you have any idea about naming?

Hmm... if this tool focuses on binary format, fluent-binlog-reader is good for me.
dump seems too general because other text files, e.g. result of out_file, are also dumped.

looks good > fluent-binlog-reader

@tagomoris
I changed a command name unpacker -> binlog-reader
@repeatedly Thank you for your help!

Two directories for test formatter plugins are needed?
test/scripts/fluent/plugin/formatter/{formatter_test1,formatter_test2}.rb seems enough.

@repeatedly
There are two tests
( https://github.com/fluent/fluentd/pull/1165/files#diff-5e97ad93fa2a92bc851090105261df21R329
and  https://github.com/fluent/fluentd/pull/1165/files#diff-5e97ad93fa2a92bc851090105261df21R339) for -p option'.
The first one is a test when two directories are passed.
The second one is a test when a directory is passed.
So  I think it is need two directories for a test.

@ganmacs Ah, I see. Got it.

@tagomoris @repeatedly
I updated. Please review the code again.

LGTM.


The datepicker not responsive on mobile devices

Describe the bug
Hello,
I am not able to use the datapicker on mobile devices,
The datapicker opens but when i resize the screen, it does
adjust to the screen size.
I don't know if i am missing some settings or i have to make it
responsible by myself.
Thank for the feedback

This jQuery plugin has been deprecated. Please search for a mobile-friendly datepicker for your case.


Improve Book card design

What I have done

improve book card size
modify button css
change font color
add responsiveness

fixes: #12

Thanks @ajaynegi45 for the feedback, I'm half way done


Fix type problem for GPU tensors

Fix type problem for GPU tensors

This fix is not great because it requires transferring the data to GPU each
time it's used.
That is slow.
Better have the device an arg to the constructor, I think.
On Fri, Jan 10, 2020 at 6:00 AM Zili Huang<EMAIL_ADDRESS>wrote:

Fix type problem for GPU tensors
You can view, comment on, or merge this pull request online at:
https://github.com/danpovey/filtering/pull/3
Commit Summary

fix type problem for GPU tensors

File Changes

M lilfilter/local_amplitude.py
https://github.com/danpovey/filtering/pull/3/files#diff-0 (4)
M lilfilter/resampler.py
https://github.com/danpovey/filtering/pull/3/files#diff-1 (6)
M lilfilter/torch_filter.py
https://github.com/danpovey/filtering/pull/3/files#diff-2 (2)

Patch Links:

https://github.com/danpovey/filtering/pull/3.patch
https://github.com/danpovey/filtering/pull/3.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/danpovey/filtering/pull/3?email_source=notifications&email_token=AAZFLOY5NOZJ3NUR4L3HADLQ46M6HA5CNFSM4KE7CZCKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IFF62LQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO4ROW4J7WRPH453MULQ46M6HANCNFSM4KE7CZCA
.

I have a project where I need gpu functionality, so I'll go ahead and merge it into my own fork. Thanks @HuangZiliAndy :)

I think it would make more sense (would be more efficient) to add a device class member and optional constructor arg.
.type_as() only changes the dtype, not the device, anyway, so it seems it wouldn't solve the stated problem.

i can go ahead and implement that, unless you want to do it :) i'll be slower to write it than you for sure

yes, please do!
On Fri, Jun 18, 2021 at 3:15 PM Kento Nishi @.***>
wrote:

i can go ahead and implement that, unless you want to do it :) i'll be
slower to write it than you for sure
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/danpovey/filtering/pull/3#issuecomment-863815626, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLOYRENTXVNSPXLQNYSDTTLXATANCNFSM4KE7CZCA
.

hm, i just found out that torchaudio.transforms.Resample exists. i'll give that a try instead :P


Update version.py

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.49%. Comparing base (1971573) to head (6b89ccc).
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #105   +/-   ##
=======================================
  Coverage   94.49%   94.49%
=======================================
  Files          17       17
  Lines         763      763
  Branches      112      112
=======================================
  Hits          721      721
  Misses         33       33
  Partials        9        9

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


Update AGPL-1.0.txt

It appears that the invalid Unicode in the header was due to an encoding mismatch.
I've replaced the invalid sequences with newlines, which I expect is the intended text.

this has been fixed in the new repo/format. thanks.


Android studio move

Moving from eclipse for android to Android studio (IntelliJ)

Merged

:worried:

What is the problem?

Just concern since not all are willing to move to AS / Gradle.
But there is still the option to use the pom files for the Android sample.

Of course, we won't move to gradle for the whole project as I currently do not see wide adoption and integration for gradle (I'm using e.g. NetBeans with good maven support). The downside is we have to sync those dependencies but that is okay for now.
Do you see other problems? Eclipse users can either use gradle, use the pom or even use the created jar files. I just want to get rid of something I don't like that much (I had to many fights with eclipse) with something which is now also the recommended system.

No, as you mentioned with Maven or jars someone can use the library in any IDE and with Gradle in AS (we follow the same couse in Mapsforge too).

By the way you have the AndroidDownloader class 2 times in Android module.
It's also outside the app folder, alone inside a src folder.

Thanks, will remove


Consider invalidating the configuration cache upon changes to the "version" Gradle property

project.version is often used as an implicit task input.
project.version can also be set via a Gradle property, e.g., gradle -Pversion=4.2 assemble.
It would be convenient if Gradle would automatically consider version an input to the configuration cache.
Expected Behavior
Gradle invalidates the configuration cache entry when a different value for the version Gradle property is given.
Current Behavior
Users are required to explicitly declare version as a configuration cache input via providers.gradleProperty("version").forUseAtConfigurationTime().orNull otherwise the configuration cache won't be invalidated when a different value for the property is given.

Fixed in #19079.


Feature/monitor rent

I added monitoring page for rent market service.

Sorry, I added to main branch for pull request. I close this pull request and send again to develop branch.


Rebuild with new conda-forge tooling

This PR adds CUDA 11.5 packages for ucx-py.
This PR bumps the build number so that we can re-build ucx-py with conda-forge's new tooling to get support for CUDA 11.5.

As part of the CEC work in conda-forge, we made this change, which relaxes the CUDA version pinning of packages built with 11.2 to support 11.2+. So it should be possible to drop anything higher than 11.2 as the package built with 11.2 will support 11.x (with x>=2).

As part of the CEC work in conda-forge, we made this change, which relaxes the CUDA version pinning of packages built with 11.2 to support 11.2+. So it should be possible to drop anything higher than 11.2 as the package built with 11.2 will support 11.x (with x>=2).

Ok. @jakirkham, I've updated this PR based on this comment and our Slack thread.

Think we need to bump this build number ( https://github.com/rapidsai/ucx-split-feedstock/pull/65 )


Feature/shadowing

Hello,
May I know how to add shadow and reflection here

I am trying to find out how but I can't. Thank you
Settings

Hello! It looks like you have created a pull request, to make code changes to Crest. I'm guessing this is not what you intended :).
If you dont mind could you please post this as an issue in the issue section? https://github.com/crest-ocean/crest/issues
There is a template for issues that has some questions including which version of crest you're using? We have 3 fairly different versions so it will depend on which version you're on, so the answer will depend.


CLI output vtk

The command line interface should have an option to enable VTK file output

see also #312

With the latest release of dune-copasi, this should be pretty easy since the state is now responsible for writing the vtk files.

resolved by #511, as all simulation settings are now saved in the sbml model & the CLI reads & uses these options when simulating


[Bug]  Follow us anchor tag not present.

Describe the bug
when click on social icon nothing happens
To Reproduce
click on social icons nothing will happen
Expected behavior
when click, it should redirect to the social sites of owner.
Screenshots

@Abhishek2634  this issue is already assigned ot someone kindly work on other issues and also work on issues which i already asssigned to you then make other issues


Any plans to implement the retention policy for reports which stored on filesystem?

Hi!
Our Bamboo Allure local storage growing is quite faster,
any plans to implement the allure retention policy on plugin level please?
We will be so happy :)
Cheers,
Gonchik

It will be good to understand the next step of that :)


How should the LayerTimelineController notify the StudioCtrl?

Hey Roman, I'm going to just file this as a question and maybe you can give me a hint when you get a chance. :D
I've been trying to figure out how to add an "Import" menu to allow SVGs to be imported via a file picker (as opposed to drag and drop), and I've basically got everything working except the part where the svg text content is somehow passed from the LayerTimelineController to the StudioCtrl.
StudioCtrl has an onDropFile(fileInfo) method that will trigger exactly the control flow I want... but I guess this is the part of angular I don't really understand yet (i.e. how communication between different controllers work and stuff).
If you have any advice that'd be cool. :D

In my CL f942bf24eab4d368c4ef950937f796a7a30ba48f, I moved a lot of the "add svg layers" logic out of StudioCtrl to StudioStateService. It's cleaner to rely on a global service to do some of this work rather than coupling LayerTimelineController to StudioCtrl.
In the future, if there's a compelling reason to couple them, you can try injecting StudioCtrl in the child controller's constructor


ENOTFOUND or ECONNREFUSED for all client functions

Just installed node-cloudservers via npm today.  So far, I haven't been able to use any of the client functions to create a server on Rackspace.  All return an err of either ECONNREFUSED (when I'm at work) or ENOTFOUND (when I'm at home).  I'm using Mac OS Snow Leopard, and I found a similar issue reported a year ago related to the use of /etc/resolv.conf for dns lookup resolution.   Other functions (npm, curl, etc.) appear to work fine.  Is there something I'm missing?

Can you paste the code you are attempting to run?
That error message is being thrown from node.js core, it generally means that your outgoing connection cannot find it's target ( bad URI / bad network / etc )

Thanks Marak.  Here's the code I'm trying to run.
var cloudservers = require('cloudservers'),
    sys = require('sys');

function createServer(serverName, client) {
  var options = {
    name: serverName,
    image: 49, // Ubuntu 10.04 (Lucid Lynx)
    flavor: 1, // 256 server
  };

  client.setAuth(function (err, res) {
    if(err) {
      console.log("Error Authenticating Client:")
      console.log(err);
    } else {
      client.createServer(options, function (err, server) {
        if(err) {
          console.log("Error Creating Server")
          console.log(err);
        } else {
          server.setWait({ status: 'ACTIVE' }, 5000, function () {
            // Our server is now built and active, so we can install node.js on it
            sys.puts('Your server ' + serverName + ' is now ready.');
            sys.puts('  IP Address: ' + server.addresses.public);
            sys.puts('  Root password: ' + server.adminPass);
          });
        }
      });
    }
  });
};

var args = process.argv.slice(2);
var serverName = args[0];
var username = args[1] || 'my-userid';
var apiKey = args[2] || 'my-api-key';

var config = {
  auth : {
    username: 'my-userid',
    apiKey: 'my-api-key'
  }
};

var client = cloudservers.createClient(config);

createServer(serverName, client);

I can successfully authenticate using curl with the following:
user$ curl -D -
-H "X-Auth-Key: (my-api-key)"
-H "X-Auth-User: (my-user-id)"
https://auth.api.rackspacecloud.com/v1.0
I agree with you.  Everything points to something funky with the outbound connection...but I don't seem to have issues with curl or npm.

Haven't resolved.  Any ideas?

Confirmed the same problem.  I used the client.getFlavors example from the readme.

Seems the current git repo works.  What is in npm doesn't.  Might need to publish version "0.2.9". Current in npm in "0.2.8"


Unify codepath for retrieveing metro location on Android

Summary:
With the current ways metro location is determined, when we want to use a different metro port this requires app to be rebuild as the port and location are stored in resource file that gets compiled to R.class. The only way to avoid app rebuild due to a port change is to use shared preferences that can be accessed from dev menu, where metro URL can be specified. However, due to a separate code-paths for retrieving bundle location and for /inspector/device calls, the setting only applies to the former. As a consequence, you can change metro URL in the shared preferences, but debugging would only work if you use the default port or you rebuild the app with the correct port number.
This PR removes the separate code-path for retrieving inspector URL including all the dependencies scattered across different files including the gradle plugin. We then replace calls to PackagerConnectionSettings.getInspectorServerHost with PackagerConnectionSettings.getDebugServerHost which respects the shared preferences and other possible ways of configuring the port.
I decided to remove the separate inspector URL code path, as the resource value for inspector port added in #23616 was never functioning properly due to a bug. In the said PR introduced a bug in AndroidInfoHelpers.java where react_native_dev_server_port was used instead react_native_inspector_proxy_port. As a result the added resource value was never read.
This can be potentially a breaking change as I'm removing some public methods. However I think it is unlikely anyone relied on said methods. As a part of this PR I'm also changing occurences of removed methods from ReactAndroid.api – I don't know how to test those changes since I don't understand how this file is used as it doesn't have any references in public code.
Changelog:
[ANDROID] [FIXED] - Make Android respect metro location from shared preferences for the debugger workflow
Test Plan:

Run android app on emulator using default port
Check the debugger works when using "Open Debugger" option from dev menu
Restart metro with custom port (--port 9090) while keeping the app running
Open dev menu, click "Settings" then "Debug server host & port", put "<IP_ADDRESS>:9090" there
Reload the app
Before this change things like hot reload would continue to work while "Open Debugger" option would do nothing
After this change both reloading and debugging will work

Important: I haven't tested changes made to ReactAndroid.api as I don't know what this files is used for with no references in the codebase.

Thanks, an alternative approach would be to keep these codepaths separate. Meaning that we'd keep separate methods for getting inspector proxy URL and for dev server URLs. I can just make it so that the inspector proxy URL honors the shared preferences and falls back to old behavior otherwise.

That's exactly why I'd like someone from DevX to share their thoughts on this


External Mosquitto Connection Fails due to SSL Error

Checklist

[x]  I am not using Home Assistant. Or: a developer has told me to come here.
[x] I have checked the troubleshooting section and my problem is not described there.
[x] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Z-Wave JS UI version
8.10.1.e5d6c21
ZwaveJS version
10.11.1
Describe the bug
I configured Z-Wave JS UI to point to my own Mosquitto server rather than the local one. Once config is complete, I see a bunch of connection errors in the server logs:
2023-03-18T18:39:26: Client connection from <IP_ADDRESS> failed: error:1408F10B:SSL routines:ssl3_get_record:wrong version number.
I'm able to publish messages with the following mosquitto_pub command, so suspect this is an issue with MQTT over TLS implementation in Z-Wave JS:
mosquitto_pub -L mqtt://MyMosquittoServer.com:1883/test -i UbuntuClientPub --capath /etc/ssl/certs/ -m "This works!"
To Reproduce
Configure Z-Wave JS to point to your own Mosquitto endpoint, configured to support SSL thusly:
listener 1883
socket_domain ipv4
protocol mqtt
allow_anonymous true
certfile /certs/fullchain.pem
keyfile /certs/privkey.pem
tls_version tlsv1.1
capath /etc/ssl/certs/

My config looks like this:

Expected behavior
MQTT connects correctly.
Additional context
No response

It seems like mqtts is used when the client is encrypting with a certificate (e.g. for mutual TLS). Given that my mosquitto_pub command works with mqtt: but breaks with mqtts: I'm assuming mqtt is the correct protocol.

mqtts was the solution, actually! Thank you!


IMU Adress Clash Issue

We have 4 imu's that need to be mounted on arm for homing, issue is that current imu we have selected for use has only 2 possible addresses.
Electrical is currently exploring two possible actions:

Use two brands of imu's with different addresses.
Address translate two of the imus.

You may want to throw one more option into the fold here.

Use two I2C buses

The Stm32f103CU MCU that we are using has two I2C buses, so for the reference IMU and shoulder IMU can be on I2C1 and elbow and wrist IMU can be on I2C2.

Noted, I have requested samples of LTC4316 i2c address translators, Ill build a prototype board and we will test to see if we can use it. If we run into issues with signal integrity and to many i2c errors we will only then order two different imu's. This is being done to avoid rewriting new drivers.

Noted, I have requested samples of LTC4316 i2c address translators, Ill build a prototype board and we will test to see if we can use it. If we run into issues with signal integrity and to many i2c errors we will only then order two different imu's.

This is good. It will be good to see if we can make these devices work properly as before, it had not been achieved. I'm certain it will work and that mistakes were made previously that prevented the translators from working.

This is being done to avoid rewriting new drivers.

I'm not sure what you mean by this. If we have an I2C peripheral driver, that driver is reusable across all of the I2C ports in general for almost all MCUs. Some devices will have slightly different register layouts to promote additional special features, but if we get 1 I2C bus working, then we have them all working.
Also, if you are looking to simplify the complexity, omitting the LTC translators would be the way to go, as we would simply need to keep the two sets of IMUs on different buses and call it a day.
And to keep perspective, the rover needs to be done somewhere in early January in order to give controls and the team a chance to make the rover work enough and tested enough to actually be able to produce a video report for competition. So if we can reduce complexity for the first iteration (we can always be fancy in the next iteration) then we should.

Update: Received Sample Chips waiting for breakout board for chip package to build prototype units.

@maximvoven may have already come up and voiced this idea prior, but I thought it bares the need to be written here.
Something I had considered a week ago regarding the IMU address clash. I was thinking back to the usage of USB-C on the Rover and I think the usage of the cable should be limited to just the RMD-X7 motors. Potentially just for Drive and the Wrist motors as they are far away from their control boards. The shoulder and elbow motors on the arm board will probably be so close to the board that a direct ZH1.5-8 connector can be used directly. Using the ZH1.5-8 connector will not increase the BOM lines as it will be used anyway for the final connection from USB-C to the motor via a breakout board.
Something to note about the USB-C 2.0 cables I've worked with is that they DO NOT include the SBUx lines, just VCC, GND, D+, D- and CCx, so the pin count is a bit more limited than expected.
My suggestion for the IMU connector is something like this:

RJ45 Pinout
Signals

PIN1-GW
SCL

PIN2-G
VCC

PIN3-OW
CS-IMU1

PIN4-B
CS-IMU2

PIN5-BW
CS-IMU3

PIN6-O
CS-IMU4

PIN7-BrW
GND

PIN8-Br
SDA

The idea here is to daisy chain ethernet connectors from the shoulder to elbow to wrist. And since each node in the chain will have access to all 8 pins of the ethernet connector, we can specialize specific pins for specific devices. In this case, power and communication take up 4 pins and the last pins are chip select pins that connect to the AD0 pin of the MPU6050. This would eliminate the need for additional ICs such as the LTC4316 and makes each daisy chain board containing the MPU6050 as simple as two ethernet adapters, a decoupling capacitor, a place to slot in the MPU6050s and a dip switch or shorting block used to select which CS changes the Address pin for that node.
Again I still believe that it is extremely useful to get the LTC4316 chip working so that we know we can get that chip working and figure out any caveats with the IC in the event we need it in the future.

LTC4316 has been forced into working, issue is non issue


Fix monochrome icon, add warning when it's not specified

Fix small typo which resulted in setting adaptive background icon to monochrome one if monochrome is specified.

@Summersay415 Can you add a warning in Export Present if monochrome icon is not provided.
Eg. No monochrome icon provided; default Godot monochrome icon will be used.

Should I do this with adaptive background too?

I don't think that's needed, default adaptive background is just a plane color.
But in case of monochrome icon, I think it would be better to add this warning. It is a new feature, so it can be overlooked during export and it is not just a plane color insted Godot monochrome logo.
CC @m4gr3d

I should this warning in export window or just print in console?

In export window, like we have for invalid pakage name.

like this?

Thanks!

No need to backport this to 4.3; it has already been addressed in https://github.com/godotengine/godot/pull/100066


Characters are garbled

Describe the Bug
When I use OHIF Viewer to display the panoramic image, I find that the name is garbled, how do I need to modify it?

Could you kindly provide the data if it has been anonymized and you can confirm that there is no patient health information present in any of the headers or embedded within the pixel data?


error: wrong arguments

Expected Behavior
My dwg file:

I have dxf file of this dwg but I have this error:
Actual Behavior

Steps to Reproduce the Problem

Cloned repository
Open Index in OpenJSCAD.org-master\packages\desktop
Upload a dxf file.

@rgrastur Thanks for using JSCAD
Sadly, V1 suppprt for 2D drawings is not good. The real focus of JSCAD is 3D designs for 3D printing.

@rgrastur if still interested then please try V2.

@rgrastur please provide an example, otherwise this issue will be closed.


How to include prelude to run .ls file?

Working on a command Iine util, I tried
lsc -wd my.ls

where I have a simple use of map. But I got:
ReferenceError: map is not defined

Using "lsc -d" interactively to test the code is all good. Also tried "--prelude", which makes no difference.
I also have "npm i -g prelude-ls" but always got "Error: Cannot find module 'prelude-ls'" when I tried to require it in node.
So, what's missing?

Ok, turns out it's old issue #290
So install prelude locally and require it inside my.ls is the way to go.
Does that mean we should have  -d --prelude really work for "lsc -d" to run .ls script ... missing feature ?

That's how node's require work: no globals


Fixed bad grammar.

Almost as annoying as Hayt being friendly.

I haven't ever noticed this.

haha, neither have I


infinite loops

compute is still subject to infinite loops

grab data, run compute from UI

How?  Max iterations should disallow infinite loops.

I wish I knew

On May 19, 2022, at 4:06 PM, Will Lehman @.***> wrote:

How? Max iterations should disallow infinite loops.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were assigned.

https://github.com/HydrologicEngineeringCenter/fda-model/blob/4e60264dfe7f427d369b64055ca7d5feab38d634/fda-model/compute/ImpactAreaScenarioSimulation.cs#L236
in this block of code, write some messages or write to console to determine how many iterations have computed (_computedIterations) and determine if it is more than the max iterations set on the convertgence criteria. if it is we should break under that scenario.

Ok. Thank you for the advice. I plan to work on this Sunday. I’ll give it a try then.

On May 20, 2022, at 9:08 AM, Will Lehman @.***> wrote:

https://github.com/HydrologicEngineeringCenter/fda-model/blob/4e60264dfe7f427d369b64055ca7d5feab38d634/fda-model/compute/ImpactAreaScenarioSimulation.cs#L236
in this block of code, write some messages or write to console to determine how many iterations have computed (_computedIterations) and determine if it is more than the max iterations set on the convertgence criteria. if it is we should break under that scenario.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were assigned.

im not sure what it is that would cause an infininte loop, the only other thing i could think of is if the fix for the background thread on inline histogram is somehow not in your branch?

I merged that fix into main a long time ago.

On May 20, 2022, at 9:22 AM, Will Lehman @.***> wrote:

im not sure what it is that would cause an infininte loop, the only other thing i could think of is if the fix for the background thread on inline histogram is somehow not in your branch?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were assigned.

The infinite loop occurs intermittently when computing a Scenario with an analytical flow-frequency using the user interface. I ran our unit tests multiple times, and we never get caught in an infinite loop. If the problem was in fda-model, we should be able to replicate the bug, right? We have a sufficient variety of study configurations that include an analytical flow-frequency at this point to rule out study configuration being the problem. I think this suggests that the problem comes from the user interface, but it does not seem intuitive that being stuck on the compute method stems from something other than the compute method.

the loop gets stuck on the results = simulation.Compute(randomProvider, cc);  in the below method. I don't see anything fishy about the below method that could lead us into issues. The only problems I am seeing is that the messaging stuff "does not exist in the current context" but I don't remember seeing that error in @codymccoy 's machine. Also, I am not sure that problems in a given line of code at run time would leave us stuck on the previous line of code. I think this would be a build error, not a run time error, right?
        {
            ImpactAreaScenarioResults results = null;

            AnalyticalFrequencyElement freqElem = (AnalyticalFrequencyElement)StudyCache.GetChildElementOfType(typeof(AnalyticalFrequencyElement), FlowFreqID);
            InflowOutflowElement inOutElem = (InflowOutflowElement)StudyCache.GetChildElementOfType(typeof(InflowOutflowElement), InflowOutflowID);
            RatingCurveElement ratElem = (RatingCurveElement)StudyCache.GetChildElementOfType(typeof(RatingCurveElement), RatingID);
            ExteriorInteriorElement extIntElem = (ExteriorInteriorElement)StudyCache.GetChildElementOfType(typeof(ExteriorInteriorElement), ExtIntStageID);
            LeveeFeatureElement leveeElem = (LeveeFeatureElement)StudyCache.GetChildElementOfType(typeof(LeveeFeatureElement), LeveeFailureID);
            AggregatedStageDamageElement stageDamageElem = (AggregatedStageDamageElement)StudyCache.GetChildElementOfType(typeof(AggregatedStageDamageElement), StageDamageID);

            SimulationCreator sc = new SimulationCreator(freqElem, inOutElem, ratElem, extIntElem, leveeElem,
                stageDamageElem, ImpactAreaID);

            int thresholdIndex = 1;
            foreach(ThresholdRowItem thresholdRow in Thresholds)
            {
                Threshold threshold = new Threshold(thresholdIndex, new ConvergenceCriteria(), thresholdRow.ThresholdType.Metric, thresholdRow.ThresholdValue);
                sc.WithAdditionalThreshold(threshold);
                thresholdIndex++;
            }

            FdaValidationResult configurationValidationResult = sc.IsConfigurationValid();
            if (configurationValidationResult.IsValid)
            {
                ImpactAreaScenarioSimulation simulation = sc.BuildSimulation();
                int seed = 999;
                RandomProvider randomProvider = new RandomProvider(seed);
                ConvergenceCriteria cc = new ConvergenceCriteria();
                try
                {
                    results = simulation.Compute(randomProvider, cc);
                    MessageBox.Show("Simulation computed successfully.", "Compute Completed", MessageBoxButton.OK, MessageBoxImage.Information);

                }
                catch (Exception ex)
                {
                    MessageBox.Show(ex.Message, "Failed Compute", MessageBoxButton.OK, MessageBoxImage.Error);
                }
            }
            else
            {
                MessageBox.Show(configurationValidationResult.ErrorMessage, "Invalid Configuration", MessageBoxButton.OK, MessageBoxImage.Exclamation);
            }
            return results;

        }

@codymccoy do you know if we're still running into this issue? I have not. Unfortunately, I don't have a record of any commits on this issue. I need to be more consistent in using "Fixes #...."

I have not encountered an infinite loop during the compute in quite a while. Seems like this is probably fixed.

thanks! I'll close this one assuming I fixed whatever the problem was and open a new one if we run into another


Mutiple headwords as seperated entries?

Hello,
I have made my own dictionary in DF format and coverted it to Kobo zip file using pyGlossary. Author suggested I ask here for information regarding valid entry format that Kobo reader can process. Original thread:
https://github.com/ilius/pyglossary/issues/356#issuecomment-1023167507
Almost all my df entries look like:
@ entry_name
: \pronanuciation\
& alias1
& alias2
noun here comes the descrition

But some can have different meaning:
@ name1
: \aaaa\
noun description

@ name2
: \aaaa\
adjective different description

Kobo doesn't seem to find this when looking at word. What is proper way to define this so Kobo can show both of the words? Should I merge somehow? Or add alias to it (&)?
I also have some headwords with optional characters that are inside parentheses. Not sure if this works.
@ backward(s)
: \baekwəd(z)\
adverb description...

Thank you for all the suggestions.

Have you tried this with dictgen instead of PyGlossary? If I understand what you're asking, it should work fine.
Regarding backward(s), you'll have to use variants to match all forms of the word (also note that kobo does more than just exact matches).

I apologize in advanced. Did not know df is your "standard" :). I was googling on how to make my own dictionary and went from penelope to PyGlossary. On your web page I only looked on how df format looks.
Tried dictgen and found I still have some issues inside my df file that PyGlossary passed but dictget did not. When I fixed them I got a seems to be valid zip file.
c:\Temp\eBooks\dictgen>dictgen-windows.exe en-sl.df
Parsing dictfiles.
Error: input "en-sl.df": dictfile: line 10677: no word after variant specifier (&).

c:\Temp\eBooks\dictgen>dictgen-windows.exe en-sl.df
Parsing dictfiles.
Error: input "en-sl.df": dictfile: line 10677: no word after variant specifier (&).

c:\Temp\eBooks\dictgen>dictgen-windows.exe en-sl.df
Parsing dictfiles.
Error: input "en-sl.df": dictfile: line 51548: header info (: or ::) specified within definition content (prepend a space if this was intended to be part of the definition itself).

c:\Temp\eBooks\dictgen>dictgen-windows.exe en-sl.df
Parsing dictfiles.
<w>
A
<var></var>abbreviation chemistry argon; absolute; Academy; America(n)</w><nil>
Opening output.
Generating dictzip.
  Using image method: optimize and encode as base64 data URL (max_width=1000, max_height=1000, grayscale=false, jpeg_quality=60) (warning: this causes segfaults in the in-book dictionary due to a bug in nickel with firmware versions below 4.20.14601).
Successfully wrote 73947 entries from 1 dictfile(s) to dictzip dicthtml.zip.

Output looks correct? That html entry looks strange :).
So if I understood correctly I can keep samename1, samename2, samename3 and they should be parsed correctly and displayed on Kobo?

Regarding backward(s), you'll have to use variants to match all forms of the word (also note that kobo does more than just exact matches).

This is not clear to me. I need to create two entries. Like this?
@ backward
& backwards

My source for dictonary are text files. I wrote my own program to parse the text files. So I can manipulate the content before assembling the df file. Just want to do it correctly.
Thank you very much.

The errors are expected. Note that if they're intended to be part of the content and are at the beginning of a line, you need to escape them by putting a space before them.
That HTML entry looks fine, but was that printed with the output (it shouldn't be doing that). Does the dictionary work correctly on your kobo?
You are correct about the backwards part.

First I need to redo the df generation. Fix this (x) words so they are without letters in parentheses and word with included letters is just an alias.
Regarding the alias words (&). Can I use alias word but still have the word definition? For example:
@ word
& alias_word
: \pronoucionation\
html code

@ alias_word
: \pronoucionation\
html code

That HTML entry looks fine, but was that printed with the output (it shouldn't be doing that). Does the dictionary work correctly on your kobo?

Yes. Html code was in output.
Not yet tested it on Kobo zip file generated by dictgen. Need to make df perfect first :). Only the one made from PyGlossary and it was working, but I saw it is not that perfect. All the words with number at end were shown with number and Kobo displayed them as the one it found later, not directly. For example word almost was not found, but he displayed almost1 that is inside dictionary.
Need to test this with dictgen. Will words like test1 test2 test3 be found in Kobo when I highlight word test? Or must I merge this into one dictionary headword?

Finished with the (x) words. Now they are aliases and there are no more words like this inside my df. I have also fixed the blank aliases errors and made them unique. No more errors in dictget when generating.
It created valid zip file and it is working on my Kobo (Libra 2). I still see the html code in output of dictgen. This code is from first headword in df file:
@ A
abbreviation chemistry argon; absolute; Academy; America(n)

@ A
1 abbreviation first class

Here is the output:
c:\Projekti\KoboDictionaryMaker\KoboDictionaryMaker\bin\Debug>dictgen-windows.exe -o dicthtml-en-sl.zip en-sl.df
Parsing dictfiles.
<w>
A
<var></var>abbreviation chemistry argon; absolute; Academy; America(n)</w><nil>
Opening output.
Generating dictzip.
  Using image method: optimize and encode as base64 data URL (max_width=1000, max_height=1000, grayscale=false, jpeg_quality=60) (warning: this causes segfaults in the in-book dictionary due to a bug in nickel with firmware versions below 4.20.14601).
Successfully wrote 73947 entries from 1 dictfile(s) to dictzip dicthtml-en-sl.zip.

I also tested this dictionary on my Kobo. It seems to work fine for 99% words. But the words with many meaning and with just different number as suffix are not found. For example:

Can you suggest how to solver this properly? Merging all this entries as one headword (disguise in my case) and seperate all word description as html lists?
What would be best example of df file that already has all this features and I could use as reference?

To merge separate entries, you'd need to do that in the tool generating the df file.
For examples, look at the GOTDict and Webster's df files (there's tools to generate them in the releases). Those have a few words with multiple separate entries for the same headword, merged entries, images, variants, and other features.

Hello. Finished my dictionary. Merged what could be merged inside df file and it looks and works great. Thank you for a great tool.


Cannot ctrl+c abort

I have a problem to abort a program with ctrl+c, namely ngrok in this case. I also turned off the intelligent ctrl+c copy, but that doesn't have any influence.

Aborting works in WSL shells
Aborting does not work in PowerShell, cmd.exe and with cmder when ConPTY is deactivated
Aborting works when ConPTY is activated

My current version is <IP_ADDRESS>1.

Yep, same. ctrl+c just adds ^C to commandline input on powershell.

Fixed in 2227f2c

@Eugeny I don't know if this is related to my issue, however shell takes long to cancel a program in node. I hit Ctrl+C and sometimes takes up to 4 minutes.


smoketest fails in convolution-inference

I just did a fresh compile of NNPACK because in my ported build I encountered a failure in the smoketest. But the same thing is happening in NNPACK.
[6/10] RUN convolution-inference-smoketest
[==========] Running 132 tests from 9 test cases.
[----------] Global test environment set-up.
[----------] 16 tests from FT8x8
[ RUN      ] FT8x8.single_tile
Floating point exception (core dumped)
FAILED: convolution-inference-smoketest
/home/Filip/NNPACK/bin/convolution-inference-smoketest --gtest_color=yes
ninja: build stopped: subcommand failed.
thanks

I tried it just now, and couldn't reproduce it. Could you post which platform you build on, and a complete sequence of commands you run during the build?

I'm on a fresh install of Ubuntu 18.04.
gcc 7.3
clang 6.0.0

Thanks, the issue is gone when using clang.


Go: Add List.IteratorAt() and use in list merging

When merge.ThreeWay() merges list splices, it frequently needs to
extract a slice of a List or check whether a given range of values is
exactly equal in two different Lists. Repeatedly Getting elements from
a List is expensive, because it creates a new cursor internally every
time. Adding IteratorAt(idx uint64) allows code to iterate over a
range of a List while only creating a cursor once. This allows a slice
of Values to be extracted or used in comparisons efficiently.
Toward #2445

PTAL

WDYT? You could even call the new API RangeFrom or something, to distinguish it.
also should add a quick test either way.

Ok. I've changed to List.Iterator() and List.IteratorAt(). PTAL

lgtm but see comment about composition. I don't want methods from the cursor being available on the iterator.

Another trickle of comments. Feel free to ping me directly if my turnaround is too slow.


Opening file with LaunchFileInfoAsync where path/filename contains space character

Describe the bug
When LaunchFileInfoAsync is used in Linux and the path/filename contains a space, then it fails, because the rest of the string after the space is interpreted as parameter.
xdg-open: unexpected argument 'rest-of-filename.txt'
Try 'xdg-open --help' for more information.
To Reproduce

Code:
var launcher = TopLevel.GetTopLevel(control).Launcher;
await launcher.LaunchFileInfoAsync(fileInfo); // fileInfo where the path or the filename contains a space character

Run under Linux

Workaround (used extra "" for filepath):
if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
{
Process.Start(new ProcessStartInfo(filePath) { UseShellExecute = true });
}
else if (RuntimeInformation.IsOSPlatform(OSPlatform.Linux))
{
Process.Start(new ProcessStartInfo
{
FileName = "xdg-open",
Arguments = $""{filePath}"",
UseShellExecute = false,
CreateNoWindow = true
});
}

Expected behavior
Space characters should be supported.
Avalonia version
11.1.4
OS
Linux
Additional context
No response

Will work on that


Add analyzer tests

Hi there! 👋
I'd like to change the parser over to using YARP instead of Syntax Tree. Going forward Syntax Tree is mostly going to be used as a wrapper around YARP anyway. This is going to provide you with quite a speed and memory improvement, so it's probably going to be worth it for this site.
This commit itself only provides a test suite for the analyzer. It's the first step to make sure I don't accidentally break anything by switching the parser.
I didn't add anything to the CI because I figured that wasn't really in the scope of this PR, but feel free to add an action that runs bin/rails test/bundle exec rake. All the tests are passing at the moment.

Hey Kevin 👋🏼
Thank you for this pull request! SyntaxTree has been a pleasure to use and it's been really exciting to follow the progress on both SyntaxTree and YARP. I'm really looking forward to the improvements and benefits coming to SyntaxTree thanks to YARP!
If it helps, I'm also using SyntaxTree in the phlexing gem which has a pretty robust test suite already.


Support RegExp in path GoRoute

Use case
Navigate packages are made to make it easier to handle routing, and now GoRouter has completely ignored this principle by making it difficult and removing a variety of commonly used methods.
In the Velocity_x package, I have the possibility to insert the route as a RegExp, and now this is not possible in GoRouter, and only the
parameters are handled through the RegExp!
Expected RegExp path:
static const resetPasswordScreen = "\/(resetPasswordProfile|profile)(\/password)?";

The above RegExp should support the following four paths:
---> resetPasswordScreen = "/resetPasswordProfile"
---> resetPasswordScreen = "/resetPasswordProfile/password"
---> resetPasswordScreen = "/profile"
---> resetPasswordScreen = "/profile/password"

GoRoute(
  name: 'resetPasswordScreen',
  path: resetPasswordScreen,
  parentNavigatorKey: rootNavigatorKey,
  pageBuilder: (context, state) => _transitionPageBuilder(context, state, const ResetPasswordProfile()),
)

I want to know this feature exist or not, and if not, can you implement it?
Proposal
static const resetPasswordScreen = "\/(resetPasswordProfile|profile)(\/password)?";

GoRoute(
  name: 'resetPasswordScreen',
  path: resetPasswordScreen,
  parentNavigatorKey: rootNavigatorKey,
  pageBuilder: (context, state) => _transitionPageBuilder(context, state, const ResetPasswordProfile()),
)

@jimpatrick
Can you take a look at this and see if it helps in your case or not ?

@jimpatrick Can you take a look at this and see if it helps in your case or not ?

No, as I said, only parameters can be checked by RegExp in GoRouter, and this is different from my case.
Please check path in GoRouter package.
This code about convert path to RegExp in GoRouter package:
RegExp patternToRegExp(String pattern, List<String> parameters) {
  final StringBuffer buffer = StringBuffer('^');
  int start = 0;
  for (final RegExpMatch match in _parameterRegExp.allMatches(pattern)) {
    if (match.start > start) {
      buffer.write(RegExp.escape(pattern.substring(start, match.start)));
    }
    final String name = match[1]!;
    final String? optionalPattern = match[2];
    final String regex = optionalPattern != null
        ? _escapeGroup(optionalPattern, name)
        : '(?<$name>[^/]+)';
    buffer.write(regex);
    parameters.add(name);
    start = match.end;
  }

  if (start < pattern.length) {
    buffer.write(RegExp.escape(pattern.substring(start)));
  }

  if (!pattern.endsWith('/')) {
    buffer.write(r'(?=/|$)');
  }
  return RegExp(buffer.toString(), caseSensitive: false);
}

Thanks for the feedback.

I think you can workaround this by define two goRoute
GoRoute(/:id(resetPasswordProfile|profile))
GoRoute(/:id(resetPasswordProfile|profile)/password)

I think you can workaround this by define two goRoute GoRoute(/:id(resetPasswordProfile|profile)) GoRoute(/:id(resetPasswordProfile|profile)/password)

Good idea but it's not perfect! Path parameter in GoRouter class should support RegEx :)


Bi-Weeklies: 22/Q4 & 23/Q1 & 23/Q2

22/Q4

https://github.com/bfh/open-source/issues/12
https://github.com/bfh/open-source/issues/25
https://github.com/bfh/open-source/issues/29

23/Q1

https://github.com/bfh/open-source/issues/34
https://github.com/bfh/open-source/issues/13 / https://github.com/bfh/open-source/issues/36
https://github.com/bfh/open-source/issues/37
https://github.com/bfh/open-source/issues/38
https://github.com/bfh/open-source/issues/39
https://github.com/bfh/open-source/issues/40
https://github.com/bfh/open-source/issues/41

23/Q2

[ ] Find deputy for Bi-Weekly moderation


Block these domains

Describe your Suggestion
A detailed description of your Suggestion.
Example: Adding example.com to the Suspicious Blocklist.
Please add these to the ads list:
ads-api.tiktok.com
ads-sg.tiktok.com
business-api.tiktok.com
adtech.yahooinc.com
https://widgets.pinterest.com
https://advertising.yandex.ru
Describe why you think that would be useful
A clear and concise description of why you think your Suggestion would benefit everyone.
Example: The Site hosts Malicious content which includes Discord Token Grabbers.
Describe alternatives you've considered or examples of similar Solutions
Are there any alternative solutions that already fulfill your Suggestion in a similar form, so that we can use it as our Baseline?
Additional context
Add any other context or screenshots about your Suggestion here.

I'm not so sure if these belong in the suspicious category and not rather into advertising, but accepted nonetheless.
Thank you for your contribution!


Add index.md for the remote topic with description and resources

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/main/CONTRIBUTING.md.
[ ] I am not the sole author or employee of a company who created either the topic I am modifying/adding or the collection entry I am modifying/adding.

Which change are you proposing?
This pull request adds an index.md file for the remote topic, including:

A detailed description of the remote topic.
Links to related repositories and tools.
An image for better presentation.

This is my first contribution to this repository, and I’m excited to contribute to GitHub Explore. Please let me know if there are any changes or improvements needed. Thank you for your time and review!

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
[x] Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[ ] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing topic or collection, or at least not a single one
[ ] My suggested edits are not about curating a new topic or collection, or at least not a single one
[ ] My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Hi, thank you for reviewing my pull request! I mistakenly checked the box stating that I am not the sole author of the topic I’m modifying. However, I am the sole author of the changes made in this pull request, and I’ve contributed them independently.
Please let me know if there’s anything else I need to clarify or update to help move this forward. Thank you for your time and assistance!

Hello! In short, do as you please, it's all the same, to be honest, no one
will appreciate it, unless you're the new A. Einstein with your brilliant
ideas! All the best and good luck!
вт, 31 дек. 2024 г. в 14:53, Tech Headhunter @.***>:

Hi, thank you for reviewing my pull request! I mistakenly checked the box
stating that I am not the sole author of the topic I’m modifying. However,
I am the sole author of the changes made in this pull request, and I’ve
contributed them independently.
Please let me know if there’s anything else I need to clarify or update to
help move this forward. Thank you for your time and assistance!
—
Reply to this email directly, view it on GitHub
https://github.com/github/explore/pull/4569#issuecomment-2566424559, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BGXS6F4EPT746VINXLDLBPT2IKHTJAVCNFSM6AAAAABUNOHIGKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRWGQZDINJVHE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Added gcc8 CI build for Linux; trying to make "skipped steps" go away.

The skipped steps business seem to suggest that the way the yaml file is organized, there's no way for the runners to discern the dependencies for each job.  Will dig into this later....


[Snyk] Security upgrade golang from 1.22.5-alpine3.20 to alpine3.20

Snyk has created this PR to fix 2 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

tool/actions-gh-release/Dockerfile

We recommend upgrading to golang:alpine3.20, as this image has only 1 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

CVE-2024-6119 SNYK-ALPINE320-OPENSSL-7895537
  364

CVE-2024-6119 SNYK-ALPINE320-OPENSSL-7895537
  364

CVE-2024-9143 SNYK-ALPINE320-OPENSSL-8235201
  364

CVE-2024-9143 SNYK-ALPINE320-OPENSSL-8235201
  364

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

https://github.com/pipe-cd/pipecd/pull/5335 seems better (format is the same)


Question about function use

In 'mmdet/models/bbox_heads/fc_bbox_head.py'
line 156-162 code is :
’cls_reg_targets = fcos_target(
pos_proposals,
neg_proposals,
pos_gt_bboxes,
pos_gt_labels,
rcnn_train_cfg)‘
I wonder why this function is ’fcos_target‘? And I check this function which is in 'mmdet/models/anchor_heads/fcos_head.py',
but they have different parameter list.

If here is the function which is the 'fc_target' in the header of this file?


Fix crashing issue on Tabs when scrollView ref is not defined

🚨  URGENT! 🚨
This PR fixes a crashing issue on src/basic/Tabs/index.js on componentDidMount() function when scrollView ref is not defined.
NOTE: This is a blocking issue in one of our projects so we are referencing the dependency from my forked repository until this PR gets merged.
Error:

SIGABRT: Unhandled JS Exception: null is not an object (evaluating 'this.scrollView.scrollTo')
Location: src/basic/Tabs/index.js:80

Thanks for the fix, this was fixed in a later PR which has already been merged!

When it will be uploaded to npm


⚠ ClamAV has degraded performance

In 2f96c5e, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 562 ms

Resolved: ClamAV performance has improved in 9b68db5.


SelfHost:  ServerSelectionTimeoutError on make nb

Following the install instructions, but failing applying migrations.   I'm running on a few VM running Ubuntu 20.04.6 LTS x64
> git clone https://github.com/samuelclay/NewsBlur.git
> cd NewsBlur
> git log --oneline
cc748198d (HEAD -> master, origin/master, origin/HEAD) Merge branch 'sictiru'
21190c18b (origin/sictiru) Android v13.0.0

> sudo make nb

Fails /bin/bash: gtimeout: command not found and sudo: /usr/bin/security: command not found.
Run sudo make nb again:
 ---> Starting NewsBlur development server...
Operations to perform:
  Apply all migrations: admin, analyzer, auth, contenttypes, django_ses, feed_import, ipn, oauth2_provider, profile, push, reader, recommendations, rss_feeds, sessions, sites, social, zebra
Running migrations:
  Applying rss_feeds.0002_remove_mongo_types...Traceback (most recent call last):
  File "/srv/newsblur/./manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 330, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 371, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 85, in wrapped
    res = handle_func(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/commands/migrate.py", line 243, in handle
    post_migrate_state = executor.migrate(
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 117, in migrate
    state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 147, in _migrate_all_forwards
    state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 227, in apply_migration
    state = migration.apply(state, schema_editor)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/migration.py", line 124, in apply
    operation.database_forwards(self.app_label, schema_editor, old_state, project_state)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/operations/special.py", line 190, in database_forwards
    self.code(from_state.apps, schema_editor)
  File "/srv/newsblur/apps/rss_feeds/migrations/0002_remove_mongo_types.py", line 8, in remove_mongo_types
    collections = db.collection_names()
  File "/usr/local/lib/python3.9/site-packages/pymongo/database.py", line 910, in collection_names
    for result in self.list_collections(session=session,
  File "/usr/local/lib/python3.9/site-packages/pymongo/database.py", line 848, in list_collections
    return self.__client._retryable_read(
  File "/usr/local/lib/python3.9/site-packages/pymongo/mongo_client.py", line 1514, in _retryable_read
    server = self._select_server(
  File "/usr/local/lib/python3.9/site-packages/pymongo/mongo_client.py", line 1346, in _select_server
    server = topology.select_server(server_selector)
  File "/usr/local/lib/python3.9/site-packages/pymongo/topology.py", line 244, in select_server
    return random.choice(self.select_servers(selector,
  File "/usr/local/lib/python3.9/site-packages/pymongo/topology.py", line 202, in select_servers
    server_descriptions = self._select_servers_loop(
  File "/usr/local/lib/python3.9/site-packages/pymongo/topology.py", line 218, in _select_servers_loop
    raise ServerSelectionTimeoutError(
pymongo.errors.ServerSelectionTimeoutError: db_mongo:29019: [Errno -3] Temporary failure in name resolution, Timeout: 30s, Topology Description: <TopologyDescription id: 64d46597d06506dab7b7619b, topology_type: Single, servers: [<ServerDescription ('db_mongo', 29019) server_type: Unknown, rtt: None, error=AutoReconnect('db_mongo:29019: [Errno -3] Temporary failure in name resolution')>]>
Sentry is attempting to send 2 pending error messages
Waiting up to 2 seconds
Press Ctrl-C to quit
make: *** [Makefile:38: migrate] Error 1

I tried deleting all docker image and re-cloning the repo.  Got a bit further this time, but still timedout:
Operations to perform:
  Apply all migrations: admin, analyzer, auth, contenttypes, django_ses, feed_import, ipn, oauth2_provider, profile, push, reader, recommendations, rss_feeds, sessions, sites, social, zebra
Running migrations:
  Applying contenttypes.0001_initial... OK
  Applying auth.0001_initial... OK
  Applying admin.0001_initial... OK
  Applying admin.0002_logentry_remove_auto_add... OK
  Applying admin.0003_logentry_add_action_flag_choices... OK
  Applying rss_feeds.0001_initial... OK
  Applying analyzer.0001_initial... OK
  Applying contenttypes.0002_remove_content_type_name... OK
  Applying auth.0002_alter_permission_name_max_length... OK
  Applying auth.0003_alter_user_email_max_length... OK
  Applying auth.0004_alter_user_username_opts... OK
  Applying auth.0005_alter_user_last_login_null... OK
  Applying auth.0006_require_contenttypes_0002... OK
  Applying auth.0007_alter_validators_add_error_messages... OK
  Applying auth.0008_alter_user_username_max_length... OK
  Applying auth.0009_alter_user_last_name_max_length... OK
  Applying auth.0010_alter_group_name_max_length... OK
  Applying auth.0011_update_proxy_permissions... OK
  Applying auth.0012_alter_user_first_name_max_length... OK
  Applying django_ses.0001_initial... OK
  Applying feed_import.0001_initial... OK
  Applying ipn.0001_initial... OK
  Applying ipn.0002_paypalipn_mp_id... OK
  Applying ipn.0003_auto_20141117_1647... OK
  Applying ipn.0004_auto_20150612_1826... OK
  Applying ipn.0005_auto_20151217_0948... OK
  Applying ipn.0006_auto_20160108_1112... OK
  Applying ipn.0007_auto_20160219_1135... OK
  Applying ipn.0008_auto_20181128_1032... OK
  Applying oauth2_provider.0001_initial... OK
  Applying oauth2_provider.0002_auto_20190406_1805... OK
  Applying profile.0001_initial... OK
  Applying profile.0002_auto_20200620_0803... OK
  Applying profile.0003_auto_20201005_0932... OK
  Applying profile.0004_auto_20220110_2106... OK
  Applying profile.0005_profile_is_archive... OK
  Applying profile.0006_profile_days_of_unread... OK
  Applying profile.0007_auto_20220125_2108... OK
  Applying profile.0008_profile_paypal_sub_id... OK
  Applying profile.0009_paypalids... OK
  Applying profile.0010_profile_active_provider... OK
  Applying profile.0011_auto_20220408_1908... OK
  Applying profile.0012_auto_20220511_1710... OK
  Applying push.0001_initial... OK
  Applying reader.0001_initial... OK
  Applying recommendations.0001_initial... OK
  Applying rss_feeds.0002_remove_mongo_types...Traceback (most recent call last):
  File "/srv/newsblur/./manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 330, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 371, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 85, in wrapped
    res = handle_func(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/commands/migrate.py", line 243, in handle
    post_migrate_state = executor.migrate(
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 117, in migrate
    state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 147, in _migrate_all_forwards
    state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 227, in apply_migration
    state = migration.apply(state, schema_editor)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/migration.py", line 124, in apply
    operation.database_forwards(self.app_label, schema_editor, old_state, project_state)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/operations/special.py", line 190, in database_forwards
    self.code(from_state.apps, schema_editor)
  File "/srv/newsblur/apps/rss_feeds/migrations/0002_remove_mongo_types.py", line 8, in remove_mongo_types
    collections = db.collection_names()
  File "/usr/local/lib/python3.9/site-packages/pymongo/database.py", line 910, in collection_names
    for result in self.list_collections(session=session,
  File "/usr/local/lib/python3.9/site-packages/pymongo/database.py", line 848, in list_collections
    return self.__client._retryable_read(
  File "/usr/local/lib/python3.9/site-packages/pymongo/mongo_client.py", line 1514, in _retryable_read
    server = self._select_server(
  File "/usr/local/lib/python3.9/site-packages/pymongo/mongo_client.py", line 1346, in _select_server
    server = topology.select_server(server_selector)
  File "/usr/local/lib/python3.9/site-packages/pymongo/topology.py", line 244, in select_server
    return random.choice(self.select_servers(selector,
  File "/usr/local/lib/python3.9/site-packages/pymongo/topology.py", line 202, in select_servers
    server_descriptions = self._select_servers_loop(
  File "/usr/local/lib/python3.9/site-packages/pymongo/topology.py", line 218, in _select_servers_loop
    raise ServerSelectionTimeoutError(
pymongo.errors.ServerSelectionTimeoutError: db_mongo:29019: [Errno -3] Temporary failure in name resolution, Timeout: 30s, Topology Description: <TopologyDescription id: 64d46821e81fd1eef804c8ae, topology_type: Single, servers: [<ServerDescription ('db_mongo', 29019) server_type: Unknown, rtt: None, error=AutoReconnect('db_mongo:29019: [Errno -3] Temporary failure in name resolution')>]>
Sentry is attempting to send 2 pending error messages
Waiting up to 2 seconds
Press Ctrl-C to quit

Looks like this is a duplicate of https://github.com/samuelclay/NewsBlur/issues/1792
Still verifying the fix of:
sudo chmod 777 docker/volumes/db_mongo/

confirmed duplicate of #1792


Unify container calls in controllers

Description
Please describe your pull request:

Why is it necessary?
All controllers have a property container we don't need global Shopware function
What does it improve?
Replaces all Shopware()->Container() with $this->container
Does it have side effects?
Nope

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?

How to test?

This has no actual benefit.


Add support for different separator values

Current:
Goodtables supports only CSV with comma separator (delimiter).
Expected:
Please support selection of value separator:

semicolon
comma
tab

CSV specification
https://en.wikipedia.org/wiki/Comma-separated_values#Specification
RFC 4180 proposes a specification for the CSV format, and this is the definition commonly used. However, in popular usage "CSV" is not a single, well-defined format. As a result, in practice the term "CSV" might refer to any file that:

is plain text using a character set such as ASCII, various Unicode character sets (e.g. UTF-8), EBCDIC, or Shift JIS,
consists of records (typically one record per line),
with the records divided into fields separated by delimiters (typically a single reserved character such as comma, semicolon, or tab; sometimes the delimiter may include optional spaces),
where every record has the same sequence of fields.

@schlos
Thanks for your feedback!
May be you'll be interested - https://discuss.okfn.org/t/launching-goodtables-io-tell-us-what-you-think/5165

Please use https://try.goodtables.io


nodejs: 7.2.1 -> 7.6.0

Motivation for this change
NodeJS v7.6.0 contains some new, helpful features such as native support for async/await from the ES7 specification.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

The release notes of node.js 7.6.0 also mention that they updated libuv to 1.11.0. So I think we should update this here as well.

libuv seems to be at 1.11.0 (https://github.com/NixOS/nixpkgs/blob/838e29d236f78cc171a53aaf5eea4b2bbd2a4fe8/pkgs/development/libraries/libuv/default.nix#L5)

@Ma27 You are right, I only looked for closed PRs but it seems like the change was commited directly on master, so I didn’t notice.

✅  I tested this on macOS, everyting works fine.

I tested this on macOS, everyting works fine.

cool, thank you :-)


@toast-ui/vue-editor window is not defined

how resolve it?

@farshidrezaei
You solve the problem through this issue?
If you have an another problem, you need to give us more detail information.


Lookbook Listening

I am having trouble getting the listener to catch preview changes. I regret bothering you with this here as an issue since I am sure it is something I am doing but if you have any suggestions about the best way to debug this please let me know. Thanks so much.

Het @jonsgreen - sorry to hear you are having troubles and no worries about opening the issue!
Before diving any deeper, if you are using Puma for your webserver can I ask if you've seen the troubleshooting note here? It's a pretty common cause of this issue so just want to check you've seen that first.
If that doesn't apply to you (or the suggested workaround there doesn't fix things!) just let me know and we can figure out some next steps to diagnose the issue.

@allmarkedup I had seen that and checked that my dev puma server is running in single mode. I am running my server using overmind instead of foreman; not sure if that could be a factor?

@jonsgreen ok great, always good to rule out the most likely culprits first :-)
Overmind could indeed be a factor - I have to admit I've never used it or looked at it in any detail but I'll try to run a quick test with it later today and see if I can recreate the issue.
In the meantime, would you mind sharing some debug data so I can eyeball it for anything unusual? In the top right of the Lookbook UI is a little ? icon in a circle which opens a dropdown debug menu:

Could you click the 'copy debug data' link and paste in the output here for me to look at? It shouldn't contain any sensitive information (apart from potentially the file paths) but obviously redact anything you feel you need to.
I'll let you know if I turn anything up once I've given overmind a spin.

Hey @allmarkedup, here is the debug data you asked for. Sorry about the delay:
{
  "version": "1.2.1",
  "env": "development",
  "config": {
    "project_name": "GTO Design System LookBook",
    "log_level": 2,
    "log_use_rails_logger": true,
    "auto_refresh": true,
    "components_path": "app/components",
    "page_controller": "Lookbook::PageController",
    "page_route": "pages",
    "page_paths": [
      "test/components/docs"
    ],
    "page_options": {
    },
    "markdown_options": {
      "tables": true,
      "fenced_code_blocks": true,
      "disable_indented_code_blocks": true,
      "strikethrough": true,
      "highlight": true,
      "with_toc_data": true,
      "lax_spacing": true
    },
    "sort_examples": false,
    "preview_paths": [
      "spec/components/previews",
      "spec/components/previews",
      "/redacted/spec/components/previews"
    ],
    "preview_display_options": {
    },
    "preview_disable_action_view_annotations": true,
    "preview_params_options_eval": false,
    "listen": true,
    "listen_paths": [
      "spec/components/previews",
      "spec/components/previews",
      "/redacted/spec/components/previews",
      "app/components"
    ],
    "listen_extensions": [
      "rb",
      "html.*"
    ],
    "listen_use_polling": false,
    "ui_theme": "indigo",
    "ui_theme_overrides": {
    },
    "ui_favicon": true,
    "debug_menu": true,
    "experimental_features": false,
    "preview_controller": "ViewComponentsController",
    "panels": {
      "main": [
        {
          "label": "Preview",
          "hotkey": "v",
          "disabled": false,
          "show": true,
          "copy": null,
          "locals": {
          },
          "partial": "lookbook/previews/panels/preview",
          "name": "preview",
          "group": "main"
        },
        {
          "label": "HTML",
          "hotkey": "h",
          "disabled": false,
          "show": true,
          "copy": null,
          "locals": {
          },
          "partial": "lookbook/previews/panels/output",
          "name": "output",
          "group": "main"
        }
      ],
      "drawer": [
        {
          "label": "Source",
          "hotkey": "s",
          "disabled": false,
          "show": true,
          "copy": "#<Proc:0x0000000114c81d00 (eval):1 (lambda)>",
          "locals": {
          },
          "partial": "lookbook/previews/panels/source",
          "name": "source",
          "group": "drawer"
        },
        {
          "label": "Notes",
          "hotkey": "n",
          "disabled": "#<Proc:0x0000000114c81800 (eval):1 (lambda)>",
          "show": true,
          "copy": null,
          "locals": {
          },
          "partial": "lookbook/previews/panels/notes",
          "name": "notes",
          "group": "drawer"
        },
        {
          "label": "Params",
          "hotkey": "p",
          "disabled": "#<Proc:0x0000000114c81440 (eval):1 (lambda)>",
          "show": true,
          "copy": null,
          "locals": {
          },
          "partial": "lookbook/previews/panels/params",
          "name": "params",
          "group": "drawer"
        }
      ]
    },
    "inputs": {
      "select": {
        "name": "select",
        "partial": "lookbook/previews/inputs/select",
        "opts": {
          "opts": {
          }
        }
      },
      "textarea": {
        "name": "textarea",
        "partial": "lookbook/previews/inputs/textarea",
        "opts": {
          "opts": {
          }
        }
      },
      "toggle": {
        "name": "toggle",
        "partial": "lookbook/previews/inputs/toggle",
        "opts": {
          "opts": {
          }
        }
      },
      "color": {
        "name": "color",
        "partial": "lookbook/previews/inputs/color",
        "opts": {
          "opts": {
          }
        }
      },
      "range": {
        "name": "range",
        "partial": "lookbook/previews/inputs/range",
        "opts": {
          "opts": {
          }
        }
      },
      "text": {
        "name": "text",
        "partial": "lookbook/previews/inputs/text",
        "opts": {
          "opts": {
          }
        }
      },
      "email": {
        "name": "email",
        "partial": "lookbook/previews/inputs/text",
        "opts": {
          "opts": {
          }
        }
      },
      "number": {
        "name": "number",
        "partial": "lookbook/previews/inputs/text",
        "opts": {
          "opts": {
          }
        }
      },
      "tel": {
        "name": "tel",
        "partial": "lookbook/previews/inputs/text",
        "opts": {
          "opts": {
          }
        }
      },
      "url": {
        "name": "url",
        "partial": "lookbook/previews/inputs/text",
        "opts": {
          "opts": {
          }
        }
      },
      "date": {
        "name": "date",
        "partial": "lookbook/previews/inputs/text",
        "opts": {
          "opts": {
          }
        }
      },
      "datetime_local": {
        "name": "datetime_local",
        "partial": "lookbook/previews/inputs/text",
        "opts": {
          "opts": {
          }
        }
      }
    },
    "tags": {
      "hidden": {
        "name": "hidden",
        "label": "Hidden status",
        "yard_parser": null,
        "opts": {
        }
      },
      "label": {
        "name": "label",
        "label": "Label",
        "yard_parser": null,
        "opts": {
        }
      },
      "display": {
        "name": "display",
        "label": "Display",
        "yard_parser": null,
        "opts": {
        }
      },
      "position": {
        "name": "position",
        "label": "Position",
        "yard_parser": null,
        "opts": {
        }
      },
      "id": {
        "name": "id",
        "label": "Id",
        "yard_parser": null,
        "opts": {
        }
      },
      "component": {
        "name": "component",
        "label": "Component",
        "yard_parser": null,
        "opts": {
        }
      },
      "param": {
        "name": "param",
        "label": "Param",
        "yard_parser": "with_name",
        "opts": {
        }
      }
    }
  }
}

Thanks for that @jonsgreen - however it all looks good to me, nothing obviously problematic there at all 😕
I've also just tried using Overmind to run an app with Lookbook installed and listening for changes seems to work fine, even when running multiple instances of each process. So i don't think it can be that.
One thing to try - can you set the value of the listen_use_polling config option to true and see if that makes any difference?
config.lookbook.listen_use_polling = true

Sorry this is not a great experience for you right now!

Just linking these discussions here: https://github.com/ViewComponent/lookbook/discussions/148 as they seem like they might be to do with the same issue (whatever it is!)

@allmarkedup turning on listen_use_polling did not seem to help. I think you know this already but it is just preview changes that are not getting updated; changes to the actual component are updated in Lookbook.
It does seem like the user in the other issue is having the same problem.

I had exactly the same issue but found out that even if the WEB_CONCURRENCY is set to 1 the Puma webserver (at least version 6.0.0) does run in a clustered mode. Try setting it to 0 instead, that fixed it for me.

@mdrbohlav thanks for the suggestion. Unfortunately that did not seem to fix it for me.

@jonsgreen really sorry for the slow reply, I've had a bit of a hectic week.
I hadn't actually picked up on the fact that it was only changes to the previews (not the component files themselves) that were the problem. That definitely helps narrow down the list of things that might be causing a problem here as it means that the file change listening isn't failing completely - it's just not registering changes in your previews directory for some reason.
If you add your preview directory (it looks like spec/components/previews from your debug data dump) directly to the lookbook.listen_paths config option does that have any effect?
config.lookbook.listen_paths = ["spec/components/previews"]

Is the repo you are having trouble with open-source? If so can you point me to it so I can clone it and see if I can replicate the issue within it?

@mdrbohlav Thanks for the heads up about the puma WEB_CONCURRENCY value - I'll update the troubleshooting section in the docs to reflect that 👍

@allmarkedup thanks for your persistence in trying to help me figure this out.
I already had that configuration. This is what I have in config/application.rb:
  config.view_component.generate_sidecar = true
  config.view_component.preview_paths += [Rails.root.join('spec/components/previews')]
  config.lookbook.project_name = 'GTO Design System LookBook'
  config.lookbook.preview_paths = ['spec/components/previews']
  config.view_component.default_preview_layout = 'lookbook'

I am not sure exactly what the behavior should be like but I should explain that I never get hot code reloading if I make changes to either the components or the previews. However, if I make changes to either the component code or its corresponding erb template and refresh Lookbook then it picks up what has changed. I also do get updates if I change the preview erb template and refresh the page. However, any changes that I make to the preview ruby file are not picked up when I refresh the preview in Lookbook.
Unfortunately, this is a private repository for a client so I cannot share. I suppose I could try to replicate the issue in a vanilla app though it would take some work to get that all setup and not sure I am up for it right now.

@jonsgreen no worries, happy to try and help although I'm afraid i'm running out of ideas here!
I've tried running the demo app using Overmind and with the preview directory changed to spec/components/preview and using the config settings you provided above and unfortunately i cannot replicate the issues you are seeing.
There must be some other config/local setup mismatch going on here but without anything else to go on I'm afraid I'm a bit stumped 😕
Totally understand you not having a lot of time to put into this but if you do have a chance to run the demo app locally and do let me know if you encounter the same issues.
FWIW I've just released a few updates, some of which include some refactoring of the websocket/listening code. I don't anticipate them having any effect on your problems but it's probably worth updating just in case!

@jonsgreen as I haven't heard back I'm going to go ahead and close this down now - if you are still having problems and manage to replicate it somewhere that I can take a look at it then please feel free to reopen this issue.


🐛 [Bug] Events not being dispatched when saving models

Before feedback (在反馈之前)

[X] I've searched the existing Issues, Discussions and Google (我已经搜索了现有的 Issues, Discussions 和 Google)
[X] The problem can be stably reproduced (这个问题可以被稳定复现)
[x] The problem is generated after upgrading (问题是在升级之后产生的)

Operating System (操作系统)
MacOS
Golang Version (Go 版本)
1.21.x
Goravel Version (Goravel 版本)
1.0.0
Describe The Problem (描述问题)

Create a model.
Add DispatchesEvents() method.
Create and save data.
Password does not get hashed

Reproduction Code (复现代码或截图)
package models

import (
	"fmt"

	corm "github.com/goravel/framework/contracts/database/orm"
	"github.com/goravel/framework/database/orm"
	"github.com/goravel/framework/facades"
)

type User struct {
	orm.Timestamps
	orm.SoftDeletes
	ID       string  `gorm:"primaryKey" json:"id"`
	Name     string  `json:"name"`
	Email    string  `json:"email"`
	Password string  `json:"-"`
	Username string  `json:"username"`
	Avatar   string  `json:"avatar,omitempty"`
	Posts    []*Post `json:"posts,omitempty"`
}

func (u *User) DispatchesEvents() map[corm.EventType]func(corm.Event) error {
	return map[corm.EventType]func(corm.Event) error{
		corm.EventCreating: func(e corm.Event) error {
			password, _ := facades.Hash().Make(u.Password)
			u.Password = password
			return nil
		},
		corm.EventSaving: func(e corm.Event) error {
			password, _ := facades.Hash().Make(u.Password)
			u.Password = password
			return nil
		},
	}
}

...
UserController
...
	user := &models.User{
		ID:       uuid.NewString(),
		Name:     request.Name,
		Email:    request.Email,
		Password: request.Password,
		Username: request.Username,
	}
	if err := c.q.Create(&user); err != nil {
		return ctx.Response().Json(http.StatusInternalServerError, http.Json{"error": err.Error()})
	}
...

You should call the SetAttribute  method to set the new value: https://www.goravel.dev/orm/getting-started.html#parameter-in-observer

Problem is the observer I created using artisan and also registered in the app/providers/event_service_provider.go is not being called, this is the repository https://github.com/tekpriest/goravel-blog

func (u *User) DispatchesEvents() and UserModelObserver shouldn't be used simultaneously. You can only use func (u *User) DispatchesEvents() first to test your basic logic.

that was my first implementation, same thing, so I commented it out and added observer bus same thing

Is it normal now?

this is what I have now
func (u *User) DispatchesEvents() map[corm.EventType]func(corm.Event) error {
	return map[corm.EventType]func(corm.Event) error{
		corm.EventCreating: func(e corm.Event) error {
			password, _ := facades.Hash().Make(u.Password)
			e.SetAttribute("password", password)
			return nil
		},
		corm.EventSaving: func(e corm.Event) error {
			password, _ := facades.Hash().Make(u.Password)
			e.SetAttribute("password", password)
			return nil
		},
	}
}

no, it doesn't work
On Mon, 19 Aug 2024 at 15:18, Wenbo Han @.***> wrote:

Is it normal now?
—
Reply to this email directly, view it on GitHub
https://github.com/goravel/goravel/issues/476#issuecomment-2296694682,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5ORRJ5TFK7JVNXP2W2LJ3ZSH5DXAVCNFSM6AAAAABMTEAL72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJWGY4TINRYGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

maybe I did have a wrong setup, I generated a new project and now it works