Headless CUDA hanging

I had issues with my headless (no monitor connected) Ubuntu 14.04 multi-gpu setup.
Running anything using cutorch would hang when finished or interruped and I would have to reboot the machine.
However the same code would finish fine if a monitor was connected to the machine.
The solution (which took me a while to figure out...) is to enable the persistence mode which keeps the gpu drivers loaded even when no process is running.
You can do this by running nvidia-smi -pm 1
You can also run it at startup by editing cron
sudo crontab -e then add @startup nvidia-smi -pm 1
I hope this help if anyone has the same problem!

Thanks! This is good to know.


Implementing a scheduler

I used to use Multicraft and the lack of a scheduler is unfortunately a negative point for PufferPanel. It is important for a user to have a way to have scheduled commands that will be run on one server whenever it's needed.
If you can implement it in the future updates, PufferPanel will become a real competitor of the no-opensource server managers.
Thank you very much.

I would also like to see this.
Duplicate of #474, closing.


Distillation training setting in DeiT

Hi, I am trying to reproduce the distillation result in DeiT peper but I am a little confused about the training detail. In table 2 of the DeiT paper there is 'pretrain' in 'Student: DeiT-B'  column and in table 3 you mentioned '300 epochs of pre-training'. I'm confused about its meaning: pretrain for teacher or student?
So take table 2 for example, to make DeiT-B achieve 83.1, the teacher model 'RegNetY-16GF' and student model 'DeiT-B' should both be pretrained,or just the teacher model? Would you please share the command to train since it's not included in README file.
Thanks a lot.

Sorry that I find the command in the README file now, it seems my downloaded README file is out of date. I'm only confused about the 'pretrain' meaning in paper now.

It seems that this "pretrain" is under 224 resolution, followed by a finetuning stage under bigger resolution

That makes sense.Thanks.


ObviousAlexC/PenPlus Fix texture handling between versions of pen+ by backporting from the experimental version.

this fixes one half of https://github.com/TurboWarp/extensions/issues/1508 the other half is from the blocks making the depth being negative.
Thanks to penta-quark-neutro for reporting this bug.
I have migrated the costume handling from the experimental version

Here @penta-quark-neutro here is the other half for your project.


Release of the pre-trained model weight?

Hi @pinakinathc,
Appreciate for your excellent work! Would you please release the pre-trained model weight on SketchyCOCO and the inference code?
Regards,
Alonzo

Dear @AlonzoLeeeooo
I can only get back to this (i.e., upload pre-trained models) in late November. However, the training and validation code is uploaded at main.py#L66.
If you download SketchyCOCO from this url and NOT change the folder structure, you should be up and running. Just set the data path at options.py#L16

Okay. Thanks for replying. By the way, what GPU do you use for training? How long does the training take?

Dear @AlonzoLeeeooo
As far as I recall, you can hit the numbers on paper in about 5 hours or so. The GPUs are RTX 3090 Ti. Note, I did try with gradient accumulation (via pytorch lightning) for a bigger batch size (which helps a lot).

Dear @AlonzoLeeeooo
As far as I recall, you can hit the numbers on paper in about 5 hours or so. The GPUs are RTX 3090 Ti. Note, I did try with gradient accumulation (via pytorch lightning) for a bigger batch size (which helps a lot).

Thanks for the information!


PGP/MIME with Protected Headers ("Memory Hole")

Enhancement request:
Please implement the PGP/MIME Protected Headers known as "memory hole" as implemented in Thunderbird's Enigmail plug-in (though it may not yet be widely enabled).
It protects mail from having altered From and Subject fields most importantly.
Filing a bug so as to track its progress.

The Memory Hole standard is documented here:
https://modernpgp.org/memoryhole/
I think it's definitely necessary to implement this standard. Please do so!

This server could not prove that it is modernpgp.org; its security certificate is from *.github.com.

Sigh.

This gets more and more widespread use, Enigmail does ship it now enabled by default.

I'm currently focusing to get Autocrypt support done. I might add simple support for encrypted subjects, though. It's not super high priority though.

BTW Memoryhole is also supported in Enigmail AFAIK since one or a few version(s).

See #3309


Ocelot Application with own Controllers/Actions

Need implementation example
Trying to create API gateway which also uses it's own Web API Controllers to handle mashups and any other custom code, using Ocelot for pipeline items like rate limiting, Authorization, and caching.
I am trying to put together an API gateway that also serves the purpose of mashing up calls to multiple APIs. I would also like to have autogenerated Swagger, so I'd like to combine the capability of Ocelot for authorization and rate limiting but would also like to be able to create the edge custom API's in .NET code to handle translations and combinations of API calls.
Is this possible with Ocelot, and are there any examples? Does Ocelot only work with downstream API's as separate applications? I'm trying to avoid an extra hop that would only exist for mashups and composite API calls.
Stack Overflow related item: https://stackoverflow.com/questions/56956831/can-ocelot-have-its-own-controller-actions

Is this sorted? Can we have custom logic in ocelet's own controller? Let's say I want to call 2nd downstream API based on the response of the 1st..

Something new?

Hi @kvansaders, @masanand  and @szymonpiekny,
Recently I was working on similar PoC for one of the projects I'm working on and my main goal was to have endpoints like @kvansaders mentioned to be able to have possibility mehsup / service enhance / add specific aggregation in case of one call to API G. with multiple walls internally or hide old / ugly API and what was more important for me  to be able to call to API G. to publish a message to a broker. My first approach was to have middleware which worked but I wasn't able to create out of box swagger. Then I get to this topic of Ocelot own APIs.
After standard Ocelot configuration on .NET 6 project with some downstream projects what I did additionally is:

In the services section (Program.cs) make sure you have
services.AddEndpointsApiExplorer();
and if you would like to have swagger use MMLib.SwaggerForOcelot nugget with configuration
services.AddSwaggerForOcelot(configuration, (o) =>
{
o.GenerateDocsForGatewayItSelf = true;
});
Then in the app section (Program.cs) for swagger make sure you have
app.UseSwaggerForOcelotUI();
and your custome Api like
app.MapGet("/testing", () => Results.Ok("It's working"));
This way swagger will start generating the doc (change the view from default first service to Gateway - there you will see your endpoint). In my case running this I was getting 404 when I called the internal API. After couple of tryouts with the configuration I figured out that I was missing:
app.UseRouting();
app.UseEndpoints(endpoints =>
{
endpoints.MapControllers();
});
With such configuration you will be able to run standard downstream / upstream setup with ocelot and swagger (need additional configuration in ocelot.json) and your custom api for any needs.
If needed let me know I may publish some demo app into the github. I hope this will help you guys.


Missing parameter for goals causes a build error in project...

I have been using your plugin for a while and it all works well, accept for the following error whenever I try to clean or save my project in eclipse.
The following is the error I receive...
Missing parameter for pluginExecutionFilter. groupId, artifactId, versionRange and goals must be specificed, but found: groupId = 'com.devskiller.jpa2ddl'
artifactId = 'jpa2ddl-maven-plugin'
versionRange = '0.9.11'
goals = '[]'

Also please see attached what I currently have in my pom...
			<plugin>
	            <groupId>com.devskiller.jpa2ddl</groupId>
	            <artifactId>jpa2ddl-maven-plugin</artifactId>
	            <version>0.9.11</version>
	            <extensions>true</extensions> <!-- required to run automatically -->
	            <goals></goals>
	            <configuration>
	                <outputPath>${basedir}/src/test/resources/sql/schema_MariaDb.sql</outputPath>
	                <packages>
	                    <package>generic.model</package>
	                    <package>generic.model</package>
	                </packages>
	                <jpaProperties>
	                    <property>
	                        <name>hibernate.dialect</name>
	                        <value>org.hibernate.dialect.MySQL5Dialect</value>
	                    </property>
	                    <property>
	                        <name>hibernate.default_schema</name>
	                        <value>databasename</value>
	                    </property>
	                </jpaProperties>
	                <formatOutput>true</formatOutput>
	                <skipSequences>true</skipSequences>
	                <delimiter>;</delimiter>
	                <action>CREATE</action>

	                <ignoredDependencies>
                		<ignoredDependency>org.javassist:javassist</ignoredDependency>
              		</ignoredDependencies>
		 			<ignoredUsedUndeclaredDependencies>
                		<ignoredUsedUndeclaredDependency>org.javassist:javassist</ignoredUsedUndeclaredDependency>
              		</ignoredUsedUndeclaredDependencies>
 					<ignoredUnusedDeclaredDependencies>
                		<ignoredUnusedDeclaredDependency>org.javassist:javassist</ignoredUnusedDeclaredDependency>
              		</ignoredUnusedDeclaredDependencies>

	            </configuration>
	        </plugin>

Any help would be greatly received, thanks.

Why do you add <goals></goals> ? It's not needed as extensions set to true enables the goal inheritance from the plugin config. Maybe Eclipse doesn't support it? Then if this error is annoying for you please just add generate goal:
<goals>
    <goal>generate</goal>
</goals>


Add screenshot testing support & workflow

Enhancements

Adds dropshot plugin for screenshot testing
Adds multiple simpler screenshot tests and helper methods
Adds screenshots for Pixel 7 API 33  as the first test device.
Adds workflow for running the screenshot tests on Firebase Test Lab
Refactors UiTestInfo to provide test method as a property and wrap the logging of events, enabling us to easily access it.
Minor cosmetic updates to UiTestActivity

TODO

[x] Create the firebase project and add service file to github
[x] Add readme

Moved back into draft to add saving of test results to bucket after failing.


[stdlib] Set, Dictionary: Prepare for per-instance hash seeds

Implement the per-instance seeding infrastructure without actually enabling per-instance seeding.
This is #19533 except for the crucial step of actually generating a new seed.
rdar://problem/44760778

@swift-ci please test source compatibility

@swift-ci please test

Ah, facepalm. Swiftpm is failing because I forgot updating mapValues to copy the seed over.
Clearly we don't have enough test coverage for mapValues.

@swift-ci smoke test

@swift-ci smoke test

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please test source compatibility

FAIL: fluent, 4.2, 270b6f, Swift Package

But how

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


Interacting with labels

Feature Description
Labels on merge requests are a key part of how I track my development and code deployment processes. Being able to display them (#152; or perhaps other methods as well) and add/delete them would be amazing.

This functionality has been added, please see MR #159 and the added documentation.


Variable member passed by value to method of same object -- Why?

@weierophinney
Good morning,
After looking at your code, I feel a bit stupid and so I prefer ask you to learn a bit more if needed.
In the Zend\ConfigAggregator\ConfigAggregator, the constructor look as follows:
    public function __construct(
        array $providers = [],
        $cachedConfigFile = null,
        array $postProcessors = []
    ) {
        if ($this->loadConfigFromCache($cachedConfigFile)) {
            return;
        }

        $this->config = $this->loadConfigFromProviders($providers);
        $this->config = $this->postProcessConfig($postProcessors, $this->config);
        $this->cacheConfig($this->config, $cachedConfigFile);
    }

So basically put, you get the configuration from the configuration providers, then you post process it with optional post-processors and finally, you cache it if requested.
The question that come to mind here is: why you're passing the config member variable by value to methods of same object. For me, doing this mean that you do a copy first in memory, involving useless overhead in my opinion. Why you don't just access the config member variable from the called methods directly? Of course these days, this can look as a micro-optimization but I'm really wondering the purpose of such a way.

It's a lesson I learned from functional programming. Doing it this way makes the methods idempotent and more easily testable.

@weierophinney
Thank you for explanation. That's much appreciated ;)


Regenerate with latest gir and mark suboptimal Object constructors as deprecated

Actually don't merge this yet. I have another improvement for the builders pending.


Kaiwa Index Page Never Loads

Running Kaiwa on my own server, with the LDAP module removed, results in the webpage hanging indefinitely.
Running node server only gives the following output:
dev environment detected
Client app ready
Cache manifest generated
Otalk running at: <URL>:8000

No further debugging output is available.
From the developer's console in Chrome, it seems like the client is hanging on trying to load app.nonCached.js.

I have the same problem using prosody 0.10 and prosody 0.9 (stanza.io with old-websocket).
After logging in Chrome console says "Unhandled rejection Error: [object Object] in app.nonCached.js" after calling _drainQueues() method. I can't do anything - I have "main screen", I can "add" room, but nothing else.

Same issue, also using Prosody 0.10. Is anyone ever going to look at this? The report is over a year old now.


"Explore" item about JBrowse?

Should we add an Explore item about JBrowse at some point?  We could probably just have a screen shot, a sentence or two introduction and then a link to the "Getting started with PomBase JBrowse" page.

Sounds like a good plan! I missed that opportunity...

@Antonialock

I uploaded an image in the spotlight folder. I can't see where to dump the text (not good at searching github outside of tickets!)
PomBase's JBrowse instance (https://www.pombase.org/jbrowse/) provides a response online environment in which to visualise published genomic datasets. Datasets can be accessed either using the track selector within JBrowse, or by following the link on the publication page of a hosted dataset. For more information please see our JBrowse documentation page https://www.pombase.org/documentation/JBrowse_quick_start.

updated the comment because my english sucks.

The text goes in the main website config file:
https://github.com/pombase/pombase-config/blob/master/website/pombase_v2_config.json
If you search for "explore" in the file you'll see what the entries look like.  The "link" field should probably be "/jbrowse/".
If add your item I'm happy to test it locally before adding the main site.

cheers, done


Refactor for code sharing

Closes #264

Codecov Report
Base: 92.06% // Head: 92.00% // Decreases project coverage by -0.06% :warning:

Coverage data is based on head (80822a1) compared to base (0ec8620).
Patch coverage: 95.74% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #286      +/-   ##
==========================================
- Coverage   92.06%   92.00%   -0.07%
==========================================
  Files          10       10
  Lines         870      850      -20
==========================================
- Hits          801      782      -19
+ Misses         69       68       -1

Impacted Files
Coverage Δ

weather_command/_builder.py
99.22% <95.74%> (+0.30%)
:arrow_up:

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.


Stopped working after several days

Hey,
i have the cli running as a docker container container. It worked perfectly, but since two days, no messages are published any more.
Can i somewhere review the logs what happened?
Thanks a lot,
Max

after restarting container, i see the following logs:
weconnect-mqtt_composed | INFO:weconnect-mqtt:Connect to WeConnect
weconnect-mqtt_composed | INFO:weconnect:Token refreshed
weconnect-mqtt_composed | INFO:weconnect-mqtt:Connected to MQTT broker
weconnect-mqtt_composed | INFO:weconnect-mqtt:Update data from WeConnect
weconnect-mqtt_composed | WARNING:weconnect:/vehicles/WVWZZZE1ZMP041013/status/climatisationTimer/timers/1: Unknown attribute singleTimer with value {'startDateTime': '1999-12-31T22:00:00Z'}
weconnect-mqtt_composed | WARNING:weconnect:/vehicles/WVWZZZE1ZMP041013/status/climatisationTimer/timers/2: Unknown attribute singleTimer with value {'startDateTime': '1999-12-31T22:00:00Z'}
weconnect-mqtt_composed | INFO:weconnect-mqtt:New intervall set to 300s by MQTT message
weconnect-mqtt_composed | INFO:weconnect-mqtt:Error setting value: /vehicles/WVWZZZE1ZMP041013/status/climatisationSettings/climatisationWithoutExternalPower: new value must be of type <class 'bool'> but is of type <class 'str'>
weconnect-mqtt_composed | INFO:weconnect-mqtt:Error setting value: /vehicles/WVWZZZE1ZMP041013/status/climatisationSettings/climatizationAtUnlock: new value must be of type <class 'bool'> but is of type <class 'str'>
weconnect-mqtt_composed | INFO:weconnect-mqtt:Error setting value: /vehicles/WVWZZZE1ZMP041013/status/climatisationSettings/windowHeatingEnabled: new value must be of type <class 'bool'> but is of type <class 'str'>
weconnect-mqtt_composed | INFO:weconnect-mqtt:Error setting value: /vehicles/WVWZZZE1ZMP041013/status/climatisationSettings/zoneFrontLeftEnabled: new value must be of type <class 'bool'> but is of type <class 'str'>
weconnect-mqtt_composed | INFO:weconnect-mqtt:Error setting value: /vehicles/WVWZZZE1ZMP041013/status/climatisationSettings/zoneFrontRightEnabled: new value must be of type <class 'bool'> but is of type <class 'str'>
weconnect-mqtt_composed | INFO:weconnect-mqtt:Error setting value: /vehicles/WVWZZZE1ZMP041013/status/climatisationSettings/zoneRearLeftEnabled: new value must be of type <class 'bool'> but is of type <class 'str'>
weconnect-mqtt_composed | INFO:weconnect-mqtt:Error setting value: /vehicles/WVWZZZE1ZMP041013/status/climatisationSettings/zoneRearRightEnabled: new value must be of type <class 'bool'> but is of type <class 'str'>
weconnect-mqtt_composed | INFO:weconnect-mqtt:Error setting value: id /vehicles/WVWZZZE1ZMP041013/controls/charging cannot be set. You need to provide it in the correct format select one of [start, stop]
weconnect-mqtt_composed | INFO:weconnect-mqtt:Error setting value: id /vehicles/WVWZZZE1ZMP041013/controls/climatization cannot be set. You need to provide it in the correct format select one of [start, stop]

Hey, I removed the full VIN from your log (for your privacy). I'll have a look tomorrow. The "Error setting value" should be already fixed in Version 0.6.2
The "singleTimer" I have not seen with my cars before, I will add that to the API as soon as I can, but this should not prevent new messages.
What would be helpful is a log of the container when it does not send anymore.

Can you please try to update to version 0.7.0 and see if the errors are gone?
For the situation where it stopped working I would need a log from that specific situation.

i'll try it, thanks! can i retrieve the logs from some files or must i be attached to the process and view the logs live?

If you use docker-compose you can go with:
docker-compose logs weconnect-mqtt

if your image has a different name than weconnect-mqtt you have to change that of course.
If not find your container ID with:
docker container ls

and then execute
docker logs <CONTAINER ID>

I'll close this for now. If the problem shows again, please re-open

Hi,
the problem reappeared. I now as able to get the logs:
weconnect-mqtt_composed | INFO:weconnect:Token refreshed
weconnect-mqtt_composed | Exception in thread Thread-118:
weconnect-mqtt_composed | Traceback (most recent call last):
weconnect-mqtt_composed |   File "/usr/local/lib/python3.9/threading.py", line 954, in _bootstrap_inner
weconnect-mqtt_composed |     self.run()
weconnect-mqtt_composed |   File "/usr/local/lib/python3.9/threading.py", line 1266, in run
weconnect-mqtt_composed |     self.function(*self.args, **self.kwargs)
weconnect-mqtt_composed |   File "/usr/local/lib/python3.9/site-packages/weconnect/weconnect.py", line 374, in __refreshToken
weconnect-mqtt_composed |     raise RetrievalError(f'Status Code from WeConnect server was: {refreshResponse.status_code}')
weconnect-mqtt_composed | weconnect.errors.RetrievalError: Status Code from WeConnect server was: 504

I hope that helps?

And there are some other exceptions some time before which do not belong to this issue i think: https://pastebin.com/1kS2rVG7

I'm currently on vacation and will have a look once I'm back!

Should be fixed in 0.12.2. But I cannot reproduce the server errors that lead to the problem. Please reopen if you still experience this crash @mdhom


How to change the line of two nodes to a curve or two non-coincident parallel lines

Can you replace a straight line with a curved line

Hi, I'm sorry but curved lines are not supported yet. It's on the roadmap though. Thank you for filling in the survey!

If the curve is not good, can a straight line be used? The main purpose is to separate the two overlapping lines in the routine. Is there a way?

I very very very need it

As much as I would like to be able to help you, this library is not ready yet to support curved or parallel edges, and my priorities are elsewhere currently. If you depend on it in a production application, would you mind if I ask you to get in touch directly to discuss a possibility of sponsored development? Otherwise I would recommend to switch to another library. Thanks for your comprehension.


Support password reset on custom domain via Medplum bot

Current Signin Form React Component exposes a function onForgotPassword().  To get a robust workflow working end-to-end we want to extend the functionality as follows.

Support a Project level setting that supreses emails from Medplum on password reset and instead triggers a Bot when a new PasswordChangeRequest is generated.
This special subscription would be behind ReCaptcha and confirm that the ProjectMembership exists for that project
The customer then would then build a custom Bot with their own messaging and landing URL to change password using the data from the PasswordChangeRequest

(Note: this is irrelevant for all customers using external identity providers.)

@reshmakh Any preferences between the following approaches?

Project setting for skipResetPasswordEmail - surgical approach, suppress this one email
Project setting for skipUserEmails - broader approach, suppress all "Welcome" and "Reset Password" emails
Project "feature", which can only be modified by super admin

I think the skipResetPasswordEmail is preferred as there may be cases (admins, auditors, executives, etc) where you want to give access to console and a downstream application using the same login.

Another tricky part to this will be reCAPTCHA - at present, password reset requires the master reCAPTCHA site key, which is associated with client side domains.
We will need to add a Project level setting for reCAPTCHA site key, and this password reset flow will need that additional data.

For reference, see docs on Custom Welcome Emails: https://www.medplum.com/docs/auth/custom-welcome-emails


CachedImage with resizeMode style cause warning

Hi kfiroo,
If I add resizeMode to CachedImage style, it will cause warning
<CachedImage style={{resizeMode:Image.resizeMode.stretch}} source={{uri:item.img}} />

Warning: Failed prop type: Invalid props.style key `resizeMode` supplied to `ActivityIndicator`.
Bad object: {
  "width": 360,
  "height": 592,
  "resizeMode": "stretch",
  "opacity": 0,
  "backgroundColor": "transparent",
  "alignItems": "center",
  "justifyContent": "center"
}
...
    in ActivityIndicator (created by CachedImage)
    in CachedImage (created by TopBannerItem)

Warning: Failed prop type: Invalid props.style key `resizeMode` supplied to `View`.
Bad object: {
  "alignItems": "center",
  "justifyContent": "center",
  "width": 360,
  "height": 592,
  "resizeMode": "stretch",
  "opacity": 0,
  "backgroundColor": "transparent"
}
...
    in View (created by ActivityIndicator)
    in ActivityIndicator (created by CachedImage)
    in CachedImage (created by TopBannerItem)

the same with #45 and #1
should not use resizeMode in style,
should use props resizeMode instead


config_default.php Cleanup

Cleaned up a handful of spacing issues, related to the use of spaces vs. tabs, that weren't obvious in the GitHub views.

And it looks like I did something wrong since it isn't just the commits I made showing in the PR...

I think you'd better fork again

Detectx lives in it's own repo now
https://github.com/munkireport/detectx

My Git inexperience is showing through. I didn't mean to include all those commits by you.

I'll give it another shot after lunch.


KCF tracker doesn't support multiscale

It will be better to support multiscale.
Maybe
https://github.com/ppaanngggg/KCFTracker
is a reference.

Can I work on this?
I have gone through the related code, and had a couple of questions.


Poetry 1.7.1 fails to install blurhash-python in Docker Build Github Actions

Poetry version: 1.7.1
Python version: 3.10.9
OS version and name: Ubuntu 22.04 (ubuntu-latest in Github Actions Runner)

Issue

poetry install fails on blurhash-python version 1.2.1 with error:
#13 94.79
#13 94.79   ChefBuildError
#13 94.79
#13 94.79   Backend subprocess exited when trying to invoke build_wheel
#13 94.79
#13 94.79   WARNING setuptools_scm.pyproject_reading toml section missing 'pyproject.toml does not contain a tool.setuptools_scm section'
#13 94.79   running bdist_wheel
#13 94.79   running build
#13 94.79   running build_py
#13 94.79   creating build
#13 94.79   creating build/lib.linux-aarch64-cpython-310
#13 94.79   creating build/lib.linux-aarch64-cpython-310/blurhash
#13 94.79   copying src/blurhash/_version.py -> build/lib.linux-aarch64-cpython-310/blurhash
#13 94.79   copying src/blurhash/__init__.py -> build/lib.linux-aarch64-cpython-310/blurhash
#13 94.79   running build_ext
#13 94.79   generating cffi module 'build/temp.linux-aarch64-cpython-310/blurhash._functions.c'
#13 94.79   creating build/temp.linux-aarch64-cpython-310
#13 94.79   building 'blurhash._functions' extension
#13 94.79   creating build/temp.linux-aarch64-cpython-310/build
#13 94.79   creating build/temp.linux-aarch64-cpython-310/build/temp.linux-aarch64-cpython-310
#13 94.79   creating build/temp.linux-aarch64-cpython-310/src
#13 94.79   gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -fPIC -Isrc/ -I/tmp/tmp1a5eylby/.venv/include -I/var/lang/include/python3.10 -c build/temp.linux-aarch64-cpython-310/blurhash._functions.c -o build/temp.linux-aarch64-cpython-310/build/temp.linux-aarch64-cpython-310/blurhash._functions.o -std=gnu99
#13 94.79   error: command 'gcc' failed: No such file or directory
#13 94.79
#13 94.79
#13 94.79   at /var/lang/lib/python3.10/site-packages/poetry/installation/chef.py:164 in _prepare
#13 95.04       160│
#13 95.04       161│                 error = ChefBuildError("\n\n".join(message_parts))
#13 95.04       162│
#13 95.04       163│             if error is not None:
#13 95.04     → 164│                 raise error from None
#13 95.04       165│
#13 95.04       166│             return path
#13 95.04       167│
#13 95.05       168│     def _prepare_sdist(self, archive: Path, destination: Path | None = None) -> Path:
#13 95.05
#13 95.05 Note: This error originates from the build backend, and is likely not a problem with poetry but with blurhash-python (1.2.1) not supporting PEP 517 builds. You can verify this by running 'pip wheel --no-cache-dir --use-pep517 "blurhash-python (==1.2.1)"'.
#13 95.05
#13 95.05   • Installing uvicorn (0.23.1)
#13 95.06   • Installing uvloop (0.19.0)
#13 95.52   • Installing watchfiles (0.19.0)
#13 96.44   • Installing websockets (11.0.3)
#13 ERROR: process "/bin/sh -c poetry config virtualenvs.create false && poetry install --no-interaction --no-ansi --no-root" did not complete successfully: exit code: 1
------
 > [8/9] RUN poetry config virtualenvs.create false && poetry install --no-interaction --no-ansi --no-root:
95.04       166│             return path
95.04       167│
95.05       168│     def _prepare_sdist(self, archive: Path, destination: Path | None = None) -> Path:
95.05
95.05 Note: This error originates from the build backend, and is likely not a problem with poetry but with blurhash-python (1.2.1) not supporting PEP 517 builds. You can verify this by running 'pip wheel --no-cache-dir --use-pep517 "blurhash-python (==1.2.1)"'.
95.05
95.05   • Installing uvicorn (0.23.1)
95.06   • Installing uvloop (0.19.0)
95.52   • Installing watchfiles (0.19.0)
96.44   • Installing websockets (11.0.3)
------
Dockerfile:39
--------------------
  37 |
  38 |     # Project initialization:
  39 | >>> RUN poetry config virtualenvs.create false && poetry install --no-interaction --no-ansi --no-root
  40 |
  41 |     # Copy function code
--------------------
ERROR: failed to solve: process "/bin/sh -c poetry config virtualenvs.create false && poetry install --no-interaction --no-ansi --no-root" did not complete successfully: exit code: 1
Error: buildx failed with: ERROR: failed to solve: process "/bin/sh -c poetry config virtualenvs.create false && poetry install --no-interaction --no-ansi --no-root" did not complete successfully: exit code: 1

I've tried everything, even downgrading Poetry and blurhash-python but it keeps failing whereas it works on my local development device (macbook air M1 14.3).
The only issue I found similar to this error is: Installing a package that depends on setuptools fails intermittently: Backend 'setuptools.build_meta:legacy' is not available. #7611 on Poetry repository but no luck.
Maybe pyproject.toml file needs updating its build backend.

Hi,
The log you pasted above contains a clue what the issue could be:

#13 94.79   gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -fPIC -Isrc/ -I/tmp/tmp1a5eylby/.venv/include -I/var/lang/include/python3.10 -c build/temp.linux-aarch64-cpython-310/blurhash._functions.c -o build/temp.linux-aarch64-cpython-310/build/temp.linux-aarch64-cpython-310/blurhash._functions.o -std=gnu99
#13 94.79   error: command 'gcc' failed: No such file or directory

That looks a lot like gcc, which is required to build the package from source, is not installed on the runner by default.
I tried installing blurhash on my computer using Poetry 1.7.1, from both wheel and source distribution. Both methods were successful, so the issue shouldn't be with the package.

I figured out a way to build wheels for aarch64. I'm releasing a new version, 1.2.2, which adds wheels for Python 3.12 and aarch64 architecture, for every supported Python version. I'm using qemu to build and test the wheels, so hopefully it'll work on real hardware too :crossed_fingers:

I have released the new version: https://pypi.org/project/blurhash-python/1.2.2/. You should be able to install it without gcc. Let me know if you're unable to install the new version.


Fix rule require path

Ran across this error after adding the plugin:
Error: Cannot find module './require-size-attributes'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:636:15)
    at Function.Module._load (internal/modules/cjs/loader.js:562:25)
    at Module.require (internal/modules/cjs/loader.js:692:17)
    at require (internal/modules/cjs/helpers.js:25:18)
    at Object.<anonymous> (project/node_modules/@mizdra/eslint-plugin-layout-shift/lib/index.js:5:32)

Updating the require call with the correct relative path to the rule file fixes the issue.

I'm curious how this plugin was working without this change. I'm using ESLint 5.x, perhaps v6 changes the module resolution behavior?

@mizdra anything you need to merge this?

Thanks for the contribution! Sorry to keep you waiting. I'll check it out now.

From the diff, it looks like it doesn't work with ESLint v6/v7 as well as v5.
Originally, I created this plugin to use in my hobby app.

However, I haven't had time to develop an app lately, and so far I've never used this plugin.
That's why I wasn't aware of the existence of this bug

Just fixing the path is enough. I'll merge it.

This patch was shipped with v1.0.1.


Feedback

Analyzer Feedback from IntelliJ
Version information

IDEA IU-182.4323.46
2.1.0-edge.ccb16f72824374163562364bf19dd18e8a882fab
IU-182.4323.46, JRE 1.8.0_152-release-1248-b8x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.13.6 unknown, screens 2560x1440  Retina

Exception
Please append the contents of:
file:///private/var/folders/8g/pm17c65n2qv73xm62yjb7vhm0000gn/T/report1.txt
Dart analysis server, SDK version 2.1.0-edge.ccb16f72824374163562364bf19dd18e8a882fab, server version 1.20.5, FATAL error: Failed to handle request: {id: 7073, method: edit.isPostfixCompletionApplicable, params: {file: /Users/alex/d/learnprojs/glenair/lib/widgets/catalog_images_list.dart, offset: 237, key: .return}, clientRequestTime:<PHONE_NUMBER>643}
NoSuchMethodError: The method 'isSubtypeOf' was called on null.
Receiver: null
Tried calling: isSubtypeOf(Instance of 'InterfaceTypeImpl')
#0      Object.noSuchMethod (dart:core/runtime/libobject_patch.dart:48:5)
#1      PostfixCompletionProcessor._findOuterExpression.<anonymous closure> (package:analysis_server/src/services/completion/postfix/postfix_completion.dart:566:16)
#2      _ListBase

It's difficult to know for sure because the stack traces don't match the current code base, but I believe that this was fixed by https://dart-review.googlesource.com/71620. Please let us know if you continue to see this after updating to a more recent build.


Data Loaded Animation

What is your feature?
Adding the loading content when user click the button
Why is this feature needed?
Because without this animation the user would see the blank of the card
Who is your feature for?
to make the user more engagement
Additional context
This is for use to feel more engagement in to the app so the user would not just wait to see the blank

closed with pr #22


WIP ica.find_bads_ecg pass ch_name to create_ecg_epochs

Redo of #3169 because I'm terrible with Git.

Currently, if you do something like
ica.find_bads_ecg(raw, "CUSTOM_ECG_NAME")
it will internally create ecg_epochs by calling create_ecg_epochs(inst). The problem with this is that if, for some reason, "CUSTOM_ECG_NAME" is not defined properly as the only ECG channel, it will pick the wrong channels, and possibly create bad ecg epochs.
This PR should in principle be very rare (which is why it's not been discovered so far), but can happen for example with the HCP files.
This change passes the ch_name to the create_ecg_epochs call.
Still needs a test.

@dengemann do you agree that with the chance in line 944 because

we don't return the created sources anyways. And there is no kwarg on the input side.
?

Current behavior is actually to completely ignore the ch_name param ...

@jona-sassenhagen still urgent?

Nope, not urgent.

@jona-sassenhagen what's our status on this one?

I'm not sure how to do a good test - the ICA object we use for tests only has one channel, so we can't really check if a call picks the correct one, only if it runs without errors ...

Current coverage is 85.48% (diff: 75.00%)

Merging #3230 into master will decrease coverage by 0.10%

@@             master      #3230   diff @@
==========================================
  Files           347        347
  Lines         61881      61887     +6
  Methods           0          0
  Messages          0          0
  Branches       9503       9491    -12
==========================================
- Hits          52962      52902    -60
- Misses         6254       6326    +72
+ Partials       2665       2659     -6

Powered by Codecov. Last update ab42e6a...c7c57f2

then update what's new and set PR to MRG
thx @jona-sassenhagen

@dengemann would be cool if you could take a look before I set something stupid to merge ...

But setting it to merge only to find you've done something stupid is half
the fun!

@dengemann do you have time to check this one?

Not right now unfortunately.
On Tue, Feb 14, 2017 at 3:39 PM Eric Larson<EMAIL_ADDRESS>wrote:

@dengemann https://github.com/dengemann do you have time to check this
one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/3230#issuecomment-279724568,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fih6acPwBuLjBA3BotsgdvcDhh86rks5rcbyugaJpZM4Idg8Y
.

@mmagnuski can you look?

For me it looks good - but I've never actually used ica.find_bads_ecg so I just read through the code.

Thanks @mmagnuski.
@jona-sassenhagen ready for merge from your end? If so I can try to look more seriously in the next day or two

I can have a look soon and try to help merging + fixing this beyond this PR.

Okay. Not a release blocker though, right?

No.

rebased. Should be good to go.

LGTM +1 for merge once the CIs are happy

CIs are green ... maybe @dengemann can take a look, otherwise let's finish this up.

Will look and merge once back on keyboard
On Fri, Mar 31, 2017 at 9:01 PM jona-sassenhagen<EMAIL_ADDRESS>wrote:

CIs are green ... maybe @dengemann https://github.com/dengemann can
take a look, otherwise let's finish this up.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/3230#issuecomment-290881717,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fisbHaPwcgD0deLGfxP0up-b3wLnNks5rraHggaJpZM4Idg8Y
.

ping @dengemann :)

Thank you very much Jona!

Thanks guys!


Improvement idea for iptables DNS forwarding

Hello,
Thank you for your work!
I was thinking about iptables DNS forwarding and I realized an improvement could be brought to it. I like to use iptables for this because it is a minimally invasive and yet simple solution.
For now, iptables rules in the consul documentation, implemented in this role, are not super useful because they require local applications to explicitly query the local consul DNS since anything running on the machine relies on resolv.conf and, most likely, it has already been set through DHCP with recursors put at disposal by your cloud provider.
It would be better with transparent forwarding while still using provided recursors. To do this, we simply need to populate consul recursors with the right value and make sure only DNS traffic that is not related to the consul user is redirected to consul.
The former is trivial since the variable is already here and we just need to set it to ansible_dns.nameservers in the playbook, the latter can be implemented by taking advantage of the iptables owner module, with something like this:
[...] -m owner ! --uid-owner consul --dport 53 -j REDIRECT --to-ports 8600

Did I miss something or it sounds like it could be of some use here ?

I think this would be a great change!


🚦 Add Seedbank to Project

Proposed Changes ⏰

Install Seedbank Gem 🏀
Organize the existing seed data 🎈

Merge Checklist 📜

[x] Add specs that demonstrate bug / test a new feature.
[x] Check if route, query, or mutation authorization looks correct.

Add tests for authorization, if required.

[x] Update docs, where applicable.
[x] Add development seeds for new tables. 🥇

Closes #19 📌

@shrihari689 could you add a note on how to organize seed on our wiki?


feat:[CDS-98038]: Added Execute on Delegate for Git Connector

Describe your changes
Comment Triggers

PR Check triggers

Build: trigger build
Sub Category Field Check: trigger subcategoryfieldcheck
gitleaks: trigger gitleaks

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


Push notifications "send" needs to reflect new API key strategy

Right now it just asks for "API Key", but we have a public/private keypair now, so it's not obvious which is being asked for:

This issue has been resolved with commit 009f6305a991.


Stupid modification

To test my personal Jenkins

test this please

test this please

test this please

test this please

test this please

test this please

test this please

test this please

test this please

test this please

test this please

test this please


Unable to use until(Runnable) with Clousure if awaitility-groovy is on classpath

With awaitility-groovy module on a classpath it is not possible to use unitl(Runnable) with Groovy Closure, e.g.
await().until({ assertThat(foo).isEqualTo(bar) })
//or
await().until({ wiremock.verifyThat(postRequestedFor(urlEqualTo("...")) }

Where verifyThat throws AssertError if condition is not fulfilled (the same as AssertJ). This is caused by the fact that AwaitilityExtensionModule treats all Clusures as Runnable:
    if (runnable instanceof Callable)
      self.until(runnable as Callable)
    else
      self.until(new AssertionCondition(runnable, self.generateConditionSettings()))

In general it is good, because Groovy by default would use until(Runnable) instead of until(Callable). Nevertheless the use case mentioned above is broken.
Unfortunately I don't see a good way to fix it. All ideas I have are far from being perfect:

Use new Runnable() { void run() { ... } } explicitly.
Add marker interface AwaitilityGroovySupplier to mark Closures which should be treated as Runnable and use it in AwaitilityExtensionModule
Add another method with different name just for Groovy users
Document this limitation in until(Runnable) JavaDoc to give people a chance to get know about that.

Maybe you will have some other ideas.
One more thing. Probably not much people use aforementioned path as AssertJ with Groovy is usually not needed and Wiremock is also only for specific scenarios. In addition people using Spock can as an alternative to Awaitility use PollingCondition from Spock. In that case maybe the 4rd point would be enough?

Thanks for your detailed description and your insights, I didn't know about it and I have to think about it a bit. If you get any new ideas on how to solve it (other than documentation) please share it.

It's the same problem when using Kotlin.. Not sure what the best option is to resolve this.

I'm thinking of renaming the Runnable method to untilAsserted. Do you have any thoughts on that?

I would be very grateful if you could try out the newly released 3.0.0-rc1 version and give me some feedback of the changes (especially the untilAsserted change).
Another idea is to create a stand-alone API for Groovy like @iggymoran is suggesting for Kotlin here. Wdyt?

I've missed your questions :(. untilAsserted() sounds good to me to avoid confusion.


Headline not fully visbile (for large screens?)

Windows 10. Auflösung Bildschirm: 2560 x 1440

Not bug, you need to scroll down.


Alternatives to SwipeableDrawer

We're using MUI's SwipeableDrawer at the moment, but we've had to hack it a bit to make the contents interactable (buttons, form fields, et cetera) at the time of writing it also has a few unfixed bugs that might cause us headaches in production.
Have another look at these issues and consider the below options.
How can I achieve this gesture behavior using vanilla JS, HTML, and CSS?
https://reactjsexample.com/tag/bottom-sheet/
Historical stuff on the React Spring attempt
https://github.com/stipsan/react-spring-bottom-sheet/blob/main/GET_STARTED.md#get-started
https://github.com/stipsan/react-spring-bottom-sheet/issues/210
https://github.com/stipsan/react-spring-bottom-sheet/blob/main/src/BottomSheet.tsx
AB#146

I´m trying with this:
https://www.npmjs.com/package/react-modal-sheet


Change configuration API response to dictionary

Description
When https://github.com/dapr/dapr/pull/4695 merged, I will test the PR, then it should be reviewed by someone.
the PR can be merged after that https://github.com/dapr/dapr/pull/4695 is merged.
Issue reference
Please reference the issue this PR will close: #411

Thanks for this PR. We will still need to wait for the dapr/dapr PR to be completed and at that time we will need to update the protos before we can merge this.

That's OK， I will update codes.

I manually verified this change in its current form against a custom build of dapr/dapr

@hunter007 please do not update this PR anymore. I will manage this PR from here.

@hunter007 please do not update this PR anymore. I will manage this PR from here.

okay

/ok-to-test

/ok-to-test

/ok-to-test


Allow barrier kissing_gate with bicycle=yes for all bicycle profiles,…

… see discussion in #407.
I additionally moved the turn_stile from the list of absolute barriers to the potentials.

Thanks!

Would you mind updating this PR due to the merged changes in #2345?

Commit 96c0e1b contains all changes I had in mind. Please check. With this change the result of the discussion in #407 should be implemented.

LGTM - thanks!

Could you please finally close #407?

I would wait until it is deployed - should happen in the next 2 days.


TypeError: Cannot read property 'annotations' of undefined

f1ef6a8b903aad00a2efb607c09ec716ef07b27f  error:
Uncaught (in promise) TypeError: Cannot read property 'annotations' of undefined {stack: (...), message: "Cannot read property 'annotations' of undefined"}
a5f4882ece790b420876e3daae2687405145d8f4  work fine

sorry,this is bug report for es6-shim not quickstart

es6-shim.js:1 Uncaught RangeError: Maximum call stack size exceeded
<!doctype html>

I

Same case for me while trying to build a todoApp.
TypeError: Cannot read property 'annotations' of undefined

TypeError: Cannot read property 'annotations' of undefined
    at ReflectionCapabilities.execute.$__export.annotations (http://localhost:8080/angular2/src/reflection/reflection_capabilities.js:83:40)
    at Reflector.execute.$__export.annotations (http://localhost:8080/angular2/src/reflection/reflector.js:81:50)
    at DirectiveMetadataReader.execute.$__export.read (http://localhost:8080/angular2/src/core/compiler/directive_metadata_reader.js:33:41)
    at http://localhost:8080/angular2/src/core/compiler/compiler.js:127:35
    at Array.map (native)
    at Function.execute.$__export.map (http://localhost:8080/angular2/src/facade/collection.js:178:26)
    at Compiler.execute.$__export.createSteps (http://localhost:8080/angular2/src/core/compiler/compiler.js:126:43)
    at Compiler.execute.$__export._compileTemplate (http://localhost:8080/angular2/src/core/compiler/compiler.js:164:53)
    at http://localhost:8080/angular2/src/core/compiler/compiler.js:154:29
    at Zone.run (http://localhost:8080/dist/es6-shim.js:29891:19)
application.js:109 Uncaught (in promise) TypeError: Cannot read property 'annotations' of undefined {stack: (...), message: "Cannot read property 'annotations' of undefined"}

I ran into the same issue.

TypeError: Cannot read property 'annotations' of undefined
TypeError: Cannot read property 'annotations' of undefined
at ReflectionCapabilities.execute.$__export.annotations (http://localhost:3456/quickstart/angular2/src/reflection/reflection_capabilities.js:81:40)
at Reflector.execute.$__export.annotations (http://localhost:3456/quickstart/angular2/src/reflection/reflector.js:81:50)
at DirectiveMetadataReader.execute.$__export.read (http://localhost:3456/quickstart/angular2/src/core/compiler/directive_metadata_reader.js:31:41)
at http://localhost:3456/quickstart/angular2/src/core/compiler/compiler.js:127:35
at Array.map (native)
at Function.execute.$__export.map (http://localhost:3456/quickstart/angular2/src/facade/collection.js:172:26)
at Compiler.execute.$__export.createSteps (http://localhost:3456/quickstart/angular2/src/core/compiler/compiler.js:126:43)
at Compiler.execute.$__export._compileTemplate (http://localhost:3456/quickstart/angular2/src/core/compiler/compiler.js:164:53)
at http://localhost:3456/quickstart/angular2/src/core/compiler/compiler.js:154:29
at Zone.run (http://localhost:3456/quickstart/dist/es6-shim.js:29889:19)
application.js:255 Uncaught (in promise) TypeError: Cannot read property 'annotations' of undefined {stack: (...), message: "Cannot read property 'annotations' of undefined"}message: "Cannot read property 'annotations' of undefined"stack: (...)get stack: function () { [native code] }set stack: function () { [native code] }proto: Error

same here: todoApp

That's probably because an error in a View's directives, there is a PR pending on angular.
see https://github.com/angular/angular/pull/1917

Same problem

Same here

+1

Closed. Obsolete.

@wardbell this should be reopened. The error is an uncaught exception that occurs in several situations. For me, it occurs when a route is defined without a component declaration. The main issue is that the error message is useless and difficult to trace. In my case, it indirectly results from the resolve.ts code:
_ => resolver.resolveComponent(<any>node.value.component).then(factory => {           node.value._resolvedComponentFactory = factory;
which cascades as an error in a wholly different call stack. The assignment above should be guarded by checking for a factory value and issuing the appropriate error. In addition, the route should be validated beforehand. Furthermore, there should be a separate task to ensure that the (forthcoming?) route documentation specifically states what is required.
This issue and its response are symptomatic of a pervasive cavalier disregard for defensive programming in the Angular development process. I appreciate how developers under pressure let issues like this slide, but technical leads and QA should know better. Code review, anyone? Unfortunately, defensive programming must be integral to the development process and is difficult to address after initial  functional goals are achieved.

@FredLoney the issue you are referring to should be reported in https://github.com/angular/angular rather than in the quickstart.
The quickstart repo aims to provide a barebones development environment and is not the place to report issues with Angular 2 itself.

Oops, I thought all Angular modules shared a common issues list.
Filed as Angular Issue #10168


Codesets: separate location argument out of the API

In current implementation of codesets, https://github.com/fuseml/fuseml-core/pull/8
"location is still showing in the REST API as a POST op parameter. I think the only way to avoid this is to leave it out of the goa design spec completely and add it manually only as a CLI command line parameter, given that the location only makes sense on the client side."

OK, so I gave up, at least for now.
If I let the location argument anywhere near the design files, it will end in the REST API definition.
When I try to modify just the client files to parse extra arguments explicitly, it does not seem to end well. It would probably work to add a new global argument, but that's just as ugly as not doing anything.
Trying to add location argument on the level of "codeset" command means it is parsed by generated code which only cares about the stuff defined in design areas. In other words, it seems I'd need to modify generated code for that which is a no go.

@jsuchome I haven't looked closely at the CLI implementation, so my point of view is mostly theoretical, but here it is anyway.
First of all, I'm pretty sure this change needs to go into the CLI main package. Not in the goa design, nor in the generated client nor the server code files. I'm aware that the CLI main package has also been initially generated from the goa design, but we can and should alter it now to suit our needs. It just has to be modified to accept an additional command line parameter to the codeset push command that doesn't come from the generated design.
However, that might not be possible without changing the generated code, because I see goa is generating not just the data structures and serialization/deserialization client logic, but also the CLI command line arguments every time it is invoked (in gen/http/cli and gen/grpc/cli. I think this is where the problem lies: goa should generate those only when the CLI code is first generated, as part of the main CLI package.
I suggest you make a copy of that gen/http/cli code into the main CLI package, alter it to suit our needs and use that instead of the generated one.

Exactly, the problem is not with cli main.go, but with (generated) gen/http/cli where the validation of arguments happen.
You solution might indeed work; I'm not sure it is worth it, but well, it could be the way.

Fixed by: https://github.com/fuseml/fuseml-core/pull/63


Update README docs for consistency

Added jsx and js highlighting to code snippets. Also formatted all code snippets for consistency and better indentation practices.
Plus, fixed a small bug in useCart code example: if itemsCount > 0 !== true, then the CartNumber component would not have returned anything, causing a React error. Component now returns null if condition is false.
These edits were made to aid any new user if their approach to learning is 'copy and pasting' code snippets.

👍


SRVKP-1479: R&D Get a pipeline running with kata containers

This change makes it possible to use kata as the default container runtime

/lgtm


[occm] Documentation for occm metrics

What this PR does / why we need it:
This PR documents the metrics of the openstack-cloud-controller-manager especially for the OpenStack API calls and loadbalancer reconciliation introduced within #1178.
Which issue this PR fixes(if applicable):
n/a
Special notes for reviewers:

n/a
Release note:

NONE

Sean Schneeweiss<EMAIL_ADDRESS>Daimler TSS GmbH, legal info/Impressum

/ok-to-test

/lgtm

/approve


Somehow the comments are not working propertly

struct abc {
a : int;

My Awesome Comment

if( a == 10){
b : int;
}
};

The parsing is giving error on with the if that was expecting a identifier, but if i remove the comment it works.

Seens to happen mostly for the last fields of the struct

Draft PR to fix this is on my repo at https://github.com/igitur/BeeSchema/pull/6


Option to change USER ID and GROUP ID ?

Hi
In the docker compose instructions, the user and group IDs often causes me issues when it comes to using mapped folders on the host. For a while now, I always create them using user UID 1005 and GID 1000. Is it possible for a change of UID/GID to be supported ?
      - ./logs:/app/logs # logs folder should exist and owned by user id 1000
      - ./tokens:/app/tokens # tokens folder should exist and owned by user id 1000

I concur, possibly     user: "${UID}:${GID}" on every container, or alternately user: "${PUID}:${PGID}" with a supplemental .env file

I am aware that linuxserver provides the settings as part of compose file, but doing that is not very straightforward. The script (or the container) runs as a non-root user (appuser) if you are concerned about security implications
RUN groupadd --gid 1000 appuser && useradd --uid 1000 --gid appuser --shell /bin/bash --create-home appuser && chown -R appuser:appuser /app
USER appuser

You can always use user: "${UID}:${GID}" in any compose file if you want to do what you want. So I am closing this issue in favor to that.

Thanks @arpanghosh8453 / @AskAlice
On a stackoverflow post some had suggested..
The docker run command can be passed a user and group (or uid / gid).

docker run --user 2000:2000 acme

Or, via compose, the user: attribute can be used.

compose.yml

services:
  acme:
    image: my-alpine:latest
    user: 2000:2000

In the case that ids are used, neither the user id or group id needs to exist in the container.

Is that a valid route too ?


Learningswitch OF Exception from InstructionApplyActions fixed

Fixed an exception caused by wrong OF version for flow

I think you're assuming that OpenFlow 1.0 isn't being used. This is definitely something that needs to be fixed though! We already have a utility function to properly set the actions depending on the OpenFlow version. It's used in ForwardingBase here:
https://github.com/floodlight/floodlight/blob/master/src/main/java/net/floodlightcontroller/routing/ForwardingBase.java#L251
If you use that function, it would check the OpenFlow version for you and either apply the actions (1.0) or apply the instruction (1.1+).
If you want to make that change and commit it, we can merge your pull request!

Look good, thanks @reime005!


WinUI3

Bare bones application running on WinUI3

This builds locally, so the issue is on the pipeline. Will attempt to configure this outside this pull request


Type safety with number of columns

The number of columns currently still isn't quite type-safe, because int is a reasonable limit, but only semi-reasonable if no structured dtype is used.
It might be easiest/best to change almost all size_t occurances to npy_intp, though.  This is mainly tedious review, right now only one compiler warning really indicates at how bad it is (which surprises me, but I guess int val = size_t_value doesn't warn).
One thing that should make things saner is that the "growing" logic will error before any actual size overflow can occur.  So we don't actually have to worry about overflows.

Compiler warnings are fixed on the NumPy branch, so this should be OK I think.


blocks for the builder job

Bro, why is there like 500 blocks 😭

inactive for a long time, merging...


[BOT RULES] DevCenter

Adding matching bot rules from code owner changes in #32341

/check-enforcer override


integrate error reporting with rollbar

https://rollbar.com/ provides a centralized place to monitor logs/alerts of your application.
Free trial for up to 5000 errors/month
setting it up is quite straightforward: https://github.com/rollbar/pyrollbar#django

We could hook into the error handling to send rollbar reports.
roolbar has support for using the tornado async library by configuring it like this: rollbar.init('POST_SERVER_ITEM_ACCESS_TOKEN', 'development', handler='tornado')
I'd suggest making this feature an "add in" via a extra config file to include/add to command line so that having a rollbar account doesn't become mandatory.


Snake_case on API Requests/Response

/retest

@Mergifyio refresh

@Mergifyio refresh

@Mergifyio refresh

@Mergifyio refresh

Will be replaced by https://github.com/RedHatInsights/edge-api/pull/416


transaction batching

Resolves #42.
I took out the memo field for now because we aren't using it in this repo and it requires special attention on batching when it is populated - txns that need it cannot be batched if the memo field is not unique because it is applied to every txn in the batch.
TODO:
[ ] decouple the burn from redemption so that we can batch redemptions
[ ] all the tests need to be fixed/added

One thing to check for would be how batching effects the response stream. I had to add that splitAttributes function to the transaction filtering events, but it might be needed across the board with an introduction like this.

One thing to check for would be how batching effects the response stream. I had to add that splitAttributes function to the transaction filtering events, but it might be needed across the board with an introduction like this.

still need to check this

One thing to check for would be how batching effects the response stream. I had to add that splitAttributes function to the transaction filtering events, but it might be needed across the board with an introduction like this.

okay so i think we're okay here because it only applies to parsing provenance.marker.v1.EventMarkerTransfer - everywhere else we parse transfers we're parsing the wasm events which come in separately
it does kind of open up question of why we would parse EventMarkerTransfer vs just wasm but I guess the wasm events don't count every EventMarkerTransfer as built


[javasrc2cpg] Add anonymous classes and captures for inner classes

This PR adds capture logic to nested and inner classes that mimics javac logic. In short, captured variables are represented as members in the capturing class and are passed as extra constructor arguments. In addition, an outerClass member/parameter is created with this from the outer class passed in, which can then be used to access captured members and methods. For example,
class Foo {
  int fooMember = 0;

  void foo(int x, int y) { ... }

  void definesClass(int capturedParameter) {
    class Bar {
      void bar() {
        foo(fooMember, capturedParameter);
      }
    }
    Bar b = new Bar();
    b.bar();
  }

is represented as
class Foo {
  int fooMember = 0;

  void foo(int x, int y) { ... }

  void definesClass(int capturedParameter) {
    class Bar {
      int capturedParameter;
      Foo outerClass;

      public Bar(Foo outerClass, int capturedParameter) {
        this.outerClass = outerClass;
        this.capturedParameter = capturedParameter;
      }

      void bar() {
        this.outerClass.foo(this.outerClass.fooMember, this.capturedParameter);
      }
    }
    Bar b = new Bar(this, capturedParameter);
    b.bar();
  }

Some TODOs that would follow this PR are:

To clean up the BindingTableAdapters. There are currently 3 (one for inner classes, one for regular classes and one for lambdas), but we probably only need one (a version of the inner class one). The best way to unify these would depend somewhat on the next point.
Maybe use this representation for captures in lambdas as well. It would be nice to not have duplicate code paths, but I'm not sure how well the open source dataflow engine would deal with this, given this representation's reliance on member initializations and accesses. @DavidBakerEffendi do you have any thoughts on this?
Clean up how javasrc deals with object creation expressions. The logic used currently is quite convoluted and it has been the single biggest issue while implementing this. It works in all the cases I've tested, but I would not be surprised if there is a case I missed.

This fixes dataflow tests in https://github.com/ShiftLeftSecurity/codescience/pull/7376

If this becomes "cannon" then we may want to document this as the official way to model lambdas in the CPG

I'm not sure if this would be a good general representation for lambdas since the way Java deals with them is somewhat convoluted, with lambdas being (at least conceptually) just syntactic sugar for a sort of anonymous class declaration. For languages that support "real" anonymous functions, this intermediate/implicit object might just be an extra layer of complexity, especially since lambdas in such languages can be called directly and not via the object they define. Then again, PHP does exactly this, where lambdas implicitly create a Closure object and this object can then be called directly, so such a representation could probably work in general.
Regardless of whether we choose this as a general representation, the functionality added here would be useful as a proof of concept to see how well dataflow tracks.

Yeah a good proof-of-concept, but also potentially a lot more to model (change x N frontends) than maybe a simpler change in the data-flow engine I suppose.


Add table oci release notes

Fix for issue #486
Integration test logs

Logs
itpod /workspace/steampipe-plugin-oci/oci-test (add-table-oci-release-notes) $ node ./tint.js oci_releasenote
Test names: [ 'tests/oci_releasenote' ]
No env file present for the current environment:  staging
 Falling back to .env config
No env file present for the current environment:  staging
SETUP: tests/oci_releasenote []
PRETEST: tests/oci_releasenote
TEST: tests/oci_releasenote
Running terraform
No changes. Your infrastructure matches the configuration.
Terraform has compared your real infrastructure against your configuration
and found no differences, so no changes are needed.
Apply complete! Resources: 0 added, 0 changed, 0 destroyed.
Running SQL query: test-list-query.sql
[
{
"release_date": "2023-03-02T00:00:00Z",
"title": "Speech service available in US Government Cloud with FedRAMP Authorization"
}
]
✔ PASSED
Running SQL query: test-not-found-query.sql
null
✔ PASSED
Running SQL query: test-turbot-query.sql
[
{
"release_date": "2023-03-02T00:00:00Z",
"title": "Speech service available in US Government Cloud with FedRAMP Authorization"
}
]
✔ PASSED
POSTTEST: tests/oci_releasenote
TEARDOWN: tests/oci_releasenote
SUMMARY:
1/1 passed.

Example query results

Results
gitpod /workspace/steampipe-plugin-oci/oci-test (add-table-oci-release-notes) $ steampipe query "select
>   title,
>   service,
>   all_services,
>   summary,
>   release_date
> from
>   oci_releasenote
> limit 10 "
| title                                                                       | service                                      | all_services                                              | summary                                                                                | release_date         |
+-----------------------------------------------------------------------------+----------------------------------------------+-----------------------------------------------------------+--------------------------------------------------------------------------------------------------------------------------------------------+----------------------+
| Data Safe: Advanced filtering using SCIM in audit and alert reports         | Data Safe                                    | ["Data Safe"]                                             | You can now add advanced filters to your audit or alerts reports.                                                                          | 2023-02-23T00:00:00Z |
| Data Safe: Support for Autonomous Database on Oracle Exadata Cloud@Customer | Data Safe                                    | ["Data Safe"]                                             | You can now register an Autonomous Database on Oracle Exadata Cloud@Customer as an Oracle Data Safe target database.                       | 2023-02-23T00:00:00Z |
| Data Flow Supports Arm-Based Compute Shapes                                 | Data Flow                                    | ["Data Flow"]                                             | You can now select Arm-based compute shapes for your drivers and executors with Data Flow. For more information, see the …                 | 2023-03-01T00:00:00Z |
| Data Safe: US Government Cloud regions are now supported                    | Data Safe                                    | ["Data Safe"]                                             | Oracle Data Safe is now supported in the US Gov West (Phoenix) and US Gov East (Ashburn) regions.                                          | 2023-02-23T00:00:00Z |
| Synthetic Monitoring Enhancements                                           | Application Performance Monitoring           | ["Application Performance Monitoring"]                    | A new release is available with the following Synthetic Monitoring features. You can now:                                                  | 2023-02-28T00:00:00Z |
| Speech service available in US Government Cloud with FedRAMP Authorization  | Oracle Cloud Infrastructure Government Cloud | ["Oracle Cloud Infrastructure Government Cloud","Speech"] | Speech is now available in the US Government Cloud with FedRAMP Authorization.For more information about this service in the …             | 2023-03-02T00:00:00Z |
| Custom Query Bars in Trace Explorer                                         | Application Performance Monitoring           | ["Application Performance Monitoring"]                    | You can create and save Custom Query Bars for later use, and edit saved query bars from the new Open …                                     | 2023-02-28T00:00:00Z |
| X9-based virtual machine GPU shapes for Compute instances                   | Compute                                      | ["Compute"]                                               | Two new Intel X9-based GPU shapes are now available for virtual machine instances.                                                         | 0001-01-01T00:00:00Z |
| Oracle Autonomous Linux 8 now available as a platform image                 | Compute                                      | ["Compute"]                                               | Oracle Autonomous Linux 8 is now available as a platform image. For more information, see Platform Images.                                 | 2023-02-23T00:00:00Z |
| APM: OpenTelemetry Native Protocol is Now Supported                         | Application Performance Monitoring           | ["Application Performance Monitoring"]                    | You can now configure OpenTelemetry data sources to upload trace and metric data using the native OpenTelemetry protocol (OTLP) directly … | 2023-02-28T00:00:00Z |
+-----------------------------------------------------------------------------+----------------------------------------------+-----------------------------------------------------------+--------------------------------------------------------------------------------------------------------------------------------------------+----------------------+

gitpod /workspace/steampipe-plugin-oci/oci-test (add-table-oci-release-notes) $ steampipe query "select

title,
service,
summary,
release_date
from
oci_releasenote
where
release_date  > current_date - interval '1' month;
"

| title                                                                                | service                                      | summary                                                                                | release_date         |
+--------------------------------------------------------------------------------------------------------------------------------------+----------------------------------------------+-----------------------------------------------------------------------------------------------------------------------------------------------------+----------------------+
| New Available Region and Additional Capabilities Introduced in Oracle Access Governance                                              | Access Governance                            | The following new capabilities are introduced in Oracle Access Governance:                                                                          | 2023-02-23T00:00:00Z |
| Data Flow Supports Arm-Based Compute Shapes                                                                                | Data Flow                                    | You can now select Arm-based compute shapes for your drivers and executors with Data Flow. For more information, see the …                          | 2023-03-01T00:00:00Z |
| Speech service available in US Government Cloud with FedRAMP Authorization                                                           | Oracle Cloud Infrastructure Government Cloud | Speech is now available in the US Government Cloud with FedRAMP Authorization.For more information about this service in the …                      | 2023-03-02T00:00:00Z |
| Custom Query Bars in Trace Explorer                                                                                | Application Performance Monitoring           | You can create and save Custom Query Bars for later use, and edit saved query bars from the new Open …                                              | 2023-02-28T00:00:00Z |
| Synthetic Monitoring Enhancements                                                                                | Application Performance Monitoring           | A new release is available with the following Synthetic Monitoring features. You can now:                                                           | 2023-02-28T00:00:00Z |
| Data Safe: US Government Cloud regions are now supported                                                                             | Data Safe                                    | Oracle Data Safe is now supported in the US Gov West (Phoenix) and US Gov East (Ashburn) regions.                                                   | 2023-02-23T00:00:00Z |
| Data Safe: Audit Insights                                                                                | Data Safe                                    | You can now use Audit Insights to view and analyze the audit volume of several items, such as target databases, schemas, and database users.        | 2023-02-23T00:00:00Z |
| Oracle Autonomous Linux 8 now available as a platform image                                                                          | Compute                                      | Oracle Autonomous Linux 8 is now available as a platform image. For more information, see Platform Images.                                          | 2023-02-23T00:00:00Z |
| Discover and Monitor External Database Systems in Database Management                                                                | Database Management                          | You can now discover an External Database System, which is an Oracle Database System located outside of Oracle Cloud Infrastructure, …              | 2023-02-23T00:00:00Z |
| APM: OpenTelemetry Native Protocol is Now Supported                                                                                | Application Performance Monitoring           | You can now configure OpenTelemetry data sources to upload trace and metric data using the native OpenTelemetry protocol (OTLP) directly …          | 2023-02-28T00:00:00Z |
| Data Safe: Advanced filtering using SCIM in audit and alert reports                                                                  | Data Safe                                    | You can now add advanced filters to your audit or alerts reports.                                                                                | 2023-02-23T00:00:00Z |
| Manage master encryption key wallets in OCI GoldenGate                                                                               | GoldenGate                                   | You can now manage master encryption key wallets in OCI GoldenGate. You can use master encryption keys to encrypt trail …                           | 2023-02-07T00:00:00Z |
| New Release for Data Integration (fixes)                                                                                | Data Integration                             | This release contains the following fixes.                                                                                | 2023-02-19T00:00:00Z |
| Data Safe: Support for Autonomous Database on Oracle Exadata Cloud@Customer                                                          | Data Safe                                    | You can now register an Autonomous Database on Oracle Exadata Cloud@Customer as an Oracle Data Safe target database.                                | 2023-02-23T00:00:00Z |
| File Storage Cross-Region Replication is Now Available in US Government Cloud                                                        | File Storage                                 | File System Replication is now available in the US Government Cloud. Cross-region replication provides protection from regional outages, aids …     | 2023-02-09T00:00:00Z |
| UI-based Discovery for PeopleSoft Resources                                                                                | Stack Monitoring                             | Users can now perform PeopleSoft resource discovery directly from the Stack Monitoring UI, greatly simplifying the discovery process.For more …     | 2023-02-09T00:00:00Z |
| Visualize Synthetic Monitor Runs in Network Data Explorer                                                                            | Application Performance Monitoring           | You can now visualize the end-to-end network data path of monitor runs on one or more vantage points in the …                                       | 2023-02-08T00:00:00Z |
| Web Application Firewall available in US Government Cloud with FedRAMP Authorization                                                 | Oracle Cloud Infrastructure Government Cloud | Web Application Firewall is now available in the US Government Cloud with FedRAMP Authorization.For more information about this service …           | 2023-02-06T00:00:00Z |
| Data Safe: New Activity Auditing reports                                                                                | Data Safe                                    | Additional Activity Auditing reports have been added to Oracle Data Safe.                                                                           | 2023-02-23T00:00:00Z |
| Oracle Database Autonomous Recovery Service is Now Available                                                                         | Oracle Cloud Infrastructure                  | Oracle Database Zero Data Loss Autonomous Recovery Service is a fully managed data protection service for Oracle databases running on …             | 2023-02-15T00:00:00Z |
| MySQL Database Service Support for MySQL Version 8.0.32                                                                              | MySQL Database                               | MySQL Database Service now supports MySQL 8.0.32. New MySQL DB systems are based on this latest MySQL database version. For …                       | 2023-02-08T00:00:00Z |
| Media Flow now supports SRT and JSON formats for creating subtitles                                                                  | Media Flow                                   | Oracle Cloud Infrastructure Media Flow now supports SRT and JSON formats for creating subtitles. For more information, see Media Flow …             | 2023-02-07T00:00:00Z |
| Artifact Registry available in US Government Cloud with FedRAMP Authorization                                                        | Artifact Registry                            | Artifact Registry is now available in the US Government Cloud with FedRAMP Authorization.For more information about this service in …               | 2023-02-13T00:00:00Z |
| Data Science notebook session JupyterLab interface enhancements                                                                      | Data Science                                 |                                                                                | 2023-02-22T00:00:00Z |
| New release for Data Catalog                                                                                | Data Catalog                                 | A new release for Data Catalog is now available. Apart from bug fixes and enhancements, you can now:                                                | 2023-02-08T00:00:00Z |
| Anomaly Detection now includes Univariate Anomaly Detection, Multivariate Anomaly Detection improvements, and Asynchronous Detection | Anomaly Detection                            | We've added support for detecting anomalies in univariate signals that allow you to detect different types of anomalies in univariate …             | 2023-02-22T00:00:00Z |
| New protection capabilities are available for use with in-region WAF                                                                 | Web Application Firewall                     | The Web Application Firewall (WAF) has been updated to include new WAF protection capabilities. These protection capabilities include rules for …   | 2023-02-10T00:00:00Z |
| Oracle Visual Builder Studio is now available                                                                                | Visual Builder Studio                        | Oracle Visual Builder Studio (VB Studio) is a robust application development platform that helps your team collaboratively develop, test, deploy, … | 2023-02-14T00:00:00Z |
| DevOps now supports validating the Helm chart                                                                                | DevOps                                       | DevOps now supports verifying the integrity of Helm chart before the deployment. If the verification fails, then the Helm chart is …                | 2023-02-07T00:00:00Z |
+--------------------------------------------------------------------------------------------------------------------------------------+----------------------------------------------+-----------------------------------------------------------------------------------------------------------------------------------------------------+----------------------+

gitpod /workspace/steampipe-plugin-oci/oci-test (add-table-oci-release-notes) $ steampipe query "select

title,
summary,
release_date,
all_services
from
oci_releasenote
where
all_services  ? 'Speech'"

| title                                                                      | summary                                                                                | release_date         | all_services                                              |
+----------------------------------------------------------------------------+--------------------------------------------------------------------------------------------------------------------------------+----------------------+-----------------------------------------------------------+
| Speech service available in US Government Cloud with FedRAMP Authorization | Speech is now available in the US Government Cloud with FedRAMP Authorization.For more information about this service in the … | 2023-03-02T00:00:00Z | ["Oracle Cloud Infrastructure Government Cloud","Speech"] |
| New Languages Supported for Speech                                         | The following new languages are supported with this release:                                                                   | 0001-01-01T00:00:00Z | ["Speech"]                                                |
| New features in Speech                                                     | You can now use these new features in Speech:                                                                                | 2022-04-28T00:00:00Z | ["Speech"]                                                |
| Speech service is now available                                            | You can use the Speech service to convert audio files to readable text that is stored in JSON format. Speech harnesses the …   | 2022-02-23T00:00:00Z | ["Speech"]                                                |
+----------------------------------------------------------------------------+--------------------------------------------------------------------------------------------------------------------------------+----------------------+-----------------------------------------------------------+

gitpod /workspace/steampipe-plugin-oci/oci-test (add-table-oci-release-notes) $ steampipe query "select

title,
summary,
release_date,
all_services
from
oci_releasenote
where
service  = 'Anomaly Detection'
"

| title                                                                                | summary                                                                                | release_date         | all_services                                                 |
+--------------------------------------------------------------------------------------------------------------------------------------+------------------------------------------------------------------------------------------------------------------------------------------------------------+----------------------+--------------------------------------------------------------+
| Anomaly Detection is now available                                                                                | The Anomaly Detection service helps you to identify anomalies in a multivariate dataset by taking advantage of the interrelationship among signals. This … | 2021-07-13T00:00:00Z | ["Anomaly Detection"]                                        |
| Addition of new regions                                                                                | Anomaly Detection is now available in the following regions:For more information about Anomaly …                                                           | 0001-01-01T00:00:00Z | ["Anomaly Detection"]                                        |
| New Services available in US Government Cloud with FedRAMP Authorization                                                             | The following services are now available in the US Government Cloud with FedRAMP Authorization:For more …                                                  | 0001-01-01T00:00:00Z | ["Anomaly Detection","Billing","DevOps","Anomaly Detection"] |
| Anomaly Detection now includes Univariate Anomaly Detection, Multivariate Anomaly Detection improvements, and Asynchronous Detection | We've added support for detecting anomalies in univariate signals that allow you to detect different types of anomalies in univariate …                    | 2023-02-22T00:00:00Z | ["Anomaly Detection"]                                        |
| Addition of new region                                                                                | Anomaly Detection is now available in the following region:For more information about Anomaly Detection and features …                                     | 2022-11-08T00:00:00Z | ["Anomaly Detection"]                                        |
| Anomaly Detection now has multivariate and univariate                                                                                | The Anomaly Detection service is a multivariate anomaly detection solution that generates a model by analyzing multiple timeseries signal data …           | 2021-12-16T00:00:00Z | ["Anomaly Detection"]                                        |
| Addition of new regions                                                                                | Anomaly Detection is now available in these regions:                                                                                | 0001-01-01T00:00:00Z | ["Anomaly Detection"]                                        |
| Required policy and addition of Johannesburg region                                                                                | A required policy is added as a prerequiste to using Anomaly Detection.Support for the Johannesburg region was added.You …                                 | 2022-01-28T00:00:00Z | ["Anomaly Detection"]                                        |
+--------------------------------------------------------------------------------------------------------------------------------------+------------------------------------------------------------------------------------------------------------------------------------------------------------+----------------------+--------------------------------------------------------------+

Hi @lucasjellema , thanks for opening this PR and also implementing the table in a clever way!
Are you able to share more on how you're looking to use this table? I'm not sure if this would help with your requirements, but I figured I would mention we have an RSS plugin that allows you to query RSS channel and item information for one or more feeds (we have some staged changes in https://github.com/turbot/steampipe-plugin-rss/tree/issue-24 to allow passing in feed links through the config).
We then use this plugin in our RSS Insights mod to create dashboards that allow us to create views like:  (source available in https://github.com/turbot/steampipe-mod-rss-insights/blob/initial-branch/dashboards/rss_reader.sp).
In this PR, I can see you're pulling out service info and allowing users to pass in a service qual, which is not possible out of the box with the rss_item column, but could be created through a control or dashboard with inputs.

This table does two things: it provides records for all release notes regarding OCI (since the beginning of time) as well as the subset of release notes for a particular service. Oracle provides an RSS feed for the first category only - and the contains only the most recent 50 items. In my implementation I make use of the web pages that Oracle publishes - one set of pages with all 3000+ notes and another set of pages that each cover all release notes for a particular service. Using the RSS plugin would only help with a fairly small portion of the data this table exposes.
I would like to use the this table myself to list/report on release notes - probably in a dashboard. For a generic overview, the 50 most recent notes are probably enough. For service specific overviews of the most recent changes, only drawing from the RSS feed definitely would not be enough (with over 30 services, most services do not appear more than once or even at all in the RSS feed).

Hey @lucasjellema , sorry for the long response! We've been going through open PRs in this repo, along with others, but have been moving through them a bit slower than we'd like.
For this PR at the moment, we don't have plans to accept it, as we'd like to actually create a plugin that returns scrapped data using the package you've used (or a similar one). If the plugin was designed correctly, we feel this approach would allow you to get the release note data you're getting in this table, but also be flexible enough to scrape other types of pages.
There is no concrete timing on the web scrapping plugin yet, but it is in our backlog. If you have any suggestions or use cases for the future plugin, please let us know!


Add thoroughly documented reconcile results.

Description of your changes
The intention behind these variables is to make the intent of each reconcile function's return value more obvious. It's difficult to determine elegant names for these variables, so we lean heavily on their godoc strings to explain their purposes and call out edge cases that will likely not be immediately obvious to Crossplane contributors.
Which issue is resolved by this Pull Request:
We don't have a specific issue for this, but it relates to https://github.com/crossplaneio/crossplane/pull/419 and https://github.com/crossplaneio/crossplane/issues/302.
Checklist:

[ ] Documentation has been updated, if necessary.
[ ] Pending release notes updated with breaking and/or notable changes, if necessary.
[ ] Upgrade from previous release is tested and upgrade user guide is updated, if necessary.
[ ] Code generation (make generate) has been run to update object specifications, if necessary.
[ ] CRD manifests generation (make manifests) has been run to update CRD manifests yaml file specifications, if necessary.
[ ] Update dependencies (make vendor) has been run to update Gopkg.lock file, if necessary
[ ] RBAC permissions in clusterrole.yaml have been updated to include new types, if necessary
[ ] All related commits have been squashed to improve readability.
[ ] Comments have been added or updated based on the standards set in CONTRIBUTING.md

Note that is was initially part of https://github.com/crossplaneio/crossplane/pull/420, which lead to some discussion around their value: https://github.com/crossplaneio/crossplane/pull/420#discussion_r279615406.

I don't feel strongly about this exact set of constants, or the exact names of these constants, and I do not expect these constants to be ideal for 100% of cases, but I do think these or something like them could improve the readability of our controllers in ~80% of cases.

I wonder whether we should stick to the "when this result should be used" naming scheme (which results in a lot of identical results with different names for convenience/readability), or perhaps try to come up with a smaller set of constants with more generic names and rely on the docstrings to explain what they're good for.

@ichekrygin @jbw976 - Coincidentally this seems to be an instance of the case we discussed in this morning's community meeting in which Sonar seems to be complaining about the package level coverage of the code I touched rather than just the diff. :(

Given we cannot reach a consensus I'm going to close this PR and move discussion to https://github.com/crossplaneio/crossplane/issues/447.


Extract prometheus metrics #1

Related to #255
This is the first part of larger refactoring.

move Dig to utils
move prettyPrint() to utils and rename to ToJSON()
cover ToJSON() by tests
move environment variable POD_NAMESPACE to depresolver
extend depresolver tests
extend depresolver validators for namespace validator
extract Prometheus metrics into standalone module
extend Prometheus metrics with Register(),Unregister()
remove controllers/metrics.go
fix controller tests
remove dead registry/k3s.yaml file (pushed by mistake)

TODO:

dnsupdate refactor, to provider pattern, will come in the next PR
finally, some utils will go into gopkg
IMetrics introduction (should be implemented with #124)

Look good, but not sure if ToJSON is properly reflecting semantics... Originally the function was created just for readable struct printing...

I see, but function accepts interface so can be used to any type. Of course I can limit to structures only, and return error if passed argument is not structure.
Do you think rename may help ? What about ToString(a interface{}) string ?
str{"Foo", 007} =>

{
	"Name": "Foo",
	"Value": 7
}

123 => "123"
true => "true"
nil => "null"  // marshalling implicitly returns this, but can be changed.

Look good, but not sure if ToJSON is properly reflecting semantics... Originally the function was created just for readable struct printing...

I see, but function accepts interface so can be used to any type. Of course I can limit to structures only, and return error if passed argument is not structure.
Do you think rename may help ? What about ToString(a interface{}) string ?
str{"Foo", 007} =>

{
	"Name": "Foo",
	"Value": 7
}

123 => "123"
true => "true"
nil => "null"  // marshalling implicitly returns this, but can be changed.

I'd probably separate Dig and prettyPrint refactoring to different PRs, as they are out of context of this PR

@somaritane, amended.

I'd probably separate Dig and prettyPrint refactoring to different PRs, as they are out of context of this PR

@somaritane, amended.


IPython.display.update_display interface failed since the second call

Environment data

VS Code version: 1.76.2 ee2b180d582a7f601fa6ecfdad8d9fd269ab1884 x64
Jupyter Extension version: v2023.2.1000592019
Python Extension version: v2023.4.1
OS and version: Windows10 Pro 21H2
Python and/or Anaconda version: Python 3.9.16/conda 4.12.0
Type of virtual environment used: conda
Jupyter server running: Local
IPython version: 8.11.0

Expected behaviour

Actual behaviour

Steps to reproduce:
Run this code in a jupyter notebook:
from IPython import display
import time

display.display(display.HTML('
A
'), display_id='1')
time.sleep(2)
display.update_display(display.HTML('
B
'), display_id='1')
time.sleep(2)
display.update_display(display.HTML('
C
'), display_id='1')

Jupyter Extension version: v2023.1.20010391206 gives the expected behaviour.
However, Jupyter Extension version v2023.2.1000592019 gives the wrong behaviour.
Please take a look, thanks!
Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

XXX

Please could you enable logging as follows:

Open settings & go into Jupyter->Logging
Change the value to verbose
Reload VS Code,
Attempt to repro the issue & then copy the output from the Jupyter output panel.

Hi @DonJayamanne, I seem to still run into this issue if the update call is made from a separate cell.
VS Code version: 1.77.3 704ed70d4fd1c6bd6342c436f1ede30d1cff4710 x64
Jupyter Extension version: v2023.3.1201040234
Cell 1:
from IPython.display import display
from time import sleep

dis = display('1', display_id=True)
sleep(1)
dis.update('2')

Cell 2:
dis.update('3')
sleep(1)
dis.update('4')

Expected output:
'4'

Actual output:
'3'

Furthermore, if there are several update calls made one after another, the output breaks as well.
from IPython.display import display

ref = display('1', display_id=True)
ref.update('2')
ref.update('3')

Expected output:
'3'

Actual output:

(empty)

Furthermore, if there are several update calls made one after

I cannot replicate this issue, please could you update the sample code.
As there's no code used to display the ref

Sorry, I may be misunderstanding, but there is ref = display('1', display_id=True) in the sample (which works as intended by itself).


Replace the method call from the deprecated method to the new one.

As the title. The deprecated method call is remained, because I forgot the replacing at #11. I would like to replace the deprecated method call to the new one.

@Fleker @atulep Could you review this pull request?

@Fleker Nick, thank you for the reviewing and merging!


fails to boot if noatime in rootflags.

Distro:Arch Linux
Version:booster-git
fails to boot if rootflags=noatime.
options section for systemd-boot is
 options root=/dev/sda2 rw rootflags=subvol=@,compress=zstd,noatime quiet nowatchdog video=DP-1:e drm.edid_firmware=DP-1:edid/edid.bin random.trust_cpu=on
If I remove noatime,It success to boot.and mounted with /dev/sda2 on / type btrfs (rw,noatime,compress=zstd:3,space_cache,subvolid=1113,subvol=/@) instead of relatime.
I think mount options is not changed since I use blank fstab.
devAdd: cannot detect block type message appears even sucseed to boot.

Thanks for the report.
It is indeed the issue with booster. It comes from the fact that mount function has 2 input flag parameters - integer mountflags and comma-separated list of strings options (see https://man.archlinux.org/man/mount.2). Some mount options needs to passed as mountflags. Booster needs to parse provided user options and separate mountflags from options.
For now the workaround is to drop noatime. It is passed to the mount() by default anyway.

Alright, this bug should be fixed now. Please install booster-git from AUR and let me know if you see any issues with it.