Segfault

Got the following crash while creating a client library. The commands being executed related to JSCAN were:
"JSCAN" "0" "COUNT" "1000"
"JSCAN" "0" "COUNT" "1000" "QUEUE" "queue17"

No other JSCAN parameters were used. I did not run a full memory check yet.
19621:C 16 Aug 03:09:30.740 # Warning: no config file specified, using the default config. In order to specify a config file use ./src/disque-server /path/to/disque.conf
19621:P 16 Aug 03:09:30.742 # You requested maxclients of 10000 requiring at least 10032 max file descriptors.
19621:P 16 Aug 03:09:30.742 # Server can't set maximum open files to 10032 because of OS error: Operation not permitted.
19621:P 16 Aug 03:09:30.742 # Current maximum open files is 2048. maxclients has been reduced to 2016 to compensate for low ulimit. If you need higher maxclients increase 'ulimit -n'.
19621:P 16 Aug 03:09:30.743 * Node configuration loaded, I'm d1dbbca17a9759ccf377a24132a7f7dc380f2918
                                        Disque 0.0.1 (6dbfa4c1/0) 64 bit
          _ -
        .                               Port: 7711
        .    o    .                     PID: 19621
                 .
               -                              http://disque.io

19621:P 16 Aug 03:09:30.743 # Server started, Disque version 0.0.1
19621:P 16 Aug 03:09:30.743 # WARNING: The TCP backlog setting of 511 cannot be enforced because /proc/sys/net/core/somaxconn is set to the lower value of 128.
19621:P 16 Aug 03:09:30.743 * The server is now ready to accept connections on port 7711
19621:P 16 Aug 03:17:17.768 #

=== DISQUE BUG REPORT START: Cut & paste starting from here ===
19621:P 16 Aug 03:17:17.768 #     Disque 0.0.1 crashed by signal: 11
19621:P 16 Aug 03:17:17.768 #     Failed assertion: <no assertion failed> (<no file>:0)
19621:P 16 Aug 03:17:17.768 # --- STACK TRACE
./src/disque-server *:7711(logStackTrace+0x33)[0x4297d3]
./src/disque-server *:7711(equalStringObjects+0x4)[0x4224d4]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x10340)[0x7f7ccb04a340]
./src/disque-server *:7711(equalStringObjects+0x4)[0x4224d4]
./src/disque-server *:7711(jscanCallback+0x20)[0x4320b0]
./src/disque-server *:7711(dictScan+0x198)[0x412f98]
./src/disque-server *:7711(jscanCommand+0x178)[0x434528]
./src/disque-server *:7711(call+0x50)[0x415440]
./src/disque-server *:7711(processCommand+0xf2)[0x416ea2]
./src/disque-server *:7711(processInputBuffer+0x97)[0x420627]
./src/disque-server *:7711(aeProcessEvents+0x250)[0x410610]
./src/disque-server *:7711(aeMain+0x2b)[0x41084b]
./src/disque-server *:7711(main+0x325)[0x40f615]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf5)[0x7f7ccac96ec5]
./src/disque-server *:7711[0x40f739]
19621:P 16 Aug 03:17:17.770 # --- INFO OUTPUT
19621:P 16 Aug 03:17:17.770 # # Server
disque_version:0.0.1
disque_git_sha1:6dbfa4c1
disque_git_dirty:0
disque_build_id:63f6d787d079142c
os:Linux 3.8.11 x86_64
arch_bits:64
multiplexing_api:epoll
gcc_version:4.8.4
process_id:19621
run_id:0b271fe02f5f2fad3fef2380e8ab1c420029b2a7
tcp_port:7711
uptime_in_seconds:467
uptime_in_days:0
hz:10
config_file:

# Clients
connected_clients:4
client_longest_output_list:0
client_biggest_input_buf:0
blocked_clients:0

# Memory
used_memory:500888
used_memory_human:489.15K
used_memory_rss:1814528
used_memory_peak:500888
used_memory_peak_human:489.15K
mem_fragmentation_ratio:3.62
mem_allocator:jemalloc-3.6.0

# Jobs
registered_jobs:23

# Queues
registered_queues:11

# Persistence
loading:0
aof_enabled:0
aof_state:off
aof_rewrite_in_progress:0
aof_rewrite_scheduled:0
aof_last_rewrite_time_sec:-1
aof_current_rewrite_time_sec:-1
aof_last_bgrewrite_status:ok
aof_last_write_status:ok

# Stats
total_connections_received:25
total_commands_processed:62
instantaneous_ops_per_sec:0
total_net_input_bytes:3691
total_net_output_bytes:9925
instantaneous_input_kbps:0.00
instantaneous_output_kbps:0.00
rejected_connections:0
latest_fork_usec:0

# CPU
used_cpu_sys:8.53
used_cpu_user:6.64
used_cpu_sys_children:0.00
used_cpu_user_children:0.00

# Commandstats
cmdstat_info:calls=1,usec=65,usec_per_call=65.00
cmdstat_monitor:calls=1,usec=5,usec_per_call=5.00
cmdstat_hello:calls=1,usec=5,usec_per_call=5.00
cmdstat_addjob:calls=26,usec=484,usec_per_call=18.62
cmdstat_getjob:calls=6,usec=272,usec_per_call=45.33
cmdstat_ackjob:calls=3,usec=15,usec_per_call=5.00
cmdstat_fastack:calls=3,usec=14,usec_per_call=4.67
cmdstat_deljob:calls=1,usec=8,usec_per_call=8.00
cmdstat_jscan:calls=18,usec=207,usec_per_call=11.50
cmdstat_enqueue:calls=1,usec=12,usec_per_call=12.00
cmdstat_dequeue:calls=1,usec=7,usec_per_call=7.00
hash_init_value:<PHONE_NUMBER>

19621:P 16 Aug 03:17:17.770 # --- CLIENT LIST OUTPUT
19621:P 16 Aug 03:17:17.770 # id=12 addr=<IP_ADDRESS>:42917 fd=9 name= age=170 idle=162 flags=N qbuf=0 qbuf-free=0 obl=0 oll=0 omem=0 events=r cmd=jscan
id=22 addr=<IP_ADDRESS>:42979 fd=12 name= age=0 idle=0 flags=N qbuf=0 qbuf-free=32768 obl=0 oll=0 omem=0 events=r cmd=getjob
id=24 addr=<IP_ADDRESS>:42981 fd=11 name= age=0 idle=0 flags=N qbuf=0 qbuf-free=32768 obl=0 oll=0 omem=0 events=r cmd=jscan
id=25 addr=<IP_ADDRESS>:42982 fd=10 name= age=0 idle=0 flags=N qbuf=0 qbuf-free=32768 obl=0 oll=0 omem=0 events=r cmd=hello

19621:P 16 Aug 03:17:17.770 # --- CURRENT CLIENT INFO
19621:P 16 Aug 03:17:17.771 # client: id=24 addr=<IP_ADDRESS>:42981 fd=11 name= age=0 idle=0 flags=N qbuf=0 qbuf-free=32768 obl=0 oll=0 omem=0 events=r cmd=jscan
19621:P 16 Aug 03:17:17.771 # argv[0]: 'JSCAN'
19621:P 16 Aug 03:17:17.771 # argv[1]: '0'
19621:P 16 Aug 03:17:17.771 # argv[2]: 'COUNT'
19621:P 16 Aug 03:17:17.771 # argv[3]: '1000'
19621:P 16 Aug 03:17:17.771 # argv[4]: 'QUEUE'
19621:P 16 Aug 03:17:17.771 # argv[5]: 'queue16'
19621:P 16 Aug 03:17:17.771 # --- REGISTERS
19621:P 16 Aug 03:17:17.771 #
RAX:0000000000000000 RBX:00007ffc5ececdc0
RCX:0000000000000000 RDX:0000000000000007
RDI:0000000000000000 RSI:00007f7cca414aa0
RBP:00007f7cca416680 RSP:00007ffc5ececcf0
R8 :aaaaaaaaaaaaaaaa R9 :000000000000001f
R10:000000000000000f R11:00007f7ccadfb870
R12:00007f7cca41c430 R13:00007f7cca410940
R14:00007f7cca49a000 R15:000000000000001f
RIP:00000000004224d4 EFL:0000000000010206
CSGSFS:0000000000000033
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcff) -> 00007f7cca49a000
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcfe) -><PHONE_NUMBER>000006
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcfd) -> 00007f7cca41c3d0
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcfc) -> 00007f7cca41c430
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcfb) -><PHONE_NUMBER>0026fc
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcfa) -> 00007f7cca41c430
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcf9) -> 00007f7cca49a000
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcf8) -><PHONE_NUMBER>000006
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcf7) -> ffffffffffffffe0
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcf6) -><PHONE_NUMBER>000030
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcf5) -><PHONE_NUMBER>412f98
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcf4) -><PHONE_NUMBER>000005
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcf3) -> 00007ffc5ececdb0
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcf2) -><PHONE_NUMBER>432090
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcf1) -><PHONE_NUMBER>4320b0
19621:P 16 Aug 03:17:17.771 # (00007ffc5ececcf0) -> ffffffffffffffff
19621:P 16 Aug 03:17:17.771 # --- FAST MEMORY TEST
19621:P 16 Aug 03:17:17.771 # Bio thread for job type #0 terminated
19621:P 16 Aug 03:17:17.771 # Bio thread for job type #1 terminated
Testing<PHONE_NUMBER>6
Testing<PHONE_NUMBER>68
Testing 7f7cc93ff000 8388608
Testing 7f7cc9c00000 16777216
Testing 7f7ccb035000 20480
Testing 7f7ccb254000 16384
Testing 7f7ccb769000 16384
Testing 7f7ccb77c000 16384
Testing 7f7ccb782000 4096
19621:P 16 Aug 03:17:18.173 # Fast memory test PASSED, however your memory can still be broken. Please run a memory test for several hours if possible.
19621:P 16 Aug 03:17:18.173 #
=== DISQUE BUG REPORT END. Make sure to include from START to END. ===

       Please report the crash by opening an issue on github:

           http://github.com/antirez/disque/issues

  Suspect RAM error? Use disque-server --test-memory to verify it.

[1]    19621 segmentation fault (core dumped)  ./src/disque-server

I was able to reproduce consistently. It fails after multiple ackjob are issued for the same job.
<IP_ADDRESS>:7711> addjob queue1 job1 0
DId1dbbca16b854c7ec98332a4019bcbf143791c3f05a0SQ
<IP_ADDRESS>:7711> ackjob DId1dbbca16b854c7ec98332a4019bcbf143791c3f05a0SQ
(integer) 1
<IP_ADDRESS>:7711> jscan 0 queue queue1
1) "0"
2) (empty list or set)
<IP_ADDRESS>:7711> ackjob DId1dbbca16b854c7ec98332a4019bcbf143791c3f05a0SQ
(integer) 0
<IP_ADDRESS>:7711> jscan 0 queue queue1
Could not connect to Disque at <IP_ADDRESS>:7711: Connection refused

Hi @seppo0010 , there is a path may cause this. :-)
ack.c-L216-217 job.c-L178 when there is no job of that ID(already ack and delete in your case), ackjobCommand create a dummy job, but the job is created with its queue as NULL.
Then job.c-L1416 use the NULL(job->queue) in equalStringObjects, so there is the segfault.

Try to fix it in https://github.com/antirez/disque/pull/114 :-)

Just confirming that I encountered the same error in my tests. It only happens when using the JSCAN command after sending a duplicate ACK.

Thanks, going to merge @sunheehnus fix between today and tomorrow. Tomorrow I'll work all day to Disque btw so I'll check all the other issues. Thanks.

p.s. would be great to get ACKs about @sunheehnus fix being enough to stop the problem in @Koed00 and @seppo0010 tests.

Yes, I guess this situation is more likely to happen in a one-node cluster :-)

I have some time to run the pull request against my tests. Will report back.

Thanks!

Works great. Tests are passing again. Carry on.

:fireworks:


[Doc] Document required explaining how user can set up proxy URL for Harvester UI to access UI from outside the world.

Document required explaining how user can set up proxy URL for Harvester UI to access UI from outside the world.

@khushboo-rancher Is this still valid?

Not valid, we have https://docs.harvesterhci.io/v1.0/settings/settings/#http-proxy in place already.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.10 to before v5.15.12",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=b94e02822debdf0cc473556aad7dcc859f216653",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=c691e7575eff76e563b0199c23ec46bd454f43e3"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "b94e02822debdf0cc473556aad7dcc859f216653",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "c691e7575eff76e563b0199c23ec46bd454f43e3",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "mm/hwpoison: clear MF_COUNT_INCREASED before retrying get_any_page()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.12 by commit c691e7575eff76e563b0199c23ec46bd454f43e3, it was introduced in version v5.10 by commit b94e02822debdf0cc473556aad7dcc859f216653. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002837


Improve image loading/performance

Let's be honest, it's HORRIBLE when you visit a page and there is a bunch of images flashing on the screen after they load, and that's precisely what happens on TechSchool today. We need to find a more performant way of loading the images or maybe show some "loading skeleton" on the image while it loads (this could be a new component for image, that is reused on the entire website)

@danielbergholz Hey, I can take this one!

Images for courses are loading faster without lazy loading! It's almost impossible without throttling to figure out "adopt liveview" loaded latter, than the others.

Images for courses (youtube videos) are loading blazing fast. I think, this is possible because of Google CDN, which gives fast speed for the most places around the world. (magic link https://i.ytimg.com)
Now I am looking for a way to hook the moment, when image ends loading and show it instead of loader/skeleton. It might be useful for low connection speed (see loading with throttling).

Super interesting! Thanks for the insights! I'll assign this issue to you then, feel free to ask any other questions here.
And feel free to remove the lazy loading for the courses images 🤝

I fixed this by using inline SVGs instead of an img tag. This way the icon is already rendered when the page loads and it reduces the load on the server to serve multiple images.
See commit https://github.com/danielbergholz/techschool.dev/commit/f8ebe995b101379402e5e6af5bc3acae9a4d51f9


Forcing restart of process?

Just wondering how to restart the VSCode Julia process for testing? For example if I add a new dependency; how can I restart Julia? (Right now I have to reload the VSCode window which seems to achieve this)

The problem is a duplicate of https://github.com/julia-vscode/julia-vscode/issues/3022. That issue is fixed in https://github.com/julia-vscode/julia-vscode/pull/3609, but that is still a bit away from being ready.
You can terminate any test process from the Julia Workspace view. But that won't solve the underlying problem, as we still have a cache of the outdated env hanging around, so even the restarted test process will not be correct. Until I merge that PR mentioned above, you'll have to restart VS Code whenever anything changes in the env...


feat: Add validate command to verify configs

Useful for testing automation

Might not actually need this...

Didn't need it


ignore zero-length edges for default epsilon

possible fix for #5263

Thanks!
Can you add a regression test as well? An example you reported in gh-5263 would be good. Also, a test with d=3 would be nice bonus to check that implementation does not assume 1d or 2d.
What about stability. How does your fix (and the master branch) fare if the points are nearly collinear?

The PR which last changed the value of default epsilon: gh-4643.
cc @jjgreen

I'll take a look at this and add tests. Playing around, nearly collinear points give small epsilon (as expected), but what this does to the interpolation seems to depend a lot on the shape function used. The stability properties here are a bit out of my depth.

Maybe just try to use collinear points + small noise and check that the result is sensible? Maybe something like this can even turned into a test.
I'd also ping  @arokem: you seemed to have a stake in the rbf implementation here?

I wrote a simple regression test in c3a10c4c71aa068ae1b9cddafd7a8a6d49b0e29f for the perfectly collinear case.
When points are nearly collinear and epsilon is small, the multiquadric basis returns reasonable results (turns into linear interpolation). The gaussian and inverse shapefunctions break down because the basis becomes very "narrow".
I have not yet turned this into another test because I'm not sure what would actually be tested.

Thanks for pinging me about this. I think this will be fine for our use-case - we've decided to go with determining epsilon before calling this, so we will not be affected by this change.

Thank you @arokem
OK, it seems to me that (i) our RBF implementation can really use a face-lift, (ii) users are best off managing their values of epsilon themselves,  (iii) we should provide a sensible default if only for backwards compat. There are likely better ways of doing it, but using a bounding hypercube is probably an OK default, and this PR is a bugfix for an otherwise OK behavior. In view of (ii) we can probably leave an issue with nearly collinear points. Therefore, I'm +1 for merging it and backporting to 0.16.x (assuming 0.16.1 is going to happen).
One last minor thing to do before merging this PR: can you squash the four commits into one @njwilson23? Ideally, the commit message would start with BUG (as per http://docs.scipy.org/doc/numpy/dev/gitwash/development_workflow.html).

Point about the test taken. I've also squashed the commits and pre-pended BUG to the commit message.

Merging, thank you @njwilson23. Keep them coming!


Data deletion service now applies deletes to correct database.

This made me realize that the re-partitioning job shouldn't be run for entity sets not in the default datastore.


/usr/bin/env: ‘node\r’: No such file or directory error

Quick question, does the package run only on Windows so far?
When I try to execute it on a Linux machine (Ubuntu), I got the following error:
/usr/bin/env: ‘node\r’: No such file or directory
error Command failed with exit code 127.

To fix it temporarily, I just have to edit the cli.js file to replace CRLF with LF in the first line and then it works great.
But it means currently I can't easily integrate it in our project for all my developers + I'd like to test it in our CI :-)

@devsalsa I still haven't tried it on Linux yet. I'll try to address this issue, in the meanwhile, if you have a quick solution, feel free to open a PR

@tnicola new release when you have time please :pray:

Ok, my bad. I think the problem was publishing directly from my local with the wrong eol. Check out the 0.2.2 version

Yep you fixed the problem :-) thank you


Packman should not fail if proto source is not under a google dir

Currently, packman seems always to assume that there is a google/{api-name} directory under both --include_path and --services_root. We should not be making this assumption.
To reproduce:
gen-api-package --api_name=myapi/v1 -l python -o ../artman/output -r ../myapi/v1 -i ../googleapis/other
Expected result:
Runs protoc on ../myapi/v1
Observed result:
Tries to run protoc on ../myapi/v1/google/myapi/v1 and fails because no such directory exists.

@tbetbetbe Do you know the cause for this? A hack is necessary in https://github.com/googleapis/artman/pull/45 to work around this.

Note: the cause for this appears to be here: https://github.com/jmuk/packman/blob/08659e628eb7689504099d949b081de32c07943e/bin/gen-api-package#L275
It should be possible to disable this via a flag. Alternatively, we could just leave this set to false in general, and have it only be set via the existing -g flag.

cc: @jmuk


Make AUCPR work with multiple query groups

Fixes #4431.
I read the paper referenced by the AUCPR implementation, and as far as I can tell, the EvalAucPR seems to be consistent with the paper.
The issue was that EvalAucPR was assuming the existence of only a single query group. More precisely, it throws an exception when sum_auc >= 1.0, where it should have checked sum_auc >= ngroup instead. Since all tests for AUCPR was performing binary classification (where there is only one group), the bug went undetected.
Fix. Have a proper check on the AUC value, in presence of multiple query groups. Also add a test with learning-to-rank example.

I'm merging this, since it involves the change of a single assertion and doesn't change the actual implementation of AUCPR.


Updater: "row index is bigger than sorter's row count"

Every time I launch the updater in either Fiji or ImageJ (under macOS X with the Java 10.0.1 JDK) I get the following error in the Console:
WARNING: row index is bigger than sorter's row count. Most likely this is a wrong sorter usage.
It doesn't seem to have much of an impact on the updater though.

Thanks @cortig; I have noticed that error as well. Haven't had time to track down where it's coming from yet. Good to have an issue here as a reminder for later.

I find this message in one of my projects as well, but only if I start the application in Java 9. I have tried to find the cause, but with no luck. As this doesn't appear in Java 8, I expect this to be a bug in Java 9.

Hello, I had same problem in my application after changing table data, solved it by adding :
itemTable.getRowSorter().allRowsChanged()


Farming modal + dark: no text

(white text)

It's depended on Farm repository (not our styles). Need to change it there


Foundation expense - new campaign - expense does not exist?

I submitted two expenses last night, but upon trying to view the details today it shows an error stating that the expense does not exist.

I can see both polls on linux, Gridcoin <IP_ADDRESS>
[
{
"Polls" : "Polls Report new_campaign_rewarding_past_14_gridcoin_community_hangouts[foundation_be116841-f8bc-44d9-9b86-68159d0ac2c8]",
"Poll #1 (11-28-2016 23:08:55 ) - Magnitude+Balance" : "new_campaign_rewarding_past_14_gridcoin_community_hangouts[foundation_be116841-f8bc-44d9-9b86-68159d0ac2c8]",
"Question" : "Approve_Campaign?",
"#1 [3.000]. Approve" :<PHONE_NUMBER>7084,
"#2 [3.000]. Deny" :<PHONE_NUMBER>7470,
"Participants" : 6.00000000,
"Total Shares" : 1740471.04224555,
"Best Answer" : "Approve"
}
]
]
[
{
"Command" : "listpollresults"
},
[
{
"Polls" : "Polls Report new_campaign__gridcoin_community_hangout_#015_distribution[foundation_de853ded-19e9-4a93-840b-82a5f469d7f2]",
"Poll #1 (11-28-2016 23:04:21 ) - Magnitude+Balance" : "new_campaign__gridcoin_community_hangout_#015_distribution[foundation_de853ded-19e9-4a93-840b-82a5f469d7f2]",
"Question" : "Approve_Campaign?",
"#1 [6.000]. Approve" : 27445913.81635790,
"#2 [2.000]. Deny" :<PHONE_NUMBER>8114,
"Participants" : 8.00000000,
"Total Shares" : 28137982.67513903,
"Best Answer" : "Approve"
}
]
]

I've resubmitted the two expenses due to the 'Sorry, this expense does not exist' glitch. The new expense attempts have 'fixed' in their title.

The resubmitted 'fixed' expenses exhibit identical behaviour.
I've crated a new expense using the 'expense reimbursement' option rather than 'new campaign' & is at least showing on gridresearchcorp: http://www.gridresearchcorp.com/gridcoin/?votedetail&t=Poll_Detail&opt=2&pname=Expense_Submission3rd_time_lucky_-_1_to_15_Gridcoin_community_hangout[Foundation_13b023ee-6fed-4a03-8894-81eded311d1a]&q=Approve_Expense?
I'll verify if I can view the details later.

https://cryptocointalk.com/topic/1331-new-coin-launch-announcement-grc-gridcoin/page-1198#entry227909
Just got verification that this 3rd attempt has failed, I'm unable to submit foundation expenses.. :-1:

This is a continued issue, is there a trick to get around this issue?

I wish I knew. But yours and startail's votes are OK, so there's some improvement. Mine two are both borked.

I just added one, so lets see if its successful in 6 blocks.
GRCTEST:  You must add the expense or campaign from a machine with port 1433 open, you must have an attachment attached, and it must be from a machine where you installed gridcoin from the MSI.
I see others have successfully added expenses other than me, so its sounding more like your firewall.

Expense submission test opens correctly on my wallet. Gridcoin is installed from the MSI package on my machine and it's allowed through firewall for all outbound traffic. Port 1433 was indeed closed for inbound, I'll make sure it's open next time.

In my latest foundation expense, I removed special characters from the title and text box, perhaps that was causing an error in the past? I also did not select start/end dates. Two possible areas of further investigation.

Great! I guess we will close this then.

One of my latest expenses for the Gridcoin community hangouts has experienced this glitch again :(


Should not be able to add "base" product to cart

This template is to be used for bug reports and small enhancements. For feature requests, please contact the project owner.
Note: For the bug to be accepted, it must be reproducible using the latest release of Spartacus. See our Contributor Documentation for more information.
Environment Details

Spartacus:
Browser:
OS:
Device:
Mac/Safari
Other environment details:

Steps to Reproduce

Open category "Compact Cameras", and click product "PowerShot A480"

In "You may also like...", click the first product

try to add this product to cart; the "add to cart" popup overrides the error message

close the popup, we can see this:

This product is a "base" product

we don't support multi-dimensional products in first release

It might be related to #4821

Couldn't reproduce the problem.
There are two types of "ngIf" conditionals controlling visibility of "Add to cart" button.

PLP - for list view there is single information to check - stockLevelStatus. There is no check for stock number nor this information is coming from backend.
PDP - gets more detailed checks with stockLevelStatus and stock number available. Although for PDP we get stock number which is absent for most PLP products.

This causes an error, because backend sends incomplete information for PLP items.

After more investigating the issue, I believe the current problem lies in the backend side:
For endpoint: /products with parameters fields=products and stock(FULL) response with list of products not always contains field stockLevel, which should be always returned, even when value is 0.

Generally speaking, a base product cannot be ordered and should therefor not have any stock assigned to it. There is however some parts in the sample data where stock has been added for base products. We should fix the sample data in this case.

Sample data PR: https://github.tools.sap/cx-commerce/spartacussampledataaddon/pull/60


Cost for penalization of impulse velocity

This task refers to write the c++ code of https://github.com/loco-3d/crocoddyl/blob/devel/unittest/python/crocoddyl/impact.py#L245.
And it also includes to add Python bindings + unittest code.
it requires to be solved first #615

duplicated in #285


Mods get deleted during BepInEx update

Everything inside data/plus/BepInEx/plugins gets deleted when the docker image auto updates the BepInEx, path data/plus/BepInEx/plugins should be excluded.

Have you checked the Readme?
Says to put your mods here: /config/valheimplus/plugins/


Use ocp base image as last build layer

Ref. https://issues.redhat.com/browse/ART-6289

/lgtm
/approve


Improved Code Snippets across Mobile SDK's docs

What do these changes do?

Improved Code Snippets across Mobile SDK's docs.

Type of documentation

[ ] New documentation
[x] Documentation update
[ ] Hotfix

Notion ticket link

Add a link to the Notion ticket.

As discussed, @desusai7 and team will verify all the code snippets across the SDK docs.

Will revisit in the new docs framework if necessary


Figure out where javadocs can be hosted, explored and link it in a badge on the Readme

This exists now at https://hashgraph.github.io/hedera-sdk-java/


build_prepare enhancements

check for changed branches in cloned github repos
separte default from local script
move code to lib

more comfort for chacksums


[Transform] Support mobile/touch input

https://www.layerchart.com/docs/components/Transform
Might need to think on how best to opt-in to this (if you're scrolling down the screen, hitting a chart/map and immediately scrolling can be frustrating, especially if there isn't enough of an edge on either side to use to continue to scroll.
Sometimes maps/etc require multiple fingers to active the scrolling, for example.
discussion

Mobile support should be handled when we switch to pointer events (#144) although this issue will be used to improve how it integrates on mobile (opt in, require 2 fingers to drag, etc).  Also, implementing pointer-based pinch/zoom gestures instead of relying on mousewheel events

Pointer events implemented and also integrated Transform into Chart as TransformContext.
Still need to implement pinch/zoom gesture


Numpy-like interaction with tensors

As our random variables are now tensors, manipulations that users of pymc3 are familiar with might no longer be valid, requiring them to use tensorflow's particular syntax for manipulating tensors. I might be wrong, but advanced indexing of numpy arrays seem to no longer work and requires tf.gather_nd. Is this deviation from pymc3-like syntax desirable?

I dont think there is good workaround for that as we rely on Tensorflow for tensor operation.

I wasn't aware that advanced indexing is not supported by TF, that's a bummer. It's the only thing were non-numpy syntax would be required, however, or am I missing something else?

Hmmm.... @junpenglao & @twiecki I am commenting from the dark on this, but jax devs explicitly want to use the NumPy API - any chance we could experiment with that? What would be needed - a re-implementation of the probability distributions and bijectors are the ones I can see at the moment - but any more?
The numpy API is idiomatic, which is why I like the tensor libraries that strive to use its API rather than invent its own. CuPy and jax devs have either a formal or an informal policy that strives to use the NumPy API, but TF and PyTorch devs seem to half-heartedly do so.

@csuter Any thoughts on this discussion?

There's some historical context here, and (substantially) more here. In particular, @shoyer points to a NEP discussing a proposal to work around shortcomings of numpy advanced indexing. I know nothing about the details of why TF doesn't have numpy-style advanced indexing but it's built and designed by a large group of smart folks so a reasonable prior is that there are at least some good reasons :)
A library of distributions and bijectors built on jax would be awesome, but would also be a ton of work.

it's built and designed by a large group of smart folks so a reasonable prior is that there are at least some good reasons

Won't deny that there's definitely a large group of smart folks! But we all know about what happens to a broth when there's many chefs involved :smile:, particularly smart (and likely) opinionated ones!
Jokes aside, @csuter, thanks for providing the context on indexing. I see the issues with numpy indexing now.
Apart from numpy-like indexing, though, I had issues with both TF and PyTorch's use of non-exact function naming (e.g. tf.reduce_sum rather than tf.sum), which makes it tedious (though not difficult) to replace old NumPy code with TF code. If exact function names were copied over, I would have moved my codebase to TF in a heartbeat. Imagine import tensorflow as np! For this reason, Jax has been attractive for writing models, because a lot of the old autograd.numpy code I wrote can be ported over by a single import change.
I have other questions, perhaps I'll raise it in a different issue.


Autorization

Issue moved from dotnet/docs#22269

Please respond to @kwaazaar.

From @kwaazaar on Friday, January 8, 2021 11:55:31 AM
Authorization on package sources is always an issue. You should describe here how it works. Especially in conjunction with Visual Studio. Eg: in VS the restore of packages from a custom Azure Devops artifact store works flawless, since I'm logged in in VS. When using dotnet restore, I get an authorization error. It seems to use the sources registered in VS, but not the credentials.

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

ID: 0b256dc4-c4ad-c115-f4e2-40310bc3505a
Version Independent ID: 0b993779-65a2-6092-4047-2fc2ec16170b
Content: dotnet restore command - .NET CLI
Content Source: docs/core/tools/dotnet-restore.md
Product: dotnet-fundamentals
Technology: dotnet-cli
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

Issue moved from dotnet/docs#22269

Please respond to @tdykstra.

From @tdykstra on Monday, January 11, 2021 6:35:43 PM
@nkolev92 Should this issue be transferred to the NuGet docs repo?

Issue moved from dotnet/docs#22269

Please respond to @tdykstra.

From @tdykstra on Monday, January 11, 2021 6:35:43 PM
@nkolev92 Should this issue be transferred to the NuGet docs repo?

Issue moved from dotnet/docs#22269

Please respond to @nkolev92.

From @nkolev92 on Monday, January 11, 2021 6:42:55 PM
It's a multi repo/product problem, but is a great starting point.
https://docs.microsoft.com/en-us/nuget/consume-packages/consuming-packages-authenticated-feeds has answers for some of the customers' questions, but not all.

Issue moved from dotnet/docs#22269

Please respond to @nkolev92.

From @nkolev92 on Monday, January 11, 2021 6:42:55 PM
It's a multi repo/product problem, but is a great starting point.
https://docs.microsoft.com/en-us/nuget/consume-packages/consuming-packages-authenticated-feeds has answers for some of the customers' questions, but not all.

Issue moved from dotnet/docs#22269

Please respond to @tdykstra.

From @tdykstra on Monday, January 11, 2021 6:55:29 PM
We already link to https://docs.microsoft.com/en-us/nuget/consume-packages/consuming-packages-authenticated-feeds in our doc, can you recommend who might know the answers to what isn't covered there?

Issue moved from dotnet/docs#22269

Please respond to @tdykstra.

From @tdykstra on Monday, January 11, 2021 6:55:29 PM
We already link to https://docs.microsoft.com/en-us/nuget/consume-packages/consuming-packages-authenticated-feeds in our doc, can you recommend who might know the answers to what isn't covered there?

Issue moved from dotnet/docs#22269

Please respond to @nkolev92.

From @nkolev92 on Monday, January 11, 2021 7:20:56 PM

can you recommend who might know the answers to what isn't covered there?

The NuGet team can answer them.
There are different plugins for core and framework tools (ie, dotnet.exe vs nuget.exe & msbuild.exe).
VS also uses a different extensibility mechanism, and all these extensibility plugins/tools are expected to do their own caching.
It's on the feed provider to ensure that the plugin they ship for nuget.exe, the plugin they ship for dotnet.exe, and the VS extensibility share a cache.

Issue moved from dotnet/docs#22269

Please respond to @nkolev92.

From @nkolev92 on Monday, January 11, 2021 7:20:56 PM

can you recommend who might know the answers to what isn't covered there?

The NuGet team can answer them.
There are different plugins for core and framework tools (ie, dotnet.exe vs nuget.exe & msbuild.exe).
VS also uses a different extensibility mechanism, and all these extensibility plugins/tools are expected to do their own caching.
It's on the feed provider to ensure that the plugin they ship for nuget.exe, the plugin they ship for dotnet.exe, and the VS extensibility share a cache.


fullcalendar <IP_ADDRESS>

Hello I have this error when I try to update fullcalendar. do you have any solution please?
TypeError: context.cmdFormatter is not a function

What version of fullcalendar do you use?
@fulcalendar/core ?
@angular/core ?
see demo: https://github.com/ng-fullcalendar/ng-fullcalendar2-demo

thanks ! i use angular<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Use<EMAIL_ADDRESS>and @fullcalendar/core 4.0.1

I have the bug with these version<EMAIL_ADDRESS>and @fullcalendar/core 4.0.1! can be a conflict with momentjs ?

show me your issue in stackblitz


Update 3.18.0 -> 3.20.0 breakes with undocumented breaking change

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
3.18.0 -> 3.20.0
Affected resource(s)
provider configuration
Terraform configuration files
provider "cloudflare" {
  email     = var.cloudflare_email
  api_token = var.cloudflare_api_token
}

Debug output
$ terraform validate
╷
│ Error: Missing required argument
│
│   with provider["registry.terraform.io/cloudflare/cloudflare"],
│   on providers.tf line 30, in provider "cloudflare":
│   30:   email     = var.cloudflare_email
│
│ "email": all of `api_key,email` must be specified

Panic output
No response
Expected output
At most a warning, that this is deprecated.
Actual output
$ terraform validate
╷
│ Error: Missing required argument
│
│   with provider["registry.terraform.io/cloudflare/cloudflare"],
│   on providers.tf line 30, in provider "cloudflare":
│   30:   email     = var.cloudflare_email
│
│ "email": all of `api_key,email` must be specified

Steps to reproduce

Specify provider as given above
User provider version 3.20.0
Run terraform init
Run terraform validate

Additional factoids
It is documented in the current docs that api_token conflicts with email. However this was only added recently and it should be part of the breaking changes in the release note.
Apart from that I think that there should be no breaking changes in minor versions. As you do not comply with semantic versioning, can anybody tell me what kind of versioning system is used? It would be also helpful to have that as part of the documentation, so people can configure there auto update tools accordingly.
References
No response

The debug output is so not relevant for this bug.

see #1768


installation guide verification

Installation guide worked well on my brand new Mac!
Except the verification:
python -c 'import pycsep'
ModuleNotFoundError: No module named 'pycsep'
But: the workshop tutorials run great in Jupyter nb.

also add note to wiki page about the conda environment


Text Message from iOS to Android is empty text

We are trying to integrate ChatSDK in our iOS & Android Apps.
for iOS we are using SDK Version -> 4.14.5
for Android we are using SDK Version -> 4.8.17
Here is the following code snippets we are using to send text message
iOS : BChatSDK.thread()?.sendMessage(withText: "Hello", withThreadEntityID: thread.entityID())
Android : ChatSDK.thread().sendMessageWithText("Hello",thread)
Message sent from iOS doesn't have json_v2

Hi @ourdex86 the first thing I would recommend if possible, is that you use version 5.0.4 of the Android project. Whether this is possible would depend on how far through you are with the integration. But version 4 of the Android project is deprecated and won't be updated any further. It will only see minimal bug fixes.
In terms of the message payload, are you seeing any problems receiving messages between the two platforms? The json_v2 payload is a legacy item that is no longer used because the data is stored in the meta branch. It is only included for backwards compatibility with old versions of the project.

With the json_v2 issue, I've added a fix which is available with 4.8.18 but I would still recommend using v5 if possible because it will offer far better performance in the future.

With the json_v2 issue, I've added a fix which is available with 4.8.18 but I would still recommend using v5 if possible because it will offer far better performance in the future.

@bensmiley .. I tried to use v5, but that is using old org.jetbrains:annotations  version. Its using strict 13.0

@ourdex86 can you show me where it is using this annotation?

Hi @bensmiley. we're having this kind of issue as well but as you said you have fix v4.8.18 but I can't find it in the tags. Also, you said there was version 5.0.4 but the tags only have v5.0.0rc21 and the latest version base on the commit is 5.0.2. Question is, where is version 5.0.4?

You could install it using Gradle? The latest version is: 5.0.16. Otherwise, you could try using the master version in Github.


fix(chunk): direct upload

Checklist

[x] I have read the coding guide.
[ ] My change requires a documentation update, and I have done it.
[ ] I have added tests to cover my changes.
[ ] I have filled out the description and linked the related issues.

Description
Closes #3227

This change is

The commit message is not correct. The part in the () needs to be the package we are changing. chunk is not the correct name. It would be better to use nothing if you are not sure. But ideally this should be api package change.

So yes this problem occurs in /bytes and /bzz endpoints but not sure about /feed since it requires an Ethereum address as a param and in dev mode, we don't have any address.

So yes this problem occurs in /bytes and /bzz endpoints but not sure about /feed since it requires an Ethereum address as a param and in dev mode, we don't have any ethereum address.

The /feed requires a feed address, that indeed is the same format as the ethereum address (because it uses the same elliptic curve cryptography to generate addresses), but it is independent of the node's own ethereum address. In fact any valid ethereum address can be used as a feed address.

So yes this problem occurs in /bytes and /bzz endpoints but not sure about /feed since it requires an Ethereum address as a param and in dev mode, we don't have any ethereum address.

The /feed requires a feed address, that indeed is the same format as the ethereum address (because it uses the same elliptic curve cryptography to generate addresses), but it is independent of the node's own ethereum address. In fact any valid ethereum address can be used as a feed address.

Thanks So /bytes , /bzz and /feed are giving bad request  in response and they don't hang the request like in /chunk endpoint. Soo just going to add message dev node not supported


(doc) Update README.md

Fix formatting in table of options - filter row

Thanks for spotting that! Would you mind renaming the commit message (doc) -> (docs) ? Just for my OCD 😄

I meant the Git commit message (with git commit --amend), thanks!

Is it OK now? :)

Seems fine, thanks for contributing :)


Broken Dependency Integration

When trying to configure the CMake-Project, the process comes to a hold.
This is due to CMake not finding libusb via find_package.
The error was found when compiling on Ubuntu by @R-Abbasi, but could also be reproduced in CI (https://github.com/skunkforce/omniview/actions/runs/7232145540/job/19706039412#step:9:1244)

After investigation, it was concluded, that CMake couldn't find libusb, since it was missing a .cmake file as is required for Config Mode according to the CMake Documentation:

In this mode, CMake searches for a file called <lowercasePackageName>-config.cmake or <PackageName>Config.cmake. It will also look for <lowercasePackageName>-config-version.cmake or <PackageName>ConfigVersion.cmake

Right now I don't know, whether this file is gone due to changes in libusb or vcpkg. tree shows the following in the vcpkg/packages:
maxclerkwell@SURFnTERF ~/r/omniview (master)> tree vcpkg/packages/libusb_x64-linux/                                                    (base) vcpkg/packages/libusb_x64-linux/
├── BUILD_INFO
├── CONTROL
├── debug
│   └── lib
│       ├── libusb-1.0.a
│       └── pkgconfig
│           └── libusb-1.0.pc
├── include
│   └── libusb-1.0
│       └── libusb.h
├── lib
│   ├── libusb-1.0.a
│   └── pkgconfig
│       └── libusb-1.0.pc
└── share
    └── libusb
        ├── copyright
        ├── usage
        ├── vcpkg.spdx.json
        └── vcpkg_abi_info.txt

9 directories, 11 files

The installed libusb version by vcpkg appears to be from 10-2022.
maxclerkwell@SURFnTERF ~/r/omniview (master)> cat vcpkg/packages/libusb_x64-linux/CONTROL                                              (base) Package: libusb
Version: <IP_ADDRESS>91
Port-Version: 6
Architecture: x64-linux
Multi-Arch: same
Abi: 374c198801d491bee24730fa65832c736702021236557405bee9e735ea3c899b
Description: a cross-platform library to access USB devices
Default-Features: udev

Package: libusb
Feature: udev
Architecture: x64-linux
Multi-Arch: same
Description: Enable udev

Release Message libusb-1.0.26

The last successful run appeared on 2023-12-12, which was the exact day of a new vcpkg release.
In that release which can be found here: https://github.com/microsoft/vcpkg/releases/tag/2023.12.12 a minor change happend.
Looking at the "updated ports" libusb changed <IP_ADDRESS>91#3 to <IP_ADDRESS>91#5.
I have no clue why, but this looks like it broke our pipeline.

There are different approaches I see here:

fix the problem by using

target_include_directories(OmniView PUBLIC "${vcpkg_SOURCE_DIR}/packages/libusb_x64-linux/include")
target_link_directories(OmniView PUBLIC "${vcpkg_SOURCE_DIR}/packages/libusb_x64-linux/lib")

fix the problem by using pkg-config

find_package(PkgConfig REQUIRED)
pkg_check_modules(libusb REQUIRED IMPORTED_TARGET libusb-1.0)

target_link_libraries(main PRIVATE PkgConfig::libusb)

fix the problem by making an issue in libusb Repository.

Modifing CMakeLists.txt to clone the 2023.11.20 Release of vcpkg makes CMake happy:
FetchContent_Populate(                                                                                vcpkg                                                                                GIT_REPOSITORY https://github.com/microsoft/vcpkg.git                                                                                GIT_TAG        2023.11.20                                                                                SOURCE_DIR     "${CMAKE_SOURCE_DIR}/vcpkg"                                                                                )

I can't tell which unintended consequences this might have for the future, but this seems to be a possibility to mitigate the problem.


Consonant - Marquee

Story
As a Helix author, I would like the ability to make a Consonant Marquee so that I can create branded experiences using a Helix block.
Design brief
Criteria of Acceptance

Can make small, medium, large, quiet, and inline variants
Can leverage solid, gradient, and image backgrounds
Supports detail, title, and body copy
Supports single or multiple actions
Meets all internal accessibility requirements
Meets all internal SEO requirements

Sub-tasks

[ ] Verify design
[ ] Content review
[ ] UAT review
[ ] Analytics review
[ ] Code review
[ ] Accessibility review
[ ] SEO review

@auniverseaway I'm tracking this ticket with the Nova team here MWPW-101551. Sounds like we'll have a meet about this and other Consonant/Helix work with you and Ryan P and JFunk too. :)

Closing this. The source of truth going forward is in JIRA.


Add test for correct transactionRoot

Rel https://github.com/paritytech/frontier/issues/180

Checks an empty transaction list matches Keccka256(56e81f171bcc55a6ff8345e692c0f86e5b48e01b996cadc001622fb5e363b421).
Move calculated sha3Uncles hash to its own step.

This PR is expected to fail until transactions_root is correctly calculated.

Going to combine this and #191 for ease of merging.

Great, did you run the tests? I was pulling from sp-transaction-root to check if they are ok.

Closing, fixed in https://github.com/paritytech/frontier/pull/191


Issue 62, Flash Calibration sector must be reserved in ld linker script.

This should be done to ensure that write of calibration data will not
clear sections of the program data (.text etc).

Testet på SaltFlow da DC/AC og Temperatur ikke bruger skriv af calibration data.

Only comment it that code could be moved to last 128 k area.


Thank you.

I'm so happy to found this project.
Now I transfer my nextcloud from KVM/Qemu to Docker.
The last step step for me is, to include gitea.
That's not so easy for me, but I will try it more.

Welcome!
Thanks for using it.

cant emphasize thank you enough................


Use lazy loading

Summary of change
Lazy load authors in book and books in author
Related issue
Closes #20
Pull request checklist
Please keep this checklist in & ensure you have done the following:

[x] Read, understood and adhered to our contributing document.

[x] Ensure that you were first assigned to a relevant issue before creating this pull request
[x] Ensure code changes pass all tests

[x] Read, understood and adhered to our style guide. A lot of our code reviews are spent on ensuring compliance with our style guide, so it would save a lot of time if this was adhered to from the outset.

[x] Filled in the summary, context (if applicable) and related issue section. Replace the square brackets and its placeholder content with your contents. For an example, see any merged in pull request

[x] Created a branch that has a descriptive name (what your branch is for in a few words and includes the issue number at the end, e.g. test-reading-goal-123

[x] Set this pull request to 'draft' if you are still working on it

[x] Resolved any merge conflicts

For any of the optional checkboxes (e.g. the screenshots one), still check it if it does not apply.
If in doubt, get in touch with us via our Slack workspace

@knjk04 let me if any change is needed

@knjk04  added test cases have look
while writing test cases, I found one issue with Book constructor we are checking not null we should also check list should not be empty if does we should throw IllegalArgumentException book must have at least one author

@knjk04 I have made suggested changes have look let me know if I missed anything I will fix it


nodeos high CPU usage

The nodeos node(non-producing node) constantly consumes 100% CPU, transactions send to that node are very likely to fail.
The system calls looks as follow:

After restart nodeos, CPU usage drops to 10% to 20%,

what's your CPU model and what's your machine config?

The machine I used:

CPU: 3.2 GH Intel Xeon E5-2667 v4（Broadwell)
memory: 32 G

By saying "machine config", are you referring to how I configed the nodeos?

Actually, we got the same situation. We host the nodeos on Google cloud platform and the process CPU usage is always 100%. Is there anything we can tweak to help?

I ruled out following possible reasons:

nodeos version
peer connections
HTTP request traffic
chain state size

and finally it seems the problem was caused by malfunctioned hardware: after migrating to a new machine, CPU usage steadily maintains at 5% to 10%.


fix: "Solve the problem that the resources have not been fully released due to early exit"

@asim

There's a bunch of random go mod changes and the build is broken

There's a bunch of random go mod changes and the build is broken

It's not mod, it's because I exposed the run error

@asim  It's solved


Nested async components error when switching components

Hello, please take a look at the following two comments.
https://github.com/sveltejs/svelte/issues/3448#issuecomment-530331354
https://github.com/sveltejs/svelte/issues/3448#issuecomment-541386356
I think as mentioned here:

In the end @halfnelson came up with a solution where the pre-built component should declare at least the svelte/internal imports as globals (rollup) or externals (webpack).
You can see that comment and my comment afterwards stating what I did here: #3671 (comment)
If this is the problem that's being discussed here then it would seem that projects distributing pre-built Svelte components should set their bundles to be built expecting at least the svelte/internal import to be provided as a global somewhere.

I guess that sveltestrap case is to expect global svelte/internal import npm install --save svelte sveltestrap.

Thanks for the heads up @telemmaite
We've had a lot of trouble know what the 'official' support for pre-bundled components are -
there's been SSR issues, suggestions to add svelte source files to bundle, etc.
Svelte 3 seems like a work in progress in this area compared to Svelte 1-2...
I'll see what we can do to improve this, thanks for the issue links


disable devtools in inertia.js and Laravel 8

Hello to all
How to disable devtools in inertia.js and Laravel 8?
Please help me

Hey 👋,
I'm going to close this issue, because it either lacks sufficient explanation regarding your exact problem, how to reproduce said problem, or how the problem relates to a bug or feature with the Inertia library itself.

If you're not sure how or where to get started with Inertia, I can highly recommend Laracasts's incredible (free!) video series: https://laracasts.com/series/build-modern-laravel-apps-using-inertia-js

If you're simply not sure on how to do a certain thing, I recommend you to ask your question in our Discord, as a GitHub Discussions topic, or perhaps even over at the Laracasts forums, as we have a very active community in those places that would be more than happy to help.

If you feel absolutely certain that your issue still points out a bug in the library, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.
Thanks!


Update version command to print GitSHA, Min/Max K8S Versions

Fixes #518 by adding additional version information:

MinimumKubeVersion supported by this version of Sonobuoy
MaximumKubeVersion supported by this version of Sonobuoy
GitSHA
K8S version (if kubeconfig is passed)

Signed-off-by: Steve Sloka<EMAIL_ADDRESS>

@johnSchnake thanks for the review! I've addressed all the comments.


Update UnturnedChat.cs

There is some issues with Om plugins cancelling that event but rm override the state instead of presenving it.

Seems reasonable! I have applied your fix in a patch released now.


KMeans

Hello, I would like to ask a question. I now have 35,000 images and feed them into the convolutional neural network in batches to obtain its features. The obtained feature dimensions are (32,512,2,2). When I use KMeans algorithm, does the parameter BS refer to the size of Batch_Size of my input data. If so what does N correspond to?
Here is my code to process the data.
def deep_clustering(model, dataloader, k):
    features = []
    original_labels = []
    for inputs, labels in dataloader:
        with torch.no_grad():  #
            outputs = model(inputs)  # VGG->(32,512,2,2)

Hi there,
sorry for the late reply, I was quite busy the last few weeks.
The dimensions of the input correspond to the following:
BS = batch size
N = number of points/nodes/objects (sample size)

spatial dimension
in case of images you usually have 3 spatial dimensions: height, width and channel.
In case of your embedded images it depends on how your feature dimension is defined.
if 32 is the batch size and each image is a feature representation of (512, 2,2) then you most likely need to flatten the embedding vector
(512, 2, 2) --> (2048, )
I hope that helps.


samples/vsphere/common/sso.py broken with pyVmomi version <IP_ADDRESS>

Describe the bug
Line 31 of samples/vsphere/common/sso.py performs an import of ThumbprintMismatchException from pyVmomi. In the latest version of pyVmomi, this cannot be found for import.
Reproduction steps
1.pip install pyVmomi
2.Set PYTHONPATH to reference samples
3.Try to import sso from samples.vsphere.common
...
Expected behavior
Would expect sso to import fine.
Additional context
No response

#350

Hi @crashingmotorbikes :
This is an incompatibility problem with pyvmomi <IP_ADDRESS>. The SDK will only be compatible with versions of pyVmomi up to 7.0.3 while we are working on the pyVmomi fix.
Thanks!

Issues has been fixed with release : https://github.com/vmware/vsphere-automation-sdk-python/releases/tag/v<IP_ADDRESS>


2016 year naming consistency

The 2016 datataking period is identified in the framework by two different year labels: 2016preVFP and 2016postVFP.
We must be consistent with this naming in all the parameters:  we need to choose between 2016preVFP and 2016_PreVFP

https://github.com/PocketCoffea/PocketCoffea/blob/34bb797feb07bf5718622371d5228a56a9da4da4/PocketCoffea/parameters/pileup.py#L2
https://github.com/PocketCoffea/PocketCoffea/blob/34bb797feb07bf5718622371d5228a56a9da4da4/PocketCoffea/parameters/jec.py#L2
https://github.com/PocketCoffea/PocketCoffea/blob/34bb797feb07bf5718622371d5228a56a9da4da4/PocketCoffea/parameters/lepton_scale_factors.py#L10

The scheme 2016_PreVFP and 2016_PostVFP has been chosen as default.


Support YARD directive comments

I have checked that this feature is not already implemented

[X] This feature does not exist

Use case
This is a feature request to improve Hover.
I would expect that I can see YARD formatted documentation about a function method in a class. This is particularly useful when I create ruby methods through metaprogramming. Currently, hovering on a method that is created dynamically would not show any information.
Code example:
# @!method foo(name, opts = {})
create_a_foo_method

As a side, Solargraph supports parsing YARD directives. And solargraph VS Code plugin shows defined function upon hovering.
Description
RubyLSP should parse YARD directives when parsing code documents.
Implementation
No response

Thanks. Good to know the plans for Ruby LSP.


Assign IPs to VLANs

We have a ton of Extreme Networks switches/routers (OK, more of an L3 switch). They're a little different in the initial setup - you don't assign an IP address to a port, you assign an IP address to a VLAN, which might not even have an assigned port.
Do you think it might be possible to add a VLAN field to the IP address, so we'd have "Device/Interface/VLAN"? Of course I could add a pseudo-Interface to the device, call it "VLAN-357" and attach the IP to that interface.
Just throwing that out there :)
Frank

Of course I could add a pseudo-Interface to the device, call it "VLAN-357" and attach the IP to that interface.

That's essentially what's happening when you assign an IP to a VLAN. The switch creates an SVI with a special MAC address to which the IP is assigned. This is how you want to define it in NetBox as well: a virtual interface named after the VLAN.

Oh, well... In that case it's all good ;) Sorry to have bothered!


GetDelegate.popRoute has push animation

When using delegate.toNamed, and then delegate.popRoute, the same animation is used, when I would expect delegate.popRoute to use the reverse animation:
https://user-images.githubusercontent.com/1630091/134519633-5031dc7d-f293-4ddc-b0cb-12e0c189e216.mov
import 'package:flutter/material.dart';
import 'package:get/get.dart';

void main() {
  Get.createDelegate();

  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return GetMaterialApp(
      initialRoute: '/',
      getPages: pages,
    );
  }
}

List<GetPage<dynamic>> pages = [
  GetPage<dynamic>(
    name: '/',
    page: () => RootScreen(),
    children: [
      GetPage<dynamic>(
        name: '/home',
        page: () => HomeScreen(),
        children: [
          GetPage<dynamic>(
            name: '/details',
            page: () => DetailsScreen(),
          ),
        ],
      ),
    ],
  ),
];

class RootScreen extends StatelessWidget {

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: GetRouterOutlet(
        delegate: HomeScreen.delegate,
        initialRoute: '/home',
        anchorRoute: '/home',
      ),
    );
  }
}

class HomeScreen extends StatelessWidget {
  static GetDelegate delegate = GetDelegate();

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: SafeArea(
        child: ElevatedButton(
          child: Text('Details'),
          onPressed: () {
            HomeScreen.delegate.toNamed('/home/details');
          },
        ),
      ),
    );
  }
}

class DetailsScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Details'),
        leading: IconButton(
          icon: BackButtonIcon(),
          onPressed: HomeScreen.delegate.popRoute,
        ),
      ),
      body: Text('Details Screen'),
    );
  }
}

To Reproduce
Steps to reproduce the behavior:

Tap "Details"
Tap the back button

Expected behavior
When tapping back button, I would expect a left-to-right animation, the opposite of the push animation.
Flutter Version:
Flutter (Channel stable, 2.5.1, on Mac OS X 10.15.7 19H524 darwin-x64, locale en-US)

Getx Version:
get:
    dependency: "direct main"
    description:
      name: get
      url: "https://pub.dartlang.org"
    source: hosted
    version: "4.3.8"

Describe on which device you found the bug:
iOS simulator

I should note that using PopMode.History does not solve the problem:
HomeScreen.delegate.popRoute(popMode: PopMode.History)

https://user-images.githubusercontent.com/1630091/134520753-e7d6d0b3-b223-4d37-a338-c594ec30ab19.mov

Also, this is providing a default transition:
GetMaterialApp(
  ...
  defaultTransition: Transition.leftToRight
  ...
)

https://user-images.githubusercontent.com/1630091/134521211-29f8ec35-aa47-44e2-b8d4-b064a1f6bae7.mov


Revert back to String based URI

Description
Fixes #33671
Reverts uri(URI) back to the previous uri(String). URI.create has issues where there is no host part when using an IP-style Azurite URL (https://devaccount1.blob.<IP_ADDRESS>:10000) where uri(String) doesn't have the same issue.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


hide av download button on chrome with css

This uses a CSS hack to hide download buttons on audio and video elements in Chrome.
This has been deployed to staging for testing.  I have tested it on all my available browsers and devices, as well as the Chrome viewport controls.  Further testing would be appreciated!  We need to ensure not only that the download button is hidden on Chrome, but also that no other controls are accidentally hidden on other browsers.
You can use these examples on staging:
Audio example: https://staging.dp.la/primary-source-sets/sources/5
Video example: https://staging.dp.la/primary-source-sets/sources/6
And compare with these examples on production:
Audio example: https://dp.la/primary-source-sets/sources/1211
Video example: https://dp.la/primary-source-sets/sources/1208
This addresses ticket DT-1145.

Did not run into problems on Chrome on my machine (macOS, Version 56.0.2924.87 (64-bit)) or my phone (55.0.2883.91 for Android)

Same thing with Chrome in Windows 10, Version 56.0.2924.87

Also looks good in Chrome in ElementaryOS, Version 56.0.2924.87

Waiting on @markbreedlove and @moltude for more testing on their systems.
It'd be good if we could find a machine with an older version of Chrome on it. I'll look around here.

When @mdellabitta is done verifying on that old Chrome version, I'll be available to merge this.


Checking if a userdata value is a certain usertype?

I'm not sure how best to go about this. I'd ultimately like to be able to do something like one of the following (with thing bound as a usertype):
local inst = thing.new()

-- one option, member function on the type
assert(inst:is_a(thing) == true)

-- another option, helper method
assert(is_usertype(inst, thing))

-- third option, reference somewhere in the userdata
assert(inst.__usertype == thing)
-- or
assert(getmetatable(inst).__usertype == thing)

In looking through the Lua objects at runtime, I couldn't find a foolproof way to check the relation between a userdata instance and a usertype table--the only things shared are the values of the userdata's metatable key/value pairs and the usertype's key/value pairs (but the metatable is not == to the usertype).
I hacked up a helper method version in Lua, but it's prone to error/false positives:
-- if there is a matching key/value pair between the inst's metatable
-- and the usertype, then assume they match
function is_usertype(inst, usertype)
  if type(inst) ~= "userdata" then return false end
  if type(usertype) ~= "table" then return false end
  local mt = getmetatable(inst)
  for k,v in pairs(usertype) do
    if mt[k] and v == mt[k] then return true end
  end
  return false
end

Any ideas on how this sort of thing could be achieved?

Write a function in C++ when you register your usertype to do the check, if it's that important:
lua.new_usertype<thing>( "thing",
     ..., ....
     "is", []( sol::stack_object something ) { return something.is<thing>(); }
);

If you're asking how you would do that in Lua, you kind've don't? Metatable implementation is an currently opaque right now, save for the memory guarantees about where the usertype stuff is. It's still subject to change because it's not full-featured yet (see the open issue #266) and we need to make sure we get all the features we can. Plus, we employ some meta-table optimizations based on what gets serialized, so guaranteeting a representation would screw everyone out of optimizations.
Write the methods you need in C++ to check this. Lua isn't very strongly typed and trying to force it to be is probably not the best idea.

Alright, I was hoping to avoid having to bind a method to/for every usertype, but that works. Thanks for the response.

You can use a template free function to make things easier to bind:
template <typename T>
bool is_check ( sol::stack_object something ) {
     return something.is<T>();
}

Then, when binding...
lua.new_usertype<thing>( "thing",
     ..., ....
     "is", &is_check<thing>
);

A bit of a necro on this issue here, see #506 and maybe drop your opinion.


DependencyNotFoundException: Unable to resolve system scoped dependency: com.sun:tools:jar:1.8.0:system

jdk 1.8 (oracle)
dependency check version 6.3.1
maven 3.5
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.owasp:dependency-check-maven:6.3.1:aggregate (default-cli) on project modernisation: One or more exceptions occurred during dependency-check analysis: One or more exceptions occurred during analysis:
[ERROR] 	DependencyNotFoundException: Unable to resolve system scoped dependency: com.sun:tools:jar:1.8.0:system
[ERROR] 	DependencyNotFoundException: Unable to resolve system scoped dependency: com.sun:tools:jar:1.8.0:system
[ERROR] 	DependencyNotFoundException: Unable to resolve system scoped dependency: com.sun:tools:jar:1.8.0:system
[ERROR] -> [Help 1]

Sounds like it's not a bug, you should check the pom, sounds like there is an invalid system path for the tools.jar in your pom

there's no sytem path at all. if I call dependency-check-maven with java 11 (open jdk) it works.

Well, there should be some item in the dependency tree that defines a dependency on com.sun:tools:jar:1.8.0:system, as otherwise DependencyCheck would not try to resolve it.
As running it with java 11 works without proble it looks as if some pom on the path triggers the dependency on the JDK tools.jar depending on the JVM it's run on.
Without access to your project structure it's impossible to determine what that would be.
You can try to get clarity from maven on it by using
mvn dependency:tree -Dincludes=com.sun:tools

(with JDK 8 of course, as apparently the dependency is absent for JDK 11)

You are right I find this dependency in plugin weblogic-maven-plugin

com.sun
tools
1.8.0
system
${java.home}/../lib/tools.jar

and within the build jenkins the property java.home does'nt exist (we define JAVA_HOME instead)

Seems strange to me that your project would have a dependency on a maven plugin. Typically a maven plugin is configured to run in a build, not to be a depended-upon library.

we have a piepline with multiple stage (build + dependency check + analyse sonar + ...). we use weblogic-maven-plugin to compile jsp in stage build. may be in stage dependency check it scan CVE in relation with this plugin

But the error you get sounds as if you have the plugin listed in <dependencies> section of the pom.xml. A maven plugin only needs to be listed in the <build><plugins> section to use it in your build.

If you look at the pom of that library you'll see why you have different behaviour between running with jdk8 and jdk11: depending on the jvm version the library takes a different profile which triggers different dependencies. For java 8 you need a tools.jar present on your system and you'll have to define the appropriate path to it in order to allow it to be resolved.
Within the dependencyManagement section of your top-level pom override the dependency of arquillian-liberty-managed
The arquillian pom-file defines it as:
        <dependency>
          <groupId>com.sun</groupId>
          <artifactId>tools</artifactId>
          <version>1.6.0</version>
          <scope>system</scope>
          <systemPath>${java.home}/../lib/tools.jar</systemPath>
        </dependency>

And as per the error your system doesn't have a tools.jar present at ${java.home}/../lib/tools.jar (does it have the openjdk JDK installed? Or do you only have the openjdk JRE?)
You'll need to put an identical entry in your top-level pom dependencyManagement  in order to override the systemPath with the location of tools.jar on your system (or configure an appropriate java.home system property).
As an alternative solution you could configure the plugin to skip system-scoped dependencies - https://jeremylong.github.io/DependencyCheck/dependency-check-maven/configuration.html.

@aikebah Thank you for the explanation. There is a tools.jar in ${java.home}/../lib for my OpenJDK 8, but I haven't got a JAVA_HOME set on my system. Setting that doesn't seem to fix the issue.
Skipping system-scoped dependencies does the trick however. Thanks.

While the solution from @aikebah proposed in this comment works, it ceases to work when JDK >8 is used.
We eventually solved it by adding
<skipSystemScope>true</skipSystemScope>
to the configuration of the dependency-check-plugin.
See also issue #729


Update preinstalled software list for TCC agents

PR created by http://unit-1592.labs.intellij.net:8111/buildConfiguration/TestProject_UpdateSoftwareInfoViaPr/3650

Closing this PR as obsolete, there is a more recent one


What is the defination of <  your Verified Domain >

Hi, it looks that this document has been updated in 2017, but e.g. the previous comment "Multiple SIP/SMTP domains" from @rgsteele has been written in 2018. So that comment has been closed, but not implemented?
But to my comment is about defination of "<  your Verified Domain >"
On the step 1 you say:

Specify a verified domain for your Skype for Business Online organization. This domain should be the same domain used as the primary SIP domain used for the cloud-based accounts.

But then on Step 7 you say:

This domain should be the same domain used as the primary SMTP domain used for the on-premises Exchange accounts. This domain is referred as  in the following procedure.

Isn't that 10x easier to say, it is the domains you get by the command get-cssipdomain? Why to make it too complicated?
So would be any possibilities to clarify what do you mean by this, and how this should affect if you have 15 SIP domains in use? What steps I need to run 15 times?
regards
Petri

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

ID: 19d818da-830e-c813-08af-6192eb403ccc
Version Independent ID: 575c0a11-f6e1-c571-4ad4-d490915faa15
Content: Configure OAuth between Skype for Business Online and Exchange on premises
Content Source: Skype/SfbServer/deploy/integrate-with-exchange-server/oauth-with-online-and-on-premises.md
Product: skype-for-business-itpro
GitHub Login: @jambirk
Microsoft Alias: jambirk

Hi Petri,
The comment has been closed in GitHub, and an issue added to our internal backlog, as @kenwith explained: "I was able to get this into the backlog for the Skype team. The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better."  I can understand frustration with the delay, but we are tracking this internally and will get to it when we're able.

Also, this new Issue that was just opened will get picked up by a new process to work on improving articles based on the valuable feedback of people like you @Petri-X . Your knowledge is what makes the docs better so please keep up the great work. Sorry for the long delay before. It took me some time to get a process worked out but we are up and running with it now and working through these issues with content actively.

Okay, great to hear :)
And @jambirk the "frustration" might be a bit too strong term :D But thanks to share this.

@officedocsbot assign @scanum


Update sbt to 1.4.7

Updates org.scala-sbt:sbt from 1.3.13 to 1.4.7.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.3.13).
You might want to review and update them manually.
src/main/resources/swagger-ui/swagger-ui-bundle.js.map
src/main/resources/swagger-ui/swagger-ui-standalone-preset.js.map
src/main/resources/swagger-ui/swagger-ui.js.map

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-minor, old-version-remains

Superseded by https://github.com/manuzhang/akka-http-petstore/pull/15


The app is crashing on "Move Funds" CTA when we are trying to move funds from a vault to hide the vault.

https://github.com/bithyve/bitcoin-keeper/assets/62699947/8fd38205-9a7a-41dd-9142-7fab03e8a91e

Still having the same issue on Keeper app v 1.1.12(303)

https://github.com/bithyve/bitcoin-keeper/assets/62699947/78f15c83-44bd-49f3-9854-965e8c77e5d1

Verified this issue on keeper app v 1.1.12(306)


Row Count Validation

As an MVP, having the option to compare a previous version of a dataset the amount of rows in a subsequent data load might be handy for some data. I'm thinking anything related to something like administrative areas where the number of states, provinces, territories, etc should remain constant.
Future development will entail being able to allow partial files to load.

need:
Min and max thresholds to generate both errors and warnings


fixed filter issue while sharing

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


N(0, scale), exp, multiply \mu OR N(\mu, scale), exp

Hi, thanks for your really amazing work!
I wonder when reparametrization is not a must, e.g., MLE, the predicted $\mu$ should be used after exp via SO3MultiplyTransform or before exp as the mean of the Euclidean Gaussian? I.e., like the title, the procedure should be N(0, scale) $\rightarrow$ exp $\rightarrow$ multiply $\mu$ or N(\mu, scale) $\rightarrow$ exp?
Note that in readme, it is the former case, while in experiment
https://github.com/pimdh/relie/blob/master/relie/experiments/so3_conditional_mle.py#L90, it is the latter case.
Is there any preference?
Thanks in advance!

I got it. Thanks for the really quick response!


Close some resources in tests

A minor thing, most for fixing warnings from linter in the $IDE.
Introduced in this PR AutoCloseableResource.resource perhaps should be in the JVM module for the 0.23 series. I'll port it in case of merging this PR.

This looks very similar to Resource.fromAutoCloseable: https://typelevel.org/cats-effect/api/2.x/cats/effect/Resource$.html#fromAutoCloseableF[_],A%3C:AutoCloseable(implicitF:cats.effect.Sync[F]):cats.effect.Resource[F,A]

@armanbilge yep. By and large for cases not dealing with F[_] and CE Resourse. I've strived to don't refactor tests much, keeping their initial semantic.

Thanks for explaining. IMHO I think it would be more idiomatic to make use of Resource.fromAutoCloseable and ResourceFixture from munit-cats-effect. But, whatever works :)

I'm pretty sure ResourceFixture works the same for any kind of test, since it's wiring CE Resource with munit's existing fixture system (which doesn't know anything about CE).

@armanbilge using ResourceFixture or CE Resource is looking promising. Besides, it would mean solid refactoring of tests, which I've tried to get around. And I agree with Ross, that one may do this subsequently. I left a comment about it for hereafter investigators.


Rails 3.2 Compatibility

Adapter name changed a bit. Support for < 3.2 is unchanged.

Cheers

Ah, thank you! I was just about to open a ticket and do a pull request myself.

[to kreeger] yep, sure. I'll publish new gem version soon with other fixes.

@electronick Thanks… you rock!


[DataTable] TableToolbarSearch placeholder text not customizable / translatable to multiple variations within one app

@joshblack
Use this template if you want to request a new feature, or a change to an existing feature.
If you are reporting a bug or problem, please use the bug template instead.
Summary
Please describe your request in one or two sentences.
DataTable / TableToolbarSearch =>translateWithId doesn't support multiple placeholder text within the search input within the same react app.  Not all search placeholder text are all alike in the same app, but if there is a workaround the translateWithId functionality, I'd be open to hearing about it.
Clarify if you are asking for both design and development, or just design, or just development.
Justification: See the previous sentences.  This likely only necessitates development, but I'm betting the documentation will need to be amended.
Provide the business reasons for this request.
This is will have impact on IBM
Desired UX and success metrics
Describe the full user experience for this feature. Also define the metrics by which we can measure success/failure for the user.
See sentences above.
"Must have" functionality
Highlight any "must have" needs and functionality for the request.
See sentences above.
This should not be a full list of functionality; the Carbon team will work with you to define functionality based on the desired UX.
Specific timeline issues / requests
Do you want this work within a specific time period? Is it related to an upcoming release?
Yes and yes.  If this could be done within the next month that would be ideal.
NB: The Carbon team will try to work with your timeline, but it's not guaranteed. The earlier you make a request in advance of a desired delivery date, the better!
Available extra resources
What resources do you have to assist this effort?
I can help if necessary.
Carbon is a collaborative system. We encourage teams to build components and submit them for integration as either add-ons or core components.

Hi @LongBu! Could you just change the translateWithId prop that you pass into the component in different areas of the app? Agreed that this makes no sense in the context of the app, if you have suggestions for how to do the message formats let me know!

Hi @joshblack,
In changing the translateWithId prop, this would manifest in different areas of the app where DataTable/ TableToolbarSearch were located.  There might be a good reason for translateWithId, but my suggestion would be to have a propsuch as placeHolderText similar to what is found in the Search Component.  That seems to be the most logical fix, but this unfortunately could break others code on update/re-build, though the amending one's build  would likely not take a large degree of man hours.  Thoughts?
Thanks!

@LongBu agreed that translateWithId could have a better 1:1 mapping with explicit text. The reason we have translateWithId is so that you could supply one function to DataTable and then it could pass it around the entire component tree. In other words, in cases where you didn't have direct access to a label you could use this method to have deep access for translation 👍

@joshblack
Can you supply an example of utilizing a function that allows one to change up the placeholderText to something besides one value per project?
Harnessing 'react-i18next' and injecting InjectedI18nProps, InjectedTranslateProps seems to only way that I've been able to get translatedWithId to accept anything (eg: a direct mapping to one value in an external file):
<DataTable
translateWithId={t}
....
/>
Maybe I'm missing something or is it just something to live with?
Thanks

Weighing in - Do anybody interested in a "hybrid" approach, where translateWithId runs only if e.g. placeHolderText is undefined? Thanks!


Add bms-c1

Insert URLs to the project here: https://github.com/LibreSolar/bms-c1

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as Good First Issue of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.
All projects listed will be posted on our Mastodon channel

@martinjaeger


Empty All in placeholder remove plugins from all the localized version of the page

Solve Issue #3880

Nice, thanks!

How about writing a test for this?

I'd like to get this into 3.0.11, which has a target "cutoff point" of 28-Feb.

@pevar do you need any help to write the tests to get this merged?

Yes please, first time for me.

@pevar steps are:

create a page
create a few plugins (see https://github.com/divio/django-cms/blob/support/3.0.x/cms/tests/plugins.py#L324-L336)
create a title (this means: translate the page at first step by using create_title) as in https://github.com/divio/django-cms/blob/support/3.0.x/cms/tests/multilingual.py#L182
add plugins to the title (as in step 2, but setting the language according to the title you just created)
use login_context to login as admin (https://github.com/divio/django-cms/blob/support/3.0.x/cms/tests/admin.py#L82)
run the clear_placeholder view (as in https://github.com/divio/django-cms/blob/support/3.0.x/cms/tests/admin.py#L1149-L1151) (by default it runs in english)
after the view has run check that the placeholder still contains plugin in the other language (something like self.assertEqual(ph_en.get_plugins().count(), #) would do, where "#" is the number of plugins you added in step 4

Join #django-cms IRC channel for further help

@yakky
Thanks, which module in test folder I should put my code in?

I think cms.tests.admin is the best one

@pevar wow, really nice!
Te failing test it's just a pyflake check, you just need to adjust the test code style (please see the failure text on travis or run develop.py test cms.AboveStaticAnalysisCode on you local machine)

@yakky, it's not really clear to me what travis is saying. I've now removed some unused variables to get the style right.

@pevar exactly :)
We have this pyflake check in the testsuite that runs pyflake on the codebase and it fails if pyflake report some issues.
You can run this check locally with develop.py test cms.AboveStaticAnalysisCode

@mkoistinen LGTM

@mkoistinen this may be ported to support/3.0.x too, I think

@pevar congrats for your first contribution to django CMS, well done!

@yakky thanks for the help! I'm very happy to contribute.


fix: #1532 upload sources file error "I/O operation on closed file"

the @pavel604 fix https://github.com/cpacker/MemGPT/issues/1352#issuecomment-2099250518 works for me to resolve @hqzhon issue #1352

lgtm, thanks!


Add Goffstown to NH city list

Please add Goffstown to the list of cities for New Hampshire. Thank you!

Hi @bethanyveilleuxmw, thanks for your interest in adding Goffstown to the website. Emails on defund12.org are all written by community members. To add your city, you must provide:

Subject line for the email
Email addresses (at least one) of the government officials you’d like to receive the message
Detailed message concerning the budgeting decisions at stake in your community

Until you provide this content, we cannot post your city.
Follow our email style guidelines for quicker acceptance. Remember that messages must be city-specific and focused on defunding the police. If your message is copied from an external source, please provide a link to the original source.
Your request will be closed until you reply with the necessary content. Thank you!


PROPOSAL - refactor the templates to be more composable to reduce the amount of repetition

At the moment, the structure of the templates is <cloud>-<vcs> (eg, aws-github or civo-gitlab). IMO, this means that there will be an awful lot of duplication in these templates. For example, when I did the recent GCP cluster work (#769) it was only for GitHub - the likelihood is that most (if not all) of this will be identical for GitLab.
This is an additional maintenance burden on our small team that we don't need.
├── akamai-github
├── aws-github
...<lots-more>
└── vultr-gitlab

I would advocate for refactoring this repo so that it incorporates a common folder. This would be for each level. My proposed structure will be something like this:
├── common
│   ├── cloud  # One folder for each cloud provider, eg `aws`, `civo`, `gcp` etc
│   │   ├── aws # Everything in here is copied to every "aws" gitops
│   │   └── <...>
│   ├── template # Everything in here is copied to every gitops
│   └── vcs # One folder for each VCS provider, eg `github`, `gitlab` etc
│       ├── github # Everything in here is copied to every "github" gitops
│       └── <...>
└── template # Each of the folders we currently have
    ├── akamai-github
    ├── aws-github
    ├── <...>
    └── vultr-gitlab

Pros

Reduce the amount of duplication we have
Make it easier to rollout updates

Cons

The gitops repo for the user is no longer a fork of the gitops-template repo
How would we maintain backwards compatibility?

with this PR merge, i think we can leverage a lot more the gitops-template  with a bit of logic in it instead of only in kubefirst-api
can be lead to somekind of custom helm rendering dedicated to K1
but will be a huge breaking change .


Need an object pool

At least in the CameraDetection scene we create/destroy objects a bunch. We should try to reuse any when possible.

A basic pool has been added.We will optimize its performance as we hit limitations (if any).


Update to using pyproject.toml

It seems like the Python community is moving towards using pyproject.toml files to manage package and project information. This commit adds a pyproject.toml, removes the setup.py and and adds some instructions for running the tests with poetry. The github CI action was also updated.

Thanks for this, I wanted to do this, too;
Could you please check the failing checks?
Also, I'd like to make the package versions as lenient as possible.
E.g. requiring python 3.11 I think is too much.

@Florents-Tselai good idea about loosening the versions and allowing older pythons. I just updated the PR to allow for pythons >= 3.9 (will that be enough, or should we go lower?). I also unpinned the dependencies to see if that will work going forward. I think I mostly just copied the requirement versions from the setup.py, but I liked the idea of leaving them looser, especially for getting newer/better versions of things as they become available.
And I think I fixed the CI build, I had forgotten to install dependencies!


ArchiveFiles: fails with file and no root folder

If the specified folder to create is a file, and the do not include root folder option is set, then the task was failing because it was attempting set the current working directory the file instead of its parent directory.

Looks good!


Spread exact shaped object cannot flow into intersection type

This case seems pretty easy and I'm not sure if it is a bug or just a way how flow works, but for me it's odd that this spread type is treated as inexact.
https://flow.org/try/#0PTAEAEDMBsHsHcBQiAuBPADgU1AYVgHaQCWA5qALygDeAPoqKMQCYBcoBArgLYBGWAJwA0DUOmzsAzigHECpEbQC+yAMaFpoEgQCG0AAo6BknFQAU6omXb4r5AGQ1QGIzu5SZc8koCUlAHygAOQCWJKc0ChBahoozkYmtiTk5pbJNoTJQvECbh6y8n4UgdSilprcgqRYzJQ0AHSNaWTZLrncoCqMoSicAgRacnqGxlhmlQLVzD6ISkA

This is likely same issue as #2405

Indeed, I think it is. It's a bummer that the other issue stays unresolved after so much time 😢


Obsoletion notice: GO:0033779 5beta-cholestane-3alpha,7alpha-diol 12alpha-hydroxylase activity and GO:0033778 7alpha-hydroxycholest-4-en-3-one 12alpha-hydroxylase activity

Dear all,
The proposal has been made to obsolete:
GO:0033779 5beta-cholestane-3alpha,7alpha-diol 12alpha-hydroxylase activity and GO:0033778 7alpha-hydroxycholest-4-en-3-one 12alpha-hydroxylase activity
The reason for obsoletion is that this corresponds to specific substrates.
There are 3 TAS annotations by Reactome.
RHEA mappings will be transfered to the parent term,  sterol 12-alpha-hydroxylase activity
You can comment on the ticket: https://github.com/geneontology/go-ontology/issues/29450
Thanks, Pascale

Duplicate of https://github.com/geneontology/go-ontology/issues/29331


Add support for wallet operations

Describe your new idea or enhancement
Add support for the /wallet endpoint

Yeah our intention is to wrap the entire  cloudagent. But it just takes some time. We will give this some priority.


Highlight and recommend projects

As a dribdat user I want to recommend certain tools, bootstraps, data repositories, or projects from previous hackathons. I want to be able to highlight a project, promoting it to event participants in the Information page, or as a direct recommendation to teams on their project pages. I need to be able to give it my recommendation for a progress level (so it appears at all events) or category (which may be event-specific). I may want to leave a feedback comment when I do this, and for this to be seen on the project page.
See also: #129 and #87

This has been partially achieved in the above PR: projects can be referenced in a "Resource Event".

Using the Resources, I should be able to specify that my project is a Fork of another.
Additionally, I would like to annotate my reference with an evaluation. This could even be a questionnaire, or based on some kind of automatic process.


How to implement double click android back to exit app

I have read the docs,but does not find any way to control the android back button behavior.
So I also read the source code.
In createNavigationContainer.js
componentDidMount() {
...
this.subs = BackAndroid.addEventListener('backPress', () =>
this.dispatch(NavigationActions.back())
);
...
}
dispatch = (action: NavigationAction) => {
const { state } = this;
if (!this._isStateful()) {
return false;
}
const nav = Component.router.getStateForAction(action, state.nav);
  if (nav && nav !== state.nav) {
    if (console.group) {
      console.group('Navigation Dispatch: ');
      console.log('Action: ', action);
      console.log('New State: ', nav);
      console.log('Last State: ', state.nav);
      console.groupEnd();
    } else {
      console.log('Navigation Dispatch: ', { action, newState: nav, lastState: state.nav });
    }
    this.setState({ nav });
    return true;
  }
  return false;
};

when I follow the docs,override Component.router.getStateForAction, reutrn null in some condition to prevent back navigation,but  "BackAndroid.addEventListener('backPress', () =>
this.dispatch(NavigationActions.back())
);" will return false. This cause the app exit.

+1

Hi! In an effort to get the hundreds of issues on this repo under control I'm closing issues tagged as questions. Please don't take this personally - it's simply something we need to do to get the issues to a manageable point. If you still have a question I encourage you to re-read the docs, ask on StackOverflow, or ask on the #react-navigation Reactiflux channel. Thank you!

@Yorkey Please have a look at this module. I hope it solves your problem. https://www.npmjs.com/package/exit-on-double-back

https://stackoverflow.com/questions/52253072/react-native-double-back-press-to-exit-app/52253073#52253073

for react native v 0.61.0 I use this solution:
import React ,{useEffect, useState}from 'react'
import {BackHandler} from 'react-native'

const Main = () => {
    const [exitApp,SETexitApp]=useState(false)

    const backAction = () => {
        if(exitApp==false){
            SETexitApp(true);
            Toast.show({
                text: 'back click for exit',
                type:'warning',
                duration:1500,
                style:[styles.toast],
                textStyle:[styles.toast_text],
            })
        }
        else if(exitApp==true){
            BackHandler.exitApp()
        }

        setTimeout(()=>{
            SETexitApp(false)
        }, 1500);
        return true;
    };
    useEffect(() =>
    {
        const backHandler = BackHandler.addEventListener(
            "hardwareBackPress",
            backAction
        );
        return () => backHandler.remove();
    },[exitApp])

}


Sort low contrast in dark mode

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog :^)

Thanks @JacobCoffee for submitting this 🙏🏼
I'm expecting a baby any second now 😍 So not sure I'll be able to ship a fix as quickly as I'd normally like. Adding good first issue in case someone wants to contribute a patch. Otherwise, I'll make sure to prioritize a fix as soon as possible for this.

What browser/OS was this @JacobCoffee ?

Windows, Firefox
Linux, Firefox works just fine fwiw

I'm working on this issue. Can you please assign it to me?

https://github.com/polarsource/polar/pull/1007

@JacobCoffee This issue is fixed. https://github.com/polarsource/polar/pull/1007 was merged.
Thanks all,
Ashhar


Removes remaining use of jackson-databind from zipkin-server

This is still in use in elasticsearch, but the server no longer uses it.
Also addresses feedback from @anuraaga in https://github.com/openzipkin/zipkin/pull/2819

I started to look into the scope of dropping databind from ES.. mainly it is the marshaling the search request. It is really odd how huge that jackson-databind is compared to what we had before. moshi is <200k. Anyway it is feasible given a long flight to drop our jar size over a meg once we axe databind from ES.

outside the server, the only thing we use databind for is elasticsearch read requests which is not performance bound (as ES results are way slower than making requests). I only now noticed this from a @bsideup tweet. Seems an easier way to kill a meg https://github.com/FasterXML/jackson-jr


Multiple admin accounts

It'll be nice if YOURLS could support multiple admin accounts, especially for orgs with different departments. Or if there's a plugin for this, or a way to develop one, please kindly help point me in the right direction.
Thank you

I don't understand what you're requesting. The config file already supports multiple username/password pairs.

Could you please point me to a documentation on how to pull that off?

Thank you

Sounds like you found the solution, but in case not:
https://github.com/YOURLS/YOURLS/blob/792f2e56916a5d089bf89228f9914ae741faa26c/user/config-sample.php#L59-L66

Also the wiki : https://github.com/YOURLS/YOURLS/wiki/Username-Passwords

Got it sorted...thanks again :)


ADQ in bigquery and on site don't match

From user feedback.
Regarding https://openprescribing.net/measure/ktt9_uti_antibiotics/

Indeed this do not match with what you get in the website.
But @richiecroker checked in in BigQuery, and you do get results matching what OP was expecting:

SELECT month, SUM(p.quantity * r.adq_per_quantity) AS numerator
from hscic.normalised_prescribing_standard p  LEFT JOIN
hscic.presentation r  ON p.bnf_code = r.bnf_code
where
    (p.bnf_code LIKE '0501015P0%AB' OR --Pivmecillinam HCl_Tab 200mg (brands and generic)
    p.bnf_code LIKE '0501080W0%AE' OR --Trimethoprim_Tab 200mg (brands and generic)
    p.bnf_code LIKE '0501130R0%AA' OR --Nitrofurantoin_Cap 50mg (brands and generic)
    p.bnf_code LIKE '0501130R0%AD' OR --Nitrofurantoin_Tab 50mg (brands and generic)
    p.bnf_code LIKE '0501130R0%AG%') --Nitrofurantoin_Cap 100mg M/R (brands and generic)
    and pct = '08E'
    group by month
    order by month

I've checked and the answer is.. with setting = 4:
Row | month | numerator |
-- | -- | -- | --
1 | 2019-04-01 00:00:00 UTC | 6699.75 |
2 | 2019-05-01 00:00:00 UTC | 6582.75 |
3 | 2019-06-01 00:00:00 UTC | 6611.583333333333 |
without setting = 4:
Row | month | numerator |
-- | -- | -- | --
1 | 2019-04-01 00:00:00 UTC | 8229.25 |
2 | 2019-05-01 00:00:00 UTC | 8450.583333333332 |
3 | 2019-06-01 00:00:00 UTC | 8252.083333333334 |