exclude /usr/lib/.build-id from RPM

with the move to Rocky 8 from CentOS 7, it appears rpm-build now embeds debug info in /usr/lib/.build-id/
Links with info (credit @grahamaj for digging into this one!)

RHEL8 rpmbuild and /usr/lib/.build-id https://access.redhat.com/discussions/5045161
What is the purpose of /usr/lib/.build-id/ dir? https://unix.stackexchange.com/questions/411727/what-is-the-purpose-of-usr-lib-build-id-dir/411736#411736
If I do not care about debug support in red hat packages, what are the drawbacks of turning build-id support off in the specs file? https://unix.stackexchange.com/questions/688839/if-i-do-not-care-about-debug-support-in-red-hat-packages-what-are-the-drawbacks

contains info about %define _build_id_links none

https://stackoverflow.com/questions/61728580/cpack-conditonal-behaviour-based-on-generator#comment109208274_61728580

comment has cmake to disable .build-id in the RPM specset(CPACK_RPM_SPEC_MORE_DEFINE "%define _build_id_links none")

CPACK_RPM_SPEC_MORE_DEFINE docs https://cmake.org/cmake/help/v3.28/cpack_gen/rpm.html#variable:CPACK_RPM_SPEC_MORE_DEFINE

testers were reporting conflicts with files from different microservice packages
https://isrhub.usurf.usu.edu/Vantage/UserService/discussions/176#discussioncomment-1140

I downloaded the server rpm of the last RC of UserService built on CentOS 7 and the first RC built on Rocky 8 and examined the contents
$ rpm -qlp UserService-<IP_ADDRESS>-Linux-server.rpm
/opt/VantageSuite/UserService
/opt/VantageSuite/UserService/.init
/opt/VantageSuite/UserService/.init/local_config.type
/opt/VantageSuite/UserService/.init/userservice.service
/opt/VantageSuite/UserService/bin
/opt/VantageSuite/UserService/bin/UserService
/opt/VantageSuite/UserService/bin/node
/opt/VantageSuite/UserService/web
/opt/VantageSuite/UserService/web/bin
/opt/VantageSuite/UserService/web/bin/1f63235f154a52af81d3a0dc94b45e6a.node
/opt/VantageSuite/UserService/web/bin/4efc97c1a519c12646c7ed680cf319fa.node
/opt/VantageSuite/UserService/web/bin/NodeLogManagerbin.node
/opt/VantageSuite/UserService/web/bin/NodeMockBrokerbin.node
/opt/VantageSuite/UserService/web/bin/NodeMockPtpProviderbin.node
/opt/VantageSuite/UserService/web/bin/NodeUtilsbin.node
/opt/VantageSuite/UserService/web/bin/NodeVantCommbin.node
/opt/VantageSuite/UserService/web/bin/ac17b83f95270357f2ae8a5a811dff20.node
/opt/VantageSuite/UserService/web/certs
/opt/VantageSuite/UserService/web/certs/SDLVantageCA.crt
/opt/VantageSuite/UserService/web/certs/server.crt
/opt/VantageSuite/UserService/web/certs/server.key
/opt/VantageSuite/UserService/web/createAdmin.js
/opt/VantageSuite/UserService/web/createAdmin.js.LICENSE.txt
/opt/VantageSuite/UserService/web/index.js
/opt/VantageSuite/UserService/web/index.js.LICENSE.txt
/opt/VantageSuite/UserService/web/user-management-app.tar.gz

$ rpm -qlp UserService-<IP_ADDRESS>-Linux-server.rpm
/opt/VantageSuite/UserService
/opt/VantageSuite/UserService/.init
/opt/VantageSuite/UserService/.init/local_config.type
/opt/VantageSuite/UserService/.init/userservice.service
/opt/VantageSuite/UserService/bin
/opt/VantageSuite/UserService/bin/UserService
/opt/VantageSuite/UserService/bin/node
/opt/VantageSuite/UserService/web
/opt/VantageSuite/UserService/web/bin
/opt/VantageSuite/UserService/web/bin/1f63235f154a52af81d3a0dc94b45e6a.node
/opt/VantageSuite/UserService/web/bin/579ba4c588fc74a1a6b4898d4ce085de.node
/opt/VantageSuite/UserService/web/bin/NodeLogManagerbin.node
/opt/VantageSuite/UserService/web/bin/NodeMockBrokerbin.node
/opt/VantageSuite/UserService/web/bin/NodeMockPtpProviderbin.node
/opt/VantageSuite/UserService/web/bin/NodeUtilsbin.node
/opt/VantageSuite/UserService/web/bin/NodeVantCommbin.node
/opt/VantageSuite/UserService/web/bin/ac17b83f95270357f2ae8a5a811dff20.node
/opt/VantageSuite/UserService/web/certs
/opt/VantageSuite/UserService/web/certs/SDLVantageCA.crt
/opt/VantageSuite/UserService/web/certs/server.crt
/opt/VantageSuite/UserService/web/certs/server.key
/opt/VantageSuite/UserService/web/createAdmin.js
/opt/VantageSuite/UserService/web/createAdmin.js.LICENSE.txt
/opt/VantageSuite/UserService/web/index.js
/opt/VantageSuite/UserService/web/index.js.LICENSE.txt
/opt/VantageSuite/UserService/web/user-management-app.tar.gz
/usr/lib/.build-id
/usr/lib/.build-id/07
/usr/lib/.build-id/07/6d63220fa54ce2174ee744ad6800045b440dc0
/usr/lib/.build-id/21
/usr/lib/.build-id/21/4aca320a113a00bfaa03b81e76b74f1b954e9a
/usr/lib/.build-id/7b
/usr/lib/.build-id/7b/69169bcf0b1874734b9fc3bdae39355f34eedc
/usr/lib/.build-id/8b
/usr/lib/.build-id/8b/b1d53c9541a7a220c9fe8799bc7f7d3ca2c5f5
/usr/lib/.build-id/ac
/usr/lib/.build-id/ac/d4122e9e6bc64a3829ddfe049bf45bfd688f87
/usr/lib/.build-id/cd
/usr/lib/.build-id/cd/9c25a1265635c11ed55f74d453e30bfd42eaec
/usr/lib/.build-id/d7
/usr/lib/.build-id/d7/5f020bbc25bce5bbce910390d09eb94789dc3a

sure enough, the Rocky 8 RPMs have /usr/lib/.build-id/ files included

completed with commit to main branch referenced above
built UserService with the changes in place -- no /usr/lib/.build-id!
$ rpm -qlp UserService-<IP_ADDRESS>-Linux-server.rpm
/opt/VantageSuite/UserService
/opt/VantageSuite/UserService/.init
/opt/VantageSuite/UserService/.init/local_config.type
/opt/VantageSuite/UserService/.init/userservice.service
/opt/VantageSuite/UserService/bin
/opt/VantageSuite/UserService/bin/UserService
/opt/VantageSuite/UserService/bin/node
/opt/VantageSuite/UserService/web
/opt/VantageSuite/UserService/web/bin
/opt/VantageSuite/UserService/web/bin/1f63235f154a52af81d3a0dc94b45e6a.node
/opt/VantageSuite/UserService/web/bin/NodeCorebin.node
/opt/VantageSuite/UserService/web/bin/NodeLogManagerbin.node
/opt/VantageSuite/UserService/web/bin/NodeMockBrokerbin.node
/opt/VantageSuite/UserService/web/bin/NodeMockPtpProviderbin.node
/opt/VantageSuite/UserService/web/bin/NodeUtilsbin.node
/opt/VantageSuite/UserService/web/bin/NodeVantCommbin.node
/opt/VantageSuite/UserService/web/bin/ac17b83f95270357f2ae8a5a811dff20.node
/opt/VantageSuite/UserService/web/bin/d2ce8de1ccc38c9ec08f91a1c3b1fae3.node
/opt/VantageSuite/UserService/web/certs
/opt/VantageSuite/UserService/web/certs/SDLVantageCA.crt
/opt/VantageSuite/UserService/web/certs/server.crt
/opt/VantageSuite/UserService/web/certs/server.key
/opt/VantageSuite/UserService/web/createAdmin.js
/opt/VantageSuite/UserService/web/createAdmin.js.LICENSE.txt
/opt/VantageSuite/UserService/web/index.js
/opt/VantageSuite/UserService/web/index.js.LICENSE.txt
/opt/VantageSuite/UserService/web/user-management-app.tar.gz


Await further progress on loading multiple mbtiles in Mapeo map server

As of this time, this script generates complete assets for an offline Mapbox map in accordance to the Mapbox style spec, including a stylesheet, tiles (mbtiles, both raster and vector), fonts, and glyphs. Many map servers (such as tileserver-gl) will be able to load this map.
For Mapeo, the legacy mapeo map server is able to load such a map, except that the tiles need to be in an XYZ dir format (and typically compressed using asar) instead of mbtiles. The new Mapeo map server is built to work with mbtiles instead of XYZ dir, and the Digital Democracy team is working on support for loading vector mbtiles in addition to raster mbtiles. However, it is as of yet unclear whether we will be able to provide maps with multiple, mixed sources (raster and vector); I asked about this in a CMI / Dd collaboration document.
The new map server will have backwards compatibility to be able to load maps formatted for the Mapbox style spec (as per the requirements of the legacy map server), but it's unclear whether it will also be able to load mbtiles; I asked about that in this issue. If not, then to get the offline map assets working via this method, we will have to adapt the script to generate tiles in XYZ directory format instead of mbtiles, and then compress them using asar. This in and of itself is not a problem, however then we will likely need to add the offline map assets using a USB connection or adb instead of using the new Mapeo background map manager UI, which is not optimal for our users.
The other alternative will be to generate a final raster mbtiles based on the output stylesheet (which overlays and styles the two mbtiles sources). To my understanding, the only way to do this without manually using a desktop application like QGIS is to use tilelive-copy from tilelive in tandem with mapbox-gl-native and tilelive-gl, and this is rather unwieldy as some of these modules (or their deps) are working with very early and outdated versions of Node and Python.
At this point, I would like to wait on input from the Mapeo team about future support for styles with multiple mbtiles sources.
For ODK/Kobo, since the tools can only take either a raster or vector mbtiles, unfortunately to get both the vector (change detection alert) and raster (satellite imagery) working at the same time, we will actually need to bake a composite raster mbtiles that overlays and styles both sources... probably using tilelive unless we can find another automated method. But I'll wait to pursue that until we decide that we actually want to start using these tools for change detection.

The other alternative will be to generate a final raster mbtiles based on the output stylesheet (which overlays and styles the two mbtiles sources). To my understanding, the only way to do this without manually using a desktop application like QGIS is to use tilelive-copy from tilelive in tandem with mapbox-gl-native and tilelive-gl, and this is rather unwieldy as some of these modules (or their deps) are working with very early and outdated versions of Node and Python.

I figured out a way to do this using tileserver-gl and added it to the script. So we have a workaround of using raster mbtiles for the time being.

As of today, there is no anticipated timeline for implementing a way to add multi-source styles (e.g. a way to add both vector and raster) to the CoMapeo Map Manager.
There will be backwards compatibility for the old Mapeo "offline map as a folder" format (which entails using xyz/ tiles, either in a directory or compressed .asar format). This is a way to have both vector and raster sources in the same style. However, the user experience for adding these is not great - it involves needing to download the files on a computer and then adding them to a mobile device file directory using USB or adb. Far too cumbersome for the timely addition of change detection alerts circulating on a regular interval, in my strong opinion. (And a big user pain point of the legacy Mapeo in general.)
ODK does not (yet) offer us an alternative - it, too, can only use one mbtiles file as an offline background map at a time, either raster or vector.
Hence, if we want to circulate offline map resources for mobile data collection to our users in Q1/Q2 of 2024, I expect we will need to find a way to get FastAPI to generate a composite raster mbtiles.

I have not evaluated or researched these options yet, but documenting what I've learned by asking around about ways to generate a rendered raster mbtiles:
Tilelive-gl and timeline-copy

You could use something like tilelive-gl and timelive-copy
https://github.com/trailbehind/tilelive-gl
But that uses old mapbox-gl-native and probably won’t build anymore so it would need to be updated for maplibre

Maplibre GL JS / capacitator

I'm using maplibre gl js, capacitor and addProtocol to achieve offline maps, works pretty well.

Martin

Martin (https://github.com/maplibre/martin) has a tool martin-cp to copy tiles from any sources supported by Martin itself. Sadly, no rasterisation support just yet.  It is a big TODO, and we urgently need API for MapLibre.native for that
the moment martin supports rusters, so will martin-cp

Using a browser wrapper

It would be possible to do this as a browser app, using the maplibre-gl rendering library to render tiles to canvas and then reading the canvas to PNGs or JPEGs, and then writing to SQLite in the browser, but it would be quite a bit of (fun) work. You could also create a node module using the maplibre node renderer (which I think tileserver-gl uses under the hood) and directly writes to mbtiles, and maybe wrap that up in an electron app or even better maybe https://tauri.app/. Bother approaches would be fun challenges, but I'm not sure if anything exists already to do this. Years back there was tilemill which does a lot of what you need - it was actually the precursor to Mapbox really. It uses mapnik to render map tiles. It looks like it's not maintained however, and seems to be limited to node v8 which is super old and probably not supported in a lot of places nowadays.

https://github.com/conservationMetrics/mapgl-tile-renderer/ was born to meet this need, so closing the issue.


fix(*-writer): remove the incomplete JSDoc annotations

To lower the expectations for JS code completion, all related JSDoc annotations have been removed. This ensures that for all UI5 modules do not declare the types of the required modules.
The addition of the tsconfig.json and the usage of the ESM types is ok and could be a started for interested developers.
If developers are interested in using code completion for JS they should best take a look into the following documentation: https://github.com/SAP-samples/ui5-cap-event-app/tree/js-with-typescript-support#applying-typescript-benefits-to-a-javascript-application
This explains in detail what is possible with JSDoc annotations and what are the limitations. But developers should decide whether to use it or not as we cannot provide a proper tooling support for this.
If developers want to benefit from full code completion support the best option is to use TypeScript:
https://sap.github.io/ui5-typescript/

Thanks for merging, just noticed it today...


Block editing of local cluster name

The local cluster's name should stay as "local" as it is hardcoded that way in many places. Disable editing of the cluster name in the UI.

There should be an isLocal property on the management cluster that denotes if it's the local one.

It looks like this was always possible in the old ember ui. In the dashboard we embed that page. For RKE2 types we do allow them to change it to local, which I guess is the same problem in reverse.
However, if we have the flag, what depends on the 'local' name? @vincent99 Is it valid to be able to rename the 'local' cluster? Plus then rename another non-local cluster as 'local'?

@deniseschannon I didn't intend for this to be in 2.6.0 or a release blocker, as Richard said it was always possible. Just something that came out of another issue.

Thanks @cbron - I'll bump to 2.6.X and remove the release blocker label

There's probably not anything wrong here?  The real metadata.name (norman "id") is never editable in k8s, but Ember/rke1 displays and lets you edit displayName any time, which you (backend/@cbron) shouldn't care about or ever look at.

Ahh, Vince is right, I was thinking we were changing the real name here and not display name. I am going to close this and re-open the other issue around potential problems when changing it.


Wrong x86_64 kernel set

In the imagehub it is set to use the standard x86_64 kernel, not the Docker one.

Fixed


No organization found

I updated my sentry and set ldap as well by following instruction, but after I try to login I face this page

when you click on please click here it navigate to our organization sentry, but how I could fix this issue to enter directly to our organization. I set defaul-auth-ldap as well but no difference.
I really appreciate to help me for solving this issue

Did you set AUTH_LDAP_SENTRY_DEFAULT_ORGANIZATION (recommended, organization slug) or AUTH_LDAP_DEFAULT_SENTRY_ORGANIZATION (deprecated, organization name)

when I set AUTH_LDAP_SENTRY_DEFAULT_ORGANIZATION I have received bad gateway

Take a look at the logs.


SAA-2223 change unfit to work to unpaid

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

Project coverage is 84.57%. Comparing base (0232862) to head (110af34).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #1206   +/-   ##
=========================================
  Coverage     84.57%   84.57%
  Complexity     3757     3757
=========================================
  Files           385      385
  Lines         12767    12767
  Branches       1098     1098
=========================================
  Hits          10798    10798
  Misses         1459     1459
  Partials        510      510

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


How do you get the current index of the current FlashcardArray card?

How do you get the current index of the current FlashcardArray card?

Since it’s a component library, I didn’t make any hooks to go along with it but you can keep track of card indexes locally using the onCardChange callback prop.
This function will be called every time the card is changed. The function will pass these parameters for you to keep track of the current card.
(id: any, index: number) => {}

Thanks, had a bit of trouble getting this to work but got it sorted in the end, thanks for super quick responses - great little component


Test control message / clean shutdown mechanism

This PR includes two new tests for ControlMessage mechanism:

block-fetch application
tx-submission application

There was a failure on windows (not reproducible on linux), captured below as a QuickCheck property:
prop_terminate_fail :: Property
prop_terminate_fail =
    prop_terminate
      (TestChainFork
         Chain.Genesis
         (Chain.Genesis Chain.:>
             Block {blockHeader = BlockHeader {headerHash = HeaderHash<PHONE_NUMBER>4089534, headerPrevHash = GenesisHash, headerSlot = SlotNo 3, headerBlockNo = BlockNo 1, headerBodyHash = BodyHash (-3649366451720810582)}, blockBody = BlockBody "VAAN"})
         Chain.Genesis)
      (Delay 0.026033306121)

Relies on #2631 which should be merged first.

bower merge

bors merge


Static Web Site and Tab only projects

Is your feature request related to a problem? Please describe.
Make it easier to create and publish client side only apps.
Describe the solution you'd like
I'd like to see an option where yo teams scaffolds a project with ONLY tab support - and with the possibility to publish to a static web site. That is - not spinning up express or anything like that when in production.

Thanks for this. Will take a look at it ASAP.

I agree...just need the results of the client side.... I am irritated by how slow the "gulp serve" is.  To make it worse, it always crash and I have to restart it.  Don't know what its problem:
FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
1: 00007FF706747DDA v8::internal::GCIdleTimeHandler::GCIdleTimeHandler+4506
2: 00007FF706722876 node::MakeCallback+4534
3: 00007FF7067231F0 node_module_register+2032
4: 00007FF706A3B6BE v8::internal::FatalProcessOutOfMemory+846
5: 00007FF706A3B5EF v8::internal::FatalProcessOutOfMemory+639
6: 00007FF706C21DE4 v8::internal::Heap::MaxHeapGrowingFactor+9620
7: 00007FF706C18DC6 v8::internal::ScavengeJob::operator=+24550
8: 00007FF706C1741C v8::internal::ScavengeJob::operator=+17980
9: 00007FF706C20167 v8::internal::Heap::MaxHeapGrowingFactor+2327
10: 00007FF706C201E6 v8::internal::Heap::MaxHeapGrowingFactor+2454
11: 00007FF706D4A487 v8::internal::Factory::NewFillerObject+55
12: 00007FF706DC7676 v8::internal::operator<<+73494
13: 000001ABE735C5C1

What if we added support for running your tabs with react-scripts, but still scaffold the server side (which don't have to use)?


Looking for some examples of basic FASTA/Q handling with Bio::DB::HTS

First, thanks for the work on this and for making it available on CPAN. I would like to know if there are any plans to add bindings to the FASTA/Q parsing methods in kseq.h of htslib? This may be possible already or by some other method but it is not clear. For reference, @cjfields has already written bindings to kseq.h in the Bio::Kseq module, which is extremely useful because it is much faster than any pure-Perl parser. I think it would be great to have those basic sequence reading methods in one package along with all the alignment methods.

Just a note: the code in Bio::Kseq hasn't been released to CPAN and has only basic XS bindings in place.  That being said, it does seem to work fine; if needed I would support it being subsumed into Bio-HTS.

Thanks for raising this. I will have a further look at the repo and if it's OK to bringing it in can do so. One non-technical question that will come up is one of the licencing - just to make it clear what the existing code stands - @cjfields would it be possible to add the Apache2 licence? I think it would simplify matters. Thanks

Fasta (indexed with samtools faidx) is already handled by Bio::DB::HTS::Faidx but the docs are a little mangled.  Here's a basic example which grabs the last 10 bases of each sequence (mouse 37 used here) and takes ~0.05 of a second over NFS (file is 2.6GB):
testFaidx.pl
#!/usr/bin/env perl
use strict;
use Bio::DB::HTS::Faidx;

my $ref = shift @ARGV;
my $index = Bio::DB::HTS::Faidx->new($ref);
my @seq_ids = $index->get_all_sequence_ids();
for my $seq_id(@seq_ids) {
  my $full_length = $index->length($seq_id);
  my $location = sprintf '%s:%d-%d', $seq_id, $full_length-9, $full_length;
  my ($seq, $this_length) = $index->get_sequence($location);
  printf "result_length %d, sequence %s, seq_id %s\n", $this_length, $seq, $seq_id;
}

Output
$ ls -lh mouse_37_genome.fa
-r-xr-x--- 1 cgpref cancer 2.6G Aug 21  2015 mouse_37_genome.fa

$ time perl testFaidx.pl mouse37_genome.fa
result_length 10, sequence ACTGGAATTC, seq_id 1
result_length 10, sequence CCTATGAATT, seq_id 10
result_length 10, sequence GTTAGGGTTA, seq_id 11
result_length 10, sequence GAATGAATTC, seq_id 12
result_length 10, sequence GAAGGAATTC, seq_id 13
result_length 10, sequence TCAGGAATTC, seq_id 14
result_length 10, sequence AATGGAGATC, seq_id 15
result_length 10, sequence AATTGAATTC, seq_id 16
result_length 10, sequence ATGTAGGATC, seq_id 17
result_length 10, sequence TTAGGGTTAG, seq_id 18
result_length 10, sequence GCCTGCTTGC, seq_id 19
result_length 10, sequence GATTACGGGT, seq_id 2
result_length 10, sequence ACTCAACTTC, seq_id 3
result_length 10, sequence GGCATTGGGT, seq_id 4
result_length 10, sequence GGTTAGGGTT, seq_id 5
result_length 10, sequence TCAGCCTTTC, seq_id 6
result_length 10, sequence GTGCTGAAGT, seq_id 7
result_length 10, sequence CTTGTACAAC, seq_id 8
result_length 10, sequence GTGGTAATAG, seq_id 9
result_length 10, sequence ACATTAACAA, seq_id MT
result_length 10, sequence GTTAGGGTTA, seq_id X
result_length 10, sequence NNNNNNNNNN, seq_id Y

real	0m0.051s
user	0m0.020s
sys	0m0.012s

@keiranmraine True, but the KSeq bindings also process FASTQ.  It is specifically tuned for use in processing and filtering large sequence data sets (seqtk is the analogous tool for this as samtools is for BAM).
@rishidev I'm open to whatever licensing works, but to be honest I prefer BSD or MIT.

@cjfields I was just providing an example specifically for FASTA in case it was of use

@keiranmraine not a problem

@cjfields Would you be OK to make a pull request with kseq added in? That would clear up licencing compatibility concerns.

@rishidev sure, I'll set up a fork and start on this.

@cjfields Thanks

Has there been any progress on this?
I'm worried that we're suggesting to introduce new C dependencies into Bio-HTS. My view on this is Bio-HTS should provide access to routines/functions available from the htslib package. If it's not in htslib then I'm against bringing the code in.

There are functions in kseq.h which is from HTSlib. The Bio::KSeq library as it exists builds for a different kseq.h. The porting plan would be to build for the HTSlib kseq.h, but I don't know if @cjfields has had a chance to look at this yet..

Ah okay I had misunderstood this point. Apologies

Nope, hadn't had a chance to be honest, the last four months have been hellish
@andrewyatz @rishidev the header file is present in the HTSLib distribution, but I don't believe the kseq.h functions are 'public' and available directly through the compiled HTS library (libhts I think).  I suspect they are primarily only used under the hood for FASTA.  Bio::KSeq has a compilation step during build to directly compile the C header for the module, but I'm not sure how much that would complicate Bio::HTS compilation.

I'm not sure it would complicate it at all @cjfields. If my understanding is correct by doing #include <kseq.h> you're going to import the header and all of kseq's code into Kseq.xs. From there you compile the XS as we need to do anyway and out pops the compiled libs.
I've done a quick check by pointing your compilation code away from the klib submodule to my local set of htslib headers and kseq.h. I got a working module out of it and 91 very happy test cases.
Looking quickly at the code if kseq was to exist as a separate module under say Bio::DB::HTS::KSeq and given POD then it shouldn't be a huge drama.

Thanks for opening the conversation again @andrewyatz. The only thing I'd like to add is that the Bio::Kseq module should probably be redesigned slightly so that the iterator returns an object with methods for accessing the id, seq, and qual string. Currently, it returns a hash. This is not a major issue, but it would be inconsistent with the behavior of methods in other Bio::DB::HTS classes.

It would be a small change. In @cjfields implementation it seems the iterator object has access to the fields in question. Anyway I think it's relatively simple to bless an object in XS though the exact way escapes me. That way the returned object can be used as a hash or object or you can just use the iterator and ignore all of this other stuff (probably the most efficient way).

I took a look at this over the weekend and built something on 55d2055 (feature/kseq). Take a look and see what you think. Usage looks like
use feature 'say';
use Bio::DB::HTS::Kseq;
my $kseq = Bio::DB::HTS::Kseq->new('path/to/file');
my $iter = $kseq->iterator();
while(my $seq = $iter->next_seq()) {
  say $seq->seq();
}

The iterator returns a blessed hash object so all of @cjfields code continues to work with minor modifications to the packages involved. Apart from that I think it works and I've not noticed any major memory leaks.

Done a quick test of the memory usage and it seems very static. I think if people are happy with the integration (and licensing) then it can go in without much more fuss.

Just done some memory tests on this and usage is static. PR #24 is open for further comment.

Seems that @rishidev has just merged this. So I'm marking the thread as closed. @cjfields and @sestaton can you take a look at the APIs and confirm you're okay with them. This work has been licensed under Apache2.0. @cjfields can you say if this is okay?

Everything looks good to me. Thanks for the work on this everyone! It is fine with me to mark this closed, but I would like to know if there will be a new CPAN release with these changes. That would be very helpful.

@andrewyatz @sestaton works for me, just tested it out and everything works fine.  Thanks all, in particular special thanks to @andrewyatz for the effort in moving the code in!

Great news and thanks @cjfields but it wouldn't be in if not for your code. @sestaton I believe @rishidev is winding up for a new release soon so this will make it onto cpan soon.


Update package.json exports to be default condition

:rocket: PR was released in v1.13.4 :rocket:


Cannot build IDE Extension project

I'm creating empty new IDE Extension project, then doesn't matter if I update the MonoDevelop.Addins package or not the build fails with "error : Could not resolve addin reference 'MonoDevelop.Core'" and "error : Could not resolve addin reference 'MonoDevelop.Ide'".
I tried rebuilding and updating the add-in registry with vstool setup reg-update and reg-build but this doesn't help.
=== Visual Studio Community 2017 for Mac ===
Version 7.1.5 (build 2) Installation UUID: dbcd7d63-12ef-4166-affa-15f92e4e31a2 Runtime: Mono <IP_ADDRESS> (d15-3/14f2c81) (64-bit) GTK+ 2.24.23 (Raleigh theme) Package version: 502000224
=== NuGet ===
Version: <IP_ADDRESS>8
=== .NET Core ===
Runtime: Not installed SDK: Not installed MSBuild SDKs: /Library/Frameworks/Mono.framework/Versions/5.2.0/lib/mono/msbuild/15.0/bin/Sdks
=== Xamarin.Profiler ===
Version: 1.5.5 Location: /Applications/Xamarin Profiler.app/Contents/MacOS/Xamarin Profiler
=== Apple Developer Tools ===
Xcode 9.0 (13247) Build 9A235
=== Xamarin.Mac ===
Version: <IP_ADDRESS> (Visual Studio Community)
=== Xamarin.iOS ===
Version: <IP_ADDRESS> (Visual Studio Community) Hash: 152b654a Branch: xcode9 Build date: 2017-09-15 02:25:56-0400
=== Xamarin.Android ===
Version: <IP_ADDRESS> (Visual Studio Community) Android SDK: /Users/mstefanov/Library/Developer/Xamarin/android-sdk-macosx Supported Android versions: 6.0 (API level 23) SDK Tools Version: 24.4.1 SDK Platform Tools Version: 23.0.1 SDK Build Tools Version: 23.0.1 Java SDK: /usr java version "1.8.0_101" Java(TM) SE Runtime Environment (build 1.8.0_101-b13) Java HotSpot(TM) 64-Bit Server VM (build 25.101-b13, mixed mode) Android Designer EPL code available here: https://github.com/xamarin/AndroidDesigner.EPL
=== Xamarin Inspector ===
Version: 1.3.1 Hash: cbc48dd Branch: 1.3-release Build date: Thu, 21 Sep 2017 19:52:53 GMT Client compatibility: 1
=== Build Information ===
Release ID: 701050002 Git revision: 7afedcaef8e7542e70e3cf8f9bdb26938b8c0876 Build date: 2017-09-15 08:39:58-04 Xamarin addins: 3262aadf811a18c12eac6742532d052b0139a808 Build lane: monodevelop-lion-d15-3-xcode9
=== Operating System ===
Mac OS X 10.12.6 Darwin 16.7.0 Darwin Kernel Version 16.7.0 Thu Jun 15 17:36:27 PDT 2017 root:xnu-3789.70.16~2/RELEASE_X86_64 x86_64
=== Enabled user installed addins ===
AddinMaker 1.3.7 Manifest.addin <IP_ADDRESS>

running into the same issue using AddinMaker 1.3.7 in MD 7.1 on ubuntu 16.04

It seems after the latest update of VS for Mac I'm able to build.
@germgerm There is a Linux specific issue you can check.


RuntimeError: Selective_scan only supports state dimension <= 256

When using mamba, I found the following RuntimeError:
  File "/data2/user/workspace/D-Mamba/d_mamba/models/mamba.py", line 171, in forward
    h = self.mamba(h)
  File "/home/user/miniconda3/envs/py39/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/user/miniconda3/envs/py39/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/user/miniconda3/envs/py39/lib/python3.9/site-packages/mamba_ssm/modules/mamba_simple.py", line 146, in forward
    out = mamba_inner_fn(
  File "/home/user/miniconda3/envs/py39/lib/python3.9/site-packages/mamba_ssm/ops/selective_scan_interface.py", line 306, in mamba_inner_fn
    return MambaInnerFn.apply(xz, conv1d_weight, conv1d_bias, x_proj_weight, delta_proj_weight,
  File "/home/user/miniconda3/envs/py39/lib/python3.9/site-packages/torch/autograd/function.py", line 539, in apply
    return super().apply(*args, **kwargs)  # type: ignore[misc]
  File "/home/user/miniconda3/envs/py39/lib/python3.9/site-packages/torch/cuda/amp/autocast_mode.py", line 113, in decorate_fwd
    return fwd(*args, **kwargs)
  File "/home/user/miniconda3/envs/py39/lib/python3.9/site-packages/mamba_ssm/ops/selective_scan_interface.py", line 217, in forward
    out, scan_intermediates, out_z = selective_scan_cuda.fwd(
RuntimeError: selective_scan only supports state dimension <= 256

Here is the code I use:
from mamba_ssm.modules.mamba_simple import Block, Mamba

batch, length, dim = 16, 64, 512
x = torch.randn(batch, length, dim).to('cuda')
model = Mamba(
    # This module uses roughly 3 * expand * d_model^2 parameters
    d_model=dim,  # Model dimension d_model; = `dim` in this case
    d_state=dim,  # SSM state expansion factor; should be the same to `dim`
    d_conv=4,  # Local convolution width
    expand=16,  # Block expansion factor; should be the same to `batch` size
).to('cuda')
y = model(x)
print(y.shape)

Why there is restriction for the dim? It is not proper for most use cases.

I don't think the comments match the parameters they purport to describe in your Mamba instantiation. E.g. d_state is  the number of states in the linear dynamic SS models, and should not in general "be the same as 'dim'". Nor do I see why expand should in general be the same as batch size.
The values you assign are radically different from those of the trained models the authors published. You have d_state = dim = 512, but d_state is 16 in the published models. You set expand=16, but it is 2 in the published models. I am doubtful that these values you are trying to use would make sense in most circumstances, at least without more explanation.
Selective scan is designed to exploit the memory cache hierarchy of GPUs; letting the SSM get too big will no longer fit the hardware.
Also, the dynamic models represented are many single-input single-output (SISO) dynamic systems corresponding to a system of uncoupled differential equation of order d_state=16. A 16-th order linear dynamic system is already a pretty high order, especially considering there are thousands of them per block. Letting it get more than an order of magnitude bigger would be of dubious value, at least in the use cases I've seen for transformer type sequence-to-sequence models.

To put a finer point on it, the authors did ablation studies and the paper shows the effect of state size in table 10. There was not much improvement in perplexity in going from state size 8 to 16. So 16 is probably as big as makes sense in the case of language models.

Yup, d_state here means each dimension of d_model will be expanded by a factor of d_state. We used d_state=16 in our experiments.
Thanks @jyegerlehner for the detailed explanation.

@jyegerlehner @tridao Thank you for your explanation!


fixed wrong I2C labels

I2C1 signal mapping was corrected while preparing rev.1.1. of the design for fabrication (see the https://github.com/antmicro/jetson-orin-baseboard/commit/94c4fd80cc312c070bfa282dfb547830983e1d5c#diff-979ce83e23918e705ca42628b0e5339dad93eee530aa5b5cca65ceab46615b37).


Switch to new Cachix in CI

Changed

Switch to new cargo2nix cache in CircleCI. This change is coupled with the addition of CACHIX_SIGNING_KEY to the build-time secrets.
Push nix-build output inside CI build step, using this configuration as reference.

Should hopefully fix CI for #89.

I think I'll build the ci attribute locally with musl, populate the new Cachix cache, and re-trigger CI here.

Local build is almost complete. I will retrigger CI very soon.

Okay, I've learned a few things about running Nix inside CircleCI on the free open-source project plan:

As described in https://github.com/tenx-tech/cargo2nix/pull/89#issuecomment-593239136, the build jobs often get killed by CircleCI once the Docker image consumes more memory and CPU than the free medium instance will allow.
You can work around this by building cargo2nix locally, pushing the build artifacts yourself up to Cachix, and then re-triggering CI, which will copy the artifacts back down from the cache.
nix-build -A ci | cachix push only pushes the final build artifact up to the cache. In this case, the resulting NAR from the test runner references none of the tests' build inputs, so a nearly empty derivation gets copied over. The real solution to populate the Cachix cache is to run:nix-build -A rustPkgs.workspace.cargo2nix | cachix push cargo2nix

Only then will nix-build -A ci in CircleCI complete within a reasonable time and with low enough memory consumption such that it won't be automatically killed.

@trha I'm documenting this here just in case we run into these CI issues again. 😄

Nice! It looks like Cachix is actually working now, since there are now lines like these appearing in the build output:
compressing and pushing /nix/store/llv0s3c99bwjd9y4ibjdizwnyf6aw035-crate-rand_core-0.5.1-x86_64-unknown-linux-musl (220.44 KiB)


Failed login not sending email!

In laravel 11 failed login email wont come. could you please take a look?

Can you provide information about your issue? Like any errors and steps for reproduction.

I actually tried this recently and noticed the same issue as well! After the configurations, i attempted a failed login  and received no emails, but the log information is saved in the database table "authentication_log"

Have you set the environmental variables or updated the config file?


TD 2658 Expose Ref for Wizard Content

Contributes to TD-2658
Changes
Updates the WizardContent to pass on a ref to the underlying box component. This is done so that the scroll postion of the scroll bar in the WizardContent can be set.
Dependencies
N/A
UI/UX
N/A
Testing notes

Not much to test here, just check that the ref forwarding looks correct.

Author checklist before assigning a reviewer

[x] Reviewed my own code-diff.
[x] Branch has been run in docker.
[x] PR assigned to me or an appropriate delegate.
[x] Relevant labels added to the PR.
[x] Appropriate tests have been added.
[ ] Lint and test workflows pass.

Tests are failing due to flaky playwright tests


[DF] Port TableScan logic to Datafusion

Is your feature request related to a problem? Please describe.
Update all existing "TableScan" logic to replace Apache Calcite data structures with Apache Datafusion data structures.
Describe the solution you'd like
Replace instances of Calcite LogicalTableScan with correlating instances of Datafusion TableScan. This includes any base classes, functions, members that might also rely on that logic.
Describe alternatives you've considered
None
Additional context
None

This issue was resolved as part of the PR


Non public field org.example.customer.IgnoreField middleName with no matching setter or constructor?

Hello.
Whether the ignored serialized fields should be filtered, because the get/set method may not be necessary for the ignored fields.
@Json
public class IgnoreField {

  private String firstName;
  @Json.Ignore
  private String middleName;

  public IgnoreField() {
  }

  public IgnoreField(String firstName, String middleName) {
    this.firstName = firstName;
    this.middleName = middleName;
  }

  public String getFirstName() {
    return firstName;
  }

  public IgnoreField setFirstName(String firstName) {
    this.firstName = firstName;
    return this;
  }

  public String getMiddleName() {
    return middleName;
  }
}

I expect to ignore the serialization of middleName and do not provide a public set method. At present, compilation will prompt errors and cause compilation failure.
Thanks.

Nice bug, thanks !!


Got Error when an input image with specific size was feed to the detection module.

Environments
Ubuntu16.04
Python 2.7
PyTorch 1.1.0
TorchVision 0.3.0

How to reproduce
Upgrade torchvision version to 0.3.0, run the following code with python 2.7:
import torchvision
import torch

model = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained=True)
model.eval()
x = [torch.rand(3, 900, 1000)]
predictions = model(x)

Got Error message:
Traceback (most recent call last):
  File "detection-test.py", line 9, in <module>
    predictions = model(x)
  File "/usr/local/lib/python2.7/dist-packages/torch/nn/modules/module.py", line 493, in __call__
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/torchvision/models/detection/generalized_rcnn.py", line 47, in forward
    images, targets = self.transform(images, targets)
  File "/usr/local/lib/python2.7/dist-packages/torch/nn/modules/module.py", line 493, in __call__
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/torchvision/models/detection/transform.py", line 45, in forward
    images = self.batch_images(images)
  File "/usr/local/lib/python2.7/dist-packages/torchvision/models/detection/transform.py", line 110, in batch_images
    pad_img[: img.shape[0], : img.shape[1], : img.shape[2]].copy_(img)
RuntimeError: The expanded size of the tensor (864) must match the existing size (888) at non-singleton dimension 2.  Target sizes: [3, 800, 864].  Tensor sizes: [3, 800, 888]

The model has a minimum maximum input range. Check it out. This can be set when training the model.

Fixed via https://github.com/pytorch/vision/pull/960


Logo

This doesn't make any sense due to we are using H5BP logo, https://github.com/learning-rust/site/issues/4 :)


Integration with GitHub merge-queue

When using GitHub merge-queue, GitHub creates new branch for merging pull requests.
This brings two issues.
First, the label checker is running only on pull requests:
The documentation specifies the trigger as follow:
on:
  pull_request:

This can be fixed by adding this trigger:
  merge_group:
    types: [checks_requested]

Second, even if we run the label checker on those branches to be merged, we have an issue as they are no pull requests for them, and no labels. The check fails with the error message:
Checking GitHub labels ...
2023/03/02 10:34:20 Error that we cannot handle, Could not resolve to a PullRequest with the number of 0.

What can we do?
Maybe we can detect if the action is run for a branch for the merge-queue, and always succeed in that case?

Related to https://github.com/orgs/community/discussions/13690

👍
I'll add some info to this issue if I find a solution.

OK I found a way to make it work. I hope this can help others.
name: Label Checker
on:
  pull_request:
    types:
      - opened
      - synchronize
      - reopened
      - labeled
      - unlabeled
  merge_group:
    types: [checks_requested]

jobs:
  check_labels:
    if: github.event_name == 'pull_request'
    name: Check labels
    runs-on: ubuntu-latest
    steps:
      - uses: docker://agilepathway/pull-request-label-checker:latest
        with:
          one_of: major,minor,patch
          repo_token: ${{ secrets.GITHUB_TOKEN }}

Explanations:

the action run on merge branches thanks to the on merge_group
the action does not do anything on merge branches thanks to the if: github.event_name == 'pull_request'. GitHub accepts this and the PR can be merged.


Amend AWS IoT CLI command in collect the correct endpoint

modified comments in sample AWS param files to use the correct CLI command to collect the recommended endpoint as per the documentation: https://docs.aws.amazon.com/iot/latest/developerguide/server-authentication.html

Thanks!


Old PyPI release

Hey there!
Would it be possible to release new blocksec2go version to PyPI? The current one is from March and it seems there's been a number of changes since then. Examples included in the repo don't work with the PyPI version as it doesn't include find_reader method for example and it can be a bit confusing at first :)
Cheers!

Can confirm, the version on PyPI is quite old. For now just clone the repo and manually install it via pip install setup.py... 😄

Hey there!
we made an update 😃
BR


Corelight and zeek kits have colliding IDs and UUIDs

What are the steps to reproduce this issue?

install corelight kit
install zeek kit
uninstall corelight kit

What happens?
Chunks of the zeek kit are uninstalled
What were you expecting to happen?
Not this
Any logs, error output, etc?
Any other comments?
This means you cannot install both, it looks like corelight kit was derived from the zeek kit and the IDs and UUIDs were not updated.
What versions of software are you using?
Zeek version 8
Corelight version 2

https://github.com/gravwell/kits/pull/136


custom tracker name not working

I've added the environment variable "TRACKER_SCRIPT_NAME" with the value of "t" (on vercel) but the script name is still umami.js
What can I do? Should I just change all references of umami.js to t.js ?

First of all, yes you have to change all umami.js to t.js, because the name of the script is changed, by adding this environment variable.
If it is still not working, try rebuilding the project on Vercel.
I hope it works!

In Vercel, if you add an environment variable, you have to redeploy or it won't pick it up.

@mikecao it looks like there was a regression somewhere.
In the latest version the code to generate the file was actually removed.
https://github.com/mikecao/umami/compare/v1.26.0...v1.27.0
It already happened in 1.27.0 and to be exact it was this commit: https://github.com/mikecao/umami/commit/460762bf3d11b6e3938fa1270e807bc43a8f75a4

Same issue on Netlify, re-deployed too

@rdgout That method was abandoned because it only worked during build time (which doesn't work for docker images). In v1.28.0 it uses middleware during runtime.
@laurent-d I've never been able to deploy to Netlify successfully. But it been a while since I've tried so I'll probably revisit at some point.

In Vercel, if you add an environment variable, you have to redeploy or it won't pick it up.

I have redeployed, removed/added EV, but it is not working.

@rdgout That method was abandoned because it only worked during build time (which doesn't work for docker images). In v1.28.0 it uses middleware during runtime.
@laurent-d I've never been able to deploy to Netlify successfully. But it been a while since I've tried so I'll probably revisit at some point.

What is the best way to achieve this? Change all instances of umami.js to customName.js?

@Silocoxyz are you sure you're running v1.28.0? That's where the middleware feature landed.

After setting the variable, umami.js can be accessed, and the custom name can also be accessed. Both are valid at the same time.

I think the misunderstanding is, (which I also had and was confused about) is that the TRACKER_SCRIPT_NAME would change the name of the umami.js file, when it actually acts as a redirect to it? Maybe changing .env value name to something like ALTERNATIVE_SCRIPT_NAME would clear this up more.

Same problem on Netlify, fresh fork so 1.31, set TRACKER_SCRIPT_NAME=u, did a new deploy also clearing the cache but /u.js is not accessible

I also just tried this in a docker container. The ENV variable is set to TRACKER_SCRIPT_NAME=u.js but /u.js is not accessible, while umami.js still is.
It runs the latest v1.31.0.

Same problem on Netlify for me, while COLLECT_API_ENDPOINT works just fine.


emailservice crashloop

The emailservice appears to be in a crashloop when testing on my personal project. The error message is:

google.api_core.exceptions.PermissionDenied: 403 Stackdriver Trace API has not been used in project<PHONE_NUMBER>799 before or it is disabled. Enable it by visiting https://console.developers.google.com/apis/api/cloudtrace.googleapis.com/overview?project=1093971809799 then retry. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry.

Do we need to enable the API before using stackdriver sandbox? Is this not done as part of the install script?

After doing some more research, it looks like we need to enable cloudtrace.googleapis.com on the stackdriver sandbox project as it's created

has this been encountered before? It seems like it should have been an issue from the beginning
why only email service? Do other services not use stackdriver trace?
is this something that can be added through terraform?

addressed in https://github.com/GoogleCloudPlatform/stackdriver-sandbox/pull/10


Delete archive.html

sorry I deleted it by mistake.

haha. that is fine. You didn't delete it by the way. You do not have access to change anything in this repository. So instead the project was forked (copied in to your account) where you could delete the file. And then you did a pull request - i.e. asking me to merge and accept the deletion (I declined)

you are welcome to keep a copy, but you probably just want to delete https://github.com/aserawayneasera/jekyll-hp-base-theme again

Oh ok that's a relief. 😅. Thanks Morton.
On Thu, 25 Feb 2021 at 22:03, Morten Høfft<EMAIL_ADDRESS>wrote:

you are welcome to keep a copy, but you probably just want to delete
https://github.com/aserawayneasera/jekyll-hp-base-theme again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gbif/jekyll-hp-base-theme/pull/5#issuecomment-785701620,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGURLDSIFLWPEDGAR27XV2DTAX745ANCNFSM4YFSV7AA
.

--
regards,
Wayne Asera


Possible docker deployment approach

What's the state of this PR? Should we resolve the conflicts and merge or close it?

Still want to figure out the best approach for this, but if you want to make some process with it, feel free.  My focus will be on Map and then IntersectionObserver until I've shipped those

Struggling with issues like grunt compatgen which needs the runtime environment to run.  For the moment we will use docker just for deployment, so I'll modify this PR to limit itself to deployment.

Hi Rowan - in Jake's absence can you look over this?  Jake and I decided to use docker just for deployment on the polyfill service.  I can build the image and run it locally, and I think I have the correct tasks configured for deployment, but I get this output from Heroku:
$ npm run deploy
The push refers to a repository [registry.heroku.com/ft-polyfill-service-qa/web]
0d48b43af4d4: Preparing
6d64942ab833: Preparing
976b0b9ce8a3: Preparing
538e81ce30d6: Preparing
ea74409b057b: Preparing
c56b7dabbc7a: Waiting
no basic auth credentials

There don't seem to be any public docs on using Heroku's docker registry for deployment so I assume you guys found out how to do this direct from Heroku somehow, or else are just smarter than me.  Anyway feel free to deploy (npm run deploy will deploy to QA) and if it works please let me know how to configure auth!

The origami build service should have documentation on how to deploy a
docker image to herokus registry, you need to be a part of the beta and
then register via the docker Cli herokus registry
On Thursday, 26 May 2016, Andrew Betts<EMAIL_ADDRESS>wrote:

Hi Rowan - in Jake's absence can you look over this? Jake and I decided to
use docker just for deployment on the polyfill service. I can build the
image and run it locally, and I think I have the correct tasks configured
for deployment, but I get this output from Heroku:
$ npm run deploy
The push refers to a repository [registry.heroku.com/ft-polyfill-service-qa/web]
0d48b43af4d4: Preparing
6d64942ab833: Preparing
976b0b9ce8a3: Preparing
538e81ce30d6: Preparing
ea74409b057b: Preparing
c56b7dabbc7a: Waiting
no basic auth credentials
There don't seem to be any public docs on using Heroku's docker registry
for deployment so I assume you guys found out how to do this direct from
Heroku somehow, or else are just smarter than me. Anyway feel free to
deploy (npm run deploy will deploy to QA) and if it works please let me
know how to configure auth!
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/Financial-Times/polyfill-service/pull/658#issuecomment-221784457

I can take a look. Also yep the Build Service documentation outlines how to do this. @triblondon: I'll send you Heroku's docker article via Slack in case you don't have access.

Thanks.  I tried that and got this:
Successfully built c15e9da43fcf
The push refers to a repository [registry.heroku.com/ft-polyfill-service-qa/web]
cda391d7472d: Pushed
e4d18a9393e5: Pushed
08a4cceb4026: Pushed
538e81ce30d6: Pushed
ea74409b057b: Pushed
c56b7dabbc7a: Pushed
received unexpected HTTP status: 500 Internal Server Error

Could you try it?

It deployed but I'm missing Fastly credentials, so deployvcl:qa failed. The Docker deploy worked though!

OK, I'm signed up to the beta now and deployment works perfectly.  Since the haikro method is broken for me for some reason, I'm merging so I can resume deploying.


Godot hides the error text after the crashed game is closed

Godot version:
Godot 3.1.1
OS/device including version:
ArchLinux
Issue description:
When my game crashes, Godot displays the error in the debugger, but when I close the crashed game to reduce the lag a bit, the error suddenly disappears.. This is really irritating, and was bothering me for a long time.

P.S. don't mind that Godot doesn't take focus after the error, something's wrong with my desktop environment

The stack trace is cleared now and the error is replaced with a message.

It does not look like a bug anymore.


Clarify how WaitRequestIdle() should be used.

Rod v0.28.0 used.
Hi I am trying to use WaitRequestIdle in following way:
package main

import (
	"fmt"
	"time"

	"github.com/ysmood/rod"
	"github.com/ysmood/rod/lib/launcher"
)

func main() {
	url := launcher.New().
		Headless(false).
		Launch()

	b := rod.New().ControlURL(url).Connect().Timeout(30 * time.Second)
	defer b.Close()

	page := b.Page("https://google.com")
	w := page.WaitRequestIdle()

	page.Navigate("https://wikipedia.org")
	fmt.Println(time.Now())
	w()
	fmt.Println(time.Now())

	page.Navigate("https://gmail.com")
	fmt.Println(time.Now())
	w()
	fmt.Println(time.Now())
}

It seems w() really waits only after first navigation attempt but not on second. If I create second instance before second attempt it will wait twice. But I am not sure that this is intended behaviour.
Am I using it correctly?

The wait is only for the next request. If you have another request you have to create another wait:
package main

import (
	"fmt"
	"time"

	"github.com/ysmood/rod"
	"github.com/ysmood/rod/lib/launcher"
)

func main() {
	url := launcher.New().
		Headless(false).
		Launch()

	b := rod.New().ControlURL(url).Connect().Timeout(30 * time.Second)
	defer b.Close()

	page := b.Page("https://google.com")
	w := page.WaitRequestIdle()

	page.Navigate("https://wikipedia.org")
	fmt.Println(time.Now())
	w()
	fmt.Println(time.Now())

	w := page.WaitRequestIdle()  // look at this line!

	page.Navigate("https://gmail.com")
	fmt.Println(time.Now())
	w()
	fmt.Println(time.Now())
}

If you want more options, you can use WaitRequestIdleE other than the high-level version WaitRequestIdle

BTW, for page load events it's better to use page.WaiteLoad(), WaitRequestIdle is designed for ajax requests when you click some button on the page.

Normally you don't need to use WaitRequestIdle or WaitLoad at all, you can directly use the selector page.Element, it will automatically retry until it finds the element.

Oh. Thanks. I misunderstood how it should work.
I really need WaitRequestIdle. In my case all elements can be already on page but background requests are still running. If I f.e. click on element before requests stop sometimes I receive unexpected behaviour. So after clicking on some element I want to wait while requests are finished before clicking next.
May I propose to slightly change doc string for WaitRequestIdle to empathize that function returned will only be used for one next request?

Agree, if you can make a PR to fix the doc, I'll be more than happy to merge!

Ok. I will do it slightly later. Leaving issue opened till then.

On May 4, 2020, at 3:25 PM, Yad Smood<EMAIL_ADDRESS>wrote:
Agree, if you can make a PR to fix the doc, I'll be more than happy to merge!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ysmood/rod/issues/23#issuecomment-623432942, or unsubscribe https://github.com/notifications/unsubscribe-auth/AO4BNGYVHKUMLXVS5EMEV7LRP2X4JANCNFSM4MYV7OZQ.

I see you already closed issue. Sorry that you had to do it yourself. Was away for a while. Won't happen next time.

Never mind. Looking forward to your idea and contribution 😃


Should Canvas Formatted Text expose word-break positions?

CanvasFormattedTextLineSegments include an isRTL and textMetrics information for the segment. Should the word break positions be exposed also in the CanvasFormattedTextLineSegment in order to help with accessibility?

Closing as this is tracked in the WICG repo now.


Could you please let me know how to deploy the examples in detail?

@kubawach
Hi, I built your examples with gradle, then upload the .zip file to the lambda function code on AWS console. It throws the error 'class not found'. Could you please let me know how to deploy the examples in detail?
Error loading class com.splunk.support.lambda.examples.TracingRequestWrapper: java.lang.ExceptionInInitializerError
java.lang.ExceptionInInitializerError
	at java.base/java.lang.Class.forName0(Native Method)
	at java.base/java.lang.Class.forName(Unknown Source)
Caused by: java.lang.RuntimeException: com.splunk.support.lambda.examples.RequestFunction not found in classpath
	at io.opentelemetry.instrumentation.awslambda.v1_0.WrappedLambda.fromConfiguration(WrappedLambda.java:51)
	at io.opentelemetry.instrumentation.awslambda.v1_0.TracingRequestWrapperBase.<clinit>(TracingRequestWrapperBase.java:30)
	... 2 more

Hi @puremjlee thanks for your report. General advice is to deploy the examples using SAM as described in README.MD files (eg https://github.com/signalfx/splunk-otel-java-lambda/tree/main/examples/splunk-wrapper)
I haven't tried uploading the ZIP manually, but I guess it should just work. If you still prefer the manual way, please describe the scenario completely (which example bundle, gradle command, AWS console steps).

@puremjlee has above ^^ fixed your problem? Let us know, otherwise I'll assume it did and close this issue.

I have tried to implement this for my own functions but got the same error.
To see if this was my implementation or the wrapper not working, I have cloned this repo and tried to build everything but this fails already due to dependencie retrieved from Bintray which gives a 403 (bintray is not public anymore)
On February 3 2021, JFrog announced that they will be shutting down Bintray and JCenter. This post tells you what you need to know and do to avoid disruptions to your build pipelines.2

This is only applicable for the splunk-wrapper so I have deleted this one and tried to deploy the splunk-layer but this fails with the same error as @puremjlee
Error loading class com.splunk.support.lambda.examples.TracingRequestWrapper: java.lang.ExceptionInInitializerError
java.lang.ExceptionInInitializerError

Also commenting to give this some traction. Done pretty much te same as Drissamri, first thing you bounce on are the Bintray dependencies in the splunk-wrapper.

FYI - PR removing offending repos has just been opened. Anyways, I recommend deploying examples using sam build && sam deploy. Let me know about the results.

Can you remove your .m2 folder and try again?
* What went wrong:
Execution failed for task ':splunk-wrapper:compileJava'.
> Could not resolve all files for configuration ':splunk-wrapper:compileClasspath'.
   > Could not find com.signalfx.public:otel-java-lambda-wrapper:0.0.12-SNAPSHOT.
     Searched in the following locations:
       - file:/Users/drissamri/.m2/repository/com/signalfx/public/otel-java-lambda-wrapper/0.0.12-SNAPSHOT/maven-metadata.xml
       - file:/Users/drissamri/.m2/repository/com/signalfx/public/otel-java-lambda-wrapper/0.0.12-SNAPSHOT/otel-java-lambda-wrapper-0.0.12-SNAPSHOT.pom
       - https://repo.maven.apache.org/maven2/com/signalfx/public/otel-java-lambda-wrapper/0.0.12-SNAPSHOT/maven-metadata.xml
       - https://repo.maven.apache.org/maven2/com/signalfx/public/otel-java-lambda-wrapper/0.0.12-SNAPSHOT/otel-java-lambda-wrapper-0.0.12-SNAPSHOT.pom
     Required by:
         project :splunk-wrapper

@drissamri how's your testing? Managed to run the examples?

Out of side conversation - seems that the examples run successfully, so closing the ticket.


chore : Update CRC GitHub issue template to be more interactive (#4494)

Description
Fixes: #4494
Relates to: #4494
CRC project is using quite old style issue template. Update it to be more interactive so that users can provide information easily.
You can check the preview of how it would look like here .
I've only ported relevant fields that were present in old template. If you think something else needs to be added, please add a comment on the pull request.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change
[X] Chore (non-breaking change which doesn't affect codebase;
test, version modification, documentation, etc.)

Proposed changes
Update GitHub Issue template to use new issue template format.
Testing
I've tested the template on my github fork. You can try it out here.
Contribution Checklist

[x] I have read the contributing guidelines
[x] My code follows the style guidelines of this project
[x] I Keep It Small and Simple: The smaller the PR is, the easier it is to review and have it merged
[x] I have performed a self-review of my code
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I tested my code on specified platforms

[ ] Linux
[ ] Windows
[ ] MacOS

Looks good to me!

/lgtm


Wrong check for the control <IP_ADDRESS>

name: "SCORED | <IP_ADDRESS> | PATCH | L1 Ensure Enumerate local users on domain-joined computers is set to Disabled MS only"
win_regedit:
path: HKLM:\Software\Policies\Microsoft\Windows\System
name: EnumerateLocalUsers
data: 0
type: String
when:

rule_18_8_28_4
tags:
level1-memberserver
rule_<IP_ADDRESS>
patch

Hey,
Looking at this one I'm not sure it should be a string. Like with other I could be wrong but based on the two below links I think it should be dword still. Let me know if I'm wrong on this one.
https://admx.help/?Category=Windows_10_2016&Policy=Microsoft.Policies.WindowsLogon::EnumerateLocalUsers
I know this is STIG, but it's the same control
https://www.stigviewer.com/stig/windows_10/2015-11-30/finding/V-63633
George

Hi,
Sorry my bad? it's correct. maybe a wrong copy/paste error from my side.
Sorry
We can close this issue.


Question about compress features of long_mem

I'd like to know if you have considered using the similarity of image features to compress long-term memory. If so, have you encountered any difficulties that prevent you from removing outdated features based on their usage?

We haven't but that might be a good idea.

thank you for your reply !


forking mode

https://github.com/brockelmore/rust-cevm/commit/30eefd0dde797554a2548ce4b44ed64d4b48c44c pretty clean
https://github.com/brockelmore/rust-cevm/search?q=fork+backend
if we can parallelize requests with joins, multithreaded tokio runtime etc. to get very high performance that'd be great

Base RPC support was added in #42. Leaving this open for implementing the other forking methods above.

Seems this can be closed since we now have forking


Add PullRequest to Issue struct

If an issue has a pull request, a pull request JSON payload is returned. That JSON payload should be parsed into a PullRequest struct and attached to the Issue.

Is this still considered open? It sounds pretty straightforward.

Looks like this was added in 32d2b56.

Agreed.  Closing issue. Thanks, @wlynch.


Add revoking certificates support

Add revoking certificates support
Why i am copy crl.pem: http://stackoverflow.com/questions/22758197/why-openvpn-cannot-read-crl-pem-xhost-cookbooks-openvpn

This is conflated with PR #19, and should be a separate commit. I see e8bda6704d2b894f6201a31f09a15208ce456d02 as containing this feature. The approach used in the template is not very Chef like - templates should only contain code that renders content, not for calling things like FileUtils#cp.

Also note that the server config has been refactored to be used in the openvpn_conf LWRP.

Relates to https://github.com/xhost-cookbooks/openvpn/pull/82.


BUG:  Poor alignment of content insid cards of  Agroblogs section

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
In the Agroblogs section, the card content is not aligned properly.the section is not responsive. The card size is much bigger than needed. The date and time on these cards are touching the boundaries, making them not properly visible. This results in a poor look for the website. I can solve this alignment issue and beautify the website. Please assign this issue to me.

#GSSOC'24 #CONTRIBUTOR
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

i want to fix this issue.please assign me to work on it.

I want to work in this issue. Please assign me this task.


Adding webhooks routes

WIP
Relates to https://github.com/lucianopf/serverless-webhooks-service/issues/3


Allow pytest

This PR assumes #82 is already merged. It's diff will be considerably smaller once #82 is merged.
I saw pytest was commented out in .circleci/config.yml and thought .. well pytest is pretty common nowadays.
So I renamed a file from test_all.py to test_replays.py and commented pytest in.

This also adds an important check, i.e.:
is this package really pip installable?

Oh! A very very good point, I totally missed that!

@cclauss and @StoicLoofah can this be merged?


Paging not working as expected

Sorry for the double post.. but since codeplex is being shut down..
I seem to have run into a snag with paging. I know Active Directory usually has a maximum page size of 1000. So, in my code, I'm using query.InPagesOf(900) just to be on the safe side.
When I first wrote that, I tested in my lab that has about 100 users. So I set query.InPagesOf(10), and verified that I'd get all results (I do). So I figured things work as desired.
Now I'm in the productive environment, and query.InPagesOf(900) gets me 900 results. query.InPagesOf(100) gets me 100 results, query.InPagesOf(1000) gets me 1000 results, and so forth. So basically, the value I give to InPagesOf is the maximum number of results I get.
When I change the code to automatic paging, so query.ToList(), then I get 500 results (I guess that's the default page size then). But that leaves out everything above 500.
Then I tried manual paging as described here: https://linqtoldap.codeplex.com/wikipage?title=Paging
But, page.HasNextPage is always false.
So, how do I get all results without blowing up whatever maximum search result size the LDAP server has set?
As a workaround, I wrote my own pagingg
private List<T> readPagedManual<T>(IQueryable<T> query, int pageSize)
    {
        if (pageSize > 0)
        {
            List<T> results = new List<T>();
            int nbPages = 1;
            List<T> page = query.Take(pageSize).ToList();
            results.AddRange(page);
            while (page.Count == pageSize)
            {
                page = query.Skip(nbPages * pageSize).Take(pageSize).ToList();
                results.AddRange(page.);
                nbPages++;
            }
            return results;
        }
        else
        {
            return query.ToList();
        }
    }

But that requires server side ordering which may be disabled so that's not really a solution either.

Sorry for the very late reply. You are most likely getting different results because the paging control needs to be enabled. I do not know the name of the control off the top of my head.

Well, I'm happy you're still around.
Speaking of controls, isn't the paging control enabled by default? After all, I have this OpenLDAP server to work against, where I can only run queries against if I call DisablePaging() on my configuration. And after all, for small data sets, it works perfectly against active directory.
And, I can't seem to talk my LDAP into going without encryption to actually look what controls it offers.
Any chance you could have another look?

Still around, but very busy with my day job. Glad you're still making use of the library. Yes, I'll take a second look. I try to stand up an instance of OpenLDAP to reproduce the issue.

So, I think I stumbled upon something important: For my large queries, I want to get all object of a certain type from my Active directory. So I found this article https://ilminator.wordpress.com/2012/01/27/retrieving-move-than-500-objects-with-linqtoldap/
.WithControls(new[] { new SearchOptionsControl(SearchOption.DomainScope) })

Is where the magic lies. This also increased the size of my search results (without it it was cut to 10'000 user objects in my prod domain).
the only question remaining is whether it is actually possible to really skip a number of results in paging... looking at what happens in Wireshark, if I page, and I want to read page 3, I see all three pages being loaded with the results from the third being returned (using the Skip/Take approach:
Result = query.InPagesOf(pageSize).Skip((page - 1) * pageSize).Take(pageSize).ToList();

) My guess is the answer is no.

Your suspicion is correct. Default paging is a black box for LDAP. You have to use a cookie returned by the server to load remaining pages and that cookie may expire if you wait too long when loading the next page. Skip / Take does work when using Virtual List Views, but you have to explicitly enable the feature in the directory.


cfi build fixes in x86-64 ghash assembly

There were some 'else' branch paths that were missing .cfi_endproc

Checklist

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

ping Travis errors should be fine. The failures were due to the doc nits failures in master.

Merged thanks all.

As indicated in #8361, this should have gone to 1.1.1 as well...  cherry-picking it now.

Done.
dbb1340314fcec37662d79720e6566fdd5a232e8 cfi build fixes in x86-64 ghash assembly


Adding support for amazonlinux

Adding this here because I know there is more work to support this but I do not know what that might be. As a result, I have not followed any of the procedures below but wanted to make this available if someone was willing to fully support amazonlinux.

[ ] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[ ] Ensure that the pull request title represents the desired changelog entry
[ ] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Feel free to close, remove, etc.

Hi @dave-juicelabs , thanks for raising the issue.
Could you describe the problem you would want to solve by adding this OS?
As a general idea, we would want to decrease the amount of supported OS in the controller image as the whole idea of providing a container image is to abstract the OS. Ideally, we should not have any OS at all inside the image (look at examples such as Nix or https://github.com/wolfi-dev/os).
Adding a lot of OS increases the amount of scenarios to support which is the absolute opposite of the problem solved by containers (this is a NP matrix problem which makes sense for packages or acceptance testing for Jenkisn of course).
I might be missing a point of course (I do not have absolute knowledge), hence my question about your use case.
Adding a new OS would make sense, though, for the agent images: a standard usage of Jenkins means adding the toolins in the agents so it would be really useful for end users with Amazon Linux tooling for instance.
This comment is neither an approval or a refusal: it is a request for clarification to ensure that if this PR is merged, then all the safety nets for having it sustainable are ok.

I think that we should not add support for Amazon Linux 2 in the Jenkins container images.  I've seen no requests from other users asking that the container image be based on Amazon Linux 2 and I've not seen indications from the submitter of this pull request that they are willing to remain actively involved in the maintenance of the container images after this is merged.  Without active maintainers to care for the Amazon Linux container image, it takes more time from the current maintainers without providing significant benefit to container users.
I consider Amazon Linux 2 to be on "extended life support".  Amazon had announced an end of life date for Amazon Linux 2 in favor of the upcoming release of Amazon Linux 2022, Amazon has extended that end of life date for Amazon Linux 2 on two different occasions.  They have stated that they intend to release Amazon Linux 2022 this year and to end support for Amazon Linux 2022 in 2024.
I feel that we already have more operating system variants than is healthy for us to support.  In my opinion, adding more operating system variants to the mix is increasing the maintainer burden without increasing the number of maintainers.  I think that's the wrong direction for our container images.
If the submitter is willing to be actively involved in maintaining and improving the container image definitions in this repository, that would reduce my concerns about that aspect of adding this container to the list of containers.  It would not completely eliminate it, since this is the first submission from @dave-juicelabs , but it would reduce my concerns.
@dave-juicelabs is this a one-time donation pull request or is this the beginning of a series of contributions to the maintenance of the Jenkins container image?  Either is fine and we're grateful for contributions, but if you don't plan to be an ongoing contributor to this repository then I'm of the opinion that we should not accept this contribution.

Closing this pull request on the assumption that it was a single contribution.  Thanks!


Remove print.

I assume this print was not intentional. (It also breaks the Python 3 compatibility.)

Ok that's embarrassing. Time to add a check for print statements in our test cycle :)


hardcoded data in funtion getOverrideTraces seems to be wrong

When I was calculating today,I found the [Effect RES] value of Set that I had equipped became lower in the result table.

Then I debugged the code and found that the hardcoded data was inaccurately defined in the function getOverrideTraces.
So I rechecked the data:
I just can't understand why you multiply RES by 2/3, but not all characters do that.
Luocha
[Constants.Stats.ATK_P]: 0.3,	=should be=》	[Constants.Stats.ATK_P]: 0.28,
March 7th、Himeko、Welt、Serval、Sampo、Bailu
[Constants.Stats.RES]: 0.067,	=should be=》	[Constants.Stats.RES]: 0.10,
Gepard\Natasha
[Constants.Stats.RES]: 0.12,	=should be=》	[Constants.Stats.RES]: 0.18,

Thank you for the find! I will check on these tonight. Can't wait to get to this task so we can get rid of the hardcoded values https://github.com/fribbels/hsr-optimizer/issues/31

Hotfix deployed, thanks again!


Add github action to build containers

Coverage remained the same at 85.182% when pulling 8ad1e4d299901645cc18de9c8b8f17e07f11b32a on feature/github-actions-build-images into 5d79bacf10d46a6ccd2f7b0dc580353d2ef0243a on master.


Change Banned indicator when a student is scanned in through kiosk entry.

Currently, if a user has their ID scanned for a Kiosk sign in at an event in a program they are banned from we give no indication on the Kiosk page to the people who are doing the scanning. We should instead show a red flash message saying something like  "Sign-in failed, please talk with an Admin."

 Change Banned indicator when a student is scanned in through kiosk entry. #1045

I am working on this issue


update example + -3dconnexion sdk

hey, sorry to ask again, but it's not quite right. a quick explanation:
git stores every change in the repo, so adding and then deleting a file means the file is still available if you checkout the point in history where the file was added. this is why you don't want to push a big file or copyrighted file (or password file...) or anything else that you might remove later: it will still be available, contributing to the download size, or causing other problems.
instead, you'll need to do something like this:
rm -rf ofxConnexion
git clone https://github.com/kylemcdonald/ofxConnexion.git
cd ofxConnexion
# update the project file
git add example/ofApp.xcodeproj
git status # check that only the project is added
git remote add manu https://github.com/moebiussurfing/ofxConnexion.git
git push manu master --force

the last couple steps will mean that your repo is force-updated, dropping any old history, and then you won't need to open a new PR. github will automatically update this one.
hope that's helpful :)


[WIP] JSON.mapping: add store_other_attributes_in flag

This PR isn't finished but I'd like to start and discuss the design around this.
Check the added specs to see what this is all about. This idea originated here
The things to discuss are:

The name of the JSON.mapping parameter
What if we want to store the extra attributes as raw strings, to later parse them as we want? Of course we can just call to_s on each hash value, but that's not memory efficient if we will lazily parse the string. So, an option to invoke pull.read_raw instead of creating a JSON::Any. I can't come up with a good API for this. Maybe a second parameter, store_other_attributes_as_raw_strings: true?

We could then copy this to YAML.mapping.

My biggest concern is that `extra´ isn't the most conflict free name one can imagine. But it's admittedly hard to come up with a good one. Which leads my thoughts towards API that lets me pick it:
 JSON.mapping({
   x: Int32, y: Int32,
 }, extra_attributes: {key: "extra", type: Hash(String, JSON::Any)})

 JSON.mapping({
   x: Int32, y: Int32,
 }, extra_attributes: {key: "extra", type: String::RawConverter})

key defines the name of the attribute to collect extra attributes in. The idea is that the passed named tuple is identical to or at least strong resembles that passed to "regular" attributes of the mapping. A bit more verbose but the usecases for this are not that common anyway.

I like that, but since type is always going to be either that has or the raw string, maybe we can have:
 JSON.mapping({
   x: Int32, y: Int32,
 }, extra_attributes: {key: "extra"}) # same as raw: false

 JSON.mapping({
   x: Int32, y: Int32,
 }, extra_attributes: {key: "extra", raw: true})

Using extra_attributes as a name was only my initial though and had the same concerns as you, but I guess it's fine (shorter and kind of obvious).
We could even support just: extra_attributes: "extra" as an equivalent to passing raw: false (similar to how in the mapping you can specify a type or a "hash" with options).

I'm actually not sure it'll stay "just those two" for every, I can't say for sure that using a user defined "collector" can't make sense there.

I really like the concept behind this PR. Just thought I'd chime in on the parameter name:
What about other?:
 JSON.mapping({
   x: Int32, y: Int32,
 }, other: {key: "extra"})

 JSON.mapping({
   x: Int32, y: Int32,
 }, other: {key: "extra", raw: true})

It would be short and (I believe) more readable.

I see only one use case for this: the JSON object you get has properties you are not interested about and you just want to keep them to forward them when serializing the object back to JSON, or when passing it somewhere else. So, they are opaque. If they weren't opaque you would mention them in the mapping itself. So I can't imagine why you'd like to treat them in a special way other than a hash of opaque data or a hash of opaque string data.
We could probably have converters, or have a callback invoked on each unknown key, passing the pull parser, but that's not really the point, and it would complicate the API a lot.
In the case of sidekiq these properties are opaque to the class defining the mapping, and they are passed to plugins which know what type to expect there. Other cases will be similar.
Also, the name key doesn't look good to me. It's not a key in the JSON mapping, it's in which instance variable or property we want to store the extra data. I also like other because it's shorter and also kind of obvious as to what it does, specially when followed by the mapping itself. Maybe then:
JSON.mapping({
   x: Int32, y: Int32,
 }, other: extra)

JSON.mapping({
   x: Int32, y: Int32,
 }, other: {property: extra})

 JSON.mapping({
   x: Int32, y: Int32,
 }, otra: {property: "extra", raw: true})

On the other hand, let me take that back. Here's the use case we are trying to solve. A plugin would like to register an extra attribute to be stored in the job, and later get that attribute back. So maybe we do want to have a callback of some sort, maybe specify which methods on the instance gets called, when an unknown attribute is reached, and pass the pull parser from there, so it's as type safe as possible.


Website compromised - ghostbsd.org

Version
N/A
What is the security issue or vulnerability?
 If I pay $5/month with PayPal does the website automatically post whatever link and text I choose?

They are sponsors. The website is not compromised. They are manually added.

I reworked the partners and sponsors section on the front page.


restore useDefaultLabel parameter in ResourceController

Is your feature request related to a problem? Please describe.
The ability to fetch text files using /foo/default/file.txt?useDefaultLabel was removed as per https://github.com/spring-cloud/spring-cloud-config/issues/1643. This was a useful feature and in use in our environment
Describe the solution you'd like
Adding the same method with a more specific RequestMapping would make this work with new spring
Describe alternatives you've considered
At the moment I have a separate class with the above RequestMapping in my config server
Additional context
I have created pull request https://github.com/spring-cloud/spring-cloud-config/pull/1822 with the changes

Duplicates #1643


setup fails to missing helm-3.1.1 RPM

It looks like helm-3.1.1 RPM is not available on the operator node (at least using Oracle-Linux-7.8-2020.09.23-0).
Terraform errors out with the following:
module.olcne.null_resource.install_nginxcontroller[0] (remote-exec): Connected!
module.olcne.null_resource.install_nginxcontroller[0] (remote-exec): /home/opc/.bashrc: line 15: helm: command not found
module.olcne.null_resource.install_nginxcontroller[0] (remote-exec): Creating Ingress Controller
module.olcne.null_resource.install_nginxcontroller[0] (remote-exec): namespace/nginx created

Error: error executing "/tmp/terraform_858443886.sh": Process exited with status 1

On the operator node, helm is not installed and the binary is not found:
[root@dev-operator opc]# helm
bash: helm: command not found

A search for helm shows only 2.9.1 is available.
[root@dev-operator opc]# yum list helm
Loaded plugins: langpacks, osmsplugin, ulninfo
This system is receiving updates from OSMS.
Available Packages
helm.x86_64                                                                                2.9.1-1.0.3.el7                                                                                ol7_developer-x86_64
[root@dev-operator opc]#

Please, check if this issue can be reproduced with latest code release.


Update .gitattributes

Files under config are mostly not generated. The CRD and role yaml, even though generated, are still easier to review than the kubebuilder annotation changes all over the go files.

/override ci/prow/integration

/lgtm

/approve

/override ci/prow/integration


Calico and flanneld

Hello,
I really liked the post, I'm implementing it and with some doubts.
Why did you use Calico and flanneld?
What is the function of each of them in the cluster?
I imagined they had the same function.
Thanks

flannel is the level 2 network, it make the cluster nodes can connect each other. calico is the level 3 network, it make cluster can use network policy function.

Ahh ok,  I'm thinking of using Weave instead of flanneld, because the weave have a great plugin ( weave scope) .  What do you think?

I never try it, why not use it and tell me how your feelings?


[Help] Why doesn't grand-child route render?

So I added a child route /list following the fractal structure.  I want to have /list show a listing, then /list/:id show an individual item from the listing.  My file structure is as follows, but /list/:id still shows the list.  What am I missing?

// routes/List/index.js
import List from './components/List'
import SingleItemRoute from './routes/Item'

export default (store) => ({
  onEnter() {
    console.log('Entered ListRoute');
  },
  path: 'list',
  component: List,
  childRoutes: [
    SingleItemRoute(store),
  ]
})

//routes/List/routes/Item/index.js
import SingleItem from './components/SingleItem'

export default (store) => ({
  onEnter() {
    console.log('Entered SingleItemRoute')
  },
  path: ':id',
  component: SingleItem
})

I finally figured it out, in order to have /list/:id render we can't have it as a nested route of /list, it actually has to be a sibling.


feat: add testnet metadata and split cfg features

Signed-off-by: Gregory Hill<EMAIL_ADDRESS>

What happens if you don't define any features? Ideally I would like to have cargo check --workspace and cargo test to work without having to define any features. Can we make standalone the default?

I actually think it's better not to have a default since it's super explicit with chain you're building for which should prevent some confusion. The alternative is to build every parachain binary with the --no-default-features flag.

The reason I was saying I prefer standalone to be default is that it's easier that way to debug/build from the ide. But I guess I'll just spend some time to configure the default features in the ide itself then.
One more thing, though, can you also update runtime/build.rs?


Console - make Failover save works

Issue
https://gravitee.atlassian.net/browse/APIM-2749
Description
Fix : "User unable to save Failover configuration"
Additional context

📚  View the storybook of this branch here

Finally, it's THE fix to do. Between 4.0 and 4.1, we switched to MAPI v2, which didn't handle the default value pour cases


add test case for cast_timestamp_before_1970

Which issue does this PR close?

Closes #3082
Rationale for this change

this is solved while arrow-rs is upgraded to v20. this pr is to add test cases
What changes are included in this PR?

Are there any user-facing changes?

Codecov Report

Merging #3163 (67e8355) into master (15a9a4b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 67e8355 differs from pull request most recent head 98a67c5. Consider uploading reports for the commit 98a67c5 to get more accurate results

@@           Coverage Diff           @@
##           master    #3163   +/-   ##
=======================================
  Coverage   85.87%   85.88%
=======================================
  Files         291      291
  Lines       52758    52769   +11
=======================================
+ Hits        45307    45320   +13
+ Misses       7451     7449    -2

Impacted Files
Coverage Δ

datafusion/core/src/physical_plan/mod.rs
88.00% <ø> (ø)

datafusion/core/tests/sql/timestamp.rs
99.81% <100.00%> (+<0.01%)
:arrow_up:

datafusion/expr/src/logical_plan/plan.rs
77.60% <0.00%> (+0.17%)
:arrow_up:

datafusion/core/src/physical_plan/metrics/value.rs
87.43% <0.00%> (+0.50%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Create BaseLearnableLayer class

Summary
Create a BaseLearnableLayer class. This is an inheritance from BaseLayerClass and has prefix_param argument, which is string.

Will close because I implemented the class.


Update alm-api-for-spfx-add-ins.md

Q
A

content fix?
yes

New article?
no

Related issues?

What's in this Pull Request?
Correct uninstall app rest call from
/_api/web/tenantappcatalog/AvailableApps/GetById('xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxx')/Upgrade
to
/_api/web/tenantappcatalog/AvailableApps/GetById('xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxx')/Uninstall

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/apis/alm-api-for-spfx-add-ins.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

thx Aaron, good to get the copy-paste issue solved.


create github action to build docker image

This is a simple workflow to build and push the docker image to https://ghcr.io (GitHub's docker container registry)
This might require changing this setting for workflows in the repository settings:

Before we start automating this sort of thing, I need to get my integration/unit test branch merged in.  We'll put this on the back burner for a little but we'll get back to this (Hopefully soon)


Revamp documentation resources

Most documentations live in /docs now, with a landing page at https://lampepfl.github.io/gears
linking to them.
README now is a simple page showing build status, release, basic build instructions etc.

Move Bartosz's report to its own directory
Move Martin Odersky's talk and README to docs
Add Maximilian's report
Add landing page
Add a simple README

I wonder what's the role of the README in this setup: Build instructions are in CONTRIBUTING and infos - or rather: links - are on the docs page. If the page didn't look as nice as it does, I would have voted for its deletion in favor of an elaborate README. But I guess it's fine, because it does look nice :)
Besides, I would add some "For an overview of the library, see..." which would be really important to be in the docs imho. We do have something like that: the release notes: https://github.com/lampepfl/gears/releases/tag/v0.1.0. Maybe we could copy/move them to the docs - and in any case keep them updated. Otherwise it's very hard to get started, given the reports, the Scala API, and the increasingly outdated Scalar README.

Besides, I would add some "For an overview of the library, see..." which would be really important to be in the docs imho.

Would the book be sufficient in this case?

Let's have this in first -- I think it's better to have a landing page first and foremost, better than nothing ;) We can improve it later.


docs(deck) rename master to main and hbagdi to kong

@hbagdi should we also add some info on how to upgrade to the new location? I thought I saw a thread about this on slack.
Or - is there any issue with leaving the instructions as is? I just tested with a brew upgrade deck and noticed it still pulled from hbagdi/deck but the detailed info links to https://github.com/kong/deck. Will this be disabled in the future?

We can document on how to upgrade but the way I've set it up requires no user action. Existing and new users will get the same artifact always.
The reasoning here is that the following repos are kept in sync:
github.com/kong/homebrew-deck/
github.com/hbagdi/homebrew-deck/


SyntaxError: '(' expected

HI,
I have been using pretty-quick for sometime, today I saw an error, which I do not know why:
I am using Angular and this line:
private readonly modalRef: MatDialogRef<any> = inject(MatDialogRef<any>) is causing this error:
SyntaxError: '(' expected.
Any ideas?
Versions details:
"pretty-quick": "3.1.3"
"prettier": "2.6.2"
"eslint-config-prettier": "^8.8.0"
node js: 16.14.0
angular: 14.3.0

That's mostly like that your prettier version does not recognize the TypeScript syntax, it has nothing related to pretty-quick.


Fix bug for when the first char is non-ascii char

Crashes with:
Index 258 out of bounds for length 256
java.lang.ArrayIndexOutOfBoundsException:
Index 258 out of bounds for length 256
This adds a check for bigger SUFFIX_LOOKUP.length indexes for the first char.

Thanks, good catch!
Now that I'm looking at that line, I'm wondering why I didn't just check the first character directly against the range of valid characters (if (firstChar < '0' || firstChar > '7') ...) and avoid the lookup altogether. The second part of the check ((SUFFIX_LOOKUP[firstChar] >>> 3) & 0x3) > 0) isn't that great either and could then be removed.
What do you think?

you are right :) updated

Thanks again :) I will push a new release later today.


[WIP] ci: Fail PRs if branch VERSION.txt is not open/dev

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/backport

this requires a bit of testing

tested in #25851 and appears to work as expected (now)


Crash when inputting x^10=10 into function data input field in Graphing Calculator

A team member will update this issue with more details soon.

This is your friendly Microsoft Issue Bot. I created this issue automatically as requested by a team member.


Hero positions and small layout improvements

What:

Rely more on flex than paddings/margins to place main hero content
Reduce the width of the content for some sections to keep text similar to design distribution.
Move donation text to left as Join Union is

Why:
Related to https://app.asana.com/0/1196225495304457/1196225495304526
Media:
https://www.loom.com/share/592104f320b049188b5a61c7a7306e5b

I think the image looks too dark, I would prefer if we reduce the overlay tint a bit

I still waiting for the overlay color. In order to avoid further issues I will follow up that on its own ticket at
https://app.asana.com/0/1196225495304457/1196579158777584 for now this one needs to be merged.

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


Increase utility panel width

Feature description
To ensure that the scratchpad is released properly, we'll need to increase the width of the right side utility panel.
Currently, for larger screens, the panel has a max-width of 25rem.
Based on the team discussion, from the max breakpoint and above, this maximum width should be increased to 32rem
Alternatives
No response
Additional context
No response

closed via https://github.com/Qiskit/platypus/pull/1251


Saint does not work if launched from USB

I noticed an unpleasant feature. If I run the file from the desktop - all works fine. But, if I put this file on flash usb, run it, disconnect - then no emails are no longer coming. Is it possible to fix this?
It turns out, you need to move the file to the "victim's" computer, which is not very convenient. The ability to insert a flash drive and pull, thereby infecting the computer without manipulation (like badUSB) disappears.

Hello,
Do you have enabled persistence?
I tested here, and when you run it from flash usb, the file are copied to %appdata%/(s)AINT (but are necessary enable the persistence when the spyware are generated).
So when you restart the OS, the spyware start.

I noticed an unpleasant feature. If I run the file from the desktop - all works fine. But, if I put this file on flash usb, run it, disconnect - then no emails are no longer coming. Is it possible to fix this?
It turns out, you need to move the file to the "victim's" computer, which is not very convenient. The ability to insert a flash drive and pull, thereby infecting the computer without manipulation (like badUSB) disappears.

Based on my experience, if the app has been removed and the target characters are not yet reached, it does not record the screenshot and cam capture. Only the keystroke logging persists. If you want to remove the app/USB after installation, make sure the target characters are hit first so the data logging shall persist.

Hey How'd you get it to work I'm getting an 'os is not supported' error on gitbash.
And when I launch it from the exe nothing happens.
It finishes building successfully but doesn't start.


AMBARI-25149. Cover all not covered common views

What changes were proposed in this pull request?
Cover all not covered common views:
views/common/log_search_ui_link_view.js
views/common/log_tail_view.js
views/common/not-scrollable-textarea.js
views/common/service_restart_view.js
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4918/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4927/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4928/
Test PASSed.