Running dragon as an executor from within a larger python program

I would like to use dragon as within the context of a larger piece of python software (Cubed - see https://github.com/cubed-dev/cubed/issues/467). In particular I want to write an equivalent to Cubed's ProcessesExecutor (or ThreadsExecutor) but which uses Dragon as the concurrent_executor.
All this executor needs to do is execute a series of stages each made up of a number of embarrasingly parallel tasks (each of which are python functions). I just want Dragon to launch the tasks in parallel for me across a whole HPC allocation.
I'm looking through the docs and I have two main questions:
1) Should I use the dragon.workflows.parsl_executor.DragonPoolExecutor?
That seems like a drop-in replacement, but if that's actually using Parsl (which I noticed got built when I built the dragon executable) and that's all I want to use then would I be better off not bothering with Dragon and just using the parsl.executors.ThreadPoolExecutor instead? What's the difference?
2) How do I launch dragon from within the context of another python program?
All the docs examples seem to say that you use dragon to launch another python program from the command line like this
dragon my_python_script.py

, and dragon works by

replacing all standard Multiprocessing classes with Dragon equivalent classes before CPython resolves the inheritance tree.

(from Inheritance and Multiple Start Methods.)
But this is inconvenient if I can't represent my workload as a single standalone python script. Instead I ideally want to be able to call the Executor from inside a running python process on an interactive job (e.g. from within a jupyter notebook cell) and have it execute across a whole allocation.
Do I need to somehow auto-generate this script and make a subprocess.call to the dragon executable?
Or if I try omitting the dragon executable (mentioned on this page) then I'm not sure what this implies:

The Dragon core library can still be imported via e.g. from dragon.managed_memory import MemoryPool and used. In this case, the "dragon" start method must not be set. The infrastructure will not be started.

This part:

Note that all other parts of the Dragon stack, in particular the Dragon Native API require the running Dragon infrastructure and are thus not supported without patching Multiprocessing.

seems to be saying that I can still use Dragon Core but not Dragon Native from within a python program that I didn't launch using the dragon executable. Is the dragon.workflows.parsl_executor.DragonPoolExecutor in Dragon Core or Dragon Native?

@applio you said you

got dragon to run the add-asarray.py example single node as the executor already

so I'm curious what your approach was?
cc @tomwhite

i am away through next week. Can we talk when i get back? if not we can
have you talk to another team member. sounds like we should talk
On Fri, Aug 2, 2024 at 12:33 PM Tom Nicholas @.***>
wrote:

I would like to use dragon as within the context of a larger piece of
python software (Cubed https://github.com/cubed-dev/cubed). In
particular I want to write an equivalent to Cubed's ProcessesExecutor
https://github.com/cubed-dev/cubed/blob/19f844e2c1ea799ae4bb0cb754f7aa92f917e598/cubed/runtime/executors/local.py#L266
(or ThreadsExecutor
https://github.com/cubed-dev/cubed/blob/19f844e2c1ea799ae4bb0cb754f7aa92f917e598/cubed/runtime/executors/local.py#L227)
but which uses Dragon as the concurrent_executor
https://github.com/cubed-dev/cubed/blob/19f844e2c1ea799ae4bb0cb754f7aa92f917e598/cubed/runtime/executors/local.py#L176
.
All this executor needs to do is execute a series of stages each made up
of a number of embarrasingly parallel tasks (each of which are python
functions). I just want Dragon to launch the tasks in parallel for me
across a whole HPC allocation.
I'm looking through the docs and I have two main questions:
1) Should I use the dragon.workflows.parsl_executor.DragonPoolExecutor
https://dragonhpc.github.io/dragon/doc/_build/html/ref/workflows/dragon.workflows.parsl_executor.html#dragon.workflows.parsl_executor.DragonPoolExecutor?
That seems like a drop-in replacement, but if that's actually using Parsl
https://parsl-project.org/ (which I noticed got built when I built the
dragon executable) and that's all I want to use then would I be better
off not bothering with Dragon and just using the
parsl.executors.ThreadPoolExecutor
https://parsl.readthedocs.io/en/stable/stubs/parsl.executors.ThreadPoolExecutor.html#parsl-executors-threadpoolexecutor
instead? What's the difference?
2) How do I launch dragon from within the context of another python
program?
All the docs examples seem to say that you use dragon to launch another
python program from the command line like this
dragon my_python_script.py
, and dragon works by
replacing all standard Multiprocessing classes with Dragon equivalent
classes before CPython resolves the inheritance tree.
(from Inheritance and Multiple Start Methods
https://dragonhpc.github.io/dragon/doc/_build/html/pguide/dragon_multiprocessing.html#inheritance-and-multiple-start-methods
.)
But this is inconvenient if I can't represent my workload as a single
standalone python script. Instead I ideally want to be able to call the
Executor from inside a running python process on an interactive job (e.g.
from within a jupyter notebook cell) and have it execute across a whole
allocation.
Do I need to somehow auto-generate this script and make a subprocess.call
to the dragon executable?
Or if I try omitting the dragon executable (mentioned on this page
https://dragonhpc.github.io/dragon/doc/_build/html/pguide/dragon_multiprocessing.html#multiprocessing-and-dragon-without-patching)
then I'm not sure what this implies:
The Dragon core library can still be imported via e.g. from
dragon.managed_memory import MemoryPool and used. In this case, the
"dragon" start method must not be set. The infrastructure will not be
started.
This part:
Note that all other parts of the Dragon stack, in particular the Dragon
Native
https://dragonhpc.github.io/dragon/doc/_build/html/ref/native/index.html#dragon-native
API require the running Dragon infrastructure and are thus not supported
without patching Multiprocessing.
seems to be saying that I can still use Dragon Core but not Dragon Native
from within a python program that I didn't launch using the dragon
executable. Is the dragon.workflows.parsl_executor.DragonPoolExecutor
https://dragonhpc.github.io/dragon/doc/_build/html/ref/workflows/dragon.workflows.parsl_executor.html#dragon.workflows.parsl_executor.DragonPoolExecutor
in Dragon Core or Dragon Native?
@applio https://github.com/applio you said you
got dragon to run the add-asarray.py example
https://github.com/cubed-dev/cubed/blob/main/examples/add-asarray.py
single node as the executor already
so I'm curious what your approach was?
cc @tomwhite https://github.com/tomwhite
—
Reply to this email directly, view it on GitHub
https://github.com/DragonHPC/dragon/issues/20, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM7WSZDFWEPS53VNO6YBPTZPPGIXAVCNFSM6AAAAABL5AUUCGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2DKNJUGY2TQNA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@TomNicholas and I chatted Monday (yesterday) and I promised a PR for cubed with the example I got working when traveling back from the SciPy Conference in July (where Tom and I got to meet in person).
Along with my PR for cubed, I will add an example for the Dragon repo as well to showcase how to use cubed and Dragon via cubed's existing use of concurrent.futures/multiprocessing.

No worries - @applio actually came to our meeting yesterday and explained how he got around this problem by spawning the processes using dragon manually. (See the Cubed on Dragon notes in here  https://docs.google.com/document/d/1_FkLZ3NjXlzlc7p4mr1GtWuPGKN3-E991dQEdPv6bNc/edit?usp=drivesdk)
Once he shares the PR I will link it here in case anyone else in future has the same question.

I have posted 2 PRs against Cubed just now:

https://github.com/cubed-dev/cubed/pull/554
https://github.com/cubed-dev/cubed/pull/555

Discussion of those PRs can probably continue there but would it be helpful to continue on any topics here in this thread?  (We can always create a new one too.)

I think the Dragon docs could probably explain how to do this spawn trick.

from inside a running python process on an interactive job

This is also still not clear to me - I raised https://github.com/cubed-dev/cubed/issues/557 on Cubed to discuss that.


Make "Model" interface "predict" function return an NDArray.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Add arm back to recipe

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] 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.

@vgvassilev In the last commit, the part of the recipe that is in this PR was removed. In the cppinterop package files on conda forge (https://anaconda.org/conda-forge/cppinterop/files) you can see that the arm builds got dropped, and don't have the latest patches.

@vgvassilev In the last commit, the part of the recipe that is in this PR was removed. In the cppinterop package files on conda forge (https://anaconda.org/conda-forge/cppinterop/files) you can see that the arm builds got dropped, and don't have the latest patches.

I just wanted to make sure that the linter does not remove it automatically. That'd mean we need to add these builds somewhere so that the renderer does not delete them. I do not remember deleting them explicitly, although they were failing at some point for me.

@conda-forge-admin, please rerender

So the linter removes the arm builds probably because we need to put them somewhere in the configuration.

I will do some investigating to determine what needs to be added so they don't get removed.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@vgvassilev modifying the conda-forge.yml file has fixed the issue with the renderer. The renderer moved the Linux arm builds to use Travis for the CI. I squashed the commits. Can you review?
After this has been reviewed as ok and merged, I will fix the issue in the cpp-argparse and xeus-cpp feedstocks, as the issue with the renderer removing the arm jobs will exist there too.


Could you publish the trained weights?

Could you publish the weights, so that its easiert to reproduce your findings?

Thanks for your interest in our work.
We are unable to publicly release the pre-trained FSOD/gFSOD models due to their substantial storage requirements. For instance, the FSOD model consists of 6 different shots (1, 2, 3, 5, 10, 30), and each shot is associated with 10 seeds, resulting in a total of 60 models for a single FSOD method.
Nonetheless, you should be able to replicate our results using the provided code. Additionally, please feel free to reach out if you have any questions or need further discussion.


Wrong empty notices in notice page

Description
Remove unnecessary empty notice from the notice page
References

report

cannot reproduce at this moment

CommentNotice notices with unhandled target.node


chore(storagechain): :alien: update to SC:v3.1.3

#108  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @anthonyra and the rest of your teammates on  Graphite

Once this passes tests it should close #104


Type not found in document [Error]

I created the "Invoices Collection".
import { createCollection } from 'meteor/vulcan:core';
import schema from './schema.js';
import resolvers from './resolvers.js';
import mutations from './mutations.js';

const Invoices = createCollection({
  collectionName: 'Invoices',
  typeName: 'Invoice',
  schema,
  resolvers,
  mutations,
});

export default Invoices;

When I add "mutations" I'm getting the following error:

Error: Type "InvoicesInput" not found in document.

I reviewed other packages and I can't found something related with GraphQL Types.
Here two links with related information:
Link #1
Link #2
Should I define Types? Where?
I found that in "vulcan:lib"/ collections.js at line 161
GraphQLSchema.addMutation(`${mutations.new.name}(document: ${collectionName}Input) : ${typeName}`);

In "vulcan:lib"/ graphql.js at line 160 I'm getting:
type Invoice {
        _id: String
  name: String
  isVisible: Boolean
  createdAt: Date
  updatedAt: Date
  userId: String
  user: User
      }

Thanks

You can review your schema with meteor shell and then Vulcan.getGraphQLSchema(). This should show you what input types are available. InvoicesInput is supposed to be generated automatically for you, I'm not sure why it wouldn't be…

Thanks.
I'm reviewed the schema with meteor shell and Vulcan.getGraphQLSchema().
I found type "User" and type "Invoice" but nothing about "InvoicesInput".

One reason why you might not have any InvoicesInput type is if none of your fields are marked as insertable or editable? (insertableBy and editableBy)

Yes. Thanks You.
Right Now I reading...

One more thing! By default, all schema fields are locked down, so we need to specify which ones the user should be able to insert as part of a “new document” operation.
Once again, we do this through the schema. We’ll add an insertableBy property to any “insertable” field and set it to [members] to indicate that a field should be insertable by any member of the members group (in other words, regular logged-in users):

Thanks.

This is so useful! Thanks!


Feature request: Add label to many projects at once

I really like the project label feature, and I use it quite a bit:

I recently started working at a new company, which means working with more than 25+ new repositories within the last months. I want to label these projects as "Company" but doing so one-by-one is quite tedious.
Simple QOL request, it would be awesome if we could do something like:

That way, you can do it once instead of picking Project / Label and hitting save 25 times

Thanks, I see how this will be beneficial. I'll add it as a feature request with not-so-high prio.

Hello @muety I would like to work on this.

This is what I came up with

So as far as I can see, every time a new label is added in the current UI, a new ProjectLabel model is created associated with the ProjectKey. In which case the above UI might not be possible.
Correct me if I am wrong, but there does not seem to be any existing model that stores projects explicitly. They are just stored as string in the heartbeat model.

Hi @IgnisDa, thanks a lot for these very elaborate thoughts! I really like the concept you came up with. Feel free to give it a go!
To give you a couple of hints as a guideline (without having fully thought through this): you'll probably have to adapt the adtionAddLabel() handlers (sorry, the settings.go is pretty messy, I have a to-do on my list to split the settings up into separate pages) to accept an array of values for key and value, instead of only a single one. See here (and potentially here). Try to use as little JavaScript as possible, but of course as much as needed.

Correct me if I am wrong, but there does not seem to be any existing model that stores projects explicitly. They are just stored as string in the heartbeat model.

Yep, that's right, projects are simply identified by their plain name strings (same for languages, editors, etc.).

Will work on this.
Can you tell me the normal development workflow you follow? I run go run . -- --config config.yml in the project root. When I make a change to the go code, I have to restart the server and have to login again in the browser. Is there a way to circumvent this?

Yes, exactly. That's my current development workflow. If you wanted to circumvent having to re-login each time, you could probably set the securecookie's secrets to some constant string, instead of a random value.

Can you tell me which setting in this table? https://github.com/muety/wakapi#-configuration-options

There's no setting for that, you'd have to do it in code here. But I always just re-logged in every time again.

Would be useful for development if we make that configurable via an env variable.

Agree! Created #463 for this.


azure board review comments

Packages impacted by this PR
Communication-call-automation
Describe the problem that is addressed by this PR
feedback from azure board review
General

callInvite is now an interface instead of a class
rename target/s to targetParticipant/s
append EventData to events
event parser parse is now a function instead of a class
update uri to url

CallMediaUpdates

move all required params in options to method signature

CallConnectionUpdates

rename callConnectionImpl to callConnection
getParticipant takes a CommunicationIdentifier instead of a participantMri string.

CallRecordingUpdates

remove "recording from method names" ex startRecording is now start.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-communication-call-automation


Serialize zero values

Motivation and Context

Generated clients currently don't serialize the value 0 and false for members that have a default value. It seems like this logic was originally added to fix issues with S3 which are technically modeling issues and have since been fixed. Not serializing zero values has caused issues in the past, for example https://github.com/awslabs/aws-sdk-rust/issues/630, and in other SDKs that do it like Go v2: https://github.com/aws/aws-sdk-go-v2/issues/2162. Also the Smithy spec says clients should serialize default values, which generated clients currently do not do if that default is 0/false.
Description

Removes logic from RustWriter that was used to not serialize zero values, ie. 0 for numbers and false for booleans, if that was also the default value for the member. With this change, the only time the default value for a member is not serialized is if the member is also @clientOptional per Smithy spec.
The Blob type was also updated to implement std::Default, since SymbolVisitor treats "" as a RustDefault.
Important: This change impacts all the protocols and will cause many members to now be sent over the wire that previously weren't.
Testing

[x] Generated the AWS SDK and inspected the diff, looks like the expected changes, removing checks for default values in serializers.
[ ] TODO: Testing with individual clients: This is a much larger task since there are so many clients that have been changed, will likely use a subset.

Checklist

[ ] I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
[ ] I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing in favor of https://github.com/smithy-lang/smithy-rs/pull/3252, until we better test the impact of default value serialization.


MySql Connection Not Closed in docker linux

Hi
i use mysql Connection in asp.net core 2.2 and open connection Successfully but when send close connection the connection not close
i dont have any idea

You might be describing connection pooling, but without more info it's impossible to be certain. MysqlConnection is not an in-box type; you will want to file a support issue with whichever product you're using as they will have more in-depth knowledge.

Closing due to the lack of response after two weeks or more.


Add CI

Closes #1

workflows/ruby.yml looks like a duplicate of workflows/rubocop.yml
@Judahmeek
Bad mistake! Fixed.


undefined is not an object (evaluating 'gl.Jimp = Jimp')

Getting this error "undefined is not an object (evaluating 'gl.Jimp = Jimp')" while starting the app. Not even able to run the app
Expected Behavior
The app should run successfully.
Current Behavior
Error:  undefined is not an object (evaluating 'gl.Jimp = Jimp')
Failure Information (for bugs)
Steps to Reproduce

Install jimp
Run the app
You get this error: undefined is not an object (evaluating 'gl.Jimp = Jimp')

Screenshots

Context

Jimp Version: 0.6.4
Operating System: Ubuntu 16.10
Node: 10.15.3
npm: 6.4.1
react: 16.8.3
react-native: 0.59.3

Failure Logs

Strangely it works if you have the debugger enabled, but otherwise, I encounter the same problem

Any updates on this issue?

Same problem here - both on iOS and Android (using Expo).
I'm using Jimp v0.10.2, the stack trace looks slightly different (attached) but the result is the same.

Same problem here - both on iOS and Android (using Expo).
I'm using Jimp v0.10.2, the stack trace looks slightly different (attached) but the result is the same.

Thanks for reporting this bug, @Josh2941! We are currently investigating several issues with Jimp in frontend tools such as React [Native], and will consolidate the discussion there. We will therefore close this issue as duplicate, but continue to work on a fix. You can follow the progress on that in #1194.


Metadata: Align entity name properties with CLR

We have EntityType.Name and EntityType.SimpleName. Seems like it would be better just to match CLR here: FullName and Name.

We decided otherwise in the API review. Those changes will be made based on notes of the API review.


Check commit message hashes before making links

Previously, when formatting commit messages, anything
that looked like SHA1 hashes was turned into a link
using regex. This meant that certain phrases or numbers
such as 777777 or deadbeef could be recognized as a commit
even if the repository has no commit with those hashes.
This change will make it so that anything that looks
like a SHA1 hash using regex will then also be checked
to ensure that there is a commit in the repository
with that hash before making a link.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0639247). Click here to learn what that means.
The diff coverage is 55.55%.

@@            Coverage Diff            @@
##             master    #7713   +/-   ##
=========================================
  Coverage          ?   41.23%
=========================================
  Files             ?      471
  Lines             ?    63771
  Branches          ?        0
=========================================
  Hits              ?    26299
  Misses            ?    34033
  Partials          ?     3439

Impacted Files
Coverage Δ

models/repo.go
48.61% <100%> (ø)

modules/markup/html.go
80.12% <33.33%> (ø)

Continue to review full report at Codecov.

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

@lafriks Could you take another look? Turns out go-git cannot be used as it does not support short hashes. Switched to using git rev-parse which should be a lot faster then git log.

@mrsdizzie please review


[DO NOT MERGE] Republish routes for Detailed Guides if ever published

Trello:
https://trello.com/c/oku0LGNO/67-detailed-guides-which-should-be-redirecting-display-a-410-page
A data migration in Content Store on 22nd December
(https://github.com/alphagov/content-store/pull/96) deleted all routes
for Whitehall content that had been redirected or withdrawn. This was
the correct thing to do for non Detailed Guides documents as everything
else is handled by the /government prefix route. As Detailed Guides
are located at the GOV.UK root, they require individual exact routes
per document, and thus shouldn’t have been removed. This resulted in a
number of documents not being accessible to the public, some of which
have been manually fixed by 2nd Line.
Whitehall itself handles returning a Redirect or Gone status for these
documents, so all Detailed Guide documents should have a backend route
to Whitehall.
This PR simply publishes backend routes for all Detailed Guides that
were ever in a published state.
/cc @jamiecobbett @heathd @alext @benilovj

Things to sort out first, but top work :+1:

Updated, @alext @binaryberry

In testing with not-entirely-up-to-date data, there were 4 routes that were owned by something other than Whitehall

The route registration part looks good to me.  I'll let someone more familiar with the whitehall workflow comment on the other parts.

More great sleuthing!

Updated. Here's the documents that aren't owned by Whitehall:
WARNING: Couldn't add route for /importing-and-exporting-by-post, owned by publisher
WARNING: Couldn't add route for /rural-development-programme-for-england, owned by publisher
WARNING: Couldn't add route for /agricultural-skills-and-training, owned by publisher
WARNING: Couldn't add route for /national-measurement-office, owned by short-url-manager
WARNING: Couldn't add route for /office-for-life-sciences, owned by short-url-manager
WARNING: Couldn't add route for /the-armed-forces-covenant, owned by short-url-manager
WARNING: Couldn't add route for /foreign-travel-advice, owned by travel-advice-publisher
WARNING: Couldn't add route for /bond-support-scheme, owned by publisher
WARNING: Couldn't add route for /export-working-capital-scheme, owned by publisher
WARNING: Couldn't add route for /letter-of-credit-guarantee-scheme, owned by publisher
WARNING: Couldn't add route for /care-to-learn, owned by publisher
WARNING: Couldn't add route for /apply-for-an-eu-ecolabel, owned by publisher

Bump @heathd / @jamiecobbett

using first_published_at IS NOT NULL is a good approach, I think that will handle all the edge cases.
Looks good :+1:


Android Auto: Crash when switching between car UI and AA app

A user reports that EVMap is crashing when switching from Android Auto to the car manufacturer's UI and back to AA when EVMap is opened on the favorites screen:
https://www.goingelectric.de/forum/viewtopic.php?f=45&t=56342&p=1670317#p1666678

closing as I cannot reproduce it.


Add shoulda-matchers Gem

It might be nice, with some of these, to know why the gems are being added…

@SteveMarshall mea culpa--I got in a rush there.  I suggested @EGiataganas add this gem to simplify some of the basic specs needed for things like the additional_visitor model.  I also missed that it needed configuration, so we'll add the explanation there.


Update travis

This PR is continuation of https://github.com/indirect/haml-rails/pull/115

@syguer thank you! I have added you as a committer to this repository.

wow! thanks! 🎉


Commands lagging for ~8 seconds due to version-check

Fixes: https://github.com/Azure/azure-storage-azcopy/issues/2482

Could you describe the change in the description?

@gapra-msft , added description.


/postcode endpoint now exposes country info

Also changed the postcode_gss_code_importer to use the offline-table method,
for a big speed-up. Refactored the reuseable parts out of the addressbase_import.sh script
to achieve this in a DRY-way.
Not 100% happy with the separation yet - it could be cleaner, and
at some point we may want to move some of the bash functions back in to Python - but this is
good enough at this stage.

all issues fixed - ready for re-review

:+1:


Redis is  Sentinel Mode and OOM occurred in Java application when Redis was down.

Bug Report

Redis is operating as Sentinel, and OOM occurred in Java application when Redis was down.
I couldn't find the cause, and below is part of Heap Dump.

Environment

Lettuce version(s): lettuce-core:6.1.8
Redis version: 4.0.14

Please limit the request queue.


WIP: Add SCP ingest API support

Introduce a new output plugin SplunkIngestApiOutput for
ingesting metrics into Splunk SCP.
This includes a fairly large refactor to introduce an
intermediate base class.
Other improvements:

Introduced rubocop - Reformatted all the code
Introduced more prometheus metrics

TODO:

Add metric support to the SCP side
Update documentation
Update the gem version and push upstream
Remove the SCP specific logging in the HEC plugin

LGTM.
Question:
If SCP is down or ingestion is at capacity, how will the plugin behave?

With release coming out, buffer to memory and then buffer to file.  Benchmarking and perf testing currently ongoing.

Thank @bcotton - Current active development is happening on the develop branch. This branch has all our updated CI and changes for the upcoming release of Splunk Connect for Kubernetes.
we will be promoting the code up to a release branch and then master.
This may affect  your PR which looks to be based of master. After we get our release out and pushed. After release, code base can take on all the changes you've provided.

Refactored and merged as part of #63. Closing older PR


Add Privacy manifests

Hello NW Web Socket Team,
I hope this message finds you well. I would like to draw your attention to an important update required by the App Store. Starting May 1, 2024, a privacy manifest will be mandatory for all apps. Given the ongoing development and support for your SDK, integrating this feature would be highly beneficial for ensuring compliance with the new App Store guidelines. This addition would greatly help developers like us in continuing to provide seamless services through our applications. We appreciate your attention to this matter and look forward to any updates you can provide.
Thank you for your continued support.

Hi, I'm also using this lib for my app, can you explain me more about what kind of update that affects this lib?


map not fit perfect screen on ios

Map view not fit flexible tag on ios device < X.
look at grey border.
It work on iphone X.

return new Padding(
      padding: new EdgeInsets.all(0.0),
      child: new Column(
        children: [
          new Flexible(
            child: new FlutterMap(
              options: new MapOptions(
                  center: new LatLng(39.213695, 9.121050),
                  interactive: false,
                  minZoom: 12.0,
                  maxZoom: 19.0,
                  zoom: 16.5,
                  onTap: (latLng) {
                    print('tap callback  $latLng');
                  },
                  onPositionChanged: (MapPosition mapPos) {
                    print("pos changed $mapPos");
                  }),
              layers: [
                new TileLayerOptions(
                    offlineMode: false,
                    urlTemplate:
                        "https://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png",
                    //  urlTemplate: "assets/offlinemaps/cagliari/{z}/{x}/{y}.png"),
                    subdomains: ['a', 'b', 'c']),
                new MarkerLayerOptions(markers: markers)
              ],
            ),
          ),
        ],
      ),
    );

Can you double check that you are using the latest version? (0.1.4)

I m using 0.1.4

Were you able to figure out what was causing this?

I m investigating,no risults for now. Flexible container fit correct all the space (ad you se in the pick has gray background). Maps appear ti ve not in the center.

ok some news... this is related to option.zoom.
this code has map without problem
 options: new MapOptions(
                 ...
                  maxZoom: 17.0,
                  zoom: 14.0,
                  ....
                 maxZoom: 18.0,
                  zoom: 17.0,

this code generate the problem
...
 maxZoom: 17.0,
 zoom: 16.5,
...
maxZoom: 17.0,
zoom: 15.5,
...
maxZoom: 17.0,
zoom: 14.5,

it seems that the problem appear when zoom is not int!!!

@gimox Thanks, saved me! zoom and maxZoom must be rounded doubles, with no decimal points.

gotcha. I would vote to re-open this now that this is a reproducible bug

@johnpryan yes, it should be reopened, it still happens as of today, and the zoom property still is a double, and should definitely be a int

mmmh, the gray bars also happen when using fitBounds() probably because it does not round the calculated zoom

@johnpryan this ticket should be reopened, it happens all the time with fitbounds


Enable Advanced User Scenarios for HttpClients

Enable the support for completely configuring the HttpClient layer, I.E. complete configuration of a Netty HttpClient.

This functionality is already allowed by passing NettyAsyncHttpClientBuilder a Reactor Netty HttpClient during instantiation.
https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/core/azure-core-http-netty/src/main/java/com/azure/core/http/netty/NettyAsyncHttpClientBuilder.java#L73
This same functionality is also offered for OkHttp by passing OkHttpAsyncHttpClientBuilder an OkHttp OkHttpClient during instantiation.
https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/core/azure-core-http-okhttp/src/main/java/com/azure/core/http/okhttp/OkHttpAsyncHttpClientBuilder.java#L53


Statements in transaction can still execute after rollback

Some statments in a transaction execute after a transaction rolls back, thus getting saved to the database. I have a full working sample at: https://github.com/luggage66/bookshelf-transaction-problem
The relevant bit of code from that:
return bookshelf.transaction(function(t) {

    var promise1 = new MyObject({name: 'This object should never get saved'}).save(null, {transacting: t});
    var promise2 = Promise.reject(new Error('SomeError'))

    return Promise.all([ promise1, promise2]);

});

Excellent bug report @luggage66!
Getting a patch out for it right away!

Ohh, awesome. I didn't expect such a fast response. I was trying to fix it myself by poking around and came up with: https://github.com/tgriesser/knex/compare/master...luggage66:master

The project leadership of Bookshelf recently changed. In an effort to advance the project we close all issues older than one year.
If you think this issue needs to be re-evaluated please post a comment on why this is still important and we will re-open it.
We also started an open discussion about the future of Bookshelf.js here https://github.com/bookshelf/bookshelf/issues/1600. Feel free to drop by and give us your opinion.
Let's make Bookshelf great again


Source .zsh-theme files by default

.zsh-theme and .theme-zsh files are treated the same way as .plugin.zsh files.
Related: #164.

Nice!


[BugFix][branch-3.1] fix compilation errors when WITH_CACHELIB=ON

Why I'm doing:
fix compilation errors in 3.1 when WITH_CACHELIB=ON:
In file included from /data/starrocks/be/src/block_cache/block_cache.cpp:22:
/data/starrocks/be/src/block_cache/cachelib_wrapper.h:53:45: error: 'std::unordered_map<std::__cxx11::basic_string<char>, double> starrocks::CacheLibWrapper::cache_stats()' marked 'override', but does not override
   53 |     std::unordered_map<std::string, double> cache_stats() override;
      |                                             ^~~~~~~~~~~

In file included from /data/starrocks/be/src/block_cache/cachelib_wrapper.cpp:15:
/data/starrocks/be/src/block_cache/cachelib_wrapper.h:53:45: error: 'std::unordered_map<std::__cxx11::basic_string<char>, double> starrocks::CacheLibWrapper::cache_stats()' marked 'override', but does not override
   53 |     std::unordered_map<std::string, double> cache_stats() override;

What I'm doing:
remove override keyword.
Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)
[ ] I have added documentation for my new feature or new function
[x] This is a backport pr

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@GavinMar Please help me review the code, thanks!


Responsive design

hey @arpitghura i would like to make this app responsive for all devices please assign this issue to me under codepeak'23

if this is under dwoc, then I would like to work on this issue, please assign it to me

Hey, I think using a png/svg for the icon will be better for the responsiveness or we can dynamically change the header color with the screen size. Then there's also the issue of the buttons position in smaller screens. Also the sizes of the notes is big compared to what it should be. I think I can solve this issue, so please assign it to me. Thanks


Upgrade to envoy 1.8.0

Envoy 1.8.0 was just released. Let's make upgrading to 1.8.0 a headline for Contour 0.8

https://groups.google.com/forum/#!topic/envoy-announce/s0h8P8mgI30


Use the new Linter API

https://github.com/steelbrain/linter/blob/master/docs/guides/upgrading-to-indie-linter-v2.md


Fix code saving in notebooks by refreshing user settings after login

This PR:

Calls a function to refresh user settings after a login. This allows us to pull in code saving preferences, which fixes an existing issue with history not getting saved in notebooks.
Refactors some viewer query logic in wandb_setup.

@adrnswanberg , is there a ticket associated with this PR?

@raubitsj no ticket that I could find, just something that I noticed while messing with jupyter.

@raubitsj no ticket that I could find, just something that I noticed while messing with jupyter.

@adrnswanberg great. i noticed it also, i just didnt want to mess with that code right now.  THanks!
Im capturing the commented out code that you removed so it can be used as reference in the kaggle internet bug:
https://wandb.atlassian.net/browse/WB-3289


Add more versioned tool binaries for (go build) gardener tools to gardenertools image

How to categorize this PR?

/area dev-productivity
/kind enhancement
What this PR does / why we need it:
This PR creates versioned binaries for tools we build with Go according to their versions in go.mod.
It allows us to add them to gardentools image in a safe way. The image contains the latest versions from master branch of gardener/gardener. These versions are copied to ./hack/tools/bin during the tests and can be used without compiling them.
If the version does not match e.g. because of older versions in release branches or newer versions in PRs, the binaries are built when the test is executed.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

/retest

/lgtm
/approve

The adapted make targets have to be in the other branches too.
/cherry-pick release-v1.80

/cherry-pick release-v1.79

/cherry-pick release-v1.78

/cherry-pick release-v1.77


[BUG] Wrong data in test run summary

I wrote and run(debug) 1 test.
In console I see test run summary

and I see 2 bugs:

Suites      │ 2 - Why run 2 suites when 1 run only 1 test from 1 test file?
Duration    │ 2m 156s - Strange duration. Total time was 156sec = 2 min 36 sec. Or display in sec, or correctly convert to min.

1, should be misunderstanding of name, Suites = projects + files + describes
see here https://playwright.dev/docs/api/class-suite
2, I will fix it next release

I have only one test in my project, so why should the number of test suites be equal to the sum of projects and files (2)? I believe that if I have one test file with one test, it should be considered as one suite. Similarly, if I have two test files and I run tests from both files, it should be counted as two suites (Suites = touched test files).

reasonable, how about following counting?
1, each case have a only parent suite
2, count all parent suites without duplicate suite

Correct

fixed in<EMAIL_ADDRESS>please have a try

In debug al, displayed cool

After normal run not cool :)


Fix TravisCI and make compatible with go-ipfs v0.4.3rc2

TravisCI previously ignored the functional tests that required an IPFS daemon to be running. #46 enabled those tests, but did not configure TravisCI to run an IPFS daemon. Now, Travis will download the release version of go-ipfs v0.4.3rc2 and run it before running the tests, ensuring that the functional tests pass too.
There have been some changes to the HTTP API responses since go-ipfs v0.4.3rc1, so this PR also updates the tests to reflect those changes.
This will resolve #50

Tested and verified that it indeed fixes all functional tests in py-ipfs-api. :+1:

Thanks for your contribution!


Md-Sidenav open mode bootstrap animation

Bug, feature request, or proposal:
Bug
What is the expected behavior?
When using universal, the sidenav (in side mode) will be rendered already open in the initial page load, and then when the app bootstraps, it will stay open.
What is the current behavior?
The initial pre-rendered page loads with the sidenav closed, and then when the app boostraps, the sidenav animates open.
What is the use-case or motivation for changing an existing behavior?
In beta 8, it worked perfectly. The initial pre-rendered html file aligned with the bootstrap re-render.
Now, the page loads with the sidenav closed, and after a number of seconds later (when the app bootstraps) the sidenav suddenly animates open, which is quite distracting.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 4.3.3, material 2.0.0-beta8, windows 10

I'd like to believe that the culprit here is a882546debb92bd9cf52937bf73eee8548d61bf0, which was for some reason left out of the CHANGELOG (this would also correspond to your timing that this worked in beta-8 and not beta-10 or beta-11).
Universal rendering doesn't support Angular Animations (yet), since it can't affect the document object (see https://github.com/angular/angular/issues/15098 and https://github.com/angular/angular/issues/14784). It looks like the beta-8 method relied exclusively on CSS-based animations, which would've rendered just fine.
cc @crisbeto @jelbourn Thoughts? (I.e. is it worth the tradeoff in perf/is there a workaround, and if not, is it worth the impact on UX for non-JS users who will never see the sidenav open?)

Might be duplicate of #7007, resolved by #7129

The problem here is actually deeper than anticipated. See here:
https://github.com/angular/material2/blob/20a23f16e8ac1ced29172709cc837fa500b6377b/src/lib/sidenav/drawer.ts#L347-L350
https://github.com/angular/material2/blob/20a23f16e8ac1ced29172709cc837fa500b6377b/src/lib/sidenav/drawer.ts#L553-L585
Since the margins for the mat-drawer-content depend on nativeElement, Universal can't possibly predict the width, leaving it static at 0, and populated on bootstrap on the client. The "animation" is really just the transition CSS firing once the width is populated.
@mmalerba Is there a possible refactoring we can do here, maybe by searching the element tree for CSS width if none other is available?

As documented in #7795, there's also an issue with transform: translate3d(0,0,0) not being applied correctly.

There's also currently an issue open https://github.com/angular/angular/issues/19235 that is tracking an issue with style.property not being applied correctly if the property name is in camelCase (in drawer, this is marginRight and marginLeft).

should be fixed by #8488

@mmalerba Issue not resolved by #8488

@crisbeto Still not fixed

Hi, any updates on this? In my case it was working OK with Angular 5.0.2 and material 5.0.0.rc0 but stop working when updated to Angular 5.2.8 and material 5.2.4


use filter instead of map

Filter sinks (only keep sinks that exist).
Should behave equal to old map function that maps non-existent sinks to an empty stream.
However, filter is faster and easier to understand.

yes of course, no idea why I did this
Can you replace Boolean with b => b? It's easier to read.

did it myself, merged


Add pe file event

Note

R 프로젝트에서만 사용되는 pe file 이벤트에 대한 정의를 rio 에 merge 하도록 pr 합니다.

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (eb5bbfa) 33.69% compared to head (d803155) 33.28%.

Files
Patch %
Lines

src/ingest/sysmon.rs
0.00%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##              rio      #96      +/-   ##
==========================================
- Coverage   33.69%   33.28%   -0.41%
==========================================
  Files          13       13
  Lines         650      655       +5
==========================================
- Hits          219      218       -1
- Misses        431      437       +6

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


adding rust-cache to improve build times

Motivation
adding rust-cache to improve build times.
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
untested
Related PRs
continuation of https://github.com/diem/diem/pull/9865
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

Findings:

rust-cache did not improve build times (original: 15m, with rust-cache: 20-54min). There is a post block that is taking a long time for rust-cache (about 20 minutes, other times 0s). I increased the timeout from 30m to 60m to allow the post block to finish, but subsequent runs after that also did not have a performance increase (still took >15min)
the build-benchmarks step is completing successfully but doesn't appear to be using the cache (still downloading and compiling)
Links:
54 minute run with increased timeout: https://github.com/diem/diem/actions/runs/1498906343/attempts/2
subsequent run after 54m run: https://github.com/diem/diem/actions/runs/1498906343 (cancelled bc it was clear it wasn't <15min)

please note I do not have permission to delete branches (the feature branches I created), so someone with that power should delete these branches:

add_rust_cache_to_build_benchmarks
add_rust_cache

For context, this is from bootcamp task T100732822. It doesn't look like rust-cache is working as the build-benchmark job continues to download and compile 3rd party crates. I am going to close this PR for the time being.  Feel free to re-open and rework the PR to optimize the build time with cache.


TypeError: init() got an unexpected keyword argument 'no_args_is_help'

blackbricks --remote<EMAIL_ADDRESS>Traceback (most recent call last):
File "/Users/henokyemam/opt/anaconda3/envs/experiment/bin/blackbricks", line 8, in
sys.exit(app())
File "/Users/henokyemam/opt/anaconda3/envs/experiment/lib/python3.6/site-packages/typer/main.py", line 214, in call
return get_command(self)(*args, **kwargs)
File "/Users/henokyemam/opt/anaconda3/envs/experiment/lib/python3.6/site-packages/typer/main.py", line 239, in get_command
click_command = get_command_from_info(typer_instance.registered_commands[0])
File "/Users/henokyemam/opt/anaconda3/envs/experiment/lib/python3.6/site-packages/typer/main.py", line 442, in get_command_from_info
deprecated=command_info.deprecated,
TypeError: init() got an unexpected keyword argument 'no_args_is_help'

Thank you for reporting this. I can't seem to reproduce this on a fresh install. The keyword argument this error message complains about is not used anywhere in the code. This leaves me to think that the issue is somehow related to some incompatible package versions in your environment.
Please provide the output of these commands, from within your conda environment:
python --version
pip --version
blackbricks --version
pip freeze

I just had the same problem. The reason was that my Anaconda installation had a very old version of click installed (v7.0 from 2018).
The no_args_is_help keyword argument was introduced in v7.1 (see the click changelog -> it is mentioned there).
So the fix is to update click, e.g. conda update -n experiment click or make sure you pin click to at least v7.1 in your conda environment file.

That makes sense. Still, I'm a bit surprised this wasn't resolved when blackbricks was installed. The setup file requires typer >= 0.3.0, which again requires click >= 7.1.1, <7.2.0. Perhaps anaconda doesn't automatically upgrade packages you already have on the system. Regardless, I don't see what I can change on "my side" to ensure this doesn't happen. At best I can put something in the FAQ.
I will close this for now, but feel free to reopen if there is something you think should be changed in blackbricks.


dev-2.0 stuck trying to exit

Prometheus dev-2.0 branch did not exit after SIGTERM.
Relevant logs: https://gist.github.com/Gouthamve/ba1c52b09034a96d3b73fc85c8939011
I see fdatasync(17) in the middle which makes me think that Prometheus is stuck trying to flush.
cc @fabxc

Could grab the output of: http://localhost:9090/debug/pprof/goroutine?debug=2
Gisted here: https://gist.github.com/Gouthamve/ba1c52b09034a96d3b73fc85c8939011#file-goroutine-debug-2

Looking at the goroutine dump, I see that this could have been caused by #2629
This is because there are 2 goroutines stuck trying to acquire db.mtx and the exiting is blocked on the db.Close() call which is trying to acquire the lock.
The panic caused by the parser might not have let Querier gracefully RUnlock causing the deadlock.

This is due to a panic.
What potentially happened was we panicked due to a bug and that go-routine had acquired a lock. And we cannot gracefully exit until that lock is released. But as this is caused due to a panic, something which should never happen, can we close this?

I think so yes. We definitely fixed a bunch of cases where we put lock releases into defer statements to prevent this. In general though, if it's a panic, these things might just happen.


User Auth on Frontend

User login:

email
hashed password
login session
user page
-- list of swot pages


fix: provide public constructors

Swift synthesizes internal constructors instead of public constructors, making it impossible to call funcs requiring a RadiusAtCoordinates or Coordinates object as a parameter.
Also provide a convience constructor for RadiusAtCoordinates that takes a latitude and longitude as parameters.

These synthesized constructors always get me. So difficult to properly test. Thanks for the fix!

These synthesized constructors always get me. So difficult to properly test. Thanks for the fix!


I think there is a conflict in dependencies between angular-ui-router and ui-select

I tried to install both packages and it ended up with 2 versions of angular:
<EMAIL_ADDRESS><EMAIL_ADDRESS>looks like there are 2 organizations for the same purpose.
what I see in chrome console is:
WARNING: Tried to load angular more than once.

looks like this is ui-select issue, @dotansimha can you update the dependencies?

Hi,
@Urigo  - Fixed it, package: https://atmospherejs.com/angularui/ui-select - the new version is 0.12.0.
@oshai - Can you try and verify?

the package was removed but I still have the warning.
from looking at the versions file of meteor I see I still have 2 versions
of angular, although I am not sure that is a problem.
<EMAIL_ADDRESS><EMAIL_ADDRESS>searching the web I found that usually it happened because of a bad
template route. I was not able to spot one like that... yet...
On Tue, Jun 2, 2015 at 12:54 AM, Dotan Simha<EMAIL_ADDRESS>wrote:

Hi,
@Urigo https://github.com/Urigo - Fixed it, package:
https://atmospherejs.com/angularui/ui-select - the new version is 0.12.0.
@oshai https://github.com/oshai - Can you try and verify?
—
Reply to this email directly or view it on GitHub
https://github.com/Urigo/angular-meteor/issues/382#issuecomment-107724211
.

@oshai , please try to debug and check for invalid files that are loaded (404), Meteor does not return "404" - instead he returns the homepage, that can cause angular to load again in some cases.

now its gone, without doing any additional thing, so it might be that it
did the trick. Thanks!
On Tue, Jun 2, 2015 at 2:54 PM, ohad shai<EMAIL_ADDRESS>wrote:

the package was removed but I still have the warning.
from looking at the versions file of meteor I see I still have 2 versions
of angular, although I am not sure that is a problem.
<EMAIL_ADDRESS><EMAIL_ADDRESS>searching the web I found that usually it happened because of a bad template route. I was not able to spot one like that... yet...
On Tue, Jun 2, 2015 at 12:54 AM, Dotan Simha<EMAIL_ADDRESS>wrote:

Hi,
@Urigo https://github.com/Urigo - Fixed it, package:
https://atmospherejs.com/angularui/ui-select - the new version is 0.12.0.
@oshai https://github.com/oshai - Can you try and verify?
—
Reply to this email directly or view it on GitHub
https://github.com/Urigo/angular-meteor/issues/382#issuecomment-107724211
.

thanks, in my case I have "otherwise" route to 404.
On Tue, Jun 2, 2015 at 3:05 PM, Dotan Simha<EMAIL_ADDRESS>wrote:

@oshai https://github.com/oshai , please try to debug and check for
invalid files that are loaded (404), Meteor does not return "404" - instead
he returns the homepage, that can cause angular to load again in some
cases.
—
Reply to this email directly or view it on GitHub
https://github.com/Urigo/angular-meteor/issues/382#issuecomment-107930274
.


bugdown: {start_tabs}…{end_tabs} produces broken HTML

In templates/zerver/help, the Markdown input
{start_tabs}

foo

{end_tabs}

renders as

None

foo

which is incoherent tag soup: there’s a
 closing a
, a
 closing a
, and a
 closing nothing at the end (because the first
 was autoclosed by the first
).
This affects all uses of {start_tabs}…{end_tabs} across /api, /help, and even test_tabbed_sections.md, whose broken output has been enshrined in test_templates.py.
The validator is not pleased.

@eeshangarg can you look into this?

@andersk: Thanks a lot for reporting! I think I remember running into this at some point, the way the Markdown preprocessor works is kinda inherently broken, to be honest! I'll investigate this. :)

It seems like this doesn't have to be inherently broken.  We should be able to use the same high-level strategy used for blockquotes (for example) to produce a block without invalid HTML.
@aero31aero @PentaHiggs this might be something you could help with, since it's likely to be something subtle like the order we runs things in.  The code is in zerver/lib/bugdown/tabbed_sections.py and http://localhost:9991/help/emoji-and-emoticons is a page that demonstrates the issue (though be careful; browsers will auto-fix HTML).

is somebody working on this, if not i can try as my, pr(https://github.com/zulip/zulip/pull/12657) is dependent on this issue?

@Arpitpundir: This issue hasn't been claimed, no! So feel free to work on this. :) Thanks!

The bug is that the RawHtmlPostProcessor in python-markdown puts the raw html in the final message as: "
%s
" % (self.md.get_html(). For us, we have two html blocks:
# <Block 1>

\n

\n

None
\n

\n

\n

# </Block 1>
			\n\nfoo\n\n
# <Block 2>

\n

\n

# </Block 2>

We need to subclass the RawHtmlPostProcessor for the engine we make in app_filters.py.

Oh, nice debugging.


how to populate IR_*_clone_id columns?

Hi @grst, I can't work out when in the analysis workflow is the IR_*_clone_id columns populated. Toggling dual_ir option doesn't seem to change that behaviour. Does it actually get populated from preprocessing/analysis performed within scirpy?

No, these columns would actually only get populated when reading in the
clone_id column from an AIRR formatted file and not flagging clone_id
as a cell-level attribute.
(Cell-level attributes must be the same in all records belonging to a cell
in the AIRR table, and would then only get a single column in adata.obs
instead of one for each chain.
But this behavior is still pretty much in the works.
Clonotype assignments called with scirpy are stored in a single clonotype
column, or cc_{sequence}_{metric} for "clonotype clusters".
On Wed, 31 Mar 2021 at 11:16, Kelvin @.***> wrote:

Hi @grst https://github.com/grst, I can't work out when in the analysis
workflow is the IR_*_clone_id columns populated. Toggling dual_ir option
doesn't seem to change that behaviour. Does it actually get populated from
preprocessing/analysis performed within scirpy?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/icbi-lab/scirpy/issues/258, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVZRV7CBHB6KJRBWWB5753TGLR77ANCNFSM42EEZIFA
.

The more I think about it: probably it's best to just use clone_id as the default output column for define_clonotypes.


Why is @cutout not a boolean?

The cutout attribute only allows the value "cutout". Is there a reason for this not to be a boolean? As far as I can see this is the only attribute with only a single value allowed.

Another thing that is not totally clear: Have mRest cutout="cutout" and mSpace cutout="cutout" the same visual expression, i.e. is the whole measure "invisible" or stayes the rest glyph for the former?


bug: turan cse 7 is wrong

@lamBOOO please fix it

I have wuestion: Which Prandtl number is used?

use $\operatorname{Pr}=0.7$


Terminal pane starts up before Python extension and virtual environment isn't activated

VS Code Version: 1.56.0 (user setup)
OS Version: Windows 10

From Help/About:
Version: 1.56.0 (user setup)
Commit: cfa2e218100323074ac1948c885448fdf4de2a7f
Date: 2021-05-04T22:09:06.405Z
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19041

Steps to Reproduce:

Start VS Code on a directory containing a Python project, with a virtual environment .venv which is set as the default Python interpreter.
Start the integrated terminal, which will run with .venv activated as expected.
Close VS Code
Reopen VS Code
The editor will open with the terminal pane open, as it was previously closed with the terminal still open. Note that the Python extension is still starting at this point (you may not see this if your machine is particularly fast, but it is consistent on my PC).
The virtual environment is not activated, because the Python extension hasn't initialised.

I checked my settings, and I see
"terminal.integrated.enablePersistentSessions": true
"terminal.integrated.environmentChangesRelaunch": true

I did not set either of these myself, so I assume they are default values. They seem to suggest that what I should expect is for my terminal session to get restarted once the Python extension has loaded.

Does this issue occur when all extensions are disabled?: No
No, it's specifically related to how extensions initialise.

🪓 If you answered No above, use 'Help: Start Extension Bisect' from Command Palette to try to identify the cause.

I don't see how this would help, but if I'm missing something I'll be happy to do this with a little clarification as to what I should do.

📣 Issues caused by an extension need to be reported directly to the extension publisher. The 'Help > Report Issue' dialog can assist with this.

I don't believe this is specific to the Python extension, but I don't have another extension that changes the terminal environment, so I don't have any way to confirm this.

@meganrogge I use different settings, but I have the same issue.
Previously, I used the following workspace settings to auto-activate the environment in the terminal:
"terminal.integrated.shellArgs.linux": [
    "/path/to/activate.sh",
],

It activated the environment right after starting up.
These settings were deprecated in the latest version:
https://code.visualstudio.com/updates/v1_56#_terminal
I tried to use the new settings:
"terminal.integrated.profiles.linux": {
    "bash": {
        "path": "bash",
        "args": [
            "/path/to/activate.sh",
        ],
    },
},

Unfortunately, it only works for New Terminals or when I have a different tab open after the start and switch to Terminal after VS Code is fully loaded.

This is fixed now and in 1.56.2

@meganrogge Which settings are supposed to work in 1.56.2?
Now I tried that the old settings work on Linux:
"terminal.integrated.shellArgs.linux": [
    "/path/to/activate.sh",
],

But it's marked as deprecated.
On the other hand the new syntax doesn't work:
"terminal.integrated.profiles.linux": {
    "bash": {
        "path": "bash",
        "args": [
            "/path/to/activate.sh",
        ],
    },
},

On Windows, neither the old syntax nor the new one worked (with 1.56.2).
Old:
"terminal.integrated.shellArgs.windows": [
    "powershell -noexit -command \"& {cd $env:VSCODE_CWD; . C:/.../Scripts/Activate.ps1}\""
],

New:
"terminal.integrated.profiles.windows": {
    "PowerShell": {
        "source": "PowerShell",
        "args": [
            "-noexit",
            "-command",
            "\"& {cd $env:VSCODE_CWD; . C:/.../Scripts/Activate.ps1}\"",
        ]
    }
},

By not working I mean it's not activated on startup. It's activated when the new Terminal is launched (or the VS Code is started with hidden Terminal and the tab is switched to it).

@FitRiver I'm assuming that what is fixed is the behaviour I reported (the terminal doesn't wait to open until the Python extension is ready, so that it gets the activated virtual environment). What you're describing is different, and should probably be reported as a separate issue (unless there's some link that I'm not seeing - I'm just a user, @meganrogge can probably advise better than I can).

Hmm, @meganrogge I just tested the behaviour again - I now have 1.56.2 - and it's still happening.
Could you re-open this issue, as it looks like it hasn't been fixed?

@FitRiver I'm assuming that what is fixed is the behaviour I reported (the terminal doesn't wait to open until the Python extension is ready, so that it gets the activated virtual environment).

I have been waiting on any response to your comment, but you are probably right it's a different issue. I'll open a new ticket.

Two things you might be hitting are that if shell or shellArgs is set, they will take precedence over the defaultProfile setting. Additionally the initial profile restored will not restore defaultProfile https://github.com/microsoft/vscode/issues/123188
The latter sounds like your issue @FitRiver. Note also that your args are a different format, in the first you're running powershell within your shell with the args as a single string, the second is separate args within a single shell. This would be the equivalent, if your shell was powershell before:
"terminal.integrated.profiles.windows": {
    "PowerShell": {
        "source": "PowerShell",
        "args": [
            "powershell -noexit -command \"& {cd $env:VSCODE_CWD; . C:/.../Scripts/Activate.ps1}\""
        ]
    }
},

@Tyriar I don't have those settings, as you can see from my settings posted above.
Can this issue please be reopened as it definitely hasn't been fixed for me? If not, I can open a new issue pointing back to this one - but I'd rather not have to do that as it seems unproductive compared to just re-opening this.

@pfmoore please try on the Insiders build  first, a lot of changes have gone into profiles since v1.56.2

Nope, still an issue with insiders build.

Install Insiders build
Install Python extension (auto-installs pylance and jupyter). I got asked if I wanted to use the insiders build of the extensions, I said yes.
Open a folder that contains a .venv directory with a virtual environment, and a .vscode directory with a settings.json containing just the line "python.pythonPath": ".venv\\Scripts\\python.exe".
Open a .py file in that folder.
Open the terminal with Ctrl-'. Terminal starts and runs the venv activate script. Terminal is pwsh (Powershell core). I assume it's picking that up from my settings.
Close VSCode, without closing the terminal.
Restart VSCode. It starts with the previous folder open, and the terminal pane open as that's how it was when last closed. The terminal opens, but it starts opening before the Python extension has finished initialising (the "Python 3.9.2 64-bit ('.venv')" note in the bottom status bar hasn't appeared yet). When the terminal finishes starting up, it does not have the venv activated.

Why do you think it's related to profiles? It seems to me that the issue here is that when starting, if the terminal was remembered as having been open when last closed, VSCode restarts the terminal without waiting for extensions to initialise so that they can add their "do this when the terminal opens" settings. Is that profile-related?

Restart VSCode. It starts with the previous folder open, and the terminal pane open as that's how it was when last closed. The terminal opens, but it starts opening before the Python extension has finished initialising (the "Python 3.9.2 64-bit ('.venv')" note in the bottom status bar hasn't appeared yet). When the terminal finishes starting up, it does not have the venv activated.

This is by design. I believe the Python team is looking into the better way of contributing to the environment via the context.environmentVariableCollection API. Not sure if they had an issue but created https://github.com/microsoft/vscode-python/issues/16318

I'm guessing that when you say "by design" you mean that a previously opened terminal is reopened without waiting for extensions because it was a deliberate choice not to wait. I can see that, but it has the unfortunate consequence that extensions that the way the Python extension activates the selected venv doesn't work.
Given that this means it's regrettably far too easy to end up working in a terminal with the wrong Python environment active, is there a way to work around this? I'm specifically thinking of a setting that lets me say "always start VS Code with the terminal window closed, even if it was open when the program last shut down". That would be for me far better than the current behaviour.

I'm specifically thinking of a setting that lets me say "always start VS Code with the terminal window closed, even if it was open when the program last shut down". That would be for me far better than the current behaviour.

The issue I linked to should hopefully result in the best of both worlds, also it's possible the Python extension could use the current mechanism to activate within restored terminals since they can access them. Both of these would be jobs for the Python terminal though.
The setting idea has come up before but we'd prefer not to do that, regardless it wouldn't fix the case where you open a terminal quickly after launching (before extension activation).


get_hashtag_posts() throws KeyError: 'graphql'

Describe the bug
When logged in, running instaloader.get_hashtag_posts('<some hashtag>') throws an error.
To Reproduce
These are the steps to reproduce the issue for me:
import instaloader
from os import getenv
import dotenv

dotenv.load_dotenv('./instagram/.env')

loader = instaloader.Instaloader()

loader.login(getenv('INSTAGRAM_USERNAME'), getenv('INSTAGRAM_PASSWORD'))

posts = loader.get_hashtag_posts('opet')

Expected behavior
I expect to run the error-giver line to run without any errors and contionue to get informatin about posts.
Error messages and tracebacks
If applicable, add error messages and tracebacks to help explain your problem.
Traceback (most recent call last):
  File "instafoo.py", line 12, in <module>
    posts = loader.get_hashtag_posts('covid19')
  File "/home/.../.venv/lib/python3.8/site-packages/instaloader/instaloader.py", line 996, in get_hashtag_posts
    return Hashtag.from_name(self.context, hashtag).get_posts()
  File "/home/.../.venv/lib/python3.8/site-packages/instaloader/structures.py", line 1302, in from_name
    hashtag._obtain_metadata()
  File "/home/.../.venv/lib/python3.8/site-packages/instaloader/structures.py", line 1316, in _obtain_metadata
    self._node = self._query({"__a": 1})
  File "/home/.../.venv/lib/python3.8/site-packages/instaloader/structures.py", line 1311, in _query
    return self._context.get_json("explore/tags/{0}/".format(self.name),
KeyError: 'graphql'

Instaloader version
Instaloader: 4.6
Python: 3.8 in a virtualenv
Ubuntu 20.04

I have checked the release notes above 4.6 but didn't see a note that resolves this issue.


Optional disk resource column tasks page

Implements #1018 based on #993

merging this into disk since they are both in stable and go together


filterItems() can execute asynchronously?

Hi @davideas,
I would like to ask filterItems () whether there is an asynchronous implementation of the method?
For example, when I filter 5000 data, if I use filterItems () directly, it will lead to ANR.
Adding asynchronous filtering will make the library more perfect, please think about it, thank you.

@LuckHard, thanks for your message.
I agree with this improvement, it can be split in 2: notifyDataSetChanged() and Async.
Currently filterItems() calls animateTo(), it calculates the removal / addition / moving, I can make the animation optional, calling notifyDataSetChanged() it will improve a lot only that, but Async will probably still help for high volume like yours.
I will try to add this option to be controlled by the user first, then I will check how I can introduce the Async in the Adapter, even if, in version 4.2.0 I removed Async, at that time it was a superfluous feature.

@LuckHard, please let me know if the addition I have done (use SNAPSHOT release) resolves the ANR, you can test with higher number as well, so you can give me a better feedback.
Async is still not in place.

@LuckHard, I'm trying to add Async processing, there are several problems to resolve.

All notifyXXX() methods cannot be called in Background because Caused by: java.lang.IllegalStateException: Cannot call this method while RecyclerView is computing a layout or scrolling.
We need to stop the previous unfinished task before a new one can start.
New listener to notify the fact a background task is started and finished
expandAll() and collaspeAll() can eventually benefit of the Async processing, but same limitations will occur.

Any ideas?

Could perform the filtering and removals/additions/moves, but instead of notifying from the async thread put together a queue of positions (or pairs, for moves). When the task is finished, pass the queues back to the UI thread and go through them to notify. Just a thought, not sure if it's super feasible. At the very least, you can do all filtering up until animateTo off the UI thread.
Would like to integrate this library into my project, but no async filtering makes it a non-starter unfortunately.
Would also be nice to have an additional hook into the filtering logic, in case we need to filter by something other than / in addition to a string. My application has search by string filtering in addition to categoric filtering, and I don't see a way to accomplish this with your library right away.

Hi @cordeveloper, a part the work already published with notifyDataSetChanged() which resolves already the speed of the filter, yesterday I worked on experiments branch where I committed it now the AsyncFilter as @LuckHard suggested: I created a queue of notifications (field of main class for the moment) to be executed in onPostExecute().
I also made changes (where possible) to use HashSet instead of List, this made a big improvements itself, unfortunately I had to made option the move animation.
Now, basic functionality works very fine, just sometimes if you insert a text, then stop, then add more text, it doesn't notify the last text.
I tested with :100:000 items in the emulator, I think it's not bad:
10:58:02.225 5003-5003/D/MainActivity: onQueryTextChange newText: 7
10:58:02.435 5003-5098/I/FilterAsyncTask: doInBackground - started Filter
10:58:02.435 5003-5098/V/FlexibleAdapter: FilterItems with searchText=7
10:58:03.535 5003-5098/V/FlexibleAdapter: Animate changes! oldSize=100001 newSize=40951
10:58:04.917 5003-5098/V/FlexibleAdapter: calculateRemovals total out=59050
10:58:04.940 5003-5098/V/FlexibleAdapter: calculateAdditions total new=0
10:58:04.940 5003-5098/I/FilterAsyncTask: doInBackground - ended Filter
10:58:04.946 5003-5003/V/FlexibleAdapter: Performing 100001 notifications
10:58:05.401 5003-5003/D/MainActivity: onUpdateEmptyView size=40951

I'll create a Fragment only for big list with dynamic values and searchFilter, for now you can use FragmentEndlessScrolling and FragmentExpandableSections to test and give me some feedback. To test it, change the numbers directly from the initialization of the RV.
And from the MainActivity you can change the delay or remove it, to call filterItems().
@cordeveloper, to Filter by categories etc, you can try to override filterObject() as I suggested in the method comment, you will tell me more.

I moved all code in dev branch. The demo Fragment for Async is ready (to be finilized), but you can already test with different configuration.
I wait your feedback.
Davide

The part 2 of this improvement (AsyncTask) is now published with the Snapshot release of today 16/7/2016.


test-branch/execute-job/wrong-commit-sha-on-pr/e8af0ef3-b9f4-4a77-8cb2-1a4d01e3d0c9

PR for testing adba5ad381c78b9d71001a6f244951910a9b635d

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


Create short Script Documentation(feature|usage)

Write short documentation about how to use the script and what it could do. Very useful for a newcomers ;)

might be related to #128 - I assume you'd want to place this in the readme?

Yeah, sounds good! Could be a starting point for all newcomers.

I'm not sure if #168 was sufficient, @w20k , if yes, would you close this? Otherwise give me a hit what I'd need to add :)

@Idrinth , I was thinking, if you could describe each functionality separately.
Like - Settings tab contains number of dfiferent settings (list of setting), when  minimize checkbox enabled will decrease space used by the script on the screen and could be accessible via pressing on the small button .
@Idrinth , remember you had an idea about initial tutorial? We could move this "manual" to an about page or help page (which could be added) in script, or add question mark in each tab, somewhere, which will provide a short info.

would want to move your first point to #128 - just way too big to be put all of that into the readme imo.
Regarding the second one, that would make sense, but I think we should do this in another ticket and after the translation feature has been implemented. Would you open one?

Sure thing will be closing this one than.


[BUG REPORT] Highlights trigger when matched phrase is inside part of a word

Please Describe Your Issue
When a user has something highlighted (e.g. test), Horace DMs them when a message has another word that has that phrase (e.g. tested).
How to Reproduce

Highlight phrase
Have another user send a message with that phrase, inside another word (e.g. highlight Sam and send Same).

Expected behavior
Horace should only alert user when the phrase they've highlighted appears on its own.
Extra Context
Probably just need to adjust a regex or two, to make sure there's only spaces or punctuation on either side of the highlighted phrase.

@ThatNerdSquared, plus there is also an issue where even when i tried to add the word sam to my highlights it shows that it already exists.

@ThatNerdSquared Did PR #274 fix this issue? Confirming before closing


Make playlist.py easier to understand by using the built-in csvreader

Using python's built in csv reader to make reading the csv file less verbose and confusing.
cc:/ @savingoyal @tuulos

On further thought, I am going to close this PR, because I think this example could distract from the larger goal of explaining Metaflow by inadvertently nerd sniping data scientists to focus instead on this csv to dataframe construction.
I'll bring this topic up in a different forum and see if there is a better way to go about this.
Thank you 🙇🏽


Testing the code

Just putting miscellaneous notes here as I test the code.

Need to add tidysdm to the packages in setup.R
Need instructions on changing your data PATH in the web instructions "Coding" page

Hmmm... tidysdm requires a special install.  Is it not installing for you?
https://github.com/BigelowLab/ColbyForecasting2025/blob/main/setup.R#L11

I just used install.packages() and it worked. Maybe it's in CRAN now?

The version on CRAN is outdated (it doesn't have our recent contributions to make it work with stars objects), and so we are installing the development version from R-universe which is a non-CRAN CRAN service for developmental packages.  The package author doesn't plan to update the stable CRAN version until after the holidays which is too late for us.


MAINT: Change c*np.ones(...) to np.full(..., c, ...) in many places

What does this implement/fix?
When c is a scalar, c * np.ones(...) and np.full(..., c, ...) produce the same output, but np.full is faster
and clearer in intent, so I (somewhat mechanically) replaced a lot of instances.
timeit np.ones((100, 1000)) / 12.
725 µs ± 37.3 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)

timeit np.full((100, 1000), 1 / 12.)
75.2 µs ± 7.44 µs per loop (mean ± std. dev. of 7 runs, 10000 loops each)

I did not change ones inside benchmarks or where I wasn't sure that it's always a scalar.

Looks like a good idea to me, let me know when everything is green and I should look @endolith

@larsoner It's green now.

When c is a scalar, c * np.ones(...) and np.full(..., c, ...) produce the same output

There is at least one difference worth mentioning; the dtype may be different. e.g.
>>> (5 * np.ones(10)).dtype
dtype('float64')
>>> np.full(10, 5).dtype
dtype('int64')

I tried to be conscious of the types while looking, but one more set of reviewer eyes would be good

Thanks @endolith


Upgrade from 1.4 to 1.5/1.6 can cause Jenkins jobs to fail

This problem occurs when a job doesn't have a Tenant ID set in its configuration. If the plugin is updated but the job is never modified or re-saved, the plugin fails when trying to read the Tenant ID:
java.lang.NullPointerException 	at hudson.plugins.octopusdeploy.OctopusDeployDeploymentRecorder.<init>(OctopusDeployDeploymentRecorder.java:77) 	at hudson.plugins.octopusdeploy.OctopusDeployReleaseRecorder.perform(OctopusDeployReleaseRecorder.java:299) 	at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20) 	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:779) 	at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:720) 	at hudson.model.Build$BuildExecution.post2(Build.java:185) 	at hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:665) 	at hudson.model.Run.execute(Run.java:1745) 	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43) 	at hudson.model.ResourceController.execute(ResourceController.java:98) 	at hudson.model.Executor.run(Executor.java:410)
The workaround is to simply save the job again, so that the job contains a tenant id in its configuration.

this will be resolved with 1.7.0

1.7.0 has been published


[fleet] add first systemd test

What does this PR do?
Add a first systemd test based on ordered journald logs.
This test only the case where the experiment crashes (no experiment installed). More tests will follow. Adding it now as it can help with updater self update tests
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge


ignore build-deps

Fixed by https://github.com/fare/gerbil-poo/commit/29d09d9a259ce74984b7db0c2d31c9524568bf59


Interoperating with CancellableTasks

While writing #77 I realised it would be logical to allow this TaskSeq thing to interoperate with a standard CancellableTask, that is
taskSeq {
    let! res = cancellableTask { ... }
    ...
}

which would pass the CancellationToken governing the iteration of the TaskSeq to the Cancellable task.  This would give robust cancellation token passing through the overall computation.
Under the alternative long term strategy in #77 this would be the same as
asyncSeq {
    let! res = async2 { ... } // re-implemented F# async
    ...
}

because F# cancellableTask and async2 are more or less the same thing.
This would mean bringing a CancellableTask into this library from IcedTasks, which may be a good thing.

This would mean bringing a CancellableTask into this library from IcedTasks, which may be a good thing.

Hmm, yes. But then the top lib name of TaskSeq should be reconsidered, as the namespace will be hosting tasks and task sequences.
Maybe just FSharp.Control.Tasks and turn it into a collection of all of our wishlist in one place? I.e., cancellable task, parallel task and delayed tasks from IcedTasks and TaskSeq, perhaps later AsyncSeq as well? And then should we include the extremely useful FsToolkit taskResult and taskOption and the like?
Many questions and strategies possible... Though the idea of a "single stop shop" for all (or most) of task/async related stuff is appealing.

Yes for the project and nuget something like FSharp.Control.TaskExtras may make sense
We can think it over a bit more but I guess it's OK to go for task/cancellableTask/taskSeq for now.
I'll think about whether it's possible to do an overall revamp for F#8, tailcalls etc too. If we did there would be no harm in a rename then.

@dsyme, see linked issue by @bartelink, #128. It's now a good time as any to discuss the scope of Task extensions and landing it. It appears that other libraries also yearn for a small, useful extension library.
Not exactly the same as what we discussed here, but this could be a precursor to unifying these efforts at some point.


lyne-timetable-transportation-walk

state: in progress
figma: https://www.figma.com/file/oA9B5W6w0449gtItDzLS86/00%3A-Components?node-id=360%3A735437
dev: https://lyne.sbb.ch/components/lyne-timetable-transportation-walk
ux quality-check
code quality-check
refactoring (optionaly)
testing

Feature Description
We want to review/refactor the lyne-timetable-transportation-walk
Figma Spec

dev: https://lyne.sbb.ch/components/lyne-timetable-transportation-walk

Design Spec
Technical Spec

Definition of Done

[ ]  ux quality-check
[ ]  code quality-check
[ ]  refactoring (optionaly)
[ ]  testing
[ ]  Component is implemented
[ ]  Tests are implemented
[ ]  E2E Tests are implemented
[ ]  Storybook stories are implemented
[ ]  Navigation via keyboard is tested
[ ]  Screen reader output is tested
[ ]  High-contrast is tested
[ ]  Remaining accessibility is tested
[ ]  UX approved

Will be tracked/decided in #1901


About hardware mapping files.

Hello ,
I just explored this repo a few days ago. Here ,https://github.com/KULeuven-MICAS/zigzag-imc) in this link , I saw some of the DIMC has been evaluated in this framework. can you kindly point me to the hardware files and mapping files for these DIMC papers ?
just like I am seeing files inside zigzag/inputs/examples/hardware andzigzag/inputs/examples/mapping/tesla_npu_like.py in these folders. I want to see the these files for DIMC mentioned in this figures.

Hi, thanks for your interest!
@JiacongSun could you have a look and point her to the correct settings please

Hi,
These chips are used for validation through specific scripts. Please have a look at https://github.com/KULeuven-MICAS/zigzag-imc/tree/master/imc_model_validation for the details.
To your question, they are not provided as hardware templates, so there are no valid templates yet. However, using the parameters in the table, you can fill these numbers into the IMC template files of DIMC and AIMC.

Thanks @JiacongSun!
Closing this issue now. Feel free to open any other issues if you have more.


Allow configuration of AppBar.titleSpacing in AppBarTheme

AppBar has a titleSpacing property that could benefit from being configured in the AppBarTheme.

@shihaohong I have submitted a PR, PTAL when you have some time. No hurries :)


Added settings object

and copied scheme object into it with spread syntax
idk if this is the approach you wanted, but it works perfectly fine

as simple as that, I found out that it's just a localStorage key, imma update the methods

chatgpt.scheme is missing methods to set the theme dark/light, do you want me to add them?

yes w/ .set('dark'|'light|'system')

@adamlui it should be done now, give the code a look


closes #389

Codecov Report

Merging #417 into experimental will decrease coverage by 0.36%.
The diff coverage is 18.18%.

@@               Coverage Diff                @@
##           experimental     #417      +/-   ##
================================================
- Coverage         72.85%   72.49%   -0.37%
================================================
  Files                30       30
  Lines              1735     1745      +10
================================================
+ Hits               1264     1265       +1
- Misses              471      480       +9

Impacted Files
Coverage Δ

vent/menus/main.py
11.11% <18.18%> (-0.07%)
:arrow_down:

Continue to review full report at Codecov.

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


Add doctest_discover_tests()

Heavily inspired on
https://github.com/catchorg/Catch2/blob/7be35af167f969433d9775f6ddc5d294f69dd4b5/docs/cmake-integration.md#catchcmake-and-catchaddtestscmake
Probably it shouldn't be merged as it is, I'm far from a cmake expert. But I didn't know how to discuss it in https://github.com/onqtam/doctest/issues/171 without actually creating the branch.

I had tried the script directly from my project, as was when I was developing it, but hadn't tried it after integrating it with the doctest installation.
I have updated the PR so CMAKE_MODULE_PATH now points to the scripts installation directory and users can just do include(doctest) to be able to use doctest_discover_tests().

decided to merge it now and review it later. Thanks for the contribution!


Update GlobalMessagesService

Add Toast method for external consumers and IToast interface
Unsure when this should get merged, but we need the NPM package to make it out the door before we can create a toast service example in the sample extension.

@clayplumridge could you take another look at this and update the message service to be how we want it now with all the various options? (corner dialog, toast, banner, etc.)

@nkirchem Updated to include the new method on the remotable service and some of the props on IGlobalMessageBanner. Should we include "settingId" that records if the message has been dismissed?


fix: write components.d.ts to correct path

🔗 Linked issue
https://github.com/nuxt/bridge/issues/622

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
components.d.ts is being generated, but the path for nuxt.d.ts is set to types/components.d.ts.

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

@danielroe
Due to the change to types/components.d.ts, it was necessary to modify the path to the component files (which I overlooked). Should I correct it to work with the types/components.d.ts path as intended?
https://github.com/nuxt/bridge/blob/main/packages/bridge/src/type-templates.ts#L18-L23
Or should I revert it back to components.d.ts and leave the path to the component files as-is?

I would keep it in types/ directory for consistency with the other files, and update the relative to be the typesDir not the buildDir.


PixelPick mouse-free annotation tool

Hello，author, thank you very much for your sharing. I'd like to take a look at the mouse-free annotation tool mentioned in this article, but I haven't found the code for it yet. I wonder if you could tell me?

Hi pranoid-cj and thanks for the enquiry.
We are currently working on providing the PixelPick annotation tool with a much better GUI on top of VGG Image Annotator (VIA). However, as per your request and also for other potential users, we uploaded a sample script for the annotation tool in annotation_tool directory.
Please find the instructions in the updated README.md file to implement it and feel free to let us know if you encounter any problems. :)
Kind regards,
Gyungin

Dear Tejasri,
Sorry for the late reply.
As a usage for a custom dataset hasn't been considered yet, we may need
some time to test the code for it.
I'll try to make it available before the end of this November if that's OK
with you.
Kind regards,
Gyungin
On Sun, 10 Oct 2021 at 14:58, Tejasri Nampally @.***>
wrote:

Dear Tejasri Thank you for your inquiry! We currently have a standalone
VIA GUI. What this means is that it has not been incorporated into the
official VIA code yet (due to securing backward compatibility) but has the
exact same VIA features. As we are working on another project at this
moment, I think the integration would take some time. However, I can make
the standalone VIA GUI if you'd like to try it first, so please let me know
your opinion. Kind regards, Gyungin
… <#m_-7979380969750779071_>
On Fri, 8 Oct 2021 at 11:52, Tejasri Nampally @.***> wrote: Hi
pranoid-cj and thanks for the enquiry. We are currently working on
providing the PixelPick annotation tool with a much better GUI on top of
VGG Image Annotator (VIA). https://www.robots.ox.ac.uk/~vgg/software/via/
However, as per your request and also for other potential users, we
uploaded a sample script for the annotation tool in annotation_tool
directory. Please find the instructions in the updated README.md
https://github.com/NoelShin/PixelPick file to implement it and feel free
to let us know if you encounter any problems. :) Kind regards, Gyungin Any
progress ? Would love to try in VGG — You are receiving this because you
commented. Reply to this email directly, view it on GitHub <#2 (comment)
https://github.com/NoelShin/PixelPick/issues/2#issuecomment-938545042>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIZBLA66Q7ABFZOX42VMR6LUF3EOHANCNFSM43SGBZ7A
.
Hi @NoelShin https://github.com/NoelShin , Thanks for your reply :)
Yes, I would like to use the standalone version on our custom dataset.
Waiting for the repo update.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NoelShin/PixelPick/issues/2#issuecomment-939487224,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIZBLA4IBBZN6OT5XTXA63TUGGLXPANCNFSM43SGBZ7A
.

Hey @NoelShin Any update done for custom dataset ?


Use unique element locators for multiproxy selector on desktop and mobile

At the moment we have to use .first when trying to interact with the multiproxySelector element eg:
topMenuItems.multiproxySelector().should('be.visible').first().click()
This is because two elements with that locator can be found in the DOM (the desktop and mobile one).
Let's give them unique names and update tests to get rid of the .first

After looking into this a little bit, I don't think this is because it's finding the selector from the desktop and mobile view.
I think it is more related to how it is rendered in the header (but can't work out exactly why).
I was able to set a unique locator for Desktop and Mobile eg
const theme = useTheme()
  const isDesktop = useMediaQuery(theme.breakpoints.up('md'))

  return (
    <Autocomplete
      className={className}
      isOptionEqualToValue={isOptionEqualToValue}
      disableClearable
      filterOptions={filterOptions}
      options={multiProxyList}
      renderOption={renderOptions}
      renderInput={renderInput}
      getOptionLabel={getOptionLabel}
      onChange={onChange}
      value={canFindMultiProxyFromUrl ? selectedMultiProxy || multiProxyList[0] : ''}
      data-cy={isDesktop ? 'select-multiproxy-desktop' : 'select-multiproxy-mobile'}
    />
  )

Which successfully set a unique one locator per layout (I set some custom viewports to check and saw the mobile one successfully applied).
However, when I removed the .first() from the element interactions in the test file I saw that Cypress would still find two of the select-multiproxy-desktop locators and I wasn't sure how to proceed.

I checked and I'm certain that the issue was because of the dektop/mobile one. The mobile is hidden, but present in the DOM. I didn't use the exact same logic as you did, it's something a little simpler, since we have the info directly from the props depending on where this multiproxy selector is called.


⚠ Bofrost.at has degraded performance

In 5d41548, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1611 ms

Resolved: Bofrost.at performance has improved in bbcffce after 6 minutes.


Add-On Kit: Drag 'n Drop

Replacement for #74 - for the not UI-Schema related logic parts

Package:  @ui-schema/kit-dnd
Docs // following

experimental

Initially released with 0.3.0-alpha.5


Send the notifications to the Ceilometer for backup service

Add the notification send-out for create_backup, delete_backup
and restore_backup.
Add the progress notification to Swift backup service.

DocImpact
Change-Id: I9835073a39aa8b2ffbec12d84147cce027ff731b
implements-bp: backup-notification
(cherry picked from commit 77452d8c05d7a6b62e84dd39930fdca40257ce09)

Fixes: http://<IP_ADDRESS>/issues/10111


Add a reticle that shows the current center

This one was tought. Let me know what you think

TBH I don't like how it looks, but it definitely helps. Let's review it later


Allow empty slices for commit parents

libgit2 allows the parents pointer to be null if there are 0 parents,
and Zig may set ptr to null for zero-length slices:
https://github.com/ziglang/zig/issues/4771

Thanks!
This library needs a lot of love, I'm planning to get back to it and get it working with every released version of libgit2 once zig 0.10 is out.


Change spinning gear to something meaningful and informative.

Monitor job status somehow.

Can you determine progress from the job.status event sequence?

downloading - 20%
queued - 40%
running - 60%
queued - 80%
running - 100%

Sadly no, its not possible to know the progress in terms of a percentage

Be sure it's consistent between tagger and vizer

The above sequence is for the visualize task (ends on running). The tagger task is a different sequence and ends on complete. So for the viz task what's the flag that signifies it's ready?

I made that change to the viz task as I thought is made more sense that the task stayed in the running state, as its not really complete unless someone terminates it. I guess I didn't make the same change for the tagger. Lets discuss what would be the best approach for the UI.

In the VM is there an elegant way to update cumulus and girder?
On Tue, Oct 20, 2015 at 7:23 AM, Chris Harris<EMAIL_ADDRESS>wrote:

I made that change to the viz task as I thought is made more sense that
the task stayed in the running state, as its not really complete unless
someone terminates it. I guess I didn't make the same change for the
tagger. Lets discuss what would be the best approach for the UI.
—
Reply to this email directly or view it on GitHub
https://github.com/Kitware/HPCCloud/issues/68#issuecomment-149567139.

--

Tristan Wright

R&D Engineer
Kitware Inc.

You can add the following to your Vagrantfile in the  config.vm.provision "ansible" do |ansible| block, after line 73:
ansible.tags = ["cumulus", "girder"]

Then run:
vagrant provision

We still need some sort of progress indicator in the mesh tagger its several seconds after we switch to the mesh tagger before we see the mesh ...