Parse out information from demo_2.0.bag

Is there any tool that can parse out information from demo_2.0.bag？

You should check out the Apollo 2.0 version and parse it with python. For example: https://github.com/ApolloAuto/apollo/blob/r2.0.0/modules/tools/rosbag/stat_mileage.py

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


time: UnmarshalJSON does not respect escaped unicode characters

What version of Go are you using (go version)?
1.16.6
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
The playground.
What did you do?
Unmarshaled a JSON string containing an escaped unicode character into a time.Time.
https://play.golang.org/p/MV6VeYcalrQ
What did you expect to see?
I expected it to parse correctly as per the JSON spec.
What did you see instead?
It did not understand the escaped character.

parsing time ""2009-11-10T23:00:00\u005a"" as ""2006-01-02T15:04:05Z07:00"": cannot parse "\u005a"" as "Z07:00"

This is unfortunate. In order to fix this, we either:

implement JSON string decoding in the time package,
have the time package depend on the json package, or
move just the JSON string decoding logic from the json package into some internal package that both time and json can access.

As an aside: it was a mistake to implement UnmarshalJSON methods on time.Time, and there should only have been the UnmarshalText methods since those take in the unescaped string.

Would it be plausible for the JSON package to special-case time.Time, ignoring (time.Time).UnmarshalJSON in favor of a correct implementation?
This obviously doesn't scale to all types everywhere, but perhaps it would suffice for standard library types where we regret adding an UnmarshalJSON method.

Would it be plausible for the JSON package to special-case time.Time, ignoring (time.Time).UnmarshalJSON in favor of a correct implementation?

That's another option. One detriment is that it fails to detect cases of embedding:
type MyTime struct { time.Time }

Here MyTime forwards all methods of time.Time and then probably adds a few more. From the perspective of the json package, it cannot accurately tell if MyTime is promoting time.Time.{Marshal,Unmarshal}JSON or not. The best it can do is check that MyTime has an anonymous field of type time.Time, but reflect provides no way to tell if the UnmarshalJSON method is explicitly declared on the MyTime type (because perhaps they really did want different formatting) or was forwarded from time.Time.

It also wouldn't fix it for other implementations of json (and yaml?) decoders

I suspect YAML decoders will be fine because they will probably use the UnmarshalText methods, but who knows.

As an aside: it was a mistake to implement UnmarshalJSON methods on time.Time, and there should only have been the UnmarshalText methods since those take in the unescaped string.

For what it's worth, time.Time.UnmarshalJSON was added before UnmarshalText existed.  It was one of the reasons that UnmarshalText was invented.  See https://golang.org/s/go12encoding.

Crazy idea: The json package can try the UnmarshalJSON method first. If it errors, the JSON token is a string, and a UnmarshalText method exists, then try the UnmarshalText method.
This ensures backwards compatibility where UnmarshalJSON takes precedence over UnmarshalText. This should be mostly compatible as all it would ever do is convert a few error cases to non-errors (presumably all desirable behavior changes).

Make it a specific sentinel error returned by UnmarshalJSON, and it'll be entirely backwards compatible.

Crazy idea: The json package can try the UnmarshalJSON method first. If it errors, the JSON token is a string, and a UnmarshalText method exists, then try the UnmarshalText method.

hey @dsnet , the new way that encoding/json/v2 works allowed us to avoid this error we had.
https://go.dev/play/p/ocmILAWHDXS
it worked wonders. Didn't see these types of bugs mentioned in the encoding/json/v2 github discussion but just so you knew that it also tackles this.


Update mockery mock class names for php 54.

Coverage remained the same at 100.0% when pulling 8d7bfd9144b2ff6444f42d7c9bfd03391e012361 on candasm:php_54_class_name_bug_fix into e2cf6d1bef39d4844bfd9bc3b271820478a282e9 on shift31:master.

Coverage remained the same at 100.0% when pulling 8d7bfd9144b2ff6444f42d7c9bfd03391e012361 on candasm:php_54_class_name_bug_fix into e2cf6d1bef39d4844bfd9bc3b271820478a282e9 on shift31:master.


[5.1] Add Pre Task hook to Scheduler

Hi. There is currently a Post Task hook for the scheduler (using then())
It would be nice if you could hook into the Pre Task, to run anything required before the command itself runs.
A specific use case for this is to Ping a monitoring service that your task is about to start - so you can track/monitor the length of time the task took to run.
If this PR is accepted I'll do up a modification for the Docs to show it there as well.
p.s. I used the naming convention of before() and beforePing() - but it might be before to call it now() and nowPing() which is kind of the opposite of then()? Let me know if you want me to change it...

:-1: You should be using middleware here. We already removed filters in the router. I don't think we should add stuff here too.

@GrahamCampbell - I'm confused - what does this have to do with filters or middleware?
We are saying we want to run a specific task(s) before we run the command.
It is not a filter (i.e .to check if we want to run the command) - the command will always run after the before() command.
And how would you apply middleware here anyway?

We are saying we want to run a specific task(s) before we run the command.

Yeh, it's the same idea.

And how would you apply middleware here anyway? Its not a HTTP request.

Middleware don't have to apply to routing. The exist in the command system of laravel too.

I don't want to add filters anywhere new. Laravel's already decided they're a bad design pattern and they're removed in 5.2.

But these are not filters?
Then a pre-coomand hooks - to run additional commands prior to the command itself running?
It is no different to the Post hook function - that allows you to run additional commands after the command itself?

These are pre-command hooks - to run additional commands prior to the command itself running?

That's exactly what middleware are for.

...so by that same logic - we should actually remove then() - since you can just use middleware to run tasks after the command is run?

Yeh. Though we don't have middleware for this yet. I'm saying we should do that instead of more filter type stuff. Currently we only have middleware for the command bus.

I'm fine with this other than the wording of beforePing. That reads awkwardly to me.

@taylorotwell - what about 'first()' and 'firstPing()'
Or
'firstly()' and 'firstlyPing()'

$schedule->command('inspire')
    ->firstPing('http://callback/endpoint')
    ->then(function () { doSomething(); });

$schedule->command('inspire')
    ->pingBefore('http://callback/endpoint')
    ->then(function () { doSomething(); });

@taylorotwell - I've changed it to first() and firstPing() - which I thinks reads the best?
$schedule->command('inspire')
    ->firstPing('http://callback/endpoint')
    ->then(function () { doSomething(); });

If this is accepted I'll do a PR on the Docs for you...


README.MD Arduino IDE installation instructions reference non-existing

For setting up the LilyGO T-Echo in Arduino IDE, the README.MD mentions to select Adafruit nRF52 board Nordic nRF52840(PCA10056), but this board does not exist (anymore?) in boards manager Adafruit nRF52 by Adafruit, version 1.6.1.

Which board shall be selected instead? Please update the README.MD.

Which board shall be selected instead?

Nordic nRF52840 DK


FIX for ruby 3 incompatibility in sidekiq middleware, fixes #229

See #229 for more details.
The test case might seem a bit weird, but I didn't find a more elegant way to reproduce Sidekiq's middleware initialization. So definitely open for suggestions.

Sidekiq middleware chain for reference: https://github.com/mperham/sidekiq/blob/main/lib/sidekiq/middleware/chain.rb

@mperham what do you think of the workaround? I guess param handling changes between 2.7 / 2.8 / 3.0 are a bit of a sore point?

Lgtm!

thanks @benzimmer  !


UX/UI design, chapter 1, Sunday Roll Call

Leave a comment with these two things:

A summary of your last chapter using only emojis
Something you'd like to share, anything goes! (within respect)

emoji cheat sheet

Super 👌

💻🤝🤜🤛✌

:slightly_smiling_face:

Good 🙂

:smile:
:heart_eyes:

🤩😊😞🤒😵💫🤧😃

👌
🤝

I was very hard week but it worths it😅🤗

👌🤝🤩😊

Great 🤓

:wave: :ok_hand:

good 😃

👌

UX/UI my favorite :smiley:

...

not good in UX/UI 😔

Lying on a bed:/


New internals

extremely early, attempting to address or make possible to address numerous issues, including but not limited to:

#324
#290
#21
#463

The general ideas are as follows:

Factor out "Stack" from "Workspace", specifically the bits in server/workspace.go that might have multiplicity N >= 0 relative to each workspace.
Refactor "Component" in to physical and logical halves. The logical half will remain called "Component" and the physical half will be called a "Resource". Main distinction is that resources are expected to be external, have a globally unique IRI of some kind, and it is expected that their lifetime is shorter than a component and subject to external influences. Components may have zero or more resources. For example, a process-component will have start & stop, and each time it starts/stops, it will create/destroy a unix-process-resource.
Keep a pending-operations log for CRUD operations on resources, so that if an operation fails, the user can debug/recover.
Rework "apply" to use initialize/start/stop/dispose, causing components to create/delete resources, instead of create/delete-ing processes, containers, etc directly. (ie fix #290)
Write a data migration codepath that takes the component state from above and moves it to some tables in sqlite instead of storing in the state.json file.

going to close this PR for now, as I'm going to come at this problem from a different direction


Unable to uninstall/install ESX VIB from master

Installed
[root@sc-rdops-vm04-dhcp-99-161:~] esxcli software vib list |grep vmdk
esx-vmdkops-service            1.0.0-0.0.1                            VMWare  PartnerSupported  2016-06-17

Attempt to uninstall failed
[root@sc-rdops-vm04-dhcp-99-161:~] esxcli software vib remove -n esx-vmdkops-service
 [LiveInstallationError]
 Error in running rm /tardisks/vmdkops.v00:
 Return code: 1
 Output: rm: can't remove '/tardisks/vmdkops.v00': No such file or directory

 It is not safe to continue. Please reboot the host immediately to discard the unfinished update.
 Please refer to the log file for more details.

Install silently fails;
[root@sc-rdops-vm04-dhcp-99-161:~] esxcli software vib install -v /tmp/docker-volume-vsphere/vmware-esx-vmdkops-1.0.beta.5ad077e.vib -f
Installation Result
   Message: Host is not changed.
   Reboot Required: false
   VIBs Installed:
   VIBs Removed:
   VIBs Skipped: VMWare_bootbank_esx-vmdkops-service_1.0.0-0.0.1

How do I recover without restarting ESX?

Used "--maintenance-mode" remove options and was able to successfully remove vib. Unable to repro issue now. Closing.


Cannot chown files or folders where path contains spaces

The error was:
Failed to change owner: 256
I assume chown is running chown user folder withspace instead of chown user 'folder withspace'

Merged.

Thanks :)


[Feature] Timeline

I'd love to see a graphic that is a timeline for the animation.
Pick the First and Last datetime from the data frame, create a timeline with tick marks at reasonable intervals.
As the animation moves forward, an arrow is displayed over (or under) the timeline to show where in the timeline the animation is currently.

Thank you for opening this. This is a great improvement, and shouldn't be too difficult to implement. You can even let the timeline depend on the supplied dataframe.
The library includes a separate class to create axis. This can be used to create this, the axis also includes a function to calculate the coordinates needed when giving a specific value.


Vertex Weirdness

I'm generating vertices with marching cubes. I split up the terrain into size 32x32x32 chunks of voxels. I'm currently generating 4x4x4 (or 64) chunks. I'm just doing some simple sin maths to generate voxels. Adjusting the f value yields "interesting" results.
let f = 1.0/16.0;
((x as f32 * f).sin() + (y as f32 * f).sin() + (z as f32 * f).sin()) / 3.0

let f = 1.0/8.0;

In the first image, the number of chunks with at least 1 triangle generated is 62. In the second image it's 64.
Here's a list with the number of triangles per chunk for the first image.
527
2868
61
527
2558
4356
1098
2868
4356
627
3891
61
1098
3891
271
527
2558
4356
1098
2558
4120
3348
3190
4356
3348
163
4215
1098
3190
4215
1673
2868
4356
627
3891
4356
3348
163
4215
627
163
473
3891
4215
473
4402
61
1098
3891
271
1098
3190
4215
1673
3891
4215
473
4402
271
1673
4402
553

Here's a list with the number of triangles per chunk in the second image.
3325
5135
5637
3322
5135
4720
4950
5304
5637
4950
5188
5960
3322
5304
5960
3096
5135
4720
4950
5304
4720
3820
3830
5002
4950
3830
3828
5253
5304
5002
5253
5481
5637
4950
5188
5960
4950
3830
3828
5253
5188
3828
3790
5436
5960
5253
5436
6237
3322
5304
5960
3096
5304
5002
5253
5481
5960
5253
5436
6237
3096
5481
6237
2858

Hey, can you add more information, like example code?

I'm pretty convinced the problem was I was adding too many vertices to a mesh (limited to 65_536).


Importing FBX of a non-Unicode encoded path causes Assimp throws error

Godot version: 3.2.1
OS/device including version: Windows 10
Issue description:
I'm using a non-English Windows system whose default encoding is not Unicode. However it turns out Assimp assumes the input path is in UTF8 encoding which causes a error when I importing a FBX of a non-Unicode encoded path.
see /thirdparty/assimp/code/Common/DefaultIOSystem.cpp, line 86, where Utf8ToWide is applied to the input path:
bool DefaultIOSystem::Exists(const char* pFile) const
{
#ifdef _WIN32
    struct __stat64 filestat;
    if (_wstat64(Utf8ToWide(pFile).c_str(), &filestat) != 0) {
        return false;
    }
#else
    FILE* file = ::fopen(pFile, "rb");
    if (!file)
        return false;

    ::fclose(file);
#endif
    return true;
}

One solution was to convert the path to UTF8 encoding before call the Assimp API, where I found String::utf8() is very handy. I just need to change /modules/assimp/editor_scene_importer_assimp.cpp, line 150 to:
String g_path = ProjectSettings::get_singleton()->globalize_path(p_path);
aiScene *scene = (aiScene *)importer.ReadFile(g_path.utf8().ptr(), post_process_Steps);

and the problem is solved.

@akien-mga @RevoluPowered We are in the process of testing a new revision of the fbx importer. I think patching the existing system is worth doing. Will look into it.

@fire are you still working on this?

We are prioritizing getting the list of pr items in FBX like blend shape support and locators.
Not working on this.

We are prioritizing getting the list of pr items in FBX like blend shape support and locators.
Not working on this.

I could send a quick pr with the fix suggested in the original comment.


ConsoleTaskList.java:10: still smelly, it's a...

The puzzle 6-a91f86e8 (from #6) in src/java/main/ConsoleTaskList.java has to be resolved: "still smelly, it's a bridge in disguise: ConsoleTasks only handle ConsoleTask"
https://github.com/paulodamaso/donkey-tasks/blob/945747994e39a21d7e8a03b9bc26d1b22dfa6bfb/src/java/main/ConsoleTaskList.java#L10-L10
The puzzle was created by Paulo Lobo on 15-Dec-17.
Estimate: 0 minutes, role: IMP.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

That's the way it's supposed to work; no smell here.


Exiting mitmdump via ctrl+c can "crash" it

Problem Description
Not sure if this is worth fixing. I noticed that during bootup, if you time it just right (or wrong), you'll get this:
10.3.0
 $ ./mitmdump
^CTraceback (most recent call last):
  File "mitmdump.py", line 2, in <module>
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 419, in exec_module
  File "mitmproxy/tools/main.py", line 15, in <module>
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 419, in exec_module
  File "mitmproxy/master.py", line 5, in <module>
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 419, in exec_module
  File "mitmproxy/addons/__init__.py", line 6, in <module>
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 419, in exec_module
  File "mitmproxy/addons/clientplayback.py", line 22, in <module>
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 419, in exec_module
  File "mitmproxy/options.py", line 4, in <module>
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 419, in exec_module
  File "mitmproxy/optmanager.py", line 17, in <module>
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 419, in exec_module
  File "ruamel/yaml/__init__.py", line 51, in <module>
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 419, in exec_module
  File "ruamel/yaml/cyaml.py", line 4, in <module>
  File "_ruamel_yaml.pyx", line 20, in init ruamel.yaml.clib._ruamel_yaml
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 400, in exec_module
  File "PyInstaller/loader/pyimod02_importers.py", line 187, in get_code
  File "PyInstaller/loader/pyimod01_archive.py", line 149, in extract
KeyboardInterrupt
[10898] Failed to execute script 'mitmdump' due to unhandled exception!

main venv
$ mitmdump
^CTraceback (most recent call last):
  File "/src/mitmproxy/venv/bin/mitmdump", line 5, in <module>
    from mitmproxy.tools.main import mitmdump
  File "/src/mitmproxy/mitmproxy/tools/main.py", line 15, in <module>
    from mitmproxy import master
  File "/src/mitmproxy/mitmproxy/master.py", line 5, in <module>
    from .addons import termlog
  File "/src/mitmproxy/mitmproxy/addons/__init__.py", line 19, in <module>
    from mitmproxy.addons import onboarding
  File "/src/mitmproxy/mitmproxy/addons/onboarding.py", line 3, in <module>
    from mitmproxy.addons.onboardingapp import app
  File "/src/mitmproxy/mitmproxy/addons/onboardingapp/__init__.py", line 3, in <module>
    from flask import Flask
  File "/src/mitmproxy/venv/lib/python3.12/site-packages/flask/__init__.py", line 6, in <module>
    from .app import Flask as Flask
  File "/src/mitmproxy/venv/lib/python3.12/site-packages/flask/app.py", line 20, in <module>
    from werkzeug.routing import BuildError
  File "/src/mitmproxy/venv/lib/python3.12/site-packages/werkzeug/routing/__init__.py", line 123, in <module>
    from .map import Map as Map
  File "/src/mitmproxy/venv/lib/python3.12/site-packages/werkzeug/routing/map.py", line 28, in <module>
    from .matcher import StateMachineMatcher
  File "/src/mitmproxy/venv/lib/python3.12/site-packages/werkzeug/routing/matcher.py", line 12, in <module>
    from .rules import Rule
  File "/src/mitmproxy/venv/lib/python3.12/site-packages/werkzeug/routing/rules.py", line 68, in <module>
    _converter_args_re = re.compile(
                         ^^^^^^^^^^^
  File "/usr/lib/python3.12/re/__init__.py", line 228, in compile
    return _compile(pattern, flags)
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/re/__init__.py", line 307, in _compile
    p = _compiler.compile(pattern, flags)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/re/_compiler.py", line 747, in compile
    code = _code(p, flags)
           ^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/re/_compiler.py", line 580, in _code
    _compile(code, p.data, flags)
  File "/usr/lib/python3.12/re/_compiler.py", line 128, in _compile
    _compile(code, p, _combine_flags(flags, add_flags, del_flags))
  File "/usr/lib/python3.12/re/_compiler.py", line 174, in _compile
    _compile(code, av, flags)
  File "/usr/lib/python3.12/re/_compiler.py", line 111, in _compile
    _compile(code, av[2], flags)
  File "/usr/lib/python3.12/re/_compiler.py", line 86, in _compile
    charset, hascased = _optimize_charset(av, iscased, tolower, fixes)
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/re/_compiler.py", line 325, in _optimize_charset
    out.append((LITERAL, p))
KeyboardInterrupt

Steps to reproduce the behavior:

mitmdump
ctrl + c just right 👌

System Information
$ ./mitmdump --version
Mitmproxy: 10.3.0 binary
Python:    3.12.3
OpenSSL:   OpenSSL 3.2.1 30 Jan 2024
Platform:  Linux-6.5.0-35-generic-x86_64-with-glibc2.38

$ mitmdump --version
Mitmproxy: 11.0.0.dev (+523, commit fa8b899)
Python:    3.12.0
OpenSSL:   OpenSSL 3.2.1 30 Jan 2024
Platform:  Linux-6.5.0-35-generic-x86_64-with-glibc2.38

But this also happens with the venv? Or are we doing some magical things that PyInstaller requires that has an effect even when running without it?

If you can confirm this is an issue exclusive to bootstrapping, then I don't see a reason fix it. I'm probably the only person to ever trigger this :smile:


Move towards using standard kubernetes provider for manifests/crds

Recently I've made a simple test running the QHub deployment on a freshly installed machine. Basically I've set up minikube + kubectl and spun up a minikube cluster. Followed the Local Testing section of the Testing docs and it all worked fine until "Deploy and render the infrastructure" where I got the following error when deploying:
[terraform]: │ Error: Failed to determine GroupVersionResource for manifest
[terraform]: │
[terraform]: │   with module.qhub.module.kubernetes-dask-gateway.kubernetes_manifest.main,
[terraform]: │   on modules/kubernetes/services/dask-gateway/crds.tf line 1, in resource "kubernetes_manifest" "main":
[terraform]: │    1: resource "kubernetes_manifest" "main" {
[terraform]: │
[terraform]: │ cannot select exact GV from REST mapper

I've tried to search about that problem but didn't find much information regarding terraform usage. Fortunatly, I was able to solve that problem removing the docker engine installation and changing it's version from latest to 20.10.8 as well as directly specifying the kubernetes version for the cluster:
minikube start --kubernetes-version=1.19.4 --driver=docker --cpus 4 --memory 4096
There might be some changes within the latest versions of the necessary components for the local deployment, so I would suggest a review in our docs.

Tried again, changing only the kubernetes version by including the flag --kubernetes-version=v1.19.4 in the minikube start command, as my Docker engine was already at version 20.10.8 (that seems like it's the current latest stable?).
It did require that I spin down and delete my existing minikube cluster, as it was running kubernetes 1.22. As such, I had to back up and repeat some of the previous steps in the Local Testing instructions to initialize the minikube cluster. As mentioned, the only step that changed was including an explicit kubernetes version in the minikube startup command.
The rest of the steps proceeded smoothly and without problems exactly as indicated.
Of course, when attempting to navigate to github-actions.qhub.dev/hub/login in the browser, Chrome nuked the connection due to an overdeveloped cert management system, but that's for a different ticket. Closing.

Going to reopen this issue seem related to https://github.com/hashicorp/terraform-provider-kubernetes-alpha/issues/235. Looks like terraform now supports a kubernetes_manifest and we should now start to use the kubernetes provider for all this work. https://www.hashicorp.com/blog/beta-support-for-crds-in-the-terraform-provider-for-kubernetes
https://github.com/hashicorp/terraform-provider-kubernetes

Done in #1003. Closing issue.


WIP [cC]onfig => [cC]onfiguration

[x] Replace config to configuration. (Let me know if I'm going too far.😅)
[x] Add deprecation message for old value usage.

Related: https://github.com/yonaskolb/XcodeGen/issues/59

Oh wow, you literally renamed every single instance of config in the project :)
I must say, I think I prefer config in code. It's just much easier to read and type.
At the moment in terms of user facing properties in the spec we have following:

project.configs
project.configFiles (not added yet)
target.configs
target.configFiles
settings.configs (in project, target and setting groups)

These should be the only things that change (as well as the documentation for them).
In addition, I'm actually still on the fence about this change. Does anyone else here have any opinions?

IMO config is very troublesome. New user might think configFiles is related to config.  Xcode or xcodebuild does not use config for shorthand name for Configuration. I think it's more user friendly to avoid potential confusion with xcconfig. Some user even don't know what Configuration is either, so we should be very very kind as possible.

Usage: xcodebuild [-project <projectname>] [[-target <targetname>]...|-alltargets] [-configuration <configurationname>] [-arch <architecture>]... [-sdk [<sdkname>|<sdkpath>]] [-showBuildSettings] [<buildsetting>=<value>]... [<buildaction>]...

I'm not planning to work on this one for now. (I got used to the code-base and I feel good.👌)


feat : quotes.py

Closes: Quote of the Day #71

Title : Quote Of The Day
Name: Akarsh Ghildyal
Idenitfy yourself: SSOC Contributor

Describe the add-ons or changes you've made 📃
Added a Quote of the day program under API Programs.
Give a clear description of what have you added or modifications made
Using zenquotes api, this fetches the quote of the day and displays it.
Type of change ☑

What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙

Ran the code locally.
Checklist: ☑

[x] My code follows the Contributing Guidelines & Code of Conduct of this project.
[x] This PR does not contain plagiarized content.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly wherever it was hard to understand.
[x] My changes generate no new warnings.

Screenshots 📷

https://github.com/Avdhesh-Varshney/Jarvis/assets/129004417/e75f6549-ca73-462e-9668-8233c035cd92
@Avdhesh-Varshney uses zenquotes api to fetch for the quote of the day


Archive websites and provide a fallback

Adds two jobs that (a) archive hackathon websites on web.archive.org and (b) tags hackathons with broken websites as "Website down". Additionally, if a hackathon is tagged with "Website down", the API will return the archive URL.
In theory this works... let's see. I'm still wrapping my head around Ruby & Rails. I'm skeptical about how simple the web archive job is.

mind if I add a couple commits to get the tests to pass?

go for it!

sweet it fixed it!

Also, tags are capitalized as titles so "Website Down" would be best for consistency.

@northeastprince: I think I managed to catch everything you requested! let me know if I missed something

I'll take a look at this today!

quick summary of the big changes:

I added a couple of methods to the Hackathon model including website_down?
In another PR, we moved from interfacing with Sidekiq to Active Job. Sorry about that!! I converted the jobs over to Active Job.
Archive website when submitted

Awesome! LGTM?

oh wait one sec! (might have found a bug)

Oh I just came across another potential bug. We already have a handful of downed websites, since these can't be archived after they're down, web archive will return a 404.
In these cases, I think it would be best to return the original website

@sampoder this is good to go if you wanna take a look at it

After merging, I'll run the following in production to achieve all the existing hackathon websites
rake to_stdout hackathons:archive_all_websites

let's merge!

324 websites were archived!

That’s awesome!
On Sun, 20 Aug 2023 at 6:15 AM, Gary Tou @.***> wrote:

324 websites were archived!
—
Reply to this email directly, view it on GitHub
https://github.com/hackclub/hackathons-backend/pull/114#issuecomment-1685283175,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJP3VRFZRWSSM72POYOQQILXWIEWFANCNFSM6AAAAAA3QNWKEM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

--
From Sam


[SoftDeleteable] Deleted entities showing up on eager fetches

Using v2.4.7 and Symfony 2.7.7 (also tested on Symfony 2.7.6), if I use fetch="EAGER" on one-to-many relations the "deleted" entities appear in the results.
E.g.
<one-to-many field="items" target-entity="BasketItem" mapped-by="basket" fetch="EAGER">       <cascade>         <cascade-all/>       </cascade>     </one-to-many>
Is this desired behaviour? Could I be doing something wrong?
orm:
        auto_generate_proxy_classes: "%kernel.debug%"
        default_entity_manager: default
        entity_managers:
            default:
                connection:    default
                mappings:
                    AppBundle: ~
                result_cache_driver:
                    type: redis
                    host: "%redis_host%"
                    instance_class: Redis
                filters:
                    softdeleteable:
                        class: Gedmo\SoftDeleteable\Filter\SoftDeleteableFilter
                        enabled: true

it might be that eager fetching does not hit filter, so far I do not know how you could by pass it.. but in the end it is better to handle softdeletion manually. managing softdeleted relations is though task.

Found someone else with the same problem it seems: http://sofd.developer-works.com/article/21370516/Doctrine+2+filtering+with+EAGER+relations

How EAGER mapping is handled has changed with doctrine 2.5. It now uses a join instead of two separate queries. The filter does not get added to the join.

I see, if that is possible, maybe there is a way to hook it in sql filter, if that issue is a concern, you could have a look

Tried to find a workaround that problem.
Added posibility to suply eager relations like that:
/**
 * @ORM\Table()
 * @ORM\Entity
 * @Gedmo\SoftDeleteable(fieldName="deletedAt", timeAware=false, eager={"template_view"})
 */

in filter listener
public function addFilterConstraint(ClassMetadata $targetEntity, $targetTableAlias)

added a code to handle configuration param
if(isset($config['eager']) && $config['eager'])
{
	$mainCondition = "({$addCondSql})";
	$softColumn = $targetEntity->getQuotedColumnName($config['fieldName'], $platform);
	$platform = $conn->getDatabasePlatform();

	foreach($config['eager'] as $eagerRelation)
	{
		$softCondition = $platform->getIsNullExpression($eagerRelation.'.'.$softColumn);
		$addCondSql = "{$mainCondition} AND ({$softCondition})";
	}
}

That would generate query:
SELECT ... FROM template t0 LEFT JOIN template_view t5 ON t5.template_id = t0.id
WHERE (((t0.deleted_at IS NULL)
AND (template_view.deleted_at IS NULL)))':

The problem is that there is no way to find out what is the alias for template_view table

I see @Wobbly-Wibbly in general softdeleteable does not fit its concept well with orm, most of the users fallback to managing it manually, same for myself in such cases sadly. Keeping it open since there is no obvious solution so far

Nowadays, Soft deletes are an anti-pattern but many legacy projects use it.
However, it might be worth mentioning this compatibility problem on the SoftDeleteable docs.

While there are better ways to solve the problems (logging, auditing, archiving) that soft delete is intended to solve. It is an easy option to use when time or resources are not available to implement a better option. Its also very easily understood.
Of course it can other problems later, depending on the application, so its a tradeoff.

Softdelete is definitely not an anti-pattern. Some computer science people may think so, but real word use cases, and RAD requires the softdeleteable pattern.
I wrote a failing test for this issue: https://github.com/Atlantic18/DoctrineExtensions/pull/1550
Unfortunately, It should be fixed by doctrine.

thanks for the input, will need to mention this in documentation.

Did somebody create an issue in Doctrine about that issue?


Add type parameter m to ReadCtx

Description
Just a small refactoring. ReadCtx previously hard-coded IO despite being used in other parametrised records.

https://github.com/IntersectMBO/lsm-tree/actions/runs/10738853253/job/29783295632#step:16:738 is passing even though it is expected to fail. How come?

https://github.com/IntersectMBO/lsm-tree/actions/runs/10738853253/job/29783295632#step:16:738 is passing even though it is expected to fail. How come?

In all 100 test runs didn't have any blobs in the write buffer when creating a cursor? That seems quite unlikely, but the write buffer in this test only has up to 3 entries. I can't imagine any other reason.


"No symbols found" in file view when open project with Project Manager

Recently, I found sometimes there are no symbols found when I press cmd + r. After searching and trying to fix the problem, I found a possibly answer. Opening a project from command line is the only workaround I could find so far.

Do you get the same result opening a project through the menu or is it just when using the project manager?

I’m having the same issue; it seems unrelated to the Project Manager package. Symbols fail to load when Atom is launched from Finder (as opposed to from the Terminal prompt), regardless of whether Project Manager is installed or not.


feat: WDS typography improvements

Short to medium-term improvement: remove font selection options.
See #34582
Only the option we call System Default stays.
This unlocks performance optimization and subsequent refinement work for design:

Refinement of weights and sizes
Matching of icon weights to fonts

Please add your planning poker estimate with Zenhub @KelvinOm


Still crashing on canCaveSpiderSpawn after updating to 1.4.1

Basically the same issue as described in https://github.com/sf-inc/sihywtcamd/issues/7, but on the mod version 1.4.1.
MC 1.17.1
Fabric API 0.37.1
sihywtcamd 1.4.1
Crash log: crash-2021-07-28_01.42.49-server.txt

Hmm okay, I'll fix that. I guess it crashes less often than before but still crash if the position is equal to 0... But I don't know how do you have the issue because when I create a world with the datapack, it never tries to spawn them because the mojang dapatack overrides every mob spawn. Do you do anything more?

Nope, I just log into my existing world and after a very short period of time (varies from 1 to 10min) I get a crash. Didn't try with new worlds as I'm playing on a server. Could it be because I don't use the official datapack, but rather this one?

This datapack seems to act the same way. Let me give it some tries, I'll check this

Okay so I've been reading your crashlog and I just saw mcmobs-dp-1-17-1.zip datapack. Could you tell me what is it or link it?

Yep, it's this one. It adds unreleased mobs, such as warden and glutton, but they never spawn below 0.

Thanks. Can't find out how you got evrything working in your world 😅 I'll release a version with spawn (normally) fixed in this issue so that you can test it for me if it doesn't bother you.

Yeah, sure, I will test it.

sihywtcamd-betatest.zip
This should work

Tested it for about 30min, just walking around, flying in creative and flying in spectator, especially around areas that previously caused a crash. So far seems to be fixed, I will let you know if the crash happens again. And I'll close the issue tomorrow if I still don't encounter a crash by then, unless you would like to close it now.
Thanks a lot!

Okay perfect, I will publish the fix on Curseforge soon then. You can still play with this version if you want, but I may add a change on the version I will release this afternoon. Have a nice day! =)


In calculate_extensions, check as runtime for calling Python 3.5 subprocess.run

The calculate_extensions code had a commented-out section for using subprocess.run in Python 3.5. This adds a static class variable subprocess_has_run which checks for this method at runtime, and to use it if available. This way, a user running Python 3.5 doesn't have to edit the code. (Otherwise, Python 3.5 gives lots of warnings for unclosed files if using subprocess.Popen.)
Also added the necessary stderr=subprocess.PIPE in the call to subprocess.run.

(Sorry for the extra commit "Merge branch 'change-assertEquals-to-assertEqual'" which appeared when I merged into myabcdefhiostw fork. If you merge the pull request, this won't affect anything. But I can re-submit and remove it if you want.)

Nice, thanks!

Thanks for the quick response on my pull requests. Now my installation runs! I'll send a pull request for an INSTALL file so others can follow the steps.


Add support for 3-tier database configurations on development

Hi! I noticed parity doesn't get along with multi-database configurations, so I was thinking it could be nice to support this. This patch shows a rough idea of the needed changes, but I wanted to make sure I'm heading in the right direction. Any comments would be greatly appreciated.

Hi @horacio I am quite sorry for missing this PR, it came up while I was very short on time. I think this looks like it's workable. If you re-open it I can approve this week.

@geoffharcourt don't worry, and thanks for re-opening it! I'd be glad to work on this again. I'll continue with the specs for this addition, if that's OK.

@horacio sounds great. Tag me when you're ready with specs and I'll review again. Thanks for your time and effort.

Is there a new version release with this change included?

@megatux v3.5.0 is released!


[INS-1786] WebSocket headers tab

Motivation: Send custom headers
Highlights

adds header tab to request pane
extends RequestHeaderEditor component to support WebSocketRequest and readOnly headers
adds outgoing header logging to ws smoke test server

Todo

[x] add headers tab
[x] init model with some defaults
[x] get everything rendering consistently
[x] wire up bulk editor toggle
[x] fix tab/button/bar css alignment
[x] make default header values accurate
[x] disable editing while connected

Possible future work

nunjucks rendering, not done in message tab yet, better tackled at the same time
prettify event log view / payload
add upgrade event, response status code/message, headers and handle set-cookies
add outgoing headers to request object?

Notes:
Since the websocket client doesn't expose the request headers which are actually sent to the server.
We can either construct it or perhaps redirect it through a local server to capture, but for now we show the headers as read only in the headers tab.

@filfreire Thanks!

Do we need to support empty value headers (e.g. only key is defined)
we do not currently allow that in HTTP requests, see parse-header-strings.ts#L89
Do we need to support some sort of header "array"
out of scope, feel free to create an investigation ticket
switching back and forth from message and headers tab - loses the contents of the message tab
out of scope, in order to avoid migrating the data model, message persistence is subject to change
create a new WS request reuses URL
not introduced by this PR but fixed in this PR
moving websocket requests in the tree view doesn't work properly
not introduced by this PR, feel free to create a ticket.


Update Exchange-Teams-interact.md

Correction to Sup # 5 note. The number was incorrectly set to 4. Sorry.

fixing this in a different PR, there are other issues to resolve here.


Fix tests mocking tracefs accesses.

If /sys/kernel/debug/tracing directory does not exist, then bpftrace assumes tracefs is available under /sys/kernel/tracing. This breaks mocks and results in test failures when /sys/kernel/debug/tracing does not exist. This pull request replaces tracefs paths in tests with tracefs::* function calls to use the same tracefs detection logic as bpftrace.
This fixes following tests:
[  FAILED  ] 17 tests, listed below:
[  FAILED  ] bpftrace.add_probes_wildcard
[  FAILED  ] bpftrace.add_probes_wildcard_no_matches
[  FAILED  ] bpftrace.add_probes_kernel_module_wildcard
[  FAILED  ] bpftrace.add_probes_tracepoint_wildcard
[  FAILED  ] bpftrace.add_probes_tracepoint_category_wildcard
[  FAILED  ] bpftrace.add_probes_tracepoint_wildcard_no_matches
[  FAILED  ] semantic_analyser.call_ntop
[  FAILED  ] semantic_analyser.builtin_args
[  FAILED  ] semantic_analyser.tracepoint_common_field
[  FAILED  ] portability_analyser.tracepoint_field_access
[  FAILED  ] codegen.args_multiple_tracepoints
[  FAILED  ] codegen.args_multiple_tracepoints_category_wild
[  FAILED  ] codegen.args_multiple_tracepoints_wild
[  FAILED  ] codegen.builtin_probe
[  FAILED  ] codegen.builtin_probe_wild
[  FAILED  ] codegen.map_key_probe
[  FAILED  ] codegen.strncmp
Checklist

[ ] Language changes are updated in man/adoc/bpftrace.adoc and if needed in docs/reference_guide.md
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

@BurntBrunch: this should fix the problems you reported in https://github.com/iovisor/bpftrace/issues/2412

Ah, yes, this seems right. I can reproduce the test failures with tests/bpftrace_test --gtest_filter='codegen.*' but not with sudo tests/bpftrace_test --gtest_filter='codegen.*' and my /sys/kernel/debug is not world-readable.


setup ci

Starting with go test and will add compiling and releasing executables

was secretly hoping this would run my pipeline as I made changes to this PR.. so may need to play with this some to see if I can't trigger this locally


Error installing on Ubuntu

Describe the bug
After adding the repository, key, apt-get update and install, I got an error.
To Reproduce
Steps (or project) to reproduce the behavior:
sudo apt-key adv --keyserver hkp://pool.sks-keyservers.net --recv-keys 023EDB0B
echo deb https://dl.bintray.com/gauge/gauge-deb stable main | sudo tee -a /etc/apt/sources.list
sudo apt-get update
sudo apt-get install gauge

Logs
#8 1.384 Err:4 https://dl.bintray.com/gauge/gauge-deb stable InRelease
#8 1.384   403  Forbidden [IP: <IP_ADDRESS> 443]

Thanks for reporting this.
The debian repo is no longer support as the host bintray dropped support.
New instructions for installing are available here
https://docs.gauge.org/getting_started/installing-gauge.html?os=linux&language=javascript&ide=vscode


[plot] resetZoom with data margins and keep aspect ratio on

This PR fixes an issue with Plot.resetZoom method with data margins and keep aspect ratio on: Closes #452

LGTM


Perl installing to wrong platform

I am trying to build openSSL vs 3.0.0. I have installed Strawberry Perl and NASM. However, when I go to run the command to build openSSL "perl Configure VC-WIN64A" I get the following error:
Configuring OpenSSL version 3.0.0-alpha7-dev for target VC-WIN64A
Using os-specific seed configuration
Failure!  makefile wasn't produced.
Please read INSTALL.md and associated NOTES-* files.  You may also have to
look over your available compiler tool chain or change your configuration.

This perl implementation doesn't produce Windows like paths (with backward
slash directory separators).  Please use an implementation that matches your
building platform.
This Perl version: 5.10.1 for i686-cygwin-thread-multi-64int

I am not sure why my Perl is build for i686-cygwin-thread-multi-64int and not what is needed to compile in Visual Studio.
I am using the x64 Native Tools Command Prompt from Visual Studio 17. How can I change which platform Perl is built for?

Probably the cyygwin perl is on your %PATH% before the strawberry perl (or the strawberry perl is not on your %PATH% at all). Ad just your %PATH% environment variable appropriate to  ensure you get the right one.

My %PATH% = C:\Program Files\NASM;C:\Strawberry\c\bin;C:\Strawberry\perl\site\bin;C:\Strawberry\perl\bin
Should those folders from Strawberry be rearrange some how?

@mattcaswell - you were correct, I had a CYGWIN folder listed before the Strawberry folders in my %PATH% definition. I didn't think to look at that, since it has been years since I installed anything related to CYGWIN.

It sounds like this has been answered, so I'll close it -- please reopen if I am misunderstanding.


I notice that you almost completely deleted everything for 2.0

Do you plan on finishing? I got part way through porting this to sponge before realizing that it was ported previously and had been deleted.

2.0 only contains the API, but I can't finish the plugin because I'm not sure what to do for the /slack send command.

What is the slack send command supposed to do / work? I might take a crack at it from the older commits unless theres code in your local repo.

https://github.com/CircuitSoftGroup/SlackMC/blob/92fa060c0209985a77563dbd709945134707950b/src/main/java/org/circuitsoft/slack/bukkit/SlackBukkit.java#L106
It is supposed to do everything that the new SlackMC API allows you to do: usernames, messages, attachments, user icons, etc


Fix/ga/code smells

Issue
No issue number
Description
Clean up
Checklist for Pull Requests

[ ] Supplied as many details as possible on this change
[ ] Included the link to the associated issue in the Issue section above
[ ] The code conforms to the general development guidelines.
[ ] The code has unit tests where applicable and is easily unit-testable
[ ] This branch is appropriately named for the associated issue

Thanks!


maintainers: add BarinovMaxim

Add myself to maintainer list.
I added myself to maintainer list in that commit https://github.com/NixOS/nixpkgs/pull/161221 where I wanted to add a new package. But this package won't be merged for a long time but I want to add another one package. I think it's not best decision to add myself in maintainer list in another commit

ping @SuperSandro2000


Document and test the documentation to move apps from Sandbox to Tools

Describe the Issue
As a user with apps in Sandbox that need to be moved to Tools, I want a document that will help to ensure my current apps in Sandbox continue working in Tools
Note: Carles currently has one test account in Assembly that have a sample app deployed using the sandbox OU. I would expect that after moving from Sandbox to Tools these apps continue working.
Additional Context
Properly speaking, the apps will not be "moved" but rebuilt in the new environment (tools). What means not only to install the app themselves but also to move any S3 bucket, dynamoDB or whatever other artifacts associated to the app.
Acceptance Criteria

A document that will explain step by step how to rebuild an application in tools.
Details how to rebuild DynamoDB, S3 buckets
Verify/Go through the document to ensure it works
After following the steps in the documents, the app works as expected from the tools account

What will happen is that a new environment (tools) will be created and it will be the responsibility of each team to move (redeploy) their Sandbox apps in Tools.
The test plan already has verified the sample app that were deployed in Sandbox can also be deployed in Tools with no problem. In this context, there is nothing else to test or document. This issue can be closed.


Perf regression: emit time

Several of the perf tests show an emit time regression since 4.2, but I focused on angular, since it seemed the clearest.

Regressions are at 68b0323b72bf51236d8cd181ed94ba9a63f933fd (aka #42676) and 7751ecb544a4e318a4f63fe1e2fb13329d7b5de4 (aka #35877).  Cumulative effect is 10-15%.

#42676 didn't seem to have any major regressions when I tested it, although it does have some impact (namely a full walk of the tree prior to emit), but there's really no other easy way to fix substitutions requiring parenthesization (other than not doing it and just adding excess parentheses around everything added via substitution).
#35877 is an ECMAScript conformance change that affects imports and necessitates adding new nodes to the tree for every call to an imported function. I'm surprised the impact there is so heavy, but unfortunately its another case where I'm not certain what we could do differently other than make the transformation optional via a flag. We opted not to do the flag, but also not to do the transform for namespaces and only external modules.
You mentioned these are the times from Angular. What version of NodeJS is this from?

@rbuckton I believe that's the node 14 chart, but I saw changes across versions.

@rbuckton it looks like #42676 is not strictly necessary - it could be backed out and while your code won't always be canonical, it will use a conservative but cheap check that can win back half of the regression. I would vote that we revert unless we can avoid the tree walk.
#35877 - well, that seems more unclear to me.

@rbuckton it looks like #42676 is not strictly necessary - it could be backed out and while the emitted code won't always be canonical, it will use a conservative but cheap check that can win back half of the regression. I would vote that we revert unless we can avoid the tree walk.

I'd like to investigate that one further as there were some other changes that were part of that change (such as the reusable trampoline for binary expressions) that might be difficult to untangle. I considered other mechanisms for handling the parenthesization cleanly, but they were all much more complicated implementation wise. The main reason I introduced this change was to prevent us from running into further one-off parenthesization-related issues. I may have some other ideas that could work, such as "pushing" a parenthesization rule prior to emitting a child node, and having substitution check for this rule on the way out. The downside being the need to ensure both the node factory and the printer remain in sync.

#35877 - well, that seems more unclear to me.

That PR fixes a longstanding issue and has support from members of TC39, rolling it back would be unfortunate. When we discussed it as a team in a design meeting, one of the important points was that anyone using --module esnext and a bundler like Webpack+Babel was already getting this output, so we were matching the ecosystem. I might also want to deep-dive into this PR to determine if there's a way to reduce the performance cost.
Perhaps it might be feasible to mark the PropertyAccessExpression resulting from substitution of an imported Identifier with an EmitFlag, and have the printer wrap the identifier in a (void 0, ...). Another possible solution would be to create a single, reusable void 0 expression to reduce allocations.

@rbuckton I will opt for us to back out the changes by tomorrow since we haven't gotten a flag in yet.

For some reason the performance dashboard isn't showing perf numbers for recent CI builds. I'm looking into that. The fix in #43652 should have significantly improved both regressions (since one was exacerbated by the other), though I'm still looking for further optimizations.


YETUS-955. Update binaries in Dockerfile

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/1/console in case of problems.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
19m 28s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m  9s
master passed

+1 :green_heart:
javadoc
0m 40s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  6s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 10s
the patch passed

+1 :green_heart:
hadolint
0m  5s
The patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)

+1 :green_heart:
shellcheck
0m  0s
There were no new shellcheck issues.

+1 :green_heart:
shelldocs
0m  3s
There were no new shelldocs issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
yamllint
0m  3s
There were no new yamllint issues.

+1 :green_heart:
javadoc
0m 40s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 10s
precommit in the patch passed.

+1 :green_heart:
unit
0m 58s
root in the patch passed.

+1 :green_heart:
asflicense
0m 22s
The patch does not generate ASF License warnings.

27m 47s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/yetus/pull/98

Optional Tests
dupname asflicense yamllint shellcheck shelldocs javac javadoc unit hadolint

uname
Linux dfaad7874657 4.15.0-76-generic #86-Ubuntu SMP Fri Jan 17 17:24:28 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
precommit/src/main/shell/personality/yetus.sh

git revision
master / 8b44644

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

Test Results
https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/1/testReport/

Max. process+thread count
100 (vs. ulimit of 2000)

modules
C: precommit . U: .

Console output
https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.7.0 hadolint=1.17.5-0-g443423c yamllint=1.20.0

Powered by
Apache Yetus 0.12.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/2/console in case of problems.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
4m 57s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 16s
master passed

+1 :green_heart:
javadoc
0m 42s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  6s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 13s
the patch passed

+1 :green_heart:
hadolint
0m  5s
The patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)

+1 :green_heart:
shellcheck
0m  0s
There were no new shellcheck issues.

+1 :green_heart:
shelldocs
0m  2s
There were no new shelldocs issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
yamllint
0m  3s
There were no new yamllint issues.

+1 :green_heart:
javadoc
0m 45s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 11s
precommit in the patch passed.

+1 :green_heart:
unit
1m  0s
root in the patch passed.

+1 :green_heart:
asflicense
0m 24s
The patch does not generate ASF License warnings.

13m 38s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/yetus/pull/98

Optional Tests
dupname asflicense yamllint shellcheck shelldocs javac javadoc unit hadolint

uname
Linux cd1e97f84207 4.15.0-76-generic #86-Ubuntu SMP Fri Jan 17 17:24:28 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
precommit/src/main/shell/personality/yetus.sh

git revision
master / 8b44644

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

Test Results
https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/2/testReport/

Max. process+thread count
101 (vs. ulimit of 2000)

modules
C: precommit . U: .

Console output
https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/2/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.7.0 hadolint=1.17.5-0-g443423c yamllint=1.20.0

Powered by
Apache Yetus 0.12.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/3/console in case of problems.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
10m 58s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+0 :ok:
mvndep
0m 11s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
1m 21s
master passed

+1 :green_heart:
javadoc
0m 37s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  5s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 26s
the patch passed

+1 :green_heart:
hadolint
0m  4s
The patch generated 0 new + 0 unchanged - 2 fixed = 0 total (was 2)

+1 :green_heart:
markdownlint
0m  1s
The patch generated 0 new + 0 unchanged - 8 fixed = 0 total (was 8)

+1 :green_heart:
rubocop
0m  4s
The patch generated 0 new + 0 unchanged - 15 fixed = 0 total (was 15)

+1 :green_heart:
shellcheck
0m  4s
There were no new shellcheck issues.

+1 :green_heart:
shelldocs
0m  2s
There were no new shelldocs issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
yamllint
0m  2s
There were no new yamllint issues.

+1 :green_heart:
javadoc
0m 40s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m  8s
precommit in the patch passed.

+1 :green_heart:
unit
0m  6s
asf-site-src in the patch passed.

+1 :green_heart:
unit
0m 39s
root in the patch passed.

+1 :green_heart:
asflicense
0m 25s
The patch does not generate ASF License warnings.

17m 28s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/yetus/pull/98

Optional Tests
dupname asflicense rubocop hadolint shellcheck shelldocs markdownlint yamllint javac javadoc unit

uname
Linux d500f5e0fd4a 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
precommit/src/main/shell/personality/yetus.sh

git revision
master / 8b44644

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

Test Results
https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/3/testReport/

Max. process+thread count
100 (vs. ulimit of 2000)

modules
C: precommit asf-site-src . U: .

Console output
https://builds.apache.org/job/yetus-github-multibranch/job/PR-98/3/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.7.0 hadolint=1.17.5-0-g443423c markdownlint=0.22.0 rubocop=0.80.1 yamllint=1.20.0

Powered by
Apache Yetus 0.12.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


localhost serve doesn't work (on Windows?)

Hi there,
first of all - great work! I'm new to web programming and Hug eases stuff a lot!
I am having a problem running simple dev server on Windows.
I am simply running unmodified the code from:
https://github.com/timothycrosley/hug/blob/develop/examples/quick_server.py
and I get following error:

...
File "C:\Python34\WinPython-64bit-<IP_ADDRESS>Qt5\python-3.4.4.amd64\lib\socketserver.py", line 444, in server_bind
self.socket.bind(self.server_address)
OSError: [WinError 10013] An attempt was made to access a socket in a way forbidden by its access permissions

Any thoughts on what may cause it?

I think you'll have to open that port in your firewall for python.
On Thu, Mar 23, 2017 at 11:03 AM, simplynail<EMAIL_ADDRESS>wrote:

Hi there,
first of all - great work! I'm new to web programming and Hug eases stuff
a lot!
I am having a problem running simple dev server on Windows.
I am simply running unmodified the code from:
https://github.com/timothycrosley/hug/blob/develop/examples/quick_server.
py
and I get following error:
...
File "C:\Python34\WinPython-64bit-<IP_ADDRESS>Qt5\python-3.4.4.amd64\lib\socketserver.py",
line 444, in server_bind
self.socket.bind(self.server_address)
OSError: [WinError 10013] An attempt was made to access a socket in a way
forbidden by its access permissions
Any thoughts on what may cause it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/timothycrosley/hug/issues/470, or mute the thread
https://github.com/notifications/unsubscribe-auth/AASNz2Ae59auLBoYHsoiJaP-AID1fA8_ks5rope4gaJpZM4Mm-bT
.

Fair enough, I didn't get any prompt from firewall that Python requests access to new port so didn't catch that. It works now, thanks!
BTW
is there some simple way to bind dev server to some other port than default 8000?


Contrast and Brightness as Visual-attribute

While annotating the Healthy Brain Network project's Surround Suppression task, I found that Opacity and Lunminence,  Hue and Saturation, but I can't find Contrast and Brightness. While these values are potentially dependent, having the full range of definitions helps describe the events.

Upon further investigation, it seems that Luminance is the correct word to use instead of brightness:
Here is the definition of Luminance and Brightness, as described in a blog post by x.rite, the color calibration tool from Pantone
Luminance: Luminance is the luminous intensity projected on a given area and direction. Luminance typically describes the intensity of emitted light.
Brightness: We perceive brightness when lumens fall on the rods and cones of our retina. When we speak of brightness, we use subjective words like “dim” or “brilliant” because brightness cannot be measured like luminance but can be scaled in percentages.
Here, I am not sure if we want to differentiate the difference between the two. My current take is to keep the luminance and not to include Brightness in the schema.

Here is the definition of Contrast:
Contrast is the difference between the luminance of two objects. Either of the following equations can formalize the definition  of  contrast:

Micbelson contrast = (Lmax - Lmin) / (Lmax + Lmin)
where  Lmax and  Lmin are the maximum and minimum luminance of an image.

Weber contrast = (L - Lback) / Lback
where L  is  the  luminance  of  the  object  and Lback  is  the  luminance  of  the  background

Source:

Legge, G. E., D. H. Parish, A. Luebker, and L. H. Wurm. 1990. “Psychophysics of Reading. XI. Comparing Color Contrast and Luminance Contrast.” Journal of the Optical Society of America. A, Optics and Image Science 7 (10): 2002–10.
Rosa, Claudio, and Carlo Aleci. 2022. “Psychophysics in the Ophthalmological Practice—II. Contrast Sensitivity.” Annals of Eye Science 7 (1): 35–35.

After HWG discussion --- it was agreed we should put in Contrast but not Brightness in the next release.  What should the parent node be?
Also, formula 1 is a relative difference ... should that be in the description?  The definition also indicates two objects.  Should this be a Relation such as "Contrast-between".
Do the two references both give both of these definitions?

Contrast should be a Visual-attribute (not a Relation).
Some examples:

An image of e.g. a grating can have a certain contrast.
One image can have multiple contrasts, as contrast can be calculated in different ways (like those noted above, or in other ways) for a certain spatial resolution at different locations.

Thanks a lot, @VisLab and @dorahermes, for your support.

What should the parent node be?

I agree with @dorahermes that it should be under Visual-attribute.

Formula 1 is a relative difference

Both formulas talk about differences in luminance. Contrast needs two objects to be defined. However, I think that Contrast-between would cause unnecessary repetition. For example, for a foreground gray circle and a white background, the short HED tag will look like:
(Visual-presentation, ((Foreground-view, (Circle, Gray)), (Background-view, White),  Contrast/0.5))

Do the two references both give both of these definitions?
No, the Weber contrast is listed only in the second source. However, the first source is well-cited and clearly describes the effect of contrast on the subject performance. The second source covers how the contrast can be manipulated for psychophysical experiments.

Talking to @smakeig, he emphasized that there are many contrasts, and what we are referring to here is luminance contrast.  The first source also emphasizes this point by comparing the attributes of color contrast and luminance contrast.
So, I wonder if it would be better to have Luminance-contrast under Visual-attributes to clarify the tag.

I agree that Luminance-contrast is clear and contrast-between does not work. Note that contrast can also be calculated using sets of filters in visual encoding models.

I wonder, though, whether luminance-contrast, as we are seeking to use in
the Present movie annotation project, is not better thought of as a measure
applied to events rather than a response attribute per se...   Here, we are
using it as defined by a particular formula (not specified in the
HED schema, of course, as it has near-arbitrary parameters).  Further,
these parameters and the formula they belong to are not accepted
standards.  Compare, for example, the use of dB to specify the volume of a
sound - here, several dB scales are recognized (Db A, etc.).  Of course,
the same '?' might be raised about many other event attributes.  For
example, "color, 'very-dark-red'" does have an agreed formal definition, as
does 'latency, 21.344 sec'.  It is doubtless of use to enter e.g. computed
luminance contrast values in an events.tsv file column -- and HED tools
should be able to incorporate this into the HED string for (here) each HED
"movie-shot, Onset" event marker - to allow HED search to pick out events
with e.g." luminance-contrast > 3".
Perhaps HED should have a list of recommended measure column name labels
that are not 'at the level of' standard HED schema terms...
Else -  perhaps we should introduce a MovingImage library schema that
defines not only 'luminance-contrast' but also parameters entering into its
definition:
e.g.
"luminance-ratio, 2.1777;
defined by  npreceding-frames, 3; nfollowing-frames, 3;
luminance--measure, RMS; luminance-smoothing, mean"
The above would seem like 'overkill' ... as meaningful variations in this
definition are not likely to be represented in a data archive, certainly
not on the scale that would make their differences across datasets
meaningfully testable...
?
On Fri, Feb 9, 2024 at 10:26 AM dorahermes @.***> wrote:

I agree that Luminance-contrast is clear and contrast-between does not
work. Note that contrast can also be calculated using sets of filters in
visual encoding models.
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/hed-standard/hed-schemas/issues/135*issuecomment-1936409084__;Iw!!Mih3wA!Ec1YQQ5cf5XXF2uzsYFG2CisA2ZlMluN_ooE2Dq1gjzSnsvsrRsG7wqz_lBcVVxzoWEMm3p6l5ULLwTfvyISxOyq$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AKN2SFQ3OYRRSFPMB4YPAELYSZS6TAVCNFSM6AAAAABCJN6A46VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZWGQYDSMBYGQ__;!!Mih3wA!Ec1YQQ5cf5XXF2uzsYFG2CisA2ZlMluN_ooE2Dq1gjzSnsvsrRsG7wqz_lBcVVxzoWEMm3p6l5ULLwTfvxXzSzO3$
.
You are receiving this because you were mentioned.Message ID:
@.***>

I meant that comment to begin, "I wonder, though, whether
luminance-contrast, as we are seeking to use in the Present movie
annotation project, is not better thought of as a measure applied to events
rather than AN EVENT attribute per se..."
On Sun, Feb 11, 2024 at 1:34 PM Scott Makeig @.***> wrote:

I wonder, though, whether luminance-contrast, as we are seeking to use in
the Present movie annotation project, is not better thought of as a measure
applied to events rather than a response attribute per se...   Here, we are
using it as defined by a particular formula (not specified in the
HED schema, of course, as it has near-arbitrary parameters).  Further,
these parameters and the formula they belong to are not accepted
standards.  Compare, for example, the use of dB to specify the volume of a
sound - here, several dB scales are recognized (Db A, etc.).  Of course,
the same '?' might be raised about many other event attributes.  For
example, "color, 'very-dark-red'" does have an agreed formal definition, as
does 'latency, 21.344 sec'.  It is doubtless of use to enter e.g. computed
luminance contrast values in an events.tsv file column -- and HED tools
should be able to incorporate this into the HED string for (here) each HED
"movie-shot, Onset" event marker - to allow HED search to pick out events
with e.g." luminance-contrast > 3".
Perhaps HED should have a list of recommended measure column name labels
that are not 'at the level of' standard HED schema terms...
Else -  perhaps we should introduce a MovingImage library schema that
defines not only 'luminance-contrast' but also parameters entering into its
definition:
e.g.
"luminance-ratio, 2.1777;
defined by  npreceding-frames, 3; nfollowing-frames, 3;
luminance--measure, RMS; luminance-smoothing, mean"
The above would seem like 'overkill' ... as meaningful variations in this
definition are not likely to be represented in a data archive, certainly
not on the scale that would make their differences across datasets
meaningfully testable...
?
On Fri, Feb 9, 2024 at 10:26 AM dorahermes @.***>
wrote:

I agree that Luminance-contrast is clear and contrast-between does not
work. Note that contrast can also be calculated using sets of filters in
visual encoding models.
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/hed-standard/hed-schemas/issues/135*issuecomment-1936409084__;Iw!!Mih3wA!Ec1YQQ5cf5XXF2uzsYFG2CisA2ZlMluN_ooE2Dq1gjzSnsvsrRsG7wqz_lBcVVxzoWEMm3p6l5ULLwTfvyISxOyq$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AKN2SFQ3OYRRSFPMB4YPAELYSZS6TAVCNFSM6AAAAABCJN6A46VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZWGQYDSMBYGQ__;!!Mih3wA!Ec1YQQ5cf5XXF2uzsYFG2CisA2ZlMluN_ooE2Dq1gjzSnsvsrRsG7wqz_lBcVVxzoWEMm3p6l5ULLwTfvxXzSzO3$
.
You are receiving this because you were mentioned.Message ID:
@.***>

@smakeig, we are not using), at least currently, Luminance-contrast for the Present movie. A good example of the use of this tag is the contrast between foreground and background in a couple of the Healthy Brain Network Project. Here is an example of the exemplary analysis of the Surround Suppression Paradigm from Langer et.al., Sci Data 2017:

The Luminance Ratio for the Present Movie (as briefly described under BIDS issue 153 does not seem to be able to benefit from Luminance-contrast, because Log Luminance Ratio (LLR) is relating two separate frames (which is different from the Luminance-contrast, which requires the elements to be in the same scene).
This is not to discount the importance of annotating temporal context; I just don't think that Luminance-contrast can be extended for that 😊.

To summarize, here is what we have discussed so far
Suggestion to include Luminance-contrast in the base schema:
Tag: Luminance-contrast
Parent: Property/Sensory-property/Sensory-attribute/Visual-attribute
Definition: Luminance contrast is the difference between the luminance of two objects. The suggested quantification method is the relative difference measure of luminance: Contrast-luminance = (Lmax - Lmin) / (Lmax + Lmin), where Lmax and Lmin are the maximum and minimum luminance of [specific portions of] an image.
Source: Legge, G. E., D. H. Parish, A. Luebker, and L. H. Wurm. 1990. “Psychophysics of Reading. XI. Comparing Color Contrast and Luminance Contrast.” Journal of the Optical Society of America. A, Optics and Image Science 7 (10): 2002–10.
Accept-value: Yes, Type = (Non-negative, Fraction), Range = 0 to 1.
Example: Represent a gray circle over a white background with a contrast of 0.5.
    (Visual-presentation,
        (
            (Foreground-view, (Circle, Gray)), (Background-view, White), (Luminance-Contrast, Fraction/0.5)
        )
    )

This definition is incorrect in many cases and only applies to a subset of luminance contrast measurements (you are suggesting Michelson) and it can be calculated in multiple ways as mentioned above. I would therefore not give a 'suggested method for quantification', as there are many methods (Michelson, Weber, RMS, and other filtering approaches). Also note that the RMS contrast is not a fraction. I think the definition should be more general and allow future sub-tree extension with such specifics, e.g:
Definition: Luminance contrast is the difference in luminance in [specific portions of] a scene or image.
Accept-value: Yes, Type = (Non-negative), Range = 0 to 1.

Thanks, @dorahermes,
Comparing the two definitions,

Luminance contrast is the difference between the luminance of two objects.
Luminance contrast is the difference in luminance in [specific portions of] a scene or image.

I don't think either of the definitions is incorrect. However, definition 1 is more general since it does not constrain Luminance-contrast to a scene or image.
There are indeed several ways for quantification of contrast (see Wikipedia page on contrast). We should encourage the inclusion of the quantification method in the definition somehow. Also, in my opinion, providing a recommended method to quantify Luminance-contrast would encourage users to either use that definition or explicitly mention their quantification method. @VisLab, I wonder how we could recommend including the quantification method in the tag?
With a quick literature search, I observed that the Michelson definition seemed to be used often, so I added that as the suggested quantification method.
I agree that the type should not be Fraction, to include the RMS method for contrast quantification.

I have calculated contrast before in encoding models using sets of Gabor filters, and while it would be useful to have some suggestions, I do not think we should recommend any one in particular. Specific recommendations/luminance contrast types should be reserved for a level deeper.

Type of value would be done with SuggestedTag.  Added as part of PR #140.

Addressed in PR#149


review relation to share.js

http://sharejs.org/

Another interesting one worth looking at: https://github.com/Studio182/Studio-History For offline editing.


Is Cloudflare still being actively supported by Serverless Framework?

Hi,
I haven't seen a lot of activities in this repo with PRs still unclosed since 2019. Is Cloudflare still being actively supported and developed by Serverless Framework team?

This looks abandoned to be honest. I wouldn't suggest this for a new project. Dependencies are massively updated as well which is not really good

The serverless framework site has taken down their docs for Cloudflare Workers. They're still accessible in the repo, but the website now 404s.


Postgres query empty when using such intersect/union/except  operator

when I use the the query (select id from a ) intersect (select id from b) the query result is empty,
although when use the sql select id from ((select id from a ) intersect (select id from b)), which can query the correct result list.

Looks like sql-limiter library will change that first query to (select id from a ) intersect (select id from b limit 100). I wonder if that is playing a role here? Can see what this library will do at https://sqlpad.github.io/sql-limiter/

So that need to change the sql-limiter library mechanism in this scenario which can parse the sql and then add limit clause outside the sql, eg: (select id from a ) intersect (select id from b ) limit 100


feat: c4 diagram improvements

Describe your changes
Ticket: https://pins-ds.atlassian.net/browse/ASB-2454
Updates descriptions and adds a BO to FO system definition for clarify as previously only the advice connection was shown which can be misleading.
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[x] I have made corresponding changes to the documentation
[x] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[x] I have provided a description of the ticket
[x] I have included unit tests to cover any testable code changes

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

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

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

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


findBy broken for inherited entity property lookups

Bug Report

Q.
A

BC Break
no

Version
2.6.3

Summary
findBy does not work correctly for inherited entities. proof here: https://github.com/themizzi/doctrine-find-by-metadata-bug
Current behavior
If EntityB has a OneToMany relationship to EntityA, and EntityC inherits (via join) from EntityA. If there is no cache, calling $entityBRepo->findBy('entityA' => $newEntityCInstance) works fine. On a subsequent request after cache has already been generated and metadata is lazy loaded, the request breaks. on sqlite this just fails, on MySQL this potentially loads up an incorrect entity.
How to reproduce
Please see link above for an example and reproduction steps.
Expected behavior
Find by can find classes by properties with an inheritance structure at all times.

@themizzi please reproduce this bug in isolation within an integration test, like in https://github.com/doctrine/doctrine2/tree/master/tests/Doctrine/Tests/ORM/Functional/Ticket.

will try to do this weekend. other priorities call for now.

Created regression test. Might have some time to looking into an appropriate fix this weekend.


Inquiry about license

I would like to ask about the license for this library.
Is there a commercial version, a closed source one?

There is no commercial version. I have added MIT license. You can use this library at will.


Change Data in the App

Does anyone have any hints on how to adjust the data for the customers, for example how would I change the customer/company A. L. Price by John Boone?  I've tried locating the file and editing the DataService files but there is no changes.  Any tips would be appreciated.

I cannot get any of the data to change in the app.  is there any suggestions to changing things like customers weather it be deleting or adding more? Help would be appreciated.


Don't use an async method if you can't be thread safe

This is for the sake of the servers that use this plugin.

Also, please make meaningful commit messages. "Updating"... what are you updating? :(

The extra line break at the end was auto-added by Github and won't affect your plugin

Wouldnt it be pretty easy just to make it thread safe?

Yes, so why don't you do that... (there's multiple ways, so I'm going to et you choose what's best there. One way is by scheduling tasks.)

Okey I will update it then


Improve Coin selection

To study:

Relationship between datum & reference script for cardano wallet's coin selection algorithm: https://github.com/geniusyield/atlas/blob/87fcce0c6eb9ee261f7f71491e2c7dd7fe7f1413/src/GeniusYield/Transaction/CoinSelection.hs#L194-L201

Currently we fix for it later: https://github.com/geniusyield/atlas/blob/87fcce0c6eb9ee261f7f71491e2c7dd7fe7f1413/src/GeniusYield/Transaction/CoinSelection.hs#L175 but as mentioned here https://github.com/geniusyield/atlas/blob/87fcce0c6eb9ee261f7f71491e2c7dd7fe7f1413/src/GeniusYield/Transaction/CoinSelection.hs#L60, we can make Address type here to uniquely identify tx output in our function to compute for minimum ada.

Study the breaking changes in cardano-wallet and how the new feature of it providing the collateral inputs to be used in transaction can be useful.
Is UnableToConstructChange thrown by cardano-wallet imply that it won't be possible build a valid tx even with change of strategy? In such a case we can communicate to user that indeed there are insufficient funds.
The useInline variable seems redundant as it is always set to True.
Fix some other TODO & comments present in files related to coin selection.

@TotallyNotChase do you think you could help with this ticket?

I did have an idea for this in mind but never had the time to implement it. I mostly left the details for it in those comments (basically making the Address associated type uniquely identify a utxo). I'm definitely down to help out by discussing/answering questions around the design I initially envisioned - though it'd probably take me too long to actually implement it myself right now - too busy with other projects :(

Changes done for this are mentioned in this PR: https://github.com/geniusyield/atlas/pull/163#issue-1771630057.


feat: testing tutor in CI

What are the relevant tickets?

Description (What does it do?)

Screenshots (if appropriate):

[ ] Desktop screenshots
[ ] Mobile width screenshots

How can this be tested?

Additional Context

This PR is waiting for the redwood release. For now, it is only working for quince release. It's failing on master because the master branch of edx-platform is broken due to a dependency version issue. For redwood, the tutor is not yet ready (see the release notes for the latest major release v17.0.0 for tutor).


Improve the layout and design of the docs site.

This is the strategy we talked about at Orchard Harvest:

Create PDF mockup for orchardproject.net. Done.
Create HTML mockup based on PDF.
Create Orchard CMS theme based on HTML mockup.
Deploy that theme for orchardproject.net
Deploy that CSS and layout to the docs.orchardproject.net website.
Consider creating a userdocs.orchardproject.net website for content developers, which will also use that CSS and layout.

Here is a diagram of the end result. Note: orchardproject.net is an Orchard CMS; the docs sites are not. The docs sites receive their content from GitHub repos.

read the docs is now used with a default theme.


Geo Limit Countries should be an array

This Improvement request (usability, performance, tech debt, etc.) affects these Traffic Control components:

Traffic Ops

Current behavior:
Currently, it's a string that represents an array of strings separated by commas e.g. "US,CA".
New behavior:
Arrays should be arrays e.g. ["US", "CA"]. If the parser is smart enough (and if we care to do it this way), that could be done in a backward-compatible manner such that if the field is a string it's parsed as a comma-delimited array, and if it's an array it's parsed as such. With the right validation (currently it has none, but country codes can't, for example, legally contain commas) the storage format could even be kept the same, which not only obviates the need for a migration but trivializes supporting old API versions.

@ocket8888 I can take this one


Landing page for role permissions

In the admin part of the application should be a "Permissions and Roles" link and the landing page should show a list of existing role permissions in the database.

GET /api/admin/permissions
The lending page will show permissions divided by roles, table for each role with edit actions

Decided to go with a select box to choose which roles the user is vieweing


oadp-1.0: Update manager image tag, Makefile quality of life updates.

#796
operator-sdk automation (will probably want this for master too..
#716
#831
tag updates

tested with a new gitpod.
https://gitpod.io/#https://github.com/kaovilai/oadp-operator/tree/1.0-tag-oadp-operator
and running make bundle. The following happen automatically.

operator-sdk installs
yq install automatically
nullables preserved.

Codecov Report

:exclamation: No coverage uploaded for pull request base (oadp-1.0@41d2459). Click here to learn what that means.
 Consider uploading reports for the commit ce5fbb9 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             oadp-1.0     #845   +/-   ##
===========================================
  Coverage            ?   38.78%
===========================================
  Files               ?       15
  Lines               ?     3161
  Branches            ?        0
===========================================
  Hits                ?     1226
  Misses              ?     1835
  Partials            ?      100

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

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


Ravi Prakash Pal

Description
I have added my profile to this project
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Hi @RaviP9973 , thanks for your interest in contributing to DevDisplay!
Before we proceed, we noticed you adding your profile on #DevDisplay but forgot to star the repo. It's like making an amazing sandwich and forgetting to take a bite! -- I think our repo's feeling a little lonely without your 🤗 So, be sure to star the DevDisplay GitHub Repository. 🌟
💌 This is a mandatory step for all contributors. & It's is the first priority of contribution in DevDisplay 💌
🚀 We look forward to your contribution once these steps are completed. 🚀
Thanks 👍🏻


Initializer does not override a designated initializer from its superclass

/WORK/Dev/vvc/platforms/ios/HelloCordova/Plugins/com.dooble.phonertc/PhoneRTCPlugin.swift:18:14: Initializer does not override a designated initializer from its superclass

How to solve this problem?I just started to learn the development of apple!

@isathish

+1 I started getting this error after updating cordova-ios from 3.8 to 4.1.1

Take a look here #190


Documentation: Term "development mode" is unclear

I'm doing some work which requires defining some context hooks for our flask app. When reading the documentation for request contexts* I came across the passage:

In production mode if an exception is not caught, the 500 internal server handler is called. In development mode however...

As a user, I want to know: What is 'development mode?' I think it's debug mode (e.g. app.run(debug=True)). I've seen several people use the terms interchangeably when searching for help, but not either the flask developers or the official documentation. The flask docs don't appear to have an index (is it hidden somewhere?), and quick search does not appear to work. Google site search of the dev docs does not immediately turn up an obvious answer.
IMO, this is a bug in the documentation. Assuming for the sake of argument I'm right about dev=debug, that passage should be either:

"In development mode (i.e. with debug=True) however..." or
The text 'development mode' is a link to a list of terms & their definitions
Something along those lines.

Maybe if I had carefully read all of the docs from beginning to end, or at least carefully read the tutorial twice, I might not be confused about this. But lots of your users won't do that. That's why I call it a bug (it's sorta like failing to validate input).
*I was in fact reading the docs for the version of flask I'm using, but pointing to dev-version docs b/c I presume they represent the current state of the documentation

In that case, it looks like it means debug mode, although it can also mean "while using the dev server" which refers to app.run, flask run or really just doing development. Do you want to submit a PR changing it to "In debug mode, however, ..."?

@davidism I sorta don't want to submit a PR inasmuch as it's almost no work to make the required change, but it requires knowledge of Flask I don't actually have.  It doesn't strike me as a very sensible way to run a project.
But supposing I was going to make a PR: what is the operational definition of "just doing development?" And am I correct in thinking that flask is in development mode both when running on the builtin server with debug=False and when running on (say) uwsgi with debug=True?

am I correct in thinking that flask is in development mode both when running on the builtin server with debug=False and when running on (say) uwsgi with debug=True?

I would avoid using "development mode" in reference to a Flask application altogether. For consistency, I think the mode should always be called "debug mode". I think calling the opposite of debug mode "production mode" is also a bit misleading, since this is a mode that you can certainly run on the Flask development server, which is not meant to be used in production.
So regarding your question @dradetsky, in my opinion there is just "debug mode", and it can be on or off, regardless of the server being the builtin one or uwsgi, gunicorn, etc.
There are a couple more references to "production mode" and 'development mode" in the docs besides the one you found that I think would also benefit if they are rephrased in terms of debug mode.

There is a distinction to be made between running the development server and running a production server, since the dev server is not intended for production, but I think that's already handled in the deployment page. Most of the docs only care about debug mode on or off.


Format by IDEA with this exception

EclipseCodeFormatter have a option:[format other file types by intellj ...]
I have input "xml;" in this settings,When I format code in a xml file, IDEA's format seems not work, what should I do? or is there any other settings I haven't noticed ?

Works as intended.
"Format other file types by IntelliJ with this exception:" - means (or at least I hope so) that IntelliJ will format everything but that what you add there.


Tuples Rework

WIP: DO NOT MERGE
This will add the full functionality of tuples as they are implemented in .NET v4.6.1.
Todo:

[x] Add basic reference Tuples.
[x] Add complex reference Tuple.
[x] Add basic value Tuples.
[x] Add complex value Tuple.
[x] Extension Methods
[ ] Optimization

Needs upstream merge to reflect "Compability" -> "Compatibility" renaming


Extend EJECT_CDROM feature to hypervisors

Description
The EJECT_CDROM feature is only supported at the moment through the windows contextualization service. This feature can be implemented at a hypervisor level by means of removing the device definition from the VM. In the case of libvirt this is achieved by using the command
"virsh --connect $LIBVIRT_URI change-media $DOMAIN $TARGET_DEVICE --eject --force"

The goal is to run this action when the contextualization service finishes after the boot process if the VM has the flag set on the template. At the moment the flag is only read internally by the context program running inside the Windows VM. This flag could be re-used, however, the implementation could be done on a disk basis
Use case
Contextualization in Guest OS is done by injecting an ISO to the guest OS. After the one-context service is run, the ISO is no longer needed, becoming an unnecessary oddity for the end user.
Additional Context

The drive removal should only occur after the contextualization process is finished in the guest
This feature could also apply to non context disks
The one.vm.updateconf call will trigger a prereconfigure and reconfigure action that remove and insert the drive. Needs to be considered to avoid in case of absent context ISO.

Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

duplicated by #805


Related Issue
Previous WIshlist is not as good and it is enlarged in size.
Description
Added wishlist page under profile section with use of bootstrap ,advance css and html.
Type of PR

[x] Bug fix
[x] Feature enhancement
[x] Documentation update
[x] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
@arghadipmanna101 plz merge my pull request under gssoc24 label. Issue no #636

There have some conflict issue check it once and make a new PR. ok

@arghadipmanna101 now i think it is verified plz check and tell changes if required.

There have some issue check it once


Interpolator for CCL Cls

Added an interpolator class to help speed up the calculation of C_ls with the newer versions of CCL
Tagging @anicola

It'd be good to check if this implementation is much slower than the old CCL one before merging


DNM: test impact of bazel's --experimental_remote_download_outputs

I am working on Bazel/RBE and noticed that you are using the new "Builds without the Bytes" feature. This PR is just to test it's impact on performance. This should not be merged. Hope that's ok.

fyi @lizan

@buchgr experimental_remote_download_outputs is required in most CI runs otherwise it will fail with out of disk.

Thanks @lizan. It looks like it's still a good indication that not downloading outputs reduces e2e times by some ~50%.