Automated cherry pick of #1080: image upload remove os type

Cherry pick of #1080 on release/2.10.0.
#1080: image upload remove os type

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@1de294b). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release/2.10.0   #1081   +/-   ##
================================================
  Coverage                  ?    7.9%
================================================
  Files                     ?     356
  Lines                     ?   56477
  Branches                  ?       0
================================================
  Hits                      ?    4464
  Misses                    ?   51618
  Partials                  ?     395

Flag
Coverage Δ

#aFlag
7.9% <ø> (?)

Continue to review full report at Codecov.

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

/lgtm
/approve


Fix UG for 2101

Codecov Report

Merging #361 (8646727) into master (06b9398) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #361   +/-   ##
=========================================
  Coverage     71.23%   71.23%
  Complexity      807      807
=========================================
  Files           129      129
  Lines          2632     2632
  Branches        327      327
=========================================
  Hits           1875     1875
  Misses          667      667
  Partials         90       90

Continue to review full report at Codecov.

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


Compile Failed(compile.sh) : mv: cannot stat ‘.zip’: No such file or directory

[root@snr dr-elephant]# ./compile.sh
Using the default configuration
Hadoop Version : 2.7.0
Spark Version  : 1.5.2
Other opts set :
############################################################################
npm installation found, we'll compile with the new user interface
############################################################################
++ sleep 3
++ ember_assets=/home/rsecar/work/repos/dr-elephant/public/assets
++ ember_resources_dir=/home/rsecar/work/repos/dr-elephant/public/assets/ember
++ ember_web_directory=/home/rsecar/work/repos/dr-elephant/web
++ cd /home/rsecar/work/repos/dr-elephant/web
++ npm install
++ node_modules/bower/bin/bower install
bower ESUDO         Cannot be run with sudo
Additional error details:
Since bower is a user command, there is no need to execute it with superuser permissions.
If you're having permission errors when using bower without sudo, please spend a few minutes learning more about how your system should work and make any necessary repairs.
http://www.joyent.com/blog/installing-node-and-npm
https://gist.github.com/isaacs/579814
You can however run a command with sudo using --allow-root option
++ node_modules/ember-cli/bin/ember build --prod
Could not start watchman; falling back to NodeWatcher for file system events.
Visit http://ember-cli.com/user-guide/#watchman for more info.
⠏ Building[WARN] (broccoli-uglify-sourcemap) Minifying: assets/vendor.js took: 63568ms (more than 20,000ms)
Built project successfully. Stored in "dist/".
File sizes:

dr-elephant.css: 8.61 KB (2.49 KB gzipped)
dr-elephant.js: 275.43 KB (28.18 KB gzipped)
aggregated-metrics.css: 207 B (156 B gzipped)
application-menu.css: 334 B (218 B gzipped)
application.css: 34 B (54 B gzipped)
breadcrumbs.css: 1.36 KB (566 B gzipped)
dashboard-header.css: 609 B (340 B gzipped)
heuristic-detail.css: 127 B (109 B gzipped)
heuristics-summary.css: 536 B (281 B gzipped)
jobs-severity.css: 333 B (202 B gzipped)
loading-panel.css: 129 B (126 B gzipped)
navbar.css: 920 B (282 B gzipped)
not-found.css: 76 B (74 B gzipped)
search-panel.css: 39 B (59 B gzipped)
search.css: 569 B (300 B gzipped)
user-detail.css: 988 B (435 B gzipped)
user-tabs.css: 725 B (321 B gzipped)
user-tags.css: 76 B (75 B gzipped)
vendor.css: 144.84 KB (24.19 KB gzipped)
vendor.js: 1.21 MB (355.91 KB gzipped)
++ rm -r /home/rsecar/work/repos/dr-elephant/public/assets/ember
++ mkdir /home/rsecar/work/repos/dr-elephant/public/assets/ember
++ cp dist/assets/dr-elephant.css /home/rsecar/work/repos/dr-elephant/public/assets/ember/
++ cp dist/assets/dr-elephant.js /home/rsecar/work/repos/dr-elephant/public/assets/ember/
++ cp dist/assets/vendor.js /home/rsecar/work/repos/dr-elephant/public/assets/ember/
++ cp dist/assets/vendor.css /home/rsecar/work/repos/dr-elephant/public/assets/ember/
++ cp -r dist/fonts /home/rsecar/work/repos/dr-elephant/public/assets/
++ cd /home/rsecar/work/repos/dr-elephant
++ trap exit SIGINT SIGTERM
++ start_script=/home/rsecar/work/repos/dr-elephant/scripts/start.sh
++ stop_script=/home/rsecar/work/repos/dr-elephant/scripts/stop.sh
++ app_conf=/home/rsecar/work/repos/dr-elephant/app-conf
++ rm -rf /home/rsecar/work/repos/dr-elephant/dist
++ mkdir dist
++ echo

++ play_command -Dhadoopversion=2.7.0 -Dsparkversion=1.5.2 clean compile dist
++ type activator
++ play -Dhadoopversion=2.7.0 -Dsparkversion=1.5.2 clean compile dist
play:      SoX v14.4.1
Usage summary: [gopts] [[fopts] infile]... [fopts] [effect [effopt]]...
SPECIAL FILENAMES (infile, outfile):

                   Pipe/redirect input/output (stdin/stdout); may need -t

-d, --default-device     Use the default audio device (where available)
-n, --null               Use the null' file handler; e.g. with synth effect -p, --sox-pipe           Alias for -t sox -'
SPECIAL FILENAMES (infile only):
"|program [options] ..." Pipe input from external program (where supported)
http://server/file       Use the given URL as input file (where supported)
GLOBAL OPTIONS (gopts) (can be specified at any point before the first effect):
--buffer BYTES           Set the size of all processing buffers (default 8192)
--clobber                Don't prompt to overwrite output file (default)
--combine concatenate    Concatenate all input files (default for sox, rec)
--combine sequence       Sequence all input files (default for play)
-D, --no-dither          Don't dither automatically
--effects-file FILENAME  File containing effects and options
-G, --guard              Use temporary files to guard against clipping
-h, --help               Display version number and usage information
--help-effect NAME       Show usage of effect NAME, or NAME=all for all
--help-format NAME       Show info on format NAME, or NAME=all for all
--i, --info              Behave as soxi(1)
--input-buffer BYTES     Override the input buffer size (default: as --buffer)
--no-clobber             Prompt to overwrite output file
-m, --combine mix        Mix multiple input files (instead of concatenating)
--combine mix-power      Mix to equal power (instead of concatenating)
-M, --combine merge      Merge multiple input files (instead of concatenating)
--multi-threaded         Enable parallel effects channels processing
--norm                   Guard (see --guard) & normalise
--play-rate-arg ARG      Default rate' argument for auto-resample with play'
--plot gnuplot|octave    Generate script to plot response of filter effect
-q, --no-show-progress   Run in quiet mode; opposite of -S
--replay-gain track|album|off  Default: off (sox, rec), track (play)
-R                       Use default random numbers (same on each run of SoX)
-S, --show-progress      Display progress while processing audio data
--single-threaded        Disable parallel effects channels processing
--temp DIRECTORY         Specify the directory to use for temporary files
-T, --combine multiply   Multiply samples of corresponding channels from all
input files (instead of concatenating)
--version                Display version number of SoX and exit
-V[LEVEL]                Increment or set verbosity level (default 2); levels:
1: failure messages
2: warnings
3: details of processing
4-6: increasing levels of debug messages
FORMAT OPTIONS (fopts):
Input file format options need only be supplied for files that are headerless.
Output files will have the same format as the input file where possible and not
overriden by any of various means including providing output format options.
-v|--volume FACTOR       Input file volume adjustment factor (real number)
--ignore-length          Ignore input file length given in header; read to EOF
-t|--type FILETYPE       File type of audio
-e|--encoding ENCODING   Set encoding (ENCODING may be one of signed-integer,
unsigned-integer, floating-point, mu-law, a-law,
ima-adpcm, ms-adpcm, gsm-full-rate)
-b|--bits BITS           Encoded sample size in bits
-N|--reverse-nibbles     Encoded nibble-order
-X|--reverse-bits        Encoded bit-order
--endian little|big|swap Encoded byte-order; swap means opposite to default
-L/-B/-x                 Short options for the above
-c|--channels CHANNELS   Number of channels of audio data; e.g. 2 = stereo
-r|--rate RATE           Sample rate of audio
-C|--compression FACTOR  Compression factor for output format
--add-comment TEXT       Append output file comment
--comment TEXT           Specify comment text for the output file
--comment-file FILENAME  File containing comment text for the output file
--no-glob                Don't `glob' wildcard match the following filename
AUDIO FILE FORMATS: 8svx aif aifc aiff aiffc al amb au avr caf cdda cdr cvs cvsd cvu dat dvms f32 f4 f64 f8 fap flac fssd gsm gsrt hcom htk ima ircam la lu mat mat4 mat5 maud nist ogg paf prc pvf raw s1 s16 s2 s24 s3 s32 s4 s8 sb sd2 sds sf sl sln smp snd sndfile sndr sndt sou sox sph sw txw u1 u16 u2 u24 u3 u32 u4 u8 ub ul uw vms voc vorbis vox w64 wav wavpcm wv wve xa xi
PLAYLIST FORMATS: m3u pls
AUDIO DEVICE DRIVERS: alsa ao oss ossdsp pulseaudio
EFFECTS: allpass band bandpass bandreject bass bend biquad chorus channels compand contrast dcshift deemph delay dither divide+ downsample earwax echo echos equalizer fade fir firfit+ flanger gain highpass hilbert input# ladspa loudness lowpass mcompand mixer* noiseprof noisered norm oops output# overdrive pad phaser pitch rate remix repeat reverb reverse riaa silence sinc spectrogram speed splice stat stats stretch swap synth tempo treble tremolo trim upsample vad vol

Deprecated effect    + Experimental effect    # LibSoX-only effect
EFFECT OPTIONS (effopts): effect dependent; see --help-effect
++ cd target/universal
+++ /bin/ls '*.zip'
/bin/ls: cannot access *.zip: No such file or directory
++ ZIP_NAME=
++ unzip
UnZip 6.00 of 20 April 2009, by Info-ZIP.  Maintained by C. Spieler.  Send
bug reports using http://www.info-zip.org/zip-bug.html; see README for details.

Usage: unzip [-Z] [-opts[modifiers]] file[.zip] [list] [-x xlist] [-d exdir]
Default action is to extract files in list, except those in xlist, to exdir;
file[.zip] may be a wildcard.  -Z => ZipInfo mode ("unzip -Z" for usage).
-p  extract files to pipe, no messages     -l  list files (short format)
-f  freshen existing files, create none    -t  test compressed archive data
-u  update files, create if necessary      -z  display archive comment only
-v  list verbosely/show version info       -T  timestamp archive to latest
-x  exclude files that follow (in xlist)   -d  extract files into exdir
modifiers:
-n  never overwrite existing files         -q  quiet mode (-qq => quieter)
-o  overwrite files WITHOUT prompting      -a  auto-convert any text files
-j  junk paths (do not make directories)   -aa treat ALL files as text
-U  use escapes for all non-ASCII Unicode  -UU ignore any Unicode fields
-C  match filenames case-insensitively     -L  make (some) names lowercase
-X  restore UID/GID info                   -V  retain VMS version numbers
-K  keep setuid/setgid/tacky permissions   -M  pipe through "more" pager
See "unzip -hh" or unzip.txt for more help.  Examples:
unzip data1 -x joe   => extract all files except joe from zipfile data1.zip
unzip -p foo | more  => send contents of foo.zip via pipe into program more
unzip -fo foo ReadMe => quietly replace existing ReadMe if archive file newer
++ rm
rm: missing operand
Try 'rm --help' for more information.
++ DIST_NAME=
++ chmod +x /bin/dr-elephant
chmod: cannot access ‘/bin/dr-elephant’: No such file or directory
++ sed -i.bak '/declare -r app_classpath/s/.$/:hadoop classpath:${ELEPHANT_CONF_DIR}"/' /bin/dr-elephant
sed: can't read /bin/dr-elephant: No such file or directory
++ cp /home/rsecar/work/repos/dr-elephant/scripts/start.sh /bin/
++ cp /home/rsecar/work/repos/dr-elephant/scripts/stop.sh /bin/
++ cp -r /home/rsecar/work/repos/dr-elephant/app-conf
cp: missing destination file operand after ‘/home/rsecar/work/repos/dr-elephant/app-conf’
Try 'cp --help' for more information.
++ zip -r .zip
zip error: Nothing to do! (.zip)
++ mv .zip /home/rsecar/work/repos/dr-elephant/dist/
mv: cannot stat ‘.zip’: No such file or directory

Able to compile after removing 'sox' package


Do not spend more than 5m trying to download caches on CI

Otherwise we're just wasting time. The default timeout is 360(!!!) minutes.

Hey it works! Check the 'sample-counter' job.


OpenStreetMap view doesn't render completely, doesn't allow zooming

When rendering the map with all its perks in the admin panel everything works flawlessly, but when doing so in the template the visualized map looks like its a screenshot with missing chunks, and doesn't allow manual zooming (the zooming effect is the same of a picture, it just gets foggy the more i zoom in).
View code:
def index(request):
return render(request, 'core/index.html', geomap_context(Location.objects.all(), map_zoom="2", auto_zoom="10", map_height="1000px"))
Template code:
 {% include "geomap/common.html" %}
Enclosed in a div with an id.
I'm using django version 4.1.7

For anyone experiencing this, the error comes from the inclusion of multiple version of JQuery in the project. Use the one provided by their package in order to avoid such trouble.

Thanks for your investigation. This project is related to Django, not to js code. It's hard for me to investigate problems with js.


MQTT & homeassistant

I am sort of lost in terms on how to control my blinds motor. I have successfully setup a wifi connection between my esp8266 and MQTT connection also reports as successfull (at that's what the console reports). Now how do I actually operate the motor? I dont see esp8266 on my homeassistant dashboard.

Hi there, if you've used the code from this repo to flash the ESP8266 you will need to manually add each MQTT topic to the Home Assistant config.
Here's my config for HA:
mqtt:
  sensor:
    - name: "Bedroom Blinds: Status"
      unique_id: "esp8266_blinds_01_status"
      object_id: "esp8266_blinds_01_status"
      state_topic: "home/esp8266_blinds_01/status"
    - name: "Bedroom Blinds: Current Mode"
      unique_id: "esp8266_blinds_01_mode"
      object_id: "esp8266_blinds_01_mode"
      state_topic: "home/esp8266_blinds_01/mode"
    - name: "Bedroom Blinds: Enabled"
      unique_id: "esp8266_blinds_01_enabled"
      object_id: "esp8266_blinds_01_enabled"
      state_topic: "home/esp8266_blinds_01/enabled"
    - name: "Bedroom Blinds: State"
      unique_id: "esp8266_blinds_01_state"
      object_id: "esp8266_blinds_01_state"
      state_topic: "home/esp8266_blinds_01/state"
    - name: "Bedroom Blinds: Current Steps"
      unique_id: "esp8266_blinds_01_steps"
      object_id: "esp8266_blinds_01_steps"
      state_topic: "home/esp8266_blinds_01/steps"
    - name: "Bedroom Blinds: Direction"
      unique_id: "esp8266_blinds_01_direction"
      object_id: "esp8266_blinds_01_direction"
      state_topic: "home/esp8266_blinds_01/direction"
    - name: "Bedroom Blinds: Vertical Steps"
      unique_id: "esp8266_blinds_01_steps_vertical"
      object_id: "esp8266_blinds_01_steps_vertical"
      state_topic: "home/esp8266_blinds_01/steps/vertical"
    - name: "Bedroom Blinds: Open Mode"
      unique_id: "esp8266_blinds_01_mode_open"
      object_id: "esp8266_blinds_01_mode_open"
      state_topic: "home/esp8266_blinds_01/mode/open"
    - name: "Bedroom Blinds: Close Mode"
      unique_id: "esp8266_blinds_01_mode_close"
      object_id: "esp8266_blinds_01_mode_close"
      state_topic: "home/esp8266_blinds_01/mode/close"
    - name: "Bedroom Blinds: Open Delay"
      unique_id: "esp8266_blinds_01_delay_open"
      object_id: "esp8266_blinds_01_delay_open"
      state_topic: "home/esp8266_blinds_01/delay/open"
    - name: "Bedroom Blinds: Close Delay"
      unique_id: "esp8266_blinds_01_delay_close"
      unique_id: "esp8266_blinds_01_delay_close"
      state_topic: "home/esp8266_blinds_01/delay/close"

  switch:
    - name: "Bedroom Blinds: Enabled"
      unique_id: "esp8266_blinds_01_enabled_switch"
      object_id: "esp8266_blinds_01_enabled_switch"
      state_topic: "home/esp8266_blinds_01/enabled"
      command_topic: "home/esp8266_blinds_01/control/enabled"
      availability_topic: "home/esp8266_blinds_01/status"
      payload_available: "connected"
      payload_not_available: "disconnected"
      qos: 1
      payload_on: 1
      payload_off: 0
    - name: "Bedroom Blinds: Opened"
      unique_id: "esp8266_blinds_01_open_switch"
      object_id: "esp8266_blinds_01_open_switch"
      state_topic: "home/esp8266_blinds_01/state"
      command_topic: "home/esp8266_blinds_01/control/blinds"
      availability_topic: "home/esp8266_blinds_01/status"
      payload_available: "connected"
      payload_not_available: "disconnected"
      qos: 1
      payload_on: "opened"
      payload_off: "closed"


[FEATURE REQUEST] Adding the dark theme toggle button in the site

I think a dark theme toggle button could be added to the site.
I'm a GSSOC contributor,
I would like to work on this issue
please assign me this issue under GSSOC23

As for now, dark theme is not required for this website. Kindly work on other issues.


Question: how to create a sub form type functionality and go back to main form

Hi,
This is not really a Jsonform question, but a Javascript question about using forms ...
I have a jsonform custom select field with a datalist. If the user enters a unknown value, I want to present a new form so that he can enter more information about the new value.
I have created the new page with a new seperate Jsonform in newStoreForm.html.
When the user enters an unkown value I do:
window.location = "newStoreForm.html";
In the onSubmit() of the newStoreForm I do:
window.history.back(); return false;
I go back but all my previous info is gone and the original page is reloaded...
By the way, I am running all this locally without a server in node-webkit http://nwjs.io/.
Is there a way to achieve this functionality without manually storing the whole state of the main page and rebuilding it manually?
Maybe use the newStoreForm jsonform in jquery Dialog?
Any examples?
thanks,
Peter

I used jquery modal dialog

I'm closing this issue since it is probably outdated (and probably out of the scope of jsonform) — Please reopen if you still need help on this. Thanks

I used jquery modal dialog

Hi @ike

I used jquery modal dialog

@ikeamanual I am currently trying to use this library in a jquery dialog and am having trouble. I can't get the onSubmit function to get called when the submit button is pressed. I can get all of the examples working fine if I don't run them in a dialog, but the submit button stops working when in a dialog. Did you happen to run into anything like this?


[WIP] Define and characterize sections for public-facing documentation

This is an attempt to define the main titles we will use for the public-facing KLT documentation.  Related discussions and issues are https://github.com/keptn/lifecycle-toolkit/discussions/303 , https://github.com/keptn/lifecycle-toolkit/issues/176 , and https://github.com/keptn/lifecycle-toolkit/issues/251 .
Discussions about the doc build tools to use are beyond the scope of this discussion, except to say that we expect to have all content either autogenerated or authored under the content/docs subdirectory in this repo and we hope to have the ability to include pages in the KLT doc set that are autogenerated or authored in other software repositories.
General notes:

We probably will not have the ability to excerpt portions of a file into the documentation, so READMEs, specs, and other such documents should not include  content that is intended for the published documentation but should instead be targeted at people who are working with the software source code.
We want to minimize duplicated content in the docs because this is hard to maintain over time.  So, for example, when discussing how to use a component, do not replicate installation instructions but instead say what must be installed and link to the instructions in the Installation section.  When using a CRD in an example, discuss only the what is directly relevant to this discussion and link to the reference page for details.  Et cetera.
As much as possible, how-to instructions should include instructions for verifying that the procedure was successful and include information about how to fix or at least analyze the problem if the procedure was not successful.

Documentation sections:

Quick tour -- a Killercoda-based exercise that illustrates what KLT is and how to use it as well as introducing  key concepts.
   Follow this with a more detailed description of the major functional characteristics, similar to what is in the Concepts articles for Keptn V1.  Pulling this information into a single section is likely to yield a document that flows more smoothly although we could do separate pages for each functional topic.
Functional architectural overview - an in-depth, under-the-hood explanation about how a KLT "project" executes, describing the components used to process each section and illustrated with log excerpts that show what is recorded when things work as they should.  This is useful for sophisticated users who need to integrate KLT with their existing environment and is indispensable for people attempting to troubleshoot an issue.
Installation -- Comprehensive instructions for installing and upgrading KLT.  Primary focus should be on installing a production environment but clear information should be included for installing an environment for demos and self-study (which may be less secure and less robust that a full installation).  The landing page for this section should include a summary list of all steps (required and optional), in order, with links to the individual pages that provide detailed instructions.
Operations/Administration -- includes pages about maintaining a KLT installation, including backing up and restoring files, monitoring resources, recovering from system errors and failures.
User Guide -- How to define and manage a KLT "project".  This should probably be divided into two sections: one about the steps used for virtually every "project" and one about different tasks and functionality can be implemented.  The landing page for each sub-section should have a summary list of the topics discussed, with each item linked to the page that provides more details.
Reference pages -- comprehensive detailed information about each entity.

API/CRD -- autogenerated, done similarly to the k8s API pages
?? Any config files that should be documented
?? Should we document each Helm chart?
?? Any miscellaneous entities that require reference pages such as environment variables
Note that KLT does not have a CLI.

Definition of documentation for developers/contributors is outside the scope of this proposal, whether we want to author a Developer Guide in the published docs or provide all information in in the CONTRIBUTING.md file and Community Docs.

Killercoda demo: https://github.com/keptn/lifecycle-toolkit/issues/250


no database connection available for a query. You need to bind the model class or the query to a knex instance.

Hello,
I'm trying to setup my own project with objection and knex. I want to create a api backend.
But I run into this error: no database connection available for a query. You need to bind the model class or the query to a knex instance.
The project can be found here: https://github.com/eazyurk/shop-backend
I create the knex and objection setup in: src/db.js
The error is in: src/api/user/user.routes.js
The error will be logged when you go to: http://localhost:5050/api/v1/user
I already tried debugging it, I tried different mysql drivers. I'm 100% sure my database is working and running properly. I'm not sure where the error is coming from so I was hoping someone else could help me. Thanks a lot!

From what I can tell, no file in your app actually uses src/db.js.  Try adding require('./db') somewhere near the root of your app to run the code in that module, perhaps inside src/app.js.

@devinivy Thanks, that fixed it. I'm new to all this so kinda dumb mistake but I'm glad that you helped me! :)


pod_target_xcconfig -> swift_version (Swift 5)

Hello.
Thank you for XCGLogger.
In this pull request I fixed merge conflicts from #274 and changed Swift version to 5.0.

This has been updated. Thanks.


Relationship support in schema and resolvers

Background
graphback version 0.3.1 creates database structure for relationships.
TODO
Add relationships in schema and resolvers
For a model
type Note {
    id: ID!
    title: String!
    comments: [Comment!]!
}

type Comment {
    id: ID!
    content: String!
}

we can create the schema with types like
type Note {
    id: ID!
    title: String!
}

type Comment {
    id: ID!
    content: String!
    noteId: ID!
}

and queries can be like
type Query {
    ....
    findCommentsOfNote  //for 1:m relation
    findCommentOfNote   //for 1:1 relation
}

Not sure how to work with n:m relations for now
Resolvers can be implemented using either

Dataloader
Graphql-query-mapper

@wtrocki What should be the methods we should provide for relationships? like findXsOfY for example.

noteId: ID!

That is not valid schema. It needs to refer to a single or array of a specific type.

findCommentsOfNote  //for 1:m relation

None of those is needed. We should be able to query comments from findAllNotes etc. Please go thru learn.graphql.com for more info

Dataloader
Graphql-query-mapper

Ideally, we need both query mapper can be used to descope the range of query.
I would see query mapper doing more heavylifting so we can drive more requirements for this library. DataLoader is must for us.
We need to have both available as part of the strongly typed context so people can easily drop their code.


Changed the module name to JSONBuilder

Hope this helps Garrett.
/sinclair

This has been fixed in 3.0. Thanks for being patient!


Feat: week interval unit

PG:
psql (15.3)
Type "help" for help.

pg=# select INTERVAL '1 week';
 interval
----------
 7 days
(1 row)

RW:
psql (15.3, server 8.3.0)
WARNING: psql major version 15, server major version 8.3.
         Some psql features might not work.
SSL connection (protocol: TLSv1.3, cipher: TLS_AES_256_GCM_SHA384, compression: off)
Type "help" for help.

dev=> select INTERVAL '1 week';
ERROR:  QueryError: Bind error: failed to bind expression: INTERVAL '1 week'

Caused by:
  Invalid input syntax: unknown unit week

'1 week'  is all over our docs. We might want to either finish this or change our doc


[openimageio:x86-windows] build failure (hash issue on downloading gzip)

Host Environment

OS: Windows 10
Compiler: VS 2019

To Reproduce
Steps to reproduce the behavior:
./vcpkg install openimageio
Failure logs
[DEBUG] Downloading https://github.com/OpenImageIO/oiio/archive/099c8585e3add6f58fab9aa438a491fa55d3f67e.tar.gz
Error: Failed to download from mirror set:
File does not have the expected hash:
url : [ https://github.com/OpenImageIO/oiio/archive/099c8585e3add6f58fab9aa438a491fa55d3f67e.tar.gz ]
File path : [ E:\VisualStudio\vcpkg\downloads\OpenImageIO-oiio-099c8585e3add6f58fab9aa438a491fa55d3f67e.tar.gz.41468.part ]
Expected hash : [ 1b6a5e41607bd68590a19672ca777c953b92b347425c9fe8ca7d096959bece789d043a0fae1f7bf00a88dcb11815dd3501414c9ad979e1fe9dd1613bb9e04b0b ]
Actual hash : [ 36e3aef52e082fcff64a16d10ad372a6335b3562c3d6e62aac88173476ae6c3694ac565084d50551724f47be83a7a8182e608c81c7a787392e576d9c401f65f5 ]
[DEBUG] C:\a\1\s\src\vcpkg\base\downloads.cpp(648)
[DEBUG] Exiting after 3005548 us (2999701 us)
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:251 (message):
  Failed to download file.
  If you use a proxy, please check your proxy setting. Possible causes are:

...
...
...
Call Stack (most recent call first):
scripts/cmake/vcpkg_from_github.cmake:173 (vcpkg_download_distfile)
ports/openimageio/portfile.cmake:8 (vcpkg_from_github)
scripts/ports.cmake:141 (include)
Error: Building package openimageio:x86-windows failed with: BUILD_FAILED
Additional context
file itself downloading without errors.

Sorry. Now another error.
`Starting package 113/113: openimageio:x86-windows
Building package openimageio[core,ffmpeg,freetype,gif,libraw,opencolorio,opencv,openjpeg,pybind11,tools,webp]:x86-windows...
-- Downloading https://github.com/OpenImageIO/oiio/archive/099c8585e3add6f58fab9aa438a491fa55d3f67e.tar.gz -> OpenImageIO-oiio-099c8585e3add6f58fab9aa438a491fa55d3f67e.tar.gz...
-- Extracting source E:/VisualStudio/vcpkg/downloads/OpenImageIO-oiio-099c8585e3add6f58fab9aa438a491fa55d3f67e.tar.gz
-- Applying patch fix-config-cmake.patch
-- Applying patch fix_static_build.patch
-- Using source at E:/VisualStudio/vcpkg/buildtrees/openimageio/src/fa55d3f67e-731af77137.clean
-- Downloading https://www.python.org/ftp/python/3.9.6/python-3.9.6-embed-win32.zip -> python-3.9.6-embed-win32.zip...
-- Found external ninja('1.10.2').
-- Configuring x86-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:127 (message):
Command failed: ninja -v
Working Directory: E:/VisualStudio/vcpkg/buildtrees/openimageio/x86-windows-rel/vcpkg-parallel-configure
Error code: 1
See logs for more information:
E:\VisualStudio\vcpkg\buildtrees\openimageio\config-x86-windows-out.log
Call Stack (most recent call first):
installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_configure.cmake:398 (vcpkg_execute_required_process)
ports/openimageio/portfile.cmake:49 (vcpkg_cmake_configure)
scripts/ports.cmake:141 (include)
Error: Building package openimageio:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: openimageio:x86-windows
Vcpkg version: 2021-08-12-85ab112d5ee102bc6eac8cdbbfdd173a71374e04
Additionally, attach any relevant sections from the log files above.`


ValidateFontFamily fails for Elements that don't have OwnerDocument

Hi,
If one creates SvgText element at runtime it's OwnerDocument property will be Null. Afterwards trying to access Bounds property will throw "Object reference not set to an instance of an object.", because ValidateFontFamily expects "SvgDocument doc" not to be null.
var svgText = new SvgText("Hello World!");
// This line will throw "Object reference not set to an instance of an object." exception.
var textWidth = svgText.Bounds.Width;
So we need to modify ValidateFontFamily method (line 436 of SvgElementStyle.cs) to check for doc not being null:
if (doc != null && doc.FontDefns().TryGetValue(f, out sFaces)) return sFaces;
I created a Pull Request #333 to resolve this issue.
Thanks

Fixed with #333.


Bad Performance on large trees

We are trying to visualize TeamCity build logs and tree can easily reach 3-5mb in size, original implementation of this lib we tried yielded 30-45seconds to build it, so I digged deeper with chrome profiler and found out that 95% of time is spent in colorHash function and 99% of it in native Regex, so I commented two lines:
    function colorHash(name) {
      // Return an rgb() color string that is a hash of the provided name,
      // and with a warm palette.
      var vector = 0;
      if (name) {
 //TWO LINES I COMMENTED:
        // name = name.replace(/.*`/, "");		// drop module name if present
        // name = name.replace(/\(.*/, "");	// drop extra info
        vector = generateHash(name);
      }
      var r = 200 + Math.round(55 * vector);
      var g = 0 + Math.round(230 * (1 - vector));
      var b = 0 + Math.round(55 * (1 - vector));
      return "rgb(" + r + "," + g + "," + b + ")";
    }

and it rendered graph in 2s with same result for us, since I am not sure what those cleanups are doing I am not ready to submit a PR. Please comment.

@Restuta I've removed the regex replaces and this should be faster in most browsers. Let me know how that performs in you tree.


Lists saved empty when the shouldn't

Summary:
Getting error: SchemaError: No list schema given for attribute: sign when sign attribute has type [Object] and saveUnknown: true. Seems to work with other types like Array even if saveUnknown is enabled. I can't use Array here though, cause this list will be populated with some objects in some cases.
Code sample:
Schema
const recSchema = new dynamoose.Schema({
  _id: {
    hashKey: true,
    required: true,
    type: String,
  },
  accountId: {
    type: String,
  },
}, {
  useDocumentTypes: true,
  saveUnknown: true,
});

Model
const Receipt = dynamoose.model('Receipt', recSchema, { update: true });

General
const main = async () => {
  const all = await Receipt.scan().exec();
  await Receipt.batchDelete(all);

  await Receipt.create({
    _id: uuid.v4(),
    accountId: '10',
    created: '2019-01-01',
    docType: 'monthlyReceipt',
    total: 0,
    payments: [],
  })
  await Receipt.create({
    _id: uuid.v4(),
    accountId: '10',
    created: '2019-01-01',
    docType: 'initialReceipt',
    total: 100,
    payments: [{ total: 100 }],
  })

  const created = await Receipt.scan().exec();
  console.log(created);
};

Current output and behavior:
Running the code for the first time, when the table wasn't created yet gives me this output:
[ Model-Receipt {
    accountId: '10',
    total: 0,
    _id: 'a63769a5-9668-4952-8ba2-09c21c2e5eac',
    created: '2019-01-01',
    payments: [] },
  Model-Rechnung {
    accountId: '10',
    total: 100,
    _id: 'da5ca6ac-5f8a-4987-b74a-ac77f3513446',
    created: '2019-01-01',
    payments: [] },
  lastKey: undefined,
  count: 2,
  scannedCount: 2,
  timesScanned: 1 ]

Note the second item in the list, which has payments: [] when actually it was expected to have an object inside it.
Running exactly the same code once again gives me an error as output:
(node:1781) UnhandledPromiseRejectionWarning: SchemaError: No list schema given for attribute: payments
    at new Attribute (/home/userx/projecty/test-dynamo/node_modules/dynamoose/dist/Attribute.js:52:23)
    at Object.createUnknownAttributeFromDynamo (/home/userx/projecty/test-dynamo/node_modules/dynamoose/dist/Attribute.js:532:18)
    at Schema.parseDynamo (/home/userx/projecty/test-dynamo/node_modules/dynamoose/dist/Schema.js:174:30)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:118:7)
(node:1781) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:1781) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Expected output and behavior:
Expected to have one item with payments = [] and the another with the payments list properly filled. Also it it's really a bug, running the same script again shouldn't pop an error.
Environment:
Operating System: Arch Linux
Operating System Version: Linux arch 5.0.5-arch1-1-ARCH #1 SMP PREEMPT Wed Mar 27 17:53:10 UTC 2019 x86_64 GNU/Linux
Node.js version (node -v): 9.8.1
NPM version: (npm -v): 5.6.0
Dynamoose version: 1.7.2
Dynamoose Plugins: No plugins used.

Dynamoose Plugins:

[ ] Yes, I believe that one or multiple 3rd party Dynamoose plugins are effecting this issue
[ ] No, I believe this issue is independent of any 3rd party Dynamoose plugins I'm using
[ ] Unknown, I'm unsure if Dynamoose plugins are effecting this issue
[x] I am not using any Dynamoose plugins

Other information (if applicable):
Type (select 1):

[x] Bug report
[ ] Feature suggestion
[ ] Question
[ ] Other suggestion
[ ] Something not listed here

Other:

[ x] I have read through the Dynamoose documentation before posting this issue
[ x] I have searched through the GitHub issues (including closed issues) and pull requests to ensure this issue has not already been raised before
[x ] I have searched the internet and Stack Overflow to ensure this issue hasn't been raised or answered before
[ x] I have tested the code provided and am confident it doesn't work as intended
[x ] I have ensured that all of my plugins that I'm using with Dynamoose are listed above
[ x] I have filled out all fields above
[ x] I am running the latest version of Dynamoose

@icaroponce It looks like you submitted two very similar tickets. They are different tho, correct?

@fishcharlie correct, from my understanding they are different, that's why I decided to create another one instead of making the previous confusing.


possibly missing an else clause in preprocessor.py

https://github.com/Khrylx/AgentFormer/blob/e4fe8dd6df3b3d2033665c5f8ac3544e81c44db5/data/preprocessor.py#L67-L69
Hi, thanks for the great work done on AgentFormer!
While going through the model's code, I noticed that there might be a missing else clause in preprocessor.py, specifically on lines 67 to 69. I'm unsure if this is a typo or if I have misunderstood something.
I would appreciate if you can give some clarification on that. Thanks in advance!

From the looks of it, the conditional does not have any effect on the definition of the data variable, as it will directly be overwritten by the assignment made on line 69.
Here, data is a block of all rows extracted from the entire .txt file, which have their timestep value equal to frame - i * self.frame_skip
Perhaps the condition imposed in line 67 was initially meant to catch the eventual case where there simply is no data to be found for the provided input frame (ie, timestep). Effectively though, the check does not have any effect
But what does happen then? That is, how does line 69 resolve when not a single row in self.gt validates the condition self.gt[:, 0] == (frame - i * self.frame_skip)?
Well, the indexing will be performed without error, but the resulting array will be essentially empty (and of shape (0, 17), with 17 corresponding to the number of columns in the initial .txt file from which the data is extracted).
Having an empty data variable here is no big deal, as this is eventually "caught" later in the call method (line 163)
https://github.com/Khrylx/AgentFormer/blob/e4fe8dd6df3b3d2033665c5f8ac3544e81c44db5/data/preprocessor.py#L163-L164
So, essentially, lines 67 and 68 can be safely removed from the script here, this won't alter the behaviour of the script (and also save a bit of computation time as well)


[Android] Other app name is not shown

on .getInstalledApp() other app names and details was not shown, it only shows the current app created.
Is is something to do with debug mode?

It turn's out that this is a problem with the android 11 needing add extra permission to the manifest.


add style field

Adding a style field to your npm package makes it compatible with sass-module-importer, which is a nice way to import css into your sass project directly from node_modules without relying on the ugly ../../ imports.
I would also recommend compiling a react-s-full.css file (that includes everything needed to use react-s-alert, including animations), and then set that as the style for your module.
The CSS files are not so big to warrant them being in their separate files, or at least for me, it was counter-intuitive. Either way, I don't think adding a full build could hurt :)

Hi, thanks. I couldn't find time to get back to this tool lately. I hope I'll be able to review this one and others  asap :)


generic poi text colors

http://localhost:8000/?animated=false#17/1.32599/103.93759

continue where we left off here:
https://github.com/tangrams/tron/commit/f45a999c676aa6d3db96601d2e13e3a16c681cfe
need generic sport icon in orange
default text still specified in red
courthouse and other icons might need new color

this is done
https://github.com/tangrams/tron/commit/ea0d6593227fe8570aade5f95fa73329d92bf610


drivers/wifi/nrfwifi: Unable to change mac address by net_mgmt

Describe the bug
When I change the mac address of nrf7002 via net_mgmt,
the function returns success, after I try to connect to the AP,
the address on the gateway remains unchanged.
To Reproduce
Add the following code to the wifi sample and run it.
static int mac_init(void)
{
    uint8_t mac[6] = {0xfa, 0xd5, 0x92, 0x57, 0xfe, 0x6f};
    struct ethernet_req_params params = {0};
    struct net_if             *iface  = net_if_get_default();

    memcpy(params.mac_address.addr, mac, sizeof(mac));
    net_mgmt(NET_REQUEST_ETHERNET_SET_MAC_ADDRESS, iface, &params, sizeof(struct ethernet_req_params));

    return 0;
}

Expected behavior
The address shown on the AP is the new address.
Impact
Unable to change mac address via net_mgmt.
Environment (please complete the following information):

OS: MacOS
Toolchain: zephyr-sdk-0.16.4
v4.0-branch

Please make sure that the network interface is down before trying to change the MAC address.

Please make sure that the network interface is down before trying to change the MAC address.

I confirmed through breakpoint debugging that it does enter nrf_wifi_if_set_config_zep, the ethernet_set_config function also checks the state of the network interface and returns an error code if it is up.

Please make sure that the network interface is down before trying to change the MAC address.

I can change the MAC address by setting the link address via net_if_set_link_addr before the network stack initialisation

Please make sure that the network interface is down before trying to change the MAC address.

I can change the MAC address by setting the link address via net_if_set_link_addr before the network stack initialisation

NET_REQUEST_ETHERNET_SET_MAC_ADDRESS

I just tried on v4.0-branch and it seems to work, right after the boot, I did net iface 1 and I could see the MAC address Link addr : FA:D5:92:57:FE:6F.

NET_REQUEST_ETHERNET_SET_MAC_ADDRESS

I just tried on v4.0-branch and it seems to work, right after the boot, I did net iface 1 and I could see the MAC address Link addr : FA:D5:92:57:FE:6F.

Thanks for your help, I found something from zephyr, I always thought that net_if_is_up returning false meant that the interface was not up yet, when I looked at the function implementation through debugging, I found that the interface was up but not running yet.

By default NRF_WIFI_IF_AUTO_START has a value of 1. In order to use net_mgmt to change the mac address I need to turn this option off and call net_if_up(iface) after setting the mac address.
static int mac_init(void)
{
    uint8_t mac[6] = {0xfa, 0xd5, 0x92, 0x57, 0xfe, 0x00};
    struct ethernet_req_params params = {0};
    struct net_if             *iface  = net_if_get_default();

    memcpy(params.mac_address.addr, mac, sizeof(mac));
    net_mgmt(NET_REQUEST_ETHERNET_SET_MAC_ADDRESS, iface, &params, sizeof(struct ethernet_req_params));
    net_if_up(iface);

    return 0;
}

I sincerely hope you can modify the following check on the status of the network interface.
static int ethernet_set_config(uint32_t mgmt_request,
			       struct net_if *iface,
			       void *data, size_t len)
{
	struct ethernet_req_params *params = (struct ethernet_req_params *)data;
	const struct device *dev = net_if_get_device(iface);
	const struct ethernet_api *api = dev->api;
	struct ethernet_config config = { 0 };
	enum ethernet_config_type type;

	......

	if (mgmt_request == NET_REQUEST_ETHERNET_SET_AUTO_NEGOTIATION) {
		if (!is_hw_caps_supported(dev,
					  ETHERNET_AUTO_NEGOTIATION_SET)) {
			return -ENOTSUP;
		}

		config.auto_negotiation = params->auto_negotiation;
		type = ETHERNET_CONFIG_TYPE_AUTO_NEG;
	}

        ......

	} else if (mgmt_request == NET_REQUEST_ETHERNET_SET_MAC_ADDRESS) {
		if (net_if_is_up(iface)) {
			return -EACCES;
		}

                 /* We need to remove the old IPv6 link layer address, that is
		 * generated from old MAC address, from network interface if
		 * needed.
		 */
		if (IS_ENABLED(CONFIG_NET_NATIVE_IPV6)) {
			struct in6_addr iid;

			net_ipv6_addr_create_iid(&iid,
						 net_if_get_link_addr(iface));

			/* No need to check the return value in this case. It
			 * is not an error if the address is not found atm.
			 */
			(void)net_if_ipv6_addr_rm(iface, &iid);
		}

		memcpy(&config.mac_address, &params->mac_address,
		       sizeof(struct net_eth_addr));
		type = ETHERNET_CONFIG_TYPE_MAC_ADDRESS;
	}
}

Replace net_if_is_up with the following code：
if (net_if_flag_is_set(iface, NET_IF_UP) ||
    net_if_flag_is_set(iface, NET_IF_RUNNING)) {
    return -EACCES;
}

A network interface in Zephyr has two active states ADMIN_UP -> administratively UP i.e., the driver is up and running, UP -> operationally UP i.e., the link can be used. For 802.11 UP is only after a successful Wi-Fi connection.
So, now to set MAC address the device shouldn't be administratively up, so, you are right about the bug, but the fix should be like below:
diff --git a/subsys/net/l2/ethernet/ethernet_mgmt.c b/subsys/net/l2/ethernet/ethernet_mgmt.c
index 703dbbccb6d..d7a49022fca 100644
--- a/subsys/net/l2/ethernet/ethernet_mgmt.c
+++ b/subsys/net/l2/ethernet/ethernet_mgmt.c
@@ -90,7 +90,7 @@ static int ethernet_set_config(uint32_t mgmt_request,
                config.full_duplex = params->full_duplex;
                type = ETHERNET_CONFIG_TYPE_DUPLEX;
        } else if (mgmt_request == NET_REQUEST_ETHERNET_SET_MAC_ADDRESS) {
-               if (net_if_is_up(iface)) {
+               if (net_if_is_admin_up(iface)) {
                        return -EACCES;
                }

And with the above patch, you can do something like https://github.com/nrfconnect/sdk-nrf/blob/main/samples/wifi/scan/src/main.c#L336.
I can submit a PR with the above fix, @jukkar @rlubos WDYT?

I can submit a PR with the above fix, @jukkar @rlubos WDYT?

Sounds reasonable.

Makes sense, just submit a proper PR. Note that your example commit has a typo in the subject (s/ehternet/ethernet/)


adding multiple columns in a single alter table statement doesn't work

When I try to parse the following valid SQL statement (it runs fine on my MySQL server) via sqlparser:
alter table sometable add column b integer, add column c integer;

Instead of successfully parsing it, I get the following error:
Expected end of statement, found: ,

It seems like sqlparser doesn't understand having multiple additions in a single alter table statement. Is it possible to make this work?

@vidmartin will work on that.
As this isn't the ANSI behaviour [(1)], wasn't a priority. I know MySQL supports it [(2)], and PostgreSQL [(3)] as well, right? Do you need it only for MySQL, or other dialect as well? If just for MySQL, I can work on that right away. If this needs to be generic, I'll have to study a little before implementing it.
[1] : https://jakewheat.github.io/sql-overview/sql-2016-foundation-grammar.html#add-column-definition
[2] : https://dev.mysql.com/doc/refman/8.0/en/alter-table.html
[3] : https://www.postgresql.org/docs/current/sql-altertable.html

I'd like to have a go at implementing this. I'd like both Postgres and MySQL support.
Looking at the documentation for Postgres, there are some "actions" that can appear in a list with multiple actions as part of a single statement, but there are some types of alter table statement that can only have a single action: https://www.postgresql.org/docs/15/sql-altertable.html
I can see two ways to implement this in terms of our types:
Option 1
Update Statement::AlterTable to take a list of operations:
    AlterTable {
        /// Table name
        #[cfg_attr(feature = "visitor", visit(with = "visit_relation"))]
        name: ObjectName,
-       operation: AlterTableOperation,
+       operations: Vec<AlterTableOperation>,
    },

Option 2
Split the AlterTableOperation into two:
pub enum AlterTableOperation {
   /// Postgres/MySQL/some others allow multiple actions in a single statement.
   /// For other dialects, this should only contain one entry
   MultipleActions(Vec<AlterTableOperationAction>),

    /// `RENAME [ COLUMN ] <old_column_name> TO <new_column_name>`
    RenameColumn {
        old_column_name: Ident,
        new_column_name: Ident,
    },

    /// `RENAME CONSTRAINT <old_constraint_name> TO <new_constraint_name>`
    ///
    /// Note: this is a PostgreSQL-specific operation.
    RenameConstraint { old_name: Ident, new_name: Ident },

    // ... other actions that cannot be grouped here...
}

pub enum AlterTableOperationAction {
    AddColumn {
        /// `[COLUMN]`.
        column_keyword: bool,
        /// `[IF NOT EXISTS]`
        if_not_exists: bool,
        /// <column_def>.
        column_def: ColumnDef,
    },
    /// `DROP [ COLUMN ] [ IF EXISTS ] <column_name> [ CASCADE ]`
    DropColumn {
        column_name: Ident,
        if_exists: bool,
        cascade: bool,
    },
    /// `ALTER [ COLUMN ]`
    AlterColumn {
        column_name: Ident,
        op: AlterColumnOperation,
    },
    // ... other actions that can be grouped in a single ALTER TABLE statement here...
}

Conclusion
My instinct is to go with Option 1. It's more flexible and I think it will be more consistent across different database systems where some support grouping certain things and others don't. I think it's more in common with the ethos of this package, which tries to be agnostic to the semantics of the queries it parses.


remove grunt-cli from peer dependencies

:+1: This works when running by hand on the server, so I'm merging it in while we figure out the Sauce Labs issues

Thank you.


📦 Publish a container image for kcctl to DockerHub

Hey @aalmiray, is this something JReleaser would help with as well?

Yup, JReleaser can make it happen

@gunnarmorling we need the following information:

a username/password for publishing to DockerHub. Store the password as a GitHub secret, I suggest using KCCTL_DOCKER_PASSWORD.
a DockerHub repository. The computed default would be kcctl/kcctl but feel free to pick a different name.
pick the base image. I think executables generated with Graal as we have them today require glibc, in which case that rules out any Linux Alpine base image. We could go with scratch or an Ubuntu based image.

Slight complication, the docker packager expects .zip or .jar files as inputs. We currently have .tar.gz for the Linux artifact. The quick workaround is to use the .zip produced by the build, making sure the zip file is not signed/released (which JReleaser 0.6.0 can do).

I'll try and register the kcctl/kcctl repo. Re the base image, the Quarkus build can produce an image itself (either based on RHEL UBI, or distroless).

That's the thing, at the moment JReleaser creates a Dockerfile based on a template, then creates the image, then published to a registry of your choice. It does not accept publishing a container image that was built from the outside, yet.

Ok, I see. I suppose we still can use the existing Dockerfile, right? One other question: can we deploy to quay.io instead of DockerHub? It seems the latter requires a paid plan now for setting up new organizations :(

Yes, we can deploy to Quai as well. You could setup a personal account at DockerHub, that's what I did for JReleaser, I did not migrate that account to org.

https://github.com/jreleaser/jreleaser/issues/358 Prevents the use of the quay.io/quarkus/quarkus-distroless-image:1.0 image however registry.access.redhat.com/ubi8/ubi-minimal:8.4 may be used. These two images are the ones defined by the Quarkus templates.

Hello All,
I'm using kcctl as container. Really a nice, well readable tool for kafka connect.
Github: https://github.com/mdwajid095/wowsome/tree/main/kcctl
DockerHub: https://hub.docker.com/r/mdwajid095/kcctl
I think, we should include the docker image part in this project here itself. So, that everyone can use it. Thanks!


⚠ Bofrost.be has degraded performance

In 38f7d39, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1818 ms

Resolved: Bofrost.be performance has improved in f4900d4.


Improve behavior and performance on Vim

Always use a workaround version in Vim so that errors that occurred in Vim become detectable (without g:denops#debug = 1)
Improve performance of batch approx 5 times

Note
Before: 43.5 char/sec (4.35 call/sec) [s=10, n=100]
After:  200  char/sec (20.0 call/sec) [s=10, n=100]

Wrong unit. Correct ones are char/msec and call/msec.


different animation on enter/leave?

Can we specify different animation for the enter/leave or hide/show events?
For exemple, I would like an fx-fade-up on enter and fx-fade-down on leave, is it possible?
Thanks

@sinedied , did you managed ?

No, still no workaround :( I ended up using animate.css directly...


Add dry_run command and associated ci_source and request_source

motivation is ability to run danger against local only repo
in condition when there is no actual PR/MR created (or is not available
to danger).

Ah nice yeah, this is basically what danger local is in Danger JS 👍
The build failed because the internal plugin you made needs documenting as it ends up in the reference page of danger.systems

Errors

Examples - DangerfileLocalOnlyPlugin (class):

You should include some examples of common use-cases for your plugin.
@see - https://github.com/dbgrandi/danger-prose/blob/v2.0.0/lib/danger_plugin.rb#L8#-L27

/home/travis/build/danger/danger/lib/danger/danger_core/plugins/dangerfile_local_only_plugin.rb:10

Oh, sorry for letting the PR broken (I was little too tired to wait for tests =/ - ashamed), will fix that today or tomorrow ...
and thanks for instruction how to fix ;o)
=)K

👍 looking good, once #1020 is ready I'll ship a release


Add a RHEL8 s390x worker

@stratakis Do you have a worker password already?

I've got worker passwords for the rest of my buildbots (different for each), but notpassword has been generated for me since the new config was deployed. Can I reuse a pass from one of my workers or does a new one have to be generated?

With some recent changes to the config (namely the one that made the worker setup more public), you can have a single password for all workers or a unique password per worker.  You currently have both; your RHEL{7,8}-x86_64 and fedora-stable workers each have their own password while your fedora-rawhide worker is using your "owner" password.  We can generate a new one for this worker, or you can use the same one as fedora-rawhide as soon as this is deployed, whichever you would prefer.

Putting it on hold for the time being.


instructions for finding serial number are wrong

https://edukit.workshop.aws/en/blinky-hello-world/device-provisioning.html
"For optional fleet management, you’ll need to replace the common name value with the device’s serial number. You can retrieve the device’s serial number by going to the Core2-for-AWS-IoT-EduKit-Factory-Firmware folder and entering the command idf.py build flash monitor -p <<DEVICE_PORT>>. Replace «DEVICE_PORT» with the system port your device is logically mounted on."
idf.py not found.  Also, it's a nit, but the device is not logically mounted - better to say "Replace «DEVICE_PORT» with the serial port device your Core2 is connected to"

If you've followed the earlier pre-req for the tutorial on https://edukit.workshop.aws/en/blinky-hello-world/prerequisites.html#installing-esp-idf-v42, the ESP-IDF. will be added to your path. You'll need to run the export.sh (Linux/OSX) or export.bat (Windows) script to add it back to your path again. Additionally, you'll need to reactivate your conda environment if it isn't already.
They are logically mounted, but they are not a logical volume since there's none. I agree that it would be simpler to write it that way.

Thank you for the great feedback @gherlein. Updated the verbiage to be simpler: https://edukit.workshop.aws/en/blinky-hello-world/device-provisioning.html#generating-a-device-certificate-manifest
Hopefully running the ESP-IDF's export script resolved the "idf.py not found" issue for you.


One way binding

We're having some trouble when binding value to a query param, with null/undefined. Is there any way to disable two-way binding? Ideally the change would only send out an action, rather than updating the value that was passed in.

Do you mean selection instead of value?

Either, really just want the select box to update if the value passed into it updates, but when user changes select box, just want it to send out an action (and not update either value or selection)

That sounds exactly like the definition of "data down, actions up".
Didn't think of that use case.
We may need to do some slight changes to make this conform that paradigm.

right. Maybe an opt-in to disable the two-way binding so it's backwards compatible.

https://github.com/miguelcobain/ember-cli-selectize/blob/master/addon%2Fcomponents%2Fember-selectize.js#L256
I'd say that, in the _updateSelection function, we would either trigger an action or update the value, but not both?

yah, again you can leave both on by default if you dont want to break bwds compatibility, but if twoWay=false don't run line 257

it's going to be a bit more than that line. I commented it out locally, but there's still some wackiness going on

Is your select multiple or single?

single. I think it has to do with _valueDidChange, but commenting out that method breaks the "data down" part (select doesn't represent value passed in)

There are a couple of  set('selection',...). Any of those may be the culprit.

right. So it looks like I actually got what I want with this:
// components/ember-selectize.js
import Component from 'ember-cli-selectize/components/ember-selectize';

Component.reopen({
  _updateSelection: function(selection) {
    // allow the observers and computed properties to run first
    Ember.run.schedule('actions', this, function() {
      var value = this.get('value');
      this.sendAction('select-item', selection, value);
    });
  },
  _valueDidChange: Ember.observer('value', function() {
  }),
});

export default Component;

This lets me pass in selection, but ember-selectize itself doesn't call .set, it just sends out the action.
This didn't work when I just passed in value though.

Glad you made it work for now.
I'll restructure the code a bit more so that we have a single point of updates.
That way we can have a better control.
Meanwhile, let's keep this open.
Thanks.

Would love a twoWay=false option.

I would prefer to infer that by checking if actions are bound or not.
Basically, if select-item action is used, don't update selection or value.
This would of course break if people are using both actions and two-way bindings. But I kind of think of that as an anti-pattern.
Thoughts?

Agreed. I'm assuming this would work for add-item and remove-item as well.

From all the actions, maybe only if any ofselect-item, add-item or remove-item is set, we don't update selection or value.
create-item and update-filter shouldn't trigger this behavior because people who wish to use two bindings may need them.

@knownasilya I've also started wrapping selectize (and other libs) in my own component code and using variables local to that wrapper for the two-way binding, but only sending actions out. This is a short-term way to enforce one-way binding in your app. (Also, I think wrapping 3rd-party components in your application is generally a good pattern).

@samselikoff feel like sharing your wrapper?

@miguelcobain were you able to make those changes for the actions?

I'm not sure what you mean by final value.
I usually just have something like {{ted-select content=content selection=selection}} and so on, and internally to the component I'd have
export default Ember.Component.extend({
  _selection: Ember.computed.oneWay('selection');

  emitActions: Ember.observer('_selection', function() {
    if (this.get('_selection') !== this.get('selection') {
      this.sendAction('on-select', this.get('_selection');
    }
  }
});

and then the component's template would be something like
{{view 'select' content=content selection=_selection}}

So input is [{ id: 1, value: 'test' }] and output is [1], which would be the final output, since ember-selectize doesn't support value and selection needs to be transformed.

output meaning, the parameter of the on-select action?

See https://gist.github.com/knownasilya/f200d462e6c3443f41ac, trying to get the selection to turn into and object, but having a hard time. It works on initial load if selection is an array of ids (internally it's replaced with object versions), but on-select it reverts back to the id version. I've tried a few things but keep running into stack overflows.. I can't wait for (mut).

did you try _selection: Ember.computed.oneWay('selection');? If you do that selectize should never mutate the thing you pass in

Is there a way to do oneWay with a full-blown computed property? Since I need to transform the users data from array of values, to an array of objects.

I dont think so, can't you just make sure your CP is only composed of other readonly properties?

From all the actions, maybe only if any of select-item, add-item or remove-item is set, we don't update selection or value.

@miguelcobain is it true in the current version?

Most people are using ember-power-select now a days.

@knownasilya thanks for the tip


Carbonless Movement v2 -- Own real tree NFT for each 10 mints

Climate change has terrible impact on planet, thus we've decided to step up game and contribute in reverting effects on the planet through NFTs. Experimentally we bought credits from offsetra and yet we've decided to go planting with RMTerra for green projects.
Despite Polkadot it lowest carbon footprint (link doc someone from recent tweets pls) we actually would like setup right scheme to help planet and incentivize creators to mint on KodaDot.
Thus my briefly enumaration would be keep numbers simple and plant 1 tree for each 10 mints you make on KodaDot. Plant 1 tree for each 30-50 mints you buy on KodaDot and so on.
I just need get proper number from Pedro and deep dive into numbers to be more in touch with reality and tha's the situation down there in Portugal where we are actually finished first phase of planting trees :)
Actually point would be nice to own one tree, so I guess you would be way more incentivized to mint on KodaDot, as we will not giving carbon credits or offsets but hard tree you can pay a visit in Portugal ;)
Ref

https://kodadot.xyz/carbonless/
https://www.youtube.com/watch?v=tFo3vftXsdg
[13:17] Lightsonemusic: Hi guys from 19 to 20 this month we will be planting more 330 trees for Koda forest

obsolate


+option for display mail format (text/plain | html) (Issue 4456)

This is based on aatdark's implementation.
Please review.

+1

+1

Looks good to me, for what that's worth.

Definitely +1

I do not understand. Is this pull request merged to the master branch now?
Then where do I find it?
Thanks

The pull request is still open. What has been closed is the old pull request for this feature.

@farmboy0: Did you solve this issue? Is this feature working on your fork? Is your fork along the master of k-9? Then I would like to fork your fork :)
Thank

@francwalter I havent rebased my master branch to k-9 master branch since I created this pr.
Youre free to do that yourself. I dont think you will encounter any merge conflicts.

I assume the goal is to preferably display text/plain parts instead of text/html when both are present as alternatives.
Currently the code is in a bit of a transitional state. Previously we went through the message and extracted both a text and an html string to write to the database. The html string was used for display in the user interface. The text string was used as the quoted text when creating replies. When only one of text/plain or text/html is present it is converted to the other format.
Now we save messages in a way that allows us to restore the original structure. To avoid redundancy in the database we no longer extract the text and html strings to write them to the database. Instead we perform this step before displaying the message.
Nowadays, we should only extract an html string because that's what we display in the WebView.
To implement the feature of allowing users to prefer text/plain parts for display we have a couple of options:

Use the old mechanism to extract text and then convert that to HTML for display in the WebView.
Use the old mechanism to extract text and then display that in the WebView using the MIME type text/plain.
Change the extraction code to preferably use text/plain parts when alternatives are available. That part is then converted to HTML and only the html result is returned.

Option 1 has the issue that we might first convert a text/html part to plain text and then back to HTML.
Option 2 isn't great because WebView is kinda bad at displaying text/plain content.
Option 3 is my preferred approach but requires a bit more work.
The code in this pull request implements option 1 only that the "conversion" to HTML is done wrong (i.e. not using HtmlConverter.textToHtml()) and in the wrong place (inside LocalMessageExtractor.buildText() instead of converting the result of LocalMessageExtractor.extractTextAndAttachments()).

The code to display messages will change a lot because of the PGP/MIME changes. Afterwards it's hopefully easier to properly implement this feature.
Closing this issue for now. I created #906 so we don't forget about this enhancement.


suggestionsTransform is not run on the first time ListBox is rendered

Steps:

Add suggestionsTransform to add some prefix (e.g. #)
Focus the input
See the listbox rendered
Expected: # is shown
Actual: # is not shown
Type a character in the input
# is now shown
Clear the input
# is still shown

Thanks for the report, should be easy to fix 👍

v7.0.0-beta.9 has been released with the fix.


Enhanced Highlighting Causes Hang On C# Files

Omnisharp has begun causing Atom to hang when switching to specific C# files.
After much turning on/off of packages and settings, I've found that the problem disappears if I turn off Enhanced Highlighting.

If the project is open source and you have the source file that hang
happens on, please let me know, that help's a ton in identifying where the
hang is at.
On Thu, Dec 3, 2015 at 1:00 PM, c.miller<EMAIL_ADDRESS>wrote:

Omnisharp has begun causing Atom to hang when switching to specific C#
files.
After much turning on/off of packages and settings, I've found that the
problem disappears if I turn off Enhanced Highlighting.
—
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-atom/issues/580.

It's not open source, but I'd be happy to send it to you if need be.

It's up to you on that part. :smile:
My email is david.driscoll at gmail.com

Files sent.

So I've pushed monkey patch, that should stop atom from locking up on C# editors when we don't quiet modify the tokenization properly.
If you notice any artifacts, like a tokenized line that goes on and on, when it shouldn't, follow that up with an issue, as we might need to address that.


CMake Error: The source directory “C:/Users/hp” does not appear to contain CMakeLists.txt.

I am trying to setting up this (aviso-fes/INSTALL.md at main · CNES/aviso-fes · GitHub).
installed successfully this command (conda install pyfes -c fbriol )
Then successfully ran this command ( mkdir build && cd build)
Problem starts here when i run this command( cmake … -DCMAKE_INSTALL_PREFIX=/opt/local )

for this third command I receive following massage
(avisofes7) C:\Users\hp\build>cmake … -DCMAKE_INSTALL_PREFIX=/opt/local
CMake Warning:
Ignoring extra path from command line:
“…”
CMake Error: The source directory “C:/Users/hp” does not appear to contain CMakeLists.txt.
Specify --help for usage, or press the help button on the CMake GUI.
I am using windows 10, anaconda env, created this particular enviroment on python 3.7, as newer pythons do not allow to install first command

If you want to build the code with cmake, you have to clone the GIT repository first, then run the setup and build commands.
git clone https://github.com/CNES/aviso-fes.git
cd aviso-fes
mkdir build
cd build
cmake … -DCMAKE_INSTALL_PREFIX=<TARGET PATH>

If you want to use only, the Python interface, you can also use the python script "setup.py" present in the root of the repository.
git clone https://github.com/CNES/aviso-fes.git
cd aviso-fes
pip install .

Thank you sir for reply. which should be the  in this command please? cmake … -DCMAKE_INSTALL_PREFIX=

Thank you sir for reply. which should be the TARGET PATH  in this/last command please? cmake … -DCMAKE_INSTALL_PREFIX=

Dear sir,
found error :
(avisofes) C:\Users\hp\aviso-fes-2.9.4\aviso-fes\build>cmake ..
-DCMAKE_INSTALL_PREFIX=/opt/local
CMake Error at CMakeLists.txt:34 (project):
Running
'nmake' '-?'
failed with:
The system cannot find the file specified
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also
"C:/Users/hp/aviso-fes-2.9.4/aviso-fes/build/CMakeFiles/CMakeOutput.log".
plz help to resolve this issue
regards
alee
On Sun, Jul 24, 2022 at 12:10 PM Frédéric BRIOL @.***>
wrote:

If you want to build the code with cmake, you have to clone the GIT
repository first, then run the setup and build commands.
git clone https://github.com/CNES/aviso-fes.git
cd aviso-fes
mkdir build
cd build
cmake … -DCMAKE_INSTALL_PREFIX=
If you want to use only, the Python interface, you can also use the python
script "setup.py" present in the root of the repository.
git clone https://github.com/CNES/aviso-fes.git
cd aviso-fes
pip install .
—
Reply to this email directly, view it on GitHub
https://github.com/CNES/aviso-fes/issues/11#issuecomment-1193261621, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APASDCLIQUZMMTJ3SSWCQQTVVTT6TANCNFSM54OGFKTA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Muhammad Ali Solangi
Assistant Professor
Department of Geology
Sindh University Campus Mirpurkhas
University of Sindh, Jamshoro
Sindh, Pakistan**Phone#  +92<PHONE_NUMBER>
sucm.usindh.edu.pk

I have generated new packages automatically for python 3.7, 3.8, 3.9 and
3.10 for windows, OSX and Linux. Don't compile your version, but use conda
to install the latest version of the library.
conda install -c fbriol pyfes
F.
Le mer. 10 août 2022 à 11:43, Muhammad Ali Solangi @.***>
a écrit :

Dear sir,
found error :
(avisofes) C:\Users\hp\aviso-fes-2.9.4\aviso-fes\build>cmake ..
-DCMAKE_INSTALL_PREFIX=/opt/local
CMake Error at CMakeLists.txt:34 (project):
Running
'nmake' '-?'
failed with:
The system cannot find the file specified
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also
"C:/Users/hp/aviso-fes-2.9.4/aviso-fes/build/CMakeFiles/CMakeOutput.log".
plz help to resolve this issue
regards
alee
On Sun, Jul 24, 2022 at 12:10 PM Frédéric BRIOL @.***>
wrote:

If you want to build the code with cmake, you have to clone the GIT
repository first, then run the setup and build commands.
git clone https://github.com/CNES/aviso-fes.git
cd aviso-fes
mkdir build
cd build
cmake … -DCMAKE_INSTALL_PREFIX=
If you want to use only, the Python interface, you can also use the
python
script "setup.py" present in the root of the repository.
git clone https://github.com/CNES/aviso-fes.git
cd aviso-fes
pip install .
—
Reply to this email directly, view it on GitHub
https://github.com/CNES/aviso-fes/issues/11#issuecomment-1193261621,
or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/APASDCLIQUZMMTJ3SSWCQQTVVTT6TANCNFSM54OGFKTA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Muhammad Ali Solangi
Assistant Professor
Department of Geology
Sindh University Campus Mirpurkhas
University of Sindh, Jamshoro
Sindh, Pakistan**Phone# +92<PHONE_NUMBER>
sucm.usindh.edu.pk
—
Reply to this email directly, view it on GitHub
https://github.com/CNES/aviso-fes/issues/11#issuecomment-1210427092, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADBAST257ZDPL5GINAP7P3VYN2VHANCNFSM54OGFKTA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Installation Instructions
Requirements
cmake: CMake
netcdf:
With reference to url pasted below, when I install cmake and hdf5 (which are mentioned as requirements in installation instructions) they result in conflict and incompatibility.  it results as that strict channel priority remove packages required for satiffiabiliy.
https://github.com/AimAlee/aviso-fes/blob/main/INSTALL.md

I have generated new packages automatically for python 3.7, 3.8, 3.9 and 3.10 for windows, OSX and Linux. Don't compile your version, but use conda to install the latest version of the library. conda install -c fbriol pyfes F. Le mer. 10 août 2022 à 11:43, Muhammad Ali Solangi @.> a écrit :
…
Dear sir, found error : (avisofes) C:\Users\hp\aviso-fes-2.9.4\aviso-fes\build>cmake .. -DCMAKE_INSTALL_PREFIX=/opt/local CMake Error at CMakeLists.txt:34 (project): Running 'nmake' '-?' failed with: The system cannot find the file specified CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage -- Configuring incomplete, errors occurred! See also "C:/Users/hp/aviso-fes-2.9.4/aviso-fes/build/CMakeFiles/CMakeOutput.log". plz help to resolve this issue regards alee On Sun, Jul 24, 2022 at 12:10 PM Frédéric BRIOL @.> wrote: > If you want to build the code with cmake, you have to clone the GIT > repository first, then run the setup and build commands. > > git clone https://github.com/CNES/aviso-fes.git > cd aviso-fes > > mkdir build > cd build > > cmake … -DCMAKE_INSTALL_PREFIX= > > If you want to use only, the Python interface, you can also use the python > script "setup.py" present in the root of the repository. > > git clone https://github.com/CNES/aviso-fes.git > cd aviso-fes > > pip install . > > — > Reply to this email directly, view it on GitHub > <#11 (comment)>, or > unsubscribe > < https://github.com/notifications/unsubscribe-auth/APASDCLIQUZMMTJ3SSWCQQTVVTT6TANCNFSM54OGFKTA > > . > You are receiving this because you authored the thread.Message ID: > @.> > -- Muhammad Ali Solangi Assistant Professor Department of Geology Sindh University Campus Mirpurkhas University of Sindh, Jamshoro *Sindh, PakistanPhone# +92<PHONE_NUMBER> sucm.usindh.edu.pk — Reply to this email directly, view it on GitHub <#11 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADBAST257ZDPL5GINAP7P3VYN2VHANCNFSM54OGFKTA . You are receiving this because you modified the open/close state.Message ID: @.***>

(new_avisfes) C:\Users\hp\aviso-fes-2.9.4\build>cmake .. -DCMAKE_INSTALL_PREFIX=/opt/local
CMake Error at CMakeLists.txt:34 (project):
Running
'nmake' '-?'
failed with:
The system cannot find the file specified
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also "C:/Users/hp/aviso-fes-2.9.4/build/CMakeFiles/CMakeOutput.log".
goth this error please


ignoreLintWarnings option

Attempts to address: https://github.com/Realytics/fork-ts-checker-webpack-plugin/issues/212
Adds a new option ignoreLintWarnings.
Question:
Should this have a test?
Thanks!

Should this have a test?

Yes definitely 😄
Take a look at the current tests here: https://github.com/Realytics/fork-ts-checker-webpack-plugin/tree/master/test
Hopefully there's a natural place for a new test to "fit" - unit or integration test I'm not sure.  Hopefully an obvious candidate should present itself.

@johnnyreilly added test. Let me know if it's not correct.

Thanks! I'll take a look soon

Hey @lizzzp1
I just wanted to check that the functionality in your PR not superseded by this PR:
https://github.com/Realytics/fork-ts-checker-webpack-plugin/pull/214
I kind of think they're different enough but I wanted to double check with you.

Hi @johnnyreilly, yeah that does look to have different functionality.

Cool - I'd like to get yours in first. Could you update the package.json and CHANGELOG.md please?

@johnnyreilly what needs to be updated in package.json?

Awesome - thanks for your work!
Once the CI is happy I'll look to merge this and get a release out. 🤗


Introduce caching for tracking files.

Reason:
The tracking file manager typically in large m2e projects will be used
hundreds of times (due to hundreds of artifacts). If the projects are
refreshed the files are re-read again and again. This could amount to as
much as 43% of time spent in m2e MavenBuilder (in my scenario on Windows
platform)
Changes:
This commit introduces the read cache for tracking files assuming that
those files do not change frequently (or at all).
The cache entries are verified once per minute if the underlying file
had changed its lastModified timestamp and if so they are re-read. If
the file timestamp had not changed the entry is valid for one more
minute.
Additionally since on Windows canonicalization is expensive operation it
is also being cached.
Cached properties are defensively copied when returned.
TODOs:

the cache is unbounded and there is no clearing policy based on LRU
and maximum size (for instance). However for typical usages this will
not be a problem since number of artifacts is probably couple of
thousands and should not pose a problem for memory.

I'm making this PR without much hope of approving it. The tracking files performance may not be something that is perceived as problematic by maintainers of this library, but I've stumbled on this issue while profiling m2e builds which are taking quite long (and I've decided to do something about it).

Please split this and canonicalization. Both things are different. Moreover, I need one JIRA issue per PR.

I believe that the TrackingFileManager suffers from the same problem as in MRESOLVER-114. See my understanding of the issue here. I started digging in the entire locking problem and the String#intern() is a bad design choice because that data is retained in memory.

Please read: https://issues.apache.org/jira/browse/MRESOLVER-68?focusedCommentId=17172644&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17172644

See: https://issues.apache.org/jira/browse/MRESOLVER-132

Superseded by #67.


Thread view broken

I made a post and astral doesn't show the replies. I can open for example this reply just fine but it doesn't show on the former link.

hmm thats very weird. I am seeing the replies when i go to the 'post' link. maybe the replies hadn't been loaded to the db yet?

I also thought it might not be in the db yet but as I could load the one reply just fine I would expect that's indicative of the event being in the db but still, loading again the parent event the reply was missing. Now it loads fine.

have you seen this happen again? can I close?

I haven't noticed it again. Closing.


Calling Self not working properly?

Godot version:

Godot Engine v4.0.dev.calinou.06ae14dd8
OS/device including version:

Win10 Home x64 desktop PC
Issue description:

New to GDS. Set up to do a tutorial or two with 4.0.
One line of code giving me resounding issues, in which a Timer needs to call on self.
As you can see from the screenshot, I did a setup in the stable build of Godot 3.2.3 of the same tutorial with the exact same code, and it worked perfectly fine.

As you can see, the 4.0 version is bugging out pretty badly. For some reason I'm not even seeing a list of methods and Node2D.tscn isn't properly indicated in the FileSystem? Am I missing something? Am I crazy? Is there some new-fangled thing I'm not finding?
Steps to reproduce:

Make Node2D
Child Sprite2D and Button to Node2D
Child Timer to Sprite 2D
Add Signal "pressed()" to Button and connect to Sprite 2D
Add script as written to Sprite 2D:

extends Sprite2D
var speed = 400
var angular_speed = PI
func _process(delta: float) -> void:
rotation += angular_speed * delta
var velocity = Vector2.UP.rotated(rotation) * speed
position += velocity * delta
func _on_Button_pressed():
set_process(not is_processing())
func _ready():
var timer = get_node("Timer")
timer.connect("timeout", self, "_on_Timer_timeout")
func _on_Timer_timeout():
visible = not visible
Minimal reproduction project:

Godot Project 01.zip

In 4.0 there is a new version of GDScript and signals are no longer connected with strings. Functions are also first class citizens now and can be passed by reference as a Callable object.
Read more in the old PR https://github.com/godotengine/godot/pull/39093


Partial ratings not rendered correctly

Hi,
I have a rating component rendering a rating eg, "4.5". The component renders it as 4. Is it possible to have partial filled starts when rating has decimal value?
Thanks

The current version doesn't support partial ratings, will add it in the next release.


Add keywords and categories to Cargo.toml

I just added some keywords (one of my first contribution to "Call for participation"): https://github.com/rust-lang-nursery/bitflags/pull/93
I did not found any relevant categories here: https://crates.io/categories

This crate is actually a good fit for the No Standard Library category :)

Thanks @rap2hpoutre !


Feature Request: Availability in US West 2?

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like.
Currently, it's available for US West and US West 3, how about US West 2? Is it just not available through the UI Portal? Does it work if I set it up using Azure CLI? The Container App sailing page says it's available for US West 2 but I can't get this option when creating through the UI Portal =/
Describe alternatives you've considered.
N/A
Additional context.
N/A

I was trying to deploy a Container Apps environment in West US 2 and got this error:

List of available regions for the resource type is 'northcentralusstage,canadacentral,westeurope,northeurope,eastus,eastus2,eastasia,australiaeast,germanywestcentral,japaneast,uksouth,westus,centralus,northcentralus,southcentralus,koreacentral,brazilsouth,westus3,francecentral,southafricanorth,norwayeast,switzerlandnorth'."

Is there any plan to enable Container Apps in West US 2 anytime soon?

Is there any plan to enable Container Apps in West US 2 anytime soon?

I would like to know too!

The region is available


Custom chart UIs show PSP switch on Kubernetes v1.25

Summary
Fixes #8434

Occurred changes and/or fixed issues
PSP checkbox to be displayed if K8S < 1.25 or PSP is already enabled.
This is applied for the charts:

OPA Gatekeeper
Istio
Logging
Monitoring
Rancher Backup
Alerting
Epinio
Longhorn
NeuVector
Prometheus Federator

Technical notes summary

Moved markup within the component to avoid empty spaces
Allowed to pass title for areas which require one
Passed cluster object to parse version
Extended tests
Created helper to get ignored questions with variables matching the path of the current cluster (ignore logic already existed) for same condition
Added tests to the helper, created separately due high complexity of the component (2500+ lines)

Areas or cases that should be tested
All the charts above for clusters with K3/RKE2 and K8S < v1.25 or >= 1.25 and PSP disabled/enabled
Areas which could experience regressions
Same as above.
Screenshot/Video
Custom charts
PSP checkbox present in all the charts for 1.24 and not 1.25
https://user-images.githubusercontent.com/5009481/227222888-c7e4e624-3a73-4af3-95c3-87e716d5f8e5.mp4
Proof of Cluster update (with error in the navigation 🤔 )
https://user-images.githubusercontent.com/5009481/227243291-86042c2e-0b8e-4917-b365-da42a900a45b.mp4
Disabling PSP but with an error in the server response
https://user-images.githubusercontent.com/5009481/227243277-c9369f63-9a5e-460b-91c1-4a22d30b84ed.mp4
Standard charts (question.yaml)
PSP checkbox present in the charts for 1.24 and not 1.25
https://user-images.githubusercontent.com/5009481/227595029-4f15e81e-92f0-4f09-a96c-81de8057c1eb.mp4
Disabling PSP but with an error in the server response
https://user-images.githubusercontent.com/5009481/227595125-055229b4-22a1-4be8-b604-98d5e6467e6a.mp4

@mantis-toboggan-md I have added the logic, tests, description and videos also for the standard charts.


inject-slf4j: Cross-build for Scala 2.13

Depends on #528
Problem
inject-slf4j is not cross-built for Scala 2.13.
Solution
Update inject-slf4j module to cross-build for Scala 2.13 using new SBT
settings.
Result
inject-slf4j is cross-built for Scala 2.13.

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

JVM 11 failure in the build seems like https://github.com/twitter/finagle/issues/832

Merged at https://github.com/twitter/finatra/commit/4b78bd5fcaef07fed205dad18402e63eb59b3a60, thanks!


Vaccinator status effect icons broken

Fixed in https://github.com/PazerOP/CastingEssentials/commit/3fbda082d09672ff29346ae5af1a8f0c9ad865f5.


New owner for broofa/node-uuid?

I'm not giving this repo the love it deserves.  Who would be the best person(s) to hand this off to?
@defunctzombie, from what I can tell you have the most popular fork at this point.  Thoughts?

@broofa thoughts are as follows:

Determine the API difference between this repo and my fork. If the differences are negligible then we can see which repo contains what fixes (iirc my fork exists because some crypto and browser fixes went into it).

Once we determine the codebase we want to use, hand it off to the node.js foundation https://github.com/nodejs. I think uuid is a simple and "core" enough module that the project itself could be the owner of it because I too don't actively work on the uuid fork all that much (since the code doesn't need active changes).

/cc @mikeal @bnoordhuis @rvagg

That seems reasonable to me.  As you say, the differences should be minimal.  The master branch has been pretty stable for a while. I've got a v2.0.0 branch that I was messing around with a while back trying to figure out how to restructure code so it'd be possible to add v3 + v5 support w/out bloating things in the simple case (e.g. where someone just wants v4 uuids)... but that's only half-done, so not worth worrying about.
If nodejs folks are willing to pick this up, I'm more than happy to hand off.

Not speaking ex cathedra but I don't expect that the Foundation would be open to taking over maintenance.  We aim to be neutral first and foremost but adopting a module is inevitably seen as picking a winner.

I don't think there's a "winner" to be picked here.  At least, not between the repos that @defunctzombie and I maintain.  I'm sure he and I can merge our two versions back together in a way that resolves any conflicts (right?)
... or are there other uuid modules out there you're worried about, @bnoordhuis?

Could always: npm deprecate node-uuid@"< 1.4.7" use uuid package

@broofa @defunctzombie So is it likely that this merge will eventually end up migrating over to the uuid package name? It feels like a bit of a misnomer to have the node- prefix on a package that can be run in either node or browser environments.
It would also help clear up issues like request/request/pull/2182 about which package name will become the new "official" uuid package.

... or are there other uuid modules out there you're worried about

The existence of an "official" uuid module may discourage competition.  Some would say that is a good thing but I speculate the Foundation would rather not go there.

@vdh yes, the intent would be to use the uuid package name.

As it seems like the foundation is probably not that much into adopting a module like node-uuid/uuid, would it make sense to look at an intermediate solution to get the project on track.
The primary thing seems to be migrating node-uuid and uuid. Is it possible under one of the current owners, or do we need to find another owner to proceed with this task? In case we need to find another owner, our organisation is willing to adopt the project to get it back on track. In case the foundation is up to adoption at a later point we can pass it on to them.

FYI, I've added @defunctzombie (Roman) as a contributor here, and collaborator on NPM.
Roman, I think it would be great to unify these modules.  The one problem I see - and it's the main issue I've struggled with trying to move this code forward - is that node-uuid/uuid.js can be dropped into a browser without any packaging or module loading system needed.  This makes it super-versatile, but anyone doing that - using the code "bare" - is very sensitive to dependency size and structure.  That you're fork has gotten away from this issue is refreshing, but also problematic as far as merging our projects back together.  I'm not sure how best to support the audience that just wants to do <script src="./uuid.js"> ... or if that's even worth worrying about at this point.
My $.02

@broofa Thank you, here is the plan I propose. If you are good with it, I will begin the transition.

Create a new repo under https://github.com/kelektiv called node-uuid and push the latest version of (defunctzombie/node-uuid) there (adding you and I as collaborators). I do not select this because it is my version but because I think the uuid.js file in that version is a simpler file to maintain (avoids browser boilerplate and allows js packaging system(s) to take care of providing the browser rng shim via package.json browser field specification. If you think this is incorrect or there are patches on broofa/node-uuid which supercede the defunctzombie fork please advise on preference.

Update node-uuid on npm to add a deprecation notice so that installing the module will tell people to just install the uuid module. Under no circumstances will we remove node-uuid from npm as this would cause unnecessary breakage.

Since the repo under kelektiv is not a move of either defunctzombie/node-uuid or broofa/node-uuid, update the README files of both projects saying that the project now lives at kelektiv/node-uuid. Close any issues or PRs in both projects and disable the issue feature in both projects. It is important that neither project on github is outright deleted as it should remain a reference for the appropriate past version of a module someone may install.

Add a section to the new repo readme on how to use one of the many npm based CDNs (https://unpkg.com/#/) for folks that just want a copy/paste solution for their script tags without learning how to use a package builder (altho we will still recommend a package builder for anything that goes beyond prototype stage).

Create a new repo under https://github.com/kelektiv called node-uuid and push the latest version of (defunctzombie/node-uuid) there (adding you and I as collaborators). I do not select this because it is my version but because I think the uuid.js file in that version is a simpler file to maintain (avoids browser boilerplate and allows js packaging system(s) to take care of providing the browser rng shim via package.json browser field specification. If you think this is incorrect or there are patches on broofa/node-uuid which supercede the defunctzombie fork please advise on preference.

Also acceptable alternative to this is to move broofa/node-uuid to the org and then apply all the changes from defunctzombie/node-uuid wholesale. This will preserve the stars and repo followers (since broofa/node-uuid has more).

That all sounds good to me.
Can you provide some background on what/who https://github.com/kelektiv is?  My apologies if I should know, but this is the first I've heard mention of that Github account, and there's zero info on the profile page.

... I know this goes without saying, but the top priority here has to be simply, "Don't break anything for the ~7K projects that depend on node-uuid and uuid, collectively".   :-)
Re: moveing node-uuid, I'm fine with that...  I'd just like to know something about who the owning person/organization would be before giving the final go-ahead.

Can you provide some background on what/who https://github.com/kelektiv is? My apologies if I should know, but this is the first I've heard mention of that Github account, and there's zero info on the profile page.

Apologies, should have also done that. It is an organization @ncb000gt and myself created to collect some of our more highly used node modules for easier contributor access (personal github repos don't allow more than the owner to be an admin or invite others). We have so far moved bcrypt and cron modules to it.

Cool, thanks.  So, one last thought.  If broofa/node-uuid moves to kelectiv and becomes the repo on which active development takes place, where would maintainence development on node-uuid take place should that be necessary?    I know  we'll be deprecating it, but you never know ... :-)
I guess a node-uuid branch would be sufficient.  Does that seem reasonable?

Hmm.  Looks like I need admin rights to kelektiv to initiate the transfer...?

@broofa I think a branch would be adequate.

@broofa please email me and we will coordinate the transfer

Transferred to kelectiv.  Go with grace. :-)

npm WARN deprecated<EMAIL_ADDRESS>use uuid module instead
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>use uuid module instead


⚠ Bofrost.be has degraded performance

In 5585ebd, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1310 ms

Resolved: Bofrost.be performance has improved in 69af693 after 6 minutes.


changed folder name from Heaps to Heap. corrected all folder names. (issue #237 )

(issue #237 ) I have changed the folder name from "heaps" to "heap". I have also corrected all folder names.

Please specify the issue number in the description

@stormfireuttam Want to review this completely?

Sure.


feat: add uploaded file sharing support

Adds support for "uploading" package.json contents via the url hash.
For URLs containing a hash of the form ...#package_json=<URI-encoded content>, the app will now treat the package_json value as it would an uploaded package.json file.
Important:  Upon "upload", the package_json portion of the hash will be immediately cleared.
This PR also adds a "share" button in the top-right corner of the inspector:

Clicking this button will copy the appropriate link to the pasteboard, _including when the graph is for a locally-uploaded file.   I.e. This is how you get links with the package_json hash prefilled.
For example, here's one for an old "message_tool" project of mine:
https://npmgraph-git-uploadhash-npmgraph.vercel.app/?q=local%3Amessage_tool%401.0.0#package_json={"name"%3A"message_tool"%2C"version"%3A"1.0.0"%2C"description"%3A"Tool+for+inspecting+the+iOS+Message+app+information"%2C"main"%3A"app.js"%2C"dependencies"%3A{"autolinker"%3A"1.6.0"%2C"colors"%3A"1.1.2"%2C"escape-html"%3A"1.0.3"%2C"express"%3A"4.16.2"%2C"morgan"%3A"1.9.0"%2C"sequelize"%3A"4.22.0"%2C"sqlite3"%3A"4.1.1"%2C"uniqolor"%3A"0.9.0"}%2C"devDependencies"%3A{}%2C"scripts"%3A{"test"%3A"echo+\"Error%3A+no+test+specified\"+%26%26+exit+1"%2C"dbm"%3A"sqlite3+%24HOME%2FLibrary%2FMessages%2Fchat.db"%2C"dbc"%3A"sqlite3+%24HOME%2FLibrary%2FApplication\\+Support%2FAddressBook%2FAddressBook-v22.abcddb"}%2C"author"%3A"Robert+Kieffer"%2C"license"%3A"MIT"%2C"_local"%3Atrue%2C"maintainers"%3A[]}

Sounds great! I have a couple of ideas:

strip the JSON file to the bare minimum before sharing, so name,dependencies,devDependencies,etc
automatically include the contents in the location.href, no need to duplicate the native Share button

Also:

if you're concerned about URL size and noise, I suggest compressing the string. I usually use https://www.npmjs.com/package/lz-string

Hmm... good points.

strip the JSON file to the bare minimum before sharing, so name,dependencies,devDependencies,etc

Good idea.  Both for size and to mitigate the risk of leaking sensitive info

automatically include the contents in the location.href, no need to duplicate the native Share button

Hmm... yeah, I probably overcomplicated this.  Simply attaching the content as part of the hash

if you're concerned about URL size and noise, I suggest compressing the string

I'm going to hold off on this for now.  Compressing the string makes it more difficult for users to generate their own URLs.

generate their own URLs.

You can still support both. if !startsWith('{'), decompress()

@fregante I've reworked this so upploaded packages are now placed in the location#hash.  This feels pretty good.  The one question I have is whether or not I should be using location.search instead.  Since I'm treating hash as a parameterized field in exactly the same way search is, the two are interchangeable from an implementation standpoint. Thoughts?
One nice benefit: I was able to remove all the localStorage-related logic and UI, since the browser history now effectively acts as the storage.

I was able to remove all the localStorage-related logic and UI

That's great! I was never a fan of that