Should anucleate and nucleated be disjoint?

Could someone comment on whether the classes indicated below should be made disjoint?
If not, could you explain the reasoning?
PATO_0001405 anucleate disjoint with PATO_0002505 nucleated
PATO_0001407 mononuclate disjoint with PATO_0001908 | multinucleate
Additionally, should 'nucleated' be renamed to 'nucleate' to be in line with the other labels (e.g., anucleate, mononucleate, multinucleate, binucleate (all without a 'd' at the end))?
@dosumis, @paolaroncaglia, is there are particular person I should mention in this issue if it is suitable for additional review/follow up?

Hi @bvarner-ebi ,
Some quick thoughts:
The topic was discussed previously, please check out related issues (pro tip: in the main issue page, add "anucleate" in the filter textbox, and inspect open and closed issues; works with CL IDs too, though IDs make for stricter search parameters);
The same cell type may be nucleated and then become anucleated in time (think red blood cells), or be mononucleated and then become multinucleate (think cell division uncoupled from karyokinesis), and I'm not sure if disjointness axioms would get in the way of representing those transitions. As I said, quick thoughts; I'd suggest to review previous threads, and if doubts or concerns persists, please note them here. I'll leave this ticket open, and assign it to you to add your comments. Thank you!

(@bvarner-ebi to clarify - I'm assigning the ticket to you so you can "own" the discussion. Edits to such high-level terms should be discussed with other editors, so please don't feel like you have to solve the puzzle all by yourself :-) but it's good practice to "lead" or at least follow up on tickets we editors open ourselves, otherwise they tend to fall through the cracks. Additional or alternative assignees may be added as the discussion evolves.)

Thank you, @paolaroncaglia. Your advice and direction make sense to me.
I had not thought about the temporal aspect. However, at any specific moment in time, I don't think a cell can be nucleate and anucleate at the same time. I'll have a look at the previous threads.
For the RBC example, I do see there are separate classes for enucleate erythrocyte (should this should be anucleate erythrocyte?) and nucleate erythrocyte. An 'enucleate' erythrocyte could never be a nucleate erythrocyte at the same time, so I think the disjoint axiom would make sense, but I will investigate more.

Well you can always add it in a branch and see if the reasoning breaks. It is good to be be careful and conservative when adding such a strong axiom.

PATO should have
bearer_of some nucleated disjointWith bearer_of some anucleate
multinucleate subClassOf nucleated

That should be sufficient for the check that you want. We should check whether these axioms makes it through the (recently revised) import process
It would be good to add these details into the relations guide
We deal with cells that transition between nucleated and anucleate by making subclasses and adding the assertions there, e.g.

I've just checked.  The disjointness axioms are not in PATO, but ~equivalent axioms are already in CL:

(I may have misremembered how this is recorded.)
@bvarner-ebi  - The pattern here is like the one we used for making red and white wine disjoint in the tutorial exercises vs making the colors red and white disjoint.

Thanks for reviewing, @dosumis!
When it comes to imported terms in CL, do we have the latitude to make those changes in CL or do we need additional communication and sign-off?
I did go ahead and make the disjoints to the PATO classes and changed 'nucleated' to 'nucleate' in a branch (issue#1393-nucleateDisjoint). I did not find any errors after running the reasoner, but did notice many changes to the merged_import.owl file. Do you know why these changes appeared and how I should test what implications they would have?

Spoke with @dosumis. He explained that edits to terms in CL that come from PATO would be overwritten with the next import, so the changes, if any, should happen in PATO. If we'd like to pursue the changes mentioned in this ticket, it was suggested to add it to one of the meeting agendas to get more feedback. I deleted my branch ((issue#1393-nucleateDisjoint).
We also discussed developing a CL style guide that could help address similar questions about word choice and modelling.


Latest build is can't connect to Mikrotik or?

Hi!
I just saw it a repo moved and a new release of bouncer. Goin to upgrade my image from 19 of March, and faced with next behaviour:
That's my stack:
  crowdsec-bouncer-mikrotik:
    image: ghcr.io/tuxtof/cs-mikrotik-bouncer:latest-working
    container_name: crowdsec-bouncer-mikrotik
    networks:
      - proxy
    depends_on:
      - crowdsec
    environment:
      - TZ=Europe/Kiev
      - CROWDSEC_BOUNCER_API_KEY=${BOUNCER_MIKROTIK_API_KEY}
      - CROWDSEC_URL=http://crowdsec:8080/
      - MIKROTIK_HOST=${MIKROTIK_HOST}
      - MIKROTIK_USER=${MIKROTIK_API_ACCESS}
      - MIKROTIK_PASS=${MIKROTIK_API_ACCESS}
      - MIKROTIK_TLS=false
      - LOG_LEVEL=0
    restart: unless-stopped
    logging:
      driver: "json-file"
      options:
        max-size: "10m"

After upgrade container (to ghcr.io/funkolab/cs-mikrotik-bouncer:latest) is loop restart every time, here is from container log:
{"level":"debug","time":"2022-05-21T17:06:00+03:00","message":"Using config: map[crowdsec_bouncer_api_key:76567b69177b455af74a227ae8a3fa43 crowdsec_url:http://crowdsec:8080/ log_level:0 mikrotik_host:<IP_ADDRESS>:8728 mikrotik_pass:pass mikrotik_tls:false mikrotik_user:user]"}
{"level":"info","time":"2022-05-21T17:06:00+03:00","message":"Connecting to mikrotik"}
{"level":"info","time":"2022-05-21T17:06:05+03:00","message":"fill 14844 entry in internal addrList\n"}
{"level":"info","time":"2022-05-21T17:06:00+03:00","message":"mikrotik ip list addr"}
{"level":"info","time":"2022-05-21T17:06:05+03:00","message":"mikrotik ipv6 list addr"}
{"level":"fatal","error":"from RouterOS device: no such command prefix","time":"2022-05-21T17:06:05+03:00","message":"address-list print failed"}
{"level":"debug","time":"2022-05-21T17:06:07+03:00","message":"Using config: map[crowdsec_bouncer_api_key:76567b69177b455af74a227ae8a3fa43 crowdsec_url:http://crowdsec:8080/ log_level:0 mikrotik_host:<IP_ADDRESS>:8728 mikrotik_pass:pass mikrotik_tls:false mikrotik_user:user]"}
{"level":"info","time":"2022-05-21T17:06:07+03:00","message":"Connecting to mikrotik"}
{"level":"info","time":"2022-05-21T17:06:07+03:00","message":"mikrotik ip list addr"}
{"level":"info","time":"2022-05-21T17:06:11+03:00","message":"fill 14844 entry in internal addrList\n"}
{"level":"info","time":"2022-05-21T17:06:11+03:00","message":"mikrotik ipv6 list addr"}
{"level":"fatal","error":"from RouterOS device: no such command prefix","time":"2022-05-21T17:06:11+03:00","message":"address-list print failed"}
{"level":"debug","time":"2022-05-21T17:06:13+03:00","message":"Using config: map[crowdsec_bouncer_api_key:76567b69177b455af74a227ae8a3fa43 crowdsec_url:http://crowdsec:8080/ log_level:0 mikrotik_host:<IP_ADDRESS>:8728 mikrotik_pass:pass mikrotik_tls:false mikrotik_user:user]"}
{"level":"info","time":"2022-05-21T17:06:13+03:00","message":"Connecting to mikrotik"}
{"level":"info","time":"2022-05-21T17:06:13+03:00","message":"mikrotik ip list addr"}
{"level":"info","time":"2022-05-21T17:06:18+03:00","message":"fill 14844 entry in internal addrList\n"}
{"level":"info","time":"2022-05-21T17:06:18+03:00","message":"mikrotik ipv6 list addr"}
{"level":"fatal","error":"from RouterOS device: no such command prefix","time":"2022-05-21T17:06:18+03:00","message":"address-list print failed"}
{"level":"debug","time":"2022-05-21T17:06:20+03:00","message":"Using config: map[crowdsec_bouncer_api_key:76567b69177b455af74a227ae8a3fa43 crowdsec_url:http://crowdsec:8080/ log_level:0 mikrotik_host:<IP_ADDRESS>:8728 mikrotik_pass:pass mikrotik_tls:false mikrotik_user:user]"}

So, what this mean: {"level":"fatal","error":"from RouterOS device: no such command prefix","time":"2022-05-21T17:06:05+03:00","message":"address-list print failed"}? - my routeros is 6.49.2 on all devices.
So I'm tagged latest working image from 19 of March, and use it now. Any suggestion what's happens? Btw, the same way on my second and third mikrotiks...

Hi @SAOPP
can you connect on your Mikrotik and launch me the following command
/ipv6 export
thanks

Hi! @tuxtof
Here is on:
/ipv6 export
# may/22/2022 18:51:55 by RouterOS 6.49.2
# software id = 3UQG-BBWY
#
# model = RouterBOARD 941-2nD
# serial number = 80ECCA707F34

I don't have a v6 to test
can you give me the output of /system/package/print
is the ipv6 activated on your device ?
can you also try the two init command I use
/ip/firewall/address-list/print where list="crowdsec"
/ipv6/firewall/address-list/print where list="crowdsec"

No, ipv6 is disable in my case.

ok I knew it was coming from there :-D
just update to the last version i release this morning v0.3.0 and add the ENV variable
MIKROTIK_IPV6=false in your docker compose

Okay, I will try it now.

Alright, the issue has been solved. Thanks homie. :+1:

Cool good job, thanks for the issue


Android 13 fix not working

I used the Android 13 fix and un-installed updates for the Files app. No Files by Google app visible.
When I click import, Valkyrie now direct to the correct location but nothing is visible inside the folder to import. There's also a message about the folder being unusable due to privacy concerns.
Samsung galaxy tab s7, latest versions of Valkyrie and MoM app.

Try a manual installation of the Import folder.
Here is the Import folder from a working android copy.
https://drive.google.com/file/d/1EmUGPvhhjriQNQNMB0JLZwoo-1RlLdeQ/view?usp=drivesdk
Download the file, unzip it and replace/create the import folder in your device.
Path: \device\Valkyrie\MoM\import

This worked perfectly
On Fri, Sep 27, 2024 at 10:28 AM Vidimus @.***> wrote:

Try a manual installation of the Import folder.
Here is the Import folder from a working android copy.
https://drive.google.com/file/d/1EmUGPvhhjriQNQNMB0JLZwoo-1RlLdeQ/view?usp=drivesdk
Download the file, unzip it and replace/create the import folder in your
device.
Path: \device\Valkyrie\MoM\import
—
Reply to this email directly, view it on GitHub
https://github.com/NPBruce/valkyrie/issues/1685#issuecomment-2379748876,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AX7UTPPGVSQKSAHCN5376ZLZYWIUJAVCNFSM6AAAAABOW2O4NCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZZG42DQOBXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fix column index to honor the skip attribute again

Fix time and space overhead of types with skip attribute.

Would you mind adding a unit test with a simple type that has the &skip attribute?

Would you mind adding a unit test

Not at all. Done.


How to integrate interactive volume objects on Webpage?

I'm looking to render the RayCastPlotter for one of my use cases and have it available as an interactive display on a webpage. Currently, I'm trying to export a sample volume using the plt.export() method,it seems that Volume objects can't be rendered on a webpage via this method.
Tried code
from vedo import Volume, dataurl
from vedo.applications import RayCastPlotter

# Load Volume data
embryo = Volume(dataurl + "embryo.slc")
embryo.mode(1).cmap("jet")  # change visual properties

# Create a Plotter instance and show
plt = RayCastPlotter(embryo, bg='black', bg2='blackboard', axes=7)
plt.show(viewup="z")
plt.export('raycastplot.x3d', binary=False)

Could you provide guidance for rendering RayCastPlotter/ any volume object with interactive sliders on a webpage? Are there any workarounds that would enable this functionality?

Hi, no that unfortunately is not possible. The upstream VTK relies on a openGL rendering window which is not supported in the browser. The x3d format only supports polygonal mesh objects.
Check out examples/notebooks/ for alternative backends.


added OSX Yosemite Terminal.app conf

Terminal.app in recent versions of OSXs (Mavericks and Yosemite) has decent color support. No need of workaround.

How about solarized light theme?


How to Enable New Collectors

I have installed wmi exporter and can't seem to enable new collectors. There's very little documentation and I can't seem to get a good answer scrolling through here or searching online. In the exporter.go file I updated defaultCollectors to pull logon and memory metrics.
const (
defaultCollectors            = "cpu,cs,logical_disk,net,os,service,system,textfile,logon,memory"
defaultCollectorsPlaceholder = "[defaults]"
serviceName                  = "wmi_exporter"
When I run the .exe those metrics are not shown under my enabled collectors. What are the actual steps to enable these non-default collectors?

If you're running the exe you can use the --collectors.enabled flag. From the project README:
.\wmi_exporter.exe --collectors.enabled "service" --collector.service.services-where "Name='wmi_exporter'"

How do I do that? Through something like powershell or the cmd line? I see in the documentation some stuff regarding the misexec, but I can't utilize that as my company won't allow the use of the msi file due to firewall security.
Also, I do have all the files from the distro on my PC, and I can set it up to run as a service instead of from the .exe. If I run it as a service, will it pull the configuration settings from the exporter.go file?

Yes, you can run the commands in Powershell or CMD. You'll need to navigate to the directory containing the exe file before running them or run the command with the full path to the executable. E.G. C:\Users\captainhistory\Downloads\wmi_exporter-0.8.3-amd64.exe --collectors.enabled=cpu,memory,net,os
You don't need all the project files on your PC, only the executable. Whether run manually or as a service, the executable will run with the default configuration as specified in exporter.go. These settings can be changed with run-time flags, such as --collectors.enabled and  --telemetry.addr.

@captainhistory please see https://github.com/martinlindhe/wmi_exporter#installation
using ENABLED_COLLECTORS you can customize what collectors to enable during installation.
How to specify enabled collectors is documented under https://github.com/martinlindhe/wmi_exporter#examples

You can install via powershell using msiexec
msiexec /i C:\path\to\msi\wmi_exporter-x.x.x-amd64.msi ENABLED_COLLECTORS="cpu, cs, logical_disk, memory, net, os, ect...." LISTEN_PORT="9182"

I have installed wmi exporter using the command:
msiexec /i wmi_exporter-0.9.0-amd64.msi ENABLED_COLLECTORS="ad,cpu,cs,dns,iis,logical_disk,logon,memory,net,os,service,system,tcp,textfile"
and the wmi_exporter is run as windows service,
The service start command is below
 "C:\Program Files\wmi_exporter\wmi_exporter.exe" --log.format logger:eventlog?name=wmi_exporter --collectors.enabled ad,cpu,cs,dns,iis,logical_disk,logon,memory,net,os,service,system,tcp,textfile --telemetry.addr :9182
Now I want to change the start command to
 "C:\Program Files\wmi_exporter\wmi_exporter.exe" --log.format logger:eventlog?name=wmi_exporter --collectors.enabled cpu,cs,logical_disk,logon,memory,net,os,service,system,tcp,textfile --telemetry.addr :9182.

https://github.com/martinlindhe/wmi_exporter/issues/440

I understand this as your problem being solved in #440? I'll close this, then.


BiConvex and Meniscus lens construction fails for large centre_thickness

While putting together tests for lens primitives I discovered another issue. The construction for large center_thickness parameter can produce lenses of wrong shape or even no shape at all, see in the picture. The left column are menisus lenses, the right contains biconvex lenses. The centre_thicknes parameter is increasing from top to bottom. The increase causes the meniscus lense to have wrong position and shape of a biconvex lens. The biconvex lens has wrong position and decreasing diameter up to a point when it "dissapears".

The lenses were generated using this code:
BiConvex(diameter, 0.015, 0.02, 0.02, parent=world, transform=translate(1.7 * diameter, 0, 0), material=schott("N-BK7"))
BiConvex(diameter, 0.03, 0.02, 0.02, parent=world, transform=translate(0.6 * diameter, 0, 0), material=schott("N-BK7"))
BiConvex(diameter, 0.075, 0.02, 0.02, parent=world, transform=translate(-0.6 * diameter, 0, 0), material=schott("N-BK7"))
BiConvex(diameter, 0.1, 0.02, 0.02, parent=world, transform=translate(-1.7 * diameter, 0, 0), material=schott("N-BK7"))

Meniscus(diameter, 0.015, 0.02, 0.02, parent=world, transform=translate(1.7 * diameter, 0, -0.1), material=schott("N-BK7"))
Meniscus(diameter, 0.03, 0.02, 0.02, parent=world, transform=translate(0.6 * diameter, 0, -0.1), material=schott("N-BK7"))
Meniscus(diameter, 0.075, 0.02, 0.02, parent=world, transform=translate(-0.6 * diameter, 0, -0.1), material=schott("N-BK7"))
Meniscus(diameter, 0.1, 0.02, 0.02, parent=world, transform=translate(-1.7 * diameter, 0, -0.1), material=schott("N-BK7"))

This is in my opinion caused by way lens objects are constructed with CSG. In both cases (BoConvex, Meniscus) the procedure is first intersection of the front and back spheres and then intersection of the result with the barrel. This fails when the distance of the sphere centers gets too large due to the center_thickness parameter.
I know these examples are not probably relevant for any realistic lenses, but I think this behaviour should not be allowed. I see two solutions:

add upper limit to the center_thickness parameters
change the lens construction procedure from the primitive shapes

I can combine a fix for this issue into a PR with tests I've been working on. I would personaly prefer the second option.

Interesting. Yes you are right the build order was never intended to cope with hugely thick lenses.
I think we may need to intersect the front and back spheres with a cylinder to cut off the spherical caps. Then union these with the barrel. I'd be very worried about numerical accuracy issues though.
This will take some thought.

The issue with convex surfaces could be solved by intersecting surface spheres with a cylinder, which is extending beyond the lens body. This could work and is as simple as what is used now.

I probably should stop posting comments late in the evening... You are right that this needs a thought, the poposed pocedure was also not handling thick lens correctly also. The best idea I have now is adding a second barrel stretching between the curvature centers to cover the center thickness , but how would that behave when the curvature and lens diameters were the same?

After some time I think introducing an upper threshold to center thickness could be a good option. It could either stop the lens init completely and we could add extra classes for thick spherical lenses or it could be used to switch to construction suitable for thick lens. This way the current simple and efficient construction would be kept and probably used in most of the applications while keeping the thick lens option.

I've implemented different construction for thin and thick lens cases into Planoconvex (1,2 from top), BiConvex (3, 4 from top) and Meniscus (5, 6 top) lenses. It still has to be checked, but looks promissing. The threshold for the thin (current construction) to be used is the maximum venter thickness it stilll works for. I have to tests still, but looks promissing by eye-metrics.

If you have a PR, I'll take a look.

Just curious, is it ever practical to have such thick lenses? I would have thought such thick lenses have very little physical relevance even if it is a legitimate bug.

I would be worried about the fact that biconvex, planoconvex and menisucs lenses can behave in very unpredictive way.
I made a draft PR  #355  to show what I've done so far. The only lens not finished is the meniscus, which has some problems close to the edges of the back surface. I will change the structure of the unittest class to introduce more versatility to the methods for surface testing, which might solve the issue.
The problem is that even though the ray is shot towards the edge of the lense, it misses. I have to investigate if it is an error in the construction or problem of the ray or just numerics.

I think I am done from my part. The new way of lens construction and tests are in PR #355.


Cart Total Calculation Is not Perfect

Expected behavior:
Total calculation show perfectly in cart section
Actual behavior:
Please saw total calculation is not perfect. (Check-in Screenshot)
Version:
V3.0 Beta 8

Issue Solved: 1 Changes on this file
vendor/tastyigniter/flame/src/Cart/Helpers/CartConditionHelper.php
Line Number 40
Find - $action['cleanValue'] = (float)$this->cleanValue($actionValue, '-');
Replace to this - $action['cleanValue'] = (float)$this->cleanValue($value, '-');

Great spot!


Intermittent SIGSEGV in Aws::S3::S3Client::PutBucketAcl()

Describe the bug
I am intermittently seeing segmentation fault while calling Aws::S3::S3Client::PutBucketAcl().
The issue was consistent/very frequent when using slightly older release [aws-sdk-cpp-1.9.319].
The issue has become intermittent since I moved to release [aws-sdk-cpp-1.9.328].
Here's the stack trace (trimmed few initial calls as it's part of proprietary code) -
210660 *** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1661395837" if you are using GNU date ***
210661 PC: @     0x7ffb1da49aa0 (unknown)
210662 *** SIGSEGV (@0x0) received by PID 31 (TID 0x7ffb1b821280) from PID 0; stack trace: ***
210663     @     0x7ffb1db383fd google::(anonymous namespace)::FailureSignalHandler()
210664     @     0x7ffb27ef0240 (unknown)
210665     @     0x7ffb1da49aa0 (unknown)
210666     @     0x7ffb1da4ae8a std::istream::tellg()
210667     @          0x1d766e4 Aws::Utils::Crypto::MD5OpenSSLImpl::Calculate()
210668     @          0x1d5f1f5 Aws::Utils::Crypto::MD5::Calculate()
210669     @          0x1d59fde Aws::Utils::HashingUtils::CalculateMD5()
210670     @          0x1d87ee4 Aws::Client::AWSClient::AddChecksumToRequest()
210671     @          0x1d8812c Aws::Client::AWSClient::BuildHttpRequest()
210672     @          0x1d9295f Aws::Client::AWSClient::AttemptOneRequest()
210673     @          0x1d93649 Aws::Client::AWSClient::AttemptExhaustively()
210674     @          0x1d94fd7 Aws::Client::AWSXMLClient::MakeRequest()
210675     @          0x1ce0120 Aws::S3::S3Client::PutBucketAcl()

Expected Behavior
The API should complete consistently, without any crash/faults.
Current Behavior
The SDK is crashing with segmentation fault.
Reproduction Steps
Calling the same API PutBucketAcl() may reproduce the issue.
I am seeing the issue with two independent binaries.
Possible Solution
No response
Additional Information/Context
No response
AWS CPP SDK version used
1.9.328
Compiler and Version used
7.4.0
Operating System and version
Linux u18-101104-01 5.4.0-80-generic #90~18.04.1-Ubuntu SMP Tue Jul 13 19:40:02 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Can you please include a minimal code snippet that reproduces this seg fault? and include sanitized logs from the sdk?


Visualize cAdvisor data via prometheus in Grafana

We are interested in metrics such as the peak memory usage, CPU efficiency, ..., of ChRIS plugins.
A feature proposed to pman might make this easier: https://github.com/FNNDSC/pman/issues/215

@jennydaman I checked the branch & looks like you have already installed the prometheus & cAdvisor image, and have configured the scrape_config. What is the requirement for this? Do we need dashbaord visualisation of them or is the configuration not working correctly?

@sakshi1215 I have configured them to run in docker-compose but I have not gotten around to writing any dashboard visualizations for them. I would be interested in being able to visualize things like

how much memory is being used on average by ChRIS plugins?
which ChRIS plugins are most memory-hungry?
what is the CPU efficiency of the ChRIS plugin pl-fshack?
...

I believe these can be accomplished using cAdvisor and prometheus. The task here is to develop dashboard visualizations which can answer these kinds of questions.
It might be easier to solve this if https://github.com/FNNDSC/pman/issues/215 is first satisfied.


Incorporate "track_until_converged" callback functionality into Tracker class

There have been multiple applications where there is a need for the Tracker class to run until it has converged on a target. This has been implemented in multiple places as a callback, but the callback mechanism can be a bit awkward. Look into incorporating this functionality directly into the Tracker class.

Meh. This feature is really only required for automation of alignment. If it fails when I get around to completing that work I can open a new issue to fix it.


SCPI_ExprChannelListEntry() crashes with whitespaces

If you enter a whitespace in your channel list, e. g. (@1, 2) instead of (@1,2), result of SCPI_ExprChannelListEntry == SCPI_EXPR_ERROR and the parser get's somehow messed up, but it doesn't throw an error.

Strictly speaking, spaces are not allowed in channel list.
If result of SCPI_ExprChannelListEntry is SCPI_EXPR_ERROR, then you can't  read other values returned by this function, because they are some random data.
Bug is, that it just return SCPI_EXPR_ERROR and does not set SCPI error.
After solving this, we can extend channel list parsing not to be so strict and allow some whitespaces.


Replicated container folder python django for containerWithTests

@bishal-pdMSFT, the realization hit me a little later that if we replicated the folder first as is with no changes and then sent an incremental PR for the changed files, reviewing will be much simpler.
This is the first PR where I've asked Vagisha to just replicate the folder with no changes whatsoever

@ShreyasRmsft if this is same as existing folder, I will go ahead and merge it.

@vagisha-nidhi please confirm whether @bishal-pdMSFT can go ahead and merge this. Double check if the files have been duplicated as is and no new files have been added

@vagisha-nidhi please update the other PR as well with latest mater


[HOLD for payment 2023-03-21] Workspace - App is crashed when clicking on View all receipts and other option while Offline

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to URL https://staging.new.expensify.com/
Log in with a HIGH TRAFFIC account.
After login in with a High Traffic account - enable Airplane mode to simulate a connection loss
Go to the following options and click on them
4.1 Reimburse expenses/ View all receipts
4.2 Pay bill/ View all bills
4.3 Send Invoices/ Send Invoice and View all invoices

Expected Result:
App is not crashed
Actual Result:
App is crashed when clicking on View all receipts and other option while Offline
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/224817992-2d8faba2-f3cd-4be9-8a11-24a4a3a68192.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Triggered auto assignment to @thienlnam (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@thienlnam gonna swoop in here to help out since this blocks the deploy of my PR for ECX stuff

caused by lack of offline testing from PR https://github.com/Expensify/App/pull/11614, fix incoming...

https://github.com/Expensify/App/pull/15923

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.2.83-3 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/15923

If no regressions arise, payment will be issued on 2023-03-21. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

Removing the blocker label as the fix has been deployed and I have verified the fix

Closing as there is no reporter to pay

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.2.84-1 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/15923

If no regressions arise, payment will be issued on 2023-03-22. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty


String comparison test fix

Trying out the fix with one test.

Fixing this via https://github.com/Microsoft/vstest/pull/1462


python3Packages.pyupgrade: 2.12.0 -> 2.16.0

Motivation for this change
Update to latest upstream release 2.16.0
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

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

2 packages built:

pyupgrade (python38Packages.pyupgrade)
python39Packages.pyupgrade

Result of nixpkgs-review pr 123236 at 25b83c01 run on aarch64-linux 1

2 packages built successfully:

pyupgrade (python38Packages.pyupgrade)
python39Packages.pyupgrade


Unable to build SWF file.

Hello! I've installed NodeJS(4.4.2)+grunt-cli(1.2.0)+grunt-air-sdk
When I try to "grunt build" I'm getting such output:
grunt build
Running "air_sdk:vpaidjs_swf" (air_sdk) task
/bin/sh: 1: -source-path: not found
/bin/sh: 1: -source-path: not found
Done, without errors.
After this, no swf file under dist directory.

Please close an issue, problem solved!
I've tried to install under root, so air_sdk was not installed.
Consider to run "npm install --unsafe-perm" if you're under root.

Noted. I'll be sure to look into it. New release coming real soon.


Gradle 5.0  error NpmPackageManager$injectDependencies$1$1 does not define or inherit a method

This plugin fails hard under Gradle 5.0

What went wrong:
Receiver class org.jetbrains.kotlin.gradle.frontend.npm.NpmPackageManager$injectDependencies$1$1 does not define or inherit an implementation of the resolved method getBuildDependencies()Lorg/gradle/api/tasks/TaskDependency; of interface org.gradle.api.file.FileCollection.

#122

Fixed in master. Will be released soon.


What  limb_path directory contains?

I am trying to run the code with FAUST dataset but unable to know what should be the input path to limb_path?

Hi, actually you don't need the limb_path, we predefined some parts/limbs of the body in the beginning to check the efficiency of regional GIH. Later in our paper, it is done by adaptively sampling the body parts.
Thanks for being interested in our work, I will organize the code and remove the redundant parts like this before the ICCV conference. Sorry, I'm quite busy these days, I cannot find time to clean it up.


Implement PlaneType with an Enum

Introduces a dependency on the Python 3.4+ enum module, but the enum34 backport is there as a dependency for older versions.

Before I merge this, I would just like to hear from @cslocum confirmation that it's OK to add a dependency to the enum34 package on py27, and have that added in as part of Ureka as well. I figure this is almost certainly acceptable given the inclusion of other packages like six and unittest2 that are back ports and compatibility layers for 2.7 vs 3.4. Agreed?

cc @jhunkeler since it sounds like you're currently working on packaging

Paging @cslocum and @jhunkeler . It's been a week, if I don't hear any objections from either of you within the next day or two to adding the dependency on enum34 to poppy , I'm going to go ahead and merge this.

@josePhoenix pinged me via email. These github messages were going to my spam folder for some reason. Anyway, yes I'll add enum24 to ssbdev tonight.


Poppable: Placement of {top:0, left:0} does not work

Prerequisites

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed

Demo Page
https://codepen.io/dietapete/pen/OJxGgVQ
Explanation

What is the expected behavior?
Poppable should show at 0,0

What is happening instead?
Poppable is not showing at all (placement-hidden class is added)

Hi @DieTapete
There are a few issues with your example:

You're not passing a reference to <Poppable/>, which is why you're modal is hidden. You should pass it like so:

const App = () => {
  const ref = useRef();
  return (
    <div>
      <button ref={ref}/>
      <Poppable reference={ref}/>
    </div>
  );
}

You're getPlacements function returns [{top: 0, left: 0}]. You should calculated those values based on the rbr (reference bounding rect, i.e. the button) and tbr (target bounding rect, i.e your modal)

I see now that you wanted it to appear at [0,0], and indeed it considers it to be hidden in that case.
@yairEO we use [0,0] as a hidden placement, see https://github.com/open-amdocs/webrix/blob/master/src/components/Poppable/Poppable.constants.js

thanks for taking care of this. i'm using [1,0] for the time being ;)

@ykadosh - what is a "hidden placement" purpose? why 0, 0 is so?
I'm unsure how to address this bug because I don't know the reason behind this logic:
export const HIDDEN_PLACEMENT = {top: 0, left: 0, name: 'hidden'};

@yairEO I don't recall all the original reasoning, but the HIDDEN_PLACEMENT is used both for hiding the <Poppable/> (initially or when the target is missing), and as a communication device between the different "recovery strategies".
So there are a few recovery strategies, i.e. trap(), reposition() and hide(), and we have some logic to use them based on their output (see: https://github.com/open-amdocs/webrix/blob/master/src/components/Poppable/strategies/index.js#L33)
Since their output is an object with top and left, we used a special object to indicate that.
We need to find a better way to communicate between those functions, but keep in mind that the final output must be a positioning object (with top and left)

Maybe it's not the "right" solution but would it be enough to set the hidden placement to something like Infinity?

The placement object must be something that can be used in CSS to position the popup, so Infinity cannot work unfortunately.


IoT-Agent Leshan allows unregistered devices to connect to dojot

Only devices registered to dojot should be able to connect to the platform, but in the current implementation any LwM2M device can connect to.

I'm not sure if Leshan allows us to reject a registration request. Actually, we use the RegistrationListener that doesn't give us any power to decide the registration response. Despite it, we don't put unregistered device's data into the platform (Kafka), to this the device data go to the message broker is necessary someone registers it on device manager, so it becomes something like 2-steps registration.
At my point of view it isn't a bug, but an architecture decision.
@rascaraficci, what do you think about it?

Ticket analyis (Dec, 27th): Rafael will analyse this consideration.


Generated Proxies Should Create Constructors Accepting Strings/Uris

Hello,
I just noticed that generated clients depend on a hard-coded string that also gets generated.  This does not account for environment-specific use cases where the URL of the service will change based on the environment.  The generated proxies should create constructors that accept both strings and URIs so that the base address can be assigned via configuration based on the target environment.  I will definitely be providing a PR for this. :)
Thank you,
Michael

That's a great idea!
In the meanwhile, here is a workaround for the C# client. You have access to the normal ASP.NET WebApi Client, just simply set the BaseAddress here:
using (var client = new PeopleClient())
{
   client.HttpClient.BaseAddress = new Uri("http://dev.some-other-url.com");
   var response = await client.GetAsync();
   response.EnsureSuccessStatusCode();
   var content = await response.Content.ReadAsAsync<object>();
}

This should be quite easy to add, any chance of a pull request?

Merged with #49


Fix tool dist-info directory normalization

.dist-info directories use a different normalization.
Fixes #4685

Thanks!


[TEST ONLY][DO NOT MERGE] Test arrow-collect after avoiding hang

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

``


Warnings running example_data set + what output files to expect? + how to launch uBin?

I've been trying to get off the ground with uBin (again) and have a few questions, that you can hopefully help me with:
When I try running uBin_wrapper.sh on the example_dataset (see output below), I get some warnings that I don't understand?
I've got diamond version 2.0.14 installed. Which one should I use?
Also I'm not sure which output files I should get? I've listed the ones I've got below, are there any missing?
And finally, how do I actually launch uBin after it has been installed?
Thanks!
Command:
bash uBin-helperscripts/bin/uBin_wrapper.sh -s uBin-helperscripts/example_dataset/scaffolds.fasta -p pdbg -r1 uBin-helperscripts/example_dataset/BG_1_S1_L001_trim_clean.PE.1.fastq.gz -r2 uBin-helperscripts/example_dataset/BG_1_S1_L001_trim_clean.PE.2.fastq.gz -e diamond -t 10 -b uBin-helperscripts/example_dataset/das_tool_DASTool_scaffolds2bin.txt
Output on screen:
all files will get generated from scratch
creating mapping index
Building a SMALL index
mapping...
calculating coverage...
calculating GC content...
parsing length of each scaffold...
cleaning up...
...done!
diamond v<IP_ADDRESS> | by Benjamin Buchfink<EMAIL_ADDRESS>Licensed under the GNU AGPL https://www.gnu.org/licenses/agpl.txt
Check http://github.com/bbuchfink/diamond for updates.
#CPU threads: 10
Scoring parameters: (Matrix=BLOSUM62 Lambda=0.267 K=0.041 Penalties=11/1)
#Target sequences to report alignments for: 1
Temporary directory:
Opening the database...  [0.006998s]
Error: Database was built with a different version of diamond as is incompatible.
/home/sara/uBin-helperscripts/bin/07_01classifier.rb:44:in initialize': No such file or directory @ rb_sysopen - pdbg_min1000.genes.faa-vs-FunTaxDB.b6 (Errno::ENOENT)         from /home/sara/uBin-helperscripts/bin/07_01classifier.rb:44:in open'
from /home/sara/uBin-helperscripts/bin/07_01classifier.rb:44:in `'
pdbg_min1000.genes.faa.bacteria.scg pdbg_min1000.genes.faa.archaea.scg
running blast for bacterial SCGs... this may take a while...
/home/sara/uBin-helperscripts/bin/08_02scg_metagenome.rb:69: warning: Insecure world writable dir /mnt/c in PATH, mode 040777
/home/sara/uBin-helperscripts/bin/src/scg_blank_diamond.rb:39: warning: Insecure world writable dir /mnt/c in PATH, mode 040777
/home/sara/uBin-helperscripts/bin/src/scg_blank_diamond.rb:39: warning: Insecure world writable dir /mnt/c in PATH, mode 040777
SCG prediction worked
Output files generated:
pdbg_min1000.genes                   pdbg_min1000.genes.faa.findSCG.b6
pdbg_min1000.fasta
pdbg_min1000.genes.faa               pdbg_min1000.genes.faa.scg.candidates.faa
pdbg_min1000.fasta.overview.txt         pdbg_min1000.genes.faa.all.b6
pdbg_min1000.genes.fna
pdbg_min1000.fasta.overview_bincol.txt  pdbg_min1000.genes.faa.archaea.scg
pdbg_min1000_SCGS.csv
pdbg_min1000.fasta.scaff2tax.txt        pdbg_min1000.genes.faa.bacteria.scg

Hi again,
I'm running it in the windows linux subsystem. You can't open GUIs (easily) in WSL1, but with wsl2 I can launch firefox so that it opens a window, so it should be ok from that side. When I installed uBin using the .deb package I found no way of calling it from the CLI so to speak other than to see that it was installed. I'll try again with WSL2 once I've got it to install everything for the helperscript side of things.
I might have to try installing uBin itself in my Ubuntu in Virtual Box and move the files from the helperscript over as you suggested. I just don't have enough memory or capacity to install and run the helperscript on that, or this would have been a lot easier. :)

Hi Dwrgi,
Sorry for the late reply, the mail notification ended up in my spam folder and i just now checked it through... . If i install either the .exe (windows) or .dmg (mac), i then find it in the programs list like any other type of program and can start it.
I just now tested opening the installed Mac app (thats the OS i have locally) via
open /Applications/uBin.app (Mac always puts installed programs into /Applications/{Appname}.app) and the GUI opened as usual. I usually open it via the programs listing and hence hadnt actually tried opening it via the CLI but at least on Mac it works. Maybe you can check where the package is installed and then use the open command with it? If you do find the program and it doesnt want to open, on Mac at least the usual cause is that it blocks any downloaded and then installed apps from executing (so you'd need to go to Safety & Security and there will then be a notice where you can basically click 'open anyway'). I could imagine something similar to happen on Linux (no idea if the surrounding windows system could actually basically enforce something like that for the WSL - in that case, other GUI-based apps should have the same problem though; maybe pretty standard applications like firefox are exempt from that since it has to do with certificates and they probably have them very much up to date).
Sorry that i cannot be of too much help. I've not used any WSL's so far and thus have limited ideas on what could be going wrong there.
Best,
Till


Update documentation for v0.2.1

Updates the documentation and README to include codecov.

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

Comparison is base (a2041a8) 100.00% compared to head (678f7ca) 100.00%.

:exclamation: Current head 678f7ca differs from pull request most recent head a2f48b6. Consider uploading reports for the commit a2f48b6 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main        #5   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines          174       174
  Branches       174       174
=========================================
  Hits           174       174

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


Add load listener

Hey guys how can i add eventlistener in stimulus way something like this
  var div = document.getElementsByTagName("div")[0];         // does not work (console is not written to when page loads)...         div.addEventListener("load", function(){console.log("div load event")}, false);

Also i think documentation is not well written in my opinion, some of custom action listener not being fully listed i have to go thorugh the codebase to find out what are the available option. it is frustrating for a user.

Hi @teekenl  - can you elaborate on what you are trying to do?  Perhaps more code.   Are you doing this in the controller connect method (which runs when the dom is connected to the stimulus controller) ?

I was trying to achieve to trigger controller action automatically when element is loaded in html page. To achieve in this raw javascript, i can achieve it to add load eventListener to the element itself which will invoke the action when it is loaded. But in the stimulus, i was trying to use data-action="load@document->MY_ACTION to achieve the same as it said in the documentatio but somehow the action is not triggered.

@robzolkos

Have you tried connect function? This gets called when the dom element connects to stimulus.

@robzolkos connect might not be suitable for my case/scenario, because i have a few places that was using controller which i need them to be triggered when they are loaded in html. putting them connect does not fit to my problem here, because it will try to re-render again and again the action for the others too at the same time

To achieve in this raw javascript, i can achieve it to add load eventListener to the element itself

There is no load event for non-media elements (like <div>s). Are you referring to the window load event?

putting them connect does not fit to my problem here, because it will try to re-render again and again the action for the others too at the same time

Stimulus doesn't do any rendering of its own so I'm not sure what you're referring to here.

I was trying to achieve to trigger controller action automatically when element is loaded in html page

Stimulus provides two callbacks for that, initialize() and connect(), documented here: https://stimulusjs.org/reference/lifecycle-callbacks

The best place for questions like this is our community forum: https://discourse.stimulusjs.org. Feel free to continue the discussion there.


An emergency

Hi luo,
i deployed your code on a new dataset.  It is bigger than COCO.  With the XE loss, the results is ok. But, when i tried SCST after 20 epoches, the negative rewards did't seem to go down too much. In addition, i  tried to use a learning parameter to balance XE loss and SCST loss( β*(SCST_loss)+(1-β)*(XE_loss)), however, it also did't work. Can you give me some advice. And, I need to correct your topdown work, the input of the att_lstm is not concat(xt,fc_feats,h2) , the fc_feats shoud be replaced by the mean of the pool5 features(resnet,  bottom up....).

try 30 epochs.


NBSP Showing as an icon instead of space

in Syngas conversion there is an empty tag

NVM it was my thousands seperator being a space


Add Dockerfiles for Windows Insider preview

Remaining work:

[ ] The readme is from the insider program.  I think the majority of the official readme should be re-used here.

@dleeapho, I hear your concern.  I don't think it is going to be an issue however.  If proven wrong, we can react as appropriate.


bytesFromNibbles error

When I try to do cargo run (but not when I do cargo build --release, I get the following error:
    /usr/bin/ld: /home/faassen/install/llama-rs/ggml-raw/ggml/ggml.c:1418: undefined reference to `bytesFromNibbles'

this is pretty mysterious as I see bytesFromNibbles is actually being defined in ggml.c, though it does use avx stuff, so perhaps that's a problem. Weirdly enough I've successfully run the c++ version on this same laptop (Ryzen 6850u), so this library does seem to compile.

Hi @faassen! Thanks, I'm aware of the issue. You have to specify the --release profile as well for cargo run. That is, cargo run --release. It just doesn't build in debug mode for some reason.

I was using cargo run --release. If I just type that by itself, it'll crash with the reported error.
Hey, if I cd into llama-rs, the sub-crate, then I don't get a crash, but that's not per instructions.
But if I then cd back to the project directory and run cargo run --release the error has gone away!

Took me a little bit of time to figure it out. These functions
https://github.com/setzer22/llama-rs/blob/main/ggml-raw/ggml/ggml.c#L367-L397
use inline instead of static inline, which means the compiler is free to assume that there exist other definitions and thus not actually inline them. The linker then attempts to find definitions of the functions, but can't because they were never emitted to begin with. This only occurs in Rust debug builds as cc passes down the debug profile, which the original Makefile doesn't do.
It can be fixed in this repo's copy of ggml, or an upstream fix can be made (to llama.cpp or to ggml?). I suggest opening a PR with upstream and then vendoring it as a submodule.

Wow, nice catch! @philpax :smile:  I wouldn't even know where to begin with this one.

It can be fixed in this repo's copy of ggml, or an upstream fix can be made (to llama.cpp or to ggml?). I suggest opening a PR with upstream and then vendoring it as a submodule.

The C way of doing things makes this more complicated than it should. :grimacing: ggml is a standalone library, but the author also copied the .c / .h files into the llama.cpp source tree, so I'm not sure which repo we should add a submodule for, or where to report issues to. My go-to place would be the ggml repo itself, not llama.cpp.

The C way of doing things makes this more complicated than it should. 😬 ggml is a standalone library, but the author also copied the .c / .h files into the llama.cpp source tree, so I'm not sure which repo we should add a submodule for, or where to report issues to. My go-to place would be the ggml repo itself, not llama.cpp.

Yeah, I have no idea. There are patches landing for the llama.cpp version of ggml, but not ggml itself, which is why I'm thinking that we should track llama.cpp.

But anyway, I think adding it as a patch here on our end first would help mitigate the issue as fast as possible, so it's a good thing to do regardless (also to verify the fix actually works).

Seems reasonable. I'll open a PR here and there at some point.

Heads up, upstream just fixed this in the llama.cpp version: https://github.com/ggerganov/llama.cpp/commit/113e685d18ac4edb20f647fd34b000941556f6a6
I'm now leaning more strongly towards submodule-vendoring in llama.cpp for as long as we have a dependency on ggml.

Solved in #32


audio out headphone jack

I am running Boombeastic on balenaOS 2.29.2+rev2 and superuser version 9.0.1 with a Raspberry Pi 3 and I want to connect the headphone jack directly to my speaker is there any way of doing this?

I want to do the same thing. Is this not possible? @ghost did you have any success?

hey @ghost @Langholt yes it can be done! you just need to avoid disabling audio (dtparam) and loading the DAC overlay (dtoverlay) during configuration

@curcuz Awesome, Thanks so much. I'll try this.

I managed this by:
1.
2. disable the copying of the asound.conf ( ae9962dfab2cb0b211576224f710a38209edbe14)
3. don't remove the snd_bcm2835 module in mopidy start.sh (bdd3984d2e20f4b75a111bcf3afe70bc9377fd9f )

@janvda Thanks for the tip here, and sorry for the slow reply just testing this now. I'm still not successful. When I run the speaker-test in the MOPIDY command the output that I get is:
`root@171317c:/usr/src/app# speaker-test -c2 -t wav
speaker-test 1.1.3
Playback device is default
Stream parameters are 48000Hz, S16_LE, 2 channels
WAV file(s)
ALSA lib confmisc.c:767:(parse_card) cannot find card '0'
ALSA lib conf.c:4528:(_snd_config_evaluate) function snd_func_card_driver returned error: No such file or directory
ALSA lib confmisc.c:392:(snd_func_concat) error evaluating strings
ALSA lib conf.c:4528:(_snd_config_evaluate) function snd_func_concat returned error: No such file or directory
ALSA lib confmisc.c:1246:(snd_func_refer) error evaluating name
ALSA lib conf.c:4528:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:5007:(snd_config_expand) Evaluate error: No such file or directory
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM default
Playback open error: -2,No such file or directory`

I think the above error indicates that the audio hardware of your raspberry pi is not accessible within your mopidy container.
Did you set privileged: true for the mopidy service in the docker-compose.yml (see example here) ?
Normally if you run the command aplay -l in a terminal window of the mopidy service you should see get something like:
root@f0bc010:/# aplay -l
**** List of PLAYBACK Hardware Devices ****
card 0: ALSA [bcm2835 ALSA], device 0: bcm2835 ALSA [bcm2835 ALSA]
  Subdevices: 7/7
  Subdevice #0: subdevice #0
  Subdevice #1: subdevice #1
  Subdevice #2: subdevice #2
  Subdevice #3: subdevice #3
  Subdevice #4: subdevice #4
  Subdevice #5: subdevice #5
  Subdevice #6: subdevice #6
card 0: ALSA [bcm2835 ALSA], device 1: bcm2835 ALSA [bcm2835 IEC958/HDMI]
  Subdevices: 1/1
  Subdevice #0: subdevice #0
root@f0bc010:/#

Thanks @janvda. I have set privileged: true for mopidy in the docker-compose file. It's set to true for all the services.
There looks like there is something up with the hardware. When I run aplay -l I get the following results:
aplay: device_list:270: no soundcards found...

@Langholt the same seems to be happening for me, what pi are you using?

@odm7341 Noted. I'll try that and report back.

@odm7341 thank you everything works now. Thank to you and @odm7341 and @janvda. I think this issue can be closed now.
Now I just have to figure out how to turn up the volume properly when listening to spotify.

@Langholt i got it working by doing what @janvda says, but you also have to make the modifications to the files in the Spotify, Airplay, AND Mopidy services. Like what I've done here

@odm7341  i downloaded your version and pushed it .
but i am getting --

root@5a00ead:/usr/src/app# aplay -l
aplay: device_list:272: no soundcards found...
root@5a00ead:/usr/src/app# speaker-test -c2 -t wav

speaker-test 1.1.8
Playback device is default
Stream parameters are 48000Hz, S16_LE, 2 channels
WAV file(s)
ALSA lib confmisc.c:767:(parse_card) cannot find card '0'
ALSA lib conf.c:4568:(_snd_config_evaluate) function snd_func_card_driver returned error: No such file or directory
ALSA lib confmisc.c:392:(snd_func_concat) error evaluating strings
ALSA lib conf.c:4568:(_snd_config_evaluate) function snd_func_concat returned error: No such file or directory
ALSA lib confmisc.c:1246:(snd_func_refer) error evaluating name
ALSA lib conf.c:4568:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:5047:(snd_config_expand) Evaluate error: No such file or directory
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM default
Playback open error: -2,No such file or directory
root@5a00ead:/usr/src/app# ^C
root@5a00ead:/usr/src/app#

nvm got it working after purging the system and let the rpi redownload it

Great to hear. One note from me, I'm still dealing with issues on very low volume, even when all my volume controls are maxed. Have any of you dealt with that? Do I need to adjust alsamixer settings? And if so in which container.
This might be off topic for this thread.


mm-scans.org Seems to be Gone

Source name
MMScans
Source language
English
Source link
https://mm-scans.org
Other details
https://www.reddit.com/r/manhwa/comments/190b48k/mmscans_does_anyone_know_what_happened_to_them/
temporarily down maybe?
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source website is down.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

It is temporary it seems:

Pulled from their discord


[stdlib] rewrite parameter of copyBytes using a primary associated type

Express the generic parameter of UnsafeMutableRawBufferPointer.copyBytes() function using some notation
(primary associated type).
Addresses rdar://99933149

@swift-ci please test macOS platform

@swift-ci please test linux platform

Blocked on rdar://99908987

@swift-ci please test macOS platform

Note: I added an exception in commit 322771d that should not be needed.

@swift-ci please test

@swift-ci please test

Given the unresolved issue with the api-digester test, should we go ahead with this, or nix it?


BUILD-191: add e2e tests

/assign @coreydaley
this should over the scenarios discussed in the README
my thought is that once we've reviewed / iterate / merge, possible next steps:

go ahead and craft "interim" PRs in openshift/release to see if PR test jobs will work where the deploy via ./deploy/deploy.sh works in CI
then circle back with the OLM work in BUILD-159 and integrate / install via that form

/lgtm

/lgtm


Use env or const instead of assertions to enable/disable optimizations

Addresses #157.
Instead of using assertions to toggle Amp's Deferred/Emitter optimizations, this PR uses either an environment variable or const AMP_OPTIMIZATIONS. If neither of these are defined, an E_USER_NOTICE is issued.
I think this is a better approach to enabling optimizations, as it is more obvious to the user and does not depend the setting for assertions, which commonly are configured incorrectly for the environment, tending to default to off.
I put the code to issue the notice in the Promise class definition file. This was a rather arbitrary place (only chosen because the optimizations relate to promises) and if someone has a better suggestion please let me know.

Closed in favor of #190.


#94 : Add namespace notion to ubiquity_<install,uninstall,deployments>.sh

All details inside #94

This change is

Review status: 0 of 7 files reviewed at latest revision, 9 unresolved discussions.

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 55 at r1 (raw file):
Previously, alonm wrote…
'kubectl .. components' =>'display all ubiquity components'

I agree, Shay, did you ask for Dima's review for all your user interactive experience?

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 56 at r1 (raw file):
Previously, alonm wrote…
'kubectl get configmap,storageclass,pv,pvc,service,daemonset,deployment,pod' => 'display  configmap,storageclass,pv,pvc,service,daemonset,deployment,pod components'

If this is to be used by users, it should explain itself better, like "display all related deployed entities" and then have a title for each entity type.

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 78 at r1 (raw file):

    kubectl create $nsf -f ${YML_DIR}/ubiquity-deployment.yml
    kubectl create $nsf -f ${YML_DIR}/ubiquity-k8s-provisioner-deployment.yml
    kubectl create $nsf -f ${YML_DIR}/ubiquity-k8s-flex-daemonset.yml

You are not checking for any return codes on any of your commands. You should have an executer function to check the return status and exit nicely on errors. A more advanced feature would be to pick up where last action failed (be idempotent) - TBD.

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 149 at r1 (raw file):

    cmd="kubectl get $nsf cm/k8s-config pv/ibm-ubiquity-db pvc/ibm-ubiquity-db svc/ubiquity svc/ubiquity-db  daemonset/ubiquity-k8s-flex deploy/ubiquity deploy/ubiquity-db deploy/ubiquity-k8s-provisioner"
    echo $cmd
    echo '---------------------------------------------------------------------'

This should be defined in a constant DASHES or something.

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 286 at r1 (raw file):

  esac
done
echo "Working in namespace [$NS]."

Deployment namespace is [$NS]

Comments from Reviewable

Review status: 0 of 7 files reviewed at latest revision, 10 unresolved discussions.

a discussion (no related file):
Are you sure it won't be easier and nicer to deploy using python with https://github.com/kubernetes-incubator/client-python/tree/master/kubernetes

Comments from Reviewable

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 73 at r1 (raw file):

function start()
{
    echo "Make sure ${UBIQUITY_DB_PVC_NAME} exist and bounded to PV (if not exit), before starting."

Make sure ${UBIQUITY_DB_PVC_NAME} exists and bound to PV (exit otherwise)...

Comments from Reviewable

Reviewed 9 of 9 files at r1.
Review status: all files reviewed at latest revision, 11 unresolved discussions.

Comments from Reviewable

Review status: all files reviewed at latest revision, 11 unresolved discussions.

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 54 at r1 (raw file):
Previously, alonm wrote…
missing space

I am not sure where is the missing space, maybe after the (). please be more specific

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 55 at r1 (raw file):

display all ubiquity components
fixes
Dima should review it soon, there is open ticket for it.

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 56 at r1 (raw file):
Previously, ranhrl (Ran Harel) wrote…
If this is to be used by users, it should explain itself better, like "display all related deployed entities" and then have a title for each entity type.

improved a bit
anyway user that run this command understand the kubectl get syntax

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 73 at r1 (raw file):
Previously, midoblgsm (Mohamed Mohamed) wrote…
Make sure ${UBIQUITY_DB_PVC_NAME} exists and bound to PV (exit otherwise)...

applied your text

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 78 at r1 (raw file):
Previously, ranhrl (Ran Harel) wrote…
You are not checking for any return codes on any of your commands. You should have an executer function to check the return status and exit nicely on errors. A more advanced feature would be to pick up where last action failed (be idempotent) - TBD.

set -e  should be good enough for these kubectl create
if there is an issue, user will see the relevant error that kubectl returns

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 81 at r1 (raw file):

Initiated ubiquity components
will consult with dima

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 97 at r1 (raw file):
Previously, alonm wrote…
'Finished to stop ubiquity deployments' => 'Stopping ubiquity components'

stopping... why, its the end of the operation. anyway will consult with Dima

scripts/k8s_scbe_all_in_one/ubiquity_deployments.sh, line 149 at r1 (raw file):

DASHES
there any tricky ways to do it, but i prefer to stay simple

Comments from Reviewable


Code is broken (still/again?)

The mergeTrees code added by tilgovi has a number if syntax errors that completely break pouchdb causing it to fail all unit tests. I have commented mergeTrees out and now tests 1 and 2 pass, but  it hangs on test 3 I think. txn.objectStore('ids').openCursor() could be the culprit. I don't know enough about IndexDB, nor the PouchDB code to be sure.
Can someone give me an idea of the status of the PouchDB project.
I think it would be brilliant to have CouchDB capabilities running in the browser (JS) with IndexedDB and Couch's Replication. I've yet to find any solution for sometimes connected apps with replication anywhere near as good as CouchDB. Nothing I'm aware of comes close.
Congrats to the folks working on PouchDB. I really would love to see this completed.
Neville

Still broken in Jan. 2012. Same errors as above.
I cannot wait for PouchDB to be working. For now, I'd be happy with just a working (tests passing) but unsynchable version - on the expectation that these smart folks will eventually get the CouchDB synch part working right. Presumably the API is not going to change much and synch is not critical for my project at this stage.
Does anyone know whether there is a commit I could use which passes all tests?

yep. some parts of the code in pouch.js are totally f*ed up. like syntax errors :|
i'm trying to fix and get a working copy with tests passing... but the code needs serious reviews... i'll try and commit some improvements... actually thinking in refactoring some stuff


Update package_info_plus

:scroll: Description
This PR allows package_info_plus to be used in its newest version 3.0.0.
:bulb: Motivation and Context
The federated two-package refactor can now be used together with sentry_flutter.
:green_heart: How did you test it?
All the workflows went through and the Sentry functionality in my app still works perfectly fine with the new version.
:pencil: Checklist

[x] I reviewed submitted code
[x] I added tests to verify changes
[x] I updated the docs if needed
[x] All tests passing
[x] No breaking changes

@ThexXTURBOXx thanks.
Please add an entry to the changelog, similar to https://github.com/getsentry/sentry-dart/blob/main/CHANGELOG.md#fixes-1

@marandaneto Done! :)

Codecov Report
Base: 89.90% // Head: 95.55% // Increases project coverage by +5.65% :tada:

Coverage data is based on head (248e930) compared to base (870f5eb).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1084      +/-   ##
==========================================
+ Coverage   89.90%   95.55%   +5.65%
==========================================
  Files         106        2     -104
  Lines        3388       45    -3343
==========================================
- Hits         3046       43    -3003
+ Misses        342        2     -340

Impacted Files
Coverage Δ

...art/lib/src/http_client/failed_request_client.dart

dart/lib/src/client_reports/client_report.dart

dart/lib/src/scope.dart

dart/lib/src/protocol/sentry_response.dart

dart/lib/src/protocol/sdk_info.dart

dart/lib/src/protocol/mechanism.dart

dart/lib/src/protocol/breadcrumb.dart

dart/lib/src/utils/hash_code.dart

dart/lib/src/default_integrations.dart

dart/lib/src/transport/rate_limit_parser.dart

... and 98 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Front Matter Cleanup

This PR removes the following front matter parameters:

toc-level
frontpage_featured
menu_order (converted to weight)
parent

It also removes one .md.orig file that was probably committed by accident, as well as fixes a link with faulty text within it.
The rest of our front matter parameters are either used by Hugo, Algolia or both.

Hi Natasa (@NatasaKralj ) You seem to have missed two files which still have "parent:" in them content\en\docs\addons\ats-addon\rg-ats\rg-one-ats\rg-one-test-dev\rg-one-std-actions-ref\rg-one-appstore-widg-acts\rg-one-simple-checkbox-set-selector\rg-one-assert-simple-checkbox-set-selector-value.md and content\en\docs\addons\ats-addon\rg-ats\rg-one-ats\rg-one-test-dev\rg-one-std-actions-ref\rg-one-appstore-widg-acts\rg-one-simple-checkbox-set-selector\rg-one-toggle-simple-checkbox-set-selector-value.md
You also haven't cleaned out all the category: items in the front matter. But I can't think of any other ones which might be there.
(I assume you have been finding and replacing ^parent:.*\n ?)
Nice to clean up these. I'll leave you to do the above.

As discussed, Category needs to stay for now as it is used by Algolia.

Removed leftover parent: parameters.


fix: remove rh-demo

What I did

Removed rh-demo

Testing Instructions

View Deploy Preview ensure rh-demo is no longer used on rh-stat

@brianferry maybe we should fold in #756 to this PR, like you suggested


Define and refactor to sub-components code standard

There are functional differences to invoking components using <ComponentSyntax /> or FunctionalSyntax(), the latter often seen when creating intermediate components for reducing clutter in the render method of a parent component.
Example, seen in many of our apps (+ this repo):
const MyComponent = () => {

  const MyHeader = () => (
    <Text>Some text here</Text>
  );

  return (
    {MyHeader()}
    <Text>Main content</Text>
  );
}

We should have some clear standards to this, and these standards need to be enforced throughout our repository.
It would be great to have devs chime in on this. When a standard has been established, a refactor of the affected packages is needed (skia draw and components).
Some reading information:
https://dev.to/igor_bykov/react-calling-functional-components-as-functions-1d3l

I think sticking to componentSyntax is the way. As the article describes, there are side effects to functionSyntax that can lead to nasty bugs. Also it looks messy and unintuitive to me.
As a sidenote, I prefer to keep as much of the JSX inside the return statement of the main function. That makes it clearer what order each component is in the render hierarchy and keeps the code nice and  segregated. Any JSX that is repeated several times or is too complex, could be treated as a sub-component and moved to its own file (either in the same folder or the components folder if it is generic)


qiita.slurm_resource_allocations

This adds the basic changes to add a new table to keep the slurm_resource_allocations info and fix #3368.

coverage: 92.86%. remained the same
when pulling dc82e12aa1e6b206549a02bf98a56975fb1d1776 on antgonza:qiita.slurm_resource_allocations
into 83476d5c3167c0309d94e983c18a27a436d9c4fa on qiita-spots:dev.


⚠ Processors has degraded performance

In 46eadfc, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 64 ms

Resolved: Processors performance has improved in 65db611.


merge

Some prj change the sdk form 1024 to 10586

Three languages should be sufficient. Don't need to add a fourth.


Add Google groups badge

A badge can be added to readme.md opening up the Google group link

Closing the issue


[FEATURE REQ] Rate limit requests to the same host

Is your feature request related to a problem? Please describe.
I've got a README.md that has two links to a hackernews post.
Since a couple of days, linkspector fails with a 503 on the 2nd URL in the file, independently of the order. Linkspector succeeds when either of the URL's is commented out.
It therefore seems that the server is responding witht 503 because linkspector fires the request to quickly, or something like that.
Describe the solution you'd like
Perhaps linkspector could wait a (configurable) amount of time between requests to the same host.

Everything works fine if I adjust the batchSize from 100 to 1 in checkHyperlinks().
https://github.com/UmbrellaDocs/linkspector/blob/91e60932db8925d690dc59145906593949c0469f/lib/batch-check-links.js#L49

Keeping the batchSize at 100 and adding a random delay of up to 10 seconds à la https://stackoverflow.com/a/45010143/2097 also works:
// returns a promise that resolves after the specified number of ms
function delay(ms) {
    return new Promise(resolve => {
        setTimeout(resolve, ms);
    });
}

async function checkHyperlinks(nodes, options = {}, filePath) {
  const { batchSize = 100, retryCount = 3, aliveStatusCodes } = options

...

    for (let i = 0; i < tempArray.length; i += batchSize) {
      const batch = tempArray.slice(i, i + batchSize)
      const promises = batch.map(async (link) => {
        await delay(Math.random() * 10000); // wait
        const page = await browser.newPage()

but 1 second is not enough.

Or maybe URL's to the same host should always be in separate batches?

Example README.md, since I've updated mine to have only a single hackernews link in there:
## Quotes

These quotes highlight the goal and status of this repository.

[kachnuv_ocasek](https://news.ycombinator.com/item?id=36354589)

[arghwhat](https://news.ycombinator.com/item?id=36354464)


Enable Solidity Converage

Description
It was disable during truffle 5 migration.
We now use: 0x/sol-coverage, which has a different strategy towards profiling. Not intrusive.
Tested
An explanation of how the changes were tested or an explanation as to why they don't need to be.
Other changes

Simplify coverage testing setup
Simplify runTests
Fix test errors (racing conditions)

Related issues

Fixes #162

I'm also confused. Don't yet know why this is failing. 🤦♂
On Tue, 6 Aug 2019 at 13:37 Aaron DeRuvo<EMAIL_ADDRESS>wrote:

@aaronmgdr commented on this pull request.
In packages/protocol/runTests.js
https://github.com/celo-org/celo-monorepo/pull/305#discussion_r311161806
:

})

try {
const closeGanache = await startGanache()

if (!argv.local) {

if (isCI) {

 // if we are running on circle ci we need to wait for ganache to be up

 // TODO(mcortesi): improvement: check for open port instead of a fix wait time.

a fix wait time => a fixed wait time
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/pull/305?email_source=notifications&email_token=AAFAHFBEQRK2MAXZQTAFQZLQDGSC3A5CNFSM4IJNAPYKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCAXFJAY#discussion_r311161806,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFAHFADVJXENW5VWNXU6ILQDGSC3ANCNFSM4IJNAPYA
.


firebase

did you have an example using firebase realtime database? thanks before

Sorry, I have no experience about that :(

I think you can search it on the StackOverflow.


Add Portuguese Language File

Translated to Pt Pt

ty, i will do as suggested. What is the name of the project on weblate?

It's hosted on https://translate-h5p.tk/ as mentioned before.

Fixed this myself now.


Feature/no endless

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[x ] a bug fix (Fix #...)
[ ] a new Ripper
[ ] a refactoring
[ ] a style change/fix
[ ] a new feature

Description
there were multiple issues reported like #20 #21. the parent class for unit tests marks the run as test: https://github.com/ripmeapp2/ripme/blob/main/src/test/java/com/rarchives/ripme/tst/ripper/rippers/RippersTest.java#L33. which then is taken into account in AbstractHTMLRipper to not test next pages: https://github.com/ripmeapp2/ripme/blob/main/src/main/java/com/rarchives/ripme/ripper/AbstractHTMLRipper.java#L168.
If this property is not tested, even the unit test would create an endless loop. Prevent this by remembering the URL.
Testing
Required verification:

[ x] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[x ] I've verified that this change works as intended.

[ ] Downloads all relevant content.
[ ] Downloads content from multiple pages (as necessary or appropriate).
[ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[x ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

wrong repo


⚠ Processors has degraded performance

In e2904f7, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 115 ms

Resolved: Processors performance has improved in 2c7e21b.


Update timeouts for vUSB

This PR:

removes the recommended way, there is no recommended way anymore
adds a timeout notification

Added one suggestion for a minor change. Thank you!

Thanks, accepted it


How to set task condition to continue on failure?

I see that the task condition can be set to "Even if a previous dependency has failed, unless the run was canceled", but I can't figure out how to configure this.  Maybe an example could be added to the page?

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

ID: 8098f527-ebdf-60d5-3989-5228b7a207c1
Version Independent ID: ce27c817-9599-00ef-5af2-3ac1dbad8dc6
Content: Build and Release Tasks in Azure Pipelines and TFS - Azure Pipelines
Content Source: docs/pipelines/process/tasks.md
Product: devops
Technology: devops-cicd
GitHub Login: @andyjlewis
Microsoft Alias: alewis

Aside from updating the documentation, if anyone had a few minutes to give a yml snippet here of the solution that would be awesome!

Not a single example of the conditions are on this page.

@lukewis - I think you want:
steps:
- task: ...
  condition: succeededOrFailed()

Which should run unless cancelled. The condition syntax is an expression, which you can read more of in the docs about conditions or expressions


Change the GCST for submission 63f8add57325c20001210db7 (PMID-36635386)

The GCSTs need to be changed to the original ones mentioned in the published article. I've attached the list of study tags and GCST. Column C has the correct list of GCST.
The user submitted 2 different submissions for the same PMID. I combined those 2 submissions and submitted it under the PMID. The submission was manually pushed through validation. Now the GCSTs need to be changed to the ones submitted by the user.
https://app.zenhub.com/files/2995118/3a37c878-9cc5-4a3f-8c76-1c7fb7a12768/download

@Santhi1901 Updated the GCST in Deposition side , couldn't find the GCST in Curation App so assuming the submission has not been imported, Kindly confirm if the data looks good

@sajo-ebi it looks good, thanks

The GCSTs from this publication appear in both unpublished and published sumstats tables, e.g. GCST90203512. There are 4443 studies affected. This needs fixing in production before the ESHG conference June 10th.

@sajo-ebi please do before DR

deleted unpublished studies which have been duplicated from Curation DB, need to wait for DR to reflect in Catalog

@ljwh2 confirm and close ticket

confirmed


PR description node has Checkout default branch in context menu

Context menu for PR item:

Context menu for description node under PR item:

Changes are not in insiders yet.


Demo code on atsign.dev no longer compiles cleanly

Describe the bug
Error when following the instructions on atsign.dev
To Reproduce
Steps to reproduce the behavior:

First go to https://atsign.dev/docs/get-started/tryatplatform/
When running the app compile time errors (tested on windows)

C:\Users\colin\test\testme>flutter run
Using hardware rendering with device Android SDK built for x86. If you notice graphics artifacts, consider enabling software rendering with "--enable-software-rendering".
Launching lib\main.dart on Android SDK built for x86 in debug mode...
/C:/tools/flutter/.pub-cache/hosted/pub.dartlang.org/at_client-3.0.13/lib/src/client/remote_secondary.dart:83:20: Error: A value of type 'int?' can't be assigned to a variable of type 'int' because 'int?' is nullable and 'int' isn't.
      ..commitId = lastSyncedId
                   ^

FAILURE: Build failed with an exception.

* Where:
Script 'C:\tools\flutter\packages\flutter_tools\gradle\flutter.gradle' line: 1102

* What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.
> Process 'command 'C:\tools\flutter\bin\flutter.bat'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 36s
Running Gradle task 'assembleDebug'...                             37.0s
Exception: Gradle task assembleDebug failed with exit code 1

C:\Users\colin\test\testme>

Expected behavior
The demo should always run and be tested when we push new code (lets automate this !)
Screenshots
See above
Smartphone (please complete the following information):

Tested on Windows but this will be the same across all platforms is my guess

Were you using an @application when the bug was found?
-demo app
Additional context
PLease use this and the snackbar sender as a simple test of client changes!

I think maybe we canary builds on the client side.. Thoughts welcome

Ideally I think we would run a straight compile only (because we won't have a device connected) on this demo (and any other 'interesting' code) every time there is a merge to trunk of a PR in the at_client_sdk, at_tools or at_libraries repos
I don't know if cross-server hooks like that are possible, so a good alternative might be to compile this demo periodically - maybe a few times per day.
I was curious about this issue because while both at_client and at_commons packages have changed recently, the latest packages are consistent with each other, and the previous versions are consistent with each other also - i.e. in theory per my understanding this problem should not have happened ... so I looked into it a bit more deeply, and I think it's a problem with how flutter pub get works
Snackbar has this in its pubspec.yaml
  at_client_mobile: ^3.0.3
  at_utils: ^3.0.0

Here's what .packages has after first running at_app create and flutter run. I've only included the subset of packages relevant to this conversation
gkc2019:snackz gary$ grep --color=none at_ .packages
at_client:file:/some/path/at_client-3.0.13/lib/
at_client_mobile:file:/some/path/at_client_mobile-3.1.9/lib/
at_commons:file:/some/path/at_commons-3.0.10/lib/
at_lookup:file:/some/path/at_lookup-3.0.15/lib/
at_utils:file:/some/path/at_utils-3.0.7/lib/

However if I change the dependency toat_client_mobile: ^3.1.10 then when I remove .packages and do another flutter pub get, everything resolves just fine, and the app builds and successfully runs on my iphone.
Here's what .packages has now:
at_client:file:///Users/[...]/at_client-3.0.14/lib/ <== up, from 3.0.13
at_client_mobile:file:/some/path/at_client_mobile-3.1.10/lib/ <== up, from 3.1.9
at_commons:file:/some/path/at_commons-3.0.9/lib/ <== down, from 3.0.10
at_lookup:file:/some/path/at_lookup-3.0.14/lib/ <== down, from 3.0.15
at_utils:file:/some/path/at_utils-3.0.6/lib/ <== down, from 3.0.7

In summary - when we start with '^3.0.3' for at_client_mobile, dependencies don't resolve correctly, even though the latest version after 3.0.3 is 3.1.10; but when we start with '^3.1.10' for at_client mobile, the dependencies all do resolve correctly. (Another point to note if anyone else picks this up to investigate more, is that at_client_mobile is restricting to specific known versions at_client: 3.0.14 at_lookup: 3.0.14 at_commons: 3.0.9 - which is fine, as we know those versions work together, and ultimately those are indeed the versions we end up with)
Not sure what to do about that. Maybe configure dependabot to nag us to update pubspec.yaml in important packages (like the snackbar demo) when new versions of dependencies are available? @XavierChanth wdyt?

So upgraded flutter to latest version and everything works just fine now...
Very odd as I cannot find any notes covering this issue..
Will look into a job to build this every so often and keep flutter etc at the latest version

That's just plain weird.

Not sure what to do about that. Maybe configure dependabot to nag us to update pubspec.yaml in important packages (like the snackbar demo) when new versions of dependencies are available? @XavierChanth wdyt?

This has been in the back of my mind with at_app, I'd like to automate as much maintenance work with the repository as possible. All verification and testing should eventually be automated, but I still have plenty of work to do before it gets there. I plan to resume my progress on this goal in May.

@XavierChanth is this an arch discussion item?

It's related to our overall testing approach, particularly automated retesting when deiendencues change  and yes we need to discuss it

Closing, as this demo code no longer exists.


Having 401 Unauthorized only with mobile

My authentication works on my desktop but when I use my mobile it does not work.
I am using ng-token-auth and devise_token_auth for my angularjs application.
SessionsController is happening correctly. I can retrieve the new logged in user.
But any subsequent request will do (mobile only):
[2015-07-14 11:27:56 +0200] Started GET "/api/posts?page=1" for <IP_ADDRESS> at 2015-07-14 11:27:56 +0200
[2015-07-14 11:27:56 +0200] Processing by Api::V5::PostsController#index as JSON
[2015-07-14 11:27:56 +0200]   Parameters: {"page"=>"1"}
[2015-07-14 11:27:56 +0200] Filter chain halted as :authenticate_user! rendered or redirected
[2015-07-14 11:27:56 +0200] Completed 401 Unauthorized in 1ms (Views: 0.2ms | ActiveRecord: 0.0ms | Solr: 18.4ms)
```
If I refresh the mobile browser (Chrome & Safari) , then I am out.

On my desktop it do:

[2015-07-14 11:38:00 +0200] Processing by Api::V5::PostsController#index as JSON
[2015-07-14 11:38:00 +0200]   Parameters: {"page"=>"1"}
[2015-07-14 11:38:00 +0200] Started GET "/api/notifications" for <IP_ADDRESS> at 2015-07-14 11:38:00 +0200

I tested with Chrome and Safari and an IPhone and it does not work.
On xcode iphone simulator it works fine.
On Chrome and Safari desktop it works also.

Does anyone have a clue?

I found a related issue: #299

This is in production right now!
Any help please?

@casertap - what version of devise_token_auth are you running, and are you using localStorage or cookies as the preferred persistence mechanism?

devise_token_auth v0.1.32.beta10
I am using ng-token-auth 0.0.25 defaults so it is set to cookies (I did try with storage 'localStorage' and it broke everything (mobile + desktop))
I switch back to cookies so my desktop login works fine.
It is working with a Nexus Phone so the problem comes from safari ios only.

my bad. It actually works with the option -> storage: 'localStorage'
Thanks.

@casertap - recently fixed an issue in ng-token-auth where an invalid cookie expiration was preventing iOS from correctly storing auth tokens as a cookie. see also: lynndylanhurley/ng-token-auth#172
Might be worth pulling ng-token-auth from master and testing cookie storage. FWIW - iOS private browsing will not support localStorage, so you'll have to do a check during configuration.


support environment variables without overexposing on client

This addresses #159 – only inserts environment variables that are being used, so it doesn't overexpose anything. Production (next build) will provide a default NODE_ENV=production, but that can be overridden if you want.

Coverage increased (+0.3%) to 34.495% when pulling 827f76d7ebe2c401e0ccd568913bebe7f429eed8 on matthewmueller:environment-variables into 36abdc77c5f79b074ff4107abf970c443f1be7b3 on zeit:master.

Coverage increased (+0.3%) to 34.495% when pulling 827f76d7ebe2c401e0ccd568913bebe7f429eed8 on matthewmueller:environment-variables into 36abdc77c5f79b074ff4107abf970c443f1be7b3 on zeit:master.

@arunoda maybe we're talking about different levels of security, but from what I understand, the client will only get a build with the environment variables that it uses. The server will only serve components that have the specified environment variables baked in.
I'm basically trying to achieve exactly what https://github.com/hughsk/envify is doing. I think having to prefix everything should be a last option though.

What if there's some piece of third party module do something like this:
console.log(process.env.SOME_SEC_ENV_VAR)
We don't know what's inside the env vars and the people who write the code and people who run that is different. So, if there's need to something to expose to the client side, that's a total responsibility of the people who run that code.
That's why we need to prefix env vars. So, they must set them explicitly.

Yah, I guess I find that threat level to be greatly outweighed by the convenience of mapping your environment variables directly.
Usually either it's your teammates running the code with shared environment variables or it's the server.
The only thing I can think of where this would be an issue is running some open source code, but you should know what you're running beforehand anyway.

@matthewmueller there could be many reasons we can't come up with.
We don't need to make it less secure. (Even there might be less chance it might happen)
So, prefixing is the way how others deals with this. (Specially CRA, Storybook )

I was trying out Next 2.0 Beta tonight, and released with the new custom Webpack config setting env vars can now be accomplished via next.config.js: https://github.com/zeit/next.js/issues/159#issuecomment-268476743
That said, I'd still like to see a first-class solution that doesn't involve dealing with Webpack directly. Once nice feature of CRA's env support is reading a .env file during development. I'd like that see that be part of the first-class env feature.

@ericf I'm never a fan of .env file of CRA.
There's so many simple ways we could do it when exposing ENV vars are supported like this.

Is this something we can purely do from webpack extensions, without having to support first-class? I'd like to not have to add support for envs whitelisting really. That should be 100% up to the dev

yah, i think so, need to test is but on the client, the next.config.js would replace the variables, on the server, it would just be a part of process.env.*
i do think this is essential for any client-side web app (something that you'd need day 2 of building any web app), but definitely it in the webpack config is easy enough, so i'll close for now.


Rust GDExtension fails to register class that extends HttpRequest

Godot version
4.0.dev beta16
System information
Windows 11 & PopOS
Issue description
Using https://github.com/godot-rust/gdextension and creating a simple class that extends HttpRequest fails when attempting to register it.
Outputted when project ran from the editor.
E 0:00:00:0400   _register_extension_class: Attempt to register an extension class 'TestHTTP' using non-existing parent class 'HttpRequest'
  <C++ Error>    Method/function failed.
  <C++ Source>   core/extension/gdextension.cpp:263 @ _register_extension_class()

I first reported it there (godot-rust/gdextension#88), but I was suggested that I report it here.
I suspect that this issue to do with class name's being case-sensitive (godot-rust/gdextension is requesting "HttpRequest" while the class name is officially "HTTPRequest") and is a matter of whether Godot allows for case-insensitive class inheritance for extensions or godot-rust/gdextension updates the name to match godot.
Steps to reproduce

Set GODOT4_BIN environment variable to the path to the Godot binary.
Build the rust project in MinRepoRustGodotHTTPRequest\rust.
Open the project in MinRepoRustGodotHTTPRequest\godot and run it.

Minimal reproduction project
MinRepoRustGodotHTTPRequest.zip

This was a bug in godot-rust, not in Godot.
The class name was passed wrong. It's totally reasonable for Godot class names to be case-sensitive 👌
As such, this issue can be closed.


Issue in points np.concatenate(s_points_list, axis=0) in centerformer-master/det3d/core/sampler/sample_ops.py

Hello,
Thanks for the open-source code.
The s_point_list is always empty in my case, the random_crop is set False in https://github.com/TuSimple/centerformer/blob/master/det3d/core/sampler/sample_ops.py#L195, even if set to True, doesn't give me s_points. Also, from the prev. condition check here https://github.com/TuSimple/centerformer/blob/master/det3d/core/sampler/sample_ops.py#L173, the s_points is empty [].
So trying to concatenate an empty array gives me an error.
What could be the issue? I'm trying using the NuScenes mini dataset, I was able to prepare date successfully.

what should be the pathlib.Path(root_path) in https://github.com/TuSimple/centerformer/blob/master/det3d/core/sampler/sample_ops.py#L181? probably there's where I'm having some issue.

Solved.