Accidentally cancelled inceptionV3 while downloading, now unable to download

The Error

tf>python3 -m scripts.retrain
Traceback (most recent call last):
File "C:\Program Files\Python36\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "C:\Program Files\Python36\lib\runpy.py", line 85, in _run_code
exec(code, run_globals)
File "D:\Kiran\Projects\ImageRecog\tf\scripts\retrain.py", line 1326, in
tf.app.run(main=main, argv=[sys.argv[0]] + unparsed)
File "C:\Program Files\Python36\lib\site-packages\tensorflow\python\platform\app.py", line 126, in run
_sys.exit(main(argv))
File "D:\Kiran\Projects\ImageRecog\tf\scripts\retrain.py", line 982, in main
maybe_download_and_extract(model_info['data_url'])
File "D:\Kiran\Projects\ImageRecog\tf\scripts\retrain.py", line 340, in maybe_download_and_extract
tarfile.open(filepath, 'r:gz').extractall(dest_directory)
File "C:\Program Files\Python36\lib\tarfile.py", line 2008, in extractall
numeric_owner=numeric_owner)
File "C:\Program Files\Python36\lib\tarfile.py", line 2050, in extract
numeric_owner=numeric_owner)
File "C:\Program Files\Python36\lib\tarfile.py", line 2120, in _extract_member
self.makefile(tarinfo, targetpath)
File "C:\Program Files\Python36\lib\tarfile.py", line 2169, in makefile
copyfileobj(source, target, tarinfo.size, ReadError, bufsize)
File "C:\Program Files\Python36\lib\tarfile.py", line 249, in copyfileobj
buf = src.read(bufsize)
File "C:\Program Files\Python36\lib\gzip.py", line 276, in read
return self._buffer.read(size)
File "C:\Program Files\Python36\lib_compression.py", line 68, in readinto
data = self.read(len(byte_view))
File "C:\Program Files\Python36\lib\gzip.py", line 482, in read
raise EOFError("Compressed file ended before the "
EOFError: Compressed file ended before the end-of-stream marker was reached

try to delete all files that has been made when you run the command.

try to run  the same retrain command with adding this argument --architecture mobilenet_1.0_224
This will download a different faster smaller model other than Inception

But I need Inception, I have, one trained using mobilenet

I tried all the above solutions, unfortunately I was running out of time and decided to do it all over again inside of a VM, will try to fix it and open another issue later. Thank you !


Periodic Schedule: Interval Parameter Issue

I'm having an issue about the interval parameter of rq scheduler.
Project spec

Django 1.10.6
django-rq 0.9.6
rq-scheduler 0.7.0
Python 2.7.13

import django_rq
from datetime import datetime

scheduler = django_rq.get_scheduler("default")
scheduler.schedule(
    scheduled_time=datetime.utcnow(),
    func=some_function,
    args=[],
    kwargs={},
    interval=30,
    repeat=None
)
scheduler.run()

This scheduler run but it does not work as intended. The function run every minute and not as specified on the interval parameter.
I checked the scheduler.__dict__ using python console and found out that there is a _interval attribute with the value of 60. It seems the scheduler uses this interval and not the interval on the schedule parameter.
{'_interval': 60, 'connection': Redis<ConnectionPool<Connection<host=localhost,port=6379,db=0>>>, 'queue_name': 'default', 'log': <logging.Logger object at 0x10213e250>}
as of now my work around is to set this _interval before running my scheduler.
scheduler._interval = 30
Is there any cleaner way to make this work. Thank you.

Hi @ryanermita,
Scheduler has two different intervals.
The first interval is how often scheduler checks Redis and push scheduled jobs to correct queues. - implemented as an active while(True) loop with sleep(interval) call.
Default value of that interval is 60 seconds
The second interval is per scheduled job and sets how often job should run.
To make your example work scheduler first interval (refresh interval) has to be smaller than or equal to lowest job interval.
To make sure your job starts at proper time your scheduler refresh interval has to be divisor of your job interval (for example 30, 15, 10, 5 etc.)
If you have more scheduled jobs your scheduler refresh interval has to be divisor of all of them.
To fix your example:
import django_rq
from datetime import datetime

scheduler = django_rq.get_scheduler("default", interval=15)  # scheduler refresh interval
scheduler.schedule(
    scheduled_time=datetime.utcnow(),
    func=some_function,
    args=[],
    kwargs={},
    # job specific interval value
    interval=30,  # should be always divisible by scheduler refresh interval
    repeat=None
)
scheduler.run()

If you use django_rq commands you shall specify this interval as a command parameter
manage.py rqscheduler --interval 15
Note:
If my project requires some jobs to run every couple of seconds I use scheduler refresh interval 1 (schedule jobs every second) so I'm sure all jobs starts in the time they should.

I understand it better now. Thank you for your response @mojeto


Status message when wallet is syncing

When certain users with larger wallets sync their transactions for the first time (or after clearing wallet data), the startup may take a while to sync up. On the app side, we have no way of telling when the wallet is syncing vs something else not working, so we show the "oops" screen.
The status api call should return the syncing status while the wallet is syncing. If there's a way to give any sort of progress, that would be a nice to have.

This should also be taken into account during sync apply when we accounts are added


AllowClear param in EnumEditor

@volkanceylan I have added the AllowClear parameter (as default true) to make possible set it to false and render enum list not clearable.

Does this work when you put [EnumEditor] attribute on a row field? Does it pass EnumKey properly?

Yes, the use is with [EnumEditor(AllowClear = false)] attribute.
And I have now see that the EnumKey is properly passed


Added Omnibox experiment explainer

@erdraud do you think we can merge this? Do you need reviews from other people before merging?

@captainbrosset I can't merge it myself, but I this is ready to go.


python312Packages.peaqevcore: 19.9.0 -> 19.9.2

Changelog: https://github.com/elden1337/peaqev-core/releases/tag/19.9.2
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

4 packages built:

python311Packages.peaqevcore
python311Packages.peaqevcore.dist
python312Packages.peaqevcore
python312Packages.peaqevcore.dist


Unhandled Exception: Instance of 'MsalInitializationException'

We are trying to initialize the azure authentication method in an android device, But getting the unhandled exception.   Here is the log:
D/DART/NATIVE(14813): onMethodCall initialize W/Telemetry(14813):  [2022-12-27 10:16:49 - thread_name : main, correlation_id : UNSET]  Android 31 Telemetry is disabled because the Telemetry context or configuration is null W/PublicClientApplicationConfiguration:validateCustomTabRedirectActivity(14813):  [2022-12-27 10:16:49 - thread_name : main, correlation_id : UNSET]  Android 31 Another application com.sample.abcd_mobile is listening for the URL scheme msauth://com.sample.abcd_mobile/xxxxxxxxxxxxxxx E/flutter (14813): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Instance of 'MsalInitializationException' E/flutter (14813): #0      AzureAdAuthentication._initialize (package:azure_ad_authentication/azure_ad_authentication.dart:142:7) E/flutter (14813): <asynchronous suspension> E/flutter (14813): #1      AzureAdAuthentication.createPublicClientApplication (package:azure_ad_authentication/azure_ad_authentication.dart:34:5) E/flutter (14813): <asynchronous suspension> E/flutter (14813): #2      _MyAppState.intPca (package:plugin_test_app/main.dart:220:12) E/flutter (14813): <asynchronous suspension> E/flutter (14813): #3      _MyAppState.getResult (package:plugin_test_app/main.dart:192:33) E/flutter (14813): <asynchronous suspension> E/flutter (14813): #4      _MyAppState._acquireToken (package:plugin_test_app/main.dart:178:5) E/flutter (14813): <asynchronous suspension> E/flutter (14813):

correctly configure the id and the other steps for operation this is wrong
msauth://com.sample.abcd_mobile/xxxxxxxxxxxxxxx


Complex Properties with Read-Only Properties

Tracking a class with a complex property does not work if the complex property has a read only property by itself. Here is the Code for this:
    public class TestSubClass {
        public virtual string Name { get; set; }

        // Adding the following Line causes an error by tracking the SubClass
        public virtual double SomeWhat => 10.5f;

        public override string ToString() => Name;

    }

    public class ErrorClass {
        public virtual TestSubClass Test { get; set; }
        public ErrorClass()  {
            Test = new TestSubClass();
        }
    }

    [TestMethod]
    public void CauseError() {
        var error = new ErrorClass().AsTrackable();
        error.Test.Name = "Changed it";
        Assert.IsTrue(error.CastToIChangeTrackable().IsChanged, "ChangeTracking for SubClass failed");

    }

You can also just add the line:
public virtual string Zip => "12345";
Into the Address class of the ChangeTracking.Tests Project

Thanks for the detailed report.
I fixed this behavior, please try the latest version 2.1.0


CLS-compliance

Can RecyclableMemoryStreamManager be made CLS-Compliant?

I don't know the answer off the top of my head. There may be reasons it is not. I will take a look and attempt it.


Please add Ilyes (@celyes) and Ahmed (@Aissaoui-ahmed) as meeting observers

Please add me and @Aissaoui-Ahmed as observers on the community-committee's meetings.
We're really excited to jump in! 🙌

See https://github.com/nodejs/community-committee/issues/450#issuecomment-463086932

Is there still a path to membership? Is CC a closed group now?

@codeekage's comment earlier links to the steps you can take to get notified when the meetings are happening. You don't have to attend the meetings to participate, but attending a meeting is a good way to get context. You'll also see notifications when discussions happen, and you're welcome to participate (by commenting, etc). Exploring and keeping up with the activities in this repo are a good way to find a space when you may enjoy contributing to with code or any other way! Stoked to have you on board, @celyes!

@keywordnew But what about membership like @adarshsaraogi asked? Is it still 3 months as an observer?

The observer to membership track has been removed from our charter. Membership can be gain through active participation, collaboration and participation. See here: https://github.com/nodejs/community-committee/blob/master/CONTRIBUTING.md
You can also start off from being a mentee on the Node.js Mentorship Programme to help provide guidance to membership and bettter understanding of the project. https://github.com/nodejs/mentorship


Add extendable.afterConstruct event

The event fires once the behaviors have been loaded.

@bennothommo can we add a fireEvent() method to the mockup or something?

@bennothommo @jaxwilko any input on this?

@mjauvin @LukeTowers I just want to be sure that you are wanting this to fire on every single instance of an extendable class (not just models)?

@bennothommo yes, in my case that I needed it for I wanted to be able to call addAssets() on a widget implementing a behaviour to implement the required assets for that behaviour so long as it's conditions for enabling itself were met. However, I was unable to call the methods that behaviour added to the widget (which were required to determine if it was eligible to run and thus add the assets in the constructor of the behaviour.
Perhaps we could also implement this by calling init() or boot() on the behavior after all the behaviours are registered instead of an event? @mjauvin would that work for your use case?

@LukeTowers personally, I'd introduce another parameter to extend to specify that a callback should be run after the implements have been applied. That way, we're not doing an event listener on every single instance.

@bennothommo I was thinking more along the lines of Laravel's bootable Eloquent traits, i.e. the behaviour would add a method called bootNameOfClass() that gets called after all the behaviours have been registered (see https://andy-carter.com/blog/using-laravel-s-eloquent-traits). That way there's no added complexity to the API and there's no extra overhead of the events system.

@LukeTowers that's a much more wild change to the current API than my suggestion though 😜

Is it? It's fairly unobtrusive IMO, it's an optional method that be implemented by the behaviour to hook into after all of the behaviours have been registered.

How would you propose that someone leverages it? Would it be part of an extend callback, or another method altogether?
My suggestion is simply an extra parameter to extend, with a default that keeps BC. I'd argue it's more graceful.

I don't remember what @mjauvin's use case of it was, but it wasn't so much external usage of it but instead internal usage by the behaviour itself in order to do setup tasks that require the method proxying to be in place on the object that's implementing them before they can do certain tasks.

My use case is to be able to call behavior's methods within the extend() callback. So @bennothommo's suggestion would work for me I believe.

I'd like to be able to do the following:
class MyModel extends Model {
    public $implement = ['MyBehavior'];
}

MyModel::extend(function ($model) {
    $model->myBeaviorMethod();
})

Replaced by #113


generated code triggers numerous shellcheck warnings

This came up while working  on a PR for apache/groovy, see: https://github.com/apache/groovy/pull/896
Basically I was helping smash some shellcheck warnings in the groovy bin scripts, including the completions scripts which I didn't realize were actually generated by picocli.
You can see the shellcheck warnings for the completion scripts here
And the changes I made here
The main issues I encountered are:

syntax of the ArrContains function is dubious, and this function doesn't really seem necessary
variables aren't properly localized and pollute the user's environment
there are unused variables (minor)

Hi, thanks for raising this!
Localizing the variables is a good idea, should definitely do that.
The ArrContains function is used to support nested subcommands, like used in grape. It is used because it supports nested sub-subcommands to any level of depth; this is quite powerful.
Can you clarify what you mean by "dubious syntax"? I would consider alternative mechanisms that still support nested sub-subcommands, can you propose something?
I'll see what we can do about the unused variables.

A few dubious constructs here:
for i in "${lArr1[@]}";{ [ -n "$i" ] && ((++tmp[$i]));}
using curly braces intead of do/done is relying on an undocumented feature meant to support backward compatibility for... an even older undocumented feature:
https://stackoverflow.com/questions/38828762/linux-curly-brackets-instead-of-if-then-else-fi-do-done
Really should be using the standard syntax of
for ... ; do ... ; done

eval lArr1=("\"\${$1[@]}\"")
I get nervous around eval.  It lends itself to abuse and has many, many pitfalls.  It should be reserved for the few cases it makes sense and is well-suited, e.g. reading in environment variables dumped by an external program or dynamically building an options string for a program, not for logic like declaring and initializing associative arrays. (also why the parens here? There's no reason to execute that in a subshell?)
https://mywiki.wooledge.org/BashFAQ/048

Finally, just the use of associative arrays in general instead of case statements is dubious in a completion script.  Pretty much the universal idiom in these scripts is to use case and switch on $cur and $prev (set from COMP_WORDS).
To your question about how to handle arbitrarily nested subcommands without an associative array, I would say since the code is being generated just generate the necessary case statements to the necessary level of depth.
I think git's own completion script is really the gold standard here, especially when it comes to handling complex nested subcommands.  I would refer to that as the 'right' way to do things in a completion script.
https://github.com/git/git/blob/master/contrib/completion/git-completion.bash

That is good feedback, thank you for the clarification!
Do you think you will be able to provide a pull request to fix (some of) these items?

I can add it to my personal FOSS queue, but I'm pretty slammed with work and other projects so not sure when or if I would get to it, so if there's someone else who'd like to take this on I will not be offended.
As far as I know nothing is actually broken, so this is probably isn't all that urgent, although I do think there's great value to generating completion scripts that adhere to the common idioms.

I like to work on the completion script generator. Variable localization, unused variable and for/if syntax are straightforward to fix.
To get ride of the ArrContains function it need a different PR I think and I'm not confident enough to go to the git completion script. Instead I can propose something based on @calid change with a tree-like solution to keep subcommand working. You can found a POC here

@AlcaYezz Great, pull requests are very welcome!
If I understand correctly you are proposing to split up the work into two PRs: one to clean up the syntax (local vars, for/if etc) and a separate one to work on a replacement of the associative array. I like that idea a lot.
I haven’t had a chance to try the completion script you linked to yet (away from PC now).

Yes that's it, to PR for this, the first one is on the way let me know if my proposal for the completion script fit your need so I can start it.

Linking to @AlcaYezz ‘s pull request: https://github.com/remkop/picocli/pull/671

Thank you, @calid and @AlcaYezz for raising this and for providing a PR.
I finally got around to improving this.
In addition to @AlcaYezz's improvements, I also rewrote the ArrContains function: it has been broken up into two functions that simply loop over an array. The use of eval and associative arrays has been eliminated. This should also make it possible for the autocompletion scripts to run on OSX (https://github.com/remkop/picocli/issues/396).


Use SYNTHIA dataset

Hello, this CVRN is a very interesing work. I am wondering how do you use SYHTHIA dataset in the experiments. Could you share like which verision of SYNTHIA dataset do you use and the format and some prepocessing steps you used in the experiments? Thanks in advance!

Hi, we have uploaded the data loaders (for all datasets) and label conversion files (SYNTHIA to COCO style), under 'data loader and processing' directory.
We used the SYNTHIA-RAND-CITYSCAPES which contains 9400 images. You could follow ADVENT (https://github.com/valeoai/ADVENT) to prepare the datasets and use our provided files to convert SYNTHIA label into panoptic segmentation (COCO style).


colums in csv file

ok...
right now all measurements and time write in one column in  csv
can you say me how i can edit script to like this:
col1      col2
time    measurement

I have added that feature for you, you can use the new --timestampstyle option:
csvexport.py output.csv --timestampstyle  first_column


Connection timed out

started librespot with sudo /usr/bin/librespot --cache /tmp/ -n "raspotify" --verbose --username *** --password ***
INFO:librespot: librespot (raspotify) 8e0d758 (2018-02-26). Built on 2018-02-26. Build ID: U2Bp36G1
DEBUG:librespot_connect::discovery: Zeroconf server listening on <IP_ADDRESS>:0
INFO:librespot_core::session: Connecting to AP "guc3-accesspoint-b-5xpp.ap.spotify.com:4070"
thread 'main' panicked at 'called Result::unwrap() on an Err value: Error { repr: Os { code: 110, message: "Connection timed out" } }', /checkout/src/libcore/result.rs:916:5
stack backtrace:
0:   0x86f057 -
1:   0x8792bb -
2:   0x8678d7 -
3:   0x8673b3 -
4:   0x867d83 -
5:   0x867bcb -
6:   0x867b53 -
7:   0x867afb -
8:   0x8d0bcf -
9:   0x449007 -
10:   0x453fd3 -
11:   0x44c4ef -
12:   0x464e5b -
13:   0x455973 -
14:   0x4658e3 -
15:   0x867a2b -
16:   0x88e65b -
17:   0x86ece3 -
18:   0x456453 - main
19: 0x76c95677 - __libc_start_main
any one else had this problem?!

I had the same issue, but after an upgrade to 0.10.4~librespot.20180226T171341Z.8e0d758 everything was working as normal again.

Upgrade and let me know if it works, @zamanisaja


o1-preview early termination

cody_1735031928433.log
Type: Bug
Extension Information

Cody Version: 1.53.1734972437
VS Code Version: 1.96.2
Extension Host: desktop

Steps to Reproduce
Open or create the MATLAB file (.m extension) in Visual Studio Code where the Cody extension is active.
Copy and paste your entire script (approximately 675 lines) into the Cody Chat window, and request an action such as:
“Fix any broken MATLAB syntax” or “Refactor this code”.
In this case, Cody abruptly stopped generating output around line 182, leaving the remainder of the code incomplete (lines 183–675 missing). This similarly occured with Flash2.0
Expected Behaviour
Cody should process the entire code snippet and provide complete suggestions or transformations, without prematurely cutting off the output.
Actual Behaviour
The o1-preview ends early or truncates the code response, causing incomplete suggestions and halting mid-code. This does not occur with o1 or o1pro in the web UI or W11 desktop app.
Logs

Extension version: 1.53.1734972437
VS Code version: Code 1.96.2 (fabdb6a30b49f79a7aba0f2ad9df9b399473380f, 2024-12-19T10:22:47.216Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i9-13950HX (32 x 2419)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
63.76GB (33.70GB free)

Process Argv
--crash-reporter-id cf5f6ea1-7412-4295-b444-0ccb15194787

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythonnoceb:30805159
pythonmypyd1:30879173
2e7ec940:31000449
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
dvdeprecation:31068756
dwnewjupytercf:31046870
newcmakeconfigv2:31071590
nativerepl2:31139839
pythonrstrctxt:31112756
nativeloc1:31192215
cf971741:31144450
iacca1:31171482
notype1cf:31157160
5fd0e150:31155592
dwcopilot:31170013
stablechunks:31184530
6074i472:31201624

Hey @AlfredoSandovalx
This is a known issue and the cause is that the output window size is capped at 4k tokens. If the reply exceeds this limit, then an error appears.
The team is evaluating if an increase makes sense in this tier plan.
Sorry for the inconvenience.


Update to modern Octane and Glimmer

fixes https://github.com/cibernox/ember-power-select-with-create/issues/113

I would like to drop ember 3.12 support and probably power select 3.   Updated the ember try scenarios and dont think it works
Build Error (TemplateCompiler)

EEXIST: file already exists, symlink '/var/folders/76/pygfd3k914100s1sf9x8xp2w0000gn/T/broccoli-5990VmdRvWU5yRfh/out-115-funnel_funnel_ember_power_select_with_create_addon/ember-power-select-with-create/components/power-select-with-create.js' -> '/var/folders/76/pygfd3k914100s1sf9x8xp2w0000gn/T/broccoli-5990VmdRvWU5yRfh/out-116-template_compiler/ember-power-select-with-create/components/power-select-with-create.js'

Stack Trace and Error Report: /var/folders/76/pygfd3k914100s1sf9x8xp2w0000gn/T/error.dump.98b8c7c2123971066108f923151e8e12.log

Result: false
---

------ RESULTS ------

Scenario ember-lts-3.12: FAIL
Command run: ember test
┌────────────────────┬────────────────────┬──────────────────────────────┬──────────┐
│ Dependency         │ Expected           │ Used                         │ Type     │
├────────────────────┼────────────────────┼──────────────────────────────┼──────────┤
│ ember-power-select │ ^3.0.0             │ 3.0.6                        │ yarn     │
├────────────────────┼────────────────────┼──────────────────────────────┼──────────┤
│ ember-source       │ ~3.12.0            │ 3.12.4                       │ yarn     │
└────────────────────┴────────────────────┴─────────────────────

Totally, I'm cool with dropping such old stuff. We will only bump a minor version number when releasing

Dropped 3.12 ember-try.  Removed the note that this was compatible with power-select 3, could add some ember tries if you wanted to keep that.

Is this ready? Looks good to me.

Yes, this is good.
I may make a couple of changes for the need I have, will follow that up with some additional PRs if required.

Merged! 🎉
If you're planning on more changes I'll hold making a release. If you don't plan on making more, I can publish.

I would publish. I an unsure if I need changes til I get to coding the other addon and since its a side project, I cant say when that will be

Released in 0.9.0


Do not set "dismissed" status in broker-db

Tied to https://github.com/ecmwf-projects/cads-broker/pull/73

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


Update logger.py

Short Description: Removes b'' from around string if no non-eng chars
Changes:

encode as utf-8 then decode utf-8

@OpenPoGo/maintainers
removes b'' from around string without utf chars

:+1:

:+1:


Embryonic fat body precursor terms

We have 'embryonic fat body', 'fat body promordium' and 'fat body/gonad primordium'
'embryonic fat body' did not previously have a definition
'fat body primordium' is described as being the primordium after the gonadal sheath primordium splits from it in stage 14, but gonadal primordia are only in PS10-12 according to FBrf0101991 and this ref also refers to segmentally repeated clusters of cells as primordia before precursors join into a contiguous fat body structure (and before stage 14).
We should probably merge 'embryonic fat body' and 'fat body primordium' and perhaps broaden the definition to cover fat body precursors at any stage of embryonic development.
I think perhaps 'fat body/gonad primordium' might refer to the 'bipotential lateral mesoderm primordia' of FBrf0102832, which can develop into either fat or gonad precursors (but is not the only source of fat precursors), rather than some kind of fused primordial entity. According to FBrf0101991, fate is chosen by around stage 10 (expression of srp in fat precursors). So this definition needs modifying.

 The fourth term is not connected to any of the others.

 The current definition of embryonic fat body ("developing fat body of the embryo") matches the intended meaning of fat body primordium.

Hi Damien,
Thanks for untangling that. I like your solution; the staging should
definitely be included in the term definitions.

 The fourth term is not connected to any of
the others.

 The current
definition of embryonic fat body ("developing fat body of the
embryo") matches the intended meaning of fat body primordium.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FlyBase/drosophila-anatomy-developmental-ontology/issues/1015#issuecomment-840470437,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADCA2CTRQDOGUB3OGIDFR5DTNOTCFANCNFSM4UTSYGDA
.


Add custom derive

Like structopt.

Thanks for the suggestion @RazrFalcon! I think this is out-of-scope for getopts directly, but if someone would like to put this together as an external crate then that would be great!


fix: increase response timeout to 30 seconds and maximum to 60 seconds

Not sure why, but some 500 series controllers take upwards of 10 seconds sometimes to respond after trying to communicate with a node, even if successfully. Having a lower timeout has no benefit, but will cause the transmission to abort earlier.
fixes: https://github.com/zwave-js/node-zwave-js/issues/6374

@zwave-js-bot automerge


Content Co-Creation: EIP Fun Weekly 15

This is a recurring issue used for co-creating EIP Fun weekly newsletter.
Expected Release Date
September 30, 2023, at 2:00 UTC
Build Our Content Pool
Please share any ideas that could help co-create our newsletter in the comments.
Your input and collaboration are appreciated! Feel free to provide a title, pitch topics, link to resources, propose sections, or anything else that comes to mind for our newsletter.
EIP Fun Substack
Currently, our weekly newsletter is primarily published through Substack, and at the same time sent to subscribers' email inboxes. Feel free to subscribe and follow along!
Links to previous newsletters: https://eipsfun.substack.com/
You can check out the current structure, content, and style of our newsletters for reference in this link above.
And the most important thing: we really want this newsletter to be community driven! So any idea from you is welcome!
Have a good day and weekend!

https://chain.link/education/blockchain-oracles

Close first, may reopen in the future.


broken lottie animation

test.json

a regression bug by this:
commit 516e8bf042ca5c5c3c11f67cdca428d508da20a7
Author: Hermet Park<EMAIL_ADDRESS>Date:   Thu Sep 7 16:29:50 2023 +0900

    loader/lottie: minimum support for 3D transform

    It lacks perspective projection information,
    so it transforms 3D axes with orthogonal projection.


Code playground should use URL templates

As discussed on CZO, we should change code playgrounds to use URL templates, like we did for linkifiers.

@PIG208 FYI -- this is not exactly high priority, but it might make sense for you to do this migration given all the context you gained doing the linkifier one, if it's not a ton of work. @gnprice @neiljp do the mobile/terminal apps do anything with realm playgrounds? If not, the API change logistics will be a lot simpler, since we wouldn't need to add a client capability.

Got it. I have this on my plate.

@gnprice @neiljp do the mobile/terminal apps do anything with realm playgrounds?

Searching for "playground", we never interact with anything in the API that contains that in its name. Is that enough to answer the question?)
(Our type annotations reflect that there's an event type realm_playgrounds, but we ignore those events.)


Proposal: pre-compile selectors

I'm looking for jq like solution to be used as a library in the hot inner loop to crunch massive amount of data.
It would be nice to expose additional functions to pre-compile selectors and re-use them to get parsing out of the inner loop.

@osidorkin the goal of jql is stay simple, as stated in the readme. Can you provide a concrete example of what you would like to achieve?

Sure. The only public API exposed by the crate is jql::walker function which parses selectors on every invocation. I want to use it in the performance critical application in the hot inner loop where every CPU cycle counts. It literally crunches billions of data points per second. I'm looking for an ability to compile selectors once and run walker against pre-compiled selectors (Groups).
Thank you for the quick response!

It can be just a few extra exports: opaque Group struct, selectors_parser function and an alternative to walker that takes &[Groups] instead of &str and doesn't call selectors_parser inside.

@osidorkin v3.1.0 has been published to address your issues.


tensorflow/go: add operation Input methods + tests

The go version of tensorflow only has operation output methods. This adds in the needed code to return operation inputs and jump from output to input and vice versa. This is required to actually be able to traverse the graph.

CCing @aselle @andrewharp since they were assigned to review my last Go PR.

I'm working on project to do federate machine learning as a service. It does both inference and training through a Go client library. These changes are part of what's needed in order to be able to quantize models on the fly to improve inference performance. I have an attributes PR in the works as well which should make it possible to fully inspect and manipulate the models.
Go client library: https://github.com/luk-ai/lukai
Site: https://luk.ai

Think I fixed all the nits.

Failure is in //tensorflow/python/data/kernel_tests:batch_dataset_op_test, which is definitely unrelated. Merging.


Fix role assumer region issue

Context
There is an error when assuming a role without having configured a default AWS region in your environment:
Executing apply on my-stack in us-east-1
Aws::Errors::MissingRegionError: missing region; use :region option or export region name to ENV['AWS_REGION']

Changes

Use the CloudFormation driver's region, which includes fallbacks when it's not specified, when assuming a role

Considerations
Which region is used doesn't actually matter when assuming a role, as IAM and STS are global services.

It seems the specs are suffering from a similar problem as described in the PR description: when a default region is defined in ~/.aws/config the tests pass (a string value is passed to the Aws::AssumeRoleCredentials initialiser). But if no default region is defined, the tests fail (nil is passed to the Aws::AssumeRoleCredentials) as exhibited in CI.


vision classification QAT tutorial: fix for DDP

Stack from ghstack:

#2191 vision classification QAT tutorial: fix for DDP

Summary:
Makes the classification QAT tutorial not crash when used
with DDP. There were two issues:

the model was moved to GPU before the observers were added, and they
are created on CPU. In the context of this repo, the fix is to finalize
the model before moving to GPU. We can potentially follow up with a
better error message in the future, in a separate PR.
the QAT conversion was running on the DDP'ed model, which had various
problems.  The fix is to unwrap the model from DDP before cloning it for
evaluation.

There is still work to do on verifying that BN is working correctly in
QAT + DDP, but saving that for a separate PR.
Test Plan:
python -m torch.distributed.launch --use_env references/classification/train_quantization.py --data-path {path_to_imagenet_1k} --output_dir {output_dir}

Reviewers:
Subscribers:
Tasks:
Tags:

Codecov Report

Merging #2191 into gh/vkuzo/1/base will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           gh/vkuzo/1/base   #2191   +/-   ##
===============================================
  Coverage             0.42%   0.42%
===============================================
  Files                   92      92
  Lines                 7448    7448
  Branches              1138    1138
===============================================
  Hits                    32      32
  Misses                7408    7408
  Partials                 8       8

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8c95855...c33e889. Read the comment docs.

@vkuzo The PR you sent went to the wrong branch -- ghstack only works for the PyTorch repo. Can you send another PR targetting the master branch?

@vkuzo The PR you sent went to the wrong branch -- ghstack only works for the PyTorch repo. Can you send another PR targetting the master branch?

ah, sorry about that, didn't realize ghstack doesn't work with this repo.  Will do!


NEW: New supervisor (IPSet)

An Alternative implementation of the Supervisor by using IPSets. It should reduce complexity by an order of magnitude.

Current coverage is 51.62% (diff: 0.00%)

Merging #80 into master will decrease coverage by 0.86%

@@             master        #80   diff @@
==========================================
  Files            19         21     +2
  Lines          2881       2929    +48
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1512       1512
- Misses         1268       1316    +48
  Partials        101        101

Powered by Codecov. Last update e5d1ef7...f54bf87


fix(data-transfer): obloader fail to load MANIFEST.bin

What type of PR is this?
type-bug

What this PR does / why we need it:
In the earlier version, ob-loader-dumper would save binary files like MANIFEST.bin in data directory.
But in obloader-<IP_ADDRESS>, it has changed the file path into root directory. So ODC has adapted this feature.
FROM
root
└── data
    ├── TABLE
    │      └── xxx.sql
    └── MANIFEST.bin

TO
root
├── data
│     └── TABLE
│          └── xxx.sql
└── MANIFEST.bin

Which issue(s) this PR fixes:

Fixes #2176

could the object provided by earlier ob-loader-dumper be imported?

yes, ODC finds MANIFEST.bin by scanning directories. As long as ODC can find the manifest file, it can be output to the working directory of ob-loader


[Bug Report] el-select component is enabled (automatic-dropdown) property is not normal

Element UI version
2.13.2
OS/Browsers version
win10_1909  Edge_81.0.416.64
Vue version
2.6.11
Reproduction Link
https://codepen.io/xiaofan007/pen/KKdJKzP?editable=true%3Dhttps%3A%2F%2Felement.eleme.cn%2F
Steps to reproduce
Mouse click on the el-select component with enabled 'automatic-dropdown' property
What is Expected?
The drop-down box displays normally when the el-select component enables the "auto drop-down" property.
What is actually happening?
When the  el-select component is enabled 'automatic-dropdown' property, the first click on the component is not normal, and it will be displayed normally only when it is clicked again is enabled.

遇到同问题 唯独点击下拉箭头时正常 点击空白处闪烁

I looked at his code and found that after the handleFocus function was executed, the toggleMenu function was also executed. So my temporary solution is:
const ElSelect = {
  mixins: [Select],
  data() {
    return { focusStatus: false }
  },
  methods: {
    toggleMenu() {
      if (this.focusStatus) return this.focusStatus = false
      Select.methods.toggleMenu.call(this)
    },
    handleFocus(event) {
      if (!this.focusStatus) {
        this.focusStatus = true
        event.stopPropagation()
      }
      Select.methods.handleFocus.call(this, event)
    },
    handleBlur(event) {
      this.focusStatus = false
      Select.methods.handleBlur.call(this, event)
    },
  },
}


Updated pandas version

Issue #, if available:
Description of changes:

Updated pandas version since 1.3.1 released fixing the regression in 1.3.0

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-1258-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1258/1/index.html


[IoTDB-3842] Fix the issue that GroupByLevel cannot use value filter

Description
Before this change, the value filter cannot be used when executing GroupByLevel query. It is because the distribution planning didn't cover this scenario where the aggregation operation should be calculated from raw data.
In this PR, I give a fix for it to use the naive query planning method when combining GroupByLevel and RawDataQuery.
See the test below:

Coverage increased (+0.01%) to 43.71% when pulling dfe5a62771c8b843ae21283a9725c252da4c3dd2 on xingtanzjr/agg_query_level into 6ad9f83cc13c1d5d46f01bf1bb2d15e16e023f9d on master.


[Feature] DNS Record Viewer

For a given hostname, fetch all the available DNS Records given below:

Record type
Record contains

'A'
IPv4 addresses (default)

'AAAA'
IPv6 addresses

'ANY'
any records

'CNAME'
canonical name records

'MX'
mail exchange records

'NAPTR'
name authority pointer records

'NS'
name server records

'PTR'
pointer records

'SOA'
start of authority records

'SRV'
service records

'TXT'
text records

Additional task: Explain briefly about all the records in UI.

@Kashish121 / @ishika1727 - Hopefully this is the next issue which you both will be working on. This involves backend as well as frontend work.

@vinitshahdeo yes will work on this one.

@vinitshahdeo @Kashish121 @ishika1727
I would like to work on backend part of this issue. Kindly assign me this.

@vinitshahdeo @Kashish121 @ishika1727
I would like to work on backend part of this issue. Kindly assign me this.

Hi @NetanMangal , this issue has been resolved and merged here and [this file] (https://github.com/vinitshahdeo/PortScanner/blob/develop/src/dnsRecord.py) is for this feature.
You can still find bugs / errors in this file / feature and report a new issue or extend its functionality and send a PR for this.
thanks,
Network Scanner team.


Add Sentry to forward-event client

So continues our quest to capture forward-event issues.
A few months ago we added Sentry to forward-event repo, but I think that we also need to add it to our client that invokes forward-event.
This will give us:

Exact page the failed forward-event request happened on.
The event payload

How to test

Pull down this branch && npm ci.
Replace xhr.send(JSON.stringify(event)); with xhr.send('test')); This will trigger a 4xx error.
Run npm start and visit the /Feedback test case.
Submit feedback for the first feedback component.
Check to see that a 400 HTTP error event came through at docs-forward-event Sentry Project.

No longer needed


Add websockets to quickstart

name: Pull Request
about: Propose changes to the codebase
title: '[PR] Add websockets to quickstart'
labels: 'documentation'
assignees: ''

Proposed Changes
Add websockets to the quickstart. In this case, bringing over the Websockets example from the By Example tutorial
Also removes the "what's new" admonition and the "What remains to be done" section.
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist
Go over all the following points, and put an x in all the boxes that apply:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] I am aware that this is an nbdev project, and I have edited, cleaned, and synced the source notebooks instead of editing .py or .md files directly.

Good idea! Although it looks like you might have accidentally deleted all the outputs in the nb, which means they won't appear in the docs.
PS: I recommend either running git diff before you do a PR, or view the PR web page (like this one) afterwards, to ensure your PR contains exactly what you want. It's easy to miss stuff like missing outputs, but it'll be very obvious in git diff!


Follow up issue

https://github.com/freescout-helpdesk/freescout/issues/2228#issuecomment-1271470784
This is a little follow-up to the convo mentioned above. A lot has been tried already.
The problem we are facing is that the custom homepage module requires settings to be resaved every so often. Only happens in the docker. Do you maybe as a creator of the image have any idea what could be wrong?
Any help is much appreciated since there are quite a few people struggling with it.

Hi I read through the issue and have not used the Custom Home Page module so not entirely sure what sort of files it modifies.
If it is like any other Laravel application I believe all of its configuration would be saved in .env (or in this image /data/config same files).
Can you tell me what is added if anything to the .env file? If you know how to restore it, or how to trigger how it loses its settings we can build some sort of timer or switch that allows you to recreate cache/routes.

These are the entries I see in my /data/config file related to the Custom Homepage extension:
CUSTOMHOMEPAGE_DASHBOARD_PATH=amigos

CUSTOMHOMEPAGE_LOGIN_PATH=

CUSTOMHOMEPAGE_HOMEPAGE_REDIRECT="https://help.amigocloud.com/hc/1806209440"

CUSTOMHOMEPAGE_HTML="PGJyLz48YnIvPjxici8+PGRpdiBjbGFzcz0iYmFubmVyIj4NCiAgICAgICAgICAgICAgICAgICAgICAgICAgICA8aW1nIGFsdD0iIiBzcmM9Imh0dHBzOi8vaGVscC5hbWlnb2Nsb3VkLmNvbS9pbWcvYmFubmVyLnBuZyI+DQogICAgICAgICAgICAgICAgICAgICAgICAgICAgPC9kaXY+DQogICAgICAgICAgICAgICAgICAgICAgICAgICAgPGgzIHN0eWxlPSJ0ZXh0LWFsaWduOiBjZW50ZXI7Ij4NCiAgICAgICAgICAgICAgICAgICAgICAgICAgICA8YSBocmVmPSJodHRwczovL2hlbHAuYW1pZ29jbG91ZC5jb20vbG9naW4iPkxvZ2luPC9hPjwvaDM+"

Thanks, and I assume that doesn't change when the custom home page stops working. How are you fixing it within the container when it stops working?

When it stops working, the login page is shown (instead of the custom homepage url), so I login and go to system settings then go to the custom homepage settings.
Once there, all the settings are as I left them, so I change the dashboard path from 'amigos' to 'amigo' and save. Everything is working then, but I just put in the wrong setting, so I change it back from 'amigo' to 'amigos' and save again.
Then everything works again until the service is restarted and it 'resets' (for lack of a better word) and shows the login page instead of the custom home page. I use the custom homepage so my help desk people can go to '/amigos' to login, and my customers can see the help center documentation.

OK thanks. So this pretty much only resets after a container restart then.
Upon container restart I do a couple rewrites to the config / .env file based on environment variables, but then issue a few commands via the included artisan command.
Specifically one of these I figure is triggering this reset.
sudo -u "${NGINX_USER}" php artisan freescout:clear-cache
sudo -u "${NGINX_USER}" php artisan config:clear
sudo -u "${NGINX_USER}" php artisan storage:link
sudo -u "${NGINX_USER}" php artisan migrate --no-ansi

Are you able to try them one by one inside your container to see if one of them is the culprit to reset?
I'm coming up to a loss here as to why this could be happening, unless of course there is a series of files stored somewhere within the source area that we aren't correctly linking to permanently reside in the /data/ folder.
To find out if that is the case - we would want to restart the container to make it revert back to the Freescout Login page. Wait 10-15 minutes, followed by doing your configuration and then immediately after enter the container and execute:
find "${NGINX_WEBROOT}" -mmin +10 -ls

This should output a list of paths and files updated within the past 10 minutes, to then we can see if there is stray files that need to link.

Thanks.
 config
  config:cache                      Create a cache file for faster configuration loading
  config:clear                      Remove the configuration cache file

Something is being cached. This may have to go back to Freescout developers, but before can you try to keep going and see if we can find any changed files within the past 10 minutes after you fix it with this command ? find "${NGINX_WEBROOT}" -mmin +10 -ls

Apologies for the lack of interaction here as an author of the issue. I got back on track. I will also test this in a few days and if I find anything that wasn't already said, I will post an update. Thank you very much to both for keeping the issue and its progress alive :)


feat: policy page

Proposed Changes

[x] Implement policy page component

policy list
link button on header

[x] Connect component with datasource
[x] test code

Implementation
Policy List Page

❓ Question
@aggre
I need to do the "createGetVotablePolicy" I couldn't figure out how to get a "PolicySetContract". I am trying to implement a function to solve this problem by referring to the dev-kit repositories. However, in the end I couldn't solve the problem. Please advise me as I am commenting on the relevant section. If you don't understand the error in the code I tried, please uncomment out and run it:pray: After the correct implementation is completed, I will implement the test.

@aggre
I am very sorry for the delay in reflecting the review:bow:
I have followed your advice and modified the implementation.
https://github.com/dev-protocol/stakes.social/blob/471ee3a4b1990f4942d19b1f5c4599bf7648ae5e/packages/web/src/fixtures/dev-kit/client.ts#L170
But, this policySet function doesn't have createGetVotablePolicy. From reading the code, it looks like DevkitContract doesn't have a function for createGetVotablePolicy. My confusion still persists, it seems...
I've added test code and refactored it, so please advise me about this part again, I'm sorry :pray:

@k3nt0w I fixed the calling method for getting policy addresses by https://github.com/dev-protocol/stakes.social/pull/368/commits/c9ddff80c17df725bf5674e1c0ac0e79a3d82659.
I'm sorry for specs are hard to convey!

@aggre
Thank you very much🙇♂ I'll fix CI now！

@aggre
I fixed test suite just now! Please check the operation:bow:

@k3nt0w Many thanks! I fixed a bit, so I merge this when after passed the CI.


[improve] Upgrade IPAddress to 5.5.0

Motivation
Upgrading to latest version of IP address parsing lib to get fixes & improvements.
Modifications

Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

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

Project coverage is 73.21%. Comparing base (bbc6224) to head (aa8e8d4).
Report is 368 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #22886      +/-   ##
============================================
- Coverage     73.57%   73.21%   -0.37%
- Complexity    32624    32985     +361
============================================
  Files          1877     1891      +14
  Lines        139502   141943    +2441
  Branches      15299    15566     +267
============================================
+ Hits         102638   103921    +1283
- Misses        28908    29999    +1091
- Partials       7956     8023      +67

Flag
Coverage Δ

inttests
27.21% <ø> (+2.63%)
:arrow_up:

systests
24.66% <ø> (+0.34%)
:arrow_up:

unittests
72.24% <ø> (-0.60%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 392 files with indirect coverage changes


[feat][Java client]: Support Reader Interceptor

Fix #14642
Motivation
Currently, the Pulsar PIP-23 support adds interceptors for producers and consumers to implement message tracing, but it didn't support adding interceptors for readers, this PR wants to support adding interceptors for readers.
Because the Pulsar reader is based on the Pulsar consumer, we could leverage consumer interceptor to implement reader interceptor, use consumer interceptor to wrap reader interceptor, the reader interceptor events should be a subset of the consumer interceptors.
Modifications

Add a new interface ReaderInterceptor, used to customize the reader interceptor.
Support set reader interceptor by ReaderBuilder.
Support set auto-update partition configurations by ReaderBuilder.

Verifying this change
Add ReaderInterceptor test.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (yes)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
Check the box below or label this PR directly (if you have committer privilege).
Need to update docs?

[x] doc-required
(If you need help on updating docs, create a doc issue)

[ ] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

@momo-jun a soft reminder: here is a PR targeted at 2.11 w/ doc-required label.

@momo-jun a soft reminder: this PR is merged, doc work can be started, thanks.

@momo-jun a soft reminder: this PR is merged, doc work can be started, thanks.

@gaoran10 will start to work on a draft as soon as he cleaned up his ongoing tasks.


After exporting the customer status of customer is showing as "0" and "1" in csv and xls file

Framework Version: 0.1.6

Cannot be done database values that are actually stored are exported as it is.


Custom color test branch

Issues to debug

GET app.js returns 404
custom colors aren't appearing in the stylesheet

can this be closed?


Improve error message for exception thrown by composite models

#358

Codecov Report

Merging #396 into dev will decrease coverage by 0.15%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##              dev     #396      +/-   ##
==========================================
- Coverage   82.77%   82.61%   -0.16%
==========================================
  Files          39       39
  Lines        2624     2612      -12
==========================================
- Hits         2172     2158      -14
- Misses        452      454       +2

Impacted Files
Coverage Δ

src/sources.jl
74.35% <75.00%> (+7.69%)
:arrow_up:

src/composition/learning_networks/nodes.jl
67.69% <92.30%> (+0.75%)
:arrow_up:

src/machines.jl
84.96% <93.33%> (+0.83%)
:arrow_up:

src/composition/models/inspection.jl
95.23% <0.00%> (-4.77%)
:arrow_down:

src/measures/finite.jl
95.32% <0.00%> (-2.93%)
:arrow_down:

src/data/datasets.jl
86.11% <0.00%> (-2.78%)
:arrow_down:

src/composition/learning_networks/inspection.jl
87.80% <0.00%> (-0.84%)
:arrow_down:

src/hyperparam/one_dimensional_range_methods.jl
91.86% <0.00%> (-0.37%)
:arrow_down:

src/univariate_finite/types.jl
90.72% <0.00%> (-0.19%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7b6a2ed...b663e41. Read the comment docs.


Create IJ ARC

Airtable is filled, how/where do I do the rest?

Done: https://nodes-dev.desci.com/community/insight-journal


Add seller country whitelist & blacklist config option

So that one can either include (exclusively) or exclude a list of countries from those considered to be valid releases.

closed by https://github.com/michaelhball/discogs_alert/pull/33 and https://github.com/michaelhball/discogs_alert/pull/34


proposal: stage-managed applications

From @wanghong230:

Thought: Consider putting Kargo more fused with Argo CD. Currently, it is two tier system, people need to worry about Argo CD and Kargo. With Kargo, it should be easier, rather than more complicated...

Defining/creating an Argo CD application within the Kargo stage definition could be a idea

I've had this thought before as well. It was even discussed early on, but we decided to start with the assumption that users are probably coming to the table already having a bunch of existing Applications that they'd like integrate with.
I agree it's now time to revisit this idea.
Note that there are significant technical challenges to this, so we shouldn't rush this.

We achieve this with a Helm Chart which creates

a kargo project
Warehouse
Stages
And ArgoCD application per stage

https://github.com/suxess-it/sx-cnp-oss/tree/feat/onboarding-apps/team-apps/onboarding-apps-charts/multi-stage-app-with-kargo-pipeline
Maybe this is a good starting point for the community. It still misses some things like adding AnalysesTemplates but it is a good start for us

I agree that charts can be a good way to approach this in the near-term.

And ArgoCD application per stage

Just note that this may work in only a minority of cases, since Argo CD Applications quite often need to exist in different clusters than the one where all the Kargo data lives.

Case I have in mind is to provide QA engineers a possibility to clickops environments. There is a wizard in 0.9.0, there is a possibility to assemble freight, but there is no possibility to manage ArgoCD applications OOTB... Even PoC of such feature could be very useful. Probably there could be a possibility to refer application template (stored in some CRD or configmap) from stage resource and probably even patch something there like Crossplane patch and transform does
spec:
  promotionMechanisms:
    gitRepoUpdates:
      - repoURL: ...
      ...
    argoCDAppUpdates:
      - appName: ...
  managedArgoCDApps:
    - ref: ...
      patch:
        - to: spec.source.path
          combine:
            fmt: >
              %s/%s/%s
            variables:
              - fromFieldPath: project.metadata.name
              - fromFieldPath: stage.metadata.name
              - literal: xxxyyy

Then kargo-controller could render application from reference, patch it with known fields like name, repoURL and patches and manage it


ConnectionError - Eli5 dataset download

Hi, I have a problem with downloading Eli5 dataset. When typing nlp.load_dataset('eli5'), I get ConnectionError: Couldn't reach https://storage.googleapis.com/huggingface-nlp/cache/datasets/eli5/LFQA_reddit/1.0.0/explain_like_im_five-train_eli5.arrow
I would appreciate if you could help me with this issue.

It should ne fixed now, thanks for reporting this one :)
It was an issue on our google storage.
Let me now if you're still facing this issue.

It works now, thanks for prompt help!


created new branch

created new branch name "Mansoor" and also add new readme file

Make sure that Main branch should be clean @Mans22r


Document tiering system

This is about documenting the implementation of the tiering system for Ehcache developers on the wiki.


devtooling: Allow PR to be used when syncing repos

Sometimes it is nice to refer to some PR when building. This allows you
to do that by:
sync_go_repo_and_patch github.com/openshift-metalkube/kni-installer https://github.com/openshift-metalkube/kni-installer.git 26

This means that instead of using kni origin/master, it will use the code
in PR #26

Build FAILURE, see build http://<IP_ADDRESS>:8080/job/dev-tools/393/

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/395/

arn't we breaking compatibility with the existing patching functionality in this patch? Actually come to think of it we can already do what your trying to do
sync_go_repo_and_patch github.com/openshift-metalkube/kni-installer https://github.com/openshift-metalkube/kni-installer.git https://patch-diff.githubusercontent.com/raw/openshift-metalkube/kni-installer/pull/13.patch
and it take multiple PR's

Yes this does break the existing patch-apply interface, personally I don't have a strong preference for one way or another - good point re the multiple PRs though @derekhiggins and also that solves the rebase problem because we apply them on master?


Typed array in parameter on RPC causes type error.

Tested versions

Reproducable in 4.3dev4, 4.2.1

System information
Godot v4.3.dev4 - Windows 10.0.22631 - GLES3 (Compatibility) - NVIDIA GeForce GTX 1660 Ti (NVIDIA; <IP_ADDRESS>58) - AMD Ryzen 7 5800X 8-Core Processor (16 Threads)
Issue description
When calling an RPC that has a parameter with type Array[Vector2i], it will error as it cannot convert the argument from array to array. This seems incorrect as the types are the same and works when just using the Array type, but I might be misunderstanding how RPCs work.
Steps to reproduce
Have 2 instances running, one as client and one as host. On host press the type bug button and it will throw an error.
Minimal reproduction project (MRP)
BugTesting.zip

Duplicate of #69215.
Fixed by #78219 for future 4.3.

Thanks for reporting the bug nonetheless!


wrong url in readme

In HoloLensForCV/Samples/py/README.md
Install and Launch the [Streamer] (https://github.com/Microsoft/HoloLensForCV/tree/master/Tools/Streamer) UWP application on your HoloLens.
The url is not correct anymore.
StreamerPV or StreamerVLC, I'm not sure which is the correct one.

Hello,
I think it depends on what you want to see. PV is the frontal ccd camera, which represents what the user will see if he wears it. VLC are those four gray-scale cameras on both sides of the hololens for environmental understanding. They are identified in this sdk as HoloLensForCV::SensorType::VisibleLightLeftLeft, VisibleLightLeftFront, VisibleLightRightFront and VisibleLightRightRight.


Release button in notification emails

It appears that there has been a further update to the notification emails, and that they now include  Release button once again that releases the email directly rather than making the user go to the Quarantine center. I'd like to link my users to this article for questions, but don't feel comfortable doing so until this item is detailed.

@lmeekscssa - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further. Thank you.

@mypil My apologies. I've not reported issues on Microsoft documentation before. The page I'm referring to is here: https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/use-spam-notifications-to-release-and-report-quarantined-messages

@lmeekscssa  - From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/windows-itpro-docs/commit/4c21ab49b8def24df3577015133c61bda1f47883 but it may take a few days for the merged content to appear in the article.
Thank you for your contribution to make the docs better! Much appreciated!

@lmeekscssa - From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/microsoft-365-docs/commit/98e9b72a001da61b0f551109d722b3215c20bdb7.
Thank you for your contribution to make the docs better! Much appreciated!


Update template to stop deprecation warnings

Fixes deprecation warnings like:
Deprecated: Found tags option '~@wip'. Support for '~@tag' will be removed from the next release of Cucumber. Please use 'not @tag' instead.

I've never contributed to a project before, so if I'm doing something wrong let me know.  Guidance is welcomed!

@ctreptow thank you.
There are some automated tests that run on Travis, and we merge the PR if the tests are all passing.
Could you please try to get them to pass, ie run your branch in Travis, and when they pass do the PR.

Ok, I'll close this until I see green.  Thanks!

Don't close failing pull requests. Just push new commits until it's green.


getDefaultTopicRouteInfoFromNameServer(final String topic, final long timeoutMillis) improper parameter

The issue tracker is used for bug reporting purposes ONLY whereas feature request needs to follow the RIP process. To avoid unnecessary duplication, please check whether there is a previous issue before filing a new one.
It is recommended to start a discussion thread in the mailing lists or github discussions in cases of discussing your deployment plan, API clarification, and other non-bug-reporting issues.
We welcome any friendly suggestions, bug fixes, collaboration, and other improvements.
Please ensure that your bug report is clear and self-contained. Otherwise, it would take additional rounds of communication, thus more time, to understand the problem itself.
Generally, fixing an issue goes through the following steps:

Understand the issue reported;
Reproduce the unexpected behavior locally;
Perform root cause analysis to identify the underlying problem;
Create test cases to cover the identified problem;
Work out a solution to rectify the behavior and make the newly created test cases pass;
Make a pull request and go through peer review;

As a result, it would be very helpful yet challenging if you could provide an isolated project reproducing your reported issue. Anyway, please ensure your issue report is informative enough for the community to pick up. At a minimum, include the following hints:
BUG REPORT

Please describe the issue you observed:

What did you do (The steps to reproduce)?
as we can see that the method name has indecated the target topic, it's unneccesary to pass a topic parameter again

What is expected to see?
just using getDefaultTopicRouteInfoFromNameServer(final long timeoutMillis) and pass default topic internally to replace

What did you see instead?

Please tell us about your environment:

Other information (e.g. detailed explanation, logs, related issues, suggestions on how to fix, etc):

FEATURE REQUEST

Please describe the feature you are requesting.

Provide any additional detail on your proposed use case for this feature.

Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?

If there are some sub-tasks involved, use -[] for each sub-task and create a corresponding issue to map to the sub-task:

sub-task1-issue-number: sub-task1 description here,
sub-task2-issue-number: sub-task2 description here,
...

Hello, @miles-ton , Welcome to submit pr to polish it :-)


Occasionally retrieving "connection timed out" errors

I get this error every so often when utilizing sqlx with pgx, and I believe it's a configuration error on my end:
error: 'write tcp [redacted-ip]:[redacted-port]->[redacted-ip]:[redacted-port]: write: connection timed out
This occurs when attempting to read from the database. I init sqlx in the startup phase:
package main

import (
    _ "github.com/jackc/pgx/stdlib"
    "github.com/jmoiron/sqlx"
)

//NewDB attempts to connect to the DB
func NewDB(connectionString string) (*sqlx.DB, error) {
    db, err := sqlx.Connect("pgx", connectionString)

    if err != nil {
	    return nil, err
    }

    return db, nil
}

Any structs responsible for interacting with the database have access to this pointer. The majority of them utilize Select or Get, and I understand those return connections to the pool on their own. There are two functions that use Exec, and they only return the result and error at the end of the function.
Other Notes

My Postgres db supports 100 max_connections
I only showed a few active connections at the time of this error
I am not using SetMaxIdleConnections or SetMaxOpenConnections
Refreshing the page and triggering the request again always works

Any tips on what might be happening here?
EDIT: I did not mention this server is on compose.io, which in turn is hosted on AWS. Is it possible AWS turns these connections into zombies because they've been open for so long and the timeout occurs after unsuccessfully trying them one by one?

This seems to work
conn := "user@ip/password"
dbConn, err = sqlx.Open("pgx", conn)
if err != nil && dbConn == nil {
	log.Fatal(err)
	time.Sleep(time.Second * 10)
	return nil
}
defer dbConn.Close()

Timeout is can try for / break / continue with sleep 10 seconds to reconnect

@KingRider I've actually never had a problem in this init code. The database is always available and ready for connections-- the issue seems to stem from the network being unhappy with long-running connections and killing them off (the client never knows when this happens). I was able to find a workaround by setting the maximum lifetime of the connection, but I have more follow-up to do on my end to see if I can shorten the TCP Keep Alives.
I will render any further discussion here


Support non optimizing ng serve

🚀 Feature request
Command (mark with an x)

[ ] new
[ ] build
[x] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Description
ng serve does not support anymore to NOT optimize the built js files. While this already wasn't present using angular.json in ng12, one could use ng serve --optimization=false, but this option is no more available.
This not only makes it much easier to debug (stacktraces are complete rather than on a.eT()), but also makes build time faster.
PS: the option --source-map=false is also gone.
Describe the solution you'd like
Be able to set optimization=false in angular.json for builds of type dev-server.
Describe alternatives you've considered
To regain speed of build, all we can play with for now is ES target and build cache. This is not as of great benefit as disable optimization.

A newly generated project will have the serve target setup to use the development configuration of the build target.  This, in turn, will be setup to not use optimization for the project by default.

Default Serve Target
      "serve": {
        "builder": "@angular-devkit/build-angular:dev-server",
        "configurations": {
          "production": {
            "browserTarget": "new-v13:build:production"
          },
          "development": {
            "browserTarget": "new-v13:build:development"
          }
        },
        "defaultConfiguration": "development"
      },

Default Build Development Configuration
          "development": {
            "buildOptimizer": false,
            "optimization": false,
            "vendorChunk": true,
            "extractLicenses": false,
            "sourceMap": true,
            "namedChunks": true
          }

If your project is not serving a development build, please review the build and serve configurations for the project based on the settings shown above.

Hi there,
Thanks, it works for me!
Do you know when the change was made in config? It may be worth include in an upgrade guide.
Best

These changes were made in version 12 and an automatic migration when using ng update was included with the release.  Please note, however, that the use of third-party builders may prevent migrations from successfully completing.


Create diagram of the KFP

Context
Create a diagram of the project and how user isolation works so it can be added to the reference architecture public docs.
What needs to get done

Create a diagram of the profile user isolation in CKF: high-level diagram of the components, their integration points and dependencies.
Add the diagram to a reference architecture draft document

Definition of Done
There is a diagram of the KFP components (not published)

Added the diagram to the reference architecture document. Closing this.


Update docs prism theme

Changes

Update prism style to vsDark
Before:

After

Update white highlight to make code more readable
Before:

After

Remove leftover git bash symbols

Update microsite README to include magic comments

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@Rugvip can you take a look?

@jhaals @Rugvip can you please take a look?


Docker improvements

Add a msgbroker service running RabbitMQ to docker-compose
Quote port numbers in docker-compose, as recommended by the spec
Configure and enable XDebug in the Dockerfile for the application

Tested locally, using PhpStorm's auto-debug functionality (no configuration required; PhpStorm autodetects) and by creating an account request and observing a message enqueued in RabbitMQ (though I had to manually create the exchange, queue, and bindings in the RabbitMQ management interface).
Note that the rabbitmq.conf file is deliberately empty, but exists is a placeholder if we want to tweak the behavior of the RabbitMQ instance in msgbroker.

I had to manually create the exchange, queue, and bindings in the RabbitMQ management interface

I think the default exchange (the empty string) is a "direct" exchange with automatic bindings to queues with the name specified in the message routing key. I wonder if we can configure a startup script within the rabbit container to automatically create the objects we need for the default configuration - it might just be a queue named for the default notification target for the default domain?
I also can't remember how the default configuration works in the container; it'd be really nice if all of this works out-of-the-box. But all of this is nice-to-haves separate from this PR 😄

I think the default exchange (the empty string) is a "direct" exchange with automatic bindings to queues with the name specified in the message routing key. I wonder if we can configure a startup script within the rabbit container to automatically create the objects we need for the default configuration - it might just be a queue named for the default notification target for the default domain?

Yeah, there's a way to pass in queues via the configuration to be created at startup if they don't exist (through a definitions.json file that'd be trivially easy to add alongside rabbitmq.conf). That being said, it looked like during my testing that the default routing key is 1; is that expected?

I also can't remember how the default configuration works in the container; it'd be really nice if all of this works out-of-the-box. But all of this is nice-to-haves separate from this PR 😄

Yeah, the default configuration probably wouldn't work because it needs the database credentials and the container names instead of localhost for connecting to the DB and MQ instances (though admittedly I haven't tried it, I just made the changes locally right off the bat). My initial thinking is we could have a Docker-specific config.local.inc.php in the docker folder that gets moved into place when the application container is started if you don't already have a config file. I'll come back to all this after addressing the code review backlog!

A default routing key of 1 really doesn't matter. It's meaningless to the ACC tool, as it's the MQ infrastructure and ultimately Helpmebot itself which deals with what the different routing keys mean.
In Helpmebot, there is a way to map arbitrary strings to channel names, but if the routing key isn't specified in the map, it'll default to being the real target of the message. Helpmebot stores the list of acceptable channels in RabbitMQ by reading from a queue which is only bound on the allowed routing keys


Update server.rb

Fix nil rack_options by defaulting to {}
#2288

FWIW, when I used this branch for Padrino it fixed my issues noted in #2288

Great, thanks for testing! @achiurizo can you review and merge into master


Station transition latency reduction (?)

Just thinking out loud here, but judging by the debug prints that the radio spits out into the serial monitor, it appears the radio issues a stop request before connecting to the next station. This leaves a considerable audio gap while  it connects to the next server, resolves metadata, etc.
Depending on how quickly the radio is able to resolve the new stream, could we not skip the stop request and just append the new stream onto the existing buffer? I read somewhere in the docs that the ring buffer should be able to hold enough for about 1.5 seconds of audio @ a certain bitrate, which although it probably wouldn't eliminate the gap entirely in all cases, it just seems like stopping the stream before doing anything else is only prolonging the gap between streams.
There's probably a good reason for doing it how it currently is done, but if not, wouldn't it make more sense to leave it playing as long as possible while transitioning servers?

I did a little bit of testing, and despite my choppy output from the most recent version (discussed in another issue) it seems with a minor code change to remove a stop request when changing stations, that the gap is significantly reduced.  Now that I've gone back to a functional version, I will test the same change to see if the results are similar.

Well it appears to work. Depending on how fast the radio picks up the next stream, it seems like at least some of the time the transition is gapless. I haven't been able to thoroughly test the modification, because I have yet to take the time to create an ini file that has only stations that work, but i'm happy to report that switching stations without issuing a stop command doesn't cause a crash or anything overly undesirable.
All I did was comment out the if structure that checks if the radio is stopped and issues the command if it isnt in the loop function (where it decides to change stations). I'd create a pull request but I haven't ever done that before so I would likely need to figure out how. It's a simple enough change to implement tho, I doubt it really warrants a PR.
Let me know if you would rather I send a PR, or if you have questions about where the changes I made were.

During the switch station process, the "loop()" function is not executed.  So there are no reads from the ringbuffer.  Switch time in normally about one second.  Some buffering is done by the VS1053 itself (size of audio FIFO is 2048 stereo (2x16 bit) samples) and that's what you can gain by postponing the stop command.

Well it seems to improve the fluidity of changing channels as there is less dead time. Unfortunately, for whatever reason the module will briefly pause for maybe a second as soon as it receives the command, and then finishes playing out whatever buffer it still has after this short pause.  The new station comes in pretty much immediately after the remaining audio data runs out from the last  station.
I haven't been able to trace the reason for the pause, but it occurs to me that it's either the heavy loading caused by the new connection, or maybe one of the various delay commands scattered throughout the sketch.
However, it seems odd that the buffer inside the mp3 module doesn't seem to offset the gap caused by the reconnection at all.
Perhaps the esp32 will be better at handling this, with its dual core, and other performance improvements.
If there are any deliberate reasons for the module to be taking a momentary break here,  off the top of (anyone's?)  head, that might point me in the right direction, would appreciate the tip.
Regardless, I find it much less noticeable if keep the audio going as much as possible.
Also, would it be fair to assume you've already considered using an interrupt to fill the ring buffer  at more tightly controlled intervals? Or maybe even the ticker library? I don't think it's a real  interrupt, but I could be wrong about that and it could be worth looking at if it allows the code to generate the data array in a more controlled & consistent fashion.


Chore: increase lecture unit test coverage

Checklist

[x] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Server: I followed the coding and design guidelines.
[x] Server: I added multiple integration tests (Spring) related to the features (with a high test coverage)
[x] Server: I added @PreAuthorize and check the course groups for all new REST Calls (security)
[x] Server: I implemented the changes with a good performance and prevented too many database calls
[x] Server: I documented the Java code using JavaDoc style.
[x] Client: I followed the coding and design guidelines.
[x] Client: I added multiple integration tests (Jest) related to the features (with a high test coverage)
[x] Client: I added authorities to all new routes and check the course groups for displaying navigation elements (links, buttons)
[x] Client: I documented the TypeScript code using JSDoc style.
[x] Client: I added multiple screenshots/screencasts of my UI changes
[x] Client: I translated all the newly inserted strings into German and English

Motivation and Context

The lecture unit feature was implemented without sufficient test coverage (especially in the client). This PR addresses this issue by greatly improving the test coverage.
Description

Client

File
TestCoverage

video-unit-form.component.ts
90%

create-video-unit.component.ts
92%

edit-video-unit.componen.ts
96%

video-unit.component.ts
100%

videoUnit.service.ts
100%

attachment-unit-form.component.ts
90%

attachment-unit.component.ts
88%

create-attachment-unit.component.ts
92%

edit-attachment-unit.component.ts
98%

attachmentUnit.service.ts
100%

exercise-unit.component.ts
100%

create-exercise-unit.component.ts
88%

exerciseUnit.service.ts
100%

unit-creation-card.component.ts
100%

lecture-unit-management.component.spec.ts
84%

Server

File
Coverage

TextUnitResource
80%

VideoUnitResource
82%

ExerciseUnitResource
87%

AttachmentUnitResource
82%

LectureUnitResource
80%

Steps for Testing

No big changes to the functionality in this PR. Just Create, Update, Delete, Reorder the different unit types and also view them as a student :)

Steps performed by me on the TestServer:

[x] Created Lecture with all kind of lecture units and check corresponding student view
[x] Reordering of the lecture units and check corresponding student view
[x] Updating all kind of lecture units and checking corresponding student view
[x] Removing all kind of lecture units and checking corresponding student view

Test Coverage

See Description
Screenshots

The view button were removed

Why did you delete this?

This should stay available and user should be able to navigate to it by either clicking the ID or the lecture title

reverted commit


drush commands using platform aliases fail env: drush: No such file or directory

Attempting to use the platform-created drush aliases results in an error:
$ drush [ALIAS.REDACTED] status
env: drush: No such file or directory

I attempted troubleshooting using the discussion in #626, but without success.

I know this issue is long past but to save anyone else this pain here's the solution to this issue.
After a very long chain of back on forth comments with support the error reported is cause by the code base missing the required .environment file to allow drush to run on the remote server.
https://docs.platform.sh/frameworks/drupal8/drush.html#accessing-drush-within-the-project
Coping the .environment from the root in from the default [Drupal 8 template] to your project (https://github.com/platformsh/template-drupal8/) solved this for me.

Ahhh... so it was the remote drush command that wasn't available in the PATH.
Thanks a lot for the solution @finteractive


An error occurred when click 'Go SDK' folder.

My vscode is installed on windows computer (with Windows 11 system). My development enviroment is an ubuntu virtual machine. I installed go and vscode-server on that ubuntu virtual machine. Everything works well until this afternoon, I got an error notification below when I click 'Go SDK' folder in 'GOMOD EXPLORER' view:

Enviroment:

vscode installed on Windows 11
ubuntu virtual machine with go 1.19.3
plugin installed on ubuntu Go Mod Explorer v0.3.8

Hope to get your help.

Hi, @jiangfangxin I'm sorry, it's my fault.  Reverted to 0.3.7 now.
Hi, @zollum Do you have any way to fix it? It doesn't seem to work on other OS.
Related #45

Thanks! It works after changing the version to 0.3.7. 👍

really sorry, please check the new PR #49

@zollum thank you very much!


there is no documentation

why there's no documentation? the docs in the source code is messed

Documentation is in the headers, for example the docs for sokol_gfx is in sokol_gfx.h. IMHO it is usually pretty clear. However the documentation assumes that you are familiar with the underlying computer graphics concepts, for example you need to know what vertices, textures, vertex and fragment shaders are. If you are not familiar with those concepts have a look at the tutorials at https://learnopengl.com.


Changing a profile using an event

Describe the bug
The profile on the tablet does not automatically switch using an event (with a simple click on the button it works)
To Reproduce

Expected behavior
Changing the profile on the tablet
Video
I switch between applications and the profile changes in the program on the computer, but not on the tablet
https://github.com/Macro-Deck-App/Macro-Deck/assets/72551624/dca03b66-7c19-4041-bca2-bcbdb4744d14
If I change profiles not through an event, but through pressing a button, then everything works

Button Settings

https://github.com/Macro-Deck-App/Macro-Deck/assets/72551624/cae59ed6-a4a2-4f3f-b6cf-fcf9e56353a7
Log file
2024-05-10.log

I realized that this can be done through folders, but the problem still remains

You need to change the target. Currently you set it to "Where executed" and because the event is executed on your computer, it changes the profile there. Change this to your tablet and it should work.


SQL Translation: modified paste to map to "CONCAT_WS" &  added paste0 mapping to "CONCAT"

Allows specification of a separator when contacting string variables, and brings behaviour more in-line with paste`paste0` in base R.

We are about to checkpoint a CRAN release so we will take very few PR to make sure there is enough time to test the merges or be very confident they won't cause regressions. My initial thought is to merge this one before CRAN since: 1) it's safe, 2) provides functionality that would otherwise be hard to workaround and 3) changes the API which would be good to avoid after a CRAN release.

LGTM as well, although there's unfortunately merge conflicts now.

Do you need me to take a look at those?

I've added this in by hand now -- thanks for the PR and sorry for not merging before conflicts arose!


Include the Engine when cloning a FromClause

Fixes #239
Turns out I was right the first time, even though my fork was out of date. If the Engine isn't preserved during FromClause.Clone(), then 2 of the 3 unit tests I added fail.

Before I launch into this long comment, I wanted to say how much I appreciate this library, and I'm sure there are considerations here that I'm missing.
Running some further tests has led me to other issues.
var query = new Query("Foo")
    .ForSqlServer(q => q.From("Bar"));

I would expect this to give me a table name of Bar for SqlServer and Foo for all other compilers. But when ClearComponent() gets called in the context of the second line, it deletes the first FromClause, because the clause's Engine is null. As a result, compiling this query fails for all other engines (because the only FromClause is for SqlServer).
public Q ClearComponent(string component, string engineCode = null)
{
    if (engineCode == null)
    {
        engineCode = EngineScope;
    }

    Clauses = Clauses
        .Where(x => !(x.Component == component && (engineCode == null || x.Engine == null || engineCode == x.Engine)))
        .ToList();

    return (Q)this;
}

I agree that if engineCode == null, the method should remove all clauses for that component. But if engineCode != null then I think it should only remove the clauses where x.Engine == engineCode. So I don't think that x.Engine == null belongs in the Where(). It looks like a couple of the Aggregate test cases fail if this change is made, because it breaks functionality in Compiler.TransformAggregateQuery(). That function does expect to clear all aggregate components – but it can accomplish that by calling query.ClearComponent("aggregate", null) instead of query.ClearComponent("aggregate", EngineCode).
It's a little complicated because I think that From() is the only method outside of a compiler that calls ClearComponent() and is therefore operating directly on the original query, not a clone, so what it does affects what all compilers will see.
There's a similar issue in BaseQuery.GetOneComponent(). This function first calls GetComponents(), which has the same kind of Where() as ClearComponent(). Here I think it's appropriate, however – you want to get the set of generic components as well as the ones that apply to the current engineCode. However, within GetOneComponent() I think that a component which matches engineCode should be prioritized over one where x.Engine == null. In my test case at the top of this comment, if I make the suggested change to ClearComponent() then all compilers work as expected except for SqlServer. When the SqlServer compiler calls GetOneComponent("from"), it gets back the generic FromClause because that happens to be first in the list, rather than getting back the FromClause which is specifically for SqlServer.
I get that this is complex because From() is a bit of an oddball. First, I think it's the only method outside of a compiler that calls ClearComponent(), so it's operating directly on the original query, not a clone, and therefore affects what all compilers will see. And second, each query can only have one FromClause per compiler.
Actually, each query can only have one Limit as well, and it looks like Limit() has some similar issues. In the following query, all compilers will get a limit of 10, because the second line overwrites the first:
var query = new Query("Foo")
    .Limit(5)
    .ForSqlServer(q => q.Limit(10));

That's a lot from what I thought was a small issue. I'm interested to hear what you think.

I'll add that both From() and Limit() can be made to behave better without changing ClearComponent() or GetOneComponent(), and I can put together a separate PR if you'd like to see what that looks like.

@asherber thanks for this great analysis, so I can conclude that we have the following issues here:

[ ] GetComponent() should prioritize engine specific when passed.
[ ] ClearComponent() should respect the engine context (This may need some discussions)

Is there something that I've missed?

While ClearComponent() doesn't work the way I would expect, I agree that it shouldn't be changed without further discussion. There may be side effects beyond what I've looked at.
Here's my current suggestion for GetOneComponent():
public C GetOneComponent<C>(string component, string engineCode = null) where C : AbstractClause
{
    engineCode = engineCode ?? EngineScope;

    var all = GetComponents<C>(component, engineCode);
    return all.FirstOrDefault(c => c.Engine == engineCode) ?? all.FirstOrDefault();
}

I think GetComponents() should continue to return components in native order from the list. But the code above tells GetOneComponent() to look first for a matching engine-specific component, and otherwise to take the first overall.
In addition, here's what I would suggest for From(). Something similar can be done for the other From() overloads, and for Limit(). The new AddOrReplaceComponent() refactors the approach currently used in Limit(), allowing us to maintain a single component for a given engine (or for a null engine).
public Q From(string table)
{
    var newClause = new FromClause
    {
        Table = table,
    };

    return AddOrReplaceComponent("from", newClause);
}

public Q AddOrReplaceComponent(string component, AbstractClause clause, string engineCode = null)
{
    engineCode = engineCode ?? EngineScope;

    var current = GetComponents(component).SingleOrDefault(c => c.Engine == engineCode);
    if (current != null)
        Clauses.Remove(current);

    return AddComponent(component, clause, engineCode);
}

I can work all of this into this PR, if you agree.

@asherber you are right,  agree from my side.

Okay, thanks. I'm a little swamped this weekend, will probably look at this early next week.

@ahmad-moussawi I think this is almost complete now. There's one thing which troubles me a little that has to do with Limit and Offset sharing a component.
var query = new Query("mytable")
    .Limit(5)
    .Offset(10)
    .ForMySql(q => q.Limit(20));

In the above example, I first set a generic limit and offset, and then I specify that MySql should have a different offset. As part of that code, the generic offset gets copied into the MySql LimitOffset. Now MySql has LIMIT 20 OFFSET 10 and all others have LIMIT 5 OFFSET 10. But now consider the following:
var query = new Query("mytable")
    .Limit(5)
    .Offset(10)
    .ForMySql(q => q.Limit(20))
    .Offset(50);

Here the generic offset of 10 gets copied into the MySql LimitOffset, but then the generic offset gets changed to 50. Looking just at the query, I would expect that MySql would use OFFSET 50, because that was the latest generic value given and there is no MySql value given. But since the MySql LimitOffset now has its own offset value (inherited from the first generic 10), it doesn't see the change to 50.
One could argue that this is a poorly formed query -- that the user shouldn't have provided two offsets. But I can see a case where a base query is set up, as in the first example, and then later in the code the generic offset is changed based on some other condition. Should the library handle this better? If so, then we'll need to stop copying the generic offset into the LimitOffset and instead have the compiler look further up the chain for the correct value.
(The generic LimitOffset would have limit 5, offset 50, and the MySql component would only have limit 20. When the MySql compiler looks at these values, it would see that there is no offset on that component and would then look at the generic component to see if there's an offset there.)
What do you think?

Hmm, not sure I can solve for this last case without one of these more drastic steps:

Include properties on LimitOffset to indicate whether the values should be inherited from the generic component.
Make Limit and Offset int?
Make Limit and Offset separate components/clauses

The problem is that a value of 0 could mean "look at the generic", or it could have been explicitly set.

@asherber You are totally right, this scenario can happen when using it dynamically, I think separating the limit and offset to two components is the best scenario here.
if you like, open another issue for it, so we can track it alone, and I will merge this PR for now.

Thanks, will do.

@ahmad-moussawi  I've got the work on LimitOffset done. Since it builds on this PR, which hasn't been merged yet, okay if I just add the commit to this PR?

@asherber yes sure 😃

Merged with master now, Thanks!


Use no-shared to stop getting libcrypto and libssl linked dependencies

I set the compile options to no-shared. It looks like this removed the dependencies. The executable runs on my machine, but I need to do more testing.
Before
otool -L /Users/ebeland/apps/ruby-packer/rubyc
/Users/ebeland/apps/ruby-packer/rubyc:
/System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 1858.112.0)
/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1311.100.3)
/usr/lib/libobjc.A.dylib (compatibility version 1.0.0, current version 228.0.0)
/var/folders/s9/mtk8pgj529j4lq_b7bl2fkkc0000gn/T/rubyc/local/lib/libssl.1.1.dylib (compatibility version 1.1.0, current version 1.1.0)
/var/folders/s9/mtk8pgj529j4lq_b7bl2fkkc0000gn/T/rubyc/local/lib/libcrypto.1.1.dylib (compatibility version 1.1.0, current version 1.1.0)
/usr/lib/libutil.dylib (compatibility version 1.0.0, current version 1.0.0)

After
otool -L /Users/ebeland/apps/ruby-packer/rubyc
/Users/ebeland/apps/ruby-packer/rubyc:
	/System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 1858.112.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1311.100.3)
	/usr/lib/libobjc.A.dylib (compatibility version 1.0.0, current version 228.0.0)
	/usr/lib/libutil.dylib (compatibility version 1.0.0, current version 1.0.0)

I also turned off some debug settings to see if we could get a little smaller executable. It works, but the difference isn't major.

@taraszka, it might be fixed here.  Does this look off-base in any way to you? It seems to have removed the visible dependencies and works on my machine so far.