[in remember of Bram]: CI: clang-17

Hm, there seems to be a problem with clang?
addr2line: DWARF error: invalid or unhandled FORM value: 0x23
regexp.c:2430:25: runtime error: call to function do_Upper through pointer to incorrect function type 'void *(*)(int *, int)'
/home/runner/work/vim/vim/src/regexp.c:1732: note: do_Upper defined here
    #0 0x55c2da0a7c23 in vim_regsub_both /home/runner/work/vim/vim/src/regexp.c:2430:25
    #1 0x55c2da0a4ee2 in vim_regsub /home/runner/work/vim/vim/src/regexp.c:1939:14
    #2 0x55c2d9b7105a in do_string_sub /home/runner/work/vim/vim/src/eval.c:7446:15
    #3 0x55c2d9bac51d in f_substitute /home/runner/work/vim/vim/src/evalfunc.c:10303:25
    #4 0x55c2da53d860 in call_bfunc /home/runner/work/vim/vim/src/vim9execute.c:1263:5
    #5 0x55c2da4eebbf in exec_instructions /home/runner/work/vim/vim/src/vim9execute.c:4257:7
    #6 0x55c2da5219cb in call_def_function /home/runner/work/vim/vim/src/vim9execute.c:6179:11
    #7 0x55c2da44cb2a in call_user_func /home/runner/work/vim/vim/src/userfunc.c:2796:6
    #8 0x55c2da44bc66 in call_user_func_check /home/runner/work/vim/vim/src/userfunc.c:3229:10
    #9 0x55c2da446406 in call_func /home/runner/work/vim/vim/src/userfunc.c:3834:15
    #10 0x55c2da444938 in get_func_tv /home/runner/work/vim/vim/src/userfunc.c:1943:8
    #11 0x55c2da471813 in ex_call_inner /home/runner/work/vim/vim/src/userfunc.c:5965:6
    #12 0x55c2da471813 in ex_call /home/runner/work/vim/vim/src/userfunc.c:6302:11
    #13 0x55c2d9c30410 in do_one_cmd /home/runner/work/vim/vim/src/ex_docmd.c:2582:2
    #14 0x55c2d9c260be in do_cmdline /home/runner/work/vim/vim/src/ex_docmd.c:994:17
    #15 0x55c2da198e16 in do_source_ext /home/runner/work/vim/vim/src/scriptfile.c:1760:5
    #16 0x55c2da196a73 in do_source /home/runner/work/vim/vim/src/scriptfile.c:1906:12
    #17 0x55c2da196a73 in cmd_source /home/runner/work/vim/vim/src/scriptfile.c:1251:14
    #18 0x55c2d9c30410 in do_one_cmd /home/runner/work/vim/vim/src/ex_docmd.c:2582:2
    #19 0x55c2d9c260be in do_cmdline /home/runner/work/vim/vim/src/ex_docmd.c:994:17
    #20 0x55c2d9f38c0a in nv_colon /home/runner/work/vim/vim/src/normal.c:3196:15
    #21 0x55c2d9f17387 in normal_cmd /home/runner/work/vim/vim/src/normal.c:939:5
    #22 0x55c2da839436 in main_loop /home/runner/work/vim/vim/src/main.c
    #23 0x55c2da837359 in vim_main2 /home/runner/work/vim/vim/src/main.c:895:5
    #24 0x55c2da82f0f4 in main /home/runner/work/vim/vim/src/main.c:441:12
    #0 0x7f0d59b4ad8f in __libc_init_first ??:?
    #1 0x7f0d59b4ae3f in __libc_start_main ??:?
    #2 0x55c2d9899f34 in _start ??:?

same error :(

same error :(

Yep, I should wait until next clang 17.0.x release and do a rebase then. ;)

I think the problem here is that the code is relying on some undefined behaviors (I think?) here. do_Upper's type is fptr_T (*)(int *d, int c), but the code is trying to cast it to fptr_T itself, which is void* (*)(int *d, int c).
Note that this is intentional. The code is trying to play with type system in C and relying on this behavior working in C compilers but I think it's undefined behavior. The relevant bits in regexp.c:
/*
 * We should define ftpr as a pointer to a function returning a pointer to
 * a function returning a pointer to a function ...
 * This is impossible, so we declare a pointer to a function returning a
 * void pointer. This should work for all compilers.
 */
typedef void (*(*fptr_T)(int *, int));

    static fptr_T
do_Upper(int *d, int c)
{
    *d = MB_TOUPPER(c);

    return (fptr_T)do_Upper;
}

//…

static int vim_regsub_both(...) {
//…
    fptr_T	func_one = (fptr_T)NULL;
//…
		case 'u':   func_one = (fptr_T)do_upper;
//…
}

I think one way to fix this is by simply defining do_Upper to return void* itself, so the types would match.
Looking more closely though, this seems… a little over-engineered? The usage of complicated function pointers imply a lot of different generic cases that need to take advantage of this but there are really 4 functions here (e.g. do_Upper) to account for upper/lower case and one-time application versus all. The ability for a function to return itself (the cause of this behavior) doesn't seem to actually be necessary at all as there are already func_one and func_all to distinguish them. If I fix it I would probably simply refactor to not use the function pointers.

Actually, is that a runtime error and not a compiler error? Did clang add some runtime check for this or something?

@ychin I guess it is an runtime error as it's showing since clang-17-rc1

I think the problem here is that the code is relying on some undefined behaviors (I think?) here. do_Upper's type is fptr_T (*)(int *d, int c), but the code is trying to cast it to fptr_T itself, which is void* (*)(int *d, int c).
Note that this is intentional. The code is trying to play fast and loose with type system in C and relying on this behavior working in C compilers but I think it's undefined behavior. The relevant bits in regexp.c:
/*
 * We should define ftpr as a pointer to a function returning a pointer to
 * a function returning a pointer to a function ...
 * This is impossible, so we declare a pointer to a function returning a
 * void pointer. This should work for all compilers.
 */
typedef void (*(*fptr_T)(int *, int));

    static fptr_T
do_Upper(int *d, int c)
{
    *d = MB_TOUPPER(c);

    return (fptr_T)do_Upper;
}

//…

static int vim_regsub_both(...) {
//…
    fptr_T	func_one = (fptr_T)NULL;
//…
		case 'U':   func_all = (fptr_T)do_Upper;
//…
	    else if (func_all != (fptr_T)NULL)
		// Turbo C complains without the typecast
		func_all = (fptr_T)(func_all(&cc, c));
//…
}

I think one way to fix this is by simply defining do_Upper to return void* itself, so the types would match.
Looking more closely though, this seems… a little over-engineered? The usage of complicated function pointers imply a lot of different generic cases that need to take advantage of this but there are really 4 functions here (e.g. do_Upper) to account for upper/lower case and one-time application versus all. The ability for a function to return itself (the cause of this behavior) doesn't seem to actually be necessary at all as there are already func_one and func_all to distinguish them. If I fix it I would probably simply refactor to not use the function pointers.

Send a PR?

Yeah I will send a new PR in a bit. Just need to test it.
I tried it locally and yes it's indeed UBSAN (undefined behavior sanitizer). Seems like clang 17 just got a little more rigorous in checking this.

#13117 should fix the issue and simplify the code as well.

See #13122.

Refactoring is still needed:
if_python.c:1061:5: runtime error: call to function run_eval through pointer to incorrect function type 'void (*)(const char *, void *, PyGILState_STATE *)'
/home/runner/work/vim/vim/src/./if_py_both.h:6132: note: run_eval defined here
    #0 0x5586bbde115f in DoPyCommand /home/runner/work/vim/vim/src/if_python.c:1061:5
    #1 0x5586bbde115f in do_pyeval /home/runner/work/vim/vim/src/if_python.c:1526:5
    #2 0x5586bb26b3f0 in f_pyeval /home/runner/work/vim/vim/src/evalfunc.c:8230:5
    #3 0x5586bb24fafe in call_internal_func /home/runner/work/vim/vim/src/evalfunc.c:3145:5
    #4 0x5586bbb1a4b5 in call_func /home/runner/work/vim/vim/src/userfunc.c:3852:14
    #5 0x5586bbb189b8 in get_func_tv /home/runner/work/vim/vim/src/userfunc.c:1943:8
    #6 0x5586bb23fb9f in eval_func /home/runner/work/vim/vim/src/eval.c:2354:8
    #7 0x5586bb23dd4a in eval9 /home/runner/work/vim/vim/src/eval.c:4258:9
    #8 0x5586bb2462e0 in eval8 /home/runner/work/vim/vim/src/eval.c:3819:11
    #9 0x5586bb244e5e in eval7 /home/runner/work/vim/vim/src/eval.c:3623:9
    #10 0x5586bb243250 in eval6 /home/runner/work/vim/vim/src/eval.c:3402:9
    #11 0x5586bb242465 in eval5 /home/runner/work/vim/vim/src/eval.c:3[29](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:30)1:9
    #12 0x5586bb2410a5 in eval4 /home/runner/work/vim/vim/src/eval.c:3142:9
    #13 0x5586bb24011d in eval3 /home/runner/work/vim/vim/src/eval.c:[30](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:31)03:9
    #14 0x5586bb20debf in eval2 /home/runner/work/vim/vim/src/eval.c:2877:9
    #15 0x5586bb20debf in eval1 /home/runner/work/vim/vim/src/eval.c:2723:9
    #16 0x5586bbb17c8e in get_func_arguments /home/runner/work/vim/vim/src/userfunc.c:1859:6
    #17 0x5586bbb1870d in get_func_tv /home/runner/work/vim/vim/src/userfunc.c:1922:11
    #18 0x5586bbb458c3 in ex_call_inner /home/runner/work/vim/vim/src/userfunc.c:5970:6
    #19 0x5586bbb458c3 in ex_call /home/runner/work/vim/vim/src/userfunc.c:6307:11
    #20 0x5586bb2fc180 in do_one_cmd /home/runner/work/vim/vim/src/ex_docmd.c:2582:2
    #21 0x5586bb2f1e2e in do_cmdline /home/runner/work/vim/vim/src/ex_docmd.c:994:17
    #22 0x5586bbb22992 in call_user_func /home/runner/work/vim/vim/src/userfunc.c:3056:2
    #23 0x5586bbb1fce6 in call_user_func_check /home/runner/work/vim/vim/src/userfunc.c:3229:10
    #24 0x5586bbb1a486 in call_func /home/runner/work/vim/vim/src/userfunc.c:3834:15
    #25 0x5586bbb189b8 in get_func_tv /home/runner/work/vim/vim/src/userfunc.c:1943:8
    #26 0x5586bbb458c3 in ex_call_inner /home/runner/work/vim/vim/src/userfunc.c:5970:6
    #27 0x5586bbb458c3 in ex_call /home/runner/work/vim/vim/src/userfunc.c:6307:11
    #28 0x5586bb2fc180 in do_one_cmd /home/runner/work/vim/vim/src/ex_docmd.c:2582:2
    #29 0x5586bb2f1e2e in do_cmdline /home/runner/work/vim/vim/src/ex_docmd.c:994:17
    #30 0x5586bb23af02 in ex_execute /home/runner/work/vim/vim/src/eval.c:7296:6
    #[31](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:32) 0x5586bb2fc180 in do_one_cmd /home/runner/work/vim/vim/src/ex_docmd.c:2582:2
    #[32](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:33) 0x5586bb2f1e2e in do_cmdline /home/runner/work/vim/vim/src/ex_docmd.c:994:17
    #[33](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:34) 0x5586bbb22992 in call_user_func /home/runner/work/vim/vim/src/userfunc.c:3056:2
    #[34](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:35) 0x5586bbb1fce6 in call_user_func_check /home/runner/work/vim/vim/src/userfunc.c:3229:10
    #[35](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:36) 0x5586bbb1a486 in call_func /home/runner/work/vim/vim/src/userfunc.c:3834:15
    #[36](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:37) 0x5586bbb189b8 in get_func_tv /home/runner/work/vim/vim/src/userfunc.c:1943:8
    #[37](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:38) 0x5586bbb458c3 in ex_call_inner /home/runner/work/vim/vim/src/userfunc.c:5970:6
    #[38](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:39) 0x5586bbb458c3 in ex_call /home/runner/work/vim/vim/src/userfunc.c:6307:11
    #[39](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:40) 0x5586bb2fc180 in do_one_cmd /home/runner/work/vim/vim/src/ex_docmd.c:2582:2
    #[40](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:41) 0x5586bb2f1e2e in do_cmdline /home/runner/work/vim/vim/src/ex_docmd.c:994:17
    #[41](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:42) 0x5586bb868438 in do_source_ext /home/runner/work/vim/vim/src/scriptfile.c:1762:5
    #[42](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:43) 0x5586bb866093 in do_source /home/runner/work/vim/vim/src/scriptfile.c:1908:12
    #[43](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:44) 0x5586bb866093 in cmd_source /home/runner/work/vim/vim/src/scriptfile.c:1253:14
    #[44](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:45) 0x5586bb2fc180 in do_one_cmd /home/runner/work/vim/vim/src/ex_docmd.c:2582:2
    #[45](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:46) 0x5586bb2f1e2e in do_cmdline /home/runner/work/vim/vim/src/ex_docmd.c:994:17
    #[46](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:47) 0x5586bbf15154 in exe_commands /home/runner/work/vim/vim/src/main.c:3173:2
    #[47](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:48) 0x5586bbf15154 in vim_main2 /home/runner/work/vim/vim/src/main.c:790:2
    #[48](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:49) 0x5586bbf0d374 in main /home/runner/work/vim/vim/src/main.c:441:12
    #[49](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:50) 0x7fba02429d8f in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #[50](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:51) 0x7fba02429e3f in __libc_start_main csu/../csu/libc-start.c:392:3
    #0 0x[55](https://github.com/vim/vim/actions/runs/6240181405/job/16939704624#step:20:56)86baf62f04 in _start ??:?

Oh, I forgot to run / test it to see what else needed to be fixed after my PR…

#13128

@ychin still undefined-behavior
https://github.com/vim/vim/actions/runs/6252669629/job/16976420178#step:20:29

Ugh, I wish these are compile-time errors rather than runtime errors. Let me turn on Clang 17 locally in a custom branch to make sure it passes CI for my next PR

Ok, #13140 should fix this. I was smarter this time and turned on clang 17 in my local branch's CI and it passed, so hopefully this is the last one.

@chrisbra it's ready to merge


Start with Windows not working

Hello,
I have a G810 and when i tick the checkbox start with windows, nothing happens at windows startup. I have to launch aurora manually.
Is it a known bug or the issue is just by my side ?
Anyway, it's not a big deal. Keep up the good work !!
OS : Windows 10 x64
LGS 8.83.85
Keyboard : G810
Mouse : G500

I have this issue too

Anything in the logs?

nothing too obvious in the log :+1:
[INFO] 18/06/2016 14:23:22: Device, Logitech, was initialized
[ERROR] 18/06/2016 14:23:22: Corsair device, CUE Exception! ErrorCode: ServerNotFound
[INFO] 18/06/2016 14:23:22: Device, Corsair, was not initialized
[INFO] 18/06/2016 14:23:22: No Chroma SDK available
[ERROR] 18/06/2016 14:23:22: Razer device, Exception! Message:System.Exception: No Chroma SDK available
à Aurora.Devices.Razer.RazerDevice.Initialize() dans d:\Repositories\Git\Aurora\Project-Aurora\Devices\Razer\RazerDevice.cs:ligne 52
[INFO] 18/06/2016 14:23:22: Device, Razer, was not initialized
18/06/2016 14:23:24: Listening for game integration calls...
[INFO] 18/06/2016 14:43:43: Device, Logitech, was shutdown

Can I see the entire log? In the beginning it shows os info and launch parameters.

same issue here, didn't manually restart the program and it gave me this log file after launching windows:
========================================
Operation System: Windows 10 Home
System Architecture: 64 bit
Environment OS Version: Microsoft Windows NT 6.2.9200.0
System Directory: C:\WINDOWS\system32
Processor Count: 4
User DomainName: xxxx-HOME
User Name: xxxx
SystemPageSize: 4096
Version: 4.0.30319.42000
========================================

[INFO] 18/06/2016 12:59:26: Program started with '-silent' parameter
[INFO] 18/06/2016 12:59:26: Program started with '-delay' parameter with delay of 5000 ms
[INFO] 18/06/2016 12:59:32: Device, Logitech, was initialized
[ERROR] 18/06/2016 12:59:32: Corsair device, CUE Exception! ErrorCode: ServerNotFound
[INFO] 18/06/2016 12:59:32: Device, Corsair, was not initialized
[INFO] 18/06/2016 12:59:32: No Chroma SDK available
[ERROR] 18/06/2016 12:59:32: Razer device, Exception! Message:System.Exception: No Chroma SDK available
   at Aurora.Devices.Razer.RazerDevice.Initialize() in d:\Repositories\Git\Aurora\Project-Aurora\Devices\Razer\RazerDevice.cs:line 52
[INFO] 18/06/2016 12:59:32: Device, Razer, was not initialized

but Aurora was not launched at all

========================================
Operation System: Windows 10 Home
System Architecture: 64 bit
Environment OS Version: Microsoft Windows NT 6.2.9200.0
System Directory: C:\WINDOWS\system32
Processor Count: 8
User DomainName: XXX-DESKTOP-PC
User Name: xxx
SystemPageSize: 4096
Version: 4.0.30319.42000
[INFO] 18/06/2016 22:03:21: Device, Logitech, was initialized
[ERROR] 18/06/2016 22:03:21: Corsair device, CUE Exception! ErrorCode: ServerNotFound
[INFO] 18/06/2016 22:03:21: Device, Corsair, was not initialized
[INFO] 18/06/2016 22:03:21: No Chroma SDK available
[ERROR] 18/06/2016 22:03:21: Razer device, Exception! Message:System.Exception: No Chroma SDK available
à Aurora.Devices.Razer.RazerDevice.Initialize() dans d:\Repositories\Git\Aurora\Project-Aurora\Devices\Razer\RazerDevice.cs:ligne 52
[INFO] 18/06/2016 22:03:21: Device, Razer, was not initialized
18/06/2016 22:03:22: Listening for game integration calls...

I suspect it's an issue with Keyboard Layouts. Will look into it after profile import/export.

Issue has been located. Turning off Skype Overlay will resolve the issue. It will be fixed in the next version.


Azure Pipelines CI

Adds Azure Pipeline builds for PRs, Merges, and Docker.
The workflows defined the Azure Pipelines currently match the Jenkins
workflows:

PRs are build and tested
Merges are build, tested, and published to the snapshot repo
Docker container are build every time changes are merged under the
docker directory, and rebuilt every day

With one main difference being artifacts are published to Artifactory
(odpi.jfrog.io) instead of Nexus.
Signed-off-by: Trevor Bramwell<EMAIL_ADDRESS>

Any last comments on this? I plan to merge tomorrow if not & we can track improvements via new issues as for now this will run in parallel


Bug: locally orderLineUpdate mutation return different error structure than on stagging

What are you trying to achieve?
I wanted to handle errors from orderLineUpdate mutation in the same way locally and on stagging.
Now when I send not valid quantity in orderLineUpdate, locally I get:
 {
  "errors": [
    {
      "message": "Int cannot represent non 32-bit signed integer value:<PHONE_NUMBER>56",
      "extensions": {
        "exception": {
          "code": "GraphQLError"
        }
      }
    }
  ]
}

When I try to do the same on stagging I get:
{
  "errors": [
    {
      "message": "Internal Server Error",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "orderLineUpdate"
      ],
      "extensions": {
        "exception": {
          "code": "DataError"
        }
      }
    }
  ],
  "data": {
    "orderLineUpdate": null
  },
  "extensions": {
    "cost": {
      "requestedQueryCost": 5,
      "maximumAvailable": 50000
    }
  }
}

Steps to reproduce the problem

Go to the master stagging dashboard
Go to order drafts
Create a new order draft and add products for them
Change quantity for a big number
You should get an error notification because the error structure is as we expect it to be
Run Dashboard locally or go to an environment created by Pull Request
Repeat the same steps
You should not see any notification because error structure is bad

What did you expect to happen?
API should return the same error structure no matter on which environment
Logs
No response
Environment
Saleor version: 3.12
OS and version: Mac os Ventura

It is expected as Saleor keeps internal errors on production envs secret. The problem is the original message: Int cannot represent non 32-bit signed integer value:<PHONE_NUMBER>56 and this should be investigated in core. Related trace:
https://saleor.sentry.io/issues/4024668381/?project=6417854&referrer=slack


Fixed comparability with unicode-math 0.8d

As of unicode-math 0.8d (see unicode-math README.md) the vargreek-shape option has been removed for compatibility. As such, I have removed the use of this option in the template.

Thanks for the contribution! Merged.


fix date >= simplify to > by mistake

pick from master #30765
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Big changes with async, synchronous functionality deleted and callback is as well

Because it is possible to get Task results sync myTask.Wait() or myTask.Result

Honestly, I wouldn't know how you would create the examples now, I've changed it to fit your code, but the examples no longer print the result.


Installing SAP HANA 2.0, express edition (Virtual Machine Method)

Hi,
I was following your instruction to install HANA 2.0 Express version. Everything was working fine, only when I tried to access webide/xsa, every time I ran into an authorization request error  as shown following:

Do you know this issue already?  Could you please help me to figure out how to fix it? Thanks!

Hi, is that with XSA_DEV user? Have you checked if the user has the proper role collections?

Closing due to lack of response


Contact Open Austin and BASTA as partner

Related to #85
Not sure if this is for your internal team, but I figured it'd be the quickest way to reach y'all. I am formally reaching out to you all on behalf of BASTA and Open Austin.
We have developed a court scraper for the OdysseyPa system which host a large portion of Texas counties court filings. We currently have developed a stable version for Travis County to address our tenant relief/support/organizing efforts in Austin but are moving on to many other counties in Texas to build a system similar to the court.bot statewide.
In the interest of not reinventing any wheels I would like to chat with you all about integrating our systems/data with yours, I am excited to see what we can come up with!

Email me at dvarnes at techlahoma.org. I already know Liani in Austin!


Regex option - could you add more info/examples in readme?

First, thank you for this card and component. I am finally able to use RSS the way I've wanted in my HA install.  It was what I had hoped the stock RSS component would do but never did.  Bless you, sir!!
But I am not quite sure what the regex option does and if it would allow me to filter out entries in the list table.  When you get a chance could you elaborate a little more in the docs on usage and perhaps provide an example or two?
If this can be used as a filter, this would solve an issue for me but I'm not sure how to go about it.  I'd like to filter out any entry beginning with "NYSTA" or "NYSDOT" but not sure if understanding how this works would be the answer or not.
Thanks again for an awesome feature enhancement to HA!

The regex feature currently is just for display of the field in question, it does not filter actual entries based on any regex, currently.

Bummer.  Still would like to know more about how it works, even if it doesn't solve my problem.  I have to assume you had a good reason for including it as a feature in the original release and it seems a shame for me not to put it to use! :)

Thanks for taking the time.  I can see where that would be useful - maybe you could include this example in the readme for others.  There are plenty of lurkers who aren't comfortable admitting they don't know.  I don't suffer from this delusion so I have no problems asking; I'd rather learn something.  LOL
Again, I really appreciate this combo and I'm looking back at ideas I had for incorporating data that I couldn't do before.   Thanks.  If you ever consider adding filters (if that's even possible), please consider me a willing beta tester!


Added Mo's algorithm [C++]

Fixes #28
By submitting this pull request I confirm I've read and complied with the below declarations.

[x] I have read the Contribution guidelines and I am confident that my PR reflects them.
[x] I have followed the coding guidelines for this project.
[x] My code follows the skeleton code structure.
[x] This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
[x] This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.

I have added the code according to the structure proposed in the issue

Hi! Thanks for showing interest in contributing to this repository.
Please make sure that you have ticked the points mentioned in PR description correctly.
Thanks again!

@Monal5031 whats the issue can you explain?

@sirjan13  It seems you are using space indentation, while this repo is configured to use tab spaces for c and cpp. Use tav indentation throughout the code and travis test will pass.

That's right @singhpratyush sir, We should not change the pattern.

I am unable to fix the indentation everytime i update it fails the tests :(

@sirjan13: You just need to use tabs over spaces.

@sirjan13 If you want some help in indenting the code properly you can use coala locally, the command to run it locally on algos directory as
coala --apply-patches -n

but first you need to install coala using
pip3 install coala-bears

This will tab indent the code itself.

Thanks for helping. 👍
Will surely update it by today :)

Closing due to inactivity.


eval: add --cue option

Partially Resolves #197
Adds the --cue option to the eval utility. This allows jammed nouns to be passed over stdin and pretty-printed to stdout.
Also includes a sub-option which will expect the incoming jammed noun to be Khan output. If the thread resulted in a goof, it's pretty-printed to stderr as a stack trace and the output is simply the conn.c header info (unique ID, %avow mark signalling Khan output, %.n loobean signalling thread failure).
Also includes the "todo"s from the comments for #155.
Testing:

Built Vere in debug mode; no new warnings introduced
Manually ran the following test cases in the terminal:

echo '42' | ~/Urbit/urbit eval -jn | ~/Urbit/urbit eval -cn
echo '[[1 2] [3 4]]' | ~/Urbit/urbit eval -jn | ~/Urbit/urbit eval -cn
echo "(cat 3 'abc' 'def')" | ~/Urbit/urbit eval -jn | ~/Urbit/urbit eval -cn

I've done more strenuous testing, but it's hard to explicitly list here since it relies on a thin client which will be committed as a separate PR after this one is merged

Not strictly necessary for this PR, but I would like to add explicit testing for both --jam and --cue. @mcevoypeter @matthew-levan If either of you can recommend how to modify Bazel to do so, it would be appreciated.

Not strictly necessary for this PR, but I would like to add explicit testing for both --jam and --cue. @mcevoypeter @matthew-levan If either of you can recommend how to modify Bazel to do so, it would be appreciated.

For testing the behavior of the command line application itself, I think the best way to do this is by writing a shell script that invokes --jam and --cue and compares the generated output to the expected output. We can then wrap that shell script in a sh_test target in Bazel.

When @joemfb removes his changes requested we'll reconsider merging this. For now, looks like this is still under deliberation.


SetValuesCommand/SetValuesRequest has additional property "columnName"

The SetValuesCommand has an additional property columnName in future. If not sent, server behaviour does not change.
If columnName is sent, the server will additionally check, if the specific column is read only, and will open an Error View.
The columnName is the column, that is edited (caused the SetValuesCommand).

The property is now renamed to editorColumnName

What if a LinkedCellEditor changes 2 columns? Which column should be named. The primary column of the editor or both edited columns?

The bound column.... every editor has 1 column.


evaluate_model() gives parsing error

I am trying to evaluate a model using evaluate_model()
evaluate_model("some.json")
but it gives a ParseException.
some.json-
{
    "model_class": "BidirectionalAttentionFlow",
    "model_serialization_prefix": "/models/bidaf",
    "encoder": {
        "word": {
          "type": "cnn",
          "ngram_filter_sizes": [5],
          "num_filters": 100
        }
    },
    "seq2seq_encoder": {
        "default": {
            "type": "bi_gru",
            "encoder_params": {
                "units": 100
            },
            "wrapper_params": {}
        }
    },
    "data_generator": {
      "dynamic_padding": true,
      "adaptive_batch_sizes": true,
      "adaptive_memory_usage_constant": 440000,
      "maximum_batch_size": 60
    },
    "patience": 3,
    "embeddings": {
      "words": {
        "dimension": 100,
        "pretrained_file": "/trained_vectors/glove.840B.300d.txt",
        "project": true,
        "fine_tune": false,
        "dropout": 0.2
      },
      "characters": {
        "dimension": 8,
        "dropout": 0.2
      }
    },
    "num_epochs": 20,
    "optimizer": {
      "type": "adadelta",
      "learning_rate": 0.5
    },
    "validation_files": ["dev_squad.json"],
    "train_files": ["train_squad.json"]
}

This is the stacktrace-
Using TensorFlow backend.
processor = {}
processor.word_splitter = simple
processor.word_filter = pass_through
processor.word_stemmer = pass_through
0 1175
Traceback (most recent call last):
  File "D:/backup/PycharmProjects/test/deep_qa/with_qa.py", line 79, in <module>
    evaluate_model("some.json")
  File "D:\backup\PycharmProjects\test\deep_qa\run.py", line 214, in evaluate_model
    model = load_model(param_path, model_class=model_class)
  File "D:\backup\PycharmProjects\test\deep_qa\run.py", line 147, in load_model
    param_dict = pyhocon.ConfigFactory.parse_file(param_path)
  File "C:\Users\mohit.badwal.NOTEBOOK546\Anaconda3\lib\site-packages\pyhocon\config_parser.py", line 51, in parse_file
    return ConfigFactory.parse_string(content, os.path.dirname(filename), resolve)
  File "C:\Users\mohit.badwal.NOTEBOOK546\Anaconda3\lib\site-packages\pyhocon\config_parser.py", line 90, in parse_string
    return ConfigParser().parse(content, basedir, resolve)
  File "C:\Users\mohit.badwal.NOTEBOOK546\Anaconda3\lib\site-packages\pyhocon\config_parser.py", line 272, in parse
    config = config_expr.parseString(content, parseAll=True)[0]
  File "C:\Users\mohit.badwal.NOTEBOOK546\Anaconda3\lib\site-packages\pyparsing.py", line 1216, in parseString
    raise exc
  File "C:\Users\mohit.badwal.NOTEBOOK546\Anaconda3\lib\site-packages\pyparsing.py", line 1210, in parseString
    se._parse( instring, loc )
  File "C:\Users\mohit.badwal.NOTEBOOK546\Anaconda3\lib\site-packages\pyparsing.py", line 1072, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "C:\Users\mohit.badwal.NOTEBOOK546\Anaconda3\lib\site-packages\pyparsing.py", line 2545, in parseImpl
    loc, exprtokens = e._parse( instring, loc, doActions )
  File "C:\Users\mohit.badwal.NOTEBOOK546\Anaconda3\lib\site-packages\pyparsing.py", line 1076, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "C:\Users\mohit.badwal.NOTEBOOK546\Anaconda3\lib\site-packages\pyparsing.py", line 2348, in parseImpl
    raise ParseException(instring, loc, self.errmsg, self)
pyparsing.ParseException: Expected end of text (at char 0), (line:1, col:1)

Where am I going wrong ?

I put that file into https://jsonlint.com/, and it said it was valid json.  I'm not sure why pyhocon is complaining when parsing it.  When I copy that file and run evaluate_model("some.json") on my linux machine, it also parses (it then crashes because it doesn't have write permission on /models, but that's a different issue).
My best guess is that pyhocon behaves differently with respect to line endings or something on windows, so it isn't parsing the file correctly.  Also, I'm pretty sure there are plenty of parts of this code that will break when running on windows.  See #335.


Rxjs 6 error

ERROR in node_modules/@ngu/carousel/src/ngu-carousel.service.d.ts(1,10): error TS2305: Module '".../ui/node_modules/rxjs/Subject"' has no exported member 'Sub
ject'.
node_modules/rxjs/Subject.d.ts(1,15): error TS2307: Cannot find module 'rxjs-compat/Subject'.
Please update the Rxjs and Angular to version 6.

duplicate #43

Install rxjs-compat

Please, update the Rxjs and Angular to version 6.
In the meantime, I have installed the rxjs-compat library. However, this will disappear when the version 7 of Rxjs comes out.

Yes, hope can support new version of rxjs, thanks  a lot~

@gfoo @beyondtheobservableuniverse @spartanPAGE @i5coding @EddieBenji Hi,
I also want to upgrade but it will only support angular 6 projects only. Then i have to drop support for angular 5 projects

As suggested by spartanPAGE, we could install rxjs-compat as a temporary solution:
npm install rxjs-compat --save

Hi @sheikalthaf, do you think in the near future you could update to angular 6?
Why don't you create a new repo where you can maintain the code for angular 6 projects?
Thank you.

@sheikalthaf just make it in separate branch and publish new major version to npm. It's annoying that we forced to use rxjs-compat for a single lib with one rxjs import. rxjs-compat using increases bundle size

I don't think it's a good idea to have the project dependency on rxjs-compat library lying around. As most of us are learning the new rxjs v6 library it's too easy to to slip back into the old way of doing things without it raising a compile time error. A bump to v6 would be highly appreciated.

Looks like last version ( v1.4.9-beta-2) does not use rxjs at all so we can remove rxjs-compat usage with it (already tried and it works)

@artaommahe @ImranAhmed @EddieBenji @DukeCaesar @i5coding @beyondtheobservableuniverse @spartanPAGE Updated the carousel to latest version of the angular. Please check release. Thanks

Thank you @sheikalthaf !
Will check!


box-sizing: border-box is missing on input field for multiple selection

Please add box-sizing: border-box as follows:
.select2-container--default .select2-selection--multiple .select2-selection__rendered {
    list-style: none;
    margin: 0;
    padding: 0 5px 5px 5px;
    width: 100%;
    box-sizing: border-box;
}

Otherwise the width is not correct calculated with width: 100% + padding / margin. Please note, padding is also corrected as 0 5px 5px 5px;. In the showcase, this setting is done for all elements in the bootstrap.css. But I don't have bootstrap.css.
I already reported a similar issue for search field https://github.com/select2/select2/issues/2978
Thanks a lot!

I also had some empty space between input field and the dropdown (multiselect onyl). This problem was due to display: inline-block for class .select2-container .select2-selection--multiple .select2-selection__rendered. I changed it to display: block as follows
.select2-container .select2-selection--multiple .select2-selection__rendered {
    display: block;
    overflow: hidden;
    padding-left: 8px;
    text-overflow: ellipsis;
}

and the empty space was gone. Can you consider this fix too? Thanks.

Thanks. Did you also add display: block for .select2-selection__rendered?

Nope, I also missed the padding issue. I'll fix those both later, but I'll reopen this for now.

Closing this off again.

Last but not least my settings for select2-selection__choice (simple compare with current defaults).

The individual selections start to look quite large with the added top/bottom padding.

This problem was due to display: inline-block for class .select2-container .select2-selection--multiple .select2-selection__rendered.

Couldn't reproduce the issue, and changing it to display: block caused the padding to be removed.

I'd need to see jsfiddles/jsbins which showed the difference with these slight changes to see what the impact is.


Add DODO to Wallet Web page

Title: DODO
Desc: Trade any asset with the best liquidity on multiple chains
Website:  http://app.dodoex.io/?utm_source=AlphaWallet
Section: BSC/Ethereum/Exchanges/Polygon
Logo：

n

Hi @colourfreak ，any update?


Containerize

Motivation
This repository makes use of sf, which, in turn, depends on still-developing system packages such as gdal, geos, and proj. This ecosystem is still evolving, and conflicts can arise. For instance, I was recently unable to run code on a branch of this repo due to such conflicts.
While a given version of sf could be enforced using something like renv, the underlying system packages cannot be managed at that level. Containerizing the workflow enables us to standardize it on specific system library versions in addition to specific R package versions, without anybody having to change their local setup (and thereby possibly mess up the ability to run other code!)
Summary of changes
This provides a docker image based on R 4.1.2, sf 1.0.7, GEOS 3.8.0, GDAL 3.0.4, and PROJ 6.3.1, which can run this project's code. The image provides an RStudio interface.

Added a docker directory with the necessary code for building the image
Added some text to the README.md saying how to run in this environment
Pushed the docker image to Docker Hub for now (if this repo is transferred to code.usgs.gov, we will set up CI to build and host the image there instead of Docker Hub)

Validation of changes
I used this container to run the pipeline successfully.
Outline of what is/is not expected from the reviewer
This is in some ways a low-stakes PR, since it does not change any of the code, and only adds to the README. However, it would be nice for a reviewer to make sure that they can pull the container and run code in it following the instructions in the README. Ideally, a full pipeline run would be a great test. I'm assigning this to @padilla410 since she has been ramping up on docker, but @amart90 might be a suitable reviewer as well!

I should be able to get to this later this afternoon or early Monday

Howdy Jesse,
I was successfully able to pull down this docker container and get things runngin while on VPN. I have a few little follow up docker questions for you, but they are unrelated to the approval of this PR.


Dev

@kbab
Here is an attempt at helping with your snakemake "resumability" problem.
You add a touch(file) flag output that snakemake will generate when the script doesn't crash, you then use that as an input to the next rule
https://snakemake.readthedocs.io/en/stable/snakefiles/rules.html#flag-files
P.s. I've not tested this feel free to commit to this branch

@kbab Did this help? Can I close/merge?

@ctr26 Yes it did - it was very helpful! Apologies for not replying sooner. You can close, no need to merge, I included it already. Thanks!


Movieclip disappears when flipping with away3d

This is a weird bug i've found and it was hard to reproduce it.
When flipping movieclip by scaleX = -1 or with matrix matrix.a = -1 movieclip disappears for both HTML5 and Native. Flash is working fine. (it applies for both horizontal and vertical flipping)
It happens only if view.render is called on enterframe.
When disabling 3d it works fine.
This bug is present for more than 6 months and was tested with latest versions of openfl and lime from git.
Here is a sample project to reproduce it. (just click on the image to flip it)
FlipBug.zip

Fix: https://github.com/openfl/openfl/commit/b514f95b0f3b5ef2f469e840c06a02d434d7749a
We need to handle state between Stage3D and our core renderer better :sweat:

Thank you!


Improve documentation on io module

Including detailed explanation of dims, units.

Add clear description of parquet files reading, because indexes are not included (so when converting csv to parquet extreme care for indices is required)


package shodan assertFalse

1.0rc6
Commercial: None
Python: 3.9.2 (default, Feb 28 2021, 17:03:44)
Flavor: Debian Python
Executable: /root/cc/bin/python
OS: Linux
Arch: x86_64
Distribution: Debian 11
I use shodan for some test. and I installed it by
pip install shodan
When I type in the command python3 -m nuitka --standalone sd.py
then cause AssertionError
Here is the Traceback
Nuitka:INFO: Generating source code for C backend compiler.
C Source Generation: 54.5%|█████████████▋           | 42/77, shodan.helpers  Problem with <Node 'STATEMENT_EXPRESSION_ONLY' with {'expression': 'EXPRESSION_STR_OPERATION_FORMAT'}> at /root/cc/lib/python3.9/site-packages/shodan/helpers.py:167
Problem with <Node 'STATEMENT_CONDITIONAL'> at /root/cc/lib/python3.9/site-packages/shodan/helpers.py:166
Problem with <Node 'STATEMENT_TRY' with {'aborting': True}> at /root/cc/lib/python3.9/site-packages/shodan/helpers.py:145
Nuitka:WARNING: Problem creating function code 'shodan$helpers$$$function__8_humanize_bytes'.

Thanks!
Update:  Just try this simply test.py with python3 -m nuitka test.py
def humanize_bytes(byte_count, precision=1):
    if byte_count == 1:
        return '1 byte'
    if byte_count < 1024:
        '{0:0.{1}f} {2}'.format(byte_count, 0, 'bytes')

    suffixes = ['KB', 'MB', 'GB', 'TB', 'PB']
    multiple = 1024.0  # .0 to force float on python 2
    for suffix in suffixes:
        byte_count /= multiple
        if byte_count < multiple:
            return '{0:0.{1}f} {2}'.format(byte_count, precision, suffix)
    return '{0:0.{1}f} {2}'.format(byte_count, precision, suffix)

That is indeed correct, and interestingly happening with the stable version too, I was kind of assuming this to be a regression of develop, as we touched the C types usage a lot there.

The following upstream code is rather unusual:
    if byte_count < 1024:
        '{0:0.{1}f} {2}'.format(byte_count, 0, 'bytes')

The result of format is not used, you might want to tell them about it. That however explains, why we have escaped noticing the bug so far. In this case, a built-in str.format is used without the proper object to result type conversion, which then runs into attempting to release a void value.

This is fixed on factory branch: https://nuitka.net/doc/factory.html
I need to contemlate making another hotfix, unfortunately I just did one earlier today.

Thanks!

This is part of 1.0 release.

Actually was already in 0.9.6 as well.


Package manager error on install

Installation Failed
Package Manager error
The package manager could not make changes to the installed system. The command
pacman -S --noconfirm linux linux-headers base-devel returned error code 1.
I selected no packages during install. Any idea how to fix?

I apologize for this, we're having an internal rewrite to an artix base along with wayland compositors which is taking some time. This bug will be gone once we get that finished,


Career menu start level select, menus, should work with controller

Currently, players using a controller cannot change the start level. These menus should be navigable, perhaps by arrowing up to the level select arrows. Or perhaps, arrowing to the left of the leftmost level should select an earlier set of levels.
Either way, there is a second issue where clicking one of the level select arrows un-highlights the currently selected level. One of these levels should always be highlighted.

The career menu has been redesigned. Players now select a chapter instead of using the "left/right" arrows at the top of the career map.


error load ui/app.js:14

i have this in F8 console

i don't see the error at line 14 !! you can help me please ?
i have the original file.

I have same problem too

Fixed in the latest version.

No fix, the 3.20 version still has this problem

does mysql in f8 console work, or does the entire vuejs app crash?
the backend and client works, just the question in how far this empty message affects the GUI for checking the performance of resources.

I also have this error and mysql does work for me.

Hello, I also have the same problem, when I do F8. When we do the mysql command in F8 there are just empty graphics, here are some screenshots:


BUGFIX: Fix uninitialized cursor_shape on windows display server

Simple bug-fix where cursor_shape is uninitialized on windows, and you get a screaming error message on debug.

Thanks!

Thanks!


Add revert_error field

This change is

crates/blockifier/src/transaction/transaction_execution.rs line 100 at r1 (raw file):
Previously, dorimedini-starkware wrote…
I will add positive + negative tests for the implemented feature; what do I want todos for?
Specifically, this is an L1 handler tx, no need to revert the sequencer txs I think.
I'll ask Yoni&Elin

I didn't mean to revert the sequencer. I meant that if/when it sequences reverted txs then revert_error won't be a hard-coded None. So it'd be a TODO to change it when the time comes.
Maybe this comment is more appropriate for the hard-coded None in account_transaction.rs where it's proper L2.


Fix matching-rule docs.

It should say any instead of all.

Coverage decreased (-0.5%) to 86.741% when pulling 31c83d9434a7e76f7062822d359defdc9be21f99 on MrQubo:patch-1 into 5abf507a7cba6a61777aaa1921edfd463185cea3 on Grokzen:master.

Very interesting, I'm not sure how it's affecting the coverage.


Please translate the FAQ into English

Please translate the FAQ into English
the FAQ issue are as follows, https://github.com/InterestingLab/seatunnel/issues/267

where to put the doc? just create another issue or add it to code repository?

where to put the doc? just create another issue or add it to code repository?

you can submit a PR to https://github.com/InterestingLab/seatunnel-docs root directory, named FAQ.md

where to put the doc? just create another issue or add it to code repository?

you can submit a PR to https://github.com/InterestingLab/seatunnel-docs root directory, named FAQ.md

OK, I want to do this.

@wolfboys which directory do you think I should put the FAQ documents under the website project?

@wolfboys which directory do you think I should put the FAQ documents under the website project?

@wolfboys which directory do you think I should put the FAQ documents under the website project?

please put the doc to this dir https://github.com/apache/incubator-seatunnel-website/tree/main/docs, happy new year

@wolfboys which directory do you think I should put the FAQ documents under the website project?

please put the doc to this dir https://github.com/apache/incubator-seatunnel-website/tree/main/docs, happy new year

PR: https://github.com/apache/incubator-seatunnel-website/pull/46

@dailidong LGTM. please PTAL.

done


Update the UCM API URL to match new scheme

Update UCM Desktop to connect to the API on
http://<IP_ADDRESS>:5858/codebase/api reflecting the recent UCM change (https://github.com/unisonweb/unison/pull/5461) to remove the autogenerated port and token in favor of defaults.

@hojberg at the moment, I cannot run ucm and then open this app to connect to ucm - I need to follow instructions from the readme.
Will this PR fix this?

@DejanMilicic hmm not quite sure. I can't really think of a reason for why this would have stopped working. Which version of UCM are you using?

ucm 0.5.29
if I start it just with ucm I get this

Just to remove any ambiguity, this is what Readme suggest for Windows users
$env:UCM_TOKEN="asdf"; $env:UCM_PORT="4444"; ucm --allow-cors-host http://tauri.localhost

are all of these still needed?
will it be possible to integrate these defaults into ucm?

@DejanMilicic all of those have been integrated in UCM with this https://github.com/unisonweb/unison/pull/5461, but haven't been released yet. I'm not sure what has happened for this to stop working on Windows.
Maybe try with https:
$env:UCM_TOKEN="asdf"; $env:UCM_PORT="4444"; ucm --allow-cors-host https://tauri.localhost

I'm going to merge this PR at least, and hold off on a release until UCM has released with the https://github.com/unisonweb/unison/pull/5461 changes.


Cannot programm TangNano wrong id-code

I set
DEVICE to GW1N-LV1QN48C6/I5
BOARD to tangnano
and create pack.fs from blink example.
When I use GOWIN programmer version <IP_ADDRESS> got error:

There is two tangNano 1k: one based on an GW1N-1 (the original) and one (the new revision) with a GW1NZ-1
Could you check idcode from your board ?:

GW1NZ-1: 0x0900281B
GW1N-1: 0x0100681B

ID code is
Info:	Cable found:  USB Debugger A/0/10545/null (USB location:10545)
Info:	Cost 0.1 second(s)
:
Info:	Target Cable: USB Debugger<EMAIL_ADDRESS>Info:	Operation "Read Device Codes" is starting on device-1...
Info:	ID code is: 0x0900281B
Info:	User code is: 0x0000496F
Info:	Status code is: 0x0001F020
Info:	Cost 0.54 second(s)
So looks like GW1NZ-1 but on chip package there is GW1N-LV1QN48C6/I5:

I change DEVICE to GW1NZ-LV1QN48C6/I5 end rerun:
nextpnr-gowin --json blinky.json --write pnrblinky.json --device $DEVICE --cst $BOARD.cst
gowin_pack -d $DEVICE -o pack1.fs pnrblinky.json

But got simmilar issue:

My fault: I've inverted idcode... It's a GW1N-1. Sorry.
Could you share your .fs (the one targeted for GW1N-LV1QN48C6/I5) ?
Thanks

pack.zip

the idcode contained into pack.fs is okay. I have tested your file with my tangnano: it works (but haven't used proprietary tool).
Seems the issue isn't apicula related but due to the programmer...

At the moment both tangnano (GW1N-1) and tangnano1k (GW1NZ-1) flash without problems.

I met similar issue that Gowin native programmer refuses to use bitstream file after gowin_pack.
It seems that native gowin programmer needs header like that:
//Copyright (C)2014-2022 Gowin Semiconductor Corporation.
//All rights reserved.

I am not opposed to generating a header in apicula, but couldn't get their programmer to work at all.
Would you be interested to make a pr for this?
I expect not all headers are required, so it'd take a bit of experimentation which parts are required and how to fill them in from apicula


Investigate alternatives for handling migrations

We've had a fair amount of pain with our current solution, especially when it comes to needing manual intervention on failures.
Let's investigate alternatives that fit our needs better.

I started this work last weekend and narrowed the playing field to Atlas which was brought up in an issue by our community but also the migrations that are supported by sqlc:
atlas
dbmate
golang-migrate
goose
sql-migrate
tern
Features that I am looking for:

Migrations that can be run from command line as well as from code
Migration history table that gives a full history of migrations
Sequential migrations so we do not have the same issues we have today with date driven migration order
Recovery paths that do nor require manual intervention
Does not enforce down migrations but simply attempt to run migrations again. This would allow us to mend a broken migration and deploy it again without having to manually intervene like we do today with golang-migrate
Allows for concurrent index creation
I have ruled out Atlas as it requires its own ddl and this is a personal preference from bad experiences with similar solutions to not choose this over native declaration.

I'd also like to add this to the list: https://github.com/graphile/migrate since we're already using graphile at the GraphQL API layer, it might be worth investigating their approach to migrations as well

Good candidate! Interesting to see the compliment comes from something postgraphile doesn't really encourage for prod.

complements PostGraphile — works with any application, but PostGraphile's watch mode means that the GraphQL schema is instantly regenerated (without server restart) whenever the database changes

We have a very good alternative the has been added to sqlq and I am proposing we continue to use the solution we have now until we have an internal tool we can consume.


vignettes filename with encoded characters

Unicode characters and special characters shows as %
in app.js variable <data.name> in line 133.

fixed in last commit


Adding two more classes to the default blacklist

Also adding

@eirbjo I've added some additional items to this pull request to allow me to dynamically change aspects of the agent at runtime.


Test examples of part of the continuous integration

I played around a bit more with the program and found an example that doesn't run.

smot color branch mono --factor-by-field 1 -c colormap foo.tre | smot push

in smot color -h
There might be more errors in other places.
It would be good to add the examples to your continuous integration set up to ensure the documentation is always providing correct examples.
This is part of https://github.com/openjournals/joss-reviews/issues/4193.

The examples are correct, you just need to substitute real files. I assume the error message that was raised said that the colormap or foo.tre files did not exist?

I just added automatic tests of the examples. And actually you were right about that example being incorrect, it needs to be smot color push.


libfmvoice: 0-unstable-2024-05-30 -> 0-unstable-2024-06-06

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for libfmvoice is: C library for loading, saving and converting FM sound chip voice files in various formats
meta.homepage for libfmvoice is: https://github.com/vampirefrog/libfmvoice
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0-unstable-2024-06-06 in filename of file in /nix/store/08n8li1h1gc6ag8yvqxrvl96ybsc7zjy-libfmvoice-0-unstable-2024-06-06

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

fmtoy
libfmvoice
vgm2x

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/08n8li1h1gc6ag8yvqxrvl96ybsc7zjy-libfmvoice-0-unstable-2024-06-06 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A libfmvoice https://github.com/r-ryantm/nixpkgs/archive/e92684942d6bb8c5ee8a13cc95f44bd614b0357b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/e92684942d6bb8c5ee8a13cc95f44bd614b0357b#libfmvoice

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/08n8li1h1gc6ag8yvqxrvl96ybsc7zjy-libfmvoice-0-unstable-2024-06-06
ls -la /nix/store/08n8li1h1gc6ag8yvqxrvl96ybsc7zjy-libfmvoice-0-unstable-2024-06-06/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package failed to build:

fmtoy

2 packages built:

libfmvoice
vgm2x

Maintainer pings
cc @OPNA2608 for testing.

Followup in https://github.com/NixOS/nixpkgs/pull/318071. Sorry for not seeing the build failure.


Support for a function as a Route's handler

R: @addyosmani @gauntface
Fixes #231

Sorry for the slow turnaround—PTAL.


Python: add additional handlebars test, to make sure built-in helpers run

Motivation and Context

added handlebars tests
Description

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

Python 3.8 Test Coverage Report • FileStmtsMissCoverMissingTOTAL5399101281% report-only-changed-files is enabled. No files were changed during this commit :)
Python 3.8 Unit Test Overview

Tests
Skipped
Failures
Errors
Time

1205
11 :zzz:
0 :x:
0 :fire:
21.330s :stopwatch:


Add dynamic target entropy ratio

Good evening,
As I understand, entropy loss makes alpha and so the entropy bigger, when entropy is lower than the target entropy.
And when entropy is bigger the the target entropy, it lowers alpha
So by setting target_entropy_ratio we are setting exploration / exploitation ratio.
For example for ter = 0.98 (2 actions),
the optimal probability would be 50.6 percents.
>>> 2.71828 ** (log(0.5) * 0.98)
0.506979971545461

Only 1 per cent for manoeuvre
For 0.7 it would be
0.6155724075787216
I think that users should be informed about this, for example this info in some way could be added to readme.md
Also, to lower exploitation/exploration ratio, we could lower ter by using scheduler. This could increase exploitation in the last phase of training.
Would be happy to hear your comments.
Best regards,
Vadims.

I think that users should be informed about this, for example this info in some way could be added to readme.md

Thank you for sharing.
I will do it later.

Also, to lower exploitation/exploration ratio, we could lower ter by using scheduler.

Maybe good to add it. (I will consider it once I have time.)


chore: add build to pr checking

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Is it possible to draw an image drawn on the pdf view? - Question

Hi
i am trying to render an imageview ontop of the pdfview,,and then saving both together as merged, into a pdf document, but it doesnt seem to work. I wanted to know if there is a way i can allow a user to draw on a page, and then save the pdf. Please assist. It is a little urgent, all help will be appreciated.
Thanks

You can use OnDrawListener to draw on pdf, but there is not possibility of saving edited documents.

is there a way i could save that ? may be by writing some code or something ?


Add coverage report

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@2126d18). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 70bbf32 differs from pull request most recent head 801fa17. Consider uploading reports for the commit 801fa17 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage        ?   89.05%
=======================================
  Files           ?       97
  Lines           ?     2174
  Branches        ?        0
=======================================
  Hits            ?     1936
  Misses          ?      214
  Partials        ?       24

Flag
Coverage Δ

unittests
89.05% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Failed to sign package; error was:<PHONE_NUMBER>

Trying to publish a simple package and receive error: Operation Failed with Error: Failed to sign package; error was:<PHONE_NUMBER>
from the %LocalAppData%\Temp log file I see this.
**** System.InvalidOperationException : Failed to sign package; error was:<PHONE_NUMBER>
I see a closed issue where it said "Also, ensure your WSUS server can access Internet to use the timestamping option."
We are in a very locked down environment and use a proxy server whitelist to only allow Microsoft update catalog connections. So I'm not sure what this is referring to. Being that this systems is pretty much air gapped is there another option to fix this?
Running WPP v1.4.2103.13 on the internal WSUS 2016 server with an Internal Microsoft Enterprise CA for the code signing certificate.

Hi ljbrusta, can you create a Reg_SZ value in HKEY_LOCAL_MACHINE\SOFTWARE\WOW6432Node\EasyCompany\Wsus Package Publisher with the data = 1 and try again. That will disable timestamping from WPP.

Just any value name for the Reg_SZ?
From: David COURTEL @.>
Sent: Monday, September 6, 2021 9:08 AM
To: DCourtel/Wsus_Package_Publisher @.>
Cc: ljbrusta @.>; Author @.>
Subject: Re: [DCourtel/Wsus_Package_Publisher] Failed to sign package; error was:<PHONE_NUMBER> (#64)
Hi ljbrusta, can you create a Reg_SZ value in HKEY_LOCAL_MACHINE\SOFTWARE\WOW6432Node\EasyCompany\Wsus Package Publisher with the data = 1 and try again. That will disable timestamping from WPP.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/DCourtel/Wsus_Package_Publisher/issues/64#issuecomment-913716148 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ALL5S5T3YSTQCITOFYGN473UATKNHANCNFSM5CDZXACA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://github.com/notifications/beacon/ALL5S5SI7FI3WG2MIZENF33UATKNHA5CNFSM5CDZXACKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGZ3DHNA.gif

Sorry, I forgot the value name: DisableTimestamping


Consider using state machine in ProxySession

May be it will be better, but only if doesn't affect code readability.

Moving this issue from Icebox because current code has became unreadable anyway :)


Pusher update of new money request clears out personal details in Onyx

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

Version Number:
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @youssef-lr
Slack conversation: https://expensify.slack.com/archives/C05LX9D6E07/p1717784608460829
Action Performed:
Create a Collect policy and enable it for approvals: Workflows => Add approvals
Invite a member
Log in from two side by side Chrome windows on both accounts, admin & submitter
Open workspace chat in both windows
Submit a new money request as the requester
Observe workspace chat as the admin
Expected Result:
Repot names should show participant names
Actual Result:
Repot names no longer display participant names
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/9680864/d2e88281-3e57-4eed-851a-3de6c0f2ae3b

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @youssef-lr

This is fixed.


Trigger renders after a course is done to catch broken links?

Noticed that old courses have quite a few broken links that we would like to catch, but we only do so when we try to re-render after making a change.
Perhaps we can trigger rendering periodically to check for these?

I think the heart issue is tracked previously here: https://github.com/jhudsl/OTTR_Template/issues/501
If the idea there doesn't cover this idea, let me know.

This is addressed with periodic URL checker!


Update tvpassport.com_us.channels.xml

Added KCNCDT1.us and KDVRDT1.us

Added WPCWDT1.us, requested by #1503


Cross travis

Added cross compiled Intercept to travis. Twice the chance to fail the tests, yay! :D


Failed to parse gcfg data: can't store data at section "http"

Hello,
I'm trying run a few simple tests with the influxsnmp tool and I'm having a hard time getting the binary to run with the config file.
Here is a sample of my config (Ubuntu 14.04):
-config.gcfg-
[common]
httpPort = 8085
; multiple snmp devices can be specified
; their config name must match a mib config name
; or an alternate config name can be used
; or there must be a wildcard mib configured -- named '*'
;[snmp "h3c-a5500"]
;[snmp "H3C-COMMON-SYSTEM-MIB"]
[snmp "inf-sw-switch"]
host = <IP_ADDRESS>
community = v3
;port = 161
port = 8086
timeout = 20
retries = 5
;repeat = 0
freq = 30
;debug = false
; if port file is ommited then all columns will be retrieved
portfile = h3cPorts.txt
; this is a wildcard -- becomes default
; if a 'snmp' section name is not otherwise specified
[mibs "inf-sw-srvroom"]
name= ifXTable
column = ifInMulticastPkts
[influx "*"]
url = http://localhost:8086/
;host = localhost
port = 8086
db = snmp
;user = username
;password = password
;web status monitor - set port to 0 to disable
[http]
port = 9501
I'm running the binary with this command:  ../bin/influxsnmp -config config.gcfg
Upon running the binary, I get:
../bin/influxsnmp -config config.gcfg
2016/09/06 15:38:22 Failed to parse gcfg data: warnings:
can't store data at section "http"
can't store data at section "http"
can't store data at section "snmp", subsection "inf-sw-srvroom", variable "portfile"
can't store data at section "mibs", subsection "inf-sw-srvroom", variable "column"
can't store data at section "influx", subsection "", variable "port"
can't store data at section "influx", subsection "", variable "db"
can't store data at section "http"
can't store data at section "http"
I believe I understand what the errors are stating but I can't seem to sort out the correct syntax for this.  I've read the docs that I can find and I wasn't sure if the 'mibs' section requires a mib entry, etc....
Would it be possible if you could point me in the right direction on this one?
I've tried running this through my favorite search engines but couldn't seem to find an answer and I'm not familiar with go enough to really figure it out.
Thank you very much for your help.
Cheers,
Mike
~

I have a similar error:

Failed to parse gcfg data: warnings:
can't store data at section "mibs", subsection "*", variable "rename"
can't store data at section "influx", subsection "switch", variable "user"

This is the extract of my config.gcfg file:

[mibs "*"]
name = sysName
rename = sysName=myNewName  ; rename entry
[influx "switch"]
url = http://<IP_ADDRESS>:8086/
database = events
user = test
password = test

Thanks for your help


feedback on connection

other than not getting an error when calling odbc.connect, is there any way to get feedback and troubleshoot?
when i connect to the datasource, i am not getting an error. when i run queries, i don't seem to be getting any results back, but my other custom odbc tools are functioning.

If there is an error you should be getting the error message because it is handled in the way recommend by the odbc documentation. If you are not getting any results, it is possible that you are pointing to a wrong driver.

thanks for the reply.
i'm using c:\windows\system32\odbc32.dll

i am connecting to an eloquence sql/r db.
like i said, i am not even getting an error when i submit an invalid query.

by the way, in case i misunderstood the directions, and you meant that i should find the driver in the "drivers" tab of the control panel, i did that.

the path to that driver is
C:\Program Files (x86)\Marxmeier\SQLR\bin\sqlro32.dll
when i try to use that as the driver path, i get this error:

Now you are on the right track. Again you are using the wrong driver but now you are closer to finding the right driver. Please refer to the documentation of your specific database vendor. From what I saw, the package for windows comes with odbc driver. Usually the name comes with an indication that it is in fact the odbc driver. The library should work without an issue given that you are using the correct driver. Good luck.

i don't understand what that means. i switched to C:\Program Files (x86)\Marxmeier\SQLR\bin\sqlro32.dll, but i get the argument error, that i showed, above.
that file is the one that shows up in the driver tab of the odbc settings panel.
here are the contents of that folder:

and here is the odbc data source admin, driver tab:

Interesting. You are right. The driver manager in fact recognizes the driver to be the odbc driver
But it is missing even the basic symbols such as SQLAllocHandle. Can you try reinstalling the odbc driver?

sure. just an fyi, i tested my other odbc clients, so i could basline, make sure my queries were correct, etc. those other clients worked.

ok, did uninstall/reinstall. then i thought i'd try to run the 64-bit admin to see if maybe the problem was a mismatch between 32 and 64 bit, and, i got a 193 error - interesting, because that's what the flutter app is giving me.

so i did a little reasearch. it seems that there might be some connection between some issue between 32 and 64-bit...something, but i don't understand what.

next, i tried to switch to using the 64 bit driver, in case that makes any difference. since we don't even make it as far as odbcConnect(), i hoped that the dsn pointing to the 32-bit driver wouldn't matter. now i get a different error:

32-bit driver error:

Interesting indeed. I am sorry for having to go through all this and still not finding a solution. Sadly there's not much I can do. If the library fails to find the method signature that sounds like an issue with the odbc driver.

any idea on why the other clients might be working, or a way for trying to get to a root cause?

Are you sure other clients are in fact using odbc driver? They could be using the native driver instead. Or there's something both of us are missing.

this is the first time i've had to specify the driver. with the other tools, i simply select the dsn. all of the dsn's point to the marxmeier 32 bit driver.

The library is still in development. And if I were to implement I would have to implement auto detecting on all platforms which comes with different configurations. But you raise a valid point. I might actually consider implementing this in the future. Thanks for your feedback 😃

it's not that it's bad, i just don't know why my code isn't working, when the other clients i've written with other dev tools do work.
i'd like to get it working. that would mean i could make flutter the default dev tool. unfrortunately, odbc is the only way we have for communcating with our erp server.

I understand your concern but I am really sorry there's nothing much I can do at this point. :(

I have reproduced your issue from my end. It is definitely an issue with the driver and not your installation itself. So, if you happen to have support for the product maybe try raising an issue?

I think I have an idea what's going on here. My driver is based on odbc 3.0 specification and apparentlySQLAllocHandle was introduced in odbc 3.0. it is possible that the sql/r driver is using an older version of the spec. I'll see what I can do. 😊

ooh, that would be cool. crossing fingers

I have updated the library. You might want to set utfType to UtfType.utf8 in the constructor. Please refer the troubleshoot section of the documentation for further details. I really hope this works out for you. ☺

i really appreciate your effort on this.
update with the updated library:

if i just use dsn, i get an error that the driver can't be found:

if i specify the driver, regardless of the utf, i get invalid application:
WITH utf specified:

WITHOUT UTF8 specified:

Thank you for the update. The error you are getting is due to erroneous configuration of driver manager. Assuming you installed 64bit version of the odbc driver you should user 64bit odbc driver manager. You can find the difference when you search odbc driver manager in windows search/start.

i have the 32 bit version installed and configured (and all the DSN's are configured with it), because the 64 bit odbc data administrator does not seem to work. i think i remember this might be an arm compatibility issue.
if i select the 64 bit odbc driver in the 64 bit odbc admin, i get this

I see. I got the driver to work on my windows vm. At some point I got the error you are getting and I fixed it by using 64bit driver manager. Also I think I know why it says driver not found. I will push another update and see if that fixes the driver not found issue

did you get a chance to mess with this, any more?

I'm sorry I've been busy with work and uni

no worries! just checking.

Hello @macMikey.
I'm really sorry for taking this long to give an update. I have applied some new fixes. This should be a major fix. You can I have published 4.1.2+1. Please do not provide path to driver this time and hopefully it will detect the driver automatically. If it is still unable to find the path please update me.

yay!
thanks for all your effort on this. i appreciate it. it'll be great if we can get this working, and start using dart + flutter for our corporate apps.

seems like we're getting a little closer.
Connecting to VM Service at ws://<IP_ADDRESS>:50924/Mq0lAKkNYfk=/ws
Connected to the VM Service.
flutter: Error: [Microsoft][ODBC Driver Manager] The specified DSN contains an architecture mismatch between the Driver and Application
flutter: ErrorCode: -1
Lost connection to device.

here's the code:
class EloqOdbc {
  String? username;
  String? password;
  DartOdbc? odbc;

  Future<void> eloqOdbc({
    required String username,
    required String password,
  }) async {
    this.username = username;
    this.password = password;

    await dotenv.load(fileName: ".env");
    final dsn = dotenv.env['DSN'];

    try {
      odbc = DartOdbc(
        dsn: dsn,
      );
      await odbc!.connect(username: username, password: password);
    } catch (e) {
      debugPrint('Error: $e');
      rethrow;
    }
  }

i don't understand.
on this machine, things are working, with other tools. i'm pretty sure that those tools are using the 32 bit driver. i don't know why, but if i was guessing, i would guess that on arm, odbc is running in a 32 bit compatibility layer (even though a 64 bit version of windows is installed).

As you can see the issue is directly thrown from the driver manager itself. So there's no doubt it's an issue with the architecture mismatch of the driver

i don't know what your code is trying to do. is it possible that you are trying to use the wrong driver? why would the other dev tools be working on this box?

That is a very good question. Up until recently I was finding the driver myself, or asking the user for the path of the driver. But zi'm not doing that anymore. Now I am directly communicating with the driver manager as a middleware. So it is very unlikely that the driver is wrong.
On second thoughts, it is possible that the flutter app is running on arm or x86_64 architecture and trying to access the 32bit driver? Sorry but there's nothing I can do about it. I suggest you try it on a 64bit pc and I bet it would work fine.

it's strange that the other tools (including excel) are working.

What do you mean

Including excel
How is it even related to this?

it's strange that even using excel as an odbc client, auto-selecting the driver from the dsn, works.
excel is running x64.

That is to be expected. Like I said it's an issue with the configuration the specific driver.
Now that you mention it, the application is running as a 64bit application right? But your driver is configured to be a 32 bit driver. Maybe that is the mismatch. Can you try downloading and installing the 64bit driver and see?

we already tried that, here and here. the other dev tools all seem to use the 32 bit driver, even if they are 64 bit apps.

https://github.com/SL-Pirate/dart_odbc/issues/8#issuecomment-2408305226
Can you try the driver you used here? Give the same path to the driver path argument. If it doesn't tell you it is not s valid win32 application, then your driver installation is fine and the problem is with your driver manager configuration (which is the most probable case). Else, you have installed the wrong driver.

you can't set up 64 bit dsn's on this setup.
in the 64 bit control panel, attempting to add a dsn gives this:

next, i tried to switch to using the 64 bit driver, in case that makes any difference. since we don't even make it as far as odbcConnect(), i hoped that the dsn pointing to the 32-bit driver wouldn't matter. now i get a different error (127). this seems to suggest that 32-bit is the only option:
32-bit driver error:

I see the issue now. The library was able to load the driver when you were using the 64bit driver. But unfortunately your setup cannot set dsn for 64bit drivers. So I can confirm the issue is with your setup. Please try this on a 64bit machine. On the other hand flutter does not support running as a 32bit application either.


Remove deprecated appendix property from AbstractArchiveTask

Fixes #16783

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

I think #17123 is going to work out, so I'm closing this.


Having non persistant methods in the model

Hello,
I am having troubles with grasping how to correctly implement methods for model entities that should not be persited (ie. helper functions to calculate something). I rewrote the code from interface to abstract class, added @Transient method, but it's crashing on NullPointerExceptions, even though if I call the get method for corresponding data it works.
Any idea what the problem could be here or how to do this correctly?
Thank you.

Hi yes abstract classes will be easier in your case. What is null that causes the NullPointerException? If you have a stack trace that would help. Is it relations or objects that are not loaded yet? for that case you would need to add a null check and only invoke the method after those have been loaded

Stack trace doesn't say much. The situation is this, I have an Entity, implemented as an abstract class, having several OneToMany relationships, implemented as Lists in the abstract class. Now, I want to have a method that calculates something from those lists.
The NullPointerException happens since the list is supposedly null, even though calling .getList returns the correct list.

Ok that helps. Those list fields will be null initially. Relations are lazily loaded, calling the getter does load them which is why it works after that. You can do a few things, make sure you only call your method after they are refreshed/loaded. You can also load them also by callingdata.refresh(Entity.RELATION), if you don't want to got the db to often you can combine that with a null or empty check.

Well the thing is, at the place I need to use the function I do not even have access to the data source. With null checks, the function doesn't work as I want it to.

So when you call getList() the proxy class will then set the variable in your entity class to an instance of io.requery.util.ObservableList. Is that member of your class still null after calling getList()? if so what type of list is getList() returning? is it possible that the method is called on an instance where getList() wasn't called yet?

The list is declared as
 @OneToMany(mappedBy = "medication", cascade = {CascadeAction.DELETE, CascadeAction.SAVE})     List<MedicationTimes> medicationTimes;
And yes, after calling getMedicationTimes() and then calling the function that should work with the data it's still null.

This seems a bit hard to reproduce can you put a breakpoint in the MEDICATION_TIMES attribute setProperty().set method where the field is actually set? You should be able to see from where your field is being overwritten back to null, the call stack information there will be helpful in finding the cause. thanks.

I am having trouble understanding the debugging output in IDEA. For some reason the breakpoint only works once, trying to return to the app causes it to get stuck, so I can't even get to the point where it crashes.


Show response timeout value in the Content Timeout exception message

Add a response timeout value in the timeout message.

Reevaluate after: https://github.com/HotelsDotCom/styx/pull/560.

Closing all issues over 3 years old. New issues can be created if problems are still occurring.


Further align with OGC API - Records etc

Main issue to keep track of work in OGC API - Records until it's released, currently especially https://github.com/opengeospatial/ogcapi-records/issues/195 and https://github.com/opengeospatial/ogcapi-records/pull/211

Seems well aligned, we may an additional recommendation to prior the query parameter over the headers as clients may inject headers without the user noticing it (e.g. weg browsers)

Seems well aligned, we may an additional recommendation to prior the query parameter over the headers as clients may inject headers without the user noticing it (e.g. weg browsers)


Remove name from user table

Is this suited for github?

[X] Yes, this is suited for github

Is your feature request related to a problem? Please describe.
Not everyone needs the name property in the user table.
Describe the solution you'd like
Make this field at least optional.
Describe alternatives you've considered
Remove it entirely, allow the user to choose what form of username/displayName/name they want on their own by using corresponding plugins for it.
Additional context
I've seen other users on Discord express concern around this as well.

I think this is duplicate of https://github.com/better-auth/better-auth/issues/424
Also, I think the image should also be optional or be removed.

I think this is duplicate of #424

Yeah, I've express agreement in that issue as well.
But that issue is closed, so I opened this as a new to bring the issue back to light.

But that issue is closed

It was reopened

But that issue is closed

It was reopened

Good.


STDERR When Firing Arrow Rain

log_snippet.txt
The log snippet shows an error after successfully firing Arrow Rain at a target. Though it doesn't affect gameplay negatively, it does contribute to considerable log spam, especially for an archer player that uses spells a lot.

Fixed with release 1.2-1.20.1
Thanks!


Preparation of meeting and guide for basic usage of dynamic systems from DynPy library

The following actions are to do, in order to prepare the meeting and the guide:

[x] #1028

[x] #1029

[x] #1030

All done.


Pull support and any common, reusable e2e files from CFO into its own repo

PR for new codeflare-common repo to be reviewed first: https://github.com/project-codeflare/codeflare-common/pull/1
PR for CFO to be reviewed following completion of PR above: https://github.com/project-codeflare/codeflare-operator/pull/336


Abiliity to analyze projects (multiple js files with inter-procedural flow)

Upon a quick glance at readme, it was evident that the analysis is limited to only one file at a time
java -jar dist/tajs-all.jar test-resources/src/google/richards.js
Is it possible to analyze an entire project (comprised of one or more files with inter procedural flow between them)
For example
java -jar dist/tajs-all.jar test-resources/nodegoat
for
https://github.com/OWASP/NodeGoat
If the project supports inter-procedural flow, I am happy to add a patch to accept a folder (to process all files) rather than an individual file.

TAJS is a whole-program analysis, so it doesn't make sense to analyze the individual files. For client-side JavaScript, TAJS automatically loads the .js files linked from the given .html file (example: java -jar dist/tajs-all.jar test-resources/src/chromeexperiments/3ddemo.html). Support for server-side JavaScript (Node.js) is under development - NodeGoat is one of the case studies we use to drive that development - but it's not yet ready for release.


MB-16448 update language in customer onboarding screens

MB-16448 Jira ticket && MB-16582 Jira ticket
Summary
This pull request updates the customer on-boarding screens to have clear, appropriate language.
The main changes include:

Removed these contractions: It's, you'll, can't, you're, don't, that's
Renamed "things" to "personal property"
Renamed "movers" to "Customer Care Representative"
Made "Signature" bold on legal agreement
Added additional context

Verification Steps for the Author and Reviewers

Start the Milmove application
Create a new user
During the customer onboarding screens, verify the language matches the designs and changes highlighted in this file

Test coverage on server failed.
Last run: Thu 20 Jul 2023 07:34:20 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against 4cfc8a586a415ba1e3426213816770542d00c27a

Everything is looking great! There are just 2 areas left to fix.

Estimated Weight / Need help estimating your PPM's weight? / Add the word tool after weight estimation calculator link
& icon.

This design shows what it should look like.

2.Legal page
The signature label should be in all capitals. This is what the PR looks like.

This is a design that shows what it should look like. (don't worry about anything besides the label content.)

Test coverage on client failed.
Last run: Thu 27 Jul 2023 01:47:41 PM UTC
Refer to this report to see coverage details for the client app.
Documentation for test coverage can be found at this link.

Test coverage on server failed.
Last run: Thu 27 Jul 2023 01:48:52 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.

@duncan-truss Is it ok to merge even though these checks are failing?

@dymonmoore I think you're ok to merge in this case, it's likely just the increase in lines slightly in the client files caused the percentage to go down.  Testing for copy isn't a great use of our time if it isn't something like a conditional error or alert being displayed.  You didn't modify any backend code so I would expect that to go away by merging in main/letting mergify do it.  There have been new commits to main since your merge yesterday at 4PM.


[RFC] We need a better error system

Is your feature request related to a problem? Please describe.
Recently we observed more and more limitation pushed out by WeChat. In some cases, connection to WeChat server got kicked out. It would be better that we emit these cases as errors so developers could handle them properly.
Currently all the error messages emitted is an Error instance, only one message on it, so it is really hard to set a solid error type for the error. So here is this issue, I would like to bring this issue up and talk about the design for WechatyError or something else :P
Describe the solution you'd like
We could have a parent error type as WechatyError, then PuppetError extends the parent, then each puppet has its own error class.
How about this design:
class WechatyError extends Error {
  public readonly type: WechatyErrorType
  public readonly message: string
  constructor (
    type: WechatyErrorType,
    message: string,
  ) {
    super()
    this.type = type
    this.message = message
  }
}

Or we could make this parent error as an abstract class, then all the other errors extends this one.
Describe alternatives you've considered
Haven't considered any alternatives...
Additional context
I've implemented one temporary solution in wechaty-puppet-padpro, but that's just a test, you could check it here:
https://github.com/botorange/wechaty-puppet-padpro/commit/47d8de1bf885201629c8dd6dd34aaf721260d1c5
Here is a small piece of code that how I use it:
bot
.on('error', error => {
  const errorMessage = error.message
  const subMessages = errorMessage.split(' ')
  if (subMessages[0] === 'PADPRO_ERROR' && subMessages[1] === 'LOGIN') {
    console.log(`-----------------------------------------------------
    AutoLogin Error:
    type: ${subMessages[2]}
    message: ${subMessages[3]}
    ------------------------------------------------------`)
  }
})
.start()

To be honest, this is ugly, but just treat this as 【抛砖引玉】(don't know how to translate this :P)
Let's make wechaty better and better.
[enhancement]

We have implemented the first version of a gRPC and ECMA compatible error class: GError.
Learn more at https://github.com/wechaty/puppet/blob/main/src/gerror/gerror.ts


Tracking issue: Nota installation and build pipeline

[ ] nota package includes nota-editor frontend which bundles a bunch of things incl. all KaTeX fonts. Better way to reduce package size?
[ ] need to simplify installation process, way too many pitfalls when installing node, npm, and global binaries from @nota-lang/nota
[ ] SSR should run until all initial mobx reactions have executed so @Ref and @TableOfContents don't flicker on page load.
[ ] Typescript should get moved into esbuild-utils (a la estrella) so it generates bindings without a separate command, and also works in watch mode

Regarding installation process, I made a Dockerfile here: https://github.com/jhvst/dockerfiles/tree/main/nota
The point is that this would package nota and its dependencies into a container which one could invoke in folders where needed with a command such as podman run -v $PWD:/nota:z -w /nota nota build examples/index.nota. Here, the -v mounts the current folder to the container at /nota, and then changes the working folder within the container to /nota with the -w flag. Then, one can run the build command. Next, the output of nota appears on the host at the ./dist folder.
However, this was not as straightforward as usually for these cases:

there is a need for Chromium
since whatever goes with Chromium requires a sandbox, the user which runs the nota command cannot be root:

[core@localhost nota]$ podman run -v /var/mnt/nota:/nota:z nota build /nota/index.nota
✘ [ERROR] [plugin ssr] Failed to launch the browser process!
[0505/223244.655939:ERROR:zygote_host_impl_linux.cc(90)] Running as root without --no-sandbox is not supported. See https://crbug.com/638180.

TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md

/usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:1603
  let error = new Error(`${text}${summary}`);
              ^
Error: Build failed with 1 error:
error: Failed to launch the browser process!
[0505/223244.655939:ERROR:zygote_host_impl_linux.cc(90)] Running as root without --no-sandbox is not supported. See https://crbug.com/638180.

TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md

    at failureErrorWithLog (/usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:1603:15)
    at /usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:1131:18
    at /usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:1126:9
    at /usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:666:9
    at handleIncomingPacket (/usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:763:9)
    at Socket.readFromStdout (/usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:632:7)
    at Socket.emit (node:events:527:28)
    at addChunk (node:internal/streams/readable:324:12)
    at readableAddChunk (node:internal/streams/readable:297:9)
    at Socket.Readable.push (node:internal/streams/readable:234:10) {
  errors: [
    {
      pluginName: 'ssr',
      text: 'Failed to launch the browser process!\n' +
        '[0505/223244.655939:ERROR:zygote_host_impl_linux.cc(90)] Running as root without --no-sandbox is not supported. See https://crbug.com/638180.\n' +
        '\n' +
        '\n' +
        'TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md\n',
      location: null,
      notes: [],
      detail: Error: Failed to launch the browser process!
      [0505/223244.655939:ERROR:zygote_host_impl_linux.cc(90)] Running as root without --no-sandbox is not supported. See https://crbug.com/638180.

      TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md

          at onClose (file:///usr/local/lib/node_modules/@nota-lang/nota/node_modules/puppeteer/src/node/BrowserRunner.ts:268:9)
          at Interface.<anonymous> (file:///usr/local/lib/node_modules/@nota-lang/nota/node_modules/puppeteer/src/node/BrowserRunner.ts:254:50)
          at Interface.emit (node:events:539:35)
          at Interface.close (node:internal/readline/interface:529:10)
          at Socket.onend (node:internal/readline/interface:258:10)
          at Socket.emit (node:events:539:35)
          at endReadableNT (node:internal/streams/readable:1345:12)
          at processTicksAndRejections (node:internal/process/task_queues:83:21)
    }
  ],
  warnings: []
}

This is about the user within the container which runs the command. I got around this by creating a user within the container, but this introduces other problems (most I think are podman centric, which is runs rootless containers):
[core@localhost nota]$ podman run -v $PWD:/nota:z -w /nota nota build examples/index.nota
✘ [ERROR] Failed to create output directory: mkdir /nota/dist: permission denied

[11:08:39 PM] info: Access to script at 'http://localhost:8000//index.mjs' from origin 'null' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
[11:08:39 PM] info: Failed to load resource: net::ERR_FAILED
✘ [ERROR] [plugin ssr] waiting for function failed: timeout 10000ms exceeded

  This error came from the "onEnd" callback registered here:

    /usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:885:10:
      885 │           await promise;
          ╵           ^

    at setup (file:///usr/local/lib/node_modules/@nota-lang/nota/node_modules/@nota-lang/esbuild-utils/lib/ssr.ts:148:11)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async handlePlugins (/usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:885:11)

/usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:1603
  let error = new Error(`${text}${summary}`);
              ^
Error: Build failed with 2 errors:
error: Failed to create output directory: mkdir /nota/dist: permission denied
error: waiting for function failed: timeout 10000ms exceeded
    at failureErrorWithLog (/usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:1603:15)
    at /usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:1249:28
    at processTicksAndRejections (node:internal/process/task_queues:96:5) {
  errors: [
    {
      detail: undefined,
      location: null,
      notes: [],
      pluginName: '',
      text: 'Failed to create output directory: mkdir /nota/dist: permission denied'
    },
    {
      pluginName: 'ssr',
      text: 'waiting for function failed: timeout 10000ms exceeded',
      location: null,
      notes: [Array],
      detail: TimeoutError: waiting for function failed: timeout 10000ms exceeded
          at new WaitTask (file:///usr/local/lib/node_modules/@nota-lang/nota/node_modules/puppeteer/src/common/DOMWorld.ts:813:28)
          at DOMWorld.waitForFunction (file:///usr/local/lib/node_modules/@nota-lang/nota/node_modules/puppeteer/src/common/DOMWorld.ts:728:22)
          at Frame.waitForFunction (file:///usr/local/lib/node_modules/@nota-lang/nota/node_modules/puppeteer/src/common/FrameManager.ts:1375:28)
          at Page.waitForFunction (file:///usr/local/lib/node_modules/@nota-lang/nota/node_modules/puppeteer/src/common/Page.ts:3358:29)
          at file:///usr/local/lib/node_modules/@nota-lang/nota/node_modules/@nota-lang/esbuild-utils/lib/ssr.ts:173:20
          at processTicksAndRejections (node:internal/process/task_queues:96:5)
          at async Promise.all (index 0)
          at async file:///usr/local/lib/node_modules/@nota-lang/nota/node_modules/@nota-lang/esbuild-utils/lib/ssr.ts:181:7
          at async /usr/local/lib/node_modules/@nota-lang/nota/node_modules/esbuild/lib/main.js:1040:15
    }
  ],
  warnings: []
}

On podman systems this requires running podman unshare chown $uid:$gid -R $folder to make the system work. Eventually one should have:
[core@localhost nota]$ podman run -v $PWD:/nota:z -w /nota nota build examples/index.nota
[11:09:29 PM] info: Built in 1.68s.
[core@localhost nota]$ ls dist/
index.html  index.mjs

My workflow requires containers for a self-inflected reason, but I'm willing to help if this seems something that you see would simplify the process.

@jhvst thanks for the heads up about containerizations. The Chromium dependency is important, because we use Chrome (via Puppeteer) for server-side rendering. But I could change the SSR plugin to pass --no-sandbox to Puppeteer when inside of a container.

Thanks, that could be in part. Though I feel like containers might not be the way the go here. I pushed some code of mine to https://github.com/jhvst/jhvst.github.io but as you can see, it became rather hacky with bash scripts involved. I think containers in general have a lot of painpoints when the host filesystem has to be modified. Though I hope this report could help anyone who wants to look into that.
That being said, do you happen to have plans to integrate Nota with VSCode via LSP? If Puppeteer is used for server-side rendering, I wonder could one use the WebView feature in VSCode for a more streamlined setup?

Puppeteer shouldn't be part of the editing loop -- it's only for generating final files that will be shipped to clients. So that shouldn't be an issue for editor integration.
For now, I am building my own Overleaf-like editor (@nota-lang/nota-editor) based on Codemirror 6. If Nota gets popular enough, I will look to add editor integrations.


SubscriptionBond (inc. Websocket, more accurate notifications)

In progress. (still dependent on new parity-js pubsub feature)

For Bonds which take an input and are not likely that the result will change frequently we maybe should stay with the api calls. E.g. bonds.blockByNumber will likely change the given number often which would lead to a new unsubscribe/subscribe its much less likely that the returned block is another one (only reorgs). However a subscriptionBond insteadof api call will always send one more message (request-response-response) instead of simple request-response.

Best, I think, to keep everything consistent and correct for now with SubscriptionBond use throughout.,
If speed becomes an issue we can revisit later.

@gavofyork parity-pubsub is merged in master. Could soon be merged?

just use #5.