Provide a way to the get the current actual

Summary
Some assertions have side effects or traverse the objects under test.
Sometimes, it is desirable to assert something, then perform additional manipulations on the object under test, before asserting more things.
In these cases, you have 2 choices:

Perform the traversal outside of assertj and then assert as you reach the relevant pieces.
Duplicate the traversal both with and without assertj

It would be useful instead if all assertions allowed for the actual object to be retrieved, removing this duplication.
Please let me know if there is already a way to do this (or a better approach) but I can't see one.
Below examples show this with succeedsWithin but the problem is not specific to succeedsWithin.
Inconvenient Alternative 1
  //no way to access what the future returned
  assertThat(future).succeedsWithin(10, TimeUnit.SECONDS);

  //Duplicate the traversal logic
  Object foo = future.get();

  //manipulate object outside of assertj
  //perform further assertions

Inconvenient Alternative 2
  //Get the underlying object first. Don't use assertj and lose the assertion failures if something goes wrong
  Object foo = future.get()

  //manipulate object outside of assertj
  //perform further assertions

Desired Outcome
  Object foo = assertThat(future)
    .succeedsWithin(10, TimeUnit.SECONDS)
    .getActual();

  //manipulate object outside of assertj
  //perform further assertions

Proposed Solution
Just add a getter to AbstractAssert:
  public ACTUAL getActual() {
    return actual;
  }

The team is not convinced exposing getActual is worth it at the moment, we believe option 1 is acceptable.
For future what we could do is something similar to optional and add get()/get(InstanceOfAssertFactory) to the API to be able to assert on the future's result.
assertThat(futureString).succeedsWithin(10, TimeUnit.SECONDS)
                        .get(InstanceOfAssertFactories.STRING)
                        .startsWith("foo");

It's a pity that this proposal was rejected. I was also looking forward to such a feature.
I sometimes have integration tests that check a certain workflow. It would be very handy to verify the structured response from one operation down to a nested field and then use that value for another operation (without having to repeat the extraction).
For example:
List<CustomerDiagnostic> customerDiagnostics = diagnosticService.getCustomerDiagnostics(todaysTimeRange);

// Online shop had one customer today (Joe), who had three different shopping sessions, the last one being still active
String sessionId = assertThat(customerDiagnostics).singleElement()
                                                  .returns("Joe", from(CustomerDiagnostic::getName))
                                                  .extracting(CustomerDiagnostic::getSessionHistory,
                                                              as(list(ShoppingSession.class)))
                                                  .hasSize(3)
                                                  .last()
                                                  .extracting(ShoppingSession::getSessionId, as(STRING))
                                                  .hasSize(32)
                                                  .getActual();

boolean isSessionActive = sessionService.isSessionActive(sessionId);
assertThat(isSessionActive).isTrue();

sessionService.doSomethingWithSession(sessionId);

@mgrafl the assertion assertThat(isSessionActive).isTrue(); can be done after .hasSize(32) with a call to satisfies.

The team has discussed again and we still think adding getActual is not worth it.

Okay. Thanks for the reconsideration though.

In case it helps, this is how I workaround this issue when I need to a) assert that a Future completes within a certain time and b) get the result of that Future so that I can use it for subsequent, non-assertion calls i.e. an integration test.
    public static <T> T assertSucceedsWithin(Future<T> future, long timeout, TimeUnit timeUnit) {
        try {
            return future.get(timeout, timeUnit);
        } catch (InterruptedException | ExecutionException | TimeoutException | CancellationException e) {
            String message = "%n"
                    + "Expecting%n"
                    + "  <%s>%n"
                    + "to be completed within %s %s.%n%n"
                    + "exception caught while trying to get the future result: ";
            fail(String.format(message, future, timeout, timeUnit), e);
            //This will never be thrown because fail throws but the compiler isn't clever enough to work that out.
            throw new RuntimeException(e);
        }
    }

It is clear that getActual() will not become part of the AssertJ API and I don't want to reheat the discussion.
Anyway, if anyone is looking for a custom extension, consider this snippet:
public class ActualReturningAssert<ACTUAL> extends AbstractAssert<ActualReturningAssert<ACTUAL>, ACTUAL> {

	protected ActualReturningAssert(ACTUAL actual) {
		super(actual, ActualReturningAssert.class);
	}

	public ACTUAL getActual() {
		return actual;
	}

	/**
	 * Allows returning the actual element of an Assertion.
	 *
	 *
Example:
	 *
String singleEntry = Assertions.assertThat(myStringList)
	 *   .singleElement(as(actualReturning(String.class)))
	 *   .getActual();

	 *
	 * @param <ACTUAL>  type of the actual element
	 * @param actualClass  class of the actual element
	 * @return AssertFactory decorator for use in asInstanceOf(...)
	 *     that provides access to the actual element
	 */
	public static <ACTUAL> InstanceOfAssertFactory<ACTUAL, ActualReturningAssert<ACTUAL>>
        actualReturning(Class<ACTUAL> actualClass) {
		return new InstanceOfAssertFactory<>(actualClass, ActualReturningAssert::new);
	}
}


RuntimeError: Terminating translation prematurely after failed translation of layer

I uploaded the following ESRI geodatabase as test, it failed to ingest
https://undpgeohub.blob.core.windows.net/userdata/a85516c81c0b78d3e89d3f00099b8b15/raw/GIS Parcels_20231102130408.gdb.zip
2023-11-02 13:08:25-ingest.py:sync_ingest:239:INFO:Starting to ingest https://undpgeohub.blob.core.windows.net/userdata/a85516c81c0b78d3e89d3f00099b8b15/raw/GIS Parcels_20231102130408.gdb.zip
2023-11-02 13:08:26-azblob.py:download_blob_sync:467:INFO:Downloading a85516c81c0b78d3e89d3f00099b8b15/raw/GIS Parcels_20231102130408.gdb.zip
2023-11-02 13:08:32-azblob.py:_progress_:448:INFO:downloaded - 10%
2023-11-02 13:08:32-azblob.py:_progress_:448:INFO:downloaded - 10%
2023-11-02 13:08:34-azblob.py:_progress_:448:INFO:downloaded - 20%
2023-11-02 13:08:34-azblob.py:_progress_:448:INFO:downloaded - 20%
2023-11-02 13:08:34-azblob.py:_progress_:448:INFO:downloaded - 20%
2023-11-02 13:08:36-azblob.py:_progress_:448:INFO:downloaded - 30%
2023-11-02 13:08:36-azblob.py:_progress_:448:INFO:downloaded - 30%
2023-11-02 13:08:36-azblob.py:_progress_:448:INFO:downloaded - 30%
2023-11-02 13:08:36-azblob.py:_progress_:448:INFO:downloaded - 30%
2023-11-02 13:08:37-azblob.py:_progress_:448:INFO:downloaded - 40%
2023-11-02 13:08:37-azblob.py:_progress_:448:INFO:downloaded - 40%
2023-11-02 13:08:37-azblob.py:_progress_:448:INFO:downloaded - 40%
2023-11-02 13:08:39-azblob.py:_progress_:448:INFO:downloaded - 50%
2023-11-02 13:08:39-azblob.py:_progress_:448:INFO:downloaded - 50%
2023-11-02 13:08:39-azblob.py:_progress_:448:INFO:downloaded - 50%
2023-11-02 13:08:39-azblob.py:_progress_:448:INFO:downloaded - 50%
2023-11-02 13:08:41-azblob.py:_progress_:448:INFO:downloaded - 60%
2023-11-02 13:08:41-azblob.py:_progress_:448:INFO:downloaded - 60%
2023-11-02 13:08:41-azblob.py:_progress_:448:INFO:downloaded - 60%
2023-11-02 13:08:43-azblob.py:_progress_:448:INFO:downloaded - 70%
2023-11-02 13:08:45-azblob.py:_progress_:448:INFO:downloaded - 80%
2023-11-02 13:08:47-azblob.py:_progress_:448:INFO:downloaded - 90%
2023-11-02 13:08:47-azblob.py:_progress_:448:INFO:downloaded - 90%
2023-11-02 13:08:49-azblob.py:_progress_:448:INFO:downloaded - 100%
2023-11-02 13:08:50-azblob.py:set_blob_metadata:227:INFO:Successfully updated metadata for a85516c81c0b78d3e89d3f00099b8b15/raw/GIS Parcels_20231102130408.gdb.zip
2023-11-02 13:08:50-utils.py:get_progress:117:INFO:/vsizip//tmp/tmpiafxybj5/GIS Parcels_20231102130408.gdb.zip contains:

2023-11-02 13:08:50-utils.py:get_progress:118:INFO:        1 vector layers
2023-11-02 13:08:50-utils.py:get_progress:119:INFO:        0 subdatasets
2023-11-02 13:08:50-utils.py:get_progress:120:INFO:        0 raster bands
2023-11-02 13:08:50-processing.py:process_geo_file:576:INFO:Opened /vsizip//tmp/tmpiafxybj5/GIS Parcels_20231102130408.gdb.zip with OpenFileGDB vector driver
2023-11-02 13:08:50-processing.py:process_geo_file:580:INFO:Found 1 vector layers
2023-11-02 13:08:51-processing.py:process_geo_file:586:INFO:Ingesting vector layer "All_Parcels".
2023-11-02 13:08:51-processing.py:should_reproject:49:ERROR:Failed to compare src and dst projections using osgeo.osr.GetAuthorityCode. Trying using osgeo.osr.IsSame
2023-11-02 13:08:52-processing.py:dataset2fgb:177:ERROR:dataset: https://undpgeohub.blob.core.windows.net/userdata/a85516c81c0b78d3e89d3f00099b8b15/raw/GIS Parcels_20231102130408.gdb.zip
layer: All_Parcels
gdal_error_message: Traceback (most recent call last):
  File "/usr/src/app/ingest/processing.py", line 149, in dataset2fgb
    fgb_ds = gdal.VectorTranslate(destNameOrDestDS=dst_path,
  File "/usr/lib/python3/dist-packages/osgeo/gdal.py", line 1472, in VectorTranslate
    return wrapper_GDALVectorTranslateDestName(destNameOrDestDS, srcDS, opts, callback, callback_data)
  File "/usr/lib/python3/dist-packages/osgeo/gdal.py", line 6085, in wrapper_GDALVectorTranslateDestName
    return _gdal.wrapper_GDALVectorTranslateDestName(*args)
RuntimeError: Terminating translation prematurely after failed
translation of layer All_Parcels (use -skipfailures to skip errors)

2023-11-02 13:08:52-processing.py:dataset2fgb:183:INFO:Uploading error message to a85516c81c0b78d3e89d3f00099b8b15/raw/GIS Parcels_20231102130408.gdb.zip.error
2023-11-02 13:08:53-azblob.py:upload_content_to_blob:206:INFO:Successfully wrote content to a85516c81c0b78d3e89d3f00099b8b15/raw/GIS Parcels_20231102130408.gdb.zip.error
2023-11-02 13:08:53-azblob.py:set_blob_metadata:227:INFO:Successfully updated metadata for a85516c81c0b78d3e89d3f00099b8b15/raw/GIS Parcels_20231102130408.gdb.zip
2023-11-02 13:08:53-processing.py:process_geo_file:639:INFO:/vsizip//tmp/tmpiafxybj5/GIS Parcels_20231102130408.gdb.zip does not contain raster GIS data
2023-11-02 13:08:53-ingest.py:sync_ingest:267:INFO:Finished ingesting https://undpgeohub.blob.core.windows.net/userdata/a85516c81c0b78d3e89d3f00099b8b15/raw/GIS Parcels_20231102130408.gdb.zip

┆Issue is synchronized with this Wrike task by Unito

@JinIgarashi, this is not an bug, it is by design
The dataset has one layer only and in this way the user knows the data is BAD. If the projection is unknown it is not possible to process the data and be sure the results are correct. We can discuss about this case and if we decide to process the data i can make small changes (use -skipfailures to skip errors). This however will produce an empty layer or an unusable layer.

that is okay if this is not a bug, but the error message should be enough for users to understand what they need to do

@iferencik please also update supported format page of GeoHub for these spec you have decided


Bump revision of kubernetes-master

@mbruzek Bump kubernetes-master to pick up https://github.com/juju-solutions/kubernetes/pull/93

+1 lgtm


[azul-zulu] Update latest versions

Based on https://docs.azul.com/core/release/january-2024/release-notes/release-notes#.
Also rollback the latest 6 and 7 version to non-commercial versions.

From https://github.com/endoflife-date/release-data/issues/331.


EnsembleModel's use of clientQueue is used incorrectly

Issue Description
What did you expect to see happen?
The number of jobs submitted should be capped by the JobHandler.maxQueueSize constant.
What did you see instead?
When running an EnsembleModel test case @PaulTalbot-INL was experiencing an unbounded submission of jobs before the queue was emptied.
Do you have a suggested fix for the development team?
The JobHandler.__clientQueue was not being checked for availability when using the EnsembleModel instead the JobHandler.__queue was checked.
In this case, new samples would be constantly appended to the clientQueue and the __queue would not be filled and thus would report empty spots available.
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or improvement?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements? should not affect current requirements
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users. It is unclear to me if any data was lost when the jobs were submitted on top of possibly existing jobs? My assumption is that the model would fail.  I don't believe that the data would be "incorrect," but rather missing as the jobs would be labeled as failed?
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)?
[x] 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided?

The users affected by this change will be updated by an upcoming milestone report; since no incorrect results can be obtained, additional notification is optional.
Note that the defect itself is not directly tested, but the mechanics surrounding it are.  We don't have a methodology to test this kind of performance improvement at this time.


Multiline Enhanced Rich Text Field in Document Library

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
Edit pop up should open on clicking of Pen (Edit button)
Observed Behavior
404 NOT FOUND error
Steps to Reproduce
Added Multiline Enhanced Rich Text field to a SharePointOnline Document Library using JSOM. The field appears properly in classic mode but gives 404 NOT FOUND error when clicking on the Pen (edit button) in modern experience.
var fieldSchema = '<Field Type="Note" DisplayName="Feedback" Required="FALSE" EnforceUniqueValues="FALSE" Indexed="FALSE" NumLines="6" UnlimitedLengthInDocumentLibrary="TRUE" RichText="TRUE" RichTextMode="FullHtml" IsolateStyles="TRUE" Sortable="FALSE" ID="{450158A2-AD0C-46A5-9108-BB1D3FA3649B}" StaticName="Feedback" Name="Feedback" ColName="ntext2" RowOrdinal="0" />'
var clientContext = new SP.ClientContext();
var targetList = clientContext.get_web().get_lists().getByTitle('Documents');
var fieldCollection = targetList.get_fields();
fieldCollection.addFieldAsXml(fieldSchema, true, SP.AddFieldOptions.addToDefaultContentType);
clientContext.load(fieldCollection);
clientContext.executeQueryAsync(function () {console.log('success');}, function() {console.log('failure');});

This could potentially be also in the xml structure. We'll have a look at some point but might take a while. I would do the field adding first from UI and then see the value in the field to mimic that in your code.

Thanks @VesaJuvonen. Same XML when added to a List (not library) works in modern experence as well.

@ahackettms Multiline Enhanced Rich Text field cannot be added to a SharePointOnline Document Library from the UI.

A generic rich-text multiline text field is not supported in the document library currently, which is why it's not visible in the UI. Underlying API has not been changed to block this, which is how you were able to get it in, even though it's not yet supported. We have a backlog item for adding rich-text multi-line support for the modern document library, but there's no ETA at this time.

Thanks for the update @VesaJuvonen


Deploy a new version of the app from the UI (Deploy Button)

List outstanding PR's against the branch for a target in a branch.   Merge via github UI, and watch the app be deployed via the status page.

Ready for design.

Design for deploying a new version of the app: https://balsamiq.cloud/s5qd59g/pks8dty/rFE77

Closing in favor of #550


fix: ensure popup commands are interpreted by sh

tmux 3.5 and 3.5a introduce some changes to the default shell used for executing commands in display-popup ^1 ^2 ^3. Our scripts are written in sh, which means if users employ a shell that is incompatible with sh syntax, such as fish, these commands may not be recognized correctly. This PR ensures tmux uses /bin/sh to execute popup commands and also properly propagates the user's default-shell option to the popup server, as the aforementioned upstream updates change $SHELL to /bin/sh.

This change sometimes seems to change the user shell permanently.
For example spawning something with sh like lazygit, then on the main window creating a new window for tmux will run it with sh since we did not quit from the lazygit yet and set default-shell "$default_shell" part does not execute if the tmux session is not finished.

This change sometimes seems to change the user shell permanently.

Should be fixed in #15; could you test it?


Form Text is all bold

Like in Edit Trip, all the text is bold, it's weird

I guess, it doesn't look terrible. I've kinda gotten used to it. Going to close this.


Fixes decision maker tag values for remote user rules and dynamic rules

This is to avoid a conflict with the duplicate value -10 used by data jobs.
The Go tracer change is also being made and will be released in 1.63.1.
Motivation

Changes

Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] Relevant label (run-parametric-scenario, run-profiling-scenario...) are presents
[ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team

There is a failure on parametric and go related with change.
I move this PR to draft

CI fails on golang? maybe you're waiting for a PR to be merged on dd-trace-go ?

Yes, there is a corresponding dd-trace-go change. But that can't be made easily for the chicken-n-egg problem (see this discussion.

CI fails on golang? maybe you're waiting for a PR to be merged on dd-trace-go ?

Yes, there is a corresponding dd-trace-go change. But that can't be made easily for the chicken-n-egg problem (see this discussion).

Pending dd-trace-go change was merged. All tests passed now.


LandingPage: Desktop view loading on iphone 6splus

Must be a breakpoint issue

flagging this again @mattwbenson as priority before monday.


delete-context deletes child objects

when deleting a context, this also deletes the referenced endpoint and user.
this should not happen as I may just delete the context to recreate with a different endpoint or user.
while there is a --cascade switch, it's defaulting to true and there is no (visible?) way of setting it to false

using --cascade=false works (= is key here)

Sweet, is there anything more we need to do with this issue?

It should probably be highlighted somewhere in docs or quick hint that the = is required (while it's not for most other parameters) but other than that  I think this is fine :)

Yes, great feedback.  I've added an example to the --help, so it is now:
C:\src\go-sqlcmd\cmd\modern>sqlcmd config delete-context --help
Delete a context
Usage:
sqlcmd config delete-context [NAME] [flags]
Examples:
Delete a context (including its endpoint and user)
sqlcmd config delete-context --name my-context --cascade
sqlcmd config delete-context my-context --cascade
Delete a context (excluding its endpoint and user)
sqlcmd config delete-context --name my-context --cascade=false
sqlcmd config delete-context my-context --cascade=false

Perfect!


Rim Disorders and Babies (1.4 Core/Biotech)

User Trooper50000 reported an error with Rim Disorders applying PTSD to babies while playing. I isolated the error to just Core/DLC and Rim Disorders.
Started dev quickstart:
Spawned in baby pawn.
Had a crib.
Had baby food.
Had care.
Approx 3 days in-game the letter PTSD came upon mouse over stated Baby had developed PTSD due to being downed. I don't know if this interaction is specifically due to how vanilla coded babies and their interactions from being placed into a crib. Or the 0% movement stat. Or if the C# patch for AgeMatters can be also applicable to vanilla age. Either way I hope this helps and reach me on Discord for any other information about said issue.

Also note that the same user contacted me again through Discord with the issue of paralytic abasia pawn suffering from PTSD just because they were suffering from paralytic abasia. IDK if you want to tweak the chances of that triggering, but I figured it's how the game treats downed pawn and how the mod interacts with that condition.

Now ignores children from biotech and pawns with paralytic abasia
8b19ca2


Support ndarrays of Delayed objects in da.from_delayed

Currently, da.from_delayed is not an inverse function of da.to_delayed.
da.to_delayed returns a NumPy array, with object dtype, where each element is a Delayed object referring to that chunk of the array.
But da.from_delayed only accepts a single Delayed object, representing a single chunk of the array. You'd have to map this over every Delayed object, then use da.stack to combine them all together. Not only is this inconvenient, it's also much less performant: the stack operation will add unnecessary complexity to the graph, plus will be quite slow to run for large numbers of chunks: https://github.com/dask/dask/issues/5913#issuecomment-587360588.
It would make sense for from_delayed to have a fastpath if the input is a NumPy array of Delayeds, which would add minimal (or, when possible, zero) complexity to the graph, and at most 1 extra HLG layer.
cc @ian-r-rose @jsignell

FYI, I'll have a PR for this up soon.


Not Working for swift 3

your code is not updated for version 1.2.1 and also there is no Edit option like Instagram . Please update it for swift3 and multiple image selection is not working.

@devender54321 the current version is 2.7.2.
 From version 1.2.1 onward it is iOS9+ only.

I am using iOS 10.3.2 , And using code 1.2.1 of your framework.
On Fri, Mar 16, 2018 at 5:28 PM, Roberto Dias<EMAIL_ADDRESS>wrote:

@devender54321 https://github.com/devender54321 the current version is
2.7.2.
 From version 1.2.1 onward it
is iOS9+ only.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Yummypets/YPImagePicker/issues/59#issuecomment-373692443,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aeo1kbmKrqlk40j3bKnrkhI5hECOfyZ1ks5te6j2gaJpZM4StOq2
.

@devender54321 multiple image selection is not available at the moment, we are working on it :)

And What about the Edit Option like Instagram
On Fri, Mar 16, 2018 at 6:50 PM, S4cha<EMAIL_ADDRESS>wrote:

@devender54321 https://github.com/devender54321 multiple image
selection is not available at the moment, we are working on it :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Yummypets/YPImagePicker/issues/59#issuecomment-373710913,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aeo1kUH4wLD3svkhfnVPzYxUS6jHmYVjks5te7wdgaJpZM4StOq2
.

https://github.com/Yummypets/YPImagePicker/pull/47

https://github.com/Yummypets/YPImagePicker/pull/47

@devender54321 not sure I understand what edit means, do you mean the filters ?

No sir the two option found in Instagram Filter and Edit (Brightness)
On Sat, Mar 17, 2018 at 2:59 PM, S4cha<EMAIL_ADDRESS>wrote:

@devender54321 https://github.com/devender54321 not sure I understand
what edit means, do you mean the filters ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Yummypets/YPImagePicker/issues/59#issuecomment-373906843,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aeo1kXKXSR25sYvTvwjnJsYeTcRzr8-Zks5tfNd3gaJpZM4StOq2
.

Edit is not supported neither at the moment but feel free to contribute :) update your version to 2.7.X to get the latest updates.


Large multiaccount wallet stops syncing, requires restart

Not sure if it actually syncs successfully
Jack has a copy of it.

This is happening now on YT sync with the pakman wallet, trying to manually upload the DB to see if that helps.
This is compounded by: https://github.com/lbryio/lbry-sdk/issues/2352


Dev

@daphneaugier Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Vickel: Watch grafixxsea(?:@gmail.com)?

Vickel requests the watch of the watch_keyword grafixxsea(?:@gmail\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

grafixxsea(?:@gmail\.com)? has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Specifying the type of child class

Hi,
So I'm sure its possible, I just can't find out how:
I have a component class, and multiple children of that class (e.g. MeshRenderer extends Component).
In my xml, I now want to specify which implementation to use. I thought it would be possible with the type argument, but this is not working:
      <Components>
        <Component type="BrokenEngine.Componenets.MeshRenderer">
          <Mesh>airboat.obj</Mesh>
          <Material>MyFirstMaterial</Material>
        </Component>
      </Components>

(Components is a List Components)
Thank you for your help!

OK cool.  My recommendation would be to create a very simple unit test (or console app) that takes a list of objects in memory, and then serialize that into the data form.  Then take the result of that and put that in the file to start with, and modify accordingly.  At least, that is how I make my files. :)
In v2 the attributes have changed so they require full namespaces now.  It should be something like the following:
<Components>
        <Component xml:exs="https://extendedxmlserializer.github.io/v2" exs:type="BrokenEngine.Componenets.MeshRenderer">
          <Mesh>airboat.obj</Mesh>
          <Material>MyFirstMaterial</Material>
        </Component>
      </Components>

Let me know how that works for you.

Okay, it didn't work at first, but then I serialized a hardcoded object.
I didn't know that the property name should be the type-name as well.
So this worked:
        <MeshRenderer xmlns="clr-namespace:BrokenEngine.Components;assembly=BrokenEngine">
          <Mesh>airboat.obj</Mesh>
          <Material>MyFirstMaterial</Material>
        </MeshRenderer>
      </Components>


why PPO is an on-policy algorithm?

https://spinningup.openai.com/en/latest/algorithms/ppo.html
PPO is considered as an on-policy algorithm but it also uses the importance sampling item when updating the parameters of policy neural network.So is it an on-policy algorithm?

It's considered "on-policy" because each update step solves an optimization problem that requires data from the most recent iteration of the policy. Using stale data is not generally recommended. (Although, to be fair, there is not a strong body of evidence to demonstrate that it would fail.)


Prisma Docker-compose is invalid

Describe the content you'd like to see
The host property is invalid
Describe alternatives you've considered
It should be the container name, not "db"
Additional context
https://cl.ly/121p1c1z321a

Can you please share the link of where it is written incorrectly? Thanks!

I'm glad I was of help, @divyenduz apologies for not answering early.


Add a pseudorandom generator for move tests

Closing #7544

Thank you Jonas, this will be super useful to Move devs!

Would you mind also adding API's for other integer types (u16, u32, u128, u156)?
Because bcs won't have peel_ functions for some of these, you may just want to roll your own version of that functionality. We will eventually have from_bytes constructors for all integer types, but in the meantime we'll have to make do.

Those are added now, and I've written my own "from bytes" functions for all types, and I'm using @emmazzz suggestion on generating values in range from modulus instead of rejection sampling. Let me know if anything is missing.


bash-completions: 2.10 -> 2.11

Motivation for this change
Failed in https://r.ryantm.com/log/2020-10-17.log because nixpkgs-review wanted to evaluate almost everything.
Things done

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

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

[ ] 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 nixpkgs-review --run "nixpkgs-review wip"
[ ] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Dupe of #100847.


Error hosting emulator container image in App Service

Testing the different Azure hosting options for the emulator:
Azure Container Instances: works (no TLS, limiting testing options)
Azure Container Apps: Works
Azure App Service: Fails
App Service fails to load a container image of the emulator.  The following error is in the deployment centre logs:
Message: failed to register layer: Error processing tar file(exit status 1): Container ID<PHONE_NUMBER> cannot be mapped to a host ID
Internet searching indicates that this could be an issue with a security layer in the hosting environment and the ID number being too large (https://circleci.com/docs/high-uid-error/).  This might be unique to my environment but recreating a new container image and new App Service didn't resolve this and @scottperham recommended logging for testing.
Repro steps with ACR:
docker build -t marketplaceapiemulator -f docker/Dockerfile .
docker tag marketplaceapiemulator .azurecr.io/marketplaceapiemulator:v1
docker push .azurecr.io/marketplaceapiemulator:v1
Enable Admin User in ACR to light-up deployment options
Select App Service as a deployment option for the container
Review the logs in the App Service deployment centre for errors
Error on browsing to public endpoint

I tested this as follows:

Creating a new App Service plan: plan (B1: 1)
From ACR > Repositories > Container Image > Select "Deploy to Web App"
Wait for deployment and browse to URL of new app service
No errors on deployment
Worked as expected.

Tested:

Cloned the repo
Build > Push
From ACR > Repositories > Container Image > Select "Deploy to Web App" - created new app service and plan selecting Linux
As with above, it's working as expected

It looks like this is specific to NPM v9. The image we use in the Dockerfile - node:18-alpine3.16 - has recently been updated to v9. Thus it works for us as we have a cached version of the image. I've been able to repro by removing my local docker image and starting from fresh.
We could either pin to the older version of the image or I can look at possible fixes....

See also #21


[🐞]

Which component is affected?
Qwik Runtime
Describe the bug
The data shaker is too aggressive and removes the myData even thought there is a code path where it is needed.
Reproduction
https://stackblitz.com/edit/qwik-starter-eywjnp?file=README.md,src%2Froutes%2Findex.tsx
Steps to reproduce
Navigate to https://stackblitz.com/edit/qwik-starter-eywjnp?file=README.md,src%2Froutes%2Findex.tsx type into the search  box.
System Info
n/a

Additional Information
I think #3355 may be related

Should be fixed already!

Should be fixed already!

Can you tell me which PR or commit hash this was fixed in? Misko came across this bug during an interview with me and I want to update my community on the fix.

@w3cj If it is related to #3355 like @mhevery thought, then it looks like this PR fixed the issue https://github.com/BuilderIO/qwik/commit/a62bbb85c1bf83b6f40399b91277560075475aa2


Which labels to use with transformers libary ?

Hello,
this looks like a great piece of work, thank you for making it available. I tried to explore clinicalBERT for some NER tasks using the transformers library. I can obtain a list of token index results from torch.argmax, but I cannot find a suitable a set of labels (predictions is containing values as large as 687). What am I doing  wrong ?
from transformers import AutoTokenizer, AutoModel
import torch

tokenizer = AutoTokenizer.from_pretrained("emilyalsentzer/Bio_ClinicalBERT")

model = AutoModel.from_pretrained("emilyalsentzer/Bio_ClinicalBERT")

label_list = ["B-IDNUM", "I-IDNUM", "B-HOSPITAL", "I-HOSPITAL", 'B-PATIENT', 'I-PATIENT', 'B-PHONE', 'I-PHONE',
        'B-DATE', 'I-DATE', 'B-DOCTOR', 'I-DOCTOR', 'B-LOCATION-OTHER', 'I-LOCATION-OTHER', 'B-AGE', 'I-AGE', 'B-BIOID', 'I-BIOID',
        'B-STATE', 'I-STATE','B-ZIP', 'I-ZIP', 'B-HEALTHPLAN', 'I-HEALTHPLAN', 'B-ORGANIZATION', 'I-ORGANIZATION',
        'B-MEDICALRECORD', 'I-MEDICALRECORD', 'B-CITY', 'I-CITY', 'B-STREET', 'I-STREET', 'B-COUNTRY', 'I-COUNTRY',
        'B-URL', 'I-URL',
        'B-USERNAME', 'I-USERNAME', 'B-PROFESSION', 'I-PROFESSION', 'B-FAX', 'I-FAX', 'B-EMAIL', 'I-EMAIL', 'B-DEVICE', 'I-DEVICE',
        'O', "X", "[CLS]", "[SEP]"]

sequence = "Patient had severe headache and took two Aspirine."

# Bit of a hack to get the tokens with the special tokens
tokens = tokenizer.tokenize(tokenizer.decode(tokenizer.encode(sequence)))
inputs = tokenizer.encode(sequence, return_tensors="pt")

outputs = model(inputs)[0]
predictions = torch.argmax(outputs, dim=2)

print([(token, label_list[prediction]) for token, prediction in zip(tokens, predictions[0].tolist())])

Lars

Hi,
I have run into a similar kind of issue. As this repository only provided clinical BERT models that were pretrained via masked LM and NSP on MIMIC data but did not release any of the models that we fine-tuned on the i2b2 or MedNLI tasks. #22
The easy workaround for NER tasks can be as follows:
from transformers import AutoTokenizer, AutoModel, AutoModelForTokenClassification import torch tokenizer = AutoTokenizer.from_pretrained("emilyalsentzer/Bio_ClinicalBERT") model = AutoModelForTokenClassification.from_pretrained("emilyalsentzer/Bio_ClinicalBERT")
AutoModelForTokenClassification will append a token classification head and will import weights for the BERT model. Don't forget to fine-tune this model.
Let me know if it helped you.
-SodhA

Hi Lars,
@vatsal-sodha is correct. We're unfortunately unable to upload the NER models trained on i2b2 to huggingface because we trained these in TF and the conversion scripts for converting TF to Pytorch models only work for the pretrained models as far as I understand. Your best bet is to do what @vatsal-sodha describes. Load in the clinicalBERT base and train the model on the downstream dataset of your choice. If you do this, let me know and we can feature it on this repo.
Closing this issue now, but feel free to reopen or open a new one if you have additional questions.


only console users are allowed to run the x server

x11docker --nxagent --desktop --gpu --fullscreen --sudouser --shell="bash" --env="SHELL=/bin/sh" --dbus --hostdbus --sharecgroup --  --  x11docker/kde-plasma

is the command i used
i got this error:
x11docker ERROR: waitforlogentry(): dockerrc: Found error message in logfile.
Last lines of logfile xinit.log:
/usr/lib/xorg/Xorg.wrap: Only console users are allowed to run the X server
any help?
thanks

Please have a look at the rest of x11docker output.
--nxagent does not support --gpu. So x11docker uses --xorg as a fallback and should have told you so.
x11docker also prints a message regarding "Only console users are allowed to run the X server" and provides a solution:
x11docker WARNING: Your configuration seems not to allow to start
  a second core Xorg server from within X. Option --xorg may fail.
  (Per default, only root or console users are allowed to run an Xorg server).

  Possible solutions:
  1.) Install one of nested X servers 'Xephyr', 'Xnest' or 'nxagent'.
      For --gpu support: install 'weston' and 'Xwayland'.
  2.) Switch to console tty1...tty6 with <CTRL><ALT><F1>...<F6>
      and start x11docker there.
  3.) Run x11docker as root.
  4.) Edit file '/etc/X11/Xwrapper.config' and replace line:
          allowed_users=console
      with lines
          allowed_users=anybody
          needs_root_rights=yes
      If the file does not exist already, you can create it.
      On Debian and Ubuntu you need package xserver-xorg-legacy.

   Be aware that switching directly between Xorg servers can crash them.
   Always switch to a black console first before switching to Xorg.

If you do not get this message, please tell me.

i thought i had installs weston but hadn't


Best practice for polling and returning a promise?

Thank you for the awesome library. I'm relatively new to Swift and trying my best to find my way.
I'm trying to implement a recursive function that returns a promise that resolves when results are not nil, otherwise wait a second and try again.
In JavaScript I'd use BlueBird's Promise.delay function, similar to this gist: https://gist.github.com/wpm/f745ea6478507c6eb72f
How would I do this with PromiseKit? Are promises even the right design pattern here?

We have after(), which appears to be the same as Bluebird’s delay.
We don’t have anything that satisfies your needs directly.

Here is the linked gist as Swift:
enum Error: ErrorType {
    case Timeout
}

func poll(signal: () -> Bool, interval: NSTimeInterval, timeout: NSTimeInterval) {
    func pollRecursive() -> Promise<Void> {
        return signal() ? Promise() : after(interval).then(pollRecursive)
    }

    return race(after(timeout).then { throw Error.Timeout }, pollRecursive())
}

Hi @mxcl great library!
This issue is helping me a lot in my quest to implement an API polling system.
I just have with this sample code. Even if it times out and catches the error, the pollRecursive and signal keeps getting called.
This is my updated version of your code.
enum PromiseError: Error {
    case Timeout
}

func poll(signal: @escaping () -> Bool, interval: TimeInterval, timeout: TimeInterval) -> Promise<Void> {
    func pollRecursive() -> Promise<Void> {
        return signal() ? Promise(value: ()) : after(interval: interval).then(execute: pollRecursive)
    }

    return race(after(interval: timeout).then { throw PromiseError.Timeout }, pollRecursive())
}

poll(signal: randomSignal, interval: 1, timeout: 2).then {
    print("done")
}.catch { error in
    print("error")
}

func randomSignal() -> Bool {
    let r = arc4random_uniform(4) == 0
    print(r ? "Complete" : "Not complete");
    return r
}

Isn't the throwing of the timeout supposed to error out and stop everything?
Thanks for your help, sorry if it's a dumb question.

Not complete        <- kept calling pollRecursive() & therefore signal()
Not complete        <- kept calling pollRecursive() & therefore signal()
Complete               <- until it completed

Sorry for adding this here. Will add this to it's own issue.
:)

Indeed, there's two separate chains, that later combined, the timeout only stops the left chain, so the right chain will continue until it errors.
Here's the fix:
enum PromiseError: Error {
    case Timeout
}

func poll(signal: @escaping () -> Bool, interval: TimeInterval, timeout: TimeInterval) -> Promise<Void> {
    let timeout = after(interval: timeout).then { throw PromiseError.Timeout }

    func pollRecursive() -> Promise<Void> {
        guard timeout.isPending else { return timeout }
        return signal() ? Promise(value: ()) : after(interval: interval).then(execute: pollRecursive)
    }

    return race(after(timeout, pollRecursive())
}

poll(signal: randomSignal, interval: 1, timeout: 2).then {
    print("done")
}.catch { error in
    print("error")
}

func randomSignal() -> Bool {
    let r = arc4random_uniform(4) == 0
    print(r ? "Complete" : "Not complete");
    return r
}


[patch] testing canonical version numbering (automatic)

Note: squash-merge makes the commit list much longer than it needs to be.


Item Stacking Enhancement

LOVE this particular mod.  However, it's not a best fit for all situations.
If a storage container doesn't have [?] (? being an integer) at the beginning of its name, it uses the default stack size defined in the config for that container.  However, if there is a [?], then the max stack size should be set to that.
The reason why is that I currently have my stack size set to 64 (Nod to minecraft), but, I don't need that kind of stack in all places, like, a cargo that contains just my food, or, a particular chest set for auto crafting items, etc.

Not planning on chaning the mod behavior.


(FACT-1361) Create networking fact acceptance test

Prior to this PR, networking facts were verified in each individual test cases in acceptance/tests/facts/.rb
This PR pulls out the networking portion of all the tests in acceptance/tests/facts into new acceptance test, acceptance/tests/facts/networking_facts.rb
This will also help to test networking facts in many more platforms in puppet-agent CI, instead of being confined to only 9 platforms in acceptance/tests/facts

CLA signed by all contributors.

looks good.  please add do/end blocks to all step's and i can merge.


iOS8 Appstore Rating URL Format

Hello! First of all, thanks for your PR! It's always nice to see some contributions. :+1:
Could you give (or link) explanations describing that the URL changed for iOS 8?
Could you review your code style (camelCase, space after comma) too please?
As well, I would rather see this for the condition checking:
    if (deviceVersion >= 8.0) {
        [...]
    }
    else if (deviceVersion >= 7.0) {
        [...]
    }

    reviewURL = [NSString stringWithFormat:iRateiOSAppStoreURLFormat, self.appStoreID];
    return [NSURL URLWithString:reviewURL];

Last, [UIDevice currentDevice].systemVersion is no longer the recommended way to check against the system version. If you can change it that's cool, otherwise I will happily do it in another PR.


Is there any way I can add multiple edges between the same set of nodes?

Please describe the feature that you want to propose

It would be great to make edges with different labels between the same pair of nodes. Something similar to the image above. If this feature is already available. Please point me to it or help me with a hint.

Hey @akshaya-venkatesh8
in a custom node you can add as many handles (ports where an edge connects to a node) if you want.
Docs: https://reactflow.dev/docs/guides/custom-nodes/
Example: https://reactflow.dev/docs/examples/nodes/custom-node/
Does this help?


[core] Mark properties as @hidden instead of @internal

Please include some reasoning behind the change. At least some of those changed locations were marked as internal very intentionally (to remove them from emitted types)


longshot and HiFi reads

To whom it may concern,
I am writing to inquire whether HiFi reads have been tested with longshot. I am testing the ability of longshot to call germline mutations with HiFi reads and I wanted to get some second option on this. In addition, I was wondering if --thread option could be added to longshot.
Best regards,
Sangjin Lee

The current version of the code has not been tested with CCS reads. It
should work in principle, but the model has no optimizations for the
technology. There's also the challenge that indel variants will not be
properly detected. With regard to a multi-threading option, we plan to
introduce an auxiliary script to process chromosomes or chunks of the
genome in parallel with different processes.
On Wed, May 29, 2019, 1:17 PM sjin09<EMAIL_ADDRESS>wrote:

To whom it may concern,
I am writing to inquire whether HiFi reads have been tested with longshot.
I am testing the ability of longshot to call germline mutations with HiFi
reads and I wanted to get some second option on this. In addition, I was
wondering if --thread option could be added to longshot.
Best regards,
Sangjin Lee
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pjedge/longshot/issues/29?email_source=notifications&email_token=ABUBV2HS4L2J6NPGTZ23RPTPX23DLA5CNFSM4HQO4ZR2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GWRLBAA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUBV2CKUUEZLAFI63TBSPLPX23DLANCNFSM4HQO4ZRQ
.


Proboscis monkey spec

Added a bit more detail on what exactly needs to be changed for the S3 ID part, because it's hard to keep track of all the parts involved.
Details on the IngestionService are kept on a higher level, specifics will be in the tickets.

Epic number 35 is Beluga Whale, the related PR is merged already, you should take the 36.


GitHub action to generate from OpenAPI and deploy to GitHub pages

What problem will this feature address?
As per the Fumadocs OpenAPI docs, I need to create several files and configs to generate OpenAPI docs.
Describe the solution you'd like
It would be much simpler if I could just add an action to my GitHub repository that will generate and deploy to GitHub pages whenever the openapi.yaml is updated on the main branch, without requiring any additional Fumadocs files to be created in the repository (except for the action yaml)
Describe alternatives you've considered

Additional context
No response

You can always write a script that generates document based on your OpenAPI schema and use a command to run the script in your GitHub action:
node ./scripts/gen-docs.mjs

I know I can do that. The aim of this feature request was to simplify deployment to GH pages (or another deployment environment) by moving all configurations to a single Action. This would eliminate the need to set up another Node.js project inside your existing Node.js project or install dev dependencies that will only be used in a deployment Action.

I think adding a dependency to the project and write a script isn't a problem. It also keeps the max flexibility since you cannot pass functions to configurations of an action. Most importantly, you still need to install Node.js and Fumadocs OpenAPI to generate documents, because the package itself is written in JavaScript.
I don't want to publish and maintain a GitHub Action while it doesn't have much benefits to users.


Docker base image info extraction fails with multistage COPY --from=

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: Docker@2
Environment

Server - Azure Pipelines or TFS on-premises?

If using TFS on-premises, provide the version:

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name: Ubuntu-20.04

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
The docker task tries to extract base image information in case the flag addBaseImageData is true. However, extraction is buggy. Check the following dockerfile:
FROM maven:3.8.1-openjdk-11 AS builder
WORKDIR /home/app
COPY pom.xml .
COPY src/ ./src/
RUN mvn clean package
FROM openjdk:11
COPY --from=builder /home/app/target/*.jar /usr/local/lib/
EXPOSE 8080
ENTRYPOINT ["/bin/sh", "-c", "java -jar /usr/local/lib/ecommerce-process-*.jar"]
The COPY --from seems to break the extraction logic. See logs below.
When I set addBaseImageData to false, the problem disappears.
Task logs
Starting: Build image
==============================================================================
Task         : Docker
Description  : Build or push Docker images, login or logout, start or stop containers, or run a Docker command
Version      : 2.187.0
Author       : Microsoft Corporation
Help         : https://aka.ms/azpipes-docker-tsg
==============================================================================
/usr/bin/docker pull =builder /home/app/target/.jar /usr/local/lib/
/usr/bin/docker inspect =builder /home/app/target/.jar /usr/local/lib/
...
Step 19/19 : LABEL image.base.ref.name==builder /home/app/target/*.jar /usr/local/lib/
---> Running in 5fca996bb04f
Removing intermediate container 5fca996bb04f
---> 8c53e6b30ccf
Successfully built 8c53e6b30ccf
Successfully tagged ecommerce:1.210610.04-pipeline-docker
Finishing: Build image
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
[Insert error from the logs here for a quick overview]

Duplicate of https://github.com/microsoft/azure-pipelines-tasks/issues/14931


Respect DockPanelSplit Percent, even when child DockPanelTabs is empty

Imagine your DockPanelHost is configured like this:
    <DockPanelHost HostLayoutInited="OnHostLayoutInited">
        <DefaultLayout>
            <DockPanelSplit Percent="0.20">
                <DockPanel ComponentType="typeof(IndexPage)" />
                <DockPanelTabs CurrentTabIndex="0">
                        <DockPanel Id="tab0" />
                        <DockPanel Id="tab1" />
                </DockPanelTabs>
            </DockPanelSplit>
        </DefaultLayout>
    </DockPanelHost>

The above renders as expected, no issues.
But if the DockPanelTabs container is empty, then the "0.20" DockPanel expands to full width. In this scenario, would it be possible for the "0.20" DockPanel to continue occupying 20% width, even when the DockPanelTabs is empty?

if DockPanelTabs is planned to be used as described in my answer here #2, then the issue with an empty number of tabs will also be solved together with task.

Ah, makes sense. Then I guess we can just close this.

until this is implemented, it's better to keep the issue opened.


[Draft] Support Collect-like Reduction Aggregations

Current PR is the last piece of the puzzle of the collect aggregations supporting, which enables collect aggregations under reduction context in spark-rapids.
Signed-off-by: sperlingxx<EMAIL_ADDRESS>

This PR is supposed to be shipped in release 22.06

Can we put this in draft until branch-22.06 is available?

Good idea! It's done.

This one can now target 22.06

build

build


User is unable to generate WiFi QR codes

Need to add option for specifying bssid and password via flags to this program
More info: https://qrcode.tec-it.com/en/Wifi

bad idea, QR password strings can be generated via bash/python script, then piped into qr
no need to bloat this tool, it does what it needs to well

On the other hand, having a guided "master" for generating textual WiFi QR codes could save a ton of time


Taskbar item text truncated with text in window title bar

Please complete the following information:

OS or distro: Windows 10 x64
neoray version: 0.1.0

Describe the bug
With set title, when window title length exceeds window title bar width but does not exceed taskbar item width, window title truncation also gets applied to taskbar item. Unsure if this is not a Neovim issue, because NeovimQt behaves exactly the same way.
I have not seen any other Windows program do this (e.g. Notepad in demonstration below does not truncate taskbar item) and consider it not very user friendly.
Demonstration:
https://user-images.githubusercontent.com/52537705/134234430-7cc90b0a-b40e-49aa-985e-51404b9fdda1.mp4
Taskbar item probably should not get truncated with window title.
Steps to reproduce
Use set title. Resize window so title gets truncated.
Your neoray config in init.vim (Only the part that concerns neoray!)
set title

Crash log if available (may be generated by neoray after crash)
N/A
Debug output (you can generate starting neoray with --verbose option)
N/A

I tried every possibilities but can't found what's causing the issue. This must be a windowing library bug/feature but I can not fix because I don't know the windows api enough. If we can't find someone who can solve it then I will open an issue on glfw. Thanks for information.

I was wrong. The neovim functions truncates title. You can set your title string to constant string and look the difference. This issue may be solved via some vimscript but I don't know how.

Ah, OK. Thank you for looking into this.
Then a very rudimentary alternative to default behavior would be something like this
function! GetTitle()
  if expand('%:p') == ""
    let &titlestring = "[No name] - NVIM"
  else
    let &titlestring = expand('%:p') . ' - NVIM'
  endif
endfunction

set title
au bufenter * call GetTitle()

Even better, I found that non-truncated length of default title can be extended with set titlelen=
https://neovim.io/doc/user/options.html#'titlelen'
https://user-images.githubusercontent.com/52537705/134549135-f3232bf9-45bc-4602-b372-d15ec4559ecc.mp4
I guess this can be closed then.


OS X: Capturing input on a child process

Hello,
I have created a MAUI app for Mac catalyst and used SharpHook to capture keyboard input. When I compile and run the app using the two commands below, capturing input works as expected.
dotnet build ./MAUI\ DEMO/MAUI\ DEMO.csproj -f net7.0-maccatalyst
./MAUI\ DEMO/bin/Debug/net7.0-maccatalyst/maccatalyst-arm64/MAUI\ DEMO.app/Contents/MacOS/MAUI\ DEMO

But building and running the application with different methods do seem to cause issues. Keyboard input is not captured when running the command below. Using Rider to compile and run gives issues as well. Moreover, when I run the application by starting a process in a different MAUI app, the input won't be captured as well.
dotnet build -t:Run ./MAUI\ DEMO/MAUI\ DEMO.csproj -f net7.0-maccatalyst

My assumption is that SharpHook won't capture input if the application in is used in, is a child process. I haven't tested this problem on other OSes, so this problem could as well occur on Linux or Windows.

Hi! Thanks for posting this issue!
The first thing that comes to mind is whether you've enabled access to accessibility APIs for the correct applications. As far as I remember, if you run your application from the terminal, then you need to give this access to the terminal itself. Same with Rider - I think you should give this access to Rider.
On the other hand, you'd get a HookException with AxApiDisabled and a system popup asking you to enabled it if that were the case so you'd most probably see it.
If the accessiblity API is enabled for these applications, then I will look into it as in that case it's a bug.

I tried to re-run the application from Finder several times. Everytime I run the application, the accessibility API permission request pops up, even though I already gave permission. Managed to fix it by resetting App permissions. It's very inconventient, because I have to reset the App permissions everytime I recompile, but it shows that this is not a bug in SharpHook. I hope I find a more conventient solution to solve this problem instead of having to execute a command after every successful recompilation. Because this could be tedious especially if you publish a new version of your software and the app permissions have to be reset by the end-user.

Yeah, this is weird - I don't think you have to re-enable this permission every time. But this does look like a quirk of macOS itself, and not SharpHook.


Adding a react-native image

Would you be willing to accept a PR adding a react-native docker image?

@mihirk I'd definitely be interested in seeing this! - Potentially as a variant of the android image?

@dantoml it would be a combination of a node and android image with react-native-cli pre installed.

@DanToml How do you propose going about this?
The major issue I see is the permutation and combinations of versioning between android, node and react-native-cli.
Suppose I use Android 25, Node 8.5 with RNCli 2.0.
The tag would look something like 25-0.8.5-2.0.
There is also another issue, most react project would be using Android Libraries whose compile android version is different from their project versions, which might require 2+ versions of android sdks and buildtools to be present in the image.
Proposition:
The react-native image should have all versions of Android installed, and the node and react-native cli version should be the tag.

@mihirk If possible could you please add a license to your OpenCHS/react-native-image repository so that I can reuse it until there is an official circleci image for this?
Thanks

@kpachar Done - https://github.com/OpenCHS/react-native-image/blob/master/LICENSE


PCP-3847: Update golang builder image

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

Double PR by mistake. Closing.


Changed mardown parsing to using marked instead of showdown

Chnaged to using 'marked' as suggested in one of your issues. This is for the ability to use github flavored markdown for markdown documents. Let me know if you see any issues.

This corresponds well with issue #32. @jbt, any plans to merge this into docker?


3 check for clean git

fixing vsix

@alexhoffman617 @brendoncaulkins @kentonbmax I got it super close to publishing, but having trouble with command 'ng-evergreen.angularEvergreen' not found error after packaging the extension. You can run npm run package to create the vsix file and then ctrl+shift+p to install from vsix and try to use it, you'll see the error message. I followed this guide (https://code.visualstudio.com/api/working-with-extensions/bundling-extension) to apply the webpack configuration, tried a whole bunch of things, but couldn't resolve the issue. Help would be most appreciated.

This might be it, not infront of laptop until later afternoon. https://stackoverflow.com/a/54797507

@duluca your refactor looks good!

latest-version had a dependency on package-json which used got.  got doesn't use webpack safe imports, which caused webpack complaints, and thus got was added to the webpack externals list.  This list is effectively peerDependencies, but VSCode doesn't handle those.
Long story short, the app was crashing on load, but VSCode was registering the commands listed in the package.json file.
The commit I added removes latest-version and swaps in get-latest-version which does the exact same thing, but with different dependencies that seem to be webpack safe.
I'm able to successfully install from VSIX and run the extension.

Also, bumped version to 0.5.2.


Failure in Galaxy Cloud lunch

I followed the directions at https://galaxyproject.org/cloudman/getting-started/ and received this meesage at the end.  I do not know what to do to handle this.
FAILURE
Create appliance task failed: An error occurred (Unsupported) when calling the RunInstances operation: The requested configuration is currently not supported. Please check the documentation for supported configurations.

@smilesang79 Can you let us know what settings you changed during launch? In particular, what instance type you selected, and what placement zone?  All settings that were changed from the defaults would be useful because it looks like some combination of settings in triggering the issue.

Closing due to inactivity.


LIBRETRO: Add files for CI/CD with libretro pipelines and various improvements

Looks excellent! I only have one request, but I'm not familiar with libretro and its build/run procedures so I don't know how feasible it is. If you could, please move libretro related files (such as the extra Makefiles and .git files) in the parent project folder to their own subfolder such as /libretro.
If this is not possible or requires too much work don't worry about it. I'd just like to keep things organized if I can. Thanks!

Hi, the .gitlab-ci yaml would need to be manually configured on our CI/CD side to move it outside the repo root, so we generally prefer not to and only do it on strong wish of the code owners or if it conflicts with their own CI pipelines
For the Makefiles itself its just a matter of defining a path in the yaml

Ok, no worries then. Thanks for the work you guys did!


Attacker yaw value can infinitely increase/decrease

What is the current behavior?
A player's yaw when you call player.getPosition().getYaw() can be under 0 or over 360. You can rotate and it can infinitely increase/decrease based on the direction you rotate in.
I doubt a video is needed but in case: https://zak.pink/2021/03/Frayed-Arachnid-11486.mp4
What is the expected behavior?
The rotation should roll over on itself. E.g -1 -> 359 and 365 -> 5.
What steps will reproduce the problem?
Rotate a couple times in the same direction and your yaw will either be under 0 or over 360.
What operating system and java version is being used?
Windows 10, Java 11 Azul

Those are actually the values given by the client.
I do however agree that this is not ideal.
Do you think that the angle should map between 0/360 or -180/180? The only change should be a small conversion here https://github.com/Minestom/Minestom/blob/master/src/main/java/net/minestom/server/utils/Position.java#L366 (same for pitch)

IMO it should be 0/360 but I'd like to see more input to see, especially from people that've practically used yaw before with spigot or whatever.

I think it would make sense to use the same values as in f3 on the client. Which is 180 - -179. Just make sure there is a utils function to find the true distance between 2 angles.

Fixed by #231


Applying ZapLine to continuous data

Hi, thanks for your package.
I was wondering if you suggest segmenting continuous data before applying the ZapLine algorithm. I tried to use it on a rather large chunk of data and it failed while trying to allocate an array of 32.2 TiB.
One option I can think of would be to cut my data into arbitrary segments of a few seconds each, apply ZapLine, and then piece the segments back together. What do you think?

Sorry, I supplied the data in the form n_channels x n_samples, not n_samples x n_channels.
For nremove = 1 it works well, but nremove > 1 always fails:
ValueError: Weights should be the same n_times as X.

I was able to reproduce this error using random data:
data = np.random.rand(300,4)
denoised_data, artifacts = meegkit.dss.dss_line(data, 50., sfreq, nremove=2)

Any ideas why?

Hi @marcpabst,
Regarding your first message, happy you got it sorted. Let me know if the issue comes back in the future, as there is an easy improvement that could be done, which is to compute the covariances by blocks instead of doing it all in one go.
Thanks for reporting into that n_remove > 1 bug, I will try to look into it this week.

@marcpabst could you please check branch nb/fix-issue-8 and see if it fixes your problem?

Thanks for your quick response! The fixed branch seems to work fine. I will report back when I had to test it in more detail.

I think it works now fine, although I can't compare it to the original MATLAB implementation atm. Thanks!


Review devDependencies

Let's make sure devDependencies in package.json contains the bare minimum to build wavesurfer.js, and strip out unnecessary (build) dependencies.

Checked everything and looks ok.


Add initialization for first state in solver

dx0 should be equal to the difference between x0 constraint as defined in the problem, and x0 variable as provided in the guess.

We are already properly handling the initial conditions in both solvers. Therefore, we don't need this PR (and most probably there is an error).
See my explanation here: https://github.com/loco-3d/crocoddyl/issues/728#issuecomment-605958175
Let me know if you agree.

Ok, I agree with that: I didn't see that initial condition was already handled.


Apply Categories to Named Individuals

How should categories be applied to named individuals. Is there a better method than the current simple scheme?

in this example:
http://rosie.crbs.ucsd.edu:9000/scigraph/dynamic/genotypes_from_gene.json?gene_id=ZDB-GENE-070117-1709
the sequence alteration ZDB-ALT-061101-19 is a named individual that doesn't have a category.  i am not even sure we want to give it a category.
same for ZDB-GENO-061101-19, it is also a named individual (i believe of type intrinsic_genotype).
since all individuals are typed as some class in the graph, should we just expose the actual type that the thing is, without having to "categorize" it?  and then if it is an instance of a node that is categorized, it would include that.  for example, by adding a "type" to the node, like:

{
  "id": "ZDB-GENO-061101-19",
  "lbl": "unm_t31392t31392",
  "type" : "GENO:0000000"              # added "type" here
  "meta":
    {
        "category": [ ]
    }
}

in this example, intrinsic_genotypes are not categorized, so it wouldn't have one.  but in the case of a gene, it might have:

{
    "id": "ZDB-GENE-070117-1709",
    "lbl": "unm_t31392",
    "type" : "SO:0001217"        #where the term is the specific type: protein_coding_gene
    "meta":
      {
        "category":  [ "gene" ]
      }
},


Routing Logins to Correct Dashboard

As a user, I want to be redirected to the correct dashboards when I log in.
Story points: 3
Priority: 1 (high)
Risk: Low
Child of: #13
Tasks:
Create a frontend interface for dashboard (no design) (1 point)
Create a backend API endpoint for forwarding user information to the correct operations (2 point)

For Admin and Managers right now. QA and Worker type users will be for future iterations .


Keyword-argument failure

Thank you for MacroTools, it's super useful! I think I found a bug
let ex = :(function foo(a, b, c; kw=1) a+b+c end)
    @capture(ex, function foo(arg1_, args__) body__ end)
    arg1
end
> :($(Expr(:parameters, :(x=1))))

It works fine without kw=1

Hmm, it works if the pattern is function foo(arg1_, args__; kwargs__), so maybe the issue is just that @capture should return false in the above example, but doesn't?

Glad you like it!
So the problem here is that the keyword arguments are becoming a kind of implicit first argument in the AST:
:(f(a,b,c; kw=1)) == :(f($(Expr(:parameters, :(kw=1)), a, b, c))

so when you try to match on the first argument, you get the :parameters expression. Unfortunately, it's the Julia AST format that's weird here – MacroTools' behaviour is completely consistent with how it operates elsewhere, which means it's tricky to fix with out making a weird special case out of it.
I'm open to suggestions, but the best thing might just be to be aware of it, and to match on keyword args explicitly where needed.

That makes sense. I'll code around it, I wouldn't want to add an ugly special-case!


Fix shutdown when ngx_pagespeed is completely disabled.

Fixes https://github.com/pagespeed/ngx_pagespeed/issues/1120

lgtm


[Feature Request] [Spark] Register Vacuum in the table history Delta 2.2

Feature request
Which Delta project/connector is this regarding?

[X] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[ ] Other (fill in here)

Overview
It is explained here: #868
PR to 2.3.0+: https://github.com/delta-io/delta/pull/1552
Cherrypick to 2.0: https://github.com/delta-io/delta/pull/1562
It is missing cherrypick to 2.1 and 2.2.
Motivation
We probably forgot to cherrypick to 2.1 and 2.2
Further details
Willingness to contribute
The Delta Lake Community encourages new feature contributions. Would you or another member of your organization be willing to contribute an implementation of this feature?

[ ] Yes. I can contribute this feature independently.
[ ] Yes. I would be willing to contribute this feature with guidance from the Delta Lake community.
[X] No. I cannot contribute this feature at this time.

I would be willing to contribute this feature with guidance from the Delta Lake community.
I assume this would be to cherry-pick 2.3.0 or 2.0 commit to 2.2
This would be my first PR. I tried creating a Draft PR but it says I need Collaborator permissions.

@saikrishnapujari102087 you should be able to open a PR without any special permissions. When does it say this?

@saikrishnapujari102087, you need to create your own fork, in case you are trying to push a new branch to delta-io/delta.
@allisonport-db, before starting this work, I'd like to confirm if this is something we should do, as it needs time from somebody to review it


nightly support?

Hello,
I'm curious if you would be willing to make some updates to support nightly?
unresolved import std::iter::AdditiveIterator
unresolved import std::num::SignedInt
Thanks!

This looks fixed?


Added Domain hwg-lu.de for Hochschule für Wirtschaft und Gesellschaft Ludwigshafen

Institution/School Name
{Hochschule für Wirtschaft und Gesellschaft Ludwigshafen}
Instiution/School Website
{https://hwg-lu.de}
IT Related Bachelor's Degree Course Website
{https://www.hwg-lu.de/bachelor-wirtschaftsinformatik.html}
IT Related Master's Degree Course Website
{https://www.hwg-lu.de/master-wirtschaftsinformatik.html}
Email Users
Please place an x between the square brackets if yes

[x] Students
[x] Academic/Teaching Staff
[x] Other/Support Staff
[] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[x] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

Fixing conflicts

Fixing conflicts


Intermittent CONNECTION_ID_LIMIT_ERROR with git master

On git master, I intermittently run into a CONNECTION_ID_LIMIT_ERROR which causes my tests to fail every few thousand iterations.  I am not sure if this is a regression in my code or in quinn-proto.

This error was introduced in 0bfd15a483d06b316119e4d0cc4b8761e934271f and almost certainly indicates that quinn is sending too many NEW_CONNECTION_ID frames due to a bug. I'll investigate tonight.

Phew!!!!  Glad to know that it is not a regression in my own code before I spend hours trying to track it down.

As always, a trace log may be helpful for diagnosing this.

Still seems to happen even after #671.  Sadly, enabling trace logs seems to make the problem go away.

Thanks for confirming!

Got a reproduction: https://gist.github.com/DemiMarie-parity/c83746d3f95f94861a446207f58e0752

Thanks. There's logs for a large number of separate connections there. Could you add tracing spans to your code which distingiush them?

Here is a gist with spans: https://gist.github.com/DemiMarie-parity/668727454cef081b34ebc876f669428f

Yeah, I see the error, but there's no way to distinguish log messages from all the different connections operating concurrently, or even the two halves of a single connection. I don't see any new spans in the second gist. To add a span, you need to do something like quinn does in its driver future.

I forgot to provide an ID for the span, sorry.  I am not sure how to distinguish the two halves of one connection, though.

@alecmocatta's repro in #650 was able to reproduce this consistently. This revealed a fundamental error in our approach to storing connection IDs that had historically been masked by formerly loose validity requirements in the draft. I've prepared a fix in #674.


Ltr 3.22

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Updates the LTR branch to follow the 3.22.x version
Bumps version to 3.22.13
Try to pin main dependencies to use same major versions as 3.22.11 (If possible they should be pinned for all the patches of a specific LTR version)

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@gillins I wonder why Python 3.11 wasn't considered in here. Is it normal?

On windows if fails on test while looking for a PROJ_DATA env variable. In the code, I can only find setting PROJ_Lib directly. Not sure if it's the recipe that is outdated or the test.

What version of proj are you using? It's PROJ_DATA in the latest but PROJ_LIB in earlier versions

What version of proj are you using? It's PROJ_DATA in the latest but PROJ_LIB in earlier versions

It's pinned to 9.0. It worked for 3.22.11.

In the post_link.bat we still only have this. No mention to PROJ_DATA
set "PROJ_LIB=%LIBRARY_PREFIX%\share\proj"

Also seems that there are some patches leftovers in this branch, so I should compare it with main.
BTW, we seem to have a main and a master branch, should we eliminate master?

This is the change that introduced PROJ_DATA (https://github.com/conda-forge/proj.4-feedstock/commit/e878f9a4bcd02af30b6bbf781441a535f0c24dba). Didn't realize we also had this in post_link.bat, that definitely didn't get set with latest pinnings, not sure what's going on...
Yes we can eliminate master, very confusing!

BTW post-link.bat only run once after installation (https://docs.conda.io/projects/conda-build/en/latest/resources/link-scripts.html) to run qgis with the --postinstall option (not sure what this does). So it doesn't set these variables permanently. We should probably check that this step is still required and update these vars....

But, testing is done with an installation, isn't it?

AFAICT the variables on post-link.bat are only set while that script is running (does qgis --postinstall need these set?). I don't think that are 'sourced' into the current session like activate.bat etc.
Anyway, post-link.bat only run once (right after install) where everything else is set every time the environment is activated (like when the test environment is created).

Starting over...


Getting input without whitespaces at the beginning and end

Needs fixing for parameter fileName

morseTranslator(fileName="control1.txt",writeToFile="control2.txt",fromMorse=True)    #works file contains "... --- ..."
Not works when "... --- ... " (space at end)

Done.


Maya: fix validate frame range on review attached to other instances

Changelog Description
Fixes situation where frame range validator can't be turned off on models if they are attached to reviewable camera in Maya.
Testing notes:

Create model publish instance in maya
create review publish instance in maya
parent model maya set under the review maya set based on this https://ayon.ynput.io/docs/artist_hosts_maya#publishing-model-with-review
turn off validate frame range on the review in the publisher GUI.
Everything should go through correctly and you should end up with review attached to the model Product.

For testing this we also need to remove the model family from project_settings/maya/publish/ValidateFrameRange/exclude_families, which is quite confusing given the label of the settings as well.
I've suggested a code fix for this PR here


Process stuck when tried to install phpcs

Output
Downloading https://squizlabs.github.io/PHP_CodeSniffer/phpcs.phar via curl extension
[================================================================= ] 495.95/507.87KB 98%

Expected Result
Downloading https://squizlabs.github.io/PHP_CodeSniffer/phpcs.phar via curl extension
[================================================================= ] 507.87/507.87KB 100%
Changing permissions to 0755
Downloaded at /home/william/.phpbrew/bin/phpcs

Command
phpbrew --debug install php-7.0.10 +default +fpm +pgsql

Build Log
https://gist.github.com/williamn/027f15fd25a80046750822a902dea473
Platform
Elementary OS 0.4 Loki (Built on "Ubuntu 16.04 LTS")
Running PHP:
PHP 7.0.10 (cli) (built: Sep 13 2016 17:04:40) ( NTS )
Copyright (c) 1997-2016 The PHP Group
Zend Engine v3.0.0, Copyright (c) 1998-2016 Zend Technologies
Installing PHP:

php-7.0.10
php-5.4.45
php-5.3.29

@williamn could you try reproducing it again? If it's reproduced, could you try using curl or wget in CLI and see if it's not a network problem?

@morozov I can confirm it is now working as expected in phpbrew 1.22.8


Usergroup minor fix

User group edit btn open to UserGroup funder and admin only

@teamdraftbox,
please don't use global store for this as well as don;t use SSR hook for this as well use memoized request call on edit button component once page is fully loaded,
the way it is done now will make initial page render even slower and this functionality is not required to be server side rendered


remove old atomicAdd from gpu_stimulus

The gpu stimulus implementation still uses the old "pre pascal" atomicAdd(double) software implementation:
https://github.com/eth-cscs/nestmc-proto/blob/master/src/backends/stimulus_gpu.hpp#L17
Replace with the new wrappers that choose the appropriate hardware or software implementation at compile time.

fixed in #208


Create discussion

Refer to Wikimedia Universal Code of Conduct

Done. Code of Conduct is separate from discussion.


Add option to disregard the flyway_schema_history table

I perform the following tasks from Gradle in this order:

flywayClean
flywayLoadSchemaAndSeed -- this is a migrate-like task that loads our baseline schema (db/init/V0-schema.sql) and seed (db/init/V1-seed.sql)

task<FlywayMigrateTask>("flywayLoadSchemaAndSeed") {
  locations = arrayOf("filesystem:${project.projectDir}/db/init")
}

runsqlFixSchemaHistory -- this drops, creates and fills the flyway_schema_history table so it lines up with the schema loaded in by previous task
flywayMigrate -- this runs our migrations from the last time we baselined (the flyway_schema_history table contains the right info so flyway knows what to do, and what not to do)

Flyway is smarter than Runsql which does not properly handle DELIMITER setting for instance (making it impossible to use Runsql for loading e.g. triggers from a dump).
I'd like to exclusively use Flyway to accomplish this, but I cannot get it to totally disregard the flyway_schema_history table. An option to have it disregard it would make, IMHO, flyway more flexible: it can then also be used as a simple script runner.
Say I could do this:
task<FlywayMigrateTask>("flywayLoadSchemaAndSeed") {
  locations = arrayOf("filesystem:${project.projectDir}/db/init")
  disregardSchemaHistory = true
}

Then the flyway_schema_history table dropping/creating/filling can be part of our db/init/V0-schema.sql file, after which the task sequence becomes like this:

flywayClean
flywayLoadSchemaAndSeed
flywayMigrate

And I do not need an additional plugin.
Thanks for considering. If you think this feature is something you'd like to add to the project, I can look into preparing a PR. Maybe such an option already exists, and I just did not discover it.

Thanks for the suggestion. We probably won't be implementing this feature so closing for now, however we will consider it in the future if there is sufficient further interest.

Ok, fair enough.


Upgrade fabric8 and archetypes version

Ah sorry I'd only run the release on the fabric8 project, had I done a full release the pipeline would have updated these deps.  I'll kick off a proper release so the other projects are updated.

@rawlingsj ah okay then I can just close this PR? And then let the release do the updates?

It wont matter as the release will just skip the step if the project already has the latest deps.  We can just leave this one.  Figured I'd say incase you saw other projects and I didn't want to cause you extra work.


Durations greater than 4 quarter notes not supported

While there is no support for dotted whole notes, dotted dotted whole notes, etc.
More generally support to override the string based duration specification and opt for purely a numerical value of quarter notes does not seem to be supported.

Hi @mitchkm,
Thanks for your message.  Here are a couple solutions for achieving durations longer than whole:
Passing an array of note durations will give you a duration equal to the sum of those durations.  Here's how you could get a dotted whole note:
const note = new MidiWriter.NoteEvent({pitch:['C4'], duration: ['1',  '2']});

You can use the tick based duration option to achieve notes longer than whole.  The library is setup to use 128 ticks per beat.  So in 4/4 that's 512 ticks per whole note.  Something like this would give you a dotted whole note:
const note = new MidiWriter.NoteEvent({pitch:['C4'], duration: 'T768'});

That does seem nice. But I will say for it to make the most sense in music theory it would be nice to provide a numeric quantity of a quarter note perhaps. ie. 10 quarter notes. I guess I can make an char. array of X quarter notes('4') but it feels a little less intuitive. Especially if arrays get changed to providing the duration for a multitude of pitches also provided in an array.

Hi @mitchkm,
Thanks, could you post an example syntax that you would prefer?
-Garrett

Following the way you specify ticks, perhaps:
const note = new MidiWriter.NoteEvent({pitch:['C4'], duration: 'Q10'});
Meaning a duration of 10 quarter notes

Hi @mitchkm,
I can see your logic with this, but I don't think it's worth implementing this way.  It introduces a new prefix Q, and presumably every prefix for all other note types; E for eighth, S for sixteenth, etc. which can get out of hand quickly and not particularly intuitive.
In it's current implementation, using an array of durations to be summed seems much more friendly and flexible from a programmatic standpoint in my opinion.  For instance, how would you create a duration of two whole notes and an eighth?
For your case, creating an array of ten "4" is fairly trivial:
const note = new MidiWriter.NoteEvent({pitch:['C4'], duration: new Array(10).fill('4')});

Appreciate your input.
-Garrett


create monitoring UI

User story
As a wis2box administrator, I want to have access to a dedicated admin user interface, so that I can monitor wis2box and take corrective action as required.
Acceptance criteria

[ ] monitoring tab removed from wis2box-ui
[ ] new container wis2box-ui-admin implemented

[ ] single page with links to:

[ ] service health (GeoHealthCheck?)

[ ] API (GHC Resource type OGC API - Features)
[ ] MQTT (needs implementation)
[ ] public storage (GHC Resource type WWW:LINK)

[ ] metrics (grafana)
[ ] container management (portainer?)

Definition of done

[ ] unit/functional tests added/updated
[ ] documentation updated
[ ] live demonstration provided

Metrics should provide an HTTP endpoint for metrics (/metrics) (OpenMetrics).

2023-01-17: @sabersd to provide feedback/comment on feasibility for custom UI for container management.


Release 1.0.0

Features

Added optional parameter 'force' in usergrants.request (186]
Added scope parameter for advertisingId (#188)
Added Play intents (#151)

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


Loss not converging for Cerebras-GPT-111M with HH dataset

I tried training with SFT loss on the Cerebras-GPT-111M model using HH dataset. I opted for the BasicTrainer. However, the loss didn't converge even after 30k steps, it wobbles within the same region. I tried learning rates from 0.1 up to 1e-6, all resulting in the same observation with the loss wobbling in different ranges.
Here's the command I used:
python -u train.py model=Cerebras-GPT-111M datasets=[hh] loss=sft exp_name=anthropic_sft_Cerebras-GPT-111M gradient_accumulation_steps=2 batch_size=16 eval_batch_size=16 trainer=FSDPTrainer sample_during_eval=false
Here goes the graph for the loss:
WanDB Log for Cerebras-GPT-111M
Note: I experimented with Pythia-2.8b as well, using the same configuration that you have provided in the readme. I tried both FSDPTrainer and BasicTrainer, just to be sure that BasicTrainer isn't causing this. However, when trying Pythia-2.8b model, I had to incorporate QLoRA since the free resources weren't sufficient. Finally I tried FSDPTrainer for the Cerebras-GPT-111M model. These all resulted in the same observation.
Graph of the Loss for Pythia-2.8b training for very few steps: WanDB Log for Pythia-2.8B
Any clues why this is happening?

After looking at your learning curves, this actually looks fine to me. It's normal for the loss to only go down a little bit during SFT for HH (see our Pythia HH demo run here, pic attached below). If you turn on smoothing, it should be clear that the train loss is going down. I'd also expect you to see some clear drop in eval loss.

We originally had a small bug that made it look like the loss was going down more dramatically than it really was during SFT, but this is fixed now. After doing SFT, you should definitely see a loss drop for the DPO phase (see the DPO demo run here), pic attached below:

I'm closing this issue because your results actually look as expected, but please don't hesitate to re-open if you don't feel the issue is resolved.

@eric-mitchell
Hello.
I already have an SFT model, so I only conducted DPO training.
The train loss doesn't seem to converge as shown below.
Is the normal situation correct?

@yunsangju thanks for the question! Can you please open a new issue where we can discuss your situation? I don't think your issue is actually related to this one (as it deals with SFT, but your issue is with the DPO stage).

@eric-mitchell Thanks for the quick response, I actually didn't want to respond until I got the final results. However, it seems it might take a while.
However, I'm affirmative that what you claimed is right and it makes sense according to the results that I've obtained so far. My concern was primarily based on the early results that it gave, with there's no direct evidence that the loss was converging. However, the general trend has been observed and the evaluation results proves it.
I'll post once again once I do have the final results. Thanks


client lib request_client has async co-routine calls not awaited

The following two calls to _request_prep should be awaited.
https://github.com/NOAA-OWP/DMOD/blob/98f8318809a68ba35e30c14a395586421672f09f/python/lib/client/dmod/client/request_clients.py#L344
https://github.com/NOAA-OWP/DMOD/blob/549aa4643fac4f36d4ea2f2efb29afd983195191/python/lib/client/dmod/client/request_clients.py#L378

Thanks for reporting this, @hellkite500! Just opened #552 to fix this.


issue with deck apply while using kong with DB

Hi,
I installed kong ingress on my local kubernetes using official helm chart with postgres database.

PS D:\projects\kong> helm list -n kong
NAME            NAMESPACE       REVISION        UPDATED                                 STATUS          CHART           APP VERSION
kong-1657894767 kong            1               2022-07-15 18:19:28.493225 +0400 +04    deployed        kong-2.11.0     2.8
When I do deck ping:
PS D:\projects\kong> deck --tls-skip-verify --kong-addr https://<IP_ADDRESS>:8444 ping
Successfully connected to Kong!
Kong version:  2.8.1

But when try sync config I got the following error:

PS D:\projects\kong> deck --tls-skip-verify --kong-addr https://<IP_ADDRESS>:8444 sync
creating service nginx-service
Summary:
Created: 0
Updated: 0
Deleted: 0
Error: 1 errors occurred:
while processing event: {Create} service nginx-service failed: HTTP status 405 (message: "cannot create or update 'services' >entities when not using a database")

But actually my kong run with DB

curl -s -k https://<IP_ADDRESS>:8444| jq .configuration.pg_host                                    "kong-1657894767-postgresql"

pg_host does not itself determine the database, it's the database host if the database is set to Postgres.  It works! I guess that part need to be defined more clearly in helm chart docs.
Thanks @rainest and @GGabriele for help. Have a nice day


Clarified plugin table renaming actions

A couple of people have been confused by this in email/IRC.
I hope that the revised wording is accurate and clearer.
@yakky @mkoistinen your opinions?

@evildmp :+1:  (test failures due to current broken develop on Django 1.8, I'm addressing this later)


Edge-aware 3DGS Optimization

As expressed in the article above，How do you represent the fringe area?
If I use the Canny algorithm to extract edge features, then just a few pixels with only the edges will increase the weight when calculating the photometric loss, does this really work? I don't seem to have any effect on using this idea in my work, am I misunderstood?

Hi! As mentioned in our paper, the method I used is from 'Pixel Difference Networks for Efficient Edge Detection.' You can refer to this repo.

Ll1 = l1_loss(edge_weight * image, edge_weight * gt_image)
Thanks so much ，I use the same way.In my test, weighting the photometric loss does not restore the edge details, have you done a comparative experiment with this strategy, and can you compare the weighted and unweighted separately?

Our edge prior is primarily used to increase the density of rays sampled in the edge regions to enhance the details of the geometric reconstruction. Additionally, we use the edge prior to improve the rendering quality in detailed regions. The ablation experiment results presented in Table 3 show an improvement in rendering quality.

Thanks a lot！


Make rounded corners for rectangles

Let's make pretty UI with rounded corners for rectangles, buttons, etc.
(possibly https://github.com/bevyengine/bevy/pull/3991 have to be released first)

why not try other bevy ui plugin like bevy_egui or belly

When I first started using Rust, I looked into using egui to add a handler for a whole table row click. However, I wasn't satisfied with the API. While the belly library is interesting, it doesn't seem to offer much for this particular project. Additionally, I don't believe that belly supports rounded corners.
While belly may not offer a solution for all of the requirements of this project, I am still open to considering it if it can bring something useful to the table that can't be done easily with raw bevy-ui. That being said, the main limitation of using raw bevy-ui for this project is not so much the lack of rounded corners, but rather the absence of editable/selectable text input.
I've spoken with bevy-ui contributors on Discord, and they have informed me that migration to cosmic-text will enable editable text input for bevy-ui. The migration is expected to occur in the upcoming 0.11 release. My goal for this project is to have rich text editing with support for markdown/org syntax in rectangles, and potentially even executable code (literate programming) in the future.

This can be re-opened if someone wants to work on it.
This is not priority anymore, we can wait until bevy has rounded corners support.


rename nimiBook to nimibook

this is a small thing but it kind of bothers me. the name was nimiBook as a calque of mdBook, but then I do not like that I have nimiBook.nimble files, hence the renaming. I guess this might be a bit bothering since we will have to change the remoteUrl of git repo (not sure what happens to the forks), so I figure I ask before proceeding. Thoughts @Clonkk, @HugoGranstrom?

Looks good to me

Go ahead! 👌

done!


first version of continueOnStepFailure configuration

Description
Note that I ended up naming the configuration 'continueOnStepFailure' instead of 'ignoreStepFailure'. The rationale is in ScenarioRuntimeTest.testcontinueOnStepFailure5() and basically is that ignore would open the door for using it as an 'if' condition, continuing still fails the test but only after the step of steps 'wrapped' with the configuration is done.
First PR for review & comments.

Relevant Issues : https://github.com/intuit/karate/issues/1446
Relevant PRs : (optional)
Type of change :

[x] New feature
[ ] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

still need to add several tests (especially called features) but base concepts ready for a first pass and review @ptrthomas

still need to add several tests (especially called features) but base concepts ready for a first pass and review @ptrthomas

All comments addressed except moving the location of the match from inside the Step class. See below how this will look in the HTML report - all failures will include the corresponding log.

All comments addressed except moving the location of the match from inside the Step class. See below how this will look in the HTML report - all failures will include the corresponding log.

@ptrthomas moved the matchingMethod to the StepResult class. Had to use reflection to be able to recreate the Result from the JSON - our alternative is to just ignore that property but this way things are consistent (and who knows if it might not come handy to re-execute).
https://github.com/intuit/karate/pull/1447/commits/1e7444e17e85a9c9474f290ffc84301c064071e7
Let me know if you're ok with this solution before I update the failing unit test (which tests conversion from an existing JSON file in the unit tests that doesn't have that property).

@ptrthomas moved the matchingMethod to the StepResult class. Had to use reflection to be able to recreate the Result from the JSON - our alternative is to just ignore that property but this way things are consistent (and who knows if it might not come handy to re-execute).
https://github.com/intuit/karate/pull/1447/commits/1e7444e17e85a9c9474f290ffc84301c064071e7
Let me know if you're ok with this solution before I update the failing unit test (which tests conversion from an existing JSON file in the unit tests that doesn't have that property).

Let me know if you're ok with this solution before I update the failing unit test (which tests conversion from an existing JSON file in the unit tests that doesn't have that property).

I think it is ok. I can have a look and tweak over your final version if needed. yes, being able to re-hydrate can be useful in future

Let me know if you're ok with this solution before I update the failing unit test (which tests conversion from an existing JSON file in the unit tests that doesn't have that property).

I think it is ok. I can have a look and tweak over your final version if needed. yes, being able to re-hydrate can be useful in future


ENH: Read/write state files in CESM driver

Related to the issues described in #457, #460 has temporarily disabled reading/writing state files in the CESM driver. Once #457 has been addressed, we should re-enable these features and test the that rpointer.lnd file is written/read properly.

closed via #556


Examples to demonstrate changing a sprite and texture

How can Bevy's documentation be improved?
Newcomers are often a bit lost when trying to learn how to idiomatically work with assets and handles.
The most common stumbling block is "how do I change a sprite once loaded?".
We should add two sibling examples, one for sprites, and one for meshes:

show how to query for sprite / mesh handles, and change what they point to
show how to programatically change the properties of a sprite / mesh (e.g. color, reflectance, translucency or so on)
show how to mutate the underlying asset to e.g. add more vertexes

This looks like good learning material for me, I'll pick this up.

Awesome. https://github.com/bevyengine/bevy/pull/15056 gets into the strategies a little bit, but it would be very nice to have a hands-on example or two.

This is possibly an obvious question, but do we mean an example for Mesh2d or Mesh here?

Either is fine: I think Mesh is more common.


re-add akka-more, akka-http

wip

test run queued: https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/988 (404 til Jenkins catches up)
this will probably progress slowly until 2.12.5 is out, but let's get the ball rolling

[akka-more] [error] Error during tests:
[akka-more] [error] 	akka.persistence.LeveldbPersistentActorSpec
[akka-more] [error] 	akka.persistence.InmemPersistentActorSpec

@2m ?

might as well attempt to re-add at the same time: twirl, play-doc, play-json, play-ws (all were published for 2.13.0-M3 recently, so very likely they will work here too)

new test run queued: https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/990 (404 til Jenkins finishes up his phone call with Rex Tillerson)

this is good:
[info] Project play-doc--------------------: SUCCESS (project rebuilt ok)
[info] Project play-json-------------------: SUCCESS (project rebuilt ok)
[info] Project twirl-----------------------: SUCCESS (project rebuilt ok)

but:
[akka-more] [error] Failed tests:
[akka-more] [error] 	akka.stream.tck.EmptyProcessorTest

intermittent? let's try again: new test run queued: https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/990 (404 til Jenkins finishes counting how many times he can rub the cat's fur backwards before it bites him)

The akka.stream.tck.EmptyProcessorTest is intermittent. Already tracked in https://github.com/akka/akka/issues/24581

we can try again after https://github.com/scalamacros/paradise/issues/114 is merged

judging from https://github.com/playframework/playframework/issues/7940 I should be able to add all of Play back in now

merging new collections took us back to square one, so it'll be a while before this is actionable again, so I'm just going to close it.

Akka ticket is https://github.com/akka/akka/issues/24535