sh(1) (ash) compliant script

Hi there,
Many scripts seem to use /bin/bash, even for the simple scripts (as for scw-nbd-disconnect-root or oc-generate-ssh-keys). After a quick review, bashisms are not often used. Do you plan to modify them to make them fully ash compliant? That could allow people to not have bash in the minimal instances.
I'm ready to give a hand if you need / want it.

Hi @Ypnose, #37 should fit your needs :)

@moul Thank you for your work! :godmode:
I have two other questions: you modified the script, bash could be also removed in the Dockerfile, or I am wrong?
Do you know when the updated image will be avaible on Scaleway?

@moul Thank you for your work! :godmode:

Credits are for @muhmuhten (https://github.com/scaleway/image-tools/pull/172), I only synced its modifications on this repo

bash could be also removed in the Dockerfile

yes we can remove it from the Dockerfile, do you want to do the PR ? (You don't need to setup all the build stuff, just remove the line, it will works :p)

Do you know when the updated image will be avaible on Scaleway? :)

@QuentinPerez will release the next version of this image, it can be done on monday

I'm really busy ATM @moul. Will try to do the tiny PR this weekend.


INSTALL ERROR

Dear, I want to use tephra, however, when I installed it, there are some error message as followings. Could you help me to address them?
[root@fat01 tephra]# export TEPHRA_DIR=/opt/software/tephra_fat/tephra
[root@fat01 tephra]# perl Makefile.PL
Warning: prerequisite App::Cmd 0 not found.
Warning: prerequisite List::UtilsBy 0 not found.
Warning: prerequisite Number::Range 0 not found.
Warning: prerequisite Parallel::ForkManager 0 not found.
Warning: prerequisite Set::IntervalTree 0 not found.
Warning: prerequisite Sort::Naturally 0 not found.
Warning: prerequisite Statistics::Descriptive 0 not found.
Warning: prerequisite autodie 0 not found.
Generating a Unix-style Makefile
Writing Makefile for Tephra
Writing MYMETA.yml and MYMETA.json
=====> Configuring Tephra, this may take some time......stopwatch.c: In function ‘format_time_string’:
stopwatch.c:84:3: internal compiler error: Illegal instruction
h  = (int) (sec / 3600.);
^
vectorops.c: In function ‘DLog’:
vectorops.c:242:5: internal compiler error: Illegal instruction
else vec[x] = -DBL_MAX;
^~~~
alistat_main.c: In function ‘main’:
alistat_main.c:222:10: internal compiler error: Illegal instruction
best * 100.,  msa->sqname[bestj],
^~~~
0x9d429f crash_signal
../.././gcc/toplev.c:337
0x9d429f crash_signal
../.././gcc/toplev.c:337
0x9d429f crash_signal
../.././gcc/toplev.c:337
Please submit a full bug report,
with preprocessed source if appropriate.
Please submit a full bug report,
with preprocessed source if appropriate.
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
Please include the complete backtrace with any bug report.
Please include the complete backtrace with any bug report.
See https://gcc.gnu.org/bugs/ for instructions.
See https://gcc.gnu.org/bugs/ for instructions.
See https://gcc.gnu.org/bugs/ for instructions.
make[1]: *** [stopwatch.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: *** [vectorops.o] Error 1
make[1]: *** [alistat_main.o] Error 1
stopwatch.c: In function ‘format_time_string’:
stopwatch.c:84:3: internal compiler error: Illegal instruction
h  = (int) (sec / 3600.);
^
vectorops.c: In function ‘DLog’:
vectorops.c:242:5: internal compiler error: Illegal instruction
else vec[x] = -DBL_MAX;
^~~~
0x9d429f crash_signal
../.././gcc/toplev.c:337
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See https://gcc.gnu.org/bugs/ for instructions.
make[1]: *** [stopwatch.o] Error 1
make[1]: *** Waiting for unfinished jobs....
0x9d429f crash_signal
../.././gcc/toplev.c:337
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See https://gcc.gnu.org/bugs/ for instructions.
make[1]: *** [vectorops.o] Error 1
core_algorithms.c: In function ‘P7ViterbiTrace’:
core_algorithms.c:729:4: internal compiler error: Illegal instruction
if (Prob2Score(hmm->begin[k+1], hmm->p1) + 1 * INTSCALE <= hmm->bsc[k+1])
^~
0x9d429f crash_signal
../.././gcc/toplev.c:337
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See https://gcc.gnu.org/bugs/ for instructions.
make[1]: *** [core_algorithms.o] Error 1
make[1]: *** Waiting for unfinished jobs....
display.c: In function ‘PrintIscore’:
display.c:335:3: internal compiler error: Illegal instruction
div = INTSCALE / 0.693147180559945;   /* == INTSCALE / log(2) */
^~~
emulation.c: In function ‘WriteProfile’:
emulation.c:179:4: internal compiler error: Illegal instruction
sc = sc * 100 / INTSCALE;
^~
core_algorithms.c: In function ‘P7ViterbiTrace’:
core_algorithms.c:729:4: internal compiler error: Illegal instruction
if (Prob2Score(hmm->begin[k+1], hmm->p1) + 1 * INTSCALE <= hmm->bsc[k+1])
^~
0x9d429f crash_signal
../.././gcc/toplev.c:337
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See https://gcc.gnu.org/bugs/ for instructions.
make[1]: *** [display.o] Error 1
make[1]: *** Waiting for unfinished jobs....
0x9d429f crash_signal
../.././gcc/toplev.c:337
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See https://gcc.gnu.org/bugs/ for instructions.
0x9d429f crash_signal
../.././gcc/toplev.c:337
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See https://gcc.gnu.org/bugs/ for instructions.
make[1]: *** [emulation.o] Error 1
make[1]: *** [core_algorithms.o] Error 1
evd_test.c: In function ‘main’:
evd_test.c:109:3: internal compiler error: Illegal instruction
mu         = -20.0;
^~
0x9d429f crash_signal
../.././gcc/toplev.c:337
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See https://gcc.gnu.org/bugs/ for instructions.
make[1]: *** [evd_test.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [all] Error 2
make failed: Inappropriate ioctl for device at lib/Tephra/Config/Install.pm line 319.

Hi,
I see a couple of issues. The first issue is that the Perl dependencies are not installed. Please see the INSTALL file for advice on installing all dependencies if you want to install from source.
Also, please let me know what operating system you are using. The compile errors are odd and might indicate an older version of gcc that has not been tested.
Thanks.

Okay, it is hard to follow now because it looks like two separate issues on two different machines. So we are on the same page, please go to the Tehpra source directory (the own you downloaded) and run:
make test

And share the output. That will be more informative.

Hi, please try the latest version (v0.12.3) if this is still of interest and then update if you have any problems.
There have been many code changes since this issue. Also, it would be helpful to show the output of 'make test' if there are any issues.

I am going to close this due to no responses. If you try the latest version and find issues, please update this or open a new issue.


Getting the anchor logic should be extracted from show anchor method

Problem
Currently it's not possible to set a unique anchor for an element if a value is not set in selectorAnchor.
Solution
There should be a getAnchor method that handles the logic of whether or not an element has anchors enabled. This logic could then be removed from api.init, api.showAnchor and showAnchor

Fixed by #41


Midi js is not working properly in Typescript

Currently, Midi JS is not working in Typescript when we move our code from HTML.
We have decided to put it aside and get back to it later.
In the worst case, we will need to abandon midi js to use the BABYLON sound to play the music.

After move midi js from HTML to typescript, there is no bug in the console but we couldn’t hear the sound.

We didn't find a way to integrate MIDI JS in to Typescript. Instead, we locate a new sound player library that is compatible with TS & Colyseus.
https://github.com/danigb/soundfont-player
This issue will be close.


[SUPPORT] hoodie.datasource.write.precombine.field not working as expected

Hi team,
In one of our tables, we have Version as a Pre combine field and in the write option we have used the same and found that it's not working as expected. Whereas in some of the tables we have changed on column as pre-combine field with timestamp as data and it's working fine for those tables
When we have the precombine key as Numbers in tables, If the numbers is less than 10, it's working as expected, if it's more than that it's inserting the Version 9 into table and not considering the values greater than 9.
And i'm not sure it's a bug or i'm using incorrect option while writing.
sample dataframe value
+-------+-------+----------+
|   NAME|VERSION|CHANGED_BY|
+-------+-------+----------+
|T009S50|      3|   USER001|
|T009S50|      2|   USER002|
|T009S50|      1|   USER002|
|T009S50|      5|   USER001|
|T009S50|      4|   USER001|
|T009S50|      6|   USER002|
|T009S50|      7|   USER002|
|T009S50|      8|   USER001|
|T009S50|      9|   USER001|
|T009S50|     10|   USER003|
+-------+-------+----------+
Write Options used
input_df.write.format("org.apache.hudi").option("hoodie.datasource.write.recordkey.field", "NAME).option("hoodie.datasource.write.precombine.field","VERSION").option("hoodie.table.name","TABLE1").option("hoodie.datasource.write.storage.type","MERGE_ON_READ").option("hoodie.datasource.hive_sync.enable","true").option("hoodie.datasource.hive_sync.table","TABLE1").option("hoodie.datasource.hive_sync.assume_date_partitioning","false").option("hoodie.datasource.hive_sync.partition_extractor_class","org.apache.hudi.hive.NonPartitionedExtractor").mode("Overwrite").save(s3:\mybucket\tablepath)
Inserted Record From Hive
T009S50  9       USER001
I have tested the above scenario for few of the datasets
Environment Details
emr-6.0.0
Hudi Version - 0.5.0

is VERSION a STRING or LONG data type? if string then alpha sort would say 9 > 10 or 9 > 12..etc

yeah.. it's string.. the dataframe is created directly on-top of s3 files

so no issue? you would need to case the df to a schema

yeah.. will take a look on the casting for the precombine key column and let you know for any concerns. Thanks @tooptoop4

THanks @tooptoop4 for the help. @rajgowtham24 : Please reopen if there is any other issue.

Hi @tooptoop4 ,
While reading the csv file i have used inferschema option as mentioned below
input_df = spark.read.format("csv").option("header","true").option("inferschema","true").load("mybucket/sample.csv")
And upon reading the file, i have validated the datatypes using input_df.dtypes and i can confirm that the Version Column is in INT type
[('NAME', 'string'), ('VERSION', 'int'), ('CHANGED_BY', 'string')]
After writing this into Hudi table, i can still see the Version 9 is getting inserted, Whereas the Hive Table metadata is now synced with the datatypes similar to the input_df.dtypes
Let me know for any addition details.

@rajgowtham24 : This is a known in 0.5.x and was fixed in 0.6.0 version

Thanks @bvaradar  will check on that


Add timestamps to every DB table

It's always useful to know when a row was created or updated, but we forgot to add these columns.
Existing records are backfilled with the current date and time. In practice, as we aren't live yet, this doesn't matter, but it allows for painless migration in development.

:+1: I wonder if it's worth adding some sort of audit trail for updates, too? Probably something for post-launch, though…


Math code crashes without a useful error message when variable isn't resolved

Example:
proto my_proto {
  parent = $PARENT
  var = "${PARENT}.$VAR"  # An example of using variables in a string
  center = {{ 0.5 * ($MAX - ${MIN}) }}
}

reference my_proto as concrete {
  $PARENT = $PARENT_NAME
  $VAR = "fuzz"
}

Because $MIN and $MAX aren't defined, this leads to a segfault. Instead, a useful exception should probably be thrown to make debugging easier.

The code fails when trying to evaluate the mathematical expression in stack::evalBack() due to the assert(vs_.size() == ops_.size() + 1)
I think the right solution is to check within replaceProtoVar that an expression (or any type) has no more variables once all proto vars have been resolved.


Chore[ingest]: adding parameter --partition-pdf-infer-table-structure

Summary

adding --partition-pdf-infer-table-structure to unstructured-ingest

Test

check there are field "text_as_html" for extracted tables in output json file

Note
all partition parameters are not passing into request correctly when use together with partition-by-api, so currently we have to stick with local partition (will be fixed soon).

Does this need to pass the --partition-pdf-infer-table-structure flag's value along to partition_via_api too? Did I miss the part in the code that does it?

yep we want to pass all partition kwargs to partition_via_api, but sadly its not working, and its using the local partition from /partition/auto.py
added a issue to fix the partition_via_api: https://github.com/Unstructured-IO/unstructured/issues/1064


contradiction in "Rights to your contributions"

These 2 points contradict each other.
1st states that you must have authored 100% and 2nd explain that it's not necessary to author 100%.
I suggest this case is raised as a good first-time issue for some friendly technical writer to fix.
Agree that you have authored 100% of the content
...
Agree that, if you did not author 100% of the content, the appropriate licenses and copyrights have been added along with any other necessary attribution.
File: https://github.com/Green-Software-Foundation/Carbon_CI_Pipeline_Tooling/blob/Dev/CONTRIBUTING.md
Screenshot:

Ah, true. Looks like the original contributing file hasn't been changed since @Avanade first donated the code.
I've updated it in Avanade's contributing file, but good flag for this project.

Thank you @nikitakoselev - have updated this in the dev branch to remove the contradiction.


Rerun failed tests in Patrol

Use case
The ability to rerun failed tests in patrol would be useful for integration testing.
Proposal
I don't have a proposal. I just suggest the ability to do it.

If you want to (1) run tests, then (2) update them, and (3) re-run, then it's non-trivial/impossible to do. For the closest thing, see patrol develop.
If you mean to just run the same tests a few times on the device, you can pass --no-uninstall flag to patrol test, and then just run it manually with adb/some obsucre xcrun` command. There's a feature request by me (#1982) to make this flow much easier, but it's not done.

@bartekpacia thanks for your reply.
It is the second point. I have an automation suite of tests that I want to run before each prod deployment.
Sometimes they fail but because the emulator in BS(BrowserStack) is flaky or an interruption occurs on BS.
So I just want to rerun the ones that didn't complete or failed or were not run.
In order to run them in BS I need to upload an apk file of the app and an apk file of the test suite.
Then BS runs the test suite against the apk file via an emulator.
Since the test suite is built already I wasn't sure how I could rerun only the tests that didn't run or failed.

It would be really useful to implement in the patrol test command an argument like --rerun_failed=n, where n would be the times to rerun a test on failure. As of today, there is no way to control/customize patrol test execution in CI/CD pipelines, we have to do it all manually.


Captcha Loop

I get into a captcha loop whenever I try to download something, please fix this.

Which website are you using?

kimcartoon for me


Installing older node on mac M1

i am trying to install npm 10.15.1 on my mac M1 with the rosetta enabled, but it seems like NVM keeps detecting the Silicon CPU despite having changed the arch to -x86_64

did i miss a step somewhere?

I’m not sure; you didn’t fill out the issue template. Please do so, it’s there for a reason.


File array issue

https://stackoverflow.com/questions/2720014/upgrading-all-packages-with-pip

https://i.imgur.com/TKe4eKe.png


doc: change go get to multi-line code block for more convenient copy

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

Comparison is base (406092b) 92.85% compared to head (e9f76e8) 92.85%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   92.85%   92.85%
=======================================
  Files           6        6
  Lines         210      210
=======================================
  Hits          195      195
  Misses         11       11
  Partials        4        4

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


Add vbs readme

Hi @solarsbeans, Thanks for the PR. But I don't think VisualBasic would be suitable for this list as it's neither done with web tech nor a cross-platform solution.

Hi @solarsbeans, Thanks for the PR. But I don't think VisualBasic would be suitable for this list as it's neither done with web tech nor a cross-platform solution.

thank you for comment!


Add initial pricing model and a http route to query it

Pricing information is gathered by using existing parts of Tenant::gc_iteration which are now separated as Tenant::refresh_gc_info. Tenant::refresh_gc_info collects branch points, and invokes Timeline::update_gc_info; nothing was supposed to be changed there. The gathered branch points (through Timeline's GcInfo::retain_lsns), GcInfo::horizon_cutoff, and GcInfo::pitr_cutoff are used to build up a Vec of updates fed into the libs/pricing_model to calculate the history size.
The gathered information is now exposed using GET /v1/tenant/{tenant_id}/history_size, which which will respond with the actual calculated size. Initially the idea was to have this delivered as tenant background task and exported via metric, but it might be too computationally expensive to run it periodically as we don't yet know if the returned are any good.
Adds one new metric:

pageserver_storage_operations_seconds with label logical_size

separating from original init_logical_size
see 5f248a7

Adds a pageserver wide configuration variable:

concurrent_pricing_logical_size_queries with default 1

TODOs:

[ ] pricing model inputs: they possibly need to only be gathered up to gc_cutoff_lsn
[x] pageserver configurable semaphore permits for concurrent logical_sizes

Decided to drop the periodic task and metric and only do http request support for now.

Re: 1/2 of tasks done: I am still fuzzy on the actual pricing model inputs: following off-github conversations, currently they are calculated by taking all of the branch points and then an additional state for next_gc_cutoff_lsn, should the gc_cutoff_lsn be after the branch point.
Alternative one would be to include the branch points only if the gc_cutoff_lsn comes after the branch point, leaving out all branches happening after it.

"Pricing model" is not a great name and this is one of the confusion points. Alternatives:

history_size
retained_history_size
ideal_history_size

Just logging progress in case you happen yo review right now. Creating a simple test reveals this doesn't currently work with empty database.

What a chance to win two flaky tests in both regression tests. Trying to fix that test.

Had to do another rebasing, added tests consume free ports.


fix broken test-dist

Background
npm run test-dist is down

Change List

run test against deck.gl es6 build

drop this PR cause team has one big refactor ongoing and it will resolve this issue.


Use get_db_prep_value when getting instance attribute value in get_pr…

…efetch_queryset
This resolves a problem for non-PostgreSQL databases when using
prefetch_related with a source model that uses a UUID primary key.

hmm .. will try to fix isort and flake8 complaints.

buildbot, add to whitelist.

I'm not sure what to do here ... the gis_tests.tests complete successfully when I run them locally.

Not your problem. That failure was present on master when the tests ran.

merged in bfb5b7150ff52571a7a3cf10e0dd0d1dbd45c4b5, thanks!


Collections Kahoot

https://create.kahoot.it/share/ds-khoot-06-python-collections/8d4560d9-4b62-4286-b50d-e50c0c4c042b


Allow boot from volume

For bigger deployments it would be nice to have an option to use "boot from volume" nova option when creating instances instead of passing an image.

I got the "boot from volume" templates. Any idea about using conditionals for unifying the templates? @jprovaznik

@jprovaznik this feature could create issues when replacing nodes, and should be addressed in some way.
heat can't manage boot-volumes when replacing host

create an host with a boot volume using the following template https://pastebin.com/quB7Ky2j
mark the host unhealthy
update the stack

results in
UPDATE_FAILED  Resource CREATE failed: BadRequest: resources.host: Invalid volume: volume 'xxx' status must be 'available'. Currently in 'in-use' (HTTP 400) (Request-ID: req-yyy)

@ioggstream I wonder whether it's a feature or a race issue on heat side - I would expect that replacing node means deleting the node first (which should make volume available again) and then creating a new node.

@ioggstream I don't think that there is much active development on this project so I wouldn't expect this (or other features) to be done though.

@jprovaznik I wrote here because I didn't find a relevant issue on trello. It's fine to continue the discussion there and forward port the stuff on the new openshift-ansible (I already made 2 PR).
afaik heat workflow is the following:
1- create new server and ensure it's online
2- remove the old one
I'm on irc  #heat looking for insights now.

@ioggstream I'm not on this project for a while, but with ceph-ansible I suppose you can define customized playbook for replacing a node (which does exactly what you want - so first deletes the node, then creates a new one) if it turns out that heat creates a new one before deleting the old one.

@jprovaznik can you point me to somebody interested to that work? I'd really like that any effort ccould be useful to a future solution in openshift-ansible. Thx++

@ioggstream this is a really old RFE which came from testing scalability of these templates. IIRC the problem was that by default nova instance boots from image, which means that this image is copied from glance to a nova compute node where the instance is being started - if you boot hundreds of nodes at the same time, it will be bottleneck. You can ping @jeremyeder from the scalability team or openshift-ansible active developers, I think @tomassedovic works on openstack integration so he might navigate you to the right people.


43_tabular.learner: create docments

Add docments to 43_tabular.learner.
See the style guide and contribution guide for more details (links will be added when created).
Decent option for a new user that has went through the tabular tutorial
Docments progress:

[ ] TabularLearner
[ ] tabular_learner

Hi, I would like to work on these tasks :)


Copa will need to write metadata information about which packages got patched.

Investigated this with update all and scanner patches:

For debian non-distroless images, the status file is properly updated with new versions of packages when comparing locally and using crane.

For debian distroless, the package files in the status.d folder are properly upgraded with new versions when checking locally. When inspecting through crane however, the changes in the same image are not reflected. This could be an issue with crane?

For rpm non-distroless, I could not find a file that would need to be updated to reflect package version changes.

For rpm distroless, the container-manifest1/container-manifest2 files are not upgraded with new versions when inspecting with crane. Unable to run the images locally to test.

For apk non-distroless, new version changes are not reflected when inspecting with crane. Unable to run the images locally to test. I believe the /lib/apk/db/installed file needs to be updated.

This metadata information was consistent in update all and scanner patches. There should probably be a follow up fix to this since this is not specific to the update all feature, and we need to add code to rpm and apk that updates the metadata.


Prevent adding tests to extensions to generic types.

When a test is added to a type extension, we can't see any information about the type being extended, so we cannot report invalid lexical contexts such as protocols or non-final classes.
One of the invalid cases is a generic type, because swift-testing does not know how to realize the type—what generic arguments is it supposed to pass?
This PR causes the compiler to emit a diagnostic when a generic type contains a test or suite. How does it work?, you ask? Generic types do not support static stored properties, while non-static types do support them. So we just add a static stored property (of type Void, because the value is unimportant.) In a non-generic type, it will "just work." In a generic type, a diagnostic is emitted:

🛑 Static stored properties not supported in generic types

The diagnostic is non-optimal (it's not fine-tuned for swift-testing) but it prevents building an invalid test that can't be resolved at runtime, so it's better than nothing.
This change works with swift-syntax-600 and swift-syntax-510.
Checklist:

[ ] Code and documentation should follow the style of the Style Guide.
[ ] If public symbols are renamed or modified, DocC references should be updated.

@swift-ci please test


Missing specs/documentation for Standard_M832ixs_v2 / Standard_M416s_8_v2

After the release of instance Standard_M832ixs_v2 9 months ago there are still no information available about availability in the regions, detailed specs or even prices inside the price calculator.
If you search for "M832ixs_v2" you won't find a single document. Even Standard_M416s_8_v2 which was released after Standard_M832ixs_v2 is better documented, but still details are missing in the documentation.
Please add the specs and details:
https://learn.microsoft.com/en-us/azure/virtual-machines/mv2-series
https://learn.microsoft.com/en-us/azure/virtual-machines/dedicated-host-memory-optimized-skus
https://learn.microsoft.com/en-us/azure/virtual-machines/how-to-enable-write-accelerator
https://learn.microsoft.com/en-us/azure/virtual-machines/linux/compute-benchmark-scores
Please add details regarding availability and costs in the price calculator for transparency.
I look forward to better quality management for new VM instances in the future.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 7a4a150d-1617-96a4-c460-3190ed8598b4
Version Independent ID: fd8dd71a-84b6-11f9-077f-80ea27980756
Content: Mv2-series - Azure Virtual Machines
Content Source: articles/virtual-machines/mv2-series.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @lauradolan
Microsoft Alias: ayshak

@gleicje
Thanks for your feedback! We will investigate and update as appropriate.

@lauradolan
Can you please check and add your comments on this doc update request as applicable.

@gleicje
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

Seems to be no update after one month - just wanted to add that the latest instance should also be added: M832ixs

Hope this issue is resolved? Please let us know if there is anything to assist with.
#please-close


[GDD] Fix text color in search bar

Mozilla Firefox 115.0b9 (64-bit)
Stylus 1.5.33 (June 11, 2023)
Microsoft Windows 10.0.19045.3086

This should be fixed with 66dfdd6 (v0.2.16). Please feel free to open another issue if you're still seeing those issues!


无法下载文件，多次出现Connection failed with request

Connection failed with request[{If-Match=[W/"16d5498b3582d41:0"], Range=[bytes=0-], User-Agent=[FileDownloader/1.7.5]}] response[{null=[HTTP/1.1 412 Precondition Failed], Connection=[close], Content-Length=[331], Content-Type=[text/html; charset=us-ascii], Date=[Thu, 22 Nov 2018 07:33:45 GMT], Server=[Microsoft-HTTPAPI/2.0], X-Android-Received-Millis=[1542871748104], X-Android-Response-Source=[NETWORK 412], X-Android-Selected-Protocol=[http/1.1], X-Android-Sent-Millis=[1542871748100]}] http-state[412] on task[-403978202--1], which is changed after verify connection, so please try again.

Duplicate with #1140


Nopause

Added most of the noPause suggestions from html5gamedevs.
This PR changes

The public-facing API

Users will be able to add noPause = true to DisplayObjects that they want to respond to events even when game.paused === true. For example, if I have a pause menu and want the buttons to still respond to mouse over and click events, I add .noPause = true to the button object.
Users will also have access to timeStep in a State's pauseUpdate function.
Minor fix was also added to get jshint task to succeed.

Nice, I'll merge this in, but could I suggest a PR that updates the docs as well to reflect this change?


No way to add a port to Server Settings

Not sure If I am missing something here, but if you're trying to join a server that's not using port 443 it seems you always get a Server IP error. I am using the : to signify a different port.

Are you sure the server is running on another port? The default is 443, you can modify the server config to run on other ports.

@Excadere I just did a test, the server side is modified to port 442, the client access through port 442 successfully, please confirm whether it is a firewall problem.

I see, so it wasn't a firewall problem. The server I set up is using a subdomain name rather than a regular IP. So it seems SubDomainName:port does not work the same as IP:port. I guess that's just a lack of trying on my side.

Can your game client connect to the GC server properly? You can try turning on or off the switch for the system proxy on the first page.

My game can connect to the server, and if I use IP instead of subdomain name grasscutter tools can connect as well. The only issue I'm seeing now is every time I try to input the code sent to the mailbox I get network error. But that may be a separate issue.

@Excadere I have no problem using subdomains now. I guess that there is a problem with your domain name resolution or server config.

If in doubt open again.


Add implementation to support ingesting a TOML file and setting configurations based on the ingested file

This draft PR aims to support ingesting a TOML file and setting configurations based on the ingested file.
A sample TOML file, configuration.toml, is attached, and necessary CMake instructions (to integrate toml++ into the build) are written into CMakeLists.txt.

Can you rebase? I made some changes to the naming scheme which unfortunately conflict with the PR.

Can you rebase? I made some changes to the naming scheme which unfortunately conflict with the PR.

I know this may be a naive question, but may I do a merge? When I attempted to do a rebase, it seems that I need to start doing rebases progressively from the very first commit on this branch, which looks/sounds daunting.

Can you rebase? I made some changes to the naming scheme which unfortunately conflict with the PR.

I know this may be a naive question, but may I do a merge? When I attempted to do a rebase, it seems that I need to start doing rebases progressively from the very first commit on this branch, which looks/sounds daunting.

Sure, that's fine. Personally, I prefer squash+merge plus rebase because merges create a lot of noise.

Can you rebase? I made some changes to the naming scheme which unfortunately conflict with the PR.

So far, all the conflicts have been resolved with the added functionalities. There are still some comments on the previous codebase that are inconsistent with the changes to the naming scheme. Should I fix them in this PR? Or should we create a new PR that is dedicated to making those changes (to the comments)?

Should I fix them in this PR?

No, that should be a separate PR. Or are those fixes related to your code?

Should I fix them in this PR?

No, that should be a separate PR. Or are those fixes related to your code?

No. I don't think so.

Should I fix them in this PR?

No, that should be a separate PR. Or are those fixes related to your code?

No. I iterated over all the changes I've made (using Ctrl-F and hitting return/enter), and I found none.

Should I fix them in this PR?

No, that should be a separate PR. Or are those fixes related to your code?

No. I iterated over all the changes I've made (using Ctrl-F and hitting return/enter), and I found none.

Issues should be addressed, hopefully. I forgot to synchronize the branch to the A branch before requesting a review. Sorry.

Alright, looks good to me. Let's get this merged. I will likely do a little bit of clean up but this is already a very comprehensive PR. Good job!

Thanks! Should I clean up the (paths of) std::nullopt within this PR? Or should I leave it and deal with it in another PR (which I will do perhaps later since I don't have the local machine with me right now)?

Alright, looks good to me. Let's get this merged. I will likely do a little bit of clean up but this is already a very comprehensive PR. Good job!

Thanks! Should I clean up the (paths of) std::nullopt within this PR (which I would do perhaps later since I don't have the local machine with me right now)? Or should I leave it and deal with it in another PR?

Update: I've resolved the concern(s) in the latest commits and will correspondingly squash and merge the commits in this PR to the master branch.

Update: I've resolved the concern(s) in the latest commits and will correspondingly squash and merge the commits in this PR to the master branch.

I have resolved all the issues and am going to squash and merge (commits in) this PR to the master branch.


Use broccoli-asset-rev asset map for manifest

broccoli-asset-rev works by renaming files inside fastboot-dist so that the file name contains an MD5 hash of their contents. Unfortunately, it is hardcoded to rewrite only specific files like index.html and therefore broke the FastBoot manifest in fastboot-dist/package.json.
This commit modifies FastBoot to run after broccoli-asset-rev, and use a generated asset map to update the FastBoot manifest to point to the fingerprinted assets.
Fixes #102

@jpadilla @danmcclain Let me know if this solves your issue.

@tomdale I can test tomorrow

@tomdale Sorry for the delay, but this did solve the issue WRT prod builds :+1:


Build with latest VS2017 15.7.1

Special fixes for VS2017 should increase _MSC_FULL_VER check which is now
#if defined(_MSC_FULL_VER) && _MSC_FULL_VER <= 191426428 && _MSC_FULL_VER >= 191200000

This is fixed now; thanks for reporting!


Merge VS For Mac support back to master

@mrward has put together a VS for Mac add-in to support Libman: https://github.com/mrward/LibraryManager/tree/monodevelop
We should integrate this back up to the trunk, but it will require some refactoring in the existing Vsix project to make IDE support more flexible to account for VS/VSfM differences.

With the announcement that VS for Mac will be deprecated in August 2024, this request is no longer on the roadmap.


Bug 1898500: Support InstallPlan steps upgrading existing ClusterIP Services.

If a ClusterIP-type Service manifest omits the immutable spec field
"ClusterIP", a value is selected automatically when the Service is
created. The catalog operator can patch existing Service resources if
the updated manifest either omits "ClusterIP" or sets "ClusterIP" to
the same value as that of the existing Service.

/lgtm

/cherry-pick release-4.6

/cherry-pick release-4.6


stacked and grouped to create waterfall

Can you currently create multiple series and have one series grouped ... so I can create a waterfall chart.  The Stacking comes from make this send bar in each year have an opacity 0 segment to make it rise above the zero line.    I considered a candlestick chart but that chart does not allow multiple series (3 bars per year) ...

Sorry, that's not possible currently.

You may use the range column charts to create something similar to what you've described.
https://apexcharts.com/javascript-chart-demos/column-charts/range-column-chart/


fix(events): Prefix emitted events with service invocation route

Each emitted event payload needs be prefixed with service name/invocation route. Event trigger resolves the latter via a map which is maintained by dispatching code which attributes id of being processed message with the route. The approach requires at least 2 calls to gstd::msg::id() which is not that cheap. Current there is no option to cache it somehow.

@osipov-mit FYI. It might affect your code where you parse event payload


[Event Request] - Codeunit CRM Sales Order to Sales Order - OnBeforeSetLineDescription

Can you create this event?

Please always put the full detailed signature of the requested events and the parameters passed in the call.
You help us implementing the requests faster when your request contains AL code copied from VSCode instead of screenshots.

Hi Oleg,
My bad. Let me try again.
local procedure SetLineDescription(SalesHeader: Record "Sales Header"; var SalesLine: Record "Sales Line"; var CRMSalesorderdetail: Record "CRM Salesorderdetail")
var
SalesReceivablesSetup: Record "Sales & Receivables Setup";
LineDescriptionInStream: InStream;
CRMSalesOrderLineDescription: Text;
ExtendedDescription: Text;
IsHandled: Boolean;
begin
//Event
OnBeforeSetLineDescription(SalesLine, CRMSalesorderdetail, IsHandled);
if IsHandled then
exit;
//EVent
CRMSalesorderdetail.CalcFields(Description);
CRMSalesorderdetail.Description.CreateInStream(LineDescriptionInStream, TEXTENCODING::UTF16);
LineDescriptionInStream.ReadText(CRMSalesOrderLineDescription);
    if CRMSalesOrderLineDescription = '' then
        CRMSalesOrderLineDescription := CRMSalesorderdetail.ProductDescription;

    ExtendedDescription := CRMSalesOrderLineDescription;

    // in case of write-in product - write the description directly in the main sales line description
    if SalesReceivablesSetup.Get() then
        if SalesLine."No." = SalesReceivablesSetup."Write-in Product No." then begin
            SalesLine.Description := CopyStr(CRMSalesOrderLineDescription, 1, MaxStrLen(SalesLine.Description));
            if StrLen(CRMSalesOrderLineDescription) > MaxStrLen(SalesLine.Description) then
                ExtendedDescription := CopyStr(CRMSalesOrderLineDescription, MaxStrLen(SalesLine.Description) + 1)
            else
                ExtendedDescription := '';
        end;

    if ExtendedDescription = SalesLine.Description then
        exit;

    // in case of inventory item - write the item name in the main line and create extended lines with the extended description
    CreateExtendedDescriptionOrderLines(SalesHeader, ExtendedDescription, SalesLine."Line No.");

    // in case of line descriptions with multple lines, add all lines of the line descirption
    while not LineDescriptionInStream.EOS() do begin
        LineDescriptionInStream.ReadText(ExtendedDescription);
        CreateExtendedDescriptionOrderLines(SalesHeader, ExtendedDescription, SalesLine."Line No.");
    end;
end;

// ----- Event --------//
[IntegrationEvent(false, false)]
local procedure OnBeforeSetLineDescription(var SalesLine: Record "Sales Line"; var CRMSalesOrderDetail: Record "CRM Salesorderdetail"; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Suggestion - pull in marionette.js list (front-end extension for backbone)

Link: https://github.com/instanceofpro/awesome-marionette

:+1:

I personally like them being separate.


Chapter 4 : Using the "squared" parameter of Scikit-learn mean_squared_error function.

Dear Aurélien,
First of all, I would like to thank you for your outstanding contribution in the field of machine learning and AI.
Your work is invaluable to those who work in this field and to those who aspire to become machine learning practitioners.
Here is my little comment, you could make use of the "squared" parameter of Scikit-learn mean_squared_error function in the basic implementation of early stopping thus avoiding the np.sqrt(...) computation.
Here below your code that creates the graph where I set the squared parameter to False so that to get directly the RMSE value. By the way  I use Scikit-learn 0.24.0.
Thanks again, BR Reda
sgd_reg = SGDRegressor(max_iter=1, tol=-np.infty, warm_start=True,
                       penalty=None, learning_rate="constant", eta0=0.0005, random_state=42)

n_epochs = 500
train_errors, val_errors = [], []
for epoch in range(n_epochs):
    sgd_reg.fit(X_train_poly_scaled, y_train)
    y_train_predict = sgd_reg.predict(X_train_poly_scaled)
    y_val_predict = sgd_reg.predict(X_val_poly_scaled)
    train_errors.append(mean_squared_error(y_train, y_train_predict,squared=False))
    val_errors.append(mean_squared_error(y_val, y_val_predict,squared=False))

best_epoch = np.argmin(val_errors)
best_val_rmse = val_errors[best_epoch]

plt.figure(figsize=(7,5))

plt.annotate('Best model',
             xy=(best_epoch, best_val_rmse),
             xytext=(best_epoch, best_val_rmse + 1),
             ha="center",
             arrowprops=dict(facecolor='black', shrink=0.05),
             fontsize=16,
            )

best_val_rmse -= 0.03  # just to make the graph look better

plt.plot([0, n_epochs], [best_val_rmse, best_val_rmse], "k:", linewidth=2)
plt.plot(val_errors, "b-", linewidth=3, label="Validation set")
plt.plot(train_errors, "r--", linewidth=2, label="Training set")
plt.legend(loc="upper right", fontsize=14)
plt.xlabel("Epoch", fontsize=14)
plt.ylabel("RMSE", fontsize=14)
save_fig("early_stopping_plot")
plt.show()

Thanks a lot for your kind words, Reda! 👍
And thanks for your suggestion, I think I'll add a comment in the notebook about squared=False.
Cheers,
Aurélien

Thank you Aurélien !


Can't install rustup because of existing rust that I can't remove

We have a copy of rustc version 1.14.0 installed on my school's server. I want to set up the latest version of rust (currently 1.15.1) in my home directory, ideally using rustup. But because of the existing copy of rust - which I can't remove, since I don't have root access - rustup-init fails with the following error:
warning: it looks like you have an existing installation of Rust
warning: rustup cannot be installed alongside Rust. Please uninstall first
warning: run `/usr/local/lib/rustlib/uninstall.sh` as root to uninstall Rust
error: cannot install while Rust is installed

I wonder why can't rustup have a special system toolchan, that cannot be installed or updated but can be chosen.

I'm not sure the best way to fix this offhand. I think the reason I did this was to reduce potential confusion after install, having seen cases where people were still invoking the old rustc afterwards. For example, there are many cases where rustup fails to configure PATH, and that would leave the old rustc active.
We could possibly loosen this to a warning, like "warning: There is an existing rustc installed to $path. Depending on how $PATH is configured, the existing toolchain may take precedence."
Could also make it interactive, but that's deeper surgery.
A 'system' toolchain is a useful feature to add, but does not solve the problem of users being confused about which toolchain they are using.

Temporary workaroud (let's call it - force install) for this, is to download rustup.rs source code and comment sanity checks in do_pre_install_sanity_checks() (or apply patch)
diff --git a/src/rustup-cli/self_update.rs b/src/rustup-cli/self_update.rs
index 7d1a641..c289962 100644
--- a/src/rustup-cli/self_update.rs
+++ b/src/rustup-cli/self_update.rs
@@ -381,7 +381,7 @@ fn do_pre_install_sanity_checks() -> Result<()> {
         warn!("it looks like you have an existing installation of Rust");
         warn!("rustup cannot be installed alongside Rust. Please uninstall first");
         warn!("run `{}` as root to uninstall Rust", uninstaller_path.display());
-        return Err("cannot install while Rust is installed".into());
+        // return Err("cannot install while Rust is installed".into());
     }

     if rustup_sh_exists {

Then compile (cargo build) and then proceed with installation, e.g. RUSTUP_HOME=/home/user/.multirust CARGO_HOME=/home/user/.cargo ./target/debug/rustup-init --no-modify-path.

The copy of rustc that we had on my school's server has since been removed, so I was able to install rustup just fine when I tried it last week. That being said, there should be some way to force installation, even if there is an existing Rust installation somewhere.

You can bypass the check by setting RUSTUP_INIT_SKIP_PATH_CHECK to yes:
$ ./rustup-init
error: it looks like you have an existing installation of Rust at:
error: /usr/bin
error: rustup cannot be installed alongside Rust. Please uninstall first
error: if this is what you want, restart the installation with `-y'
error: cannot install while Rust is installed
$ RUSTUP_INIT_SKIP_PATH_CHECK=yes ./rustup-init

Welcome to Rust!

This will download and install the official compiler for the Rust programming
language, and its package manager, Cargo.
# ...

I think the error message should at least mention this environment variable -- I only found it after looking through the source of rustup-init.

I meet the some question too, and I use
$RUSTUP_INIT_SKIP_PATH_CHECK=yes sh ./rustup-init.sh

to reinstall rust, It's work.
Thanks @mgeisler

Sorry for the late reply.
You just need to run:
curl -sSf https://static.rust-lang.org/rustup.sh | sh
and then Build Cargo

@anjalianjali1234 This no longer works:
$ curl -sSf https://static.rust-lang.org/rustup.sh | sh
The location of rustup.sh has moved.
Run the following command to install from the new location:
    curl https://sh.rustup.rs -sSf | sh

Not sure what has happened, using -y or RUSTUP_INIT_SKIP_PATH_CHECK to stop checking when installing rustup did not work for me:
./rustup-init --version
rustup-init 1.16.0 (beab5ac2b 2018-12-06)

RUSTUP_INIT_SKIP_PATH_CHECK=yes ./rustup-init
warning: it looks like you have an existing installation of Rust
warning: rustup cannot be installed alongside Rust. Please uninstall first
warning: run `/usr/local/lib/rustlib/uninstall.sh` as root to uninstall Rust
error: cannot install while Rust is installed

./rustup-init -y./rustup-init -y
warning: it looks like you have an existing installation of Rust
warning: rustup cannot be installed alongside Rust. Please uninstall first
warning: run `/usr/local/lib/rustlib/uninstall.sh` as root to uninstall Rust
error: cannot install while Rust is installed

warning: it looks like you have an existing installation of Rust
warning: rustup cannot be installed alongside Rust. Please uninstall first
warning: run `/usr/local/lib/rustlib/uninstall.sh` as root to uninstall Rust
error: cannot install while Rust is installed

Is it possible to install rustup without un-installing?

https://doc.rust-lang.org/1.0.0/book/installing-rust.html Please follow this link.
…
On Thu, Jan 17, 2019 at 4:25 AM Michael Hewson @.***> wrote: I don't know. Probably one of the admins removed it. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#953 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AgKDbJJMUwF5vmUhzS8ynn6RbEw0HmmZks5vD63xgaJpZM4L-YjL .

tks!

thank you very much !

After compiling all above comments, I used following command and  it worked perfectly with previously installed rust.
curl https://sh.rustup.rs -sSf | RUSTUP_INIT_SKIP_PATH_CHECK=yes sh

@ssowellsvt - Spot on - Thanks!

Yes, use
export RUSTUP_INIT_SKIP_PATH_CHECK=yes
curl https://sh.rustup.rs -sSf | sh
source $HOME/.cargo/env

Just run:
sudo apt remove rustc
then
curl https://sh.rustup.rs/ -sSf | sh

This worked for me

On OSX, I had to manually go into /usr/local/bin and uninstall all the rust* binaries and the cargo binary. For some reason rustup self uninstall was not removing these.

@m-cat rustup-init would never have installed binaries into /usr/local/bin unless you did something very odd, so that's expected.

On OSX, I had to manually go into /usr/local/bin and uninstall all the rust* binaries and the cargo binary. For some reason rustup self uninstall was not removing these.

Same thing happened to me (OSX Big Sur, Intel Mac) after updating to either 1.56 or 1.57. Somehow I had the binaries also in /usr/local/bin and this messed up cross-compilation, i.e. not recognising installed targets like wasm32-unknown-unknown.
after running rustup self uninstall I removed the rust* and cargo bin from /usr/local/bin, and was then able to reinstall Rust. This also solved my cross-compilation problems.

If an existing rust installation is left there, is there a way to force rustup-init to always setup the PATH so that its installed version takes precedence?


[delete/ freme-ner/datasets] response body not displayed

When a dataset is deleted with DELETE e-entity/freme-ner/datasets/{name} via postman, the following line is written to the body .
 The DatasetMetadata: NAME was removed sucessfully.'
But in the documentation, it always prints 'no content' in the response body field...

This is ok. The http status response code is 204 and i think that in this case no content should be send. so the bug is not in the documentation but in the api that sends content. but it is ok we do not need to change anything.


concat development

UUID for filenames.
Either alter SIPcreator to accept files, or build a sipcreator type workflow for the concatenated file.
logfile needed to record events
Check if P2 can use the same script, XDCAM almost certainly can.

This has been created - P2 is less of a priority right now.


Here's an example.  Ignore the ticks, but any geometry below 0 latitude should not be shown.  This PR will ensure that.
Consider the following:

fig = Figure(size=(1500, 1500))

settings = "+proj=leac +lat_1=45.0 +ellps=sphere "

ga = GeoAxis(
	fig[1, 1];
	dest=settings,
	title=settings,
)
lines!(ga, GeoMakie.coastlines())
ylims!(ga, 0, 90)
fig

This is priority after #261, #245 and #260


Testing Issue #83, DO NOT MERGE

Testing a file with one missing clade for Issue #83

The file with a missing clade passes all checks, like @nickreich suggested there must be a bug someplace.

hubValidations 0.7.0 has been released. I reran the check and it correctly fails now:
── Individual check results ──

── variant-nowcast-hub ────

✔ [valid_config]: All hub config files are valid.

── 2024-10-02-UMass-HMLR.parquet ────

✔ [file_exists]: File exists at path
  'model-output/UMass-HMLR/2024-10-02-UMass-HMLR.parquet'.
✔ [file_name]: File name "2024-10-02-UMass-HMLR.parquet" is valid.
✔ [file_location]: File directory name matches `model_id` metadata in file
  name.
✔ [round_id_valid]: `round_id` is valid.
✔ [file_format]: File is accepted hub format.
✔ [metadata_exists]: Metadata file exists at path
  'model-metadata/UMass-HMLR.yml'.
✔ [file_read]: File could be read successfully.
✔ [valid_round_id_col]: `round_id_col` name is valid.
✔ [unique_round_id]: `round_id` column "nowcast_date" contains a single, unique
  round ID value.
✔ [match_round_id]: All `round_id_col` "nowcast_date" values match submission
  `round_id` from file name.
✔ [colnames]: Column names are consistent with expected round task IDs and std
  column names.
✔ [col_types]: Column data types match hub schema.
✔ [valid_vals]: `tbl` contains valid values/value combinations.
✔ [rows_unique]: All combinations of task ID
  column/`output_type`/`output_type_id` values are unique.
✖ [req_vals]: Required task ID/output type/output type ID combinations missing.
  See `missing` attribute for details.
✔ [value_col_valid]: Values in column `value` all valid with respect to
  modeling task config.
ℹ [value_col_non_desc]: No quantile or cdf output types to check for
  non-descending values. Check skipped.
ℹ [value_col_sum1]: No pmf output types to check for sum of 1. Check skipped.
✔ [spl_compound_taskid_set]: All samples in a model task conform to single,
  unique compound task ID set that matches or is coarser than the configured
  `compound_taksid_set`.
✔ [spl_compound_tid]: Each sample compound task ID contains single, unique
  value.
✔ [spl_non_compound_tid]: Task ID combinations of non compound task id values
  consistent across modeling task samples.
✔ [spl_n]: Required samples per compound idx task present.
✔ [submission_time]: Submission time is within accepted submission window for
  round.

── Overall validation result ───────────────────────────────────────────────────

── 2024-10-02-UMass-HMLR.parquet ────

✖ [req_vals]: Required task ID/output type/output type ID combinations missing.
  See `missing` attribute for details.
Error in `hubValidations::check_for_errors()`:
!
The validation checks produced some failures/errors reported above.
Backtrace:
    ▆
 1. └─hubValidations::check_for_errors(v, verbose = TRUE)
 2.   └─rlang::abort("\nThe validation checks produced some failures/errors reported above.")
Execution halted

Success! This closes #83 .


bootengine: add local-fs-pre.target to<EMAIL_ADDRESS>

This is to prevent the possibility of fsck during ignition-disks, which
is ordered before local-fs-pre.target.
<EMAIL_ADDRESS>was already ordered after local-fs-pre.target but
had no wants or requires of it, leaving the potential for fsck to start
before local-fs-pre.target becomes activated.

Still need to test this

closing this as I'm pretty sure this isn't what we want


composite: add COMPOSITE_DEVICES config for composite device

Summary
Add COMPOSITE_DEVICES config for composite device, mentioned in the https://github.com/apache/nuttx/pull/10070.
Impact
composite device.
Testing
sim:usbdev

please add a commit message describing the change

done


Endless loop in start error.

If you mistype an iohyve start command, or try to start a guest that does not exist, bad things happen.

This has been fixed by using a test to see if a directory exists where /iohyve/guestname


Adds bugfix for lwip multicast group commands

Problem

The function to join the multicast group was not getting invoked due to netif being returned as null from GetNetIfByInterface() which calls netif_get_by_index() with index value 0. The function returns NULL if index 0 is passed as a reference to find the netif from the list of netifs maintained by LwIP.

Change overview
Adds fix for group commands based on UDP multicast.
Testing
No new unit test cases added, since this is part of TC-SC-5.1 and TC-SC-5.2.
Platforms tested:
RPI4 (chip-tool) + EFR32 + (rs9116/wf200)
Depends upon #20123
Reopening on #20133

fixes #18984

Fixes cert blocker issue #20286

@andy31415 I believe this should be more inline with the existing implementation using auto variables.
#if LWIP_IPV4 && LWIP_IGMP
    const ip4_addr_t lIPv4Address = aAddress.ToIPv4();
    err_t lStatus;
    if (aInterfaceId.IsPresent())
    {
        struct netif * const lNetif = FindNetifFromInterfaceId(aInterfaceId);
        VerifyOrReturnError(lNetif != nullptr, INET_ERROR_UNKNOWN_INTERFACE);
        const auto method = join ? igmp_joingroup_netif : igmp_leavegroup_netif;
        lStatus           = method(lNetif, &lIPv4Address);
    }
    else
    {
        const auto method = join ? igmp_joingroup : igmp_leavegroup;
        lStatus           = method(IP4_ADDR_ANY4, &lIPv4Address);
    }
    if (lStatus == ERR_MEM)
    {
        return CHIP_ERROR_NO_MEMORY;
    }
    return chip::System::MapErrorLwIP(lStatus);
#else  // LWIP_IPV4 && LWIP_IGMP

Updated code to be consistent. Ended up to making a hybrid of the two that is hopefully readable (after slack discussion with @rosahay-silabs )


Embed consul ui in a spring cloud consul app

@EnableConsulUi? or maybe via actuator if a s-c-consul-ui is on the classpath.

Why you need this, if you may see UI on consul nodes?

I kind of like the thought that it could be protected using the same facilities as other spring-boot resources.  But your right, the reason I haven't moved on this earlier is because I just use the ui in the consul process.


PackageId:CPython39.Exe.x64;PackageAction:DownloadPackage;ReturnCode:0x80070133;

This issue has been moved from a ticket on Developer Community.

Describe your issue here
===DO NOT EDIT BELOW THIS LINE===
PackageId:CPython39.Exe.x64; PackageAction:DownloadPackage; ReturnCode:0x80070133;

Original Comments
Feedback Bot on 3/19/2023, 07:29 PM:
(private comment, text removed)

Original Solutions
(no solutions)

It seems the problem occurs when the CPython installer fails. This could be due to network timeouts, corporate firewall policy, VPN configuration, anti-virus, or other problems. See if a retry helps, and if not, see if you can download the Python runtime directly from https://python.org/ and install it manually.
If that doesn't solve your issue, we could use more detailed logs to see what's really going on. Can you please collect more logs by running http://aka.ms/vscollect.exe and uploading %temp%\vslogs.zip and reopen the ticket if it persists? Thank you!😊


socket timeout error

When trying to run the example, I get this traceback:
Traceback (most recent call last):
File "C:/Users/Minichota/PycharmProjects/Global_Automation/lights.py", line 4, in
data = d.status()  # NOTE this does NOT require a valid key
File "C:\Python\lib\site-packages\pytuya_init_.py", line 255, in status
data = self.send_receive(payload)
File "C:\Python\lib\site-packages\pytuya_init.py", line 165, in _send_receive
s.connect((self.address, self.port))
socket.timeout: timed out
I hope you can help me or guide me through to the solution

@Minichota  is this tuyapi issue 57 - https://github.com/codetheweb/tuyapi/issues/57 ? If so I'm going to hazard a guess that something has that device open already (https://github.com/codetheweb/tuyapi/issues/57#issuecomment-412610094 indicates that maybe the phone app has it open).
This is one of those "common knowledge" things where we know the devices can only handle one active connection at a time but to know this you either need to have been burnt by this or read alot of the history comments on issues in this and the https://github.com/codetheweb/tuyapi/issues/ project (https://github.com/codetheweb/tuyapi/issues/5 has a ton of information). This has sparked me to create a FAQ page in the wiki. I just need to get this into an entry there https://github.com/clach04/python-tuya/wiki/FAQ :-)

well, I'm quite sure that the device isn't connected to by multiple things as when the indicator light is blinking it is disconnected.

I've also unplugged the device and replugged it.

@Minichota if the device light is blinking, then the device is not on the network. This project expect the device to already be provisioned and on the network. I'm guessing this isn't working with https://github.com/codetheweb/tuyapi/ either?

So are you saying that I should run it through the setup.md example you gave again? @clach04

I also haven't tried to use the other tuya api as I don't have a compiler for that language.

@Minichota you mean "Key extraction", yes. You'll need to provision on your network and extract the key (using your preferred method). Unfortunately it looks like the guide I linked to was updated 6 days ago to do provisioning with the CLI tool so you either need to install Node to follow the current guide or read the older guide https://github.com/codetheweb/tuyapi/blob/9c835da1139a3436f43b359b86abeb3529ae90a1/docs/SETUP.md

@clach04 Do you mean to do this if I wasn't able to extract the information?

@Minichota  I don't understand the question. If the light is blinking that means its not connected to the network and its entered provisioning mode. I've updated https://github.com/clach04/python-tuya/wiki and copied the old readme.
From your description it sounds like you need to provision the device. This will change any key(s) you may have already extracted.

I have connected to it using the 'old' method you posted and I have fixed the blinking light issue and its simply when I run the example you provide in the README I get a socket.timeout error as I pasted above. I have access to the all the data from the key extraction and it should technically be connected so I'm not sure.

@clach04 If I up the connection timeout to 50 seconds, I get this error: TimeoutError: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond

@Minichota so definitely a network problem. I'll close this down. If its any consolation I had a similar problem once. I was super tired 😴  and my laptop connected to an open WiFi network and I couldn't work out why I couldn't access any devices on my LAN!

@clach04 by the way, the old setup instructions can be found here.


home-assistant: 0.74.2 -> 0.75.1

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

cc @peterhoeg

Success on x86_64-linux (full log)
Attempted: home-assistant
Partial log (click to expand)
tests/components/test_init.py ................                           [ 71%]
tests/components/test_introduction.py .                                  [ 71%]
tests/components/test_logger.py ......                                   [ 74%]
tests/components/test_script.py .....                                    [ 77%]
tests/components/test_shell_command.py ........                          [ 81%]
tests/components/test_system_log.py ................                     [ 89%]
tests/components/test_websocket_api.py .....................             [100%]

========================== 197 passed in 7.01 seconds ==========================
/nix/store/ay9zxvypbh2vfbbv550k17kr44rbc966-homeassistant-0.75.1

Success on aarch64-linux (full log)
Attempted: home-assistant
Partial log (click to expand)
tests/components/test_init.py ................                           [ 71%]
tests/components/test_introduction.py .                                  [ 71%]
tests/components/test_logger.py ......                                   [ 74%]
tests/components/test_script.py .....                                    [ 77%]
tests/components/test_shell_command.py ........                          [ 81%]
tests/components/test_system_log.py ................                     [ 89%]
tests/components/test_websocket_api.py .....................             [100%]

========================= 197 passed in 26.19 seconds ==========================
/nix/store/bfvjbkgnjjfni5vs2pi3w16dkardgk97-homeassistant-0.75.1

I will not have time to try this out, but it looks great. On a separate note, I think we may need to package pypandoc with a vendored pandoc v1 as the move to pandoc 2 has broken it.


Deprecate instrumentations in favor of opentracing-contrib

Many of the frameworks we instrumented in jaeger-client-java are also instrumented in opentracing-contrib. We should deprecate Jaeger-specific instrumentation.
Criteria:

[x] Need to implement #179 to support active span
[ ] Deprecate jaeger-context (make it a wrapper around active span, if possible)
[ ] Need to investigate compatibility of opentracing-contrib libs with how Jaeger instrumentation was done. Ideally we want to avoid forcing changes to existing code by keeping Jaeger classes as thin wrappers around opentracing-contrib
[ ] If complete backwards compatibility is not possible, document exact migration steps

I'm assigning it to myself.
Later on, do we want to deprecate jaeger wrappers too?

Later on, do we want to deprecate jaeger wrappers too?

If by deprecate, you mean adding @Deprecated to the wrappers, I feel that we should do it when the wrappers are introduced.
I think we should remove jaeger wrappers with the first major release.

I'm more in favor of deprecating and let users upgrade to the contrib versions.
There are some potential problems:

jax-rs adds PEER_SERVICE tag when com.uber.jaeger.Constants.X_UBER_SOURCE(x-uber-source) is present. Users might provide custom span decorator to do this but I think it's inconvenient.
dropwizzard: sampler can be specified when in config. OT contrib integration constructrs tracer in Application class.

Re x-uber-source, at Uber applications usually do not import Jaeger directly, but via central service frameworks, so a custom decorator might work.

But who should provide this decorator? Can we expect that they will correctly implement it? Or should be provided as jaeger-jaxrs2 module?

the decorator that actually contains the x-uber-source string will be provided by our internal infra libs.

I have to do some work on dropwizard contrib lib:
The configuration will accept:
tracing:
   serviceName: // string
   serverAddress: // string could be `http://localhost:9411/api/v1/spans` or `localhost:5984`
   enabled: // to quickly disable tracing by using NoopTracer

Then in the Application class users will provide tracer instance. Is this acceptable for Uber users?
Other problem is that jaeger metrics won't work  https://github.com/uber/jaeger-client-java/tree/master/jaeger-dropwizard#metrics, Could be https://github.com/uber/jaeger-client-java/blob/master/jaeger-dropwizard/src/main/java/com/uber/jaeger/dropwizard/StatsFactory.java and other referenced classes also provided by internal infra libs?

Why do we need to change configuration from its current form?
StatsFactory delegates to com.codahale.metrics.MetricRegistry. I don't think we have any other internal impl for it (if we do, it's backed but the internal metrics API).

@yurishkuro we don't have to change jaeger configuration. I'm talking about dropwizzard configuration which could be used with any OT tracer.

Oh... That sounds like an anti pattern. There's never going to be a generic configuration that will satisfy ALL tracers, so why even bother? Why can't each implementation have its own config? Is this somehow forced by dropwizard?

It's not forced by dropwizard. It's just an idea how to simplify integration for most tracer. In the Application users would still have to instantiate tracer manually and pass it to integration code which would initialize jax-rs instrumentation

@yurishkuro For 1.0 milestone, would you be ok with the framework instrumentations being moved into a separate repo, remaining under the com.uber package - while the other modules would be moved under io.jaegertracing as part of #287 ?

Current plan, as discussed on the last bi-weekly call, is to retain the com.uber maven artifacts for both the tracer and framework instrumentations - but in a separate repo. The tracer, configuration classes, etc would then become wrappers around the io.jaegertracing versions.

I don't want to move it back to uber org because of approvals, CLAs, etc.
How about jaegertracing/legacy-client-java ?

sounds good to me
unfortunately we cannot fork the repo in the same org, so I will create a repo and push sources there (with full history).

Done, instrumentations are now in legacy-client-java repo.

Not really the same, so I rebooked it in the legacy repo.


Add example that leads to a stack overflow error

Simplified from here: https://github.com/AngusJohnson/Clipper2/discussions/345#discussioncomment-4626340

Thanks heaps once again Juha. Hopefully fixed now.

Still failing for this non-simplified input: https://github.com/AngusJohnson/Clipper2/discussions/345#discussioncomment-4626340
Will try to simplify again, now for the latest version.


merge tree settings

How do i override merge tree settings? I want to include this
<merge_tree>
        <max_suspicious_broken_parts>20</max_suspicious_broken_parts>
</merge_tree>

into the configuration definition

spec:
  configuration:
    settings:
      mergree/max_suspicious_broken_parts: 20


adding AMLFS alerting

Overview/Summary
Adding changes for AMLFS alerting and added a new section for the AI pattern
This PR fixes/adds/changes/removes

Adds folder for new AI AMBA pattern
Adds AMLFS alerts for HPC
Adds BYOM scenario for AI pattern

Breaking Changes
NA
As part of this Pull Request I have

[ X] Read the Contribution Guide and ensured this PR is compliant with the guide
[ X] Checked for duplicate Pull Requests
[ X] Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
[ X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ X] Ensured PR tests are passing
[ X] Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@JoeyBarnes - Added GUIDs


[BUG] UserAgent in azure-spring-data-cosmos will be overrided by spring-cloud-azure-autoconfigure

Describe the bug
The method getUserAgentSuffix() will return wrong value if the classpath has more than one META-INF/project.properties
Exception or Stack Trace
No.
To Reproduce
Include dependency
<dependency>
  <groupId>com.azure.spring</groupId>
  <artifactId>spring-cloud-azure-starter-data-cosmos</artifactId>
  <version>4.0.0-beta.3</version>
</dependency>

This is a mini sample to reproduce this issue.
Code Snippet
This is a mini sample to reproduce this issue.
Expected behavior
Expected behavior
userAgentSuffix='az-sd-cos/3.17.0az-sp-cos/4.0.0-beta.3'
it's better to be userAgentSuffix='az-sd-cos/3.17.0  az-sp-cos/4.0.0-beta.3'
but the result is userAgentSuffix='az-sd-cos/4.0.0-beta.3az-sp-cos/4.0.0-beta.3'

Screenshots
The reason is
Setup (please complete the following information):

OS: [e.g. Mac]
IDE: [e.g. IntelliJ]
Library/Libraries: [ azure-spring-data-cosmos:3.17]

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

Hi @kushagraThapar , can you help look into this issue?

@backwind1233 - can you please review this PR ?
https://github.com/Azure/azure-sdk-for-java/pull/27311


Home assistant sruck

Hi
I have this behavior for some time. If I want restart from console I get:. TASK ERROR: can't lock file '/var/lock/qemu-server/lock-201.conf' - got timeout
In console I have multiple task stop pending timeout.
I have to stop server and manually turn it on and then I have

I gave it 8gb of RAM and 32gb disc and it has lot of free space.
It was installed quite long time ago should I install from newest script or this is not related?
Thanks in advance cheers

Read the last line in the console log. The kind of sounds like to me like you're either over provisioning your RAM and it is overcashing and causing deadlocking issues.  I would definitely try to test your RAM to see if the RAM has some faults in it or something like that. But I would also check to make sure you're not using up all of your RAM and more. if you have 8 gig on your system I would stick to 7.5 available between all of your systems running. Do you have a system running 4 gb, in a system running 2 gb, probably shouldn't run a third system on another 4 GB. for having a gigabytes of RAM I see an awful lot of containers. I'm just questioning provisioning the ram correctly.

Hi thanks for response
8 gb is for this vm only I started with 2..   inside vm in glances it never go higher than 2gb used. I was told that quemu agent doesn't show real value in proxmox. I can give it 32gb but is there any point or rather problem is somewhere else

Show me your hardware tab, please. Thanks.
I still don't know for sure what the issue is, but ram availability shouldn't be it. Show me your hardware tab, and the options tab. I'll compare it to my functioning install.

Machine is quite new some 8 months. Xeon e 2246 32 gb of RAM dual nvme in zfs ram. So I don't think that can be hardware.here are screenshots

I have also another home assistant ( dev) it has minimal add-ons and I don't have same problems. There is a difference in time of installation script. The dev is around 3 months old and main one is same as server so around 8 months.

My first thing I would try would be to turn off ram ballooning. In the ram setting set it to at least 3 gig minimum. And turn off ballooning perhaps. One or the other or both. I see your minimum is quite low so perhaps turn up the minimum to something like 3 gig. But if that doesn't work turning off ballooning entirely would just give it the 8 gig and be done with it. I don't know maybe start there. At this point you're not going to break anything else. At least not with doing that.

Thanks I will try removing ballooning. In fact since I write first post all is ok so maybe lastest trimming of home assistant has something to do with it

Thanks I will try removing ballooning. In fact since I write first post all is ok so maybe lastest trimming of home assistant has something to do with it
To be honest I thought that I have minimal 1 gb of ram

do me a favor bud? and scrub/delete your photos? close the ticket perhaps? but definitely scrub the photos, I don't see anything that sticks out but there is alot there.
cheers.


Add option ignoremissingsubtests to ignore missing t.Parallel() in subtests

In our codebase we've coded ourselves into a bit of a corner wherein
although all our top-level tests are safe to run concurrently, subtests
within any particular test are not due to the use of a shared set of
variables. It'd be nice to fix this of course, but doing so would
involved changing thousands of tests and many dozens of hours of work.
For any Go project, if you've managed to get to a place where all your
top-level tests within a package can run in parallel, that's pretty
good. You might not get the same granularity of work balancing that you
would if all your subtests were parallel too, but (depending somewhat on
the variance in timing between your tests) it gets like you like 80% of
the way there.
So here, I propose that the package adds a new -ignoremissingsubtests
flag. It's similar to the existing -i, but only ignores missing calls
to t.Parallel in subtests and ranges.

@kunwardeep Hey, I saw that the project seems to be in a semi-maintained state, but it's still in golangci-lint's core, and AFAIK there isn't a better maintained alternative. Thoughts on this idea?

Hey @brandur i am glad this liner is helping your and your team. I have been sick for a week, hence the delay in response.
I'll have a look at this PR this week. Thanks for contributing.

Thanks Kunwardeep! And no worries — hope you're feeling better soon.

Thanks @kunwardeep. Opened https://github.com/golangci/golangci-lint/pull/3985 to see if we can get this into golangci-lint!


AzureRmWebAppDeployment ignores AdditionalArguments for PublishProfile provider

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: AzureRmWebAppDeployment
Environment

Server - Azure Pipelines or TFS on-premises?

NA

Agent - Hosted or Private:

NA

Issue Description
PublishProfileWebAppDeploymentProvider uses AdditionalArguments parameter that allows to provide additional parameters to the MsDeploy tool.
But this line in TaskParameters.ts always returns taskParameters with default values for the PublishProfile connection type.
Hence AdditionalArguments is always ignored for the PublishProfileWebAppDeploymentProvider and default values are used.
So there is no way to change or remove default -retryAttempts:6 -retryInterval:10000 MsDeploy parameters for this provider.
Task logs
NA
Troubleshooting
NA
Error logs
NA

This issue is not fixed yet


No way to quit Dictater & other UI issues

It doesn't seem possible to quit Dictate. I right-click to select quit, but it's greyed out:

Also, the options menu is wonky in dark mode, but it's not really a big deal:

Finally, I really miss the gear icon that called up the options (The gear-icon in the floating menu that has the controls), but I guess right-clicking is okay. :)

Silly me, you have to click the tick icon to close the app! Whoops. The wonky dark mode still remains, but it's not a major issue.


Notification is not sent when a service parameter is changed

When a service is added or removed the Python API receives an service notification message from the backend, something like this:
<notification xmlns="urn:ietf:params:xml:ns:netconf:notification:1.0"><eventTime>2024-01-02T12:14:42.480596Z</eventTime><services-commit xmlns="http://clicon.org/controller"><tid>23</tid><source>actions</source><target>actions</target><service>as-path-filter[service-name='as-test']</service></services-commit></notification>
However, if I change a service parameter in the CLI the Python API doesn't receive any notification.

Seems to work, closing.


Remove clarify rootfs item from the ROADMAP

We discussed this in the face to face meeting and agreed
that it makes sense to keep the rootfs as is for flexibility.
Signed-off-by: Mrunal Patel<EMAIL_ADDRESS>

@mrunalp: Why does the rootfs have to be part of the Spec?

@vishh We are taking this item out of the roadmap as we don't think that we need any modifications.

What is the rationale to keep the rootfs path configurable?
On Wed, Jan 13, 2016 at 12:07 PM, Mrunal Patel<EMAIL_ADDRESS>wrote:

@vishh https://github.com/vishh We are taking this item out of the
roadmap as we don't think that we need any modifications.
—
Reply to this email directly or view it on GitHub
https://github.com/opencontainers/specs/pull/304#issuecomment-171417679.

On Wed, Jan 13, 2016 at 11:58:15AM -0800, Mrunal Patel wrote:

We discussed this in the face to face meeting and agreed that it
makes sense to keep the rootfs as is for flexibility.

Looking over 1, the only references I see to ‘rootfs’ are in the
archive-format discussion, the layering discussion, and the “Split b/w
runtime.json” discussion (which seems to be about unifying the
config?).  Are you saying “we want to keep the current ‘root.path’
setting so folks can use other names” (which sounds fine to me)?  Or
are you saying “we want to keep the current spec and charter that
require bundles to contain a rootfs”.  I was pushing back on the
latter in 2, and having an optional rootfs certainly seems more
flexible to me.
 Subject: Dropping the rootfs requirement and restoring arbitrary bundle content
 Date: Wed, 26 Aug 2015 12:54:47 -0700
 Message-ID<EMAIL_ADDRESS>

This was discussed yesterday and the conclusion was that bundle authors should be able to name their rootfs whatever they want to.

Ok. Thanks for the clarity.
On Wed, Jan 13, 2016 at 12:10 PM, Mrunal Patel<EMAIL_ADDRESS>wrote:

This was discussed yesterday and the conclusion was that bundle authors
should be able to name their rootfs whatever they want to.
—
Reply to this email directly or view it on GitHub
https://github.com/opencontainers/specs/pull/304#issuecomment-171418881.

Eh, somehow I missed this point yesterday, but I agree with Vish, this
seems totally unnecessary in the spec until we have a good use
case/rationale for why it should be configurable.
On Wed, Jan 13, 2016 at 9:12 PM, Vish Kannan<EMAIL_ADDRESS>wrote:

Ok. Thanks for the clarity.
On Wed, Jan 13, 2016 at 12:10 PM, Mrunal Patel<EMAIL_ADDRESS>wrote:

This was discussed yesterday and the conclusion was that bundle authors
should be able to name their rootfs whatever they want to.
—
Reply to this email directly or view it on GitHub
<https://github.com/opencontainers/specs/pull/304#issuecomment-171418881
.

—
Reply to this email directly or view it on GitHub
https://github.com/opencontainers/specs/pull/304#issuecomment-171419334.

Allowing changing the rootfs allows flexibility in the config. For e.g. one can have two configs point to the same rootfs and invoke the runtime to start readonly containers.
We discussed that the bundle verification is separate from what the runtime sees. So, after the bundle is verified, there could be a prepare tool to merge the layers and prepare a rootfs. Next step might be to have two different configs point to the same prepared rootfs. If we don't allow modifying the path to the runtime then we lose this flexibility.

@mrunalp: My understanding is that the current bundle format will contain a json configration along with a directory named rootfs that will contain the application's rootfs.
If the rootfs were to be re-used, there can be multiple configs with different top level directories.
If the configs have to be re-used, again multiple top level directories can be used with a combination of bind-mounts to avoid copying the rootfs.
I agree that we need to provide an ability to mark the rootfs as readOnly, but I'm having trouble understanding why the directory path rootfs needs to be configurable?

On Wed, Jan 13, 2016 at 12:25:54PM -0800, Vish Kannan wrote:

My understanding is that the current bundle format will contain a
json configration along with a directory named rootfs that will
contain the application's rootfs.

The current bundle format does not require that name 1 (fine with
me), but it does require a rootfs directory (which seems like it's out
of scope for this issue).

If the configs have to be re-used, again multiple top level
directories can be used with a combination of bind-mounts to avoid
copying the rootfs.

It's true that you don't need configurable paths (because you can just
bind-mount anything).  But allowing a configurable path makes this
sort of thing easier.  For example, my editor lets me specify a path
on the command line, so I don't have to bind-mount the file to
‘./edit.txt’ before invoking the editor ;).

Discussing this more the general idea is that it is better for the config to be explicit from the user and not infer anything were we don't have to even if by convention it will most likely be rootfs.
LGTM

LGTM


Fleet CLI update

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
az fleet
General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (`pip

Please fix CI issues

Please fix CI issues.

@kairu-ms CI issues resolved. Good to merge.


add 'k' as argument to FaissRM.forward()

add 'k' as argument to FaissRM.forward()

LGTM! tagging @isaacbmiller as it seems like the ruff fix workflow is erroring out. Is this something on the workflow end? or does it need to be handled within the PR?

If you run ruff check --fix-only, then commit, it will pass CI.

LGTM after that

If you run ruff check --fix-only, then commit, it will pass CI.

@software-artisan


Error building AWS Lambda function

Environment
How are you running Flogo Web?

[x] Docker (started with docker run --rm -it -p 3303:3303 flogo/flogo-docker eula-accept)
[ ] Other (Provide details of your environment)

Describe the bug
Clicking on build -> flogo-aws-lambda -> Linux/amd64 results in an error in the logs and no executable being built.
info: [17:04:40.199] Exec command: flogo create -f /flogo-web/local/engines/exported-app-build.json app-build in /flogo-web/local/engines
info: [17:04:40.203] run command: flogo create -f /flogo-web/local/engines/exported-app-build.json app-build
engine:create: 8283.393ms
info: [17:04:48.483] undefined {"durationMs":8285}
info: [17:04:48.484] [log] Build flogo: "flogo build -o,-e,--shim,aws_lambda_trigger" compileOpts:
info: [17:04:48.486] run command: flogo build -o -e --shim aws_lambda_trigger
[log] build output: Using build.go to build shim......

error: [17:04:54.896] /flogo-web/local/engines/app-build/bin is empty {"stack":"Error: /flogo-web/local/engines/app-build/bin is empty\n    at Object.recursivelyFindFirstFile (/flogo-web/apps/server/main.js:1732:15)"}

My flow is a Lambda trigger and a log activity.
Running Flogo Web version flogo/flogo-docker                        latest              fb9649528a83
As an additional suggestion, clicking build -> flogo-aws-lambda gives you the ability to select multiple operating systems for AWS Lambda, but only Linux/amd64 should be allowed given that AWS Lambda uses either Amazon Linux or Amazon Linux 2.

The lambda trigger is handled specially in the UI and it looks like we forgot to update the reference of the lambda trigger to the new one 😅
https://github.com/project-flogo/flogo-web/blob/8efb2055bc3335031434aef9f5570d74a7c8dc2c/libs/core/src/lib/constants/contributions-ref-placeholder.ts#L3-L4
I wonder if it's okay to handle only the new reference and ignore the old one? @mellistibco

@fcastill we can replace the ref with the new one. No need to maintain a ref for the old repo anymore.

@mellistibco could you please confirm if github.com/project-flogo/aws-contrib/trigger/lambda is the new Lambda trigger ref?

@apandurangi yup, thats the one.


fix .end() calling store.end() multiple times on error

remove a store from the .stores array if it has been closed successfully, so if you need to retry the .end() call a store.end() isn't going to be called although it has already been ended.
if this gets merged, i can do the same for .close()

@mafintosh check this out

fixed in master. forgot you opened this pr sry


Youtube 403 error

Hi!
I recently noticed that my bot does not play youtube tracks anymore, so I updated to newest lavaplayer and youtube-source.
Has something changed recently, so that I would have to authorize it myself? This worked fine about a week ago.
I have never used OAuth or poTokens before, and I assume those are not required still?
Anyways, I still cant play youtube tracks. Any help would be appreciated.
Thanks in advance!
onTrackException:
Exception message: Something broke when playing the track.
Severity: FAULT
Cause:
java.lang.RuntimeException: Not success status code: 403
at com.sedmelluq.discord.lavaplayer.tools.io.PersistentHttpStream.validateStatusCode(PersistentHttpStream.java:88)
at com.sedmelluq.discord.lavaplayer.tools.io.PersistentHttpStream.attemptConnect(PersistentHttpStream.java:124)
at com.sedmelluq.discord.lavaplayer.tools.io.PersistentHttpStream.connect(PersistentHttpStream.java:106)
at com.sedmelluq.discord.lavaplayer.tools.io.PersistentHttpStream.internalRead(PersistentHttpStream.java:158)
at com.sedmelluq.discord.lavaplayer.tools.io.PersistentHttpStream.read(PersistentHttpStream.java:174)
at java.base/java.io.DataInputStream.readByte(DataInputStream.java:271)
at com.sedmelluq.discord.lavaplayer.container.matroska.format.MatroskaEbmlReader.readEbmlInteger(MatroskaEbmlReader.java:39)
at com.sedmelluq.discord.lavaplayer.container.matroska.format.MatroskaFileReader.readNextElement(MatroskaFileReader.java:46)
at com.sedmelluq.discord.lavaplayer.container.matroska.MatroskaStreamingFile.readFile(MatroskaStreamingFile.java:93)
at com.sedmelluq.discord.lavaplayer.container.matroska.MatroskaAudioTrack.loadMatroskaFile(MatroskaAudioTrack.java:52)
at com.sedmelluq.discord.lavaplayer.container.matroska.MatroskaAudioTrack.process(MatroskaAudioTrack.java:35)
at com.sedmelluq.discord.lavaplayer.track.DelegatedAudioTrack.processDelegate(DelegatedAudioTrack.java:25)
at dev.lavalink.youtube.track.YoutubeAudioTrack.processStatic(YoutubeAudioTrack.java:148)
at dev.lavalink.youtube.track.YoutubeAudioTrack.processWithClient(YoutubeAudioTrack.java:116)
at dev.lavalink.youtube.track.YoutubeAudioTrack.process(YoutubeAudioTrack.java:71)
at com.sedmelluq.discord.lavaplayer.track.playback.LocalAudioTrackExecutor.execute(LocalAudioTrackExecutor.java:109)
at com.sedmelluq.discord.lavaplayer.player.DefaultAudioPlayerManager.lambda$executeTrack$2(DefaultAudioPlayerManager.java:330)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
at java.base/java.lang.Thread.run(Thread.java:833)

Follow the issue template.


[release-4.16] Delete pods and lease using etcd container

As of now when user consume the bundle to provision the cluster, old data around pods and lease is consumed and timestamp for those pods are also around same time when we first provisioned this cluster. This PR try to use etcd image to run etcd server using same db which created by the cluster  and uses etcdctl command to delete the pods and lease details. It will help to make sure every pods comes up in clean state.
Something similar done in recert project

https://github.com/openshift-kni/lifecycle-agent/blob/fd1cf27502b9c3f136fa1d161bcbd7ecf7c19da4/lca-cli/ops/ops.go#L186-L221

/retest

/close
#959 surpass it.


fix: hide language/text details on non-editor tabs

[ ] Added an entry to CHANGELOG.md if this change could be valuable to users

Codecov Report

Merging #1776 (38d7706) into master (5cd779f) will increase coverage by 0.02%.
The diff coverage is 0.00%.

 Learn more


Error dialog needs to be displayed when findAnnotations call fails

Sometimes, fetching annotations from the server fails. The error has been logged, but is not displayed to the users. Need to show dialog to inform users.

I've attached the log that reported this problem.
vars-annotation.20181031.log
This is resolved in code I just need to build a new release.


Prompt asking for a list returns 1 random item

I have a super basic script that I want to use for querying an index.
`print("Enter prompt")
prompt = input()
index = GPTSimpleVectorIndex.load_from_disk('testIndex.json')
print(index.query(prompt))`
I am asking it to give me 10 key points from the document and it responds with a valid point that is indeed from the index but only one. It seems to give me back a single item. It might be something like

INFO:llama_index.token_counter.token_counter:> [query] Total LLM token usage: 4794 tokens
INFO:llama_index.token_counter.token_counter:> [query] Total embedding token usage: 9 tokens

some valid and important fact...etc.

Since the data is returned is valid data and its exactly what I wanted, I know its definitely querying the indexc but it has me wondering if i am not outputting this correctly.

Hey, this might be because simple vector index retrieves the top 1 most relevant document chunk by default. You can try to change similarity_top_k parameter, or consider using another index that's better suited for summarization tasks. Read more at https://gpt-index.readthedocs.io/en/latest/use_cases/queries.html#summarization

Over what size of text are you searching for the points? Also if you give the specific prompt it might help since it might take some prompt engineering.


⚠ sandbox-api.piste.gouv.fr healthcheck has degraded performance

In b5d4870, sandbox-api.piste.gouv.fr healthcheck (https://sandbox-api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 2626 ms

Resolved: sandbox-api.piste.gouv.fr healthcheck performance has improved in b49e3c1.


Feature request: Nonce edit

Feature request: A settings toggle that would add a nonce field for all transactions, allowing to override the default nonce.
Scenario: On Ethereum network, when a txn is underpriced with gas, instead of speeding it, the transaction might become redundant at certain time, so a replacing it with a completely different transaction is useful, but requires to override nonce for the new transaction.

@tobutridaz it is already possible to set custom nonce for any transaction. It has been implemented in this PR https://github.com/poanetwork/nifty-wallet/pull/379. And this feature is available in the currently published Nifty wallet 5.1.1. Isn't this you are looking for?

I'm running 5.1.1 and below is a transaction popup after interacting with a dapp. Where do I edit the nonce?

Oh yes. I see it now. The nonce's editing is available in a previous screen when someone sends tx from the wallet itself. And there is no way to get to that previous screen if transaction originated from dApp. I should open access to that previous screen from the confirmation screen.

Yes please, or just add a nonce field on the confirm tx window as well.


Add app_registry.ca property to trust registries with a self signed cert

We are seeing smoke tests fail with the following error message:
[2020-09-24 17:29:51.27 (UTC)]> cf push cf-for-k8s-smoke-1-app-99029be0dfaba098 -p assets/test-node-app --no-route
...
...
Waiting for API to complete processing files...
Job (0f30b04d-672f-42b7-abc9-a98c44cff879) failed: An unknown error occurred.
FAILED

The cf-api-server pod contains the following error message (among a few others)
cf-api-server-699d49df87-zqwh4 > package-image-uploader | 2020/09/24 17:29:55 Error from uploadFunc(/tmp/packages/registry_bits_packer20200924-1-1qu1y1d/copied_app_package.zip, harbor.pig.cf-app.com/ci-app-workloads-pr/40b6d190-a263-404b-9865-86836c5b7e5d): Get "https://harbor.pig.cf-app.com/v2/": x509: certificate signed by unknown authority
ccdb-migrate-n6m6k > istio-proxy | 2020-09-24T17:27:10.546624Z	info	sds	resource:ROOTCA connection is terminated: rpc error: code = Canceled desc = context canceled

It looks like the issue might be due to capi not having the app registry CA cert configured. But it also looks like we don’t have a way to do that. Can y'all expose a property to configure the app registry CA please?

Hi @DennisDenuto, we started some work on adding support for this on our end but discovered we are blocked on kpack supporting the usage of untrusted CAs: https://github.com/pivotal/kpack/issues/207
We will be discussing with them soon to hopefully get that work prioritized or perhaps contribute the changes ourselves.
Thanks!

Thanks @jspawar
FYI, We are going to work around this by injecting the ca cert via a webhook.


Configurable logging for libraries

Proposed changes:

Fixes https://github.com/RasaHQ/rasa/issues/10203

Add environment variables to configure logging at library level. This is enabled for matplotlib, rabbitmq (pika etc.) and kafka at the moment.

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

Does this solution look good? I'll add tests and docs once we agree on a path to take.


Feature: keyboard shortcuts to move nodes up, down, indent, unindent

Ctrl+shift up / ctrl+shift down to move node up or down
ctrl+shift left / right to indent / unindent

Any news on this one?

@zurfyx Can you assign this to me? I would like to work.


fix: encrypt stored sdk config

Background
Following the Slack discussion, we realized that we should encrypt the SDK config to prevent possible reverse engineering and unauthorized extraction of keys.
Changes

Utilized EncryptedSharedPreferences for secure storage of the SDK config
Renamed the existing preferences file for the SDK config to avoid corrupt values
Disabled backup for encrypted preferences, as recommended in the Android docs.

Testing

Native SDK users will not experience any impact, as they initialize the SDK in the Application class, eliminating the need for auto-initialization
For existing wrapper customers, the SDK won't be auto-initialized from the killed state until the app is launched after this update. As soon as the app is launched, stored config values will be updated, and everything will proceed as expected.
To test auto backup changes, enable allowBackup in the app and run the following command:

adb shell bmgr backupnow <app-package-name>

Closing this PR based on our discussion on Slack. We'll continue exploring improvements for this and revisit later.


Update for Swift 5

Lib won't compile because of few changes needed.
Can't even access customHeight property (Fixed in this commit)
Because of -- 'customHeight' is inaccessable due to 'internal' protection level -- error

Friends, thanks for your work!
But your changes not support swift 5.
For example UIScrollViewDecelerationRateFast renamed in swift 4. I am attach screenshoot.
You can create empty project and see it.

Now I am fix internal protect level and publish new version.