Prevent crash and wrong display of version alert

Using dispatch_get_global_queue results in YT app crashing. and since we're showing an alert,  we might as well use dispatch_get_main_queue that is meant to be used for UI updates.
The version comparison isn't right either, it evaluates to true when the current version is not lesser than the  MIN_YOUTUBE_VERSION, and shows the alert on yt 16.46.5

Oh. Thanks heaps.


WatchedCDXSource; replacement for #184.

As per #184, code for the new WatchedCDXSource:
<property name="resourceIndex">
    <bean class="org.archive.wayback.resourceindex.LocalResourceIndex">
        <property name="canonicalizer" ref="waybackCanonicalizer" />
        <property name="source">
            <bean class="org.archive.wayback.resourceindex.WatchedCDXSource">
                <property name="path" value="${wayback.basedir}/cdx-index/" />
            </bean>
        </property>
        <property name="maxRecords" value="10000" />
        <property name="dedupeRecords" value="true" />
    </bean>
</property>

Once all the configuration options have been added we should really document this properly on the wiki..

Agreed. I've added some initial documentation, linked from the Advanced configuration page. I'll update it when #219 is done.


olap tests waiting for janusgraph fix

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@284f425). Click here to learn what that means.
The diff coverage is 50%.

@@            Coverage Diff             @@
##             develop      #17   +/-   ##
==========================================
  Coverage           ?   87.24%
  Complexity         ?      147
==========================================
  Files              ?       11
  Lines              ?      580
  Branches           ?       54
==========================================
  Hits               ?      506
  Misses             ?       49
  Partials           ?       25

Impacted Files
Coverage Δ
Complexity Δ

...ka/janusgraph/aerospike/AerospikeStoreManager.java
88.61% <50%> (ø)
38 <0> (?)

Continue to review full report at Codecov.

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


[Dcmtk] Error using vcpkg with VS2022

Hi, I tried to use Dcmtk in visual studio 2022.
But theres some error in osconfig.h
I don't know what I have to do.
The error is in the line 1137.

I would guess you have to do the following: https://devblogs.microsoft.com/cppblog/msvc-now-correctly-reports-__cplusplus/

Thank you that was my problem.


Spring Cloud use case

Developer creates a (server-side) project using Spring Initializr.
While the developer has many options to choose from, we are going to focus on a distinct subset of features. The developer selects the following features:
Web
Actuator
Config
Eureka Discovery
JPA
Rest Repository

Developer adds configuration capabilities, reading application configuration through a ConfigMap
Developer extends application by externalizing configuration. During development of a service, this is done using an application.properties file exposed through a ConfigMap mounted volume.

Developer creates and deploys Eureka Server
Developer creates and Eureka Server (using @EnableEurekaServer) and provisions it using the fabric8 maven plugin during development

Developer creates business entities
Developer uses JPA and the Spring JPA Repository to build a data model, and then exposes those as restful endpoints using the Rest Repository API. The service is also registered with the Eureka Server

Developer provision Hystrix Dashboard in OpenShift

Developer creates client application Spring Initializr that invokes the service
Developer creates a client application that invokes the prior service. It looks up the service in the Eureka Registry (using DiscoveryClient). The client also uses Hystrix with a simple failover web page. The client also binds a stream to the Hystrix Dashboard. The client application is provisioned using the fabric8 maven plugin during development.

Remarks :

Note that a Spring developer may decide to develop applications completely outside of openshift, including running a standalone eureka server, etc, and later deploy resulting artifacts to OpenShift
I understand that I need to add something about S2I and/or the CI/CD pipeline, perhaps to simulate a more formal release process.
Also, I could use help with the fabric8 maven plugin. I created a simple Spring app (using initializer) and tried to run it in openshift. It deploys but never starts :-/ I really need to get some hands-on. Was trying to validate the above use case, but I think you can help with that (limited time).

Config

Which Spring Dependency do you refer here (Spring Cloud Config, Spring Configuration Processor, ...) ? @jclingan

To be discussed with John (@jclingan)

[ ] Web : https://spring.io/guides/gs/serving-web-content/
[ ] JPA : https://spring.io/guides/gs/accessing-data-jpa/

Spring Cloud is discussed through Trello Card - https://trello.com/c/dR1SBnWY/26-7-1-m-certify-spring-boot


When watch is triggered build.js.dev displays lots of errors.

Since I merged the pull request of #1099 I get the following type error messages from the build.js.dev task, when I have npm start running and make a change in the code which triggers the watch to re-execute the given tasks.
Could not find input file ....\src\client\src\client\app\+home\home.component.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "..../src/client/"
sources = ["src\\client\\app\\+home\\home.component.ts"]

(This errors shows up for basically every TS file it compiles.)
After a bit of debugging I found out this is happening because the projectFiles are not cached any more on the typed builds, see build.js.dev.ts#L30.
For now I have added the line projectFiles = projectFiles.pipe(plugins.cached()); to my typed builds as well such that the errors don't show up any more.
In my debugging I found out that on the first run of the npm start tasks there is no sourceRoot specified in the tsProject variable. However on every new 'watch execution' the tsProject contains a sourceRoot = APP_CLIENT entry, I believe this should actually be APP_BASE instead.
@brian428 any idea why this is happening? Can you replicate this behaviour?

Hmm...have you changed the glob for the source files that is passed into the TS compiler? Because if it's including all of your source files and it's not cached, I don't see how it could be "missing" a file?
The reason that gulp-cached was removed is because it was actually hiding errors. gulp-cached only passes changed files downstream, so the "typed" compile wasn't actually checking things correctly. For example, if you changed a class name, that file might still compile fine, but any other (currently unchanged) files that depended on the old class name would fail to trigger compile errors (because they're unchanged and aren't passed downstream for recompilation).
The new setup ensures that errors like that would now be caught. The new TYPED_COMPILE_INTERVAL config option is meant to help with compile speed. Take a look at the wiki content I added and let me know if using that helps at all.

Also not sure what you mean about APP_CLIENT or APP_BASE in this context. There's nothing in tsproject.ts or build.js.dev.ts that uses either of those values.

Thanks for your answer. I did take a look at the wiki, but sadly nothing really new for me there.
The reason I mentioned APP_CLIENT and APP_BASE is because if you look in the error of my initial post you can see sourceRoot ending in ../src/client (which is basically where APP_CLIENT is pointing at), and sources start at src\\client\\... (only missing the APP_BASE part).
It turns out, it might not have anything to do with that. I did find some new information, I only get this long list of error messages when I change a SASS file and not when I change a TS file. So it is somehow related to the SASS files and only when the watch gets triggered.
Tomorrow morning I will check if I can reproduce this with a clean checkout from the seed.

Well, I made a fresh clone from the seed, but sadly can't reproduce the problem then.
It is still weird that these errors show up when I change an SASS file and not TS file.

Could there be some difference between your larger project and the master seed? Something you changed or tweaked that makes it diverge from master? Beyond that, I guess just keep us posted. If you find the cause and it is actually due to the changes I made to the TS compile task, let me know.

I've kept the changes to the master seed as little as possible (in order to hopefully prevent these kind of differences). I made some minor changes here and there and extended it, but can't think of anything that would affect this.
Anyway, thanks for your help! For now I will just keep using the cached version, I can live with that. If I find the cause + solution in the future you will hear from me.

@brian428 I got the same issue @Sjiep have.
And found out the same thing:

In my debugging I found out that on the first run of the npm start tasks there is no sourceRoot specified in the tsProject variable.

Like @Sjiep, i could not reproduce with the original seed.
In my project, i have SASS enabled too, but have realized that those problem happen when i change some html file as well.

Yeah, still no idea how this could have anything to do with SASS. The glob used for the TS source only includes *.ts files (https://github.com/mgechev/angular2-seed/blob/master/tools/tasks/seed/build.js.dev.ts#L24), so this isn't even including HTML or SASS files.

@brian428, i guessing that the previouly task called by the gulp "build.html_css.ts" (in the watch task) is changing something that the projectFiles = projectFiles.pipe(plugins.cached()); is solving.
Still, have not figured out what it is.

Hmm, I don't see how. build.html_css.ts specifies only .html/.css/.scss extensions in it's globs.
If you specify a cache key in build.js.dev.ts, does that matter? e.g.:

if (typedBuildCounter < TYPED_COMPILE_INTERVAL) {
    isFullCompile = false;
    tsProject = makeTsProject({isolatedModules: true});
    projectFiles = projectFiles.pipe(plugins.cached('build-js-dev'));
    util.log('Performing typeless TypeScript compile.');
  } else {

I don't think it should, but the empty cache key is about the only thing I can think of that would affect anything.

To solve the issue, between lines 30 and 43 i put the following:
  let projectFiles = gulp.src(src).pipe(plugins.cached());
  let result: any;
  let isFullCompile = true;

  // Only do a typed build every X builds, otherwise do a typeless build to speed things up
  if (typedBuildCounter < TYPED_COMPILE_INTERVAL) {
    isFullCompile = false;
    tsProject = makeTsProject({isolatedModules: true});
    util.log('Performing typeless TypeScript compile.');
  } else {
    tsProject = makeTsProject();
    projectFiles = merge(typings, projectFiles);
  }

Basically is:

remove line 38 (build.js.dev.ts#L38)
and changing the line 30 (build.js.dev.ts#L30) to let projectFiles = gulp.src(src).pipe(plugins.cached());

This way the problem is solved. But i don't know if it has any side-effects.

Yeah the whole problem is that we don't want the typed compile to be cached, because gulp-cached only forwards changed files downstream. Meaning a change in one file could result errors in other (currently unchanged) files that are not caught because the unchanged files aren't recompiled.
What about putting it back the way it was, but removing the use of cached from the untyped build (so that cached is basically not used at all in build.js.dev.ts)?

@jvitor83 your change is what the code used to do before the pull request by @brian428 got merged.
For now I've done a similar thing to 'supress' the error message (strangely even though they are error message, the build just runs fine and does its thing, it's just very annoying that a billion error messages are displayed in the console on every file change).
Still am clueless why this is happening, as @brian428 mentioned, SASS has nothing to do with the TS compiler.

To be clear: does this only occur when using SASS?
Everyone who's tried things using a fresh pull of the seed says the problem doesn't manifest (even with SASS enabled, as far as I can tell), so it seems like it's got to be something specific to the projects that are seeing this problem. But what it could be I have no idea. It doesn't seem to make any sense. :-/

I indeed can't replicate the problem on a fresh pull of the seed with SASS enabled.
For me errors show up when the watch is running and I change a SASS file, not when I change a TS file. I'm not sure if this only happens when using SASS though. I've only used the seed in one project which has SASS enabled.

Does removing the use of gulp-cached from the isolated module compile (so that gulp-cached isn't used at all in build.js.dev.ts), or adding a cache key to gulp-cache call in the isolated module compile (e.g. plugins.cached('build-js-dev')) affect things at all?

Does removing the use of gulp-cached from the isolated module compile (so that gulp-cached isn't used at all in build.js.dev.ts), or adding a cache key to the gulp-cache call in the isolated module compile (e.g. plugins.cached('build-js-dev')) affect things at all?

My TYPED_COMPILE_INTERVAL variable is 0 (default value) so without my edit of adding gulp-cached to the projectFiles pipe, it isn't using gulp-cached at all when the errors appear. Also regarding the key, my code doesn't execute the typedBuild if statement, so adding a cache key there shouldn't have any affect.

Understood, it was just an idea. I guess all we can hope for is a reproducible case then. Use of gulp-cached in the TS compile was going to come out anyway (for the reasons already mentioned), so it sounds like this would have affected you no matter what. But I'm still totally baffled as to how anything in the TS compile task could be affecting (or be affected by) the SASS task...

Not sure if this is the same, but I got similar error messages. Just found this issue because I noticed the "double" path \src\client\src\client  and searched for it.
It started to happen after adding a third party npm module and added it as a service in the AppModule.
I could reproduce it without all the hassle of my project, just adding the third party module and add it as service to AppModule.
Probably this could help to find the reason.

git clone https://github.com/mgechev/angular2-seed.git
cd .\angular2-seed
npm install
npm start -> no errors
touched file app.module.ts without changes -> reloaded without errors
killed npm process
npm install angular2-busy --save
modified app.module.ts (adding import of service and added to providers):

import { NgModule } from '@angular/core';
import { BrowserModule } from '@angular/platform-browser';
import { APP_BASE_HREF } from '@angular/common';
import { RouterModule } from '@angular/router';
import { HttpModule } from '@angular/http';
import { AppComponent } from './app.component';
import { routes } from './app.routes';

import { AboutModule } from './+about/about.module';
import { HomeModule } from './+home/home.module';
import { SharedModule } from './shared/shared.module';
import { BusyService } from 'angular2-busy';

@NgModule({
  imports: [BrowserModule, HttpModule, RouterModule.forRoot(routes), AboutModule, HomeModule, SharedModule.forRoot()],
  declarations: [AppComponent],
  providers: [{
    provide: APP_BASE_HREF,
    useValue: '<%= APP_BASE %>'
  },
  BusyService],
  bootstrap: [AppComponent]

})

export class AppModule { }

npm start -> no errors
touched file app.module.ts without changes -> reloaded with errors:

[13:33:29] Starting 'build.js.dev'...
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\shared\name-list\name-list.service.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\shared\\name-list\\name-list.service.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\shared\name-list\index.ts. This is probably an issue of gulp-typescript.

Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\shared\\name-list\\index.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\shared\navbar\navbar.component.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\shared\\navbar\\navbar.component.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\shared\navbar\index.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\shared\\navbar\\index.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\shared\toolbar\toolbar.component.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\shared\\toolbar\\toolbar.component.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\shared\toolbar\index.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\shared\\toolbar\\index.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\shared\config\env.config.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\shared\\config\\env.config.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\shared\index.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\shared\\index.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\app.component.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\app.component.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\+about\about.component.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\+about\\about.component.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\+about\about.routes.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\+about\\about.routes.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\+about\index.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\+about\\index.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\+home\home.component.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\+home\\home.component.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\+home\home.routes.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\+home\\home.routes.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\+home\index.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\+home\\index.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\app.routes.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\app.routes.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\+about\about.module.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\+about\\about.module.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\shared\shared.module.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\shared\\shared.module.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\+home\home.module.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\+home\\home.module.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\node_modules\angular2-busy\index.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["node_modules\\angular2-busy\\index.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\app.module.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\app.module.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\hot_loader_main.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\hot_loader_main.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\app\main.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\app\\main.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\testing\router\mock-location-strategy.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\testing\\router\\mock-location-strategy.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\testing\router\router-testing-providers.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\testing\\router\\router-testing-providers.ts"]
Could not find input file C:\Data\_git\irt-web\test\angular2-seed\src\client\src\client\________________empty.ts. This is probably an issue of gulp-typescript.
Please report it at https://github.com/ivogabe/gulp-typescript/issues
Debug information:
sourceRoot = "C:/Data/_git/irt-web/test/angular2-seed/src/client/"
sources = ["src\\client\\________________empty.ts"]
[13:33:29] Finished 'build.js.dev' after 80 ms

Sounds like a separate issue. Check out https://github.com/mgechev/angular2-seed/issues/1157 and https://github.com/ivogabe/gulp-typescript/issues/307.

@brian428 Recently I merged the latest changes from the seed into my project.
Out of curiosity I removed my line that cached the project files on a typed build to see if the warning/error messages still appeared. And behold, they didn't show up any more!

Great, sounds like this can be closed then. Given what you said, it sounds like the issue was likely with one of the dependent libraries rather than the seed itself.

I am having this issue after angular2-seed update rc6.
But only happen when I change html file.
When I change something in ts file, it's still working.


Please Help! active_storage and multiple_active_storage, how shall I show filename?

Hello,
I would like to know how to show filename when multiple_active_storage.

It works for has_one_attached, but can't find the way to show the filename when multiple_active_storage.
Thanks in advance.

Hi. Did you tries this?

Try this code in your edit section of model:
field :my_attribute, :active_storage do
  pretty_value do
    if value
      tags = value.blobs.map do |blob|
        path = Rails.application.routes.url_helpers.rails_blob_path(blob, only_path: true)
        bindings[:view].content_tag(:div,
          bindings[:view].content_tag(:a, blob.filename, href: path)
        )
      end
      tags.join.html_safe
    end
  end
end

thanks aired I tryied that way with no luck, even I tried to do it via partial, where the name is shown but no luck to delete file :-(. this is a multiple_active_storage field :-(.
The problem with my partial is routing the delete method because I do not know where to handle the controller action for that route:
here is the partial:

<% if @object.obligatorio.attached? %>
	<% @object.obligatorio.map do |archivo| %>

		      <%= link_to archivo.filename, Rails.application.routes.url_helpers.rails_blob_path(archivo, only_path: true), target: '_blank' %>

<%= link_to 'Eliminar Archivo', estandar_etapa_certificacion_delete_obligatorio_path(:todo_id => @object.id, :upload_id => archivo.id),
          method: :delete, data: { confirm: '¿Confirma Eliminar el Archivo?' } %>
	   </div>

	</div>
<% end %>

<% end %>

<%= form.file_field :obligatorio, multiple: true %>

I did not ry partial but I tried my code above and it worked well. Can you provide your rails_admin.rb config to show me how you use it?

field :obligatorio do
partial 'documentos'
end

Can you try to use my code above in rails_admin.rb?

I did,
field :obligatorio, :active_storage do         pretty_value do           if value             tags = value.blobs.map do |blob|               path = Rails.application.routes.url_helpers.rails_blob_path(blob, only_path: true)               bindings[:view].content_tag(:div, bindings[:view].content_tag(:a, blob.filename, href: path))             end           tags.join.html_safe           end         end       end
and I got this error message:
undefined method blobs' for #ActiveStorage::Attached::Many:0x00007f85344cb1c8`
in line:
tags = value.blobs.map do |blob|

when I change first line in rails_admin.rb to:
field :obligatorio, :multiple_active_storage
it run but do not show filename

@erniux I think, I found the solution.
In model:
has_many_attached :my_attrbute

Rails_admin.rb:
field :my_attrbute, :multiple_active_storage do
  partial 'my_partial'
end

app/views/rails_admin/main/_my_partial.html.haml:
- field.attachments.each_with_index do |attachment, i|
  .toggle{class: field.reorderable? ? 'sortables' : ''}
    = link_to attachment.value.blob.filename, Rails.application.routes.url_helpers.rails_blob_path(attachment.value.blob, only_path: true), target: '_blank'
    %a.btn.btn-info.btn-remove-image{href: '#', :'data-toggle' => 'button', role: 'button'}
      %i.icon-white.icon-trash
      = I18n.t('admin.actions.delete.menu').capitalize + " #{field.label.downcase} ##{i + 1}"
    = form.check_box("my_attrbute", {multiple:true, checked: true, style: 'display:none;'}, attachment.value.blob.signed_id, nil)
= form.file_field(field.name, field.html_attributes.reverse_merge({ data: { :"multiple-fileupload" => true }, multiple: true }))

Replace all my_attrbute to your value.

Thank you, It looks fine, but, it is not deleting the files, I have some problems with this line:
.toggle{class: field.reorderable? ? 'sortables' : ''}

Your code does not work for me. You just copy partial from rails_admin gem. It expects either field.delete_method or field.keep_method. They both return nil so delete buttons does not showing. My version of gem is gem 'rails_admin', '~> 2.0', git: 'https://github.com/sferik/rails_admin.git'.
I have just tested my code one more time and it works absolutely fine. Strange, that it does not work for you. What is your gem version? Anyway, glad it works now.

yes you help me to set the name, because the partial doesn't show the filename, so I change third line with your solution. And really appreciate your time to help me. Very valuable.


Make tls support optional and add rustls

Tls support was recently added by forcing the usage of openssl via hyper-tls. It means that either this lib is installed or it needs to be compiled. For users unconcerned with tls, we could remove the dependency. Alternatively, rustls provides a pure rust tls implementation.

I am also facing an issue because of the added dependency on hyper-tls.  I have to cross compile a project to a statically linked binary to be able to run on aarch64-unknown-linux-gnu, which is currently impossible to do with the dependency on hyper-tls added in version 0.13.0 of metrics-exporter-prometheus.
Please switch either to rustls or provide a way to opt out of HTTPS (which would be my preference since I don't need a secure connection for my use case).

Are there any plans on moving forward with @nstinus' suggestions? On my end, either switching to rustls and removing hyper-tls outright would be acceptable.

I left feedback on their PR, which hasn't (yet?) been incorporated. I don't have the time/need to modularize TLS support, so this is waiting on someone to submit a PR doing so, or an update to the aforementioned PR based on my feedback.

@tobz if you are ok with switching to rustls, I can create a PR for that. Tomorrow that is. It's way past my bedtime. ;-)


Hook for debugging shared libraries in Visual Studio

Debugging shared libraries in Visual Studio without its original source wasn't possible before.
This post-package hook solves it by copying the .pdb (symbol files) needed for debugging to the package folder so that Visual can find them.
It searches for the corresponding .pdb for every .dll and copies it next to it so that Visual can find them.
It searches for every .dll and copies it's corresponging .pdb next to it so that Visual can find it.
This features isn't enabled by default because .pdb files are heavy compared to the rest of the files in the package.
Test in progress

 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.

I like the approach of the hook, we just need to finally come up with an idea to not enable this by default - Separate repo?

@RubenRBS  We chose to have the hook name starting with _hook_... by default so users will need to rename it to use the hook.
This works because hooks only run if their name starts with hook_...
As we discussed we don't want this hook running by default as it greatly increases the size of packages.


Enable maven central sync

I'll sync up 4.4.2 manually.

NetflixOSS » denominator » denominator-pull-requests #77 SUCCESS
This pull request looks good

thx. picked into 4.x nonetheless


[Scenario] Countdown to Devastation

What kind of mission do you want to add (mark with an X, e.g. [X] Descent)?

[X] Mansions of Madness
[ ] Descent

What expansions does the scenario require?

[ ] Conversion Kit (CK)
[ ] First Edition (MoM1E) - Also Recurring Nightmares
[ ] Fatal Alchemy (FA) - Also Suppressed Memories
[ ] Call of the Wild (CotW) - Also Suppressed Memories
[ ] Beyond the Threshold (BtT)
[ ] Streets of Arkham (SoA)
[ ] Sunctum of Twilight (SoT)
[ ] Horrific Journeys (HJ)
[ ] Path of the Serpent (PotS)

What expansions can the scenario optionally use?

[ ] Conversion Kit (CK)
[ ] First Edition (MoM1E) - Also Recurring Nightmares
[ ] Fatal Alchemy (FA) - Also Suppressed Memories
[ ] Call of the Wild (CotW) - Also Suppressed Memories
[ ] Beyond the Threshold (BtT)
[ ] Streets of Arkham (SoA)
[ ] Sunctum of Twilight (SoT)
[ ] Horrific Journeys (HJ)
[ ] Path of the Serpent (PotS)

Please enter the GitHub Url to your scenario here
https://github.com/vidimus/DomeAcolyte/tree/main/Lost_chronicles_08

Added to the manifest.ini
@antontimmermans , would you mind to add it to the statistics?

added to statistics


Instance storage reported incorrectly for all but 'm' type instances

I remember instance storage being reported correctly, but not it doesn't seem to be the case.
If you filter by instance storage (add a minimum of 1gb), all you get are the 'm' type instances, which isn't correct.

Indeed, it appears the webpage we rely on for this info has been updated to split the instances into different tables instead of one large table. Will get this fixed https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/InstanceStorage.html
https://github.com/vantage-sh/ec2instances.info/blob/master/scrape.py#L462


#31 10/25 features that sponsor wanted

Thoroughly tested, both features working as intended

Closes #37


Command line arguments to app when testing

I'd like to invoke a switch in the native part of my app when testing, in this case to mock audio input. I've been looking for a way to pass a command line argument to the app but not found anything - is there a way to do this with Detox? If not I'm happy to create a PR.

Looking at the code, my proposal would be to add command line arguments here
https://github.com/adamski/detox/blob/master/detox/src/devices/DeviceLoader.js#L22
e.g.
await fbsimctl.launch(simulatorUdid, bundleId, [{"-test", "true"}])

We've been discussing this internally for a while. In general, I don't like people having if(testing) { ... } else { ... } code in their apps, be it native or JS. Just doesn't look like a good idea in long term code management, in addition to localized problems of enabling test code by mistake.
On the JS front, we are working on a solution that allows you to mock stuff at the "compiler" level - during packaging. I'd love to have a similar solution for native, where at detox build time, we would be able to perform injections. This has many challenges, such as Swift support, how to inject, etc. I have not given thought on how to approach a solution yet.

This feature will be released with Detox 5.5.0
https://github.com/wix/detox/blob/master/docs/APIRef.DeviceObjectAPI.md#6-additional-launch-arguments


Feature request: support renaming/dropping default tags

Describe the Feature
Allow changing the name of default tags, or dropping them entirely.
Expected Behavior

Ability to rename default tags, eg rename Stage to environment
Ability to remove default tags, eg prevent namespace tag from being created

Use Case
We use this module a lot, but some of the tag names don't match our internal standard. For example, we use "environment" rather than "stage". As a workaround, we set additional_tag_map = { environment = "dev" }, but this means we have a Stage tag which duplicates the value.
In AWS this is a problem since there is a limit of 10 tags per resource, and we add other tags globally for cost /security control.
Describe Ideal Solution
module "context" {
  ...
  stage_tag_name = "environment" # use 'environment' instead of 'stage'
  namespace_tag_name = "" # don't create a namespace tag at all
}

Additional Context
I suggested this idea in the CloudPosse slack, some feedback exists there: https://sweetops.slack.com/archives/CB6GHNLG0/p1621557927034000

I share the same problem/concern as Alex. We find this very opinionated and not aligned with our tagging strategy

It is true, as @ricbartm said, this module is opinionated: it reflects Cloud Posse's opinion about providing a consistent naming and tagging system. However, it is not as bad as @alexjurkiewicz seems to think.
It is not exactly clear what results @alexjurkiewicz wants, because this module does require all generated tag keys to have the same letter case pattern, either UPPER, lower, or Title. The proposal wants to replace Stage with environment. You can set label_key_case = "lower" to get all the tag keys to be all lower case, but, yes, our opinion is that they should all be formatted the same way, so this module does not provide the ability to keep the name tag as "Name" while having the environment tag as "environment', you either get "Name" and "Environment" or "name" and "environment".  We are not going to change that.
Otherwise, the changes @alexjurkiewicz wants are already available via the label_order variable. Setting
label_order = ["environment", "name", "attributes"]

will prevent the namespace and stage tags from being created (and prevent namespace and stage from appearing in the generated IDs) and will use the environment variable as the value of the tag named Environment.
It seems the customizations available via label_order and label_key_case are sufficient to handle enough use cases that I am reluctant to offer further customizations. @alexjurkiewicz If you agree they address your use case, please close this issue. If you disagree, please explain further why they are not enough for you.

Since opening this, we decided to migrate to AWS provider default_tags. Thanks for explaining the workarounds though! They are not very intuitive, but it's nice to know some customisation is possible.

Also, a couple of other notes:

In this particular case, we already support environment and all parameters are optional, so if stage is not desirable, omit it and use environment instead (I realize this was just an example and get your point)
With the context.tf pattern, any additional tags can be passed very easily through to all modules with more granular control than with the provider's default tags

@ricbartm can you provide an example of your tagging strategy?


Usage of citation.js in Browser with webpack fails

Usage of citation.js in Browser with webpack fails
This is the browser example:
const example = new Cite({title: 'Thing A'})
example.add([{title: 'Thing B'}, {title: 'Thing C'}])
console.log(example.data)
example.format('bibliography', {
format: 'html',
template: 'apa'
})
Error:
TypeError: Cannot read property 'system_id' of undefined
(anonymous function)
node_modules/@citation-js/plugin-csl/lib-mjs/engines.js:15
12 |
13 | const getWrapperProxy = original => function (state, entry) {
14 |   if (state.sys.wrapBibliographyEntry) {

15 |     let _state$sys$wrapBiblio = state.sys.wrapBibliographyEntry(this.system_id),
16 |         _state$sys$wrapBiblio2 = _slicedToArray(_state$sys$wrapBiblio, 2),
17 |         prefix = _state$sys$wrapBiblio2[0],
18 |         postfix = _state$sys$wrapBiblio2[1];

The issue has been discussed here as well:
https://github.com/parcel-bundler/parcel/issues/3372
Any idea how to solve this?

I'll take a look (next week, I'm not available right now). This shouldn't happen but I'm not sure if that's because of citeproc or not.

Okay, figured it out. Background: to add affixes to bibliography entries (like DOIs for Vancouver, Altmetric badges etc.), the internal formatting instructions of citeproc-js have to be amended. This is done by replacing the @bibliography/entry style by a proxy function, which computes and adds the affixes before calling the original @bibliography/entry styles.
Two things going wrong here:

The @bibliography/entry style is proxied twice, once for @citation-js/plugin-csl/lib-mjs/engine.js (required by @citation-js/plugin-csl) and once for @citation-js/plugin-csl/lib/engine.js (required by citation-js for backwards compatibility). This does not matter for NodeJS or the Browserify bundle since they don't use lib-mjs/.
The original @bibliography/entry style is called without passing along this, which is fine for the original style since it does not use this, but it does matter when the "original" of the outer proxy is actual the inner proxy (which does use this).

This should be fixed in v0.4.9.


Remove all Full Fx TFMs from SBRP

As part of https://github.com/dotnet/source-build/issues/3014, all net4* TFMs are being trimmed from source-build.  Once this work is completed, we expect a significant number of SBRPs will no longer be referenced and therefore will be removed.  Of the remaining SBRPs, we expect there will be some that target net4* and net core TFMs.  Because of this, we would still need to have the net4* targeting packs in SBRP which is not desirable.
In order to address this problem, the generate SBRP tooling will need to be updated to filter out the net4* TFMs.  We can then regenerate the existing packages that reference net4* TFMs in order to "clean them up".
Full Fx TFMs to remove

[x] net45 - https://github.com/dotnet/source-build-reference-packages/pull/598
[x] net451 - https://github.com/dotnet/source-build-reference-packages/pull/596
[x] net46
[x] net461
[x] net462
[x] net47
[x] net472

Congratulations - great milestone 👍


gcimporter: go archive is missing __.PKGDEF

This only happens on openSUSE packages, because I believe that OBS (the Open Build Service) is modifying the entries in fmt.a and similar archives.
% go version
go version go1.7 linux/amd64
% go env
GOARCH="amd64"
GOBIN="/home/cyphar/go/bin"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/cyphar/.local"
GORACE=""
GOROOT="/usr/lib64/go"
GOTOOLDIR="/usr/lib64/go/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build022377736=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
% rpm -qi go
Name        : go
Version     : 1.7.0
Release     : 1.1
Architecture: x86_64
Install Date: Sun 06 Nov 2016 11:22:21 PM AEDT
Group       : Development/Languages/Other
Size        : 199522326
License     : BSD-3-Clause
Signature   : RSA/SHA256, Sat 05 Nov 2016 08:00:32 AM AEDT, Key ID b88b2fd43dbdc284
Source RPM  : go-1.7.0-1.1.src.rpm
Build Date  : Sat 05 Nov 2016 07:57:13 AM AEDT
Build Host  : cloud127
Relocations : (not relocatable)
Packager    : http://bugs.opensuse.org
Vendor      : openSUSE
URL         : http://golang.org
Summary     : A compiled, garbage-collected, concurrent programming language
Description :
Go is an expressive, concurrent, garbage collected systems programming language
that is type safe and memory safe. It has pointers but no pointer arithmetic.
Go has fast builds, clean syntax, garbage collection, methods for any type, and
run-time reflection. It feels like a dynamic language but has the speed and
safety of a static language.
Distribution: openSUSE Tumbleweed

If you run a tool like go vet on any project you'll get errors:
What did you expect to see?
No errors.
What did you see instead?
% go vet github.com/cyphar/umoci/image/cas
vet: import failed: reading export data: /usr/lib64/go/pkg/linux_amd64/fmt.a: go archive is missing __.PKGDEF
vet: import failed: reading export data: /usr/lib64/go/pkg/linux_amd64/fmt.a: go archive is missing __.PKGDEF
exit status 1

I've figured out why this is happening by the way:
It's because (for some reason), the packaged version of the fmt.a and similar static archives have been "cleaned" in a way. You can see it if you modify the source of go/internal/gcimporter/exportdata.go:
// Copyright 2011 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.

// This file implements FindExportData.

package main

import (
	"bufio"
	"fmt"
	"io"
	"os"
	"path/filepath"
	"strconv"
	"strings"
)

func readGopackHeader(r *bufio.Reader) (name string, size int, err error) {
	// See $GOROOT/include/ar.h.
	hdr := make([]byte, 16+12+6+6+8+10+2)
	_, err = io.ReadFull(r, hdr)
	if err != nil {
		return
	}
	// leave for debugging
	if false {
		fmt.Printf("header: %s", hdr)
	}
	s := strings.TrimSpace(string(hdr[16+12+6+6+8:][:10]))
	size, err = strconv.Atoi(s)
	if err != nil || hdr[len(hdr)-2] != '`' || hdr[len(hdr)-1] != '\n' {
		err = fmt.Errorf("invalid archive header")
		return
	}
	name = strings.TrimSpace(string(hdr[:16]))
	return
}

// FindExportData positions the reader r at the beginning of the
// export data section of an underlying GC-created object/archive
// file by reading from it. The reader must be positioned at the
// start of the file before calling this function. The hdr result
// is the string before the export data, either "$$" or "$$B".
//
func FindExportData(r *bufio.Reader) (hdr string, err error) {
	// Read first line to make sure this is an object file.
	line, err := r.ReadSlice('\n')
	if err != nil {
		err = fmt.Errorf("can't find export data (%v)", err)
		return
	}

	if string(line) == "!<arch>\n" {
		// Archive file. Scan to __.PKGDEF.
		var name string
		if name, _, err = readGopackHeader(r); err != nil {
			return
		}

		// First entry should be __.PKGDEF.
		if name != "__.PKGDEF" {
			fmt.Printf("name=%s\n", name)
			err = fmt.Errorf("go archive is missing __.PKGDEF")
			return
		}

		// Read first line of __.PKGDEF data, so that line
		// is once again the first line of the input.
		if line, err = r.ReadSlice('\n'); err != nil {
			err = fmt.Errorf("can't find export data (%v)", err)
			return
		}
	}

	// Now at __.PKGDEF in archive or still at beginning of file.
	// Either way, line should begin with "go object ".
	if !strings.HasPrefix(string(line), "go object ") {
		err = fmt.Errorf("not a Go object file")
		return
	}

	// Skip over object header to export data.
	// Begins after first line starting with $$.
	for line[0] != '$' {
		if line, err = r.ReadSlice('\n'); err != nil {
			err = fmt.Errorf("can't find export data (%v)", err)
			return
		}
	}
	hdr = string(line)

	return
}

func main() {
	fh, err := os.Open("/usr/lib64/go/pkg/linux_amd64/fmt.a")
	if err != nil {
		panic(err)
	}
	defer fh.Close()

	bio := bufio.NewReader(fh)
	hdr, err := FindExportData(bio)
	if err != nil {
		panic(err)
	}

	fmt.Printf("hdr: %s\n", hdr)
}

On openSUSE you get the error:
% go run exportdata.go
name=__.PKGDEF/
panic: go archive is missing __.PKGDEF

goroutine 1 [running]:
panic(0x49c240, 0xc42000a540)
        /usr/lib64/go/src/runtime/panic.go:500 +0x1a1
main.main()
        /home/cyphar/tmp/archive/exportdata.go:105 +0x1b7
exit status 2

So I do have a fix, which is this patch:
From f254166f521a3dfa172eed0bb8c63b058462a86e Mon Sep 17 00:00:00 2001
From: Aleksa Sarai<EMAIL_ADDRESS>Date: Fri, 11 Nov 2016 15:00:35 +1100
Subject: [PATCH] gcimporter: clean header name when checking for __.PKGDEF

In some distributions, the build system will "sanitise" archives and do
silly things like append '/' to all directory entries. This caused all
tools that used "go/internal/gcimporter" to not work on such
distributions, even though the actual compiler was fully operational.

Fix this by cleaning the path before checking if it is equal to
"__.PKGDEF", which ensures that we are doing a proper lexical
comparison of paths.

Fixes: #17890
Signed-off-by: Aleksa Sarai<EMAIL_ADDRESS>---
 src/go/internal/gcimporter/exportdata.go | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/go/internal/gcimporter/exportdata.go b/src/go/internal/gcimporter/exportdata.go
index c12e459c3db1..0cd31fde9f23 100644
--- a/src/go/internal/gcimporter/exportdata.go
+++ b/src/go/internal/gcimporter/exportdata.go
@@ -10,6 +10,7 @@ import (
 	"bufio"
 	"fmt"
 	"io"
+	"path/filepath"
 	"strconv"
 	"strings"
 )
@@ -57,7 +58,7 @@ func FindExportData(r *bufio.Reader) (hdr string, err error) {
 		}

 		// First entry should be __.PKGDEF.
-		if name != "__.PKGDEF" {
+		if filepath.Clean(name) != "__.PKGDEF" {
 			err = fmt.Errorf("go archive is missing __.PKGDEF")
 			return
 		}
--
2.10.1

Can you comment if that patch is valid? Would you be interested in me making an actual merge request? Or should we just carry that patch inside openSUSE?

Perhaps we should figure out why the build tool is treating the entry as a
directory instead.

@minux I figured it out. It's not because of OBS, it's actually because if you run strip on a Go archive it will convert it to become invalid (and OBS automatically strips binaries in packages). We can fix this on the openSUSE side, but IMO a language's runtime breaking if you run strip on it is not the best thing in the world.

Note that dropping strip has increased our package sizes by 4x. :cry:

It does not make sense to run strip on an archive, whether the archive is Go or C.  Can you explain precisely what you are doing?

@ianlancetaylor I'm trying to fix the Go installation in openSUSE. By default, the Open Build Service (openSUSE's build system) will run strip on every file which looks like a binary file. This is probably not the best idea in the world, and can be disabled. However, the reason for this is quite simple -- strip should not remove headers it doesn't know are safe to remove. It turns out however that strip changes the entries in a .a archive even after stating that it doesn't understand the entries. While this is a strip bug, it's also probably not a general bug (I find it hard to believe that this causes serious issues with C archives).
Since opening this issue I have fixed the core issue (stripping anything in the Go package). Now our Go package works perfectly fine. However, our Go package now is 45MB as opposed to the old size of 12MB. Which is a huge price to pay.
So, this issue needs to be fixed somewhere so that we can go back to stripping the binaries in our Go packages that are safe to strip.

When I run strip runtime.a I see
strip:runtime.a(__.PKGDEF): Unable to recognise the format of file: File format not recognized
strip:runtime.a(_go_.o): Unable to recognise the format of file: File format not recognized
strip:runtime.a(asm.o): Unable to recognise the format of file: File format not recognized

and the file size is unchanged.
Can you tell us precisely what you are doing to the archive to make it so much smaller?  I don't see how that is possible without breaking it.  Or, do you mean that the package gets smaller because you run strip on the executables?  In that case, keep running strip, just skip the .a files.  As the error messages show, strip doesn't understand Go .a files anyhow.

@iangudger Check the sha256sum and you'll see it's different. I know that it produces an error message, but strip still modifies the archive:
% cp pkg/linux_amd64/runtime.a .
% strip ./runtime.a
strip:runtime.a(__.PKGDEF): Unable to recognise the format of file: File format not recognized
strip:runtime.a(_go_.o): Unable to recognise the format of file: File format not recognized
strip:runtime.a(asm.o): Unable to recognise the format of file: File format not recognized
% ls -la pkg/linux_amd64/runtime.a runtime.a
-rw-r--r-- 1 cyphar users 1490238 Nov 11 15:19 pkg/linux_amd64/runtime.a
-rw-r--r-- 1 cyphar users 1490238 Nov 12 15:11 runtime.a
% sha256sum pkg/linux_amd64/runtime.a runtime.a
4379f032997a03a8f443abfd26bd52c9d43c247c55acfd1a08388e34e6ae6add  pkg/linux_amd64/runtime.a
ec91aa9008898e9cf4a5b50f50d1f3f4980e184eba746f54b426b7ebc9e776e4  runtime.a

Namely what's happening is that strip is appending / to __.PKGDEF. I'm not sure what else is happening in the file to make the filesize not change, but I'm sure that this is what's changing. Just look:
% cat -v runtime.a | head -n2
!<arch>
__.PKGDEF/<PHONE_NUMBER>  1000  100   100644  12766     `
% cat -v pkg/linux_amd64/runtime.a | head -n2
!<arch>
__.PKGDEF       0           0     0     644     12766     `

@iangudger The problem is that the build system doesn't really give you a choice of what files to strip (I'm investigating that though). At the very least, I thought you might want to know what strip does to your archives...

Actually, that script has several bugs in it. I will fix those, so this issue is not really applicable anymore.


Added ignoreBlacklistedUsers and ignoreBlacklistedGuilds Options

Description of the PR
Adds ignoreBlacklistedUsers & ignoreBlacklistedGuilds to Monitors
Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)

Added ignoreBlacklistedUsers to monitors
Added ignoreBlacklistedGuilds to monitors

Semver Classification

[ ] This PR only includes documentation or non-code changes.
[ ] This PR fixes a bug and does not change the (intended) framework interface.
[x] This PR adds methods or properties to the framework interface.
[ ] This PR removes or renames methods or properties in the framework interface.

need to add defaults to constants, and remove the blacklisted inhibitor still


Cant open?

I am new to all of this so maybe this is my fault but i cant seem to open the tiktok-bot.py file.
I have python, selenium and the correct chrome driver installed but whenever I try to open the file it opens then closes automatically.

Hello! So the problem might be a module that you didn't install yet, most probably the colorama module. Here's what to do, open cmd/terminal and type the following command : pip install colorama
And be sure that your chromedriver is in the same directory/folder of the tiktok-bot.py file.
Now try to run the script, it should work. If not, please open the file using cmd/terminal and tell us what errors you are getting.

Yes thank you it was the colorama module. Working now :)

Great to hear that!


contact_list backup support?

Is it possible to modify the script to make a backup of names and phones from the contact list instead of the dialog list?
I was able to disable the message backup feature and just get the names and phones, but they are taken from dialog list, not contact list.
Thanks

I was able to achieve this, thanks.
My modification is an ugly hack and breaks other things, I would send a pull request otherwise.
(You can close this issue if you want).


nightly re4 remake not working

Apologise if this is not the place for this question. Downloaded the latest nightly re4.zip from 10 hours ago at it says at the time of writing this message extracted the dinput8.dll along with nvngx_dlss (3.1.11) and the latest PDPerfPlugin.dll at the root directory of the game. The menu of the mod comes up but the temporal upscaler option is not there. What am i doing wrong? Thanks for any input.

This feature is in beta and not fully released. You need the pd-upscaler branch, and it will only work on the RT (new) versions of the games.
https://github.com/praydog/REFramework/actions


System shall allow users to create a Home, edit a Home, create a Zone and like a Zone to a Home

Overview
Use Case Information

Use Case Name: Allow users to create a Home, edit a Home, create a Zone and like a Zone to a Home
Level: User level
Brief Description: System shall allow users to create a Home, edit a Home, create a Zone and like a Zone to a Home
Preconditions:
Triggering Event: Submitting one of the forms on the page
Main Flow: User opens Create Home page, user creates a home, edits a home or creates a zone
Extension:
Postcondition:

Closed by #19 #13 #15


Dependency upgrade

Coverage increased (+0.06%) to 73.555% when pulling 51f8f5e9cce1955d84a8f4f15d06181de0236f34 on dep_upgrades into 69620b47458c27bca8f54abeb6321074ecb97992 on dev.


Language pack

There is no language choice to interact with the bot. Multi-language interface for add/edit/delete menu would be nice.

#a0f20e0

a0f20e0


Add App component to storybook

RPC requests made by the App component and its children are mocked out using Mock Service Worker.

So storybook is failing out, but there is a fix here #50 .

So I am not sure the mocks are working as intended?

Good catch. Merging in this PR and will address in a follow up PR.


Support windows as operating system

As mentioned in https://github.com/async-profiler/async-profiler#community-supported-builds there is a community supported windows version available for the async profiler.
Maybe we can make windows a first class citizen and also support it in the java agent.

looking at releases of async profiler, I don't see continued support for Windows port, we are considering using JFR on Windows to capture profiler data and use it as input for pyroscope. I know this is limited in terms of types of frames it will capture but seems better then nothing ;) We are planning to start work on it if anyone will be interested to use it (and test it :) )

Here is the initial implementation which uses JFR recordings, this will collect events using jcmd JFR commands, which works on Windows as well, https://github.com/grafana/pyroscope-java/pull/136.

As mentioned in https://github.com/async-profiler/async-profiler#community-supported-builds there is a community supported windows version available for the async profiler.

The link seems broken.
I know Jebtrains / Intellij IDEA has a windows fork, but not sure if it is opensource. Seomeone may consider plugging it in via ProfilerDelegate from #136 . I doubt closed source profiler would be bundled with pyroscope-java so users would have to bundle it themselves.
Async-profiler windows support issue was closed as Not Planned https://github.com/async-profiler/async-profiler/issues/188


Wrap kubectl calls to properly output an error

We've started experiencing issues with K8S cluster (unrelated to k8s-buildkite-plugin) and they slip in as kubectl errors in the middle of our build logs on Buildkite like this:
Uploading: /tmp/tmp.N7WoODSLLy/dist/symbols/armeabi-v7a.objdump.gz for arch: armeabi-v7a with sharedObjectName:gl.so to https://upload.bugsnag.com/so-symbol
--
  | error: You must be logged in to the server (Unauthorized)
  | error: You must be logged in to the server (Unauthorized)
  | error: You must be logged in to the server (Unauthorized)
  | error: You must be logged in to the server (Unauthorized)
  | NDK mapping NdkMappings(sharedObjectName=x.so, arch=arm64-v8a, ndkMappingFilePath=/tmp/tmp.N7WoODSLLy/capture_symbols/arm64-v8a.objdump.gz), NdkMappings(sharedObjectName=x.so, arch=armeabi-v7a, ndkMappingFilePath=/tmp/tmp.N7WoODSLLy/capture_symbols/armeabi-v7a.objdump.gz) has been uploaded successfully!
  | ProGuard mapping /tmp/tmp.X9Eev60Pxz has been uploaded successfully!
  | The operation took 27.154 sec.

This is due to k8s-buildkite-plugin outputting logs in parallel to the while true control loops where we check job/pod status, etc and ANY kubectl command can fail and print an error that doesn't at all indicate that it is coming from kubectl 🙃
PR wraps all kubectl calls, captures stderr output and prints it with the kubectl command parameters so that it is very clear

kubectl failed
What kubectl command failed (I've checked the invocations, parameter values don't seem to include any sensitive info like tokens)

Friendly ping @tgolsson 😊


mismatching results on compositional task

hi, when I try to reproduce the results on ARO, I can't get the scores, the code is
for dataset in VG_Relation VG_Attribution do     for resume in scratch/open_clip/src/Outputs/negclip/checkpoints/epoch_0.pt      do         python3 main_aro.py --dataset=$dataset --model-name=$model --resume=$resume --batch-size=$bs --device=cuda --download      done done
and I just got VG_Relation 68.11 VG_Attribution 42.16 instead of 81 and 71 as reported in table6

I was also confused, in table2 you report clip on VG_Relation scores 59 while in table6 It becomes 63, am I missing something?

Hello!!
which model is this negclip/checkpoints/epoch_0.p? how was it trained with which parameters?
0.63 is for CLIP-FT (the one fine-tuned on mscoco), CLIP score is still 0.59.

thanks for quick response, i just rename the checkpoint from your gdown script, however, I tried this  for dataset in VG_Relation VG_Attribution  do     python3 main_aro.py --dataset=$dataset --model-name=NegCLIP --device=cuda --batch-size=$bs done
the results are still the same, 68.11 and 42.17

by the way, i calculate the acc by acc=(df['Accuracy']*df['Count']).mean() don't know if it should be this way

Hello!!
which model is this negclip/checkpoints/epoch_0.p? how was it trained? with which parameters?
0.63 is for CLIP-FT (the one fine-tuned on mscoco), CLIP score is still 0.59.

but when you take a look at this one, it doesn't match

oh you are right, that is a typo from the arxiv version, we should update that. This is the camera ready version of the paper: https://openreview.net/forum?id=KRLUvxh8uaX
(preparing an answer for the reproducibility soon)

i see, you compute marco accuracy instead of accuracy, could you share the code for computing the macro accuracy?

You should find all you need in the reproducibility notebook described here.
Macro accuracy on the relation dataset is just the average of the scores computed on each relation. If you use our evaluation wrapper it should be just: df["Accuracy"].mean().
Quick colab version

thank you, still, when i test the openai-clip:ViT-B/32, the macro accuracy on VG-Relation is 63, which matches the preprint instead of camera ready version, suggesting 59 is calculated by acc=(df['Accuracy']*df['Count']).mean() while 63 is acc=df['Accuracy'].mean(), I guess you should report 63 instead of 59

just run the colab with openAI's CLIP and got 0.59, could you try to see what's missing starting from that?

just run the colab with openAI's CLIP and got 0.59, could you try to see what's missing starting from that?

why use symmetric
df = pd.DataFrame(vgr_records) df = df[~df.Relation.isin(symmetric)] print(f"VG-Relation Macro Accuracy: {df.Accuracy.mean()}")
instead of directly take the mean?

Closing this for now, let me know if you have other questions!

Hi @vinid
I was trying out the colab you shared above, but i changed the model to NegCLIP. iN particular i changed one line to
model, preprocess = get_model(model_name="NegCLIP", device="cuda", root_dir=root_dir)
I am getting
VG-Relation Macro Accuracy: 0.8021811864440539
VG-Attribution Macro Accuracy: 0.7055937135374111
Just wanted to confirm if this is correct, especially the Relation accuracy.

I didn't try that earlier. Uncommeting it gives:
VG-Relation Macro Accuracy: 0.8038109510723876

the difference is small but let me look into this

yup, not a big issue, but just wanted to confirm if this is the correct number.
Thanks for looking into it

yea makes total sense, thanks for pointing this out

Also, on OpenAI Clip I get
df = df[df["Count"] > 9 -> VG-Relation Macro Accuracy: 0.5923217479726929
Commenting it gives -> VG-Relation Macro Accuracy: 0.5946534905762514

Hi! thanks! The CLIP one matches the one in the paper

Also, funnily if you use torch tensors (and not numpy) +cuda to compute the scores and then the accuracy, you get
VG-Relation Macro Accuracy for CLIP of 0.599128631916311
with df = df[df["Count"] > 9

Thank you all! I think, somehow, most of the comments above are correct. 😄

If you look at Table 2 we wrote 0.80 for VG-R, vs in Table 3 we wrote 0.81. This is our mistake, we are sorry about this.

As for the reason, I think @DianeBouchacourt is spot on here. This appears to be due to the non-determinism in cuda. (e.g. see here). If you do the computations in cuda you get that ~0.01ish difference in perf, and our legacy code (before cleaning and releasing) is doing that.

I think overall nondeterminism can get pretty tricky in the context of the VG-R dataset. The embeddings can be pretty close, and minor differences due to nondeterminism can lead to differences around ~0.01


Sort issues I have no calls for to the bottom

Sometimes, issues pop up that have no calls for me to make.  Can these be sorted to the bottom and/or greyed out to indicate that there's no action necessary?

Thanks! I agree that this should be better, and I think a bigger issue here is that you probably live in a split district and didn't enter a specific enough address to get your reps in the house, and the app needs to do a better job alerting you to that.
To get reps for all the issues, try entering a more full address into the location page -- including a street, for example. (Let me know if that isn't why you end up with no reps for some issues.)

In fact, we should never return issues that have no calls to make -- so we should solve this by telling the user about split district.


Remove 'using namespace std' from all headers

Closes https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/issues/538
Remove using namespace std from all headers in the project, so as to not irreversibly modify the global namespace for users of this library.

@mushmal requesting review

Ah - I didn't have the gstreamer plugin enabled in my local build

Ah - I didn't have the gstreamer plugin enabled in my local build

Thanks for taking care of it. The changes look good. Let's wait for the clean build. I am pretty sure the tests will still fail due to the need to have credentials to run against the backend. We will address that by re-submitting the PR

Few more compile errors.

Ok looks like tests also don't build by default - noting that I am now using configuration
cmake .. -DBUILD_GSTREAMER_PLUGIN=ON -DBUILD_JNI=OFF -DBUILD_DEPENDENCIES=ON -DBUILD_TEST=ON

@MushMal from what I can tell it is now test failures due to credentials, is that correct?

@MushMal from what I can tell it is now test failures due to credentials, is that correct?

Oh yes, these are auth failures. Let us take care of it and push the commit

Resolved by https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/pull/544


Unable to provision Learning Pathway site

Category

[X ] Question
[ ] Typo
[ ] Additional article idea
[x] Checked items have no spaces between the [] (delete this line)

Question

I am attempting to provision a learning pathways site for our SharePoint.  Every time I attempt to do so it says provisioning site and it just spins.  I have left it sitting for hours with no response or change.  I have checked all prerequisites and I cannot figure out what I am missing. Any suggestions will be helpful.  I do not get any type of error messages and I have tried to do this using Edge, Chrome and Firefox.

Environment details (if applicable)

SharePoint online

OS: Windows 10
Framework: [e.g. .NET Framework v3.x, .NET Core v3.x, Node.js v6/v8/v10/v12]
Browser(s): Tried Chrome and Edge
Tooling: [e.g. VS Code, SPFx v1.10.0, Visual Studio 2019]

Please submit your issue to the Learning Pathways project as this repo is for general SharePoint development questions: https://github.com/pnp/custom-learning-office-365


[ENG-12057][eas-cli] don't prompt user to set android.package and ios.bundleIdentifier value when runing in non-interactive mode

Why
https://linear.app/expo/issue/ENG-12057/fail-with-a-nice-and-meaningful-error-message-if-iosbundleidentifier
How
Throw meaningful error instead
Test Plan
Tests

/changelog-entry bug-fix Don't prompt to set android.package and ios.bundleIdentifier values when running in non-interactive mode


tickets/DM-32983: Convert Jenkins Lab builds to sciplat-lab and make

We're just going to build it with GHA instead.


FG-2933: Add CompressedAudio schema

Changelog

Add CompressedAudio schema
Docs

None
Description

Foxglove does not currently support audio messages of any kind. In order to support playback of audio in Foxglove Studio, this schema provides the wire format for audio frames that Foxglove can play.

Tested locally using the MCAP recording demo and a new Audio panel.

FG-2933

The CI / python workflow is failing: https://github.com/foxglove/schemas/actions/runs/12341162294/job/34439471378?pr=159
It seems like Python 3.7 is not supported for Ubuntu 24.04 x64 - not sure if runners recently changed. According to this issue, we would need to fix to ubuntu-22.04 instead of ubuntu-latest or upgrade Python version to something supported in Ubuntu 24.04.


Added class option to include journal publication information in arti…

Added class option to include journal publication information in article footer. Sample documents updated to include parameter definitions and instructions regarding pubversion class option.
Since we're not 100% certain of the DOI format yet, I made this generic; the DOI is not auto-generated and could be entirely independent of the article number.
May need to write some instructions for authors regarding the "pubversion" class option that triggers the footer.

Excellent, @dwsideriusNIST . Yes, definitely need some instructions...

(Can merge this, or could bring in instructions at the same time. Preferences?)

I placed some instructions in the sample TeX files, perhaps we add a short section to https://github.com/livecomsjournal/article_templates/blob/master/information_for_authors.md ? I was initially reluctant to do so, as that file is not about the nitty-gritty of LaTeX. Opinion/advice is kindly received.

There's two choices; we do the fix ourselves to add date, version, issue, or they do it.
One thought; we might not assign an issue or number in the issue until after it's published. We might want the options of having some intermediate marker (ASAP?) until the release is triggered.  Which would be a giant pain for 5-10 papers, just updating the numbering. Or, after an issue is released, we start numbering with the next issue, but say that issue N is released when all of the articles from N are published (we just essentially do a press release . . . )

@mrshirts:

There probably should be an "ASAP" or "Accepted" descriptor before the issue/article number is available. I am in favor of minimal granularity, so max of three categories: "Just Accepted", "ASAP" (meaning in final form but without issue and/or number), and "In Print."

There should probably be a date field to indicate when the reviewed version (should be vX.0, right?) was in final form. In a sense, the publication date of the reviewed version. It can be another line in the footer, very easy to add.

@mrshirts

There's two choices; we do the fix ourselves to add date, version, issue, or they do it.

I'm ambivalent as to which; preferences?
@dwsideriusNIST - what would be the differnece between "just accepted" and "ASAP"? It seems to me we could just do this:

As soon as accepted, get them (or us) to make a final publication-ready version
Post that in advance of the issue
Then once we are ready to release an "issue" we:
Change the numbering on it to indicate what issue it is in and re-post

I don't think it would necessarily be a big pain to update the numbers; it'd just be adding the issue number on all of the papers, or am I missing something, @mrshirts ?

If there was a release-tagged version of the paper when it was accepted, then we could go in and just add the issue, and have them generate the .pdf.  Then they would have to mail it to the journal, and we could post it.  So, doable for now, but kind of a pain.

Slightly painful, but seems tolerable perhaps.
I did hear of the Journal fo Open Source Software which basically does everything we are trying to do (including, I believe, automatic releases from GitHub being published) except only for software. It's possible longer-term we could shift to use their infrastructure, as I think it basically does everything we want in a more automated way, or at least that was my superficial understanding.

We are planning a submission there soon, so I will investigate. https://joss.theoj.org/.  I worry there might be too many differences in how they want to do things, but we could at least learn from them.

@davidlmobley My definitions were:
"Just Accepted" : the version accepted by the editors, but without any post-processing. "As is"
"ASAP" : in final form, but without an issue number or other metadata that can't be known until the issue is assembled
That distinction may be less relevant for this journal since most of the formatting responsibility is on the authors.

Since authors are doing their own editing, I think 'accepted' and 'final' become the same thing.  @mrshirts ?
I think we'd better list both received and accepted dates.  Despite our total reliance on github, our authors may be seeking tenure or some other weird status (Nobel prize for lessons learned?) where dates on the paper will matter.  We have promised 'academic credit' for previously unheralded types of work, so we should deliver the full package.

OK, I'm going to work on a set of class functions that put the received and accepted dates somewhere on the first page. I'll likely reproduce something like the ACS / JPC:x style.
This will all require a small amount of documentation in the template guide. Due to the non-traditional nature of the journal (by that I mean the option to revise post-review), we should also clearly define what "received" and "accepted" mean for LiveCoMS. Just to be clear, I would define them as:
Received: Date of submission to LiveCoMS
Accepted: Date (following review) when the reviewed version was accepted by the editors.
There is still the "This version dated <---->" blurb on the front page, which is created by the class file using the date that the PDF was compiled. Or perhaps we need to revisit that aspect of the LaTeX class, as the PDF will have to be compiled post-acceptance, making that date always more recent than the Accepted date. Thoughts?

I'm closing this PR so that the BibTeX style can be updated in a different PR, then we'll settle on the footers for the article template.

I wonder if there's a way to get that date from the last entry in the Git log. For now, though, I'm going to start a new PR with the updated footer and, possibly, instructions for entering the article ID, etc. Stay tuned.

That would be cool.

OK, PR #58 addresses the footer information in the document, leaving the "This version dated...." issue unaddressed.
A few stack overflow searches came up with ways to supply the git log information to the document, but they're NOT user friendly. Often requires a command-line directive during the latex compilation, which I think we should avoid. So, no proper answer yet.

Here are some options for getting the Git log into the LaTeX document automatically:

Use either gitinfo or gitinfo2 (packages available in TeXLive); these rely on a post-commit hook in the git configuration that creates a metadata file for LaTeX to interpret. Upside: simple integration, transparent use, platform-independent. Downside: Authors have to create the necessary hook; we would have to include directions in the author checklist that involves adding a file to the .git/hooks directory. Additional downside: the .git directory is not version-controlled, so extra git magic is required for the hook to propagate to clones/forks.

Use some variation of \immediate\write18{git status > git.tex}\include{git.tex} to pipe command-line output to a text file. Upside: no packages required. Downside: POSIX-specific implementation, plus the compiler has to be run with a --shell-escape or --enable-write18 directive.

I cannot recommend either option as a workable solution for LiveCoMS. Both require manual user-level interactions with LaTeX and/or git that is not conducive to generation of consistently-styled PDFs for the journal.
Does anyone else have any other methods for automatically placing Git information in a LaTeX document?

@dwsideriusNIST thanks a lot for investigating.  I'm sorry but all this is beyond my expertise.

I don't, @dwsideriusNIST - BUT, since my last experiment was such a success, let me ask Twitter. :)

@HaoZeke My intention wasn't to put a lot of git metadata in the document, just either 1) the last commit date or 2) an indicator that the current compiled PDF is dirty compared to the commit log.
A makefile would be my solution also (I actually use makefiles to do a lot of TeX / git / BibTeX operations), but that is still not platform independent. Despite the pain of saying this, I don't want to lock out a Windows/something else (Overleaf? ShareLaTeX?) user from straightforward document formatting for the journal.

@dwsideriusNIST ah yes, I also noticed that late and modified the comment...
yarn with gulp and shx is actually very cross platform....
I've tested it for windows, *nix and MacOS as well, for example to run this pure js platform independent document generator and this tup based tool which uses shx to run on multiple OSs'.
However, Overleaf and ShareLaTeX will be a major stumbling block as I believe they only support a small subset of the git features. However we can still provide for those fringe cases by having CI like Semaphore or Travis or wercker build the final pdfs on each push.
That way we can have a standardized build system throughout, controlled by the CI running a custom docker image, like this one.

Twitter agrees that one either needs to use a build system or a script to get a date from git into an article, e.g. see https://twitter.com/the_mabraham/status/1015477503660576769


Fix build/script script names in electron

When I run yarn electron:package:mac I get the error "Command "build" not found". When I update the packages/electron package.json script to remove yarn build from the "package:mac" script, electron build throws the error: "Application entry file "build/electron.js in Contents/Resources/app.asar" does not exist".
Is there a way to resolve this? Thanks.

Ok solved the latter issue by running the cra:build script first.

Ha! @whenmoon this is actually a bug, good call —  these two commands should probably drop the cra: prefix.

Hi @mmazzarolo, ok , I'll open a PR to fix..

Remove cra: prefix and then remevoe ^ at devDependencies of electron. like that ^14.0.0 -> 14.0.0

In my opinion, this issue can be closed.


Add documentation for core_names.h fields and update code based on usage

Checklist

[x] documentation is added or updated
[ ] tests are added or updated

Still quit a bit to do.

Ready for feedback.

ping

Apart from the concerns I just mentioned, this is great work.  I'm especially happy to see the EVP_SIGNATURE and EVP_KEYEXCH pages appear.  Yay :-)

rebased

another rebase

travis error is s390 resource unavailable..

I'm generally happy with what I see, there were just a few nits.  I'll make another sweep later today to see that I haven't missed something, but other than that and nit fixups, this should be good to go.

This is good work, @slontis!  Thanks for your patience

Cheers.. Thanks for reviewing - I hope to get better at documenting with less NITS at some point..

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Thanks. Merged to master.


Embedded image

please expand the pr description.

please expand the pr description.

Pull Request
Description
This PR introduces the following changes:

Initialization of New Attributes:

Added _mlm_client and _mlm_model attributes to the PptxConverter class, initialized using the kwargs dictionary.

Handling of Image Shapes:

Integrated a new method _convert_image_to_markdown to handle the conversion of image shapes to markdown within the presentation slides processing loop.

Addition of _convert_image_to_markdown Method:

Added a new method _convert_image_to_markdown to the PptxConverter class to convert image shapes to markdown format.

Related Issue
Link to the related issue (if any).
Motivation and Context

The new attributes _mlm_client and _mlm_model are required for additional functionality.
The _convert_image_to_markdown method improves the handling of image shapes by converting them to markdown format, enhancing the overall functionality of the PptxConverter class.

How Has This Been Tested?

[ ] Unit tests
[ ] Integration tests
[ ] Manual testing

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Screenshots (if appropriate):
Types of changes

[ ] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Documentation update

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] The title of my pull request is a short description of the requested changes.

Additional Notes
Add any additional information or context.


Create zemerik.dev.org.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.

Description
I will be using this domain to host my personal portfolio.
Link to Website
https://zemerik.is-a.dev

CLA Assistant Lite bot: 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 can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA


Nexmo request parameter format change

Nexmo API has changed the Request parameter formatting from underscore concatenation to hyphen concatenation. Parameter formattin changed accordingly:
When sending SMSs the request parameters are concatenated with underscore:
https://github.com/ConnectCorp/nexmo-client/blob/master/src/Service/Message.php#L96
but Nexmo API says that they should be hyphenated:
https://docs.nexmo.com/api-ref/sms-api/request

@apeltosa great work! :+1: Could you please address the CI issues so I can merge it? Thanks.

@vood CI tests updated and issue solved


[WIP] Add add-slaves and remove-slaves commands

This PR is a work in progress.
This PR adds add-slaves and remove-slaves commands which enable the user to resize existing clusters.
TODO:

[ ] Implement add-slaves.
[x] Implement remove-slaves.
[ ] Add acceptance tests.
[ ] Update README.

Open questions:

[ ] Does Spark need to be restarted after slaves are removed from a running cluster?
[ ] How about when they are added?

Fixes #16.
Fixes #113.
Fixes #140.

Not sure why the Travis build is failing. Am investigating here: https://github.com/pyinstaller/pyinstaller/issues/2123

I am ready to merge this PR in.
Pinging @ereed-tesla, @serialx, and @soyeonbaek-dev, since I know you are interested in this feature.
Do you have any feedback or questions about this before I merge it in? Especially regarding the some of the open questions noted above.

First, hooray for adding cluster resize! Very awaited feature indeed! 😃
For those who don't know: We have been cooking our own cluster resize patches to flintrock and have been using that for over two weeks. We mainly only use Spark without HDFS, so our experiences with dynamic cluster sizes are limited to Spark. And we don't restart master. It seems to work fine with both adding and removing slaves. Since the system itself is fault tolerant, Spark seems to work fine with this.
About removing slaves. The code simply removes N slaves in the list. I think it would be better to have some kind of prioritisation, like removing spot instances first. You can see @soyeonbaek-dev's code

About removing slaves. ... I think it would be better to have some kind of prioritisation, like removing spot instances first.

This is a good suggestion. If you have a mix of spot and on-demand instances (which this PR now makes possible), you probably do want to remove the spot instances first.
I'll add this to the PR.

OK, that's done. I'll leave this open for a few more days and then merge it in if there's no more feedback.

Testing this branch revealed a bug related to configuring EC2 instance profiles. When I use --ec2-instance-profile-name settings to create a cluster, add-slaves command fails with the following error:
Traceback (most recent call last):
  File "/usr/local/bin/flintrock", line 9, in <module>
    load_entry_point('Flintrock==0.6.0.dev0', 'console_scripts', 'flintrock')()
  File "/usr/local/lib/python3.5/site-packages/flintrock/flintrock.py", line 1031, in main
    cli(obj={})
  File "/usr/local/lib/python3.5/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.5/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.5/site-packages/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.5/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.5/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.5/site-packages/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/local/lib/python3.5/site-packages/flintrock/flintrock.py", line 653, in add_slaves
    **provider_options)
  File "/usr/local/lib/python3.5/site-packages/flintrock/ec2.py", line 46, in wrapper
    res = func(*args, **kwargs)
  File "/usr/local/lib/python3.5/site-packages/flintrock/ec2.py", line 255, in add_slaves
    instance_profile_name = profiles.instance_profile_name
AttributeError: 'filter' object has no attribute 'instance_profile_name'

I was able to fix this bug using a simple patch. But I don't really like how this works. Perhaps you have a better solution to this. :)
diff --git a/flintrock/ec2.py b/flintrock/ec2.py
index 0f3dd1a..630d8fa 100644
--- a/flintrock/ec2.py
+++ b/flintrock/ec2.py
@@ -249,7 +249,10 @@ class EC2Cluster(FlintrockCluster):
         if not self.master_instance.iam_instance_profile:
             instance_profile_name = ''
         else:
-            instance_profile_name = self.master_instance.iam_instance_profile['Id']
+            iam = boto3.resource('iam')
+            instance_profile_id = self.master_instance.iam_instance_profile['Id']
+            profiles = filter(lambda x: x.instance_profile_id == instance_profile_id, iam.instance_profiles.all())
+            instance_profile_name = list(profiles)[0].instance_profile_name
         instance_initiated_shutdown_behavior = response['InstanceInitiatedShutdownBehavior']['Value']

         self.add_slaves_check()

Hmm, so the IAM profile ID is different from the IAM profile name? I presume iam_instance_profile['Arn'] is also not what we want?
I'll take a closer look at this tonight or tomorrow.

I tried the Arn approach. Also failed:
Traceback (most recent call last):
  File "/usr/local/Cellar/python3/3.5.1/Frameworks/Python.framework/Versions/3.5/lib/python3.5/runpy.py", line 170, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/local/Cellar/python3/3.5.1/Frameworks/Python.framework/Versions/3.5/lib/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/Users/serialx/workspace/flintrock/flintrock/__main__.py", line 8, in <module>
    sys.exit(main())
  File "/Users/serialx/workspace/flintrock/flintrock/flintrock.py", line 1031, in main
    cli(obj={})
  File "/Users/serialx/.virtualenvs/flintrock/lib/python3.5/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/Users/serialx/.virtualenvs/flintrock/lib/python3.5/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/Users/serialx/.virtualenvs/flintrock/lib/python3.5/site-packages/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/serialx/.virtualenvs/flintrock/lib/python3.5/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/serialx/.virtualenvs/flintrock/lib/python3.5/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "/Users/serialx/.virtualenvs/flintrock/lib/python3.5/site-packages/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/Users/serialx/workspace/flintrock/flintrock/flintrock.py", line 653, in add_slaves
    **provider_options)
  File "/Users/serialx/workspace/flintrock/flintrock/ec2.py", line 46, in wrapper
    res = func(*args, **kwargs)
  File "/Users/serialx/workspace/flintrock/flintrock/ec2.py", line 275, in add_slaves
    instance_initiated_shutdown_behavior=instance_initiated_shutdown_behavior)
  File "/Users/serialx/workspace/flintrock/flintrock/ec2.py", line 680, in _create_instances
    'EbsOptimized': ebs_optimized})['SpotInstanceRequests']
  File "/Users/serialx/.virtualenvs/flintrock/lib/python3.5/site-packages/botocore/client.py", line 278, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/Users/serialx/.virtualenvs/flintrock/lib/python3.5/site-packages/botocore/client.py", line 572, in _make_api_call
    raise ClientError(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (InvalidParameterValue) when calling the RequestSpotInstances operation: Value (arn:aws:iam::527973214447:instance-profile/zeppelin-role) for parameter iamInstanceProfile.name is invalid. Invalid IAM Instance Profile name

I guess we need to feed it with IAM Profile Names. :(

Thanks for investigating @serialx, it looks like you're right. The only way to get the profile by name is with the heavy-handed pull and filter. I've reported a couple of issues against Boto3 to see if there is a way this can be fixed at some point:

https://github.com/boto/boto3/issues/768
https://github.com/boto/boto3/issues/769

I've pushed a new commit that fixes this based off of your patch above. Take a look and let me know if you have any other concerns. Thanks again for finding and reporting this issue.


Search exception

Seems like queries that should return results in the applications list cause exception.
Failures:
http://apidoc.me/search?q=org%3Agilt+test
http://apidoc.me/search?q=org%3Agilt+install
Returns no results successfully:
http://apidoc.me/search?q=org%3Agilt+test2

Thanks for reporting. This comes from an API change to use the new discriminator. We have cleared the search table and queued up the applications for re-indexing which means they should shortly be available.


Connection error HTTPSConnectionPool

Hi,
I'm having the following issue:
HTTPSConnectionPool(host='archive.sec-api.iohttps', port=443): Max retries exceeded with url: ...
Caused by NameResolutionError("<urllib3.connection.HTTPSConnection object at ...>: Failed to resolve 'archive.sec-api.iohttps' ([Errno -2] Name or service not known)"
The links are working, my connection is stable. Is there maybe a way to increase retries for renderApi.get_filing() or another workaround?
Thanks

I added a manual retry function, now it seems to work like a charm. But maybe it would make sense to hardcode this into the get_filing() function as I got this error quite a few times.


Save caller-location information for root node/function as well.

Since currently, no caller/callee information saved for the root node of call-graph, we can't jump to the location of root function.

Fixed in recent commit.


CI fails

@tshemsedinov It seems that this is a problem with the mongodb.provider at globalstorage. More on that, previous CI for 0.5.2 failed with the same error: https://travis-ci.org/metarhia/impress/builds/371805434

@SemenchenkoVitaliy to fix the CI build, you can revert https://github.com/metarhia/impress/commit/a051d8a29310a3653b2a0baadfec93fdb89fb778 here.


bugzilla 754 -- test codes with valgrind

Bob Grumbine 2022-12-02 18:40:42 UTC
(In reply to Simon Hsiao from comment #0)

Please check the potential memory leak in seaice executables, you can
perform the memory leak check: valgrind --leak-check=yes myexec.

I've gone looking and found valgrind is under module valgrind4hpc on wcoss2. I'll include that in my testing / building.

Per email with Steven Earle, changed to a test with fortran compilations under -check all and -ftrapuv
No messages, no changes to outputs.


Warning:(14, 24) Application namespace for attribute bind:visibilityOnIndex will be ignored.

After open and import your code app to my project, this warning occured after I rebuild project.
any solution?

I wasn't able to reproduce this issue on my end. Which file are you getting this error in?

is it still an issue ? I think i had a similar issue in my project with proguard minification enabled.

Proguard is disabled by default in this project, so that shouldn't be a problem. I'm still unable to repro this hence closing this issue.


Extract node_modules cache flake in pipeline

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=211471&view=logs&j=9833cdaa-2e92-5f8d-56a8-239fd18fd628&t=ba20f3c3-572a-5fba-98e2-dcb31723a3fa&l=60423
A similar error has appeared for other Azure Pipelines tasks https://github.com/microsoft/azure-pipelines-tasks/issues/13785

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=211852&view=logs&s=d26761af-f5e8-5e3e-3ed3-ef1ee2069094&j=78322c78-e078-5e20-184d-1b2bb6aed118
https://dev.azure.com/monacotools/Monaco/_build/results?buildId=211871&view=logs&j=260a512b-e5ca-5cf1-3f8e-20c0fa41efc9&t=22113586-ca8b-59fc-5d74-ba15236b4ee3

Odd:

@joyceerhl's one is about extracting
@meganrogge's first one shows no error at all (copy paste mistake?)
@meganrogge's second one is about creating a zip

@joaomoreno I reran the failure for the first one which is why it no longer shows an error
here's the precise link to the failure
https://dev.azure.com/monacotools/Monaco/_build/results?buildId=211852&view=logs&j=55ac390b-ffb1-50ea-3650-525dd9a3fd80&t=7a41f864-1411-5635-a6d8-563d674ed440&s=a56e7651-4213-56af-9488-c25fe636c10c

Thanks!

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=212170&view=logs&j=9833cdaa-2e92-5f8d-56a8-239fd18fd628&t=ba20f3c3-572a-5fba-98e2-dcb31723a3fa


D2E Shop Issues - missing item, item error, fame behaviour

Description of Problem or Question
I'm testing a shopping stage for a campaign.
In the campaign initialisation I setup fame thresholds and party fame, for testing purposes I then show a button to add 20,000 gold at a time... Then you can choose to go to the shop.
The city-style shop has a restock button and a button to increase party fame to/past each threshold in turn, so I can test buying everything...
First issue: missing items
The following items do not seem to be present in the D2E shop:
Archaic Scroll (Shadow of Nerekhall)
Jinn's Lamp (Labyrinth of Ruin)
Mapstone (The Trollfens)
I guess Jinn's Lamp and the Mapstone don't work in the app and therefore shouldn't be there, but Archaic Scroll seems legitimate?
Second issue: item error
The Act II item 'Blessed Armor' is incorrectly shown as Act I
Third issue: minimum fame error
Potentially mostly caused by an error on my part... If you initialise fame like this:
operations=$%fameimpressive,=,2 $%famecelebrated,=,12 $%fameheroic,=,22 $%famelegendary,=,32 $%fame,=,0
then I have noticed that Valkyrie won't go anywhere near a shop - my call to the shopSubfolder/quest.ini just causes Valkyrie to stop (not crash, but Menu->Main Menu does nothing) without triggering the 'EventStart' trigger opening event within that subfolder quest...
That might be because at that point fame is indetermined - it could be Insignificant or Noteworthy and the app can't tell since those thresholds aren't set?
As I said, mostly user error but an interesting failure mechanism I thought was worth reporting :)
Valkyrie Version
2.3.04

In general I think there's something weird going on with shop items.
I don't think the minfame/maxfame is working - if I go into my shop with fame=2, when the insignificant threshold is 0 and noteworthy is 10, I only see one item (Magic Staff). That's the only item with a maxfame of insignificant...
If I then increase fame by 10 with my button in the shop, it breaks Valkyrie - I'm not sure it's finding any Noteworthy items...
If I get my fame up to the Heroic level, I can only buy 14 items - all the ones with maxfame set to Heroic... minfame doesn't seem to affect anything?

First issue:
This is from RtL data, I'm inclinded to leave it as-is.
Second issue:
Fixed in 2.4

Third issue:
Definately 2 bugs here that I have found and now looks better. Building as 2.4.01 now.

Just to add to this: Archaic scroll is unavailable, because you have to return the card after using it, but there is no way to tell the game you used it, and to remove it from inventory in RtL. It might theoretically be possible to do this in Valkyrie by using the MoM item inspect mechanic, but this would feel out of place in the Descent module.


Invoice expiry

Invoices should expire after some pre-determined timeout is reached. Especially important for invoices quoted in fiat currencies, where the exchange rate should only be locked-in for limited time.

It appear like c-lightning already expires invoices internally, but doesn't provide a way to control it over RPC.

Added in 68ff2997882c670378c80790a5590f64ce0d9fdf (expiry configuration over REST API), 37699ec9a45b2144c9e1954f4eb60efbe638ffb8 (expire API calls), fabee83b5d8d444ab32019e7f21b306ea4d96494 (expire checkout page) and d456ca69406da3dbade8383193851ff90fefa20f (checkout client-side).